-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Fix bugs in the provider models and standard models #5427
Merged
Merged
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
…ance/OpenBBTerminal into feature/add-data-providers
…ance/OpenBBTerminal into feature/add-data-providers
…B-finance/OpenBBTerminal into feature/add-data-providers
* boilerplate for av provider * adding stock eod for av * ruff n black * changing available functions for better ux * docstring * move debug_mode out of system settings (#5366) * move debug_mode out of system settings * Update router.py * update readme with debug mode * dev_mode hub service + openbb_ namespace to Env * update readme * copy env to avoid problems w global state * Linting * rebuild python interface + fix py38 * Add @classproperty to fetcher for nested return types (#5371) * add classproperty to fetcher and cpi.py * type var bound * validate model in registry map * better exception msg * fix fred mypy and add data_type * typo * specific model validation * specific model validation * reformat error msg * better * msg * add black & ruff to core deps * supress linter errors if not debug mode * pylint + regen fixedincome * py38 compat * fix bug getting provider for placeholder cmds + ruff * Revert "fix bug getting provider for placeholder cmds + ruff" This reverts commit 1328c0b. * ruff * avoid bug for placeholder and no provider * expose available providers in provider interface * poetry toml to match ci ruff version * first we ruff and then we black * add back noqa: E501 to docstrings inside package * use singleton for provider interface: same functionality (#5372) * use singleton for provider interface: same functionality * ruff * docstring typo * update readme with tuna reference * move to correct readme * Merge * Some error handling (#5357) * Removed prints (#5367) * Removed prints * FIxed typing * FIxed typing * Fixed typing * Fixed typing * Fixed typing * Fixed typing * Fixed typing * Fix * Reverted stuff * Feature/move package (#5374) * move package to openbb_sdk * avoid test writing files * black * revert toml changes, will do it in another PR * run build in a separate process * change error msg * rename func * change msg * rename func + msg * change param name * update readme * lazy load some deps * Update README.md * modules is better * doc * ruff * pylint * fix autocomplete (#5383) * Core tests (#5375) * makeing the function easier - @montezdesousa pls review this commit * improve update_provider_choices readbility * unncessary classmethod * command runner tests * fixing wrong attribution * some tests for the charting manager * changing fetcher and models to have a shorter name * @the-prax suggestions * fixing references to av * making function a private fild so the user don't access it * unbreaking changes * Improve snake_case generation * adding av to pyproject.toml * stating the function as a private field while keeping validation for function specific fields --------- Co-authored-by: montezdesousa <[email protected]> Co-authored-by: Diogo Sousa <[email protected]> Co-authored-by: Igor Radovanovic <[email protected]> Co-authored-by: Colin Delahunty <[email protected]>
…ance/OpenBBTerminal into feature/add-data-providers
* rename EOD to Historical in providers * renamed EOD to Historical in sdk/provider * rename _eod to _historical in standard_models * rename _eod to _historical in providers directory * cleanup * rename EOD to Historical in extensions * static * Revert "rename EOD to Historical in extensions" * rename EOD to Historical in relevant extensions * fix spelling in init * static * linting * intrinio y u alone * so sorry intrinio * static * ruff --------- Co-authored-by: hjoaquim <[email protected]>
…ance/OpenBBTerminal into feature/add-data-providers
IgorWounds
approved these changes
Sep 19, 2023
the-praxs
force-pushed
the
fix/provider-bugs
branch
from
September 19, 2023 19:58
c0aca80
to
b0bc14e
Compare
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.
A lot of fixes to stabilize the working of mostly Pro data providers.
Breaking changes for Stock News, Global News and Options.