Skip to content

Commit

Permalink
refactor: update stats/base/dnanmean native addon from C++ to C
Browse files Browse the repository at this point in the history
PR-URL: #4112
Reviewed-by: Athan Reines <kgryte@gmail.com>
  • Loading branch information
Neerajpathak07 authored Dec 25, 2024
1 parent e473425 commit 53da41e
Show file tree
Hide file tree
Showing 4 changed files with 82 additions and 118 deletions.
2 changes: 1 addition & 1 deletion lib/node_modules/@stdlib/stats/base/dnanmean/include.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@

# Source files:
'src_files': [
'<(src_dir)/addon.cpp',
'<(src_dir)/addon.c',
'<!@(node -e "var arr = require(\'@stdlib/utils/library-manifest\')(\'./manifest.json\',{},{\'basedir\':process.cwd(),\'paths\':\'posix\'}).src; for ( var i = 0; i < arr.length; i++ ) { console.log( arr[ i ] ); }")',
],

Expand Down
38 changes: 38 additions & 0 deletions lib/node_modules/@stdlib/stats/base/dnanmean/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,44 @@
],
"confs": [
{
"task": "build",
"src": [
"./src/dnanmean.c"
],
"include": [
"./include"
],
"libraries": [
"-lm"
],
"libpath": [],
"dependencies": [
"@stdlib/stats/base/dnanmeanpn",
"@stdlib/napi/export",
"@stdlib/napi/argv",
"@stdlib/napi/argv-int64",
"@stdlib/napi/argv-strided-float64array",
"@stdlib/napi/create-double"
]
},
{
"task": "benchmark",
"src": [
"./src/dnanmean.c"
],
"include": [
"./include"
],
"libraries": [
"-lm"
],
"libpath": [],
"dependencies": [
"@stdlib/stats/base/dnanmeanpn"
]
},
{
"task": "examples",
"src": [
"./src/dnanmean.c"
],
Expand Down
43 changes: 43 additions & 0 deletions lib/node_modules/@stdlib/stats/base/dnanmean/src/addon.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
/**
* @license Apache-2.0
*
* Copyright (c) 2024 The Stdlib Authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

#include "stdlib/stats/base/dnanmean.h"
#include "stdlib/napi/argv.h"
#include "stdlib/napi/argv_int64.h"
#include "stdlib/napi/argv_strided_float64array.h"
#include "stdlib/napi/create_double.h"
#include "stdlib/napi/export.h"
#include <node_api.h>

/**
* Receives JavaScript callback invocation data.
*
* @param env environment under which the function is invoked
* @param info callback data
* @return Node-API value
*/
static napi_value addon( napi_env env, napi_callback_info info ) {
STDLIB_NAPI_ARGV( env, info, argv, argc, 3 );
STDLIB_NAPI_ARGV_INT64( env, N, argv, 0 );
STDLIB_NAPI_ARGV_INT64( env, stride, argv, 2 );
STDLIB_NAPI_ARGV_STRIDED_FLOAT64ARRAY( env, X, N, stride, argv, 1 );
STDLIB_NAPI_CREATE_DOUBLE( env, stdlib_strided_dnanmean( N, X, stride ), v );
return v;
}

STDLIB_NAPI_MODULE_EXPORT_FCN( addon )
117 changes: 0 additions & 117 deletions lib/node_modules/@stdlib/stats/base/dnanmean/src/addon.cpp

This file was deleted.

1 comment on commit 53da41e

@stdlib-bot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage Report

Package Statements Branches Functions Lines
stats/base/dnanmean $\color{green}359/359$
$\color{green}+100.00\%$
$\color{green}14/14$
$\color{green}+100.00\%$
$\color{green}4/4$
$\color{green}+100.00\%$
$\color{green}359/359$
$\color{green}+100.00\%$

The above coverage report was generated for the changes in this push.

Please sign in to comment.