Project

General

Profile

Actions

Bug #599

closed

need to avoid conflict real Leginon sessions and instruments from appion image uploading ones

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

Status:
Closed
Priority:
Urgent
Assignee:
Eric Hou
Category:
Web interface
Target version:
Start date:
05/28/2010
Due date:
% Done:

100%

Estimated time:
Affected Version:
Show in known bugs:
Workaround:

Description

It will write to people's running session and change the microscope-camera pixel calibration for all Leginon user if it is input accidentally. This is too serious for me to put off for next release.


Related issues 2 (0 open2 closed)

Related to Appion - Bug #592: Image uploader should start with letter z not aClosedEric Hou05/28/2010

Actions
Related to Leginon - Feature #489: Automatically create a SimCCDCamera and SimTEM Instruments for localhost during leginon installClosedAnchi Cheng05/17/2010

Actions
Actions #1

Updated by Anchi Cheng over 14 years ago

fix it by
1. Searching for unused session name from z to a instead of Leginon's default a to z.
2. Create and force Appion users to upload images from hostname appion.

Committed to branch myami-2.0 as r14216, r14218 and to trunk as r14209 by Neil on part one and r14218 on part two

I have added the same host and TEM, camera names in dbemdata on cronus4.

Test Case 1-New installation:
1. Go through installation and create a project.
2. Select in the project to upload images to a new session.
The uploadimage.php page should show session name starting from z and only have the host as appion, TEM default is AppionTEM and Camrea default is AppionCamera

Test Case 2-Upgrade:
1. Go through upgrade wizard
2. Select an existing project to upload images to a new session.
Should see the same as case one.

Actions #2

Updated by Anchi Cheng over 14 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Anchi Cheng to Eric Hou
  • % Done changed from 0 to 100
Actions #3

Updated by Eric Hou over 14 years ago

code review is good.
I also test it on brand new database.

Actions #4

Updated by Eric Hou over 14 years ago

  • Status changed from In Code Review to In Test
Actions #5

Updated by Eric Hou over 14 years ago

  • Status changed from In Test to Closed

Also tested on upgrade.

Thanks.

Eric

Actions

Also available in: Atom PDF