Skip to content

fix: remove deprecated Project Score cross-references#203

Merged
DSuveges merged 1 commit into
devfrom
fix/remove-project-scores-cross-refs
Jun 5, 2026
Merged

fix: remove deprecated Project Score cross-references#203
DSuveges merged 1 commit into
devfrom
fix/remove-project-scores-cross-refs

Conversation

@d0choa

@d0choa d0choa commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Mirror the pis and pts cleanup of the deprecated Project Score resource. No new code — this just keeps the orchestration configs aligned with the upstream changes.

  • dags/config/pis.yaml — drop the two cog.sanger.ac.uk/cmp/download/ fetches under input/target/project-scores/
  • dags/config/pts.yaml — drop the 3 pre-processing tasks (unzip essentiality matrix, csv_to_parquet essentiality matrix, csv_to_parquet gene identifier) and the 2 project_scores_* source entries on the target pyspark step

Out of scope (intentionally unaffected):

  • dags/config/clusters.yaml — Project Score v2 CRISPR evidence parser (different dataset, still maintained)
  • DepMap-backed target_gene_essentiality step

Companion PRs:

Refs opentargets/issues#4367

Test plan

  • Sanity-check yamlfmt locally (orchestration CI requires single-space before inline comments)
  • After all three PRs merge, run an end-to-end release to confirm the target step assembles without input/target/project-scores/ and dbXrefs no longer contains source: ProjectScore entries

Mirror the pis and pts cleanup: drop the Sanger Project Score fetch
tasks from pis.yaml and the matching pre-processing + source entries
from pts.yaml. The deprecated resource (cog.sanger.ac.uk/cmp/download/
gene_identifiers_latest.csv.gz + essentiality_matrices.zip) is no
longer consumed by the pts target step.

The Sanger Cell Model Passport download (model_list_latest.csv.gz,
used by the Project Score v2 CRISPR evidence parser) is untouched.

Companion PRs:
- pts: opentargets/pts#138
- pis: opentargets/pis#201

Refs opentargets/issues#4367
@d0choa d0choa marked this pull request as ready for review June 1, 2026 16:34
@d0choa d0choa requested a review from DSuveges June 1, 2026 16:34
@DSuveges DSuveges merged commit d31923a into dev Jun 5, 2026
2 checks passed
@DSuveges DSuveges deleted the fix/remove-project-scores-cross-refs branch June 5, 2026 13:35
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