search

tusharmath / observable-air

Observable for the calorie conscious
MIT License
34 stars 3 forks source link

fix(marble): fix toMarble() logic #73

Closed tusharmath closed 7 years ago

tusharmath commented 7 years ago

toMarble() prefixes an additional hyphen in some cases

tusharmath commented 7 years ago

observable-air@ benchmark /home/travis/build/tusharmath/observable-air node .dist/benchmarks/run

Node: v7.2.0 V8: 5.4.500.43

┌────────────────────────┬─────────────────────┬─────────┐
│ name                   │ ops/sec             │ samples │
├────────────────────────┼─────────────────────┼─────────┤
│ file -> map -> reduce  │ 221 (±1.36%)        │ 81      │
├────────────────────────┼─────────────────────┼─────────┤
│ file -> takeN(0, n/10) │ 548 (±1.24%)        │ 83      │
├────────────────────────┼─────────────────────┼─────────┤
│ file -> scan -> reduce │ 57 (±1.45%)         │ 66      │
├────────────────────────┼─────────────────────┼─────────┤
│ tryCatch               │ 13,286,210 (±2.13%) │ 83      │
├────────────────────────┼─────────────────────┼─────────┤
│ scheduleImmediately    │ 355,854 (±1.41%)    │ 82      │
└────────────────────────┴─────────────────────┴─────────┘
coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.02%) to 91.447% when pulling b71dc242b94e281e3ef2aa2f6d51f7c89942cfb1 on marble-fix into 6bf6fbe542d9a113ca6bd996752e4b3c469bc20b on master.