This repository was archived by the owner on Nov 17, 2025. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 20
Fix the design issues in MeasurableTransformΒ #206
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededrefactoringA change that improves the codebase but doesn't necessarily introduce a new featureA change that improves the codebase but doesn't necessarily introduce a new featurerv-transformsInvolves transforms applied to random variablesInvolves transforms applied to random variables
Description
Currently, MeasurableTransform keeps a fixed set of valid_scalar_types consisting of supported scalar Ops. This needs to be completely changed so that supported Ops can be extended in a reasonable way, and without the need to import/specify them in AePPL itself, or perform hackish updates to static lists in AePPL classes.
The entire approach needs to be reconsidered carefully, although the aforementioned improvements are still a good start.
rlouf
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededrefactoringA change that improves the codebase but doesn't necessarily introduce a new featureA change that improves the codebase but doesn't necessarily introduce a new featurerv-transformsInvolves transforms applied to random variablesInvolves transforms applied to random variables