Skip to content
This repository was archived by the owner on Oct 31, 2024. It is now read-only.

Speed up parametric estimation, extract function locpoly into kernreg package, and set up continuous integration workflow with tox and GitHub Actions#183

Merged
segsell merged 73 commits intomasterfrom
speed_up_and_extract
Mar 1, 2021
Merged

Speed up parametric estimation, extract function locpoly into kernreg package, and set up continuous integration workflow with tox and GitHub Actions#183
segsell merged 73 commits intomasterfrom
speed_up_and_extract

Conversation

@segsell
Copy link
Copy Markdown
Member

@segsell segsell commented Feb 22, 2021 •
edited by peisenha
Loading

Parametric estimation:

  • Use gradient functions to speed up estimation
  • Align par and semipar output
  • Include docstrings

Semiparametric estimation:

  • The function locpoly has been extracted and moved to the kernreg package. It is available as an import.

Continuous Integration:

  • tox and GitHub Actions now run the continuous integration workflow. Travis is still left in place until Philipp gives green light to remove it.

This change is Reviewable

SeBecker and others added 30 commits May 3, 2020 13:11
@SeBecker
implemented new estimation routine for the usage of estimagic
55223e6
@SeBecker
Added gradient function to old framework
8908023
@SeBecker
preparations for comparison
1e478a6
@SeBecker
small changes an estimate_par
ff95840
@SeBecker
Added random start value option to estimate_par and included gradient…
ae58dbf 
… for optimization
@SeBecker
Adjusted the computation process of the hessian matrix, solved bug in…
f34885b 
… the estimation output function
@SeBecker
Added unit test for the gradient functions
0524140
@SeBecker
adjusted tutorial files
014ac01
@SeBecker
Adjusted promotion notebook
f724636
@SeBecker
Travis Fixing commit
569289b
@SeBecker
Some small adjustments
6dfafbf
@SeBecker
Merging master into estimation speed up branch
5a06662
@SeBecker
Fixed grmpy promotion notebook
db7853e
@SeBecker
Start Restructuring of estimate_par, added docstrings to most of the …
59c3d08 
…functions in estimate_par, Adjusted test battery for new inputs formats
@SeBecker
Fixed promotion notebook
0554af4
@SeBecker
Fixed test6 in test_integration.py
f6fdfd6
@SeBecker
Reorganized gradient function
8c1e8b2
@SeBecker
Fixed error in start value function
b9dbbdd
@SeBecker
Fixed test_integration.py::test6
ea74c98
@SeBecker
Adjusted output of the parametric estimation results
8729f24
@SeBecker
Adjusted output table format
fcaebd0
@SeBecker
small fix for upcoming changes
59fd0e5
@SeBecker
Adjusted fit_par output such that it is aligned with the semipar vers…
77405a1 
…ion, changed the container of the optimization results to DataFrame
@SeBecker
Adjusted test battery and plotting function
a15eac4
@SeBecker
Adjusted param tutorial notebook, the plot functions and the test bat…
996d8a7 
…tery
@segsell
Fix isinstance bug in plot_auxiliary.py: Type to check is pd.Series.
753fe30
@segsell
Pin sphinx bibtex version to 3.2.1.
3554521
@segsell
Pin sphinx bibtex to <2.0.0.
65b8cd2
@segsell
Pin sphinx to <2.0.0 and fix typo.
9429847
@segsell
Specify sohinxcontrib version in environment.yml
f8ec280
segsell added 19 commits February 6, 2021 15:42
@segsell
Use setup-python v1 and tox conda.
4aa0fb3
@segsell
Go back to conda-incubator.
16c8085
@segsell
Comment out long pytest command.
28a7cb6
@segsell
Add flake8 as pytest dependency in tox.ini.
231606d
@segsell
Add pre-commit and docs to continuous integration.
80b6b92
@segsell
Set pre-commit language version to python3.8.
f5542c2
@segsell
Add files changed by isort.
64184a9
@segsell
Add package kernreg as pip dependency.
eec97bd
@segsell
Import locpoly from external package kernreg.
453c745
@segsell
Remove KernReg folder.
3c9256c
@segsell
Try to fix tox error on windows. Remove requirements.txt
9f1cf77
@segsell
Specify llvmlite as conda dependency and set numpy version to >= 1.2
01e72fb
@segsell
Remove llvmlite from tox conda deps.
cec63b7
@segsell
Add llmvlite installation for windows in continuous-integration-workf…
df3056f 
…low.yml
@segsell
Add llmvlite>=0.35.0 to tox conda deps.
4075911
@segsell
Add OS ubuntu and mac again.
1eadeba
@segsell
Comment out setenv in [testenv:pytest] and remove conda dep llvmlite.
05a77d4
@segsell
Add llvmlite as conda dep again.
e149d95
@segsell
Comment in setenv in [testenv:pytest].
4377692
@review-notebook-app
Copy link
Copy Markdown

review-notebook-app Bot commented Feb 22, 2021

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@segsell segsell changed the title Speed up parametric estimation, extract function locpoly into kernreg package, and set up continuous integration workflow on GitHub Actions Speed up parametric estimation, extract function locpoly into kernreg package, and set up continuous integration workflow with tox and GitHub Actions Feb 22, 2021
@segsell segsell requested review from SeBecker and removed request for SeBecker February 22, 2021 13:52
@segsell segsell merged commit 5d87f2c into master Mar 1, 2021
@segsell segsell deleted the speed_up_and_extract branch March 1, 2021 11:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@segsell @SeBecker