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.

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

FEAT: Report configuration refactor #5168

Merged
merged 70 commits into from
Sep 26, 2024
Merged

Conversation

maxcapodi78
Copy link
Collaborator

No description provided.

maxcapodi78 added 4 commits September 12, 2024 18:22
moved all report files to new post folder
added export_config method to reports
moved all report files to new post folder
added export_config method to reports
@ansys-reviewer-bot
Copy link
Contributor

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

@ansys-reviewer-bot review

@MaxJPRey
Copy link
Collaborator

Nice and impressive work @maxcapodi78 !

@github-actions github-actions bot added the examples Anything related to the examples label Sep 13, 2024
Copy link

codecov bot commented Sep 13, 2024

Codecov Report

Attention: Patch coverage is 86.41528% with 562 lines in your changes missing coverage. Please review.

Project coverage is 83.35%. Comparing base (8161171) to head (1dbb6b0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5168      +/-   ##
==========================================
- Coverage   83.51%   83.35%   -0.16%     
==========================================
  Files         129      142      +13     
  Lines       57728    58182     +454     
==========================================
+ Hits        48212    48500     +288     
- Misses       9516     9682     +166     

@maxcapodi78 maxcapodi78 changed the title Report configuration refactor REFACT: Report configuration refactor Sep 13, 2024
@maxcapodi78 maxcapodi78 changed the title REFACT: Report configuration refactor FEAT: Report configuration refactor Sep 13, 2024
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 18, 2024
@gmalinve gmalinve linked an issue Sep 25, 2024 that may be closed by this pull request
@Samuelopez-ansys Samuelopez-ansys marked this pull request as ready for review September 26, 2024 11:46
@maxcapodi78 maxcapodi78 merged commit 4404594 into main Sep 26, 2024
42 of 43 checks passed
@maxcapodi78 maxcapodi78 deleted the report_configuration_refactor branch September 26, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples Anything related to the examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOCS: fix advanced fields calc docs
5 participants