helloadhavan/github_issues huggingface dataset:
Repository: electron/electron
Pull request: test: run script to help target discovery and reduce flakes

#### Description of Change

Breaking out a potential flaky test fix from https://github.com/electron/electron/pull/44665

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [ ] PR description included and stakeholders cc'd
- [ ] `npm test` passes
- [ ] tests are [changed or added](https://github.com/electron/electron/blob/main/docs/development/testing.md)
- [ ] relevant documentation, tutorials, templates and examples are changed or added
- [ ] [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: spec/api-debugger-spec.ts
patch: @@ -227,20 +227,25 @@ describe('debugger module', () => {
     it('creates unique session id for each target', (done) => {
       w.webContents.loadFile(path.join(__dirname, 'fixtures', 'sub-frames', 'debug-frames.html'));
       w.webContents.debugger.attach();
-      let session: String;
+      let debuggerSessionId: string;
 
       w.webContents.debugger.on('message', (event, ...args) => {
         const [method, params, sessionId] = args;
         if (method === 'Target.targetCreated') {
           w.webContents.debugger.sendCommand('Target.attachToTarget', { targetId: params.targetInfo.targetId, flatten: true }).then(result => {
-            session = result.sessionId;
+            debuggerSessionId = result.sessionId;
             w.webContents.debugger.sendCommand('Debugger.enable', {}, result.sessionId);
+
+            // Ensure debugger finds a script to pause to possibly reduce flaky
+            // tests.
+            w.webContents.mainFrame.executeJavaScript('void 0;');
           });
         }
         if (method === 'Debugger.scriptParsed') {
-          expect(sessionId).to.equal(session);
-          w.webContents.debugger.detach();
-          done();
+          if (sessionId === debuggerSessionId) {
+            w.webContents.debugger.detach();
+            done();
+          }
         }
       });
       w.webContents.debugger.sendCommand('Target.setDiscoverTargets', { discover: true });
Repository: denoland/deno
Pull request: fix(node): Add missing `inspector/promises`

Add missing `inspector/promises` in node builtin module list, that causes types checking error

<img width="713" alt="Screenshot 2024-12-29 at 20 05 30" src="https://github.com/user-attachments/assets/4024ff14-bff6-4369-b6c2-d6bec285ad23" />


Changed files:
filename: cli/tsc/00_typescript.js
patch: @@ -136063,6 +136063,7 @@ var unprefixedNodeCoreModuleList = [
   "https",
   "http2",
   "inspector",
+  "inspector/promises",
   "module",
   "net",
   "os",

filename: ext/node/polyfill.rs
patch: @@ -57,6 +57,7 @@ generate_builtin_node_module_lists! {
   "http2",
   "https",
   "inspector",
+  "inspector/promises",
   "module",
   "net",
   "os",

filename: tests/integration/lsp_tests.rs
patch: @@ -8712,6 +8712,7 @@ fn lsp_completions_node_specifier() {
       "node:http2",
       "node:https",
       "node:inspector",
+      "node:inspector/promises",
       "node:module",
       "node:net",
       "node:os",

filename: tests/unit_node/inspector_test.ts
patch: @@ -1,5 +1,8 @@
 // Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
 import inspector, { Session } from "node:inspector";
+import inspectorPromises, {
+  Session as SessionPromise,
+} from "node:inspector/promises";
 import { assertEquals } from "@std/assert/equals";
 
 Deno.test("[node/inspector] - importing inspector works", () => {
@@ -9,3 +12,11 @@ Deno.test("[node/inspector] - importing inspector works", () => {
 Deno.test("[node/inspector] - Session constructor should not throw", () => {
   new Session();
 });
+
+Deno.test("[node/inspector/promises] - importing inspector works", () => {
+  assertEquals(typeof inspectorPromises.open, "function");
+});
+
+Deno.test("[node/inspector/promises] - Session constructor should not throw", () => {
+  new SessionPromise();
+});
Repository: vercel/next.js
Pull request: fix(turbopack): Fix config caching for turbopack + React Compiler

### What?

Modify the caching logic to include the whole filename when compiling with the React Compiler.

### Why?

When RC is enabled, we cannot cache the config merely using the extension.



Closes PACK-4837

Changed files:
filename: packages/next/src/build/babel/loader/get-config.ts
patch: @@ -37,27 +37,31 @@ interface CharacteristicsGermaneToCaching {
   isPageFile: boolean
   isNextDist: boolean
   hasModuleExports: boolean
-  fileExt: string
+  fileNameOrExt: string
 }
 
 const fileExtensionRegex = /\.([a-z]+)$/
 function getCacheCharacteristics(
   loaderOptions: NextBabelLoaderOptions,
   source: string,
-  filename: string
+  filename: string,
+  transformMode: 'default' | 'standalone'
 ): CharacteristicsGermaneToCaching {
   const { isServer, pagesDir } = loaderOptions
   const isPageFile = filename.startsWith(pagesDir)
   const isNextDist = nextDistPath.test(filename)
   const hasModuleExports = source.indexOf('module.exports') !== -1
-  const fileExt = fileExtensionRegex.exec(filename)?.[1] || 'unknown'
+  const fileNameOrExt =
+    transformMode === 'default'
+      ? fileExtensionRegex.exec(filename)?.[1] || 'unknown'
+      : filename
 
   return {
     isServer,
     isPageFile,
     isNextDist,
     hasModuleExports,
-    fileExt,
+    fileNameOrExt,
   }
 }
 
@@ -436,7 +440,7 @@ async function getFreshConfig(
  * file attributes and Next.js compiler states: `CharacteristicsGermaneToCaching`.
  */
 function getCacheKey(cacheCharacteristics: CharacteristicsGermaneToCaching) {
-  const { isServer, isPageFile, isNextDist, hasModuleExports, fileExt } =
+  const { isServer, isPageFile, isNextDist, hasModuleExports, fileNameOrExt } =
     cacheCharacteristics
 
   const flags =
@@ -446,7 +450,7 @@ function getCacheKey(cacheCharacteristics: CharacteristicsGermaneToCaching) {
     (isNextDist ? 0b0100 : 0) |
     (hasModuleExports ? 0b1000 : 0)
 
-  return fileExt + flags
+  return fileNameOrExt + flags
 }
 
 type BabelConfig = any
@@ -472,7 +476,8 @@ export default async function getConfig(
   const cacheCharacteristics = getCacheCharacteristics(
     loaderOptions,
     source,
-    filename
+    filename,
+    loaderOptions.transformMode
   )
 
   if (loaderOptions.transformMode === 'default' && loaderOptions.configFile) {
Repository: facebook/react
Pull request: [Internal tests] Close MessageChannel port to prevent leak

Node's MessageChannel implementation will leak if you don't explicitly close the port. This updates the enqueueTask function we use in our internal testing helpers to close the port once it's no longer needed.

Changed files:
filename: packages/internal-test-utils/enqueueTask.js
patch: @@ -11,6 +11,9 @@ const {MessageChannel} = require('node:worker_threads');
 
 export default function enqueueTask(task: () => void): void {
   const channel = new MessageChannel();
-  channel.port1.onmessage = task;
+  channel.port1.onmessage = () => {
+    channel.port1.close();
+    task();
+  };
   channel.port2.postMessage(undefined);
 }
Repository: facebook/react
Pull request: Support Iterables in Flight

We support any super type of anything that we can serialize. Meaning that as long as the Type that's passed through is less precise, it means that we can encoded it as any subtype and therefore the incoming type doesn't have to be the subtype in that case. Basically, as long as you're only passing through an `Iterable<T>` in TypeScript, then you can pass any `Iterable<T>` and we'll treat it as an array.

For example we support Promises *and* Thenables but both are encoded as Promises.

We support Arrays and since Arrays are also Iterables, we can support Iterables.

For @wongmjane

Changed files:
filename: packages/react-client/src/__tests__/ReactFlight-test.js
patch: @@ -169,6 +169,34 @@ describe('ReactFlight', () => {
     expect(ReactNoop).toMatchRenderedOutput(<span>Hello, Seb Smith</span>);
   });
 
+  it('can render an iterable as an array', async () => {
+    function ItemListClient(props) {
+      return <span>{props.items}</span>;
+    }
+    const ItemList = clientReference(ItemListClient);
+
+    function Items() {
+      const iterable = {
+        [Symbol.iterator]: function* () {
+          yield 'A';
+          yield 'B';
+          yield 'C';
+        },
+      };
+      return <ItemList items={iterable} />;
+    }
+
+    const model = <Items />;
+
+    const transport = ReactNoopFlightServer.render(model);
+
+    await act(async () => {
+      ReactNoop.render(await ReactNoopFlightClient.read(transport));
+    });
+
+    expect(ReactNoop).toMatchRenderedOutput(<span>ABC</span>);
+  });
+
   it('can render a lazy component as a shared component on the server', async () => {
     function SharedComponent({text}) {
       return (

filename: packages/react-server/src/ReactFlightServer.js
patch: @@ -73,6 +73,7 @@ import {
 } from './ReactFlightNewContext';
 
 import {
+  getIteratorFn,
   REACT_ELEMENT_TYPE,
   REACT_FORWARD_REF_TYPE,
   REACT_FRAGMENT_TYPE,
@@ -1059,6 +1060,12 @@ export function resolveModelToJSON(
       }
       return (undefined: any);
     }
+    if (!isArray(value)) {
+      const iteratorFn = getIteratorFn(value);
+      if (iteratorFn) {
+        return Array.from((value: any));
+      }
+    }
 
     if (__DEV__) {
       if (value !== null && !isArray(value)) {
Repository: huggingface/transformers
Pull request: build: :pushpin: Remove upper bound on PyTorch

# What does this PR do?
Removes the upper bound on the PyTorch dependency, as the original issue in PyTorch (https://github.com/pytorch/pytorch/issues/152275) has been resolved and released.

Fixes # (issue)
Fixes https://github.com/huggingface/transformers/issues/37824

## Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [X] Did you read the [contributor guideline](https://github.com/huggingface/transformers/blob/main/CONTRIBUTING.md#create-a-pull-request),
      Pull Request section?
- [ ] Was this discussed/approved via a Github issue or the [forum](https://discuss.huggingface.co/)? Please add a link
      to it if that's the case.
- [ ] Did you make sure to update the documentation with your changes? Here are the
      [documentation guidelines](https://github.com/huggingface/transformers/tree/main/docs), and
      [here are tips on formatting docstrings](https://github.com/huggingface/transformers/tree/main/docs#writing-source-documentation).
- [ ] Did you write any new necessary tests?


## Who can review?
@anijain2305 


Changed files:
filename: setup.py
patch: @@ -189,7 +189,7 @@
     "tiktoken",
     "timm<=1.0.11",
     "tokenizers>=0.21,<0.22",
-    "torch>=2.1,<2.7",  # Installing torch 2.7 results in slower compiled LLMs. Pinned while we investigate.
+    "torch>=2.1",
     "torchaudio",
     "torchvision",
     "pyctcdecode>=0.4.0",

filename: src/transformers/dependency_versions_table.py
patch: @@ -91,7 +91,7 @@
     "tiktoken": "tiktoken",
     "timm": "timm<=1.0.11",
     "tokenizers": "tokenizers>=0.21,<0.22",
-    "torch": "torch>=2.1,<2.7",
+    "torch": "torch>=2.1",
     "torchaudio": "torchaudio",
     "torchvision": "torchvision",
     "pyctcdecode": "pyctcdecode>=0.4.0",
Repository: denoland/deno
Pull request: perf(node/fs/copy): reduce metadata lookups copying directory

```
> cat main.mjs
import { cpSync, rmdirSync } from "node:fs";

for (let i = 0; i < 100_000; i++) {
  cpSync("folder", "other", {
    force: true,
    recursive: true
  });
  rmdirSync("other");
}
> Measure-Command { node main.mjs }
32313.49
> Measure-Command { deno run -A main.mjs }
27379.9646
> Measure-Command { ../deno/target/debug/deno run -A main.mjs } # notice this is debug--i'm too lazy to check release
11628.361
```

Changed files:
filename: ext/fs/std_fs.rs
patch: @@ -723,30 +723,34 @@ fn cp(from: &Path, to: &Path) -> FsResult<()> {
     }
   }
 
-  match (fs::metadata(to), fs::symlink_metadata(to)) {
-    (Ok(m), _) if m.is_dir() => cp_(
-      source_meta,
-      from,
-      &to.join(from.file_name().ok_or_else(|| {
-        io::Error::new(
-          io::ErrorKind::InvalidInput,
-          "the source path is not a valid file",
-        )
-      })?),
-    )?,
-    (_, Ok(m)) if is_identical(&source_meta, &m) => {
+  if let Ok(m) = fs::metadata(to) {
+    if m.is_dir() {
+      return cp_(
+        source_meta,
+        from,
+        &to.join(from.file_name().ok_or_else(|| {
+          io::Error::new(
+            io::ErrorKind::InvalidInput,
+            "the source path is not a valid file",
+          )
+        })?),
+      );
+    }
+  }
+
+  if let Ok(m) = fs::symlink_metadata(to) {
+    if is_identical(&source_meta, &m) {
       return Err(
         io::Error::new(
           io::ErrorKind::InvalidInput,
           "the source and destination are the same file",
         )
         .into(),
-      )
+      );
     }
-    _ => cp_(source_meta, from, to)?,
   }
 
-  Ok(())
+  cp_(source_meta, from, to)
 }
 
 #[cfg(not(windows))]
Repository: ggml-org/llama.cpp
Pull request: llama : add simple option to enable CPU for MoE weights (--cpu-moe)

This is intended to be a simple and curated way to use the CPU for the MoE weights. Internally, it is just setting up the appropriate tensor overrides, but this should be easier to use.

Changed files:
filename: common/arg.cpp
patch: @@ -2380,6 +2380,15 @@ common_params_context common_params_parser_init(common_params & params, llama_ex
             }
         }
     ));
+    add_opt(common_arg(
+        {"--cpu-moe"},
+        "use CPU for Mixture of Experts (MoE) weights",
+        [](common_params & params) {
+            params.tensor_buft_overrides.push_back({"\\.ffn_up_exps\\.weight$",   ggml_backend_cpu_buffer_type()});
+            params.tensor_buft_overrides.push_back({"\\.ffn_down_exps\\.weight$", ggml_backend_cpu_buffer_type()});
+            params.tensor_buft_overrides.push_back({"\\.ffn_gate_exps\\.weight$", ggml_backend_cpu_buffer_type()});
+        }
+    ).set_env("LLAMA_ARG_CPU_MOE"));
     add_opt(common_arg(
         {"-ngl", "--gpu-layers", "--n-gpu-layers"}, "N",
         "number of layers to store in VRAM",
Repository: ggml-org/llama.cpp
Pull request: graph : fix equal_seq() check

fix #14893
cont #14482 

The check for `ubatch.equal_seqs()` was inverted.

Also add `LLAMA_GRAPH_REUSE_DISABLE` env as a debugging option for disable graph reuse logic.

Changed files:
filename: src/llama-context.cpp
patch: @@ -113,6 +113,15 @@ llama_context::llama_context(
         }
     }
 
+    {
+        const char * LLAMA_GRAPH_REUSE_DISABLE = getenv("LLAMA_GRAPH_REUSE_DISABLE");
+        graph_reuse_disable = LLAMA_GRAPH_REUSE_DISABLE ? (atoi(LLAMA_GRAPH_REUSE_DISABLE) != 0) : graph_reuse_disable;
+
+        if (graph_reuse_disable) {
+            LLAMA_LOG_WARN("%s: graph reuse disabled\n", __func__);
+        }
+    }
+
     const uint32_t n_ctx_per_seq = cparams.n_ctx / cparams.n_seq_max;
 
     LLAMA_LOG_INFO("%s: n_seq_max     = %u\n",   __func__, cparams.n_seq_max);
@@ -716,7 +725,7 @@ llm_graph_result * llama_context::process_ubatch(const llama_ubatch & ubatch, ll
     // in order to correctly reuse a graph, it's full topology has to be uniquely determined by these parameters
     const auto gparams = graph_params(res, ubatch, mctx, gtype);
 
-    if (res->can_reuse(gparams)) {
+    if (!graph_reuse_disable && res->can_reuse(gparams)) {
         //LLAMA_LOG_DEBUG("%s: reusing previous graph\n", __func__);
 
         n_reused++;

filename: src/llama-context.h
patch: @@ -291,6 +291,9 @@ struct llama_context {
     // ref: https://github.com/ggml-org/llama.cpp/pull/14285
     bool supports_set_rows = false;
 
+    // env: LLAMA_GRAPH_REUSE_DISABLE
+    bool graph_reuse_disable = false;
+
     // perf
     mutable int64_t t_start_us  = 0;
     mutable int64_t t_load_us   = 0;

filename: src/llama-graph.h
patch: @@ -423,7 +423,9 @@ struct llm_graph_params {
                 (!ubatch.embd  && !other.ubatch.embd)
             );
 
-        if (can_reuse_ubatch && !ubatch.equal_seqs()) {
+        // when we split the batch using "equal_seqs" we have to verify that the participating sequences are the same
+        //   the reason is because the set of attention streams would be different for different sequences
+        if (can_reuse_ubatch && ubatch.equal_seqs()) {
             if (!ubatch.data) {
                 // if the old ubatch does not own it's data, then we cannot guarantee that it is still alive, and
                 //   therefore we cannot perform the sequence id check. normally should never happen
Repository: denoland/deno
Pull request: fix(check): expose more globals from @types/node

Extracted out of https://github.com/denoland/deno/pull/26558

Closes https://github.com/denoland/deno/issues/26578

Changed files:
filename: cli/tsc/99_main_compiler.js
patch: @@ -1337,18 +1337,12 @@ delete Object.prototype.__proto__;
     "console",
     "Console",
     "ErrorConstructor",
-    "exports",
     "gc",
     "Global",
     "ImportMeta",
     "localStorage",
-    "module",
-    "NodeModule",
-    "NodeRequire",
-    "process",
     "queueMicrotask",
     "RequestInit",
-    "require",
     "ResponseInit",
     "sessionStorage",
     "setImmediate",

filename: tests/specs/npm/compare_globals/__test__.jsonc
patch: @@ -1,4 +1,4 @@
 {
-  "args": "run --allow-read --check=all compare_globals/main.ts",
-  "output": "compare_globals/main.out"
+  "args": "run --allow-read --check=all main.ts",
+  "output": "main.out"
 }

filename: tests/specs/npm/compare_globals/main.out
patch: @@ -8,10 +8,11 @@ Download http://localhost:4260/@denotest/globals/1.0.0.tgz
 Download http://localhost:4260/@types/node/node-22.5.4.tgz
 Download http://localhost:4260/undici-types/undici-types-6.19.8.tgz
 [UNORDERED_END]
-Check file:///[WILDCARD]/compare_globals/main.ts
+Check file:///[WILDCARD]/main.ts
 true
 true
 []
+process equals process true
 setTimeout 1 false
 setTimeout 2 function
 setTimeout 3 function

filename: tests/specs/npm/compare_globals/main.ts
patch: @@ -5,10 +5,17 @@ console.log(globals.global === globals.globalThis);
 // @ts-expect-error even though these are the same object, they have different types
 console.log(globals.globalThis === globalThis);
 console.log(globals.process.execArgv);
+console.log("process equals process", process === globals.process);
 
 type AssertTrue<T extends true> = never;
-type _TestNoProcessGlobal = AssertTrue<
-  typeof globalThis extends { process: any } ? false : true
+type _TestHasProcessGlobal = AssertTrue<
+  typeof globalThis extends { process: any } ? true : false
+>;
+type _TestProcessGlobalVersion = AssertTrue<
+  typeof process.versions.node extends string ? true : false
+>;
+type _TestNoBufferGlogal = AssertTrue<
+  typeof globalThis extends { Buffer: any } ? false : true
 >;
 type _TestHasNodeJsGlobal = NodeJS.Architecture;
Repository: facebook/react
Pull request: Use setTimeout to schedule work on the server in Edge environments

We rely heavily on being able to batch rendering after multiple fetches etc. have completed on the server. However, we only do this in the Node.js build. Node.js `setImmediate` has the exact semantics we need. To be after the current cycle of I/O so that we can collect after all those I/O events already in the queue has been processed.

This doesn't exist in standard browsers, so we ended up not using it there. We could've used `setTimeout` but that risks being throttled which would severely negatively affect the performance so we just did it synchronously there. We probably could just use the `scheduler` there.

Now we have a separate build for Edge where `setTimeout(..., 0)` actually behaves like `setImmediate` which is what we want. So we can just use that in that build.

@Jarred-Sumner not sure what you want for Bun.

Changed files:
filename: packages/react-server-dom-webpack/src/__tests__/ReactFlightDOMEdge-test.js
patch: @@ -15,6 +15,10 @@ global.ReadableStream =
 global.TextEncoder = require('util').TextEncoder;
 global.TextDecoder = require('util').TextDecoder;
 
+// Don't wait before processing work on the server.
+// TODO: we can replace this with FlightServer.act().
+global.setTimeout = cb => cb();
+
 let clientExports;
 let webpackMap;
 let webpackModules;

filename: packages/react-server/src/ReactServerStreamConfigEdge.js
patch: @@ -13,7 +13,7 @@ export type PrecomputedChunk = Uint8Array;
 export opaque type Chunk = Uint8Array;
 
 export function scheduleWork(callback: () => void) {
-  callback();
+  setTimeout(callback, 0);
 }
 
 export function flushBuffered(destination: Destination) {

filename: packages/react/src/__tests__/ReactFetchEdge-test.js
patch: @@ -20,6 +20,10 @@ global.Response = require('node-fetch').Response;
 // Patch for Edge environments for global scope
 global.AsyncLocalStorage = require('async_hooks').AsyncLocalStorage;
 
+// Don't wait before processing work on the server.
+// TODO: we can replace this with FlightServer.act().
+global.setTimeout = cb => cb();
+
 let fetchCount = 0;
 async function fetchMock(resource, options) {
   fetchCount++;
Repository: huggingface/transformers
Pull request: Fix `mllama`

# What does this PR do?

`test_11b_model_integration_forward` has dtype issue, it fails, and affect other tests (GPU OOM).

I also update the expected values for A10.

All tests pass now on T4/A10 + torch 2.6/2.7

Changed files:
filename: tests/models/mllama/test_modeling_mllama.py
patch: @@ -534,7 +534,7 @@ def test_11b_model_integration_generate(self):
                 {
                     ("xpu", 3): "If I had to write a haiku for this one, it would be:.\\nA dock on a lake.\\nA mountain in the distance.\\nA long exposure.",
                     ("cuda", 7): "If I had to write a haiku for this one, it would be:.\\nA dock in the lake.\\nA mountain in the distance.\\nA long exposure.",
-                    ("cuda", 8): "If I had to write a haiku for this one, it would be:.\\nA dock on a lake.\\nA mountain in the distance.\\nA long exposure.",
+                    ("cuda", 8): 'If I had to write a haiku for this one, it would be:.\\nA dock in the lake.\\nA mountain in the distance.\\nA long exposure.',
                 }
             )  # fmt: skip
         expected_output = expected_outputs.get_expectation()
@@ -618,9 +618,9 @@ def test_11b_model_integration_forward(self):
         actual_logits = output.logits[0, -1, :5].cpu()
         expected_logits_all = Expectations(
             {
-                ("xpu", 3): torch.tensor([9.1562, 8.9141, 5.0664, 1.6855, 3.2324]),
-                ("cuda", 7): torch.tensor([9.0781, 8.8750, 5.0781, 1.6221, 3.2207]),
-                ("cuda", 8): torch.tensor([9.0703, 8.8750, 5.0781, 1.6279, 3.2207]),
+                ("xpu", 3): torch.tensor([9.1562, 8.9141, 5.0664, 1.6855, 3.2324], dtype=actual_logits.dtype),
+                ("cuda", 7): torch.tensor([9.0781, 8.8750, 5.0781, 1.6221, 3.2207], dtype=actual_logits.dtype),
+                ("cuda", 8): torch.tensor([9.0703, 8.8750, 5.0781, 1.6279, 3.2207], dtype=actual_logits.dtype),
             }
         )
 
@@ -665,7 +665,7 @@ def test_11b_model_integration_batched_generate(self):
                 {
                     ("xpu", 3): "If I had to write a haiku for this one, it would be:.\\nA dock on a lake.\\nA mountain in the distance.\\nA long exposure.",
                     ("cuda", 7): "If I had to write a haiku for this one, it would be:.\\nA dock on a lake.\\nA mountain in the distance.\\nA long exposure.",
-                    ("cuda", 8): "If I had to write a haiku for this one, it would be:.\\nA dock on a lake.\\nA mountain in the distance.\\nA long exposure.",
+                    ("cuda", 8): 'If I had to write a haiku for this one, it would be:.\\nA dock in the lake.\\nA mountain in the distance.\\nA long exposure.',
                  }
             )  # fmt: skip
         expected_output = expected_outputs.get_expectation()