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

Update AMRFinderPlus column name #73

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

apcamargo
Copy link

@apcamargo apcamargo commented Dec 16, 2024

In version 4.0 of AMRFinderPlus, the Accession of closest sequence column has been renamed to Closest reference accession. This PR updates the code to select the correct column.

@luispedro
Copy link
Member

Thanks @apcamargo. We probably want to support both the older and the newer formats, so @Vedanth-Ramji will work on a patch that supercedes this.

@apcamargo
Copy link
Author

Thanks!
I figured that supporting older versions was important here. But since I had this local fix in handy, it would be a waste not to at least bring arte to this

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.

2 participants