Skip to content

Conversation

souhailaS
Copy link
Contributor

@souhailaS souhailaS commented Jul 16, 2025

Brief Information

There is a an optimization opportunity for the ci.yml workflow, by taken advantage of shared cashing across the two jobs. There is no need to have a separate cache for each of the jobs.

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

The PR simplifies the workflow while preserving all it functionalities. Both jobs install the same dependencies, so they can share the same cache instead of having separate lint-and build-cache keys. Which increase cache hit rates.

Change impact

  • Shared caching: Both jobs now use the same cache key, so the second job can reuse the first job's cached dependencies
  • Cleaner configuration: Removed unnecessary environment variables
  • Same functionality: All the same commands run in the same order with identical results

You can compare the CPU time taken by the workflow's old runs with the test runs in here: https://github.com/souhailaS/echarts/actions/runs/16315916920

Other information

We are a team of researchers from University of Zurich (https://www.ifi.uzh.ch/en/zest.html) currently working on automating energy optimizations in GitHub Actions workflows. This optimization maintains full functionality while reducing computational overhead and energy consumption.

[email protected]

@souhailaS souhailaS changed the title [CI Energy Waste] Optimized ci [CI] Optimized ci Jul 16, 2025
Copy link
Contributor

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-21110@6e8e514

@plainheart plainheart merged commit c8a12c6 into apache:master Jul 16, 2025
2 checks passed
@souhailaS souhailaS changed the title [CI] Optimized ci [ CI ] Optimized ci Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants