Actions
Support #3505
closed[Sample pipelines] Support science team
Start date:
08/07/2014
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
Story points:
2.0
Updated by Ward Vandewege over 10 years ago
- Subject changed from Support science team to [Sample pipelines] Support science team
Updated by Peter Amstutz over 10 years ago
- Status changed from New to In Progress
Updated by Brett Smith over 10 years ago
Reviewing 3505-virtual-work-dir at 02d6330. Looks pretty good overall, just a few small comments:
- Would it make sense to organize these Crunch script utility modules, like robust_put and vwd, under a subdirectory of crunch_scripts? I feel like it would be nice if people could
ls crunch_scripts
and see at a glance a list of the scripts that ship with Arvados. Tucking the non-runnable modules into a subdirectory would make that easier. - robust_put's exception handlers use the logging module, but it isn't imported.
- As a general rule,
var is None
is more natural Python style thanvar == None
.
Thanks.
Updated by Tom Clegg over 10 years ago
- Assigned To changed from Peter Amstutz to Tom Clegg
Updated by Tom Clegg over 10 years ago
- Status changed from In Progress to Resolved
Actions