Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Print runtime for tests as suite runs #369

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
Merged
xylar merged 2 commits into from
Apr 21, 2022
Merged

Print runtime for tests as suite runs #369

xylar merged 2 commits into from
Apr 21, 2022

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Apr 21, 2022

This way you don't have to wait until the end of the suite to see how long things are taking.

closes #368

@xylar
Print runtime for tests as suite runs
dd114d8 
This way you don't have to wait until the end of the suite to
see how long things are taking.
@xylar xylar added enhancement New feature or request framework labels Apr 21, 2022
@xylar xylar requested a review from matthewhoffman April 21, 2022 17:39
@xylar xylar self-assigned this Apr 21, 2022
@xylar
Copy link
Collaborator Author

xylar commented Apr 21, 2022

Testing

I ran the nightly test suite on my laptop. I see:

ocean/baroclinic_channel/10km/default
  * step: initial_state
  * step: forward
  test execution:      SUCCESS
  test runtime:        00:01
ocean/baroclinic_channel/10km/threads_test
  * step: initial_state
  * step: 1thread
  * step: 2thread
  test execution:      SUCCESS
  test validation:     PASS
  baseline comparison: PASS
  test runtime:        00:03
ocean/baroclinic_channel/10km/decomp_test
  * step: initial_state
  * step: 4proc
  * step: 8proc
  test execution:      SUCCESS
  test validation:     PASS
  baseline comparison: PASS
  test runtime:        00:03
...

@xylar
Copy link
Collaborator Author

xylar commented Apr 21, 2022

@matthewhoffman, do you want the time in some color? It's currently black which somehow looks weird next to the green "PASS" text...

@xylar
Copy link
Collaborator Author

xylar commented Apr 21, 2022

I switched it to be in blue. Much, much better!

matthewhoffman
matthewhoffman approved these changes Apr 21, 2022
View reviewed changes
Copy link
Member

@matthewhoffman matthewhoffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great! Thanks, @xylar

@xylar xylar merged commit 9701cfa into MPAS-Dev:master Apr 21, 2022
@xylar xylar deleted the print_test_time_during_suite branch April 21, 2022 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matthewhoffman matthewhoffman matthewhoffman approved these changes

Assignees

@xylar xylar

Labels

enhancement New feature or request framework

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature request: print test runtime during execution within a suite

2 participants

@xylar @matthewhoffman
Morty Proxy This is a proxified and sanitized view of the page, visit original site.