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.

Sign up for GitHub

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

Jump to bottom

Remove DeterministicProcess class #335

Closed
dylanhmorris opened this issue Jul 26, 2024 · 1 comment
Closed

Remove DeterministicProcess class #335

dylanhmorris opened this issue Jul 26, 2024 · 1 comment
Labels
clean up Good code that could be better
Milestone
🦖 Rajasaurus

Comments

@dylanhmorris
Copy link
Collaborator

Made an attempt in 9ff9dea but I increasingly think we should get rid of the DeterministicProcess class, as I think it should be replaced by other more time-aware ways of handling temporal broadcasting of values. Beyond the scope of this PR, though.

Originally posted by @dylanhmorris in #262 (comment)
@damonbayer damonbayer added the clean up Good code that could be better label Jul 29, 2024
@damonbayer damonbayer mentioned this issue Aug 19, 2024
Closed
@damonbayer damonbayer added this to the 🦖 Rajasaurus milestone Sep 11, 2024
@damonbayer
Copy link
Collaborator

Closed in #441

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clean up Good code that could be better
Projects
None yet
Milestone
🦖 Rajasaurus
Development

No branches or pull requests
2 participants
@dylanhmorris @damonbayer