google-research / tf-slim

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

why does this not mirror models/research/slim

kr-ish opened this issue · comments

commented

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't

thanks!

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..

commented

@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 nets piece by piece- not sure if there's a better way to go about it

@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...