Open DeflateAwning opened 4 months ago
Chatted with @DeemDeem52 on the path forward with this repo and using telecommands! He's working on it, and will get a unit test done for Tuesday so we can discuss any issues had!
I've gotten started updating the functions for unit testing and telecommands. It's going to take some time, though. (The ADCS team is having problems right now, so we're not actually losing any time, though)
Refactoring is making excellent progress. I've got unit tests up and running, which are showing me that I need to do some bugfixing. The ADCS team has also managed to fix the problem they were having, so testing can also resume!
Fantastic news and updates! Happy to hear that the unit tests are helping fix bugs before they're hard to detect later on; that's the ideal scenario!
Refactoring of the existing functions is just about complete (barring some minor things which don't affect how the code runs -- those will be fixed after I've got all the code up and functional)! Additional news, the latest bugfix combined with some troubleshooting from the ADCS team has fixed the issues with the ADCS, so we're back up and running for testing again. Although for some reason I seem to have created two divergent branches in my attempts to rename the ADCSFirmware branch... which somehow still has the latest commits. I'll get on fixing that once I can figure out how it happened.
I've made some really good progress on writing telecommands for each of the major functions. Out of 60 or so telecommands, there should be 9 remaining. Up next after finishing those 9 is writing more unit tests and testing... then this will be complete!
How's the progress of the testing? I'd love to see testing and finalize the structure of just a couple telecommands to minimize any reworking we have to do across all the telecommands if there are issues/opportunities for improvement.
Our next testing date will be Saturday the 20th, so we'll likely be testing the telecommands then.
ADCS functions (I2C interface) are being developed separately. Integrate them into this repo when complete.