-
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
Release/3.0.1 #5025
Merged
Merged
Release/3.0.1 #5025
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
Co-authored-by: Henrique Joaquim <[email protected]>
* handle defi anchor exceptions * reason * mypy --------- Co-authored-by: Henrique Joaquim <[email protected]>
Release/3.0.0
Co-authored-by: Henrique Joaquim <[email protected]>
* handle defi anchor exceptions * reason * mypy --------- Co-authored-by: Henrique Joaquim <[email protected]>
* Create and update tests * Create and update tests --------- Co-authored-by: montezdesousa <[email protected]>
hofix / sync with main post
* Remove stocks/disc/cnews command from terminal and sdk * Remove disc/cnews tests and test data * Remove cnews from default sources, translations and intros * Remove old test data files that are not used anymore * Add pydocstyle ignore statement to pyproject toml * Fix print help in stocks/disc controller test
) * improve setting/loading streamlit session state variables * Update streamlit_helpers.py * fix margins * fix hex color markdown showing on forecast tables --------- Co-authored-by: James Maslek <[email protected]>
* futures curve: prettifies contract names in raw table and in excel export. change column header to Last Price. set x axis of curve chart to correspond to contract months * fixed linting and updated tests --------- Co-authored-by: Danglewood <[email protected]> Co-authored-by: James Maslek <[email protected]>
* init * updates * Update loggers.py * Update loggers.py * sdk handling * feature flag * fix MOCKING_USER_PATH for windows * Update loggers.py * Update plotly.html * updates * Update parent_classes.py * Update plotly.html * add posthog to deps * Update loggers.py
* fix: remove old commands * fix: ruff * fix: ruff * fix: james suggestions * fix: added to csv * fix: generate sdk cmds * bump pywry * fix rm command (#4890) Co-authored-by: Henrique Joaquim <[email protected]> * Fix `crypto/defi/anchor` (#4892) * handle defi anchor exceptions * reason * mypy --------- Co-authored-by: Henrique Joaquim <[email protected]> * fix windows pyinstaller subprocess fail if space in path * remove API_IEX_TOKEN * remove API_SENTIMENTINVESTOR_TOKEN * make -h actually work * black ./ --------- Co-authored-by: James Maslek <[email protected]> Co-authored-by: teh_coderer <[email protected]> Co-authored-by: montezdesousa <[email protected]> Co-authored-by: Henrique Joaquim <[email protected]>
* Removed empty row in OpenBB SDK table under Installation * Revamped `get_estimates` function * Linting * Fixed bugs, tests and modified args * Fixed test
…e yFinance. (#4937) * adds options helper function for DTE * removes logging decorator * More decorators --------- Co-authored-by: James Maslek <[email protected]>
…o for Mutual Funds (#4917) * updates Terminal Intro Guide for Equity Options * adds the images. * grammar * makes changes * adds Terminal Intro Guide for Mutual Funds * adds screenshots to mutual funds intro * html correction * Translate Swedish description into English for Codespell * remove unusual options table * updates some links in Terminal FAQ
* Fix type check of std argument in bbands * Fix std input handling * Fix chart legend * Fix linting * Fix int display in legend --------- Co-authored-by: Danglewood <[email protected]>
* enhances stocks/quote to allow for multiple tickers * fix for MyPy type error * fix pylint * formats index string as title * adds test for multi-ticker quote and for bad tickers * adds integration test for multi-ticker quote * updates docstrings and adds 'openbb.stocks.quote_chart' to SDK * updates syntax in equity report for quote_data * ruff * gets rid of empty lines that got created. * adds logic to handle input as string when not a list * re-records tests * decorators
Fix api-key md file in docs
…tocks/candle (#4979) * Adds a flag for Heikin Ashi Candles on stocks/candle * ruff * pylint * doc strings * fix typo * update integration test file * updates the docs accordingly. * remove second if statement * requested change * adds check for the expected columns --------- Co-authored-by: teh_coderer <[email protected]> Co-authored-by: James Maslek <[email protected]>
* fix biztoc print on launch * missed something
* init * fix tables * add shortcuts * handle svgs * test * test2 * test3 * Update loggers.py * Update sdk_helpers.py * Update loggers.py * Update loggers.py * updates * updates * update * handle add text high/low for candlestick traces when updating * adding save overlay to tables * maybe fix * prints * asdf * working * working * removing prints * update * fix no figure title text * add trace labels space * download finished popup for MacOS * bump pywry * update * update table * Update plotly_helper.py * only format numbers over 10_000 on tables * Delete generate_index.py * turn off logs in generations * update tables * bump pywry, fix tables, added ichimoku ta indicator * Update backend.py * mypy * update tables * updates * juan more * tables * test * bump pywry * bump pywry * Update loggers.py * test * fixing some formatting * add update button to indicators dashboard * fix oecd dates to be full dates * prepare charts/tables for pywry update * Update poetry.lock * Update finviz_view.py * tests * tests * tests * tests * testss? * tests * Create test_call_func[call_sec-nasdaq_view.sec_filings-other_args37-called_with37].yaml * testtss please * Update loggers.py * testss * `with open(newline="\n")` * tests * tests -__- * why * Revert "`with open(newline="\n")`" This reverts commit 554e0f5. * Revert "why" This reverts commit 89f1d9b. * Revert "tests -__-" This reverts commit a9c694a. * Revert "tests" This reverts commit 5463249. * whelp * . * rename `streamlit.py` -> `streamlit_run.py`, bump pywry/finviz/streamlit * Update backend.py * update * Revert "update file" --------- Co-authored-by: andrewkenreich <[email protected]> Co-authored-by: James Maslek <[email protected]>
* improving handling of logging handlers * removing alias * lint * removing debug print * adding alias back * adding the commithash to the system so it gets logged * doing posthog on hub_user login * ruff * tacking this on - docs update * identify and alias together * fixing conditions * the juan way * ruff * don't send logs to S3 if posthog is an active handler * branch name on the posthog logs * removing duplicate import * fixing tests --------- Co-authored-by: andrewkenreich <[email protected]> Co-authored-by: James Maslek <[email protected]>
* init * fix tables save * Update .gitignore * Update utils.ts * add volume to csv, fix `-` filename * fixes #5020 * tests * License year * tests v1 * tests? * Revert "tests?" This reverts commit f837a6e. * fix posthog * Update test_get_debt.yaml * tests --------- Co-authored-by: James Maslek <[email protected]>
Co-authored-by: James Maslek <[email protected]>
# Description Simply edit to explain new feature - [ ] Summary of the change / bug fix. - [ ] Link # issue, if applicable. - [ ] Screenshot of the feature or the bug before/after fix, if applicable. - [ ] Relevant motivation and context. - [ ] List any dependencies that are required for this change. # How has this been tested? * Please describe the tests that you ran to verify your changes. * Provide instructions so we can reproduce. * Please also list any relevant details for your test configuration. - [ ] Make sure affected commands still run in terminal - [ ] Ensure the SDK still works - [ ] Check any related reports # Checklist: - [ ] I have adhered to the GitFlow naming convention and my branch name is in the format of `feature/feature-name` or `hotfix/hotfix-name`. - [ ] Update [our documentation](https://openbb-finance.github.io/OpenBBTerminal/) following [these guidelines](https://github.com/OpenBB-finance/OpenBBTerminal/tree/main/website). Update any user guides that are affected by the changes. - [ ] Update our tests following [these guidelines](https://github.com/OpenBB-finance/OpenBBTerminal/tree/main/tests). - [ ] Make sure you are following our [CONTRIBUTING guidelines](https://github.com/OpenBB-finance/OpenBBTerminal/blob/main/CONTRIBUTING.md). - [ ] If a feature was added make sure to add it to the corresponding [integration test script](https://github.com/OpenBB-finance/OpenBBTerminal/tree/develop/openbb_terminal/miscellaneous/integration_tests_scripts). # Others - [ ] I have performed a self-review of my own code. - [ ] I have commented my code, particularly in hard-to-understand areas. Co-authored-by: James Maslek <[email protected]>
* Update backend.py * bump pywry * bump again :/
Reviewpad Report Fatal
|
tehcoderer
approved these changes
May 16, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM SHIP IT 🚢🛳️
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.
Title