helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: test: log html output for a flaky test to debug it

this test has been flaky recently but we don't know why and can't repro locally. this PR adds a log to inspect the full HTML when it fails

x-ref: https://vercel.slack.com/archives/C05KYT5S9FF/p1745259844464719

Changed files:
filename: test/e2e/app-dir/rsc-basic/rsc-basic.test.ts
patch: @@ -94,7 +94,19 @@ describe('app dir - rsc basics', () => {
   it('should correctly render page returning undefined', async () => {
     const homeHTML = await next.render('/return-undefined/page')
     const $ = cheerio.load(homeHTML)
-    expect(stripHTMLComments($('#return-undefined-layout').html())).toBeEmpty()
+    try {
+      expect(
+        stripHTMLComments($('#return-undefined-layout').html())
+      ).toBeEmpty()
+    } catch (err) {
+      const sep = '='.repeat(40) + ' PAGE HTML ' + '='.repeat(40)
+      console.log()
+      console.log(sep)
+      console.log($('html').html())
+      console.log(sep)
+      console.log()
+      throw err
+    }
   })
 
   it('should correctly render component returning undefined', async () => {
Repository: electron/electron
Pull request: fix: confirm a `v8::Value` is a `v8::Object` before casting it

Backport of #43575

See that PR for details.


Notes: none.

Changed files:
filename: shell/browser/api/message_port.cc
patch: @@ -30,12 +30,12 @@ namespace electron {
 
 namespace {
 
-bool IsValidWrappable(const v8::Local<v8::Value>& obj) {
-  v8::Local<v8::Object> port = v8::Local<v8::Object>::Cast(obj);
-
-  if (!port->IsObject())
+bool IsValidWrappable(const v8::Local<v8::Value>& val) {
+  if (!val->IsObject())
     return false;
 
+  v8::Local<v8::Object> port = val.As<v8::Object>();
+
   if (port->InternalFieldCount() != gin::kNumberOfInternalFields)
     return false;
Repository: huggingface/transformers
Pull request: Don't auto-assign reviewers when the author is in HF

As the title says, we don't want to auto-assign reviewers for staff authors because they can do it themselves (and probably more accurately)

Changed files:
filename: .github/scripts/assign_reviewers.py
patch: @@ -54,6 +54,21 @@ def get_file_owners(file_path, codeowners_lines):
             return owners  # Remember, can still be empty!
     return []  # Should never happen, but just in case
 
+def pr_author_is_in_hf(pr_author, codeowners_lines):
+    # Check if the PR author is in the codeowners file
+    for line in codeowners_lines:
+        line = line.split('#')[0].strip()
+        if not line:
+            continue
+
+        # Split into pattern and owners
+        parts = line.split()
+        owners = [owner.removeprefix("@") for owner in parts[1:]]
+
+        if pr_author in owners:
+            return True
+    return False
+
 def main():
     script_dir = Path(__file__).parent.absolute()
     with open(script_dir / "codeowners_for_review_action") as f:
@@ -68,6 +83,9 @@ def main():
     pr_number = event['pull_request']['number']
     pr = repo.get_pull(pr_number)
     pr_author = pr.user.login
+    if pr_author_is_in_hf(pr_author, codeowners_lines):
+        print(f"PR author {pr_author} is in codeowners, skipping review request.")
+        return
 
     existing_reviews = list(pr.get_reviews())
     if existing_reviews:
Repository: ggml-org/llama.cpp
Pull request: llama : plamo rope type is neox

Plamo was broken for over a year due to setting wrong rope_type.

Changed files:
filename: src/llama-model.cpp
patch: @@ -13226,7 +13226,6 @@ llama_rope_type llama_model_rope_type(const llama_model * model) {
         case LLM_ARCH_DECI:
         case LLM_ARCH_BAICHUAN:
         case LLM_ARCH_STARCODER:
-        case LLM_ARCH_PLAMO:
         case LLM_ARCH_ORION:
         case LLM_ARCH_INTERNLM2:
         case LLM_ARCH_MINICPM:
@@ -13265,6 +13264,7 @@ llama_rope_type llama_model_rope_type(const llama_model * model) {
         case LLM_ARCH_PHI2:
         case LLM_ARCH_PHI3:
         case LLM_ARCH_PHIMOE:
+        case LLM_ARCH_PLAMO:
         case LLM_ARCH_GEMMA:
         case LLM_ARCH_GEMMA2:
         case LLM_ARCH_GEMMA3:
Repository: denoland/deno
Pull request: fix: regressions around Error.prepareStackTrace

Includes:
- https://github.com/denoland/deno_core/pull/858
- https://github.com/denoland/deno_core/pull/856

Fixes https://github.com/denoland/deno/issues/24782
Fixes https://github.com/denoland/deno/issues/24825

Changed files:
filename: Cargo.lock
patch: @@ -1350,9 +1350,9 @@ dependencies = [
 
 [[package]]
 name = "deno_core"
-version = "0.300.0"
+version = "0.301.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "eb79d0494b8c15f97761645f4afca5d52a414d4dc8f8aef8ca3b3ca82240ab55"
+checksum = "d595cedb88ce9972e35b6b6ea3832792a570412c06c4ef2fe2b1e54505595e51"
 dependencies = [
  "anyhow",
  "bincode",
@@ -1836,9 +1836,9 @@ dependencies = [
 
 [[package]]
 name = "deno_ops"
-version = "0.176.0"
+version = "0.177.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8ecb75c43c9441f413f0595f694746f90d8ffd292337287da309dbd93773928c"
+checksum = "f9e89d5b94f766a73ecfa03b8481c9b4932d137670bdd9b227e5fcf283175af7"
 dependencies = [
  "proc-macro-rules",
  "proc-macro2",
@@ -6047,9 +6047,9 @@ dependencies = [
 
 [[package]]
 name = "serde_v8"
-version = "0.209.0"
+version = "0.210.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "15d9095a8a725125fff1706f940af182e2a8ca01da8b0cd019aa47da93b28b94"
+checksum = "ef949bc9a8f4eb6c6620ed5e52c60f8d7239ef31c9c5e1eb3ad88d882ad9bde2"
 dependencies = [
  "num-bigint",
  "serde",

filename: Cargo.toml
patch: @@ -45,7 +45,7 @@ repository = "https://github.com/denoland/deno"
 
 [workspace.dependencies]
 deno_ast = { version = "=0.41.2", features = ["transpiling"] }
-deno_core = { version = "0.300.0" }
+deno_core = { version = "0.301.0" }
 
 deno_bench_util = { version = "0.158.0", path = "./bench_util" }
 deno_lockfile = "0.20.0"

filename: cli/main.rs
patch: @@ -399,7 +399,7 @@ fn resolve_flags_and_init(
   };
 
   init_v8_flags(&default_v8_flags, &flags.v8_flags, get_v8_flags_from_env());
-  deno_core::JsRuntime::init_platform(None);
+  deno_core::JsRuntime::init_platform(None, !*DENO_FUTURE);
   util::logger::init(flags.log_level);
 
   Ok(flags)

filename: cli/standalone/mod.rs
patch: @@ -728,7 +728,7 @@ pub async fn run(
 
   // Initialize v8 once from the main thread.
   v8_set_flags(construct_v8_flags(&[], &metadata.v8_flags, vec![]));
-  deno_core::JsRuntime::init_platform(None);
+  deno_core::JsRuntime::init_platform(None, true);
 
   let mut worker = worker_factory
     .create_main_worker(WorkerExecutionMode::Run, main_module, permissions)

filename: ext/node/ops/vm.rs
patch: @@ -140,7 +140,7 @@ mod tests {
   #[test]
   fn test_run_in_this_context() {
     let platform = v8::new_default_platform(0, false).make_shared();
-    deno_core::JsRuntime::init_platform(Some(platform));
+    deno_core::JsRuntime::init_platform(Some(platform), false);
 
     let isolate = &mut v8::Isolate::new(Default::default());
Repository: vercel/next.js
Pull request: fix bad next.browser() arguments in socket.io test

i guess we forgot to remove the `next.url` when this was converted from `webdriver()` to `next.browser()`

Changed files:
filename: test/deploy-tests-manifest.json
patch: @@ -53,7 +53,7 @@
       "test/e2e/app-dir/rsc-basic/rsc-basic.test.ts",
       "test/e2e/basepath/basepath.test.ts",
       "test/e2e/postcss-config-cjs/index.test.ts",
-      "test/e2e/socket-io/index.test.js",
+      "test/e2e/socket-io/index.test.ts",
       "test/e2e/middleware-matcher/index.test.ts",
       "test/e2e/next-script/index.test.ts",
       "test/production/standalone-mode/**/*"

filename: test/e2e/socket-io/index.test.ts
patch: @@ -10,13 +10,16 @@ describe('socket-io', () => {
       'utf-8-validate': '6.0.3',
       bufferutil: '4.0.8',
     },
+    // the socket.io setup relies on patching next's `http.Server` instance,
+    // which we can't do when deployed
+    skipDeployment: true,
   })
 
   it('should support socket.io without falling back to polling', async () => {
     let requestsCount = 0
 
-    const browser1 = await next.browser(next.url, '/')
-    const browser2 = await next.browser(next.url, '/', {
+    const browser1 = await next.browser('/')
+    const browser2 = await next.browser('/', {
       beforePageLoad(page) {
         page.on('request', () => {
           requestsCount++
@@ -30,6 +33,7 @@ describe('socket-io', () => {
     await input1.fill('hello world')
     await check(() => input2.inputValue(), /hello world/)
 
+    expect(requestsCount).toBeGreaterThan(0)
     const currentRequestsCount = requestsCount
 
     await input1.fill('123456')
Repository: electron/electron
Pull request: fix: confirm a `v8::Value` is a `v8::Object` before casting it

#### Description of Change

Small fix to confirm that a `Local<Value>` is a `Local<Object>` before casting it as one.

Looks like doing the other way around can cause a crash when`Local.As()` calls `Local.Cast()` if `V8_ENABLE_CHECKS` is defined and the Value isn't an Object?

#### Checklist

- [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: shell/browser/api/message_port.cc
patch: @@ -30,12 +30,12 @@ namespace electron {
 
 namespace {
 
-bool IsValidWrappable(const v8::Local<v8::Value>& obj) {
-  v8::Local<v8::Object> port = v8::Local<v8::Object>::Cast(obj);
-
-  if (!port->IsObject())
+bool IsValidWrappable(const v8::Local<v8::Value>& val) {
+  if (!val->IsObject())
     return false;
 
+  v8::Local<v8::Object> port = val.As<v8::Object>();
+
   if (port->InternalFieldCount() != gin::kNumberOfInternalFields)
     return false;
Repository: facebook/react
Pull request: Add exports field to react-refresh's package.json

<!--
  Thanks for submitting a pull request!
  We appreciate you spending the time to work on these changes. Please provide enough information so that others can review your pull request. The three fields below are mandatory.

  Before submitting a pull request, please make sure the following is done:

  1. Fork [the repository](https://github.com/facebook/react) and create your branch from `main`.
  2. Run `yarn` in the repository root.
  3. If you've fixed a bug or added code that should be tested, add tests!
  4. Ensure the test suite passes (`yarn test`). Tip: `yarn test --watch TestName` is helpful in development.
  5. Run `yarn test --prod` to test in the production environment. It supports the same options as `yarn test`.
  6. If you need a debugger, run `yarn debug-test --watch TestName`, open `chrome://inspect`, and press "Inspect".
  7. Format your code with [prettier](https://github.com/prettier/prettier) (`yarn prettier`).
  8. Make sure your code lints (`yarn lint`). Tip: `yarn linc` to only check changed files.
  9. Run the [Flow](https://flowtype.org/) type checks (`yarn flow`).
  10. If you haven't already, complete the CLA.

  Learn more about contributing: https://reactjs.org/docs/how-to-contribute.html
-->

## Summary

We were migrating our build tools to pure ESM format, and found we cannot import `react-refresh/babel` from a pure ESM module, it indicates to add an extension:

```
Cannot find module '/Users/otakustay/Develop/reskript/packages/config-babel/node_modules/react-refresh/babel' imported from /Users/otakustay/Develop/reskript/packages/config-babel/dist/index.js
```

In order to make pure ESM able to import `react-refresh/babel`, we need to add an `exports` field into `package.json` mapping `./babel` to `./babel.js`.

<!--
 Explain the **motivation** for making this change. What existing problem does the pull request solve?
-->

## How did you test this change?

I have manually modified local `node_modules/react-refresh/package.json` and tested our build.

<!--
  Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes the user interface.
  How exactly did you verify that your PR solves the issue you wanted to solve?
  If you leave this empty, your PR will very likely be closed.
-->


Changed files:
filename: packages/react-refresh/package.json
patch: @@ -18,6 +18,12 @@
     "umd/"
   ],
   "main": "runtime.js",
+  "exports": {
+    ".": "./runtime.js",
+    "./runtime": "./runtime.js",
+    "./babel": "./babel.js",
+    "./package.json": "./package.json"
+  },
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/react.git",
@@ -31,4 +37,4 @@
     "react-dom-16-8": "npm:react-dom@16.8.0",
     "scheduler-0-13": "npm:scheduler@0.13.0"
   }
-}
\ No newline at end of file
+}
Repository: denoland/deno
Pull request: fix(ext/webgpu): assign missing `constants` property of shader about `GPUDevice.createRenderPipeline[Async]`

fixes https://github.com/denoland/deno/issues/24287

Changed files:
filename: ext/webgpu/01_webgpu.js
patch: @@ -1490,6 +1490,7 @@ class GPUDevice extends EventTarget {
       fragment = {
         module,
         entryPoint: descriptor.fragment.entryPoint,
+        constants: descriptor.fragment.constants,
         targets: descriptor.fragment.targets,
       };
     }
@@ -1501,6 +1502,7 @@ class GPUDevice extends EventTarget {
       vertex: {
         module,
         entryPoint: descriptor.vertex.entryPoint,
+        constants: descriptor.vertex.constants,
         buffers: descriptor.vertex.buffers,
       },
       primitive: descriptor.primitive,
@@ -1638,6 +1640,7 @@ class GPUDevice extends EventTarget {
       fragment = {
         module,
         entryPoint: descriptor.fragment.entryPoint,
+        constants: descriptor.fragment.constants,
         targets: descriptor.fragment.targets,
       };
     }
@@ -1649,6 +1652,7 @@ class GPUDevice extends EventTarget {
       vertex: {
         module,
         entryPoint: descriptor.vertex.entryPoint,
+        constants: descriptor.vertex.constants,
         buffers: descriptor.vertex.buffers,
       },
       primitive: descriptor.primitive,

filename: tests/testdata/webgpu/hellotriangle_shader.wgsl
patch: @@ -1,11 +1,20 @@
+// only test purpose
+override value: f32;
+
 @vertex
 fn vs_main(@builtin(vertex_index) in_vertex_index: u32) -> @builtin(position) vec4<f32> {
+    // only test purpose
+    _ = value;
+
     let x = f32(i32(in_vertex_index) - 1);
     let y = f32(i32(in_vertex_index & 1u) * 2 - 1);
     return vec4<f32>(x, y, 0.0, 1.0);
 }
 
 @fragment
 fn fs_main() -> @location(0) vec4<f32> {
+    // only test purpose
+    _ = value;
+
     return vec4<f32>(1.0, 0.0, 0.0, 1.0);
 }

filename: tests/unit/webgpu_test.ts
patch: @@ -129,10 +129,18 @@ Deno.test({
     vertex: {
       module: shaderModule,
       entryPoint: "vs_main",
+      // only test purpose
+      constants: {
+        value: 0.5,
+      },
     },
     fragment: {
       module: shaderModule,
       entryPoint: "fs_main",
+      // only test purpose
+      constants: {
+        value: 0.5,
+      },
       targets: [
         {
           format: "rgba8unorm-srgb",
Repository: ggml-org/llama.cpp
Pull request: server : add cache reuse card link to help

Add a link to a visual card in the help for the `--cache-reuse` argument:

```bash
$ llama-server --help

--cache-reuse N                         min chunk size to attempt reusing from the cache via KV shifting
                                        (default: 0)
                                        [(card)](https://ggml.ai/f0.png)
                                        (env: LLAMA_ARG_CACHE_REUSE)

```

This link will open the following image:

![image](https://ggml.ai/f0.png)

Could be useful to click on it from the terminal to get some extra information about how the feature works. Maybe we can improve the help by adding more cards like this.

Changed files:
filename: common/arg.cpp
patch: @@ -2783,7 +2783,10 @@ common_params_context common_params_parser_init(common_params & params, llama_ex
     ).set_examples({LLAMA_EXAMPLE_SERVER}).set_env("LLAMA_ARG_THREADS_HTTP"));
     add_opt(common_arg(
         {"--cache-reuse"}, "N",
-        string_format("min chunk size to attempt reusing from the cache via KV shifting (default: %d)", params.n_cache_reuse),
+        string_format(
+            "min chunk size to attempt reusing from the cache via KV shifting (default: %d)\n"
+            "[(card)](https://ggml.ai/f0.png)", params.n_cache_reuse
+        ),
         [](common_params & params, int value) {
             params.n_cache_reuse = value;
         }

filename: examples/server/README.md
patch: @@ -154,7 +154,7 @@ The project is under active development, and we are [looking for feedback and co
 | `--ssl-cert-file FNAME` | path to file a PEM-encoded SSL certificate<br/>(env: LLAMA_ARG_SSL_CERT_FILE) |
 | `-to, --timeout N` | server read/write timeout in seconds (default: 600)<br/>(env: LLAMA_ARG_TIMEOUT) |
 | `--threads-http N` | number of threads used to process HTTP requests (default: -1)<br/>(env: LLAMA_ARG_THREADS_HTTP) |
-| `--cache-reuse N` | min chunk size to attempt reusing from the cache via KV shifting (default: 0)<br/>(env: LLAMA_ARG_CACHE_REUSE) |
+| `--cache-reuse N` | min chunk size to attempt reusing from the cache via KV shifting (default: 0)<br/>[(card)](https://ggml.ai/f0.png)<br/>(env: LLAMA_ARG_CACHE_REUSE) |
 | `--metrics` | enable prometheus compatible metrics endpoint (default: disabled)<br/>(env: LLAMA_ARG_ENDPOINT_METRICS) |
 | `--slots` | enable slots monitoring endpoint (default: disabled)<br/>(env: LLAMA_ARG_ENDPOINT_SLOTS) |
 | `--props` | enable changing global properties via POST /props (default: disabled)<br/>(env: LLAMA_ARG_ENDPOINT_PROPS) |
Repository: denoland/deno
Pull request: fix(ext/console): render properties of Intl.Locale

Fixes #21271

Changed files:
filename: ext/console/01_console.js
patch: @@ -790,6 +790,24 @@ function formatRaw(ctx, value, recurseTimes, typedArray, proxyDetails) {
             return ctx.stylize(base, "date");
           }
         }
+      } else if (
+        proxyDetails === null &&
+        ObjectPrototypeIsPrototypeOf(globalThis.Intl.Locale.prototype, value)
+      ) {
+        braces[0] = `${getPrefix(constructor, tag, "Intl.Locale")}{`;
+        ArrayPrototypeUnshift(
+          keys,
+          "baseName",
+          "calendar",
+          "caseFirst",
+          "collation",
+          "hourCycle",
+          "language",
+          "numberingSystem",
+          "numeric",
+          "region",
+          "script",
+        );
       } else if (
         proxyDetails === null &&
         typeof globalThis.Temporal !== "undefined" &&

filename: tests/unit/console_test.ts
patch: @@ -1066,6 +1066,24 @@ Deno.test(async function consoleTestStringifyPromises() {
   assertEquals(strLines[1], "  <rejected> Error: Whoops");
 });
 
+Deno.test(function consoleTestStringifyIntlLocale() {
+  assertEquals(
+    stringify(new Intl.Locale("zh-Hant-TW", { hourCycle: "h12" })),
+    `Locale [Intl.Locale] {
+  baseName: "zh-Hant-TW",
+  calendar: undefined,
+  caseFirst: undefined,
+  collation: undefined,
+  hourCycle: "h12",
+  language: "zh",
+  numberingSystem: undefined,
+  numeric: false,
+  region: "TW",
+  script: "Hant"
+}`,
+  );
+});
+
 Deno.test(function consoleTestWithCustomInspector() {
   class A {
     [customInspect](
Repository: huggingface/transformers
Pull request: Fixing gated repo issues

# What does this PR do?

Using unsloth model alternative for gated repos in quark quantization, instead of `require_read_token`.

https://github.com/huggingface/transformers/actions/runs/14415368554/job/40431139879

Changed files:
filename: tests/quantization/quark_integration/test_quark.py
patch: @@ -19,7 +19,6 @@
     is_torch_available,
     require_accelerate,
     require_quark,
-    require_read_token,
     require_torch_gpu,
     require_torch_multi_gpu,
     slow,
@@ -44,7 +43,7 @@ def test_commmon_args(self):
 @require_quark
 @require_torch_gpu
 class QuarkTest(unittest.TestCase):
-    reference_model_name = "meta-llama/Llama-3.1-8B-Instruct"
+    reference_model_name = "unsloth/Meta-Llama-3.1-8B-Instruct"
     quantized_model_name = "amd/Llama-3.1-8B-Instruct-w-int8-a-int8-sym-test"
 
     input_text = "Today I am in Paris and"
@@ -76,13 +75,11 @@ def setUpClass(cls):
             device_map=cls.device_map,
         )
 
-    @require_read_token
     def test_memory_footprint(self):
         mem_quantized = self.quantized_model.get_memory_footprint()
 
         self.assertTrue(self.mem_fp16 / mem_quantized > self.EXPECTED_RELATIVE_DIFFERENCE)
 
-    @require_read_token
     def test_device_and_dtype_assignment(self):
         r"""
         Test whether trying to cast (or assigning a device to) a model after quantization will throw an error.
@@ -96,7 +93,6 @@ def test_device_and_dtype_assignment(self):
             # Tries with a `dtype``
             self.quantized_model.to(torch.float16)
 
-    @require_read_token
     def test_original_dtype(self):
         r"""
         A simple test to check if the model succesfully stores the original dtype
@@ -107,7 +103,6 @@ def test_original_dtype(self):
 
         self.assertTrue(isinstance(self.quantized_model.model.layers[0].mlp.gate_proj, QParamsLinear))
 
-    @require_read_token
     def check_inference_correctness(self, model):
         r"""
         Test the generation quality of the quantized model and see that we are matching the expected output.
@@ -131,7 +126,6 @@ def check_inference_correctness(self, model):
         # Get the generation
         self.assertIn(self.tokenizer.decode(output_sequences[0], skip_special_tokens=True), self.EXPECTED_OUTPUTS)
 
-    @require_read_token
     def test_generate_quality(self):
         """
         Simple test to check the quality of the model by comparing the generated tokens with the expected tokens
Repository: ggml-org/llama.cpp
Pull request: convert : explicitly disable trust_remote_code for AutoConfig

This is needed because without `trust_remote_code=False`, the `transformers` library will ask the user to allow remote code (the answer is `y/n`). This breaks automated pipelines where `stdin` is not available


Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -419,7 +419,9 @@ def get_model_part_names(dir_model: Path, prefix: str, suffix: str) -> list[str]
     @staticmethod
     def load_hparams(dir_model: Path):
         try:
-            return AutoConfig.from_pretrained(dir_model).to_dict()
+            # for security reason, we don't allow loading remote code by default
+            # if a model need remote code, we will fallback to config.json
+            return AutoConfig.from_pretrained(dir_model, trust_remote_code=False).to_dict()
         except Exception as e:
             logger.warning(f"Failed to load model config from {dir_model}: {e}")
             logger.warning("Trying to load config.json instead")