-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes to get rpi mission working with recent NMF #2
base: master
Are you sure you want to change the base?
Conversation
Hi Sacha, Thanks for your contribution! Best regards, |
Yes, it actually only works with the phi-sat-2 branch, as the code already expects some dependencies from it. |
I think you compile it against the dev branch because the phi-sat-2 has the new AI service that you removed. |
I added the updated readme and will check a clean install to see if i indeed mixed the branches up. I was playing around with the framework before, so that is very much possible. I will alter this PR accordingly.
|
I just checked everything again, and compiled against phi-sat-2
Adding orekit solves this error |
Hi @CesarCoelho sorry for the commit mess, but I have now removed orekit from the code, so the dependency could be removed again, and further fixed the docker and rp environment so it works with the phi-sat-2 branch. Except for the PR still open in that branch, let me know if you rather have me add that here, although I think my proposed change in esa/nanosat-mo-framework#184 makes sense to implement never-the-less. |
This PR includes four small fixes to the code to get it to compile and upload to an RPi, including updated documentation to reflect changes.
Another issue seemed to be better to fix at the framework level, that is described in the tiny pull request at: esa/nanosat-mo-framework#184
Let me know if you want me to split this up, or alter in any way,
thanks!