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

Re-enable HW dithering on Kindle #1034

Merged
merged 3 commits into from Jan 31, 2020
Merged

Re-enable HW dithering on Kindle #1034

merged 3 commits into from Jan 31, 2020

Conversation

NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Jan 31, 2020

Without doing something stupid this time around, like requesting monochromatic output when we really, really shouldn't ;).

c.f., #819 & #820
As well as koreader/koreader#4602


This change is Reviewable

Only comment tweaks.
This time without fucking up the waveform modes...
(The previous attempt was requesting monochromatic dithering on
partial/reagl updates, which obviously lead to terrible results).

c.f., koreader#819 & koreader#820
As well as koreader/koreader#4602
@NiLuJe
Copy link
Member Author

NiLuJe commented Jan 31, 2020

Why, yes, I did just realize that I had made a similar, but not quite the same, mistake in FBInk... :D.

Because that's highly counter-productive, as the work buffer gets
squashed to B&W *before* dithering ^^.
@NiLuJe
Copy link
Member Author

NiLuJe commented Jan 31, 2020

Oh, and also don't wreck B&W dithering (applies to Kobo Mk. 7, too).

Sidenote: q1 dithering looks like ass, though. But it's free.
Legacy Y1 dithering looks better (except on Kindle, where it's a Sierra Lite matrix, which I find doesn't work out all that well). But we're not doing that, anyway ^^.

We shouldn't actually be doing any of that (dithering @ A2/DU) in any active codepath, though, because I think I disabled A2 pans in PDF (because it looked like ass ^^).

Copy link
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems sane enough on the surface but that's all I can say about it. ;-)

@NiLuJe
Copy link
Member Author

NiLuJe commented Jan 31, 2020

Let's see if I got it right this time :D.

@NiLuJe NiLuJe merged commit a779522 into koreader:master Jan 31, 2020
NiLuJe added a commit to NiLuJe/koreader that referenced this pull request Feb 4, 2020
* Re-enable HW dithering on Kindle
  (koreader/koreader-base#1034)
* Update SQLite to 3.31.1
  (koreader/koreader-base#1035)
* reMarkable port
  (koreader/koreader-base#1023)
* zsync2
  (koreader/koreader-base#1036)
NiLuJe added a commit to koreader/koreader that referenced this pull request Feb 4, 2020
* Switch to zsync2

Requires koreader/koreader-base#1036

* Simplify FBInk syntax

The weird-ass workarounds for -s's subopts handling are
no longer needed w/ FBInk >= 1.21.0

* Update base

* Re-enable HW dithering on Kindle
  (koreader/koreader-base#1034)
* Update SQLite to 3.31.1
  (koreader/koreader-base#1035)
* reMarkable port
  (koreader/koreader-base#1023)
* zsync2
  (koreader/koreader-base#1036)

* zsync2 means we can finally have nice things

(OpenStack backed storage)

* We also no longer need that insane workaround on ARM

* And comment out @chrox's mirror, which appears to be down.

* Warn that a malformed URL will horribly blow up in fun and interesting
ways
yparitcher added a commit to yparitcher/koreader-base that referenced this pull request Feb 7, 2020
NiLuJe pushed a commit that referenced this pull request Feb 7, 2020
Much like the fancy nightmode waveforms, this appears unsupported here.

I guess we'll see if it turns out *none* of 'em actually do ^^.
mwoz123 pushed a commit to mwoz123/koreader that referenced this pull request Mar 29, 2020
* Switch to zsync2

Requires koreader/koreader-base#1036

* Simplify FBInk syntax

The weird-ass workarounds for -s's subopts handling are
no longer needed w/ FBInk >= 1.21.0

* Update base

* Re-enable HW dithering on Kindle
  (koreader/koreader-base#1034)
* Update SQLite to 3.31.1
  (koreader/koreader-base#1035)
* reMarkable port
  (koreader/koreader-base#1023)
* zsync2
  (koreader/koreader-base#1036)

* zsync2 means we can finally have nice things

(OpenStack backed storage)

* We also no longer need that insane workaround on ARM

* And comment out @chrox's mirror, which appears to be down.

* Warn that a malformed URL will horribly blow up in fun and interesting
ways
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

2 participants