helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: fix `check_bad commit.py` gives wrong results

# What does this PR do?

During bisecting, it uses

> elif f"{target_test} FAILED" in result.stdout:

to determine a commit fails a test or not.

However, since #35912, the output is kind altered with `live log call`, and the results are wrong (it think every commit passes).

This PR fixes the issue by checking 

> elif f"FAILED {target_test}" in result.stdout:

(at `short test summary info`)

#### current output running on main

> tests/models/instructblipvideo/test_modeling_instructblipvideo.py::InstructBlipVideoModelIntegrationTest::test_expansion_in_processing 
-------------------------------- live log call ---------------------------------
WARNING  transformers.video_processing_utils:logging.py:[32](https://github.com/huggingface/transformers/actions/runs/14999267405/job/42141539393#step:5:33)8 You have video processor config saved in `preprocessor.json` file which is deprecated. Video processor configs should be saved in their own `video_preprocessor.json` file. You can rename the file or load and save the processor back which renames it automatically. Loading from `preprocessor.json` will be removed in v5.0.
FAILED

Changed files:
filename: utils/check_bad_commit.py
patch: @@ -55,7 +55,7 @@ def create_script(target_test):
     else:
         print(f"pytest failed to run: {{result.stderr}}")
         exit(-1)
-elif f"{target_test} FAILED" in result.stdout:
+elif f"FAILED {target_test}" in result.stdout:
     print("test failed")
     exit(2)
Repository: denoland/deno
Pull request: tests: enable auto_discovered tests of run/no_deno_json

Towards https://github.com/denoland/deno/issues/25241

Changed files:
filename: tests/specs/run/no_deno_json/__test__.jsonc
patch: @@ -29,19 +29,33 @@
       "cwd": "code"
     },
     "auto_discovered": {
-      // TODO(2.0): most likely needs to change output to not expect auto-install
-      "ignore": true,
-      // auto-discovered node_modules relative package.json
-      "args": "run -A main.js",
-      "output": "code/sub_dir/main.out",
-      "cwd": "code/sub_dir"
+      "steps": [
+        {
+          "args": "install",
+          "output": "code/install.out",
+          "cwd": "code"
+        },
+        {
+          // auto-discovered node_modules relative package.json
+          "args": "run -A main.js",
+          "output": "code/sub_dir/main.out",
+          "cwd": "code/sub_dir"
+        }
+      ]
     },
     "auto_discovered_arg": {
-      // TODO(2.0): most likely needs to change output to not expect auto-install
-      "ignore": true,
-      // auto-discovered for local script arg
-      "args": "run -L debug -A code/main.ts", // notice this is not in the sub dir
-      "output": "main.out"
+      "steps": [
+        {
+          "args": "install",
+          "output": "code/install.out",
+          "cwd": "code"
+        },
+        {
+          // auto-discovered for local script arg
+          "args": "run -L debug -A code/main.ts", // notice this is not in the sub dir
+          "output": "main.out"
+        }
+      ]
     }
   }
 }

filename: tests/specs/run/no_deno_json/code/install.out
patch: @@ -0,0 +1,26 @@
+[UNORDERED_START]
+Download http://localhost:4260/@denotest/check-error
+Download http://localhost:4260/@denotest/cjs-default-export
+Download http://localhost:4260/chalk
+Download http://localhost:4260/ansi-styles
+Download http://localhost:4260/supports-color
+Download http://localhost:4260/color-convert
+Download http://localhost:4260/has-flag
+Download http://localhost:4260/color-name
+Download http://localhost:4260/@denotest/check-error/1.0.0.tgz
+Download http://localhost:4260/@denotest/cjs-default-export/1.0.0.tgz
+Download http://localhost:4260/chalk/chalk-4.1.2.tgz
+Download http://localhost:4260/ansi-styles/ansi-styles-4.3.0.tgz
+Download http://localhost:4260/supports-color/supports-color-7.2.0.tgz
+Download http://localhost:4260/color-convert/color-convert-2.0.1.tgz
+Download http://localhost:4260/has-flag/has-flag-4.0.0.tgz
+Download http://localhost:4260/color-name/color-name-1.1.4.tgz
+Initialize @denotest/cjs-default-export@1.0.0
+Initialize color-name@1.1.4
+Initialize ansi-styles@4.3.0
+Initialize has-flag@4.0.0
+Initialize supports-color@7.2.0
+Initialize color-convert@2.0.1
+Initialize chalk@4.1.2
+Initialize @denotest/check-error@1.0.0
+[UNORDERED_END]

