Lazy initialize tensorboard #4852
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
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D43863071 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D43863071 |
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
Mar 11, 2023
Summary: Pull Request resolved: facebookresearch#4852 Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints). Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/ Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180 Reviewed By: Reubend Differential Revision: D43863071 fbshipit-source-id: c007cd8c1f07d7376df278663e12f6164a5777a6
wat3rBro
force-pushed
the
export-D43863071
branch
from
48e33c2
to
eb48da9
Compare
|
This pull request was exported from Phabricator. Differential Revision: D43863071 |
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
Mar 16, 2023
Summary: Pull Request resolved: facebookresearch#4852 Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints). Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/ Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180 Reviewed By: Reubend Differential Revision: D43863071 fbshipit-source-id: a338d67f70ce7c43ceaeafba54fcefedc36b78b2
wat3rBro
force-pushed
the
export-D43863071
branch
from
eb48da9
to
1f1fca1
Compare
|
This pull request was exported from Phabricator. Differential Revision: D43863071 |
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
Mar 23, 2023
Summary: Pull Request resolved: facebookresearch#4852 Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints). Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/ Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180 Reviewed By: Reubend, crassirostris Differential Revision: D43863071 fbshipit-source-id: 391a74cf7684c2f7a7664d7ae2463b99a2eab4b6
wat3rBro
force-pushed
the
export-D43863071
branch
from
1f1fca1
to
efd98d0
Compare
Summary: Pull Request resolved: facebookresearch#4852 Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints). Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/ Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180 Reviewed By: Reubend, crassirostris Differential Revision: D43863071 fbshipit-source-id: b632444aaf87cb60423518a69257623789b44741
|
This pull request was exported from Phabricator. Differential Revision: D43863071 |
wat3rBro
force-pushed
the
export-D43863071
branch
from
efd98d0
to
9ae7d8e
Compare
|
This pull request has been merged in 1bc3a33. |
danielm322
pushed a commit
to danielm322/detectron2
that referenced
this pull request
Jun 9, 2023
Summary: Pull Request resolved: facebookresearch#4852 Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints). Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/ Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180 Reviewed By: Reubend, crassirostris Differential Revision: D43863071 fbshipit-source-id: 3fb749199772837697911d802d7e43d157597573
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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:
Every time a summary writer object is created it'll end up creating a file in the passed in logdir with just metadata of 40 bytes. This will cause a lot of spam with every rank initializing the file aggressively (I've done this myself when adding support for Adhoc logging to PyPer to a lot of angry complaints).
Report at https://fb.workplace.com/groups/723537759122220/posts/745046770304652/
Sample code that accesses "_writer" directly: https://www.internalfb.com/code/fbsource/[6b39948cd8e179fcc49f08526efa5dc26fc00843]/fbcode/mobile-vision/d2go/d2go/utils/visualization.py?lines=180
Reviewed By: Reubend
Differential Revision: D43863071