Skip to content

Commit

Permalink
Remove tensorboard mapping from our default mapping (#19673)
Browse files Browse the repository at this point in the history
`torch.utils.tensorboard` imports `tensorboard` (or dies trying). The
original intent is "if a file imports it, it should receive a dependency
on `tensorboard`". Unfortunately the actual behavior is that _only_
tensorboard is inferred (and not `torch`). Worse, an import from
`tensorboard` is now ambiguous (because it's missing from the value).

So we'll remove it and cherry-pick to 2.17.x (when it was introduced).
Later we can fix the underlying issue of being able to model an import
mapping to multiple packages.

See #19672
  • Loading branch information
thejcannon authored and WorkerPants committed Aug 29, 2023
1 parent f1ec650 commit 1d501f7
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,6 @@
"snowflake-sqlalchemy": ("snowflake.sqlalchemy",),
"strawberry-graphql": ("strawberry",),
"streamlit-aggrid": ("st_aggrid",),
"tensorboard": ("torch.utils.tensorboard",),
"unleashclient": ("UnleashClient",),
"websocket-client": ("websocket",),
}
Expand Down

0 comments on commit 1d501f7

Please sign in to comment.