Skip to content

Commit

Permalink
Merge pull request #256 from UCL/saransh/fix-dask
Browse files Browse the repository at this point in the history
fix: graphviz must be installed for dask visualizations
  • Loading branch information
Saransh-cpp authored Sep 29, 2024
2 parents 38cee26 + 1b4fd33 commit f874121
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/build_site.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,10 @@ jobs:
with:
python-version: 3.12
- name: Install dependencies
run: python -m pip install -r .github/python/requirements.txt
run: |
sudo apt-get update
sudo apt-get install python3-pydot graphviz
python -m pip install -r .github/python/requirements.txt
- name: Building notes
run: make ready
- uses: ruby/setup-ruby@v1
Expand Down
2 changes: 2 additions & 0 deletions ch08performance/030misc-libs.ipynb.py
Original file line number Diff line number Diff line change
Expand Up @@ -329,6 +329,8 @@ def mandel_numpy(position,limit=50):

# %%
df

# %% [markdown]
# The computation gave us a dask object and not the actual answer. Why is that?
# Displaying the dataframe just displays the metadata of the variable, and not any
# data. This is because of the "lazy" nature of dask. Dask has "lazy" execution,
Expand Down

0 comments on commit f874121

Please sign in to comment.