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

Fix active design selection behavior for Emit designs #4587

Merged
merged 3 commits into from
Apr 26, 2024

Conversation

myoung301
Copy link
Contributor

If the project and design names are not specified, the active project and active design are supposed to be used. However, a new project is always being created when the Emit() object is created.

If the project and design names are not specified, the active project and active design are supposed to be used. However, a new project is always being created when the Emit() object is created.
@myoung301 myoung301 requested a review from jsalant22 as a code owner April 24, 2024 16:45
@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

@codecov-commenter
Copy link

codecov-commenter commented Apr 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.53%. Comparing base (87739a9) to head (f89c709).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4587      +/-   ##
==========================================
+ Coverage   81.52%   81.53%   +0.01%     
==========================================
  Files         110      110              
  Lines       53765    53772       +7     
==========================================
+ Hits        43831    43843      +12     
+ Misses       9934     9929       -5     

jsalant22
jsalant22 previously approved these changes Apr 24, 2024
@SMoraisAnsys
Copy link
Collaborator

@myoung301 Thanks for the contribution ! Could you also update the tests such that your modification passes our CI ?
Seems like modifying the behavior of automatically creating project was leveraged in some tests.

Copy link
Collaborator

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for updating the tests

@myoung301 myoung301 merged commit feb9369 into main Apr 26, 2024
15 checks passed
@myoung301 myoung301 deleted the myoung301-fix-emit-use-active-project branch April 26, 2024 17:44
myoung301 added a commit that referenced this pull request May 1, 2024
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.

5 participants