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

Pepteditor: Helix function is warping side chains. #282

Open
electronicsbyjulie opened this issue Jun 24, 2023 · 2 comments
Open

Pepteditor: Helix function is warping side chains. #282

electronicsbyjulie opened this issue Jun 24, 2023 · 2 comments
Labels
high priority Above average priority.

Comments

@electronicsbyjulie
Copy link
Contributor

From this script:

GEN "ARNDCEQGHIKLMFPSTWYV"
HELIX ALPHA 1 20
SAVE "test/received/all20.pdb"

all20.pdb.txt

Can't seem to attach a PDB extension, but if rename the attachment from .pdb.txt to .pdb, can see in the viewer just how much it's distorting certain side chains, e.g. arginine, leucine, all four of the aromatics...

Doesn't happen if no HELIX command.

@primaryodors primaryodors added CRITICAL Something is very wrong. on hold Waiting for something else to be working first. labels Jun 24, 2023
@primaryodors
Copy link
Owner

Since our predictions use ready-made PDBs, and therefore do not involve creating helices, this issue is not urgent.

@primaryodors primaryodors added high priority Above average priority. and removed CRITICAL Something is very wrong. on hold Waiting for something else to be working first. labels Jun 24, 2023
@primaryodors primaryodors linked a pull request Jun 29, 2023 that will close this issue
@electronicsbyjulie electronicsbyjulie linked a pull request Jul 12, 2023 that will close this issue
@electronicsbyjulie
Copy link
Contributor Author

It is happening even without helixing them.

This was unlinked from pull requests Jul 12, 2023
primaryodors added a commit that referenced this issue Jul 13, 2023
* Fixed an unrelated segfault that was an obstacle.

* Fix for #282 before work can be done on #287.

* Upgrade of Point class to double precision.

* Test file for coplanar pi hydrogenation.

* Pepteditor script for coplanar pi hydrogen test.

* Test of coplanar pi atoms after hydrogenation.

* Updated tests affected by the Point precision.

* Fixed array deletion before array usage.

* Observe pi coplanarity when hydrogenating.

* Updates to tests.

* Retrying...

* Fixed infinite loop.

* Temporarily increasing limit. See #288.

* Unit test.

* Unit test enhancements.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Above average priority.
Projects
None yet
Development

No branches or pull requests

2 participants