Skip to content

Predictors cleanup - #121

Open
stephaniesamm wants to merge 14 commits into
mainfrom
predictors_cleanup
Open

Predictors cleanup#121
stephaniesamm wants to merge 14 commits into
mainfrom
predictors_cleanup

Conversation

@stephaniesamm

@stephaniesamm stephaniesamm commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator
  • Every pipeline runs with uv run
  • Shared biological resources are downloaded on the first run that needs them.
  • Shared resources are reused by later pipelines and are ignored by Git.
  • Predictor-specific downloads and intermediate files stay inside each predictor’s data/ directory and are ignored.
  • Standardized prediction tsvs stay under data/predictions// and remain ignored.
  • Pipeline logs also go under data/predictions//, but logs are tracked by Git.
  • Biomart mapping stays under mirDB.
  • Harmonised CLI commands
  • Updated all relevant READMEs
  • Added data/common_resources/README.md that explains the reason of existence of this directory.
  • .gitignore cleanup
  • Reran all predictors

Comment thread pipelines/standardized_predictors/targetscan/utils.py Outdated
Comment thread data/predictions/microt_cnn/microt_cnn_pipeline.log Outdated
Comment thread .gitignore
@zacharopoulou

Copy link
Copy Markdown
Contributor

Important question: Is the check of "new"/"old" predictions be done? Are we ok?

@stephaniesamm

Copy link
Copy Markdown
Collaborator Author

@zacharopoulou if all looks good, we can merge and I can publish the new Zenodo version: https://zenodo.org/uploads/21671476

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