-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Description
It is concluded from our stand-up that:
- Indeed, the above suggestion from @jakobsj is neat and intuitive;
- @amal-ghamdi mentioned that, however, implementing such a feature requires direct changes to
density/distribution, which we have deliberately avoided so far. Currently, we do sampling via the interface ofBayesianProblemorsampler, instead of withdensity/distributiondirectly. Introducing this feature will break this consistency and convention. recommenderdoes not seem to fit well at the very high level ofcuqi,cuqi.utilitiesorcuqi.sampler.- We agree that more careful consideration is needed on this issue.
- Therefore, it's decided that we put
recommenderback to theexperimentalmodule, which is appropriate in the sense that the feature is still experimental.
Originally posted by @chaozg in #693 (comment)
Metadata
Metadata
Assignees
Labels
No labels