Skip to content

[BI-2803] Fix broken preview table on Sub Entity Appends and Observation Variable Stat#461

Open
jloux-brapi wants to merge 5 commits intodevelopfrom
bug/BI-2803
Open

[BI-2803] Fix broken preview table on Sub Entity Appends and Observation Variable Stat#461
jloux-brapi wants to merge 5 commits intodevelopfrom
bug/BI-2803

Conversation

@jloux-brapi
Copy link
Contributor

@jloux-brapi jloux-brapi commented Mar 18, 2026

Description

Story: BI-2803

Please include a summary of the change that was made.

ImportExperiment:

  • Add a mount function to fetch the traits for the activeProgram on load of the Import Experiment tab
  • Utilize the traits on initial upload preview to filter out dynamic columns which are not traits/ontology terms. This was effectively preventing the table from being initialized in the sub entity dataset appends use case.
  • Utilize new backend obsVarCount statistic instead of a filter on the dynamicCols to display the observation variable count. The old variable count filter was displaying the wrong number of observation variables in the sub entity dataset appends use case.

ImportTemplate:

  • Remove the filter on obsUnitId dynamic columns, as the new code introduced takes care of this use case.

Dependencies

bi-api PR

Testing

Prerequisites

To test, you will want to try creating a program from scratch, and complete a germplasm and ontology upload. You will need to reference the ontology terms you created as "observation variables" during the testing.

Tests

  • Verify that on Experiment Import creation workflow, the table preview correctly displays in multiple use cases:

    • When no observation variables are present in the import
    • When observation variables are present in the import; additionally verify the observations appear in the import preview
  • Verify that on Experiment Import appends workflow on a top level dataset, the table preview correctly displays in multiple use cases:

    • When no observation variables are present in the import
    • When observation variables are present in the import: additionally verify the observations appear in the import preview
  • Verify that on Experiment Import appends workflow on a sub entity level dataset, the table preview correctly displays in multiple use cases:

    • When no observation variables are present in the import
    • When observation variables are present in the import: additionally verify the observations appear in the import preview
  • Verify Observation Variable count is correct in the preview in multiple use cases in the Experiment Import Creation workflow:

    • No obs vars
    • Singular obs vars
    • Multiple obs vars
      *Verify Observation Variable count is correct in the preview in multiple use cases in the Experiment Import Appends workflow, for both top-level and sub-entity dataset appends:
    • No obs vars
    • Singular obs vars
    • Multiple obs vars

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 commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth
  • I have run SiteImprove on pages impacted by changes

OUIDs will now be filtered by function which filters on dynCols
that exist in the associated trait list for a program.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant