chainercv icon indicating copy to clipboard operation
chainercv copied to clipboard

Accept pathlib.Path type for read_image.

Open sergeant-wizard opened this issue 4 years ago • 5 comments

This patch enables users to pass a pathlib.Path object to read_image, which is currently not possible due to the limitations cv2.imread ( cv2.imread(pathlib.Path('hoge.png')) throws a cryptic error )

sergeant-wizard avatar Oct 08 '19 19:10 sergeant-wizard

Looks like we need to wait for the drop of python 2 support :sob:

sergeant-wizard avatar Oct 08 '19 19:10 sergeant-wizard

FYI: Our policy about py2 support is here.

Hakuyume avatar Oct 08 '19 22:10 Hakuyume

Will ChainerCV update to Chainer 7 after its stable release and then drop Py2 support?

sergeant-wizard avatar Oct 08 '19 23:10 sergeant-wizard

After Chainer v7 becomes stable, we can drop py2 support from ChainerCV.

Hakuyume avatar Oct 08 '19 23:10 Hakuyume

Ok the test should pass once PY2 test is removed. I'll rebase once that happens.

On Tue, Oct 8, 2019, 4:51 PM Toru Ogawa [email protected] wrote:

After Chainer v7 becomes stable, we can drop py2 support from ChainerCV.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/chainer/chainercv/pull/933?email_source=notifications&email_token=ABOL4ZDGUBQKGYDP5SNVT6DQNUMHZA5CNFSM4I6WDEOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAV7JFQ#issuecomment-539751574, or mute the thread https://github.com/notifications/unsubscribe-auth/ABOL4ZHLXI6XHTKSZ4LVYDDQNUMHZANCNFSM4I6WDEOA .

sergeant-wizard avatar Oct 09 '19 01:10 sergeant-wizard