Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Great article!

I think there's a small problem with intermediate values in this code snippet:

  const maxValue = Math.max(...samplingVector)

  samplingVector = samplingVector.map((value) => {
    value = x / maxValue; // Normalize
    value = Math.pow(x, exponent);
    value = x * maxValue; // Denormalize
    return value;
  })
Replace x by value.




Just pushed a fix, should be live in a minute or two, thanks again!

Good catch, thanks! I’ll push a fix once I’m home



Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: