benjaminbolling / DynaGUI

A Dynamic Graphical User Interface package, which gives users a method to construct temporary, permanent and/or a set of GUI:s for users in a simple and fast manner combined with diagnostics tools (with advance 1D and 2D plotting methods).
GNU General Public License v3.0
6 stars 3 forks source link

General query #4

Closed pythonpanda2 closed 4 years ago

pythonpanda2 commented 4 years ago

@benjaminbolling

It seems that the package has a strong dependency on PyTango . How widely is this package used in the domain addressed in this article ? Would you envision using the DynaGUI package in application where PyTango packages are not used ?

benjaminbolling commented 4 years ago

In order to decrease the dependency of PyTango. All codes now enables user to select between the packages "Tango" (using PyTango), "EPICS" (using PyEPICS), "Finance" (using Pandas Datareader) and "Randomizer" (for testing and demonstration purposes using random values). Other packages used for obtaining values from various data polling sources can be added in later stages with ease. @pythonpanda2 : Since you opened this issue, I leave it up to you to decide if this issue is to be considered as closed or not.

benjaminbolling commented 4 years ago

As for how much PyTango is used: "The TANGO control system is a free open source device-oriented controls toolkit for controlling any kind of hardware or software and building SCADA systems" (from TANGO wikipedia page), and PyTango is a package used in Python for communicating with the TANGO control system.

EPICS (Experimental Physics and Industrial Control System) is used at some other facilities (one of which I am working at), and hence, pyEPICS is used in the Python package to communicate with this control system.

In order to demonstrate the openness of the DynaGUI package, the two additional 'data analysis' packages "Finance" and "Random" were added.

pythonpanda2 commented 4 years ago

Thanks a lot for the explanation. I can clearly see some of the explanation in requirements. So i will close the issue and add further comments in the requirements issue.