-
Notifications
You must be signed in to change notification settings - Fork 23
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
major refactor of select_mock_targets #264
Conversation
This is nearly ready to merge, although I have not yet tested at NERSC. On my laptop (with 4 cores) generating roughly 23k targets (of all types) in a single |
…into fix-densities
OK, I think all conflicts have been fixed -- thanks for the off-list help, @geordie666! @sbailey once your tests have completed (and hopefully Travis tests pass and the documentation builds successfully!) please feel free to merge, otherwise post any crashes or bugs here or off-list. |
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.
yea, verily, let us be done with this PR and merge it (and face the consequences of our rash deeds)
This PR (which would close #261) includes a near-total rewrite of
select_mock_targets
(which really just callsmock.build.targets_truth
) and introduces target-specific classes (inmock.mockmaker
) which make it much easier, transparent, and extendable to connect mock catalogs to spectra. There are still a variety of outstanding issues (see below), but I'd like to get the current (functional) code merged nowish and go from there.I have tried to demonstrate some of the core functionality in this notebook, which I'm hopeful will be used by various Working Groups to build smaller spectroscopic datasets. Suggestions for additional tutorials or demonstrations of capabilities are welcome.
I tested this at NERSC with a single healpixel (using the
fix-densities
branch ofdesitarget
):then
which took about 7 minutes, with QA output here:
http://portal.nersc.gov/project/cosmo/temp/ioannis/refactor
Major improvements include:
targets
catalog.In addition:
obsconditions
simqso
).Known issues / problems / shortcomings:
quicksurvey
(I'm sorry, @forero!) but I have a plan for putting it back together.seed
provides reproducibility, it's non-trivial to regenerate the spectra from the outputtruth
catalogs, particularly for the QSOs. This is probably OK but medium-term we should probably implement something like what was proposed in update/change data model for template metadata? desisim#290.Additional missing features:
mockmaker
.