Skip to content

Conversation

@frano-m
Copy link
Contributor

@frano-m frano-m commented Dec 15, 2025

Closes #4618.

This pull request adds support for displaying the "Data Modality" field for files across all major entity tables (Activities, BioSamples, Datasets, Donors, Files) in the AnVIL CMG UI. It introduces new configuration keys and labels, updates view model builders to aggregate and display data modality values, and removes some unused transformer functions related to activities. The changes ensure that users can view and filter by data modality in the UI tables, improving data discoverability and consistency.

Entity Table Enhancements:

  • Added the "Data Modality" column (using the new key files.data_modality) to the Activities, BioSamples, Datasets, Donors, and Files entity tables, with appropriate view builders and initial column visibility set to hidden. [1] [2] [3] [4] [5] [6]

Configuration and Labeling:

  • Introduced new configuration key and label for file data modality in category.ts, and updated the dev config to include the new field in the file category. [1] [2] [3]

View Model and Response Types:

  • Added the EntityResponse type to unify response types for easier handling and updated view model builders to support aggregated data modality extraction and rendering for the new column. [1] [2] [3] [4]

Transformer and Utility Cleanup:

  • Removed unused transformer functions for activity data modalities, and cleaned up related imports. [1] [2] [3]

These changes collectively improve the consistency and usability of data modality representation in the AnVIL CMG interface.

image

@frano-m frano-m force-pushed the fran/4618-data-modality branch from 06f8de0 to 5eb2776 Compare December 15, 2025 03:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@frano-m
Copy link
Contributor Author

frano-m commented Dec 15, 2025

@copilot fix the failing test please.

Copy link
Contributor

Copilot AI commented Dec 15, 2025

@frano-m I've opened a new pull request, #4624, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits December 15, 2025 18:09
…es (#4618) (#4624)

* Initial plan

* fix: add FILE_DATA_MODALITY to selectable columns for all entity tables (#4618)

Co-authored-by: frano-m <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: frano-m <[email protected]>
@NoopDog NoopDog merged commit 9e44bb3 into main Dec 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AnVIL Explorer - Add file.data_modality to the facets and tables

3 participants