helloadhavan/github_issues huggingface dataset:
Repository: rust-lang/rust
Pull request: Add regression test for #154189

<!-- homu-ignore:start -->
<!--
If this PR is related to an unstable feature or an otherwise tracked effort,
please link to the relevant tracking issue here. If you don't know of a related
tracking issue or there are none, feel free to ignore this.

This PR will get automatically assigned to a reviewer. In case you would like
a specific user to review your work, you can assign it to them by using

    r? <reviewer name>
-->
<!-- homu-ignore:end -->

Fixes rust-lang/rust#154189


Changed files:
filename: tests/ui/coherence/generalize-associated-type-alias.rs
patch: @@ -0,0 +1,22 @@
+// Regression test for https://github.com/rust-lang/rust/issues/154189.
+#![feature(unboxed_closures)]
+
+trait ToUnit<'a> {
+    type Unit;
+}
+
+impl ToUnit<'_> for *const u32 {
+    type Unit = ();
+}
+
+trait Overlap<T> {}
+
+type Assoc<'a, T> = <*const T as ToUnit<'a>>::Unit;
+
+impl<T> Overlap<T> for T {}
+
+impl<'a, T> Overlap<(&'a (), Assoc<'a, T>)> for T {}
+//~^ ERROR the trait bound `*const T: ToUnit<'a>` is not satisfied
+//~| ERROR the trait bound `T: Overlap<(&'a (), _)>` is not satisfied
+
+fn main() {}

filename: tests/ui/coherence/generalize-associated-type-alias.stderr
patch: @@ -0,0 +1,28 @@
+error[E0277]: the trait bound `*const T: ToUnit<'a>` is not satisfied
+  --> $DIR/generalize-associated-type-alias.rs:18:13
+   |
+LL | impl<'a, T> Overlap<(&'a (), Assoc<'a, T>)> for T {}
+   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `ToUnit<'a>` is not implemented for `*const T`
+   |
+help: consider introducing a `where` clause, but there might be an alternative better way to express this requirement
+   |
+LL | impl<'a, T> Overlap<(&'a (), Assoc<'a, T>)> for T where *const T: ToUnit<'a> {}
+   |                                                   ++++++++++++++++++++++++++
+
+error[E0277]: the trait bound `T: Overlap<(&'a (), _)>` is not satisfied
+  --> $DIR/generalize-associated-type-alias.rs:18:49
+   |
+LL | impl<'a, T> Overlap<(&'a (), Assoc<'a, T>)> for T {}
+   |                                                 ^ the trait `Overlap<(&'a (), _)>` is not implemented for `T`
+   |
+help: the trait `Overlap<(&'a (), _)>` is not implemented for `T`
+      but trait `Overlap<(&(), ())>` is implemented for `u32`
+  --> $DIR/generalize-associated-type-alias.rs:18:1
+   |
+LL | impl<'a, T> Overlap<(&'a (), Assoc<'a, T>)> for T {}
+   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+   = help: for that trait implementation, expected `u32`, found `T`
+
+error: aborting due to 2 previous errors
+
+For more information about this error, try `rustc --explain E0277`.
Repository: nodejs/node
Pull request: tools: fix example in release proposal linter

`changelog-maker` [writes 10 character length short form SHA's](https://github.com/nodejs/changelog-maker/blob/44e442d3cc503cc94485e07d30d451761bb6ea3c/commit-to-output.js#L82). Explicitly set the `git log format` in the example to match.

<!--
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: tools/actions/lint-release-proposal-commit-list.mjs
patch: @@ -6,6 +6,7 @@
 // Example:
 // $ git log upstream/vXX.x...upstream/vX.X.X-proposal \
 //     --reverse --format='{"prURL":"%(trailers:key=PR-URL,valueonly,separator=)","title":"%s","smallSha":"%h"}' \
+//     --abbrev=10 \
 //   | sed 's/,"title":"Revert "\([^"]\+\)""/,"title":"Revert \\"\1\\""/g' \
 //   | ./lint-release-proposal-commit-list.mjs "path/to/CHANGELOG.md" "$(git rev-parse upstream/vX.X.X-proposal)"
Repository: ollama/ollama
Pull request: models: enable deepseek2 (deepseek v3.1 w/ MLA) on the new engine


Changed files:
filename: fs/ggml/ggml.go
patch: @@ -250,6 +250,7 @@ func (kv KV) OllamaEngineRequired() bool {
 		"qwen3", "qwen3moe",
 		"qwen3vl", "qwen3vlmoe",
 		"deepseekocr",
+		"deepseek2",
 	}, kv.Architecture())
 }
 

