Bug #1097
closedrunClusterCoran.php does not produce runname in its command
0%
Description
r15151 removed the runname variable pulled from the feature analysis. In this special case, the clustering and the feature analysis use the same runname.
Updated by Anchi Cheng about 14 years ago
- Status changed from Assigned to In Code Review
- Assignee set to Lauren Fisher
put that part back.
Updated by Lauren Fisher about 14 years ago
I see what I did wrong here...I moved that section to the function above so that it would set the default output directory, but I shouldn't have removed it from the second function because runname wasn't set anymore. However, if feature analysis and clustering need the same runname and need to be in the same directory, do we even want the output directory to be editable by the user? This is probably another one of those cases where it is too dangerous to give them that ability because if they don't know the directories need to be the same, it will cause problems. Should I reset runClusterCoran.php to the original so outdir is not an option on the page?
Updated by Lauren Fisher about 14 years ago
r15200 reset the variables under the function runClusterCoran to the original format, disabled the output directory so the user can see it, but can not edit it since it has to be the same for clustering and feature analysis, and added a note under the output directory explaining why it's disabled.
Updated by Lauren Fisher about 14 years ago
- Assignee changed from Lauren Fisher to Anchi Cheng
Updated by Anchi Cheng almost 14 years ago
- Status changed from In Code Review to Closed
code reviewed and tested