helloadhavan/github_issues huggingface dataset:
Repository: vuejs/vue
Pull request: fix #4041, warn overriding Vue's internal methods


Changed files:
filename: src/core/instance/state.js
patch: @@ -21,6 +21,8 @@ import {
   noop
 } from '../util/index'
 
+import BuiltinVue from '../index'
+
 export function initState (vm: Component) {
   vm._watchers = []
   initProps(vm)
@@ -143,12 +145,16 @@ function initMethods (vm: Component) {
   if (methods) {
     for (const key in methods) {
       vm[key] = methods[key] == null ? noop : bind(methods[key], vm)
-      if (process.env.NODE_ENV !== 'production' && methods[key] == null) {
-        warn(
+      if (process.env.NODE_ENV !== 'production') {
+        methods[key] == null && warn(
           `method "${key}" has an undefined value in the component definition. ` +
           `Did you reference the function correctly?`,
           vm
         )
+        hasOwn(BuiltinVue.prototype, key) && warn(
+          `Avoid overriding Vue's internal method "${key}".`,
+          vm
+        )
       }
     }
   }

filename: test/unit/features/options/methods.spec.js
patch: @@ -24,4 +24,14 @@ describe('Options methods', () => {
     })
     expect(`method "hello" has an undefined value in the component definition`).toHaveBeenWarned()
   })
+
+  it('should warn overriding builtin methods', () => {
+    new Vue({
+      methods: {
+        $emit () {
+        }
+      }
+    })
+    expect(`Avoid overriding Vue's internal method "$emit".`).toHaveBeenWarned()
+  })
 })
Repository: vuejs/vue
Pull request: Fix xhtml namespace typo


