[1.0.0] Visibility
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Quiet
- Tech stack
- python
- Domain
- data, documentation, release
Research direction
Start with the README feature matrix and xrspatial/{viewshed,visibility}.py, then review the unresolved decisions in #2930. Run the listed /deep-sweep for viewshed.py and track the linked bug issues, notebook example, and optional script. Done means the checklist is cleared with no CRITICAL or HIGH findings.
Written by the indexing model from the issue text.
Description
1.0.0 readiness: Visibility
Area: Surface Functions: viewshed, cumulative_viewshed, visibility_frequency, line_of_sight
Source: xrspatial/{viewshed,visibility}.py
Scope
- Public surface frozen against the README feature matrix
- Dark-corner decisions resolved (see #2930)
Sweep clearance, no CRITICAL or HIGH (accuracy and security first)
- Accuracy
/deep-sweep xrspatial/viewshed.py - Security
- Performance
- API consistency
Last swept: none yet
Examples
- Notebook example
- (optional) standalone script
Linked bug issues
- #2786 (High) viewshed() mutates the caller's input DataArray (int16 becomes float64)
- #2789 (Medium) viewshed: validate regular grid spacing instead of silently assuming it
- #2792 (Medium) viewshed: ValueError 'node not found' on NaN input (CPU sweep)
- #2793 (High) Broken bounds-check guard in viewshed _calculate_event_row_col()
- #2794 (Medium) viewshed(): validate scalar params (max_distance >= 0, finite observer/target elev) before dispatch
- Dominant language
- Python
- Stars
- 972
- Forks
- 92
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 7
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 xarray-contrib/xarray-spatial
-
Difficulty 1/5 Under an hour Newbie friendliness 68/100
xarray-contrib/xarray-spatial#3726 ·
-
api area:surface bug severity:medium sweep-api-consistency
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
xarray-contrib/xarray-spatial#3712 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
xarray-contrib/xarray-spatial#3710 ·
-
bug
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
xarray-contrib/xarray-spatial#3707 ·
-
area:surface documentation user-guide-example
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
xarray-contrib/xarray-spatial#3464 ·
All issues in xarray-contrib/xarray-spatial
Similar issues
-
agent-ready documentation needs-triage
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
-
documentation
Difficulty 1/5 Under an hour Newbie friendliness 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Open
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
instance instance add
Difficulty 1/5 Under an hour Newbie friendliness 72/100
searxng/searx-instances#939 · 1 comment ·
-
area-deployment area-integrations triage:bot-seen
Difficulty 2/5 Half a day Newbie friendliness 86/100