Feature #3833
closed
Change myamiweb form default for makeDEAlignedSum.py avoid error
Added by Anchi Cheng almost 9 years ago.
Updated almost 7 years ago.
Description
Traceback (most recent call last):
File "/opt/myamisnap/bin/makeDEAlignedSum.py", line 483, in
makeSum.run()
File "/opt/myamisnap/lib/appionlib/appionPBS.py", line 154, in run
command=self.generateCommand(targetdict)
TypeError: generateCommand() takes exactly 3 arguments (2 given)
- Project changed from 138 to Appion
- Status changed from Assigned to In Code Review
- Assignee changed from Anchi Cheng to Scott Stagg
- Affected Version set to Appion/Leginon 3.2
r19432 fix it.
Scott, do you still use non-queuing method ? We don't. Maybe default should be changed if neither of us use that. Please review the change.
- Status changed from In Code Review to Assigned
- Priority changed from Normal to Immediate
We got more error after that. It is just buggy without using queue.
NYSBC defaults different from the current:
File handling: link
Border: 60
Use Queue: True
Number of Jobs: 4
Queue ppn: 4
Queue memory 4
I really would like to Use Queue to be default. The border is also very important for us. I think it is true for all DE20 that the corner is no good.
Yes, running without the queue is terribly buggy. I would like to eventually devote some time to it. It is fine with me to have Use Queue be the default. I agree that the border should be masked, but the value varies from camera to camera. I don't know how to make a proper default for it.
What is the border mask at your end ? If we put a default that work for both ends, it would be ok.
- Tracker changed from Bug to Feature
- Subject changed from makeDEAlignedSum.py error when queue is not used to Change myamiweb form default for makeDEAlignedSum.py avoid error
- Status changed from Assigned to In Code Review
r19483 changes the defaults and adds validation for some obvious mistakes.
r19484 clean up buildCommand
Scott, please check if these defaults are acceptable.
Thanks.
- Status changed from In Code Review to Closed
Also available in: Atom
PDF