From f9bfbb51d17265d87615647de9dbb0a896c701a8 Mon Sep 17 00:00:00 2001 From: Shing Chan Date: Sun, 9 Nov 2025 12:15:23 +0000 Subject: [PATCH] refactor: change default output folder to outputs/{filename}/ - Change default --outputFolder from same directory as input file to 'outputs/' - Output files now organized in subdirectories: outputs/{inputFileName}/ - Simplify code by setting default in argparse instead of conditional logic Changes to src/accelerometer/accProcess.py: - Set default='outputs' in parser.add_argument for --outputFolder - Remove conditional check for None, create subdirectory path directly - Maintain backward compatibility: users can still override with --outputFolder flag --- src/accelerometer/accProcess.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/accelerometer/accProcess.py b/src/accelerometer/accProcess.py index cc1fa281..5587ee22 100644 --- a/src/accelerometer/accProcess.py +++ b/src/accelerometer/accProcess.py @@ -214,7 +214,7 @@ def main(): # noqa: C901 least active acceleration periods for circadian rhythm analysis (default : %(default)s)""") # optional outputs - parser.add_argument('--outputFolder', '-o', metavar='filename', default=None, + parser.add_argument('--outputFolder', '-o', metavar='filename', default='outputs', help="""folder for all of the output files (default : %(default)s)""") parser.add_argument('--verbose', metavar='True/False', default=False, type=str2bool, @@ -274,9 +274,9 @@ def main(): # noqa: C901 inputFileFolder = os.path.dirname(args.inputFile) inputFileName = os.path.basename(args.inputFile).split(".")[0] - # Set default output folder if not specified - if args.outputFolder is None: - args.outputFolder = os.path.abspath(inputFileFolder) + # Create subdirectory within output folder based on input file name + args.outputFolder = os.path.join(args.outputFolder, inputFileName) + args.outputFolder = os.path.abspath(args.outputFolder) os.makedirs(args.outputFolder, exist_ok=True)