Repository navigation
Conversation
Updated debug output step to provide more detailed file listings.
Added script metadata and dependencies for matchedfiltering.py
Added filter_and_plot_edit and plot_td_and_fd_edit functions for enhanced plotting and filtering capabilities.
Added runtime configuration and scripts for marimo tool.
Removed debug output step from CI workflow.
Add a step to show resolved marimo config in CI workflow
Removed the upload document_root directory step from CI workflow.
Set maximum output bytes for Marimo conversion.
Added metadata for script requirements and dependencies.
Removed unused cell function that prints import statement.
|
Looks great. Here are some things to fix.
|
Updated notebook description and removed unused imports and functions.
|
Here's what Claude has to say about the memory error: Memory leak in the browser version: equalizer sliders crash the kernelIn the WASM (browser) version, dragging a few of the equalizer sliders down to −200 makes the kernel run out of memory: After that, the notebook stops responding until the page is reloaded. CauseEvery slider movement reruns the equalizer cell, and each rerun makes a new Matplotlib figure. These figures are expensive. Each holds
MeasurementsI dragged sliders to −200 in headless Chrome and recorded the WebAssembly heap and the number of live figures after each drag:
Suggested fixes
I tested the gc.collect() fix on my scratch copy of the notebook, in which the helpers are folded into the notebook and the data loads from |
Removed environment.yml and deploy.yml used for binder hosting. Added pyproject.yoml and ci.yml for github pages hosting. Added matchedfiltering.py (Marimo notebook using plotly) and _matchedfiltering_mpl.py (Marimo notebook using matplotlib). Configured pyproject.toml and ci.yml to host _matchedfiltering_mpl.py.