Changed files:
filename: src/platforms/web/util/element.js
patch: @@ -6,7 +6,7 @@ import { makeMap } from 'shared/util'
 export const namespaceMap = {
   svg: 'http://www.w3.org/2000/svg',
   math: 'http://www.w3.org/1998/Math/MathML',
-  xhtml: 'http://www.w3.org/1999/xhtm'
+  xhtml: 'http://www.w3.org/1999/xhtml'
 }
 
 export const isHTMLTag = makeMap(
Repository: huggingface/transformers
Pull request: Tiny doc fix

# What does this PR do?

Checked out the contributing guide, found 2 dead links

Changed files:
filename: CONTRIBUTING.md
patch: @@ -125,9 +125,9 @@ If you're contributing a **vision-language model** (or any multimodal model that
 All new models should use the modular architecture pattern. Create a `modular_<model_name>.py` file using the modular model converter:
 
 - Use the CLI, [`transformers add-new-model-like`](https://github.com/huggingface/transformers/blob/main/src/transformers/cli/add_new_model_like.py) to generate a modular skeleton and get started
-- All code should be in the modular file if possible. Modeling must be in it, it's better if configuration is in it as well. [Modular guide](./modular_transformers#implementing-a-modular-file) shows a quick way to set up a modular file.
+- All code should be in the modular file if possible. Modeling must be in it, it's better if configuration is in it as well. [Modular guide](./docs/source/en/modular_transformers.md#implementing-a-modular-file) shows a quick way to set up a modular file.
 - Reuse existing patterns from similar models as much as possible
-- You can make the model compatible with inference engines such as vLLM or SGLang, and enable zero-effort integration. See specific requirements for model implementation in ["Transformers modeling backend"](./transformers_as_backend#multimodal-models)
+- You can make the model compatible with inference engines such as vLLM or SGLang, and enable zero-effort integration. See specific requirements for model implementation in ["Transformers modeling backend"](./docs/source/en/transformers_as_backend.md#multimodal-models)
 
 To verify your modular file is correct, run:
Repository: facebook/react
Pull request: fix[DevTools]: fix HostComponent naming in filters for Native

Right now we mention DOM elements as Host elements for React Native, which doesn't make sense.

Changed files:
filename: .eslintrc.js
patch: @@ -496,6 +496,7 @@ module.exports = {
     {
       files: [
         'packages/react-devtools-extensions/**/*.js',
+        'packages/react-devtools-shared/src/devtools/views/**/*.js',
         'packages/react-devtools-shared/src/hook.js',
         'packages/react-devtools-shared/src/backend/console.js',
         'packages/react-devtools-shared/src/backend/shared/DevToolsComponentStackFrame.js',

filename: packages/react-devtools-core/webpack.standalone.js
patch: @@ -87,6 +87,7 @@ module.exports = {
       __EXTENSION__: false,
       __PROFILE__: false,
       __TEST__: NODE_ENV === 'test',
+      __IS_NATIVE__: true,
       'process.env.DEVTOOLS_PACKAGE': `"react-devtools-core"`,
       'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`,
       'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null,

filename: packages/react-devtools-fusebox/webpack.config.frontend.js
patch: @@ -82,6 +82,7 @@ module.exports = {
       __EXTENSION__: false,
       __PROFILE__: false,
       __TEST__: NODE_ENV === 'test',
+      __IS_NATIVE__: true,
       'process.env.DEVTOOLS_PACKAGE': `"react-devtools-fusebox"`,
       'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`,
       'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null,

filename: packages/react-devtools-shared/src/devtools/views/Settings/ComponentsSettings.js
patch: @@ -477,7 +477,9 @@ export default function ComponentsSettings({
                     <option value={ElementTypeFunction}>function</option>
                     <option value={ElementTypeForwardRef}>forward ref</option>
                     <option value={ElementTypeHostComponent}>
-                      dom nodes (e.g. &lt;div&gt;)
+                      {__IS_NATIVE__
+                        ? 'host components (e.g. &lt;RCTText&gt;)'
+                        : 'dom nodes (e.g. &lt;div&gt;)'}
                     </option>
                     <option value={ElementTypeMemo}>memo</option>
                     <option value={ElementTypeOtherOrUnknown}>other</option>
Repository: ggml-org/llama.cpp
Pull request: Add missing unordered_map include to jinja/value.h

On macos Sequoia 15.7.3, x86_64, the build has recently started failing with
```
In file included from .../code/cpp/llama.cpp/common/jinja/string.cpp:2:
.../code/cpp/llama.cpp/common/./jinja/value.h:478:10: error: no template named 'unordered_map' in namespace 'std'
  478 |     std::unordered_map<value, value, value_hasher, value_equivalence> unordered;
      |     ~~~~~^
In file included from .../code/cpp/llama.cpp/common/jinja/caps.cpp:1:
.../code/cpp/llama.cpp/common/jinja/value.h:478:10: error: no template named 'unordered_map' in namespace 'std'
  478 |     std::unordered_map<value, value, value_hasher, value_equivalence> unordered;
      |     ~~~~~^
In file included from .../code/cpp/llama.cpp/common/jinja/value.cpp:1:
In file included from .../code/cpp/llama.cpp/common/jinja/runtime.h:4:
.../code/cpp/llama.cpp/common/jinja/value.h:478:10: error: no template named 'unordered_map' in namespace 'std'
  478 |     std::unordered_map<value, value, value_hasher, value_equivalence> unordered;
[...]
```

After a bit of digging to make sure all the appropriate flags were used, I notifced that the necessary header was not included. This fixes the build for me and should not affect negatively other builds that for some reasons were already succeeding.

Changed files:
filename: common/jinja/value.h
patch: @@ -12,6 +12,7 @@
 #include <set>
 #include <sstream>
 #include <string>
+#include <unordered_map>
 #include <vector>
 
 namespace jinja {
Repository: ollama/ollama
Pull request: Set *.png and *.ico to be treated as binary files.

The change b732beba6 makes all files text files and sets lf as eol. This will automatically change all files to have lf if they are touched by git (e.g. via git status). This change cannot be stashed and makes it hard to work with the repo (rebase and checkout don't really work). See also #6183.

Here, we set the offending files (*.png and *.ico, but that might be more in the future) to be treated as binary files and not be changed by git.

Changed files:
filename: .gitattributes
patch: @@ -1,2 +1,3 @@
 llm/ext_server/* linguist-vendored
-* text eol=lf
+* text=auto
+*.go text eol=lf
Repository: vercel/next.js
Pull request: [test] Update snapshots

Not affecting `canary` CI because that only runs on React 18 where this test is skipped.

Changed files:
filename: test/production/next-server-nft/next-server-nft.test.ts
patch: @@ -123,8 +123,7 @@ const isReact18 = parseInt(process.env.NEXT_TEST_REACT_VERSION) === 18
          "/node_modules/client-only/*",
          "/node_modules/detect-libc/*",
          "/node_modules/next/dist/build/output/log.js",
-         "/node_modules/next/dist/build/segment-config/app/app-segment-config.js",
-         "/node_modules/next/dist/build/segment-config/app/app-segments.js",
+         "/node_modules/next/dist/build/static-paths/app/extract-pathname-route-param-segments-from-loader-tree.js",
          "/node_modules/next/dist/build/static-paths/utils.js",
          "/node_modules/next/dist/client/*",
          "/node_modules/next/dist/compiled/@edge-runtime/cookies/index.js",
@@ -163,8 +162,6 @@ const isReact18 = parseInt(process.env.NEXT_TEST_REACT_VERSION) === 18
          "/node_modules/next/dist/compiled/strip-ansi/index.js",
          "/node_modules/next/dist/compiled/superstruct/index.cjs",
          "/node_modules/next/dist/compiled/ws/index.js",
-         "/node_modules/next/dist/compiled/zod-validation-error/index.js",
-         "/node_modules/next/dist/compiled/zod/index.cjs",
          "/node_modules/next/dist/experimental/testmode/context.js",
          "/node_modules/next/dist/experimental/testmode/fetch.js",
          "/node_modules/next/dist/experimental/testmode/httpget.js",
Repository: ggml-org/llama.cpp
Pull request: add tensor type checking as part of cuda graph properties

Motivated by https://github.com/ggml-org/llama.cpp/pull/15805#issuecomment-3818986820

Changed files:
filename: ggml/src/ggml-cuda/common.cuh
patch: @@ -1124,6 +1124,7 @@ struct ggml_tensor_extra_gpu {
 struct ggml_cuda_graph_node_properties {
     void * node_data;
     ggml_op node_op;
+    enum ggml_type node_type;
     int32_t flags;
     int64_t ne[GGML_MAX_DIMS];
     size_t nb[GGML_MAX_DIMS];

filename: ggml/src/ggml-cuda/ggml-cuda.cu
patch: @@ -2920,6 +2920,7 @@ static void ggml_cuda_graph_node_set_properties(ggml_cuda_graph_node_properties
     memset(props, 0, sizeof(ggml_cuda_graph_node_properties));
     props->node_data = node->data;
     props->node_op = node->op;
+    props->node_type = node->type;
     props->flags = node->flags;
     for (int i = 0; i < GGML_MAX_DIMS; i++) {
         props->ne[i] = node->ne[i];
@@ -2944,6 +2945,10 @@ static bool ggml_cuda_graph_node_properties_match(ggml_tensor * node, ggml_cuda_
         return false;
     }
 
+    if (node->type != props->node_type) {
+        return false;
+    }
+
     for (int i = 0; i < GGML_MAX_DIMS; i++) {
         if (node->ne[i] != props->ne[i]) {
             return false;
Repository: vuejs/vue
Pull request: [1.x]Empty string should not be casted to 0 (fix #3577, #3541)

Backport the behavior from vue2.

fix #3577 #3541 . Maybe a breaking fix as discused.

Changed files:
filename: src/util/lang.js
patch: @@ -126,14 +126,13 @@ export function _toString (value) {
  */
 
 export function toNumber (value) {
-  if (typeof value !== 'string') {
+  if (typeof value !== 'string' || value.trim() === '') {
     return value
-  } else {
-    var parsed = Number(value)
-    return isNaN(parsed)
-      ? value
-      : parsed
   }
+  var parsed = Number(value)
+  return isNaN(parsed)
+    ? value
+    : parsed
 }
 
 /**
Repository: nodejs/node
Pull request: tools: remove undici from daily wpt.fyi job

The [Daily WPT report](https://github.com/nodejs/node/actions/workflows/daily-wpt-fyi.yml) job responsible for uploading Node.js WPT reports to https://wpt.fyi started failing since v25.0.0-nightly2025091229738c7b42 (last success Sep 12), now also on 24.x (last success Sep 26) and 25.x (never succeeded)

This is because undici changed its WPT fixtures location and I suspect even pre-requisites for running the WPTs.

This PR removes running undici WPTs from the daily job to resume stable daily uploads.

Refs: https://github.com/nodejs/undici/issues/4644

It is possible to re-introduce this later again albeit with a switch dependant on the undici version.

Changed files:
filename: .github/workflows/daily-wpt-fyi.yml
patch: @@ -96,31 +96,6 @@ jobs:
             echo "WPT_REPORT=$(pwd)/out/wpt/wptreport.json" >> $GITHUB_ENV
           fi
 
-      # undici WPT Runner
-      - name: Set env.UNDICI_VERSION
-        if: ${{ env.WPT_REPORT != '' }}
-        run: echo "UNDICI_VERSION=v$(jq -r '.version' < deps/undici/src/package.json)" >> $GITHUB_ENV
-      - name: Remove deps/undici
-        if: ${{ env.WPT_REPORT != '' }}
-        run: rm -rf deps/undici
-      - name: Checkout undici
-        if: ${{ env.WPT_REPORT != '' }}
-        uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8  # v5.0.0
-        with:
-          repository: nodejs/undici
-          persist-credentials: false
-          path: deps/undici
-          clean: false
-          ref: ${{ env.UNDICI_VERSION }}
-      - name: Add undici WPTs to the report
-        if: ${{ env.WPT_REPORT != '' }}
-        run: |
-          rm -rf test/wpt/tests
-          mv ../../test/fixtures/wpt/ test/wpt/tests/
-          npm install
-          npm run test:wpt || true
-        working-directory: deps/undici
-
       # Upload artifacts
       - name: Clone report for upload
         if: ${{ env.WPT_REPORT != '' }}
Repository: facebook/react
Pull request: [actions] split out compiler notify


Changed files:
filename: .github/workflows/shared_discord_notify.yml
patch: @@ -5,8 +5,8 @@ on:
     types: [labeled]
 
 jobs:
-  notify:
-    if: ${{ github.event.label.name == 'React Core Team' }}
+  notify_core:
+    if: ${{ !contains(github.event.pull_request.title, '[compiler]') && github.event.label.name == 'React Core Team' }}
     runs-on: ubuntu-latest
     steps:
       - name: Discord Webhook Action
@@ -19,3 +19,17 @@ jobs:
           embed-title: '#${{ github.event.number }} (+${{github.event.pull_request.additions}} -${{github.event.pull_request.deletions}}): ${{ github.event.pull_request.title }}'
           embed-description: ${{ github.event.pull_request.body }}
           embed-url: ${{ github.event.pull_request.html_url }}
+  notify_compiler:
+    if: ${{ contains(github.event.pull_request.title, '[compiler]') && github.event.label.name == 'React Core Team' }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Discord Webhook Action
+        uses: tsickert/discord-webhook@v6.0.0
+        with:
+          webhook-url: ${{ secrets.COMPILER_DISCORD_WEBHOOK_URL }}
+          embed-author-name: ${{ github.event.pull_request.user.login }}
+          embed-author-url: ${{ github.event.pull_request.user.html_url }}
+          embed-author-icon-url: ${{ github.event.pull_request.user.avatar_url }}
+          embed-title: '#${{ github.event.number }} (+${{github.event.pull_request.additions}} -${{github.event.pull_request.deletions}}): ${{ github.event.pull_request.title }}'
+          embed-description: ${{ github.event.pull_request.body }}
+          embed-url: ${{ github.event.pull_request.html_url }}
Repository: ggml-org/llama.cpp
Pull request: tests : add GQA=20 FA test

Might be a good idea to have a test that exercises GQA=20 in order to catch any potential regressions.

Changed files:
filename: tests/test-backend-ops.cpp
patch: @@ -8213,11 +8213,13 @@ static std::vector<std::unique_ptr<test_case>> make_test_cases_eval() {
                         if (!mask && max_bias > 0.0f) continue;
                         for (float logit_softcap : {0.0f, 10.0f}) {
                             if (hsk != 128 && logit_softcap != 0.0f) continue;
-                            for (int nh : { 4, }) {
+                            for (int nh : { 1, 4 }) {
+                                if (nh == 1 && hsk != 576) continue; // GLM 4.7 Flash
                                 for (int nr3 : { 1, 3, }) {
                                     if (hsk > 64 && nr3 > 1) continue; // skip broadcast for large head sizes
-                                    for (int nr2 : { 1, 4, 12 }) {
+                                    for (int nr2 : { 1, 4, 12, 20 }) {
                                         if (nr2 == 12 && hsk != 128) continue;
+                                        if (nr2 == 20 && (nh != 1 || hsk != 576)) continue;
                                         //for (int kv : { 1, 17, 31, 33, 61, 113, 65, 127, 129, 130, 255, 260, 371, 380, 407, 512, 1024, }) {
                                         for (int kv : { 113, 512, 1024, }) {
                                             if (nr2 != 1 && kv != 512) continue;
Repository: facebook/react
Pull request: [internal] fix console patch, add RN

The forking for `shared/ReactFeatureFlags` doesn't work in the console patches. Since they're already forked, we can import the internal ReactFeatureFlags files directly. 

Would have caught this in testing a PR sync, but the PR syncs are broken right now.

Changed files:
filename: packages/shared/forks/consoleWithStackDev.rn.js
patch: @@ -6,9 +6,15 @@
  */
 
 import ReactSharedInternals from 'shared/ReactSharedInternals';
+import * as dynamicFlagsUntyped from 'ReactNativeInternalFeatureFlags';
+const enableRemoveConsolePatches =
+  dynamicFlagsUntyped && dynamicFlagsUntyped.enableRemoveConsolePatches;
 
 let suppressWarning = false;
 export function setSuppressWarning(newSuppressWarning) {
+  if (enableRemoveConsolePatches) {
+    return;
+  }
   if (__DEV__) {
     suppressWarning = newSuppressWarning;
   }
@@ -21,22 +27,33 @@ export function setSuppressWarning(newSuppressWarning) {
 // they are left as they are instead.
 
 export function warn(format, ...args) {
-  if (__DEV__) {
+  if (enableRemoveConsolePatches) {
+    if (__DEV__) {
+      console['warn'](format, ...args);
+    }
+  } else if (__DEV__) {
     if (!suppressWarning) {
       printWarning('warn', format, args);
     }
   }
 }
 
 export function error(format, ...args) {
-  if (__DEV__) {
+  if (enableRemoveConsolePatches) {
+    if (__DEV__) {
+      console['error'](format, ...args);
+    }
+  } else if (__DEV__) {
     if (!suppressWarning) {
       printWarning('error', format, args);
     }
   }
 }
 
 function printWarning(level, format, args) {
+  if (enableRemoveConsolePatches) {
+    return;
+  }
   if (__DEV__) {
     if (ReactSharedInternals.getCurrentStack) {
       const stack = ReactSharedInternals.getCurrentStack();

filename: packages/shared/forks/consoleWithStackDev.www.js
patch: @@ -5,7 +5,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-const {enableRemoveConsolePatches} = require('shared/ReactFeatureFlags');
+const {enableRemoveConsolePatches} = require('ReactFeatureFlags');
 
 // This refers to a WWW module.
 const warningWWW = require('warning');

filename: scripts/rollup/bundles.js
patch: @@ -783,7 +783,9 @@ const bundles = [
     moduleType: RENDERER,
     entry: 'react-native-renderer',
     global: 'ReactNativeRenderer',
-    externals: ['react-native'],
+    // ReactNativeInternalFeatureFlags temporary until we land enableRemoveConsolePatches.
+    // Needs to be done before the next RN OSS release.
+    externals: ['react-native', 'ReactNativeInternalFeatureFlags'],
     minifyWithProdErrorCodes: false,
     wrapWithModuleBoundaries: true,
     babel: opts =>
@@ -817,7 +819,9 @@ const bundles = [
     moduleType: RENDERER,
     entry: 'react-native-renderer/fabric',
     global: 'ReactFabric',
-    externals: ['react-native'],
+    // ReactNativeInternalFeatureFlags temporary until we land enableRemoveConsolePatches.
+    // Needs to be done before the next RN OSS release.
+    externals: ['react-native', 'ReactNativeInternalFeatureFlags'],
     minifyWithProdErrorCodes: false,
     wrapWithModuleBoundaries: true,
     babel: opts =>
Repository: vuejs/vue
Pull request: keep consistency in dev and prod env


Changed files:
filename: src/core/global-api/extend.js
patch: @@ -22,14 +22,13 @@ export function initExtend (Vue: GlobalAPI) {
     if (isFirstExtend && extendOptions._Ctor) {
       return extendOptions._Ctor
     }
-    let name = extendOptions.name || Super.options.name
+    const name = extendOptions.name || Super.options.name
     if (process.env.NODE_ENV !== 'production') {
       if (!/^[a-zA-Z][\w-]*$/.test(name)) {
         warn(
           'Invalid component name: "' + name + '". Component names ' +
           'can only contain alphanumeric characaters and the hyphen.'
         )
-        name = null
       }
     }
     const Sub = function VueComponent (options) {

filename: test/unit/features/options/name.spec.js
patch: @@ -20,14 +20,6 @@ describe('Options name', () => {
     /* eslint-enable */
   })
 
-  it('when incorrect name given it should not contain itself in self components', () => {
-    const vm = Vue.extend({
-      name: 'Hyper*Vue'
-    })
-
-    expect(vm.options.components['Hyper*Vue']).toBeUndefined()
-  })
-
   it('id should not override given name when using Vue.component', () => {
     const SuperComponent = Vue.component('super-component', {
       name: 'SuperVue'
Repository: electron/electron
Pull request: fix: auth required websocket crash

#### Description of Change

Closes https://github.com/electron/electron/issues/48505.

Fixes an issue where authentication via websockets can crash due to `NOTREACHED` being a CHECK since M120.

#### 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: Fixes an issue where authentication via websockets can crash.

Changed files:
filename: shell/browser/net/proxying_websocket.cc
patch: @@ -401,7 +401,7 @@ void ProxyingWebSocket::OnHeadersReceivedCompleteForAuth(
 
   auto continuation = base::BindRepeating(
       &ProxyingWebSocket::OnAuthRequiredComplete, weak_factory_.GetWeakPtr());
-  auto auth_rv = AuthRequiredResponse::kIoPending;
+  auto auth_rv = AuthRequiredResponse::kCancelAuth;
   PauseIncomingMethodCallProcessing();
 
   OnAuthRequiredComplete(auth_rv);
Repository: ollama/ollama
Pull request: Don't hard fail on sparse setup error

It seems this can fail in some casees, but proceed with the download anyway.

I haven't reproduced the users failure, but based on the log message and the recent regression, this seems highly likely to be the cause.

Fixes #6263 

Changed files:
filename: server/download.go
patch: @@ -216,9 +216,7 @@ func (b *blobDownload) run(ctx context.Context, requestURL *url.URL, opts *regis
 		return err
 	}
 	defer file.Close()
-	if err := setSparse(file); err != nil {
-		return err
-	}
+	setSparse(file)
 
 	_ = file.Truncate(b.Total)
 

filename: server/sparse_common.go
patch: @@ -4,6 +4,5 @@ package server
 
 import "os"
 
-func setSparse(file *os.File) error {
-	return nil
+func setSparse(*os.File) {
 }

filename: server/sparse_windows.go
patch: @@ -6,8 +6,9 @@ import (
 	"golang.org/x/sys/windows"
 )
 
-func setSparse(file *os.File) error {
-	return windows.DeviceIoControl(
+func setSparse(file *os.File) {
+	// exFat (and other FS types) don't support sparse files, so ignore errors
+	windows.DeviceIoControl( //nolint:errcheck
 		windows.Handle(file.Fd()), windows.FSCTL_SET_SPARSE,
 		nil, 0,
 		nil, 0,