helloadhavan/github_issues huggingface dataset:
Repository: electron/electron
Pull request: ci: set git `core.longpaths` to true

Backport of #47695

See that PR for details.


Notes: none

Changed files:
filename: .github/actions/install-build-tools/action.yml
patch: @@ -11,6 +11,7 @@ runs:
         git config --global core.autocrlf false
         git config --global branch.autosetuprebase always
         git config --global core.fscache true
+        git config --global core.longpaths true
         git config --global core.preloadindex true
       fi
       export BUILD_TOOLS_SHA=0a7f6bef9453ceee45612442660ca16d2c40171b

filename: .github/workflows/pipeline-segment-electron-test.yml
patch: @@ -135,6 +135,7 @@ jobs:
         git config --global core.autocrlf false
         git config --global branch.autosetuprebase always
         git config --global core.fscache true
+        git config --global core.longpaths true
         git config --global core.preloadindex true
         git clone --filter=tree:0 https://chromium.googlesource.com/chromium/tools/depot_tools.git
         # Ensure depot_tools does not update.
Repository: nodejs/node
Pull request: src: improve error reporting in node_mksnapshot

~~It's possible for NODE_OPTIONS to contain options that may not be valid during snapshot generation, such as `--use-openssl-ca` which is not supported when building with the `--without-ssl` option and causes the snapshot generation to fail. This commits causes node_mksnapshot to ignore the NODE_OPTIONS environment variable.~~

~~Also, while we're at it,~~ improve error reporting
when args processing fails.

Changed files:
filename: tools/snapshot/node_mksnapshot.cc
patch: @@ -66,8 +66,16 @@ int BuildSnapshot(int argc, char* argv[]) {
           std::vector<std::string>(argv, argv + argc),
           node::ProcessInitializationFlags::kGeneratePredictableSnapshot);
 
+  if (result->exit_code() != 0) {
+    for (const std::string& error : result->errors()) {
+      std::cerr << error << "\n";
+    }
+    std::cerr << "node_mksnapshot failed with exit code " << result->exit_code()
+              << "\n";
+    return static_cast<int>(result->exit_code());
+  }
+
   CHECK(!result->early_return());
-  CHECK_EQ(result->exit_code(), 0);
 
   std::string out_path;
   std::optional<std::string_view> builder_script_path = std::nullopt;
Repository: ggml-org/llama.cpp
Pull request: server: (router) disable SSL on child process

Because router already acts like a reversed proxy (similar to nginx), SSL should be handled by the router

Communication between router <> child process never go outside of localhost, so it doesn't need to be encrypted

Fix https://github.com/ggml-org/llama.cpp/issues/18131


Changed files:
filename: tools/server/server-models.cpp
patch: @@ -168,7 +168,9 @@ server_presets::server_presets(int argc, char ** argv, common_params & base_para
             env == "LLAMA_ARG_MODEL" ||
             env == "LLAMA_ARG_MMPROJ" ||
             env == "LLAMA_ARG_HF_REPO" ||
-            env == "LLAMA_ARG_NO_MODELS_AUTOLOAD") {
+            env == "LLAMA_ARG_NO_MODELS_AUTOLOAD" ||
+            env == "LLAMA_ARG_SSL_KEY_FILE" ||
+            env == "LLAMA_ARG_SSL_CERT_FILE") {
             control_args[env] = opt;
         }
     }
@@ -222,6 +224,9 @@ void server_presets::render_args(server_model_meta & meta) {
             preset.options[control_args["LLAMA_ARG_MMPROJ"]] = meta.path_mmproj;
         }
     }
+    // disable SSL for child processes (HTTPS already handled by router)
+    preset.options[control_args["LLAMA_ARG_SSL_KEY_FILE"]] = "";
+    preset.options[control_args["LLAMA_ARG_SSL_CERT_FILE"]] = "";
     meta.args = preset.to_args();
     // add back the binary path at the front
     meta.args.insert(meta.args.begin(), get_server_exec_path().string());
Repository: ollama/ollama
Pull request: Fix relative path lookup


Changed files:
filename: gpu/assets.go
patch: @@ -40,7 +40,7 @@ func PayloadsDir() (string, error) {
 			}
 
 			var paths []string
