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

helical_nstart option for refinement is broken #799

Closed
robertbuecker opened this issue Sep 13, 2021 · 0 comments · Fixed by #800
Closed

helical_nstart option for refinement is broken #799

robertbuecker opened this issue Sep 13, 2021 · 0 comments · Fixed by #800

Comments

@robertbuecker
Copy link
Contributor

The --helical_nstart option of relion_refine does not work in the current main version - instead of reverting the twist back to the n-start convention, it adds a multiple of 360 on top (e.g. a 2-start helix with 180.5° 1-start twist becomes 721° instead of 1° in 2-start convention). For my application - a C2_1 amyloid, where rot priors don't work out of the box - using --helical_nstart=2 hence made matters even worse. The problem results from a typo in helix.cpp (line 4425). After fixing it, it works well (at least for the C2_1 amyloid in both right-handed and left-handed config) and improves the reconstruction significantly.

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 a pull request may close this issue.

2 participants