-
Notifications
You must be signed in to change notification settings - Fork 25
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
Pin aliases #2059
base: master
Are you sure you want to change the base?
Pin aliases #2059
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2059 +/- ##
==========================================
- Coverage 86.40% 82.84% -3.57%
==========================================
Files 90 90
Lines 10233 10248 +15
==========================================
- Hits 8842 8490 -352
- Misses 1391 1758 +367 |
fe19730
to
cb64399
Compare
cb64399
to
b0a557e
Compare
b0a557e
to
ee192a5
Compare
Here is an example of a server sync update with the current changes (plus a few others due to diff with master): |
@PProfizi This looks nice, also the operator code update. Can we add a test to ensure that we get the warning when we try to connect |
This PR exposes APIs for pin name aliases in PyDPF-Core, along with a template update for operator module generation to enable retro-compatibility of scripts using aliases, as well as warning about deprecation of said aliases.