-
-
Notifications
You must be signed in to change notification settings - Fork 31k
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
Support REPL import
completion (needs investigation first)
#128786
Comments
I'd be glad to have this but I'm afraid it will be inefficient. It's possible to autocomplete attributes because the namespace already exists at runtime. However, if one writes cc @pablogsal |
This is the case but is not unheard of. For example, IPython (and many other REPL) does it: We could profile it and if we see is not too bad we can try to add support for it |
Maybe we could also make this feature on demand? that way we wouldn't penalize current users if they don't need that feature. But I would definitely be interested in knowing whether it's feasible for us or not (and would be happy to review such PR if we find a good solution) |
We should first benchmark to know what is the cost. My guess is that is close to be negligible. We can also try to back off if there are too many packages or add some other stopgaps to control the impact |
import
completion (needs investigation first)
I've added two tasks for users that want to work on that. We need a PoC first and it's better for anyone wanting to help NOT to create any PR first but link their own fork in this issue as well as their results. We'll decide on how to proceed afterwards. |
See #69605 for my attempt ten years ago |
Oh there was a precedent. Maybe it's better to close this issue then (it seems the IPython case was also mentioned in the original issue). EDIT: Considering that #117312 was closed as duplicate, I'll also close this one as a duplicate. |
Feature or enhancement
Proposal:
press
TAB
aftershows
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
Tasks
The text was updated successfully, but these errors were encountered: