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

Fix missing dependency in torch.utils.tensorboard #115598

Closed

Conversation

sachahu1
Copy link
Contributor

Fixes #114591

Version package was removed in this pull request: #114108 but is still used in torch.utils.tensorboard causing import errors. The fix removes the import and uses a simpler check.

Copy link

pytorch-bot bot commented Dec 11, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/115598

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (3 Unrelated Failures)

As of commit ba27f5a with merge base 617c228 (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

Copy link

pytorch-bot bot commented Dec 12, 2023

Didn't find following labels among repository labels: release notes: fix missing import from torch.utils.tensorboard

@sachahu1
Copy link
Contributor Author

@pytorchbot label "release notes: fx"

@pytorch-bot pytorch-bot bot added the release notes: fx release notes category label Dec 12, 2023
Copy link
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It feels like the same can be achieved by replacing the 2nd line with
from torch._vendor.packaging.version import Version, can't it?

@sachahu1
Copy link
Contributor Author

@malfet Yes, you're right, my mistake! Thanks!

@sachahu1 sachahu1 requested a review from malfet December 12, 2023 22:45
@soulitzer soulitzer added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Dec 13, 2023
@sachahu1
Copy link
Contributor Author

@malfet @soulitzer Are we happy to merge this ?

Copy link
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the fix.

@malfet malfet added this to the 2.2.0 milestone Dec 20, 2023
@malfet
Copy link
Contributor

malfet commented Dec 20, 2023

@pytorchbot merge -f "It's not tested anyway"

@malfet malfet added release notes: profiler release notes category topic: bug fixes topic category and removed release notes: fx release notes category labels Dec 20, 2023
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

hyperfraise pushed a commit to hyperfraise/pytorch that referenced this pull request Dec 21, 2023
Fixes pytorch#114591

Version package was removed in this pull request: pytorch#114108 but is still used in `torch.utils.tensorboard` causing import errors. The fix removes the import and uses a simpler check.
Pull Request resolved: pytorch#115598
Approved by: https://github.com/malfet
hyperfraise pushed a commit to hyperfraise/pytorch that referenced this pull request Dec 21, 2023
Fixes pytorch#114591

Version package was removed in this pull request: pytorch#114108 but is still used in `torch.utils.tensorboard` causing import errors. The fix removes the import and uses a simpler check.
Pull Request resolved: pytorch#115598
Approved by: https://github.com/malfet
ZhiweiYan-96 pushed a commit to ZhiweiYan-96/pytorch that referenced this pull request Dec 22, 2023
Fixes pytorch#114591

Version package was removed in this pull request: pytorch#114108 but is still used in `torch.utils.tensorboard` causing import errors. The fix removes the import and uses a simpler check.
Pull Request resolved: pytorch#115598
Approved by: https://github.com/malfet
atalman pushed a commit to atalman/pytorch that referenced this pull request Dec 28, 2023
Fixes pytorch#114591

Version package was removed in this pull request: pytorch#114108 but is still used in `torch.utils.tensorboard` causing import errors. The fix removes the import and uses a simpler check.
Pull Request resolved: pytorch#115598
Approved by: https://github.com/malfet
atalman added a commit that referenced this pull request Dec 28, 2023
Fixes #114591

Version package was removed in this pull request: #114108 but is still used in `torch.utils.tensorboard` causing import errors. The fix removes the import and uses a simpler check.
Pull Request resolved: #115598
Approved by: https://github.com/malfet

Co-authored-by: Sacha <sachahu@hotmail.fr>
@atalman
Copy link
Contributor

atalman commented Jan 18, 2024

Validated via:

>>> import torch
>>> from torch.utils.tensorboard import SummaryWriter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged open source release notes: profiler release notes category topic: bug fixes topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing packaging dependency in torch 2.1.x
6 participants