Skip to content

DOC: update 1.13 release note for MaskedArray, masked constants ellipisis indexing#9122

Merged
charris merged 1 commit intonumpy:masterfrom
ahaldane:1.13_note_ma_scalar
May 16, 2017
Merged

DOC: update 1.13 release note for MaskedArray, masked constants ellipisis indexing#9122
charris merged 1 commit intonumpy:masterfrom
ahaldane:1.13_note_ma_scalar

Conversation

@ahaldane
Copy link
Copy Markdown
Member

Fixes #9121

(backport coming up)

@ahaldane ahaldane force-pushed the 1.13_note_ma_scalar branch from b1e2559 to cb0331d Compare May 16, 2017 03:51
@ahaldane ahaldane changed the title DOC: update 1.13 release note for MA ... indexing REL: update 1.13 release note for MA ... indexing May 16, 2017
@ahaldane ahaldane force-pushed the 1.13_note_ma_scalar branch from cb0331d to adf8345 Compare May 16, 2017 03:59
@eric-wieser
Copy link
Copy Markdown
Member

Surely "wraps", not "unwraps". Don't forget that this also changes arr[i,...] on 1d arrays

@ahaldane ahaldane force-pushed the 1.13_note_ma_scalar branch from adf8345 to c04f359 Compare May 16, 2017 14:57
@ahaldane
Copy link
Copy Markdown
Member Author

ahaldane commented May 16, 2017

@eric-wieser , updated. Does this version sound accurate?

@charris charris changed the title REL: update 1.13 release note for MA ... indexing DOC: update 1.13 release note for MA ... indexing May 16, 2017
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Use dashes, '-', not ~ and put the ellipsis in quotes.

@charris charris changed the title DOC: update 1.13 release note for MA ... indexing DOC: update 1.13 release note for MaskedArray, masked constants ellipisis indexing May 16, 2017
@charris charris added the 09 - Backport-Candidate PRs tagged should be backported label May 16, 2017
@charris charris added this to the 1.13.0 release milestone May 16, 2017
@ahaldane ahaldane force-pushed the 1.13_note_ma_scalar branch from c04f359 to 28b3597 Compare May 16, 2017 16:10
@ahaldane
Copy link
Copy Markdown
Member Author

OK, updated as you suggested, and changed the commit title to be DOC too.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That should be Ellipsis, not ellipsis. I think @charris' comment was requesting ``...``

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oh! that makes more sense

@ahaldane ahaldane force-pushed the 1.13_note_ma_scalar branch from 28b3597 to f19f01f Compare May 16, 2017 16:19
@charris charris merged commit c610444 into numpy:master May 16, 2017
@charris
Copy link
Copy Markdown
Member

charris commented May 16, 2017

Looks good, thanks Allan.

@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label May 16, 2017
charris added a commit that referenced this pull request May 16, 2017
DOC: update 1.13 release note for MaskedArray, masked constants ellipisis indexing (Backport #9122)
@charris charris removed this from the 1.13.0 release milestone May 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants