load_store and dump_to_store
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4.2k
- Forks
- 1.4k
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 14
Description
Continuing from #3602, load_store and dump_to_store look like they are old and unmaintained functions:
load_storeis referenced once inapi.rst(I assume the reference tofrom_storewas toload_store), but never tested, used or mentioned anywhere elsedump_to_storeis tested (and probably used), but never mentioned except from the section on backends
what should we do with these? Are they obsolete and should be removed or just unmaintained (then we should properly document and test them).
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
Read #3602 and the api.rst reference, then inspect the backend section and the existing tests for dump_to_store; also check whether load_store has any coverage or uses. Done requires deciding whether both functions are obsolete or should remain, and then defining the corresponding removal or documentation and testing work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100