Opened 15 months ago

Closed 14 months ago

Last modified 8 months ago

#27916 closed defect (fixed)

Don't print default values for arguments twice

Reported by: pastly Owned by:
Priority: Medium Milestone: sbws: 1.0.x-final
Component: Core Tor/sbws Version:
Severity: Normal Keywords:
Cc: pastly, juga, teor Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

$ sbws generate -h
[... snip ...]
  -m TORFLOW_BW_MARGIN, --torflow-bw-margin TORFLOW_BW_MARGIN
                        Cap maximum bw when scaling as Torflow. (Default:
                        0.05) (default: 0.05)
  -r TORFLOW_ROUND_DIGS, --torflow-round-digs TORFLOW_ROUND_DIGS
                        Number of most significant digits to round bw when
                        scaling as Torflow. (Default: 3) (default: 3)
  -p SECS_RECENT, --secs-recent SECS_RECENT
                        How many secs in the past are results being still
                        considered. Note this value will supersede data_period
                        in the configuration. (Default: None) (default: None)
  -a SECS_AWAY, --secs-away SECS_AWAY
                        How many secs results have to be away from each other.
                        (Default: 86400 - one day -) (default: 86400)
  -n MIN_NUM, --min-num MIN_NUM
                        Mininum number of a results to consider them.(Default:
                        2) (default: 2)

The argument parsers are ArgumentDefaultsHelpFormatter types, so the defaults are printed ... by default. No need to add them to the help strings.

Child Tickets

Change History (3)

comment:1 Changed 14 months ago by pastly

Status: newneeds_review

comment:2 Changed 14 months ago by juga

Resolution: fixed
Status: needs_reviewclosed

Merged

comment:3 Changed 8 months ago by teor

Milestone: sbws: 1.0.x-final

Moving closed sbws tickets to sbws: 1.0.x-final.

Note: See TracTickets for help on using tickets.