Skip to content
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

ENH: add action to collate genomes to GenomeData[DNASequences] #99

Merged

Conversation

DorielaGrabocka
Copy link
Contributor

Closes #97

Copy link

codecov bot commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.39%. Comparing base (789cc26) to head (939df6f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
+ Coverage   98.32%   98.39%   +0.07%     
==========================================
  Files          39       39              
  Lines        2631     2748     +117     
==========================================
+ Hits         2587     2704     +117     
  Misses         44       44              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@misialq misialq left a comment

Choose a reason for hiding this comment

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

Hey @DorielaGrabocka, can you please rebase this branch on top of main? The changes here should not have anything to do with the changes from the other PR, right? It's a completely independent action.

@DorielaGrabocka DorielaGrabocka force-pushed the issue-97-feature-data-to-genome-data branch from 217e2bb to d895f92 Compare August 27, 2024 14:44
@DorielaGrabocka DorielaGrabocka force-pushed the issue-97-feature-data-to-genome-data branch from d98f67c to c72d5e9 Compare August 27, 2024 15:27
q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
q2_assembly/plugin_setup.py Outdated Show resolved Hide resolved
@DorielaGrabocka DorielaGrabocka force-pushed the issue-97-feature-data-to-genome-data branch from bc188d7 to 1bc46f6 Compare August 29, 2024 13:28
@misialq misialq changed the title ENH: Action to collate genomes to GenomeData[DNASequences] ENH: add action to collate genomes to GenomeData[DNASequences] Aug 29, 2024
misialq
misialq previously approved these changes Aug 29, 2024
Copy link
Contributor

@misialq misialq left a comment

Choose a reason for hiding this comment

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

Looks good! Before we merge, could you please just update the description in plugin_setup? (see below) - thanks! 🙏

q2_assembly/plugin_setup.py Outdated Show resolved Hide resolved
@misialq misialq dismissed their stale review August 29, 2024 14:13

More work needs to be done.

Copy link
Contributor

@misialq misialq left a comment

Choose a reason for hiding this comment

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

Hey @DorielaGrabocka, sorry to un-approve but I just had another look and realized that your tests actually need a bit more work - please see the comment below.

q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
q2_assembly/tests/test_helpers.py Outdated Show resolved Hide resolved
q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
q2_assembly/tests/test_helpers.py Outdated Show resolved Hide resolved
Copy link
Contributor

@misialq misialq left a comment

Choose a reason for hiding this comment

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

Looks good! 🏅

q2_assembly/helpers/helpers.py Outdated Show resolved Hide resolved
@misialq misialq merged commit 2dbb69f into bokulich-lab:main Sep 20, 2024
9 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.

ENH: add action to combine FeatureData[Sequence] artifacts into one GenomeData[DNASequence] artifact
3 participants