Skip to content

Commit

Permalink
chore(cmorizer): cleanup some comments in the dask init part
Browse files Browse the repository at this point in the history
  • Loading branch information
pgierz committed Nov 28, 2024
1 parent e0ae49f commit eb5fb02
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/pymorize/cmorizer.py
Original file line number Diff line number Diff line change
Expand Up @@ -153,14 +153,13 @@ def _post_init_create_dask_cluster(self):
raise ValueError(
"You need to specify adapt or fixed for pymorize.dask_cluster_scaling_mode"
)
# Wait for at least min_jobs to be available...
# FIXME: Client needs to be available here?
# FIXME: Include the gateway option if possible
# FIXME: Does ``Client`` needs to be available here?
logger.info(f"Cluster can be found at: {self._cluster=}")
logger.info(f"Dashboard {self._cluster.dashboard_link}")
# NOTE(PG): In CI context, os.getlogin and nodename may not be available (???)

username = getpass.getuser()
nodename = getattr(os.uname(), "nodename", "UNKNOWN")
# FIXME: Include the gateway option if possible
logger.info(
"To see the dashboards run the following command in your computer's "
"terminal:\n"
Expand Down

0 comments on commit eb5fb02

Please sign in to comment.