-
Notifications
You must be signed in to change notification settings - Fork 4
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
update boss paths for v6_2+ #65
Conversation
The changes here are to support sdss/tree#94, so the tests are failing since the new path templates are not available to the tests |
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.
There are some tests failing in the CI, but this could be due to the waiting PR on tree. Will wait to see if that fixes them before approving.
@Sean-Morrison I merged in the tree PR. The CI for this PR is failing with some linting issues. Looks like a syntax error. Can you check that? |
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.
This all looks good now. Thanks @Sean-Morrison
Made modifications with the addition of new special functions to support the v6_2_0+ BOSS DRP paths while maintaining backward compatibility for older pipeline reductions