-
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
Removed duplicate chart #4511
Removed duplicate chart #4511
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #4511 +/- ##
==========================================
Coverage ? 54.54%
==========================================
Files ? 584
Lines ? 52653
Branches ? 0
==========================================
Hits ? 28722
Misses ? 23931
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
It's truly fixed now🤣... They had the view inside of the iterate loop in the controller🤦♂️. Improved the controller and view overall. 😘🥰🚀 |
* Removed duplicate chart * Removed duplicate chart * improved and fixed `call_index` in economy controller * Update preferences_model.py * Improve handling for dot in ticker --------- Co-authored-by: teh_coderer <me@tehcoderer.com>
Description
Fixes #3442
Stops showing the old, hard to read, chart. Only shows the new one with dual axises.
How has this been tested?
economy/index --indices sp500,cboe_vix -s 2022-10-10/plot --y1 sp500 --y2 cboe_vix
Checklist:
feature/feature-name
orhotfix/hotfix-name
.Others