You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
tazarov
changed the title
[Bug]: New EF refactor breaks static analysis of imports for IDEs
[Bug][DX]: New EF refactor breaks static analysis of imports for IDEs
Jun 21, 2024
This is probably happening because of the way we do dynamic imports. I agree it's a bad user experience. @tazarov if you can take some time to see how this can be improved while retaining dynamic importing, that would be helpful. Maybe we need to populate __all__ ?
What happened?
The new refactoring of the EFs with dynamic imports #2034 breaks IDEs static analysis for imports. The dynamic imports work fine at runtime.
PyCharm
VSCode
Google Colab
Legacy import:
**New Import from EF file: **
![Screenshot 2024-06-21 at 11 00 28](https://private-user-images.githubusercontent.com/1157440/341707545-0bbd4081-a844-42a2-8c45-d568c5b0bb75.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk2MDA1NzcsIm5iZiI6MTcxOTYwMDI3NywicGF0aCI6Ii8xMTU3NDQwLzM0MTcwNzU0NS0wYmJkNDA4MS1hODQ0LTQyYTItOGM0NS1kNTY4YzViMGJiNzUucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDYyOCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA2MjhUMTg0NDM3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MGE1ZmJiMDYyMDZkMDQyZWJhODI0ZmRiYjg1YzI2ODZhZTc2ZDc0ZTM2ODM5N2JhNzUwM2ZjOTdjY2ZmYzA0MiZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.nNQOYbtxhG_xSS493vZbyslGhCzSzENXZGVPoV1mtXc)
Versions
Chroma@main
Relevant log output
No response
The text was updated successfully, but these errors were encountered: