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.

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

reload functionality #11

Open
dvolgyes opened this issue Dec 10, 2022 · 1 comment
Open

reload functionality #11

dvolgyes opened this issue Dec 10, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@dvolgyes
Copy link

The regular tensorboard approach with event accumulators has a reload functionality.
That would be most helpful for dynamic situations where the data changes,
or new directories are created (see the children).

It doesn't have to be automatic, but it would be nice to make a .reload() which can update
things, instead of deleting the object and creating a new one.

@j3soon
Copy link
Owner

j3soon commented Dec 10, 2022

Are you encountering slow parsing speed when dealing with large event logs?

Exposing the EventAccumulator.reload function could save some time by allowing us to avoid loading from scratch. However, this would still require re-parsing everything in the EventAccumulator, and since the bottleneck currently lies in the parsing logic, I don't think this would significantly improve performance.

Despite this, I think it's worth adding this feature to help dealing with larger files. I may consider implementing it after addressing the current parsing bottleneck (#8).

@j3soon j3soon added the enhancement New feature or request label Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants