Update generated code for DPF 252_daily on feat/handle_pin_aliases #2129
+67,792
−82,225
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pyansys-ci-bot
added
the
server-sync
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/handle_pin_aliases #2129 +/- ##
===========================================================
+ Coverage 86.46% 86.51% +0.04%
===========================================================
Files 90 90
Lines 10300 10300
===========================================================
+ Hits 8906 8911 +5
+ Misses 1394 1389 -5 |
PProfizi
deleted the
maint/update_code_for_252_daily_on_feat/handle_pin_aliases
branch
PProfizi
added a commit
that referenced
this pull request
Mar 4, 2025
Co-authored-by: PProfizi <[email protected]>
PProfizi
added a commit
that referenced
this pull request
Mar 4, 2025
* Expose aliases in Input, Output, and PinSpecification * Add alias to template and operator code generation * Handle retro in Specification * Expose alias in operator_specification_grpcapi.py * Try getattr in template * Handle default __getattr__ * Add a deprecation warning when using pin aliases * Fix Specification doctests * Limit changes to operators with aliases * Fix retro compatibility condition * Make operator_specification_get_pin_num_aliases retro-compatible * Remove server version check * Revert "Remove server version check" This reverts commit 143aa6e. * Add server version check * Do not remove translator.py during code_generation.py * Add default value to variable for error reporting in build.py * Add aliases as valid operator inputs in the signature * Fix indent in operator.mustache * Fix indentation in operator.mustache * Print server path for operator generation * Limit tests to mac operator * Add aliases to operator signature * Fix dict creation * Revert "Limit tests to mac operator" This reverts commit 1537c46. * update generated code (#2129) Co-authored-by: PProfizi <[email protected]> * Remove redundant code following rebase * Reactivate retro testing for test_animation.py and test_animator.py * Add deprecation warning when using an alias as operator instantiation parameter * Add test_specification.py with test_pin_alias * update generated code (#2131) Co-authored-by: PProfizi <[email protected]> * Skip test_pin_alias during retro tests --------- Co-authored-by: PyAnsys CI Bot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
An update of generated code has been triggered either manually or by an update in the dpf-standalone repository.