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

running with phenix 1.21 creates unstacked anomalous MTZs #61

Closed
dennisbrookner opened this issue Sep 4, 2024 · 0 comments · Fixed by #62
Closed

running with phenix 1.21 creates unstacked anomalous MTZs #61

dennisbrookner opened this issue Sep 4, 2024 · 0 comments · Fixed by #62
Labels
bug Something isn't working

Comments

@dennisbrookner
Copy link
Member

This is a bug, to be fixed urgently.

When revising the phenix 1.20 eff template to be applicable for phenix 1.21, I did not include any equivalent of the force_anomalous_to_be_equal_to flag found here:

force_anomalous_flag_to_be_equal_to = False

For some MTZ inputs (e.g. those grabbed from the PDB), this is fine. However, MTZ that come from phenix and contain anomalous information seem to revert back to unstacked format, even when you feed phenix non-anomalous structure factors. This causes an error here:

new_mtz = mtz[~mtz[F].isnull()]
because the user-supplied column no longer exists, having been replaced by the (+) and (-) versions of itself.

Potential solutions

  1. Update the phenix 1.21 eff template to include the 1.21 syntax version of the force_anomalous_to_be_equal_to flag.
  2. Detect that the MTZ is unstacked, and stack it. I recall having difficulties implementing this in the past, but it may be possible.
  3. If this issue cannot be easily resolved, drop support (either permanently or temporarily) for phenix 1.21.
@dennisbrookner dennisbrookner added the bug Something isn't working label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant