Closed conorheins closed 2 years ago
@daphnedemekas : can you verify that the update_A
works in your setup with arbitrary observation structure? Let me know if you still were having that issue where you had to reformat your observations, and I'll fix it. You shouldn't have to hack around it manually - the utility function process_observation
is supposed to be able to deal with arbitrary observation structure.
This PR:
Agent()
class and gives informative error messages when things don't line up (either certain distributions are not normalized, or dimensions don't line up)Agent()
num_controls
are automatically inferred from structure of user-givenB
array, if not provided.control_fac_idx
is tested more rigorously against other model dimensions (like theB
array andnum_factors
)learning.update_likelihood_dirichlet(...)
, based on an issue raised by @daphnedemekas