helloadhavan/github_issues huggingface dataset:
Repository: ggml-org/llama.cpp
Pull request: [SYCL] fix norm kernels: l2_norm, group_norm, rms_norm by remove assert

fix norm kernels: l2_norm, group_norm, rms_norm by remove assert. 
all ut cases of norm are 100% passed.
no crash of UT cases.
update the ops.md


Changed files:
filename: ggml/src/ggml-sycl/ggml-sycl.cpp
patch: @@ -4606,14 +4606,12 @@ static bool ggml_backend_sycl_device_supports_op(ggml_backend_dev_t dev, const g
             return (op->type == GGML_TYPE_F32 && op->src[0]->type == GGML_TYPE_F32) && (op->type == op->src[0]->type);
 #endif
         case GGML_OP_NORM:
-            return true;
         case GGML_OP_L2_NORM:
         case GGML_OP_GROUP_NORM:
-            return ggml_is_contiguous(op->src[0]);
         case GGML_OP_RMS_NORM:
-            return ((op->src[0]->ne[0] % WARP_SIZE) == 0);
+            return true;
         case GGML_OP_RMS_NORM_BACK:
-            return ((op->src[0]->ne[0] % WARP_SIZE) == 0);
+            return ggml_is_contiguous(op->src[0]);
         case GGML_OP_SCALE:
             return true;
         case GGML_OP_CONT:

filename: ggml/src/ggml-sycl/norm.cpp
patch: @@ -251,7 +251,6 @@ static void norm_f32_sycl(const float * x, float * dst, const int ncols, const i
         const float eps, queue_ptr stream, int device) {
 
     const sycl::range<3> global_dims(nsamples, nchannels, nrows);
-    GGML_ASSERT(ncols % WARP_SIZE == 0);
     if (ncols < 1024) {
         const sycl::range<3> block_dims(1, 1, WARP_SIZE);
         stream->submit([&](sycl::handler& cgh) {
@@ -334,7 +333,6 @@ static void group_norm_f32_sycl(const float* x, float* dst,
 
 static void rms_norm_f32_sycl(const float* x, float* dst, const int ncols, const int nrows, const int nchannels, const int nsamples,
         const int64_t stride_row, const int64_t stride_channel, const int64_t stride_sample, const float eps, queue_ptr stream, int device) {
-    GGML_ASSERT(ncols % WARP_SIZE == 0);
     // printf("%s ncols=%d, nrows=%d, WARP_SIZE=%d\n", __func__, ncols, nrows, WARP_SIZE);
 
     const sycl::range<3> global_dims(nsamples, nchannels, nrows);
@@ -374,7 +372,6 @@ static void rms_norm_f32_sycl(const float* x, float* dst, const int ncols, const
 static void l2_norm_f32_sycl(const float* x, float* dst, const int ncols,
     const int nrows, const float eps,
     queue_ptr stream, int device) {
-    GGML_ASSERT(ncols % WARP_SIZE == 0);
     // printf("%s ncols=%d, nrows=%d, WARP_SIZE=%d\n", __func__, ncols, nrows, WARP_SIZE);
     if (ncols < 1024) {
         const sycl::range<3> block_dims(1, 1, WARP_SIZE);
Repository: vuejs/vue
Pull request: Fix links on vue-template-compiler README


Changed files:
filename: packages/vue-template-compiler/README.md
patch: @@ -38,7 +38,7 @@ The optional `options` object can contain the following:
 
 - `modules`
 
-  An array of compiler modules. For details on compiler modules, refer to its [type definition](https://github.com/vuejs/vue/blob/dev/flow/compiler.js#L31) and the [built-in modules](https://github.com/vuejs/vue/tree/dev/src/platforms/web/compiler/modules).
+  An array of compiler modules. For details on compiler modules, refer to its [type definition](https://github.com/vuejs/vue/blob/dev/flow/compiler.js#L35) and the [built-in modules](https://github.com/vuejs/vue/tree/dev/src/platforms/web/compiler/modules).
 
 - `directives`
 
@@ -55,7 +55,7 @@ The optional `options` object can contain the following:
 
   By default, a compile-time directive will extract the directive and the directive will not be present at runtime. If you want the directive to also be handled by a runtime definition, return `true` in the transform function.
 
-  Refer to the implementation of some [built-in compile-time directives](https://github.com/vuejs/vue/tree/next/src/platforms/web/compiler/directives).
+  Refer to the implementation of some [built-in compile-time directives](https://github.com/vuejs/vue/tree/dev/src/platforms/web/compiler/directives).
 
 - `preserveWhitespace`
 
@@ -80,7 +80,7 @@ This is only useful at runtime with pre-configured builds, so it doesn't accept
 
 ### compiler.parseComponent(file, [options])
 
-Parse a SFC (single-file component, or `*.vue` file) into a [descriptor](https://github.com/vuejs/vue/blob/dev/flow/compiler.js#L131). This is used in SFC build tools like `vue-loader` and `vueify`.
+Parse a SFC (single-file component, or `*.vue` file) into a [descriptor](https://github.com/vuejs/vue/blob/dev/flow/compiler.js#L137). This is used in SFC build tools like `vue-loader` and `vueify`.
 
 #### Options
Repository: ollama/ollama
Pull request: Sort Batch Embed Results

batch embed returns results out of order, fix by sorting results by task id

Resolves #6187

Changed files:
filename: llm/ext_server/server.cpp
patch: @@ -44,6 +44,7 @@
 #include <errhandlingapi.h>
 #endif
 
+#include <algorithm>
 #include <cstddef>
 #include <thread>
 #include <chrono>
@@ -1220,6 +1221,7 @@ struct llama_server_context
 
                 res.result_json = json
                 {
+                    {"id", res.id},
                     {"embedding", std::vector<float>(embd, embd + n_embd)},
                     {"timings",             slot.get_formated_timings()},
                 };
@@ -3203,6 +3205,10 @@ int main(int argc, char **argv) {
                     }
 
                     responses = result.result_json.value("results", std::vector<json>{result.result_json});
+                    std::sort(responses.begin(), responses.end(), [](const json& a, const json& b) {
+                        return a["id"] < b["id"];
+                    });
+
                     json embeddings = json::array();
 
                     int prompt_n = 0;
Repository: ggml-org/llama.cpp
Pull request: ci : find latest release with asset for winget

Find latest release that contains the Windows Vulkan asset in order to prevent failures like this:
https://github.com/ggml-org/llama.cpp/actions/runs/21426531680/job/61696642632

Successful dummy test here:
https://github.com/CISC/llama.cpp/actions/runs/21444347462/job/61755778295

Changed files:
filename: .github/workflows/winget.yml
patch: @@ -28,16 +28,17 @@ jobs:
               owner: context.repo.owner,
               repo: context.repo.repo,
             });
-            console.log("Latest release:", releases[0].tag_name);
-            return releases[0].tag_name;
+            const { tag_name: version, assets: assets } = releases.find(({assets}) => assets.find(asset => asset.name.includes('win-vulkan')));
+            const { browser_download_url: asset_url } = assets.find(asset => asset.name.includes('win-vulkan'));
+            console.log("Latest release:", version);
+            core.setOutput('VERSION', version);
+            core.setOutput('ASSETURL', asset_url);
 
       - name: Update manifest
-        env:
-          VERSION: ${{ steps.find_latest_release.outputs.result }}
         run: |
           echo "Updating manifest..."
-          komac update --version ${{ env.VERSION }} \
-            --urls "https://github.com/ggml-org/llama.cpp/releases/download/${{ env.VERSION }}/llama-${{ env.VERSION }}-bin-win-vulkan-x64.zip" \
+          komac update --version ${{ steps.find_latest_release.outputs.VERSION }} \
+            --urls "${{ steps.find_latest_release.outputs.ASSETURL }}" \
             --token ${{ secrets.WINGET_GITHUB_TOKEN }} \
             --submit \
             ggml.llamacpp
Repository: vuejs/vue
Pull request: Typo fix: optimizier -> optimizer

Fixed the spelling error of optimizier to optimizer in compiler src. 


Changed files:
filename: src/compiler/optimizer.js
patch: @@ -8,7 +8,7 @@ let isPlatformReservedTag
 const genStaticKeysCached = cached(genStaticKeys)
 
 /**
- * Goal of the optimizier: walk the generated template AST tree
+ * Goal of the optimizer: walk the generated template AST tree
  * and detect sub-trees that are purely static, i.e. parts of
  * the DOM that never needs to change.
  *
Repository: facebook/react
Pull request: [ci] Make gh workflow names consistent


Super minor change to keep our naming scheme consistent for gh workflows


Changed files:
filename: .github/workflows/shared_discord_notify.yml
patch: @@ -1,8 +1,8 @@
-name: Discord Notify
+name: (Shared) Discord Notify
 
 on:
   pull_request_target:
-    types: [ labeled ]
+    types: [labeled]
 
 jobs:
   notify:
@@ -18,4 +18,4 @@ jobs:
           embed-author-icon-url: ${{ github.event.pull_request.user.avatar_url }}
           embed-title: '#${{ github.event.number }} (+${{github.event.pull_request.additions}} -${{github.event.pull_request.deletions}}): ${{ github.event.pull_request.title }}'
           embed-description: ${{ github.event.pull_request.body }}
-          embed-url: ${{ github.event.pull_request.html_url }}
\ No newline at end of file
+          embed-url: ${{ github.event.pull_request.html_url }}
Repository: nodejs/node
Pull request: lib: fix typo in createBlobReaderStream

Corrects the misspelling of "proecss" to "process".

Changed files:
filename: lib/internal/blob.js
patch: @@ -478,7 +478,7 @@ function createBlobReaderStream(reader) {
           // We keep reading until we either reach EOS, some error, or we
           // hit the flow rate of the stream (c.desiredSize).
           // We use set immediate here because we have to allow the event
-          // loop to turn in order to proecss any pending i/o. Using
+          // loop to turn in order to process any pending i/o. Using
           // queueMicrotask won't allow the event loop to turn.
           setImmediate(() => {
             if (c.desiredSize < 0) {
Repository: electron/electron
Pull request: fix: `dialog.showMessageBox` defaultid on Windows

Backport of #48216

See that PR for details.


Notes: Fixes Windows `dialog.showMessageBox` default button handling.


Changed files:
filename: shell/browser/ui/message_box_win.cc
patch: @@ -163,8 +163,19 @@ DialogResult ShowTaskDialogWstr(gfx::AcceleratedWidget parent,
     config.dwFlags |= TDF_POSITION_RELATIVE_TO_WINDOW;
   }
 
-  if (default_id > 0)
-    config.nDefaultButton = kIDStart + default_id;
+  if (default_id >= 0 &&
+      base::checked_cast<size_t>(default_id) < buttons.size()) {
+    if (!no_link) {
+      auto common = GetCommonID(buttons[default_id]);
+      if (common.button != -1) {
+        config.nDefaultButton = common.id;
+      } else {
+        config.nDefaultButton = kIDStart + default_id;
+      }
+    } else {
+      config.nDefaultButton = kIDStart + default_id;
+    }
+  }
 
   // TaskDialogIndirect doesn't allow empty name, if we set empty title it
   // will show "electron.exe" in title.
Repository: vuejs/vue
Pull request: Fix minor typos

Fix minor typos


Changed files:
filename: src/core/vdom/create-component.js
patch: @@ -236,7 +236,7 @@ function resolveAsyncComponent (
 }
 
 function extractProps (data: VNodeData, Ctor: Class<Component>): ?Object {
-  // we are only extrating raw values here.
+  // we are only extracting raw values here.
   // validation and default values are handled in the child
   // component itself.
   const propOptions = Ctor.options.props

filename: src/platforms/web/runtime/directives/model.js
patch: @@ -59,7 +59,7 @@ export default {
       setSelected(el, binding, vnode.context)
       // in case the options rendered by v-for have changed,
       // it's possible that the value is out-of-sync with the rendered options.
-      // detect such cases and filter out values that no longer has a matchig
+      // detect such cases and filter out values that no longer has a matching
       // option in the DOM.
       const needReset = el.multiple
         ? binding.value.some(v => hasNoMatchingOption(v, el.options))

filename: src/server/render-stream.js
patch: @@ -1,7 +1,7 @@
 /* @flow */
 
 /**
- * Original RenderStream implmentation by Sasha Aickin (@aickin)
+ * Original RenderStream implementation by Sasha Aickin (@aickin)
  * Licensed under the Apache License, Version 2.0
  * http://www.apache.org/licenses/LICENSE-2.0
  *
Repository: ollama/ollama
Pull request: Implement linux NUMA detection

If the system has multiple numa nodes, enable numa support in llama.cpp If we detect `numactl` in the path, use that, else use the basic "distribute" mode.

This also removes the `use_numa` setting as the bool type is no longer useful given this parameter now requires a mode.

Fixes #6093 
Fixes #2496 

Changed files:
filename: api/types.go
patch: @@ -231,7 +231,6 @@ type Options struct {
 
 // Runner options which must be set when the model is loaded into memory
 type Runner struct {
-	UseNUMA   bool  `json:"numa,omitempty"`
 	NumCtx    int   `json:"num_ctx,omitempty"`
 	NumBatch  int   `json:"num_batch,omitempty"`
 	NumGPU    int   `json:"num_gpu,omitempty"`
@@ -615,7 +614,6 @@ func DefaultOptions() Options {
 			F16KV:     true,
 			UseMLock:  false,
 			UseMMap:   nil,
-			UseNUMA:   false,
 		},
 	}
 }

filename: gpu/cpu_common.go
patch: @@ -1,6 +1,11 @@
 package gpu
 
 import (
+	"os"
+	"path/filepath"
+	"runtime"
+	"strings"
+
 	"golang.org/x/sys/cpu"
 )
 
@@ -14,3 +19,19 @@ func GetCPUCapability() CPUCapability {
 	// else LCD
 	return CPUCapabilityNone
 }
+
+func IsNUMA() bool {
+	if runtime.GOOS != "linux" {
+		// numa support in llama.cpp is linux only
+		return false
+	}
+	ids := map[string]interface{}{}
+	packageIds, _ := filepath.Glob("/sys/devices/system/cpu/cpu*/topology/physical_package_id")
+	for _, packageId := range packageIds {
+		id, err := os.ReadFile(packageId)
+		if err == nil {
+			ids[strings.TrimSpace(string(id))] = struct{}{}
+		}
+	}
+	return len(ids) > 1
+}

filename: llm/server.go
patch: @@ -256,8 +256,14 @@ func NewLlamaServer(gpus gpu.GpuInfoList, model string, ggml *GGML, adapters, pr
 		params = append(params, "--mlock")
 	}
 
-	if opts.UseNUMA {
-		params = append(params, "--numa")
+	if gpu.IsNUMA() {
+		numaMode := "distribute"
+		if runtime.GOOS == "linux" {
+			if _, err := exec.LookPath("numactl"); err == nil {
+				numaMode = "numactl"
+			}
+		}
+		params = append(params, "--numa", numaMode)
 	}
 
 	params = append(params, "--parallel", strconv.Itoa(numParallel))
Repository: ggml-org/llama.cpp
Pull request: convert : yield Mamba2Model/GraniteMoeModel modify_tensors

This commit updates the `GraniteHybridModel` class' modify_tensors function to properly delegate to `Mamba2Model.modify_tensors` and `GraniteMoeModel.modify_tensors` using 'yield from' instead of 'return'.

The motivation for this is that modify_tensors is a generator function (it uses 'yield from'), but the two calls above use return statements but don't yield anything which means that the the caller of this function will not receive any yielded values from it. And this causes layer tensors to be silently dropped during conversion.


Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -8912,13 +8912,16 @@ def modify_tensors(
             name.endswith("block_sparse_moe.input_linear.weight")
             or "shared_mlp" in name
         ):
-            return GraniteMoeModel.modify_tensors(self, data_torch, name, bid)
+            yield from GraniteMoeModel.modify_tensors(self, data_torch, name, bid)
+            return
 
         # Determine whether this is a mamba layer or an attention layer
         if bid in self._ssm_layers:
-            return Mamba2Model.modify_tensors(self, data_torch, name, bid)
+            yield from Mamba2Model.modify_tensors(self, data_torch, name, bid)
+            return
         elif bid in self._attn_layers:
-            return GraniteMoeModel.modify_tensors(self, data_torch, name, bid)
+            yield from GraniteMoeModel.modify_tensors(self, data_torch, name, bid)
+            return
         yield from ModelBase.modify_tensors(self, data_torch, name, bid)
 
     def set_gguf_parameters(self):
Repository: ollama/ollama
Pull request: use testing tempdirs

setting OLLAMA_MODELS in createRequest ensures the request is handled in a temporary directory

Changed files:
filename: server/routes_create_test.go
patch: @@ -2,6 +2,7 @@ package server
 
 import (
 	"bytes"
+	"cmp"
 	"encoding/json"
 	"fmt"
 	"io"
@@ -53,6 +54,8 @@ func (t *responseRecorder) CloseNotify() <-chan bool {
 
 func createRequest(t *testing.T, fn func(*gin.Context), body any) *httptest.ResponseRecorder {
 	t.Helper()
+	// if OLLAMA_MODELS is not set, set it to the temp directory
+	t.Setenv("OLLAMA_MODELS", cmp.Or(os.Getenv("OLLAMA_MODELS"), t.TempDir()))
 
 	w := NewRecorder()
 	c, _ := gin.CreateTestContext(w)
Repository: facebook/react
Pull request: [playground] Use default compiler config


The playground's compilation mode is currently set to 'all' along with reporting all errors.

This tends to be misleading since people usually expect a 1:1 match between how the playground works with what the compiler does in their codebase, eg https://github.com/reactwg/react-compiler/discussions/51.


Changed files:
filename: compiler/apps/playground/components/Editor/EditorImpl.tsx
patch: @@ -78,8 +78,6 @@ function invokeCompiler(
       logEvent: () => {},
     },
     environment,
-    compilationMode: 'all',
-    panicThreshold: 'all_errors',
   });
   const ast = parseInput(source, language);
   let result = transformFromAstSync(ast, source, {
Repository: electron/electron
Pull request: fix: `dialog.showMessageBox` defaultid on Windows

#### Description of Change

Closes https://github.com/electron/electron/issues/22600.

Fixes Windows message box default button handling.

On Windows, `dialog.showMessageBox` / `showMessageBoxSync` ignored defaultId when it was 0 (first button) and failed whenever the target button text matched a common button (OK/Yes/No/Cancel/etc.), because we always offset with `kIDStart instead of using the native common button ID. This meant the default focus could be wrong. The fix accepts `defaultId >= 0`, bounds‑checks it, and maps to the proper ID (common button ID or custom offset) before calling TaskDialogIndirect.

Using the following code:

```js
dialog.showMessageBoxSync(null, {
    type:'warning',
    buttons: ['Yes', 'No'],
    defaultId: 1,
    cancelId: 1,
    title: 'Power Control',
    message: "Warning!"
});
```

<details><summary>Before</summary>
<p>

<img width="366" height="142" alt="Screenshot 2025-08-29 at 9 54 35 AM" src="https://github.com/user-attachments/assets/6df67b59-9986-4d94-aa57-6b21327e2d2c" />

</p>
</details> 

<details><summary>After</summary>
<p>

<img width="365" height="139" alt="Screenshot 2025-08-29 at 9 54 55 AM" src="https://github.com/user-attachments/assets/004cbce1-56cc-416a-9bce-852d25f87620" />

</p>
</details> 

#### 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: Fixes Windows `dialog.showMessageBox` default button handling.


Changed files:
filename: shell/browser/ui/message_box_win.cc
patch: @@ -163,8 +163,19 @@ DialogResult ShowTaskDialogWstr(gfx::AcceleratedWidget parent,
     config.dwFlags |= TDF_POSITION_RELATIVE_TO_WINDOW;
   }
 
-  if (default_id > 0)
-    config.nDefaultButton = kIDStart + default_id;
+  if (default_id >= 0 &&
+      base::checked_cast<size_t>(default_id) < buttons.size()) {
+    if (!no_link) {
+      auto common = GetCommonID(buttons[default_id]);
+      if (common.button != -1) {
+        config.nDefaultButton = common.id;
+      } else {
+        config.nDefaultButton = kIDStart + default_id;
+      }
+    } else {
+      config.nDefaultButton = kIDStart + default_id;
+    }
+  }
 
   // TaskDialogIndirect doesn't allow empty name, if we set empty title it
   // will show "electron.exe" in title.
Repository: vuejs/vue
Pull request: Fix typo in 'component'

<!--
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/core/vdom/vnode.js
patch: @@ -12,7 +12,7 @@ export default class VNode {
   key: string | number | void;
   componentOptions: VNodeComponentOptions | void;
   child: Component | void; // component instance
-  parent: VNode | void; // compoennt placeholder node
+  parent: VNode | void; // component placeholder node
   raw: boolean; // contains raw HTML? (server only)
   isStatic: boolean; // hoisted static node
   isRootInsert: boolean; // necessary for enter transition check
Repository: ggml-org/llama.cpp
Pull request: ggml-sycl: remove unused syclcompat header

The `syclcompat/math.hpp` is not used anymore. The change that introduced it was successfully reverted (https://github.com/ggml-org/llama.cpp/pull/17826). This include path will become obsolete and dropped in oneAPI 2026.0 effectively breaking `ggml-sycl` builds.

*Make sure to read the [contributing guidelines](https://github.com/ggml-org/llama.cpp/blob/master/CONTRIBUTING.md) before submitting a PR*


Changed files:
filename: ggml/src/ggml-sycl/dpct/helper.hpp
patch: @@ -15,7 +15,6 @@
 
 #include <sycl/sycl.hpp>
 #include <sycl/half_type.hpp>
-#include <syclcompat/math.hpp>
 #include <map>
 
 #ifdef GGML_SYCL_USE_INTEL_ONEMKL
Repository: vuejs/vue
Pull request: Fixed typos.

<!--
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: build/config.js
patch: @@ -30,7 +30,7 @@ const builds = {
     env: 'production',
     banner
   },
-  // Runtime+compiler standalone developement build.
+  // Runtime+compiler standalone development build.
   'web-standalone-dev': {
     entry: path.resolve(__dirname, '../src/entries/web-runtime-with-compiler.js'),
     dest: path.resolve(__dirname, '../dist/vue.js'),

filename: src/core/observer/watcher.js
patch: @@ -204,7 +204,7 @@ export default class Watcher {
   }
 
   /**
-   * Remove self from all dependencies' subcriber list.
+   * Remove self from all dependencies' subscriber list.
    */
   teardown () {
     if (this.active) {

filename: src/server/create-renderer.js
patch: @@ -21,7 +21,7 @@ export function createRenderer ({
 } {
   if (process.env.VUE_ENV !== 'server') {
     warn(
-      'You are using createRenderer without setting VUE_ENV enviroment variable to "server". ' +
+      'You are using createRenderer without setting VUE_ENV environment variable to "server". ' +
       'It is recommended to set VUE_ENV=server this will help rendering performance, ' +
       'by turning data observation off.'
     )

filename: test/ssr/ssr-env.spec.js
patch: @@ -39,7 +39,7 @@ describe('SSR: VUE_ENV=server', () => {
   it('should warn when not set', () => {
     process.env.VUE_ENV = ''
     createRenderer()
-    expect('You are using createRenderer without setting VUE_ENV enviroment').toHaveBeenWarned()
+    expect('You are using createRenderer without setting VUE_ENV environment').toHaveBeenWarned()
     process.env.VUE_ENV = 'server'
   })
 })

filename: test/unit/modules/vdom/modules/class.spec.js
patch: @@ -2,7 +2,7 @@ import { patch } from 'web/runtime/patch'
 import VNode from 'core/vdom/vnode'
 
 describe('vdom class module', () => {
-  it('shuold create an element with staticClass', () => {
+  it('should create an element with staticClass', () => {
     const vnode = new VNode('p', { staticClass: 'class1' })
     const elm = patch(null, vnode)
     expect(elm).toHaveClass('class1')