-
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
SIP Generator not outputting correctly for spice example bundle #39
Comments
@nutjob4life btw, I downloaded this data and put it out on our test server |
Note for |
I'd like to split this into two issues, one effecting the AIP generation and one this SIP generation, in order to make more controlled commits & pulls to address these issues. |
nutjob4life
added a commit
that referenced
this issue
Apr 22, 2020
- #39: SIP contains just one product for Insight Spice example - We match the lidvid against all primaries found - Take advantage of a new LogicalReference class for lid+lidvid manipulations - This probably already exists in Java land? - Look for either lid_reference or lidvid_reference in bundle XML - Add a way to compare, hash, and manipulate lidvids with optional vids: class LogicalReference - Representation, stringification - Hashing, equality, comparisons - Compare version IDs smartly (i.e., 2.9 < 2.10) - Matching based on partial lid or full lidvid - Battery of unit tests - Begin support for #24 with command-line handling and flag passing - But default it to True until we have time to actually put it in - Include prominent warning that we're doing that - Remove redundant database connection use and commit
jordanpadams
pushed a commit
that referenced
this issue
Apr 22, 2020
- #39: SIP contains just one product for Insight Spice example - We match the lidvid against all primaries found - Take advantage of a new LogicalReference class for lid+lidvid manipulations - This probably already exists in Java land? - Look for either lid_reference or lidvid_reference in bundle XML - Add a way to compare, hash, and manipulate lidvids with optional vids: class LogicalReference - Representation, stringification - Hashing, equality, comparisons - Compare version IDs smartly (i.e., 2.9 < 2.10) - Matching based on partial lid or full lidvid - Battery of unit tests - Begin support for #24 with command-line handling and flag passing - But default it to True until we have time to actually put it in - Include prominent warning that we're doing that - Remove redundant database connection use and commit
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
When I run the latest
pds-deep-archive
software, there are a few issues with the output:Note: I pointed the software at v1 of the bundle (
bundle_insight_spice_v001.xml
) so it should take that version and the versions it is referencing into account.The checksum manifest and transfer manifest both include all versions of the collections, but these should only contain the versions referenced fromSee AIP Generator not outputting correctly for spice example bundle #41bundle_insight_spice_v001.xml
.The text was updated successfully, but these errors were encountered: