helloadhavan/github_issues huggingface dataset:
Repository: denoland/deno
Pull request: fix: dont strip n-api symbols in `denort` on mac

Fixes https://github.com/denoland/deno/issues/24614

Changed files:
filename: .github/workflows/ci.generate.ts
patch: @@ -828,7 +828,7 @@ const ci = {
             "cd target/release",
             "zip -r deno-${{ matrix.arch }}-apple-darwin.zip deno",
             "shasum -a 256 deno-${{ matrix.arch }}-apple-darwin.zip > deno-${{ matrix.arch }}-apple-darwin.zip.sha256sum",
-            "strip ./denort",
+            "strip -x -S ./denort",
             "zip -r denort-${{ matrix.arch }}-apple-darwin.zip denort",
             "shasum -a 256 denort-${{ matrix.arch }}-apple-darwin.zip > denort-${{ matrix.arch }}-apple-darwin.zip.sha256sum",
           ]

filename: .github/workflows/ci.yml
patch: @@ -492,7 +492,7 @@ jobs:
           cd target/release
           zip -r deno-${{ matrix.arch }}-apple-darwin.zip deno
           shasum -a 256 deno-${{ matrix.arch }}-apple-darwin.zip > deno-${{ matrix.arch }}-apple-darwin.zip.sha256sum
-          strip ./denort
+          strip -x -S ./denort
           zip -r denort-${{ matrix.arch }}-apple-darwin.zip denort
           shasum -a 256 denort-${{ matrix.arch }}-apple-darwin.zip > denort-${{ matrix.arch }}-apple-darwin.zip.sha256sum
       - name: Pre-release (windows)
Repository: ggml-org/llama.cpp
Pull request: batched-bench : fix unified KV cache handling + pp timing

- Take into account split KV cache when computing `N_KV`
- Do not measure KV cache copying when `-pps` is enabled

