ChimeraTK / ChimeraTK/DeviceAccess

Move TypeChangingDecorator from ControlSystemAdapter to DeviceAccess

Open
#44 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C++
Stars
12
Forks
12
Avg merge
2d 14h
Merged PRs (30d)
13

Description

Move the TypeChangingDecorator from the ControlSystemAdapter project to DeviceAccess. Only the decorator itself should be moved, the factory functions (which are defined in the same files) are specific to the use case in the ControlSystemAdapter and hence should stay there.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search the ControlSystemAdapter and DeviceAccess projects for TypeChangingDecorator and the factory functions defined alongside it. Read how the decorator is used, then move only the decorator while keeping the factory functions in ControlSystemAdapter. Done means DeviceAccess provides the decorator and the ControlSystemAdapter-specific factories remain unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.