filename: tests/specs/run/no_deno_json/code/sub_dir/main.out
patch: @@ -1,4 +1,3 @@
-Download http://[WILDCARD]
 [WILDCARD]sub_dir
 {
   [WILDCARD]
Repository: vercel/next.js
Pull request: Do not run CI for modifying changeset related files

<!-- Thanks for opening a PR! Your contribution is much appreciated.
To make sure your PR is handled as smoothly as possible we request that you follow the checklist sections below.
Choose the right checklist for the change(s) that you're making:

## For Contributors

### Improving Documentation

- Run `pnpm prettier-fix` to fix formatting issues before opening the PR.
- Read the Docs Contribution Guide to ensure your contribution follows the docs guidelines: https://nextjs.org/docs/community/contribution-guide

### Adding or Updating Examples

- The "examples guidelines" are followed from our contributing doc https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md
- Make sure the linting passes by running `pnpm build && pnpm lint`. See https://github.com/vercel/next.js/blob/canary/contributing/repository/linting.md

### Fixing a bug

- Related issues linked using `fixes #number`
- Tests added. See: https://github.com/vercel/next.js/blob/canary/contributing/core/testing.md#writing-tests-for-nextjs
- Errors have a helpful link attached, see https://github.com/vercel/next.js/blob/canary/contributing.md

### Adding a feature

- Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR. (A discussion must be opened, see https://github.com/vercel/next.js/discussions/new?category=ideas)
- Related issues/discussions are linked using `fixes #number`
- e2e tests added (https://github.com/vercel/next.js/blob/canary/contributing/core/testing.md#writing-tests-for-nextjs)
- Documentation added
- Telemetry added. In case of a feature if it's used or not.
- Errors have a helpful link attached, see https://github.com/vercel/next.js/blob/canary/contributing.md


## For Maintainers

- Minimal description (aim for explaining to someone not on the team to understand the PR)
- When linking to a Slack thread, you might want to share details of the conclusion
- Link both the Linear (Fixes NEXT-xxx) and the GitHub issues
- Add review comments if necessary to explain to the reviewer the logic behind a change

### What?

### Why?

### How?

Closes NEXT-
Fixes #

-->


Changed files:
filename: scripts/run-for-change.js
patch: @@ -9,6 +9,7 @@ const CHANGE_ITEM_GROUPS = {
     'docs',
     'errors',
     'examples',
+    '.changeset',
     'UPGRADING.md',
     'contributing.md',
     'contributing',
Repository: ggml-org/llama.cpp
Pull request: [Ascend NPU] Enable labeler

*Make sure to read the [contributing guidelines](https://github.com/ggml-org/llama.cpp/blob/master/CONTRIBUTING.md) before submitting a PR*

cc: @hipudding 


Changed files:
filename: .github/labeler.yml
patch: @@ -86,3 +86,10 @@ nix:
 embedding:
     - changed-files:
         - any-glob-to-any-file: examples/embedding/
+
+Ascend NPU:
+    - changed-files:
+        - any-glob-to-any-file:
+            - ggml/include/ggml-cann.h
+            - ggml/src/ggml-cann/**
+            - docs/backend/CANN.md
Repository: denoland/deno
Pull request: feat: make 'globalThis.location' a configurable property

This commit changes `globalThis.location` property to be configurable
so that packages wanting to override it (or delete it) work properly.

Towards https://github.com/denoland/deno/issues/23882

This change makes reproduction from https://github.com/denoland/deno/issues/23882#issuecomment-2340783437
pass properly.

Changed files:
filename: runtime/js/99_main.js
patch: @@ -654,6 +654,7 @@ function bootstrapMainRuntime(runtimeOptions, warmup = false) {
     if (location_ == null) {
       mainRuntimeGlobalProperties.location = {
         writable: true,
+        configurable: true,
       };
     } else {
       location.setLocationHref(location_);

filename: tests/specs/run/location/__test__.jsonc
patch: @@ -0,0 +1,8 @@
+{
+  "tests": {
+    "location_object_define_property": {
+      "args": "run location.js",
+      "output": "location.out"
+    }
+  }
+}

filename: tests/specs/run/location/location.js
patch: @@ -0,0 +1,24 @@
+let _location = undefined;
+
+console.log(globalThis.location);
+
+Object.defineProperty(globalThis, "location", {
+  get() {
+    return _location;
+  },
+  set(v) {
+    _location = v;
+  },
+  configurable: true,
+});
+
+console.log(globalThis.location);
+
+globalThis.location = "https://deno.com";
+
+console.log(_location);
+console.log(location);
+
+delete globalThis["location"];
+
+console.log(globalThis.location);

filename: tests/specs/run/location/location.out
patch: @@ -0,0 +1,5 @@
+undefined
+undefined
+https://deno.com
+https://deno.com
+undefined
Repository: nodejs/node
Pull request: deps: update amaro to 0.3.1

This is an automated update of amaro to 0.3.1.

Fixes: https://github.com/nodejs/node/issues/56597

Changed files:
filename: deps/amaro/dist/package.json
patch: @@ -4,7 +4,7 @@
     "강동윤 <kdy1997.dev@gmail.com>"
   ],
   "description": "wasm module for swc",
-  "version": "1.10.7",
+  "version": "1.10.11",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

filename: deps/amaro/dist/strip-loader.js
patch: @@ -3,7 +3,7 @@ import { isSwcError, wrapAndReThrowSwcError } from "./errors.js";
 import { transformSync } from "./index.js";
 export async function load(url, context, nextLoad) {
   const { format } = context;
-  if (format.endsWith("-typescript")) {
+  if (format?.endsWith("-typescript")) {
     try {
       const { source } = await nextLoad(url, {
         ...context,

filename: deps/amaro/dist/transform-loader.js
patch: @@ -3,7 +3,7 @@ import { isSwcError, wrapAndReThrowSwcError } from "./errors.js";
 import { transformSync } from "./index.js";
 export async function load(url, context, nextLoad) {
   const { format } = context;
-  if (format.endsWith("-typescript")) {
+  if (format?.endsWith("-typescript")) {
     try {
       const { source } = await nextLoad(url, {
         ...context,

filename: deps/amaro/package.json
patch: @@ -1,6 +1,6 @@
 {
 	"name": "amaro",
-	"version": "0.3.0",
+	"version": "0.3.1",
 	"description": "Node.js TypeScript wrapper",
 	"license": "MIT",
 	"type": "commonjs",

filename: src/amaro_version.h
patch: @@ -2,5 +2,5 @@
 // Refer to tools/dep_updaters/update-amaro.sh
 #ifndef SRC_AMARO_VERSION_H_
 #define SRC_AMARO_VERSION_H_
-#define AMARO_VERSION "0.3.0"
+#define AMARO_VERSION "0.3.1"
 #endif  // SRC_AMARO_VERSION_H_
Repository: vercel/next.js
Pull request: fix(turbopack): prepend a question mark in AssetIdent display

## What?
Just update AssetIdent display, prepend "?" like url query. See printed ident with query like below
<img width="1500" alt="image" src="https://github.com/user-attachments/assets/bd37733f-cdd1-496e-b7f0-5c372e842d14" />


Changed files:
filename: turbopack/crates/turbopack-core/src/ident.rs
patch: @@ -59,7 +59,7 @@ impl ValueToString for AssetIdent {
 
         let query = self.query.await?;
         if !query.is_empty() {
-            write!(s, "{}", &*query)?;
+            write!(s, "?{}", &*query)?;
         }
 
         if let Some(fragment) = &self.fragment {
Repository: ggml-org/llama.cpp
Pull request: cuda : fix buffer type check with integrated GPUs

The CUDA backend now also accepts host buffers with integrated GPUs.

Fixes #14068

Changed files:
filename: ggml/src/ggml-cuda/ggml-cuda.cu
patch: @@ -1144,7 +1144,6 @@ typedef void (*ggml_cuda_op_mul_mat_t)(
 static cudaError_t ggml_cuda_cpy_tensor_2d(
     void * dst, const struct ggml_tensor * src, int64_t i3, int64_t i2, int64_t i1_low, int64_t i1_high, cudaStream_t stream) {
 
-    GGML_ASSERT(ggml_backend_buffer_is_cuda(src->buffer));
     const char * src_ptr = (const char *) src->data;
     char       * dst_ptr = (char       *) dst;
 
@@ -1427,8 +1426,6 @@ static void ggml_cuda_op_mul_mat(
     const int64_t nb2 = dst->nb[2];
     const int64_t nb3 = dst->nb[3];
 
-    GGML_ASSERT(ggml_backend_buffer_is_cuda(dst->buffer));
-    GGML_ASSERT(ggml_backend_buffer_is_cuda(src1->buffer));
     ggml_backend_cuda_buffer_context * src1_ctx = (ggml_backend_cuda_buffer_context *) src1->buffer->context;
     ggml_backend_cuda_buffer_context * dst_ctx  = (ggml_backend_cuda_buffer_context *) dst->buffer->context;
 
@@ -1750,7 +1747,7 @@ static void ggml_cuda_mul_mat_batched_cublas(ggml_backend_cuda_context & ctx, co
     GGML_ASSERT(!ggml_is_transposed(src0));
     GGML_ASSERT(!ggml_is_transposed(src1));
 
-    GGML_ASSERT(ggml_backend_buffer_is_cuda(src0->buffer));
+    GGML_ASSERT(!ggml_backend_buft_is_cuda_split(src0->buffer->buft));
     GGML_ASSERT(src0->type == GGML_TYPE_F16);
 
     // Byte offsets and tensor dimensions are currently used in an inconsistent way for dst.
Repository: denoland/deno
Pull request: fix(ext/web): acknowledge the capture option on EventTarget.removeEventListener

Fixes https://github.com/denoland/deno/issues/25787

Changed files:
filename: ext/web/02_event.js
patch: @@ -7,7 +7,6 @@
 
 import { core, primordials } from "ext:core/mod.js";
 const {
-  ArrayPrototypeFilter,
   ArrayPrototypeIncludes,
   ArrayPrototypeIndexOf,
   ArrayPrototypeMap,
@@ -982,12 +981,7 @@ class EventTarget {
     );
 
     const { listeners } = self[eventTargetData];
-    if (callback !== null && listeners[type]) {
-      listeners[type] = ArrayPrototypeFilter(
-        listeners[type],
-        (listener) => listener.callback !== callback,
-      );
-    } else if (callback === null || !listeners[type]) {
+    if (callback === null || !listeners[type]) {
       return;
     }
 

filename: tests/unit/event_target_test.ts
patch: @@ -67,6 +67,64 @@ Deno.test(function anEventTargetCanBeSubclassed() {
   assertEquals(callCount, 0);
 });
 
+Deno.test(function removeEventListenerTest() {
+  const target = new EventTarget();
+  let callCount = 0;
+  const listener = () => {
+    ++callCount;
+  };
+
+  target.addEventListener("incr", listener, true);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 1);
+
+  // Should not remove the listener because useCapture does not match
+  target.removeEventListener("incr", listener, false);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 2);
+
+  // Should remove the listener because useCapture matches
+  target.removeEventListener("incr", listener, true);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 2);
+
+  // Only the capture setting matters to removeEventListener
+  target.addEventListener("incr", listener, { passive: true });
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 3);
+
+  // Should not remove the listener because useCapture does not match
+  target.removeEventListener("incr", listener, { capture: true });
+  target.removeEventListener("incr", listener, true);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 4);
+
+  // Should remove the listener because useCapture matches
+  target.removeEventListener("incr", listener);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 4);
+
+  // Again, should remove the listener because useCapture matches
+  target.addEventListener("incr", listener, { passive: true });
+  target.removeEventListener("incr", listener, false);
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 4);
+
+  // Again, should remove the listener because useCapture matches
+  target.addEventListener("incr", listener, { passive: true });
+  target.removeEventListener("incr", listener, { capture: false });
+
+  target.dispatchEvent(new Event("incr"));
+  assertEquals(callCount, 4);
+});
+
 Deno.test(function removingNullEventListenerShouldSucceed() {
   const document = new EventTarget();
   assertEquals(document.removeEventListener("x", null, false), undefined);
Repository: nodejs/node
Pull request: tools: fix license-builder for inspector_protocol

Refs: https://github.com/nodejs/node/pull/56649



Changed files:
filename: tools/license-builder.sh
patch: @@ -99,8 +99,8 @@ addlicense "npm" "deps/npm" "$licenseText"
 # Build tools
 licenseText="$(cat "${rootdir}/tools/gyp/LICENSE")"
 addlicense "GYP" "tools/gyp" "$licenseText"
-licenseText="$(cat "${rootdir}/tools/inspector_protocol/LICENSE")"
-addlicense "inspector_protocol" "tools/inspector_protocol" "$licenseText"
+licenseText="$(cat "${rootdir}/deps/inspector_protocol/LICENSE")"
+addlicense "inspector_protocol" "deps/inspector_protocol" "$licenseText"
 licenseText="$(cat "${rootdir}/tools/inspector_protocol/jinja2/LICENSE")"
 addlicense "jinja2" "tools/inspector_protocol/jinja2" "$licenseText"
 licenseText="$(cat "${rootdir}/tools/inspector_protocol/markupsafe/LICENSE")"
Repository: vercel/next.js
Pull request: Update Turbopack development test manifest

This auto-generated PR updates the development integration test manifest used when testing Turbopack.

Changed files:
filename: test/turbopack-dev-tests-manifest.json
patch: @@ -1930,9 +1930,10 @@
       "Error recovery app render error not shown right after syntax error",
       "Error recovery app server component can recover from a component error",
       "Error recovery app server component can recover from syntax error",
-      "Error recovery app stuck error"
+      "Error recovery app stuck error",
+      "Error recovery app syntax > runtime error"
     ],
-    "failed": ["Error recovery app syntax > runtime error"],
+    "failed": [],
     "pending": [],
     "flakey": [],
     "runtimeError": false
@@ -2722,6 +2723,16 @@
     "flakey": [],
     "runtimeError": false
   },
+  "test/development/app-dir/use-cache-errors/use-cache-errors.test.ts": {
+    "passed": [
+      "use-cache-errors should not show a false-positive compiler error about a misplaced \"use cache\" directive",
+      "use-cache-errors should show a runtime error when calling the incorrectly used cache function"
+    ],
+    "failed": [],
+    "pending": [],
+    "flakey": [],
+    "runtimeError": false
+  },
   "test/development/app-hmr/hmr.test.ts": {
     "passed": [
       "app-dir-hmr filesystem changes can navigate cleanly to a page that requires a change in the Webpack runtime",
@@ -3896,6 +3907,17 @@
     "flakey": [],
     "runtimeError": false
   },
+  "test/development/typescript-plugin/client-boundary/client-boundary.test.ts": {
+    "passed": [
+      "typescript-plugin - client-boundary should have diagnostics for non-serializable props",
+      "typescript-plugin - client-boundary should not have diagnostics for non-serializable action props",
+      "typescript-plugin - client-boundary should not have diagnostics for serializable props"
+    ],
+    "failed": [],
+    "pending": [],
+    "flakey": [],
+    "runtimeError": false
+  },
   "test/development/typescript-plugin/index.test.ts": {
     "passed": ["typescript-plugin should be able to get the language service"],
     "failed": [],
@@ -7014,6 +7036,13 @@
     "flakey": [],
     "runtimeError": false
   },
+  "test/e2e/app-dir/monaco-editor/monaco-editor.test.ts": {
+    "passed": ["monaco-editor should load monaco-editor"],
+    "failed": [],
+    "pending": [],
+    "flakey": [],
+    "runtimeError": false
+  },
   "test/e2e/app-dir/navigation-layout-suspense/navigation-layout-suspense.test.ts": {
     "passed": [
       "app dir - navigation with Suspense in nested layout resolves data after client navigation to a nested layout with Suspense"
Repository: nodejs/node
Pull request: test: mark `test-esm-loader-hooks-inspect-wait` flaky

`test-esm-loader-hooks-inspect-wait` is occasionally crashing on more platforms than Linux. Mark the test as flaky on those platforms as well.

Refs: https://github.com/nodejs/node/issues/54346

<!--
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: test/parallel/parallel.status
patch: @@ -69,12 +69,20 @@ test-domain-no-error-handler-abort-on-uncaught-8: PASS, FLAKY
 test-domain-no-error-handler-abort-on-uncaught-9: PASS, FLAKY
 test-domain-throw-error-then-throw-from-uncaught-exception-handler: PASS, FLAKY
 test-domain-with-abort-on-uncaught-exception: PASS, FLAKY
+# https://github.com/nodejs/node/issues/54346
+test-esm-loader-hooks-inspect-wait: PASS, FLAKY
 # https://github.com/nodejs/node/issues/50050
 test-tick-processor-arguments: SKIP
 # https://github.com/nodejs/node/issues/54534
 test-runner-run-watch: PASS, FLAKY
 
 [$system==freebsd]
+# https://github.com/nodejs/node/issues/54346
+test-esm-loader-hooks-inspect-wait: PASS, FLAKY
+
+[$system==aix]
+# https://github.com/nodejs/node/issues/54346
+test-esm-loader-hooks-inspect-wait: PASS, FLAKY
 
 [$system==ibmi]
 # https://github.com/nodejs/node/pull/30819