Small Fix in Docs #51

Merged
merged 1 commit into from Oct 6, 2012

2 participants

@hermanjunge

Changes routes for test and benchmarking.

Maybe, we have to change the repos for benchmarking. It seems that uuid is not maintained anymore. Although I could npm install 'ed it. I couldn't make it work.

Correct me in the last point if I'm wrong, of course.

Herman Junge Small Fix in Docs 7ea6fe0
@broofa
Owner

Thanks for the pull request.

Can you provide more info about what problems you're having? The tests and benchmarks are working for me:

rkieffer@rkieffer-mbp$ pwd
/Users/rkieffer/projects/node-uuid

rkieffer@rkieffer-mbp$ node test/test.js
Pass: uuids with current time have expected order
Pass: uuids with time option have expected order
Pass: IDs created at same msec are different
Pass: Exception thrown when > 10K ids created in 1 ms
Pass: Clock regression by msec increments the clockseq
Pass: Clock regression by nsec increments the clockseq
Pass: Explicit options produce expected id
Pass: Ids spanning 1ms boundary are 100ns apart
Pass: Short parse
Pass: Dirty parse

Sanity check 10000 v1 uuids

Sanity check 10000 v4 uuids

Checking v4 randomness.  Distribution of Hex Digits (% deviation from ideal)
Pass: 0 |================================| 18781 (0.17% < 2%)
Pass: 1 |================================| 18869 (0.63% < 2%)
Pass: 2 |================================| 18805 (0.29% < 2%)
Pass: 3 |================================| 18656 (-0.5% < 2%)
Pass: 4 |==================================================| 28853 (0.36% < 2%)
Pass: 5 |================================| 18755 (0.03% < 2%)
Pass: 6 |================================| 18828 (0.42% < 2%)
Pass: 7 |================================| 18829 (0.42% < 2%)
Pass: 8 |====================================| 21066 (-0.87% < 2%)
Pass: 9 |====================================| 21084 (-0.78% < 2%)
Pass: a |====================================| 21231 (-0.09% < 2%)
Pass: b |====================================| 21220 (-0.14% < 2%)
Pass: c |================================| 18785 (0.19% < 2%)
Pass: d |================================| 18731 (-0.1% < 2%)
Pass: e |================================| 18888 (0.74% < 2%)
Pass: f |================================| 18619 (-0.7% < 2%)

Performance testing v1 UUIDs
uuid.v1(): 588235 uuids/second
uuid.v1('binary'): 243902 uuids/second
uuid.v1('binary', buffer): 666666 uuids/second

Performance testing v4 UUIDs
uuid.v4() with mathRNG: not defined
uuid.v4() with whatwgRNG: not defined
uuid.v4() with nodeRNG: not defined
uuid.v4('binary'): 76335 uuids/second
uuid.v4('binary', buffer): 94339 uuids/second

rkieffer@rkieffer-mbp$ node benchmark/benchmark.js 
uuid require failed - skipping tests
uuid-js require failed - skipping tests
# v4
nodeuuid.v4() - using node.js crypto RNG: 113610 uuids/second
nodeuuid.v4() - using Math.random() RNG: 109553 uuids/second
nodeuuid.v4('binary'): 98580 uuids/second
nodeuuid.v4('binary', buffer): 85543 uuids/second
140byte.es_v4: 92609 uuids/second

# v1
nodeuuid.v1(): 506072 uuids/second
nodeuuid.v1('binary'): 218340 uuids/second
nodeuuid.v1('binary', buffer): 439367 uuids/second

rkieffer@rkieffer-mbp$ 
@hermanjunge

Hi. The library is working like a charm. I'm using it to generate keys for some tests I'm running in RIAK.

I just was updating the information in the readme.md file :D

  • Test is in test/test.js

  • Benchmark is in benchmark/benchmark.js

And, of course, thanks a lot for this module!!! (Now I'm looking for something similar for C, but that's another story).

@broofa broofa merged commit 7066c63 into broofa:master Oct 6, 2012
@broofa
Owner

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment