'Parallel' object has no attribute '_name' in Parallel.warn

[percevalw]

Hi,

Since v1.3.1, calling Parallel.warn produces this error

.../parallel.py in ...
    109       executor = Parallel(
    110            n_jobs, backend="multiprocessing", prefer="processes", verbose=verbose
    111         )
--> 112     executor.warn(message)
    113 

~/.../site-packages/joblib/logger.py in warn(self, msg)
     78 
     79     def warn(self, msg):
---> 80         logging.getLogger(self._name).warning("[%s]: %s" % (self, msg))
     81 
     82     def info(self, msg):

AttributeError: 'Parallel' object has no attribute '_name'

I think Parallel.__init__ is missing a super().__init__ call to let the Logger class set the _name attribute.

[fcharras]

Thank you for the feedback, I think this can be fix quickly by adding super().__init__ indeed.

But I'm not sure why Parallel inherits from Logger here, since the methods it adds are not used by Parallel like it is in memory.py, was there really an intent to expose the logging methods as public methods to the users ? @GaelVaroquaux maybe you have more insight to provide on the initial intent ? I had a discussion with @tomMoral that suggests exposing the Logger methods in Parallel could be deprecated.

[fcharras]

The issue should be solved after yesterday patch release. Let's keep open for discussion about possible deprecation ?