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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Stream reader #91

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

[WIP] Stream reader #91

wants to merge 3 commits into from

Conversation

glenn2223
Copy link

@glenn2223 glenn2223 commented Dec 13, 2022

Added stream reading functionality - hope you can help @Redth 馃榿

WIP Reasons:

  • Windows: not happy with using Skia - is there an alternative? Couldn't really find anything myself (see Improvements push)
  • IOS/MAC: unable to test myself

Sidenote - android is very slow decoding (thought it was my code, but it wasn't) - see images. Windows completes the whole process in a little over a second.
Edit: Improvements push has sped things up a little (letting Zxing choose format)

Old screengrabs
Code timer Android timer
image Android Screenshot

Implemented stream decoding

Not happy with: (WIP - for now)
- Windows: having to use `Skia`
- Android: seems slow but it's no the stream conversion
Extended the example project to use stream decoding on two new pages
@glenn2223
Copy link
Author

Hey @Redth, don't suppose you've had time to look at this?

@glenn2223
Copy link
Author

Hey @Redth, me again, nudge, nudge... wink, wink 馃榿

- Improved speed slightly on android
- Dropped requirement for `Microsoft.Maui.Graphics.Skia`
- Let `RGBLuminanceSource` do the hard work of discovering the `BitmapFormat`
@glenn2223
Copy link
Author

If anyone can test this on IOS and mac then I can remove the draft 馃榿

@glenn2223 glenn2223 marked this pull request as ready for review February 15, 2023 11:26
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.

None yet

1 participant