Skip to content
This repository was archived by the owner on Mar 23, 2025. It is now read-only.

add P-touch CUBE Plus PTP710BT to list of supported devices#2

Merged
NotAFile merged 1 commit intoNotAFile:masterfrom
gabrielgrant:add-ptp710bt-to-supported-devices
Jul 26, 2020
Merged

add P-touch CUBE Plus PTP710BT to list of supported devices#2
NotAFile merged 1 commit intoNotAFile:masterfrom
gabrielgrant:add-ptp710bt-to-supported-devices

Conversation

@gabrielgrant
Copy link
Contributor

@gabrielgrant gabrielgrant commented Jul 20, 2020

the raster command reference for the PT-E550W, PT-P750W and PT-P710BT is here: https://download.brother.com/welcome/docp100064/cv_pte550wp750wp710bt_eng_raster_101.pdf

Looking through, the command set seems to be nearly identical to that of the PT-H500/PT-P700/PT-E500 whose command reference is located here: https://download.brother.com/welcome/docp000771/cv_pth500p700e500_eng_raster_110.pdf

In testing (along with my bluetooth backend in #3), it seems to mostly work. the issues i've noticed are

  1. the leading whitespace on the label is not trimmed before printing (not sure whether this is an issue with PT700 too?)
  2. There is an error at the end of printing when trying to fetch the final status. I believe this is occurring because the code assumes the printer is operating in "Concurrent printing" mode (which seems to be the default over USB), but when connected over bluetooth, it seems to operate in "Buffered printing" mode. Increasing the timeout on the serial connection seems to fix the issue, but then causes delays with the _debug_status() checks (since they seem to usually not return any data). So probably best would be to just increase the timeout for that final status check (or to fully implement the status messages and phase changes to completely support buffered printing mode, but that seems like a fair bit of work)

@NotAFile NotAFile merged commit 5208f40 into NotAFile:master Jul 26, 2020
@NotAFile
Copy link
Owner

Thanks for testing!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants