Skip to content

Drop deprecated auto-self-downcasts - #213

Draft
inducer wants to merge 1 commit into
mainfrom
no-auto-self-downcast
Draft

Drop deprecated auto-self-downcasts#213
inducer wants to merge 1 commit into
mainfrom
no-auto-self-downcast

Conversation

@inducer

@inducer inducer commented Jul 14, 2026

Copy link
Copy Markdown
Owner

No description provided.

@inducer

inducer commented Aug 11, 2026

Copy link
Copy Markdown
Owner Author

@connorjward This is another thing that might be worth a look. The gunk that this drops will no longer be needed by loopy after inducer/loopy#1023, so I'd like to get rid of it before too long. It has been deprecated (with warnings) for a while. Merging this will break old loopy.

I'd be happy to coordinate with you on the timeline for merging and releasing this.

@inducer
inducer force-pushed the no-auto-self-downcast branch from 28d7bfa to a29fc69 Compare August 11, 2026 12:49
@connorjward

Copy link
Copy Markdown

@connorjward This is another thing that might be worth a look. The gunk that this drops will no longer be needed by loopy after inducer/loopy#1023, so I'd like to get rid of it before too long. It has been deprecated (with warnings) for a while. Merging this will break old loopy.

I'd be happy to coordinate with you on the timeline for merging and releasing this.

Our next release is scheduled for early October. We are quite used to handling upstream breaking changes (looking at you PETSc) so I am not concerned. And since our releases use versioned releases of loopy this isn't an immediately breaking changes for our users (and if you make a new release that breaks us we can make a patch release with a pin in short order).

In short: early warning of things breaking is super welcome, but don't let us hold you up and merging early (and hence well in advance of our next release) is actually best.

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.

2 participants