-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
feat: add C implementation for math/base/special/negafibonacci
#1813
Conversation
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.
Minor changes requested, thanks for your work @gunjjoshi !
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/benchmark.native.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/benchmark.native.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
…mark/benchmark.native.js Co-authored-by: Pranav <85227306+Pranavchiku@users.noreply.github.com> Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
…ative.js Co-authored-by: Pranav <85227306+Pranavchiku@users.noreply.github.com> Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
…ain.c Co-authored-by: Pranav <85227306+Pranavchiku@users.noreply.github.com> Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
@stdlib/math/base/special/negafibonacci
math/base/special/negafibonacci
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/manifest.json
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/manifest.json
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/manifest.json
Outdated
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/c/native/benchmark.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/c/native/benchmark.c
Outdated
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
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.
LGTM. Thanks @gunjjoshi !
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/c/native/benchmark.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/benchmark/c/native/benchmark.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/math/base/special/negafibonacci/src/main.c
Outdated
Show resolved
Hide resolved
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
…ain.c Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com>
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.
Looks good to me, let's get this in!
PR-URL: stdlib-js#1813 Closes: stdlib-js#1758 Ref: stdlib-js#649 --------- Signed-off-by: GUNJ JOSHI <gunjjoshi8372@gmail.com> Signed-off-by: Pranav Goswami <goswami.4@iitj.ac.in> Co-authored-by: Pranav Goswami <goswami.4@iitj.ac.in> Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Reviewed-by: Philipp Burckhardt <pburckhardt@outlook.com> Reviewed-by: Athan Reines <kgryte@gmail.com> Reviewed-by: Pranav Goswami <goswami.4@iitj.ac.in>
Resolves #1758
Description
This PR adds the C implementation of
@stdlib/math/base/special/negafibonacci
.This pull request:
@stdlib/math/base/special/negafibonacci
.Related Issues
This pull request:
@stdlib/math/base/special/negafibonacci
#1758Questions
No.
Other
In the implementation of negafibonacci, I have used the same constant as that in
@stdlib/math/base/special/fibonacci
, i.e.,stdlib/constants/float64/max_safe_nth_fibonacci.h
, as both fibonacci and negafibonacci exist till the same constant, i.e., 78.Checklist
@stdlib-js/reviewers