Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adding GARCH #4815

Merged
merged 81 commits into from
May 10, 2023
Merged

Adding GARCH #4815

merged 81 commits into from
May 10, 2023

Conversation

northern-64bit
Copy link
Contributor

Description

Adding GARCH, its model fit and forecasts. It also reorganizes the econometrics menu.

How has this been tested?

Comparing to sample GARCH results from Hull.

Checklist:

Others

  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.

@reviewpad reviewpad bot added the feat S Small T-Shirt size Feature label Apr 17, 2023
@codecov
Copy link

codecov bot commented Apr 17, 2023

Codecov Report

Patch coverage: 55.04% and project coverage change: -0.24 ⚠️

Comparison is base (df9a154) 58.29% compared to head (103b1a8) 58.05%.

❗ Current head 103b1a8 differs from pull request most recent head 3e52b2d. Consider uploading reports for the commit 3e52b2d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4815      +/-   ##
===========================================
- Coverage    58.29%   58.05%   -0.24%     
===========================================
  Files          588      588              
  Lines        53666    53877     +211     
===========================================
- Hits         31283    31277       -6     
- Misses       22383    22600     +217     
Impacted Files Coverage Δ
.../core/plots/plotly_ta/plugins/volatility_plugin.py 31.37% <0.00%> (-1.29%) ⬇️
...inal/core/sdk/controllers/crypto_sdk_controller.py 0.00% <ø> (ø)
...inal/core/sdk/controllers/stocks_sdk_controller.py 0.00% <ø> (ø)
...penbb_terminal/core/sdk/models/crypto_sdk_model.py 0.00% <0.00%> (ø)
...penbb_terminal/core/sdk/models/stocks_sdk_model.py 0.00% <0.00%> (ø)
openbb_terminal/core/sdk/sdk_helpers.py 36.92% <ø> (ø)
openbb_terminal/core/sdk/sdk_init.py 92.75% <ø> (ø)
openbb_terminal/core/sdk/trailmap.py 92.68% <ø> (ø)
openbb_terminal/dashboards/stream/Forecasting.py 0.00% <0.00%> (ø)
...bb_terminal/dashboards/stream/streamlit_helpers.py 0.00% <0.00%> (ø)
... and 41 more

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@reviewpad reviewpad bot added feat M Medium T-Shirt size feature and removed feat S Small T-Shirt size Feature labels Apr 20, 2023
@northern-64bit
Copy link
Contributor Author

Note that running doctest on the example in the test fails:

I've now fixed it :)

@jmaslek
Copy link
Collaborator

jmaslek commented May 9, 2023

Note that running doctest on the example in the test fails:

I've now fixed it :)

The docstring does not represent a functional example

    51         return np.asarray(x.iloc[:, 0])

ValueError: y must be squeezable to 1 dimension

You need to specify a column from the load. Also linters are failing

@jmaslek jmaslek enabled auto-merge May 10, 2023 14:22
@jmaslek jmaslek added this pull request to the merge queue May 10, 2023
Merged via the queue into OpenBB-finance:develop with commit bde21f0 May 10, 2023
11 checks passed
@northern-64bit northern-64bit deleted the feature/garch branch May 10, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat M Medium T-Shirt size feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants