benchmarks icon indicating copy to clipboard operation
benchmarks copied to clipboard

Imports moved in TF1.10

Open don-smith2 opened this issue 7 years ago • 3 comments

Hi all: I needed to make two minor fixes to run these benchmarks in a fresh TF1.10 (conda pkg) install:

  1. Update three relocated imports (report to Tensorflow), and
  2. Manually install multi_device_iterator_ops.py (report to Continuum). Apologies and please disregard if this is a known issue. Thanks, Don Smith Dell EMC HPC & AI Innovation Lab

preprocessing.py, line 32: orig: from tensorflow.python.data.experimental.ops import prefetching_ops new: from tensorflow.contrib.data.python.ops import prefetching_ops

benchmark_cnn.py, line 48: Code moved. orig: from tensorflow.contrib.compiler import xla new: from tensorflow.compiler import xla

benchmark_cnn.py, line 53: Code moved. orig: from tensorflow.python.data.experimental.ops import prefetching_ops new: from tensorflow.contrib.data.python.ops import prefetching_ops

don-smith2 avatar Oct 05 '18 18:10 don-smith2

The benchmark code is updated to match tensorflow/head. When there are API changes or additions it will stop working with older versions of TensorFlow. I am somewhat surprised that the XLA import worked at all with TF 1.10.

tfboyd avatar Oct 05 '18 18:10 tfboyd

I had additional import errors, so I stayed on branch, cnn_tf_v1.10_compatible

aurotripathy avatar Oct 07 '18 07:10 aurotripathy

Yes, that is the best path for me as well. Thanks, -Don

From: aurotripathy [email protected] Sent: Sunday, October 7, 2018 2:02 AM To: tensorflow/benchmarks Cc: Smith, Don - Dell Team; Author Subject: Re: [tensorflow/benchmarks] Imports moved in TF1.10 (#256)

[EXTERNAL EMAIL] Please report any suspicious attachments, links, or requests for sensitive information.

I had additional import errors, so I stayed on branch, cnn_tf_v1.10_compatible

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/benchmarks/issues/256#issuecomment-427630958, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AmEdBWmLCj-FRf3dNFmhomm-3PPfWGFvks5uiacBgaJpZM4XKt5x.

don-smith2 avatar Oct 16 '18 15:10 don-smith2