Problem Description
In src/modelinfo/parsers/huggingface.py, the network request timeout is hardcoded to 10 seconds: urllib.request.urlopen(req, timeout=10). For users with slow connections fetching massive sharded index manifests, this can cause unexpected timeouts.
Proposed Solution
- Add a
--timeout argument to argparse in src/modelinfo/cli.py (default: 10.0).
- Pass this argument down through
analyze_model to fetch_huggingface_repo.
- Replace the hardcoded
10 with the user-provided timeout value.
Alternatives Considered
No response
Additional Context
This is a great first issue for those wanting to touch the CLI arguments.
Problem Description
In
src/modelinfo/parsers/huggingface.py, the network request timeout is hardcoded to 10 seconds:urllib.request.urlopen(req, timeout=10). For users with slow connections fetching massive sharded index manifests, this can cause unexpected timeouts.Proposed Solution
--timeoutargument toargparseinsrc/modelinfo/cli.py(default: 10.0).analyze_modeltofetch_huggingface_repo.10with the user-provided timeout value.Alternatives Considered
No response
Additional Context
This is a great first issue for those wanting to touch the CLI arguments.