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

Reland new hvcat design (#406) #407

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Reland new hvcat design (#406) #407

merged 2 commits into from
Jul 12, 2023

Conversation

vtjnash
Copy link
Contributor

@vtjnash vtjnash commented Jul 10, 2023

Opening the reland commit as a followup after JuliaLang/julia#48977 is finished
Relands #384
Reverts #406

@ViralBShah
Copy link
Member

We should have new nightlies in a couple of days and be able to rerun these tests.

@vtjnash vtjnash merged commit c93589c into main Jul 12, 2023
@vtjnash vtjnash deleted the revert-406-vs/revert-384 branch July 12, 2023 15:34
@ViralBShah
Copy link
Member

Do we need to bump SparseArrays on Julia master for this?

vtjnash added a commit that referenced this pull request Jul 12, 2023
Missed in #407, but causing doctest build to fail when we try to use this.
@dkarrasch
Copy link
Member

We need to be careful about what we merge into main now. Because everything we merge now, will get backported to v1.10. So, yes, we definitely need to bump SparseArrays.

@ViralBShah
Copy link
Member

Ok, let's not merge any new features for now - and stabilize what we have including hvcat.

@dkarrasch
Copy link
Member

Well, I do think we should include #392, possibly #395, #396, #398, and #405 as a prerequisite to #396. They have been waiting while we were fixing the SuiteSparse stuff.

@ViralBShah
Copy link
Member

ViralBShah commented Jul 12, 2023

The nightly is in decent shape now. So perhaps we can start merging. Do you want to do one merge and bump at a time so that we know things are in good shape incrementally?

vtjnash added a commit that referenced this pull request Jul 12, 2023
Missed in #407, but
causing doctest build to fail when we try to use this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants