-
Notifications
You must be signed in to change notification settings - Fork 25
Pin aliases #2059
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2059 +/- ##
==========================================
- Coverage 86.48% 86.34% -0.14%
==========================================
Files 90 90
Lines 10285 10300 +15
==========================================
- Hits 8895 8894 -1
- Misses 1390 1406 +16 |
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 |
I'd actually like to merge this without having to wait for the PR server-side which does the switch of |
720e249
to
143aa6e
Compare
cf56020
to
e95a12e
Compare
106e8c0
to
42be3fd
Compare
Co-authored-by: PProfizi <100710998+PProfizi@users.noreply.github.com>
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.