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

Add plotly figure support #381

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Add plotly figure support #381

merged 4 commits into from
Sep 24, 2024

Conversation

thomass-dev
Copy link
Collaborator

@thomass-dev thomass-dev commented Sep 23, 2024

  • backend
  • frontend

@thomass-dev thomass-dev linked an issue Sep 23, 2024 that may be closed by this pull request
requirements-test.txt Show resolved Hide resolved
requirements-tools.txt Show resolved Hide resolved
requirements.txt Show resolved Hide resolved
@rouk1
Copy link
Contributor

rouk1 commented Sep 24, 2024

Frontend widget is now available.
Screenshot 2024-09-24 at 13 25 07

@rouk1 rouk1 marked this pull request as ready for review September 24, 2024 11:26
Copy link
Contributor

Coverage Report for ./frontend

Status Category Percentage Covered / Total
🔵 Lines 83.8% 1195 / 1426
🔵 Statements 83.8% 1195 / 1426
🔵 Functions 60.81% 45 / 74
🔵 Branches 82.99% 122 / 147
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
frontend/src/components/PlotlyWidget.vue 100% 100% 100% 100%
frontend/src/components/ReportCanvas.vue 100% 100% 100% 100%
frontend/src/views/ReportBuilderView.vue 73.38% 80% 16.66% 73.38% 1, 33-34, 50-61, 63-65, 67-73, 75-77, 79-83
Generated in workflow #12

@thomass-dev
Copy link
Collaborator Author

approved ✅

Copy link
Contributor

@augustebaum augustebaum left a comment

Choose a reason for hiding this comment

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

Ok for me!

@augustebaum augustebaum merged commit 06d84b0 into main Sep 24, 2024
6 checks passed
@augustebaum augustebaum deleted the plotly-support branch September 24, 2024 12:24
thomass-dev added a commit that referenced this pull request Dec 2, 2024
Co-authored-by: Matthieu Jouis <matthieu@probabl.ai>
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.

Support for Plotly
3 participants