helloadhavan/github_issues huggingface dataset:
Repository: nodejs/node
Pull request: meta: remove unneeded ignore rules from ruff

Ruff doesn't need these ignore rules, as the files are not breaking them. Additionally, these files *should* be placed under the `exclude` section, as they are updated automatically via the dependency updaters, rather than via manual editing.

```
$ make lint-py      
tools/pip/site-packages/bin/ruff --version
ruff 0.5.2
tools/pip/site-packages/bin/ruff check .
All checks passed!
```

Changed files:
filename: pyproject.toml
patch: @@ -5,6 +5,7 @@ exclude = [
   "tools/gyp",
   "tools/inspector_protocol",
   "tools/eslint/node_modules",
+  "tools/github_reporter"
 ]
 line-length = 172
 target-version = "py38"
@@ -40,12 +41,6 @@ ignore = [
 [tool.ruff.lint.mccabe]
 max-complexity = 100
 
-[tool.ruff.lint.per-file-ignores]
-"tools/checkimports.py" = ["W605"]
-"tools/gyp/pylib/gyp/xcodeproj_file.py" = ["PLE0101"]
-"tools/icu/shrink-icu-src.py" = ["W605"]
-"tools/mkssldef.py" = ["W605"]
-
 [tool.ruff.lint.pylint]
 max-args = 12
 max-branches = 110

filename: tools/icu/shrink-icu-src.py
patch: @@ -48,7 +48,7 @@ def print_size(fn):
     size = (os.stat(fn).st_size) / 1024000
     print('%dM\t%s' % (size, fn))
 
-ignore_regex = re.compile('^.*\.(vcxproj|filters|nrm|icu|dat|xml|txt|ac|guess|m4|in|sub|py|mak)$')
+ignore_regex = re.compile(r'^.*\.(vcxproj|filters|nrm|icu|dat|xml|txt|ac|guess|m4|in|sub|py|mak)$')
 
 def icu_ignore(dir, files):
     subdir = dir[len(options.icusrc)+1::]
Repository: vercel/next.js
Pull request: [ts-next-plugin] fix: properly exit when failed to initialize

### Why?

The plugin false-negatively logged that the initialization failed. Also, it didn't properly exit when it failed.

Changed files:
filename: packages/next/src/server/typescript/index.ts
patch: @@ -39,7 +39,12 @@ export const createTSPlugin: tsModule.server.PluginModuleFactory = ({
     // The default user config is { "name": "next" }
     const isPluginEnabled = info.config.enabled ?? true
 
-    if (!isPluginEnabled) {
+    const isPluginInitialized = init({
+      ts,
+      info,
+    })
+
+    if (!isPluginEnabled || !isPluginInitialized) {
       return info.languageService
     }
 
@@ -51,11 +56,6 @@ export const createTSPlugin: tsModule.server.PluginModuleFactory = ({
       proxy[k] = (...args: Array<{}>) => x.apply(info.languageService, args)
     }
 
-    init({
-      ts,
-      info,
-    })
-
     // Auto completion
     proxy.getCompletionsAtPosition = (
       fileName: string,

filename: packages/next/src/server/typescript/utils.ts
patch: @@ -23,7 +23,7 @@ export function log(message: string) {
 export function init(opts: {
   ts: TypeScript
   info: tsModule.server.PluginCreateInfo
-}) {
+}): boolean {
   const projectDir = opts.info.project.getCurrentDirectory()
   ts = opts.ts
   info = opts.info
@@ -48,14 +48,15 @@ export function init(opts: {
     compilerOptions
   )
 
-  if (virtualTsEnv) {
+  if (!virtualTsEnv) {
     log(
       'Failed to create virtual TypeScript environment. This is a bug in Next.js TypeScript plugin. Please report it by opening an issue at https://github.com/vercel/next.js/issues.'
     )
-    return
+    return false
   }
 
   log('Successfully initialized Next.js TypeScript plugin!')
+  return true
 }
 
 export function getTs() {
Repository: ggml-org/llama.cpp
Pull request: SYCL: switch to SYCL namespace

The intel namespace for attribute intel::reqd_sub_group_size has been deprecated in the latest oneapi base toolkit.

Use 'sycl::reqd_sub_group_size' instead.

This is a tiny fix for a compiler warning.


Changed files:
filename: ggml/src/ggml-sycl/norm.cpp
patch: @@ -367,7 +367,7 @@ static void l2_norm_f32_sycl(const float* x, float* dst, const int ncols,
                 sycl::nd_range<3>(sycl::range<3>(1, 1, nrows) * block_dims,
                     block_dims),
                 [=](sycl::nd_item<3> item_ct1)
-                [[intel::reqd_sub_group_size(WARP_SIZE)]] {
+                [[sycl::reqd_sub_group_size(WARP_SIZE)]] {
                     l2_norm_f32(x, dst, ncols, eps, item_ct1,
                         nullptr, WARP_SIZE);
                 });
@@ -389,7 +389,7 @@ static void l2_norm_f32_sycl(const float* x, float* dst, const int ncols,
                 sycl::nd_range<3>(sycl::range<3>(1, 1, nrows) * block_dims,
                     block_dims),
                 [=](sycl::nd_item<3> item_ct1)
-                [[intel::reqd_sub_group_size(WARP_SIZE)]] {
+                [[sycl::reqd_sub_group_size(WARP_SIZE)]] {
                     l2_norm_f32(x, dst, ncols, eps, item_ct1,
                         get_pointer(s_sum_acc_ct1), work_group_size);
                 });
Repository: denoland/deno
Pull request: ci: label for running CI on draft PRs


Changed files:
filename: .github/workflows/ci.generate.ts
patch: @@ -339,10 +339,11 @@ const ci = {
         ...cloneRepoStep,
         {
           id: "check",
+          if: "!contains(github.event.pull_request.labels.*.name, 'ci-draft')",
           run: [
             "GIT_MESSAGE=$(git log --format=%s -n 1 ${{github.event.after}})",
             "echo Commit message: $GIT_MESSAGE",
-            "echo $GIT_MESSAGE | grep '\\[ci\\]' || (echo 'Exiting due to draft PR. Commit with [ci] to bypass.' ; echo 'skip_build=true' >> $GITHUB_OUTPUT)",
+            "echo $GIT_MESSAGE | grep '\\[ci\\]' || (echo 'Exiting due to draft PR. Commit with [ci] to bypass or add the ci-draft label.' ; echo 'skip_build=true' >> $GITHUB_OUTPUT)",
           ].join("\n"),
         },
       ]),

filename: .github/workflows/ci.yml
patch: @@ -37,11 +37,11 @@ jobs:
           submodules: false
         if: github.event.pull_request.draft == true
       - id: check
+        if: 'github.event.pull_request.draft == true && (!contains(github.event.pull_request.labels.*.name, ''ci-draft''))'
         run: |-
           GIT_MESSAGE=$(git log --format=%s -n 1 ${{github.event.after}})
           echo Commit message: $GIT_MESSAGE
-          echo $GIT_MESSAGE | grep '\[ci\]' || (echo 'Exiting due to draft PR. Commit with [ci] to bypass.' ; echo 'skip_build=true' >> $GITHUB_OUTPUT)
-        if: github.event.pull_request.draft == true
+          echo $GIT_MESSAGE | grep '\[ci\]' || (echo 'Exiting due to draft PR. Commit with [ci] to bypass or add the ci-draft label.' ; echo 'skip_build=true' >> $GITHUB_OUTPUT)
   build:
     name: '${{ matrix.job }} ${{ matrix.profile }} ${{ matrix.os }}-${{ matrix.arch }}'
     needs:
Repository: facebook/react
Pull request: [DevTools] Fix Issue in release script where commits for the last DevTools release are undefined

Fixes issue in release script where commits for the last DevTools release are undefined

Before:
![image](https://user-images.githubusercontent.com/2735514/131733011-dceb51c8-bb76-4430-831c-3b793e9d9513.png)

After: 
![image](https://user-images.githubusercontent.com/2735514/131733175-366c386a-7c99-444b-959c-017e3b411807.png)

Verified that this works both from the `root` folder and the `scripts/devtools` folder

Changed files:
filename: scripts/devtools/prepare-release.js
patch: @@ -115,8 +115,12 @@ async function getPreviousCommitSha() {
   const choices = [];
 
   const lines = await execRead(`
-    git log --max-count=5 --topo-order --pretty=format:'%H:::%s:::%as' HEAD -- ${PACKAGE_PATHS[0]}
+    git log --max-count=5 --topo-order --pretty=format:'%H:::%s:::%as' HEAD -- ${join(
+      ROOT_PATH,
+      PACKAGE_PATHS[0]
+    )}
   `);
+
   lines.split('\n').forEach((line, index) => {
     const [hash, message, date] = line.split(':::');
     choices.push({
Repository: facebook/react
Pull request: [DevTools] Remove `dist` build folder in release script before rebuilding

Previously, we weren't removing the `dist` build folder for `react-devtools-inline` and `react-devtools-core` packages before rebuilding in the release script, which caused issues like #22220. This PR updates the release script to do fix this.

Changed files:
filename: scripts/devtools/build-and-test.js
patch: @@ -93,6 +93,9 @@ async function buildAndTestExtensions() {
 
 async function buildAndTestStandalonePackage() {
   const corePackagePath = join(ROOT_PATH, 'packages', 'react-devtools-core');
+  const corePackageDest = join(corePackagePath, 'dist');
+
+  await exec(`rm -rf ${corePackageDest}`);
   const buildCorePromise = exec('yarn build', {cwd: corePackagePath});
 
   await logger(
@@ -133,6 +136,9 @@ async function buildAndTestInlinePackage() {
     'packages',
     'react-devtools-inline'
   );
+  const inlinePackageDest = join(inlinePackagePath, 'dist');
+
+  await exec(`rm -rf ${inlinePackageDest}`);
   const buildPromise = exec('yarn build', {cwd: inlinePackagePath});
 
   await logger(
Repository: vercel/next.js
Pull request: fix: don't reset the prefetch segment data routes on loop

Looping through the built output paths, this doesn't reset the prefetch segment data routes on every loop.

<!-- Thanks for opening a PR! Your contribution is much appreciated.
To make sure your PR is handled as smoothly as possible we request that you follow the checklist sections below.
Choose the right checklist for the change(s) that you're making:

## For Contributors

### Improving Documentation

- Run `pnpm prettier-fix` to fix formatting issues before opening the PR.
- Read the Docs Contribution Guide to ensure your contribution follows the docs guidelines: https://nextjs.org/docs/community/contribution-guide

### Adding or Updating Examples

- The "examples guidelines" are followed from our contributing doc https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md
- Make sure the linting passes by running `pnpm build && pnpm lint`. See https://github.com/vercel/next.js/blob/canary/contributing/repository/linting.md

### Fixing a bug

- Related issues linked using `fixes #number`
- Tests added. See: https://github.com/vercel/next.js/blob/canary/contributing/core/testing.md#writing-tests-for-nextjs
- Errors have a helpful link attached, see https://github.com/vercel/next.js/blob/canary/contributing.md

### Adding a feature

- Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR. (A discussion must be opened, see https://github.com/vercel/next.js/discussions/new?category=ideas)
- Related issues/discussions are linked using `fixes #number`
- e2e tests added (https://github.com/vercel/next.js/blob/canary/contributing/core/testing.md#writing-tests-for-nextjs)
- Documentation added
- Telemetry added. In case of a feature if it's used or not.
- Errors have a helpful link attached, see https://github.com/vercel/next.js/blob/canary/contributing.md


## For Maintainers

- Minimal description (aim for explaining to someone not on the team to understand the PR)
- When linking to a Slack thread, you might want to share details of the conclusion
- Link both the Linear (Fixes NEXT-xxx) and the GitHub issues
- Add review comments if necessary to explain to the reviewer the logic behind a change

### What?

### Why?

### How?

Closes NEXT-
Fixes #

-->


Changed files:
filename: packages/next/src/build/index.ts
patch: @@ -3065,7 +3065,7 @@ export default async function build(
                     throw new Error('Dynamic route not found')
                   }
 
-                  dynamicRoute.prefetchSegmentDataRoutes = []
+                  dynamicRoute.prefetchSegmentDataRoutes ??= []
                   for (const segmentPath of metadata.segmentPaths) {
                     const result = buildPrefetchSegmentDataRoute(
                       route.pathname,
Repository: ggml-org/llama.cpp
Pull request: vulkan: fix build when glslc doesn't support coopmat

@0cc4m


Changed files:
filename: ggml/src/ggml-vulkan/ggml-vulkan.cpp
patch: @@ -3136,7 +3136,9 @@ static void ggml_vk_print_gpu_info(size_t idx) {
                        && shader_integer_dot_product_features.shaderIntegerDotProduct;
 
     coopmat_support = coopmat_support
+#if defined(GGML_VULKAN_COOPMAT_GLSLC_SUPPORT)
                    && coopmat_features.cooperativeMatrix
+#endif
                    && ggml_vk_khr_cooperative_matrix_support(props2.properties, driver_props, device_architecture);
 
     std::string matrix_cores = coopmat2_support ? "NV_coopmat2" : coopmat_support ? "KHR_coopmat" : "none";
Repository: nodejs/node
Pull request: doc: fix typo callsite.lineNumber


Changed files:
filename: doc/api/util.md
patch: @@ -395,7 +395,7 @@ function exampleFunction() {
     console.log(`CallSite ${index + 1}:`);
     console.log(`Function Name: ${callSite.functionName}`);
     console.log(`Script Name: ${callSite.scriptName}`);
-    console.log(`Line Number: ${callSite.lineNumer}`);
+    console.log(`Line Number: ${callSite.lineNumber}`);
     console.log(`Column Number: ${callSite.column}`);
   });
   // CallSite 1:
Repository: denoland/deno
Pull request: fix(console): add missing AssertionError to js

Previously it did not exist, so `assert` would crash the wrong way if it failed.

No issue filed but I can create one if needed, a repro to cause this if you want to test is pasting this in REPL: (sorry)
```js
new Proxy({}, { ownKeys: () => ['foo'], getOwnPropertyDescriptor: () => ({ configurable: false }) })
```

Changed files:
filename: ext/console/01_console.js
patch: @@ -172,6 +172,13 @@ function getStderrNoColor() {
   return noColorStderr();
 }
 
+class AssertionError extends Error {
+  name = "AssertionError";
+  constructor(message) {
+    super(message);
+  }
+}
+
 function assert(cond, msg = "Assertion failed.") {
   if (!cond) {
     throw new AssertionError(msg);
Repository: ggml-org/llama.cpp
Pull request: convert : BailingMoE : avoid setting rope_dim to 0

The Ling-lite-base model has `head_dim` set to 0:
https://huggingface.co/inclusionAI/Ling-lite-base/blob/main/config.json#L44

@bartowski1182 @nicoboss

Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -5146,7 +5146,7 @@ def set_vocab(self):
     def set_gguf_parameters(self):
         super().set_gguf_parameters()
         hparams = self.hparams
-        if "head_dim" in hparams:
+        if hparams.get("head_dim"):
             rope_dim = hparams["head_dim"]
         else:
             rope_dim = hparams["hidden_size"] // hparams["num_attention_heads"]
Repository: vercel/next.js
Pull request: fix(turbopack-cli): Make turbopack_cli::dev::source a persistent (non-transient) task

This was identified using the extra checks in #77760 and debugged using the tracing in #77798.

![Screenshot 2025-04-03 at 1.30.53 PM.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/HAZVitxRNnZz8QMiPn4a/a576884b-f8ab-4266-b9df-916e9733bc85.png)

Closes PACK-4280

Changed files:
filename: turbopack/crates/turbo-tasks/src/task/task_input.rs
patch: @@ -1,4 +1,4 @@
-use std::{any::Any, fmt::Debug, future::Future, hash::Hash, time::Duration};
+use std::{any::Any, fmt::Debug, future::Future, hash::Hash, sync::Arc, time::Duration};
 
 use anyhow::Result;
 use either::Either;
@@ -89,6 +89,23 @@ where
     }
 }
 
+impl<T> TaskInput for Arc<T>
+where
+    T: TaskInput,
+{
+    fn is_resolved(&self) -> bool {
+        self.as_ref().is_resolved()
+    }
+
+    fn is_transient(&self) -> bool {
+        self.as_ref().is_transient()
+    }
+
+    async fn resolve_input(&self) -> Result<Self> {
+        Ok(Arc::new(Box::pin(self.as_ref().resolve_input()).await?))
+    }
+}
+
 impl<T> TaskInput for Option<T>
 where
     T: TaskInput,

filename: turbopack/crates/turbopack-cli/src/dev/mod.rs
patch: @@ -209,7 +209,7 @@ impl TurbopackDevServerBuilder {
             log_detail,
             log_level: self.log_level,
         });
-        let entry_requests = TransientInstance::new(self.entry_requests);
+        let entry_requests = Arc::new(self.entry_requests);
         let tasks = turbo_tasks.clone();
         let issue_provider = self.issue_reporter.unwrap_or_else(|| {
             // Initialize a ConsoleUi reporter if no custom reporter was provided
@@ -220,7 +220,7 @@ impl TurbopackDevServerBuilder {
         struct ServerSourceProvider {
             root_dir: RcStr,
             project_dir: RcStr,
-            entry_requests: TransientInstance<Vec<EntryRequest>>,
+            entry_requests: Arc<Vec<EntryRequest>>,
             eager_compile: bool,
             browserslist_query: RcStr,
         }
@@ -252,7 +252,7 @@ impl TurbopackDevServerBuilder {
 async fn source(
     root_dir: RcStr,
     project_dir: RcStr,
-    entry_requests: TransientInstance<Vec<EntryRequest>>,
+    entry_requests: Arc<Vec<EntryRequest>>,
     eager_compile: bool,
     browserslist_query: RcStr,
 ) -> Result<Vc<Box<dyn ContentSource>>> {
Repository: facebook/react
Pull request: devtools: Don't display hook index of useContext  


## Summary

Resolves https://github.com/facebook/react/pull/20998#issuecomment-865491587

`useContext` will no longer display a (wrong) hook index since it has none (with regards to the hook indices displayed during profiling). This should also affect `useDebugValue` since it also has no ID.

Alternatively, we could start giving non-stateful hooks an index as well (https://github.com/facebook/react/blob/main/packages/react-debug-tools/src/ReactDebugHooks.js). But this can happen in a follow-up since the guard against `hookID` being `null` should happen anyway.

I think this makes more sense to give `useContext` a number from the perspective of a developer using React. Hooks being stateful is somewhat of an implementation detail in my opinion. It's also a bit confusing if the mental model one has doesn't differentiate between the context hook (not stateful) and the actual context (stateful).
Though I don't know why non-stateful hooks were not given an ID. Might be that the use case for the ID simply changed over time since [it was introduced](https://github.com/facebook/react/pull/14906).
There's also an argument for maintainability since `ReactDebugHooks` always needs to know what hooks (by name) are not stateful.

Before:
![context-numbering-before](https://user-images.githubusercontent.com/12292047/131214241-45daefa2-338e-4da6-903f-73c0c1c79b56.png)

After:
![context-numbering-after](https://user-images.githubusercontent.com/12292047/131214244-0b68e58b-1f81-42c7-99b3-f0d6de021015.png)


## Test Plan

- [x] Verified manually with sheel app by including context hooks
- [x] CI green

Changed files:
filename: packages/react-devtools-shared/src/devtools/views/Components/InspectedElementHooksTree.js
patch: @@ -222,7 +222,7 @@ function HookView({
 
   let name = hook.name;
   if (enableProfilerChangedHookIndices) {
-    if (!isCustomHook) {
+    if (hookID !== null) {
       name = (
         <>
           <span className={styles.PrimitiveHookNumber}>{hookID + 1}</span>

filename: packages/react-devtools-shell/src/app/InspectableElements/CustomHooks.js
patch: @@ -9,10 +9,12 @@
 
 import * as React from 'react';
 import {
+  createContext,
   forwardRef,
   Fragment,
   memo,
   useCallback,
+  useContext,
   useDebugValue,
   useEffect,
   useState,
@@ -64,8 +66,13 @@ function useDeepHookF() {
   useDebugValue('useDeepHookF');
 }
 
+const ContextA = createContext('A');
+const ContextB = createContext('B');
+
 function FunctionWithHooks(props: any, ref: React$Ref<any>) {
   const [count, updateCount] = useState(0);
+  // eslint-disable-next-line no-unused-vars
+  const contextValueA = useContext(ContextA);
 
   // eslint-disable-next-line no-unused-vars
   const [_, __] = useState(object);
@@ -85,6 +92,9 @@ function FunctionWithHooks(props: any, ref: React$Ref<any>) {
   // Tests nested custom hooks
   useNestedOuterHook();
 
+  // eslint-disable-next-line no-unused-vars
+  const contextValueB = useContext(ContextB);
+
   // Verify deep nesting doesn't break
   useDeepHookA();
Repository: vercel/next.js
Pull request: [test] temporarily disable flaky test for react 18

temporarily disable the react 18 tests for toggling the frames, it's not a quick fix so I'll follow up in another PR.

Changed files:
filename: test/development/acceptance/ReactRefreshLogBox.test.ts
patch: @@ -1161,7 +1161,10 @@ describe('ReactRefreshLogBox', () => {
       `)
     }
 
-    await toggleCollapseCallStackFrames(browser)
+    // TODO: fix the broken collapsing in callstack
+    if (!isReact18) {
+      await toggleCollapseCallStackFrames(browser)
+    }
 
     // Expect more than the default amount of frames
     // The default stackTraceLimit results in max 9 [data-nextjs-call-stack-frame] elements
Repository: ggml-org/llama.cpp
Pull request: convert : Qwerky : use lora_rank_tokenshift and lora_rank_decay if present

The config items `lora_rank_tokenshift` and `lora_rank_decay` were introduced in new release, see:
https://huggingface.co/featherless-ai/Qwerky-72B/blob/main/modeling_rwkv6qwen2.py#L268-L279

Fixes #12662

Changed files:
filename: convert_hf_to_gguf.py
patch: @@ -3557,8 +3557,8 @@ def set_gguf_parameters(self):
         head_size = hidden_size // num_attention_heads
         rms_norm_eps = self.hparams["rms_norm_eps"]
         intermediate_size = self.hparams["intermediate_size"]
-        time_mix_extra_dim = 64 if hidden_size >= 4096 else 32
-        time_decay_extra_dim = 128 if hidden_size >= 4096 else 64
+        time_mix_extra_dim = self.hparams.get("lora_rank_tokenshift", 64 if hidden_size >= 4096 else 32)
+        time_decay_extra_dim = self.hparams.get("lora_rank_decay", 128 if hidden_size >= 4096 else 64)
 
         # RWKV isn't context limited
         self.gguf_writer.add_context_length(1048576)
Repository: huggingface/transformers
Pull request: fixed typo


Changed files:
filename: src/transformers/generation/utils.py
patch: @@ -1232,7 +1232,7 @@ def compute_transition_scores(
     ) -> torch.Tensor:
         """
         Computes the transition scores of sequences given the generation scores (and beam indices, if beam search was
-        used). This is a convenient method to quicky obtain the scores of the selected tokens at generation time.
+        used). This is a convenient method to quickly obtain the scores of the selected tokens at generation time.
 
         Parameters:
             sequences (`torch.LongTensor`):
Repository: ggml-org/llama.cpp
Pull request: sync : ggml


Changed files:
filename: ggml/src/ggml-vulkan/CMakeLists.txt
patch: @@ -59,7 +59,6 @@ if (Vulkan_FOUND)
             set(GGML_VULKAN_COOPMAT2_GLSLC_SUPPORT OFF CACHE INTERNAL "Whether coopmat2 is supported by glslc")
         else()
             message(STATUS "GL_NV_cooperative_matrix2 supported by glslc")
-            
             set(GGML_VULKAN_COOPMAT2_GLSLC_SUPPORT ON CACHE INTERNAL "Whether coopmat2 is supported by glslc")
         endif()
     else()
Repository: ggml-org/llama.cpp
Pull request: llava : proper description fix

Use `gguf_find_key` instead of `get_key_idx` to avoid throwing error.

Changed files:
filename: examples/llava/clip.cpp
patch: @@ -1401,7 +1401,7 @@ struct clip_ctx * clip_init(const char * fname, struct clip_context_params ctx_p
             const std::string name = gguf_get_val_str(ctx, idx_name);
             LOG_INF("%s: model name:   %s\n", __func__, name.c_str());
         }
-        const int idx_desc = get_key_idx(ctx, KEY_DESCRIPTION);
+        const int idx_desc = gguf_find_key(ctx, KEY_DESCRIPTION);
         if (idx_desc != -1) { // ditto
             const std::string description = gguf_get_val_str(ctx, idx_desc);
             LOG_INF("%s: description:  %s\n", __func__, description.c_str());