MetOffice / MetOffice/CDDS

Refactor StreamInfo (and those Classes that inherit from it) so that StreamStore can be eliminated.

Open
#665 0 comments 0 reactions 0 assignees View on GitHub
project plugins
Dominant language
Python
Stars
8
Forks
7
Avg merge
1d 15h
Merged PRs (30d)
23

Description

From a brief look at this code I think think that all `StreamStore` (and related classes) can be removed. I don't really see the utility in having this information provided as a singleton and feel that this introduces a lot of unnecessary code and complexity as a consequence. If we could remove this, it would leave only one type of "Store" I.e the `PluginStore` which I think would be a nice simplification.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating StreamInfo, the classes that inherit from it, StreamStore, and its related classes, then trace how the singleton information is used. The issue is complete when StreamStore and the unnecessary related code are eliminated while the remaining PluginStore design continues to work.

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.