eive/eive-obsw
eive/eive-obsw
13
4
Fork 0
Code Issues 4 Pull Requests 1 Releases 120 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
there are actually 8 heaters
Some checks are pending
EIVE/eive-obsw/pipeline/head Build started...
Details
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
7b611810df
some docs
Some checks are pending
EIVE/eive-obsw/pipeline/head Build started...
Details
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
c558a117b0
muellerr added 1 commit 2023-07-07 11:07:39 +02:00
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
0b673fcdec
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
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
7388c4f661
muellerr added 1 commit 2023-07-07 11:48:02 +02:00
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
977e9f366f
muellerr added 1 commit 2023-07-07 11:57:04 +02:00
compile fixes
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
ff7d0b3989
muellerr added 1 commit 2023-07-07 12:05:32 +02:00
add event
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
7337595d47
muellerr added 1 commit 2023-07-07 12:06:44 +02:00
bump tmtc
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
b662e220e1
muellerr added 1 commit 2023-07-07 12:22:46 +02:00
pls compile
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
fa85cc4d78
muellerr added 1 commit 2023-07-07 16:18:19 +02:00
done
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
df3755d6cc
muellerr added 2 commits 2023-07-10 13:28:46 +02:00
this should be better
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
33de08eafc
muellerr added 1 commit 2023-07-10 13:56:37 +02:00
that should be it
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
ce2755e161
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
disable test case
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
53d53e8771
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
Merge remote-tracking branch 'origin/main' into tcs-heater-upper-limit
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
Details
8800b20409
muellerr added 1 commit 2023-07-11 10:45:17 +02:00
thermal controller
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
ed7d75a777
muellerr added 1 commit 2023-07-11 10:46:06 +02:00
Merge branch 'main' into tcs-heater-upper-limit
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
Details
914a22f642
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?