Skip to content

Updated build of tango dummy-cs#72

Merged
JeanLucPons merged 1 commit intomainfrom
fix-dummy-cs-name-conflicts
Nov 17, 2025
Merged

Updated build of tango dummy-cs#72
JeanLucPons merged 1 commit intomainfrom
fix-dummy-cs-name-conflicts

Conversation

@JeanLucPons
Copy link
Copy Markdown
Contributor

This PR solves issue when switching between dummy-cs and true tango cs.

@gubaidulinvadim
Copy link
Copy Markdown
Contributor

I actually fail to reproduce the issue either here or on the main branch. @TeresiaOlsson, could you check that there's no conflict anymore with tango?

@JeanLucPons
Copy link
Copy Markdown
Contributor Author

JeanLucPons commented Nov 14, 2025

To reproduce the issue on the main you should:
Make a succesfull test using a true control system.
Install the dummy cs:

pip install tests/dummy_cs/tango-pyaml

then it breaks tango:

>>> import tango
>>> ds = tango.DeviceProxy("1/2/3")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'tango' has no attribute 'DeviceProxy'

To go back in a clean situation you have to uninstall all (tango-pyaml and pytango) and reinstall.

This PR prevents to break tango and to switch between tests/dummy_cs/tango-pyaml and tango-pyaml

@TeresiaOlsson
Copy link
Copy Markdown
Contributor

I actually fail to reproduce the issue either here or on the main branch. @TeresiaOlsson, could you check that there's no conflict anymore with tango?

I don't understand. What should I check? I have had no issues with dummy-cs because I never tried to run any of the tests.

@JeanLucPons
Copy link
Copy Markdown
Contributor Author

It would be nice that this patch is applied.
It is annoying when you switch from true to dummy CS to reinstall all each time.
Thanks.

@JeanLucPons JeanLucPons merged commit 45ebab2 into main Nov 17, 2025
2 checks passed
@TeresiaOlsson
Copy link
Copy Markdown
Contributor

TeresiaOlsson commented Nov 17, 2025

Should this work now? I just pull the latest version of the main branch and I have the problem.

I have the problem when trying to run the scripts in the examples folder if that makes a difference?

@gubaidulinvadim gubaidulinvadim deleted the fix-dummy-cs-name-conflicts branch November 25, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants