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

Uniprot #62

Open
wants to merge 61 commits into
base: dev
Choose a base branch
from
Open

Uniprot #62

wants to merge 61 commits into from

Conversation

tgurbich
Copy link
Contributor

@tgurbich tgurbich commented Oct 1, 2023

Added scripts to generate import files for Uniprot. Scripts are executed outside of the main pipeline from file_organisher.sh (to be folded into a future pipeline version). Scripts generate individual Uniprot import files for each species representative genome and one Uniprot metadata file per catalogue.

@tgurbich tgurbich requested a review from mberacochea October 1, 2023 02:24
Adjusted pytest to include the required folders in the pythonpath
Copy link
Member

@mberacochea mberacochea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff.. I think we should merge this one.

We should start migrating the UniProt, RNAcentral, and file_organizer bash scripts as subworkflows in the pipeline. They are becoming unwieldy. Or as prefect flows... not sure yet.

Another thing that is missing is running the linters, I ran the pre-commits and bumped into loads os warning and errors (we need to fix them). I would do this after the merge in another PR, and probably not now.

Cheers

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