helloadhavan/github_issues huggingface dataset:
Repository: electron/electron
Pull request: fix: crash on empty dialog extensions array on Windows

#### Description of Change

Closes https://github.com/electron/electron/issues/48611

Fixes a crash caused by the following:

```
out_of_range was thrown in -fno-exceptions mode with message "basic_string"
```

Due to a stricter `-fno-exceptions` introduced in Chromium via https://github.com/electron/electron/compare/v33.2.0...v33.2.1.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd
- [x] `npm test` passes
- [x] [PR release notes](https://github.com/electron/clerk/blob/main/README.md) describe the change in a way relevant to app developers, and are [capitalized, punctuated, and past tense](https://github.com/electron/clerk/blob/main/README.md#examples).

#### Release Notes

Notes: Fixed a potential crash in `dialog.showOpenDialog` on Windows with an empty extension filter array.

Changed files:
filename: shell/browser/ui/file_dialog_win.cc
patch: @@ -44,8 +44,7 @@ void ConvertFilters(const Filters& filters,
                     std::vector<std::wstring>* buffer,
                     std::vector<COMDLG_FILTERSPEC>* filterspec) {
   if (filters.empty()) {
-    COMDLG_FILTERSPEC spec = {L"All Files (*.*)", L"*.*"};
-    filterspec->push_back(spec);
+    filterspec->push_back({L"All Files (*.*)", L"*.*"});
     return;
   }
 
@@ -55,11 +54,16 @@ void ConvertFilters(const Filters& filters,
     buffer->push_back(base::UTF8ToWide(filter.first));
     spec.pszName = buffer->back().c_str();
 
-    std::vector<std::string> extensions(filter.second);
-    for (std::string& extension : extensions)
-      extension.insert(0, "*.");
-    buffer->push_back(base::UTF8ToWide(base::JoinString(extensions, ";")));
-    spec.pszSpec = buffer->back().c_str();
+    if (filter.second.empty()) {
+      buffer->push_back(L"*.*");
+      spec.pszSpec = buffer->back().c_str();
+    } else {
+      std::vector<std::string> extensions(filter.second);
+      for (std::string& extension : extensions)
+        extension.insert(0, "*.");
+      buffer->push_back(base::UTF8ToWide(base::JoinString(extensions, ";")));
+      spec.pszSpec = buffer->back().c_str();
+    }
 
     filterspec->push_back(spec);
   }
Repository: vercel/next.js
Pull request: [next-upgrade] Fall back to `npx` if `yarn dlx` is not available

Affected Yarn v1. This is the same logic we use in Next.js' CLI.

Changed files:
filename: packages/next-codemod/bin/upgrade.ts
patch: @@ -203,7 +203,7 @@ export async function runUpgrade(
 
   let shouldRunReactCodemods = false
   let shouldRunReactTypesCodemods = false
-  let execCommand = 'npx'
+  let execCommand = 'npx --yes'
   // The following React codemods are for React 19
   if (
     !shouldStayOnReact18 &&
@@ -213,13 +213,7 @@ export async function runUpgrade(
     shouldRunReactCodemods = await suggestReactCodemods()
     shouldRunReactTypesCodemods = await suggestReactTypesCodemods()
 
-    const execCommandMap = {
-      yarn: 'yarn dlx',
-      pnpm: 'pnpx',
-      bun: 'bunx',
-      npm: 'npx',
-    }
-    execCommand = execCommandMap[packageManager]
+    execCommand = getNpxCommand(packageManager)
   }
 
   fs.writeFileSync(appPackageJsonPath, JSON.stringify(appPackageJson, null, 2))
@@ -740,3 +734,19 @@ function warnDependenciesOutOfRange(
     })
   }
 }
+
+function getNpxCommand(pkgManager: PackageManager) {
+  let command = 'npx --yes'
+  if (pkgManager === 'pnpm') {
+    command = 'pnpm --silent dlx'
+  } else if (pkgManager === 'yarn') {
+    try {
+      execSync('yarn dlx --help', { stdio: 'ignore', cwd })
+      command = 'yarn --quiet dlx'
+    } catch {}
+  } else if (pkgManager === 'bun') {
+    command = 'bunx'
+  }
+
+  return command
+}
Repository: ggml-org/llama.cpp
Pull request: codeowners : add danbev for examples/debug


Changed files:
filename: CODEOWNERS
patch: @@ -27,6 +27,7 @@
 /examples/batched.swift/                @ggerganov
 /examples/batched/                      @ggerganov
 /examples/convert-llama2c-to-ggml/      @ggerganov
+/examples/debug/                        @danbev @pwilkin
 /examples/deprecation-warning/          @ggerganov
 /examples/diffusion/                    @am17an
 /examples/embedding/                    @ggerganov
Repository: ollama/ollama
Pull request: fix(cmd): show info may have nil ModelInfo

add a `nil` check for `ModelInfo`
fixes: #6578 

Changed files:
filename: cmd/cmd.go
patch: @@ -726,14 +726,17 @@ func ShowHandler(cmd *cobra.Command, args []string) error {
 }
 
 func showInfo(resp *api.ShowResponse) {
-	arch := resp.ModelInfo["general.architecture"].(string)
-
 	modelData := [][]string{
-		{"arch", arch},
 		{"parameters", resp.Details.ParameterSize},
 		{"quantization", resp.Details.QuantizationLevel},
-		{"context length", fmt.Sprintf("%v", resp.ModelInfo[fmt.Sprintf("%s.context_length", arch)].(float64))},
-		{"embedding length", fmt.Sprintf("%v", resp.ModelInfo[fmt.Sprintf("%s.embedding_length", arch)].(float64))},
+	}
+	if resp.ModelInfo != nil {
+		arch := resp.ModelInfo["general.architecture"].(string)
+		modelData = append(modelData,
+			[]string{"arch", arch},
+			[]string{"context length", fmt.Sprintf("%v", resp.ModelInfo[fmt.Sprintf("%s.context_length", arch)].(float64))},
+			[]string{"embedding length", fmt.Sprintf("%v", resp.ModelInfo[fmt.Sprintf("%s.embedding_length", arch)].(float64))},
+		)
 	}
 
 	mainTableData := [][]string{
Repository: vuejs/vue
Pull request: Update util.js

Typo fix


Changed files:
filename: src/shared/util.js
patch: @@ -82,7 +82,7 @@ export function cached<F: Function> (fn: F): F {
 }
 
 /**
- * Camelize a hyphen-delmited string.
+ * Camelize a hyphen-delimited string.
  */
 const camelizeRE = /-(\w)/g
 export const camelize = cached((str: string): string => {
Repository: electron/electron
Pull request: fix: crash on empty dialog extensions array on Windows

Backport of #48640

See that PR for details.


Notes: Fixed a potential crash in `dialog.showOpenDialog` on Windows with an empty extension filter array.

Changed files:
filename: shell/browser/ui/file_dialog_win.cc
patch: @@ -44,8 +44,7 @@ void ConvertFilters(const Filters& filters,
                     std::vector<std::wstring>* buffer,
                     std::vector<COMDLG_FILTERSPEC>* filterspec) {
   if (filters.empty()) {
-    COMDLG_FILTERSPEC spec = {L"All Files (*.*)", L"*.*"};
-    filterspec->push_back(spec);
+    filterspec->push_back({L"All Files (*.*)", L"*.*"});
     return;
   }
 
@@ -55,11 +54,16 @@ void ConvertFilters(const Filters& filters,
     buffer->push_back(base::UTF8ToWide(filter.first));
     spec.pszName = buffer->back().c_str();
 
-    std::vector<std::string> extensions(filter.second);
-    for (std::string& extension : extensions)
-      extension.insert(0, "*.");
-    buffer->push_back(base::UTF8ToWide(base::JoinString(extensions, ";")));
-    spec.pszSpec = buffer->back().c_str();
+    if (filter.second.empty()) {
+      buffer->push_back(L"*.*");
+      spec.pszSpec = buffer->back().c_str();
+    } else {
+      std::vector<std::string> extensions(filter.second);
+      for (std::string& extension : extensions)
+        extension.insert(0, "*.");
+      buffer->push_back(base::UTF8ToWide(base::JoinString(extensions, ";")));
+      spec.pszSpec = buffer->back().c_str();
+    }
 
     filterspec->push_back(spec);
   }
Repository: facebook/react
Pull request: [ci] Use 'opened' event for discord notifications


We don't need to wait for it to be labeled now that we have the shared maintainer check workflow.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/react/pull/32332).
* #32333
* __->__ #32332

Changed files:
filename: .github/workflows/compiler_discord_notify.yml
patch: @@ -2,7 +2,7 @@ name: (Compiler) Discord Notify
 
 on:
   pull_request_target:
-    types: [labeled]
+    types: [opened]
     paths:
       - compiler/**
       - .github/workflows/compiler_**.yml
@@ -14,7 +14,7 @@ jobs:
       actor: ${{ github.event.pull_request.user.login }}
 
   notify:
-    if: ${{ needs.check_maintainer.outputs.is_core_team == 'true' && github.event.label.name == 'React Core Team' }}
+    if: ${{ needs.check_maintainer.outputs.is_core_team == 'true' }}
     needs: check_maintainer
     runs-on: ubuntu-latest
     steps:

filename: .github/workflows/runtime_discord_notify.yml
patch: @@ -2,7 +2,7 @@ name: (Runtime) Discord Notify
 
 on:
   pull_request_target:
-    types: [labeled]
+    types: [opened]
     paths-ignore:
       - compiler/**
       - .github/workflows/compiler_**.yml
@@ -14,7 +14,7 @@ jobs:
       actor: ${{ github.event.pull_request.user.login }}
 
   notify:
-    if: ${{ needs.check_maintainer.outputs.is_core_team == 'true' && github.event.label.name == 'React Core Team' }}
+    if: ${{ needs.check_maintainer.outputs.is_core_team == 'true' }}
     needs: check_maintainer
     runs-on: ubuntu-latest
     steps:
Repository: ollama/ollama
Pull request: Update documentation: Change .bin to .gguf in GGUF file and adapter examples

This pull request updates the documentation to reflect the change from GGML to GGUF format.

Changes made:
- In the "Build from a GGUF file" section, updated the example Modelfile to use the .gguf extension instead of .bin
- Modified the explanatory text to refer to "GGUF file" instead of "GGUF bin file"
- In the "GGUF adapter" section, updated the example Modelfile to use the .gguf extension for the adapter file

(This is my first contribution to OSS, so I'm excited about it. Thank you.)

Changed files:
filename: docs/modelfile.md
patch: @@ -128,10 +128,10 @@ Currently supported model architectures:
 #### Build from a GGUF file
 
 ```modelfile
-FROM ./ollama-model.bin
+FROM ./ollama-model.gguf
 ```
 
-The GGUF bin file location should be specified as an absolute path or relative to the `Modelfile` location.
+The GGUF file location should be specified as an absolute path or relative to the `Modelfile` location.
 
 
 ### PARAMETER
@@ -208,7 +208,7 @@ Currently supported Safetensor adapters:
 #### GGUF adapter
 
 ```modelfile
-ADAPTER ./ollama-lora.bin
+ADAPTER ./ollama-lora.gguf
 ```
 
 ### LICENSE
Repository: ggml-org/llama.cpp
Pull request: llama : rename llama-sampling to llama-sampler

This commit addresses the TODO in llama-sampling.h to rename that header and the implementation to llama-sampler.

Changed files:
filename: src/CMakeLists.txt
patch: @@ -31,7 +31,7 @@ add_library(llama
             llama-model-saver.cpp
             llama-model.cpp
             llama-quant.cpp
-            llama-sampling.cpp
+            llama-sampler.cpp
             llama-vocab.cpp
             unicode-data.cpp
             unicode.cpp

filename: src/llama-grammar.cpp
patch: @@ -2,7 +2,7 @@
 
 #include "llama-impl.h"
 #include "llama-vocab.h"
-#include "llama-sampling.h"
+#include "llama-sampler.h"
 
 #include <cmath>
 #include <algorithm>

filename: src/llama-sampler.cpp
patch: @@ -1,4 +1,4 @@
-#include "llama-sampling.h"
+#include "llama-sampler.h"
 
 #include "llama-impl.h"
 #include "llama-vocab.h"

filename: src/llama-sampler.h
patch: @@ -1,7 +1,5 @@
 #pragma once
 
-// TODO: rename llama-sampling.h/.cpp to llama-sampler.h/.cpp ?
-
 #include "llama.h"
 
 #include <vector>
Repository: vuejs/vue
Pull request: [weex] Use weex.requireModule instead of weex.require

After a discussion, we feel the `weex.require` api is ambiguity, use `weex.requireModule` is more reliable.

Changed files:
filename: src/entries/weex-framework.js
patch: @@ -91,7 +91,7 @@ export function createInstance (
   const weexInstanceVar = {
     config,
     document,
-    require: moduleGetter
+    requireModule: moduleGetter
   }
   Object.freeze(weexInstanceVar)
 
@@ -112,7 +112,7 @@ export function createInstance (
   const instanceVars = Object.assign({
     Vue: subVue,
     weex: weexInstanceVar,
-    __weex_require_module__: weexInstanceVar.require // deprecated
+    __weex_require_module__: weexInstanceVar.requireModule // deprecated
   }, timerAPIs)
   callFunction(instanceVars, appCode)
Repository: vuejs/vue
Pull request: fix checkbox test case

Related with #4639 

Changed files:
filename: test/unit/features/directives/model-checkbox.spec.js
patch: @@ -243,17 +243,17 @@ describe('Directive v-model checkbox', () => {
   it('should get updated with model when in focus', (done) => {
     const vm = new Vue({
       data: {
-        a: '2'
+        a: 2
       },
-      template: '<input type="checkbox" value="1" v-model="a"/>'
+      template: '<input type="checkbox" v-model="a"/>'
     }).$mount()
     document.body.appendChild(vm.$el)
     vm.$el.click()
     waitForUpdate(() => {
-      expect(vm.$el.checked).toBe(true)
+      expect(vm.$el.checked).toBe(false)
       vm.a = 2
     }).then(() => {
-      expect(vm.$el.checked).toBe(false)
+      expect(vm.$el.checked).toBe(true)
     }).then(done)
   })
Repository: vercel/next.js
Pull request: Turbopack: Use `Debug` instead of `Display` for `ValueDebugFormat` impl on `RcStr`

Stuff like empty `RcStr`s look really confusing if we use `Display`. Also stop using the alternate debug mode for `String`, as it looks like the alternate flag doesn't do anything in this case.

Changed files:
filename: turbopack/crates/turbo-tasks/src/debug/mod.rs
patch: @@ -74,13 +74,13 @@ pub trait ValueDebugFormat {
 
 impl ValueDebugFormat for String {
     fn value_debug_format(&self, _depth: usize) -> ValueDebugFormatString<'_> {
-        ValueDebugFormatString::Sync(format!("{self:#?}"))
+        ValueDebugFormatString::Sync(format!("{self:?}"))
     }
 }
 
 impl ValueDebugFormat for RcStr {
-    fn value_debug_format(&self, _: usize) -> ValueDebugFormatString<'_> {
-        ValueDebugFormatString::Sync(self.to_string())
+    fn value_debug_format(&self, _depth: usize) -> ValueDebugFormatString<'_> {
+        ValueDebugFormatString::Sync(format!("{self:?}"))
     }
 }
Repository: ollama/ollama
Pull request: Add findutils to base images

This caused missing internal files

Changed files:
filename: scripts/rh_linux_deps.sh
patch: @@ -30,7 +30,7 @@ if grep -i "centos" /etc/system-release >/dev/null; then
         dnf install -y rh-git227-git
         ln -s /opt/rh/rh-git227/root/usr/bin/git /usr/local/bin/git
     fi
-    dnf install -y devtoolset-10-gcc devtoolset-10-gcc-c++ pigz
+    dnf install -y devtoolset-10-gcc devtoolset-10-gcc-c++ pigz findutils
 elif grep -i "rocky" /etc/system-release >/dev/null; then
     # Temporary workaround until rocky 8 AppStream ships GCC 10.4 (10.3 is incompatible with NVCC)
     cat << EOF > /etc/yum.repos.d/Rocky-Vault.repo
@@ -45,6 +45,7 @@ EOF
     dnf install -y git \
         gcc-toolset-10-gcc-10.2.1-8.2.el8 \
         gcc-toolset-10-gcc-c++-10.2.1-8.2.el8 \
+        findutils \
         pigz
 else
     echo "ERROR Unexpected distro"
Repository: ggml-org/llama.cpp
Pull request: metal : adaptive CPU/GPU interleave based on number of nodes

Put a bit more work on the main thread when encoding the graph. This helps to interleave better the CPU/GPU work, especially for larger graphs.

| Model                    | Test   |   t/s master |   t/s gg/metal-adaptive-cpu-interleave |   Speedup |
|:-------------------------|:-------|-------------:|---------------------------------------:|----------:|
| deepseek2 30B.A3B Q8_0   | pp512  |      1643.86 |                                1650.41 |      1.00 |
| deepseek2 30B.A3B Q8_0   | tg32   |        60.99 |                                  64.10 |      1.05 |
| gemma3 1B Q4_0           | pp512  |     11084.12 |                               11096.19 |      1.00 |
| gemma3 1B Q4_0           | tg32   |       221.23 |                                 229.51 |      1.04 |
| gemma3 4B Q4_0           | pp512  |      2801.46 |                                2801.59 |      1.00 |
| gemma3 4B Q4_0           | tg32   |       136.59 |                                 140.30 |      1.03 |
| gpt-oss 120B MXFP4 MoE   | pp512  |      1216.02 |                                1216.82 |      1.00 |
| gpt-oss 120B MXFP4 MoE   | tg32   |        87.14 |                                  89.91 |      1.03 |
| gpt-oss 20B MXFP4 MoE    | pp512  |      2407.47 |                                2407.12 |      1.00 |
| gpt-oss 20B MXFP4 MoE    | tg32   |       131.83 |                                 134.63 |      1.02 |
| qwen3 0.6B Q4_0          | pp512  |     14327.76 |                               14248.38 |      0.99 |
| qwen3 0.6B Q4_0          | tg32   |       329.70 |                                 341.94 |      1.04 |
| qwen3 0.6B Q8_0          | pp512  |     14197.99 |                               14173.35 |      1.00 |
| qwen3 0.6B Q8_0          | tg32   |       269.92 |                                 279.47 |      1.04 |
| qwen3 4B Q8_0            | pp512  |      2468.90 |                                2463.35 |      1.00 |
| qwen3 4B Q8_0            | tg32   |       113.25 |                                 114.03 |      1.01 |
| qwen3moe 30B.A3B Q4_0    | pp512  |      2140.49 |                                2158.52 |      1.01 |
| qwen3moe 30B.A3B Q4_0    | tg32   |       100.31 |                                 106.69 |      1.06 |
| qwen3next 80B.A3B Q4_K_M | pp512  |       840.10 |                                 844.97 |      1.01 |
| qwen3next 80B.A3B Q4_K_M | tg32   |        34.61 |                                  36.55 |      1.06 |

Changed files:
filename: ggml/src/ggml-metal/ggml-metal-context.m
patch: @@ -415,7 +415,7 @@ bool ggml_metal_cpy_tensor_async(ggml_metal_t ctx_src, ggml_metal_t ctx_dst, con
 
 enum ggml_status ggml_metal_graph_compute(ggml_metal_t ctx, struct ggml_cgraph * gf) {
     // number of nodes encoded by the main thread (empirically determined)
-    const int n_main = 64;
+    const int n_main = MAX(64, 0.1*gf->n_nodes);
 
     // number of threads in addition to the main thread
     const int n_cb = ctx->n_cb;
Repository: vuejs/vue
Pull request: fix #4344 #4445 #4511

<!--
Please make sure to read the Pull Request Guidelines:
https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#pull-request-guidelines
-->


Changed files:
filename: src/compiler/parser/text-parser.js
patch: @@ -4,7 +4,7 @@ import { cached } from 'shared/util'
 import { parseFilters } from './filter-parser'
 
 const defaultTagRE = /\{\{((?:.|\n)+?)\}\}/g
-const regexEscapeRE = /[-.*+?^${}()|[\]/\\]/g
+const regexEscapeRE = /[-.*+?^${}()|[\]\/\\]/g
 
 const buildRegex = cached(delimiters => {
   const open = delimiters[0].replace(regexEscapeRE, '\\$&')
Repository: ollama/ollama
Pull request: remove any unneeded build artifacts

metal lib is embedded so the file isn't necessary. this shaves off roughly 50KB

Changed files:
filename: llm/generate/gen_common.sh
patch: @@ -87,6 +87,8 @@ apply_patches() {
 build() {
     cmake -S ${LLAMACPP_DIR} -B ${BUILD_DIR} ${CMAKE_DEFS}
     cmake --build ${BUILD_DIR} ${CMAKE_TARGETS} -j8
+    # remove unnecessary build artifacts
+    rm -f ${BUILD_DIR}/bin/ggml-common.h ${BUILD_DIR}/bin/ggml-metal.metal
 }
 
 compress() {
Repository: facebook/react
Pull request: fix[react-devtools-standalone]: define missing globals

Same as what we did for `react-devtools-fusebox` in https://github.com/facebook/react/pull/32297.

Changed files:
filename: packages/react-devtools-core/webpack.standalone.js
patch: @@ -88,6 +88,9 @@ module.exports = {
       __PROFILE__: false,
       __TEST__: NODE_ENV === 'test',
       __IS_NATIVE__: true,
+      __IS_FIREFOX__: false,
+      __IS_CHROME__: false,
+      __IS_EDGE__: false,
       'process.env.DEVTOOLS_PACKAGE': `"react-devtools-core"`,
       'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`,
       'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null,
Repository: huggingface/transformers
Pull request: Fix `pr_build_doc_with_comment.yml` not reporting correctly

# What does this PR do?

The condition that determines if the run is ok or not uses the wrong job, so the reported comment was wrong.

Changed files:
filename: .github/workflows/pr_build_doc_with_comment.yml
patch: @@ -110,7 +110,7 @@ jobs:
     env:
       GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       GITHUB_RUN_URL: https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}
-      STATUS_OK: ${{ contains(fromJSON('["skipped", "success"]'), needs.create_run.result) }}
+      STATUS_OK: ${{ contains(fromJSON('["skipped", "success"]'), needs.build-doc.result) }}
     steps:
       - name: Get `build-doc` job status
         run: |
Repository: facebook/react
Pull request: [Fiber] `getHoistableRoot()` should account for Document containers

While modern DOM implementations all support getRootNode if you are running React in a runtime which does not the fallback logic which uses `.ownerDocument` works everywhere except when the container is a Document itself. This change corrects this by returning the container intsance if it is a Document type.

Changed files:
filename: packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js
patch: @@ -2523,10 +2523,13 @@ export type HoistableRoot = Document | ShadowRoot;
 export function getHoistableRoot(container: Container): HoistableRoot {
   // $FlowFixMe[method-unbinding]
   return typeof container.getRootNode === 'function'
-    ? /* $FlowFixMe[incompatible-return] Flow types this as returning a `Node`,
+    ? /* $FlowFixMe[incompatible-cast] Flow types this as returning a `Node`,
        * but it's either a `Document` or `ShadowRoot`. */
-      container.getRootNode()
-    : container.ownerDocument;
+      (container.getRootNode(): Document | ShadowRoot)
+    : container.nodeType === DOCUMENT_NODE
+      ? // $FlowFixMe[incompatible-cast] We've constrained this to be a Document which satisfies the return type
+        (container: Document)
+      : container.ownerDocument;
 }
 
 function getCurrentResourceRoot(): null | HoistableRoot {