-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/unified variants #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Deleted templates, models, repositories, and controllers related to old variant group and alias handling. - Refactored dependent views and logic to remove references to deprecated components. - Cleaned up accompanying documentation and comments.
…ce genome and variant metadata handling** - Replaced `genbankContigId` with `refGenome` and `contig` in forms and backend logic. - Enhanced `editForm` and `createForm` to support clearer metadata fields and additional variant types. - Made genomic coordinates immutable, restricting edits to metadata fields only. - Added improved validation and context for editable and optional fields. - Streamlined templates and controller logic to align with `VariantV2` schema.
… models** - Deleted unused templates (`createForm`, `editForm`, `detailPanel`) for Cytobands and STR Markers. - Removed associated Scala models (`Cytoband`, `StrMarker`), streamlining codebase maintenance. - Cleaned up controllers and routes referencing deprecated Cytoband and STR Marker logic.
… models** - Deleted unused templates (`createForm`, `editForm`, `detailPanel`) for Cytobands and STR Markers. - Removed associated Scala models (`Cytoband`, `StrMarker`), streamlining codebase maintenance. - Cleaned up controllers and routes referencing deprecated Cytoband and STR Marker logic.
…ties** - Extracted reusable badge class and variant formatting logic into `CuratorViewUtils` and `VariantViewUtils`. - Updated multiple views to utilize the new utilities, reducing redundancy and improving maintainability. - Replaced inline helper methods with calls to centralized utility functions across various curator and variant templates.
…earch input components** - Added reusable templates for breadcrumbs, flash messages, and search inputs in `fragments`. - Updated variant browser and curator views to utilize the new components, reducing redundancy. - Simplified and centralized layout structures for improved maintainability.
…earch input components** - Added reusable templates for breadcrumbs, flash messages, and search inputs in `fragments`. - Updated variant browser and curator views to utilize the new components, reducing redundancy. - Simplified and centralized layout structures for improved maintainability.
This feature allows users to toggle between the standard tree layout and the block layout on the main tree pages. The preference is saved in a cookie and respected across sessions.
…d clarity** - Added a temporary index to speed up grouping and joining during variant migration. - Combined variant name and coordinates insertion into a single pass. - Simplified alias aggregation using filtered JSONB operations. - Improved maintainability by removing redundant steps and optimizing query structure. - Dropped temporary index after migration steps to clean up.
…tion with dynamic fragment loading** - Replaced server-side pagination logic for variants and haplogroups with HTMX-driven fragment updates for improved responsiveness. - Simplified controller actions and templates, removing unused parameters and reducing complexity. - Added loading spinner and placeholder text for better user experience during content updates.
- Deleted `StrMarker` model and its corresponding table definition. - Cleaned up related dependencies and imports, reflecting the shift away from legacy STR marker handling.
…ariants** - Introduced `findMatches` method in `VariantV2Repository` for matching variants by coordinates or aliases. - Enhanced `YBrowseVariantIngestionService` to support GFF3 ingestion with coordinate normalization, alias grouping, and liftover handling. - Added `smartUpsertVariant` logic to merge or create variants based on matching results. - Improved metadata merging for aliases, coordinates, evidence, and primers during upsert process. - Included support for parsing and processing Y-DNA variants from both VCF and GFF files, with batch processing for efficiency.
…r better performance and maintainability** - Enhanced migration with GIN index management for faster inserts. - Redesigned aggregation queries using Common Table Expressions (CTEs) for aliases and coordinates. - Simplified deduplication steps for `haplogroup_variant` FK updates. - Added verification and manual cleanup steps for improved integrity and safety.
… ingestion to use. Prioritizing the user experience.
…problems with PostgreSQL's planner and inducing full table scans.
…ll see progress, but not tons of 0 new variants. This is the DEFAULT case.
… original as a link source to the final status.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.