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

Use non-blocking action logic for first point #61

Merged

Conversation

olliewalsh
Copy link
Collaborator

The current sync/blocking action can quite often trigger battery undervoltage as low voltage condition will not abort until the first path is reached.
Also skip_area does not apply until the first point is reached.

@ClemensElflein
Copy link
Owner

This should not be affected by merging #60, (if skip_path == true we need to do the approach as normal and then it should work fine). please confirm if you have the time

The current sync/blocking action can quite often trigger battery undervoltage
as low voltage condition will not abort until the first path is reached.
Also skip_area/skip_path does not apply until the first point is reached.
@olliewalsh
Copy link
Collaborator Author

This should not be affected by merging #60, (if skip_path == true we need to do the approach as normal and then it should work fine). please confirm if you have the time

Rebased/updated. Better for skip_path to apply to the first point too, otherwise like skip_area it can be very tedious to skip through multiple paths

@ClemensElflein ClemensElflein merged commit b9d2df0 into ClemensElflein:main Oct 28, 2023
3 checks passed
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.

2 participants