search

JuliaPOMDP / MCVI.jl

Other
5 stars 3 forks source link

get rid of zero argument MCVISolver constructor #6

Closed zsunberg closed 7 years ago

zsunberg commented 7 years ago

@rejuvyesh can we get rid of the zero argument MCVISolver constructor here?

If someone uses it, they just run into "undefined" errors that are hard to understand, so, unless there is another reason for it, I think getting rid of it would be the best for usability.

zsunberg commented 7 years ago

@ekhlasssonu and I did this and we were confused

rejuvyesh commented 7 years ago

I see. Sure. Not completely sure why I allowed it. I guess was an even worse julia noob then.