helloadhavan/github_issues huggingface dataset:
Repository: denoland/deno
Pull request: fix(fmt): upgrade to dprint-plugin-markdown 0.17.4

Closes https://github.com/denoland/deno/issues/20917
Closes https://github.com/denoland/deno/issues/21285

Changed files:
filename: .dprint.json
patch: @@ -67,7 +67,7 @@
   "plugins": [
     "https://plugins.dprint.dev/typescript-0.91.6.wasm",
     "https://plugins.dprint.dev/json-0.19.3.wasm",
-    "https://plugins.dprint.dev/markdown-0.17.2.wasm",
+    "https://plugins.dprint.dev/markdown-0.17.4.wasm",
     "https://plugins.dprint.dev/toml-0.6.2.wasm",
     "https://plugins.dprint.dev/exec-0.5.0.json@8d9972eee71fa1590e04873540421f3eda7674d0f1aae3d7c788615e7b7413d0",
     "https://plugins.dprint.dev/g-plane/pretty_yaml-v0.4.0.wasm"

filename: .github/ISSUE_TEMPLATE/bug_report.md
patch: @@ -4,7 +4,6 @@ about: Report an issue found in the Deno CLI.
 title: ''
 labels: ''
 assignees: ''
-
 ---
 
 Version: Deno x.x.x

filename: .github/ISSUE_TEMPLATE/feature_request.md
patch: @@ -4,5 +4,4 @@ about: Suggest a feature for the Deno CLI.
 title: ''
 labels: ''
 assignees: ''
-
 ---

filename: Cargo.lock
patch: @@ -2481,9 +2481,9 @@ dependencies = [
 
 [[package]]
 name = "dprint-plugin-markdown"
-version = "0.17.2"
+version = "0.17.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a38c16d6d22fb0eb3c2ec94c35d074334d0a0d1da8e4c62e182c9679bdb8ac31"
+checksum = "8b2748618896a9206d21762a64a1d57be380f13e3e6774163d42395ca5bdf2f1"
 dependencies = [
  "anyhow",
  "dprint-core",
@@ -5420,9 +5420,9 @@ dependencies = [
 
 [[package]]
 name = "pulldown-cmark"
-version = "0.9.6"
+version = "0.11.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "57206b407293d2bcd3af849ce869d52068623f19e1b5ff8e8778e3309439682b"
+checksum = "cb4e75767fbc9d92b90e4d0c011f61358cde9513b31ef07ea3631b15ffc3b4fd"
 dependencies = [
  "bitflags 2.5.0",
  "memchr",

filename: cli/Cargo.toml
patch: @@ -101,7 +101,7 @@ dissimilar = "=1.0.4"
 dotenvy = "0.15.7"
 dprint-plugin-json = "=0.19.3"
 dprint-plugin-jupyter = "=0.1.3"
-dprint-plugin-markdown = "=0.17.2"
+dprint-plugin-markdown = "=0.17.4"
 dprint-plugin-typescript = "=0.91.6"
 env_logger = "=0.10.0"
 fancy-regex = "=0.10.0"
Repository: facebook/react
Pull request: Fix use of hydrateRoot in DevTools test

I noticed while working on a different PR that this test was not using hydrateRoot correctly. You're meant to pass the initial children as the second argument.

Changed files:
filename: packages/react-devtools-shared/src/__tests__/inspectedElement-test.js
patch: @@ -2099,7 +2099,7 @@ describe('InspectedElement', () => {
     await utils.actAsync(() => {
       const container = document.createElement('div');
       container.innerHTML = '<div></div>';
-      ReactDOMClient.hydrateRoot(container).render(<Example />);
+      ReactDOMClient.hydrateRoot(container, <Example />);
     }, false);
 
     const inspectedElement = await inspectElementAtIndex(0);
Repository: huggingface/transformers
Pull request: Remove torchvision requirement from AutoImageProcessor


Changed files:
filename: src/transformers/models/auto/image_processing_auto.py
patch: @@ -327,7 +327,7 @@ def _warning_fast_image_processor_available(fast_class):
     )
 
 
-@requires(backends=("vision", "torchvision"))
+@requires(backends=("vision",))
 class AutoImageProcessor:
     r"""
     This is a generic image processor class that will be instantiated as one of the image processor classes of the
Repository: ggml-org/llama.cpp
Pull request: CUDA: fix misaligned synchronization in FA

See discussion following https://github.com/ggml-org/llama.cpp/issues/13430#issuecomment-2869309424 .

I added a `__syncthreads` instruction in https://github.com/ggml-org/llama.cpp/pull/13438 to fix a race condition. However, this instruction is for some kernel configuration only executed for part of the warps so the points at which warps wait for each other become misaligned. On a sufficiently long context this then causes incorrect results.

Changed files:
filename: ggml/src/ggml-cuda/fattn-mma-f16.cuh
patch: @@ -895,6 +895,11 @@ static __device__ __forceinline__ void flash_attn_ext_f16_process_tile(
             float2 * dstk_fixup_meta = dstk_fixup + (gridDim.x + blockIdx.x)*ncols;
             dstk_fixup_meta[(threadIdx.y/np)*cols_per_warp + threadIdx.x] = make_float2(KQ_cmn, KQ_crs);
         }
+    } else if (np > 1) {
+        // Warps with threadIdx.y % np == 0 execute a __syncthreads() in the if branch.
+        // Therefore, all other warps also need to execute a __syncthreads().
+        // Otherwise the points at which warps synchronize with each other would become misaligned.
+        __syncthreads();
     }
 
 #pragma unroll
Repository: nodejs/node
Pull request: build: add major release action

This action reminds collaborators of the upcoming
major release date. In the future, this action can
also update and create the branches (that's why the
action name is generic).

Refs: https://github.com/nodejs/node/pull/55732

Changed files:
filename: .github/workflows/major-release.yml
patch: @@ -0,0 +1,48 @@
+name: Major Release
+
+on:
+  schedule:
+    - cron: 0 0 15 2,8 *  # runs at midnight UTC every 15 February and 15 August
+
+permissions:
+  contents: read
+
+jobs:
+  create-issue:
+    runs-on: ubuntu-latest
+    permissions:
+      issues: write
+    steps:
+      - name: Check for release schedule
+        id: check-date
+        run: |
+          # Get the current month and day
+          MONTH=$(date +'%m')
+          DAY=$(date +'%d')
+          # We'll create the reminder issue two months prior the release
+          if [[ "$MONTH" == "02" || "$MONTH" == "08" ]] && [[ "$DAY" == "15" ]]; then
+            echo "create_issue=true" >> "$GITHUB_ENV"
+          fi
+      - name: Retrieve next major release info from nodejs/Release
+        if: env.create_issue == 'true'
+        run: |
+          curl -L https://github.com/nodejs/Release/raw/HEAD/schedule.json | \
+          jq -r 'to_entries | map(select(.value.start | strptime("%Y-%m-%d") | mktime > now)) | first | "VERSION=" + .key + "\nRELEASE_DATE=" + .value.start' >> "$GITHUB_ENV"
+      - name: Compute max date for landing semver-major PRs
+        if: env.create_issue == 'true'
+        run: |
+          echo "PR_MAX_DATE=$(date -d "$RELEASE_DATE -1 month" +%Y-%m-%d)" >> "$GITHUB_ENV"
+      - name: Create release announcement issue
+        if: env.create_issue == 'true'
+        run: |
+         gh issue create --repo "${GITHUB_REPOSITORY}" \
+           --title "Upcoming Node.js Major Release ($VERSION)" \
+           --body-file -<<EOF
+            A reminder that the next Node.js **SemVer Major release** is scheduled for **${RELEASE_DATE}**.
+            All commits that were landed until **${PR_MAX_DATE}** (one month prior to the release) will be included in the next semver major release. Please ensure that any necessary preparations are made in advance.
+            For more details on the release process, consult the [Node.js Release Working Group repository](https://github.com/nodejs/release).
+
+            cc: @nodejs/collaborators
+         EOF
+        env:
+          GH_TOKEN: ${{ github.token }}
Repository: vercel/next.js
Pull request: Use `node:` prefixed in ESM emit of standalone server.js

This enables running this directly in Deno, even when bare node builtin imports are not enabled. By default Deno does not resolve bare node builtins. You have to either prefix them with `node:`, or use `--unstable-bare-node-builtins`


Changed files:
filename: packages/next/src/build/utils.ts
patch: @@ -1698,9 +1698,9 @@ export async function copyTracedFiles(
     `${
       moduleType
         ? `performance.mark('next-start');
-import path from 'path'
-import { fileURLToPath } from 'url'
-import module from 'module'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+import module from 'node:module'
 const require = module.createRequire(import.meta.url)
 const __dirname = fileURLToPath(new URL('.', import.meta.url))
 `
Repository: huggingface/transformers
Pull request: [kernels] use original forward at compile time

# What does this PR do?

`kernels` and `torch.compile` are not yet compatible with each other. Although we can skip custom kernels when the package is not installed, adding an error message is also not feasible -- we can't throw exceptions at compile time.

This PR hijacks the `kernels` decorator to add a compile-friendly path: until `kernels` supports `torch.compile`, let's use the original `forward`. 

Changed files:
filename: src/transformers/integrations/hub_kernels.py
patch: @@ -13,14 +13,18 @@
 # limitations under the License.
 from typing import Dict, Union
 
+from ..utils import is_torchdynamo_compiling
+
 
 try:
     from kernels import (
         Device,
         LayerRepository,
         register_kernel_mapping,
         replace_kernel_forward_from_hub,
-        use_kernel_forward_from_hub,
+    )
+    from kernels import (
+        use_kernel_forward_from_hub as original_use_kernel_forward_from_hub,
     )
 
     _hub_kernels_available = True
@@ -56,6 +60,40 @@
 
     register_kernel_mapping(_KERNEL_MAPPING)
 
+    def use_kernel_forward_from_hub(*args, **kwargs):
+        """
+        Expands `kernels`' `use_kernel_forward_from_hub` to NOT use a kernel at compile time. This should be removed
+        when `kernels` supports `torch.compile`.
+
+        If the layer has a `config` attribute, we can also set `config.disable_custom_kernels = True` to disable the
+        kernel.
+        """
+
+        def decorator_with_compile_path(cls):
+            # Keeps a reference to the original forward method
+            original_forward = cls.forward
+
+            # Applies the original decorator
+            decorator = original_use_kernel_forward_from_hub(*args, **kwargs)
+            cls = decorator(cls)
+
+            # Replaces the kernel forward with a compile-friendly version
+            kernel_forward = cls.forward
+
+            def forward_with_compile_path(*forward_args, **forward_kwargs):
+                disable_custom_kernels = hasattr(cls, "config") and getattr(cls.config, "disable_custom_kernels", None)
+                if is_torchdynamo_compiling() or disable_custom_kernels:
+                    return original_forward(*forward_args, **forward_kwargs)
+                else:
+                    return kernel_forward(*forward_args, **forward_kwargs)
+
+            cls.forward = forward_with_compile_path
+
+            return cls
+
+        return decorator_with_compile_path
+
+
 except ImportError:
     # Stub to make decorators int transformers work when `kernels`
     # is not installed.
Repository: facebook/react
Pull request: Match select value against primitives to string but not undefined

This is the source of the difference in symbol throwing behavior noted in https://github.com/facebook/react/pull/22522#discussion_r756444082

We always did throw for symbols when a "value" is provided to the parent `<select>` component. It's just that I treated undefined as providing a value. However, we also didn't toString that value so if it's one of the other primitives it didn't match.

Changed files:
filename: packages/react-dom/src/__tests__/ReactDOMServerIntegrationSelect-test.js
patch: @@ -246,4 +246,34 @@ describe('ReactDOMServerIntegrationSelect', () => {
       expect(option.selected).toBe(true);
     },
   );
+
+  itRenders(
+    'a boolean true select value match the string "true"',
+    async render => {
+      const e = await render(
+        <select value={true} readOnly={true}>
+          <option value="first">First</option>
+          <option value="true">True</option>
+        </select>,
+        1,
+      );
+      expect(e.firstChild.selected).toBe(false);
+      expect(e.lastChild.selected).toBe(true);
+    },
+  );
+
+  itRenders(
+    'a missing select value does not match the string "undefined"',
+    async render => {
+      const e = await render(
+        <select readOnly={true}>
+          <option value="first">First</option>
+          <option value="undefined">Undefined</option>
+        </select>,
+        1,
+      );
+      expect(e.firstChild.selected).toBe(true);
+      expect(e.lastChild.selected).toBe(false);
+    },
+  );
 });

filename: packages/react-dom/src/server/ReactDOMServerFormatConfig.js
patch: @@ -749,7 +749,7 @@ function pushStartOption(
     }
   }
 
-  if (selectedValue !== null) {
+  if (selectedValue != null) {
     let stringValue;
     if (value !== null) {
       if (__DEV__) {
@@ -782,8 +782,13 @@ function pushStartOption(
           break;
         }
       }
-    } else if (selectedValue === stringValue) {
-      target.push(selectedMarkerAttribute);
+    } else {
+      if (__DEV__) {
+        checkAttributeStringCoercion(selectedValue, 'select.value');
+      }
+      if ('' + selectedValue === stringValue) {
+        target.push(selectedMarkerAttribute);
+      }
     }
   } else if (selected) {
     target.push(selectedMarkerAttribute);
Repository: ggml-org/llama.cpp
Pull request: CUDA: fix race conditions in FlashAttention kernels

There are 2 race conditions in the CUDA FlashAttention code as reported by `compute-sanitizer --tool=racecheck ./build/bin/test-backend-ops`. This PR adds the necessary calls to `__syncthreads` to fix them.

Changed files:
filename: ggml/src/ggml-cuda/fattn-mma-f16.cuh
patch: @@ -874,6 +874,8 @@ static __device__ __forceinline__ void flash_attn_ext_f16_process_tile(
             }
         }
 
+        __syncthreads();
+
         // Write back combined meta data:
 #pragma unroll
         for (int imeta = 0; imeta < nmeta; ++imeta) {

filename: ggml/src/ggml-cuda/fattn-vec-f16.cuh
patch: @@ -168,6 +168,7 @@ static __global__ void flash_attn_vec_ext_f16(
     for (int j = 0; j < ncols; ++j) {
         KQ[j*D + tid] = -HALF_MAX_HALF;
     }
+    __syncthreads();
 
     half2 VKQ[ncols] = {{0.0f, 0.0f}};
Repository: vercel/next.js
Pull request: feat: add ravendb library to server-external-packages.json

Hello from the RavenDB team (https://github.com/ravendb/ravendb)

We've seen multiple reports of issues with Next.js and our ravendb package. This PR resolves those issues by adding ravendb package to `server-external-package.json`.

i've included one library.
- [ravendb](https://www.npmjs.com/package/ravendb)

let me know if any changes are needed.

Changed files:
filename: docs/01-app/05-api-reference/05-config/01-next-config-js/serverExternalPackages.mdx
patch: @@ -74,6 +74,7 @@ Next.js includes a [short list of popular packages](https://github.com/vercel/ne
 - `prisma`
 - `puppeteer-core`
 - `puppeteer`
+- `ravendb`
 - `require-in-the-middle`
 - `rimraf`
 - `sharp`

filename: docs/02-pages/04-api-reference/04-config/01-next-config-js/serverExternalPackages.mdx
patch: @@ -74,6 +74,7 @@ Next.js includes a [short list of popular packages](https://github.com/vercel/ne
 - `prisma`
 - `puppeteer-core`
 - `puppeteer`
+- `ravendb`
 - `require-in-the-middle`
 - `rimraf`
 - `sharp`

filename: packages/next/src/lib/server-external-packages.json
patch: @@ -55,6 +55,7 @@
   "prisma",
   "puppeteer",
   "puppeteer-core",
+  "ravendb",
   "require-in-the-middle",
   "rimraf",
   "sharp",
Repository: denoland/deno
Pull request: feat(publish): suggest importing `jsr:@std/` for `deno.land/std` urls


Changed files:
filename: cli/tools/registry/api.rs
patch: @@ -149,14 +149,62 @@ pub async fn get_package(
 }
 
 pub fn get_jsr_alternative(imported: &Url) -> Option<String> {
-  if !matches!(imported.host_str(), Some("esm.sh")) {
-    return None;
+  if matches!(imported.host_str(), Some("esm.sh")) {
+    let mut segments = imported.path_segments()?;
+    match segments.next()? {
+      "gh" => None,
+      module => Some(format!("\"npm:{module}\"")),
+    }
+  } else if imported.as_str().starts_with("https://deno.land/") {
+    let mut segments = imported.path_segments()?;
+    let maybe_std = segments.next()?;
+    if maybe_std != "std" && !maybe_std.starts_with("std@") {
+      return None;
+    }
+    let module = segments.next()?;
+    let export = segments
+      .next()
+      .filter(|s| *s != "mod.ts")
+      .map(|s| s.strip_suffix(".ts").unwrap_or(s).replace("_", "-"));
+    Some(format!(
+      "\"jsr:@std/{}@1{}\"",
+      module,
+      export.map(|s| format!("/{}", s)).unwrap_or_default()
+    ))
+  } else {
+    None
   }
+}
+
+#[cfg(test)]
+mod test {
+  use super::*;
+
+  #[test]
+  fn test_jsr_alternative() {
+    #[track_caller]
+    fn run_test(imported: &str, output: Option<&str>) {
+      let imported = Url::parse(imported).unwrap();
+      let output = output.map(|s| s.to_string());
+      assert_eq!(get_jsr_alternative(&imported), output);
+    }
 
-  let mut segments = imported.path_segments().unwrap();
-  match segments.next() {
-    Some("gh") => None,
-    Some(module) => Some(format!("\"npm:{module}\"")),
-    None => None,
+    run_test("https://esm.sh/ts-morph", Some("\"npm:ts-morph\""));
+    run_test(
+      "https://deno.land/std/path/mod.ts",
+      Some("\"jsr:@std/path@1\""),
+    );
+    run_test(
+      "https://deno.land/std/path/join.ts",
+      Some("\"jsr:@std/path@1/join\""),
+    );
+    run_test(
+      "https://deno.land/std@0.229.0/path/join.ts",
+      Some("\"jsr:@std/path@1/join\""),
+    );
+    run_test(
+      "https://deno.land/std@0.229.0/path/something_underscore.ts",
+      Some("\"jsr:@std/path@1/something-underscore\""),
+    );
   }
 }

filename: tests/specs/publish/prefer_fast_check_graph/main.out
patch: @@ -9,6 +9,10 @@ error[invalid-external-import]: invalid import to a non-JSR 'https' specifier
   |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the specifier
   | 
   = hint: replace this import with one from jsr or npm, or vendor the dependency into your package
+  | 
+1 | "jsr:@std/assert@1/assert"
+  | -------------------------- try this specifier
+  | 
 
   info: the import was resolved to 'https://deno.land/std/assert/assert.ts'
   info: this specifier is not allowed to be imported on jsr
Repository: ggml-org/llama.cpp
Pull request: sycl: enable dpcpp nightly builds with oneMKL and oneDNN

Currently if one tries to build the SYCL backend of LLaMA with nightly dpcpp with oneDNN and oneMKL libraries included in the release, it disables the oneDNN support because of GPU_VENDOR not being defined, and the CMake errors out with the following error:

```bash
CMake Error at ggml/src/ggml-sycl/CMakeLists.txt:112 (target_link_libraries):
  Target "ggml-sycl" links to:

    MKL::MKL_SYCL::BLAS

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

```

This is because the MKL's Cmake does not export the MKL_SYCL target when it's a nightly compiler.
This PR enables the same. 

Changed files:
filename: ggml/src/ggml-sycl/CMakeLists.txt
patch: @@ -52,9 +52,8 @@ target_compile_options(ggml-sycl PRIVATE "-Wno-narrowing")
 find_package(DNNL)
 set(GGML_SYCL_DNNL 0)
 if(DNNL_FOUND)
-    if (DEFINED ENV{ONEAPI_ROOT} AND NOT DEFINED DNNL_GPU_VENDOR)
-        # Assuming oneDNN packaged with oneapi release is used which
-        # supports only intel target
+    if (NOT DEFINED DNNL_GPU_VENDOR)
+        # default to intel target
         set(DNNL_GPU_VENDOR "INTEL")
         if(NOT "${GGML_SYCL_TARGET}" STREQUAL "INTEL")
             message(WARNING "oneDNN builds bundled with oneapi release only support INTEL target")
@@ -108,6 +107,9 @@ endif()
 if (GGML_SYCL_TARGET STREQUAL "INTEL")
     # Intel devices use Intel oneMKL directly instead of oneMath to avoid the limitation of linking Intel oneMKL statically
     # See https://github.com/uxlfoundation/oneMath/issues/654
+    if (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
+        set(SYCL_COMPILER ON)
+    endif()
     find_package(MKL REQUIRED)
     target_link_libraries(ggml-sycl PRIVATE MKL::MKL_SYCL::BLAS)
     target_compile_definitions(ggml-sycl PRIVATE GGML_SYCL_USE_INTEL_ONEMKL)
Repository: facebook/react
Pull request: React DevTools 4.23.0 -> 4.24.0

Reverts #23400 and updates CHANGELOG to include #24031 and #24065. Preparation for upcoming 4.24.0 release.

Changed files:
filename: packages/react-devtools/CHANGELOG.md
patch: @@ -5,13 +5,15 @@
 ---
 
 ### 4.24.0
-March 2, 2022
+March 10, 2022
 
 #### Feature
 * Show DevTools backend and frontend versions in UI ([bvaughn](https://github.com/bvaughn) in [#23399](https://github.com/facebook/react/pull/23399))
 * Timeline profiler refactored to support reading basic profiling data directly from React ([bvaughn](https://github.com/bvaughn) in [#22529](https://github.com/facebook/react/issues/22529))
 
 #### Bugfix
+* Better handle undefined `Error` stacks in DevTools error boundary ([bvaughn](https://github.com/bvaughn) in [#24065](https://github.com/facebook/react/pull/24065))
+* Fixed edge case bug in Profiler commit filtering ([bvaughn](https://github.com/bvaughn) in [#24031](https://github.com/facebook/react/pull/24031))
 * Gracefully handle empty "xstyle" prop values ([lunaruan](https://github.com/lunaruan) in [#23279](https://github.com/facebook/react/pull/23279) and [bvaughn](https://github.com/bvaughn) in [#23190](https://github.com/facebook/react/pull/23190))
 * Add `<TracingMarker>` component boilerplate ([lunaruan](https://github.com/lunaruan) in [#23275](https://github.com/facebook/react/pull/23275))
Repository: ggml-org/llama.cpp
Pull request: mtmd : use RMS norm for InternVL 3 38B and 78B mmproj

Follow-up to https://github.com/ggml-org/llama.cpp/pull/13443 - in my original PR I was only checking the config file for the actual ViT and failed to realize the norm type was also different, since it wasn't included in that specific config. See referenced PR for more information.

Due to the norm type not being stored in the gguf metadata, we check the hidden size and block count instead.

<details>

<summary>Quick test</summary>

Before (topk=1, greedy sampling, black and white image)

![before](https://github.com/user-attachments/assets/4a341225-a76e-4b3a-9373-db92efc2de6a)

After (topk=1, greedy sampling, same image)

![after](https://github.com/user-attachments/assets/7f9dbafd-f09e-4f78-8473-3d4484ce68b1)

</details>

Changed files:
filename: tools/mtmd/clip.cpp
patch: @@ -879,9 +879,15 @@ struct clip_graph {
         // add CLS token
         inp = ggml_concat(ctx0, inp, model.class_embedding, 1);
 
+        // The larger models use a different ViT, which uses RMS norm instead of layer norm
+        // ref: https://github.com/ggml-org/llama.cpp/pull/13443#issuecomment-2869786188
+        norm_type norm_t = (hparams.n_embd == 3200 && hparams.n_layer == 45)
+            ? NORM_TYPE_RMS // 6B ViT (Used by InternVL 2.5/3 - 26B, 38B, 78B)
+            : NORM_TYPE_NORMAL; // 300M ViT (Used by all smaller InternVL models)
+
         ggml_tensor * cur = build_vit(
                                 inp, n_pos,
-                                NORM_TYPE_NORMAL,
+                                norm_t,
                                 hparams.ffn_op,
                                 model.position_embeddings,
                                 nullptr);