Skip to content

beaconobject: Use property name when dealing with settings/config

Valentin Valls requested to merge fix-attr-name-without-same-prop-name into master

Closes #3313 (closed)

By design the BeaconObject was expecting attr name and prop name to match.

class Foo(BeaconObject):
    foo = BeaconObject.property(name="foo")

This MR remove the limitation my patching few part of the BeaconObject. Trying not to change much of the code.

This allows to have a clean Redis setting/config API without taking care of changes of the Python implementation.

class Foo(BeaconObject):
    _not_exposed_to_the_scientist_api = BeaconObject.property(name="foo")

I was using more and more that feature in the ebs-tomo project until @muzelle found it was not properly working.

I really want it to be part of BLISS 1.10 and 1.9. I can add extra tests.

This was tested at BM05 for now (on top of BLISS 1.9).

Merge request reports