-
Notifications
You must be signed in to change notification settings - Fork 174
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
Add real-time throughput to links and custom alert in pyGUI #496
Comments
In other words, what APIs/functionalities does the PYGUI support, by utilizing the grpc client, without modifying the source code? |
The throughput option is specific to the GUI at this point. There is not an alert option by way of gRPC at the moment, may be reasonable enough to add though for a future release. An example of how that would be useful to you, would be good to know. Everything else is mostly specific to creating and configuring sessions that get rendered by the GUI, since the GUI is querying that information when joining session etc. |
Thanks @bharnden What I am trying to do right now is associated with this issue . I somehow managed to create a session, associated it with a grpc server and then started a pygui instance connecting to it. Everything works really well, so I am thinking if I can utilize the alert dialog to show some inference results from our backend it will be even sweeter (we are building a co-simulation demo so showing up an alert inside the CORE GUI will be reasonablly straightforward to explain). That's the major reason why I want to have this custom alert functionality. Speaking of this, again, if it is possible for you to add an optional argument to the core-pygui command so that we can specify the session, this will make the demo process fully automatic. |
If that is the case, how to update the GUI when the session is running? Like the change of topology or change of ip address? |
That is a bit more nuanced, I believe link additions and nodes would appear now, with changes on the develop branch (maybe it is on master). Some editing would bubble up to the GUI for nodes. For ip address changing, that would rely more on having to delete/add a link, as there is not a specific edit to modify an existing links addresses. But editing and adding links during runtime brings up a separate issue. Service that run on nodes may often generate files and run using information about the interfaces at the time of startup (traditionally, everything being setup and starting has been the expected case). Modifying things after the fact, node services do not have any way to account for this or try to restart/etc. I can look into quickly adding the alert like api by way of grpc, although these are currently called "Exceptions" internally. Although there can be an INFO Exception, which is a bit odd naming and should likely be refactored to alerts formally. I can also see how much trouble pushing an option for a session id on startup would be too. If trivial I can get into the next release which should be soon. |
added the grpc alert API and did a small sprucing up for the alert handling to denote a number for alerts and properly handle changing the color etc based on severity. these are on develop |
Howdy!
I just wonder is there a way to add the real-time throughput info to links that show up in the new pygui, by only using the python script? Also is it possible to send a custom message (from a grpc client to the pygui/or to the grpc server) to trigger an alert and show up in the alert dialog?
Thanks in advance!
The text was updated successfully, but these errors were encountered: