< gwillen> I just learned about Golomb-Rice codes from BIP 158. That's neat.
< gmaxwell> there are more general/efficient techniques, but golomb coding is at least easy to understand and simple to implement.
< gmaxwell> (and actually efficient in the unusual case that you can make your probablities just right...)
< sipa> gmaxwell: you mean range/arithmetic/rANS ?
< gwillen> the unary quotient/binary remainder scheme is very cute
< gmaxwell> right (there are another half dozen techniques that also achieve effectively the same thing)
< gwillen> I am surprised it wasn't presented in the course I had in college where we went over various clever self-delimiting number encoding schemes in an exercise
< gmaxwell> did you get told the story about how huffman codes were invented?
< gwillen> we certainly learned about huffman codes, I don't recall a story
< * sipa> did not learn any stories about huffman
< gmaxwell> fano basically tricked huffman into solving the problem by presenting it as coarsework, and not mentioning that it was considered a hard open problem.
< sipa> coarse or course?
< gmaxwell> of course.
< gwillen> is there a trick people use when reviewing to see the commits in the proper order? Do you just not use the github interface?
< gwillen> (github sorts them in a nonsense order, using 'author timestamp' instead of commit timestamp or topological order)
< sipa> gwillen: yeah, it's a pain and i don't understand why they do that
< sipa> i complained once and iirc they said they wouldn't fix it
< gmaxwell> can you use this to make a PR, get some acks on it, then add an out of order commit that appears above the acks?
< sipa> the overview page always shows things in "edit order"; if you rewrite an earlier commit it gets moved to the bottom etc
< sipa> so no
< sipa> it's only in the commit list order (including on the overview page, within groups of multiple simultaneously-pushed commits) that's sorted by author date
< jb55> gwillen: add fetch = +refs/pull/*/head:refs/pull/origin/* and fetch = +refs/pull/*/merge:refs/merge/origin/*
< jb55> to your .git/config
< sipa> ah, yes!
< sipa> that was such a help when i discovered it
< jb55> then you can git log -p refs/merge/origin/<pr>^..refs/pull/origin/<pr>
< jb55> I do this call with (magit-log) to do reviews in magit
< jb55> if you use something like notmuch I have scripts that pulls the PR number automatically out of the subject and launches a magit review: jb55/notmuch-github
< jb55> only thing that it can't do is code review inside magit, but the magit guy is working on that atm
< * jb55> isn't a fan of the github interface
< gwillen> aha, thanks jb55
< dongcarl> gwillen: Here are some more tips (including the above)! https://github.com/bitcoin/bitcoin/blob/master/doc/productivity.md#reference-prs-easily-with-refspecs
< bitcoin-git> [bitcoin] dongcarl closed pull request #11915: Qt: Reflect correct dbcache value in options panel. (master...dbcache_gui_fix) https://github.com/bitcoin/bitcoin/pull/11915
< IZooo> Hello :)
< fanquake> wumpus 15273 is trivial to merge, otherwise probably close
< fanquake> luke-jr can you have a look at #14740
< gribble> https://github.com/bitcoin/bitcoin/issues/14740 | Update `rpcbind` doc to match the manpage by sp4ke · Pull Request #14740 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] Sjors closed pull request #15320: [Do Not Merge] break < Qt5.6 compatibility for addAction to test Travis (master...2019/02/do-not-merge-qt52) https://github.com/bitcoin/bitcoin/pull/15320
< provoostenator> ^ TL&DR Travis actually catches QT > 5.5.1 syntax so we don't have to find out months after merge :-)
< bitcoin-git> [bitcoin] Sjors opened pull request #15461: [depends] update to Boost 1.66 (master...2019/02/depends_boost) https://github.com/bitcoin/bitcoin/pull/15461
< provoostenator> ^ no rush
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a83aedca1889...a094b5433242
< bitcoin-git> bitcoin/master a786c3b Douglas Roark: Slight tweak to the verify-commits script directions
< bitcoin-git> bitcoin/master a094b54 Wladimir J. van der Laan: Merge #15273: docs: Slight tweak to the verify-commits script directions
< bitcoin-git> [bitcoin] laanwj merged pull request #15273: docs: Slight tweak to the verify-commits script directions (master...tweak_verify_commit_dirs) https://github.com/bitcoin/bitcoin/pull/15273
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a094b5433242...77fcf252f65e
< bitcoin-git> bitcoin/master 53b3103 João Barbosa: refactor: Drop redundant wallet reference
< bitcoin-git> bitcoin/master 77fcf25 MarcoFalke: Merge #15458: refactor: Drop redundant wallet reference
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15458: refactor: Drop redundant wallet reference (master...2019-02-interfaces-wallet) https://github.com/bitcoin/bitcoin/pull/15458
< bitcoin-git> [bitcoin] promag opened pull request #15462: gui: Fix async open wallet call order (master...2019-02-fix-15455) https://github.com/bitcoin/bitcoin/pull/15462
< promag_> tag 0.18 ^
< decouverte> hey!
< bitcoin-git> [bitcoin] promag opened pull request #15463: rpc: Speedup getaddressesbylabel (master...2019-02-fix-15447) https://github.com/bitcoin/bitcoin/pull/15463
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/77fcf252f65e...169dced9a42b
< bitcoin-git> bitcoin/master fa9b60c MarcoFalke: Remove unused TransactionError constants
< bitcoin-git> bitcoin/master 169dced MarcoFalke: Merge #15408: Remove unused TransactionError constants
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15408: Remove unused TransactionError constants (master...Mf1902-TransactionErrorConst) https://github.com/bitcoin/bitcoin/pull/15408
< bitcoin-git> [bitcoin] promag opened pull request #15464: gui: Drop unused return values in WalletFrame (master...2019-02-walletframe) https://github.com/bitcoin/bitcoin/pull/15464
< bitcoin-git> [bitcoin] stevenroose opened pull request #15466: tests: Print remaining jobs in test_runner.py (master...remaining-jobs) https://github.com/bitcoin/bitcoin/pull/15466
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/169dced9a42b...f3f9c1de19e6
< bitcoin-git> bitcoin/master 39e20fc Daniel Kraft: Add missing #include.
< bitcoin-git> bitcoin/master f3f9c1d MarcoFalke: Merge #15435: rpc: Add missing #include
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15435: rpc: Add missing #include (master...fix-include) https://github.com/bitcoin/bitcoin/pull/15435