Skip to content
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.

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

Revert "ENH: sparse: Add _array version of diags creation functions… #18599

Merged
merged 1 commit into from Jun 1, 2023

Conversation

jarrodmillman
Copy link
Member

…. (#18538)"

This reverts commit 763a25e.

@tylerjereddy @rossbar See #18592 (comment)

Copy link
Member

@perimosocordiae perimosocordiae left a comment

Choose a reason for hiding this comment

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

This works for me. If diags_array is in the release notes, we should remove it.

Copy link
Contributor

@tylerjereddy tylerjereddy left a comment

Choose a reason for hiding this comment

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

The 1 CI failure is an unrelated and intentional version bound check on the release branch, so that's fine.

CJ's approval is good enough for me, and it would indeed be helpful for sparse folks to check what I drafted in for the "Highlights" section of the release notes when they make any other adjustments..

I've already pushed the version bump to rc2 so in it goes, thanks.

@tylerjereddy tylerjereddy merged commit 628b3a2 into scipy:maintenance/1.11.x Jun 1, 2023
22 of 23 checks passed
tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request Jun 15, 2023
* remove `diags_array` from release notes per scipygh-18599
tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request Jun 16, 2023
* update the SciPy 1.11.0rc2 release notes
following additional backports

DOC: suggestions for `scipy.sparse` release notes

Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>

Reword some sentences

Remove mentions of n-dimensional sparse arrays

Apply suggestions from code review

DOC: update release notes for scipy.stats

DOC: PR 18671 revisions

* remove `diags_array` from release notes per scipygh-18599

DOC: PR 18671 revisions

* update release notes after more activity
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.

None yet

3 participants