Description

Story: BI-1993 - Accept NA Observations

Enables import of NA observations. Previously "NA" observations were set to blank on import due to default tablesaw behavior and "na" observations would be marked as invalid by validations for certain observation types.

Modify experiment import processing to skip observation ontology validations in the case where the entry is NA (case insensitive).

Modify pom/etc to utilize BI branch of tablesaw.

Dependencies

This utilizes a BI fork of tablesaw created as a consequence of this card due to the need to remove NA from the inbuilt missingValueString list that automatically converts cells with values on the list to empty cells.

Testing

Create an experiment and multiple types of ontology terms
Import observations for different ontology term types with the following observation values:

• Valid value (should be displayed correctly on import and be imported properly)
• Invalid value (should get an error)
• NA (should be NA on import and be imported properly)
• na (should be na on import and be imported properly)
• n/a (should still be set to blank due to existing tablesaw functionality)

Checklist:

• I have performed a self-review of my own code
• I have tested my code and ensured it meets the acceptance criteria of the story
• I have tested that my code works with both the brapi-java-server and BreedBase
• I have create/modified unit tests to cover this change
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to documentation
• I have run TAF: <please include a link to TAF run>