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

Strategy performance chart - convert to ChartIQ #355

Closed
Tracked by #350
kenkunz opened this issue Feb 14, 2023 · 0 comments · Fixed by #397
Closed
Tracked by #350

Strategy performance chart - convert to ChartIQ #355

kenkunz opened this issue Feb 14, 2023 · 0 comments · Fixed by #397
Assignees
@kenkunz
Labels
enhancement New feature or request priority: P2 Default priority. An issue might not yet be fixed in the next release. size: M An ordinary task that can be completed in a few days or in a week

Comments

@kenkunz
Copy link
Contributor

kenkunz commented Feb 14, 2023 •
edited
Loading

Goal

Improve usability, visual style, interaction consistency and rendering performance of Strategy performance chart.

Details

The Strategy performance chart (example below) currently uses the Plotly charting library. There are several reasons it would be beneficial to migrate this to use ChartIQ.

  • Plotly does not give us as much control over interactivity and visual style as ChartIQ – we would like to provide a more consistent UX / UI experience for charts
  • Current performance chart doesn't support light/dark color modes (has light style only)
  • Plotly dependency currently results in the largest page build size on our site by a wide margin (1150+ KiB)

Upgrading to use ChartIQ should allow us to address all of these issues.

image
@kenkunz kenkunz mentioned this issue Feb 14, 2023
Open
11 tasks
@kenkunz kenkunz self-assigned this Feb 14, 2023
@kenkunz kenkunz added enhancement New feature or request priority: P2 Default priority. An issue might not yet be fixed in the next release. size: M An ordinary task that can be completed in a few days or in a week labels Feb 14, 2023
@kenkunz kenkunz mentioned this issue Feb 14, 2023
Closed
@kenkunz kenkunz mentioned this issue Mar 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kenkunz kenkunz

Labels
enhancement New feature or request priority: P2 Default priority. An issue might not yet be fixed in the next release. size: M An ordinary task that can be completed in a few days or in a week
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Strategy performance chart -> ChartIQ tradingstrategy-ai/frontend
1 participant
@kenkunz