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.

Sign up for GitHub

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

Jump to bottom

Feat/strategy performance v 0.20 #100

Merged
merged 17 commits into from
Nov 24, 2023
Merged

Feat/strategy performance v 0.20 #100

merged 17 commits into from
Nov 24, 2023

Conversation

tomasgaudino
Copy link
Collaborator

@tomasgaudino tomasgaudino commented Nov 20, 2023 •
edited
Loading

VERSION 0.20

  • Features:
    • Add position executor data into summary table
    • Add position executor sunburst
    • Add positions to candlestick
    • Improve candlestick's hovertext
    • Add linemode for min interval resolution
    • Add metrics hints to avoid confusion
    • Improve dashboard layout

@tomasgaudino
(feat) add performance graphs class
78051f3
@tomasgaudino
(feat) add get databases function
2e4e485
@tomasgaudino
(feat) add download csv button and style metrics cards functions
a54a1e3
@tomasgaudino
(feat) implement performance charts class in strategy performance pag…
4def4d6 
…e + general refactor
@tomasgaudino
(refactor) remove unnecessary variables
79f53de
@tomasgaudino
(feat) add position executor data
a3527be
@tomasgaudino
(refactor) rename download csv button
0d5bfed
@tomasgaudino
(feat) improve has summary table property
dedfbca
@tomasgaudino
(feat) add position executor download csv
6f03027
@tomasgaudino
(feat) add position executors sunburst + general refactor
a3f81e9
@tomasgaudino
(feat) add position executor data to summary table
0a21a89
@tomasgaudino
(feat) add positions to candlesgraph
1125a17
@tomasgaudino
(feat) improve market activity layout
814d409
@tomasgaudino
(feat) add metrics hint texts + rename returns tab
d2fc254
@tomasgaudino
(fix) handle missing position executor data
723c948
@tomasgaudino
(feat) add candles graph line mode for min interval resolution
52c7067
@tomasgaudino
(feat) improve candles hovertexts
f6995c7
@tomasgaudino tomasgaudino marked this pull request as ready for review November 23, 2023 20:53
cardosofede
cardosofede approved these changes Nov 24, 2023
View reviewed changes
Copy link
Contributor

@cardosofede cardosofede left a comment

Choose a reason for hiding this comment

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

LGTM!

@cardosofede cardosofede merged commit 16acd78 into main Nov 24, 2023
@cardosofede cardosofede deleted the feat/strategy-performance-v-0.20 branch November 24, 2023 20:25
@cardosofede cardosofede restored the feat/strategy-performance-v-0.20 branch November 24, 2023 20:28
@cardosofede cardosofede deleted the feat/strategy-performance-v-0.20 branch November 24, 2023 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cardosofede cardosofede cardosofede approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tomasgaudino @cardosofede