-
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
Halo -> MTL wrapper #26
Comments
I'm not sure whether we need a function that tries to meet all cases, vs. just documentation with a worked example that can be adapted to a specific analysis. See the FiberAssign tutorial in desihub/tutorials as a starting point for how to create an MTL file from scratch with the minimal set of columns required. Basically we'd need code to take a halo catalog and add those columns. |
Yes now we are digging on this option, creating the MTL file from scratch , I think for our purposes is enought ,the interest to run select_mock_targets is to generate again Dark Sky Mocks being completly compatible with codes (i.e mostly fiber assign). I will update on this soon. |
Sounds good. Not sure what you mean by meets all cases, given that a mock will largely be ra, dec, and redshift. There seems to be a bottleneck where it takes a lot of compute time, and even John, to keep mocks up-to-date with data systems. I do wonder about a 'pro', but lightweight, version of select_mock_targets. i.e the goal shouldn't be Anyway, happy with whatever works for everyone. |
+1 for a lightweight-but-pro version of select_mock_targets that is aimed at mocks through fiberassign including standards and sky, but is not aimed at simulations beyond that, e.g. not trying to generate realistic distributions of magnitudes that pass target selection, etc. |
Apologies for the radio silence. I'll take this as-assigned, but I'm preparing for some travel next week and so it may be a couple weeks before I can get something functional into the software stack. Will post additional questions here but in the meantime feel free to add feature requests. |
I suggest that QGC/sim groups should identify someone within their group to take ownership of this, and then work with experts like John, Jaime, Sarah, Ted, myself to make it happen. Over multiple conversations there appears to be a misconception (due to lack of documentation and lack of broadly spread expertise) that everything has to run through select_mock_targets and make_mtl, which is more complex than many cases need. We should train the working groups in the underlying tools so that all requests don't have to route via @moustakas . Put another way, much of this request is to make a simplified mocks -> fiberassign that doesn't need @moustakas, so we should try to do that without needing @moustakas. We can save him for the more complex end-to-end simulations that do need consistency between target selection, magnitudes, spectra, etc. |
@sbailey I agree that this it is probably a misconception, that everything has to run through select_mock_targets I am happy to take this on me (I am working with my student @CesarArroyo09 who is preparing a jupyter notebook with the kind of things the clustering goup needs that is mostly running fiber assignement on randoms, etc...) |
Thanks @mmarianav and @CesarArroyo09 . When you've had a chance to try |
There's a need for a simple transition from halo catalogues (at the right target density) to an MTL file format that skips the cost/expertise of running select_mock_targets and generates input for assignment. Ideally, this should also be implemented by minitest to ensure everything is kept up to date with the (MTL) format choice.
This facilitates the GQC/Sim group simply creating desi-like output for fiber assignment and as a start to the code base.
Could well be questions on how to best select targets etc, but the aim is to be simple than select_mock_targets, which people have been having trouble running successfully.
Interested parties:
Mariana V
Srivatsan
Albert
That may have something on this already.
The text was updated successfully, but these errors were encountered: