Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Report and Aggregate default duration is 8 days, not 7 #354

Open
pceBuildMaster opened this issue Feb 14, 2020 · 1 comment · May be fixed by #458
Open

Report and Aggregate default duration is 8 days, not 7 #354

pceBuildMaster opened this issue Feb 14, 2020 · 1 comment · May be fixed by #458
Labels
improvement

Comments

@pceBuildMaster
Copy link

The docs state for both Report and Aggregate: By default, the time spent the last 7 days is printed.

But both put out 8 days. Example from today:

Fri 07 February 2020 -> Fri 14 February 2020

Company training - 1h 45m 38s
	[overhead  1h 45m 38s]

Aggregate clearly shows time from both Fridays is included:

Fri 07 February 2020 - 8h 00m 00s
  Full automation run on hosted Testrail - 2h 00m 00s
	[run-analysis  2h 00m 00s]

  Migrate Testlog to Testrail - 6h 00m 00s
	[coding  6h 00m 00s]


Sat 08 February 2020 - 2h 00m 00s
  Full automation run on hosted Testrail - 1h 00m 00s
	[run-analysis  1h 00m 00s]

  Full-time remote proposal - 1h 00m 00s
	[overhead  1h 00m 00s]

...

Thu 13 February 2020 - 10h 41m 50s
  Full-time remote proposal - 2h 03m 12s
	[overhead  2h 03m 12s]

  Meeting - 1h 29m 33s
	[overhead  1h 29m 33s]

  Migrate Testlog to Testrail - 6h 12m 49s
	[coding  5h 13m 14s]
	[overhead     59m 35s]

  Setup time tracking system - 56m 16s
	[overhead     56m 16s]


Fri 14 February 2020 - 2h 05m 18s
  Meeting - 35m 42s
	[overhead     35m 42s]

  Migrate Testlog to Testrail - 1h 10m 01s
	[coding  1h 10m 01s]

  Planning - 19m 35s
	[overhead     19m 35s]
@pceBuildMaster
Copy link
Author

To be clear 'the last 7 days' is not the same as '7 days ago'. The first includes today, the last does not.

Most importantly, if the default behavior for a report is to cover the work done in the last week than 'the last 7 days' is desired.

Work-around using date that only has concept of days ago:

weekend="$(date '+%Y-%m-%d')"
weekstart="$(date --date='6 days ago' '+%Y-%m-%d')"
watson report --from $weekstart --to $weekend -G | sed 's/\t/    /g'

(note watson output is mix of spaces and tabs for formatting thus the sed to convert tabs to spaces)

@kvnloughead kvnloughead linked a pull request Dec 19, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

change default lookback period for report, aggregate and log to 7 days kvnloughead/Watson
2 participants
@jmaupetit @pceBuildMaster