teobaluta / qr-linux-kernel

Library for QR encoding for OPW
Other
7 stars 2 forks source link

Explanation option #9

Open Two-Tone opened 10 years ago

Two-Tone commented 10 years ago

When the QR code prints also print out an obvious message that states what the QR code is for and that there is an android app on the play store for scanning the code to report the error.

Two-Tone commented 10 years ago

I said I'd make this issue months ago. Then I forgot. Oops

levex commented 10 years ago

Hi,

I am sure we can do that. I'll try to do it over this week.

teobaluta commented 10 years ago

No problem, thanks!

levex commented 10 years ago

The attached patch will display a message. I wonder if the message could be made better, or is there any better way to make it sure that it appears?

Patch: https://gist.github.com/levex/df3466e3c0f1de0fa5c6

Thanks, Levente Kurusa

Two-Tone commented 10 years ago

Isn't the oops contained within the QR code and compressed with zlib, making it so that you need a special app to be able to scan and report it?

levex commented 10 years ago

Yea, it's being discussed... I personally don't like having a special app required for this, but if people think that is the way we should go, then so be it.

Two-Tone commented 10 years ago

Well, if it's not heavily compressed, how are you suppose to get enough data in there to make the oops usable?

levex commented 10 years ago

I have been thinking about a QR content like this:

https://oops.kernel.org/?qr=[encoded oops here]

Choosing a correct encoding mode (i.e base64 won't work since that has chars which aren't allowed in URLs) will make it difficult but I think generality as in availability is more important.

Two-Tone commented 10 years ago

What's the point of having any of this then? The current method allows for a very easy way to get the oops data and report it. Having just some url doesn't really help as those who would take the time and effort to grab the data by hand would be the same people who don't need a QR code. It defeats the original purpose of this branch entirely.

On Fri, Jun 13, 2014 at 11:41 AM, Levente Kurusa notifications@github.com wrote:

I have been thinking about a QR content like this:

https://oops.kernel.org/?qr=

Choosing a correct encoding mode (i.e base64 won't work since that has chars which aren't allowed in URLs) will make it difficult but I think generality as in availability is more important.

— Reply to this email directly or view it on GitHub https://github.com/teobaluta/qr-linux-kernel/issues/9#issuecomment-46033054 .

levex commented 10 years ago

The QR code itself will contain the URL by which you can report the oops. Currently, the QR code in itself is meaningless as it only contains a zlib'd oops. Normal scanners can't do a thing with it. The app Teodora is creating will be the sole app that is able to decode and report it if the project remains in this state.

To add normal scanners to the party, we need a format they understand too, and a URL is quite common.