8000 MAINT: Simplify npymath by mattip · Pull Request #22090 · numpy/numpy · GitHub
[go: up one dir, main page]
Skip to content

MAINT: Simplify npymath #22090

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.

Sign up for GitHub

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

Merged
merged 16 commits into from
Aug 23, 2022
Merged

MAINT: Simplify npymath #22090

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f33e579
remove obsolete support for Solaris
mattip Aug 7, 2022
e9e8cee
rearrange headers to make npy_config.h only about config
mattip Aug 7, 2022
9e481ad
make optional C99 double routines mandatory
mattip Aug 7, 2022
aceba28
remove ifndef alterantives for mandatory functions
mattip Aug 10, 2022
c35cb18
restore HAVE_LOG2 for cygwin blocklistingw
mattip Aug 11, 2022
16a6405
restore HAVE_POWL for macOS blocklisting
mattip Aug 11, 2022
c304f6f
move hypot to optional for windows, fix pow and log2
mattip Aug 14, 2022
37b2cf1
frexp is now mandatory
mattip Aug 14, 2022
b0878cf
copysign is mandatory, no HAVE_COPYSIGN is defined
mattip Aug 14, 2022
ee2103b
rearrange badly placed header includes
mattip Aug 14, 2022
759e0cd
strtoull and strtoll are mandatory
mattip Aug 15, 2022
2c4cc10
restore atan2 implementation, clean up c99 mandatory functions
mattip Aug 15, 2022
704ca94
nextafter is mandatory
mattip Aug 15, 2022
b6d52fc
remove duplicate C99 functions, ensure optional variant HAVE_ macros …
mattip Aug 16, 2022
d1cbf3f
BUILD: convert HAVE_ macros to BLOCK_, cleanup
mattip Aug 16, 2022
25629dd
TASK: rename macros (from review)
mattip Aug 21, 2022
8000
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
frexp is now mandatory
  • Loading branch information
@mattip
mattip committed Aug 21, 2022
commit 37b2cf17cbf50122283f9737e4419a79fb32d6fb
5 changes: 1 addition & 4 deletions numpy/core/src/npymath/npy_math_internal.h.src
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ NPY_INPLACE double npy_log2(double x)
{
#ifdef HAVE_LOG2
return log2(x);
#elif defined(HAVE_FREXP)
#else
if (!npy_isfinite(x) || x <= 0.) {
/* special value result */
return npy_log(x);
Expand All @@ -123,9 +123,6 @@ NPY_INPLACE double npy_log2(double x)
return log(m) / log(2.0) + e;
}
}
#else
/* does not provide int(log(2**i)) == i */
return NPY_LOG2E * npy_log(x);
#endif
}

Expand Down
0