-
Notifications
You must be signed in to change notification settings - Fork 0
Creating default print methods for class sr_model #114
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
base: main
Are you sure you want to change the base?
Conversation
Merge commit '4ee4b99f9f601b2f95fab79fb25a7d781b37e1a7' #Conflicts: # R/Run_Mod.R
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
📖 https://ucd-serg.github.io/serodynamics/preview/pr114 |
I am a little confused with the warning/error I am getting here. It looks like there is a scoping issue relating to the base dataset when using dplyr, but when I fix the issue the tests are producing a warning. It seems like the solution might be in conflicting with each other. Grateful for any help! |
Merge commit '34b08d3a8c2659714e89031c368c3979912e8d6f' #Conflicts: # NEWS.md
Merge commit '4660563dd819748b6850e0b60c2eb51fd3e263e4' #Conflicts: # DESCRIPTION
Hi Ezra, I am having a harder time with this PR. I am not totally sure why one of the tests is not passing. Can you take a look when you have time? Also would be happy to hop on a quick chat to discuss. Thank you, |
It looks like the test snap is recording different values than what are produced when going through github reviews. I am not totally sure why this would be happening -- maybe the median() command is being processed differently in the github reviews. @d-morrison Any pointers are greatly appreciated here! |
I am creating a default print methods for sr_model using the S3 object system. The default print provides the mean value of the posterior distribution for each parameter and all stratification iso_type combinations.