-
Notifications
You must be signed in to change notification settings - Fork 202
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
Refine3D starting from 1/2 maps doesn't appear to initialise references from 1/2 map 2. #1055
Comments
I think this works only when you specify @scheres, what is the intended behaviour? |
Hmm, this has been wrong for a long time. Thanks for reporting, I hope the commit above fixes this. Please re-open if it doesn't. |
Thanks for the quick fix! I can confirm that with 8514526 the initial references for half set 1 and half set 2 are different. Does this fix need back-porting to RELION-4 as the same code is present there? |
@scheres I am looking at the codes and your changes but where does "half1" in the @huwjenkins I can |
That was already done ml_optimiser_mpi.cpp line 730: |
deleted as answered by @scheres whilst I was writing the comment. |
@scheres @huwjenkins Thanks! I applied the same patch to the |
The RELION 5 (and 4) tutorial states:
However it is not clear how this works. In the
relion50_tutorial_precalculated_results.tar.gz
file the lastRefine3D
(job029) is run with:running
relion_image_handler
on the half maps from job025 show they have different average pixel values:however, the initial 1/2 maps in job029 do not:
in fact these files are identical:
most concerningly if I replace
Refine3D/job025/run_half2_class001_unfil.mrc
with a map with all 0 pixels then the refinement works fine. However if I keepRefine3D/job025/run_half2_class001_unfil.mrc
intact but instead replaceRefine3D/job025/run_half1_class001_unfil.mrc
with all zeros then the refinement starts off very badly:in this case both initial half maps are all zeros:
and in the first few iterations both the 1/2 maps look very poor. Does this suggests that the initial references are only made from the first 1/2 map?
The text was updated successfully, but these errors were encountered: