helloadhavan/github_issues huggingface dataset:
Repository: facebook/react
Pull request: console test utils fix: match entire string, not just first letter

Fixes issue where if the first letter of the expected string appeared anywhere in actual message, the assertion would pass, leading to false negatives. We should check the entire expected string.

Changed files:
filename: packages/internal-test-utils/__tests__/ReactInternalTestUtils-test.js
patch: @@ -2129,6 +2129,28 @@ describe('ReactInternalTestUtils console assertions', () => {
       `);
     });
 
+    // @gate __DEV__
+    it('regression: checks entire string, not just the first letter', async () => {
+      const message = expectToThrowFailure(() => {
+        console.error('Message that happens to contain a "T"\n    in div');
+
+        assertConsoleErrorDev([
+          'This is a completely different message that happens to start with "T"',
+        ]);
+      });
+      expect(message).toMatchInlineSnapshot(`
+        "assertConsoleErrorDev(expected)
+
+        Unexpected error(s) recorded.
+
+        - Expected errors
+        + Received errors
+
+        - This is a complete different message that happens to start with "T"
+        + Message that happens to contain a "T" <component stack>"
+      `);
+    });
+
     describe('global withoutStack', () => {
       // @gate __DEV__
       it('passes if errors without stack explicitly opt out', () => {

filename: packages/internal-test-utils/consoleMock.js
patch: @@ -386,7 +386,7 @@ export function createLogAssertion(
           expectedWithoutStack = expectedMessageOrArray[1].withoutStack;
         } else if (typeof expectedMessageOrArray === 'string') {
           // Should be in the form assert(['log']) or assert(['log'], {withoutStack: true})
-          expectedMessage = replaceComponentStack(expectedMessageOrArray[0]);
+          expectedMessage = replaceComponentStack(expectedMessageOrArray);
           if (consoleMethod === 'log') {
             expectedWithoutStack = true;
           } else {
Repository: ggml-org/llama.cpp
Pull request: mtmd: allow QwenVL to process larger image by default

Since we have flash attn enabled by default now, QwenVL can use max=4096 tokens with less memory

Also removed some obsoleted comments


Changed files:
filename: tools/mtmd/clip.cpp
patch: @@ -2791,14 +2791,8 @@ struct clip_model_loader {
                         get_u32(KEY_SPATIAL_MERGE_SIZE, hparams.n_merge, false);
                         get_u32(KEY_WIN_ATTN_PATTERN, hparams.n_wa_pattern, model.proj_type == PROJECTOR_TYPE_QWEN25VL); // only 2.5 requires it
                         // ref: https://huggingface.co/Qwen/Qwen2.5-VL-7B-Instruct/blob/main/preprocessor_config.json
-                        // the actual max limit is 12845056/14/14/2/2/4 = 4096 tokens
-                        // but we set a lower value to avoid OOM
-                        // TODO: make it configurable by user
-                        // TODO (2): bbox coordinates become inaccurate with small number of tokens,
-                        //           therefore we need to increase the min_tokens
-                        //           see: https://github.com/ggml-org/llama.cpp/issues/16842#issuecomment-3475144858
-                        hparams.set_limit_image_tokens(8, 2048);
-                        hparams.set_warmup_n_tokens(256); // avoid OOM on warmup
+                        hparams.set_limit_image_tokens(8, 4096);
+                        hparams.set_warmup_n_tokens(46*46); // avoid OOM on warmup
                         const int warn_min_pixels = 1024 * hparams.n_merge * hparams.n_merge * hparams.patch_size * hparams.patch_size;
                         if (hparams.image_min_pixels < warn_min_pixels) {
                             LOG_WRN("%s: Qwen-VL models require at minimum 1024 image tokens to function correctly on grounding tasks\n", __func__);
@@ -4814,7 +4808,7 @@ bool clip_image_batch_encode(clip_ctx * ctx, const int n_threads, const clip_ima
         case PROJECTOR_TYPE_QWEN2VL:
         case PROJECTOR_TYPE_QWEN3VL:
             {
-                const int merge_ratio = 2;
+                const int merge_ratio = hparams.n_merge;
                 const int pw = image_size_width  / patch_size;
                 const int ph = image_size_height / patch_size;
                 std::vector<int> positions(n_pos * 4);
Repository: huggingface/transformers
Pull request: Redundant code removal

# What does this PR do?

vocab_file and _extra_ids are already assigned on line 146-147, so their reassignment is redundant.


## Before submitting
- [x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [ ] 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?

Anyone in the community is free to review the PR once the tests have passed. Tagging @ArthurZucker as this relates to text encoders.

<!-- Your PR will be replied to more quickly if you can figure out the right person to tag with @

 If you know how to use git blame, that is the easiest way, otherwise, here is a rough guide of **who to tag**.
 Please tag fewer than 3 people.

Models:

- text models: @ArthurZucker
- vision models: @amyeroberts, @qubvel
- speech models: @eustlb
- graph models: @clefourrier

Library:

- flax: @gante and @Rocketknight1
- generate: @zucchini-nlp (visual-language models) or @gante (all others)
- pipelines: @Rocketknight1
- tensorflow: @gante and @Rocketknight1
- tokenizers: @ArthurZucker
- trainer: @zach-huggingface, @SunMarc and @qgallouedec
- chat templates: @Rocketknight1

Integrations:

- deepspeed: HF Trainer/Accelerate: @SunMarc @zach-huggingface
- ray/raytune: @richardliaw, @amogkam
- Big Model Inference: @SunMarc
- quantization (bitsandbytes, autogpt): @SunMarc @MekkCyber

Documentation: @stevhliu

HF projects:

- accelerate: [different repo](https://github.com/huggingface/accelerate)
- datasets: [different repo](https://github.com/huggingface/datasets)
- diffusers: [different repo](https://github.com/huggingface/diffusers)
- rust tokenizers: [different repo](https://github.com/huggingface/tokenizers)

Maintained examples (not research project or legacy):

- Flax: @Rocketknight1
- PyTorch: See Models above and tag the person corresponding to the modality of the example.
- TensorFlow: @Rocketknight1

 -->


Changed files:
filename: src/transformers/models/t5/tokenization_t5.py
patch: @@ -182,8 +182,6 @@ def __init__(
 
         self.legacy = legacy
         self.sp_model = self.get_spm_processor(kwargs.pop("from_slow", False))
-        self.vocab_file = vocab_file
-        self._extra_ids = extra_ids
         self.add_prefix_space = add_prefix_space
 
         super().__init__(
Repository: huggingface/transformers
Pull request: Oupsy 

# What does this PR do?
Need to pin greater or equal and smaller

Changed files:
filename: setup.py
patch: @@ -189,7 +189,7 @@
     "timeout-decorator",
     "tiktoken",
     "timm<=1.0.19,!=1.0.18",
-    "tokenizers==0.22.0",
+    "tokenizers>=0.22.0,<=0.23.0",
     "torch>=2.2",
     "torchaudio",
     "torchvision",

filename: src/transformers/dependency_versions_table.py
patch: @@ -91,7 +91,7 @@
     "timeout-decorator": "timeout-decorator",
     "tiktoken": "tiktoken",
     "timm": "timm<=1.0.19,!=1.0.18",
-    "tokenizers": "tokenizers==0.22.0",
+    "tokenizers": "tokenizers>=0.22.0,<=0.23.0",
     "torch": "torch>=2.2",
     "torchaudio": "torchaudio",
     "torchvision": "torchvision",
Repository: facebook/react
Pull request: Test top level fragment inside lazy semantics

This wasn't clearly articulated and tested why the code structure is like this but I think the logic is correct - or at least consistent with the weird semantics.

We place this top-level fragment check inside the recursion so that you can resolve how many every Lazy or Usable wrappers you want and it still preserves the same semantics if they weren't there (which they might not be as a matter of a race condition).

However, we don't actually recurse with the top-level fragment unwrapping itself because nesting a bunch of keyless fragments isn't the same as a single fragment/element.

Changed files:
filename: packages/react-reconciler/src/ReactChildFiber.js
patch: @@ -1460,15 +1460,18 @@ function createChildReconciler(
     lanes: Lanes,
     debugInfo: ReactDebugInfo | null,
   ): Fiber | null {
-    // This function is not recursive.
+    // This function is only recursive for Usables/Lazy and not nested arrays.
+    // That's so that using a Lazy wrapper is unobservable to the Fragment
+    // convention.
     // If the top level item is an array, we treat it as a set of children,
     // not as a fragment. Nested arrays on the other hand will be treated as
     // fragment nodes. Recursion happens at the normal flow.
 
     // Handle top level unkeyed fragments as if they were arrays.
     // This leads to an ambiguity between <>{[...]}</> and <>...</>.
     // We treat the ambiguous cases above the same.
-    // TODO: Let's use recursion like we do for Usable nodes?
+    // We don't use recursion here because a fragment inside a fragment
+    // is no longer considered "top level" for these purposes.
     const isUnkeyedTopLevelFragment =
       typeof newChild === 'object' &&
       newChild !== null &&

filename: packages/react-reconciler/src/__tests__/ReactFragment-test.js
patch: @@ -965,4 +965,51 @@ describe('ReactFragment', () => {
       </>,
     );
   });
+
+  it('should preserve state of children when adding a fragment wrapped in Lazy', async function () {
+    const ops = [];
+
+    class Stateful extends React.Component {
+      componentDidUpdate() {
+        ops.push('Update Stateful');
+      }
+
+      render() {
+        return <div>Hello</div>;
+      }
+    }
+
+    const lazyChild = React.lazy(async () => ({
+      default: (
+        <>
+          <Stateful key="a" />
+          <div key="b">World</div>
+        </>
+      ),
+    }));
+
+    function Foo({condition}) {
+      return condition ? <Stateful key="a" /> : lazyChild;
+    }
+
+    ReactNoop.render(<Foo condition={true} />);
+    await waitForAll([]);
+
+    ReactNoop.render(<Foo condition={false} />);
+    await waitForAll([]);
+
+    expect(ops).toEqual(['Update Stateful']);
+    expect(ReactNoop).toMatchRenderedOutput(
+      <>
+        <div>Hello</div>
+        <div>World</div>
+      </>,
+    );
+
+    ReactNoop.render(<Foo condition={true} />);
+    await waitForAll([]);
+
+    expect(ops).toEqual(['Update Stateful', 'Update Stateful']);
+    expect(ReactNoop).toMatchRenderedOutput(<div>Hello</div>);
+  });
 });
Repository: facebook/react
Pull request: Promote ASYNC_ITERATOR symbol to React Symbols

So that when we end up referring to it in more places, it's only one.

We don't do this same pattern for regular `Symbol.iterator` because we also support the string `"@@iterator"` for backwards compatibility.

Changed files:
filename: packages/react-client/src/ReactFlightClient.js
patch: @@ -65,6 +65,7 @@ import {
   REACT_LAZY_TYPE,
   REACT_ELEMENT_TYPE,
   REACT_POSTPONE_TYPE,
+  ASYNC_ITERATOR,
 } from 'shared/ReactSymbols';
 
 export type {CallServerCallback, EncodeFormActionCallback};
@@ -1245,8 +1246,6 @@ function startReadableStream<T>(
   resolveStream(response, id, stream, flightController);
 }
 
-const ASYNC_ITERATOR = Symbol.asyncIterator;
-
 function asyncIterator(this: $AsyncIterator<any, any, void>) {
   // Self referencing iterator.
   return this;

filename: packages/react-server/src/ReactFlightServer.js
patch: @@ -99,6 +99,7 @@ import {
   REACT_LAZY_TYPE,
   REACT_MEMO_TYPE,
   REACT_POSTPONE_TYPE,
+  ASYNC_ITERATOR,
 } from 'shared/ReactSymbols';
 
 import {
@@ -201,8 +202,6 @@ if (
 
 const ObjectPrototype = Object.prototype;
 
-const ASYNC_ITERATOR = Symbol.asyncIterator;
-
 type JSONValue =
   | string
   | boolean

filename: packages/shared/ReactSymbols.js
patch: @@ -60,3 +60,5 @@ export function getIteratorFn(maybeIterable: ?any): ?() => ?Iterator<any> {
   }
   return null;
 }
+
+export const ASYNC_ITERATOR = Symbol.asyncIterator;
Repository: nodejs/node
Pull request: doc: add missing colon to headers in `globals.md`

I noticed this while working on the new documentation generator.

Changed files:
filename: doc/api/globals.md
patch: @@ -621,7 +621,7 @@ added: v20.0.0
 
 See {File}.
 
-## Class `FormData`
+## Class: `FormData`
 
 <!-- YAML
 added:
@@ -660,7 +660,7 @@ the global scope; `var something` inside a Node.js module will be local to that
 module, regardless of whether it is a [CommonJS module][] or an
 [ECMAScript module][].
 
-## Class `Headers`
+## Class: `Headers`
 
 <!-- YAML
 added:
Repository: denoland/deno
Pull request: fix: regex pattern syntax in config-file.v1.json

Into `config-file.v1.json`, `compilerOptions.moduleResolution` has in invalid regexp pattern.

<img width="546" height="370" alt="image" src="https://github.com/user-attachments/assets/193e74d7-0188-4b5d-a1be-2e1fde13ef25" />

I've added the missing opening parenthesis.

Changed files:
filename: cli/schemas/config-file.v1.json
patch: @@ -247,7 +247,7 @@
               ]
             },
             {
-              "pattern": "^([Nn]ode[Nn]ext)|([Bb]undler))$"
+              "pattern": "^([Nn]ode[Nn]ext|[Bb]undler)$"
             }
           ],
           "markdownDescription": "Specify how TypeScript looks up a file from a given module specifier.\n\nSee more: https://www.typescriptlang.org/tsconfig#moduleResolution"
Repository: ollama/ollama
Pull request: progressbar: make start and end seamless

This just makes the bars limiting the progressbar width hug the progressbar because it looks nicer.

Changed files:
filename: progressbar/progressbar.go
patch: @@ -291,7 +291,7 @@ func OptionShowDescriptionAtLineEnd() Option {
 	}
 }
 
-var defaultTheme = Theme{Saucer: "█", SaucerPadding: " ", BarStart: "|", BarEnd: "|"}
+var defaultTheme = Theme{Saucer: "█", SaucerPadding: " ", BarStart: "▕", BarEnd: "▏"}
 
 // NewOptions constructs a new instance of ProgressBar, with any options you specify
 func NewOptions(max int, options ...Option) *ProgressBar {
Repository: huggingface/transformers
Pull request: `tokenizers` bump tokenizers version

# What does this PR do?
Mostly breaking for the DecodeStream API but also adds `async` usage for `tokenizers`

Changed files:
filename: setup.py
patch: @@ -189,7 +189,7 @@
     "timeout-decorator",
     "tiktoken",
     "timm<=1.0.19,!=1.0.18",
-    "tokenizers>=0.21,<0.22",
+    "tokenizers==0.22.0",
     "torch>=2.2",
     "torchaudio",
     "torchvision",

filename: src/transformers/dependency_versions_table.py
patch: @@ -91,7 +91,7 @@
     "timeout-decorator": "timeout-decorator",
     "tiktoken": "tiktoken",
     "timm": "timm<=1.0.19,!=1.0.18",
-    "tokenizers": "tokenizers>=0.21,<0.22",
+    "tokenizers": "tokenizers==0.22.0",
     "torch": "torch>=2.2",
     "torchaudio": "torchaudio",
     "torchvision": "torchvision",
Repository: ollama/ollama
Pull request: Add Ollamac to community integrations

Hi @jmorganca, I appreciate the time and effort you've put into developing Ollama—it's a fantastic tool! I noticed there isn't a native macOS application listed under community integrations, so I went ahead and created one called [Ollamac](https://github.com/kevinhermawan/Ollamac). I'm looking forward to hearing your thoughts on it! 😃

![Ollamac](https://github.com/jmorganca/ollama/assets/84965338/dcae2e9e-3f8c-4d19-9221-7567cf64ce93)

Changed files:
filename: README.md
patch: @@ -223,6 +223,7 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [Typescript UI](https://github.com/ollama-interface/Ollama-Gui?tab=readme-ov-file)
 - [Minimalistic React UI for Ollama Models](https://github.com/richawo/minimal-llm-ui)
 - [Web UI](https://github.com/ollama-webui/ollama-webui)
+- [Ollamac](https://github.com/kevinhermawan/Ollamac)
 
 ### Terminal
Repository: denoland/deno
Pull request: feat(unstable): add process id to permission broker messages

Closes https://github.com/denoland/deno/issues/31005

Changed files:
filename: cli/schemas/permission-broker-request.v1.json
patch: @@ -12,6 +12,10 @@
         "1"
       ]
     },
+    "pid": {
+      "type": "number",
+      "description": "The process ID of the Deno process making the request."
+    },
     "id": {
       "type": "number",
       "description": "The numeric ID of the request. The response must have a matching ID."

filename: runtime/permissions/broker.rs
patch: @@ -17,9 +17,11 @@ use crate::ipc_pipe::IpcPipe;
 static BROKER_EXIT_CODE: i32 = 87;
 
 static PERMISSION_BROKER: OnceLock<PermissionBroker> = OnceLock::new();
+static PID: OnceLock<u32> = OnceLock::new();
 
 pub fn set_broker(broker: PermissionBroker) {
   assert!(PERMISSION_BROKER.set(broker).is_ok());
+  assert!(PID.set(std::process::id()).is_ok());
 }
 
 pub fn has_broker() -> bool {
@@ -30,6 +32,7 @@ pub fn has_broker() -> bool {
 #[serde(rename_all = "camelCase")]
 struct PermissionBrokerRequest<'a> {
   v: u32,
+  pid: u32,
   id: u32,
   datetime: String,
   permission: &'a str,
@@ -76,6 +79,7 @@ impl PermissionBroker {
       .fetch_add(1, std::sync::atomic::Ordering::Relaxed);
     let request = PermissionBrokerRequest {
       v: 1,
+      pid: *PID.get().unwrap(),
       id,
       datetime: chrono::Utc::now().to_rfc3339(),
       permission,

filename: tests/integration/run_tests.rs
patch: @@ -3582,11 +3582,11 @@ fn test_permission_broker() {
   test_util::assertions::assert_wildcard_match(
     &line,
     r#"[WILDCARD]
-{"v":1,"id":1,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/scratch.txt"}
-{"v":1,"id":2,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/scratch.txt"}
-{"v":1,"id":3,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/log.txt"}
-{"v":1,"id":4,"datetime":"[WILDCARD]","permission":"write","value":"./run/permission_broker/log.txt"}
-{"v":1,"id":5,"datetime":"[WILDCARD]","permission":"env","value":null}
+{"v":1,"pid":[WILDCARD],"id":1,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/scratch.txt"}
+{"v":1,"pid":[WILDCARD],"id":2,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/scratch.txt"}
+{"v":1,"pid":[WILDCARD],"id":3,"datetime":"[WILDCARD]","permission":"read","value":"./run/permission_broker/log.txt"}
+{"v":1,"pid":[WILDCARD],"id":4,"datetime":"[WILDCARD]","permission":"write","value":"./run/permission_broker/log.txt"}
+{"v":1,"pid":[WILDCARD],"id":5,"datetime":"[WILDCARD]","permission":"env","value":null}
 [WILDCARD]"#,
   );
 }
Repository: nodejs/node
Pull request: test_runner: correct "already mocked" error punctuation placement

The current message contains a period within the quotes around the specifier, making it look malformed.

Changed files:
filename: lib/internal/test_runner/mock/mock.js
patch: @@ -671,7 +671,7 @@ class MockTracker {
 
     if (baseURL.searchParams.has(kMockSearchParam)) {
       throw new ERR_INVALID_STATE(
-        `Cannot mock '${specifier}.' The module is already mocked.`,
+        `Cannot mock '${specifier}'. The module is already mocked.`,
       );
     }
Repository: ollama/ollama
Pull request: Added Ollama4j (Java library) to community integrations

Hi @jmorganca, thank you so much for your efforts in building and constantly updating such a cool piece of software. 
I really like Ollama and have been experimenting with it regularly. I also plan to integrate it into many more Java applications. 
I noticed there isn't a Java library listed under community integrations, that's why I felt the need to create a Java library for interacting with Ollama's REST APIs, and as a result, I've published Ollama4j as a package and I plan to constantly improve it. This way, others who work with Java can easily interact with the Ollama API. 

Thanks again, and keep up the great work!

Changed files:
filename: README.md
patch: @@ -239,6 +239,7 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [LiteLLM](https://github.com/BerriAI/litellm)
 - [OllamaSharp for .NET](https://github.com/awaescher/OllamaSharp)
 - [Ollama-rs for Rust](https://github.com/pepperoni21/ollama-rs)
+- [Ollama4j for Java](https://github.com/amithkoujalgi/ollama4j)
 - [ModelFusion Typescript Library](https://modelfusion.dev/integration/model-provider/ollama)
 
 ### Extensions & Plugins
Repository: ollama/ollama
Pull request: Fix sudo variable in install.sh

It was forgotten to replace sudo at one place with the variable for sudo.

Changed files:
filename: scripts/install.sh
patch: @@ -180,7 +180,7 @@ install_cuda_driver_apt() {
     case $1 in
         debian)
             status 'Enabling contrib sources...'
-            $SUDO sed 's/main/contrib/' < /etc/apt/sources.list | sudo tee /etc/apt/sources.list.d/contrib.list > /dev/null
+            $SUDO sed 's/main/contrib/' < /etc/apt/sources.list | $SUDO tee /etc/apt/sources.list.d/contrib.list > /dev/null
             ;;
     esac
Repository: ollama/ollama
Pull request: Added logseq ollama plugin

Adds a plugin I made to integrate ollama with [logseq](https://github.com/logseq/logseq)

Changed files:
filename: README.md
patch: @@ -243,7 +243,8 @@ See the [API documentation](./docs/api.md) for all endpoints.
 - [Discollama](https://github.com/mxyng/discollama) (Discord bot inside the Ollama discord channel)
 - [Continue](https://github.com/continuedev/continue)
 - [Obsidian Ollama plugin](https://github.com/hinterdupfinger/obsidian-ollama)
+- [Logseq Ollama plugin](https://github.com/omagdy7/ollama-logseq)
 - [Dagger Chatbot](https://github.com/samalba/dagger-chatbot)
 - [Discord AI Bot](https://github.com/mekb-turtle/discord-ai-bot)
 - [Dumbar](https://github.com/JerrySievert/Dumbar)
-- [Hass Ollama Conversation](https://github.com/ej52/hass-ollama-conversation)
\ No newline at end of file
+- [Hass Ollama Conversation](https://github.com/ej52/hass-ollama-conversation)