Better to include Chainer as external dependencies

[yoheikikuta]

Version

import dezero
dezero.__version__
# '0.0.11'

Some tests depend on Chainer (for example, https://github.com/oreilly-japan/deep-learning-from-scratch-3/blob/master/tests/test_conv2d.py#L6).
After Chainer is installed via pip, all tests run successfully.

I guess it's better to add Chainer is one of the external dependencies to README.

[wakame1367]

I think so. And this code depend on Chainer(

deep-learning-from-scratch-3/steps/step59.py

Lines 4 to 6 in 81cbe3f

	from chainer import Chain
	from chainer.optimizers import SGD
	import chainer.links as L

)

[ftnext]

I think so too.

proposal (setup.py)

setup(name='dezero',
      ...
      extras_require={
            'develop': ['chainer']
      },
      ...
      )

And pip install -e .[develop]

ref: https://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies

[koki0702]

Thanks for suggestions!

Now Chainer is only needed at the unittests.
So Chainer is not included at setup.py, but added at travis CI setup.
https://github.com/oreilly-japan/deep-learning-from-scratch-3/blob/master/.travis.yml#L7

([develop] option is a good idea !)