helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: [tests] tag serve tests as slow 

# What does this PR do?

These tests are not slow and don't require GPUs, but they're being flaky on the CI setup. This PR tags them as slow to unblock CI.

Changed files:
filename: tests/commands/test_serving.py
patch: @@ -197,6 +197,7 @@ def test_generation_config_in_request(self):
     # TODO: speed-based test to confirm that KV cache is working across requests
 
 
+@slow  # TODO (joao): this shouldn't be needed
 class ServeCompletionsGenerateTest(ServeCompletionsMixin, unittest.TestCase):
     """Tests the `generate` version of the Completions API."""
 
@@ -286,6 +287,7 @@ def test_tool_call(self):
         self.assertTrue(all(reason is None for reason in finish_reasons[:-1]))
 
 
+@slow  # TODO (joao): this shouldn't be needed
 class ServeCompletionsContinuousBatchingTest(ServeCompletionsMixin, unittest.TestCase):
     """Tests the `continuous_batching` version of the Completions API."""
Repository: denoland/deno
Pull request: fix(compile): revert multi-icon support in compile

Current understanding is that this produced an invalid resource table on certain versions of Windows which led to `deno compile`'ed binaries silently failing.

Fixes https://github.com/denoland/deno/issues/28982
Reverts https://github.com/denoland/deno/pull/28771


