8000 ndarray.put should respect the writeable flag. Fixes #4465 by jarl-haggerty · Pull Request #5132 · numpy/numpy · GitHub
[go: up one dir, main page]
Skip to content

ndarray.put should respect the writeable flag. Fixes #4465 #5132

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

ndarray.put should respect the writeable flag. Fixes #4465 #5132

wants to merge 3 commits into from

Conversation

jarl-haggerty
Copy link
Contributor

@juliantaylor
Copy link
Contributor

please remove the wrong commits with git rebase -i
this should have some tests, probably somewhere near the putmask test in test_multiarray.py

@charris
Copy link
Member
charris commented Sep 27, 2014

Also replace #4465 by something more descriptive. How does this relate to your #5130 PR? It looks like the same change set it in both that PR and this one.

@charris charris changed the title 4465 ndarray.put should respect the writeable flag. Fixes #4465 Sep 30, 2014
@charris
Copy link
Member
charris commented Oct 10, 2014

Needs a test.

@charris charris mentioned this pull request Oct 21, 2014
Closed
@charris
Copy link
Member
charris commented Oct 21, 2014

#5215 is a cleaned up verstion of this with your commit part of it. Thanks

@charris charris closed this Oct 21, 2014
@charris charris mentioned this pull request Oct 22, 2014
Merged
juliantaylor referenced this pull request Oct 24, 2014
@juliantaylor
Merge pull request #5216 from charris/cleanup-gh-5132
fe7816d 
BUG: Make PyArray_PutTo respect writeable flag.
juliantaylor referenced this pull request Oct 24, 2014
@juliantaylor
Merge pull request #5216 from charris/cleanup-gh-5132
42d890f 
BUG: Make PyArray_PutTo respect writeable flag.
@njsmith njsmith mentioned this pull request Sep 24, 2015
Closed
@adlerandy adlerandy mentioned this pull request Nov 29, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jarl-haggerty @juliantaylor @charris
0