forked from zed-industries/zed
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] main from zed-industries:main #26
Merged
Merged
Conversation
This file contains 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
It should be 200k (with 100k output). I can't find anything that puts it at 20k and the changeover in 2f82374 only changed the name from o1-mini to o3-mini References: * https://docs.github.com/en/copilot/using-github-copilot/asking-github-copilot-questions-in-github#ai-models-for-copilot-chat * https://github.com/marketplace/models/azure-openai/o3-mini * https://platform.openai.com/docs/models#o3-mini Release Notes: - Corrected Github Copilot o3-mini context length --------- Co-authored-by: Marshall Bowers <[email protected]>
Adds a button to the theme selector to help people find more themes in the extension store.  Release Notes: - Added a way to access the extension store from the theme selector to make it easier to find new themes.
Reverts #24138 Nightly build failed, I believe because of this. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95189  CC: @ConradIrwin Release Notes: - N/A
Closes #ISSUE Release Notes: - N/A
…24186) Also send the `speculated_output` (which is just the editable region) to the llm backend Closes #ISSUE Release Notes: - N/A --------- Co-authored-by: Antonio Scandurra <[email protected]>
Release Notes: - N/A Co-authored-by: Marshall <[email protected]>
…ad of ellipsis (#24202) https://github.com/user-attachments/assets/06e14893-c285-4cea-927c-75e82a378b15 Release Notes: - N/A --------- Co-authored-by: Ben <[email protected]>
Release Notes: - N/A --------- Co-authored-by: Ben <[email protected]>
…ffects loop (#24208) Release Notes: - N/A
#24213) This fixes a "subtract with overflow" error that could happen in debug mode when viewing the project diagnostics. From git bisecting, I think that this behavior was introduced by #21942. It seems like it's possible in some cases for the excerpt-expansion heuristic to cause the excerpt's `context` range to start *after* the excerpt's `primary` range. We should probably revisit that heuristic at some point, but it also seems reasonable to handle that situation at this layer, rather than overflowing. Release Notes: - N/A
## Context While looking through the client crate, I noticed that some of the old functions and variables were still using gpui::model name that was deprecated during the gpui3 transition. This PR renames those instances of model to entity to be more inline with gpui3. In addition, I also renamed `model` to `entity` in cases found by the below search terms given by @someone13574 - model = cx. - model: Entity - model: &Entity - OpenedModelHandle - model.update - model.upgrade - model = .*\.root (regex) - parent_model - model = cx.new - cx.spawn(move |model Release Notes: - N/A
Note that this does *not* involve any breaking code changes. cc @0xtimsb - I didn't change any settings or anything here. That can happen separately! Release Notes: - N/A
) Chatted with @as-cii about limitations in the `Window::parent_view_id()` API (see: 662153d) and realized that I shouldn't be using the dispatch tree's data structures as they are layout dependent. I've introduced a new stack to `Window`, `rendered_entity_stack`, that tracks exactly which view's elements are being drawn. As such, I've also been able to remove the `Option<>` around the previous API. Release Notes: - N/A
Release Notes: - N/A
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )