AllenInstitute / AllenInstitute/ConnectsCommonConnectivity

HierarchyCategory: per-taxonomy vocabulary; fix level type

Open
#9 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug schema
Dominant language
Jupyter Notebook
Stars
4
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Problem: category ids like class/subclass/cluster are global, but taxonomies don't share a level structure. Two taxonomies writing class collide — the last writer's level/description wins. Merge-scoped writes (#13) don't fix this. Also level generates as Optional[str] while Cluster.level is int.

Fix: add a hierarchy/taxonomy discriminator slot so category vocabularies are per-taxonomy; declare level: integer.

Context: Cluster holds hierarchical categorical features generally, not only cell types — we don't want to force a common level structure.

Contributor guide

No contributing guide indexed for this repository

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

Locate the category vocabulary and Cluster model/schema definitions referenced by the issue, then inspect how category IDs, taxonomy context, level, and description are stored. Confirm the change with the existing model or schema checks: vocabularies should remain separate per taxonomy, and Cluster.level should be represented as an integer rather than Optional[str].

Written by the indexing model from the issue text.

Assessment

Tech stack
jupyter-notebook
Domain
data
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.