-
Notifications
You must be signed in to change notification settings - Fork 12
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 to support ComplexLSB8 data types and investigate Floating point exception #7
Comments
Needs a sample product for testing. |
@rchenatjpl can you help find us an example? |
I'll create one. I'll look, but I assume I've never seen one, else I would have started a ticket. |
OR I have seen one, and it worked. On https://pds-ppi.igpp.ucla.edu/search/view/?f=null&id=pds://PPI/mess-rs-raw/data-rsr/2015/mess_rs_20150010130_2b1_rsr&o=1, there's 1 field with data_type=ComplexMSB16. It validates, and you can muck around and set it to MSB8 or LSB8 or LSB16, though for the 8, you'll have to change field_length to 8. pds4_viewer does the right thing as well. |
The schematron has another place that allows ComplexLSB8 et al: Element_Array/data_type. I could try to dummy something up if desired, but please let me know if so. Can we press the original guy for a sample? |
thanks @rchenatjpl we will try to use that test data noted above. @al-niessner does that suffice? |
@al-niessner original test data from user added to top comment |
Note:The XML provided was heavily flawed and had to correct the schema/schematron portions to reduce it to just the bad value. |
Um, this is a really deep problem because it seems as though pds4-jparser does not support complex types at all. All of the data that we process goes through this interface: gov.nasa.pds.objectAccess.array.DataTypeAdapter. It only has the ability for scalars. What is happening is that gov.nasa.pds.objectAccess.array.ElementType does not have a ComplexLSB8 - should it be IEEE754ComplexLSB8? - and is thus generating a null which then is accessed which then throws a NullPointerException and is caught way over in validate and turned into a bad_field_value message. Can improve error messaging and will to say that given However, need breakout time today to discuss how to move forward with complex data types. It is probably going to really break a lot of jparser and validate to support it. |
@al-niessner Complex Binary Data Types should be supported by PDS4: https://pds.nasa.gov/datastandards/documents/im/v1/index_1J00.html#class_pds_complex |
Are there more validate issues? Mine passes it % validate -V |
From an e-mail from Shubham Gupta from ISRO:
Test Data
SAC.zip
The text was updated successfully, but these errors were encountered: