-
Notifications
You must be signed in to change notification settings - Fork 648
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
Enable Flow Control UI menu entry #3343
Comments
No, you cannot enable this without bigger changes in underlaying main flow structure to ensure proper user feedback. |
what is missing? |
If I remember right (This feels long time ago, beginning of last year when I proposed the main flow refactoring #2188) it is only working with autostart active, disabling autostart, main flow is not active anymore. It's still the same issue that main flow is bulky and not controlable and therefore quite limited to properly interact with it... This was one of the reasons I forked at this point. Some references:
It could be that the functionality can be provided partly, but the main benefit to trigger a start if no autostart is active is not possible. If actual reaction and functionality is sufficient for you, feel free to activate it. :-) |
Yes, the code seems still to be that way that it only works if the autostart is enabled. As a workaround to work around disabling the autostart, one can just set a very large interval. Refactoring the whole loop thing is out of scope from my side. |
Once it's on the WebUI, hopes will be raised and questions will come, so I didn't activate it at the time. As soon as I use the REST API call, I'm already deeper into the topic and usually also more experienced with software. Just my cents... |
we also just could disable the disabling option and instead let it set to max. 24h. IMO there speaks nothing against it to start the round directly at startup, even when we only want to trigger it manually. With a high enough interval there should be no disturbance to manually triggering it. |
This could be a solution |
@Slider0007 can we enable this now?
AI-on-the-edge-device/sd-card/html/index.html
Line 138 in dadf18f
It seems the REST entrypoint exists now.
(although I would prefer we return JSON instead of plain text, but that would require a general API refactoring)
The text was updated successfully, but these errors were encountered: