Project

General

Profile

Actions

Bug #1002

closed

navigator final image shift uses image shift of the parent preset plus the final image shift

Added by Anchi Cheng over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Leginon subsystem
Start date:
11/08/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
Appion/Leginon 2.0.0
Show in known bugs:
No
Workaround:

Description

See 10nov05b_00005hl_v01.mrc

The target is chosen on the hl image (504,184), the navigator did move the stage to approximately the target with 7.5e-8 m move error. However, an image shift of (1.2e-6,-1.4e-6) is applied to the en (0,0) preset,

This might be related the navigation error reported by Dmitry and Scott

Actions #1

Updated by Jim Pulokas over 13 years ago

still searching for the cause, but I am pretty sure it has something to do with navigator assuming that the current preset is the parent image preset (hl in this case). This would be true for the first target on the hl image because it was just at hl, but for all other targets, the current preset is probably en.

Actions #2

Updated by Jim Pulokas over 13 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Jim Pulokas to Anchi Cheng

r15005 adds preset change before anything else happens. That way presets manager will know what navigator has done.

Actions #3

Updated by Jim Pulokas over 13 years ago

I tested using SimTEM/SimCCDCamera, and this seems to fix the problem. Now for all en images, the image shift reported in the database only includes the amount of the final image shift done by navigator. Before the fix, the en image shift included the hl preset image shift + the final image shift done by navigator (except for the first target, which always worked)

Actions #4

Updated by Anchi Cheng over 13 years ago

  • Status changed from In Code Review to Closed

reviewed and tested with r15019 debug. This fix causes some problem with preset image alignment and other things that moved with navigator. Will open a separate issue and fix them there.

Actions #5

Updated by Anchi Cheng over 13 years ago

  • Category set to Leginon subsystem
  • Status changed from Closed to In Code Review
  • Assignee changed from Anchi Cheng to Jim Pulokas
  • Priority changed from Immediate to High
  • Target version set to Appion/Leginon 2.1.1

modify presets.py in r15023 so that cycleToScope does not try to change scope condition while navigator.moveToTarget is trying to do the same. Also use _cycleToScope instead of cycleToScope to avoid sending self.panel.presetsEvent()

The changes from this bug fix will need to be merged into 2,0 and 2.1

Actions #6

Updated by Anchi Cheng over 13 years ago

  • Subject changed from navigator final image shift does not take into account of the move to navigator final image shift uses image shift of the parent preset plus the final image shift
Actions #7

Updated by Anchi Cheng over 13 years ago

merge changes from the three revisions to branch 2.1 as r15051. Tested

Actions #8

Updated by Anchi Cheng over 13 years ago

  • Status changed from In Code Review to Closed
  • Affected Version changed from Appion/Leginon 2.1.0 to Appion/Leginon 2.0.0

r15052 merges the same changes to branch 2,0

tested.

Actions

Also available in: Atom PDF