-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some constants don't match break_infinity.js documentation #171
Comments
For the methods, I wrote the documentation in a way that I think is more informative about the functions than the documentation you gave here. Fair point about the constants, though - I'll add documentation for those. ...I feel like I missed the point of what you're saying. Did you just mean "the documentation isn't worded the same", or were you trying to say something else? |
Well, seems like I missed you updating the ReadMe to mention index.d.ts so my point is now partially lost, sorry! And by the way, if my past point or coming comments sounds like criticism then sorry, it really is not meant that way, it's a great library. Still:
So apropos point 2 I thought perhaps that list could be provided for the readme, which is why I gave the list. I figured either it could be used in some simple list form, or as a basis for transferring the break_infinity.js docs. As it turns out, you have done something with index.d.ts so it's not so important, though maybe a simpler 'readme' form would be good in addition to the index? Hope that helps. |
* Added penta_log and linear_penta_root (the two inverses of pentation) * Added the "floored" parameter to mod (when this parameter is true, it performs floored modulo instead of truncated division modulo) * Fixed several issues ( #166, #167, #170, and #171 ); #170 in particular is a new feature, that being constants for the maximum value of a Decimal.
v2.1.0 has added documentation for the constants.
I'm still not sure if I actually covered your whole issue, so I'm not going to close it myself. Feel free to close the issue if the documentation for the constants is sufficient for what you were looking for. |
Yeah, I took it to mean 'like-for-like' replacement where the methods, constants etc. match except in the few cases specified as not matching. Anyway, there's documentation, I'm a happy bunny (and sorry for missing the update before I posted). Calling it closed, have a great one! |
You didn't "miss the update before you posted", I just released the update an hour ago. |
No, I meant not seeing the change to the ReadMe where you added a reference to index.d.ts, thus giving info on where we could find the methods, before I posted the original issue. I read it a few months back, when it just mentioned the ReadMe and not index.d.ts, and I never spotted the change. |
Some of the constants don't match the break_infinity.js documentation; I'm not sure about the methods.
I produced an automated list, I have not manually checked this, but it might be a useful start for any additional documentation.
Here are the constants:
Here is the list of functions, with their parameters and brief explanations:
The text was updated successfully, but these errors were encountered: