Skip to content

Navigation Menu

Sign in
Sign up

update for abr function - #82

Open
jaxxzer wants to merge 2 commits into
bluerobotics:master from
jaxxzer:abr
Open

update for abr function #82
jaxxzer wants to merge 2 commits into
bluerobotics:master from
jaxxzer:abr

Conversation

@jaxxzer

@jaxxzer jaxxzer commented Feb 20, 2020

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread examples/autoBaudrate.py
print("failed to initialize at %d bps", baudrate)
exit(1)
print("ok")
time.sleep(0.01)

@patrickelectric patrickelectric Feb 21, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When will this finish ?

@jaxxzer jaxxzer Feb 21, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when you push ctrl + c like ping, or the sensor fails to communicate

if baudrate is not None:
self.iodev.baudrate = baudrate
self.iodev.send_break()
self.iodev.write("U".encode("utf-8"))

@patrickelectric patrickelectric Feb 21, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that this should be ascii, since it's the U char that we want, and not the utf-8 representation.

@jaxxzer jaxxzer Feb 21, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks correct on the wire with a probe (0x55), I will try ascii though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@patrickelectric patrickelectric patrickelectric requested changes
@Williangalvani Williangalvani Awaiting requested review from Williangalvani

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /