GaloisInc / BESSPIN-Tool-Suite

The core tool of the BESSPIN Framework.
Other
6 stars 2 forks source link

Cyberphys/can display/UI skin #1183

Closed lolsborn closed 3 years ago

lolsborn commented 3 years ago

There are three separate states here that need to be represented and I'm not sure how to trigger state changes here. Any ideas @EthanJamesLew @podhrmic

podhrmic commented 3 years ago

@lolsborn I am leaning towards using something like RabbitMQ to call Python from Node JS - I think that rewriting all the logic we already have in Python into Js would be a lot of work. So for state changes for example, you can query some python function that will return the current state (see my comment in https://github.com/GaloisInc/BESSPIN-Tool-Suite/issues/1061 )

lolsborn commented 3 years ago

Lets talk about this when you have some time. I have some ideas. There are some ways we could do this that would eliminate the need for the IPC between the background and foreground threads including wrapping the python stuff in a webservice or websocket that would not take but a couple hours to implement.

On Sun, May 30, 2021, at 10:08 AM, Michal Podhradsky wrote:

@lolsborn https://github.com/lolsborn I am leaning towards using something like RabbitMQ https://medium.com/@HolmesLaurence/integrating-node-and-python-6b8454bfc272 to call Python from Node JS - I think that rewriting all the logic we already have in Python into Js would be a lot of work. So for state changes for example, you can query some python function that will return the current state (see my comment in #1061 https://github.com/GaloisInc/BESSPIN-Tool-Suite/issues/1061 )

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/GaloisInc/BESSPIN-Tool-Suite/pull/1183#issuecomment-851031189, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAOZBAQIP3WEJ5FTTEEORDTQJWJHANCNFSM45YZZOFA.

dmzimmerman commented 3 years ago

Has the discussion about this happened yet? If not, let me know when it is going to...