Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

Update ontology versions #543

Closed
wants to merge 2 commits into from
Closed

Conversation

Zethson
Copy link
Member

@Zethson Zethson commented Mar 8, 2024

Fixes #541

Copy link

netlify bot commented Mar 8, 2024

Deploy Preview for bionty-base-091d failed.

Name Link
🔨 Latest commit c5fea49
🔍 Latest deploy log https://app.netlify.com/sites/bionty-base-091d/deploys/65eb2203c0cc7d0008ba9624

Signed-off-by: zethson <[email protected]>
@sunnyosun
Copy link
Member

I think the CI failing here is because you didn't upload any of the new ontology files to s3:
Screenshot 2024-03-13 at 12 40 35

@sunnyosun
Copy link
Member

sunnyosun commented Mar 13, 2024

I made a guideline for the process here: laminlabs/bionty#33

@sunnyosun
Copy link
Member

sunnyosun commented Mar 13, 2024

Also could you make 10 PRs for this? Each PR should make a single update with a clear title.

Note that CellLine, NCBItaxon and ExperimentalFactor need manual curation (see guideline), so they also require a paired PR in bionty-assets.

@Zethson
Copy link
Member Author

Zethson commented Mar 13, 2024

Yaa, I remember the parquet stuff but got distracted when the CI failed an I thought that it's due to the recent changes.

I'll do so!

@Zethson Zethson closed this May 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update ontologies
2 participants