-			for _, root := range []string{appExe, cwd} {
+			for _, root := range []string{filepath.Dir(appExe), cwd} {
 				paths = append(paths,
 					filepath.Join(root),
 					filepath.Join(root, "windows-"+runtime.GOARCH),
Repository: denoland/deno
Pull request: fix(ext/node): add missing http2 header constants

Adds missing HTTP2_HEADER_ACCEPT_ENCODING and related constants that grpc-js depends on. Without these, grpc-js server crashes with:

```
TypeError: Cannot read properties of undefined (reading 'toLowerCase')
    at normalizeKey (metadata.js:38:16)
    at Metadata.remove (metadata.js:103:15)
```

when trying to normalize header keys in `BaseServerInterceptingCall`.

### Added constants:
- `HTTP2_HEADER_ACCEPT`
- `HTTP2_HEADER_ACCEPT_ENCODING`
- `HTTP2_HEADER_ACCEPT_LANGUAGE`
- `HTTP2_HEADER_ACCEPT_RANGES`

### Testing
With this fix, grpc-js server can handle requests:
```
$ grpcurl -plaintext -d '{"name":"Deno"}' localhost:50051 test.Greeter/SayHello
{
  "message": "Hello Deno"
}
```

Found while testing #26436 (which is fixed by the http2 rewrite).

Changed files:
filename: ext/node/ops/http2/types.rs
patch: @@ -152,6 +152,10 @@ pub struct Http2Constants {
   http2_header_transfer_encoding: &'static str,
   http2_header_keep_alive: &'static str,
   http2_header_proxy_connection: &'static str,
+  http2_header_accept: &'static str,
+  http2_header_accept_encoding: &'static str,
+  http2_header_accept_language: &'static str,
+  http2_header_accept_ranges: &'static str,
 
   http2_method_connect: &'static str,
   http2_method_delete: &'static str,
@@ -302,6 +306,10 @@ pub fn op_http2_constants() -> Http2Constants {
     http2_header_transfer_encoding: "transfer-encoding",
     http2_header_keep_alive: "keep-alive",
     http2_header_proxy_connection: "proxy-connection",
+    http2_header_accept: "accept",
+    http2_header_accept_encoding: "accept-encoding",
+    http2_header_accept_language: "accept-language",
+    http2_header_accept_ranges: "accept-ranges",
 
     http2_method_connect: "CONNECT",
     http2_method_delete: "DELETE",
Repository: huggingface/transformers
Pull request: Update issue template 

# What does this PR do?

This PR updates the issue template. 

Changed files:
filename: .github/ISSUE_TEMPLATE/bug-report.yml
patch: @@ -48,18 +48,17 @@ body:
           - continuous batching: @remi-or @ArthurZucker @McPatate
           - pipelines: @Rocketknight1
           - tokenizers: @ArthurZucker and @itazap
-          - trainer: @zach-huggingface @SunMarc
+          - trainer: @SunMarc
           - attention: @vasqu @ArthurZucker @CyrilVallez
           - model loading (from pretrained, etc): @CyrilVallez
-          - distributed: @3outeille @ArthurZucker @S1ro1
+          - distributed: @3outeille @ArthurZucker
           - CIs: @ydshieh
 
         Integrations:
 
-          - deepspeed: HF Trainer/Accelerate: @SunMarc @zach-huggingface
           - ray/raytune: @richardliaw, @amogkam
           - Big Model Inference: @SunMarc
-          - quantization (bitsandbytes, autogpt): @SunMarc @MekkCyber
+          - quantization: @SunMarc @MekkCyber
           - kernels: @MekkCyber @drbh
           - peft: @BenjaminBossan @githubnemo
         

filename: .github/PULL_REQUEST_TEMPLATE.md
patch: @@ -51,18 +51,17 @@ Library:
 - continuous batching: @remi-or @ArthurZucker @McPatate
 - pipelines: @Rocketknight1
 - tokenizers: @ArthurZucker and @itazap
-- trainer: @zach-huggingface @SunMarc
+- trainer: @SunMarc
 - attention: @vasqu @ArthurZucker @CyrilVallez
 - model loading (from pretrained, etc): @CyrilVallez
-- distributed: @3outeille @ArthurZucker @S1ro1
+- distributed: @3outeille @ArthurZucker
 - CIs: @ydshieh
 
 Integrations:
 
-- deepspeed: HF Trainer/Accelerate: @SunMarc @zach-huggingface
 - ray/raytune: @richardliaw, @amogkam
 - Big Model Inference: @SunMarc
-- quantization (bitsandbytes, autogpt): @SunMarc @MekkCyber
+- quantization: @SunMarc @MekkCyber
 - kernels: @MekkCyber @drbh
 - peft: @BenjaminBossan @githubnemo
Repository: huggingface/transformers
Pull request: remove ray_scope and check_quantized_param

# What does this PR do?

This PR removes a few things that we don't need in v5. 

Changed files:
filename: src/transformers/quantizers/base.py
patch: @@ -190,14 +190,6 @@ def adjust_max_memory(self, max_memory: dict[str, Union[int, str]]) -> dict[str,
         """adjust max_memory argument for infer_auto_device_map() if extra memory is needed for quantization"""
         return max_memory
 
-    def check_quantized_param(self, *args, **kwargs) -> bool:
-        """DEPRECATED -> remove in v5"""
-        logger.warning_once(
-            "`check_quantized_param` is deprecated in favor of `param_needs_quantization`, which is a much "
-            "more self.explanatory name for what the method achieves. It will be removed in v5"
-        )
-        return self.param_needs_quantization(*args, **kwargs)
-
     def param_needs_quantization(self, model: "PreTrainedModel", param_name: str, **kwargs) -> bool:
         """
         Check whether a given param needs quantization as defined by `create_quantized_param`.

filename: src/transformers/training_args.py
patch: @@ -1291,20 +1291,6 @@ class TrainingArguments:
             )
         },
     )
-    ray_scope: str = field(
-        default=None,
-        metadata={
-            "help": (
-                "This argument is deprecated and will be removed in v5.2. Set env var RAY_SCOPE instead."
-                'The scope to use when doing hyperparameter search with Ray. By default, `"last"` will be used. Ray'
-                " will then use the last checkpoint of all trials, compare those, and select the best one. However,"
-                " other options are also available. See the Ray documentation"
-                " (https://docs.ray.io/en/latest/tune/api_docs/analysis.html"
-                "#ray.tune.ExperimentAnalysis.get_best_trial)"
-                " for more options."
-            )
-        },
-    )
     ddp_timeout: int = field(
         default=1800,
         metadata={
Repository: nodejs/node
Pull request: deps: update googletest to eb2d85e

This is an automated update of googletest to eb2d85e.

Changed files:
filename: deps/googletest/include/gtest/gtest-typed-test.h
patch: @@ -48,15 +48,15 @@ template <typename T>
 class FooTest : public testing::Test {
  public:
   ...
-  typedef std::list<T> List;
+  using List = ::std::list<T>;
   static T shared_;
   T value_;
 };
 
 // Next, associate a list of types with the test suite, which will be
-// repeated for each type in the list.  The typedef is necessary for
+// repeated for each type in the list.  The using-declaration is necessary for
 // the macro to parse correctly.
-typedef testing::Types<char, int, unsigned int> MyTypes;
+using MyTypes = ::testing::Types<char, int, unsigned int>;
 TYPED_TEST_SUITE(FooTest, MyTypes);
 
 // If the type list contains only one type, you can write that type
@@ -157,7 +157,7 @@ REGISTER_TYPED_TEST_SUITE_P(FooTest,
 // argument to the INSTANTIATE_* macro is a prefix that will be added
 // to the actual test suite name.  Remember to pick unique prefixes for
 // different instances.
-typedef testing::Types<char, int, unsigned int> MyTypes;
+using MyTypes = ::testing::Types<char, int, unsigned int>;
 INSTANTIATE_TYPED_TEST_SUITE_P(My, FooTest, MyTypes);
 
 // If the type list contains only one type, you can write that type

filename: deps/googletest/include/gtest/gtest.h
patch: @@ -1610,6 +1610,8 @@ GTEST_API_ AssertionResult DoubleNearPredFormat(const char* expr1,
                                                 double val1, double val2,
                                                 double abs_error);
 
+using GoogleTest_NotSupported_OnFunctionReturningNonVoid = void;
+
 // INTERNAL IMPLEMENTATION - DO NOT USE IN USER CODE.
 // A class that enables one to stream messages to assertion macros
 class GTEST_API_ AssertHelper {
@@ -1621,7 +1623,8 @@ class GTEST_API_ AssertHelper {
 
   // Message assignment is a semantic trick to enable assertion
   // streaming; see the GTEST_MESSAGE_ macro below.
-  void operator=(const Message& message) const;
+  GoogleTest_NotSupported_OnFunctionReturningNonVoid operator=(
+      const Message& message) const;
 
  private:
   // We put our data in a struct so that the size of the AssertHelper class can
Repository: electron/electron
Pull request: ci: update audit CI workflow dependencies

#### Description of Change

<!--
Thank you for your Pull Request. Please provide a description above and review
the requirements below.

Contributors guide: https://github.com/electron/electron/blob/main/CONTRIBUTING.md
-->

The npm dependencies weren't being pinned before, and use the new `@electron/fiddle-core` release.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd

#### Release Notes

Notes: none <!-- Please add a one-line description for app developers to read in the release notes, or 'none' if no notes relevant to app developers. Examples and help on special cases: https://github.com/electron/clerk/blob/main/README.md#examples -->


Changed files:
filename: .github/workflows/audit-branch-ci.yml
patch: @@ -15,7 +15,11 @@ jobs:
     permissions:
       contents: read
     steps:
-      - run: npm install @actions/cache @electron/fiddle-core
+      - name: Setup Node.js
+        uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
+        with:
+          node-version: 22.17.x
+      - run: npm install @actions/cache@4.0.3 @electron/fiddle-core@2.0.1
       - uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
         id: audit-errors
         with:
@@ -29,7 +33,7 @@ jobs:
             // Only want the most recent workflow run that wasn't skipped or cancelled
             const isValidWorkflowRun = (run) => !['skipped', 'cancelled'].includes(run.conclusion);
 
-            const versions = await ElectronVersions.create(undefined, { ignoreCache: true });
+            const versions = await ElectronVersions.create({ ignoreCache: true });
             const branches = versions.supportedMajors.map((branch) => `${branch}-x-y`);
 
             for (const branch of ["main", ...branches]) {
Repository: ggml-org/llama.cpp
Pull request: convert : force patch_merger tensors to f16/f32

Same fix as in #15367

Fixes #18122

Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -1838,7 +1838,7 @@ def _find_param(self, obj: dict[str, Any], keys: Iterable[str], optional: bool =
 
     def tensor_force_quant(self, name, new_name, bid, n_dims):
         del bid, name, n_dims  # unused
-        if ".patch_embd.weight" in new_name:
+        if ".patch_embd.weight" in new_name or ".patch_merger.weight" in new_name:
             return gguf.GGMLQuantizationType.F16 if self.ftype == gguf.LlamaFileType.MOSTLY_F16 else gguf.GGMLQuantizationType.F32
         return False
Repository: facebook/react
Pull request: [compiler][eslint] remove compilationMode override; report bailouts on first line

Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom):
* __->__ #30423
* #30394

---

Before:
- observe that both functions are compiled, even though `useFoo` is not defined using [component syntax](https://flow.org/en/docs/react/component-syntax/)
- eslint diagnostic highlights entire function body

https://github.com/user-attachments/assets/e4a26fa9-cebb-4a44-8789-4810a015a67b

After:
- `compilationMode` config is respected
- eslint diagnostic highlights only first line

https://github.com/user-attachments/assets/eb3f9b1a-b204-4f10-a920-d6fd6b821adc



Changed files:
filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/bug-fbt-preserve-whitespace-param.expect.md
patch: @@ -2,7 +2,7 @@
 ## Input
 
 ```javascript
-import fbt from "fbt";
+import fbt from 'fbt';
 
 /**
  * Currently fails with the following:
@@ -32,7 +32,7 @@ function Foo(props) {
 
 export const FIXTURE_ENTRYPOINT = {
   fn: Foo,
-  params: [{ name: "Jason" }],
+  params: [{name: 'Jason'}],
 };
 
 ```

filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/bug-fbt-preserve-whitespace-param.tsx
patch: @@ -1,4 +1,4 @@
-import fbt from "fbt";
+import fbt from 'fbt';
 
 /**
  * Currently fails with the following:
@@ -28,5 +28,5 @@ function Foo(props) {
 
 export const FIXTURE_ENTRYPOINT = {
   fn: Foo,
-  params: [{ name: "Jason" }],
+  params: [{name: 'Jason'}],
 };
Repository: ollama/ollama
Pull request: types/model: remove old comment


Changed files:
filename: types/model/name.go
patch: @@ -80,9 +80,6 @@ func (k partKind) String() string {
 //
 // It is not guaranteed to be valid. Use [Name.IsValid] to check if the name
 // is valid.
-//
-// It is not directly comparable with other Names. Use [Name.Equal] and
-// [Name.MapHash] for determining equality and using as a map key.
 type Name struct {
 	Host      string
 	Namespace string
Repository: denoland/deno
Pull request: test: support `--list` flag in custom test harnesses

The `specs`, `unit`, and `unit_node` test harnesses use custom `main()` functions that don't go through libtest, so `cargo test <harness> -- --list` didn't work. 

This is a problem, because AI agents often use `cargo test -p <package> -- --list` to look for tests and instead of print the list our custom harnesses were just running these tests (which is really bad because specs test harness has >2000 tests).

Changed files:
filename: tests/specs/mod.rs
patch: @@ -300,6 +300,10 @@ pub fn main() {
     return; // all tests filtered out
   }
 
+  if test_util::test_runner::print_tests_if_list_flag(&root_category) {
+    return;
+  }
+
   let _http_guard = test_util::http_server();
   let parallelism = Parallelism::default();
   let flaky_test_tracker = Arc::new(FlakyTestTracker::default());

filename: tests/unit/mod.rs
patch: @@ -39,6 +39,9 @@ fn main() {
   if category.is_empty() {
     return; // no tests to run for the filter
   }
+  if test_util::test_runner::print_tests_if_list_flag(&category) {
+    return;
+  }
   let parallelism = Parallelism::default();
   let flaky_test_tracker = Arc::new(FlakyTestTracker::default());
   let _g = util::http_server();

filename: tests/unit_node/mod.rs
patch: @@ -41,6 +41,9 @@ fn main() {
   if category.is_empty() {
     return;
   }
+  if test_util::test_runner::print_tests_if_list_flag(&category) {
+    return;
+  }
   let parallelism = Parallelism::default();
   let flaky_test_tracker = Arc::new(FlakyTestTracker::default());
   let _g = util::http_server();

filename: tests/util/lib/test_runner.rs
patch: @@ -586,6 +586,31 @@ pub trait ReporterData {
   fn times_flaky() -> usize;
 }
 
+/// Checks if `--list` was passed in CLI args. If so, prints all test names
+/// in libtest format and returns `true` (caller should exit early).
+pub fn print_tests_if_list_flag<TData>(
+  category: &CollectedTestCategory<TData>,
+) -> bool {
+  if !std::env::args().any(|arg| arg == "--list") {
+    return false;
+  }
+  fn print_tests_recursive<T>(category: &CollectedTestCategory<T>) {
+    for child in &category.children {
+      match child {
+        CollectedCategoryOrTest::Category(c) => print_tests_recursive(c),
+        CollectedCategoryOrTest::Test(t) => {
+          #[allow(clippy::print_stdout)]
+          {
+            println!("{}: test", t.name);
+          }
+        }
+      }
+    }
+  }
+  print_tests_recursive(category);
+  true
+}
+
 pub fn get_test_reporter<TData: 'static>(
   test_module_name: &str,
   flaky_test_tracker: Arc<FlakyTestTracker>,
Repository: huggingface/transformers
Pull request: fix some case failures lead by "`torch.compile` recompiled part of th…

…e forward pass" in xpu

# What does this PR do?

fix test failure. including
cases include

tests/models/chameleon/test_modeling_chameleon.py::ChameleonVision2SeqModelTest::test_generate_compile_model_forward_fullgraph
tests/models/dia/test_modeling_dia.py::DiaModelTest::test_generate_compile_model_forward_fullgraph
tests/models/emu3/test_modeling_emu3.py::Emu3Text2TextModelTest::test_generate_compile_model_forward_fullgraph
tests/models/emu3/test_modeling_emu3.py::Emu3Vision2TextModelTest::test_generate_compile_model_forward_fullgraph
tests/models/glm4v/test_modeling_glm4v.py::Glm4vModelTest::test_generate_compile_model_forward_fullgraph
tests/models/got_ocr2/test_modeling_got_ocr2.py::GotOcr2ModelTest::test_generate_compile_model_forward_fullgraph
tests/models/gpt2/test_modeling_gpt2.py::GPT2ModelTest::test_generate_compile_model_forward_fullgraph
tests/models/internvl/test_modeling_internvl.py::InternVLModelTest::test_generate_compile_model_forward_fullgraph
tests/models/llava/test_modeling_llava.py::LlavaForConditionalGenerationModelTest::test_generate_compile_model_forward_fullgraph
tests/models/mistral3/test_modeling_mistral3.py::Mistral3ModelTest::test_generate_compile_model_forward_fullgraph
tests/models/ovis2/test_modeling_ovis2.py::Ovis2ModelTest::test_generate_compile_model_forward_fullgraph
tests/models/perception_lm/test_modeling_perception_lm.py::PerceptionLMForConditionalGenerationModelTest::test_generate_compile_model_forward_fullgraph
tests/models/qwen2_5_vl/test_modeling_qwen2_5_vl.py::Qwen2_5_VLModelTest::test_generate_compile_model_forward_fullgraph
tests/models/vipllava/test_modeling_vipllava.py::VipLlavaForConditionalGenerationModelTest::test_generate_compile_model_forward_fullgraph
tests/models/whisper/test_modeling_whisper.py::WhisperStandaloneDecoderModelTest::test_generate_compile_model_forward_fullgraph


@IlyasMoutawwakil


Changed files:
filename: src/transformers/masking_utils.py
patch: @@ -839,7 +839,8 @@ def create_causal_mask(
     # Do not allow skip if we are compiling (this is to match BC)
     # TODO: cyril -> probably revisit and remove this, but a lot of tests rely on it
     if _is_torch_xpu_available:
-        allow_is_causal_skip = True
+        # Do not allow skip if we are compiling for decoding, but for prefill, we still allow skip to optimization the perf of 1st token generation
+        allow_is_causal_skip = not (getattr(past_key_values, "is_compileable", False) and cache_position.shape[0] == 1)
     else:
         allow_is_causal_skip = not getattr(past_key_values, "is_compileable", False)
Repository: ggml-org/llama.cpp
Pull request: llama-fit-params: fix memory print

Fixes https://github.com/ggml-org/llama.cpp/issues/18119 .

The problem is that the memory for the best fit is not being updated in some cases so the wrong values are being printed.

Changed files:
filename: src/llama.cpp
patch: @@ -542,6 +542,7 @@ static void llama_params_fit_impl(
             } else {
                 assert(ngl_per_device_high[id].n_layer == n_unassigned);
                 ngl_per_device = ngl_per_device_high;
+                mem            = mem_high;
                 LLAMA_LOG_DEBUG("%s: set ngl_per_device[%d].n_layer=%" PRIu32 "\n", __func__, id, ngl_per_device[id].n_layer);
             }
         }
@@ -629,6 +630,7 @@ static void llama_params_fit_impl(
             }
         } else {
             ngl_per_device = ngl_per_device_high;
+            mem            = mem_high;
             id_dense_start = id_dense_start_high;
             LLAMA_LOG_DEBUG("%s: set ngl_per_device[%zu].(n_layer, n_part)=(%" PRIu32 ", %" PRIu32 "), id_dense_start=%zu\n",
                 __func__, id, ngl_per_device[id].n_layer, ngl_per_device[id].n_part, id_dense_start);