Skip to content

Conversation

@pyansys-ci-bot
Copy link
Collaborator

An update of generated code has been triggered either manually or by an update in the dpf-standalone repository.

@pyansys-ci-bot pyansys-ci-bot requested a review from a team as a code owner March 18, 2025 00:18
@pyansys-ci-bot pyansys-ci-bot added the server-sync DO NOT USE, Related to automatic synchronization with the server label Mar 18, 2025
@codecov
Copy link

codecov bot commented Mar 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.02%. Comparing base (256f7f4) to head (4a9d3e6).
Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2164      +/-   ##
==========================================
- Coverage   86.67%   84.02%   -2.65%     
==========================================
  Files          90       90              
  Lines       10303    10303              
==========================================
- Hits         8930     8657     -273     
- Misses       1373     1646     +273     

@pyansys-ci-bot pyansys-ci-bot force-pushed the maint/update_code_for_252_daily_on_master branch from 838eb3e to 4a9d3e6 Compare March 18, 2025 13:03
@PProfizi
Copy link
Contributor

We'll need to discuss removal of all the cyclic operator Python modules. Is this compatible with older servers where these are still the operators to call?

@PProfizi PProfizi merged commit a71146a into master Mar 19, 2025
42 of 46 checks passed
@PProfizi PProfizi deleted the maint/update_code_for_252_daily_on_master branch March 19, 2025 09:32
@rafacanton
Copy link
Contributor

rafacanton commented Mar 19, 2025

We'll need to discuss removal of all the cyclic operator Python modules. Is this compatible with older servers where these are still the operators to call?

Actually with your alias solution we can make them alive again and use the mapdl::rst::XXX alias there. That would be the best for backwards compatibility... And in the spec they are marked as deprecated, so...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

server-sync DO NOT USE, Related to automatic synchronization with the server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants