Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

surface(): document the InputError for a team on a no-team surface; pitch_updates=None

Open Beginner friendly
  • Is something wrong?
#100 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
82/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
python
Domain
data-visualization, documentation

Research direction

Start in src/sdvplot/_surface.py, in surface(): the Raises block and the merge {**SOCCER_PITCH, **sportypy_kwargs.get("pitch_updates", {})}. Reproduce with surface("soccer", pitch_updates=None) to see the TypeError. Done when None falls back to the defaults like an absent key, and the Raises block lists the InputError for a team on a league in NO_TEAMS ("fiba").

Written by the indexing model from the issue text.

Description

From the soccer/FIBA program's final review. (1) surface()'s Raises block (src/sdvplot/_surface.py) omits the InputError raised for a team on a league in NO_TEAMS ("fiba"). (2) surface("soccer", pitch_updates=None) raises TypeError: 'NoneType' object is not a mapping at the {**SOCCER_PITCH, **sportypy_kwargs.get("pitch_updates", {})} merge; R's pitch_updates = NULL gives the 105 x 68 defaults. Treat None like absent.

Dominant language
Python
Stars
1
Forks
0
Avg merge
3h 26m
Merged PRs (30d)
129

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from sportsdataverse/sdvplot

All issues in sportsdataverse/sdvplot

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.