El aft rebased #462
Merged
El aft rebased #462
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… univariate variance
… have default search limits.
…I was printing inner optimization results
Also briefly went through can changed int to float per Joesf's suggestion. I will continue to keep a lookout for these as I'm sure I didn't catch them all.
…tion for contour plot.
…ot return an error. It seems to be able to recover from bad nuisance parameters by itself
…n is through the origin
… empirical likelihood. The EL part will only be used for inference.
…has a quadruply nested optimization and is computationally intensive.
…ssion parameter takes a long time because it involves a quadruply nested optimization, which is noted in the documentation.
josef-pkt
added a commit
that referenced
this pull request
Sep 16, 2012
El aft rebased some cleanup will come in a future branch by Justin
Closed
PierreBdR
pushed a commit
to PierreBdR/statsmodels
that referenced
this pull request
Sep 2, 2014
El aft rebased some cleanup will come in a future branch by Justin
|
@ShuguangSun A comment in a merged PR/commit will get lost. Please open new issues if you find some problems, so we have an open issue to track problems with existing code. AFAIK, we never have |
|
koul_and_mc.py is an example. It throws and an error for fit().params. In aft_el.py, params is a method, and not a attribute. I will open an issue for it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
rebased version of el_aft PR #427
tests run clean
might need refactoring in future for consistency with a more general survival/AFT subpackage