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

stellar mass function #49

Closed
3 of 4 tasks
yymao opened this issue Dec 14, 2017 · 10 comments
Closed
3 of 4 tasks

stellar mass function #49

yymao opened this issue Dec 14, 2017 · 10 comments

Comments

@yymao
Copy link
Member

yymao commented Dec 14, 2017

I know @evevkovacs is working on the SMF test so I opened this issue to track progress.

  • code to reduce mock data
  • code that works within DESCQA framework
  • validation data
  • validation criteria
@yymao
Copy link
Member Author

yymao commented Dec 26, 2017

A SMF test with PRIMUS 2013 data is now available from the merged PR #51.

@rmandelb
Copy link

@janewman-pitt-edu @sschmidt23 @morriscb - was this test of the stellar mass function in DC2 intended for the photo-z working group?

I hope @yymao will correct me if I am wrong, but my understanding is that the test has been implemented currently without a validation criterion, so we just need a validation criterion to close this issue. Might PZ be able to supply a validation criterion? You can see the format of the test here:

https://portal.nersc.gov/project/lsst/descqa/v2/?run=2017-12-21_14&test=SMF_PRIMUS

it's implemented in redshift bins based on comparison with PRIMUS.

@yymao
Copy link
Member Author

yymao commented Jan 30, 2018

Yes, @rmandelb. This test is already implemented. We need a validation criterion. If the PZ WG would like to provide a different validation data set, we can do that too.

@evevkovacs
Copy link
Contributor

@rmandelb @janewman-pitt-edu @sschmidt23 This test was implemented in DESCQA1 and I ported it to DESCQA2 because it was such a basic test. I added the PRIMUS data for the higher redshifts. It wasn't formally requested by any WG to my knowledge.

@janewman-pitt-edu
Copy link

No, PZ has little interest in the mass function. I think it was for LSS.

I would be wary of PRIMUS constraints as the sample is highly incomplete and has a relatively high catastrophic redshift error rate. Low-z constraints are almost certainly more useful (especially because they can probe below L*).

@rmandelb
Copy link

rmandelb commented Feb 1, 2018

@slosar - can you confirm whether LSS would like to have a validation test of the stellar mass function for DC2?

If no, then I think we can safely dispense with this validation test for DC2 since it's not needed to support any working group's goals. If yes, then we need to define validation criteria etc.

@janewman-pitt-edu
Copy link

Just a note -- if we don't do a stellar mass constraint (and I'd be happy not to, as unlike magnitudes stellar masses are not directly observable but rather rely on models/assumptions to infer), we do want to have some sort of dN/dmag or dN/dz test to make sure the numbers of objects are sane.

@rmandelb
Copy link

rmandelb commented Feb 1, 2018

Yes, I agree. I don't believe there is any thought that the dN/dz or dN/dmag test will disappear (despite the fact that the exact form of these is still being finalized).

@yymao
Copy link
Member Author

yymao commented Feb 1, 2018

Given that this is not a required test and it is already implemented (though without validation criteria). We can probably close this issue. Do you agree @rmandelb?

@rmandelb
Copy link

rmandelb commented Feb 2, 2018

@yymao - I agree. People can look at the plot, and that's it - we are done. Thank you!

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

No branches or pull requests

4 participants