Changed files:
filename: Cargo.lock
patch: @@ -5205,12 +5205,11 @@ dependencies = [
 
 [[package]]
 name = "libsui"
-version = "0.7.0"
+version = "0.5.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7550a110415b4780bf6064d280f2dc5b761fb80d9af92258a72e5dc82ca6f05a"
+checksum = "89795977654ad6250d6c0915411b622bac22f9efb4f852af94b2e00964cab832"
 dependencies = [
  "editpe",
- "image",
  "libc",
  "sha2",
  "windows-sys 0.48.0",

filename: Cargo.toml
patch: @@ -292,7 +292,7 @@ dprint-plugin-typescript = "=0.94.0"
 env_logger = "=0.11.6"
 eszip = "=0.85.0"
 fancy-regex = "=0.14.0"
-libsui = "0.7.0"
+libsui = "0.5.0"
 malva = "=0.11.2"
 markup_fmt = "=0.19.1"
 open = "5.0.1"
Repository: ggml-org/llama.cpp
Pull request: Change to info instead of debug, to explain reason for stopping.

Closes #15553 


Changed files:
filename: tools/main/main.cpp
patch: @@ -587,12 +587,12 @@ int main(int argc, char ** argv) {
 
                 if (n_past + (int) embd.size() >= n_ctx) {
                     if (!params.ctx_shift){
-                        LOG_DBG("\n\n%s: context full and context shift is disabled => stopping\n", __func__);
+                        LOG_WRN("\n\n%s: context full and context shift is disabled => stopping\n", __func__);
                         break;
                     }
 
                     if (params.n_predict == -2) {
-                        LOG_DBG("\n\n%s: context full and n_predict == -%d => stopping\n", __func__, params.n_predict);
+                        LOG_WRN("\n\n%s: context full and n_predict == %d => stopping\n", __func__, params.n_predict);
                         break;
                     }
Repository: nodejs/node
Pull request: doc: fix typo of file `zlib.md`

<!--
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: doc/api/zlib.md
patch: @@ -12,7 +12,7 @@ Gzip, Deflate/Inflate, Brotli, and Zstd.
 To access it:
 
 ```mjs
-import os from 'node:zlib';
+import zlib from 'node:zlib';
 ```
 
 ```cjs
Repository: vercel/next.js
Pull request:  Turbopack: sourceMappingURL for Node.js runtime chunk

The sourcemap chunk was generated, but the comment referencing it was missing.
Copy the logic over from the browser runtime chunk

Closes PACK-5011

Changed files:
filename: turbopack/crates/turbopack-nodejs/src/ecmascript/node/entry/runtime.rs
patch: @@ -4,7 +4,7 @@ use anyhow::{Result, bail};
 use indoc::writedoc;
 use turbo_rcstr::{RcStr, rcstr};
 use turbo_tasks::{ResolvedVc, ValueToString, Vc};
-use turbo_tasks_fs::{File, FileSystem, FileSystemPath};
+use turbo_tasks_fs::{File, FileSystem, FileSystemPath, rope::RopeBuilder};
 use turbopack_core::{
     asset::{Asset, AssetContent},
     chunk::ChunkingContext,
@@ -157,9 +157,22 @@ impl Asset for EcmascriptBuildNodeRuntimeChunk {
     #[turbo_tasks::function]
     async fn content(self: Vc<Self>) -> Result<Vc<AssetContent>> {
         let code = self.code().await?;
-        Ok(AssetContent::file(
-            File::from(code.source_code().clone()).into(),
-        ))
+
+        let rope = if code.has_source_map() {
+            let mut rope_builder = RopeBuilder::default();
+            rope_builder.concat(code.source_code());
+            let source_map_path = self.source_map().path().await?;
+            write!(
+                rope_builder,
+                "\n\n//# sourceMappingURL={}",
+                urlencoding::encode(source_map_path.file_name())
+            )?;
+            rope_builder.build()
+        } else {
+            code.source_code().clone()
+        };
+
+        Ok(AssetContent::file(File::from(rope).into()))
     }
 }
 

filename: turbopack/crates/turbopack-tests/tests/snapshot/runtime/default_build_runtime/output/[turbopack]_runtime.js
patch: @@ -713,4 +713,7 @@ const regexJsUrl = /\.js(?:\?[^#]*)?(?:#.*)?$/;
 module.exports = {
     getOrInstantiateRuntimeModule,
     loadChunk
-};
\ No newline at end of file
+};
+
+
+//# sourceMappingURL=%5Bturbopack%5D_runtime.js.map
\ No newline at end of file
Repository: ggml-org/llama.cpp
Pull request: scripts: add sqlite3 check for compare-commits.sh

Check for sqlite3 before running the bench

Changed files:
filename: scripts/compare-commits.sh
patch: @@ -25,6 +25,12 @@ fi
 # verify at the start that the compare script has all the necessary dependencies installed
 ./scripts/compare-llama-bench.py --check
 
+if ! command -v sqlite3 >/dev/null 2>&1; then
+    echo "Error: sqlite3 is not installed or not in PATH"
+    echo "Please install sqlite3 to use this script"
+    exit 1
+fi
+
 if [ "$tool" = "llama-bench" ]; then
     db_file="llama-bench.sqlite"
     target="llama-bench"
Repository: denoland/deno
Pull request: fix(ext/node): honor flags with respect to reuseAddress when binding to a UDP socket

Issue #28155 

Honor flags with respect to reuseAddress when binding to  a UDP socket in node compatibility polyfill.



Changed files:
filename: ext/node/polyfills/internal_binding/udp_wrap.ts
patch: @@ -42,6 +42,7 @@ import type { ErrnoException } from "ext:deno_node/internal/errors.ts";
 import { isIP } from "ext:deno_node/internal/net.ts";
 import * as net from "ext:deno_net/01_net.js";
 import { isLinux, isWindows } from "ext:deno_node/_util/os.ts";
+import { os } from "ext:deno_node/internal_binding/constants.ts";
 
 const DenoListenDatagram = net.createListenDatagram(
   op_node_unstable_net_listen_udp,
@@ -324,12 +325,12 @@ export class UDP extends HandleWrap {
     this.#unrefed = true;
   }
 
-  #doBind(ip: string, port: number, _flags: number, family: number): number {
-    // TODO(cmorten): use flags to inform socket reuse etc.
+  #doBind(ip: string, port: number, flags: number, family: number): number {
     const listenOptions = {
       port,
       hostname: ip,
       transport: "udp" as const,
+      reuseAddress: (flags & os.UV_UDP_REUSEADDR) !== 0,
     };
 
     let listener;

filename: tests/unit_node/dgram_test.ts
patch: @@ -2,7 +2,7 @@
 
 import { assertEquals } from "@std/assert";
 import { execCode } from "../unit/test_util.ts";
-import { createSocket } from "node:dgram";
+import { createSocket, type Socket } from "node:dgram";
 
 const listenPort = 4503;
 const listenPort2 = 4504;
@@ -57,3 +57,28 @@ Deno.test("[node/dgram] udp unref", {
     `);
   assertEquals(statusCode, 0);
 });
+
+Deno.test("[node/dgram] createSocket, reuseAddr option", async () => {
+  const { promise, resolve } = Promise.withResolvers<string>();
+  const socket0 = createSocket({ type: "udp4", reuseAddr: true });
+  let socket1: Socket | undefined;
+  socket0.bind(0, "0.0.0.0", () => {
+    const port = socket0.address().port;
+    socket1 = createSocket({ type: "udp4", reuseAddr: true });
+    socket1.bind(port, "0.0.0.0", () => {
+      const socket = createSocket({ type: "udp4" });
+      socket.send("hello", port, "localhost", () => {
+        socket.close();
+      });
+    });
+    socket1.on("message", (msg) => {
+      resolve(msg.toString());
+    });
+  });
+  socket0.on("message", (msg) => {
+    resolve(msg.toString());
+  });
+  assertEquals(await promise, "hello");
+  socket0.close();
+  socket1?.close();
+});
Repository: huggingface/transformers
Pull request: Handle DAC conversion when using weight_norm with newer PyTorch versions

# What does this PR do?
`nn.utils.weight_norm` has been moved to `nn.utils.parametrizations.weight_norm`. As a result, parametrized layers no longer expose `weight_g` or `weight_v`, but only `weight`. This causes an AttributeError when DAC conversion attempts to access weights:
```bash
AttributeError: 'ParametrizedConv1d' object has no attribute 'weight_g'. Did you mean: 'weight'?
```
This PR moves `apply_weight_norm` directly into the script, explicitly setting to use `weight_norm = nn.utils.weight_norm` to ensure compatibility.

Changed files:
filename: src/transformers/models/dac/convert_dac_checkpoint.py
patch: @@ -17,6 +17,7 @@
 import re
 
 import torch
+import torch.nn as nn
 
 from transformers import (
     DacConfig,
@@ -185,6 +186,38 @@ def recursively_load_weights(orig_dict, hf_model, model_name):
     logger.warning(f"Unused weights: {unused_weights}")
 
 
+def apply_weight_norm(model):
+    weight_norm = nn.utils.weight_norm
+
+    for layer in model.quantizer.quantizers:
+        weight_norm(layer.in_proj)
+        weight_norm(layer.out_proj)
+
+    weight_norm(model.encoder.conv1)
+    weight_norm(model.encoder.conv2)
+
+    for layer in model.encoder.block:
+        weight_norm(layer.conv1)
+        weight_norm(layer.res_unit1.conv1)
+        weight_norm(layer.res_unit1.conv2)
+        weight_norm(layer.res_unit2.conv1)
+        weight_norm(layer.res_unit2.conv2)
+        weight_norm(layer.res_unit3.conv1)
+        weight_norm(layer.res_unit3.conv2)
+
+    weight_norm(model.decoder.conv1)
+    weight_norm(model.decoder.conv2)
+
+    for layer in model.decoder.block:
+        weight_norm(layer.conv_t1)
+        weight_norm(layer.res_unit1.conv1)
+        weight_norm(layer.res_unit1.conv2)
+        weight_norm(layer.res_unit2.conv1)
+        weight_norm(layer.res_unit2.conv2)
+        weight_norm(layer.res_unit3.conv1)
+        weight_norm(layer.res_unit3.conv2)
+
+
 @torch.no_grad()
 def convert_checkpoint(
     model_name,
@@ -214,7 +247,7 @@ def convert_checkpoint(
 
     original_checkpoint = model_dict["state_dict"]
 
-    model.apply_weight_norm()
+    apply_weight_norm(model)
     recursively_load_weights(original_checkpoint, model, model_name)
     model.remove_weight_norm()
Repository: vercel/next.js
Pull request: Turbopack: fix instrumentation-edge layer name

A typo

Regression from https://github.com/vercel/next.js/pull/80388

Closes PACK-5014

Changed files:
filename: crates/next-api/src/project.rs
patch: @@ -1567,7 +1567,7 @@ impl Project {
                 self.execution_context(),
             ),
             Layer::new_with_user_friendly_name(
-                rcstr!("instrumentation"),
+                rcstr!("instrumentation-edge"),
                 rcstr!("Edge Instrumentation"),
             ),
         )))
Repository: nodejs/node
Pull request: tools: ignore V8 tests in CodeQL scans

CodeQL reports hundreds of errors in V8 tests indicating 'A parse error occurred'. Code quality issues (real or spurious) in V8 tests are not a concern for Node.js. Exclude the tests from CodeQL scans.

<!--
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/codeql-config.yml
patch: @@ -2,3 +2,4 @@ name: CodeQL config
 
 paths-ignore:
   - test
+  - deps/v8/test
Repository: facebook/react
Pull request: SchedulerPostTask: Reuse original TaskController

## Summary

It's not clear to me why we currently create a new TaskController in `runTask` – ultimately, we use the same signal and priority from the original created in `unstable_scheduleCallback`

## How did you test this change?
```
yarn test SchedulerPostTask
```

Changed files:
filename: packages/scheduler/src/forks/SchedulerPostTask.js
patch: @@ -129,15 +129,9 @@ function runTask<T>(
     if (typeof result === 'function') {
       // Assume this is a continuation
       const continuation: SchedulerCallback<T> = (result: any);
-      const continuationController = new TaskController({
-        priority: postTaskPriority,
-      });
       const continuationOptions = {
-        signal: continuationController.signal,
+        signal: node._controller.signal,
       };
-      // Update the original callback node's controller, since even though we're
-      // posting a new task, conceptually it's the same one.
-      node._controller = continuationController;
 
       const nextTask = runTask.bind(
         null,
Repository: electron/electron
Pull request: fix: cppgc crashes on Linux with 16KiB pages

Backport of #45571

See that PR for details.


Notes: Fixed an issue where Electron may experience crashes on Linux with 16KiB pages.


Changed files:
filename: shell/browser/javascript_environment.cc
patch: @@ -86,6 +86,10 @@ v8::Isolate* JavascriptEnvironment::Initialize(uv_loop_t* event_loop,
   auto* cmd = base::CommandLine::ForCurrentProcess();
   // --js-flags.
   std::string js_flags = "--no-freeze-flags-after-init ";
+#if defined(OS_LINUX)
+  // See https://issues.chromium.org/issues/378017037 - fixed in M134.
+  js_flags.append("--nodecommit_pooled_pages ");
+#endif
   js_flags.append(cmd->GetSwitchValueASCII(blink::switches::kJavaScriptFlags));
   v8::V8::SetFlagsFromString(js_flags.c_str(), js_flags.size());
Repository: denoland/deno
Pull request: fix(publish): support `virtual:` and `cloudflare:` schemes

For https://github.com/jsr-io/jsr/pull/1035

Changed files:
filename: cli/graph_util.rs
patch: @@ -429,7 +429,7 @@ impl ModuleGraphCreator {
         specifier: &deno_ast::ModuleSpecifier,
         options: deno_graph::source::LoadOptions,
       ) -> deno_graph::source::LoadFuture {
-        if specifier.scheme() == "bun" {
+        if matches!(specifier.scheme(), "bun" | "virtual" | "cloudflare") {
           return Box::pin(std::future::ready(Ok(Some(
             deno_graph::source::LoadResponse::External {
               specifier: specifier.clone(),

filename: cli/tools/publish/graph.rs
patch: @@ -46,7 +46,7 @@ impl GraphDiagnosticsCollector {
        resolution: &ResolutionResolved| {
         if visited.insert(resolution.specifier.clone()) {
           match resolution.specifier.scheme() {
-            "file" | "data" | "node" | "bun" => {}
+            "file" | "data" | "node" | "bun" | "virtual" | "cloudflare" => {}
             "jsr" => {
               skip_specifiers.insert(resolution.specifier.clone());
 

filename: tests/specs/publish/bun_specifier/mod.ts
patch: @@ -1 +0,0 @@
-import "bun:sqlite";

filename: tests/specs/publish/special_specifiers/__test__.jsonc
patch: @@ -1,4 +1,4 @@
 {
   "args": "publish --token 'sadfasdf'",
-  "output": "bun_specifier.out"
+  "output": "special_specifiers.out"
 }

filename: tests/specs/publish/special_specifiers/mod.ts
patch: @@ -0,0 +1,3 @@
+import "bun:sqlite";
+import "virtual:test";
+import "cloudflare:sockets";
Repository: vercel/next.js
Pull request: Allow `pnpm build` to be run when offline

When running `pnpm build` in the Next.js repo without an internet connection, this currently fails while trying to download the AMP `validator_wasm.js` file. We can fix this, and thus enable offline support for `pnpm build`, by moving the download of this file from the `precompile` task to the `ncc_amphtml_validator` task. The file is then committed to `src/compiled/amphtml-validator`, and later copied into `dist/compiled/amphtml-validator` as part of the build command, without the need to trigger a request.

Changed files:
filename: packages/next/taskfile.js
patch: @@ -676,6 +676,23 @@ export async function ncc_amphtml_validator(task, opts) {
     .source(relative(__dirname, require.resolve('amphtml-validator')))
     .ncc({ packageName: 'amphtml-validator', externals })
     .target('src/compiled/amphtml-validator')
+
+  const validatorRes = await fetch(
+    'https://cdn.ampproject.org/v0/validator_wasm.js'
+  ).catch((err) => {
+    throw new Error('Failed to fetch AMP validator', { cause: err })
+  })
+
+  if (!validatorRes.ok) {
+    throw new Error(
+      `Failed to get the AMP validator, status: ${validatorRes.status}`
+    )
+  }
+
+  await fs.writeFile(
+    join(__dirname, 'src/compiled/amphtml-validator/validator_wasm.js'),
+    require('buffer').Buffer.from(await validatorRes.arrayBuffer())
+  )
 }
 
 // eslint-disable-next-line camelcase
@@ -2244,23 +2261,6 @@ export async function precompile(task, opts) {
     ['browser_polyfills', 'copy_ncced', 'copy_styled_jsx_assets'],
     opts
   )
-
-  const validatorRes = await fetch(
-    'https://cdn.ampproject.org/v0/validator_wasm.js'
-  ).catch((err) => {
-    throw new Error('Failed to fetch AMP validator', { cause: err })
-  })
-
-  if (!validatorRes.ok) {
-    throw new Error(
-      `Failed to get the AMP validator, status: ${validatorRes.status}`
-    )
-  }
-
-  await fs.writeFile(
-    join(__dirname, 'dist/compiled/amphtml-validator/validator_wasm.js'),
-    require('buffer').Buffer.from(await validatorRes.arrayBuffer())
-  )
 }
 
 // eslint-disable-next-line camelcase
Repository: electron/electron
Pull request: fix: cppgc crashes on Linux with 16KiB pages

#### Description of Change

Closes https://github.com/electron/electron/issues/45560.
Refs https://issues.chromium.org/issues/378017037.

Fixes an issue where Electron may experience crashes on Linux with 16KiB pages. This happened due to hardcoded `kGuardPageSize` in cppgc - see above CRBUG for more. Fixed in M134 and above.

#### 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 issue where Electron may experience crashes on Linux with 16KiB pages.


Changed files:
filename: shell/browser/javascript_environment.cc
patch: @@ -85,6 +85,10 @@ v8::Isolate* JavascriptEnvironment::Initialize(uv_loop_t* event_loop,
   auto* cmd = base::CommandLine::ForCurrentProcess();
   // --js-flags.
   std::string js_flags = "--no-freeze-flags-after-init ";
+#if defined(OS_LINUX)
+  // See https://issues.chromium.org/issues/378017037 - fixed in M134.
+  js_flags.append("--nodecommit_pooled_pages ");
+#endif
   js_flags.append(cmd->GetSwitchValueASCII(blink::switches::kJavaScriptFlags));
   v8::V8::SetFlagsFromString(js_flags.c_str(), js_flags.size());
Repository: denoland/deno
Pull request: fix(coverage): improve responsiveness of HTML reporter on small screens

Excuse the oversized screenshots. They were taken when scrolled to the right on a small screen. @kt3k

### Before (all files view)
![_Users_asher_GitHub_redis_coverage_html_index html(iPhone SE) (1)](https://github.com/user-attachments/assets/10dc45f7-bf88-4386-b734-59d3060022ec)
### After (all files view)
![_Users_asher_GitHub_redis_coverage_html_index html(iPhone SE) (2)](https://github.com/user-attachments/assets/67790353-a9d0-4e98-88a5-55aaea9a0d79)
### Before (file view)
![_Users_asher_GitHub_redis_coverage_html_mod ts html(iPhone SE)](https://github.com/user-attachments/assets/30ad69d2-9a52-46f2-a48d-cadacc8cbe52)
### After (file view)
![_Users_asher_GitHub_redis_coverage_html_mod ts html(iPhone SE) (1)](https://github.com/user-attachments/assets/03378d50-194e-4885-b057-519d31b6e1b7)






Changed files:
filename: cli/tools/coverage/reporter.rs
patch: @@ -510,7 +510,7 @@ impl HtmlCoverageReporter {
         <body>
           <div class='wrapper'>
             {header}
-            <div class='pad1'>
+            <div class='pad1 overflow-auto'>
               {main_content}
             </div>
             <div class='push'></div>

filename: cli/tools/coverage/style.css
patch: @@ -369,3 +369,6 @@ pre.prettyprint {
 .push {
   height: 48px;
 }
+.overflow-auto {
+  overflow: auto;
+}
\ No newline at end of file