helloadhavan/github_issues huggingface dataset:
Repository: vercel/next.js
Pull request: Bump eslint-plugin-react-hooks to v7

Closes: https://github.com/vercel/next.js/pull/84484

Changed files:
filename: packages/eslint-config-next/package.json
patch: @@ -19,7 +19,7 @@
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-jsx-a11y": "^6.10.0",
     "eslint-plugin-react": "^7.37.0",
-    "eslint-plugin-react-hooks": "^5.0.0"
+    "eslint-plugin-react-hooks": "^7.0.0"
   },
   "peerDependencies": {
     "eslint": "^7.23.0 || ^8.0.0 || ^9.0.0",

filename: pnpm-lock.yaml
patch: @@ -929,8 +929,8 @@ importers:
         specifier: ^7.37.0
         version: 7.37.1(eslint@9.12.0(jiti@2.5.1))
       eslint-plugin-react-hooks:
-        specifier: ^5.0.0
-        version: 5.0.0(eslint@9.12.0(jiti@2.5.1))
+        specifier: ^7.0.0
+        version: 7.0.0(eslint@9.12.0(jiti@2.5.1))
       typescript:
         specifier: '>=3.3.1'
         version: 5.6.3
@@ -9302,6 +9302,12 @@ packages:
     peerDependencies:
       eslint: ^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0-0 || ^9.0.0
 
+  eslint-plugin-react-hooks@7.0.0:
+    resolution: {integrity: sha512-fNXaOwvKwq2+pXiRpXc825Vd63+KM4DLL40Rtlycb8m7fYpp6efrTp1sa6ZbP/Ap58K2bEKFXRmhURE+CJAQWw==}
+    engines: {node: '>=18'}
+    peerDependencies:
+      eslint: ^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0-0 || ^9.0.0
+
   eslint-plugin-react@7.35.0:
     resolution: {integrity: sha512-v501SSMOWv8gerHkk+IIQBkcGRGrO2nfybfj5pLxuJNFTPxxA3PSryhXTK+9pNbtkggheDdsC0E9Q8CuPk6JKA==}
     engines: {node: '>=4'}
@@ -27076,6 +27082,17 @@ snapshots:
     dependencies:
       eslint: 9.12.0(jiti@2.5.1)
 
+  eslint-plugin-react-hooks@7.0.0(eslint@9.12.0(jiti@2.5.1)):
+    dependencies:
+      '@babel/core': 7.26.10
+      '@babel/parser': 7.27.0
+      eslint: 9.12.0(jiti@2.5.1)
+      hermes-parser: 0.25.1
+      zod: 3.25.76
+      zod-validation-error: 3.4.0(zod@3.25.76)
+    transitivePeerDependencies:
+      - supports-color
+
   eslint-plugin-react@7.35.0(eslint@9.12.0(jiti@2.5.1)):
     dependencies:
       array-includes: 3.1.8
Repository: nodejs/node
Pull request: deps: update histogram to 0.11.9

This is an automated update of histogram to 0.11.9.

Changed files:
filename: deps/histogram/include/hdr/hdr_histogram_version.h
patch: @@ -7,6 +7,6 @@
 #ifndef HDR_HISTOGRAM_VERSION_H
 #define HDR_HISTOGRAM_VERSION_H
 
-#define HDR_HISTOGRAM_VERSION "0.11.8"
+#define HDR_HISTOGRAM_VERSION "0.11.9"
 
 #endif  // HDR_HISTOGRAM_VERSION_H

filename: deps/histogram/src/hdr_histogram.c
patch: @@ -341,10 +341,10 @@ static int32_t buckets_needed_to_cover_value(int64_t value, int32_t sub_bucket_c
 /* ##     ## ######## ##     ##  #######  ##     ##    ##    */
 
 int hdr_calculate_bucket_config(
-        int64_t lowest_discernible_value,
-        int64_t highest_trackable_value,
-        int significant_figures,
-        struct hdr_histogram_bucket_config* cfg)
+    int64_t lowest_discernible_value,
+    int64_t highest_trackable_value,
+    int significant_figures,
+    struct hdr_histogram_bucket_config* cfg)
 {
     int32_t sub_bucket_count_magnitude;
     int64_t largest_value_with_single_unit_resolution;
@@ -406,10 +406,10 @@ void hdr_init_preallocated(struct hdr_histogram* h, struct hdr_histogram_bucket_
 }
 
 int hdr_init(
-        int64_t lowest_discernible_value,
-        int64_t highest_trackable_value,
-        int significant_figures,
-        struct hdr_histogram** result)
+    int64_t lowest_discernible_value,
+    int64_t highest_trackable_value,
+    int significant_figures,
+    struct hdr_histogram** result)
 {
     int64_t* counts;
     struct hdr_histogram_bucket_config cfg;
@@ -492,13 +492,12 @@ bool hdr_record_values(struct hdr_histogram* h, int64_t value, int64_t count)
 {
     int32_t counts_index;
 
-    if (value < 0)
+    if (value < 0 || h->highest_trackable_value < value)
     {
         return false;
     }
 
     counts_index = counts_index_for(h, value);
-
     if (counts_index < 0 || h->counts_len <= counts_index)
     {
         return false;
@@ -514,7 +513,7 @@ bool hdr_record_values_atomic(struct hdr_histogram* h, int64_t value, int64_t co
 {
     int32_t counts_index;
 
-    if (value < 0)
+    if (value < 0 || h->highest_trackable_value < value)
     {
         return false;
     }
Repository: facebook/react
Pull request: [compiler] Typedefs for Array.prototype.flatMap

Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom):
* #30801
* #30797
* #30796
* #30795
* __->__ #30794
* #30793



Changed files:
filename: compiler/packages/babel-plugin-react-compiler/src/HIR/ObjectShape.ts
patch: @@ -317,6 +317,23 @@ addObject(BUILTIN_SHAPES, BuiltInArrayId, [
       mutableOnlyIfOperandsAreMutable: true,
     }),
   ],
+  [
+    'flatMap',
+    addFunction(BUILTIN_SHAPES, [], {
+      positionalParams: [],
+      restParam: Effect.ConditionallyMutate,
+      returnType: {kind: 'Object', shapeId: BuiltInArrayId},
+      /*
+       * callee is ConditionallyMutate because items of the array
+       * flow into the lambda and may be mutated there, even though
+       * the array object itself is not modified
+       */
+      calleeEffect: Effect.ConditionallyMutate,
+      returnValueKind: ValueKind.Mutable,
+      noAlias: true,
+      mutableOnlyIfOperandsAreMutable: true,
+    }),
+  ],
   [
     'filter',
     addFunction(BUILTIN_SHAPES, [], {
@@ -534,6 +551,17 @@ addObject(BUILTIN_SHAPES, BuiltInMixedReadonlyId, [
       noAlias: true,
     }),
   ],
+  [
+    'flatMap',
+    addFunction(BUILTIN_SHAPES, [], {
+      positionalParams: [],
+      restParam: Effect.Read,
+      returnType: {kind: 'Object', shapeId: BuiltInArrayId},
+      calleeEffect: Effect.ConditionallyMutate,
+      returnValueKind: ValueKind.Mutable,
+      noAlias: true,
+    }),
+  ],
   [
     'filter',
     addFunction(BUILTIN_SHAPES, [], {
Repository: denoland/deno
Pull request: fix(ext/signals): prevent panic on FreeBSD

This PR fixes a panic in Deno on FreeBSD when running tasks (e.g., deno task georg). The issue occurs because the signal dictionary for FreeBSD incorrectly includes SIGLIBRT (signal 33), which is not defined on FreeBSD, leading to an EINVAL error when attempting to bind or handle signals.

## Changes
1. **Removed `(33, "SIGLIBRT")` from the FreeBSD signal dictionary** (`ext/signals/dict.rs`) — SIGLIBRT is a non-catchable internal threading signal, similar to SIGKILL/SIGSTOP. It should not be in the dictionary of handleable signals.

2. **Replaced `.unwrap()` with error propagation on `add_signal`** (`ext/signals/lib.rs`) — The bare `.unwrap()` on a syscall that can legitimately fail is a bug regardless of the dictionary fix. All existing callers already handle errors gracefully:
   - `listen_and_forward_all_signals` uses `let Ok(...) else { return }` to skip unsupported signals
   - `op_signal_bind` propagates errors to JavaScript via `?`

Related Issue
fixes #31087 

Changed files:
filename: ext/signals/dict.rs
patch: @@ -104,8 +104,7 @@ signal_dict!(
   (29, "SIGINFO"),
   (30, "SIGUSR1"),
   (31, "SIGUSR2"),
-  (32, "SIGTHR"),
-  (33, "SIGLIBRT")
+  (32, "SIGTHR")
 );
 
 #[cfg(target_os = "openbsd")]

filename: ext/signals/lib.rs
patch: @@ -193,7 +193,11 @@ pub fn register(
 
       #[cfg(unix)]
       {
-        handle.0.add_signal(signal).unwrap();
+        handle.0.add_signal(signal).map_err(|e| {
+          std::io::Error::other(format!(
+            "Failed to register signal {signal}: {e}"
+          ))
+        })?;
       }
       #[cfg(windows)]
       {
Repository: electron/electron
Pull request: build: cleanup symlinks in cache

Backport of #47727

See that PR for details.


Notes: <!-- 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 -->none


Changed files:
filename: .github/actions/restore-cache-azcopy/action.yml
patch: @@ -96,7 +96,7 @@ runs:
 
       $TEMP_DIR=New-Item -ItemType Directory -Path temp-cache
       $TEMP_DIR_PATH = $TEMP_DIR.FullName
-      C:\ProgramData\Chocolatey\bin\7z.exe -y x $src_cache -o"$TEMP_DIR_PATH"
+      C:\ProgramData\Chocolatey\bin\7z.exe -y -snld x $src_cache -o"$TEMP_DIR_PATH"
 
   - name: Move Src Cache (Windows)
     if: ${{ inputs.target-platform == 'win' }}
Repository: denoland/deno
Pull request: ci: rename `lint ci status` to `ci status`

Having "lint" in the title has been confusing to all of us.

Changed files:
filename: .github/workflows/ci.generate.ts
patch: @@ -1641,10 +1641,10 @@ const denoCoreMiriJob = job("deno-core-miri", {
   ),
 });
 
-// === lint ci status job (status check gate) ===
+// === ci status job (status check gate) ===
 
-const lintCiStatusJob = job("lint-ci-status", {
-  name: "lint ci status",
+const ciStatusJob = job("ci-status", {
+  name: "ci status",
   // We use this job in the main branch rule status checks for PRs.
   // All jobs that are required to pass on a PR should be listed here.
   needs: [
@@ -1704,7 +1704,7 @@ const workflow = createWorkflow({
     lintJob,
     denoCoreTestJob,
     denoCoreMiriJob,
-    lintCiStatusJob,
+    ciStatusJob,
     publishCanaryJob,
   ],
 });

filename: .github/workflows/ci.yml
patch: @@ -5907,8 +5907,8 @@ jobs:
           rustup component add --toolchain nightly-2025-11-12 miri
           # This somehow prints errors in CI that don't show up locally
           RUSTFLAGS=-Awarnings cargo +nightly-2025-11-12 miri test -p deno_core
-  lint-ci-status:
-    name: lint ci status
+  ci-status:
+    name: ci status
     needs:
       - bench
       - build-debug-macos-x86_64
Repository: ggml-org/llama.cpp
Pull request: Fixing Windows --no-mmap model loading

In the llama_model_loader.cpp the typedef `off_t` was used in a code path which is also used on Windows (PR #18012). On Windows this type is defined as `signed long`, which is too small for file offsets. Changing in this PR `off_t` to `size_t`.

Changed files:
filename: src/llama-model-loader.cpp
patch: @@ -1086,10 +1086,10 @@ bool llama_model_loader::load_all_data(
             } else {
                 // If upload_backend is valid load the tensor in chunks to pinned memory and upload the buffers asynchronously to the GPU.
                 if (upload_backend) {
-                    auto offset = (off_t) weight->offs;
+                    size_t offset = weight->offs;
                     alignment = file->read_alignment();
-                    off_t aligned_offset = offset & ~(alignment - 1);
-                    off_t offset_from_alignment = offset - aligned_offset;
+                    size_t aligned_offset = offset & ~(alignment - 1);
+                    size_t offset_from_alignment = offset - aligned_offset;
                     file->seek(aligned_offset, SEEK_SET);
 
                     // Calculate aligned read boundaries
Repository: electron/electron
Pull request: build: cleanup symlinks in cache

#### 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
-->
This fixes the issue we are seeing on Windows builds: `Dangerous symbolic link path was ignored`
#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd
- [ ] `npm test` passes
- [x] [PR release notes](https://github.com/electron/clerk/blob/main/README.md) describe the change in a way relevant to app developers, and are [capitalized, punctuated, and past tense](https://github.com/electron/clerk/blob/main/README.md#examples).

#### Release Notes

Notes: <!-- 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 -->none


Changed files:
filename: .github/actions/restore-cache-azcopy/action.yml
patch: @@ -96,7 +96,7 @@ runs:
 
       $TEMP_DIR=New-Item -ItemType Directory -Path temp-cache
       $TEMP_DIR_PATH = $TEMP_DIR.FullName
-      C:\ProgramData\Chocolatey\bin\7z.exe -y x $src_cache -o"$TEMP_DIR_PATH"
+      C:\ProgramData\Chocolatey\bin\7z.exe -y -snld x $src_cache -o"$TEMP_DIR_PATH"
 
   - name: Move Src Cache (Windows)
     if: ${{ inputs.target-platform == 'win' }}
Repository: ollama/ollama
Pull request: server: destination invalid

update cmd help too

Changed files:
filename: cmd/cmd.go
patch: @@ -1125,7 +1125,7 @@ Environment Variables:
 		RunE:    ListHandler,
 	}
 	copyCmd := &cobra.Command{
-		Use:     "cp SOURCE TARGET",
+		Use:     "cp SOURCE DESTINATION",
 		Short:   "Copy a model",
 		Args:    cobra.ExactArgs(2),
 		PreRunE: checkServerHeartbeat,

filename: server/routes.go
patch: @@ -810,16 +810,13 @@ func (s *Server) CopyModelHandler(c *gin.Context) {
 
 	src := model.ParseName(r.Source)
 	if !src.IsValid() {
-		_ = c.Error(fmt.Errorf("source %q is invalid", r.Source))
+		c.AbortWithStatusJSON(http.StatusBadRequest, gin.H{"error": fmt.Sprintf("source %q is invalid", r.Source)})
+		return
 	}
 
 	dst := model.ParseName(r.Destination)
 	if !dst.IsValid() {
-		_ = c.Error(fmt.Errorf("destination %q is invalid", r.Destination))
-	}
-
-	if len(c.Errors) > 0 {
-		c.AbortWithStatusJSON(http.StatusBadRequest, gin.H{"error": c.Errors.Errors()})
+		c.AbortWithStatusJSON(http.StatusBadRequest, gin.H{"error": fmt.Sprintf("destination %q is invalid", r.Source)})
 		return
 	}
Repository: ggml-org/llama.cpp
Pull request: llama-server: friendlier error msg when ctx < input

This PR adds formatted strings to the server's send_error function

I was testing out the new cli and I thought that these might be more helpful to debug when reading too large an input 

Changed files:
filename: tools/server/server-context.cpp
patch: @@ -1974,19 +1974,33 @@ struct server_context_impl {
 
                         if (!slot.can_split()) {
                             if (slot.task->n_tokens() > n_ubatch) {
-                                send_error(slot, "input is too large to process. increase the physical batch size", ERROR_TYPE_SERVER);
+                                send_error(slot,
+                                           string_format(
+                                               "input (%d tokens) is too large to process. increase the physical batch "
+                                               "size (current batch size: %d)",
+                                               slot.task->n_tokens(), n_ubatch),
+                                           ERROR_TYPE_SERVER);
                                 slot.release();
                                 continue;
                             }
 
                             if (slot.task->n_tokens() > slot.n_ctx) {
-                                send_error(slot, "input is larger than the max context size. skipping", ERROR_TYPE_EXCEED_CONTEXT_SIZE);
+                                send_error(
+                                    slot,
+                                    string_format(
+                                        "input (%d tokens) is larger than the max context size (%d tokens). skipping",
+                                        slot.task->n_tokens(), slot.n_ctx),
+                                    ERROR_TYPE_EXCEED_CONTEXT_SIZE);
                                 slot.release();
                                 continue;
                             }
                         } else {
                             if (slot.task->n_tokens() >= slot.n_ctx) {
-                                send_error(slot, "the request exceeds the available context size, try increasing it", ERROR_TYPE_EXCEED_CONTEXT_SIZE);
+                                send_error(slot,
+                                           string_format("request (%d tokens) exceeds the available context size (%d "
+                                                         "tokens), try increasing it",
+                                                         slot.task->n_tokens(), slot.n_ctx),
+                                           ERROR_TYPE_EXCEED_CONTEXT_SIZE);
                                 slot.release();
                                 continue;
                             }
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=f2a960b4d82e6b5c9dbbd437378a39489f399c50

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: huggingface/transformers
Pull request: Fix quantization base class 

# What does this PR do?

This PR fixes issues that was created from this [PR](https://github.com/huggingface/transformers/pull/41445) regarding quantization. 

Changed files:
filename: src/transformers/quantizers/base.py
patch: @@ -211,6 +211,9 @@ def update_ep_plan(self, config):
         "updates the tp plan for the scales"
         return config
 
+    def _process_model_before_weight_loading(self, model, **kwargs):
+        return model
+
     def preprocess_model(self, model: "PreTrainedModel", config, dtype=None, checkpoint_files=None, **kwargs):
         """
         Setting model attributes and/or converting model before weights loading. At this point
@@ -345,12 +348,6 @@ def update_state_dict_with_metadata(self, state_dict, metadata):
         """Update state dict with metadata. Default behaviour returns state_dict"""
         return state_dict
 
-    @abstractmethod
-    def _process_model_before_weight_loading(self, model, **kwargs): ...
-
-    @abstractmethod
-    def _process_model_after_weight_loading(self, model, **kwargs): ...
-
     @abstractmethod
     def is_serializable(self, safe_serialization=None): ...
 

filename: src/transformers/quantizers/quantizer_torchao.py
patch: @@ -364,7 +364,7 @@ def preprocess_model(self, model: "PreTrainedModel", config, dtype=None, checkpo
         """
         super().preprocess_model(model, config, dtype, checkpoint_files, **kwargs)
         # Torchao needs access to all metadata later
-        model.set_metadata(checkpoint_files)
+        self.set_metadata(checkpoint_files)
 
     def _process_model_after_weight_loading(self, model, **kwargs):
         """No process required for torchao quantized model"""

filename: tests/quantization/bnb/test_mixed_int8.py
patch: @@ -645,7 +645,7 @@ def test_pipeline(self):
         self.pipe = pipeline(
             "text-generation",
             model=self.model_name,
-            model_kwargs={"device_map": "auto", "load_in_8bit": True},
+            model_kwargs={"device_map": "auto", "quantization_config": BitsAndBytesConfig(load_in_8bit=True)},
             max_new_tokens=self.MAX_NEW_TOKENS,
         )
 
@@ -851,8 +851,7 @@ def test_cpu_accelerator_disk_loading_custom_device_map_kwargs(self):
             model_8bit = AutoModelForCausalLM.from_pretrained(
                 self.model_name,
                 device_map=device_map,
-                quantization_config=BitsAndBytesConfig(load_in_8bit=True),
-                llm_int8_enable_fp32_cpu_offload=True,
+                quantization_config=BitsAndBytesConfig(load_in_8bit=True, llm_int8_enable_fp32_cpu_offload=True),
                 offload_folder=tmpdirname,
             )
Repository: denoland/deno
Pull request: fix(ext/node): handle signal 0 in child_process.kill()

## Summary

- `child_process.kill(0)` threw `Unknown signal: 0` because `toDenoSignal()` didn't recognize signal 0
- In POSIX, signal 0 checks if a process exists without sending a signal (`kill(pid, 0)`)
- Now handles signal `0` (both numeric and string `"0"`) as a special case in `ChildProcess.kill()`, using `process.kill(pid, 0)` which already supports signal 0
- This broke npm packages like `nuxt` that use `kill(0)` to check if a child process is alive on exit

Closes #25776

## Test plan

- [x] Added regression test: `killSignalZero` in `child_process_test.ts`
- [x] `cargo test unit_node::child_process_test` passes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Changed files:
filename: ext/node/polyfills/internal/child_process.ts
patch: @@ -576,6 +576,17 @@ export class ChildProcess extends EventEmitter {
       return this.killed;
     }
 
+    // Signal 0 is a special case: it checks if the process exists
+    // without sending a signal (POSIX kill(pid, 0)).
+    if (signal === 0 || signal === "0") {
+      try {
+        process.kill(this.pid!, 0);
+        return true;
+      } catch {
+        return false;
+      }
+    }
+
     const denoSignal = signal == null ? "SIGTERM" : toDenoSignal(signal);
     this.#closePipes();
     try {

filename: tests/unit_node/child_process_test.ts
patch: @@ -1263,6 +1263,24 @@ Deno.test({
   },
 });
 
+// Regression test for https://github.com/denoland/deno/issues/25776
+Deno.test(async function killSignalZero() {
+  const child = CP.spawn(Deno.execPath(), [
+    "eval",
+    "setTimeout(() => {}, 60_000)",
+  ]);
+  try {
+    // Signal 0 checks if the process exists without sending a signal
+    const alive = child.kill(0);
+    assertEquals(alive, true);
+    // The child should not be marked as killed
+    assertEquals(child.killed, false);
+  } finally {
+    child.kill();
+    await new Promise((resolve) => child.on("close", resolve));
+  }
+});
+
 Deno.test(async function experimentalFlag() {
   const code = ``;
   const file = await Deno.makeTempFile();
Repository: facebook/react
Pull request: Revert "[compiler][eslint] remove compilationMode override; report bailouts on first line"

Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom):
* __->__ #30792

This reverts commit b34b750729bcbcfd80f72f82f46da5bc3e72158f.

This hack doesn't play well internally so I'm reverting this for now
(but keeping the compilationMode override). I'll audit the locations we
report later and try to make them more accurate so we won't need this
workaround.

Changed files:
filename: compiler/packages/eslint-plugin-react-compiler/src/rules/ReactCompilerRule.ts
patch: @@ -166,16 +166,9 @@ const rule: Rule.RuleModule = {
               detail.loc != null && typeof detail.loc !== 'symbol'
                 ? ` (@:${detail.loc.start.line}:${detail.loc.start.column})`
                 : '';
-            const firstLineLoc = {
-              start: event.fnLoc.start,
-              end: {
-                line: event.fnLoc.start.line,
-                column: 10e3,
-              },
-            };
             context.report({
               message: `[ReactCompilerBailout] ${detail.reason}${locStr}`,
-              loc: firstLineLoc,
+              loc: event.fnLoc,
               suggest,
             });
           }