helloadhavan/github_issues huggingface dataset:
Repository: huggingface/transformers
Pull request: Fix failing `PhiIntegrationTests`

# What does this PR do?
Fixes these failing [PhiIntegrationTests](https://github.com/huggingface/transformers/actions/runs/20706072009/job/59437072132#step:14:2409).

<img width="1333" height="195" alt="image" src="https://github.com/user-attachments/assets/16c2ba7d-539a-4147-b0c0-2d26da311df2" />

## 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?
@vasqu 

Changed files:
filename: tests/models/phi/test_modeling_phi.py
patch: @@ -85,7 +85,7 @@ def test_model_phi_1_5_logits(self):
             )
         }
 
-        model = PhiForCausalLM.from_pretrained("microsoft/phi-1_5").to(torch_device)
+        model = PhiForCausalLM.from_pretrained("microsoft/phi-1_5", dtype=torch.float32).to(torch_device)
         model.eval()
 
         output = model(**input_ids).logits
@@ -101,7 +101,7 @@ def test_model_phi_2_logits(self):
             )
         }
 
-        model = PhiForCausalLM.from_pretrained("microsoft/phi-2").to(torch_device)
+        model = PhiForCausalLM.from_pretrained("microsoft/phi-2", dtype=torch.float32).to(torch_device)
         model.eval()
 
         output = model(**input_ids).logits

filename: tests/models/phi3/test_modeling_phi3.py
patch: @@ -103,7 +103,9 @@ def test_model_phi3_mini_4k_instruct_logits(self):
             )
         }
 
-        model = Phi3ForCausalLM.from_pretrained("microsoft/phi-3-mini-4k-instruct").to(torch_device)
+        model = Phi3ForCausalLM.from_pretrained("microsoft/phi-3-mini-4k-instruct", dtype=torch.float32).to(
+            torch_device
+        )
         model.eval()
 
         output = model(**input_ids).logits
@@ -169,7 +171,9 @@ def test_model_phi3_mini_128k_instruct_logits(self):
             )
         }
 
-        model = Phi3ForCausalLM.from_pretrained("microsoft/phi-3-mini-128k-instruct").to(torch_device)
+        model = Phi3ForCausalLM.from_pretrained("microsoft/phi-3-mini-128k-instruct", dtype=torch.float32).to(
+            torch_device
+        )
         model.eval()
 
         output = model(**input_ids).logits
Repository: ollama/ollama
Pull request: cuda: enable flash attention

ggml added an option to disable flash attention so explicitly enable it. this caused flash attention kernels to not be build thus forced those operation onto the cpu and caused slowdowns

resolves #9415 

Changed files:
filename: CMakeLists.txt
patch: @@ -23,6 +23,7 @@ set(GGML_SCHED_MAX_COPIES 4)
 set(GGML_LLAMAFILE ON)
 set(GGML_CUDA_PEER_MAX_BATCH_SIZE 128)
 set(GGML_CUDA_GRAPHS ON)
+set(GGML_CUDA_FA ON)
 
 if((CMAKE_OSX_ARCHITECTURES AND NOT CMAKE_OSX_ARCHITECTURES MATCHES "arm64")
     OR (NOT CMAKE_OSX_ARCHITECTURES AND NOT CMAKE_SYSTEM_PROCESSOR MATCHES "arm|aarch64|ARM64|ARMv[0-9]+"))
Repository: ggml-org/llama.cpp
Pull request: common : fix `--n-cpu-moe`, `--cpu-moe` for models with fused gate + up

Changed the regex that matches conditional experts from:

```cpp
const char * const LLM_FFN_EXPS_REGEX = "\\.ffn_(up|down|gate)_(ch|)exps";
```

to:

```cpp
const char * const LLM_FFN_EXPS_REGEX = "\\.ffn_(up|down|gate|gate_up)_(ch|)exps";
```

I think this should fix #20414.

Changed files:
filename: common/common.h
patch: @@ -926,7 +926,7 @@ const char * const LLM_KV_SPLIT_TENSORS_COUNT = "split.tensors.count";
 // MoE utils
 //
 
