🐛[BUG]: MLFlow Logging Typically fails for Multi-process

[NickGeneva]

Version

0.1.0

On which installation method(s) does this occur?

Source

Describe the issue

Probably need to have it just log on root process.

Minimum reproducible example

No response

Relevant log output

No response

Environment details

No response

[mariusaurus]

mine works when I tried using the two GPUs on my desktop machine, but each process was logged individually. Could we have this fixed as I have a customer who wants to use the MLFlow logger in multi-GPU and multi-node settings

[mariusaurus]

since this is related to logging and only requires small changes: file logging at https://github.com/NVIDIA/modulus-launch/blob/main/modulus/launch/logging/console.py#L40C5-L47C48 should have only the first line after the if statement indented:

    def file_logging(self, file_name: str = "launch.log"):
        """Log to file"""
        if os.path.exists(file_name):
            os.remove(file_name)
        filehandler = logging.FileHandler(file_name)
        filehandler.setFormatter(self.formatter)
        filehandler.setLevel(logging.DEBUG)
        self.logger.addHandler(filehandler)

Also, formatter has to be passed from the init routine, ideally by making it self.formatter in line 28. And in the current form, this only logs process one, if this could include the whole logging would be great! LMK if I should open a seperate issue for all that.