Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[einsum] Fix opt_einsum defaults to be more reasonable (pytorch#86985)
Fixes the confusing situation mentioned here pytorch#85224 (comment) by - setting better OG defaults - changing warnings to errors now that we have better defaults Test plan: - Ran einsum tests locally + CI - Uninstalled opt-einsum and ran through setting - `enabled` to False (doesn't throw error) - `strategy` to anything that's not None (errors) - `strategy` to None (noops) - Installed opt-einsum and ran through setting - `enabled` to False (doesn't throw error) - `enabled` to True (doesn't throw error, no ops + defaults to 'auto') - `strategy` to random string (errors) - `strategy` to None (noops, still is 'auto') - `strategy` to 'greedy' (is set to 'greedy') Pull Request resolved: pytorch#86985 Approved by: https://github.com/soulitzer
- Loading branch information