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

Fix 3188 - UX for Feature Engineering functions #3224

Merged
merged 10 commits into from
Nov 1, 2022
Merged

Fix 3188 - UX for Feature Engineering functions #3224

merged 10 commits into from
Nov 1, 2022

Conversation

martinb-ai
Copy link
Contributor

Description

Fixes all issues mentioned in: #3188

Checklist:

Others

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

@martinb-ai martinb-ai added bug Fix bug feat S Small T-Shirt size Feature labels Oct 31, 2022
@martinb-ai martinb-ai self-assigned this Oct 31, 2022
@martinb-ai martinb-ai changed the title Feature Engineering UX improvements UX improvements: Feature Engineering functions Oct 31, 2022
@martinb-ai martinb-ai changed the title UX improvements: Feature Engineering functions Fix 3188 - UX for Feature Engineering functions Oct 31, 2022
Copy link
Collaborator

@DidierRLopes DidierRLopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

@jmaslek jmaslek merged commit 58d2799 into main Nov 1, 2022
@martinb-ai martinb-ai deleted the iloc-updates branch November 1, 2022 02:10
hjoaquim pushed a commit to hjoaquim/OpenBBTerminal that referenced this pull request Nov 1, 2022
* fix depr. warning

* console printout enhancment

* improvement to sto for UX

* average true range UX

* signal UX improv.
jmaslek added a commit that referenced this pull request Nov 1, 2022
* improved integration tests for forex

* integration test for dashboards

* added more tests to economy

* added test for forecast report

* added eedbt integration test

* added hist tests

* added eval command

* updatedeconometrics tests

* changed order

* test removing warning

* Revert "test removing warning"

This reverts commit 09ca09012156771428b2041b478296dbf2bfc780.

* adding todo

* Fix top categories and sorting not working (#3025)

* Fix categories and sorting not working

* Allow sort in ascending order

* Documented the new flag

* Fixed tests

* Fix bad practises

Co-authored-by: James Maslek <[email protected]>

* [FEAT] StatsForecast AutoETS forecasting model (#2988)

* feat: add autoets model

* feat: update autoets index

* feat: add verbose option

* feat: freq preprocessing

* feat: improve test size call

* feat: add statsforecast dep to conda env full

Co-authored-by: James Maslek <[email protected]>
Co-authored-by: martinb-bb <[email protected]>

* Improve ETF menu (#3124)

* improve etf menu

* improve docs

* fix tests

* should cover more edge cases

Co-authored-by: James Maslek <[email protected]>

* improve export (#3176)

* When renaming funcs to remove source at start, the translating keys were missing update (#3177)

* fix the en.yml key for some funcs

* fix tests

Co-authored-by: James Maslek <[email protected]>

* Adds guide for SDK/Keys (#3111)

* Adds guide for SDK/Keys

* Linter

* Applied requested changes, and improved the examples.

* Removed bullet point

Co-authored-by: Jeroen Bouma <[email protected]>

* Adds Introduction Guide for the Economy SDK Module (#3142)

* Adds Introduction Guide for the Economy SDK Module

* Changes title

Co-authored-by: Jeroen Bouma <[email protected]>

* version check for statforecast (#3227)

* Fix 3188 - UX for `Feature Engineering` functions  (#3224)

* fix depr. warning

* console printout enhancment

* improvement to sto for UX

* average true range UX

* signal UX improv.

* Fix Some Econometrics Bugs (#3210)

* Added suggestions for load

* Fixed a command

* Fixed panel

Co-authored-by: Henrique Joaquim <[email protected]>
Co-authored-by: James Maslek <[email protected]>

* Fixing 6 crypto bugs (#3228)

* fixing 6 crypto bugs

* fixing filter stuff

Co-authored-by: Colin Delahunty <[email protected]>

* Fix SDK plot colors without needing openbb.stocks.candle (#3178)

* fix SDK plot colors without needing openbb.stocks.candle

* lint

* ordering

Co-authored-by: James Maslek <[email protected]>
Co-authored-by: Colin Delahunty <[email protected]>

* Updating crypto integration tests (#3222)

* disc, ov, and base

* more integration tests and fixing previous commited ones

* some more fixes

Co-authored-by: Colin Delahunty <[email protected]>

* removing commented tests

* removing commented tests

Co-authored-by: Gerard <[email protected]>
Co-authored-by: James Maslek <[email protected]>
Co-authored-by: fede <[email protected]>
Co-authored-by: martinb-bb <[email protected]>
Co-authored-by: DidierRLopes <[email protected]>
Co-authored-by: Danglewood <[email protected]>
Co-authored-by: Jeroen Bouma <[email protected]>
Co-authored-by: Colin Delahunty <[email protected]>
Co-authored-by: James Simmons <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug feat S Small T-Shirt size Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants