-
Notifications
You must be signed in to change notification settings - Fork 317
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
DelegateParameters inherit their source's settable/gettable/snapshot_value/unit/label #6648
base: main
Are you sure you want to change the base?
DelegateParameters inherit their source's settable/gettable/snapshot_value/unit/label #6648
Conversation
@jenshnielsen Not sure if this is a big enough issue to rethink the implementation. What are your thoughts? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6648 +/- ##
==========================================
+ Coverage 69.37% 69.38% +0.01%
==========================================
Files 340 340
Lines 31293 31310 +17
==========================================
+ Hits 21708 21725 +17
Misses 9585 9585 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
b0a2781
to
a1752ba
Compare
…s://github.com/samantha-ho/Qcodes into samanthaho/delegate_inherits_settable_gettable
This PR fixes a bug observed when chaining multiple DelegateParameters, and changing the source at the bottom of the chain.
In the previous implementation, a DelegateParameter only updated it's internal
_settable
and_gettable
attributes when thesource
was changed. But it is possible to set up a chain of DelegateParameters and then change the 'root' source Parameter. This updates the bottom-most DelegateParameter, but all of the other DelegateParameters in the chain will not update.This PR changes the implementation to override the
ParameterBase
implementation of thegettable
andsettable
properties, so that they reflect the underlyingsource
parameter, no matter what it may be.