Actions
Bug #2493
closedDownload best ctf reults to emx file not returning best results
Status:
Closed
Priority:
Normal
Assignee:
Amber Herold
Category:
Web interface
Target version:
Start date:
08/28/2013
Due date:
% Done:
0%
Estimated time:
Affected Version:
Appion/Leginon 3.0.0
Show in known bugs:
No
Workaround:
Updated by Amber Herold about 11 years ago
- Status changed from Assigned to In Code Review
- Assignee changed from Amber Herold to Neil Voss
Updated by Neil Voss about 11 years ago
- Assignee changed from Neil Voss to Amber Herold
Hi Amber,
The code is good, but I prefer:
coalesce(resolution_80_percent,100)+coalesce(resolution_50_percent,100) ASC
As opposed to:
coalesce(resolution_80_percent,999) ASC,coalesce(resolution_50_percent,999) ASC
because otherwise resolution_50 does not matter that much.
Updated by Anchi Cheng about 11 years ago
I don't think using addition of the two is better. Let's say we have two results:
run | 80% res | 50% res | + |
1 | 20 | 8 | 28 |
2 | 15 | 13 | 28 |
I would take result 2 over result 1.
Updated by Neil Voss about 11 years ago
Hi Anchi,
I understand, but I am not sure if I agree. I usually don't see something like you have, I usually have something that it is more like:
run | 80% res | 50% res | + |
1 | 16 | 8 | 24 |
2 | 15 | 13 | 28 |
Updated by Amber Herold over 10 years ago
- Status changed from In Code Review to Closed
Actions