Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Effect staalname maand en Poisson distributie op alfa-diversiteit #22

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

slambrechts
Copy link
Collaborator

@slambrechts slambrechts commented Apr 24, 2024

Voorlopig enkel voor Annelida

  1. Variabele Maand_staalname toegevoegd aan sample data phyloseq object
  2. Deze ook toegevoegd in verkenning designvariabelen
  3. alfa-diversiteit eens gelopen met Maand_staalname ipv Landgebruik_MBAG
  4. 1e poging Maand_staalname meenemen in de modellering van effect landgebruik (samen met Diepte en aantal reads). Interactie Landgebruik_MBAG met Maand_staalname hier ook nodig? Voorlopig in aparte chunk gezet en originele alfa-div modellering dus niet aangepast
  5. aparte chunk met alfa-diversiteit modellering obv Poisson ipv glmmTMB::nbinom2, maar nog zonder Maand_staalname

@slambrechts
Copy link
Collaborator Author

  1. Maand_staalname meenemen in de modellering van effect landgebruik (samen met Diepte en aantal reads), chunk [m-annelida-richness-predictions-2] dus, werkt nog niet:
Error in `names(out) <- colnames(stats::vcov(model, full = TRUE))`:
! 'names' attribute [68] must be the same length as the vector [39]
Backtrace:
 1. marginaleffects::plot_predictions(...)
 2. marginaleffects::predictions(...)
 4. marginaleffects (local) `<fn>`(...)
 6. marginaleffects:::get_coef.glmmTMB(model, ...)
Execution halted

Ik vraag mij in eerste instantie af of ik onderstaande juist heb aangevuld met Maand_staalname:

samples_totcount_10k <- tidy_physeq$samples %>%
  filter(total_count > 1e+4)

form_annelida_richness <- formula(
  observed ~
    log(total_count)
  + Landgebruik_MBAG
  + Diepte
  + Maand_staalname
  + Landgebruik_MBAG:Diepte
  + Landgebruik_MBAG:Maand_staalname
  + (1 | Cmon_PlotID)
  )

m_annelida_richness <- glmmTMB::glmmTMB(
  formula = form_annelida_richness,
  data = samples_totcount_10k,
  family = glmmTMB::nbinom2())

@slambrechts
Copy link
Collaborator Author

model ~ log(total_reads) + maand + landgebruik toegevoegd, en origineel nbinom2 model er uit.

Maakt de volgorde van de variabelen Landgebruik_MBAG, Diepte, Maand_staalname uit in onderstaand model?

form_annelida_richness <- formula(
  observed ~
    log(total_count)
  + Landgebruik_MBAG
  + Diepte
  + Maand_staalname
  + Landgebruik_MBAG:Diepte
  + (1 | Cmon_PlotID)
  )

m_annelida_richness <- glmmTMB::glmmTMB(
  formula = form_annelida_richness,
  data = samples_totcount_10k,
  family = poisson())

@hansvancalster
Copy link
Collaborator

Maakt de volgorde van de variabelen Landgebruik_MBAG, Diepte, Maand_staalname uit in onderstaand model?

Maakt niet uit zolang we er geen sequentiële anova op toepassen (type I anova). Je kan op de output van dit model bv een type II anova verkrijgen via car::Anova(m_annelida_richness). Hier speelt de volgorde geen rol.

…iteit modellering met enkel gekende soorten (taxonomische diversiteit)
@slambrechts
Copy link
Collaborator Author

  1. Alfa-diversiteit opgesplitst in genetische diversiteit (aantal OTUs) en taxonomische diversiteit (aantal gekende soorten)
  2. Shannon diversity index proberen modelleren als een gamma verdeling met log link, eens zonder en eens met staalname maand als covariaat

labs(y = "Voorspelde aantal taxa")
```

### Staalname maand als covariaat
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De titel strookt niet met wat volgt

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bedoel je dat staalname maand geen covariaat is als we de interactie met landgebruik verwijderen uit het model?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nee. Ik denk dat ik hiermee bedoelde dat er naast staalname maand ook nog andere covariaten in het model zitten dat in deze sectie gefit werd. Terwijl de titel suggereert dat enkel staalname maand als covariaat bekeken wordt.


### Shannon diversity incl staalname maand als covariaat

nog aan te vullen
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier staat nog een to do

source/rmarkdown/annelida_data_analyse.Rmd Outdated Show resolved Hide resolved
source/rmarkdown/annelida_data_analyse.Rmd Outdated Show resolved Hide resolved
```


### Staalname maand als covariaat
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Titel best aanpassen zodat die wat beter weergeeft wat er onder komt

Comment on lines +882 to +886
Er zijn waarschuwingen die aangeven dat de interactie tussen maand en landgebruik niet mogelijk is.
Dit is te verwachten omdat landgebruik en maand niet volledig gekruist zijn.
We verwijderen deze interactie tussen maand en landgebruik dus uit het model.
Dit is ook duidelijk uit de al eerder getoonde tabel:

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zie eerdere suggesties

@hansvancalster
Copy link
Collaborator

Om de "this branch is out-of-date with the base branch" op te lossen, kan je op de "update branch" knop drukken en de stappen volgen (dit is dan op de remote, om het daarna lokaal te hebben moet je nog git pull uitvoeren lokaal). Andere manier is het lokaal oplossen (git pull origin main) en daarna de wijzigingen pushen naar hier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants