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

A few improvements #32

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

A few improvements #32

wants to merge 4 commits into from

Conversation

TheZoc
Copy link

@TheZoc TheZoc commented Jul 26, 2024

Hi @AndBondStyle :)

Here are the changes I made a few months ago.
I'm happy to subdivide them into smaller PRs if you prefer, but I'm unable to do substantial changes due to time constraints on my side.

These were tested on my Niimbot B1 and seem to work well :)

Changes for this PR:

  • Fixes Wait for finish to send end_print #12, where printer stickers are not printing completely
  • Added USB Serial Port auto-detection of model
  • Made a small change on include paths, so it doesn't require PYTHONDIR (or Poetry) to run. I'm happily running this in a local venv
  • I upgraded pillow to 10.3 - I remember I ran into an issue with the old one, but I can't recall which one now.

Please let me know if you want me to change something before merging

TheZoc added 4 commits April 22, 2024 22:16
- Added printer_models.py that defines the properties of each supported printer
- Added basic error checking before attempting to print (Open paper compartment and busy)
- Unpacking on get_print_status() now works. Tagged unknown values for the time being.
- Added two unknown values to InfoEnum
@MultiMote
Copy link

MultiMote commented Aug 2, 2024

You can detect model by sending GetInfo packet.
See this.

Also checked port serial numbers of my devices so your solution will not work:

  • B1 - B1-G32707118
  • D110 - 00000000050C

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