-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adding validation test for stellar mass distribution of galaxies and their number densities #146
Conversation
@Andromedanita thanks very much for the PR! I guess you are still working on this? (which is totally OK -- we encourage early PR, just let me know when it is ready to be reviewed.) I did spot a few things already: to import from descqa submodule, you'll need to add a from .base import BaseValidationTest, TestResult
from .plotting import plt Also, remove these: import matplotlib.pylab as plt
%matplotlib inline And, you'll need to add a yaml config file too. Let me know if you have any questions! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have edited the code so that it runs and passes basic QA. However, this PR is still missing a test config file, and also an actual DESCQA run.
@yymao I have edited the code by a tiny bit (changed the title of the plot and also includes cmass plot). Also, what is the step that I need to do to add a test config file and actual DESCQA run that you mentioned? |
@Andromedanita yes, please commit. Just watch out for potential merge conflicts. To add a test config, just add a yaml file (example below) inside subclass_name: StellarMassDistribution.StellarMassTest
description: <add a description here> |
7fbdae9
to
b9068f2
Compare
b9068f2
to
9359b92
Compare
@Andromedanita (cc @evevkovacs), I have updated this branch to make the test work within the DESCQA framework, and also utilize @Andromedanita Can you review my updates and see if I implemented your test correctly. If you want to make future changes, make sure you pull from this branch first before you add new changes. @evevkovacs Can you test if this updated test now works? |
@yymao I reviewed the code and it seems perfect! Thanks for reviewing it and making the necessary changes! |
I've just run this test on |
Validation test for issue #109
[edited by @yymao: fix #109]