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

adding start.sh to run the python file. #136

Closed
wants to merge 2 commits into from
Closed

adding start.sh to run the python file. #136

wants to merge 2 commits into from

Conversation

sunny1401
Copy link

Describe the contribution
A clear and concise description of what the contribution is.

Testing performed
Steps taken to test the contribution:
Ran the ground system using:

  • python GroundSystem.py
  • python3 GroundSystem.py
  • bash start.sh

Expected behavior changes
All three of the above commands open the form with further buttons for The groundSystem.

System(s) tested on
MacOS Catalina

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

  • Sankranti Joshi (Personal)

@sunny1401
Copy link
Author

@CDKnightNASA - Hi - do you think this is what you had in mind. I am not quite sure. I just added a bash file to run the code.

@sunny1401 sunny1401 closed this Oct 30, 2020
@skliper
Copy link
Contributor

skliper commented Nov 2, 2020

Any context as to why this was closed? @astrogeco

@sunny1401
Copy link
Author

sunny1401 commented Nov 2, 2020

sorry - I closed this.. I saw another PR [https://github.com//pull/137], which is on the same issue.. and I felt maybe that was what you were looking for. Hence I closed it. Just to avoid confusions. I think I misunderstood the problem. Sorry. @skliper

@skliper skliper added the duplicate This issue or pull request already exists label Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants