-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates for tensorflow on cc8 #5603
Conversation
please test |
The tests are being triggered in jenkins. |
-1 |
please test for cc8_amd64_gcc820 |
The tests are being triggered in jenkins.
|
-1 |
test parameters:
|
please test |
The tests are being triggered in jenkins.
|
-1 Tested at: b181ace
I found compilation error when building: �[32m[761 / 1,769]�[0m checking cached actions �[1A�[K�[31m�[1mFAILED:�[0m Build did NOT complete successfully �[1A�[K�[31m�[1mFAILED:�[0m Build did NOT complete successfully �[0m ERROR: Could not build Bazel error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.wBV8gA (%build) RPM build errors: line 37: It's not recommended to have unversioned Obsoletes: Obsoletes: external+bazel+0.29.1-45459d Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.wBV8gA (%build) You can see the results of the tests here: |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@slava77 could you have a look please there are comparison differences |
@mrodozov Is there a way to set the baseline to be from the same IB/arch that is used for the (special) tests like this? |
I have to extend my understanding about published baselines, from what I see it's only the production architecture. which explains the differences.
there is, I think what we did in the past was to produce the baseline by hand for the arch and publish it, but I've never done it myself. we did it at least once for this PR: |
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_11_1_X/cc8 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
Changes taken from the slc7 PR for tensorflow.
The forward porting of cmsdist master to cc8 is not straightforward, so it had to be done by hand.
To be tested when the next cc8 IB is available