Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
DM-40740: Create an obs_package to handle the fiber spectrographs #1
DM-40740: Create an obs_package to handle the fiber spectrographs #1
Changes from all commits
3c8ad40
3b4a2ae
fd56cc4
3c5637a
78f5e7d
f34a6c9
7d98960
06acd12
d320618
6441a4f
c8ce2b7
97f125e
229442a
9ce6b42
8b21fde
a71b9ca
00b1681
24106db
924193a
275fc7c
ac6aa91
9d4a7f1
6b3ab9c
a58c5fc
e9a0e6e
6eeb8b5
7995fda
425f43b
abd1cdf
c5cce55
0ff0dd4
4245831
39c280e
7848cd4
c2ba25b
740e31d
0dbe239
de5467a
6a73187
9a9a293
8fb4874
c255fbc
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
This file was deleted.
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.
I would be open to you patching obs_lsst to add a package_dir class property so you could avoid copying this method.
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.
sounds good - is it ok to push this to a future ticket to keep things simple for now?