Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does object detection API has a plan to migrate to python3 #4830

Closed
irmowan opened this issue Jul 19, 2018 · 10 comments
Closed

Does object detection API has a plan to migrate to python3 #4830

irmowan opened this issue Jul 19, 2018 · 10 comments
Assignees

Comments

@irmowan
Copy link
Contributor

irmowan commented Jul 19, 2018

I have to use python3 on object detection API for project. But there is many conflicts when I use python3. Does API has a plan to migrate to python3, as python2.7 will not be supported soon?

@daavoo
Copy link

daavoo commented Jul 19, 2018

I managed to get the object detection API in Python3 by simply using the 2to3 official Python conversor and patching a few parts of the code (I don't remember all but I remember to at least have to manually patch https://github.com/tensorflow/models/blob/master/research/object_detection/utils/config_util.py#L326 ).

It should be pretty straightforward, just run the 2to3 conversor and fix the exceptions that you will be encountering.

@TedLLH
Copy link

TedLLH commented Jul 20, 2018

#4844 someone try to do so without any success, @daavoo could you have a look please?

@daavoo
Copy link

daavoo commented Jul 20, 2018

@TedLLH #4844 tried to covert the entire models/research. I have not tried that, I managed to convert models/research/object_detection which was what I needed. The errors in #4844 are not from the object_detection module

@TedLLH
Copy link

TedLLH commented Jul 23, 2018

@daavoo i tried with models/research/object_detection and it failed. The error was:

TypeError: Expected float32, got 'Loss/BoxClassifierLoss/localization_loss' of type 'str' instead

which was mentioned #4832 , it seems like I will have to stick with python2 at the moment until they fix the issue......

@TedLLH
Copy link

TedLLH commented Jul 23, 2018

Nope, even with python2, still does not work and encountered the same error, right here: #4832

@D-X-Y
Copy link

D-X-Y commented Jul 24, 2018

@TedLLH You can try my solution at #4832

@armandgurgu23
Copy link

hey @daavoo do you mind posting the Python-3 compatible object detection library as a separate repo online so that the open source community can make use of it?

@Shreeyak
Copy link

I second the motion to port the Object Detection API to python3!

@doraemon96
Copy link

Can someone try this? Im too busy right now to do it but it seems to work with py3.6!
https://tensorflow-object-detection-api-tutorial.readthedocs.io/en/latest/index.html

@tensorflowbutler
Copy link
Member

Hi There,
We are checking to see if you still need help on this, as this seems to be considerably old issue. Please update this issue with the latest information, code snippet to reproduce your issue and error you are seeing.
If we don't hear from you in the next 7 days, this issue will be closed automatically. If you don't need help on this issue any more, please consider closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants