Skip to content

Conversation

@bencap
Copy link
Collaborator

@bencap bencap commented Nov 26, 2025

This pull request updates several components to support a class-based calibration model introduced by VariantEffect/mavedb-api#589.

Migration to new schema types and properties:

  • Replaced usage of functionalRanges and the FunctionalRange type with functionalClassifications and the corresponding OpenAPI-generated types throughout AssayFactSheet.vue, CalibrationTable.vue, and ScoreSetHeatmap.vue. This includes updating computed properties, method signatures, and template logic to use the new structure. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Enhancements to CalibrationTable functionality:

  • Added support for class-based calibrations: the table now conditionally displays class names and the number of variants in each class when functionalClassifications are present and class-based. This includes new computed properties and template sections for rendering this information. [1] [2] [3]

Visualization Improvements:

  • Added a series classifier for class-based calibrations, since thresholds can be misleading when overlapping.
  • Added a warning to the histogram when a class-based calibration is selected which notes why thresholds are not drawn.

Template and display improvements:

  • Updated template logic in AssayFactSheet.vue to explicitly type keyword lookups and improve readability, ensuring correct access to experiment keywords and their labels. [1] [2] [3]
  • Changed the "Classification" row header in CalibrationTable.vue to "Functional classification" for clarity.

Type safety and prop improvements:

  • Updated component prop types to use OpenAPI-generated types (e.g., components['schemas']['ScoreSet'], components['schemas']['ScoreCalibration']) for improved type safety and code clarity. [1] [2] [3]

Sorting and utility method updates:

  • Refactored calibration range sorting logic to handle both class-based and interval-based calibrations, sorting alphabetically by class and then by range boundaries as needed.

@bencap bencap linked an issue Nov 26, 2025 that may be closed by this pull request
@bencap bencap requested a review from sallybg November 26, 2025 18:34
…ns and allow class based calibration creation/modification

- Allowed class based calibration creation in CalibrationEditor.vue.
- Transitioned to form data in CalibrationEditor.vue invokers to reflect class based file upload.
- Updated CalibrationTable.vue to replace functional ranges with functional classifications.
- Modified ScoreSetHeatmap.vue to handle functional classifications in heatmap calculations.
- Adjusted ScoreSetHistogram.vue to utilize functional classifications for histogram shaders.
- Revised VariantMeasurementView.vue to check against functional classifications instead of ranges.
- Enhanced ScoreSetCalibrationsView.vue to display functional classification counts and handle empty states.
- Updated ScoreSetCreator.vue to validate functional classifications during calibration creation.
- Refined ScoreSetView.vue to include classes draft reference in the calibration editor.
- Improved SearchVariantsScreen.vue to filter score calibrations based on functional classifications.
- Cleaned up calibrations.ts by removing outdated interfaces and adapting functions to use new schema components.
… from variant scores when a calibration is class based
@bencap bencap force-pushed the feature/bencap/categorical-calibrations branch from 8b48379 to f3e7d9a Compare December 30, 2025 23:53
@bencap bencap changed the base branch from release-2025.6.0 to release-2025.5.3 December 31, 2025 00:00
@bencap bencap changed the base branch from release-2025.5.3 to release-2025.6.0 December 31, 2025 00:01
bencap added 4 commits January 5, 2026 11:27
…leRef in ScoreSetView

This annotation in a JS file was causing the automatic linter to strip parenthesis from the ref definition, making the classes flie undefined in all cases and preventing it from being saved when set.
Includes some automatic formatting changes from linter.
@jstone-dev
Copy link
Collaborator

I know there's still some rebasing or merging to do, but the current version is behaving as expected for all the cases I've tested.

@bencap bencap merged commit f04c98c into release-2025.6.0 Jan 14, 2026
@bencap bencap deleted the feature/bencap/categorical-calibrations branch January 14, 2026 17:36
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.

Calibrations without score ranges

3 participants