Skip to content

MAINT: update config for jb==0.15.1#219

Merged
doctor-phil merged 7 commits into
mainfrom
update-config
Mar 24, 2023
Merged

MAINT: update config for jb==0.15.1#219
doctor-phil merged 7 commits into
mainfrom
update-config

Conversation

@mmcky

@mmcky mmcky commented Mar 22, 2023

Copy link
Copy Markdown
Contributor

This PR adjusts the config for the project due to some changes in jupyter-book

@mmcky

mmcky commented Mar 22, 2023

Copy link
Copy Markdown
Contributor Author

Issues that need to be resolved

  • /home/runner/work/lecture-datascience.myst/lecture-datascience.myst/lectures/applications/maps.md:466: WARNING: skipping unknown output mime type: application/vnd.bokehjs_load.v0+json [mystnb.unknown_mime_type]
  • /home/runner/work/lecture-datascience.myst/lecture-datascience.myst/lectures/applications/maps.md:480: WARNING: skipping unknown output mime type: application/vnd.bokehjs_exec.v0+json [mystnb.unknown_mime_type]
  • /home/runner/work/lecture-datascience.myst/lecture-datascience.myst/lectures/applications/recidivism.md: WARNING: Executing notebook failed: CellExecutionError [mystnb.exec]
    /home/runner/work/lecture-datascience.myst/lecture-datascience.myst/lectures/applications/recidivism.md: WARNING: Notebook exception traceback saved in: /home/runner/work/lecture-datascience.myst/lecture-datascience.myst/_build/html/reports/applications/recidivism.err.log [mystnb.exec]

@mmcky

mmcky commented Mar 22, 2023

Copy link
Copy Markdown
Contributor Author

@doctor-phil will need to think about the first two. Not sure what is driving that.

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author
  • check if ecda245 fixes the execution error once completed

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author

The following error is being issued:

ValueError: keyword grid_b is not recognized; valid keywords are ['size', 'width', 'color', 'tickdir', 'pad', 'labelsize', 'labelcolor', 'zorder', 'gridOn', 'tick1On', 'tick2On', 'label1On', 'label2On', 'length', 'direction', 'left', 'bottom', 'right', 'top', 'labelleft', 'labelbottom', 'labelright', 'labeltop', 'labelrotation', 'grid_agg_filter', 'grid_alpha', 'grid_animated', 'grid_antialiased', 'grid_clip_box', 'grid_clip_on', 'grid_clip_path', 'grid_color', 'grid_dash_capstyle', 'grid_dash_joinstyle', 'grid_dashes', 'grid_data', 'grid_drawstyle', 'grid_figure', 'grid_fillstyle', 'grid_gapcolor', 'grid_gid', 'grid_in_layout', 'grid_label', 'grid_linestyle', 'grid_linewidth', 'grid_marker', 'grid_markeredgecolor', 'grid_markeredgewidth', 'grid_markerfacecolor', 'grid_markerfacecoloralt', 'grid_markersize', 'grid_markevery', 'grid_mouseover', 'grid_path_effects', 'grid_picker', 'grid_pickradius', 'grid_rasterized', 'grid_sketch_params', 'grid_snap', 'grid_solid_capstyle', 'grid_solid_joinstyle', 'grid_transform', 'grid_url', 'grid_visible', 'grid_xdata', 'grid_ydata', 'grid_zorder', 'grid_aa', 'grid_c', 'grid_ds', 'grid_ls', 'grid_lw', 'grid_mec', 'grid_mew', 'grid_mfc', 'grid_mfcalt', 'grid_ms']

from

nbclient.exceptions.CellExecutionError: An error occurred while executing the following cell:
------------------
# plots illustrating regularization parameter choice
scores=lasso_mod.scores_[1.0].mean(axis=0)
logpenalties=np.log(lasso_mod.Cs_)
nnonzero=(np.abs(lasso_mod.coefs_paths_[1.0])>1e-6).sum(axis=2).mean(axis=0)
colors=plt.rcParams["axes.prop_cycle"].by_key()["color"]
fig, ax1 = plt.subplots()
ax1.plot(logpenalties,scores, color=colors[0])
ax1.set_ylabel("CV log likelihood", color=colors[0])
ax1.set_xlabel("-log(penalty)")
ax1.tick_params('y', colors=colors[0])
ax2 = ax1.twinx()
ax2.plot(logpenalties,nnonzero, color=colors[1])
ax2.set_ylabel("nonzero coefficients", color=colors[1])
ax2.tick_params('y', colors=colors[1])
ax2.grid(b=None);
------------------

for recidivism

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author
  • update to use jupyter-book build -nW --keep-going lectures

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author

@doctor-phil I have forced a full rebuild as a test (without the cache) to see if that uses the latest code in that cell.

@mmcky

mmcky commented Mar 23, 2023

Copy link
Copy Markdown
Contributor Author

@doctor-phil I have an issue with Fiona when trying to install this environment.

Perhaps #200 needs to be done :-)

@doctor-phil

Copy link
Copy Markdown
Collaborator

@mmcky 2/3 windows devices that I have tried to build on have had errors involving Fiona or Geopandas. That was before the current environment build failures involving the mysterious Python 3.11 error

Fixing recidivism error on this branch re: #219
@doctor-phil

Copy link
Copy Markdown
Collaborator

The following error is being issued:

ValueError: keyword grid_b is not recognized; valid keywords are ['size', 'width', 'color', 'tickdir', 'pad', 'labelsize', 'labelcolor', 'zorder', 'gridOn', 'tick1On', 'tick2On', 'label1On', 'label2On', 'length', 'direction', 'left', 'bottom', 'right', 'top', 'labelleft', 'labelbottom', 'labelright', 'labeltop', 'labelrotation', 'grid_agg_filter', 'grid_alpha', 'grid_animated', 'grid_antialiased', 'grid_clip_box', 'grid_clip_on', 'grid_clip_path', 'grid_color', 'grid_dash_capstyle', 'grid_dash_joinstyle', 'grid_dashes', 'grid_data', 'grid_drawstyle', 'grid_figure', 'grid_fillstyle', 'grid_gapcolor', 'grid_gid', 'grid_in_layout', 'grid_label', 'grid_linestyle', 'grid_linewidth', 'grid_marker', 'grid_markeredgecolor', 'grid_markeredgewidth', 'grid_markerfacecolor', 'grid_markerfacecoloralt', 'grid_markersize', 'grid_markevery', 'grid_mouseover', 'grid_path_effects', 'grid_picker', 'grid_pickradius', 'grid_rasterized', 'grid_sketch_params', 'grid_snap', 'grid_solid_capstyle', 'grid_solid_joinstyle', 'grid_transform', 'grid_url', 'grid_visible', 'grid_xdata', 'grid_ydata', 'grid_zorder', 'grid_aa', 'grid_c', 'grid_ds', 'grid_ls', 'grid_lw', 'grid_mec', 'grid_mew', 'grid_mfc', 'grid_mfcalt', 'grid_ms']

from

nbclient.exceptions.CellExecutionError: An error occurred while executing the following cell:
------------------
# plots illustrating regularization parameter choice
scores=lasso_mod.scores_[1.0].mean(axis=0)
logpenalties=np.log(lasso_mod.Cs_)
nnonzero=(np.abs(lasso_mod.coefs_paths_[1.0])>1e-6).sum(axis=2).mean(axis=0)
colors=plt.rcParams["axes.prop_cycle"].by_key()["color"]
fig, ax1 = plt.subplots()
ax1.plot(logpenalties,scores, color=colors[0])
ax1.set_ylabel("CV log likelihood", color=colors[0])
ax1.set_xlabel("-log(penalty)")
ax1.tick_params('y', colors=colors[0])
ax2 = ax1.twinx()
ax2.plot(logpenalties,nnonzero, color=colors[1])
ax2.set_ylabel("nonzero coefficients", color=colors[1])
ax2.tick_params('y', colors=colors[1])
ax2.grid(b=None);
------------------

for recidivism

@mmcky I changed the code on this branch to remove the depreciated keyword argument b=None. This error should no longer appear, we'll see what happens

@github-actions github-actions Bot temporarily deployed to commit March 23, 2023 22:27 Inactive
@doctor-phil

Copy link
Copy Markdown
Collaborator

@mmcky it's building now. Won't merge now in case you're still working on

  • update to use jupyter-book build -nW --keep-going lectures

@doctor-phil doctor-phil self-requested a review March 23, 2023 23:44
@doctor-phil doctor-phil merged commit 791ca19 into main Mar 24, 2023
@doctor-phil doctor-phil deleted the update-config branch March 29, 2023 18:42
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.

2 participants