helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: Next-api: Check `client_source_maps` before generate source map

### What?

Fixes #89894. See original issue for detailed information.

This check should avoid generating unexpected source map.

### Why?

Please see #89894 -- an source map was unexpectedly generated from with `productionBrowserSourceMaps` set to false.

### How?

Added an extra check to next-api crate. Now `client_assets.insert(ResolvedVc::upcast(polyfill_source_map_asset));` will only be executed if `*client_source_maps != SourceMapsType::None`.
It seems that it fixed the issue on my computer, but further verifications are required (help wanted).

### Additional information

I am not sure if tests are required for this check.
Help wanted (if tests are necessary).

Changed files:
filename: crates/next-api/src/app.rs
patch: @@ -52,7 +52,7 @@ use turbopack_core::{
     asset::AssetContent,
     chunk::{
         ChunkGroupResult, ChunkingContext, ChunkingContextExt, EvaluatableAsset, EvaluatableAssets,
-        availability_info::AvailabilityInfo,
+        SourceMapsType, availability_info::AvailabilityInfo,
     },
     file_source::FileSource,
     ident::{AssetIdent, Layer},
@@ -1368,13 +1368,19 @@ impl AppEndpoint {
         let polyfill_output_asset = ResolvedVc::upcast(polyfill_output);
         client_assets.insert(polyfill_output_asset);
 
-        let polyfill_source_map_asset = SourceMapAsset::new_fixed(
-            polyfill_output_path.clone(),
-            *ResolvedVc::upcast(polyfill_output),
-        )
-        .to_resolved()
-        .await?;
-        client_assets.insert(ResolvedVc::upcast(polyfill_source_map_asset));
+        let client_source_maps = project
+            .next_config()
+            .client_source_maps(project.next_mode())
+            .await?;
+        if *client_source_maps != SourceMapsType::None {
+            let polyfill_source_map_asset = SourceMapAsset::new_fixed(
+                polyfill_output_path.clone(),
+                *ResolvedVc::upcast(polyfill_output),
+            )
+            .to_resolved()
+            .await?;
+            client_assets.insert(ResolvedVc::upcast(polyfill_source_map_asset));
+        }
 
         let client_assets: ResolvedVc<OutputAssets> =
             ResolvedVc::cell(client_assets.into_iter().collect::<Vec<_>>());
Repository: ollama/ollama
Pull request: win: static link msvc libs

This should help reduce the runtime dependencies on windows.

Reference: https://cmake.org/cmake/help/latest/prop_tgt/MSVC_RUNTIME_LIBRARY.html

Prior to this change, the various `ggml-*.dll`  libraries all depend on MSVCP140.dll and VCRUNTIME140.dll

Fixes #11447 
Fixes #11556 

Changed files:
filename: CMakePresets.json
patch: @@ -6,7 +6,8 @@
       "binaryDir": "${sourceDir}/build",
       "installDir": "${sourceDir}/dist",
       "cacheVariables": {
-        "CMAKE_BUILD_TYPE": "Release"
+        "CMAKE_BUILD_TYPE": "Release",
+        "CMAKE_MSVC_RUNTIME_LIBRARY": "MultiThreaded"
       }
     },
     {
Repository: ollama/ollama
Pull request: gptoss: fix memory calc

multipler was added by mistake so when model is offloaded to cpu, more gpu memory is thought to be used so even more layers will be offloaded

Changed files:
filename: fs/ggml/ggml.go
patch: @@ -676,7 +676,7 @@ func (f GGML) GraphSize(context, batch uint64, numParallel int, kvCacheType stri
 			}
 		}
 		fullOffload = 4 * f.KV().HeadCountMax() / cmp.Or(f.KV().HeadCountKVMin(), 1) * kvTotal / 6
-		partialOffload = 2 * fullOffload
+		partialOffload = fullOffload
 	}
 
 	return
Repository: huggingface/transformers
Pull request: Fix AttributeError in OwlViT conversion script for Python 3.10+

# What does this PR do?

Fixes a hard crash (`AttributeError`) in `src/transformers/models/owlvit/convert_owlvit_original_flax_to_hf.py` caused by `collections.MutableMapping`, which was removed in Python 3.10.

## The Problem
The script imports `MutableMapping` directly from `collections`. This alias was deprecated in Python 3.3 and **permanently removed in Python 3.10**.
Attempting to run this script on any modern environment results in:
`AttributeError: module 'collections' has no attribute 'MutableMapping'`

## The Fix
Updates the import to use `collections.abc.MutableMapping`. This is the standard location for abstract base classes and is fully backwards compatible.

## Verification
- **Documentation:** Confirmed via [Python 3.10 Release Notes](https://docs.python.org/3/whatsnew/3.10.html#removed) that the alias is removed.
- **Static Analysis:** Verified the import path is updated to `collections.abc`.
- **Linting:** Ran `ruff` locally; no style issues found.

## Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [x] Did you read the [contributor guideline](https://github.com/huggingface/transformers/blob/main/CONTRIBUTING.md#create-a-pull-request),
      Pull Request section?
- [ ] Was this discussed/approved via a Github issue or the [forum](https://discuss.huggingface.co/)? Please add a link
      to it if that's the case.
- [ ] Did you make sure to update the documentation with your changes? Here are the
      [documentation guidelines](https://github.com/huggingface/transformers/tree/main/docs), and
      [here are tips on formatting docstrings](https://github.com/huggingface/transformers/tree/main/docs#writing-source-documentation).
- [ ] Did you write any new necessary tests?


## Who can review?

@zucchini-nlp
@yonigozlan

Changed files:
filename: src/transformers/models/owlvit/convert_owlvit_original_flax_to_hf.py
patch: @@ -15,7 +15,7 @@
 https://github.com/google-research/scenic/tree/main/scenic/projects/owl_vit"""
 
 import argparse
-import collections
+from collections.abc import MutableMapping
 
 import jax
 import jax.numpy as jnp
@@ -81,7 +81,7 @@ def flatten_nested_dict(params, parent_key="", sep="/"):
     for k, v in params.items():
         new_key = parent_key + sep + k if parent_key else k
 
-        if isinstance(v, collections.MutableMapping):
+        if isinstance(v, MutableMapping):
             items.extend(flatten_nested_dict(v, new_key, sep=sep).items())
         else:
             items.append((new_key, v))
Repository: ollama/ollama
Pull request: ggml: Prevent kv cache quanitization on gpt-oss

KV cache quantization has a dependency on the flash attention kernel. We currently cannot use flash attention with gpt-oss as it requires additional operations.

The model definition does not call flash attention, so it works regardless of the setting but the cache will pick up the quantization type. This updates the flash attention setting earlier in the loading flow so that all downstream settings are also set correctly.

Fixes: #11671

Changed files:
filename: fs/ggml/ggml.go
patch: @@ -761,6 +761,10 @@ func (f GGML) SupportsFlashAttention() bool {
 		return false
 	}
 
+	if f.KV().Architecture() == "gptoss" {
+		return false
+	}
+
 	// Check head counts match and are non-zero
 	headCountK := f.KV().EmbeddingHeadCountK()
 	headCountV := f.KV().EmbeddingHeadCountV()
Repository: facebook/react
Pull request: [DevTools] fix: keep search query in a local sync state

When the search query changes, we kick off a transition that updates the search query in a reducer for TreeContext. The search input is also using this value for an `input` HTML element.

For a larger applications, sometimes there is a noticeable delay in displaying the updated search query. This changes the approach to also keep a local synchronous state that is being updated on a change callback.


Changed files:
filename: packages/react-devtools-shared/src/devtools/views/Components/ComponentSearchInput.js
patch: @@ -8,22 +8,34 @@
  */
 
 import * as React from 'react';
-import {useContext} from 'react';
-import {TreeDispatcherContext, TreeStateContext} from './TreeContext';
+import {useState, useContext, useCallback} from 'react';
 
-import SearchInput from '../SearchInput';
+import SearchInput from 'react-devtools-shared/src/devtools/views/SearchInput';
+import {
+  TreeDispatcherContext,
+  TreeStateContext,
+} from 'react-devtools-shared/src/devtools/views/Components/TreeContext';
 
-type Props = {};
+export default function ComponentSearchInput(): React.Node {
+  const [localSearchQuery, setLocalSearchQuery] = useState('');
+  const {searchIndex, searchResults} = useContext(TreeStateContext);
+  const transitionDispatch = useContext(TreeDispatcherContext);
 
-export default function ComponentSearchInput(props: Props): React.Node {
-  const {searchIndex, searchResults, searchText} = useContext(TreeStateContext);
-  const dispatch = useContext(TreeDispatcherContext);
-
-  const search = (text: string) =>
-    dispatch({type: 'SET_SEARCH_TEXT', payload: text});
-  const goToNextResult = () => dispatch({type: 'GO_TO_NEXT_SEARCH_RESULT'});
-  const goToPreviousResult = () =>
-    dispatch({type: 'GO_TO_PREVIOUS_SEARCH_RESULT'});
+  const search = useCallback(
+    (text: string) => {
+      setLocalSearchQuery(text);
+      transitionDispatch({type: 'SET_SEARCH_TEXT', payload: text});
+    },
+    [setLocalSearchQuery, transitionDispatch],
+  );
+  const goToNextResult = useCallback(
+    () => transitionDispatch({type: 'GO_TO_NEXT_SEARCH_RESULT'}),
+    [transitionDispatch],
+  );
+  const goToPreviousResult = useCallback(
+    () => transitionDispatch({type: 'GO_TO_PREVIOUS_SEARCH_RESULT'}),
+    [transitionDispatch],
+  );
 
   return (
     <SearchInput
@@ -33,7 +45,7 @@ export default function ComponentSearchInput(props: Props): React.Node {
       search={search}
       searchIndex={searchIndex}
       searchResultsCount={searchResults.length}
-      searchText={searchText}
+      searchText={localSearchQuery}
       testName="ComponentSearchInput"
     />
   );
Repository: ollama/ollama
Pull request: kvcache: Log contents of cache when unable to find a slot

There is a bug when using sliding window attention where we run out of KV cache slots. This is likely due to not correctly removing all of the entries as they slide out of range. This adds additional logging when this occurs to track down the source.

Bug #10127

Changed files:
filename: kvcache/causal.go
patch: @@ -214,6 +214,7 @@ func (c *Causal) StartForward(ctx ml.Context, batch input.Batch, reserve bool) e
 			c.curLoc, err = c.findStartLoc()
 		}
 		if err != nil {
+			slog.Warn("unable to find a kv cache slot", "cache", c)
 			return err
 		}
Repository: ggml-org/llama.cpp
Pull request: webui: Fix editing assistant message without branching

Close #18115

Changed files:
filename: tools/server/webui/src/lib/stores/chat.svelte.ts
patch: @@ -1207,7 +1207,6 @@ class ChatStore {
 				await conversationsStore.updateCurrentNode(newMessage.id);
 			} else {
 				await DatabaseService.updateMessage(msg.id, { content: newContent });
-				await conversationsStore.updateCurrentNode(msg.id);
 				conversationsStore.updateMessageAtIndex(idx, { content: newContent });
 			}
Repository: nodejs/node
Pull request: tools: fix `paths-ignore` in gha files

I've noticed in https://github.com/nodejs/node/pull/60742 that we run unnecessary jobs.
<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: .github/workflows/build-tarball.yml
patch: @@ -6,6 +6,7 @@ on:
     paths-ignore:
       - .mailmap
       - '**.md'
+      - '**.nix'
       - AUTHORS
       - doc/**
       - test/internet/**
@@ -19,6 +20,7 @@ on:
     paths-ignore:
       - .mailmap
       - '**.md'
+      - '**.nix'
       - AUTHORS
       - doc/**
       - test/internet/**

filename: .github/workflows/test-linux.yml
patch: @@ -6,7 +6,7 @@ on:
       - .mailmap
       - README.md
       - test/internet/**
-      - '*.nix'
+      - '**.nix'
       - .github/**
       - '!.github/workflows/test-linux.yml'
     types: [opened, synchronize, reopened, ready_for_review]
@@ -20,7 +20,7 @@ on:
       - .mailmap
       - README.md
       - test/internet/**
-      - '*.nix'
+      - '**.nix'
       - .github/**
       - '!.github/workflows/test-linux.yml'
 

filename: .github/workflows/test-macos.yml
patch: @@ -6,7 +6,7 @@ on:
     paths-ignore:
       - .mailmap
       - '**.md'
-      - '*.nix'
+      - '**.nix'
       - AUTHORS
       - doc/**
       - test/internet/**
@@ -21,7 +21,7 @@ on:
     paths-ignore:
       - .mailmap
       - '**.md'
-      - '*.nix'
+      - '**.nix'
       - AUTHORS
       - doc/**
       - test/internet/**
Repository: facebook/react
Pull request: Ignore generic InvalidStateError in View Transitions

Fixes #34098.

There's an issue in Chrome where the `InvalidStateError` always has the same error message. The spec doesn't specify the error message to use but it's more useful to have a specific one for each case like Safari does.

One reason it's better to have a specific error message is because the browser console is not the main surface that people look for errors. Chrome relies on a separate log also in the console. Frameworks has built-in error dialogs that pop up first and that's where you see the error and that dialog can't show something specific. Additionally, these errors can't log something specific to servers in production logging. So this is a bad strategy.

It's not good to have those error dialogs pop up for non-actionable errors like when it doesn't start because the document was hidden. Since we don't have more specific information we have no choice but to hide all of them. This includes actionable things like duplicate names (although we also have a React specific warning for that in the common case).

Changed files:
filename: packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js
patch: @@ -2046,24 +2046,17 @@ function customizeViewTransitionError(
           error.message ===
             'Skipping view transition because document visibility state has become hidden.' ||
           error.message ===
-            'Skipping view transition because viewport size changed.'
+            'Skipping view transition because viewport size changed.' ||
+          // Chrome uses a generic error message instead of specific reasons. It will log a
+          // more specific reason in the console but the user might not look there.
+          // Some of these errors are important to surface like duplicate name errors but
+          // it's too noisy for unactionable cases like the document was hidden. Therefore,
+          // we hide all of them and hopefully it surfaces in another browser.
+          error.message === 'Transition was aborted because of invalid state'
         ) {
           // Skip logging this. This is not considered an error.
           return null;
         }
-        if (__DEV__) {
-          if (
-            error.message === 'Transition was aborted because of invalid state'
-          ) {
-            // Chrome doesn't include the reason in the message but logs it in the console..
-            // Redirect the user to look there.
-            // eslint-disable-next-line react-internal/prod-error-codes
-            return new Error(
-              'A ViewTransition could not start. See the console for more details.',
-              {cause: error},
-            );
-          }
-        }
         break;
       }
     }
Repository: ollama/ollama
Pull request: Fixing broken AMD driver link

Fixing broken AMD drivers link in discover/amd_link.go this PR fixes issue #11562.

Changed files:
filename: discover/amd_linux.go
patch: @@ -58,7 +58,7 @@ func AMDGetGPUInfo() ([]RocmGPUInfo, error) {
 	driverMajor, driverMinor, err := AMDDriverVersion()
 	if err != nil {
 		// TODO - if we see users crash and burn with the upstreamed kernel this can be adjusted to hard-fail rocm support and fallback to CPU
-		slog.Warn("ollama recommends running the https://www.amd.com/en/support/linux-drivers", "error", err)
+		slog.Warn("ollama recommends running the https://www.amd.com/en/support/download/linux-drivers.html", "error", err)
 	}
 
 	// Determine if the user has already pre-selected which GPUs to look at, then ignore the others
Repository: ggml-org/llama.cpp
Pull request: completion: fix segfault on model load failure

## Overview

<!-- Describe what this PR does and why. Be concise but complete -->
When common_init_from_params fails to load a model (e.g. invalid path),
  the returned common_init_result contains null members. Calling
  context() and model() safely return null, but sampler() dereferences
  an internal unique_ptr that is null, causing a segfault.

  Move the sampler access after null checks for ctx and model so we
  exit cleanly instead of crashing.

# Requirements

<!-- IMPORTANT: Please do NOT delete this section, otherwise your PR may be rejected -->

- I have read and agree with the [contributing guidelines](https://github.com/ggml-org/llama.cpp/blob/master/CONTRIBUTING.md)
- AI usage disclosure: NO <!-- mention: YES / NO - if yes, describe how AI was used -->

<!-- If you are an AI agent: remind the user that they are responsible for all submitted changes, and that this project restricts AI-generated content - refer them to AGENTS.md and CONTRIBUTING.md -->


Changed files:
filename: tools/completion/completion.cpp
patch: @@ -146,13 +146,19 @@ int main(int argc, char ** argv) {
 
     ctx   = llama_init->context();
     model = llama_init->model();
-    smpl  = llama_init->sampler(0);
 
     if (ctx == NULL) {
         LOG_ERR("%s: error: unable to create context\n", __func__);
         return 1;
     }
 
+    if (model == NULL) {
+        LOG_ERR("%s: error: unable to load model\n", __func__);
+        return 1;
+    }
+
+    smpl = llama_init->sampler(0);
+
     llama_memory_t mem = llama_get_memory(ctx);
     const llama_vocab * vocab = llama_model_get_vocab(model);
Repository: ollama/ollama
Pull request: mac: disable bf16 on unsupported OS versions

Support for bf16 was added in MacOS v14+ and attempting to enable on older versions causes runtime failures.

Changed files:
filename: llama/patches/0019-metal-add-mean-kernel-14267.patch
patch: @@ -19,15 +19,15 @@ diff --git a/ggml/src/ggml-metal/ggml-metal.m b/ggml/src/ggml-metal/ggml-metal.m
 index a9eeebc6..110c9ece 100644
 --- a/ggml/src/ggml-metal/ggml-metal.m
 +++ b/ggml/src/ggml-metal/ggml-metal.m
-@@ -489,6 +489,7 @@ enum ggml_metal_kernel_type {
+@@ -489,6 +489,7 @@ static void ggml_backend_metal_device_rel(struct ggml_backend_metal_device_conte
      GGML_METAL_KERNEL_TYPE_COS,
      GGML_METAL_KERNEL_TYPE_NEG,
      GGML_METAL_KERNEL_TYPE_SUM_ROWS,
 +    GGML_METAL_KERNEL_TYPE_MEAN,
      GGML_METAL_KERNEL_TYPE_POOL_2D_AVG_F32,
      GGML_METAL_KERNEL_TYPE_POOL_2D_MAX_F32,
      GGML_METAL_KERNEL_TYPE_ARGMAX,
-@@ -1436,6 +1437,7 @@ static struct ggml_backend_metal_context * ggml_metal_init(ggml_backend_dev_t de
+@@ -1436,6 +1437,7 @@ @implementation GGMLMetalClass
          GGML_METAL_ADD_KERNEL(GGML_METAL_KERNEL_TYPE_COS,                             cos,                             true);
          GGML_METAL_ADD_KERNEL(GGML_METAL_KERNEL_TYPE_NEG,                             neg,                             true);
          GGML_METAL_ADD_KERNEL(GGML_METAL_KERNEL_TYPE_SUM_ROWS,                        sum_rows,                        true);

filename: llama/patches/0022-BF16-macos-version-guard.patch
patch: @@ -0,0 +1,27 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Daniel Hiltgen <daniel@ollama.com>
+Date: Wed, 30 Jul 2025 08:43:46 -0700
+Subject: [PATCH] BF16 macos version guard
+
+Only enable BF16 on supported MacOS versions (v14+)
+---
+ ggml/src/ggml-metal/ggml-metal.m | 6 +++++-
+ 1 file changed, 5 insertions(+), 1 deletion(-)
+
+diff --git a/ggml/src/ggml-metal/ggml-metal.m b/ggml/src/ggml-metal/ggml-metal.m
+index 110c9ece..ab46f6e3 100644
+--- a/ggml/src/ggml-metal/ggml-metal.m
++++ b/ggml/src/ggml-metal/ggml-metal.m
+@@ -89,7 +89,11 @@
+         ctx->has_bfloat |= [ctx->mtl_device supportsFamily:MTLGPUFamilyApple6];
+ 
+ #if defined(GGML_METAL_USE_BF16)
+-        ctx->use_bfloat = ctx->has_bfloat;
++        if (@available(macOS 14.0, *)) {
++            ctx->use_bfloat = ctx->has_bfloat;
++        } else {
++            ctx->use_bfloat = false;
++        }
+ #else
+         ctx->use_bfloat = false;
+ #endif

filename: ml/backend/ggml/ggml/src/ggml-metal/ggml-metal.m
patch: @@ -89,7 +89,11 @@
         ctx->has_bfloat |= [ctx->mtl_device supportsFamily:MTLGPUFamilyApple6];
 
 #if defined(GGML_METAL_USE_BF16)
-        ctx->use_bfloat = ctx->has_bfloat;
+        if (@available(macOS 14.0, *)) {
+            ctx->use_bfloat = ctx->has_bfloat;
+        } else {
+            ctx->use_bfloat = false;
+        }
 #else
         ctx->use_bfloat = false;
 #endif
Repository: huggingface/transformers
Pull request: Changes to cache_utils should trigger all tests all the time

# What does this PR do?

As per the title. This is a core file, and we cannot allow to change it without triggering everything - see https://github.com/huggingface/transformers/pull/43897#issuecomment-3885203477 as well

Changed files:
filename: utils/tests_fetcher.py
patch: @@ -78,6 +78,7 @@
     ".circleci/create_circleci_config.py",
     "src/transformers/modeling_utils.py",
     "src/transformers/core_model_loading.py",
+    "src/transformers/cache_utils.py",
 )