Autoreload error when running a script

[jewfro-cuban]

When running a script I get an error:

Error in callback <bound method AutoreloadMagics.post_execute_hook of <autoreload.AutoreloadMagics object at 0x00000000045E1B70>> (for post_execute):
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
C:\<path_to_env>\lib\site-packages\IPython\extensions\autoreload.py in post_execute_hook(self)
    523         newly_loaded_modules = set(sys.modules) - self.loaded_modules
    524         for modname in newly_loaded_modules:
--> 525             _, pymtime = self._reloader.filename_and_mtime(sys.modules[modname])
    526             if pymtime is not None:
    527                 self._reloader.modules_mtimes[modname] = pymtime

C:\<path_to_env>\lib\site-packages\IPython\extensions\autoreload.py in filename_and_mtime(self, module)
    188
    189         # if hasattr(module, '__name__'):
--> 190         if module.__name__ == '__main__':
    191             # we cannot reload(__main__)
    192             return None, None

AttributeError: 'module' object has no attribute '__name__'

I've fixed it locally with:

        if hasattr(module, '__name__'):
            if module.__name__ == '__main__':
                # we cannot reload(__main__)
                return None, None

[Carreau]

Do you want to make a PR ? I suppose another fix would be getattr(module, '__name__', None). What king of module do not have __name__ ?

[jewfro-cuban]

I haven't tried too much to figure what exactly is the reason but it happened to me in the past as well few times and the above fix let me continue working. Whenever I update to a new release I encounter this behavior so it might be a good idea to make that code part immune to this possibility even if I can't pinpoint the exact reason.
Agreed?

[Carreau]

Sure. Are you familiar with git/github ? Do you want to try sending a fix through a Pull-Request ?
(I see from your joining date that you are new, feel free to ask questions on how to do things)

[jewfro-cuban]

I've sent a PR:
#8568

[Carreau]

Great ! Will review !

Tip : When you commit, you can also add in the commit message fixes #8564 , when you push on github it will automatically link the PR (Pull Request) and the target issue. Then when the PR is merged it will auto close the issue !

[Carreau]

Also I'm still curious of what kind of module might not have a __name__ can you share one of the things that make the bug happend ?

[jewfro-cuban]

Thanks.
As I've said, at the moment it's hard to pinpoint why it's happening.
Maybe there's something unusual in my workflow that leads to that.
If I'll have more time I'll try to investigate and extract a sample test script.

[nedlrichards]

I had the same problem with a ipython 4.0.0 and python3.4
I added the hasattr logic above to line 190. It seemed to work, but now it seems like I have new problems. This error is raised from autoreload.
I do not think that I am doing any multicore processing currently, which seems to be an issue with related cases. #7830

[autoreload of mp_main failed: Traceback (most recent call last):
File "/home/me/.local/lib/python3.4/site-packages/IPython/extensions/autoreload.py", line 248, in check
superreload(m, reload, self.old_objects)
AttributeError: 'module' object has no attribute 'name'

[lukepfister]

I'm running into this as well- ipython 4.0.0 and python 3.4. For what it's worth, I'm installing using Anaconda.

Running this as a script triggers the problem:

from scipy.interpolate import interp1d

if __name__ == "__main__":
    print("Hello")

My ipython_config.py file contains

c.InteractiveShellApp.exec_lines = [
'%load_ext autoreload',
'%autoreload 2',
]

The proposed fix seems to work.

[lukepfister]

I wound having the same problem described by oceanEngineer.

[autoreload of __mp_main__ failed: Traceback (most recent call last):
  File "/home/luke/software/python/miniconda/envs/nsp/lib/python3.4/site-packages/IPython/extensions/autoreload.py", line 248, in check
    superreload(m, reload, self.old_objects)
AttributeError: 'module' object has no attribute '__name__'
]

A simple workaround is to have autoreload skip attempting to reload any modules with module name __mp__main__.

So at line 223 of autoreload.py:

 223         for modname in modules:                                                                     
 224             if modname != '__mp_main__':                                                            
 225                 m = sys.modules.get(modname, None)                           

and indent the rest as necessary.

[takluyver]

#8568 got merged, so I think this is fixed.

[jewfro-cuban]

thanks

On Tue, Jan 26, 2016 at 6:39 PM, Thomas Kluyver notifications@github.com
wrote:

Closed #8564 #8564.

—
Reply to this email directly or view it on GitHub
#8564 (comment).