Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

All products are assumed to be Product_Observational by PLAID #25

Closed
jordanpadams opened this issue Jun 3, 2021 · 13 comments
Closed

All products are assumed to be Product_Observational by PLAID #25

jordanpadams opened this issue Jun 3, 2021 · 13 comments

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Jun 3, 2021

馃悰 Describe the bug

Whenever I try to create a product not Observational (e.g. Document, Context, etc.), the software tries to drop this into Product_Observational and it does not work.

Minimal support should include:
See https://pds.nasa.gov/datastandards/documents/im/v1/index_1G00.html#class_pds_observation_area for:

  • Product_Observational

See https://pds.nasa.gov/datastandards/documents/im/v1/index_1G00.html#class_pds_context_area for:

  • Product_Context
  • Product_Document
  • Product_Collection
  • Product_Bundle
@jordanpadams
Copy link
Member Author

bug may be here:

$filename = "../workspace/observational.xml";

@voxparcxls
Copy link
Collaborator

@jordanpadams

The two criteria for Issue#25 to include 2 product types (Collection, Bundle) and fix XML mutator to ascribe correct outer <Product_Type> tag to all 7 types

  • Types: Collection , Bundle
  • <Product_Type> fix not complete as of yet

Screen Shot 2021-07-14 at 1 41 24 PM

Product Type Collection, Bundle product manually tested using cases from: (https://pds.nasa.gov/data/pds4)

@jordanpadams
Copy link
Member Author

@voxparcxls is there anything I can do to get this moving forward? Is there somewhere you are blocked?

@voxparcxls
Copy link
Collaborator

@jordanpadams For the product_type XML bug, there's nothing large thats blocking me. I just need to retrieve the product selection POST data without it interrupting the following Label pages.

I can use the rest of today to narrow that down. And open PR when it's ready.

@voxparcxls
Copy link
Collaborator

voxparcxls commented Jul 14, 2021

@jordanpadams

P.S., the dev version is up on: https://miplvm-docker1.jpl.nasa.gov/plaid if you would want to try it.

I'll be fully migrating the DB and updating for Issue#25

@voxparcxls
Copy link
Collaborator

@jordanpadams

Product_Type names are now prepended to each respective products:

  • Product_Observational
  • Product_Document
  • Product_Context
  • Product_Collection
  • Product_Bundle

Test for Product_Context : https://pds.nasa.gov/data/pds4/context-pds4/node/node.geoscience_1.0.xml

Screen Shot 2021-07-21 at 2 06 14 PM

@jordanpadams
Copy link
Member Author

@voxparcxls looks like there is still something funky going on.

It looks like it is inserting an extra xml version and Product_Context into the label?

@voxparcxls
Copy link
Collaborator

Here's the Label product type selection.

Screen Shot 2021-07-22 at 12 55 19 AM

I'll open a branch for the Product_Context fixes.

@voxparcxls
Copy link
Collaborator

@jordanpadams

Context Label Output is fixed. Removed resting label error.

  • Removed empty <Discipline_Area></Discipline_Area> tag from Product_Context labels
  • Product_Context no longer has

Screen Shot 2021-07-23 at 12 50 13 PM

Both fixed included in Branch: PLAID_Issue25

@jordanpadams
Copy link
Member Author

@voxparcxls is this branch ready for a PR?

@voxparcxls
Copy link
Collaborator

@jordanpadams

Yes #25 PR is almost ready. I'm writing up the summary for it, and should be opened in a few hours.

Dev-Deployment UPDATE:

  • The new image is pushed to Artifactory:
    cae-artifactory.jpl.nasa.gov:16001/gov/nasa/jpl/ammos/ids/plaid/plaidimage:V3
  • Now configuring the DB container for dev, mariadb:10.3.23. (same version as live PLAID)

I will give status on dev deploy this afternoon.

@voxparcxls
Copy link
Collaborator

@jordanpadams

Test cases took longer than expected and sidetracked opening the PR.

Working on publishing the dev deployment ASAP.

@jordanpadams
Copy link
Member Author

closed per #27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants