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

Issue when running runBoth.sh #38

Closed
ecruz480 opened this issue May 18, 2019 · 6 comments
Closed

Issue when running runBoth.sh #38

ecruz480 opened this issue May 18, 2019 · 6 comments

Comments

@ecruz480
Copy link

ecruz480 commented May 18, 2019

I have successfully ran make rebuild_rpi but am running into issues when trying to launch the program itself with ./runBoth.sh.
Photo with runBoth issue.
Any help is appreciated, currently running on RPI 3b+ with Raspian 4.14 (Debian Stretch)

@timcanham
Copy link
Collaborator

It looks like a bug in the scripts when we converted to sh from tcsh:

if !($?BUILD_ROOT) then
    set curdir = "${PWD}"
    setenv BUILD_ROOT `dirname $0`/../..
    cd $BUILD_ROOT
    setenv BUILD_ROOT ${PWD}
    cd ${curdir}
endif

It should be something like:

if !($?BUILD_ROOT) then
    set curdir = "${PWD}"
    export BUILD_ROOT=`dirname $0`/../..
    cd $BUILD_ROOT
    export BUILD_ROOT=${PWD}
    cd ${curdir}
endif

Next time I get on a Pi, I'll try it, but perhaps you could try it for us.

@timcanham
Copy link
Collaborator

Note that sometimes the GUI has issues running on the Pi even if the scripts are working and it can be sloooooow. The alternate pattern is to run the GUI on a Linux machine (runGui.sh) then run the app on the Pi (runRPI.sh or runRPICross.sh if you cross-compiled). You will have to edit the two scripts and change the IP addresses to whatever network you are running on.

@ecruz480
Copy link
Author

Yes! Using export cleared up my prior issues but I am now stuck on python's importing tools.

utils.PortFinder import error

Just from reading a few posts online, it might be an issue with sys.path and how it is initialized by the program. I am currently running python 3.5.3 but I seems like there are still problems with how the system interprets what is a package directory and the careful use of __int__.py. I am an electrical engineer by trade and heart so my usefulness is trying to catch up to my eagerness, but I plan on trying to cross-compile it soon.

Link on import issue.

@LeStarch
Copy link
Collaborator

Yeah, the Gse/Gds layer was merged together, however; the "runBoth" script was not updated to reflect this.

export PYTHONPATH="${BUILD_ROOT}/Gse/src:${BUILD_ROOT}/Gse/generated/FlightDemo/serializable"

Should read something like:

export PYTHONPATH="${BUILD_ROOT}/Gds/src:${BUILD_ROOT}/RPI/py_dict/serializable"

@LeStarch
Copy link
Collaborator

@timcanham can we close this, or do we need to create a run-on-rpi wrapper?

@LeStarch
Copy link
Collaborator

LeStarch commented Sep 3, 2019

Closing.

@LeStarch LeStarch closed this as completed Sep 3, 2019
r9-pena added a commit to r9-pena/fprime that referenced this issue Jul 23, 2021
LeStarch pushed a commit that referenced this issue Aug 4, 2021
* Create tutorial-support.yml

* Update tutorial-support.yml

GpsApp tutorial to be pulled for build test and submodules to be pulled recursively

* Implementation of tutorial workflow

* typo in repo address

* Added missing github action file

* Changed workflow path

* Changed dir paths

* Work around for workflow

* work around

* troubleshooting

* troubleshoot

* troubleshoot

* Performance update

* Path correction

* troubleshoot

* Dockerfile correction

* path change

* path change

* path fix

* path fix

* path fix

* path fix

* paht fix

* Revert to changes

* Completed workflow for tutorial support

* troubleshoot

* Build path error

* Optimization of workflow

* Path fix

* Path fix

* Path fix #1

* path fix #2

* path check

* Workflow test

* Workflow test #2

* Added logging feature

* Logs Troubleshoot

* Typo fix

* Troubleshoot #1

* Troubleshoot #2

* Corrected typo on path

* Troubleshoot #3

* Troubleshoot #4

* Troubleshoot #5

* Removed log archive feature

* Changed repo path to conform pull request merge

* Test run #1

* test #2

* test #3

* added branch for workflow test

* test #4

* test #4

* test #5

* test #6

* Test #7

* test #8

* test #8

* Test #9

* Test #9

* Test #10

* Test #11

* Test #12

* Test #13

* Test #14

* Test #15

* Test #16

* Test #17

* Test #18

* Test #20

* Test #21

* Test #22

* Test #23

* Test #24

* Test #25

* Split test files

* Split tests #2

* Added executable permissions

* Delete RPI.bash

* Delete Ref.bash

* Delete Framework.bash

* exe files

* Make Framework.bash executable

* Make RPI.bash and Ref.bash executable

* Test #26

* Test #27

* Test #28

* Attached Integration test to Ref test

* Test #29

* Test #30

* Test #31

* Test #32

* Test #33

* Test #34

* Test #35

* Test #35

* Test #37

* Test #38

* Test #39

* Test #40

* Test #41

* Test #42

* Test #43

* Test #43

* Test #44

* Test #45

* Cleaned up files to remove commented code

* Remove CI test branch from workflow

* Incorporated comments for PR

* Modified path for framework job test

* Incoporate reviewer comment for PR

* Incoporated reviewer comment for PR

* Was using incorrect directory for the test

* Revised args call for jobs

* Passing args #1

* Define entrypoint directly from workflow

* Changed entrypoint method

* Changed entrypoint method #2

* Corrected test path

* Syntax correction

* Path Test #1

* Path test #2

* Path Test #4

* Path Test $5

* Test Path #6

* Path Test #6

* Path Test #7

* Path Test #8

* Refactored scripts to remove master.bash

* Corrected test path

* Fixed log problems

* Entrypoint alternative

* Bypassed master.bash file

* Added line for better CI error messages
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

No branches or pull requests

3 participants