Skip to content

stochastic state#657

Draft
ValerianRey wants to merge 12 commits intomainfrom
stochastic-state
Draft

stochastic state#657
ValerianRey wants to merge 12 commits intomainfrom
stochastic-state

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

@ValerianRey ValerianRey commented Apr 17, 2026

@PierreQuinton Just made a draft PR to show you an example of what I had in mind. I only made the implementation for pcgrad.

I also forgot the reset method of PCGrad and PCGradWeighting.

Copy link
Copy Markdown
Contributor

@PierreQuinton PierreQuinton left a comment

Choose a reason for hiding this comment

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

I like the idea of Stochastic state. But this raises the question of having a class for states. It is slightly weird (yet true) that a state is stateful.

In principle, we could have an abstract class State, it would have an abstract static method initial() -> State where the return state is a state of same type as this. I think Stateful would then be generic over the State type (so in python I guess it must take a State class type). We can have two States, one that contains data directly, and another would be stochastic state. The structure isn't clear in my mind, but there may be a path.

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