probcomp / GenExperimental.jl

Featherweight embedded probabilistic programming language and compositional inference programming library
MIT License
17 stars 2 forks source link

@register_module does not work from outside Gen package #7

Closed marcoct closed 7 years ago

marcoct commented 7 years ago

You cannot currently use @register_macro from outside Gen itself it seems. It is essential to be able to declare new modules in user space, outside of Gen. There is a workaround which is to add a different registration function that automates less, requiring a little more boilerplate for the module declaration.

marcoct commented 7 years ago

This is not acceptable for user experience. I will assume now that users will not be writing their own modules. This will need to be fixed before we expect users to write their own modules.

marcoct commented 7 years ago

Obviated by new method for registering modules: https://github.com/probcomp/Gen.jl/commit/cbbdcab715e0cfb2e49597a4c2aa0054f8923a1b