deepseek-ai/DeepSeek-VL2

Fork 0

mirror of https://github.com/deepseek-ai/DeepSeek-VL2.git synced 2026-11-05 01:07:23 +00:00

[PR #149] fix: remove print pollution, add version, and relax version constraints #149

New issue

Open

opened 2026-10-26 21:12:43 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 21:12:43 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/deepseek-ai/DeepSeek-VL2/pull/149
Author: @yurekami
Created: 12/25/2025
Status: 🔄 Open

Base: main ← Head: fix-print-pollution-and-versions

📝 Commits (1)

32db1b2 fix: remove print pollution, add version, and relax version constraints

📊 Changes

3 files changed (+8 additions, -7 deletions)

View changed files

📝 deepseek_vl2/__init__.py (+4 -3)
📝 pyproject.toml (+2 -2)
📝 requirements.txt (+2 -2)

📄 Description

Summary

Remove unconditional print statement on import in __init__.py
Add __version__ = "1.0.0" to package __init__.py for better package introspection
Improve comment to clarify why collections patching is needed
Relax torch and transformers version constraints from == to >= in both requirements.txt and pyproject.toml

Problem

The print statement print("Python version is above 3.10, patching the collections module.") was polluting stdout on every import, which can interfere with logging and scripting.
Pinned exact versions (torch==2.0.1, transformers==4.38.2) cause unnecessary installation conflicts when users have newer compatible versions already installed.

Test plan

Verify Python syntax with python3 -m py_compile
Verify package imports without print pollution
Verify installation works with newer torch/transformers versions

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/deepseek-ai/DeepSeek-VL2/pull/149 **Author:** [@yurekami](https://github.com/yurekami) **Created:** 12/25/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix-print-pollution-and-versions` --- ### 📝 Commits (1) - [`32db1b2`](https://github.com/deepseek-ai/DeepSeek-VL2/commit/32db1b2030f1cebc24c7e352381b61c1e1dcd5c3) fix: remove print pollution, add __version__, and relax version constraints ### 📊 Changes **3 files changed** (+8 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `deepseek_vl2/__init__.py` (+4 -3) 📝 `pyproject.toml` (+2 -2) 📝 `requirements.txt` (+2 -2) </details> ### 📄 Description ## Summary - Remove unconditional print statement on import in `__init__.py` - Add `__version__ = "1.0.0"` to package `__init__.py` for better package introspection - Improve comment to clarify why collections patching is needed - Relax `torch` and `transformers` version constraints from `==` to `>=` in both `requirements.txt` and `pyproject.toml` ## Problem 1. The print statement `print("Python version is above 3.10, patching the collections module.")` was polluting stdout on every import, which can interfere with logging and scripting. 2. Pinned exact versions (`torch==2.0.1`, `transformers==4.38.2`) cause unnecessary installation conflicts when users have newer compatible versions already installed. ## Test plan - [x] Verify Python syntax with `python3 -m py_compile` - [ ] Verify package imports without print pollution - [ ] Verify installation works with newer torch/transformers versions 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi added the

pull-request

label

2026-10-26 21:12:43 +00:00

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

deepseek-ai/DeepSeek-VL2#149

No description provided.

Rows
Columns

[PR #149] fix: remove print pollution, add __version__, and relax version constraints #149