[Code scan] Package metadata allows incompatible Pydantic 1 installations

オープン 初心者向け
  • 何か問題がありますか？
#443 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
86/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
静か
技術スタック
python
領域
build-system

調査の方向性

pyproject.toml のパッケージ依存関係の宣言から始め、次に lambench/models/basemodel.py と lambench/metrics/post_process.py を読んで、issue で指定されている Pydantic 2 API を確認します。依存関係のメタデータを更新して、意図したメジャーバージョン互換性を表現し、その結果得られる制約でパッケージを解決およびインポートできることを確認します。

索引モデルが issue の本文から書いたものです。

説明

This issue was found by a Codex global repository scan of tracked non-test files at commit 8c93925cb10b401b2b83c738bd9263fd74474468.

Relevant code

https://github.com/deepmodeling/LAMBench/blob/8c93925cb10b401b2b83c738bd9263fd74474468/pyproject.toml#L8-L19
https://github.com/deepmodeling/LAMBench/blob/8c93925cb10b401b2b83c738bd9263fd74474468/lambench/models/basemodel.py#L1-L22
https://github.com/deepmodeling/LAMBench/blob/8c93925cb10b401b2b83c738bd9263fd74474468/lambench/metrics/post_process.py#L180-L190

Impact

The dependency is declared as bare pydantic, but the code uses Pydantic 2 APIs such as ConfigDict and model_dump(). A resolver can legally install Pydantic 1 when another package constrains it, which would make imports or runtime calls fail.

Suggested fix

Declare the intended major version explicitly, for example pydantic>=2,<3, or remove Pydantic 2-only APIs if Pydantic 1 compatibility is intended.

主要言語
Python
スター
21
フォーク
11
平均マージ
23日 6時間
マージ済み PR（30日）
1

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

deepmodeling/LAMBench のほかの issue

deepmodeling/LAMBench の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。