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

Add papyrus protein targets #336

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

phalem
Copy link
Contributor

@phalem phalem commented Jun 27, 2023

Add papyrus data 05.4_combined_set_protein_targets.tsv.gz which have a target and each information of 7k target

Add  papyrus data 05.4_combined_set_protein_targets.tsv.gz which have a target and each information of 7k target
@phalem
Copy link
Contributor Author

phalem commented Jun 27, 2023

@MicPie Can you deal with the problem related to line length etc.. ??

@phalem phalem mentioned this pull request Jun 30, 2023
@MicPie MicPie self-requested a review July 26, 2023 12:29
@MicPie
Copy link
Contributor

MicPie commented Jul 26, 2023

Hi @phalem
The line length issue you can simply disable by adding # noqa: E501 to the end of the line.

@MicPie
Copy link
Contributor

MicPie commented Jul 26, 2023

Ok, I run the pre commit hooks and fixed the too long lines.
Be sure to run this from your newly checkout main repo dir:

  1. pip install -e .[dataset_creation] to install all the needed packages for the datasets, and
  2. pre-commit install so that you have the pre commit hooks run locally when you commit changes.

@MicPie
Copy link
Contributor

MicPie commented Jul 26, 2023

Ok, I'll need to check out the other Papyrus dataset in order to fully understand it. :-)

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