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 ECO-CI Project token #2090

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add ECO-CI Project token #2090

wants to merge 5 commits into from

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Feb 28, 2025

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • Tests for the changes have been added (for bug fixes / features)
    • (If applicable) Documentation has been added / updated (for bug fixes / features)
  • CHANGELOG.rst has been updated (with summary of main changes)
    • Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

  • Adds project tracking to the ECO-CI Action

Does this PR introduce a breaking change?

No.

Other information:

Project and organization-level tracking is a "paid" service. We have access to a relatively liberal monthly allowance of minutes thanks to the generous offer of the creator of the project.

One thing worth looking at is reducing the number of full builds we do. The monthly minute usage of xclim alone is really high (60-70% of minutes used in Ouranosinc are here). We can reduce this and still maintain effective testing/coverage.

I also needed to create a commercial (free) API token with https://www.electricitymaps.com/ in order to receive power and carbon intensity statistics.

@Zeitsperre Zeitsperre added the enhancement New feature or request label Feb 28, 2025
@Zeitsperre Zeitsperre self-assigned this Feb 28, 2025
@github-actions github-actions bot added the CI Automation and Contiunous Integration label Feb 28, 2025
Copy link

Note

It appears that this Pull Request modifies the main.yml workflow.

On inspection, the XCLIM_TESTDATA_BRANCH environment variable is set to the most recent tag (v2025.1.8).

No further action is required.

@Ouranosinc Ouranosinc deleted a comment from github-actions bot Feb 28, 2025
@Zeitsperre Zeitsperre marked this pull request as draft February 28, 2025 19:00
@Zeitsperre
Copy link
Collaborator Author

This PR is waiting on some clarifications from the ECO-CI developers concerning the ElectricityMaps integration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Automation and Contiunous Integration enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant