You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
document the change to .base semantics introduced by Add maskna= and ownmaskna= parameters to np.asarray and friends (6c0ad59) #438
Right...I don't fully understand what the maskna and ownmaskna parameters are, or how they were meant to be used. But their implementation also included changes to .base, which I'm guessing were necessary anyways. And I remember seeing some mailing list discussions about changing .base to only include arrays that own their data (a change a welcome, which makes walking .base a lot easier and closes some reference cycles that I've had to fight with in my own code).
But it's sort of unrelated to the original change that it rode in on.
Activity
njsmith commentedon Sep 12, 2012
Those parameters were removed again in #297, so there's nothing to document...
njsmith commentedon Sep 12, 2012
Oh, you mean, document the change in the .base parameter that was also included in that commit... sorry.
embray commentedon Sep 12, 2012
Right...I don't fully understand what the maskna and ownmaskna parameters are, or how they were meant to be used. But their implementation also included changes to
.base, which I'm guessing were necessary anyways. And I remember seeing some mailing list discussions about changing.baseto only include arrays that own their data (a change a welcome, which makes walking .base a lot easier and closes some reference cycles that I've had to fight with in my own code).But it's sort of unrelated to the original change that it rode in on.
teoliphant commentedon Nov 6, 2012
This is related to issue #470 so I'm closing this one.
certik commentedon Nov 14, 2012
I've added a note about
.baseinto the release notes in PR #2737. Let me know if you think it is sufficient.embray commentedon Nov 14, 2012
Works for me.
DOC: document the change in .base
certik commentedon Dec 27, 2012
This is now fixed by #2737.