Update TensorFlow interface for TF 1.6.

[riga]

This PR updates the custom NTSession and TBBSession implementations in PhysicsTools/TensorFlow to work with TensorFlow 1.6. As before, changes w.r.t. upstream code are minimized in order to simplify future updates.

Upstream changes can be found here: tensorflow/tensorflow@v1.5.0...v1.6.0

The switch to TF 1.6 was proposed in #22568 (comment), the associated PR in cmsdist is cms-sw/cmsdist#3835.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks
explicitly set code checks passed (by @smuzaffar)

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22603/3943

Code check has found code style and quality issues which could be resolved by applying a patch in https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22603/3943/git-diff.patch
e.g. curl https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22603/3943/git-diff.patch | patch -p1

You can run scram build code-checks to apply code checks directly

[smuzaffar]

please test with #22568 cms-sw/cmsdist#3835

[cmsbuild]

The tests are being triggered in jenkins.
Using externals from cms-sw/cmsdist#3835
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/26841/console

[cmsbuild]

A new Pull Request was created by @riga (Marcel R.) for master.

It involves the following packages:

PhysicsTools/TensorFlow

@perrotta, @monttj, @cmsbuild, @slava77, @gpetruc, @arizzi can you please review it and eventually sign? Thanks.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: ef5a03c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22603/26841/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
7aefbe8
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22603/26841/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22603/26841/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22603/26841/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 29
• DQMHistoTests: Total histograms compared: 2477528
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2477351
• DQMHistoTests: Total skipped: 176
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 1.5700000002 KiB( 23 files compared)
• Checked 118 log files, 9 edm output root files, 29 DQM output files

[slava77]

+1

for #22603 ef5a03c

• updates in NTSession and TBBSession are consistent with changes in the upstream tensorflow re the following files

• https://github.com/tensorflow/tensorflow/raw/v1.5.0/tensorflow/core/common_runtime/direct_session.h
• https://github.com/tensorflow/tensorflow/raw/v1.6.0/tensorflow/core/common_runtime/direct_session.h
• https://github.com/tensorflow/tensorflow/raw/v1.5.0/tensorflow/core/common_runtime/direct_session.cc
• https://github.com/tensorflow/tensorflow/raw/v1.6.0/tensorflow/core/common_runtime/direct_session.cc

• jenkins tests pass and comparisons show no differences

[smuzaffar]

@fabiocos , can we get this PR + #22568 cms-sw/cmsdist#3835 ?

[slava77]

@fabiocos
please clarify if this can make it to the master/10_1_X or if it should wait for 10_2_X to open
Thank you.

[cmsbot]

please test with #22568 cms-sw/cmsdist#3835
testing updates for the external

[fabiocos]

@smuzaffar I would prefer to wait 10_2_X for this

[smuzaffar]

@fabiocos , can we get this and external in 10.2.X now?

[fabiocos]

+1

[fabiocos]

merge