You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
During the refactoring --prefix option seems to be dropped. Lya WG group used this option to run desi_zcatalog script on quickquasars mocks (see issue #1370 by @alxogm ). Maybe the script got too complicated to support this argument, @sbailey ?
Workaround:
Do not load the main branch (tested by Andrei): source /global/common/software/desi/desi_environment.sh 23.1
detail: edited by Stephen to explicitly load environment 23.1 rather than the default, which will change in the future)
Oh, I was not aware there was a refactoring of desi_zcatalog. Lets see if @sbailey confirms if this script will not continue to support other prefix's, in which case we can add @p-slash script, or a similar one, maybe to desisim or igmhub for further use of the LyaWG. or its just a matter of including the feature in the new version.
Apologies for breaking the --prefix option. Supporting it in the refactor was getting complicated, though from looking at the code again I don't remember why. I thought I was dropping a leftover option from pre-data days, not realizing it was still in use.
Two requests:
please post an example directory at NERSC and what --prefix option previously would have worked but is now broken (for testing that the right feature is actually being re-implemented usefully)
does anyone in the LyA WG have time to try to fix this? Normally "I broke it I should fix it", but that might take longer for me to find the time than if someone else could jump in to help (with my thanks...)
During the refactoring
--prefix
option seems to be dropped. Lya WG group used this option to rundesi_zcatalog
script on quickquasars mocks (see issue #1370 by @alxogm ). Maybe the script got too complicated to support this argument, @sbailey ?Workaround:
source /global/common/software/desi/desi_environment.sh 23.1
detail: edited by Stephen to explicitly load environment 23.1 rather than the default, which will change in the future)
The text was updated successfully, but these errors were encountered: