.NET port (work in progress) about hdrhistogram HOT 5 CLOSED

Cool!

I'd be happy to chat directly, but we'll need some way to exchange e-mails for that (without putting them here where bots can find them). Message me @giltene via twitter if you want.

Comments:

1. As you probably saw (#14), I recently deprecated HistogramData and folded it's functionality onto AbstractHistogram. For a .NET port, I'd get rid of it from the start.
2. On licensing: Since HdrHistogram is public domain via CC0, you can do whatever you want with it (including no attribution, and any license you want). But thanks for the attribution! If you wish to license it the same way I did, and keep an attribution, I'd change the header to say e.g.:

/**

• Written by Matt Warren, and released to the public domain,
• as explained at http://creativecommons.org/publicdomain/zero/1.0/
• This is a .NET port of the original Java version, which was written by
• Gil Tene as described in https://github.com/HdrHistogram/HdrHistogram
  */

In addition, I've had people ask for an OSI license, so I added a BSD style license in addition. See https://github.com/HdrHistogram/HdrHistogram/blob/master/LICENSE.txt for that (you can just add a similar file at your base directory if you also want to do this.).

And if you want, I'll be happy to add you to the HdrHistogram org and have you include this .NET port under the same directory tree, similarly to what Mike Barker did with his C port.

from hdrhistogram.

Just sent a DM.

I'll try and get the C# port comparable with the Java one in the next few weeks. I've also got to check the perf, last time I looked C# was slower on all the perf tests (even testRawRecordingSpeed), so I need to work out why. Maybe it's just a case of Java Hotspot doing a better job when the code is jitted.

Once I've got the code in decent shape, I'll submit a pull-request that incorporates it into the main HdrHistorgram branch, thanks for the offer.

from hdrhistogram.

The Java v.2.0.1 is now up and ready for porting to C# ;-)

from hdrhistogram.

While this topic marked as work in progress, I'll post here.
In AbstractHistogram.cs incorrect exception message:
"numberOfSignificantValueDigits must be between 0 and 6"
should be
"numberOfSignificantValueDigits must be between 0 and 5"

Thanks for a project!

from hdrhistogram.

This work has been moved to https://github.com/HdrHistogram/HdrHistogram. I think this can be closed now.

from hdrhistogram.