You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Explain what n_samples does in LikelihoodResult.se() (It says to look at the implementation of constraints, but glancing over it I cannot find anything there.
Make the warning about a missing seed more constructive in LikelihoodResult.se(). E.g., a core estimagic dev wrote some code that produces:
estimagic/estimation/estimate_ml.py:429:
UserWarning: seed is set to None and constraints are transforming.
This leads to randomness in the result. To avoid random behavior, choose a
non-None seed.
and it took me quite some time to find out where it is coming from (pytask does not show more and it wasn't obvious to me that LikelihoodResult lives in estimate_ml.py. So I had initially thought this happens during the estimation phase... Would be enough to add that this needs to be done in .se() or .summary().
The text was updated successfully, but these errors were encountered:
Add a docs improvement issues template
Explain what
n_samples
does inLikelihoodResult.se()
(It says to look at the implementation of constraints, but glancing over it I cannot find anything there.Make the warning about a missing seed more constructive in
LikelihoodResult.se()
. E.g., a core estimagic dev wrote some code that produces:and it took me quite some time to find out where it is coming from (pytask does not show more and it wasn't obvious to me that
LikelihoodResult
lives inestimate_ml.py
. So I had initially thought this happens during the estimation phase... Would be enough to add that this needs to be done in.se()
or.summary()
.The text was updated successfully, but these errors were encountered: