-
Notifications
You must be signed in to change notification settings - Fork 1
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
Going through tutorial #7
Comments
Hey, Spaak. Thanks for the comprehensive feedback. I'll run through each topic according to your sorting list.
We could enhance the Joss article with some of this discussion. Thank you! |
Thanks for the clear responses! Pinging @AJQuinn (JOSS editor) here. I'm not entirely sure how JOSS reviews proceed after something like this. The tutorial is part of the documentation, and I gave some suggestions to improve it. Can I safely move on to the rest of the review checklist, or should I aim to check whether the tutorial and paper have indeed improved after I leave such comments? (I'd prefer moving on and leaving a second/third/etc check to either the editor or a re-review, if it comes to that. Otherwise I'm worried the review process will take ages.) Thanks :) |
(As mentioned before, I'm a reviewer for JOSS, see openjournals/joss-reviews#6037)
I'm going through the tutorial here https://jeancmaia.github.io/posts/tutorial-mcglm/tutorial_mcglm.html and wanted to highlight a few things.
modelresults.pearson_residuals
is allNaN
s. At the step before that, you emphasize checking these residuals, so a natural thing would be to also do that here. Likely relevant warning:mc_ma
doing for us exactly? (The documentation ofmc_ma
is not sufficient; also the naming could be a bit more informative.)The text was updated successfully, but these errors were encountered: