helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: [`Qwen Omni/VL`] Fix fa tests

Mrope is dependent on the head dim so we need to update it as well

cc @zucchini-nlp (since vlms and such have more potential to fuck things up here)

Changed files:
filename: tests/test_modeling_common.py
patch: @@ -4739,6 +4739,7 @@ def _prepare_config_headdim(config, requested_dim):
         (There are many more examples especially now that the `kernels` library is
         supported)
         """
+        config = copy.deepcopy(config)
 
         def update_config_headdim(config, requested_dim):
             # Flex Attention cannot use dropout
@@ -4784,6 +4785,14 @@ def update_config_headdim(config, requested_dim):
                 )
                 config.cross_hidden_size *= max(requested_dim // cross_head_dim, 1)
 
+            # 3d rope also depends on the head dim
+            # (we assume easy shapes here where we get to the requested head dim at least)
+            if hasattr(config, "rope_scaling") and len(config.rope_scaling.get("mrope_section", None)) > 0:
+                scaling_factor = max(requested_dim // (sum(config.rope_scaling["mrope_section"]) * 2), 1)
+                config.rope_scaling["mrope_section"] = [
+                    section * scaling_factor for section in config.rope_scaling["mrope_section"]
+                ]
+
         # Update config values
         update_config_headdim(config, requested_dim)
         for key in config.sub_configs:
Repository: electron/electron
Pull request: fix: handle potential missing close event property

Backport of #46605

See that PR for details.


Notes: Fixed an error when calling `window.emit('close')` after toggling fullscreen mode.

Changed files:
filename: lib/browser/api/browser-window.ts
patch: @@ -54,7 +54,7 @@ BrowserWindow.prototype._init = function (this: BWT) {
   });
   this.on('close', (event) => {
     queueMicrotask(() => {
-      if (!unresponsiveEvent && !event.defaultPrevented) {
+      if (!unresponsiveEvent && !event?.defaultPrevented) {
         unresponsiveEvent = setTimeout(emitUnresponsiveEvent, 5000);
       }
     });
Repository: ollama/ollama
Pull request: Add missing "be" to GPU support warning message


Changed files:
filename: server/routes.go
patch: @@ -697,7 +697,7 @@ func Serve(ln net.Listener, allowOrigins []string) error {
 	if runtime.GOOS == "linux" {
 		// check compatibility to log warnings
 		if _, err := llm.CheckVRAM(); err != nil {
-			log.Printf("Warning: GPU support may not enabled, check you have installed install GPU drivers: %v", err)
+			log.Printf("Warning: GPU support may not be enabled, check you have installed install GPU drivers: %v", err)
 		}
 	}
Repository: vercel/next.js
Pull request: Added 2 values to the props table at 02-components/image.mdx

Added `unoptimized` and `decoding` props to the props table (the table at the top of the doc page).

I was looking for the `unoptimized` prop and thought it wasn't in this page because I could not find it at the props table. I think it's better included there since it's not deprecated. The same goes for `decoding`.

Added them with the same syntax as the rest values in the table. 

Thanks.

Changed files:
filename: docs/01-app/03-api-reference/02-components/image.mdx
patch: @@ -53,7 +53,9 @@ The following props are available:
 | [`onError`](#onerror)                     | `onError(event => fail()}`               | Function        | -          |
 | [`loading`](#loading)                     | `loading="lazy"`                         | String          | -          |
 | [`blurDataURL`](#blurdataurl)             | `blurDataURL="data:image/jpeg..."`       | String          | -          |
+| [`unoptimized`](#unoptimized)             | `unoptimized={true}`                     | Boolean         | -          |
 | [`overrideSrc`](#overridesrc)             | `overrideSrc="/seo.png"`                 | String          | -          |
+| [`decoding`](#decoding)                   | `decoding="async"`                       | String          | -          |
 </div>
 
 #### `src`
Repository: ollama/ollama
Pull request: remove modelfile context deprecated in v0.0.7

This modelfile variable was deprecated in ollama v0.0.7, which was a very early stage of the project. It was also not documented anywhere, and no longer used in our library images. It should be ok to remove this now.

Changed files:
filename: server/images.go
patch: @@ -63,15 +63,11 @@ func (m *Model) Prompt(request api.GenerateRequest) (string, error) {
 		First  bool
 		System string
 		Prompt string
-
-		// deprecated: versions <= 0.0.7 used this to omit the system prompt
-		Context []int
 	}
 
 	vars.First = len(request.Context) == 0
 
 	vars.Prompt = request.Prompt
-	vars.Context = request.Context
 
 	if len(vars.Context) == 0 {
 		vars.System = m.System
Repository: ollama/ollama
Pull request: Set `NumKeep` to `4` by default


Changed files:
filename: api/types.go
patch: @@ -293,7 +293,7 @@ func DefaultOptions() Options {
 	return Options{
 		// options set on request to runner
 		NumPredict:       -1,
-		NumKeep:          0,
+		NumKeep:          4,
 		Temperature:      0.8,
 		TopK:             40,
 		TopP:             0.9,
Repository: denoland/deno
Pull request: feat(ext/http): make websocket response headers mutable

its very annoying to not be able to set more headers

Changed files:
filename: ext/http/02_websocket.ts
patch: @@ -123,7 +123,7 @@ function upgradeWebSocket(request, options = { __proto__: null }) {
     })();
   }
 
-  const response = fromInnerResponse(r, "immutable");
+  const response = fromInnerResponse(r, "response");
 
   response[_ws] = socket;
 

filename: tests/unit/http_test.ts
patch: @@ -836,6 +836,7 @@ Deno.test(function httpUpgradeWebSocket() {
     response.headers.get("sec-websocket-accept"),
     "s3pPLMBiTxaQ9kYGzzhZRbK+xOo=",
   );
+  response.headers.set("x-new-header", "hello");
 });
 
 Deno.test(function httpUpgradeWebSocketMultipleConnectionOptions() {
Repository: electron/electron
Pull request: fix: handle potential missing close event property

#### Description of Change

Closes https://github.com/electron/electron/issues/46574.

If an app for some reason emits `close` themselves the `close` event won't have an `event` property necessarily. Ensure an error isn't thrown in that case.

#### 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 an error when calling `window.emit('close')` after toggling fullscreen mode.

Changed files:
filename: lib/browser/api/browser-window.ts
patch: @@ -54,7 +54,7 @@ BrowserWindow.prototype._init = function (this: BWT) {
   });
   this.on('close', (event) => {
     queueMicrotask(() => {
-      if (!unresponsiveEvent && !event.defaultPrevented) {
+      if (!unresponsiveEvent && !event?.defaultPrevented) {
         unresponsiveEvent = setTimeout(emitUnresponsiveEvent, 5000);
       }
     });
Repository: denoland/deno
Pull request: fix(lockfile): purged package reqs should be removed from the jsr deps when changing workspace config

* https://github.com/denoland/deno_lockfile/pull/62

Reduces the chance of:

```
> deno i
error: Failed reading lockfile at '.../deno.lock'

Caused by:
    0: Failed deserializing. Lockfile may be corrupt
    1: Invalid jsr dependency 'npm:preact@^10.22.1' for '@preact-icons/common@1.1.0'
```

Changed files:
filename: Cargo.lock
patch: @@ -2378,9 +2378,9 @@ dependencies = [
 
 [[package]]
 name = "deno_lockfile"
-version = "0.32.1"
+version = "0.32.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "64deefbf7192aab1428bf68d2d221cf3161c47481d03a50f4ffc43c7ee10450f"
+checksum = "3d71c0df1464034be21a9472e7ec8f9a21958418d203fa2c40507fb5cafe799d"
 dependencies = [
  "async-trait",
  "deno_semver",

filename: Cargo.toml
patch: @@ -69,7 +69,7 @@ deno_doc = "=0.185.0"
 deno_error = "=0.7.0"
 deno_graph = { version = "=0.102.1", default-features = false }
 deno_lint = "=0.80.0"
-deno_lockfile = "=0.32.1"
+deno_lockfile = "=0.32.2"
 deno_media_type = { version = "=0.2.9", features = ["module_specifier"] }
 deno_native_certs = "0.3.0"
 deno_npm = "=0.41.0"
Repository: ggml-org/llama.cpp
Pull request: ci : apply model label to models

Apply `model` label to `src/models/` commits.

Changed files:
filename: .github/labeler.yml
patch: @@ -76,6 +76,10 @@ ggml:
     - changed-files:
         - any-glob-to-any-file:
             - ggml/**
+model:
+    - changed-files:
+        - any-glob-to-any-file:
+            - src/models/**
 nix:
     - changed-files:
         - any-glob-to-any-file:
Repository: ollama/ollama
Pull request: Only insert system prompt if first request


Changed files:
filename: server/images.go
patch: @@ -69,10 +69,14 @@ func (m *Model) Prompt(request api.GenerateRequest) (string, error) {
 	}
 
 	vars.First = len(request.Context) == 0
-	vars.System = m.System
+
 	vars.Prompt = request.Prompt
 	vars.Context = request.Context
 
+	if len(vars.Context) == 0 {
+		vars.System = m.System
+	}
+
 	if request.System != "" {
 		vars.System = request.System
 	}
Repository: denoland/deno
Pull request: fix(rt): prevent `Event` to be overwritten on bootstrap

Closes #30953

Ensures that the `Event` class always use the imported one from `ext:deno_web/02_event.js`.

Changed files:
filename: runtime/js/99_main.js
patch: @@ -459,17 +459,17 @@ function processRejectionHandled(promise, reason) {
 }
 
 function dispatchLoadEvent() {
-  globalThis_.dispatchEvent(new Event("load"));
+  globalThis_.dispatchEvent(new event.Event("load"));
 }
 
 function dispatchBeforeUnloadEvent() {
   return globalThis_.dispatchEvent(
-    new Event("beforeunload", { cancelable: true }),
+    new event.Event("beforeunload", { cancelable: true }),
   );
 }
 
 function dispatchUnloadEvent() {
-  globalThis_.dispatchEvent(new Event("unload"));
+  globalThis_.dispatchEvent(new event.Event("unload"));
 }
 
 let hasBootstrapped = false;
@@ -975,7 +975,7 @@ event.saveGlobalThisReference(globalThis);
 event.defineEventHandler(globalThis, "unhandledrejection");
 
 // Nothing listens to this, but it warms up the code paths for event dispatch
-(new event.EventTarget()).dispatchEvent(new Event("warmup"));
+(new event.EventTarget()).dispatchEvent(new event.Event("warmup"));
 
 removeImportedOps();
 

filename: tests/unit/globals_test.ts
patch: @@ -1,5 +1,6 @@
 // Copyright 2018-2025 the Deno authors. MIT license.
 
+import { join } from "@std/path";
 import { assert, assertEquals, assertRejects } from "./test_util.ts";
 
 Deno.test(function globalThisExists() {
@@ -208,3 +209,27 @@ Deno.test(function globalGlobalIsWritable() {
   // @ts-ignore the typings here are wrong
   globalThis.global = globalThis;
 });
+
+Deno.test(async function overwriteEventOnExternalModuleShouldNotCrash() {
+  const tmpDir = await Deno.makeTempDir();
+
+  const externalModulePath = join(tmpDir, "overwrite_event.ts");
+  const externalModuleContent =
+    `globalThis.Event = class {}; export default {};`;
+  await Deno.writeTextFile(externalModulePath, externalModuleContent);
+
+  const entrypointPath = join(tmpDir, "index.ts");
+  const entrypointContent = `import("./overwrite_event.ts");`;
+  await Deno.writeTextFile(entrypointPath, entrypointContent);
+
+  const command = new Deno.Command(Deno.execPath(), {
+    args: ["run", "-A", entrypointPath],
+    stdout: "null",
+    stderr: "null",
+  });
+  const child = command.spawn();
+
+  const status = await child.status;
+  assert(status.success);
+  await Deno.remove(tmpDir, { recursive: true });
+});
Repository: facebook/react
Pull request: Move enableUseDeferredValueInitialArg to canary

Per team discussion, this upgrades the `initialValue` argument for `useDeferredValue` from experimental to canary.

- Original implementation PR: https://github.com/facebook/react/pull/27500
- API documentation PR: https://github.com/reactjs/react.dev/pull/6747

I left it disabled at Meta for now in case there's old code somewhere that is still passing an `options` object as the second argument.

Changed files:
filename: packages/shared/ReactFeatureFlags.js
patch: @@ -116,8 +116,6 @@ export const alwaysThrottleRetries = true;
 
 export const passChildrenWhenCloningPersistedNodes = false;
 
-export const enableUseDeferredValueInitialArg = __EXPERIMENTAL__;
-
 export const enableServerComponentLogs = __EXPERIMENTAL__;
 
 /**
@@ -191,6 +189,9 @@ export const disableDOMTestUtils = true;
 // Make <Context> equivalent to <Context.Provider> instead of <Context.Consumer>
 export const enableRenderableContext = true;
 
+// Enables the `initialValue` option for `useDeferredValue`
+export const enableUseDeferredValueInitialArg = true;
+
 // -----------------------------------------------------------------------------
 // Chopping Block
 //
Repository: electron/electron
Pull request: fix: re-enable OOP printing on Linux

Backport of #46529

See that PR for details.


Notes: none

Changed files:
filename: patches/chromium/printing.patch
patch: @@ -932,3 +932,19 @@ index 63f170c95050416c595e62f4c460c4cd6b7dbd1c..157e3d046889f9c63fdf0fd5d503890f
    // Does bookkeeping when an error occurs.
    virtual mojom::ResultCode OnError();
  
+diff --git a/printing/printing_context_linux.cc b/printing/printing_context_linux.cc
+index fa55d2a1c5f4207cb8eda9305e8bf2af29352c91..13ca65c2c7c3c24882061d8e224bf93899d9a7ab 100644
+--- a/printing/printing_context_linux.cc
++++ b/printing/printing_context_linux.cc
+@@ -52,9 +52,8 @@ void PrintingContextLinux::AskUserForSettings(int max_pages,
+                                               bool is_scripted,
+                                               PrintSettingsCallback callback) {
+   if (!print_dialog_) {
+-    // Can only get here if the renderer is sending bad messages.
+-    // http://crbug.com/341777
+-    NOTREACHED();
++    print_dialog_ = ui::LinuxUi::instance()->CreatePrintDialog(this);
++    print_dialog_->UseDefaultSettings();
+   }
+ 
+   print_dialog_->ShowDialog(delegate_->GetParentView(), has_selection,

filename: shell/browser/feature_list.cc
patch: @@ -61,14 +61,6 @@ void InitializeFeatureList() {
       std::string(",") + features::kMacWebContentsOcclusion.name;
 #endif
 
-#if BUILDFLAG(IS_LINUX) && BUILDFLAG(ENABLE_PRINTING)
-  disable_features +=
-      // EnableOopPrintDrivers is still a bit half-baked on Linux and
-      // causes crashes when trying to show dialogs.
-      // TODO(codebytere): figure out how to re-enable this with our patches.
-      std::string(",") + printing::features::kEnableOopPrintDrivers.name;
-#endif
-
 #if BUILDFLAG(ENABLE_PDF_VIEWER)
   // Enable window.showSaveFilePicker api for saving pdf files.
   // Refs https://issues.chromium.org/issues/373852607
Repository: huggingface/transformers
Pull request: Fix mistral3 tests after "[Kosmos 2.5] Rename checkpoints"

Partially  Reverts huggingface/transformers#40338

Changed files:
filename: tests/models/mistral3/test_modeling_mistral3.py
patch: @@ -334,7 +334,10 @@ def test_mistral3_integration_batched_generate(self):
                 {
                     "role": "user",
                     "content": [
-                        {"type": "image", "url": "https://huggingface.co/microsoft/kosmos-2.5/resolve/main/view.jpg"},
+                        {
+                            "type": "image",
+                            "url": "https://huggingface.co/ydshieh/mistral3-test-data/resolve/main/view.jpg",
+                        },
                         {"type": "text", "text": "Write a haiku for this image"},
                     ],
                 },
@@ -402,7 +405,10 @@ def test_mistral3_integration_batched_generate_multi_image(self):
                 {
                     "role": "user",
                     "content": [
-                        {"type": "image", "url": "https://huggingface.co/microsoft/kosmos-2.5/resolve/main/view.jpg"},
+                        {
+                            "type": "image",
+                            "url": "https://huggingface.co/ydshieh/mistral3-test-data/resolve/main/view.jpg",
+                        },
                         {"type": "text", "text": "Write a haiku for this image"},
                     ],
                 },
@@ -413,11 +419,11 @@ def test_mistral3_integration_batched_generate_multi_image(self):
                     "content": [
                         {
                             "type": "image",
-                            "url": "https://huggingface.co/microsoft/kosmos-2.5/resolve/main/Statue-of-Liberty-Island-New-York-Bay.jpg",
+                            "url": "https://huggingface.co/ydshieh/mistral3-test-data/resolve/main/Statue-of-Liberty-Island-New-York-Bay.jpg",
                         },
                         {
                             "type": "image",
-                            "url": "https://huggingface.co/microsoft/kosmos-2.5/resolve/main/golden-gate-bridge-san-francisco-purple-flowers-california-echium-candicans-36805947.jpg",
+                            "url": "https://huggingface.co/ydshieh/mistral3-test-data/resolve/main/golden-gate-bridge-san-francisco-purple-flowers-california-echium-candicans-36805947.jpg",
                         },
                         {
                             "type": "text",
Repository: vercel/next.js
Pull request: fix dynamic param extraction in edge-ssr-app

Fixes edge runtime pages with dynamic routes returning 500 errors when the URL segment literally contains bracket syntax that matches the folder name (e.g., accessing `/[id]` for a route defined as `/[id]/page.tsx`).

The removal of web-server.ts in #81389 modified how route params were extracted for dynamic routes. The route matcher in prepare() was being created with the full internal page path (e.g., /[id]/page) which generated a regex expecting /page at the end. However, actual URL pathnames don't include the /page suffix, causing the matcher to fail when trying to extract params. This issue specifically manifested when the URL contained literal brackets (like /[id]), as the fallback param extraction would fail and leave params empty, leading to the error. 

This PR normalizes the page path using before creating the route matcher in prepare(). This ensures the regex pattern matches the actual URL pathname format (without /page suffix). 

Fixes NEXT-4698

Changed files:
filename: packages/next/src/server/route-modules/route-module.ts
patch: @@ -598,8 +598,13 @@ export abstract class RouteModule<
       }
     }
 
+    // Normalize the page path for route matching. The srcPage contains the
+    // internal page path (e.g., /app/[slug]/page), but route matchers expect
+    // the pathname format (e.g., /app/[slug]).
+    const normalizedSrcPage = normalizeAppPath(srcPage)
+
     const serverUtils = getServerUtils({
-      page: srcPage,
+      page: normalizedSrcPage,
       i18n,
       basePath,
       rewrites,
@@ -807,7 +812,6 @@ export abstract class RouteModule<
     const nextConfig =
       routerServerContext?.nextConfig || serverFilesManifest.config
 
-    const normalizedSrcPage = normalizeAppPath(srcPage)
     let resolvedPathname =
       getRequestMeta(req, 'rewroteURL') || normalizedSrcPage
 

filename: test/e2e/app-dir/rsc-basic/app/node/dynamic/[id]/page.js
patch: @@ -0,0 +1,3 @@
+export default function page() {
+  return 'dynamic route [id] page'
+}

filename: test/e2e/app-dir/rsc-basic/app/node/dynamic/page.js
patch: @@ -0,0 +1,3 @@
+export default function page() {
+  return 'dynamic route index page'
+}

filename: test/e2e/app-dir/rsc-basic/rsc-basic.test.ts
patch: @@ -382,6 +382,15 @@ describe('app dir - rsc basics', () => {
     expect(dynamicRouteUrl).toBe(`${next.url}/edge/dynamic/123`)
   })
 
+  describe.each(['node', 'edge'])(`%s`, (runtime) => {
+    it('should handle dynamic routes when URL segment matches the folder bracket syntax', async () => {
+      const browser = await next.browser(`/${runtime}/dynamic/[id]`)
+      expect(await browser.elementByCss('body').text()).toBe(
+        'dynamic route [id] page'
+      )
+    })
+  })
+
   it('should support streaming for flight response', async () => {
     await next
       .fetch(`/?${NEXT_RSC_UNION_QUERY}`, {
Repository: ollama/ollama
Pull request: update default NumKeep


Changed files:
filename: api/types.go
patch: @@ -293,7 +293,7 @@ func DefaultOptions() Options {
 	return Options{
 		// options set on request to runner
 		NumPredict:       -1,
-		NumKeep:          -1,
+		NumKeep:          0,
 		Temperature:      0.8,
 		TopK:             40,
 		TopP:             0.9,
Repository: electron/electron
Pull request: fix: re-enable OOP printing on Linux

#### Description of Change

Follow up to https://github.com/electron/electron/pull/46087 - we should be using the new OOP printing standard on Linux. Our initial call to update the print settings triggers an unexpected pathway in Linux OOP print code, which expected that the dialog would be initialized by a call to request default print settings that never occurs.

#### 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: none

Changed files:
filename: patches/chromium/printing.patch
patch: @@ -932,3 +932,19 @@ index 63f170c95050416c595e62f4c460c4cd6b7dbd1c..157e3d046889f9c63fdf0fd5d503890f
    // Does bookkeeping when an error occurs.
    virtual mojom::ResultCode OnError();
  
+diff --git a/printing/printing_context_linux.cc b/printing/printing_context_linux.cc
+index fa55d2a1c5f4207cb8eda9305e8bf2af29352c91..13ca65c2c7c3c24882061d8e224bf93899d9a7ab 100644
+--- a/printing/printing_context_linux.cc
++++ b/printing/printing_context_linux.cc
+@@ -52,9 +52,8 @@ void PrintingContextLinux::AskUserForSettings(int max_pages,
+                                               bool is_scripted,
+                                               PrintSettingsCallback callback) {
+   if (!print_dialog_) {
+-    // Can only get here if the renderer is sending bad messages.
+-    // http://crbug.com/341777
+-    NOTREACHED();
++    print_dialog_ = ui::LinuxUi::instance()->CreatePrintDialog(this);
++    print_dialog_->UseDefaultSettings();
+   }
+ 
+   print_dialog_->ShowDialog(delegate_->GetParentView(), has_selection,

filename: shell/browser/feature_list.cc
patch: @@ -61,14 +61,6 @@ void InitializeFeatureList() {
       std::string(",") + features::kMacWebContentsOcclusion.name;
 #endif
 
-#if BUILDFLAG(IS_LINUX) && BUILDFLAG(ENABLE_PRINTING)
-  disable_features +=
-      // EnableOopPrintDrivers is still a bit half-baked on Linux and
-      // causes crashes when trying to show dialogs.
-      // TODO(codebytere): figure out how to re-enable this with our patches.
-      std::string(",") + printing::features::kEnableOopPrintDrivers.name;
-#endif
-
 #if BUILDFLAG(ENABLE_PDF_VIEWER)
   // Enable window.showSaveFilePicker api for saving pdf files.
   // Refs https://issues.chromium.org/issues/373852607