Skip to content

Conversation

@jtarraga
Copy link
Member

No description provided.

@jtarraga jtarraga requested a review from Copilot May 29, 2025 10:11
Copy link

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.

Pull Request Overview

This PR introduces two new annotation features: integrating genomic sequence context annotation and ACMG classification into the variant annotation workflow, along with a minor checkstyle configuration update.

  • Updated checkstyle.xml to increase the maximum method length.
  • Modified test expectations in VariantAnnotationCalculatorTest.java to accommodate changes in the ConsequenceType constructor.
  • Added FutureGenomicSequenceContextAnnotator to generate genomic sequence context annotations and integrated ACMG classification in VariantAnnotationCalculator.java.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
checkstyle.xml Increased the max value for MethodLength from 250 to 300, updating coding style constraints.
cellbase-lib/src/test/java/org/opencb/cellbase/lib/impl/core/VariantAnnotationCalculatorTest.java Updated test cases to reflect the additional parameter in ConsequenceType instantiation.
cellbase-lib/src/main/java/org/opencb/cellbase/lib/variant/annotation/futures/FutureGenomicSequenceContextAnnotator.java Introduced a new annotator implementation to fetch genomic sequence context asynchronously.
cellbase-lib/src/main/java/org/opencb/cellbase/lib/variant/annotation/VariantAnnotationCalculator.java Integrated genomic sequence context processing and added ACMG classification for consequence types.
Comments suppressed due to low confidence (1)

cellbase-lib/src/main/java/org/opencb/cellbase/lib/variant/annotation/VariantAnnotationCalculator.java:699

  • Ensure that 'java.util.stream.Collectors' is imported to support the usage of 'Collectors.toList()'.
consequenceType.setAcmg(acmgs.stream().map(ClinicalAcmg::getClassification).collect(Collectors.toList()));

@jtarraga jtarraga requested a review from pfurio May 29, 2025 10:13
pfurio
pfurio previously approved these changes May 29, 2025
pfurio
pfurio previously approved these changes May 29, 2025
@juanfeSanahuja juanfeSanahuja requested a review from pfurio June 27, 2025 09:06
pfurio
pfurio previously approved these changes Jun 27, 2025
pfurio
pfurio previously approved these changes Jul 15, 2025
@jtarraga jtarraga requested a review from j-coll September 10, 2025 09:54
j-coll
j-coll previously approved these changes Sep 10, 2025
@j-coll j-coll changed the title Include new annotation data: ACMG and variant context TASK-7549 - Include new annotation data: ACMG and variant context Sep 10, 2025
j-coll
j-coll previously approved these changes Sep 11, 2025
@juanfeSanahuja juanfeSanahuja self-requested a review October 30, 2025 14:26
juanfeSanahuja
juanfeSanahuja previously approved these changes Oct 30, 2025
@juanfeSanahuja juanfeSanahuja dismissed stale reviews from j-coll and themself via 76a6f36 October 30, 2025 15:17
@jtarraga jtarraga merged commit 1e5aa89 into release-6.x.x Nov 5, 2025
6 of 7 checks passed
@jtarraga jtarraga deleted the TASK-7549 branch November 5, 2025 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants