search

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
10 stars 3 forks source link

collect warning messages and filter out duplicated #81

Closed mirpedrol closed 6 months ago

mirpedrol commented 7 months ago

This is to avoid having the same warning printed for every process. Collects all warning messages and filters out the duplicated ones.

image

skrakau commented 7 months ago

I would add an extra warning if the cpu_model is null, since this is a different issue one should be aware of.

skrakau commented 7 months ago

Great, I would still wait a bit to make sure there is no other, intended way of handling redundant warnings

skrakau commented 6 months ago

I added some changes, have a look if you think that make sense or if you would want soemthing to be changed

mirpedrol commented 6 months ago

I agree with your changes :) is it ready to be merged?