Project

General

Profile

Actions

Feature #2970

closed

Show higher resolution in ctf estimation displays

Added by Anchi Cheng about 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
11/05/2014
Due date:
% Done:

0%

Estimated time:

Description

Higher resolution data needs fits shown to higher resolution

Actions #1

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

Actions #2

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?

Actions #3

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.

Actions #4

Updated by Anchi Cheng about 10 years ago

  • Assignee changed from Anchi Cheng to Neil Voss

Your code is good.

Actions #5

Updated by Anchi Cheng about 9 years ago

  • Project changed from Leginon to Appion
  • Status changed from In Code Review to Closed
Actions

Also available in: Atom PDF