-
-
Notifications
You must be signed in to change notification settings - Fork 834
chore: address commit comment #5879
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
base: develop
Are you sure you want to change the base?
Conversation
Hello! Thank you for your contribution to stdlib. We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:
This acknowledgment confirms that you've read the guidelines, which include:
We can't review or accept contributions without this acknowledgment. Thank you for your understanding and cooperation. We look forward to reviewing your contribution! |
/stdlib merge |
@@ -46,5 +46,5 @@ double stdlib_base_dists_uniform_mgf( const double t, const double a, const doub | |||
return 1.0; | |||
} | |||
// Case: t not equal to zero | |||
return ( stdlib_base_exp( t * b ) - stdlib_base_exp( t * a ) ) / ( t * ( b - a ) ); | |||
return stdlib_base_exp( v ) * ( stdlib_base_sinh ( u ) / u ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes in this PR are incomplete. As mentioned in #5861, you also need to update the JS implementations and adjust the test tolerances.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left initial comments.
Resolves #5861
Description
This pull request:
Related Issues
This pull request:
bc90eba
) #5861Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers