eive/eive-obsw
eive
/
eive-obsw
12
3
Fork 0
Code Issues 3 Pull Requests 1 Releases 117 Wiki Activity

TCS heater upper burn limit #732

Merged
muellerr merged 18 commits from tcs-heater-upper-limit into main 2023-07-11 10:56:44 +02:00
Conversation 0 Commits 18 Files Changed 15 +163 -29
muellerr commented 2023-07-07 11:04:48 +02:00
Owner
Copy Link

TMTC PR: eive/eive-tmtc#213

TMTC PR: https://egit.irs.uni-stuttgart.de/eive/eive-tmtc/pulls/213
muellerr added 4 commits 2023-07-07 11:04:49 +02:00
EIVE/eive-obsw/pipeline/head Build started... Details
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
7b611810df
there are actually 8 heaters
EIVE/eive-obsw/pipeline/head Build started... Details
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
c558a117b0
some docs
muellerr added 1 commit 2023-07-07 11:07:39 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
0b673fcdec
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
muellerr added this to the v6.1.0 milestone 2023-07-07 11:07:57 +02:00
muellerr added 1 commit 2023-07-07 11:47:08 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
7388c4f661
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
muellerr added 1 commit 2023-07-07 11:48:02 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
977e9f366f
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
muellerr added 1 commit 2023-07-07 11:57:04 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
ff7d0b3989
compile fixes
muellerr added 1 commit 2023-07-07 12:05:32 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
7337595d47
add event
muellerr added 1 commit 2023-07-07 12:06:44 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
b662e220e1
bump tmtc
muellerr added 1 commit 2023-07-07 12:22:46 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
fa85cc4d78
pls compile
muellerr added 1 commit 2023-07-07 16:18:19 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
df3755d6cc
done
muellerr added 2 commits 2023-07-10 13:28:46 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
33de08eafc
this should be better
muellerr added 1 commit 2023-07-10 13:56:37 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
ce2755e161
that should be it
muellerr changed title from WIP: TCS heater upper burn limit to TCS heater upper burn limit 2023-07-10 13:56:50 +02:00
muellerr requested review from meggert 2023-07-10 13:57:46 +02:00
muellerr added 1 commit 2023-07-10 16:33:34 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
53d53e8771
disable test case
muellerr added a new dependency 2023-07-10 18:51:15 +02:00
#733 TCS Observability
meggert approved these changes 2023-07-11 09:17:20 +02:00
mission/controller/ThermalController.h Outdated
@ -103,2 +109,4 @@
static constexpr int16_t SANITY_LIMIT_UPPER_TEMP = 160;
// 1 hour
static constexpr uint32_t DEFAULT_MAX_HEATER_ON_DURATION_MS = 60 * 60 * 1000;
meggert commented 2023-07-11 09:16:44 +02:00
Owner
Copy Link

maybe we should change this to being a parameter someday

maybe we should change this to being a parameter someday
muellerr commented 2023-07-11 09:36:58 +02:00
Author
Owner
Copy Link

If all the changes fix the current heater issue, this mechanism will hopefully never have be used at all.. at least that is what I am hoping.

If all the changes fix the current heater issue, this mechanism will hopefully never have be used at all.. at least that is what I am hoping.
muellerr added 1 commit 2023-07-11 09:45:03 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
8800b20409
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
muellerr added 1 commit 2023-07-11 10:45:17 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
ed7d75a777
thermal controller
muellerr added 1 commit 2023-07-11 10:46:06 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
914a22f642 Merge branch 'main' into tcs-heater-upper-limit
muellerr merged commit 55affe5a42 into main 2023-07-11 10:56:44 +02:00
muellerr deleted branch tcs-heater-upper-limit 2023-07-11 10:56:44 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
meggert
Labels
Clear labels   
OPS TODO

Issues identified by OPS

  
api change

API change

  
breaking api change

API change which can break existing code

  
bug

Something is not working

  
documentation

Documentation

  
duplicate

This issue or pull request already exists

  
feature

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
OPS TODO
api change
breaking api change
bug
documentation
duplicate
feature
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
v6.1.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
#733 TCS Observability
eive/eive-obsw
Reference: eive/eive-obsw#732
No description provided.
Delete Branch "tcs-heater-upper-limit"

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?