fsfw/fsfw
fsfw
/
fsfw
10
3
Fork 6
Code Issues 31 Pull Requests 4 Releases 10 Wiki Activity

refactor power module #590

Merged
gaisser merged 20 commits from eive/fsfw:mueller/refactor-power-switch-if-etc into development 2022-05-12 18:37:46 +02:00
Conversation 0 Commits 20 Files Changed 17 +403 -51
muellerr commented 2022-03-28 12:54:25 +02:00
Owner
Copy Link
  • Remove const specifier from sendSwitchCommand and sendFuseOnCommand and also specify a ReturnValue_t return type. The const specifier can make the API cumbersome and awkward to use.

Concerning repercusions if thread-safety is/might be an issue:

In my opinion, this should be mentioned in the code documentations and implemented by the user, not be enforced with const, which can make the API difficult and/or awkward to use.

  • Add a DummyPowerSwitcher module, which can be useful for test setups when no PCDU is available.
  • Add new definitions file, add new typedef for switches
  • Extend PowerSwitcher to optionally check current state when calling turnOn or turnOff. No need to command the switches if they are already in the correct state
  • Apply auto-formatter
  • Add power switch mock object for unit tests
- Remove `const` specifier from `sendSwitchCommand` and `sendFuseOnCommand` and also specify a `ReturnValue_t` return type. The const specifier can make the API cumbersome and awkward to use. Concerning repercusions if thread-safety is/might be an issue: In my opinion, this should be mentioned in the code documentations and implemented by the user, not be enforced with `const`, which can make the API difficult and/or awkward to use. - Add a `DummyPowerSwitcher` module, which can be useful for test setups when no PCDU is available. - Add new definitions file, add new typedef for switches - Extend `PowerSwitcher` to optionally check current state when calling `turnOn` or `turnOff`. No need to command the switches if they are already in the correct state - Apply auto-formatter - Add power switch mock object for unit tests
muellerr added 1 commit 2022-03-28 12:54:29 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
6f0362b956
refactor power module
muellerr added 1 commit 2022-03-28 12:57:21 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b32d1da421
this should compile
muellerr added 1 commit 2022-03-28 12:59:13 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
23af170229
small tweaks
muellerr added the
Breaking API Change
 label 2022-03-28 13:01:48 +02:00
muellerr changed title from refactor power module to WIP: refactor power module 2022-03-28 13:29:39 +02:00
muellerr added 1 commit 2022-03-30 17:41:55 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
b42987059a
make dummy power switcher a system object
muellerr added 1 commit 2022-04-01 16:40:40 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c20be13733
change switch type in header as well
muellerr added 1 commit 2022-04-01 17:28:19 +02:00
muellerr added a new dependency 2022-04-01 17:32:53 +02:00
#596 Power Switcher Component
muellerr added 2 commits 2022-04-01 18:38:42 +02:00
muellerr added 1 commit 2022-04-01 18:39:03 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2d0e4ba951
applied afmt
muellerr added 1 commit 2022-04-01 18:43:55 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b764194ed0
added more unit tests
muellerr added 1 commit 2022-04-11 16:11:30 +02:00
muellerr added this to the v5.0.0 milestone 2022-04-11 17:34:58 +02:00
muellerr added 1 commit 2022-04-25 15:37:10 +02:00
muellerr added 1 commit 2022-04-25 15:42:55 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
572d602b72
improve changelog, add entry
muellerr added 1 commit 2022-04-25 15:44:54 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b94685e045
added missing PR cross-ref
muellerr added 1 commit 2022-04-27 08:45:09 +02:00
muellerr added 1 commit 2022-05-09 11:02:52 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
c78b7c432b Merge branch 'development' into mueller/refactor-power-switch-if-etc
muellerr changed title from WIP: refactor power module to refactor power module 2022-05-09 11:02:54 +02:00
muellerr added 1 commit 2022-05-09 15:37:38 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
15352b539d Merge remote-tracking branch 'upstream/development' into mueller/refactor-power-switch-if-etc
muellerr added 1 commit 2022-05-10 09:58:51 +02:00
gaisser reviewed 2022-05-11 14:00:50 +02:00
src/fsfw/power/DummyPowerSwitcher.h Outdated
@ -0,0 +8,4 @@
#include "definitions.h"
#include "fsfw/objectmanager/SystemObject.h"
class DummyPowerSwitcher : public SystemObject, public PowerSwitchIF {
gaisser commented 2022-05-11 14:00:50 +02:00
Owner
Copy Link

I think we should mention the intenion of this object.

I think we should mention the intenion of this object.
muellerr commented 2022-05-11 14:02:23 +02:00
Author
Owner
Copy Link

good ideal will add

good ideal will add
gaisser marked this conversation as resolved
muellerr added 1 commit 2022-05-12 15:02:55 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
d11f898f70
update dummy power switcher docs
muellerr added 1 commit 2022-05-12 17:09:30 +02:00
gaisser approved these changes 2022-05-12 18:37:25 +02:00
gaisser left a comment
Owner
Copy Link

LGTM

LGTM
gaisser merged commit a83b86ccd2 into development 2022-05-12 18:37:46 +02:00
gaisser deleted branch mueller/refactor-power-switch-if-etc 2022-05-12 18:37:49 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
gaisser
Labels
Clear labels   
API Change

Changes the API, so Users need to adapted

  
Breaking API Change

Changes the API in a non backwards compatible way

  
bug

Something is not working

  
build

Issues related to the buildsystem, including CI

  
cosmetics

Small change most users won't notice

  
Documentation

Improvement of code documentation

  
duplicate

This issue or pull request already exists

  
feature

New feature

  
help wanted

Need some help

  
hotfix

Quick or important fix that is merged directly into master, causing a new revision

  
invalid

#worksforme

  
question

More information is needed

  
Refactor

Contains a refactoring of some code

  
Tests

Issues related to testing code

  
wontfix

This won't be fixed

No Label
API Change
Breaking API Change
bug
build
cosmetics
Documentation
duplicate
feature
help wanted
hotfix
invalid
question
Refactor
Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
v5.0.0
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Blocks
#596 Power Switcher Component
fsfw/fsfw
Reference: fsfw/fsfw#590
No description provided.
Delete Branch "eive/fsfw:mueller/refactor-power-switch-if-etc"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?