helloadhavan/github_issues huggingface dataset:
Repository: facebook/react
Pull request: Update utils-test.js desciption

<!--
  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 test --debug --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
   Updated the typo in test description
<!--
 Explain the **motivation** for making this change. What existing problem does the pull request solve?
-->

## How did you test this change?
   NA, correct test description improves readability of the code and confusion for anyone who is new to the codebase.
<!--
  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-devtools-shared/src/__tests__/utils-test.js
patch: @@ -283,12 +283,12 @@ describe('utils', () => {
       expect(isPlainObject(new (class C {})())).toBe(false);
     });
 
-    it('should retun false for objects, which have not only Object in its prototype chain', () => {
+    it('should return false for objects, which have not only Object in its prototype chain', () => {
       expect(isPlainObject([])).toBe(false);
       expect(isPlainObject(Symbol())).toBe(false);
     });
 
-    it('should retun false for primitives', () => {
+    it('should return false for primitives', () => {
       expect(isPlainObject(5)).toBe(false);
       expect(isPlainObject(true)).toBe(false);
     });
Repository: ggml-org/llama.cpp
Pull request: SYCL: fix rms_norm_mul_add for tensor dim not a multiple of sg_size

The original implementation unconditionally returned true for this operation, leading to a failure when the src0 tensor's columns were not a multiple of the device sub group size. This caused a GGML_ASSERT(ncols % WARP_SIZE == 0) failure in ggml-sycl/norm.cpp.

This change updates the ggml_backend_sycl_device_supports_op check to correctly return true for GGML_OP_RMS_NORM only when the ncols are a multiple of the device sg_size, ensuring the operation can be performed without error.

--

@CISC @ggerganov  Given the current situation, I think the SYCL backend will be replaced by the new ggml openVINO backend for Intel devices, so this is just adjusting and definitely not a proper fix.

Others can feel free to correct me on this. I will try to support everything other than MUL_MAT kernels until we get a definite response from the backend's original maintainers from Intel. Leaving an unmaintained backend around isn’t ideal, so this should at least prevent immediate failures.


Changed files:
filename: ggml/src/ggml-sycl/ggml-sycl.cpp
patch: @@ -4364,11 +4364,12 @@ static bool ggml_backend_sycl_device_supports_op(ggml_backend_dev_t dev, const g
             return (op->type == GGML_TYPE_F32 && op->src[0]->type == GGML_TYPE_F32) && (op->type == op->src[0]->type);
 #endif
         case GGML_OP_NORM:
-        case GGML_OP_RMS_NORM:
             return true;
         case GGML_OP_L2_NORM:
         case GGML_OP_GROUP_NORM:
             return ggml_is_contiguous(op->src[0]);
+        case GGML_OP_RMS_NORM:
+            return ((op->src[0]->ne[0] % WARP_SIZE) == 0);
         case GGML_OP_SCALE:
             return true;
         case GGML_OP_CONT:
Repository: denoland/deno
Pull request: fix(npm): panic when using tag with patched package

* https://github.com/denoland/deno_npm/pull/99

Closes https://github.com/denoland/deno/issues/28890

Changed files:
filename: Cargo.lock
patch: @@ -2337,9 +2337,9 @@ dependencies = [
 
 [[package]]
 name = "deno_npm"
-version = "0.31.0"
+version = "0.31.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "352510920cdb72896495893f9baa344175df4f9317a24858c3c224a58f6f8d2d"
+checksum = "343a30beea9bee6241affc3b1f7cb583c66bb8abbab281169cb18d9cc3344077"
 dependencies = [
  "async-trait",
  "capacity_builder",

filename: Cargo.toml
patch: @@ -62,7 +62,7 @@ deno_lint = "=0.74.0"
 deno_lockfile = "=0.26.1"
 deno_media_type = { version = "=0.2.8", features = ["module_specifier"] }
 deno_native_certs = "0.3.0"
-deno_npm = "=0.31.0"
+deno_npm = "=0.31.1"
 deno_package_json = { version = "=0.6.0", default-features = false }
 deno_path_util = "=0.3.2"
 deno_semver = "=0.7.1"
Repository: huggingface/transformers
Pull request: [Tests] Update model_id in AIMv2 Tests

As per the title, updates model_id in Aimv2 integration tests with that of official ones as the PRs for HF compatible weights are merged.

Changed files:
filename: tests/models/aimv2/test_modeling_aimv2.py
patch: @@ -598,7 +598,7 @@ def test_eager_matches_sdpa_inference(
 class Aimv2ModelIntegrationTest(unittest.TestCase):
     @slow
     def test_inference(self):
-        model_name = "yaswanthgali/aimv2-large-patch14-224-lit-HF"
+        model_name = "apple/aimv2-large-patch14-224-lit"
         model = Aimv2Model.from_pretrained(model_name, device_map=torch_device)
         processor = AutoProcessor.from_pretrained(model_name)
 
@@ -622,16 +622,16 @@ def test_inference(self):
         )
 
         # handle device
-        expected_logits = torch.tensor([[34.2415, 24.6724]]).to(model.device)
-        self.assertTrue(torch.allclose(outputs.logits_per_image, expected_logits, atol=1e-3))
+        expected_logits = torch.tensor([[33.3550, 26.4255]]).to(model.device)
+        torch.testing.assert_close(outputs.logits_per_image, expected_logits, atol=1e-3, rtol=1e-3)
 
 
 @require_vision
 @require_torch
 class Aimv2VisionModelIntegrationTests(unittest.TestCase):
     @slow
     def test_inference(self):
-        model_name = "yaswanthgali/aimv2-large-patch14-224-HF"
+        model_name = "apple/aimv2-large-patch14-224"
 
         model = Aimv2VisionModel.from_pretrained(model_name, device_map=torch_device)
         processor = AutoImageProcessor.from_pretrained(model_name)
@@ -662,7 +662,7 @@ def test_inference(self):
 
     @slow
     def test_inference_for_native_resolution(self):
-        model_name = "yaswanthgali/aimv2-large-patch14-native-HF"
+        model_name = "apple/aimv2-large-patch14-native"
 
         model = Aimv2VisionModel.from_pretrained(model_name, device_map="auto")
         processor = AutoImageProcessor.from_pretrained(model_name)
Repository: ggml-org/llama.cpp
Pull request: tests: add test-backend-ops performance test for mul mat id

This adds a `test-backend-ops perf` test for mul_mat_id using the matrix sizes from qwen3-30b-a3b and gpt-oss-20b. As this is really only meant for testing optimizations I'm only including a limited number of quants and batch sizes for now.

Changed files:
filename: tests/test-backend-ops.cpp
patch: @@ -6400,6 +6400,24 @@ static std::vector<std::unique_ptr<test_case>> make_test_cases_perf() {
         }
     }
 
+    // qwen3-30b-a3b
+    for (int bs : {1, 4, 8, 512}) {
+        for (ggml_type type_a : {GGML_TYPE_F32, GGML_TYPE_F16, GGML_TYPE_Q4_0, GGML_TYPE_Q8_0, GGML_TYPE_Q4_K, GGML_TYPE_Q6_K, GGML_TYPE_IQ2_XS}) {
+            for (ggml_type type_b : {GGML_TYPE_F32}) {
+                test_cases.emplace_back(new test_mul_mat_id(type_a, type_b, 128, 8, false, 768, bs, 2048, 1));
+            }
+        }
+    }
+
+    // gpt-oss-20b
+    for (int bs : {1, 4, 8, 512}) {
+        for (ggml_type type_a : {GGML_TYPE_MXFP4}) {
+            for (ggml_type type_b : {GGML_TYPE_F32}) {
+                test_cases.emplace_back(new test_mul_mat_id(type_a, type_b, 32, 4, false, 2880, bs, 2880, 1));
+            }
+        }
+    }
+
     for (int K : {3, 5}) {
         for (int IC : {256, 2560}) {
             for (int IW_IH : {32, 64, 256}) {
Repository: electron/electron
Pull request: fix: fix mksnapshot gen/v8 path

Backport of #45488

See that PR for details.


Notes: Fixed the path for a needed library used for mksnapshot


Changed files:
filename: .github/actions/build-electron/action.yml
patch: @@ -109,7 +109,9 @@ runs:
         if [ "${{ inputs.target-platform }}" = "win" ]; then
           cd out/Default
           powershell Compress-Archive -update mksnapshot_args mksnapshot.zip
-          powershell Compress-Archive -update gen/v8/embedded.S mksnapshot.zip
+          powershell mkdir mktmp\\gen\\v8
+          powershell Copy-Item gen\\v8\\embedded.S mktmp\\gen\\v8
+          powershell Compress-Archive -update -Path mktmp\\gen mksnapshot.zip
         else
           (cd out/Default; zip mksnapshot.zip mksnapshot_args gen/v8/embedded.S)
         fi
Repository: facebook/react
Pull request: Bugfix: useFormState queues actions in wrong order

I neglected to update the "last" pointer of the action queue. Since the queue is circular, rather than dropping the update, the effect was to add the update to the front of the queue instead of the back. I didn't notice earlier because in my demos/tests, the actions would either resolve really quickly or the actions weren't order dependent (like incrementing a counter).

Changed files:
filename: packages/react-dom/src/__tests__/ReactDOMForm-test.js
patch: @@ -90,7 +90,7 @@ describe('ReactDOMForm', () => {
       const thenable = record.value;
       record.status = 'resolved';
       record.value = text;
-      thenable.pings.forEach(t => t());
+      thenable.pings.forEach(t => t(text));
     }
   }
 
@@ -1082,6 +1082,35 @@ describe('ReactDOMForm', () => {
     });
   });
 
+  // @gate enableFormActions
+  // @gate enableAsyncActions
+  test('queues multiple actions and runs them in order', async () => {
+    let action;
+    function App() {
+      const [state, dispatch] = useFormState(
+        async (s, a) => await getText(a),
+        'A',
+      );
+      action = dispatch;
+      return <Text text={state} />;
+    }
+
+    const root = ReactDOMClient.createRoot(container);
+    await act(() => root.render(<App />));
+    assertLog(['A']);
+
+    await act(() => action('B'));
+    await act(() => action('C'));
+    await act(() => action('D'));
+
+    await act(() => resolveText('B'));
+    await act(() => resolveText('C'));
+    await act(() => resolveText('D'));
+
+    assertLog(['D']);
+    expect(container.textContent).toBe('D');
+  });
+
   // @gate enableFormActions
   // @gate enableAsyncActions
   test('useFormState: warns if action is not async', async () => {

filename: packages/react-reconciler/src/ReactFiberHooks.js
patch: @@ -1915,7 +1915,7 @@ function dispatchFormState<S, P>(
       payload,
       next: first,
     };
-    last.next = newLast;
+    actionQueue.pending = last.next = newLast;
   }
 }
Repository: vercel/next.js
Pull request: Ensure resolvedPathname normalizes trailing slash

Fixes case with trailing slash from refactoring done in https://github.com/vercel/next.js/pull/81144

Changed files:
filename: packages/next/src/build/templates/pages.ts
patch: @@ -225,7 +225,9 @@ export async function handler(
     const method = req.method || 'GET'
 
     const resolvedUrl = formatUrl({
-      pathname: parsedUrl.pathname,
+      pathname: nextConfig.trailingSlash
+        ? parsedUrl.pathname
+        : removeTrailingSlash(parsedUrl.pathname || '/'),
       // make sure to only add query values from original URL
       query: hasStaticProps ? {} : originalQuery,
     })

filename: packages/next/src/server/route-modules/route-module.ts
patch: @@ -59,6 +59,7 @@ import {
   type RouterServerContext,
 } from '../lib/router-utils/router-server-context'
 import { decodePathParams } from '../lib/router-utils/decode-path-params'
+import { removeTrailingSlash } from '../../shared/lib/router/utils/remove-trailing-slash'
 
 /**
  * RouteModuleOptions is the options that are passed to the route module, other
@@ -782,6 +783,8 @@ export abstract class RouteModule<
       resolvedPathname = decodePathParams(resolvedPathname)
     } catch (_) {}
 
+    resolvedPathname = removeTrailingSlash(resolvedPathname)
+
     return {
       query,
       originalQuery,
Repository: electron/electron
Pull request: test: make sure test window is on top for focus tests

Backport of #45435

See that PR for details.


Notes: <!-- Please add a one-line description for app developers to read in the release notes, or 'none' if no notes relevant to app developers. Examples and help on special cases: https://github.com/electron/clerk/blob/main/README.md#examples -->none


Changed files:
filename: spec/chromium-spec.ts
patch: @@ -98,14 +98,13 @@ describe('window.postMessage', () => {
   });
 });
 
-// Tests disabled due to regression in Chromium upgrade
-// https://github.com/electron/electron/issues/45322
-ifdescribe(!(process.platform === 'win32' && process.arch === 'ia32'))('focus handling', () => {
+describe('focus handling', () => {
   let webviewContents: WebContents;
   let w: BrowserWindow;
 
   beforeEach(async () => {
     w = new BrowserWindow({
+      alwaysOnTop: true,
       show: true,
       webPreferences: {
         nodeIntegration: true,

filename: spec/lib/screen-helpers.ts
patch: @@ -122,6 +122,14 @@ export class ScreenCapture {
     return this._expectImpl(findPoint(this.display.size), hexColor, true);
   }
 
+  public async takeScreenshot (filePrefix: string) {
+    const frame = await this.captureFrame();
+    return await createArtifactWithRandomId(
+      (id) => `${filePrefix}-${id}.png`,
+      frame.toPNG()
+    );
+  }
+
   private async captureFrame (): Promise<NativeImage> {
     const sources = await desktopCapturer.getSources({
       types: ['screen'],
Repository: huggingface/transformers
Pull request: fix flaky `test_generate_compile_model_forward`

# What does this PR do?

Found it flaky today in https://app.circleci.com/jobs/github/huggingface/transformers/1816845

(7 failed in a run of 5000 times)

After this PR, all 5000 runs pass

Changed files:
filename: tests/generation/test_utils.py
patch: @@ -2088,14 +2088,17 @@ def test_generate_compile_model_forward(self):
 
         ⚠️ Runs two sequential generations to ensure the cache doesn't get stuck after the first compiled run! ⚠️
         """
+        set_model_tester_for_less_flaky_test(self)
         for model_class in self.all_generative_model_classes:
             # 1. Test exclusion criteria
             if not model_class._supports_static_cache:
                 self.skipTest("This model doesn't support static cache (= no expectations of compilation support)")
 
             # 2. Prepares two sets of inputs
             config, inputs_dict = self.prepare_config_and_inputs_for_generate(batch_size=4)
+            set_config_for_less_flaky_test(config)
             model = model_class(config).to(torch_device)
+            set_model_for_less_flaky_test(model)
             model.eval()  # otherwise `self.training` is `True` -- this flag is used at attn mask creation time
 
             # Some composite models have a custom generate and will call an inner model's generate -> that inner model
Repository: nodejs/node
Pull request: tools: enable CodeQL config file

A previous change designed to ignore test files in CodeQL scans had multiple problems. This fixes the CodeQL scan breakage. It adds a CodeQL config file, which allows us to ignore the test directory in our scans.

Refs: https://github.com/nodejs/node/pull/57978#issuecomment-2829182983
Refs: https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/customizing-your-advanced-setup-for-code-scanning#specifying-directories-to-scan

<!--
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: @@ -0,0 +1,4 @@
+name: CodeQL config
+
+paths-ignore:
+  - test

filename: .github/workflows/codeql.yml
patch: @@ -7,9 +7,6 @@ on:
 permissions:
   contents: read
 
-paths-to-ignore:
-  - test
-
 jobs:
   analyze:
     name: Analyze
@@ -33,6 +30,7 @@ jobs:
         uses: github/codeql-action/init@6bb031afdd8eb862ea3fc1848194185e076637e5  # v3.28.11
         with:
           languages: ${{ matrix.language }}
+          config-file: ./.github/codeql-config.yml
 
       - name: Autobuild
         uses: github/codeql-action/autobuild@6bb031afdd8eb862ea3fc1848194185e076637e5  # v3.28.11
Repository: ggml-org/llama.cpp
Pull request: graph : fix assert in memory-less build_attn

cont #15586 

This assert will be removed after the TODOs with tag `[TAG_NO_CACHE_PAD]` from #14363 are completed. For now guarantee that we are not calling this path with `equal_seqs && n_seqs_unq > 1`, which is guaranteed in the `encode()` method of `llama_context`:

https://github.com/ggml-org/llama.cpp/blob/dc7e58d2b0f392e5cb69d7bd5bd50aa0f441d685/src/llama-context.cpp#L780-L784



Changed files:
filename: src/llama-graph.cpp
patch: @@ -1376,7 +1376,7 @@ ggml_tensor * llm_graph_context::build_attn(
 
     // [TAG_NO_CACHE_PAD]
     // TODO: if ubatch.equal_seqs() == true, we can split the three tensors below into ubatch.n_seqs_unq streams
-    assert(!ubatch.equal_seqs());
+    assert(!ubatch.equal_seqs() || (k_cur->ne[3] == 1 && k_cur->ne[3] == ubatch.n_seqs_unq));
 
     ggml_tensor * q = q_cur;
     ggml_tensor * k = k_cur;
Repository: huggingface/transformers
Pull request: [smollm3] add tokenizer mapping for `smollm3`

# What does this PR do?

Adds a tokenizer mapping for `smollm3`, which fixes the following script

```py
from transformers import pipeline

# tokenizer is infered from the model, through the tokenizer mapping
pipe = pipeline("text-generation", model="HuggingFaceTB/SmolLM3-3B-Base")
pipe("Once upon a time,")
```

Changed files:
filename: src/transformers/models/auto/tokenization_auto.py
patch: @@ -580,6 +580,7 @@
                 "GemmaTokenizerFast" if is_tokenizers_available() else None,
             ),
         ),
+        ("smollm3", (None, "PreTrainedTokenizerFast" if is_tokenizers_available() else None)),
         ("speech_to_text", ("Speech2TextTokenizer" if is_sentencepiece_available() else None, None)),
         ("speech_to_text_2", ("Speech2Text2Tokenizer", None)),
         ("speecht5", ("SpeechT5Tokenizer" if is_sentencepiece_available() else None, None)),
Repository: nodejs/node
Pull request: src: fix EnvironmentOptions.async_context_frame default value

`default_is_true` in bool OptionsParser is a hint for help text. The
default value for an option is still required to be set in the option
struct.

Changed files:
filename: src/node_options.h
patch: @@ -146,7 +146,7 @@ class EnvironmentOptions : public Options {
   bool allow_worker_threads = false;
   bool experimental_repl_await = true;
   bool experimental_vm_modules = false;
-  bool async_context_frame = false;
+  bool async_context_frame = true;
   bool expose_internals = false;
   bool force_node_api_uncaught_exceptions_policy = false;
   bool frozen_intrinsics = false;
@@ -408,6 +408,10 @@ class OptionsParser {
   // These methods add a single option to the parser. Optionally, it can be
   // specified whether the option should be allowed from environment variable
   // sources (i.e. NODE_OPTIONS).
+
+  // default_is_true is only a hint in printing help text, it does not
+  // affect the default value of the option. Set the default value in the
+  // Options struct instead.
   void AddOption(const char* name,
                  const char* help_text,
                  bool Options::*field,

filename: test/parallel/test-internal-async-context-frame-disable.js
patch: @@ -0,0 +1,9 @@
+// Flags: --expose-internals --no-async-context-frame
+'use strict';
+
+require('../common');
+const assert = require('assert');
+const AsyncContextFrame = require('internal/async_context_frame');
+
+// Test that AsyncContextFrame can be disabled.
+assert(!AsyncContextFrame.enabled);

filename: test/parallel/test-internal-async-context-frame-enabled.js
patch: @@ -0,0 +1,9 @@
+// Flags: --expose-internals
+'use strict';
+
+require('../common');
+const assert = require('assert');
+const AsyncContextFrame = require('internal/async_context_frame');
+
+// Test that AsyncContextFrame is enabled by default.
+assert(AsyncContextFrame.enabled);
Repository: denoland/deno
Pull request: fix(ext/node): export test as property of default export

Currently the pattern below is not supported. This PR fixes it.

```js
const { test } = require('node:test');
```

towards #28837 

Changed files:
filename: ext/node/polyfills/testing.ts
patch: @@ -222,4 +222,6 @@ export const mock = {
   },
 };
 
+test.test = test;
+
 export default test;

filename: tests/specs/node/node_test_module/test.js
patch: @@ -13,7 +13,7 @@ test("sync pass todo", (t) => {
   t.todo();
 });
 
-test("sync pass todo with message", (t) => {
+test.test("sync pass todo with message", (t) => {
   t.todo("this is a passing todo");
 });