helloadhavan/github_issues huggingface dataset:
Repository: ggml-org/llama.cpp
Pull request: minor: root cause in error message if loading backend library fails

This PR adds additional information to an error message when loading backend library via `ld_load_library()` fails. This helps spotting why backend library did not load (missing library, missing dependency or unresolved symbol etc.).


Changed files:
filename: ggml/src/ggml-backend-reg.cpp
patch: @@ -135,6 +135,10 @@ static void * dl_get_sym(dl_handle * handle, const char * name) {
     return p;
 }
 
+static const char * dl_error() {
+    return "";
+}
+
 #else
 
 using dl_handle = void;
@@ -155,6 +159,11 @@ static void * dl_get_sym(dl_handle * handle, const char * name) {
     return dlsym(handle, name);
 }
 
+static const char * dl_error() {
+    const char *rslt = dlerror();
+    return rslt != nullptr ? rslt : "";
+}
+
 #endif
 
 using dl_handle_ptr = std::unique_ptr<dl_handle, dl_handle_deleter>;
@@ -240,7 +249,7 @@ struct ggml_backend_registry {
         dl_handle_ptr handle { dl_load_library(path) };
         if (!handle) {
             if (!silent) {
-                GGML_LOG_ERROR("%s: failed to load %s\n", __func__, path_str(path).c_str());
+                GGML_LOG_ERROR("%s: failed to load %s: %s\n", __func__, path_str(path).c_str(), dl_error());
             }
             return nullptr;
         }
@@ -530,7 +539,7 @@ static ggml_backend_reg_t ggml_backend_load_best(const char * name, bool silent,
                 if (filename.native().find(file_prefix) == 0 && ext == file_extension) {
                     dl_handle_ptr handle { dl_load_library(entry) };
                     if (!handle && !silent) {
-                        GGML_LOG_ERROR("%s: failed to load %s\n", __func__, path_str(entry.path()).c_str());
+                        GGML_LOG_ERROR("%s: failed to load %s: %s\n", __func__, path_str(entry.path()).c_str(), dl_error());
                     }
                     if (handle) {
                         auto score_fn = (ggml_backend_score_t) dl_get_sym(handle.get(), "ggml_backend_score");
Repository: denoland/deno
Pull request: fix: support virtio vsock only on systems on which it is available (remainder)

See https://github.com/denoland/deno/issues/29268 for the related discussion. @devsnek @marvinhagemeister

These are two patches I missed with the initial PR (https://github.com/denoland/deno/pull/29354)

Changed files:
filename: ext/http/http_next.rs
patch: @@ -1054,7 +1054,7 @@ where
     NetworkStream::Unix(conn) => {
       serve_http(conn, connection_properties, lifetime, tx, options)
     }
-    #[cfg(unix)]
+    #[cfg(any(target_os = "linux", target_os = "macos"))]
     NetworkStream::Vsock(conn) => {
       serve_http(conn, connection_properties, lifetime, tx, options)
     }

filename: ext/http/request_properties.rs
patch: @@ -165,14 +165,14 @@ impl HttpPropertyExtractor for DefaultHttpPropertyExtractor {
       NetworkStreamAddress::Ip(ip) => Some(ip.port() as _),
       #[cfg(unix)]
       NetworkStreamAddress::Unix(_) => None,
-      #[cfg(unix)]
+      #[cfg(any(target_os = "linux", target_os = "macos"))]
       NetworkStreamAddress::Vsock(vsock) => Some(vsock.port()),
     };
     let peer_address = match peer_address {
       NetworkStreamAddress::Ip(addr) => Rc::from(addr.ip().to_string()),
       #[cfg(unix)]
       NetworkStreamAddress::Unix(_) => Rc::from("unix"),
-      #[cfg(unix)]
+      #[cfg(any(target_os = "linux", target_os = "macos"))]
       NetworkStreamAddress::Vsock(addr) => {
         Rc::from(format!("vsock:{}", addr.cid()))
       }
@@ -214,7 +214,7 @@ fn listener_properties(
     NetworkStreamAddress::Ip(ip) => Some(ip.port() as _),
     #[cfg(unix)]
     NetworkStreamAddress::Unix(_) => None,
-    #[cfg(unix)]
+    #[cfg(any(target_os = "linux", target_os = "macos"))]
     NetworkStreamAddress::Vsock(vsock) => Some(vsock.port()),
   };
   Ok(HttpListenProperties {
@@ -260,7 +260,7 @@ fn req_host_from_addr(
       percent_encoding::NON_ALPHANUMERIC,
     )
     .to_string(),
-    #[cfg(unix)]
+    #[cfg(any(target_os = "linux", target_os = "macos"))]
     NetworkStreamAddress::Vsock(vsock) => {
       format!("{}:{}", vsock.cid(), vsock.port())
     }
@@ -273,7 +273,7 @@ fn req_scheme_from_stream_type(stream_type: NetworkStreamType) -> &'static str {
     NetworkStreamType::Tls => "https://",
     #[cfg(unix)]
     NetworkStreamType::Unix => "http+unix://",
-    #[cfg(unix)]
+    #[cfg(any(target_os = "linux", target_os = "macos"))]
     NetworkStreamType::Vsock => "http+vsock://",
   }
 }
@@ -299,7 +299,7 @@ fn req_host<'a>(
       }
       #[cfg(unix)]
       NetworkStreamType::Unix => {}
-      #[cfg(unix)]
+      #[cfg(any(target_os = "linux", target_os = "macos"))]
       NetworkStreamType::Vsock => {}
     }
     return Some(Cow::Borrowed(auth.as_str()));
Repository: denoland/deno
Pull request: fix(bundle): don't panic on broken pipe

`deno bundle main.js | less` and then `Ctrl + C` caused
a panic because of `Broken pipe` error.

Changed files:
filename: cli/tools/bundle/mod.rs
patch: @@ -233,10 +233,7 @@ pub async fn bundle(
 
   if let Some(stdout) = response.write_to_stdout {
     let stdout = replace_require_shim(&String::from_utf8_lossy(&stdout));
-    #[allow(clippy::print_stdout)]
-    {
-      println!("{}", stdout);
-    }
+    crate::display::write_to_stdout_ignore_sigpipe(stdout.as_bytes())?;
   } else if response.errors.is_empty() {
     if bundle_flags.output_dir.is_none()
       && std::env::var("NO_DENO_BUNDLE_HACK").is_err()
Repository: denoland/deno
Pull request: test: disable _fs_watch_test.ts test case

This test was added in https://github.com/denoland/deno/pull/29659 and
an attempt to make it more stable was done in https://github.com/denoland/deno/pull/29699.

Unfortunately it's still very flaky. Ignoring for now.

CC @kt3k 

Changed files:
filename: tests/unit_node/_fs/_fs_watch_test.ts
patch: @@ -30,6 +30,8 @@ Deno.test({
 
 Deno.test({
   name: "watching a file with options",
+  // TODO(bartlomieju): this test is flaky on CI
+  ignore: true,
   async fn() {
     const file = Deno.makeTempFileSync();
     const spyFn = spy();
Repository: facebook/react
Pull request: [Flight] Serialize deduped elements by direct reference even if they suspend

In #28123 I switched these to be lazy references. However that creates a lazy wrapper even if they're synchronously available. We try to as much as possible preserve the original data structure in these cases.

E.g. here in the dev outlining I only use a lazy wrapper if it didn't complete synchronously: https://github.com/facebook/react/pull/28272/files#diff-d4c9c509922b3671d3ecce4e051df66dd5c3d38ff913c7a7fe94abc3ba2ed72eR638

Unfortunately we don't have a data structure that tracks the status of each emitted row. We could store the task in the map but then they couldn't be GC:ed as they complete. We could maybe store the status of each element but seems so heavy.

For now I just went back to direct reference which might be an issue since it can suspend something higher up when deduped.

Changed files:
filename: packages/react-server/src/ReactFlightServer.js
patch: @@ -1192,12 +1192,16 @@ function renderModelDestructive(
             // but that is able to reuse the same task if we're already in one but then that
             // will be a lazy future value rather than guaranteed to exist but maybe that's good.
             const newId = outlineModel(request, (value: any));
-            return serializeLazyID(newId);
+            return serializeByValueID(newId);
           } else {
             // We've already emitted this as an outlined object, so we can refer to that by its
-            // existing ID. We use a lazy reference since, unlike plain objects, elements might
-            // suspend so it might not have emitted yet even if we have the ID for it.
-            return serializeLazyID(existingId);
+            // existing ID. TODO: We should use a lazy reference since, unlike plain objects,
+            // elements might suspend so it might not have emitted yet even if we have the ID for
+            // it. However, this creates an extra wrapper when it's not needed. We should really
+            // detect whether this already was emitted and synchronously available. In that
+            // case we can refer to it synchronously and only make it lazy otherwise.
+            // We currently don't have a data structure that lets us see that though.
+            return serializeByValueID(existingId);
           }
         } else {
           // This is the first time we've seen this object. We may never see it again
Repository: electron/electron
Pull request: feat: add ffmpeg.dll to delay load configuration

Backport of #46151

See that PR for details.


Notes: Added `ffmpeg.dll` to delay load configuration.


Changed files:
filename: BUILD.gn
patch: @@ -1246,7 +1246,7 @@ if (is_mac) {
         "//components/crash/core/app:run_as_crashpad_handler",
       ]
 
-      ldflags = []
+      ldflags = [ "/DELAYLOAD:ffmpeg.dll" ]
 
       libs = [
         "comctl32.lib",
Repository: huggingface/transformers
Pull request: Fix link to models in README

# What does this PR do?

Fix links and models in README

Changed files:
filename: README.md
patch: @@ -280,8 +280,8 @@ Expand each modality below to see a few example models for various use cases.
 - Automatic mask generation with [SAM](https://huggingface.co/facebook/sam-vit-base)
 - Depth estimation with [DepthPro](https://huggingface.co/apple/DepthPro-hf)
 - Image classification with [DINO v2](https://huggingface.co/facebook/dinov2-base)
-- Keypoint detection with [SuperGlue](https://huggingface.co/magic-leap-community/superglue_outdoor)
-- Keypoint matching with [SuperGlue](https://huggingface.co/magic-leap-community/superglue)
+- Keypoint detection with [SuperPoint](https://huggingface.co/magic-leap-community/superpoint)
+- Keypoint matching with [SuperGlue](https://huggingface.co/magic-leap-community/superglue_outdoor)
 - Object detection with [RT-DETRv2](https://huggingface.co/PekingU/rtdetr_v2_r50vd)
 - Pose Estimation with [VitPose](https://huggingface.co/usyd-community/vitpose-base-simple)
 - Universal segmentation with [OneFormer](https://huggingface.co/shi-labs/oneformer_ade20k_swin_large)
Repository: vercel/next.js
Pull request: [test] Allow running `lint-eslint` on a specific directory

`pnpm lint-eslint` no longer has a default pattern. This allows running it on a sub-directory in the monorepo e.g. `pnpm lint-eslint packages/next/src/next-devtools`

`lint-no-typescript` (and therefore CI) still defaults to `pnpm lint-eslint .`

Changed files:
filename: package.json
patch: @@ -41,14 +41,14 @@
     "git-clean": "git clean -d -x -e node_modules -e packages -f",
     "typescript": "tsc --noEmit",
     "lint-typescript": "turbo run typescript",
-    "lint-eslint": "cross-env ESLINT_USE_FLAT_CONFIG=false eslint . --ext js,jsx,mjs,ts,tsx,mts,mdx --config .eslintrc.cli.json --no-eslintrc",
+    "lint-eslint": "cross-env ESLINT_USE_FLAT_CONFIG=false eslint --ext js,jsx,mjs,ts,tsx,mts,mdx --config .eslintrc.cli.json --no-eslintrc",
     "lint-ast-grep": "ast-grep scan",
-    "lint-no-typescript": "run-p prettier-check lint-eslint lint-language",
+    "lint-no-typescript": "run-p prettier-check \"lint-eslint .\" lint-language",
     "types-and-precompiled": "run-p lint-typescript check-compiler-fixtures types:test-lib && pnpm check-precompiled",
     "check-compiler-fixtures": "find crates/next-custom-transforms/tests/fixture -type f -name 'tsconfig.json' -print0 | xargs --null -n1 -I'{}' pnpm tsc --noEmit --project '{}'",
     "validate-externals-doc": "node ./scripts/validate-externals-doc.js",
-    "lint": "run-p test-types lint-typescript prettier-check lint-eslint lint-ast-grep lint-language",
-    "lint-fix": "pnpm prettier-fix && pnpm lint-eslint --fix",
+    "lint": "run-p test-types lint-typescript prettier-check \"lint-eslint .\" lint-ast-grep lint-language",
+    "lint-fix": "pnpm prettier-fix && pnpm lint-eslint --fix .",
     "lint-language": "alex .",
     "prettier-check": "prettier --check .",
     "check-examples": "./scripts/check-examples.sh",
Repository: huggingface/transformers
Pull request: Better return type hint for `AutoModelForCausalLM` and `AutoModelForImageTextToText`

# What does this PR do?

Defines a better return type hint for `AutoModelForCausalLM` and `AutoModelForImageTextToText`, from_pretrained method returns `_BaseModelWithGenerate` so anyone can see `generate` signature (see images)

#### On main
<img width="757" height="204" alt="Screenshot 2025-08-04 at 12 12 35" src="https://github.com/user-attachments/assets/1cfb7abb-9a82-4521-843f-0a7bec15d8ef" />

#### On branch
<img width="738" height="456" alt="Screenshot 2025-08-04 at 12 12 10" src="https://github.com/user-attachments/assets/8e8f9138-7dd9-4333-995d-88baceb671a1" />




Changed files:
filename: src/transformers/models/auto/modeling_auto.py
patch: @@ -14,8 +14,10 @@
 # limitations under the License.
 """Auto Model class."""
 
+import os
 import warnings
 from collections import OrderedDict
+from typing import TYPE_CHECKING, Union
 
 from ...utils import logging
 from .auto_factory import (
@@ -27,6 +29,15 @@
 from .configuration_auto import CONFIG_MAPPING_NAMES
 
 
+if TYPE_CHECKING:
+    from ...generation import GenerationMixin
+    from ...modeling_utils import PreTrainedModel
+
+    # class for better type annotations
+    class _BaseModelWithGenerate(PreTrainedModel, GenerationMixin):
+        pass
+
+
 logger = logging.get_logger(__name__)
 
 MODEL_MAPPING_NAMES = OrderedDict(
@@ -1863,6 +1874,16 @@ class _AutoModelWithLMHead(_BaseAutoModelClass):
 class AutoModelForCausalLM(_BaseAutoModelClass):
     _model_mapping = MODEL_FOR_CAUSAL_LM_MAPPING
 
+    # override to give better return typehint
+    @classmethod
+    def from_pretrained(
+        cls: type["AutoModelForCausalLM"],
+        pretrained_model_name_or_path: Union[str, os.PathLike[str]],
+        *model_args,
+        **kwargs,
+    ) -> "_BaseModelWithGenerate":
+        return super().from_pretrained(pretrained_model_name_or_path, *model_args, **kwargs)
+
 
 AutoModelForCausalLM = auto_class_update(AutoModelForCausalLM, head_doc="causal language modeling")
 
@@ -2057,6 +2078,16 @@ class _AutoModelForVision2Seq(_BaseAutoModelClass):
 class AutoModelForImageTextToText(_BaseAutoModelClass):
     _model_mapping = MODEL_FOR_IMAGE_TEXT_TO_TEXT_MAPPING
 
+    # override to give better return typehint
+    @classmethod
+    def from_pretrained(
+        cls: type["AutoModelForImageTextToText"],
+        pretrained_model_name_or_path: Union[str, os.PathLike[str]],
+        *model_args,
+        **kwargs,
+    ) -> "_BaseModelWithGenerate":
+        return super().from_pretrained(pretrained_model_name_or_path, *model_args, **kwargs)
+
 
 AutoModelForImageTextToText = auto_class_update(AutoModelForImageTextToText, head_doc="image-text-to-text modeling")
Repository: ggml-org/llama.cpp
Pull request: Preserve zero values in WebUI chat settings inputs and textareas

Close #16267 

Fixes the issue by switching to nullish coalescing for field values and default placeholders.


Changed files:
filename: tools/server/webui/src/lib/components/app/chat/ChatSettings/ChatSettingsFields.svelte
patch: @@ -30,9 +30,9 @@
 
 			<Input
 				id={field.key}
-				value={String(localConfig[field.key] || '')}
+				value={String(localConfig[field.key] ?? '')}
 				onchange={(e) => onConfigChange(field.key, e.currentTarget.value)}
-				placeholder={`Default: ${SETTING_CONFIG_DEFAULT[field.key] || 'none'}`}
+				placeholder={`Default: ${SETTING_CONFIG_DEFAULT[field.key] ?? 'none'}`}
 				class={isMobile ? 'w-full' : 'max-w-md'}
 			/>
 			{#if field.help || SETTING_CONFIG_INFO[field.key]}
@@ -47,9 +47,9 @@
 
 			<Textarea
 				id={field.key}
-				value={String(localConfig[field.key] || '')}
+				value={String(localConfig[field.key] ?? '')}
 				onchange={(e) => onConfigChange(field.key, e.currentTarget.value)}
-				placeholder={`Default: ${SETTING_CONFIG_DEFAULT[field.key] || 'none'}`}
+				placeholder={`Default: ${SETTING_CONFIG_DEFAULT[field.key] ?? 'none'}`}
 				class={isMobile ? 'min-h-[100px] w-full' : 'min-h-[100px] max-w-2xl'}
 			/>
 			{#if field.help || SETTING_CONFIG_INFO[field.key]}
Repository: denoland/deno
Pull request: test(ext/http): reduce flakiness of httpServerVsockWebSocketUpgrade

This PR tries to reduce the flakiness of the case `httpServerVsockWebSocketUpgrade` by adding await for close event of server-side websocket connection (This event seems usually awaited in other test cases).

Changed files:
filename: tests/unit/serve_test.ts
patch: @@ -4437,6 +4437,7 @@ Deno.test(
   async function httpServerVsockWebSocketUpgrade() {
     const ac = new AbortController();
     const { promise, resolve } = Promise.withResolvers<Deno.VsockAddr>();
+    const serverWebSocketClosed = Promise.withResolvers<void>();
 
     await using server = Deno.serve(
       {
@@ -4458,6 +4459,10 @@ Deno.test(
           }
         };
 
+        socket.onclose = () => {
+          serverWebSocketClosed.resolve();
+        };
+
         return response;
       },
     );
@@ -4487,6 +4492,7 @@ Deno.test(
     assert(responseText.includes("101 Switching Protocols"));
 
     await conn.close();
+    await serverWebSocketClosed.promise;
     ac.abort();
     await server.finished;
   },
Repository: huggingface/transformers
Pull request: Set `torch.backends.cudnn.allow_tf32 = False` for CI

# What does this PR do?

See https://github.com/pytorch/pytorch/issues/157274#issuecomment-3090791615

(if we don't do this, we have to increase the thresholds in many tests when torch 2.8 is released) 

Changed files:
filename: conftest.py
patch: @@ -23,7 +23,7 @@
 import _pytest
 import pytest
 
-from transformers.testing_utils import HfDoctestModule, HfDocTestParser
+from transformers.testing_utils import HfDoctestModule, HfDocTestParser, is_torch_available
 
 
 NOT_DEVICE_TESTS = {
@@ -127,3 +127,9 @@ def check_output(self, want, got, optionflags):
 doctest.OutputChecker = CustomOutputChecker
 _pytest.doctest.DoctestModule = HfDoctestModule
 doctest.DocTestParser = HfDocTestParser
+
+if is_torch_available():
+    import torch
+    # The flag below controls whether to allow TF32 on cuDNN. This flag defaults to True.
+    # We set it to `False` for CI. See https://github.com/pytorch/pytorch/issues/157274#issuecomment-3090791615
+    torch.backends.cudnn.allow_tf32 = False
Repository: nodejs/node
Pull request: doc: correct formatting of example definitions for `--test-shard`

The original formatting intention was pretty clear in the raw markdown, but it doesn't render that way.

Changed files:
filename: doc/api/cli.md
patch: @@ -2578,8 +2578,9 @@ added:
 
 Test suite shard to execute in a format of `<index>/<total>`, where
 
-`index` is a positive integer, index of divided parts
-`total` is a positive integer, total of divided part
+* `index` is a positive integer, index of divided parts.
+* `total` is a positive integer, total of divided part.
+
 This command will divide all tests files into `total` equal parts,
 and will run only those that happen to be in an `index` part.
Repository: denoland/deno
Pull request: test(ext/node): reduce flakiness of _fs_watch_test.ts

A test case in `tests/unit_node/_fs/_fs_watch_test.ts` seems flaky after landing #29659 

This PR tries to fix it

Changed files:
filename: tests/unit_node/_fs/_fs_watch_test.ts
patch: @@ -44,6 +44,8 @@ Deno.test({
     await wait(100);
     assertEquals(spyFn.calls.length, 1);
     unwatchFile(file);
+    await wait(100);
+    assertEquals(spyFn.calls.length, 1);
   },
 });
Repository: electron/electron
Pull request: fix: add missing `cpp_heap` to Node.js worker `CreateParams`

Backport of #46104

See that PR for details.


Notes: none


Changed files:
filename: patches/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch
patch: @@ -165,10 +165,20 @@ index 4119ac1b002681d39711eac810ca2fcc2702ffc7..790347056cde949ffe6cf8498a7eca0c
  
  ExitCode NodeMainInstance::Run() {
 diff --git a/src/node_worker.cc b/src/node_worker.cc
-index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..a610ee24ff18bddc3849aec3a43c2037b9ab5d53 100644
+index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..9d35cbf3dff538f38e8d5b8660d40c1fbaa56474 100644
 --- a/src/node_worker.cc
 +++ b/src/node_worker.cc
-@@ -230,13 +230,8 @@ class WorkerThreadData {
+@@ -162,6 +162,9 @@ class WorkerThreadData {
+     SetIsolateCreateParamsForNode(&params);
+     w->UpdateResourceConstraints(&params.constraints);
+     params.array_buffer_allocator_shared = allocator;
++    params.cpp_heap =
++        v8::CppHeap::Create(w->platform_, v8::CppHeapCreateParams{{}})
++            .release();
+     Isolate* isolate =
+         NewIsolate(&params, &loop_, w->platform_, w->snapshot_data());
+     if (isolate == nullptr) {
+@@ -230,13 +233,8 @@ class WorkerThreadData {
          *static_cast<bool*>(data) = true;
        }, &platform_finished);
Repository: electron/electron
Pull request: fix: add missing `cpp_heap` to Node.js worker `CreateParams`

#### Description of Change

Ref https://github.com/electron/electron/pull/45922
Ref https://github.com/v8/node/pull/209

Add missing `cpp_heap` to Node.js worker `CreateParams` in `src/node_worker.cc`.

#### 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/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch
patch: @@ -165,10 +165,20 @@ index 4119ac1b002681d39711eac810ca2fcc2702ffc7..790347056cde949ffe6cf8498a7eca0c
  
  ExitCode NodeMainInstance::Run() {
 diff --git a/src/node_worker.cc b/src/node_worker.cc
-index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..a610ee24ff18bddc3849aec3a43c2037b9ab5d53 100644
+index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..9d35cbf3dff538f38e8d5b8660d40c1fbaa56474 100644
 --- a/src/node_worker.cc
 +++ b/src/node_worker.cc
-@@ -230,13 +230,8 @@ class WorkerThreadData {
+@@ -162,6 +162,9 @@ class WorkerThreadData {
+     SetIsolateCreateParamsForNode(&params);
+     w->UpdateResourceConstraints(&params.constraints);
+     params.array_buffer_allocator_shared = allocator;
++    params.cpp_heap =
++        v8::CppHeap::Create(w->platform_, v8::CppHeapCreateParams{{}})
++            .release();
+     Isolate* isolate =
+         NewIsolate(&params, &loop_, w->platform_, w->snapshot_data());
+     if (isolate == nullptr) {
+@@ -230,13 +233,8 @@ class WorkerThreadData {
          *static_cast<bool*>(data) = true;
        }, &platform_finished);
Repository: huggingface/transformers
Pull request: Fix quant docker for fp-quant 

# What does this PR do?

This PR fixes an issue with the quantization [docker](https://github.com/huggingface/transformers/actions/runs/16484958005/job/46652740114). fp-quant requires to have [py3.11](https://github.com/IST-DASLab/FP-Quant/blob/b7f7462b54c8d572a63bb7a8668da22a4e93079a/inference_lib/setup.py#L19) but our ci uses 3.9. Hence we don't install for now. Thanks @gante for spotting this. 


Changed files:
filename: docker/transformers-quantization-latest-gpu/Dockerfile
patch: @@ -79,7 +79,8 @@ RUN git clone https://github.com/NetEase-FuXi/EETQ.git && cd EETQ/ && git submod
 # RUN python3 -m pip install --no-cache-dir git+https://github.com/Dao-AILab/fast-hadamard-transform.git
 
 # Add fp-quant for quantization testing
-RUN python3 -m pip install --no-cache-dir "fp-quant>=0.1.6"
+# Requires py3.11 but our CI runs on 3.9
+# RUN python3 -m pip install --no-cache-dir "fp-quant>=0.1.6"
 
 # Add compressed-tensors for quantization testing
 RUN python3 -m pip install --no-cache-dir compressed-tensors