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

TensorFlowEvent metrics collector doesn't find events written by torch / tensorboardX #2466

Closed
garymm opened this issue Dec 18, 2024 · 0 comments · Fixed by #2467
Closed

TensorFlowEvent metrics collector doesn't find events written by torch / tensorboardX #2466

garymm opened this issue Dec 18, 2024 · 0 comments · Fixed by #2467
Labels
kind/bug

Comments

@garymm
Copy link
Contributor

garymm commented Dec 18, 2024

What happened?

Trial code writes tensorboard summary using tensorboardX or torch.utils.tensorboard.
Experiment configured to use TensorFlowEvent metrics collector.
The metrics collector reports the metric is unavailable.

What did you expect to happen?

I expect the metrics collector to report the metric accurately.

Environment

Katib v0.17.0

Impacted by this bug?

Give it a 👍 We prioritize the issues with most 👍
garymm added a commit to garymm/katib that referenced this issue Dec 18, 2024
@garymm
Support old-style TensorFlow events (tensorboard)
d1c626e 
Fixes: kubeflow#2466
@garymm garymm mentioned this issue Dec 18, 2024
Merged
garymm added a commit to garymm/katib that referenced this issue Feb 4, 2025
@garymm
Support old-style TensorFlow events (tensorboard)
2942637 
Fixes: kubeflow#2466
Signed-off-by: Gary Miguel <[email protected]>
garymm added a commit to garymm/katib that referenced this issue Feb 6, 2025
@garymm
Support old-style TensorFlow events (tensorboard)
8898605 
Fixes: kubeflow#2466
Signed-off-by: Gary Miguel <[email protected]>
garymm added a commit to garymm/katib that referenced this issue Feb 12, 2025
@garymm
Support old-style TensorFlow events (tensorboard)
829092a 
Fixes: kubeflow#2466
Signed-off-by: Gary Miguel <[email protected]>
google-oss-prow bot pushed a commit that referenced this issue Feb 15, 2025
@garymm
Support old-style TensorFlow events (tensorboard) (#2467)
c18035e 
* Support old-style TensorFlow events (tensorboard)

Fixes: #2466
Signed-off-by: Gary Miguel <[email protected]>

* format

Signed-off-by: Gary Miguel <[email protected]>

* test

Signed-off-by: Gary Miguel <[email protected]>

* don't continue loops

Signed-off-by: Gary Miguel <[email protected]>

* format

Signed-off-by: Gary Miguel <[email protected]>

---------

Signed-off-by: Gary Miguel <[email protected]>
mahdikhashan pushed a commit to mahdikhashan/katib that referenced this issue Feb 17, 2025
@garymm @mahdikhashan
Support old-style TensorFlow events (tensorboard) (kubeflow#2467)
a545e86 
* Support old-style TensorFlow events (tensorboard)

Fixes: kubeflow#2466
Signed-off-by: Gary Miguel <[email protected]>

* format

Signed-off-by: Gary Miguel <[email protected]>

* test

Signed-off-by: Gary Miguel <[email protected]>

* don't continue loops

Signed-off-by: Gary Miguel <[email protected]>

* format

Signed-off-by: Gary Miguel <[email protected]>

---------

Signed-off-by: Gary Miguel <[email protected]>
@mahdikhashan mahdikhashan mentioned this issue Feb 17, 2025
Merged
1 task
google-oss-prow bot pushed a commit that referenced this issue Feb 17, 2025
@mahdikhashan @garymm
cherry-picked c18035e1 (#2517)
7dcdde7 
* Support old-style TensorFlow events (tensorboard)

Fixes: #2466


* format



* test



* don't continue loops



* format



---------

Signed-off-by: Gary Miguel <[email protected]>
Co-authored-by: Gary Miguel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support old-style TensorFlow events (tensorboard) garymm/katib
2 participants
@garymm @andreyvelich