Skip to content

Make the format of the PDB flexible so that we can read ZDOCK file#81

Closed
NicoRenaud wants to merge 6 commits into
masterfrom
iss74
Closed

Make the format of the PDB flexible so that we can read ZDOCK file#81
NicoRenaud wants to merge 6 commits into
masterfrom
iss74

Conversation

@NicoRenaud

@NicoRenaud NicoRenaud commented Aug 24, 2022

Copy link
Copy Markdown
Member

I'm not sure if that's something we really want to do, but since pdb files are not all the time in the exact format we would like them to be, this PR tries to make pdb2sql a bit more flexible so that we can read more pdb files. See #74

There is now a flexible_format arg in the constructor of pdb2sql that if turned to true allow to only read the data up to the xyz coordinates. The default value is false and will therefore crash when trying to open a non ideal pdb files

@NicoRenaud

Copy link
Copy Markdown
Member Author

We can now read zdock files (and most likely other format that complies with the PDB format up to the xyz coordinate).
I don't know if that's a feature we want nor if that would be the best way of implementing it though

@github-actions

github-actions Bot commented Sep 9, 2022

Copy link
Copy Markdown

This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the stale label Sep 9, 2022
@NicoRenaud NicoRenaud removed the stale label Sep 9, 2022
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the stale label Sep 26, 2022
@github-actions

github-actions Bot commented Oct 5, 2022

Copy link
Copy Markdown

This PR was closed because it has been inactive for 7 days since being marked as stale.

@github-actions github-actions Bot closed this Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant