This repository was archived by the owner on Nov 22, 2022. It is now read-only.
Make TensorBoard work with RNNG #195
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
the
CLA Signed
geof90
force-pushed
the
export-D13596297
branch
from
67a1b11 to
dfa8d1d
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: d4d767371ad01904b84771e6602440ed9e2ddc93
geof90
force-pushed
the
export-D13596297
branch
from
dfa8d1d to
b241dfc
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: fb31013604ba9872257aeec46808a53aa05925c2
geof90
force-pushed
the
export-D13596297
branch
from
b241dfc to
4bdf448
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 46c9c332900bc9dfab446ad6d66e693570781467
geof90
force-pushed
the
export-D13596297
branch
from
4bdf448 to
28b8349
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 28a2aed10e56fd68bf1c0a46f6724af42db120d6
geof90
force-pushed
the
export-D13596297
branch
from
28b8349 to
5723adf
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 567066e05856f0edfd034e12b680ec17fb47a004
geof90
force-pushed
the
export-D13596297
branch
from
5723adf to
9c7972e
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 9b95e024b2feb3a30e5390c5d2a4e497a4ab46b0
geof90
force-pushed
the
export-D13596297
branch
from
9c7972e to
64c7b17
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 725d5b5c6c2010f790af8d53e0e40a7051be62e5
geof90
force-pushed
the
export-D13596297
branch
from
64c7b17 to
f72753f
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Differential Revision: D13596297 fbshipit-source-id: 889ef038f055ca73b4e4a8a5e5b031483a36ea7f
geof90
force-pushed
the
export-D13596297
branch
from
f72753f to
8dc893c
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: 3f0006f4083b4bedcd5db4080628a8079c96c1b0
geof90
force-pushed
the
export-D13596297
branch
from
8dc893c to
8ccb957
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: ea22b2b9d51d52f9a47077c2038400cc47b285e2
geof90
force-pushed
the
export-D13596297
branch
from
8ccb957 to
1c89b81
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 8, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: 6d5d9f91c7b3527019b1f78ec34c542ac8e84890
geof90
force-pushed
the
export-D13596297
branch
from
1c89b81 to
7ae8969
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 9, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: ce0e8f378a03664376b81e79100e823b3efe594e
geof90
force-pushed
the
export-D13596297
branch
from
7ae8969 to
bf920a0
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 9, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: 46bc376d2bbe23622b788f224ff2190ea5d1ddf0
geof90
force-pushed
the
export-D13596297
branch
from
bf920a0 to
c091b27
Compare
geof90
added a commit
to geof90/pytext
that referenced
this pull request
Jan 9, 2019
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: orionr Differential Revision: D13596297 fbshipit-source-id: 7a1402de3625d8d242dcf7d54f7cfab516872503
Summary: Pull Request resolved: facebookresearch#195 Make TensorBoard work with RNNG. Exporter was crashing with use_tensorboard=true because RNNG exporter does not have a dummy_input, this is fixed to add the c2 net directly when calling add_graph. Also, reworked throttling in Manifold Writer (recent throttling caused data to not show up on the dashboard until the summary_writer is closed) Reviewed By: hikushalhere Differential Revision: D13596297 fbshipit-source-id: 45167b35e71a87da0c6820fb7db069bf3473c41b
geof90
force-pushed
the
export-D13596297
branch
from
c091b27 to
2360bf4
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary: Differential Revision: D13596297