Project

General

Profile

Actions
Copy link

Feature #662

closed

Pass all command line results from "Just Show Command" to a common function

Added by Neil Voss over 14 years ago. Updated about 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Amber Herold
Category:
Web interface
Target version:
Appion/Leginon 2.1.0
Start date:
06/15/2010
Due date:
% Done:

100%

Estimated time:

Description

Pass all command line results from "Just Show Command" to a common function. This will allow better control and use of Christopher's appion script

Related issues 5 (2 open — 3 closed)

Related to Appion - Feature #319: Determine how appion/bin programs should be usedClosedAmber Herold04/02/2010

Actions
Related to Appion - Feature #676: Instead of installing appion scripts into bin, use the appion wrapperNew06/16/2010

Actions
Related to Appion - Task #675: Document appion wrapper config in install guideNew06/16/2010

Actions
Related to Appion - Feature #692: Implement showOrSubmitCommand() for as many Appion functions as possibleClosed06/21/2010

Actions
Blocked by Appion - Feature #661: Add use_appion wrapper and path to appion wrapper to config.phpClosedChristopher Irving06/15/2010

Actions
Actions
Copy link
 #1

Updated by Neil Voss over 14 years ago

I've created an incredibly simple function to show the command or submit it to Appion.

// submit command
$errors = showOrSubmitCommand($command, $headinfo, 'partalign', $nproc);

// if error display them
if ($errors)
    createMaxLikeAlignForm($errors);
exit;

Actions
Copy link
 #2

Updated by Neil Voss over 14 years ago

I added --expId and --jobtype to python for improved job submission info.

Actions
Copy link
 #3

Updated by Neil Voss over 14 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Neil Voss to Eric Hou
Actions
Copy link
 #4

Updated by Eric Hou over 14 years ago

  • Status changed from In Code Review to In Test
  • Assignee changed from Eric Hou to Amber Herold
  • % Done changed from 0 to 100

Looks ok to me. The only thing I will change is not using if statement but switch statement.

Thanks.

Eric

Actions
Copy link
 #5

Updated by Amber Herold about 14 years ago

  • Status changed from In Test to Closed
Actions
Copy link

Also available in: Atom PDF