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

ADD argNorm version number to tsv outputs #69

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

Vedanth-Ramji
Copy link
Member

The current argNorm version number is now added to tsv outputs under the 'argNorm_version' column to match with the way the software versions of ARG annotation tools are listed

The current argNorm version number is now added to tsv outputs under the 'argNorm_version' column
@luispedro
Copy link
Member

I don't like this format. I know hAMRonizer does it like this, so it could be fine for hamronized results, but not for the argNorm ones.

I would prefer it as a comment at the top of the file.

@luispedro
Copy link
Member

Also, there needs to be documentation of the output formats

Tsv outputs will now have the argNorm version on the top as a comment, such as: "# argNorm version: 0.6.0".

Tests in test_normalizers.py now skip the first row of the golden files that contain the argNorm version. `get_golden_file` function was added to test_normalizers.py to make changing the procurement of golden files easier.
tests/test_normalizers.py Show resolved Hide resolved
@luispedro luispedro merged commit 737e414 into BigDataBiology:main Oct 8, 2024
6 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.

2 participants