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

Implementation of ByteMatrixQRCode #449

Closed
wants to merge 1 commit into from

Conversation

Benjlet
Copy link

@Benjlet Benjlet commented Apr 2, 2023

Summary

This PR implements 'ByteMatrixQRCode', which allows a byte matrix representation of the generated QR Code data.

I found this to be a very portable format for writing pixels blocks to other image libraries, where these simply translate to a grid of 1s/0s. Originally I was using the ZXing library to achieve this and had to develop that conversion as a wrapper, so believe this is adding something useful.

Unit tests included and all passing for me - I ran this on .NET 6 in VS2022.

I did not update the main README in this request as this would likely be done after/if this is merged, so it can link to the Wiki.

Cheers - love the library.

This implements ByteMatrixQRCode, which allows a byte matrix representation of the generated QR Code data
@codebude
Copy link
Owner

codebude commented Apr 7, 2024

Hi @Benjlet ,

late (probably too late) reply, but doesn't this somehow duplicate the "raw data" export of QRCoder? The raw data export aims to export the "black/white" information in the most space efficient way to be shared with other applications.

See: https://github.com/codebude/QRCoder/wiki/Advanced-usage---QR-Code-raw-data-export

@Benjlet
Copy link
Author

Benjlet commented Apr 8, 2024

Hey @codebude, I didn't see that was already an option - I personally found this implementation easier to port to other libraries and thought I'd share it out. I will set this to closed but feel free to open if you think there's some value in it being shared in future - cheers.

@Benjlet Benjlet closed this Apr 8, 2024
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