MetOffice / MetOffice/CDDS

Simplify CddsPlugin, BasePlugin, and Cmip6Plugin classes

Open
#649 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

See whether it is possible to simplify the three levels of inheritance by combining `CddsPlugin` and `BasePlugin` and have `Cmip6Plugin` inherit from this simplified Class.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the definitions of CddsPlugin, BasePlugin, and Cmip6Plugin and read how the three inheritance levels are used. Compare their responsibilities and callers before deciding whether the hierarchy can be reduced without changing behavior. Done means the simplified inheritance structure is justified and the affected plugin behavior remains intact.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-engineering
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.