dimensionalOS / dimensionalOS/dimos
Codebase Dept cleanup
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4.5k
- Forks
- 808
- Avg merge
- 3d 5h
- Merged PRs (30d)
- 233
Description
this is a work in progress document
generic binary encodings need to be unified
we have two parallel systems, should have one
protocol/pubsub/encoders.py
dimos/memory2/codecs/base.py
streams need to be unified
we have a concept of a stream within module, stream within memory2, and rxpy stream.
this needs to be the same class
probably all streams should be memory2 in-memory streams by default
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 comparing protocol/pubsub/encoders.py with dimos/memory2/codecs/base.py to document the two binary-encoding systems. Then trace the module, memory2, and RxPy stream concepts and identify their current boundaries. Done means the encoding approach and stream abstraction are unified across the named areas, with the intended default behavior confirmed.
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
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100