diff --git a/CHANGELOG.md b/CHANGELOG.md index 44c98cb5..8b2c84ad 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -47,6 +47,7 @@ will consitute of a breaking change warranting a new major release: maximum size of 255 bytes. - TCS CTRL: Limit number of heater handler messages sent in case there are not sensors available anymore. +- Fix to allow adding real STR device for EM. # Added diff --git a/bsp_q7s/em/emObjectFactory.cpp b/bsp_q7s/em/emObjectFactory.cpp index a7b05e3d..e5d6abc2 100644 --- a/bsp_q7s/em/emObjectFactory.cpp +++ b/bsp_q7s/em/emObjectFactory.cpp @@ -82,6 +82,9 @@ void ObjectFactory::produce(void* args) { // The ACU broke. dummyCfg.addOnlyAcuDummy = true; #endif +#if OBSW_ADD_STAR_TRACKER == 1 + dummyCfg.addStrDummy = false; +#endif #if OBSW_ADD_BPX_BATTERY_HANDLER == 1 dummyCfg.addBpxBattDummy = false; #endif diff --git a/bsp_q7s/objectFactory.cpp b/bsp_q7s/objectFactory.cpp index e6ae6603..326ba8ef 100644 --- a/bsp_q7s/objectFactory.cpp +++ b/bsp_q7s/objectFactory.cpp @@ -937,16 +937,7 @@ void ObjectFactory::createStrComponents(PowerSwitchIF* pwrSwitcher) { starTrackerCookie->setNoFixedSizeReply(); StrComHandler* strComIF = new StrComHandler(objects::STR_COM_IF); - const char* paramJsonFile = nullptr; -#ifdef EGSE - paramJsonFile = "/home/pi/arcsec/json/flight-config.json"; -#else -#if OBSW_STAR_TRACKER_GROUND_CONFIG == 1 - paramJsonFile = "/mnt/sd0/startracker/ground-config.json"; -#else - paramJsonFile = "/mnt/sd0/startracker/flight-config.json"; -#endif -#endif + const char* paramJsonFile = "/mnt/sd0/startracker/flight-config.json"; if (paramJsonFile == nullptr) { sif::error << "No valid Star Tracker parameter JSON file" << std::endl; } diff --git a/dummies/helperFactory.cpp b/dummies/helperFactory.cpp index 84d6acc9..e085dae0 100644 --- a/dummies/helperFactory.cpp +++ b/dummies/helperFactory.cpp @@ -63,11 +63,14 @@ void dummy::createDummies(DummyCfg cfg, PowerSwitchIF& pwrSwitcher, GpioIF* gpio rws[3] = new RwDummy(objects::RW4, objects::DUMMY_COM_IF, comCookieDummy); ObjectFactory::createRwAssy(pwrSwitcher, power::Switches::PDU2_CH2_RW_5V, rws, rwIds); new SaDeplDummy(objects::SOLAR_ARRAY_DEPL_HANDLER); - auto* strAssy = new StrAssembly(objects::STR_ASSY); - strAssy->connectModeTreeParent(satsystem::acs::ACS_SUBSYSTEM); - auto* strDummy = - new StarTrackerDummy(objects::STAR_TRACKER, objects::DUMMY_COM_IF, comCookieDummy); - strDummy->connectModeTreeParent(*strAssy); + + if (cfg.addStrDummy) { + auto* strAssy = new StrAssembly(objects::STR_ASSY); + strAssy->connectModeTreeParent(satsystem::acs::ACS_SUBSYSTEM); + auto* strDummy = + new StarTrackerDummy(objects::STAR_TRACKER, objects::DUMMY_COM_IF, comCookieDummy); + strDummy->connectModeTreeParent(*strAssy); + } if (cfg.addSyrlinksDummies) { auto* syrlinksDummy = new SyrlinksDummy(objects::SYRLINKS_HANDLER, objects::DUMMY_COM_IF, comCookieDummy); diff --git a/dummies/helperFactory.h b/dummies/helperFactory.h index 9c1a93d3..fb272bb8 100644 --- a/dummies/helperFactory.h +++ b/dummies/helperFactory.h @@ -27,6 +27,7 @@ struct DummyCfg { bool addTempSensorDummies = true; bool addRtdComIFDummy = true; bool addPlocDummies = true; + bool addStrDummy = true; bool addTmpDummies = true; Tmp1075Cfg tmp1075Cfg; bool addCamSwitcherDummy = false;