Skip to content

Commit

Permalink
Fix multiple crypto bugs (#1903)
Browse files Browse the repository at this point in the history
* fix #1850

* fix #1831

* add extra check to Reddit API keys

* ignore warning message to update praw api

* improve OpenBB links

* fix quick performance only on stocks class because I'm James bitch

* fix quick performance only on stocks class because I'm James bitch

* fix #1829

* fix #1821

* add messari to keys - fix #1819

* example of multiple oclumns to check on options/chains

* minor improvement in xlabel re. #1814

* remove repeated command

* fix #1698

* fix line too long

* fix #1814 fr now

* fix tests

* Fix multiple bugs

* Disable false positive pylint in some crypto models

* Fix website not building because of expo command removed

Co-authored-by: DidierRLopes <dro.lopes@campus.fct.unl.pt>
Co-authored-by: Theodore Aptekarev <aptekarev@gmail.com>
  • Loading branch information
3 people authored Jun 6, 2022
1 parent 335b89c commit 4c02dc2
Show file tree
Hide file tree
Showing 7 changed files with 5 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ def display_crypto_sentiment_analysis(
)
ax.axhline(y=0, linestyle="--")
ax.set_xlabel("Time")
ax.set_ylabel("Sentiment")
ax.set_ylabel("Finbrain's Sentiment Score")
start_date = sentiment.index[-1].strftime("%Y/%m/%d")
ax.set_title(f"FinBrain's Sentiment Analysis for {coin}-USD since {start_date}")
ax.set_ylim([-1.1, 1.1])
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ def display_marketcap_dominance(
else:
return

ax.plot(df.index, df["values"])
ax.plot(df.index, df["marketcap_dominance"])

ax.set_title(f"{coin}'s Market Cap Dominance over time")
ax.set_ylabel(f"{coin} Percentage share")
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
"""CoinGecko model"""
__docformat__ = "numpy"
# pylint:disable=unsupported-assignment-operation

import logging
from typing import Any, Dict, List, Optional, Tuple, Union
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ def display_global_market(export: str) -> None:

df = paprika.get_global_market()
df_data = df.copy()
df["Value"] = df["Value"].apply(
df["Value"] = df["Value"].apply( # pylint:disable=unsupported-assignment-operation
lambda x: lambda_long_number_format_with_type_check(x)
)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,7 @@ def filter_stocks(
industry=industry,
exclude_exchanges=exclude_exchanges,
)
print(data.keys())
else: # no industry
data = fd.select_equities(
country=country,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ def vcr_config():
}


@pytest.mark.skip(reason="`subplots` not mocked => calling displays the plot")
@pytest.mark.vcr
@pytest.mark.record_stdout
def test_display_btc_rainbow(mocker):
Expand Down
4 changes: 0 additions & 4 deletions website/data/menu/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1197,8 +1197,6 @@ main:
ref: "/terminal/common/prediction_techniques/conv1d"
- name: mc
ref: "/terminal/common/prediction_techniques/mc"
- name: expo
ref: "/terminal/common/prediction_techniques/expo"
- name: funds
ref: "/terminal/funds"
sub:
Expand Down Expand Up @@ -1662,8 +1660,6 @@ main:
ref: "/terminal/common/prediction_techniques/conv1d"
- name: mc
ref: "/terminal/common/prediction_techniques/mc"
- name: expo
ref: "/terminal/common/prediction_techniques/expo"
- name: "OpenBB Python API"
ref: "/api"
- name: "OpenBB Bots"
Expand Down

0 comments on commit 4c02dc2

Please sign in to comment.