Skip to content
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

catch cpuinfo ImportError #4138

Merged
merged 3 commits into from
May 1, 2018
Merged

catch cpuinfo ImportError #4138

merged 3 commits into from
May 1, 2018

Conversation

robieta
Copy link
Contributor

@robieta robieta commented May 1, 2018

No description provided.

@robieta robieta requested review from karmel and a team as code owners May 1, 2018 17:16
@robieta robieta requested a review from qlzh727 May 1, 2018 17:16
Copy link
Member

@qlzh727 qlzh727 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also update the psutil import for the memory info collection.

@robieta
Copy link
Contributor Author

robieta commented May 1, 2018

@qlzh727 Done.

run_info["machine_config"]["memory_total"] = vmem.total
run_info["machine_config"]["memory_available"] = vmem.available
except ImportError:
tf.logging.warn("'psutil' not imported. Memort info will not be logged.")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Memory"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops.

@robieta robieta merged commit 5c78b9d into master May 1, 2018
@robieta robieta deleted the fix/swallow_cpuinfo_failure branch May 1, 2018 17:49
xueeinstein added a commit to xueeinstein/models that referenced this pull request May 2, 2018
* 'master' of https://github.com/tensorflow/models: (404 commits)
  official/mnist: Updates with the release of TensorFlow 1.8.
  Internal changes to slim and object detection (tensorflow#4100)
  catch cpuinfo ImportError (tensorflow#4138)
  [samples/core/get_starter/eager]: Update with API simplifications in 1.8
  Add vincentvanhoucke as owner for research/marco
  Add marco subdirectory to README.md
  Create request.json
  Create jpeg2json.py
  Fix width / height.
  Create README.md
  Add new import paths to mnist_tpu.py (tensorflow#4133)
  fix model call (tensorflow#4132)
  Remove useless fields in resnet_model (tensorflow#4110)
  official/mnist: Add input_shape parameter to initial layer (tensorflow#4104) (tensorflow#4118)
  Increase minimum TF version for DEFINE_enum and rename variable mappings for change to RNN variable names. (tensorflow#4123)
  fix import errors
  Minor update the resnet runloop. (tensorflow#4113)
  Two new plot-generating scripts and changes in support of the ICLR poster. (tensorflow#4092)
  Add export savedmodel to wide_deep (tensorflow#4041)
  Fix 2 typos
  ...
omegafragger pushed a commit to omegafragger/models that referenced this pull request May 15, 2018
* catch cpuinfo ImportError

* add psutil import catch

* fix typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants