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

170 add csv and latex export buttons #212

Merged
merged 39 commits into from
Feb 6, 2025
Merged

170 add csv and latex export buttons #212

merged 39 commits into from
Feb 6, 2025

Conversation

JavierCladellas
Copy link
Collaborator

@JavierCladellas JavierCladellas commented Jan 16, 2025 •
edited
Loading

Done:

  • Add an "Export CSV" button for each figure
  • Add an "Export LaTeX" button for each figure
  • Refactored figureFactory
  • Changed templates to not use python anymore. Directly use HTML for figures.
  • Bug fixes: Handle report rendering for incomplete or empty data. (Now exceptions are raised on unsuccessful figure generation)

About LaTeX plot generation

A new interface for Tikz/PGF plot generation has been created using templates. At the moment, grouped and stacked bar charts, scatter plots and tables are supported.

[NOTE]
Antora compilation time is now VERY slow (~10 mins), maybe due to large .adoc files (~20MB each)

@JavierCladellas JavierCladellas added enhancement A new feature or request report Tasks related to project reporting visualization Issues related to visualization benchmarking Issues related to benchmarking data-visualization Creating visual representations of data labels Jan 16, 2025
@JavierCladellas JavierCladellas self-assigned this Jan 16, 2025
@JavierCladellas JavierCladellas linked an issue Jan 16, 2025 that may be closed by this pull request
Add CSV and LaTeX export buttons #170
Closed
2 tasks
@netlify Netlify
Copy link

netlify bot commented Jan 16, 2025 •
edited
Loading

✅ Deploy Preview for benchmarking-polite-crostata-92f389 ready!

Name Link
🔨 Latest commit df03043
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/67a0c409c12f290008de125d
😎 Deploy Preview https://deploy-preview-212--benchmarking-polite-crostata-92f389.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JavierCladellas JavierCladellas marked this pull request as ready for review January 16, 2025 13:01
JavierCladellas and others added 20 commits January 16, 2025 16:38
@JavierCladellas
StrategyFactory -> TransformationStrategyFactory
bbb6907
@JavierCladellas
add mock params
f919cb6
@JavierCladellas
add dataGenerationStrategies tests #170
034f6dc
@JavierCladellas
transformationStrategies -> transformationFactory
e8e3da9
@JavierCladellas
dataGenerationStrategies -> dataGenerationFactory #170
d00043e
@JavierCladellas
TestStrategy -> TestFactory
c266413
@JavierCladellas
up download & upload artifact v3 -> v4
6f3c309
@JavierCladellas
Merge pull request #213 from feelpp/up-upload-download-artifact
dbc7645 
up download & upload artifact v3 -> v4
@JavierCladellas
Figure -> PlotlyFigure #170
833d03b
@JavierCladellas
figure -> plotly figure
a133998
@JavierCladellas
start tikz fig generation (bar + scatter) #170
6c05bd4
@JavierCladellas
huge figure refactoring #170
6f4ba18
@JavierCladellas
up scatter figure tikz #170
9ab5344
@JavierCladellas
small fixes and dosctrings #170
3fb2cbc
@JavierCladellas
rm tikzplotly from reqs #170
b2fdf50
@JavierCladellas
rm ymin = 0 #170
34dad25
@JavierCladellas
treat special chars (underscore) #170
dde9410
@JavierCladellas
pgf : support categorical xaxis in plots #170
e900114
@JavierCladellas
fix stacked bar plot #170
c684aed
@JavierCladellas
add quick tikz table #170
df03043
@JavierCladellas JavierCladellas marked this pull request as draft February 3, 2025 13:33
@JavierCladellas JavierCladellas marked this pull request as ready for review February 3, 2025 13:33
@JavierCladellas
Copy link
Collaborator Author

Data should not be on latex files, export csv with latex

@vincentchabannes vincentchabannes merged commit 64c1515 into master Feb 6, 2025
10 checks passed
@vincentchabannes vincentchabannes deleted the 170-add-csv-and-latex-export-buttons branch February 6, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincentchabannes vincentchabannes vincentchabannes approved these changes

@prudhomm prudhomm Awaiting requested review from prudhomm

Assignees

@JavierCladellas JavierCladellas

Labels
benchmarking Issues related to benchmarking data-visualization Creating visual representations of data enhancement A new feature or request report Tasks related to project reporting visualization Issues related to visualization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add CSV and LaTeX export buttons
2 participants
@JavierCladellas @vincentchabannes