Skip to content

Adjust fields in ncbi-datasets to match current NCBI terms#7988

Merged
SaimMomin12 merged 2 commits into
galaxyproject:mainfrom
jennaj:fix-ncbi-datasets-field-names
May 17, 2026
Merged

Adjust fields in ncbi-datasets to match current NCBI terms#7988
SaimMomin12 merged 2 commits into
galaxyproject:mainfrom
jennaj:fix-ncbi-datasets-field-names

Conversation

@jennaj
Copy link
Copy Markdown
Member

@jennaj jennaj commented May 14, 2026

Per https://www.ncbi.nlm.nih.gov/datasets/docs/v2/command-line-tools/using-dataformat/genome-data-reports/

Reported at https://help.galaxyproject.org/t/wrapper-for-datasets-download-genome-force/17711/4

Confirmed at https://usegalaxy.eu/u/jenj/h/error-ncbi-datasets-genomes

Resolves error:

Error: field(s) [annotinfo-busco-ver], [assminfo-linked-assmaccession], [assminfo-linked-assmtype] not recognized.

Renames:

  • annotinfo-busco-ver -> annotinfo-busco-version
  • assminfo-linked-assmaccession -> assminfo-linked-assm-accession
  • assminfo-linked-assmtype -> assminfo-linked-assm-type

Wrapper revision bumped (@VERSION_SUFFIX@ 0 -> 1).

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

jennaj and others added 2 commits May 14, 2026 10:54
Per https://www.ncbi.nlm.nih.gov/datasets/docs/v2/command-line-tools/using-dataformat/genome-data-reports/

Resolves error:
> Error: field(s) [annotinfo-busco-ver], [assminfo-linked-assmaccession], [assminfo-linked-assmtype] not recognized.

Renames:
- annotinfo-busco-ver -> annotinfo-busco-version
- assminfo-linked-assmaccession -> assminfo-linked-assm-accession
- assminfo-linked-assmtype -> assminfo-linked-assm-type

Bump @VERSION_SUFFIX@ to 1.
Wrapper revision bumped (@VERSION_SUFFIX@ 0 → 1).
@SaimMomin12 SaimMomin12 merged commit 12bdfb2 into galaxyproject:main May 17, 2026
20 of 22 checks passed
@SaimMomin12
Copy link
Copy Markdown
Contributor

Thanks both!

@mvdbeek
Copy link
Copy Markdown
Member

mvdbeek commented May 17, 2026

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.

4 participants