Add pass-through filter for shortcuts#14080
Merged
Carreau merged 2 commits intoipython:mainfrom Jun 2, 2023
Merged
Conversation
|
I can confirm this fixes the issue for me with Python 3.11.3. |
|
Nice! Is it possible to backport this to 8.12.x? We're stuck with Python 3.8 for now. |
Member
let's see ... @meeseeksdev backport to 8.12.x |
meeseeksmachine
pushed a commit
to meeseeksmachine/ipython
that referenced
this pull request
Sep 12, 2023
Member
|
Backport seem to be clean, so I'll likely make a release by end of month. In general feel free to backport and send a PR on older branches. I'm likely to miss notification on closed issues/PRs. |
EliahKagan
added a commit
to EliahKagan/EmbeddingScratchwork
that referenced
this pull request
Dec 4, 2023
This constrains openai<1 for now. Showing how to work with and wrap the old Python API is a central part of this project, so it's unclear what will be done with that moving forward. The options are to stay with <1 indefinitely, move up to 1, or support both conditionally (which should usually be avoided, but for this project it might be nice to show how to do so when needed). Note that embeddings_utils is not present in openai>=1. Less importantly, this also pins ipython back to ~8.12.3, which is actually a downgrade. This is to support Python 3.8. Before, we were installing an older version just on older Pythons, as we do with numpy and scipy. For ipython, the reason to give newer Pythons the newer version was that the convenient and popular way to autocomplete a full line, by pressing right-arrow, was broken, and the fix initially had not been backported. This is now fixed, due to the backporting of ipython/ipython#14080 to 8.12, which is in the 8.12.3 patch.
EliahKagan
added a commit
to EliahKagan/EmbeddingScratchwork
that referenced
this pull request
Dec 4, 2023
This constrains openai<1 for now. Showing how to work with and wrap the old Python API is a central part of this project, so it's unclear what will be done with that moving forward. The options are to stay with <1 indefinitely, move up to 1, or support both conditionally (which should usually be avoided, but for this project it might be nice to show how to do so when needed). Note that embeddings_utils is not present in openai>=1. Less importantly, this also pins ipython back to ~8.12.3, which is actually a downgrade. This is to support Python 3.8. Before, we were installing an older version just on older Pythons, as we do with numpy and scipy. For ipython, the reason to give newer Pythons the newer version was that the convenient and popular way to autocomplete a full line, by pressing right-arrow, was broken, and the fix initially had not been backported. This is now fixed, due to the backporting of ipython/ipython#14080 to 8.12, which is in the 8.12.3 patch.
EliahKagan
added a commit
to dmvassallo/EmbeddingScratchwork
that referenced
this pull request
Dec 4, 2023
This constrains openai<1 for now. Showing how to work with and wrap the old Python API is a central part of this project, so it's unclear what will be done with that moving forward. The options are to stay with <1 indefinitely, move up to 1, or support both conditionally (which should usually be avoided, but for this project it might be nice to show how to do so when needed). Note that embeddings_utils is not present in openai>=1. Less importantly, this also pins ipython back to ~8.12.3, which is actually a downgrade. This is to support Python 3.8. Before, we were installing an older version just on older Pythons, as we do with numpy and scipy. For ipython, the reason to give newer Pythons the newer version was that the convenient and popular way to autocomplete a full line, by pressing right-arrow, was broken, and the fix initially had not been backported. This is now fixed, due to the backporting of ipython/ipython#14080 to 8.12, which is in the 8.12.3 patch.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is one way to fix #14070.
The alternatives are:
eagerflag)