feat: add C implementation for stats/base/dists/f/pdf#11201
feat: add C implementation for stats/base/dists/f/pdf#11201rautelaKamal wants to merge 1 commit intostdlib-js:developfrom
Conversation
Coverage Report
The above coverage report was generated for the changes in this PR. |
|
Just a quick heads up regarding the minor coverage drop flagged by Codecov: The native.js file is showing 0% function coverage because the addon.node binary correctly skips compilation during the CI coverage step, which triggers the expected opts.skip = true fallback inside test.native.js. The math logic is fully tested and verified when the native addon actually builds! Also, the drop in ibeta_power_terms.js relates to the original pure JavaScript implementations and was untouched in this PR. @Planeshifter |
|
@rautelaKamal FYI: I think you have misunderstood what we do for adding native implementation. Here we expect to add the corresponding C files for benchmark, examples and the main implementation alongside the relevant dependencies and documentations. |
|
@Neerajpathak07 Thank you for catching this and pointing me to the reference PR! I totally misunderstood the scope, I was focused solely on evaluating the N-API/Node.js boundary and completely neglected the standalone C ecosystem deliverables. I have just pushed an update that adds the benchmark/c/ and examples/c/ directories, the src/Makefile for cleanup, and the full C APIs documentation section in the README.md. Really appreciate you taking the time to explain the standard.Please let me know if there's anything else I missed! |
|
@rautelaKamal Update your commit message
to the following
|
9744905 to
197e55f
Compare
8be84d0 to
4bf98b2
Compare
Description
This pull request:
@stdlib/stats/base/dists/f/pdf).include.gypi,binding.gyp, andmanifest.jsondependencies via internal N-API mechanisms (STDLIB_MATH_BASE_NAPI_MODULE_DDD_D).NaN.test.native.jsfixtures to validate the C-port accuracy and benchmarks it.Related Issues
This pull request has no related issues.
Questions
No.
Other
This is a Proof of Concept submission towards my GSoC 2026 application regarding porting distributions and specific math functions to high-performance C native addons.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
@stdlib-js/reviewers