ENH: Changed repr of np.bool_ by ganesh-k13 · Pull Request #17592 · numpy/numpy

ganesh-k13 · 2020-10-20T10:02:24Z

Changes:

Changed repr of np.bool_

After Fix:

>>> str(np.bool_(False))
'numpy.False_'
>>> str(np.bool_(True))
'numpy.True_'
>>>

Note:

I expect a lot of UT and Doc failures, will fix as we go along.

cc: @eric-wieser , @seberg

numpy/core/src/multiarray/scalartypes.c.src

ganesh-k13 · 2020-10-20T10:57:08Z

Some tests are forming cores, will send the mail to the mailing list, once that's fixed

numpy/core/src/multiarray/scalartypes.c.src

ganesh-k13 · 2020-10-21T04:53:14Z

Do macs have a different set of tests? I noticed this is failing.

ganesh-k13 · 2020-10-21T09:11:09Z

It's working in linux:

(Pdb) p numpy_f
<function any at 0x7f78f15e7310>
(Pdb) p numpy_f(a, axis=1, keepdims=True)[...,:-1]
masked_array(
  data=[[[True, False, True]],

        [[True, False, True]]],
  mask=[[[False, False, False]],

        [[False, False, False]]],
  fill_value=numpy.True_)

> /home/ganesh/open-source/numpy/build/testenv/lib/python3.8/site-packages/numpy/ma/tests/test_core.py(4948)testkeepdims()
-> assert_equal(ma_f(a, keepdims=True).shape,
(Pdb) p ma_f(a, axis=1, keepdims=True)[...,:-1]
masked_array(
  data=[[[True, False, True]],

        [[True, False, True]]],
  mask=[[[False, False, False]],

        [[False, False, False]]],
  fill_value=numpy.True_)
(Pdb) p f
'any'

numpy/numpy/ma/tests/test_core.py

Lines 4958 to 4960 in 32f1359

    
           for f in ['sum', 'prod', 'mean', 'var', 'std']: 
        
               testaxis(f, a, d) 
        
               testkeepdims(f, a, d)

Any idea why it's failing only on Mac?

eric-wieser · 2020-10-21T09:30:22Z

The test that is failing is a documentation test, and I guess the documentation tests are only run on mac? You need to update the docstrings to use the new printing.

mattip · 2020-10-21T10:23:59Z

The incantation is python runtests.py -g --refguide-check

ganesh-k13 · 2020-10-24T06:06:19Z

I have emailed the mailing list (nabble link, pipermail link)for their opinion @eric-wieser, as mentioned in the issue.

eric-wieser

Can you add an "improvements" release note, like we did when we improved void.__repr__ with https://numpy.org/doc/stable/release/1.14.0-notes.html#void-datatype-elements-are-now-printed-in-hex-notation?

ganesh-k13 · 2020-11-01T22:04:17Z

@eric-wieser , any more changes needed?

doc/release/upcoming_changes/17592.improvement.rst

eric-wieser

Looks good to me, and the mailing list seems not to care. Will leave this for another maintainer to push the button on if they agree.

I reworded the release note slightly to make clear also what did not change.

ganesh-k13 · 2020-11-01T22:22:00Z

Thanks Eric, sure

seberg · 2020-11-02T14:58:23Z

Happy to put this in as such, although I wonder a bit whether this has the potential to be a slightly noisy transition, and 1.20 is already a big release.

hameerabbasi · 2020-11-04T19:34:41Z

This is different from every other NumPy scalar, np.int*, np.float*, ...

Also reprs don't usually include the module.

seberg · 2020-11-04T19:37:10Z

We are a bit unsure whether this is a good idea at all, could you propose this on the mailing list, since it is an API change?

eric-wieser · 2020-11-04T20:54:57Z

This already hit the mailing list and was meet with silence

mattip · 2022-04-18T18:09:22Z

Summarizing the current status:

this PR should be retitled to reflect the pivot to changing the repr of all scalars
tests are failing and it is still marked as a draft
@eric-wieser could you remove your approval of the PR, it is confusing when looking at PR statuses.

seberg · 2023-07-26T17:59:12Z

The changes here were included in gh-22449 so closing this one.

github-actions bot added the 01 - Enhancement label Oct 20, 2020

eric-wieser reviewed Oct 20, 2020

View reviewed changes