-const char * const LLM_FFN_EXPS_REGEX = "\\.ffn_(up|down|gate)_(ch|)exps";
+const char * const LLM_FFN_EXPS_REGEX = "\\.ffn_(up|down|gate|gate_up)_(ch|)exps";
 
 inline std::string llm_ffn_exps_block_regex(int idx) {
     return string_format("blk\\.%d%s", idx, LLM_FFN_EXPS_REGEX);
Repository: electron/electron
Pull request: test: fix version color output when running prebuilt Electron

#### 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
-->

Fixes what looks like was a brain fart in #37840. Not going to bother to backport since it's just a minor logging issue.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included

#### 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: script/spec-runner.js
patch: @@ -88,7 +88,7 @@ async function main () {
     }
 
     const versionString = `v${args.electronVersion}`;
-    console.log(`Running against Electron ${versionString.green}`);
+    console.log(`Running against Electron ${chalk.green(versionString)}`);
   }
 
   const [lastSpecHash, lastSpecInstallHash] = loadLastSpecHash();
Repository: ggml-org/llama.cpp
Pull request: llama : whitespace cleanup


Changed files:
filename: src/llama-model.h
patch: @@ -299,7 +299,7 @@ struct llama_layer {
     struct ggml_tensor * ffn_gate_exps_s   = nullptr;
     struct ggml_tensor * ffn_down_exps_s   = nullptr;
     struct ggml_tensor * ffn_up_exps_s     = nullptr;
-  
+
     // ff MoE latent proj
     struct ggml_tensor * ffn_latent_down = nullptr;
     struct ggml_tensor * ffn_latent_up   = nullptr;
Repository: vuejs/vue
Pull request: Make sure global state is restored in the case of an exception

**What kind of change does this PR introduce?** (check at least one)

- [x] Bugfix

**Does this PR introduce a breaking change?** (check one)

- [ ] Yes
- [x] No

**The PR fulfills these requirements:**

- [x] It's submitted to the `dev` branch for v2.x (or to a previous version branch), _not_ the `master` branch
- [ ] When resolving a specific issue, it's referenced in the PR's title (e.g. `fix #xxx[,#xxx]`, where "xxx" is the issue number)
- [x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
- [ ] New/updated tests are included

Changed files:
filename: src/core/util/next-tick.js
patch: @@ -81,9 +81,11 @@ if (typeof Promise !== 'undefined' && isNative(Promise)) {
 export function withMacroTask (fn: Function): Function {
   return fn._withTask || (fn._withTask = function () {
     useMacroTask = true
-    const res = fn.apply(null, arguments)
-    useMacroTask = false
-    return res
+    try {
+      return fn.apply(null, arguments)
+    } finally {
+      useMacroTask = false    
+    }
   })
 }
Repository: vuejs/vue
Pull request: fix(ssr): computed properties first argument

<!--
Please make sure to read the Pull Request Guidelines:
https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#pull-request-guidelines
-->

<!-- PULL REQUEST TEMPLATE -->
<!-- (Update "[ ]" to "[x]" to check a box) -->

**What kind of change does this PR introduce?** (check at least one)

- [x] Bugfix
- [ ] Feature
- [ ] Code style update
- [ ] Refactor
- [ ] Build-related changes
- [ ] Other, please describe:

**Does this PR introduce a breaking change?** (check one)

- [ ] Yes
- [x] No

If yes, please describe the impact and migration path for existing applications:

**The PR fulfills these requirements:**

- [x] It's submitted to the `dev` branch for v2.x (or to a previous version branch), _not_ the `master` branch
- [ ] When resolving a specific issue, it's referenced in the PR's title (e.g. `fix #xxx[,#xxx]`, where "xxx" is the issue number)
- [ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
- [x] New/updated tests are included

**Other information:**

Fixes #8977

Changed files:
filename: src/core/instance/state.js
patch: @@ -216,13 +216,13 @@ export function defineComputed (
   if (typeof userDef === 'function') {
     sharedPropertyDefinition.get = shouldCache
       ? createComputedGetter(key)
-      : userDef
+      : createGetterInvoker(userDef)
     sharedPropertyDefinition.set = noop
   } else {
     sharedPropertyDefinition.get = userDef.get
       ? shouldCache && userDef.cache !== false
         ? createComputedGetter(key)
-        : userDef.get
+        : createGetterInvoker(userDef.get)
       : noop
     sharedPropertyDefinition.set = userDef.set || noop
   }
@@ -253,6 +253,12 @@ function createComputedGetter (key) {
   }
 }
 
+function createGetterInvoker(fn) {
+  return function computedGetter () {
+    return fn.call(this, this)
+  }
+}
+
 function initMethods (vm: Component, methods: Object) {
   const props = vm.$options.props
   for (const key in methods) {

filename: test/ssr/ssr-string.spec.js
patch: @@ -1076,6 +1076,24 @@ describe('SSR: renderToString', () => {
     })
   })
 
+  // #8977
+  it('should call computed properties with vm as first argument', done => {
+    renderToString(new Vue({
+      data: {
+        firstName: 'Evan',
+        lastName: 'You'
+      },
+      computed: {
+        fullName: ({ firstName, lastName }) => `${firstName} ${lastName}`,
+      },
+      template: '<div>{{ fullName }}</div>',
+    }), (err, result) => {
+      expect(err).toBeNull()
+      expect(result).toContain('<div data-server-rendered="true">Evan You</div>')
+      done()
+    })
+  })
+
   it('return Promise', done => {
     renderToString(new Vue({
       template: `<div>{{ foo }}</div>`,
Repository: vercel/next.js
Pull request: fix(turbo-tasks-backend): use correct TaskDataCategory for is_immutable check

## Fix task immutability check by using correct TaskDataCategory

### What?
This PR fixes a bug in the task immutability check by ensuring we use `TaskDataCategory::Meta` instead of `TaskDataCategory::Data` when checking if dependent tasks are immutable.

### Why?
The immutable flag is stored in the Meta category, but we were incorrectly checking the Data category. This could prevent us from reading the immutable flag if Meta hadn't been restored for the task.

### How?
- Changed the category from `TaskDataCategory::Data` to `TaskDataCategory::Meta` when reading tasks where we needed `immutability`
- Added a clarifying comment explaining why we use the Meta category
- Moved the `is_immutable()` implementation from `TaskGuardImpl` to the `TaskGuard` trait with a default implementation that uses `has_key(&CachedDataItemKey::Immutable {})` which enforces correct access modes


Changed files:
filename: turbopack/crates/turbo-tasks-backend/src/backend/mod.rs
patch: @@ -2133,7 +2133,7 @@ impl<B: BackingStorage> TurboTasksBackendInner<B> {
         if let Some(dependencies) = task_dependencies_for_immutable
             && dependencies
                 .iter()
-                .all(|&task_id| ctx.task(task_id, TaskDataCategory::Data).is_immutable())
+                .all(|&task_id| ctx.task(task_id, TaskDataCategory::Meta).is_immutable())
         {
             is_now_immutable = true;
         }

filename: turbopack/crates/turbo-tasks-backend/src/backend/operation/mod.rs
patch: @@ -662,7 +662,9 @@ pub trait TaskGuard: Debug {
     /// Only returns Some once per task.
     /// It returns a set of tasks and which info is needed.
     fn prefetch(&mut self) -> Option<FxIndexMap<TaskId, TaskDataCategory>>;
-    fn is_immutable(&self) -> bool;
+    fn is_immutable(&self) -> bool {
+        self.has_key(&CachedDataItemKey::Immutable {})
+    }
     fn is_dirty(&self) -> bool {
         get!(self, Dirty).is_some_and(|dirtyness| match dirtyness {
             Dirtyness::Dirty => true,
@@ -1007,10 +1009,6 @@ impl<B: BackingStorage> TaskGuard for TaskGuardImpl<'_, B> {
             .collect::<FxIndexMap<_, _>>();
         (map.len() > 1).then_some(map)
     }
-
-    fn is_immutable(&self) -> bool {
-        self.task.contains_key(&CachedDataItemKey::Immutable {})
-    }
 }
 
 macro_rules! impl_operation {
Repository: huggingface/transformers
Pull request: Fix missing UTF-8 encoding in check_repo.py for Windows compatibility

## What does this PR do?

This PR fixes an inconsistent file-opening pattern in `utils/check_repo.py` by explicitly specifying `encoding="utf-8"` when reading Python source files.

Several utilities in `utils/` already open files with an explicit UTF-8 encoding, but this particular code path relied on the platform default. On Windows, this can lead to `UnicodeDecodeError` or inconsistent behavior when parsing Python files containing non-ASCII characters.

This change aligns the implementation with existing patterns across the repository and improves cross-platform robustness without altering any functional logic.

---

## Before submitting
- [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? (Not required for a small mechanical fix)
- [ ] Did you make sure to update the documentation with your changes? (Not applicable)
- [ ] Did you write any new necessary tests? (Not required for this change)

---

## Who can review?

@ydshieh or anyone else in the community after the checks have passed!


Changed files:
filename: utils/check_repo.py
patch: @@ -1230,7 +1230,7 @@ def check_models_have_kwargs():
         if model_dir.is_dir() and (modeling_file := list(model_dir.glob("modeling_*.py"))):
             modeling_file = modeling_file[0]
 
-            with open(modeling_file, "r") as f:
+            with open(modeling_file, "r", encoding="utf-8") as f:
                 tree = ast.parse(f.read())
 
             # Map all classes in the file to their base classes
Repository: nodejs/node
Pull request: fs: validate statfs path

Fixes #61224 

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: lib/internal/fs/promises.js
patch: @@ -1047,7 +1047,7 @@ async function stat(path, options = { bigint: false }) {
 
 async function statfs(path, options = { bigint: false }) {
   const result = await PromisePrototypeThen(
-    binding.statfs(path, options.bigint, kUsePromises),
+    binding.statfs(getValidatedPath(path), options.bigint, kUsePromises),
     undefined,
     handleErrorFromBinding,
   );

filename: test/parallel/test-fs-promises-statfs-validate-path.js
patch: @@ -0,0 +1,12 @@
+'use strict';
+
+const common = require('../common');
+const fs = require('fs');
+const assert = require('assert');
+
+(async () => {
+  await assert.rejects(
+    fs.promises.statfs(),
+    { code: 'ERR_INVALID_ARG_TYPE' },
+  );
+})().then(common.mustCall());
Repository: ggml-org/llama.cpp
Pull request: metal : fix capture_compute counter logic

cont #20398

Changed files:
filename: ggml/src/ggml-metal/ggml-metal-context.m
patch: @@ -465,7 +465,7 @@ enum ggml_status ggml_metal_graph_compute(ggml_metal_t ctx, struct ggml_cgraph *
 
         ctx->n_nodes_per_cb = (ctx->n_nodes_1 + ctx->n_cb - 1) / ctx->n_cb;
 
-        if (ctx->capture_compute > 0) {
+        if (ctx->capture_compute >= 0) {
             ctx->capture_compute--;
         }
Repository: vuejs/vue
Pull request: Fix the env detection of ssr for wechat mini-program

<!--
Please make sure to read the Pull Request Guidelines:
https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#pull-request-guidelines
-->

<!-- PULL REQUEST TEMPLATE -->
<!-- (Update "[ ]" to "[x]" to check a box) -->

**What kind of change does this PR introduce?** (check at least one)

- [x] Bugfix
- [ ] Feature
- [ ] Code style update
- [ ] Refactor
- [ ] Build-related changes
- [ ] Other, please describe:

**Does this PR introduce a breaking change?** (check one)

- [ ] Yes
- [x] No

**Other information:**

In wechat mini-program, `global` does exist, but `global['process']` does not:

![image](https://user-images.githubusercontent.com/13151189/48593214-cc766200-e986-11e8-93d2-d69783288e58.png)


Changed files:
filename: src/core/util/env.js
patch: @@ -41,7 +41,7 @@ export const isServerRendering = () => {
     if (!inBrowser && !inWeex && typeof global !== 'undefined') {
       // detect presence of vue-server-renderer and avoid
       // Webpack shimming the process
-      _isServer = global['process'].env.VUE_ENV === 'server'
+      _isServer = global['process'] && global['process'].env.VUE_ENV === 'server'
     } else {
       _isServer = false
     }
Repository: vuejs/vue
Pull request: fix(ts types): Issue # 8983 Fixing $once type declaration on vue.d.ts file. $once …

Issue #8983 
$once should accept an event or array

Fixing $once type declaration on vue.d.ts file. $once should accept an event or array of events as
first parameter

fix #8983

<!--
Please make sure to read the Pull Request Guidelines:
https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#pull-request-guidelines
-->

<!-- PULL REQUEST TEMPLATE -->
<!-- (Update "[ ]" to "[x]" to check a box) -->

**What kind of change does this PR introduce?** (check at least one)

- [x] Bugfix
- [ ] Feature
- [ ] Code style update
- [ ] Refactor
- [ ] Build-related changes
- [ ] Other, please describe:

**Does this PR introduce a breaking change?** (check one)

- [ ] Yes
- [x] No

If yes, please describe the impact and migration path for existing applications:

**The PR fulfills these requirements:**

- [x] It's submitted to the `dev` branch for v2.x (or to a previous version branch), _not_ the `master` branch
- [x] When resolving a specific issue, it's referenced in the PR's title (e.g. `fix #xxx[,#xxx]`, where "xxx" is the issue number)
- [x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
- [ ] New/updated tests are included

If adding a **new feature**, the PR's description includes:
- [ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

**Other information:**


Changed files:
filename: types/vue.d.ts
patch: @@ -53,7 +53,7 @@ export interface Vue {
     options?: WatchOptions
   ): (() => void);
   $on(event: string | string[], callback: Function): this;
-  $once(event: string, callback: Function): this;
+  $once(event: string | string[], callback: Function): this;
   $off(event?: string | string[], callback?: Function): this;
   $emit(event: string, ...args: any[]): this;
   $nextTick(callback: (this: this) => void): void;
Repository: ollama/ollama
Pull request: runner: default to greedy sampler for performance

As we are adding support for weighted sampling we have seen some performance regressions, bypassing the sampler logic for now and defaulting to greedy until we can benchmark the new sampler logic.

Changed files:
filename: runner/ollamarunner/runner.go
patch: @@ -575,23 +575,11 @@ func (s *Server) completion(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	sampler, err := sample.NewSampler(
-		req.Temperature,
-		req.TopK,
-		req.TopP,
-		req.MinP,
-		req.Seed,
-	)
-	if err != nil {
-		http.Error(w, fmt.Sprintf("Failed to create sampler: %v", err), http.StatusInternalServerError)
-		return
-	}
-
 	seq, err := s.NewSequence(req.Prompt, req.Images, NewSequenceParams{
 		numPredict: req.NumPredict,
 		stop:       req.Stop,
 		numKeep:    int32(req.NumKeep),
-		sampler:    sampler,
+		sampler:    sample.Greedy(), // TODO: add support for different samplers when performance is optimized
 		embedding:  false,
 	})
 	if err != nil {
Repository: electron/electron
Pull request: ci: detect patch needs update error with problem matcher

#### 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
-->

We had a patch failure on `main` which wasn't caught by our audit, so extending this problem matcher to catch this case.

Example run using that most recent failure: https://github.com/electron/electron/actions/runs/21024844609/job/60446804629?pr=49405

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included

#### 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/actions/checkout/action.yml
patch: @@ -143,11 +143,12 @@ runs:
           echo "No changes to patches detected"
         fi
       fi
-  - name: Remove patch conflict problem matcher
+  - name: Remove patch conflict problem matchers
     shell: bash
     run: |
       echo "::remove-matcher owner=merge-conflict::"
       echo "::remove-matcher owner=patch-conflict::"
+      echo "::remove-matcher owner=patch-needs-update::"
   - name: Upload patches stats
     if: ${{ inputs.target-platform == 'linux' && github.ref == 'refs/heads/main' }}
     shell: bash

filename: .github/problem-matchers/patch-conflict.json
patch: @@ -19,6 +19,16 @@
           "line": 3
         }
       ]
+    },
+    {
+      "owner": "patch-needs-update",
+      "pattern": [
+        {
+          "regexp": "^((patches\/.*): needs update)$",
+          "message": 1,
+          "file": 2
+        }
+      ]
     }
   ]
 }