Add --samples_per_plugin to specify explicit sampling counts · Pull Request #1138 · tensorflow/tensorboard

…ification of samples to keep per summary type.

Previously, TensorBoard always downsampled summaries for OOM reasons but some users wanted the ability to keep all their summaries of some types.

With this flag, --samples_per_plugin='histogram=2,images=0'
 keeps TensorBoard defaults for other summaries, restricts the number of histograms to 2, and keeps all image summaries.

nfelt

approved these changes Apr 18, 2018

@nfelt nfelt changed the title Add TensorBoard flag samples_per_plugin that allows for custom spec… Add --samples_per_plugin to specify sampling counts per plugin

Apr 18, 2018

@nfelt nfelt changed the title Add --samples_per_plugin to specify sampling counts per plugin Add --samples_per_plugin to specify explicit sampling counts

Apr 18, 2018

This was referenced

Jun 15, 2018

@nfelt nfelt mentioned this pull request

Jun 25, 2018

@nfelt nfelt mentioned this pull request

Jul 20, 2018

@lanpa lanpa mentioned this pull request

Oct 18, 2018

wchargin added a commit that referenced this pull request

Aug 18, 2020
Summary:
Before `--samples_per_plugin` existed, the only way to change the
sampling thresholds was to patch `application.py` manually. But the flag
has been the right way to do this since #1138, and manual patching has
been insufficient since the flag value was exposed to plugins in #3271.
There’s no need to discuss this implementation detail any more.

wchargin-branch: readme-remove-application-reference
wchargin-source: b79027ec8b279537bdf38f7b5d7f36cf5dd38498

wchargin added a commit that referenced this pull request

Aug 18, 2020
Summary:
Before `--samples_per_plugin` existed, the only way to change the
sampling thresholds was to patch `application.py` manually. But the flag
has been the right way to do this since #1138, and manual patching has
been insufficient since the flag value was exposed to plugins in #3271.
There’s no need to discuss this implementation detail any more.

wchargin-branch: readme-remove-application-reference