search

michitaro / healpix

An implementation of HEALPix in JavaScript / TypeScript
MIT License
13 stars 4 forks source link

Rename clamp to clip #15

Closed cdeil closed 6 years ago

cdeil commented 6 years ago

I suggest to rename the clamp utility function to clip.

Personally I didn't know the word "clamp", but "clip" I know and there's numpy.clip.

With Google I see that both "clamp" and "clip" are sometimes used, so it's just a personal preference.

@michitaro - Feel free to just close the PR if you prefer to kee "clamp".

michitaro commented 6 years ago

I just bring name "clamp" from glsl. Of course, popular words are preferred.