Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Create default fallback #65

Merged

Conversation

jacobtomlinson
Copy link
Contributor

Added a default lookup location for create_cluster(). If you fail to specify the path to a valid spec it will try and find it at dask-cluster.yaml.

>>> import dask_ctl.lifecycle
>>> dask_ctl.lifecycle.create_cluster()
DaskClusterConfigNotFound: Unable to find dask-cluster.yaml

You can also set the local_fallback=True option to start a LocalCluster if a spec file is not found.

>>> import dask_ctl.lifecycle
>>> dask_ctl.lifecycle.create_cluster(local_fallback=True)
LocalCluster(3ce576d4, 'tcp://127.0.0.1:8787', workers=4, threads=12, memory=93.02 GiB)

@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (afc0e5c) 71.14% compared to head (d00ace7) 72.01%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
+ Coverage   71.14%   72.01%   +0.86%     
==========================================
  Files          10       11       +1     
  Lines         357      368      +11     
==========================================
+ Hits          254      265      +11     
  Misses        103      103              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jacobtomlinson jacobtomlinson merged commit 1aed52e into dask-contrib:main Jan 25, 2024
12 checks passed
@jacobtomlinson jacobtomlinson deleted the create-default-fallback branch January 25, 2024 14:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants