search

airavata-courses / scapsulators

Looking to understand how weather phenomena in a region, affect your research? Well look no further!
https://github.com/airavata-courses/scapsulators/wiki/Weather-enthusiasts-assemble
Apache License 2.0
1 stars 2 forks source link

Project Setup Review (SuperNova) #66

Open sanket1211996 opened 2 years ago

sanket1211996 commented 2 years ago

Setup instructions are mostly clear. I am able to download the docker files. Following things could be more clearer.

  1. "Simply click on the entry-point script: start-Weatherpedia.sh" - Must provide direct shell script run command, because by simply clicking on the shell script it opened the file on VS Code Code Editor. That could be confusing for some user. Since I know how to execute script with cmd I was able to run it.

  2. You should provide the list on required port for you services in the instructions, to check and make them available before running the script of it will fail if any port is already in use. I have got the error port already in used for 5000. I tried to kill the port, but some type of default service is getting up again with new process ID. May be some default system application daemon is using that port. Do you have any suggestion.

Screen Shot 2022-02-09 at 11 26 35 PM
shubhpatr commented 2 years ago

Hey @sanket1211996 ,

Thanks for pointing out the issue. There is an option to change the port number in the gateway's environment config file.

If you want you can clone the gateway-feature branch, and follow the steps in the readme file of that branch to manually configure the gateway. Do not forget to change the port number.

Let me know if you get stuck again.

Regards, Shubham

sanket1211996 commented 2 years ago

Thanks for solution. Appreciated.

vikrantdeshpande09876 commented 2 years ago

This is a good point @sanket1211996. The same issue was highlighted in #63 where an Apple MacBook runs the Air Play application by default on port 5000, and we should have some script to check for ports-currently-in-use. We'll keep this in mind for our next release.