NASA-PDS / PLAID

APPS PDS Label Assistant for Interactive Design (PLAID). See an overview of the software on YouTube. https://www.youtube.com/watch?v=WCo8erW_rL8
https://plaid.jpl.nasa.gov
Other
9 stars 0 forks source link

All products are assumed to be Product_Observational by PLAID #25

Closed jordanpadams closed 2 years ago

jordanpadams commented 3 years ago

🐛 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:

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

jordanpadams commented 3 years ago

bug may be here: https://github.com/NASA-PDS/PLAID/blob/2511e5cd6e40499e5ce7edca0699bb74cc92eab8/php/interact_db.php#L570

voxparcxls commented 2 years ago

@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

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 commented 2 years ago

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

voxparcxls commented 2 years ago

@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 commented 2 years ago

@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 commented 2 years ago

@jordanpadams

Product_Type names are now prepended to each respective products:

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 commented 2 years ago

@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 commented 2 years ago

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 commented 2 years ago

@jordanpadams

Context Label Output is fixed. Removed resting label error.

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

Both fixed included in Branch: PLAID_Issue25

jordanpadams commented 2 years ago

@voxparcxls is this branch ready for a PR?

voxparcxls commented 2 years ago

@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:

I will give status on dev deploy this afternoon.

voxparcxls commented 2 years ago

@jordanpadams

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

Working on publishing the dev deployment ASAP.

jordanpadams commented 2 years ago

closed per #27