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

Feature: single_statistic_table heatmap #1163

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

emhbrine
Copy link
Contributor

@emhbrine emhbrine commented Oct 30, 2023

Additional parameters added to SignalsReturns.single_statistic_table():

:param <bool> show_heatmap: if True, the table is visualized as a
        heatmap. Default is False.
:param <str> title: plot title; if none given default title is printed.
:param <List[str]> row_names: specifies the labels of rows in the heatmap.
        If None, the indices of the generated DataFrame are used.
:param <List[str]> column_names: specifies the labels of columns in the
        heatmap. If None, the columns of the generated DataFrame are used.
:param <float> min_color: minimum value of the color scale. Default
        is None, in which case the minimum value of the table is used.
:param <float> max_color: maximum value of the color scale. Default
        is None, in which case the maximum value of the table is used.
:param <Tuple[float]> figsize: Tuple (w, h) of width and height of graph.
:param <bool> annotate: if True, the values are annotated in the heatmap.

single_stat_heatmap

@emhbrine emhbrine force-pushed the feature/signal_returns_visuals branch 2 times, most recently from 93e307d to e0a25a3 Compare October 30, 2023 16:55
@emhbrine emhbrine changed the title plotting for signal returns Feature: single_statistic_table heatmap Oct 30, 2023
@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #1163 (76ce9b5) into develop (240e735) will increase coverage by 0.19%.
Report is 1 commits behind head on develop.
The diff coverage is 100.00%.

❗ Current head 76ce9b5 differs from pull request most recent head 8ab1e44. Consider uploading reports for the commit 8ab1e44 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1163      +/-   ##
===========================================
+ Coverage    75.78%   75.98%   +0.19%     
===========================================
  Files           57       58       +1     
  Lines         4923     4963      +40     
===========================================
+ Hits          3731     3771      +40     
  Misses        1192     1192              
Files Coverage Δ
macrosynergy/signal/signal_return.py 94.82% <100.00%> (+0.31%) ⬆️
macrosynergy/visuals/__init__.py 100.00% <100.00%> (ø)
macrosynergy/visuals/table.py 100.00% <100.00%> (ø)

Impacted file tree graph

@emhbrine emhbrine force-pushed the feature/signal_returns_visuals branch from 0c3da73 to beb8a51 Compare October 31, 2023 11:38
@emhbrine emhbrine force-pushed the feature/signal_returns_visuals branch from beb8a51 to db1e7e3 Compare October 31, 2023 11:39
@emhbrine emhbrine marked this pull request as ready for review October 31, 2023 11:41
@emhbrine emhbrine requested review from Magnus167, a team and rsueppel as code owners October 31, 2023 11:41
@emhbrine emhbrine merged commit 3b04514 into develop Oct 31, 2023
4 checks passed
@emhbrine emhbrine deleted the feature/signal_returns_visuals branch October 31, 2023 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants