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.

Sign up for GitHub

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

scalar: fix scalar_demos #5866

Merged
merged 1 commit into from
Aug 12, 2022
Merged

scalar: fix scalar_demos #5866

merged 1 commit into from
Aug 12, 2022

Conversation

yatbear
Copy link
Member

@yatbear yatbear commented Aug 11, 2022 •
edited

Resets the default writer every time before writing scalar summaries, to fix the error: tensorflow/tensorflow#25707 (comment)

Node: 'temperature/current/write_summary'
Resource localhost/_AnonymousVar0/N10tensorflow22SummaryWriterInterfaceE does not exist.
	 [[{{node temperature/current/write_summary}}]] [Op:__inference_one_step_62]

Googlers, see b/239742608.

Tested locally:
image

Also tested generating demo data internally in cl/466983681.

#onduty

@yatbear yatbear requested a review from nfelt August 11, 2022 17:03
nfelt
nfelt approved these changes Aug 11, 2022
View reviewed changes
Copy link
Collaborator

@nfelt nfelt left a comment

Choose a reason for hiding this comment

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

Thanks!

@yatbear yatbear merged commit a319dd2 into tensorflow:master Aug 12, 2022
@yatbear yatbear deleted the scalars_demo branch August 12, 2022 13:06
@yatbear yatbear mentioned this pull request Sep 29, 2022
Open
yatbear added a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
@yatbear
scalar: fix scalar_demos (tensorflow#5866)
c7f2a97 
Set the default writer every time before writing scalar summaries in tf function.
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
@yatbear @dna2github
scalar: fix scalar_demos (tensorflow#5866)
21f4ce9 
Set the default writer every time before writing scalar summaries in tf function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nfelt nfelt nfelt approved these changes

Assignees
No one assigned
Labels
core:summaries plugin:scalars
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@yatbear @nfelt