29 adding a field for catalog name for the control system and a catalog registration method new impl#30
Conversation
… support (tango.pyaml.attribute_indexed). TangoCatalog handles the attr@index syntax with SPECTRUM validation in connected mode.
…e attribute_indexed.py and attribute_indexed_read_only.py, update catalog and tests.
|
Tests are failing due to the wrong PyAML version. It is not using the correct branch. |
- Use the attached TangoControlSystem tango_host when resolving Tango metadata - Keep shared TangoCatalog instances isolated through per-control-system resolvers - Add public accessors to avoid cross-object private config access - Validate attached control system type explicitly - Add tests for shared catalogs with different Tango databases and metadata ranges
Move public device reference resolution into TangoControlSystem.get_device(). Resolve string refs through the runtime catalog, construct supported Tango DeviceAccess objects from backend config models, and expose get_catalog_config(). Remove the runtime dependency on PyAML CatalogResolver by making TangoCatalog resolve keys with an explicit TangoControlSystem context. Keep Tango key parsing, indexed SPECTRUM handling, and per-control-system catalog caches in the backend. Add distinct read-only config models and cover static catalog, Tango catalog, indexed lookup, config-model construction, missing catalog, and unknown key errors in tests.
…-catalog-name-for-the-control-system-and-a-catalog-registration-method-new-impl # Conflicts: # tango/pyaml/controlsystem.py
|
@JeanLucPons, it's ready for review |
|
Did you remove |
Actually, you did. It was in the |
|
OK so i will have to update catalog in pyaml test suite as i use this to check on ESRF VA. |
|
I removed the name field: |
|
Yes, I need to update it regarding the last merge. I'm on it. |
|
Ok. |
|
It should be good now for tango-pyaml too. |
|
Next: 
|
|
DeviceAccessList is imutable: |
Forgot calls to supper()...
|
Corrected |
|
Seems OK to me |
JeanLucPons
deleted the
29-adding-a-field-for-catalog-name-for-the-control-system-and-a-catalog-registration-method-new-impl
branch
3 participants
Summary
This PR adds catalog support to the Tango PyAML bridge.
TangoControlSystemconfiguration now accepts acatalogfield.TangoCatalogresolves direct Tango attribute references, with support for both connected and disconnected modes.StaticCatalogandStaticCatalogEntryprovide explicit key-to-device mappings.Attribute(index=...), replacing the separate indexed attribute classes.Closes #29
Note
The corresponding
pyamldevelopments are available in branch187-adding-catalogs-section-in-the-pyaml-configuration-file-new-implementation(PR #234).