Skip to content

Commit

Permalink
Fix some small things (#874)
Browse files Browse the repository at this point in the history
  • Loading branch information
carolineechen authored Jun 18, 2024
1 parent 302b111 commit 9c1e248
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion runhouse/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -604,7 +604,7 @@ def _start_server(
flags.append(default_env_flag)

conda_env_flag = f" --conda-env {conda_env}" if conda_env else ""
if default_env_flag:
if conda_env_flag:
logger.info(f"Creating runtime env for conda env: {conda_env}")
flags.append(conda_env_flag)

Expand Down
1 change: 1 addition & 0 deletions runhouse/resources/packages/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ def _install(self, env: Union[str, "Env"] = None, cluster: "Cluster" = None):
path = self.install_target.path
else:
path = self.to(cluster).install_target.path
install_cmd = self._install_cmd(cluster=cluster)

if not path:
return
Expand Down
4 changes: 3 additions & 1 deletion runhouse/rns/defaults.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,9 @@ def set(self, key: str, value: Any, config_path: Optional[str] = None):

def set_nested(self, key: str, value: Any, config_path: Optional[str] = None):
"""Set a config key that has multiple key/value pairs"""
self.defaults_cache.setdefault(key, {}).update(value)
if not self.defaults_cache.get(key):
self.defaults_cache.setdefault(key, {})
self.defaults_cache[key].update(value)
self.save_defaults(config_path=config_path)

def set_many(self, key_value_pairs: Dict, config_path: Optional[str] = None):
Expand Down

0 comments on commit 9c1e248

Please sign in to comment.