Skip to content

ci: adding-profiling-to-unit-tests#3676

Merged
germa89 merged 8 commits into
mainfrom
ci/adding-profiling-to-unit-tests
Jan 21, 2025
Merged

ci: adding-profiling-to-unit-tests#3676
germa89 merged 8 commits into
mainfrom
ci/adding-profiling-to-unit-tests

Conversation

@germa89

@germa89 germa89 commented Jan 17, 2025

Copy link
Copy Markdown
Collaborator

Description

This PR runs the profiler along with pytest. The profiler output is written to the directory prof which I am uploading as an artifact. Additionally, if you use the flag --profile-svg, an SVG file with the profiling information is written, like this:

image

Issue linked

NA

Checklist

@germa89 germa89 requested a review from a team as a code owner January 17, 2025 12:18
@ansys-reviewer-bot

Copy link
Copy Markdown
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions Bot added CI/CD Related with CICD, Github Actions, etc dependencies maintenance General maintenance of the repo (libraries, cicd, etc) labels Jan 17, 2025
@germa89 germa89 changed the title ci:adding-profiling-to-unit-tests ci: adding-profiling-to-unit-tests Jan 17, 2025
@codecov

codecov Bot commented Jan 17, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.09%. Comparing base (e33b5a1) to head (9d62d35).
Report is 12 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3676   +/-   ##
=======================================
  Coverage   87.09%   87.09%           
=======================================
  Files         187      187           
  Lines       14694    14695    +1     
=======================================
+ Hits        12798    12799    +1     
  Misses       1896     1896           

@germa89 germa89 self-assigned this Jan 20, 2025
@germa89

germa89 commented Jan 21, 2025

Copy link
Copy Markdown
Collaborator Author

@pyansys-ci-bot LGTM.

@pyansys-ci-bot pyansys-ci-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

LGTM

@germa89 germa89 merged commit 8be236d into main Jan 21, 2025
@germa89 germa89 deleted the ci/adding-profiling-to-unit-tests branch January 21, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related with CICD, Github Actions, etc dependencies maintenance General maintenance of the repo (libraries, cicd, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants