Skip to content
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

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

sachavg
Copy link

@sachavg sachavg commented Jun 13, 2023

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!

@CesarCoelho
Copy link
Contributor

Hi Sacha,

Thanks for your contribution!
Is this working with the phi-sat-2 branch? Eventually the phi-sat-2 will become the main branch, so I don't want to go back and then to upgrade it again to phi-sat-2. That would be double effort.

Best regards,
César Coelho

@sachavg
Copy link
Author

sachavg commented Jun 13, 2023

Yes, it actually only works with the phi-sat-2 branch, as the code already expects some dependencies from it.
(it however only works if esa/nanosat-mo-framework#184 is merged to the phi-sat-2 branch)

@CesarCoelho
Copy link
Contributor

I think you compile it against the dev branch because the phi-sat-2 has the new AI service that you removed.
Also, I removed the orekit dependency that was just bloating the deployment, I see that you are adding it back.
Try deleting your .m2 repository and compiling from scratch with the phi-sat-2 branch. :)

@sachavg
Copy link
Author

sachavg commented Jun 13, 2023

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.

ArtificialIntelligenceInheritanceSkeleton() was not removed, but it was also defined at the bottom of the same file, so the compiler did not like that, and i removed one of the instances. (not sure if I removed the right one though)

@sachavg
Copy link
Author

sachavg commented Jun 13, 2023

I just checked everything again, and compiled against phi-sat-2
Without orekit I get the following error regarding GPSSoftSimAdapter:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:compile (default-compile) on project nmf-platform-impl-raspberrypi: Compilation failure: Compilation failure: 
[ERROR] /home/sacha/os3/nmf-mission-raspberry-pi/platform-service-impl/src/main/java/esa/mo/platform/impl/provider/raspberrypi/GPSSoftSimAdapter.java:[30,45] package org.orekit.propagation.analytical.tle does not exist
[ERROR] /home/sacha/os3/nmf-mission-raspberry-pi/platform-service-impl/src/main/java/esa/mo/platform/impl/provider/raspberrypi/GPSSoftSimAdapter.java:[63,12] cannot find symbol
[ERROR]   symbol:   class TLE
[ERROR]   location: class esa.mo.platform.impl.provider.raspberrypi.GPSSoftSimAdapter
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

Adding orekit solves this error

@sachavg
Copy link
Author

sachavg commented Jul 14, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants