full API can't currently apply pyright
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 25/100
Research direction
Start by reviewing policyengine_api/api and its pyproject.toml dependencies, then run pyright against the full API to identify the reported failures and performance constraints. The issue is done when the API can be type-checked without unusable directory exclusions and its required dependencies are correctly declared, or when the obsolete API code is removed following a decided scope.
Written by the indexing model from the issue text.
Description
Creating this as part of removing the "lib" packages for the various services in api-v2.
The code from "policyengine_api/api" is broken in a number of ways (for one we don't actually include the required dependencies in the pyproject.toml). pyright for some reason (https://github.com/microsoft/pyright/issues/10159) is unusably slow if you try to exclude directories so I've turned it off, but this means we have no type checking on the full API which is going to lead pretty quickly to code rott.
@anth-volk should we fix the offending api code, delete it, other?
- Dominant language
- Python
- Stars
- 1
- Forks
- 1
- Avg merge
- 15h 10m
- Merged PRs (30d)
- 12
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from PolicyEngine/policyengine-sim-api
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 3/5 1-2 days Newbie friendliness 65/100
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
PolicyEngine/policyengine-sim-api#674 · 3 comments ·
All issues in PolicyEngine/policyengine-sim-api
Similar issues
-
bug confirmed issue
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
open-webui/open-webui#30750 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
OpenwaterHealth/openmotion-bloodflow-app#604 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
good first issue
Difficulty 1/5 Under an hour Newbie friendliness 90/100