-
Notifications
You must be signed in to change notification settings - Fork 6
Upgrade JupyterBook to v2 #1918
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
base: main
Are you sure you want to change the base?
Conversation
* sb125_fund_split_analysis * high_quality_transit_areas - needed to migrate to notebook GCSPandas * ahsc
57cea77 to
f6cef4b
Compare
* remove logos from README files - images were stretched and we have the logo in the header now * move all markdown H1 headings to top of their files - so that MyST handles de-duping * some pre-commit reformatting
f6cef4b to
fc2448b
Compare
|
currently using fixed widths for webapp maps -- can we make that dynamic somehow? data-analyses/_shared_utils/shared_utils/webmap_utils.py Lines 149 to 157 in d473f0c
|
3539f34 to
79d830a
Compare
|
Hey @edasmalchi , we noticed that JupyterBook v2 changes the way URLs for the sites and subpages are generated. For instance, with v1 we have https://analysis.dds.dot.ca.gov/hqta_explorer/00__19_spatial_pathway__.html and or v2 we have https://analysis-staging.dds.dot.ca.gov/hqta_explorer/spatial-pathway/index.html. Is this ok? Do you all reference these v1 URLs elsewhere or know of people relying on these subpage URLs? |




This PR upgrades JupyterBook to v2 and also removes and gitignores compiled portfolio sites. Many of the file changes in this PR are a result of removing the portfolio site files, so there is quite a bit of noise.
You can see the result of this upgrade on staging for these portfolio sites:
When attempting to deploy other sites, I ran into the following errors that analysts will need to address in the notebooks before we can deploy.
Relates to #1854