Forward fix for broken update and attend test#21292
Conversation
Summary: [executorch] Add missing exir:_warnings dep to llm cache library The `cache` python_library sources `reference_cache.py`, which imports `from executorch.exir._warnings import experimental` for the `experimental` decorators on `CacheConfig` and `ContiguousReferenceCache`. The target only depended on `//caffe2:torch`, so `executorch.exir` was absent at runtime and every test in `test_update_and_attend` failed at import with `ModuleNotFoundError: No module named 'executorch.exir'`. Add `//executorch/exir:_warnings` to the `cache` library's deps. Reviewed By: metascroy Differential Revision: D113428489
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/21292
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New FailuresAs of commit 3b579dc with merge base 179c4ee ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@kiymetakdemir has exported this pull request. If you are a Meta employee, you can view the originating Diff in D113428489. |
This PR needs a
|
Summary:
[executorch] Add missing exir:_warnings dep to llm cache library
The
cachepython_library sourcesreference_cache.py, which importsfrom executorch.exir._warnings import experimentalfor theexperimentaldecorators onCacheConfigandContiguousReferenceCache. The target onlydepended on
//caffe2:torch, soexecutorch.exirwas absent at runtime and every test intest_update_and_attendfailed at import withModuleNotFoundError: No module named 'executorch.exir'.Add
//executorch/exir:_warningsto thecachelibrary's deps.Reviewed By: metascroy
Differential Revision: D113428489