-
Notifications
You must be signed in to change notification settings - Fork 16
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
first version of new continuous_create_residuals #991
Conversation
We have a set of test for the Xmipp programs: . |
Ok |
@jamesmkrieger we have to update to nolan de golds |
How do I get the input files to do this? |
We need to upload the tmpLink/xmipp_continuous_create_residuals_01/output.xmd |
How do I get input/aFewProjections.sel and input/phantomBacteriorhodopsin.vol to generate output.xmd? |
ok, I think I found them |
I have sent you an email to your cnb address with the 3 files and also a query about the command failing to run in one directory. |
Are the test ready? can I upload the files:
|
I think everything is ready, yes. We still need to regenerate the test outputs with the revised program though. |
I have added the test outputs to this pull request. I don't know if that is a good idea or will work but I guess you can take them from here |
Is not necessary to upload the data test in the repository. I have uploaded it to nolan, the server where the test data is located. |
src/xmipp/tests/test_programs.py
Outdated
|
||
def test_case3(self): | ||
"Test output xmd and projections only" | ||
self.runCase("-i input/aFewProjections.sel -o %o/output.xmd --ref input/phantomBacteriorhodopsin.vol--oprojections %o/projections.stk", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--ref input/phantomBacteriorhodopsin.vol--oprojections
needs a blank space
--ref input/phantomBacteriorhodopsin.vol --oprojections
|
This is based on angular_continuous_assign2, but has 2 changes:
angular_continuous_assign2 already applies continuous A and B to the projection mP, not mIfilterp like I thought, so no change there, except for using mPp instead.
I can see with simulated data that the lack of low-pass filter definitely has a good effect in reducing positive log likelihoods.