-
Notifications
You must be signed in to change notification settings - Fork 95
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
Add test priors 2 #695
Add test priors 2 #695
Conversation
Travis will fail on PLS because of edge artefacts. Here is a difference image. PLS uses a uniform 1's anatomical image and the same randomised densitity (between 0 and 1 voxel values) as QP and RDP Codacy is still not happy for the same reasons as mentioned #519 (comment) |
I believe PLS is failing at the (max) edge because of continue statements in the directional gradient computation STIR/src/recon_buildblock/PLSPrior.cxx Lines 318 to 333 in 7b4c274
The solution to this is not obvious. The failure at the other edge (top and left) is harder to reason but I have seen similar issues with boundary conditions with RDP and QP. I have not had a chance to look through the PLS code. |
@robbietuk, what about disabling the PLS test for now, adding logcosh and merging this PR? |
Yes, I think we also expand the testing for RDP and logcosh priors to have more than one evaluation with different Ill get on this shortly, did you want it for release4.1? |
currently failing...
reduced value of epsilon
ad5ad2e
to
52f4fa8
Compare
|
This PR is to replace and expand upon that of #519.
Addresses #153