Skip to content
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

chore: minor clean-up of math/base/special/acsc #1298

Merged
merged 27 commits into from
Jun 25, 2024

Conversation

Pranavchiku
Copy link
Member

@Pranavchiku Pranavchiku commented Jan 30, 2024

Resolves #859.

Checklist

  • update readme.md
  • include.gypi
  • binding.gyp
  • include/stdlib/math/base/special/
  • src
  • manifest.json
  • lib
  • examples
  • benchmark
  • test

Feels good to be back :)

@Pranavchiku
Copy link
Member Author

Not sure how can I run benchmarks and tests, I followed previous commands like

% make benchmark-c BENCHMARKS_FILTER=.*/math/base/special/acsc/.*                                            
zsh: no matches found: BENCHMARKS_FILTER=.*/math/base/special/acsc/.*
% make examples-c EXAMPLES_FILTER=.*/math/base/special/acsc.*                                               
zsh: no matches found: EXAMPLES_FILTER=.*/math/base/special/acsc/.*

How may I run javascript tests?

@Pranavchiku Pranavchiku marked this pull request as ready for review January 30, 2024 17:11
@kgryte kgryte added Feature Issue or pull request for adding a new feature. Math Issue or pull request specific to math functionality. Native Addons Issue involves or relates to Node.js native add-ons. labels Jan 31, 2024
@kgryte
Copy link
Member

kgryte commented Jan 31, 2024

make test TESTS_FILTER=".*/math/base/special/acsc/.*"

should work.

Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
@kgryte
Copy link
Member

kgryte commented Jan 31, 2024

@Pranavchiku I am not sure why those commands are not working for you. Can you trying surrounding the filter in quotes?

make benchmark-c BENCHMARKS_FILTER=".*/math/base/special/acsc/.*"

@kgryte
Copy link
Member

kgryte commented Jan 31, 2024

@Pranavchiku This is looking good. Thanks for working on this. The builtin C benchmark can be removed as acsc is not a part of C99, TMK. We also need to add a gypfile field to the package.json. Once resolved, this PR should be ready for merge.

@kgryte kgryte added C Issue involves or relates to C. Needs Changes Pull request which needs changes before being merged. labels Feb 23, 2024
@kgryte kgryte changed the title Add C implementation for @stdlib/math/base/special/acsc feat: add C implementation for math/base/special/acsc Mar 3, 2024
@kgryte
Copy link
Member

kgryte commented Jun 11, 2024

@Pranavchiku Are you interested in working to get this PR over the finish line?

@Pranavchiku
Copy link
Member Author

I am afraid I will not be able to do it atm. I am fine if anyone else pick it up or if it is not a blocker I can look it up later at end of June.

@Planeshifter Planeshifter self-requested a review June 25, 2024 02:02
@Planeshifter Planeshifter changed the title feat: add C implementation for math/base/special/acsc chore: minor clean-up of math/base/special/acsc Jun 25, 2024
@Planeshifter
Copy link
Member

We already merged a PR for the C implementation . Merged latest changes from develop into the PR branch and will merge the now much smaller PR, which just consists of a few documentation and style improvements, shortly.

@Planeshifter Planeshifter merged commit 9e56edf into stdlib-js:develop Jun 25, 2024
8 checks passed
@kgryte kgryte removed the Needs Changes Pull request which needs changes before being merged. label Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Issue involves or relates to C. Feature Issue or pull request for adding a new feature. Math Issue or pull request specific to math functionality. Native Addons Issue involves or relates to Node.js native add-ons.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RFC]: Add C implementation for @stdlib/math/base/special/acsc
3 participants