INCATools / INCATools/ontology-access-kit
split cli.py into sub-packages
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 198
- Forks
- 35
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 1
Description
it's getting a little unwieldy
there is also some work to move some of the logic in there for processing multi-term arguments, but that is a separate piece of work
We could have one package per command:
- cli
- viz
- tree
- info
- ...
or we could modularize by interface
- interfaces
- basic_ontology_interface <- info, search, etc go here
- obograph_interface <- viz, tree, ancestors, etc go here
- etc
or parallel:
- cli
- basic_cli
- obograph_cli
- ...
See:
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.
Research direction
Start by reading cli.py and cataloging its commands and shared logic. Compare the proposed command-based, interface-based, and parallel package layouts, noting that multi-term argument processing is separate work. Done means the CLI is split into agreed sub-packages without taking on that separate argument-processing change, with existing commands still accounted for.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- cli
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100