FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add "svg-compressed" to create a simple classic QR Code with compressed file size. Allow execution from a development environment. by johnkw · Pull Request #342 · lincolnloop/python-qrcode · GitHub

Add "svg-compressed" to create a simple classic QR Code with compressed file size. Allow execution from a development environment. - #342

Merged
bartTC merged 12 commits into
lincolnloop:mainfrom
johnkw:main
Jul 23, 2025
Merged

Add "svg-compressed" to create a simple classic QR Code with compressed file size. Allow execution from a development environment.#342
bartTC merged 12 commits into
lincolnloop:mainfrom
johnkw:main

Conversation

johnkw commented Sep 10, 2023
edited
Loading

Copy link
Copy Markdown
Contributor

For a large code an example original svg is 1479k, svg-path is 286k, and svg-compressed is 59k.
For a small code it's 22k, 3.7k vs 0.8k.

johnkw closed this Sep 15, 2023
johnkw reopened this Sep 15, 2023
j and others added 4 commits September 16, 2023 15:09
…rder to avoid the O(n^2)-ish perf hit for splicing a python list repeatedly. Only store spliceable turns, and assert that we dealt with all of them correctly.
bartTC self-assigned this Aug 13, 2024
bartTC mentioned this pull request Aug 13, 2024
maribedran closed this Sep 27, 2024

johnkw commented Sep 27, 2024

Copy link
Copy Markdown
Contributor Author

Was this actually merged? It says merged here but I don't see it in the commit.

Copy link
Copy Markdown
Contributor

I'm so sorry, @johnkw ! I saw this PR listed on #370 and didn't check if all the changes were incorporated there. The PNG import was fixed, but not the SVG classes. I'll reopen the PR.

maribedran reopened this Sep 28, 2024
johnkw changed the title Only import png if needed. Add "svg-compressed" to create a simple classic QR Code with compressed file size. Add "svg-compressed" to create a simple classic QR Code with compressed file size. Allow execution from a development environment. Sep 28, 2024

johnkw commented Sep 28, 2024

Copy link
Copy Markdown
Contributor Author

Re-merged and complied with ruff and apparent v3.9 compatibility requirement.

bartTC commented Jul 23, 2025

Copy link
Copy Markdown
Member

Continued the work and cleanup in #407 as I'm not able to push to this branch.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL