scverse / scverse/squidpy

replace manual variable checks with wrapper functions across codebase

Open
#1,137 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
598
Forks
121
Avg merge
3d 11h
Merged PRs (30d)
3

Description

Description of feature

Currently we do a lot of

if x not in y:
  raise ValueError(...)

across many files - let's replace this with semantically easier to parse functions across the entire codebase

Contributor guide

Open the contributing guide

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.

Research direction

Start by inventorying the repeated if x not in y checks across the codebase and reading the surrounding error-handling patterns. Identify the wrapper functions and replacement scope before changing callers; done means the manual checks are replaced consistently across the codebase while preserving the existing ValueError behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.