Skip to content

add options for pytest duration logging#656

Merged
pstjohn merged 2 commits into
NVIDIA-BioNeMo:mainfrom
pstjohn:pstjohn/pytest-notebook-timing
Jan 27, 2025
Merged

add options for pytest duration logging#656
pstjohn merged 2 commits into
NVIDIA-BioNeMo:mainfrom
pstjohn:pstjohn/pytest-notebook-timing

Conversation

@pstjohn
Copy link
Copy Markdown
Collaborator

@pstjohn pstjohn commented Jan 24, 2025

Adds duration logging for slow and notebook steps in the github CI

Signed-off-by: Peter St. John <pstjohn@nvidia.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.75%. Comparing base (b589fc6) to head (5b02d25).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #656   +/-   ##
=======================================
  Coverage   86.75%   86.75%           
=======================================
  Files         118      118           
  Lines        7058     7058           
=======================================
  Hits         6123     6123           
  Misses        935      935           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pstjohn pstjohn enabled auto-merge January 27, 2025 01:47
@pstjohn pstjohn added this pull request to the merge queue Jan 27, 2025
Merged via the queue into NVIDIA-BioNeMo:main with commit de83817 Jan 27, 2025
@pstjohn pstjohn deleted the pstjohn/pytest-notebook-timing branch January 27, 2025 15:33
sichu2023 pushed a commit that referenced this pull request Feb 10, 2025
Adds duration logging for slow and notebook steps in the github CI

Signed-off-by: Peter St. John <pstjohn@nvidia.com>
Signed-off-by: sichu <sichu@nvidia.com>
sichu2023 pushed a commit that referenced this pull request Feb 10, 2025
Adds duration logging for slow and notebook steps in the github CI

Signed-off-by: Peter St. John <pstjohn@nvidia.com>
Signed-off-by: sichu <sichu@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants