Skip to content

43 read support log file#52

Open
rogerkuou wants to merge 5 commits into
mainfrom
43_read_support_log_file
Open

43 read support log file#52
rogerkuou wants to merge 5 commits into
mainfrom
43_read_support_log_file

Conversation

@rogerkuou

@rogerkuou rogerkuou commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

fix #43

  • add timestamp suffix
  • add a logfile reader utility functin using tbparse -> there seems no need?

@CLAassistant

CLAassistant commented Jun 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@rogerkuou rogerkuou marked this pull request as ready for review June 10, 2026 11:26
@rogerkuou rogerkuou requested a review from SarahAlidoost June 10, 2026 11:27
@rogerkuou

Copy link
Copy Markdown
Collaborator Author

Hi @SarahAlidoost , I added the timestamp to logfile suffix. However I found tbparse already provides very nice interface reading logs as DataFrames, see these lines of usages.
For now, I do not see the need to create a utility functions to read log files. Unless you thin the logfile should be read as a specific format? Or maybe I can add the use of tbparse to the example notebook?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support to read log files of SummaryWriter

2 participants