Should we have import of Engine in ignite/__init__.py?

@elanmart thanks for the DCGAN PR - looks great!

Just noticed in the PR that

from ignite.engine.engine import Engine, Events, State

was added to ignite/__init__.py?

I understand that it does add some convenience so users can just use the Engine, but this also creates inconsistent usage patterns i.e.

import ignite
engine = ignite.Engine(...)

and

from ignite.engine import Engine, State, Events
engine = Engine()

perhaps we could just add

import ignite.engine
import ignite.handlers

etc. in the __init__.py file in ignite/, similar to what’s done in pytorch (i.e. here)

cc @vfdev-5

[vfdev-5]

Right, so we can change the imports of ignite/__init__.py to

import ignite.engine
import ignite.handlers

So, I can change it if you do not have any objections

[alykhantejani]

Merged in #180