helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: Disable report callbacks for certain training tests

This fixes the `No module named 'pynvml'` error in the affected tests on non-cuda devices.

~It does not address `test_auto_batch_size_finder` however, as that test runs `run_glue.py` directly.~

I tried adding `--report_to none` as an argument to the `run_glue.py` call. I'll check if it actually works.

Changed files:
filename: tests/trainer/test_trainer.py
patch: @@ -1368,6 +1368,7 @@ def test_torch_compile_loss_func_compatibility(self):
             per_device_train_batch_size=2,
             torch_compile=True,
             max_steps=1,  # compile happens on the first step
+            report_to="none",
         )
         trainer = Trainer(model=tiny_llama, args=args, train_dataset=train_dataset)  # noqa
         trainer.train()
@@ -3300,6 +3301,7 @@ def test_auto_batch_size_finder(self):
                 --num_train_epochs 1
                 --output_dir {tmpdir}
                 --auto_find_batch_size 0
+                --report_to none
                 """.split()
             with self.assertRaises(RuntimeError):
                 with patch.object(sys, "argv", testargs):
@@ -4560,7 +4562,7 @@ def test_trainer_saves_image_processor(self):
             config = RegressionModelConfig(a=1.5, b=2.5)
             trainer = Trainer(
                 model=RegressionPreTrainedModel(config),
-                args=TrainingArguments(output_dir=tmp_dir),
+                args=TrainingArguments(output_dir=tmp_dir, report_to="none"),
                 processing_class=image_processor,
             )
             trainer.save_model()
@@ -4576,7 +4578,7 @@ def test_trainer_saves_feature_extractor(self):
             config = RegressionModelConfig(a=1.5, b=2.5)
             trainer = Trainer(
                 model=RegressionPreTrainedModel(config),
-                args=TrainingArguments(output_dir=tmp_dir),
+                args=TrainingArguments(output_dir=tmp_dir, report_to="none"),
                 processing_class=feature_extractor,
             )
             trainer.save_model()
@@ -4596,7 +4598,7 @@ def test_trainer_saves_processor(self):
             config = RegressionModelConfig(a=1.5, b=2.5)
             trainer = Trainer(
                 model=RegressionPreTrainedModel(config),
-                args=TrainingArguments(output_dir=tmp_dir),
+                args=TrainingArguments(output_dir=tmp_dir, report_to="none"),
                 processing_class=processor,
             )
             trainer.save_model()
Repository: denoland/deno
Pull request: feat(ext/node): Add a stub implementation of cpu_info() for OpenBSD (#2)

* Add a stub implementation of cpu_info() for OpenBSD

Add an implementation of cpu_info() for OpenBSD, that returns a correctly-sized array. Since Rust's libc bindings for OpenBSD do not contain all symbols necessary for a full implementation and it is not planned to add them, this solution at least avoids problems with code that relies on cpu_info() purely for the size of the returned array to derive the number of available CPUs.

This addresses https://github.com/denoland/deno/issues/25621

<!--
Before submitting a PR, please read https://docs.deno.com/runtime/manual/references/contributing

1. Give the PR a descriptive title.

  Examples of good title:
    - fix(std/http): Fix race condition in server
    - docs(console): Update docstrings
    - feat(doc): Handle nested reexports

  Examples of bad title:
    - fix #7123
    - update docs
    - fix bugs

2. Ensure there is a related issue and it is referenced in the PR text.
3. Ensure there are tests that cover the changes.
4. Ensure `cargo test` passes.
5. Ensure `./tools/format.js` passes without changing files.
6. Ensure `./tools/lint.js` passes.
7. Open as a draft PR if your work is still in progress. The CI won't run
   all steps, but you can add '[ci]' to a commit message to force it to.
8. If you would like to run the benchmarks on the CI, add the 'ci-bench' label.
-->


Changed files:
filename: ext/node/ops/os/cpus.rs
patch: @@ -294,6 +294,54 @@ pub fn cpu_info() -> Option<Vec<CpuInfo>> {
   Some(cpus)
 }
 
+#[cfg(target_os = "openbsd")]
+pub fn cpu_info() -> Option<Vec<CpuInfo>> {
+  // Stub implementation for OpenBSD that returns an array of the correct size
+  // but with dummy values.
+  // Rust's OpenBSD libc bindings don't contain all the symbols needed for a
+  // full implementation, and including them is not planned.
+  let mut mib = [libc::CTL_HW, libc::HW_NCPUONLINE];
+
+  // SAFETY: Assumes correct behavior of platform-specific
+  // sysctls and data structures. Relies on specific sysctl
+  // names and parameter existence.
+  unsafe {
+    let mut ncpu: libc::c_uint = 0;
+    let mut size = std::mem::size_of_val(&ncpu) as libc::size_t;
+
+    // Get number of CPUs online
+    let res = libc::sysctl(
+      mib.as_mut_ptr(),
+      mib.len() as _,
+      &mut ncpu as *mut _ as *mut _,
+      &mut size,
+      std::ptr::null_mut(),
+      0,
+    );
+    // If res == 0, the sysctl call was succesful and
+    // ncpuonline contains the number of online CPUs.
+    if res != 0 {
+      return None;
+    } else {
+      let mut cpus = vec![CpuInfo::new(); ncpu as usize];
+
+      for (_, cpu) in cpus.iter_mut().enumerate() {
+        cpu.model = "Undisclosed CPU".to_string();
+        // Return 1 as a dummy value so the tests won't
+        // fail.
+        cpu.speed = 1;
+        cpu.times.user = 1;
+        cpu.times.nice = 1;
+        cpu.times.sys = 1;
+        cpu.times.idle = 1;
+        cpu.times.irq = 1;
+      }
+
+      return Some(cpus);
+    }
+  }
+}
+
 #[cfg(test)]
 mod tests {
   use super::*;
Repository: vercel/next.js
Pull request: Update Turbopack development test manifest

This auto-generated PR updates the development integration test manifest used when testing Turbopack.

Changed files:
filename: test/turbopack-dev-tests-manifest.json
patch: @@ -2758,7 +2758,8 @@
     "passed": [
       "asset-prefix rewrites rewrites that do not start with assetPrefix should still work",
       "asset-prefix rewrites should respect rewrites that start with assetPrefix",
-      "asset-prefix should load the app properly without reloading"
+      "asset-prefix should load the app properly without reloading",
+      "asset-prefix should navigate to another page without hard navigating"
     ],
     "failed": [],
     "pending": [],
Repository: facebook/react
Pull request: [DevTools][Bugfix] Don't hide fragment if it has a key

resolves #25187 

---

We shouldn't hide fragments that have a key because this is important information that the user might want in the future.

**Should we do the same thing for Offscreen/HostText/HostPortals. Will update PR with photos after**

Changed files:
filename: packages/react-devtools-shared/src/backend/renderer.js
patch: @@ -468,8 +468,9 @@ export function getInternalReactConstants(
         return type;
       case HostPortal:
       case HostText:
-      case Fragment:
         return null;
+      case Fragment:
+        return 'Fragment';
       case LazyComponent:
         // This display name will not be user visible.
         // Once a Lazy component loads its inner component, React replaces the tag and type.
@@ -981,7 +982,7 @@ export function attach(
 
   // NOTICE Keep in sync with get*ForFiber methods
   function shouldFilterFiber(fiber: Fiber): boolean {
-    const {_debugSource, tag, type} = fiber;
+    const {_debugSource, tag, type, key} = fiber;
 
     switch (tag) {
       case DehydratedSuspenseComponent:
@@ -993,13 +994,14 @@ export function attach(
         return true;
       case HostPortal:
       case HostText:
-      case Fragment:
       case LegacyHiddenComponent:
       case OffscreenComponent:
         return true;
       case HostRoot:
         // It is never valid to filter the root element.
         return false;
+      case Fragment:
+        return key === null;
       default:
         const typeSymbol = getTypeSymbol(type);
Repository: nodejs/node
Pull request: meta: add @nodejs/url as codeowner

Refs: https://github.com/nodejs/node/pull/56779#issuecomment-2614673782

Changed files:
filename: .github/CODEOWNERS
patch: @@ -35,7 +35,6 @@
 
 # net
 
-/deps/ada @nodejs/url
 /deps/cares @nodejs/net
 /doc/api/dgram.md @nodejs/net
 /doc/api/dns.md @nodejs/net
@@ -47,14 +46,11 @@
 /lib/internal/js_stream_socket.js @nodejs/net
 /lib/internal/net.js @nodejs/net
 /lib/internal/socket_list.js @nodejs/net
-/lib/internal/url.js @nodejs/url
 /lib/net.js @nodejs/net
-/lib/url.js @nodejs/url
 /src/cares_wrap.cc @nodejs/net
 /src/connect_wrap.* @nodejs/net
 /src/connection_wrap.* @nodejs/net
 /src/node_sockaddr* @nodejs/net
-/src/node_url.* @nodejs/url
 /src/tcp_wrap.* @nodejs/net
 /src/udp_wrap.* @nodejs/net
 
@@ -197,3 +193,10 @@
 
 # Performance
 /benchmark/* @nodejs/performance
+
+# URL
+/deps/ada @nodejs/url
+/lib/internal/url.js @nodejs/url
+/lib/url.js @nodejs/url
+/src/node_url.* @nodejs/url
+/test/fixtures/wpt/url @nodejs/url
Repository: electron/electron
Pull request: build: remove github actions dependabot on release branches

Looks like github actions dependabot can't be configured to only do security updates, so let's just keep `main` up to date. Otherwise this is expensive

Notes: none

Changed files:
filename: .github/dependabot.yml
patch: @@ -11,38 +11,6 @@ updates:
       - "no-backport"
       - "semver/none"
     target-branch: main
-  - package-ecosystem: github-actions
-    directory: /
-    schedule:
-      interval: weekly
-    labels:
-      - "backport-check-skip"
-      - "semver/none"
-    target-branch: 33-x-y
-  - package-ecosystem: github-actions
-    directory: /
-    schedule:
-      interval: weekly
-    labels:
-      - "backport-check-skip"
-      - "semver/none"
-    target-branch: 32-x-y
-  - package-ecosystem: github-actions
-    directory: /
-    schedule:
-      interval: weekly
-    labels:
-      - "backport-check-skip"
-      - "semver/none"
-    target-branch: 31-x-y
-  - package-ecosystem: github-actions
-    directory: /
-    schedule:
-      interval: weekly
-    labels:
-      - "backport-check-skip"
-      - "semver/none"
-    target-branch: 30-x-y
   - package-ecosystem: npm
     directories:
       - /
Repository: vercel/next.js
Pull request: [ts-next-plugin] add testing utils

Ported from https://github.com/vercel/next.js/pull/78505, this PR adds testing utils and a basic log test for the Next.js TypeScript language service plugin.

Closes NEXT-4007

Changed files:
filename: test/development/typescript-plugin/index.test.ts
patch: @@ -0,0 +1,18 @@
+import type { PluginLanguageService } from './test-utils'
+import { getPluginLanguageService } from './test-utils'
+
+describe('typescript-plugin', () => {
+  let languageService: PluginLanguageService
+
+  beforeAll(() => {
+    languageService = getPluginLanguageService(__dirname)
+  })
+
+  it('should be able to get the language service', () => {
+    expect(languageService).toBeDefined()
+    const capturedLogs = languageService.getCapturedLogs()
+    expect(capturedLogs).toContain(
+      '[next] Initialized Next.js TypeScript plugin'
+    )
+  })
+})

filename: test/development/typescript-plugin/test-utils.ts
patch: @@ -0,0 +1,70 @@
+import tsNextPluginFactory from 'next'
+import ts from 'typescript'
+export { NEXT_TS_ERRORS } from 'next/dist/server/typescript/constant'
+
+export type PluginLanguageService = ts.LanguageService & {
+  getCapturedLogs: () => string
+}
+
+export function getPluginLanguageService(dir: string): PluginLanguageService {
+  const files = ts.sys.readDirectory(dir)
+
+  const compilerOptions = ts.getDefaultCompilerOptions()
+  const compilerHost = ts.createCompilerHost(compilerOptions)
+
+  let logs = ''
+  const logger = {
+    info: (...args: any[]) => {
+      const message = args
+        .map((arg) => (typeof arg === 'string' ? arg : JSON.stringify(arg)))
+        .join(' ')
+      logs += message + '\n'
+      console.log(...args)
+    },
+  }
+
+  const languageServiceHost: ts.LanguageServiceHost = {
+    ...compilerHost,
+    getCompilationSettings: () => compilerOptions,
+    getScriptFileNames: () => files,
+    getScriptSnapshot: (fileName) => {
+      const contents = ts.sys.readFile(fileName)
+      if (contents && typeof contents === 'string') {
+        return ts.ScriptSnapshot.fromString(contents)
+      }
+      return
+    },
+    getScriptVersion: () => '0',
+    writeFile: ts.sys.writeFile,
+  }
+
+  const languageService = ts.createLanguageService(languageServiceHost)
+
+  const pluginCreateInfo: ts.server.PluginCreateInfo = {
+    project: {
+      projectService: {
+        logger,
+      },
+      getCurrentDirectory: () => dir,
+    } as unknown as ts.server.Project,
+    languageService,
+    languageServiceHost,
+    serverHost: null,
+    config: {},
+  }
+
+  const plugin: ts.server.PluginModule = (
+    tsNextPluginFactory as unknown as ts.server.PluginModuleFactory
+  )({ typescript: ts })
+
+  const service = plugin.create(pluginCreateInfo) as PluginLanguageService
+
+  // Add a custom method to get captured logs
+  service.getCapturedLogs = () => logs
+
+  return service
+}
+
+export function getTsFiles(dir: string): string[] {
+  return ts.sys.readDirectory(dir)
+}
Repository: nodejs/node
Pull request: test: update WPT for url to a23788b77a

This is an automated update of the WPT for url to https://github.com/web-platform-tests/wpt/commit/a23788b77a947304b2b3159d8efda492727ab623.

Changed files:
filename: test/fixtures/wpt/README.md
patch: @@ -28,7 +28,7 @@ Last update:
 - resource-timing: https://github.com/web-platform-tests/wpt/tree/22d38586d0/resource-timing
 - resources: https://github.com/web-platform-tests/wpt/tree/1e140d63ec/resources
 - streams: https://github.com/web-platform-tests/wpt/tree/bc9dcbbf1a/streams
-- url: https://github.com/web-platform-tests/wpt/tree/6fa3fe8a92/url
+- url: https://github.com/web-platform-tests/wpt/tree/a23788b77a/url
 - user-timing: https://github.com/web-platform-tests/wpt/tree/5ae85bf826/user-timing
 - wasm/jsapi: https://github.com/web-platform-tests/wpt/tree/cde25e7e3c/wasm/jsapi
 - wasm/webapi: https://github.com/web-platform-tests/wpt/tree/fd1b23eeaa/wasm/webapi

filename: test/fixtures/wpt/url/resources/urltestdata.json
patch: @@ -3137,6 +3137,21 @@
     "search": "",
     "hash": ""
   },
+  {
+    "input": "https://example.com/aaa/bbb/%2e%2e?query",
+    "base": null,
+    "href": "https://example.com/aaa/?query",
+    "origin": "https://example.com",
+    "protocol": "https:",
+    "username": "",
+    "password": "",
+    "host": "example.com",
+    "hostname": "example.com",
+    "port": "",
+    "pathname": "/aaa/",
+    "search": "?query",
+    "hash": ""
+  },
   "# Based on http://trac.webkit.org/browser/trunk/LayoutTests/fast/url/segments-userinfo-vs-host.html",
   {
     "input": "http:@www.example.com",

filename: test/fixtures/wpt/versions.json
patch: @@ -72,7 +72,7 @@
     "path": "streams"
   },
   "url": {
-    "commit": "6fa3fe8a929be45422cd46a8961e647e13d0cab8",
+    "commit": "a23788b77a947304b2b3159d8efda492727ab623",
     "path": "url"
   },
   "user-timing": {
Repository: denoland/deno
Pull request: feat(fmt): upgrade markup_fmt

Fixes #25259 
Fixes #25687 

Changed files:
filename: Cargo.lock
patch: @@ -4341,9 +4341,9 @@ dependencies = [
 
 [[package]]
 name = "markup_fmt"
-version = "0.12.0"
+version = "0.13.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "33edf0c364d53d5518cf663824e7d5d5339c844daa54bb0208bee5202dde88f1"
+checksum = "74fc137a4a591720176339bf7e857586a48ff35c0caee7ad6cf709327901232c"
 dependencies = [
  "aho-corasick",
  "css_dataset",

filename: cli/Cargo.toml
patch: @@ -124,7 +124,7 @@ libz-sys.workspace = true
 log = { workspace = true, features = ["serde"] }
 lsp-types.workspace = true
 malva = "=0.10.1"
-markup_fmt = "=0.12.0"
+markup_fmt = "=0.13.0"
 memmem.workspace = true
 monch.workspace = true
 notify.workspace = true
Repository: huggingface/transformers
Pull request: add timeout for downloading the `librispeech_asr` dataset

## What does this PR do?
When running the `run_wav2vec2_pretraining_no_trainer.py` example, I got `fsspec.exceptions.FSTimeoutError`.

The reason is that the default timeout (5 * 60s) for the `aiohttp` library used internally by fsspec is too short for large files, e.g. 5.9G. 

I also found an issue discussed in the dataset library, which resolved the issue by the same approach: https://github.com/huggingface/datasets/issues/7164. 




Changed files:
filename: examples/pytorch/speech-pretraining/run_wav2vec2_pretraining_no_trainer.py
patch: @@ -21,6 +21,7 @@
 from pathlib import Path
 from typing import Optional, Union
 
+import aiohttp
 import datasets
 import torch
 from accelerate import Accelerator
@@ -454,6 +455,7 @@ def main():
             split=train_split_name,
             cache_dir=args.cache_dir,
             trust_remote_code=args.trust_remote_code,
+            storage_options={"client_kwargs": {"timeout": aiohttp.ClientTimeout(total=60 * 60)}},
         )
         datasets_splits.append(dataset_split)
Repository: electron/electron
Pull request: build: use ts-node to spawn the version-bumper

Missed this in the tsification, we should probably call this via API instead of spawning a sub-proc?

Notes: none

Changed files:
filename: script/release/prepare-release.ts
patch: @@ -64,8 +64,13 @@ async function getNewVersion (dryRunMode: DryRunMode) {
   if (dryRunMode === DryRunMode.REAL_RUN) {
     console.log(`Bumping for new "${bumpType}" version.`);
   }
-  const bumpScript = join(__dirname, 'version-bumper.js');
-  const scriptArgs = ['node', bumpScript, `--bump=${bumpType}`];
+  const bumpScript = join(__dirname, 'version-bumper.ts');
+  const scriptArgs = [
+    'node',
+    'node_modules/.bin/ts-node',
+    bumpScript,
+    `--bump=${bumpType}`
+  ];
   if (dryRunMode === DryRunMode.DRY_RUN) scriptArgs.push('--dryRun');
   try {
     let bumpVersion = execSync(scriptArgs.join(' '), { encoding: 'utf-8' });
Repository: nodejs/node
Pull request: meta: move one or more collaborators to emeritus

This PR was generated by the [`find-inactive-collaborators.yml` workflow](https://github.com/nodejs/node/blob/main/.github/workflows/find-inactive-collaborators.yml).

@nodejs/tsc Please follow up with the [offboarding tasks](https://github.com/nodejs/node/blob/main/doc/contributing/offboarding.md).

Changed files:
filename: README.md
patch: @@ -291,8 +291,6 @@ For information about the governance of the Node.js project, see
   **Antoine du Hamel** <<duhamelantoine1995@gmail.com>> (he/him) - [Support me](https://github.com/sponsors/aduh95)
 * [anonrig](https://github.com/anonrig) -
   **Yagiz Nizipli** <<yagiz@nizipli.com>> (he/him) - [Support me](https://github.com/sponsors/anonrig)
-* [apapirovski](https://github.com/apapirovski) -
-  **Anatoli Papirovski** <<apapirovski@mac.com>> (he/him)
 * [atlowChemi](https://github.com/atlowChemi) -
   **Chemi Atlow** <<chemi@atlow.co.il>> (he/him)
 * [Ayase-252](https://github.com/Ayase-252) -
@@ -475,6 +473,8 @@ For information about the governance of the Node.js project, see
   **Anna M. Kedzierska** <<anna.m.kedzierska@gmail.com>>
 * [antsmartian](https://github.com/antsmartian) -
   **Anto Aravinth** <<anto.aravinth.cse@gmail.com>> (he/him)
+* [apapirovski](https://github.com/apapirovski) -
+  **Anatoli Papirovski** <<apapirovski@mac.com>> (he/him)
 * [aqrln](https://github.com/aqrln) -
   **Alexey Orlenko** <<eaglexrlnk@gmail.com>> (he/him)
 * [AshCripps](https://github.com/AshCripps) -
Repository: ggml-org/llama.cpp
Pull request: llama : fix llama_model_chat_template with template name (LLM_KV with suffix)

Suffix was never applied, and in the case of `llama_model_chat_template` arch was appended instead of suffix.

Changed files:
filename: src/llama-arch.cpp
patch: @@ -200,7 +200,6 @@ static const std::map<llm_kv, const char *> LLM_KV_NAMES = {
     { LLM_KV_TOKENIZER_HF_JSON,              "tokenizer.huggingface.json"              },
     { LLM_KV_TOKENIZER_RWKV,                 "tokenizer.rwkv.world"                    },
     { LLM_KV_TOKENIZER_CHAT_TEMPLATE,        "tokenizer.chat_template"                 },
-    { LLM_KV_TOKENIZER_CHAT_TEMPLATE_N,      "tokenizer.chat_template.%s"              },
     { LLM_KV_TOKENIZER_FIM_PRE_ID,           "tokenizer.ggml.fim_pre_token_id"         },
     { LLM_KV_TOKENIZER_FIM_SUF_ID,           "tokenizer.ggml.fim_suf_token_id"         },
     { LLM_KV_TOKENIZER_FIM_MID_ID,           "tokenizer.ggml.fim_mid_token_id"         },
@@ -1707,8 +1706,14 @@ static const std::map<llm_tensor, llm_tensor_info> LLM_TENSOR_INFOS = {
 LLM_KV::LLM_KV(llm_arch arch, const char * suffix) : arch(arch), suffix(suffix) {}
 
 std::string LLM_KV::operator()(llm_kv kv) const {
-    return suffix ? ::format(LLM_KV_NAMES.at(kv), LLM_ARCH_NAMES.at(arch), suffix)
-        : ::format(LLM_KV_NAMES.at(kv), LLM_ARCH_NAMES.at(arch));
+    std::string name = ::format(LLM_KV_NAMES.at(kv), LLM_ARCH_NAMES.at(arch));
+
+    if (suffix != nullptr) {
+        name += ".";
+        name += suffix;
+    }
+
+    return name;
 }
 
 std::string LLM_TN_IMPL::str() const {

filename: src/llama-arch.h
patch: @@ -196,7 +196,6 @@ enum llm_kv {
     LLM_KV_TOKENIZER_HF_JSON,
     LLM_KV_TOKENIZER_RWKV,
     LLM_KV_TOKENIZER_CHAT_TEMPLATE,
-    LLM_KV_TOKENIZER_CHAT_TEMPLATE_N,
     LLM_KV_TOKENIZER_FIM_PRE_ID,
     LLM_KV_TOKENIZER_FIM_SUF_ID,
     LLM_KV_TOKENIZER_FIM_MID_ID,

filename: src/llama-model.cpp
patch: @@ -13788,7 +13788,7 @@ uint64_t llama_model_size(const llama_model * model) {
 }
 
 const char * llama_model_chat_template(const llama_model * model, const char * name) {
-    const auto key = name ? LLM_KV(model->arch, name)(LLM_KV_TOKENIZER_CHAT_TEMPLATE_N)
+    const auto key = name ? LLM_KV(model->arch, name)(LLM_KV_TOKENIZER_CHAT_TEMPLATE)
         : LLM_KV(model->arch)(LLM_KV_TOKENIZER_CHAT_TEMPLATE);
     const auto & it = model->gguf_kv.find(key);
     if (it == model->gguf_kv.end()) {