X Tutup
The Wayback Machine - https://web.archive.org/web/20201014194024/https://github.com/tensorflow/tensorflow/pull/25673
Skip to content

/ tensorflow

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

Jump to bottom

Update the environment capture script for system information. #25673

Merged
merged 1 commit into from Jun 20, 2019
Merged

Update the environment capture script for system information. #25673

merged 1 commit into from Jun 20, 2019

Conversation

@Ryan-Qiyu-Jiang
Copy link
Contributor

@Ryan-Qiyu-Jiang Ryan-Qiyu-Jiang commented Feb 11, 2019

for @dynamicwebpaige 's issue #25461
added

  • Python version
  • Bazel version (if compiling from source)

tensorflow installed info (pip show tensorflow) to better discern if tf was installed from binary or source
it adds package location and author-email info to output
@Ryan-Qiyu-Jiang
added python version, bazel version, and tf installed info to env_cap…
Loading status checks…
8593e24 
…ture_script
@googlebot googlebot added the cla: yes label Feb 11, 2019
@Ryan-Qiyu-Jiang Ryan-Qiyu-Jiang mentioned this pull request Feb 11, 2019
Closed
@pragyaak pragyaak self-assigned this Feb 11, 2019
@pragyaak pragyaak added the size:S label Feb 11, 2019
@pragyaak pragyaak added this to Assigned Reviewer in PR Queue via automation Feb 11, 2019
@pragyaak pragyaak requested a review from aselle Feb 11, 2019
@pragyaak pragyaak requested a review from jianlijianli Feb 15, 2019
@pragyaak pragyaak assigned rthadur and unassigned pragyaak Feb 23, 2019
@jianlijianli
Copy link
Contributor

@jianlijianli jianlijianli commented Mar 29, 2019

Adding miaout17@ for review.
@jianlijianli jianlijianli requested review from miaout17 and removed request for jianlijianli Mar 29, 2019
@tensorflowbutler
Copy link
Member

@tensorflowbutler tensorflowbutler commented Jun 11, 2019

Nagging Reviewer @aselle, @miaout17: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 74 days with no activity and the awaiting review label has been applied.
@rthadur rthadur added the comp:lite label Jun 11, 2019
@rthadur rthadur removed the request for review from aselle Jun 11, 2019
@miaout17
miaout17 approved these changes Jun 20, 2019
View changes
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jun 20, 2019
@tensorflow-copybara tensorflow-copybara merged commit 8593e24 into tensorflow:master Jun 20, 2019
11 of 16 checks passed
11 of 16 checks passed
Linux GPU Internal CI build failed
Details
MacOS Contrib Internal CI build failed
Details
Ubuntu Python3 Internal CI build failed
Details
Windows Bazel Internal CI build failed
Details
Windows Bazel GPU Internal CI build failed
Details
Android Demo App Internal CI build successful
Details
MacOS Python2 and CC Internal CI build successful
Details
Ubuntu CC Internal CI build successful
Details
Ubuntu Makefile Internal CI build successful
Details
Ubuntu Python2 Internal CI build successful
Details
Ubuntu Python3 PIP Internal CI build successful
Details
Ubuntu Sanity Internal CI build successful
Details
Ubuntu contrib Internal CI build successful
Details
cla/google All necessary CLAs are signed
feedback/copybara Google internal checks PASS for runs with create time 2019-06-20T18:50:53.331256935Z,2019-06-20T18:50:42.063723823Z.
Details
import/copybara Change imported to the internal review system
Details
PR Queue automation moved this from Approved by Reviewer to Merged Jun 20, 2019
tensorflow-copybara pushed a commit that referenced this pull request Jun 20, 2019
@tensorflower-gardener
Merge pull request #25673 from Ryan-Qiyu-Jiang:env_capture_script_mor…
Loading status checks…
b6dedcb 
…e_system_info_update

PiperOrigin-RevId: 254264543
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miaout17 miaout17

Assignees

@rthadur rthadur

Labels
awaiting review cla: yes comp:lite ready to pull size:S
Projects
PR Queue
  
Merged
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

9 participants
@Ryan-Qiyu-Jiang @jianlijianli @tensorflowbutler @miaout17 @googlebot @kokoro-team @tensorflow-copybara @rthadur @pragyaak
You can’t perform that action at this time.
X Tutup