gajus/approximate-now

Approximate (fast) current UNIX time.

approximate-now's Issues

API access differs from that of Date

Is it intentional for the package's API access to differ from that of Date? I initially expected the package to work as a drop-in replacement of Date.now() by exporting a function by the name of now() rather than an object with a property by the name of now.

Is this something you'd allow a PR to change? I have a branch with performance increases adding up to a little over 100%. I'd like to create a PR for it but it's based on an API refactor that would close this issue.

Optimization

There's an opportunity to optimize a majority of the calculations in the library. Namely, the assignments to maxNow:

approximate-now/src/approximateNow.ts

Line 2 in 6ea8bab

let maxNow = 50 * Math.ceil((now + 50) / 50);

let now = 50 * Math.ceil((now + 50) / 50);
let maxNow = now + 50;

---

1. t₀ := scale × ceil( x₀ ∕ scale )
   where x₀ is t

2. t₁ := scale × ceil( x₁ ∕ scale )
   where x₁ is t₀ + scale

Using the definition of t₀:

t₁ = scale × ceil( x₀ ∕ scale ) + scale
t₁ = t₀ + scale

Regarding lock files

There isn't a lock file in this repository. It appears that you don't typically use them in any of your projects. Is there a particular reason for that? I didn't want to commit a lock file without knowing first.