Changed files:
filename: tools/batched-bench/batched-bench.cpp
patch: @@ -124,7 +124,7 @@ int main(int argc, char ** argv) {
                 const int tg = n_tg[i_tg];
                 const int pl = n_pl[i_pl];
 
-                const int n_ctx_req = is_pp_shared ? pp + pl*tg : pl*(pp + tg);
+                const int n_ctx_req = is_pp_shared ? (params.kv_unified ? pp : pl*pp) + pl*tg : pl*(pp + tg);
 
                 if (n_ctx_req > n_kv_max) {
                     continue;
@@ -147,13 +147,24 @@ int main(int argc, char ** argv) {
                     return 1;
                 }
 
+                const auto t_pp_end = ggml_time_us();
+
                 if (is_pp_shared) {
                     for (int32_t i = 1; i < pl; ++i) {
                         llama_memory_seq_cp(mem, 0, i, -1, -1);
                     }
-                }
 
-                const auto t_pp_end = ggml_time_us();
+                    if (!params.kv_unified) {
+                        // run one dummy token to apply the memory copy
+                        common_batch_clear(batch);
+                        common_batch_add(batch, get_token_rand(), pp + 0, { 0 }, true);
+                        if (!decode_helper(ctx, batch, ctx_params.n_batch)) {
+                            LOG_ERR("%s: llama_decode() failed\n", __func__);
+                            return 1;
+                        }
+                        llama_memory_seq_rm(mem, 0, pp, -1);
+                    }
+                }
 
                 const auto t_tg_start = ggml_time_us();
Repository: nodejs/node
Pull request: http2: use args.This() instead of args.Holder()

args.Holder() has been removed in newer version of V8.

Blocks https://github.com/nodejs/node/pull/57753
<!--
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: src/node_http2.cc
patch: @@ -768,7 +768,7 @@ void Http2Stream::EmitStatistics() {
 
 void Http2Session::HasPendingData(const FunctionCallbackInfo<Value>& args) {
   Http2Session* session;
-  ASSIGN_OR_RETURN_UNWRAP(&session, args.Holder());
+  ASSIGN_OR_RETURN_UNWRAP(&session, args.This());
   args.GetReturnValue().Set(session->HasPendingData());
 }
 
@@ -3578,7 +3578,7 @@ void Initialize(Local<Object> target,
 
 void Http2Session::SetGracefulClose(const FunctionCallbackInfo<Value>& args) {
   Http2Session* session;
-  ASSIGN_OR_RETURN_UNWRAP(&session, args.Holder());
+  ASSIGN_OR_RETURN_UNWRAP(&session, args.This());
   CHECK_NOT_NULL(session);
   // Set the graceful close flag
   session->SetGracefulCloseInitiated(true);
Repository: denoland/deno
Pull request: fix: use full SHA for canary panic URLs

Use full commit sha as one may not be able to resolve commits with the short sha because of duplicates.

Changed files:
filename: cli/lib/version.rs
patch: @@ -15,7 +15,7 @@ pub fn otel_runtime_config() -> OtelRuntimeConfig {
 
 const GIT_COMMIT_HASH: &str = env!("GIT_COMMIT_HASH");
 const TYPESCRIPT: &str = "5.7.3";
-const DENO_VERSION: &str = env!("DENO_VERSION");
+pub const DENO_VERSION: &str = env!("DENO_VERSION");
 // TODO(bartlomieju): ideally we could remove this const.
 const IS_CANARY: bool = option_env!("DENO_CANARY").is_some();
 // TODO(bartlomieju): this is temporary, to allow Homebrew to cut RC releases as well

filename: cli/main.rs
patch: @@ -370,11 +370,19 @@ fn setup_panic_hook() {
     eprintln!("Args: {:?}", env::args().collect::<Vec<_>>());
     eprintln!();
 
+    let info = &deno_lib::version::DENO_VERSION_INFO;
+    let version =
+      if info.release_channel == deno_lib::shared::ReleaseChannel::Canary {
+        format!("{}+{}", deno_lib::version::DENO_VERSION, info.git_hash)
+      } else {
+        info.deno.to_string()
+      };
+
     let trace = deno_panic::trace();
     eprintln!("View stack trace at:");
     eprintln!(
       "https://panic.deno.com/v{}/{}/{}",
-      deno_lib::version::DENO_VERSION_INFO.deno,
+      version,
       env!("TARGET"),
       trace
     );
Repository: vercel/next.js
Pull request: [segment explorer] fix route path in top navbar

* Remove the `app` prefix in top bar of route info as we only need to know the pathname. It's also removed in the new design
* `state.page` was only passed for panel ui but missing in segment explorer of dock indicator 

Changed files:
filename: packages/next/src/next-devtools/dev-overlay/components/errors/dev-tools-indicator/dev-tools-indicator.tsx
patch: @@ -64,6 +64,7 @@ export function DevToolsIndicator({
       isTurbopack={!!process.env.TURBOPACK}
       disabled={state.disableDevIndicator || !isDevToolsIndicatorVisible}
       isBuildError={isBuildError}
+      page={state.page}
       {...props}
     />
   )
@@ -104,6 +105,7 @@ function DevToolsPopover({
   dispatch,
   scale,
   setScale,
+  page,
 }: {
   routerType: 'pages' | 'app'
   disabled: boolean
@@ -118,6 +120,7 @@ function DevToolsPopover({
   dispatch: OverlayDispatch
   scale: DevToolsScale
   setScale: (value: DevToolsScale) => void
+  page: string
 }) {
   const menuRef = useRef<HTMLDivElement>(null)
   const triggerRef = useRef<HTMLButtonElement | null>(null)
@@ -343,8 +346,7 @@ function DevToolsPopover({
           triggerRef={triggerRef}
           style={popover}
           routerType={routerType}
-          // dummy page for legacy segment explorer, will be removed in the future
-          page={''}
+          page={page}
         />
       ) : null}
 

filename: packages/next/src/next-devtools/dev-overlay/components/overview/segment-explorer.tsx
patch: @@ -18,11 +18,10 @@ const isFileNode = (node: SegmentTrieNode) => {
 }
 
 function PageRouteBar({ page }: { page: string }) {
-  const pagePath = `/app${page}`
   return (
     <div className="segment-explorer-page-route-bar">
       <BackArrowIcon />
-      <span className="segment-explorer-page-route-bar-path">{pagePath}</span>
+      <span className="segment-explorer-page-route-bar-path">{page}</span>
     </div>
   )
 }
Repository: ggml-org/llama.cpp
Pull request: convert : update Ernie 4.5 0.3B model name

 Baidu Ernie 4.5 0.3B model have the name change from `Ernie4_5_ForCausalLM` to `Ernie4_5ForCausalLM`.

- https://huggingface.co/baidu/ERNIE-4.5-0.3B-PT/discussions/4
- https://huggingface.co/baidu/ERNIE-4.5-0.3B-Base-PT/discussions/4


Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -3159,7 +3159,7 @@ def modify_tensors(self, data_torch: Tensor, name: str, bid: int | None) -> Iter
         yield from super().modify_tensors(data_torch, name, bid)
 
 
-@ModelBase.register("Ernie4_5_ForCausalLM")
+@ModelBase.register("Ernie4_5_ForCausalLM", "Ernie4_5ForCausalLM")
 class Ernie4_5Model(TextModel):
     model_arch = gguf.MODEL_ARCH.ERNIE4_5
Repository: denoland/deno
Pull request: fix(build): upgrade libffi to 4.0.0


Changed files:
filename: Cargo.lock
patch: @@ -5113,19 +5113,19 @@ checksum = "c19937216e9d3aa9956d9bb8dfc0b0c8beb6058fc4f7a4dc4d850edf86a237d6"
 
 [[package]]
 name = "libffi"
-version = "3.2.0"
+version = "4.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ce826c243048e3d5cec441799724de52e2d42f820468431fc3fceee2341871e2"
+checksum = "4a9434b6fc77375fb624698d5f8c49d7e80b10d59eb1219afda27d1f824d4074"
 dependencies = [
  "libc",
  "libffi-sys",
 ]
 
 [[package]]
 name = "libffi-sys"
-version = "2.3.0"
+version = "3.2.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f36115160c57e8529781b4183c2bb51fdc1f6d6d1ed345591d84be7703befb3c"
+checksum = "ead36a2496acfc8edd6cc32352110e9478ac5b9b5f5b9856ebd3d28019addb84"
 dependencies = [
  "cc",
 ]

filename: Cargo.toml
patch: @@ -215,8 +215,8 @@ k256 = "0.13.1"
 lazy-regex = "3"
 lcms2 = "6.1.0"
 libc = "0.2.168"
-libffi = "=3.2.0"
-libffi-sys = "=2.3.0"
+libffi = "=4.0.0"
+libffi-sys = "=3.2.0"
 libloading = "0.7"
 libsqlite3-sys = "0.30.1"
 libsui = "0.7.0"
Repository: electron/electron
Pull request: fix: Update widget visibility in `NativeWindowMac::ShowInactive`

Manual backport of #45427

See that PR for details

Notes: none

Changed files:
filename: shell/browser/native_window_mac.mm
patch: @@ -429,7 +429,13 @@ static bool FromV8(v8::Isolate* isolate,
   if (parent())
     InternalSetParentWindow(parent(), true);
 
+  // Triggers `NativeWidgetMacNSWindowHost::OnVisibilityChanged`.
+  widget()->ShowInactive();
+  // `Widget::ShowInactive` is not sufficient to bring window to front.
   [window_ orderFrontRegardless];
+  // Above calls do not trigger `orderWindow: relativeTo:` in which headless
+  // mode is being disabled.
+  [window_ disableHeadlessMode];
 }
 
 void NativeWindowMac::Hide() {

filename: shell/browser/ui/cocoa/electron_ns_window.h
patch: @@ -49,6 +49,7 @@ class ElectronNativeWindowObserver;
 - (NSRect)originalContentRectForFrameRect:(NSRect)frameRect;
 - (BOOL)toggleFullScreenMode:(id)sender;
 - (NSImage*)_cornerMask;
+- (void)disableHeadlessMode;
 @end
 
 #endif  // ELECTRON_SHELL_BROWSER_UI_COCOA_ELECTRON_NS_WINDOW_H_

filename: shell/browser/ui/cocoa/electron_ns_window.mm
patch: @@ -259,13 +259,7 @@ - (void)setFrame:(NSRect)windowFrame display:(BOOL)displayViews {
 }
 
 - (void)orderWindow:(NSWindowOrderingMode)place relativeTo:(NSInteger)otherWin {
-  if (shell_) {
-    // We initialize the window in headless mode to allow painting before it is
-    // shown, but we don't want the headless behavior of allowing the window to
-    // be placed unconstrained.
-    self.isHeadless = false;
-    shell_->widget()->DisableHeadlessMode();
-  }
+  [self disableHeadlessMode];
   [super orderWindow:place relativeTo:otherWin];
 }
 
@@ -328,6 +322,16 @@ - (NSImage*)_cornerMask {
   }
 }
 
+- (void)disableHeadlessMode {
+  if (shell_) {
+    // We initialize the window in headless mode to allow painting before it is
+    // shown, but we don't want the headless behavior of allowing the window to
+    // be placed unconstrained.
+    self.isHeadless = false;
+    shell_->widget()->DisableHeadlessMode();
+  }
+}
+
 // Quicklook methods
 
 - (BOOL)acceptsPreviewPanelControl:(QLPreviewPanel*)panel {
Repository: vercel/next.js
Pull request: [dynamicIO] recognize prerender interrupted errors as well known

all Next.js errors should pass through the RSC boundary with an in tact digest

Changed files:
filename: packages/next/src/server/app-render/create-error-handler.tsx
patch: @@ -7,6 +7,7 @@ import { isAbortError } from '../pipe-readable'
 import { isBailoutToCSRError } from '../../shared/lib/lazy-dynamic/bailout-to-csr'
 import { isDynamicServerError } from '../../client/components/hooks-server-context'
 import { isNextRouterError } from '../../client/components/is-next-router-error'
+import { isPrerenderInterruptedError } from './dynamic-rendering'
 import { getProperError } from '../../lib/is-error'
 import { createDigestWithErrorCode } from '../../lib/error-telemetry-utils'
 import { isReactLargeShellError } from './react-large-shell-error'
@@ -41,6 +42,9 @@ export function getDigestForWellKnownError(error: unknown): string | undefined {
   // tell the user about this error, as it's not actionable.
   if (isDynamicServerError(error)) return error.digest
 
+  // If this is a prerender interrupted error, we don't need to log the error.
+  if (isPrerenderInterruptedError(error)) return error.digest
+
   return undefined
 }
Repository: electron/electron
Pull request: test: dump Electron process PID to path if environment variable set

Backport of #45481

See that PR for details.


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: @@ -164,6 +164,9 @@ async function asyncSpawn (exe, runnerArgs) {
     const child = childProcess.spawn(exe, runnerArgs, {
       cwd: path.resolve(__dirname, '../..')
     });
+    if (process.env.ELECTRON_TEST_PID_DUMP_PATH && child.pid) {
+      fs.writeFileSync(process.env.ELECTRON_TEST_PID_DUMP_PATH, child.pid.toString());
+    }
     child.stdout.pipe(process.stdout);
     child.stderr.pipe(process.stderr);
     if (process.env.ELECTRON_FORCE_TEST_SUITE_EXIT) {
Repository: huggingface/transformers
Pull request: Fix patch helper

# What does this PR do?

Since it's for a patch there should not be a -1

Changed files:
filename: utils/patch_helper.py
patch: @@ -56,8 +56,6 @@ def get_release_branch_name():
         major -= 1
         # You'll need logic to determine the last minor of the previous major version
         raise ValueError("Minor version is 0; need logic to find previous major version's last minor")
-    else:
-        minor -= 1
 
     return f"v{major}.{minor}-release"
Repository: facebook/react
Pull request: [Flight Reply] Update error message

This is in the reverse direction.

Changed files:
filename: packages/react-client/src/ReactFlightReplyClient.js
patch: @@ -304,22 +304,22 @@ export function processReply(
           );
         } else if (objectName(value) !== 'Object') {
           console.error(
-            'Only plain objects can be passed to Client Components from Server Components. ' +
+            'Only plain objects can be passed to Server Functions from the Client. ' +
               '%s objects are not supported.%s',
             objectName(value),
             describeObjectForErrorMessage(parent, key),
           );
         } else if (!isSimpleObject(value)) {
           console.error(
-            'Only plain objects can be passed to Client Components from Server Components. ' +
+            'Only plain objects can be passed to Server Functions from the Client. ' +
               'Classes or other objects with methods are not supported.%s',
             describeObjectForErrorMessage(parent, key),
           );
         } else if (Object.getOwnPropertySymbols) {
           const symbols = Object.getOwnPropertySymbols(value);
           if (symbols.length > 0) {
             console.error(
-              'Only plain objects can be passed to Client Components from Server Components. ' +
+              'Only plain objects can be passed to Server Functions from the Client. ' +
                 'Objects with symbol properties like %s are not supported.%s',
               symbols[0].description,
               describeObjectForErrorMessage(parent, key),
Repository: vercel/next.js
Pull request: Remove unneccesary step that's making workflow fail

This step was causing the front-sync job to fail. I'm not totally sure how it was working before, some other workflow must have logged in at some point. This is just a debug step anyway, so I think we can delete it. **I confirmed that the next step worked just fine.** We could also add a step that runs `gh login` if we think it would actually be useful.


Closes PACK-4976

Changed files:
filename: .github/workflows/test_e2e_deploy_release.yml
patch: @@ -108,10 +108,6 @@ jobs:
       - id: test-result
         name: Set test result variable
         run: echo 'immediately-close=${{ needs.test-deploy.result != 'success' && 'true' || 'false' }}' >> "$GITHUB_OUTPUT"
-      - name: Check token
-        run: gh auth status
-        env:
-          github-token: ${{ secrets.GH_UPDATE_NEXT_WORKFLOW_TRIGGER }}
       - uses: actions/github-script@v7
         name: Trigger vercel/front sync
         id: trigger-front-sync
Repository: electron/electron
Pull request: test: dump Electron process PID to path if environment variable set

#### 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
-->

Tiny change in service of making it easier to attach a native debugger to the Electron test process, e.g. `ELECTRON_TEST_PID_DUMP_PATH=/foo/bar e test --wait-for-debugger`.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd
- [x] `npm test` passes

#### 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: @@ -164,6 +164,9 @@ async function asyncSpawn (exe, runnerArgs) {
     const child = childProcess.spawn(exe, runnerArgs, {
       cwd: path.resolve(__dirname, '../..')
     });
+    if (process.env.ELECTRON_TEST_PID_DUMP_PATH && child.pid) {
+      fs.writeFileSync(process.env.ELECTRON_TEST_PID_DUMP_PATH, child.pid.toString());
+    }
     child.stdout.pipe(process.stdout);
     child.stderr.pipe(process.stderr);
     if (process.env.ELECTRON_FORCE_TEST_SUITE_EXIT) {
Repository: facebook/react
Pull request: [meta-only] Move ReactServerStreamConfigFB to `react-server` from `react-server-dom-fb`

Code organization PR.

It looks like the `ReactServerStreamConfigFB` is only used in the `relay-server` package. This PR moves it to `react-server` from `react-server-dom-fb` (similar to how we have config for bun, for example). This avoids cross-package imports from `react-server` to `react-server-dom-fb.`

Changed files:
filename: packages/react-server/src/ReactServerStreamConfigFB.js
patch: @@ -84,4 +84,4 @@ export function closeWithError(destination: Destination, error: mixed): void {
   destination.error = error;
 }
 
-export {createFastHashJS as createFastHash} from 'react-server/src/createFastHashJS';
+export {createFastHashJS as createFastHash} from './createFastHashJS';

filename: packages/react-server/src/forks/ReactServerStreamConfig.dom-fb.js
patch: @@ -7,4 +7,4 @@
  * @flow
  */
 
-export * from '../../../react-server-dom-fb/src/ReactServerStreamConfigFB';
+export * from '../ReactServerStreamConfigFB';
Repository: nodejs/node
Pull request: doc: fix `AsyncLocalStorage` example response changes after node v18

I can confirm that the change in behavior started with **Node.js 20.x**.

### ✅ Node.js v18.20.4 – Works as expected:
```
0: start  
1: start  
0: finish  
1: finish
```

### ❗ Node.js v20.17.0 and v22.11.0 – Changed behavior:
```
0: start  
0: finish  
1: start  
1: finish
```

According to the official [Node.js changelog](https://nodejs.org/en/blog/release/v20.0.0#notable-changes), this difference is due to **bug fixes and improvements in async context handling** introduced in v20:

> "Several subtle bugs and inconsistencies in edge cases (e.g., using `AsyncResource`, event emitters, timers) have been fixed in v20.  
> Node.js now handles `setTimeout`, `setImmediate`, and event emitters more predictably in relation to the async context."

So if your code depends on `AsyncLocalStorage` or similar context-tracking features, be aware that behavior may differ between Node 18 and 20+ due to these changes.


Changed files:
filename: doc/api/async_context.md
patch: @@ -75,8 +75,8 @@ http.get('http://localhost:8080');
 http.get('http://localhost:8080');
 // Prints:
 //   0: start
-//   1: start
 //   0: finish
+//   1: start
 //   1: finish
 ```
 
@@ -107,8 +107,8 @@ http.get('http://localhost:8080');
 http.get('http://localhost:8080');
 // Prints:
 //   0: start
-//   1: start
 //   0: finish
+//   1: start
 //   1: finish
 ```