-
Notifications
You must be signed in to change notification settings - Fork 2
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
Imswitch integration #27
Comments
On the side of ImSwitch the imperative steps to make an easy integration of ExEngine, at least after my personal analysis, are the following:
After that we should be in a golden state where we can integrate the functionalities of the ExEngine and create new template classes to use in integration with the engine itself (if necessary, but it should not be critical). I could potentially skip the first steps of integrating the latest PRs but I'm scared of the mess we would be in. |
Seems like phishing. I blocked them and deleted the comments so no one accidentally clicks on it |
It sounds like a good plan! Let me know when you need my help |
To be clear, my plan was to build a singleton engine object to use inside the master controller. In the end what I imagine is that the engine and its notification API will go inside the ImSwitch framework to be used by developers for their controller implementation. It will be considered the "preferred" way, while leaving the old framework still existing. After some time I will probably make it deprecated and remove it entirely |
Yeah that makes sense. And the engine is designed as a singleton so that should be straightforward. |
@jacopoabramo @nornil @kasasxav
I'm making this issue to have a centralized place for discussion about the integration of ExEngine into Imswitch. The documentation is now live and mostly complete.
I think the first steps we discussed were:
Adding the imswitch managers as a device backend once they are split into their own package. There's now a guide on how to do this.
Building a new master controller on top of ExEngine, and using the notification API to update the GUI from hardware as needed.
I know there were some things to take care of before this could begin. When you do get to it, feel free to ping me if you have more question
The text was updated successfully, but these errors were encountered: