Higher ACS Modes STR Only #818

Merged
meggert merged 38 commits from higher-acs-modes-only-str into dev-7.5.0 2023-12-06 17:15:52 +01:00
Owner
No description provided.
meggert added this to the v7.3.0 milestone 2023-11-02 17:00:00 +01:00
meggert added 17 commits 2023-11-02 17:00:01 +01:00
monotonic for pwr ctrl
Some checks are pending
EIVE/eive-obsw/pipeline/head Build started...
EIVE/eive-obsw/pipeline/pr-main This commit looks good
740167ce99
Merge branch 'main' into make-robin-happy-about-timestuff
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
e7276f282a
breaking the ACS controller only for Robin
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
f61da1002f
i hope i get a medal from Robin for this
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
25354ee7b4
this might just work
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
17c253d19b
Merge branch 'main' into make-robin-happy-about-timestuff
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
dc0c24ce96
fixed deprecated functions
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
823aa70954
noice
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
9482ceb206
Merge branch 'main' into make-robin-happy-about-timestuff
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
8d8eb6bd88
small improvement
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
77a555debc
div by 0 check
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
d5e57501be
Merge branch 'main' into make-robin-happy-about-timestuff
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
3c383f6d01
Merge branch 'main' into make-robin-happy-about-timestuff
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
b8672d3453
this gonna be fun
Some checks failed
EIVE/eive-obsw/pipeline/head Build queued...
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
60a348a08f
muellerr modified the milestone from v7.3.0 to v7.4.0 2023-11-06 14:41:44 +01:00
meggert added 1 commit 2023-11-14 13:22:42 +01:00
.
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
c155f399b1
meggert added 1 commit 2023-11-14 16:07:19 +01:00
calc rot rate from STR
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
74e7785c68
meggert added 1 commit 2023-11-14 16:07:35 +01:00
Merge branch 'main' into higher-acs-modes-only-str
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
c465558543
meggert added 1 commit 2023-11-14 16:23:16 +01:00
param to enable/disable STR for rot rate calc
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
5512605cd7
meggert added 1 commit 2023-11-14 16:58:00 +01:00
fdir needs to change
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
9482f3cae9
meggert added 1 commit 2023-11-15 16:54:59 +01:00
changed fdir to new variant
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
4a86d4ba4b
meggert added 1 commit 2023-11-22 13:36:08 +01:00
quest attitude estimation
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
a45e96b772
meggert added 1 commit 2023-11-22 13:56:37 +01:00
this is better
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
0aa09bd516
meggert added 1 commit 2023-11-23 11:50:33 +01:00
dataset output
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
886dd17e4a
meggert added 1 commit 2023-11-23 16:56:42 +01:00
this is a mess
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
4a67f9ffe5
meggert added 1 commit 2023-11-24 10:52:42 +01:00
this is much better
Some checks failed
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit
647d5fda7c
meggert added 1 commit 2023-11-24 11:30:33 +01:00
this should work
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
c7ec9726c4
meggert added 1 commit 2023-11-24 11:32:25 +01:00
small fix
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
7ef55dcab1
meggert added 1 commit 2023-11-24 11:45:16 +01:00
select the according quaternion and rotational rate
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
f0247a9ab3
meggert added a new dependency 2023-11-24 11:48:23 +01:00
meggert modified the milestone from v7.4.0 to v7.5.0 2023-11-24 14:30:31 +01:00
meggert added 2 commits 2023-11-27 10:41:25 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
70be396b62
meggert added 2 commits 2023-11-27 11:15:44 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
e3271b6b4d
meggert changed target branch from main to dev-7.5.0 2023-11-29 10:38:30 +01:00
meggert added 1 commit 2023-12-01 15:20:34 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
08d0619b11
meggert added 1 commit 2023-12-04 13:15:34 +01:00
Merge branch 'dev-7.5.0' into higher-acs-modes-only-str
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
d0588b144a
meggert added 1 commit 2023-12-04 13:19:34 +01:00
Merge branch 'dev-7.5.0' into higher-acs-modes-only-str
Some checks failed
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 There was a failure building this commit
b0b279e313
meggert added 1 commit 2023-12-04 17:13:46 +01:00
small fix
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
9f5a198c5d
meggert added 1 commit 2023-12-04 17:54:29 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
fec4f64a07
meggert added 1 commit 2023-12-04 18:07:29 +01:00
fixed datasets
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
f22236b419
meggert added 1 commit 2023-12-05 13:12:53 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
a05fd75828
meggert added 2 commits 2023-12-05 15:11:27 +01:00
this is less confusing
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
6c6b7ff53f
meggert added 1 commit 2023-12-06 10:41:57 +01:00
bump fsfw
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
c6a0518515
meggert added 1 commit 2023-12-06 13:06:43 +01:00
Merge branch 'dev-7.5.0' into higher-acs-modes-only-str
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
09aa7bb439
meggert added 1 commit 2023-12-06 14:36:30 +01:00
welp i guess i also deleted stuff i still needed
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
71193495f3
meggert requested review from muellerr 2023-12-06 14:41:48 +01:00
meggert changed title from WIP: Higher ACS Modes STR Only to Higher ACS Modes STR Only 2023-12-06 14:41:52 +01:00
meggert added 1 commit 2023-12-06 14:46:42 +01:00
changelog
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
4cad1176d0
muellerr reviewed 2023-12-06 15:34:43 +01:00
@ -0,0 +51,4 @@
double beta = VectorOperations<double>::dot(helperSum, kSumVec);
double gamma = std::sqrt(std::pow(alpha, 2) + std::pow(beta, 2));
// I don't even know what this is supposed to be
Owner

