Feature #2970
closedShow higher resolution in ctf estimation displays
0%
Description
Higher resolution data needs fits shown to higher resolution
Updated by Anchi Cheng about 10 years ago
- Tracker changed from Task to Feature
- Status changed from New to In Code Review
- Assignee changed from Anchi Cheng to Neil Voss
Neil moved the 1D-plot resolution limit from 5.0 A to 3.0 A in r18376
r18659 makes 1D and 2D limit defined at the object initialization so that it is easier to find and modify
r18660 change 2D limit to 5 A from 7.7 A
r18661 fixes the bug that occurs when goodpixellimit radius is larger than half image length
Updated by Neil Voss about 10 years ago
- Assignee changed from Neil Voss to Anchi Cheng
Hi Anchi, I had to make a fix, you missed missed two instances that also needed to be changed, so I just removed the old variables 'outerresolution' and 'outerbound'. Can you check my code?
I do not understand the 'goodpixellimitradius' can you explain?
Updated by Anchi Cheng about 10 years ago
goodpixellimitradius was in the original code. I believe it was meant to extend the radius in unit of pixel to next number that is efficient for Fourier transform. I added the new code when it crashed on an image whose pixel size was larger than 1.5 A. I traced it down to this: the plot is default to 3.0 A, pixellimitradius of truncate the pixel radius in the Fourier space to Nyquist. However, getNextEvenPrime functions returns a value larger than Nyquist. Since there was no more check on the requirement that the variable goodpixellimitradius need to be less than or equal to Nyquist radius, the program failed.
Since I don't know what the nextEvenPrime is supposed to be, I wrote the while loop to reduce it incrementally until it is in good range.
Updated by Anchi Cheng about 10 years ago
- Assignee changed from Anchi Cheng to Neil Voss
Your code is good.
Updated by Anchi Cheng about 9 years ago
- Project changed from Leginon to Appion
- Status changed from In Code Review to Closed