We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
In get_dataloader in training.py we are adding dataset_seq_length_stats to the w&b summary which produces a lot of clutter.
get_dataloader
dataset_seq_length_stats
We should just log this to the console and remove it from the summary.
The dataset_seq_length_stats_summary is also being logged twice
dataset_seq_length_stats_summary
logger.summary({"dataset_seq_len_stats_summary": length_stats.summary()}) logger.summary( {"dataset_seq_len_stats": length_stats.serialize(typecast=lambda x: str(x))} )
this is because the call to lengh_stats.serialize also includes the summary. We should remove the redundant first line.
lengh_stats.serialize
The text was updated successfully, but these errors were encountered:
Seems to not happen anymore in the code; unsure, low-confidence, likely wrong, but also probably worth simply de-prioing
Sorry, something went wrong.
No branches or pull requests
In
get_dataloader
in training.py we are addingdataset_seq_length_stats
to the w&b summary which produces a lot of clutter.We should just log this to the console and remove it from the summary.
The
dataset_seq_length_stats_summary
is also being logged twicethis is because the call to
lengh_stats.serialize
also includes the summary. We should remove the redundant first line.The text was updated successfully, but these errors were encountered: