Project

General

Profile

Actions

Bug #415

closed

imageloader gives wrong pixel size if someone uploaded images of different pixel size after your last try

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

Status:
Closed
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
05/03/2010
Due date:
% Done:

100%

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

Description

Ed found this bug

Actions #1

Updated by Anchi Cheng over 14 years ago

Force db insert when a pixel size calibration is not the last inserted at the given mag during the upload. This is needed because the query for pixel size of an image in the database is based on the most recent calibration relative to the image timestamp. Didn't want to insert the pixel size for every image if not changed so that not filled with db with too many calibrations.

The bug could have been avoided if people don't use the same host for uploading images of different pixel size, but I don't think we can police the users that easily.

Actions #2

Updated by Anchi Cheng over 14 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Anchi Cheng to Neil Voss

forgot to log that the svn revision is r13678

Actions #3

Updated by Anchi Cheng over 14 years ago

Anchi Cheng wrote:

forgot to log that the svn revision is r13678

Testing:
1. using the same host to upload images of different pixel sizes at the same magnification. They can either be
in the same session by using batch text file or in separate upload from different sessions or users.

Actions #4

Updated by Neil Voss over 14 years ago

  • % Done changed from 0 to 100
Actions #5

Updated by Neil Voss over 14 years ago

  • Status changed from In Code Review to In Test
  • Assignee changed from Neil Voss to Anchi Cheng

Code looks good. Assign someone to test.

Actions #6

Updated by Anchi Cheng over 14 years ago

  • Assignee changed from Anchi Cheng to Ed Brignole
Actions #7

Updated by Amber Herold over 14 years ago

  • Status changed from In Test to Closed
  • Affected Version set to Appion/Leginon 2.0.1

Tested on Cronus3

Actions

Also available in: Atom PDF