Handle PUE param - Githubissues

nextflow-io / nf-co2footprint

[WIP] A Nextflow plugin to estimate the CO2 footprint of pipeline runs.

https://nextflow-io.github.io/nf-co2footprint/

Apache License 2.0

11 stars 4 forks source link

Handle PUE param #16

Closed skrakau closed 1 year ago

skrakau commented 1 year ago

I added the PUE value to the config and added another test for this.

Defining the default PUE value in the CO2FootprintConfig class is inconsistent with the handling of the file and summaryFile parameters, whose defaults are defined in the CO2FootprintFactory class (CO2FootprintTextFileObserver subclass). But maybe this can be restructured when we have all parameters handled properly. Or do you have already a suggestion?

skrakau commented 1 year ago

Thanks @mirpedrol !