These are chat archives for HdrHistogram/HdrHistogram

23rd
Feb 2017
Alexandre Victoor
@Alex_Victoor_twitter
Feb 23 2017 09:12
@giltene I have just tried to migrate my HdrHistogramJS repo. I got a strange error: "You don’t have the permission to create repositories on HdrHistogram". Indeed If I try to create a new repository in the HdrHistogram org, I get "insufficient permission"
Alexandre Victoor
@Alex_Victoor_twitter
Feb 23 2017 21:43
@giltene If you go to the organization settings page, on https://github.com/organizations/HdrHistogram/settings/member_privileges , there is maybe a "Allow members to create repositories for this organization" checkbox that need to be checked. I do not understand why other members succeed creating repos in the past though
Marshall Pierce
@marshallpierce
Feb 23 2017 21:55
@giltene I'll check with @jonhoo but I think it would be ideal to get the rust version in the HdrHistogram org. I'm continuing to work on it, porting stuff from my old implementation to this one. Normalization is next, then serialization.
(specifically, https://github.com/jonhoo/hdrsample is where rust effort is going)
Marshall Pierce
@marshallpierce
Feb 23 2017 22:03
@Alex_Victoor_twitter also really looking forward to using the JS port. :) it will make it a lot easier to have historical views of benchmark data.
Alexandre Victoor
@Alex_Victoor_twitter
Feb 23 2017 22:06
@marshallpierce glad to hear it :smile:
Marshall Pierce
@marshallpierce
Feb 23 2017 22:11
how unpleasant was managing JS's "unusual" numeric type?
Alexandre Victoor
@Alex_Victoor_twitter
Feb 23 2017 22:14
Well, quite unpleasant when I figure it out that js bitwise operator only works on bits...
Math.pow(2, 31) << 1 === 0
Marshall Pierce
@marshallpierce
Feb 23 2017 22:14
Doh.
Maybe in a few months when Rust's WebAssembly backend is further along it might be interesting to see if that behaves better.
I have no idea if WebAssembly has richer numeric types, though.
Alexandre Victoor
@Alex_Victoor_twitter
Feb 23 2017 22:15
There might be 64 bits integers in future JS versions as well
Marshall Pierce
@marshallpierce
Feb 23 2017 22:16
ah, good
Jon Gjengset
@jonhoo
Feb 23 2017 23:06
Yeah, moving the Rust version into the HdrHistogram org seems like a good idea. We might want to hold off until normalization and serialization lands, since those are pretty core features that I think people would expect from an "official" port. @marshallpierce has been doing some great work, so I think he should also be made an admin of the Rust port when it is adopted.