green-coding-solutions / eco-ci-energy-estimation

Eco CI Energy estimation for Github Actions Runner VMs
https://metrics.green-coding.io/ci-index.html
MIT License
55 stars 16 forks source link

Show the numbers of the calculation on the same line #9

Closed rajbos closed 1 year ago

rajbos commented 1 year ago

Perhaps a table would help with the readability, but for me the current setup does not look nice and makes it harder to read. This could be combined with the work in #7.

image

ArneTR commented 1 year ago

Yeah, I agree, that can be optimized.

@dan-mm Can you try a markdown table here?

dan-mm commented 1 year ago

@rajbos I updated the eco-ci so it now displays a graph, and only on display-results.

The downside is now you have to call display-results to see anything, as it no longer displays spot results on get-measurement. I could display a mini-table on each get-measurement call, but I suspect just one final table is neater and more in line with what people want anyways. Thoughts?

image

rajbos commented 1 year ago

I prefer to explicitly call for the summary then having that info in multiple places.

Love they the labels are included of the measurements in between 👍.

dan-mm commented 1 year ago

Awesome, thanks for taking a look at it!