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

MAINT: update config for jb==0.15.1 #219

Merged
merged 7 commits into from
Mar 24, 2023
Merged

MAINT: update config for jb==0.15.1 #219

merged 7 commits into from
Mar 24, 2023

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented Mar 22, 2023

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

@mmcky
Copy link
Contributor Author

mmcky commented Mar 22, 2023

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
Copy link
Contributor Author

mmcky commented Mar 22, 2023

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

@mmcky
Copy link
Contributor Author

mmcky commented Mar 23, 2023

  • check if ecda245 fixes the execution error once completed

@mmcky
Copy link
Contributor Author

mmcky commented Mar 23, 2023

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
Copy link
Contributor Author

mmcky commented Mar 23, 2023

@mmcky
Copy link
Contributor Author

mmcky commented Mar 23, 2023

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

@mmcky
Copy link
Contributor Author

mmcky commented Mar 23, 2023

@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
Copy link
Contributor Author

mmcky commented Mar 23, 2023

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

Perhaps #200 needs to be done :-)

@doctor-phil
Copy link
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
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
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