Make confSummary_test default#91
Conversation
havok2063
left a comment
There was a problem hiding this comment.
Does this change need to also be in the ipl-3.cfg? If so, can you update the paths there as well.
|
I'm not sure since IPL-3 may have been run using the old directories (although I'm also not sure that matters since at the end of the day the environment variable does not change). @Sean-Morrison had thoughts on this I think. |
|
For the BOSS DRP, the confSummary(F)_test paths from sdss_access and tree were used. Therefore maintaining the _test paths is important. As for the other paths, I don't have a strong feeling especially since I think the Apogee DRP (@dnidever) used hard-coded paths. |
|
apogee_drp 1.3 for IPL-3 used hardcoded confSummary paths in apload.py |
|
@havok2063 @joelbrownstein When you have a change can you double check everything and if it looks fine approve the PR? |
Replaces the definition for
confSummaryandconfSummaryFwith the same one as forconfSummary(F)_test(i.e., with the thousands grouping in addition to the hundreds one).For now we are keeping the definitions for
confSummary(F)_test, which are now identical to the ones forconfSummary(F).