Skip to content

Conversation

ribalba
Copy link
Member

@ribalba ribalba commented Sep 2, 2025

Something that annoyed me all the time is that when I do install_linux.sh the install script just overwrites my config files. Especially if I have set up something not trivial and then I switch branches and install to get the right libs everything is gone. Now I just save the config files to a .backup so I can copy paste the things I need.

@ribalba ribalba requested a review from ArneTR September 2, 2025 13:26
Copy link

github-actions bot commented Sep 2, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17405005416]:

🌳 CO2 Data:
City: , Lat: , Lon:
IP:
CO₂ from energy is: 1.540902720 g
CO₂ from manufacturing (embodied carbon) is: 0.388169436 g
Carbon Intensity for this location: 321 gCO₂eq/kWh
SCI: 1.929072 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #118.36874800.323.531360.50
Total Run18.374800.323.531360.50
Additional overhead from Eco CIN/A16.304.543.59

Copy link

github-actions bot commented Sep 2, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17405053110]:

🌳 CO2 Data:
City: Washington, Lat: 38.7095, Lon: -78.1539
IP: 20.42.8.150
CO₂ from energy is: 1.816907950 g
CO₂ from manufacturing (embodied carbon) is: 0.416592397 g
Carbon Intensity for this location: 365 gCO₂eq/kWh
SCI: 2.233500 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #117.93834977.833.411460.12
Total Run17.944977.833.411460.12
Additional overhead from Eco CIN/A15.344.123.72

Copy link
Member

@ArneTR ArneTR left a comment

Choose a reason for hiding this comment

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

I do that always manually, but do agree that this is insider knowledge and nice to have as QoL.

To my understandig these files also need to be in .gitignore:

  • docker/nginx/block.conf
    
  • docker/nginx/frontend.conf

Can you confirm and if so add. Then please merge from your side

@ribalba
Copy link
Member Author

ribalba commented Sep 3, 2025

After thinking about this I decided to add *.backup as a general gitignore. This could be debated. I think this is cleaner and quite clear. But could also go for the option of naming each file.

@ArneTR
Copy link
Member

ArneTR commented Sep 3, 2025

Please explicitely name the files. Makes it also clearer where they are created.

Catch-Alls should at least be directory restricted IMHO unless they are a standard (like .DS_Store etc.)

Copy link

github-actions bot commented Sep 3, 2025

Eco CI Output - Old Energy Estimation

Eco CI Output [RUN-ID: 17425884754]:

🌳 CO2 Data:
City: Des Moines, Lat: 41.6015, Lon: -93.6127
IP: 135.119.239.34
CO₂ from energy is: 2.108809080 g
CO₂ from manufacturing (embodied carbon) is: 0.288691924 g
Carbon Intensity for this location: 518 gCO₂eq/kWh
SCI: 2.397501 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #125.32274071.064.021011.84
Total Run25.324071.064.021011.84
Additional overhead from Eco CIN/A11.374.122.76

@ArneTR
Copy link
Member

ArneTR commented Sep 6, 2025

@ribalba ping

@ribalba
Copy link
Member Author

ribalba commented Sep 16, 2025

done

Copy link

Eco CI Output [RUN-ID: 17763595356]:

🌳 CO2 Data:
City: Phoenix, Lat: 33.4532, Lon: -112.0748
IP: 172.182.225.134
CO₂ from energy is: 1.601524450 g
CO₂ from manufacturing (embodied carbon) is: 0.344901597 g
Carbon Intensity for this location: 335 gCO₂eq/kWh
SCI: 1.946426 gCO₂eq / pipeline run emitted


Total cost of whole PR so far:

Label🖥 avg. CPU utilization [%]🔋 Total Energy [Joules]🔌 avg. Power [Watts]Duration [Seconds]
Measurement #123.78494780.673.951208.85
Total Run23.784780.673.951208.85
Additional overhead from Eco CIN/A13.414.163.22

@ArneTR ArneTR merged commit b170583 into main Sep 16, 2025
1 check passed
@ArneTR ArneTR deleted the config-backup branch September 16, 2025 16:52
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