imagine me

imagine me
meggert marked this conversation as resolved
muellerr requested changes 2023-12-06 15:42:40 +01:00
@ -0,0 +6,4 @@
AttitudeEstimation::~AttitudeEstimation() {}
void AttitudeEstimation::quest(acsctrl::SusDataProcessed *susData,
Owner

some subroutines might be a good idea, a lot of if else if else blocks in this function..

some subroutines might be a good idea, a lot of if else if else blocks in this function..
meggert marked this conversation as resolved
@ -0,0 +11,4 @@
acsctrl::AttitudeEstimationData *attitudeEstimation) {
if (susData->susVecTot.isValid() and susData->sunIjkModel.isValid() and
mgmData->mgmVecTot.value and mgmData->magIgrfModel.isValid()) {
// Normalize Data
Owner

maybe this block can be put into a subfunction?

maybe this block can be put into a subfunction?
meggert marked this conversation as resolved
@ -10,0 +73,4 @@
void FusedRotationEstimation::estimateFusedRotationRateStr(
ACS::SensorValues *sensorValues, const double timeDelta,
acsctrl::FusedRotRateSourcesData *fusedRotRateSourcesData) {
if ((sensorValues->strSet.caliQw.isValid() and sensorValues->strSet.caliQx.isValid() and
Owner

maybe use the negation of all conditions here, move the invalid setting handling into the block, and put the block below without a return?

maybe use the negation of all conditions here, move the invalid setting handling into the block, and put the block below without a return?
meggert marked this conversation as resolved
@ -10,0 +136,4 @@
void FusedRotationEstimation::estimateFusedRotationRateQuest(
acsctrl::AttitudeEstimationData *attitudeEstimationData, const double timeDelta,
acsctrl::FusedRotRateSourcesData *fusedRotRateSourcesData) {
if (attitudeEstimationData->quatQuest.isValid()) {
Owner

same as above: Put the error handling code with the pre-emptive return at the top, save one level of indentation for the regular code.

same as above: Put the error handling code with the pre-emptive return at the top, save one level of indentation for the regular code.
meggert marked this conversation as resolved
@ -499,3 +498,1 @@
bool rw2valid = (sensorValues->rw2Set.state.value && sensorValues->rw2Set.state.isValid());
bool rw3valid = (sensorValues->rw3Set.state.value && sensorValues->rw3Set.state.isValid());
bool rw4valid = (sensorValues->rw4Set.state.value && sensorValues->rw4Set.state.isValid());
bool rw1valid = (sensorValues->rw1Set.state.value and sensorValues->rw1Set.state.isValid());
Owner

value is a ptr?

value is a ptr?
meggert marked this conversation as resolved
muellerr added 1 commit 2023-12-06 15:43:34 +01:00
Merge branch 'dev-7.5.0' into higher-acs-modes-only-str
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
42036f45f9
meggert added 1 commit 2023-12-06 15:56:37 +01:00
maybe this makes him happy
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
8cd773d18b
meggert added 1 commit 2023-12-06 16:02:16 +01:00
fix
All checks were successful
EIVE/eive-obsw/pipeline/pr-dev-7.5.0 This commit looks good
d762a2b703
muellerr approved these changes 2023-12-06 17:01:49 +01:00
meggert merged commit 2c9500c7aa into dev-7.5.0 2023-12-06 17:15:52 +01:00
meggert deleted branch higher-acs-modes-only-str 2023-12-06 17:15:52 +01:00
Sign in to join this conversation.
No description provided.