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

Rework attributes in pathogen.json #97

Merged
merged 9 commits into from
Nov 2, 2023

Conversation

ivan-aksamentov
Copy link
Member

@ivan-aksamentov ivan-aksamentov commented Oct 23, 2023

Sibling PR in software: nextstrain/nextclade#1290

Attributes is now a simple dict of strings, anything can be in it. The non-friendly name field is removed (it's no longer in use). Our 3 remaining old attributes might be treated in a special way in UI, and the rest of attributes (which authors might add freely) will be displayed as key-value pairs in some way.

I added meta section, which contains author info: source code URL, bugreport URL, list of strings for authors (e.g. in John Doe <[email protected]> or John Doe <https://john-doe.website.org> format).

@ivan-aksamentov ivan-aksamentov temporarily deployed to refs/pull/97/merge October 23, 2023 20:33 — with GitHub Actions Inactive
@ivan-aksamentov ivan-aksamentov changed the title feat: move adenovirus dataset into author-named subdirectory Rework attributes in pathogen.json Oct 23, 2023
@ivan-aksamentov ivan-aksamentov force-pushed the feat/another-dataset-format branch from a1ab46e to 1540e2e Compare October 24, 2023 11:05
@ivan-aksamentov ivan-aksamentov temporarily deployed to refs/pull/97/merge October 24, 2023 11:05 — with GitHub Actions Inactive
@ivan-aksamentov ivan-aksamentov temporarily deployed to refs/pull/97/merge November 2, 2023 20:24 — with GitHub Actions Inactive
@ivan-aksamentov ivan-aksamentov marked this pull request as ready for review November 2, 2023 20:25
@ivan-aksamentov ivan-aksamentov temporarily deployed to refs/pull/97/merge November 2, 2023 20:43 — with GitHub Actions Inactive
@ivan-aksamentov ivan-aksamentov deployed to refs/pull/97/merge November 2, 2023 20:55 — with GitHub Actions Active
@ivan-aksamentov ivan-aksamentov merged commit 1c48275 into master Nov 2, 2023
1 check passed
@ivan-aksamentov ivan-aksamentov deleted the feat/another-dataset-format branch November 2, 2023 20:57
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