a-r-j / graphein

Protein Graph Library
https://graphein.ai/
MIT License
994 stars 125 forks source link

change mmseqs pdb data path #377

Closed kierandidi closed 3 months ago

kierandidi commented 3 months ago

Reference Issues/PRs

What does this implement/fix? Explain your changes

Problem: the demo notebook as well as local tests fail when one tries to use pdb_manager for clustering with the following mmseqs2 error:

FileNotFoundError: [Errno 2] No such file or directory: 'pdb_cluster_rep_seq_id_0.3_c_0.8.fasta'

This is due to this line, where only fasta_fname is passed to mmseqs, but it should be the whole path str(self.root_dir / fasta_fname) to which the fasta file was written.

What testing did you do to verify the changes in this PR?

Locally making this change and running again allows successful clustering.

Pull Request Checklist

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud