This is a small change where master_host and master_port are read from optional environment variables (if they are not present behavior stays the same). In our use case we need to specify both the host and port because we are running dist-keras on Docker. I thought it would be useful for others who are using Docker or in the case where the Spark master is running in a container. For us using environment variables rather than passing these in as parameters makes more sense because we don't want everyone using this package in our system to have to worry about this. Hope this is helpful.
Hi,
This is a small change where
master_hostandmaster_portare read from optional environment variables (if they are not present behavior stays the same). In our use case we need to specify both the host and port because we are running dist-keras on Docker. I thought it would be useful for others who are using Docker or in the case where the Spark master is running in a container. For us using environment variables rather than passing these in as parameters makes more sense because we don't want everyone using this package in our system to have to worry about this. Hope this is helpful.