disable unit test for ppc64le due to lack of tfcompiler

[smuzaffar]

PR description:

We do not have tfcompiler build for ppc64le this causes build errors in ppc64le IBs
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_ppc64le_gcc820/CMSSW_11_1_X_2019-12-02-2300/PhysicsTools/TensorFlow

This PR proposes to not build this unit test on ppc64le.

[smuzaffar]

please test

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28526/12997

• This PR adds an extra 12KB to repository

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/3746/console Started: 2019/12/03 11:47

[cmsbuild]

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master.

It involves the following packages:

PhysicsTools/TensorFlow

@perrotta, @cmsbuild, @santocch, @slava77 can you please review it and eventually sign? Thanks.
@makortel, @riga this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: 7ebca8e
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-dbaebd/3746/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2793840
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2793498
• DQMHistoTests: Total skipped: 341
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
• Checked 147 log files, 16 edm output root files, 34 DQM output files

[slava77]

+1

for #28526 7ebca8e

• code changes are in line with the PR description; the change is consistent with the update in the cmsdist made early this year in cms-sw/cmsdist@f73480e
• jenkins tests pass and comparisons with the baseline show no differences for the default ARCH tests

@smuzaffar
is the situation expected to improve in a more recent version of the tensorflow?
What is the status of updating to it, BTW?

@vlimant is tf v2.0 something to target now (we are still using 1.6)? Is v2.0 compatible with models made in the older versions?

[santocch]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[davidlange6]

On Dec 5, 2019, at 4:20 AM, Slava Krutelyov <notifications@github.com<mailto:notifications@github.com>> wrote: +1 for #28526<#28526> 7ebca8e<7ebca8e> * code changes are in line with the PR description; the change is consistent with the update in the cmsdist made early this year in cms-sw/cmsdist@f73480e<cms-sw/cmsdist@f73480e> * jenkins tests pass and comparisons with the baseline show no differences for the default ARCH tests @smuzaffar<https://github.com/smuzaffar> is the situation expected to improve in a more recent version of the tensorflow? What is the status of updating to it, BTW? I had understood that the c++ interface (which is done in an unsupported way) has still not been made to work in v2.0 (or versions in between) @vlimant<https://github.com/vlimant> is tf v2.0 something to target now (we are still using 1.6)? Is v2.0 compatible with models made in the older versions? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#28526?email_source=notifications&email_token=ABGPFQ7NMSEAMEJBJFYXPRLQXBXRFA5CNFSM4JUVZFG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF7KZGI#issuecomment-561949849>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGPFQ4WBQNWOOIJTJAIOH3QXBXRFANCNFSM4JUVZFGQ>.

[fabiocos]

+1