Fix PropertyLayer visualization tutorial to use new SpaceRenderer setup API#3223
Fix PropertyLayer visualization tutorial to use new SpaceRenderer setup API#3223ShreyasN707 wants to merge 19 commits intomesa:mainfrom
Conversation
|
@EwoutH I have updated the documentation. |
| "if mesa.__version__.startswith((\"3.0\", \"3.1\", \"3.2\")):\n", | ||
| " print(\n", | ||
| " f\"⚠️ Mesa {mesa.__version__} detected. Visualization features require Mesa 3.3+\"\n", | ||
| " f\"\u26a0\ufe0f Mesa {mesa.__version__} detected. Visualization features require Mesa 3.3+\"\n", |
There was a problem hiding this comment.
what's going on here? and in other places with weird unicode related stuff?
There was a problem hiding this comment.
Both lines render the same warning symbol (
There was a problem hiding this comment.
For me instead of the word "warning", I'd prefer this symbol as it's easy to see.
|
I believe the |
|
The changes of this PR are good, but there seems to be some problem with Failed to load model class 'HtmlModel' from module 'jupyter-vuetify'
Error: Script error for "jupyter-vue", needed by: jupyter-vuetify
http://requirejs.org/docs/errors.html#scripterror
at makeError (https://cdnjs.cloudflare.com/ajax/libs/require.js/2.3.4/require.min.js:1:1067)
at HTMLScriptElement.onScriptError (https://cdnjs.cloudflare.com/ajax/libs/require.js/2.3.4/require.min.js:1:13218)@EwoutH do you have any idea what might be causing this? Because upon searching I could only find this old issue which may not be entirely relevant: voila-dashboards/voila#619 |
Even I faced the same. This seems to be related to the Jupyter widget environment (likely Failed to load model class 'HtmlModel' from module 'jupyter-vuetify'
Script error for "jupyter-vue" |
|
Can you take a look at the conflict and update this PR accordingly. |
|
yes will do it |
|
Closing this as stale |





This updates the Property Layer Visualization tutorial to use setup_structure, setup_agents, and setup_propertylayer instead of deprecated render(...) arguments, fixing the Binder crash reported in #3216.
issue : #3216