REF: Rename jac -> score_obs. Closes #1785. #1982

Merged
merged 2 commits into from Sep 23, 2014

Projects

None yet

3 participants

@jseabold
Member

see #1785

@jseabold jseabold REF: Rename jac -> score_obs. Closes #1785.
5c41b60
@josef-pkt
Member

Don't merge this yet.

My guess is that this might break some user code. I need to check if we need to keep jac as alias, which I thought we do.

@coveralls

Coverage Status

Coverage decreased (-0.0%) when pulling 5c41b60 on jseabold:jac-rename into 0529da7 on statsmodels:master.

@jseabold jseabold MAINT: Deprecate jac methods.
0f24c10
@jseabold
Member

I deprecated it. Need to add to release notes, but I suspect they'll need a lot of work anyway.

@coveralls

Coverage Status

Coverage increased (+0.01%) when pulling 0f24c10 on jseabold:jac-rename into 0529da7 on statsmodels:master.

@josef-pkt
Member

Looks good to me, shouldn't cause any compatibility problems

merging

@josef-pkt josef-pkt merged commit 2fcc5ae into statsmodels:master Sep 23, 2014

2 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment