Skip to content

pulls changes from upstream master #3

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

Open
wants to merge 25 commits into
base: pull-from-brandon-w
Choose a base branch
from

Conversation

sprintf
Copy link
Owner

@sprintf sprintf commented Nov 1, 2023

No description provided.

nyukhalov and others added 25 commits April 20, 2022 08:03
Right now __read() will stop reading at "OK" even if a ">" is still coming.
This can cause the ">" to be seen as the response to the next command, which
confuses the initialization sequence, since the initialization sequence
expects a very specific set of responses to its commands.

This changes __read() so that by default it only treats ">" as the response
terminator. When we issue the "ATLP" command to enter low-power mode, we will
use "OK" as the response terminator instead, since that's the only time we
don't expect to see a prompt.

This should fix #226 and should also fix #227.
Signed-off-by: Alistair Francis <[email protected]>
Moving to a feature branch

This reverts commit 3ecbf6f.
This reverts commit eb8679e.
Also fixed some of the doc headers for more modern versions of mkdocs
Without this fix, Python 3.13.1 yields this warning during "import odb":

    …/obd/utils.py:177: SyntaxWarning: invalid escape sequence '\C'

I'm not a Windows expert, but the correct naming convention seems to
be described here:

https://support.microsoft.com/en-us/topic/howto-specify-serial-ports-larger-than-com9-db9078a5-b7b6-bf00-240f-f749ebfd913e
Also corrected some of the package classifiers to indicate that this
python library no longer supports python 2. It didn't even with the
older 0.20.* Pint, though Pint itself now only supports python 3.9+
Uprev pint to 0.24.* for python 3.13+ support
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.

9 participants