Skip to content
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

[choreo] Remove preprocess interval guessing #695

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shueja
Copy link
Collaborator

@shueja shueja commented Sep 4, 2024

If this works we can remove part of the tauri command api as well.

@shueja shueja marked this pull request as draft September 4, 2024 16:12
@oh-yes-0-fps
Copy link
Collaborator

big fan

@bruingineer
Copy link
Collaborator

I think this works. the counts do not update on the UI now. It would be helpful if the counts still updated on the UI since that informs any manually set override counts.

@calcmogul calcmogul changed the title Remove preprocess interval guessing [choreo] Remove preprocess interval guessing Sep 29, 2024
@shueja
Copy link
Collaborator Author

shueja commented Oct 10, 2024

A way to fix this is to send the interval counts back over the IPC after they're calculated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants