Skip to content

Conversation

@davedrp
Copy link
Contributor

@davedrp davedrp commented Feb 3, 2026

Description

BI-2716 Exp preview observation variable count is wrong

Dependencies

bi-api: develop

Testing

Import new Experiment with Observation Variables (Ontology) and Time stamps.
Under Import Summary the Observation Variables should be the count of the Observation Variables ONLY (and not counting the Time Stamps.

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 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: <link to TAF run>
  • I have run SiteImprove on pages impacted by changes

@davedrp davedrp requested review from a team, HMS17 and mlm483 and removed request for a team February 3, 2026 14:54
@github-actions github-actions bot added the bug Something isn't working label Feb 3, 2026
@davedrp
Copy link
Contributor Author

davedrp commented Feb 3, 2026

Sorry, I did not mean to check-in the version of the code with the format corrections. Most of the changes are just indentations. This could make reviewing the code more complicated than it needs to be. The only Significant code change are the lines that start with <br>Observation Variables:

@HMS17 HMS17 self-assigned this Feb 3, 2026
Copy link
Contributor

@HMS17 HMS17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Observation variable count on preview behaves as expected

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.

3 participants