Skip to content

update state docstring to be clearer; simplify its use by stop-gradient'ing it automatically#213

Merged
patrick-kidger merged 1 commit intomainfrom
linear-state
Mar 10, 2026
Merged

update state docstring to be clearer; simplify its use by stop-gradient'ing it automatically#213
patrick-kidger merged 1 commit intomainfrom
linear-state

Conversation

@patrick-kidger
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown
Contributor

@jpbrodrick89 jpbrodrick89 left a comment

Choose a reason for hiding this comment

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

This is nice and makes the API a lot simpler making my comment in #206 is unnecessary. When merging #212 we can either do both and and or put this in an else.

@patrick-kidger patrick-kidger merged commit fbd294c into main Mar 10, 2026
1 of 2 checks passed
@patrick-kidger patrick-kidger deleted the linear-state branch March 10, 2026 00:36
@patrick-kidger
Copy link
Copy Markdown
Owner Author

When merging #212 we can either do both and and or put this in an else.

I think technically 'both', since in principle the solver could close over some autodiff'd arrays. (Bit weird but in principle possible!)

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