search

emanca / wproperties-analysis

Git repo gathering all submodules in the w properties analysis in CMS
0 stars 6 forks source link

Review python argparser options in each config #60

Open sroychow opened 3 years ago

sroychow commented 3 years ago

Several configs have wrong argument type or wrong default values. Along with it, short options should be one character. Should be reviewed and fixed. E.g.

parser.add_argument('-r', '--plotter',type=int, default=False, help="run the prefit plotter, the result are saved in outputDir/plot/") parser.add_argument('-sb', '--SBana',type=int, default=False, help="run also on the sideband (clousure test)")

Minor issue. I will take care of this.

emanca commented 3 years ago

along with that, I was thinking of adding the date and time to the output folder, so that we don't risk to overwrite. What do you think?

sroychow commented 3 years ago

@emanca agreed.