-
Notifications
You must be signed in to change notification settings - Fork 3
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
Easily output merged MTZ #179
Comments
Another option would be checkboxes for the dials.scale job, as both the unmerged and merged mtz files can be generated there too. |
This is true, but the DUI workflow kind of pushes the user towards an explicit export. Whether that is right or not is debatable... |
dagewa
added a commit
that referenced
this issue
Nov 5, 2021
Merged
dagewa
added a commit
that referenced
this issue
Nov 12, 2021
* Run dials.merge instead of dials.export when requested. Fixes #179 * Use QFileDialog to select output MTZ name. Fixes #147 and #181 * Replace output.json with manifest.json, which tracks full paths of MTZs, their types (integrated|scaled|merged) and the associated report HTML * Remove attempt to automatically set filename, which was fragile
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
DUI does not make it very easy to get a merged MTZ, so CCP4 users are having to run an Aimless "only merge" job after getting
scaled.mtz
from DUI for downstream steps. A simple solution would be to add another checkbox to the export task to perform merging if scaled intensities are selected .The text was updated successfully, but these errors were encountered: