Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

Are the tf_cnn_benchmarks deprecated and replaced by MLPerf? #436

Closed
vilmara opened this issue Nov 14, 2019 · 8 comments
Closed

Are the tf_cnn_benchmarks deprecated and replaced by MLPerf? #436

vilmara opened this issue Nov 14, 2019 · 8 comments

Comments

@vilmara
Copy link

vilmara commented Nov 14, 2019

Hi @tfboyd, are the tf_cnn_benchmarks deprecated and replaced by MLPerf?. You have mentioned at #407 that "tf_cnn_benchmarks is deprecated going into TF 2.0. It is still a great tool pre-TF 2.0. The new "perf tools" are different and focus on end user performance". For "the new perf tools" do you mean MLPerf training benchmarks?.

I am testing the tf_cnn_benchmarks master branch with the TF version tf-nightly-gpu 2.1.0.dev20191112 and it is throwing the error ImportError No module named contrib.data.python.ops. I see also the repo hasn't been updated for a while (the last update was 7 months ago).

Could you please provide further comments and recommendations on what benchmarks we should use according to the test goal, and maybe updating the README file with that info?

@tfboyd
Copy link
Member

tfboyd commented Nov 14, 2019 via email

@vilmara
Copy link
Author

vilmara commented Nov 14, 2019

@tfboyd thanks for your prompt reply, do you know if there is somebody else still administrating this benchmark repo?

@tfboyd
Copy link
Member

tfboyd commented Nov 14, 2019 via email

@vilmara
Copy link
Author

vilmara commented Nov 14, 2019

thanks @tfboyd. Hi @reedwm, could you please assist?

@vilmara vilmara closed this as completed Jan 15, 2020
@reedwm
Copy link
Member

reedwm commented Jan 16, 2020

Yes, tf_cnn_benchmarks is no longer maintained. I just updated the repo to support TF2, but will not do any further updates. It is now recommended to use the TensorFlow Official
Models

@vilmara
Copy link
Author

vilmara commented Jan 16, 2020

hi @reedwm. Does the tf_cnn_benchmark.py script work with TensorFlow Official Models?. Or do you recommend getting rid of this repo at all?

@reedwm
Copy link
Member

reedwm commented Jan 16, 2020

tf_cnn_benchmarks.py does not work with the TensorFlow Official Models. Each official model has their own Python script you need to run. I recommend not using this repo anymore (except for PerfZero), and using the Official Models instead.

@vilmara
Copy link
Author

vilmara commented Jan 17, 2020

@reedwm duly noted, thanks!

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

No branches or pull requests

3 participants