Skip to content

Fix broadcasting in discretizer's Dirac-observation branch#71

Merged
DanWaxman merged 7 commits intomainfrom
ml-perfect-observations3-patch
Feb 6, 2026
Merged

Fix broadcasting in discretizer's Dirac-observation branch#71
DanWaxman merged 7 commits intomainfrom
ml-perfect-observations3-patch

Conversation

@mattlevine22
Copy link
Collaborator

Patches #70.
May interact with batching choices in #63.

Copy link
Collaborator

@DanWaxman DanWaxman left a comment

Choose a reason for hiding this comment

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

It's great to see such a speedup! I think some naming conventions and duplicate code may be confusing, though.

Copy link
Collaborator Author

@mattlevine22 mattlevine22 left a comment

Choose a reason for hiding this comment

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

should be ready

@DanWaxman DanWaxman self-requested a review February 6, 2026 23:10
Copy link
Collaborator

@DanWaxman DanWaxman left a comment

Choose a reason for hiding this comment

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

This looks good to me!!

@DanWaxman DanWaxman merged commit ad77e3b into main Feb 6, 2026
2 checks passed
@mattlevine22 mattlevine22 deleted the ml-perfect-observations3-patch branch February 6, 2026 23:21
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