Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

implement update method for coef vector #567

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

apoorvalal
Copy link
Member

@apoorvalal apoorvalal commented Jul 28, 2024

added simple update method to feols object to update coefficient vector in online-learning settings. Use demonstrated in quickstart notebook.
Also fleshed out confint section of quickstart notebook.

…lso link to sup-t reference paper for confint.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

codecov bot commented Jul 28, 2024

Codecov Report

Attention: Patch coverage is 97.05882% with 1 line in your changes missing coverage. Please review.

Files Coverage Δ
tests/test_others.py 100.00% <100.00%> (ø)
pyfixest/estimation/feols_.py 90.64% <93.75%> (ø)

... and 27 files with indirect coverage changes

@s3alfisc
Copy link
Member

Thanks for adding the tests @apoorvalal - should I merge this as is & we can add inference for the update method in a separate PR?

@apoorvalal
Copy link
Member Author

apoorvalal commented Jul 31, 2024 via email

@s3alfisc s3alfisc merged commit 3bc70e6 into py-econometrics:master Jul 31, 2024
7 checks passed
@apoorvalal apoorvalal deleted the onlineOLS branch August 23, 2024 22:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants