-
Notifications
You must be signed in to change notification settings - Fork 98
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
why does this not mirror models/research/slim #6
Comments
You are not missing anything. This happened for mostly historical reasons with tensorflow team locking down contrib directory from adding latest and greatest models, however now that tf_slim moved out of contrib, it makes sense re-evaluate and potentially merge these two repositories somehow.. |
@marksandler2 do you have an ideas of how to approach merging the two repositories? I could look into merging in some of the newer work in |
@kr-ish - to update the nets here you could go one-by-one without running into many issues if any at all since now (as of may - see commit ) tensorflow/models/research/slim imports tf_slim directly as opposed to importing it from tf.contrib as it once did. I suppose at that point the issue would be where is best to keep the nets to avoid maintaining duplicates of the same code here and there. Not sure what @marksandler2 or sergio's opinions on this would be now... |
forgive me if I'm missing something obvious, but why doesn't this repo mirror https://github.com/tensorflow/models/tree/master/research/slim? Seems the latter is more up to date- for example the latter contains mobilenet under
nets
whereas this repo doesn'tthanks!
The text was updated successfully, but these errors were encountered: