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

Refactor reporters and install script #920

Merged
merged 46 commits into from
Oct 1, 2024
Merged

Conversation

ArneTR
Copy link
Member

@ArneTR ArneTR commented Sep 27, 2024

No description provided.

ArneTR added 26 commits June 30, 2024 15:22
* main: (79 commits)
  Missing commit hash should still allow comparison
  Removed non ASCII character
  Missing notes should not throw error
  Simplied access to phase_stats_object
  Bump python from 3.12.5-slim-bookworm to 3.12.6-slim-bookworm in /docker (#898)
  Bump psycopg[binary] from 3.2.1 to 3.2.2 (#900)
  Bump fastapi[standard] from 0.114.0 to 0.114.2 (#899)
  Added new demo data [skip ci] (#892)
  Added re-licensing option
  Bump pytest from 8.3.2 to 8.3.3 (#890)
  Updated Cloud Energy
  Bump pydantic from 2.9.0 to 2.9.1 (#889)
  Adds checks to the install scripts (#886)
  Bump fastapi[standard] from 0.113.0 to 0.114.0 (#887)
  Fetch mode named digestable (#885)
  Clarifications on units in CarbonDB
  Guard clause when non-git root supplied in CLI (#874)
  Bump pydantic from 2.8.2 to 2.9.0 (#883)
  Bump fastapi[standard] from 0.112.2 to 0.113.0 (#884)
  Bump pylint from 3.2.6 to 3.2.7 (#881)
  ...
* main:
  Authentication in GMT (#872)
  Phase name check (#893)
  Carbondb unique (#877)
  Added DiskIO Reporter (#825)
  Bump pylint from 3.3.0 to 3.3.1 (#913)
  Bump aiohttp from 3.10.5 to 3.10.6 (#912)
  Bump plotext from 5.3.1 to 5.3.2 (#914)
  Bump pandas from 2.2.2 to 2.2.3 (#910)
  Bump plotext from 5.2.8 to 5.3.1 (#909)
  Bump pylint from 3.2.7 to 3.3.0 (#911)
  Bump psycopg-pool from 3.2.2 to 3.2.3 (#901)
  Bump pydantic from 2.9.1 to 2.9.2 (#904)
  Bump fastapi[standard] from 0.114.2 to 0.115.0 (#905)
  Importer for Measurements (#908)
@ArneTR ArneTR changed the base branch from main to blue-angel-reporters September 27, 2024 15:19
* blue-angel-reporters:
  Slimmed testing workflow a little
  Skipping a test for GitHub Actions
  Made tests more resilient
  Added memory.used provider to config
  Decimal conversion
  Removed guard clause again. No help
  Renamed file
  Memory test must be longer apparently
  Using 512 if path is not accessible. This can happen in VMs
@ArneTR ArneTR force-pushed the refactor-repoters-and-install branch from 8eadfb8 to 93b8806 Compare September 28, 2024 14:46
Copy link

github-actions bot commented Sep 29, 2024

Old Energy Estimation

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run (incl. overhead) 27.0676 1851.89 4.03 459
Measurement #1 27.0181 1851.89 4.05 457

🌳 CO2 Data:
City: Washington, Lat: 38.7095, Lon: -78.1539
IP: 20.42.51.64
CO₂ from energy is: 0.687051190 g
CO₂ from manufacturing (embodied carbon) is: 0.130959038 g
Carbon Intensity for this location: 371 gCO₂eq/kWh
SCI: 0.818010 gCO₂eq / pipeline run emitted

@ArneTR ArneTR marked this pull request as ready for review September 29, 2024 07:35
@ArneTR ArneTR requested a review from ribalba September 29, 2024 07:35
Base automatically changed from blue-angel-reporters to main September 29, 2024 07:36
@ArneTR ArneTR changed the base branch from main to blue-angel-reporters September 29, 2024 13:29
* main:
  Renamed LM SEnsors
  Removed disk provider from always on
  Blue angel for Software reporters (#916)
@ArneTR ArneTR changed the base branch from blue-angel-reporters to main September 29, 2024 13:35
Copy link

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run (incl. overhead) 25.9238 1802.76 3.98 453
Measurement #1 25.8682 1802.76 4.00 451

🌳 CO2 Data:
City: Boydton, Lat: 36.6534, Lon: -78.375
IP: 20.161.77.240
CO₂ from energy is: 0.596713560 g
CO₂ from manufacturing (embodied carbon) is: 0.129247155 g
Carbon Intensity for this location: 331 gCO₂eq/kWh
SCI: 0.725961 gCO₂eq / pipeline run emitted

Copy link
Member

@ribalba ribalba left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -20,3 +20,4 @@ venv/
lib/hardware_info_root.py
tools/cluster/cleanup.sh
node_modules/
lib/c/parse_int.o
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't we just ignore all *.o files?

@ArneTR ArneTR merged commit 876081f into main Oct 1, 2024
1 check passed
@ArneTR ArneTR deleted the refactor-repoters-and-install branch October 1, 2024 13:24
ArneTR added a commit that referenced this pull request Oct 1, 2024
* main:
  Refactor reporters and install script (#920)
  Makes the disk optional (#926)
  Added uninstall script (#927)
  Bump aiohttp from 3.10.6 to 3.10.8 (#925)
  Bump psycopg[binary] from 3.2.2 to 3.2.3 (#924)
  Bump uvicorn[standard] from 0.30.6 to 0.31.0 (#923)
  Bump redis from 5.0.8 to 5.1.0 (#922)
  Adding dev requirements again
  Typo
  Catching response failure for API request and handling gracefully
  Changing default to ip_api_com as this has shown to be more reliable
  Added timeline_project to demo data
  Changed to -solutions
  Changed to -solutions in container cache and gitmodules
  Groupby error in API
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.

2 participants