add handling of different lengths of scalarTags and correct Usage in ReadMe#2
Open
Strandtasche wants to merge 2 commits intoanderskm:masterfrom
Open
add handling of different lengths of scalarTags and correct Usage in ReadMe#2Strandtasche wants to merge 2 commits intoanderskm:masterfrom
Strandtasche wants to merge 2 commits intoanderskm:masterfrom
Conversation
Owner
|
@Strandtasche Thank you for submitting your pull request. I'm glad you have found the script helpful :-) |
Author
|
What kind of example do you want? Do you want an tfevents file that has that property or some python/tensorflow code that generates an tfevents file that has different lengths of scalarTags? |
Owner
|
@Strandtasche Some python/tensorflow code is preferable :-) |
Author
|
Sorry for the delay. I didn't manage to produce a minimal working example. I don't even know why my code produces different lengths of scalarTags, maybe I made a mistake somewhere. |
This file contains hidden or 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
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.
Hey,
I've used your script and noticed some things that might be improved.
In one of my examples the scalar "global steps / sec" is slightly shorter than the first column.
in your version the script crashes with an index error, so I added some error handling (substituting the missing values with NaN)
I also noticed some typos in your readme, which I fixed.
Thanks for the script, it helped me a lot