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

remove deprecated scripts #285

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

JarrodBaker
Copy link
Contributor

It is confusing having so many old scripts laying around which aren't used anywhere.

This PR removes scripts which either are now part of the ETL (old scripts), or
platform-output-support (elasticsearch / clickhouse).

  • Remove elasticsearch and clickhouse loading scripts.
  • Remove schema directory.
  • Remove old scripts.

These scripts are now part of the
[platform-output-support](git@github.com:opentargets/platform-output-support.git)
repository, and should only be referenced there.
It appears to have been added by mistake as it's only history is an
unrelated commit, and it has not been updated for years.
These are a bunch of work-in-progress/proof-of-concept scripts which are
not used anywhere. Most of them were working towards the literature
pipeline.

The main exception is the target safety script which was used to
deprecate some old spreadsheets. Since we've been using the new data now
for several releases, it does not retain any value.
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