filename: model/models/deepseek2/model.go
patch: @@ -236,6 +236,11 @@ type Model struct {
 }
 
 func New(c fs.Config) (model.Model, error) {
+	if c.Uint("attention.key_length_mla") == 0 {
+		// non-MLA models aren't yet supported
+		return nil, model.ErrUnsupportedModel
+	}
+
 	layers := make([]Layer, c.Uint("block_count"))
 
 	firstDenseLayerIndex := int(c.Uint("leading_dense_block_count"))
Repository: facebook/react
Pull request: [DevTools] Title color tweak

<img width="521" height="365" alt="Screenshot 2025-10-20 at 11 53 50 AM" src="https://github.com/user-attachments/assets/1a073c09-d440-4498-b2b3-c0dcb2272c96" />


Changed files:
filename: packages/react-devtools-shared/src/devtools/views/SuspenseTab/SuspenseRects.css
patch: @@ -41,17 +41,18 @@
 
 .SuspenseRectsTitle {
   pointer-events: none;
-  color: var(--color-text);
+  color: color-mix(in srgb, var(--color-suspense) 50%, var(--color-text));
   overflow: hidden;
   text-overflow: ellipsis;
+  font-family: var(--font-family-sans);
   font-size: var(--font-size-sans-small);
-  line-height: var(--font-size-sans-small);
-  padding: .25rem;
+  line-height: var(--line-height-data);
+  padding: 0 .25rem;
   container-type: size;
   container-name: title;
 }
 
-@container title (width < 30px) or (height < 12px) {
+@container title (width < 30px) or (height < 18px) {
   .SuspenseRectsTitle > span {
     display: none;
   }
Repository: vercel/next.js
Pull request: Turbopack: Annotate ReadVcFuture as must_use instead of annotating functions that return it

It's a little less error-prone if it's in one place (on the type) than in many (on the functions). I also added it to `ReadRawVcFuture`.

https://doc.rust-lang.org/reference/attributes/diagnostics.html#the-must_use-attribute

- **Are these equivalent?** Yes.
- **The** **`Future`** **trait already has this annotation, why do we need it on the type?** The annotation on the trait only applies if the function returns `impl Future` or `dyn Future`. Our functions return the named type.

Changed files:
filename: turbopack/crates/turbo-tasks/src/raw_vc.rs
patch: @@ -316,6 +316,7 @@ impl<F: Future> Future for SuppressTopLevelTaskCheckFuture<F> {
     }
 }
 
+#[must_use]
 pub struct ReadRawVcFuture {
     current: RawVc,
     read_output_options: ReadOutputOptions,

filename: turbopack/crates/turbo-tasks/src/vc/mod.rs
patch: @@ -566,7 +566,6 @@ where
 {
     /// Do not use this: Use [`OperationVc::read_strongly_consistent`] instead.
     #[cfg(feature = "non_operation_vc_strongly_consistent")]
-    #[must_use]
     pub fn strongly_consistent(self) -> ReadVcFuture<T> {
         self.node
             .into_read(T::has_serialization())
@@ -576,7 +575,6 @@ where
 
     /// Returns a untracked read of the value. This will not invalidate the current function when
     /// the read value changed.
-    #[must_use]
     pub fn untracked(self) -> ReadVcFuture<T> {
         self.node
             .into_read(T::has_serialization())
@@ -586,7 +584,6 @@ where
 
     /// Read the value with the hint that this is the final read of the value. This might drop the
     /// cell content. Future reads might need to recompute the value.
-    #[must_use]
     pub fn final_read_hint(self) -> ReadVcFuture<T> {
         self.node
             .into_read(T::has_serialization())

filename: turbopack/crates/turbo-tasks/src/vc/operation.rs
patch: @@ -130,7 +130,6 @@ impl<T: ?Sized> OperationVc<T> {
     /// consistent][crate::ReadConsistency::Strong] read of the value.
     ///
     /// This ensures that all internal tasks are finished before the read is returned.
-    #[must_use]
     pub fn read_strongly_consistent(self) -> ReadVcFuture<T>
     where
         T: VcValueType,

filename: turbopack/crates/turbo-tasks/src/vc/read.rs
patch: @@ -144,6 +144,7 @@ where
     }
 }
 
+#[must_use]
 pub struct ReadVcFuture<T, Cast = VcValueTypeCast<T>>
 where
     T: ?Sized,
Repository: facebook/react
Pull request: [DevTools] Text layout fixes for stack traces with badges

The badge wasn't vertically centerred and the gap didn't match the gap in the Owner Stack. The added space after `@` also had no effect with badges. The spacing was due to hardcoded 2px padding not an actual space.

Before:
<img width="482" height="252" alt="CleanShot 2025-10-20 at 19 21 44@2x" src="https://github.com/user-attachments/assets/4f6dcb06-957b-47f5-bf9d-229131991ec4" />

After:

<img width="470" height="246" alt="CleanShot 2025-10-20 at 19 22 59@2x" src="https://github.com/user-attachments/assets/e39a487d-eea6-4891-bd06-e035e9fa0b8a" />



Changed files:
filename: packages/react-devtools-shared/src/devtools/views/Components/StackTraceView.css
patch: @@ -3,8 +3,9 @@
 }
 
 .CallSite, .BuiltInCallSite {
-  display: block;
+  display: flex;
   padding-left: 1rem;
+  white-space-collapse: preserve;
 }
 
 .IgnoredCallSite, .BuiltInCallSite {
@@ -20,13 +21,15 @@
   white-space: pre;
   overflow: hidden;
   text-overflow: ellipsis;
-  flex: 1;
   cursor: pointer;
   border-radius: 0.125rem;
-  padding: 0px 2px;
 }
 
 .Link:hover {
   background-color: var(--color-background-hover);
 }
 
+.ElementBadges {
+  margin-left: 0.25rem;
+}
+

filename: packages/react-devtools-shared/src/devtools/views/Components/StackTraceView.js
patch: @@ -86,8 +86,10 @@ export function CallSiteView({
           </span>
         </>
       )}
-
-      <ElementBadges environmentName={environmentName} />
+      <ElementBadges
+        className={styles.ElementBadges}
+        environmentName={environmentName}
+      />
     </div>
   );
 }
Repository: facebook/react
Pull request: Fix react-tools module

I messed this up pretty badly and didn't include react _at all_.

Test Plan: npm pack && npm install <packed.tgz>, then require('react-tools')`

Fixes #10


Changed files:
filename: main.js
patch: @@ -1,6 +1,6 @@
 'use strict';
 
-var React = require('./build/React');
+var React = require('./build/react');
 var visitors = require('./vendor/fbtransform/visitors').transformVisitors;
 var transform = require('./vendor/fbtransform/lib/transform').transform;
 

filename: package.json
patch: @@ -16,7 +16,7 @@
     "README.md",
     "main.js",
     "bin/jsx",
-    "build/React.js",
+    "build/react.js",
     "vendor/fbtransform/",
     "vendor/woodchipper.js"
   ],
Repository: vercel/next.js
Pull request: Prefix pr-status replies with :robot: emoji

### What?

Prefixes replies posted by `scripts/pr-status.js` with the `:robot:` (🤖) emoji.

### Why?

When the pr-status script replies to PR review threads, there's no visual indicator that the reply was generated by AI rather than a human. Adding the robot emoji makes it immediately clear.

### How?

One-line change in the `replyToThread()` function in `scripts/pr-status.js` that prepends `:robot: ` to the reply body before posting it via the GitHub GraphQL API.

Changed files:
filename: scripts/pr-status.js
patch: @@ -372,6 +372,7 @@ function getPRComments(prNumber) {
 // ============================================================================
 
 function replyToThread(threadId, body) {
+  body = ':robot: ' + body
   const mutation = `
     mutation($threadId: ID!, $body: String!) {
       addPullRequestReviewThreadReply(input: {
Repository: nodejs/node
Pull request: doc: fix markdown for `expectFailure` values

Currently, about half the test API doc is broken (text outside of codeblocks below here is blank).

<img width="917" height="259" alt="2026-03-04_11-35 test api doc broken link" src="https://github.com/user-attachments/assets/3018f976-399a-4e00-b6fc-be2737053733" />


Changed files:
filename: doc/api/test.md
patch: @@ -287,11 +287,7 @@ it('should do the thing', { expectFailure: 'feature not implemented' }, () => {
 });
 ```
 
-If the value of `expectFailure` is a
-[<RegExp>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp) |
-[<Function>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function) |
-[<Object>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object) |
-[<Error>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error),
+If the value of `expectFailure` is a {RegExp|Function|Object|Error}
 the tests will pass only if they throw a matching value.
 See [`assert.throws`][] for how each value type is handled.
 
@@ -1734,11 +1730,7 @@ changes:
     **Default:** `false`.
   * `expectFailure` {boolean|string|RegExp|Function|Object|Error} If truthy, the
     test is expected to fail. If a non-empty string is provided, that string is displayed
-    in the test results as the reason why the test is expected to fail. If a
-    [<RegExp>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp),
-    [<Function>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function),
-    [<Object>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object), or
-    [<Error>](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error)
+    in the test results as the reason why the test is expected to fail. If a {RegExp|Function|Object|Error}
     is provided directly (without wrapping in `{ match: … }`), the test passes
     only if the thrown error matches, following the behavior of
     [`assert.throws`][]. To provide both a reason and validation, pass an object
Repository: electron/electron
Pull request: build: remove redundant bits of ncrypto node patch

#### Description of Change

As in title. No longer necessary.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included
- [x] I have built and tested this PR
- [x] `npm test` passes(https://github.com/electron/electron/blob/main/docs/development/style-guide.md)
- [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: patches/node/fix_handle_boringssl_and_openssl_incompatibilities.patch
patch: @@ -17,7 +17,7 @@ Upstreams:
 - https://github.com/nodejs/node/pull/39136
 
 diff --git a/deps/ncrypto/ncrypto.cc b/deps/ncrypto/ncrypto.cc
-index 461819ce0fa732048e4365c40a86ef55d984c35f..fa55c980a9c4f373723a867fd41276d67b0b9413 100644
+index 461819ce0fa732048e4365c40a86ef55d984c35f..f1c85e94cf526d0255f47c003664680d26413ec3 100644
 --- a/deps/ncrypto/ncrypto.cc
 +++ b/deps/ncrypto/ncrypto.cc
 @@ -11,6 +11,7 @@
@@ -28,38 +28,6 @@ index 461819ce0fa732048e4365c40a86ef55d984c35f..fa55c980a9c4f373723a867fd41276d6
  #if OPENSSL_VERSION_MAJOR >= 3
  #include <openssl/core_names.h>
  #include <openssl/params.h>
-@@ -1130,7 +1131,9 @@ int64_t X509View::getValidToTime() const {
-   return tp;
- #else
-   struct tm tp;
--  ASN1_TIME_to_tm(X509_get0_notAfter(cert_), &tp);
-+#ifndef OPENSSL_IS_BORINGSSL
-+   ASN1_TIME_to_tm(X509_get0_notAfter(cert_), &tp);
-+#endif
-   return PortableTimeGM(&tp);
- #endif
- }
-@@ -1142,7 +1145,9 @@ int64_t X509View::getValidFromTime() const {
-   return tp;
- #else
-   struct tm tp;
-+#ifndef OPENSSL_IS_BORINGSSL
-   ASN1_TIME_to_tm(X509_get0_notBefore(cert_), &tp);
-+#endif
-   return PortableTimeGM(&tp);
- #endif
- }
-@@ -2886,10 +2891,6 @@ std::optional<uint32_t> SSLPointer::verifyPeerCertificate() const {
- const char* SSLPointer::getClientHelloAlpn() const {
-   if (ssl_ == nullptr) return {};
- #ifndef OPENSSL_IS_BORINGSSL
--  const unsigned char* buf;
--  size_t len;
--  size_t rem;
--
-   if (!SSL_client_hello_get0_ext(
-           get(),
-           TLSEXT_TYPE_application_layer_protocol_negotiation,
 @@ -3090,9 +3091,11 @@ const Cipher Cipher::AES_256_GCM = Cipher::FromNid(NID_aes_256_gcm);
  const Cipher Cipher::AES_128_KW = Cipher::FromNid(NID_id_aes128_wrap);
  const Cipher Cipher::AES_192_KW = Cipher::FromNid(NID_id_aes192_wrap);
Repository: ollama/ollama
Pull request: win: exit instead of abort

Calling abort on windows may trigger the C++ runtime to attempt a debugger attach, which causes the crashed runners to hang instead of exit, leading to a timeout instead of a fast failure during discovery.

Fixes #12264 

Ref: https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/abort?view=msvc-170


Changed files:
filename: llama/patches/0036-win-exit-instead-of-abort.patch
patch: @@ -0,0 +1,28 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Daniel Hiltgen <daniel@ollama.com>
+Date: Tue, 18 Nov 2025 09:58:23 -0800
+Subject: [PATCH] win: exit instead of abort
+
+---
+ ggml/src/ggml.c | 7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/ggml/src/ggml.c b/ggml/src/ggml.c
+index 9be35c1be..923c33d05 100644
+--- a/ggml/src/ggml.c
++++ b/ggml/src/ggml.c
+@@ -229,8 +229,13 @@ void ggml_abort(const char * file, int line, const char * fmt, ...) {
+         fprintf(stderr, "%s\n", message);
+         ggml_print_backtrace();
+     }
+-
++#if defined(_WIN32)
++    fflush(stderr);
++    fflush(stdout);
++    exit(1);
++#else
+     abort();
++#endif
+ }
+ 
+ // ggml_print_backtrace is registered with std::set_terminate by ggml.cpp

filename: ml/backend/ggml/ggml/src/ggml.c
patch: @@ -229,8 +229,13 @@ void ggml_abort(const char * file, int line, const char * fmt, ...) {
         fprintf(stderr, "%s\n", message);
         ggml_print_backtrace();
     }
-
+#if defined(_WIN32)
+    fflush(stderr);
+    fflush(stdout);
+    exit(1);
+#else
     abort();
+#endif
 }
 
 // ggml_print_backtrace is registered with std::set_terminate by ggml.cpp
Repository: rust-lang/rust
Pull request: Migrate UI tests

In this pull request, I am migrating the following files and adding a comment at the top, including a link to the issue they were regression tests for:
- `tests/ui/issues/issue-4735.rs` ➝ `tests/ui/drop/drop-noncopyable-raw-pointer.rs`
- `tests/ui/issues/issue-17734.rs` ➝ `tests/ui/codegen/box-str-drop-glue.rs`

r? Kivooeo

Changed files:
filename: tests/ui/codegen/box-str-drop-glue.rs
patch: @@ -1,3 +1,4 @@
+//! regression test for <https://github.com/rust-lang/rust/issues/17734>
 //@ run-pass
 // Test that generating drop glue for Box<str> doesn't ICE
 

filename: tests/ui/drop/drop-noncopyable-raw-pointer.rs
patch: @@ -1,3 +1,4 @@
+//! regression test for <https://github.com/rust-lang/rust/issues/4735>
 //@ run-pass
 
 use std::mem::transmute;
Repository: electron/electron
Pull request: build: remove redundant bits of ncrypto node patch

Backport of #50252

See that PR for details.


Notes: none


Changed files:
filename: patches/node/fix_handle_boringssl_and_openssl_incompatibilities.patch
patch: @@ -17,7 +17,7 @@ Upstreams:
 - https://github.com/nodejs/node/pull/39136
 
 diff --git a/deps/ncrypto/ncrypto.cc b/deps/ncrypto/ncrypto.cc
-index 461819ce0fa732048e4365c40a86ef55d984c35f..fa55c980a9c4f373723a867fd41276d67b0b9413 100644
+index 461819ce0fa732048e4365c40a86ef55d984c35f..f1c85e94cf526d0255f47c003664680d26413ec3 100644
 --- a/deps/ncrypto/ncrypto.cc
 +++ b/deps/ncrypto/ncrypto.cc
 @@ -11,6 +11,7 @@
@@ -28,38 +28,6 @@ index 461819ce0fa732048e4365c40a86ef55d984c35f..fa55c980a9c4f373723a867fd41276d6
  #if OPENSSL_VERSION_MAJOR >= 3
  #include <openssl/core_names.h>
  #include <openssl/params.h>
-@@ -1130,7 +1131,9 @@ int64_t X509View::getValidToTime() const {
-   return tp;
- #else
-   struct tm tp;
--  ASN1_TIME_to_tm(X509_get0_notAfter(cert_), &tp);
-+#ifndef OPENSSL_IS_BORINGSSL
-+   ASN1_TIME_to_tm(X509_get0_notAfter(cert_), &tp);
-+#endif
-   return PortableTimeGM(&tp);
- #endif
- }
-@@ -1142,7 +1145,9 @@ int64_t X509View::getValidFromTime() const {
-   return tp;
- #else
-   struct tm tp;
-+#ifndef OPENSSL_IS_BORINGSSL
-   ASN1_TIME_to_tm(X509_get0_notBefore(cert_), &tp);
-+#endif
-   return PortableTimeGM(&tp);
- #endif
- }
-@@ -2886,10 +2891,6 @@ std::optional<uint32_t> SSLPointer::verifyPeerCertificate() const {
- const char* SSLPointer::getClientHelloAlpn() const {
-   if (ssl_ == nullptr) return {};
- #ifndef OPENSSL_IS_BORINGSSL
--  const unsigned char* buf;
--  size_t len;
--  size_t rem;
--
-   if (!SSL_client_hello_get0_ext(
-           get(),
-           TLSEXT_TYPE_application_layer_protocol_negotiation,
 @@ -3090,9 +3091,11 @@ const Cipher Cipher::AES_256_GCM = Cipher::FromNid(NID_aes_256_gcm);
  const Cipher Cipher::AES_128_KW = Cipher::FromNid(NID_id_aes128_wrap);
  const Cipher Cipher::AES_192_KW = Cipher::FromNid(NID_id_aes192_wrap);
Repository: huggingface/transformers
Pull request: Supplement skip logic for XPU in the CPU-only tp tests

Per the title. Supplement skip logic for XPU in the CPU-only tp tests.

Hi @IlyasMoutawwakil, please help review, thanks!

Changed files:
filename: tests/test_tensor_parallel_mixin.py
patch: @@ -387,8 +387,8 @@ def _skip_if_not_supported(self):
         if not is_torch_greater_or_equal("2.9"):
             self.skipTest("Tensor parallel tests require torch >= 2.9")
 
-        if torch.cuda.is_available():
-            self.skipTest("Tensor parallel mixin tests are CPU-only and should not run on GPU machines")
+        if torch.cuda.is_available() or torch.xpu.is_available():
+            self.skipTest("Tensor parallel mixin tests are CPU-only and should not run on GPU or XPU machines")
 
         if os.cpu_count() < self.tensor_parallel_size:
             self.skipTest(