Remove the unused economy comparison module
メンテナーはふだん 2 日以内に返信
評価
調査の方向性
Start with policyengine_api/services/economy_comparison.py and grep the repo for imports of it, then review its unit and snapshot tests and the snapshot data. Check the api.routes.no-domain-logic-in-http rule so only the economy comparison node is dropped. Done means the module, its tests and snapshots are gone and the test suite still passes. Pull request #3936 is already open against this issue, so review it rather than starting a parallel change.
索引モデルが issue の本文から書いたものです。
説明
Problem
policyengine_api/services/economy_comparison.py (915 lines) has had no caller in the package since #2569, which removed the Public API's own society-wide simulations. Only its own unit and snapshot tests use it: about 1,600 lines of tests, plus an 8,000-line snapshot file.
It still carries code that other work has to account for:
- The enhanced FRS weight matrix.
uk_constituency_breakdownanduk_local_authority_breakdowndownload it from Hugging Face. PolicyEngine/policyengine.py#553 lists that as a Public API dependency to resolve before the UK default moves to a Microcosm dataset. - The only microdf use in the package. The weighted-microdata rules (step 1.9) would otherwise guard and baseline code that never runs.
- Rules for dead code. A route-boundary rule (
api.routes.no-domain-logic-in-http) checks that the module stays out of the HTTP layer.
Proposal
Remove the module, its unit and snapshot tests, and the snapshot data.
api.routes.no-domain-logic-in-httpkeeps its calculation-route check and drops the economy comparison node.microdf_pythonstays declared. Removing a dependency nothing imports is a separate cleanup, which the dependency check planned for step 2.7 would flag.
- 主要言語
- Python
- スター
- 18
- フォーク
- 33
- 平均マージ
- 1日 18時間
- マージ済み PR（30日）
- 23
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
PolicyEngine/policyengine-api のほかの issue
-
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
PolicyEngine/policyengine-api#3911 ·
メンテナーはふだん 2 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
PolicyEngine/policyengine-api#3910 ·
メンテナーはふだん 2 日以内に返信
-
Register weighted-microdata rules: weights aren't values対応中かも @anth-volk が今日担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 18/100
PolicyEngine/policyengine-api#3937 ·
メンテナーはふだん 2 日以内に返信
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)対応中かも @anth-volk が今日担当しました。 オープン
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
PolicyEngine/policyengine-api#3931 ·
メンテナーはふだん 2 日以内に返信
-
Refactor the application package within ruff's complexity limits対応中かも @anth-volk が今日担当しました。 オープン
難易度 4/5 1週間以上 初心者へのやさしさ 18/100
PolicyEngine/policyengine-api#3930 ·
メンテナーはふだん 2 日以内に返信
PolicyEngine/policyengine-api の issue をすべて見る
似ている issue
-
approved Good First Issue scope: docs
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
hiero-ledger/hiero-sdk-python#2730 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
wemake-services/wemake-django-template#3178 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
md/asciidoc/csv: an explicit `encoding="utf-8"` keeps a leading BOM, the auto-detect path drops itオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
docling-project/docling#4846 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
neurostuff/NiMARE#1181 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
pytest-dev/pytest#15180 ·
メンテナーはふだん 1 日以内に返信