On Wed, 28 Jun 2023, Anubhav Shelat wrote:
> rteval-cmd: fixed typo
> __init__.py, rtevalConfig: changed comments to use argparse instead of
> optparse
> ---
> rteval-cmd | 2 +-
> rteval/modules/__init__.py | 4 ++--
> rteval/rtevalConfig.py | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/rteval-cmd b/rteval-cmd
> index ae94bd98cc11..1f2ea08ef4ff 100755
> --- a/rteval-cmd
> +++ b/rteval-cmd
> @@ -183,7 +183,7 @@ def parse_options(cfg, parser, cmdargs):
> ind = cmdargs.index('--summarize')
> cmd_args = cmdargs[ind+1:]
> cmdargs = cmdargs[:ind+1]
> - # if -H/--raw-histogram is specified, add the files to be summarized to cmd_args, and add -Z to cmd_opts
> + # if -H/--raw-histogram is specified, add the files to be summarized to cmd_args, and add -H to cmd_opts
> elif (sys.argv.count('-H')+sys.argv.count('--raw-histogram')) > 0:
> try:
> ind = cmdargs.index('-H')
> diff --git a/rteval/modules/__init__.py b/rteval/modules/__init__.py
> index 253e72abf8aa..794135d277b5 100644
> --- a/rteval/modules/__init__.py
> +++ b/rteval/modules/__init__.py
> @@ -292,7 +292,7 @@ the information provided by the module"""
>
>
> def SetupModuleOptions(self, parser, config):
> - """Sets up a separate optptarse OptionGroup per module with its supported parameters"""
> + """Sets up a separate argparse ArgumentGroup per module with its supported parameters"""
>
> grparser = parser.add_argument_group("Group Options for %s modules" % self.__modtype)
> grparser.add_argument('--%s-cpulist' % self.__modtype,
> @@ -451,7 +451,7 @@ class RtEvalModules:
> return self.__modules.GetModulesList()
>
> def SetupModuleOptions(self, parser):
> - "Sets up optparse based option groups for the loaded modules"
> + "Sets up argparse based argument groups for the loaded modules"
> return self.__modules.SetupModuleOptions(parser, self._cfg)
>
> def GetNamedModuleObject(self, modname):
> diff --git a/rteval/rtevalConfig.py b/rteval/rtevalConfig.py
> index ec14a13adcd9..d0115a052324 100644
> --- a/rteval/rtevalConfig.py
> +++ b/rteval/rtevalConfig.py
> @@ -291,7 +291,7 @@ class rtevalConfig:
>
> last_sect = None
> for sk, v in sorted(vars(cmd_opts).items()):
> - # optparse key template: {sectionname}___{key}
> + # argparse key template: {sectionname}___{key}
> k = sk.split('___')
> if k[0] != last_sect:
> # If the section name changed, retrieve the section variables
> --
> 2.39.3
>
>
You forgot your SOB, please resend with the SOB
