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

move get_branch request to logging_settings #5348

Merged
merged 6 commits into from
Aug 18, 2023

Conversation

montezdesousa
Copy link
Contributor

Description

  • 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:

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 XS Extra small feature label Aug 18, 2023
@montezdesousa montezdesousa marked this pull request as ready for review August 18, 2023 15:49
@montezdesousa montezdesousa merged commit 07dd848 into feature/openbb-sdk-v4 Aug 18, 2023
3 of 4 checks passed
@montezdesousa montezdesousa deleted the hotfix/system_init branch August 18, 2023 15:56
github-merge-queue bot pushed a commit that referenced this pull request Apr 29, 2024
* rename sdk_token to pat to follow hub convention

* black

* move OpenBBError class to error model

* add OBBject class docstring

* pylint

* add futures to app docstring

* Linting

* Fix chart-related mypy errors. Co-Author: ’hjoaquim’

* Fix more mypy warnings

* Build static

* MyPy improvements

* move get_branch request to logging_settings (#5348)

* move get_branch request to logging_settings

* black

* add todo

* Revert "add todo"

This reverts commit 7e3af83.

* add todo

* Update logging_settings.py

* mac/linux install script python output suppressed

* rename .settings to .user for consistency

* update readme

* update readme

* Linting

* Linting and improvements

* Regenerate static files

* Snake_case, mypy, typing, improvements. | Co-Author: montezdesousa

* Rebuild

* Revert Fetcher

* better cpi

* Snake case

* fixes: major_indices_eod for intraday,  optional company_overview atrs

* Feature/remove unncessary stuff (#5346)

* removing the journal service from the command_runner

* remove the journal service and fix tests

* removing journal from the command_runner

* removing journal entry references and adding optional metadata

* rebuilding static assets

* removing all references to journal entry

* removing all references to journal and journal repository

* removing journal query references

* restructuring the command runner

* fix argument mismatch and settings to user mismatch

* dropping command runner session

* re generation of static assets

* ruff n black

* removing introduced bug

* removing duplicate reference due to merging

* 3.8 compatible static assets

* changing the way we deal with erros - removing the Error model

* removing unncessary filters

* fixing tests

* regenerating static assets

* remving mongodb references from user service

* remving mongodb references

* updating dependencies with removed pymongo

* updating dependencies with removed pymongo

* removing useless call

* fix typo

* ruff

* fixing tests

* ruff

* removing leftovers

* making registry map 3.8 compatible

* static assets

* ruff

* Hotfix/squeeze import (#5358)

* remove commit_hash and branch references

* lazy load stats, scipy, pandas_ta

* black & ruff

* adding commithash back to log prefix

* fix test

---------

Co-authored-by: hjoaquim <h.joaquim@campus.fct.unl.pt>

* .user got lost in merge

* improve __repr__ for sdk classes

* changing posthog project key (#5362)

* Linting and improvements

* Rebuild QA

* fix py38 + metadata ref in docstrings

* doc ident

* restore qa

* update readme: remove journal

* doc detail

* disable black in App docstring

* update readme: command execution

* ruff

* remove run_in_isolation: cannot pickle nested classes

* remove outdated dir openbb_sdk

* outdated workflow

* update ruff

* dependency related changes

* using model validator instead

* adjustments to the system model

* minor adjustments

* revert all changes on stocks controller

* working historical price

* controller_builder file

* process class recursivly

* adding translators prop

* poc with platform as a menu and as command

* paths feature

* adding subcontrollers

* offline access

* menu text as obj variable

* adjustments

* move help to base class; print tables using rich table (for now); clear the choices for better autocompletion

* introducing the platform controller factory

* remove unnecessary file

* revert changes to files

* dinamically generate platform menus

* generated platform commands

* poetry lock

* fix for literals and choices

* adjustments - recursive iteration only if it's a class

* router name should be lower() to match the class processor

* deps

* better output

* handling base models

* comment call_news

* Feature/v4 deprecate everything (#6113)

* comment call_news

* remove news

* remove keys controller

* remove settings controller

* remove ta forex

* remove qa economy

* remove ta stocks

* remove qa stocks

* remove fa stocks

* remove ins stocks

* remove gov stocks

* remove dps stocks

* remove ba stocks

* remove bt stocks

* remove stocks

* remove stocks base controller class

* ruff

* remove qa forex

* remove oanda

* remove forex

* remove forex

* remove forex

* remove mention to fx_tickers

* remove ta crypto

* remove qa crypto

* remove dd crypto

* remove crypto controller

* remove crypto references

* remove economy controller

* remove economy folder

* remoe etf controller

* remove reports controller

* remove dashboards controller

* remove alt controller

* rmeove etf files

* remove alt files

* remove custom reports

* remove dashboard files

* remove econometrics controller

* remove forecast controller

* remove reports files

* remove forecast files

* remove econometrics files

* remove portfolio

* remove futures

* remove fixedincome

* remove funds

* remove sdk and common

* remove all from init

* remove reference to ipykernel

* remove reference to ta_helpers

* remove reference to ta_helpers

* cleanup

* adjustments to choices

* remove old sdk generator

* remove mention to dashboards

* ruff

* cleanup

* remove logging infra

* remove decorators

* remove sdk audit

* remove tought of the day

* remove tought of the day

* fix commands andmenus choices

* fixing lints

* ruffs

* remove old tests

* deps trim

* using container as condition for recursion

* remove unnecessary dependencies

* move dep to better section

* python versions

* poetry lock

* better custmo annotations

* handling properties

* handling edge cases: no func_doc and when signarute doesn't match type hints (self)

* reverting treating models as props

* provider groups

* comment debug stuff

* Revert "comment debug stuff"

This reverts commit f0a86f0.

* comment debug stuff

* little adjustment for annotated types

* injecting the reference to make the custom argument groups available

* comments

* custom model_dump to drop choices

* deprecated

* remove update option

* remove update

* remove translations

* remove account controller

* command descriptions

* menu descriptions

* removing interactive tables

* limit the number of columns and rows

* menu descriptions

* show indexes

* More cleanups (#6245)

* renames

* remove .init files

* remove unused files

* remove session stuff

* remove session and sources stuff

* remove unused models

* remove sdk session

* remove charting infra

* remove charting stuff but allow console style

* updates in models

* raise directly

* call terminal directly and remove artifacts

* remove reference to sources

* remove reference to source arg

* remove reference to source arg

* remove unused stuff

* remove screenshot and raise directly

* remove unnecessary helper classes

* update the config file

* remove references to old charting infra

* call terminal directly

* update deps

* make user always local for now

* rename to CLI

* ruff

* export options

* export functionality

* update rich

* update example routine

* fix issues with queue

* temporary workaround for username

* deprecate main menu commands

* deprecate commands and remove unused stuff

* deprecate commands and remove unused stuff

* deprecate commands and remove unused stuff

* adjustments and cleanups

* routines syncing w hub

* small adjustments to point to the right instance

* whoami

* custom terminal style

* adjustments related to preferences

* deprecate stuff

* rename terminal style

* deprecate more things

* more deprecation

* more adjustments

* current settings - deprecating preferences, user and system

* terminal settings

* revmap custom terminal settings

* cli version

* ruff

* black

* linting fixes

* happy mypy

* ruff

* remove old terminal build files

* remove devcontainer

* update dependencies

* trim commands and its descriptions

* better warnings

* remove commands and menus str and show warnings if dev mode

* get command descriptions from the translators instead

* Add username in brackets if logged in

* fix buggy command descriptions

* [Feature] Bug fixing the Platform CLI (#6291)

* fix buggy command descriptions

* new reference structure

* Seggregate data commands from others (settings etc)

* fix table html for charting extension

* number of rows and columns as a setting

* interactive tables

* adding menu descriptions

---------

Co-authored-by: hjoaquim <h.joaquim@campus.fct.unl.pt>

* fix the way reference was being passed

* fix choices parsing for providers

* loosen to get instead

* obbject registry class

* obbject registry class

* using the obbject registry to show available options

* link obbject results to data processing commands data argument

* small improvements to correctly get the standard args

* intersecting data processing commands and matching the obbject trough its id

* poetry lock

* use openbb field

* safe json conversion and trimming

* better at finding nargs when union

* improved obbject registry using a stack

* data processing section

* Update argparse_translator/argparse_class_processor.py

Co-authored-by: montezdesousa <79287829+montezdesousa@users.noreply.github.com>

* fix: remove unused function

* fix: terminal.py -> openbb_terminal/cli.py

* fix: move local model function to utils & delete module

* fix: rename hub_model.py to hub_service.py

* fix: delete paths_helper.py, not used

* fix: circular import console

* fix: move terminal_style.py from plots/ to config, delete plots/

* fix: delele base_helpers.py, function moved to env_handler.py

* fix: move custom_prompt_toolkit.py to core/completer

* fix: delete requirements.txt, we can export them later if needed

* fix: move routines stuff to routines/

* fix: move rich_config.py to core/config

* fix: rename parent_classes.py to base_controller.py

* Danger: create controllers/ and move stuff

* fix: move custom_prompt_toolkit.py to config and delete completer/

* fix: rename custom_prompt_toolkit.py to completer.py

* fix: move rouines stuff to folder

* ^ oops

* Edits

* fix: settings import

* fix: openbb command

* Fix routine record when local user

* Lint

* Remove quickexit

* Fix timezone change

* fix data processing commands: not allowing multiple items + need to allow 0 (which evalutes as false)

* split comma separated strings into a list so that the cli can consume them

* Feature/move terminal files (#6344)

* Move terminal files into openbb_terminal

* move routines

* Create session to handle console, user, settings, etc.

* Reset...

* Bring back frontend-components

* Move frontend-components to openbb_terminal

* rename vars

* remove doc

* rename method

* rename method

* fix routines

* read settings from .env

* rename methods

* ^

* ^

* better

* bug

* ^

* move frontend-components back to root level

* revert changes

* type checking only

* docstring

* fix: settings autocomplete

* comment

* use openbb_core singleton

* rename src to fix openbb command

* fix poetry build

* readme

---------

Co-authored-by: hjoaquim <h.joaquim@campus.fct.unl.pt>

---------

Co-authored-by: Diogo Sousa <montezdesousa@gmail.com>
Co-authored-by: Igor Radovanovic <74266147+IgorWounds@users.noreply.github.com>
Co-authored-by: montezdesousa <79287829+montezdesousa@users.noreply.github.com>
Co-authored-by: Pratyush Shukla <ps4534@nyu.edu>
Co-authored-by: tehcoderer <me@tehcoderer.com>
Co-authored-by: Theodore Aptekarev <aptekarev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat XS Extra small feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants