IQSS / Zelig

A statistical framework that serves as a common interface to a large range of models
http://zeligproject.org
111 stars 43 forks source link

Setx breaking for exp model #254

Closed tercer closed 7 years ago

tercer commented 7 years ago

The setx method is throwing up errors in the exp model. This was mentioned on the docs queue too ( https://github.com/IQSS/docs.zeligproject.org/issues/15 ) but seems to be an actual code problem, and not a Sphinx/VM issue.

screen shot 2017-04-13 at 1 18 36 pm
christophergandrud commented 7 years ago

Need to load the survival package, e.g. https://github.com/IQSS/docs.zeligproject.org/commit/83bf6c6cb3a76faf0dad42dce3f0748a866615b0

christophergandrud commented 7 years ago

Need to think of a more elegant solution, but closing for now.

tercer commented 7 years ago

One solution is to upgrade the survival package to Depends to make sure it is attached rather than just loaded. Here is it working in this example:

screen shot 2017-04-19 at 3 44 09 pm

Here's a patch with that solution to pull if that works for now: 375d2efb5eb4d60d867b8a8967af194d9dd6d58a

christophergandrud commented 7 years ago

Patch merged into master in d865edcbdc31f43649e5052533fdd666443bf367

And docs updated in: https://github.com/IQSS/docs.zeligproject.org/commit/5a0756c88dcff679176f716a25b1ff51a5d75630