Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update coiled tutorials #278

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

scharlottej13
Copy link
Contributor

Small update since Coiled is no longer doing live tutorials. More context in #275.

cc @jacobtomlinson @jrbourbeau

@dask-bot
Copy link

Beep boop! Here is a Binder link where you can try out this change. Binder

@scharlottej13
Copy link
Contributor Author

Looks like CI is failing because of a dependency in distributed, but looks like the dask + distributed versions are pinned in https://github.com/dask/dask-tutorial/blob/main/binder/environment.yml so not sure why:

ModuleNotFoundError                       Traceback (most recent call last)
File /usr/share/miniconda3/envs/dask-tutorial/lib/python3.10/site-packages/distributed/security.py:165, in Security.temporary(cls, **kwargs)
    164 try:
--> 165     from cryptography import x509
    166     from cryptography.hazmat.backends import default_backend

ModuleNotFoundError: No module named 'cryptography'

During handling of the above exception, another exception occurred:

ImportError                               Traceback (most recent call last)
Cell In[9], line 4
      1 from dask.distributed import Scheduler, Worker, Client
      2 from distributed.security import Security
----> 4 security = Security.temporary()
      6 async with Scheduler(security=security) as scheduler:
      7     async with Worker(scheduler.address, security=security) as worker:

File /usr/share/miniconda3/envs/dask-tutorial/lib/python3.10/site-packages/distributed/security.py:171, in Security.temporary(cls, **kwargs)
    169     from cryptography.x509.oid import NameOID
    170 except ImportError:
--> 171     raise ImportError(
    172         "Using `Security.temporary` requires `cryptography`, please "
    173         "install it using either pip or conda"
    174     )
    175 key = rsa.generate_private_key(
    176     public_exponent=65537, key_size=2048, backend=default_backend()
    177 )
    178 key_contents = key.private_bytes(
    179     encoding=serialization.Encoding.PEM,
    180     format=serialization.PrivateFormat.PKCS8,
    181     encryption_algorithm=serialization.NoEncryption(),
    182 ).decode()

ImportError: Using `Security.temporary` requires `cryptography`, please install it using either pip or conda

Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great thanks. I'm not too worries about the CI failures given this is just a text change.

@jacobtomlinson jacobtomlinson merged commit 3157ed8 into dask:main Nov 17, 2023
2 of 3 checks passed
@scharlottej13
Copy link
Contributor Author

This looks great thanks. I'm not too worries about the CI failures given this is just a text change.

Thanks @jacobtomlinson!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants