v0.1.9 #123

akshayka · 2023-09-19T02:32:39Z

No description provided.

vercel · 2023-09-19T02:32:46Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
marimo-docs	✅ Ready (Inspect)	Visit Preview	Sep 19, 2023 2:32am
marimo-storybook	✅ Ready (Inspect)	Visit Preview	Sep 19, 2023 2:32am

@akshayka

## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://discord.gg/JE7nhX6mD8), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [x] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary Add a quotation mark for the pip install command for `marimo[sql]`. I was confused why it didn't work when I ran this command. You would get a `zsh: no matches found` error by default. This is just a small QoL improvement.  ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

…rt` (#2850) ## 📝 Summary  Adds support for extracting inline script metadata from notebook cells, and lifting the comment block to the top of the converted marimo Python script. This allows for nice interchange between a tool like juv and marimo: ```sh juv init juv add Untitled.ipynb anywidget polars marimo convert Untitled.ipynb > Untitled.py marimo edit Untitled.py ``` ## 🔍 Description of Changes Adds regex to extract metadata from inline script comments, and lifts the comment block to the top of the converted marimo Python script.  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [x] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

## 📝 Summary  Fixes #2060 . Adding MathJax library in PlotlyPlugin to render LaTeX for Plotly. ## 🔍 Description of Changes  - Using MathJax 2 library from https://www.mathjax.org/#gettingstarted . Should use v3 in the future once all v2 features are ported over. - created a custom hook as the uidotdev useScript hook isn't suitable when there are multiple components calling `useScript` at the same time ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  WIP

@mscolnick

## 📝 Summary  Fixes #2337 . Upgrade `plotly.js` by running `pnpm up plotly.js` ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @mscolnick

@akshayka

## 📝 Summary  <img width="821" alt="Screenshot 2024-11-22 at 12 40 23 AM" src="https://github.com/user-attachments/assets/dec51f62-c9c5-477c-ac94-03be253c0f20"> Long continuous text (no spaces) is not wrapped, this change is to force text to wrap. ## 🔍 Description of Changes  - Use `break-words` as it will add line breaks mid-word if needed. (https://tailwindcss.com/docs/word-break#break-words) - Sample code: ``` df = pl.DataFrame({ "numbers": [list(range(1,100))], "long_string": ['abc' * 100], "long_string_spaced": ['abc ' * 30] }) mo.ui.table(df, wrapped_columns=["numbers", "long_string", "long_string_spaced"]) ``` As a result: <img width="799" alt="Screenshot 2024-11-22 at 12 29 32 AM" src="https://github.com/user-attachments/assets/0363d0c0-bc53-49d8-a5fd-caef76629e78"> ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  useEventListener wasn't working properly when you passed in ref.current. For example, the `ArrowUp` & `ArrowDown` keys to move between cells. This is because changes to ref.current will not trigger a re-render. ## 🔍 Description of Changes  useEventListener lets you pass in a ref object instead of ref.current and ensure it re-renders correctly when ref.current is no longer null. Stumbled upon this as I was trying to add new eventListeners. Shoutout to Claude for helping me understand & write 😅. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

…2974) ## 📝 Summary  Fixes #2678 . Markdown code will be completely hidden to offer less distraction. This PR adds Double click / Enter as shortcuts to display markdown code, similar to Jupyter notebooks. https://github.com/user-attachments/assets/8fd832de-14cf-4835-a31a-87c6843bca50 <img width="806" alt="Screenshot 2024-12-01 at 12 47 36 AM" src="https://github.com/user-attachments/assets/42290d9f-e5b6-457d-9918-84026526a28b"> - Had to make some design compromises (widening the gap btwn left buttons when there is a Collapse toggle, rly hard to get this right with various markdown sizes). If you have another idea in mind, let me know. ## 🔍 Description of Changes  - attach a resizeObserver for markdown cells, when the cell is too short, the icons on the right will overlap. A resize observer is used to add a css class to make the icon inline - changed some props to be optional so that scratchpad will work fine - some refactors, including `temporarilyShowCode` function change from async to sync ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

@akshayka

## 📝 Summary  Follow up on #2507 . <img width="834" alt="Screenshot 2024-12-03 at 9 02 17 PM" src="https://github.com/user-attachments/assets/7f17749f-01aa-42ef-b2de-ee4c58c2bed5"> ## 🔍 Description of Changes  - There were also some UI bugs from before that are fixed here. (I didn't account for Cell Output below the cell 🙏) ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  There is a bad bug when you're in app view, since the borders are removed, so hovering over the cell will cause borders to be re-added. Borderless view is not needed in 'present' or 'view' mode. ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@mscolnick

## 📝 Summary Detects `GITHUB_REPO` and `REPO_ROOT` environment variables if set, falling back to default in marimo. (I believe this is the desired behavior of the script)  ## 🔍 Description of Changes Just detects `GITHUB_REPO` and `REPO_ROOT` environment variables if set.  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers @mscolnick

@akshayka

## 📝 Summary  Fixes #3073 . Each individual part can be scrolled (code editor, output area, console output) and the whole scratchpad can be scrolled too. I think it maintains the feel of scratchpad while handling long inputs / outputs. I'm not sure if it breaks anything else, so plan to test it locally a while. <img width="300" alt="Screenshot 2024-12-06 at 7 26 57 PM" src="https://github.com/user-attachments/assets/0c777a17-8eca-4070-8ce2-bb5d7bf181ef"> ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@mscolnick

## 📝 Summary  Add a minimal example of launching the marimo notebook and having the ability to run editable marimo notebooks on Modal. ## 🔍 Description of Changes  Update `README.md` in modal examples, and add the example file `modal_edit.py` defining the modal container. Note: Not sure what the naming convention is for examples, happy to rename! ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @mscolnick

@akshayka

## 📝 Summary  Fixes #3074. Adjusts z-index of grid-controls to be the same as app header like below: ```.tsx <AppHeader connection={connection} className={cn( "pt-4 sm:pt-12 pb-2 mb-4 print:hidden z-50", // Keep the header sticky when scrolling horizontally, for column mode "sticky left-0", )} > ``` ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

Wrap default filepath in backticks to avoid double underscore rendering as bold ## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [ ] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@mscolnick

## 📝 Summary This adds a mapping for the `quaternion` package, which is named `numpy-quaternion` on PyPI.  ## 🔍 Description of Changes Just added the key:value pair to the `module_name_to_pypi_name` dict. I'm hoping a test is not needed for such a simple change, though I have tried it locally and verified that it works by automatically installing `numpy-quaternion`. ## 📋 Checklist - [X] I have read the [contributor guidelines]([[https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md]]). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @mscolnick

@akshayka

## 📝 Summary  I can't test this locally, but it's a suspicion I have. Please close if incorrect. Related to [discord link](https://discord.com/channels/1059888774789730424/1059891311190229082/1319980533106737162) ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary This is a simple typo fix.  ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Fixes #2898. A v1 tracing panel for observability efforts https://github.com/user-attachments/assets/1c042e0b-2f70-4ec0-ad63-a0c59d5a89a4 ## 🔍 Description of Changes  - view cell runs, status, code snippet and timestamps - hover sync between react components and vega chart - supports dark theme - each run now has a run_id context from the backend - note: the timestamps here should be 'more accurate' than the frontend timers since they come from code execution in the backend. Hence, there can also be a small discrepancy btwn them. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [X] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: Myles Scolnick <[email protected]>

@akshayka

## 📝 Summary This is a simple typo fix.  ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Fixes #2898. A v1 tracing panel for observability efforts https://github.com/user-attachments/assets/1c042e0b-2f70-4ec0-ad63-a0c59d5a89a4 ## 🔍 Description of Changes  - view cell runs, status, code snippet and timestamps - hover sync between react components and vega chart - supports dark theme - each run now has a run_id context from the backend - note: the timestamps here should be 'more accurate' than the frontend timers since they come from code execution in the backend. Hence, there can also be a small discrepancy btwn them. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [X] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: Myles Scolnick <[email protected]>

@akshayka

## 📝 Summary  If you don't install pyarrow, Marimo will throw a `ModuleNotFoundError` and ask you to install pyarrow. Fix is to follow this guide: https://duckdb.org/docs/guides/python/polars.html Not sure if we can update the versioning of the packages. ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@mscolnick

## 📝 Summary  Adds an engineering formatter to the Dataframe Viewer ## 🔍 Description of Changes  I am enjoying the Marimo Dataframe viewer and in particular the ability to change the format of the numbers it shows. I am often working with physical units and as such prefer to work with numbers in engineering notation (including SI prefixes). This PR adds engineering notation as an option in the formatting window. ### Example Output: <img src="https://github.com/user-attachments/assets/acdf83b2-27e3-4bfb-8240-72b9f5f77dc4" width="200"> ### Selection: <img src="https://github.com/user-attachments/assets/0e185a2d-4086-4569-be25-c35c80d8816c" width="300"> ### Implementation: My goal was to keep the engineering notation formatting code as similar as possible to the scientific format code. I therefore used `Intl.NumberFormat` with some modifications to get prefixes for negative exponent cases. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @mscolnick

@akshayka

## 📝 Summary  Fixes #3103. Adds sqlglot as an optional dependency to handle complex sql refs parsing. Also supports DML statements (insert, update and delete). ## 🔍 Description of Changes  - set sqlglot version to lowest possible (likely >23.4, due to ibis lib) - If we're keen on supporting dml's, then I could add more tests (I added some here alrdy) - `def find_sql_def()` can also be modified to use sqlglot in the future, not included in this PR. - small change: updated duckdb version to 1.1.0 to be synchronised across all installations (some were 1.0 and some were 1.1) ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [x] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Fixes #3317 . Should display hidden cells similar to pre-borderless-markdown era. Code may be a bit rough. <img width="750" alt="Screenshot 2025-01-01 at 2 05 14 AM" src="https://github.com/user-attachments/assets/ac9bb7f5-3dfb-4617-bee2-b1bfa685f052" /> ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@mscolnick

## 📝 Summary  This PR makes it possible for multiple users to edit the same notebook in real-time. This is using [CRDTs](https://en.wikipedia.org/wiki/Conflict-free_replicated_data_type) and more precisely [Yjs](https://github.com/yjs/yjs) in the frontend and [pycrdt](https://github.com/jupyter-server/pycrdt) in the backend. Unlike the "kiosk mode" which only shows a read-only view of the outputs, the collaborative mode allows editing cell inputs. Users can edit the same cell at the same time: CRDTs ensure that all users eventually see the same text, potentially resolving conflicts. ## 🔍 Description of Changes  A new WebSocket endpoint is created at `ws/{cell_id}` for each notebook cell. When connecting to it, a cell input represented as a [shared text](https://docs.yjs.dev/api/shared-types/y.text) in the frontend is synchronized with [its peer](https://jupyter-server.github.io/pycrdt/api_reference/#pycrdt.Text) in the backend. The backend shared cell receives changes from every connected frontend, and broadcast them to every connected frontend. ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @mscolnick

@akshayka

## 📝 Summary  <img width="650" alt="Screenshot 2025-01-13 at 1 24 50 AM" src="https://github.com/user-attachments/assets/e6027e14-41a0-4d16-b289-af237684e948" /> Fixes #3401 . Use codemirror as chatbot input, `/` triggers autocomplete. ## 🔍 Description of Changes  - multiline input - multiline chat bubble - removed an unnecessary extension (codemirror-mentions) and implemented custom one note: dataset atom is not finding any tables, hence `@` is not working. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Fixes #3407 . This will raise an error for `data_editor`, `dataframe` and `table` when the `page_size` configured is > 200. <img width="450" alt="image" src="https://github.com/user-attachments/assets/8ec47f57-492b-45df-bff5-b5c2d481ebf1" /> ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

@akshayka

## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [x] I have added tests for the changes made. - [x] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Myles Scolnick <[email protected]>

@akshayka

) ## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

) ## 📝 Summary  Fixes #3540 . Allow to replace with empty strings + reset matches if replace is succesful. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Fixes #3090 . Previously, the `isEqual` check will always return true. onInvalidChange -> sets `internalValue` to `newValue` from the form onChange -> checks whether `newValue` and `internalValue` are the same. Yes, they would be. https://github.com/user-attachments/assets/936f648d-de7d-43c7-9895-6df070287205 There's some flickering going on (not new issue) when toggling from no cols to 1 col, hard to figure out.. ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  Adds `--watch` command to export `html-wasm`, and some docs on how to set up this experience. Ref: [discord thread](https://discord.com/channels/1059888774789730424/1231697382870093845/1332308364008357888) ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  I branched out to experiment. `mo.sql` can execute most SQL statements. Added some frontend. Todo: - [ ] no longer existing connections aren't being deleted - [ ] update selection input display, icons etc (see @/mscolnick's comment below) **[might not do this here]** - [ ] fix/add some tests - [ ] add some docs on how to use this - [ ] sql regex doesn't handle different ordering of engine & output params **[probs won't do it now too]** future work: - handle data source panel, sql parsing, so that dataset info is obtained from custom sql engines ## 🔍 Description of Changes  - by default, frontend will set duckdb as engine (this is hardcoded in frontend, not provided by backend) - added a new atom to store the latestEngineSelected, so it's sort of "smart" when you create new cells ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [X] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary Fixes No mention of CLA in contributing guide #3632  ## 🔍 Description of Changes  ## 📋 Checklist - [ ] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  When there is console output, the 'Clear Output' option is not shown. This is to fix that. <img width="600" alt="image" src="https://github.com/user-attachments/assets/3af06b17-4443-4c20-aeea-bd0426388f11" /> Context: Have a lot of print statements and the console output got really long, but I couldn't clear it. Another solution is to not hide this option at all. Or to merge the hasOutput & hasConsoleOutput prop into one. ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

…ted engine (#3682) ## 📝 Summary  <img width="500" alt="image" src="https://github.com/user-attachments/assets/29b156db-d56a-42c6-851f-e98025dd771f" /> Disconnected: <img width="500" alt="image" src="https://github.com/user-attachments/assets/ae580d6d-eabd-46ac-9e49-b370f17dad06" /> Todo: - I believe you were thinking of adding a select option, where it will open up a link to the docs? - Once this is good, then I should update the images in the docs too. ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  When you delete an engine and create a new SQL cell, the cell's code still uses the deleted engine. This fixes that. However, it's not an ideal fix, using the constant variable triggers a `Uncaught ReferenceError: can't access lexical declaration 'DEFAULT_ENGINE' before initialization` makes me think I have designed the latestEngineSelected feature wrongly. another smaller issue: - delete an engine, refresh the page, undelete the engine, the sql cells's dropdown do not update ## 🔍 Description of Changes  ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: Myles Scolnick <[email protected]>

@akshayka

## 📝 Summary  Borderless mode isn't applied in dark mode, this is to fix that <img width="793" alt="image" src="https://github.com/user-attachments/assets/2e0cedeb-9782-455d-9923-249999f3ece2" /> ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

Narwhals 1.26 ~~comes out later today~~ is out - this PR ensures the test suite keeps passing We made a couple of changes which affect the test suite but which we don't expect to affect users ~~This fails now (as expected) but that will change once the nw release is published~~ ## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [ ] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

@akshayka

## 📝 Summary  fixes [discord: mo.stop disappearing upon restart](https://discord.com/channels/1059888774789730424/1254862968068243476/threads/1338534110586605639) Will not allow non-sql code to transform to SQL cell. This means the following is not transformable too. ```.py table_name = x _df = mo.sql("SELECT * FROM {table_name}") ``` ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  ## 🔍 Description of Changes  ## 📋 Checklist - [ ] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [ ] I have added tests for the changes made. - [ ] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  https://github.com/user-attachments/assets/1da6aef9-6f69-4886-a648-398e8cd4e518 --- **TODOs:** - [x] search tables function is not working correctly, silent previewColumn firing may not work - [X] add tests, fix typing - [ ] stress test with large db - [X] test with different db's. The worst thing to do is it hangs when there's an error in the bg. It should fail and return no - [X] snowflake - [X] bigquery - [x] mysql - [X] postgres - [X] sqlite - [x] mssql - [ ] test with `session.bind` flow - [ ] multi-db / not specifying db. Idt sqlalchemy supports db discovery, so 1 way is to execute raw sql queries depending on dialect (i.e `show databases;`) ## 🔍 Description of Changes  Experimenting a little, I find an optimal request solution is something like below. Make an initial req for db, schemas and table list, upon expanding a table, make another request to get column info, PKs, indexes. This avoids too long of an initial request time while fetching sufficient info. With snowflake sample db (7 schemas, 10+ tables each), initial request takes about 3s. If I fetched all data, this would take ~25s <img width="450" alt="image" src="https://github.com/user-attachments/assets/f15e5dd4-e72a-4f56-8dc4-340ad01cf4e0" /> ## 📋 Checklist - [x] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [x] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

@akshayka

## 📝 Summary  For postgres, it will connect to the default database if unspecified. This fixes the display for that (by finding the default db). For others like Snowflake, it's in a detached state, hence we show 'Not connected' <img width="444" alt="image" src="https://github.com/user-attachments/assets/6bdfbc01-4b88-46db-9449-9d4bcc47d30e" /> ## 🔍 Description of Changes  ## 📋 Checklist - [X] I have read the [contributor guidelines](https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md). - [ ] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on [Discord](https://marimo.io/discord?ref=pr), or the community [discussions](https://github.com/marimo-team/marimo/discussions) (Please provide a link if applicable). - [X] I have added tests for the changes made. - [X] I have run the code and verified that it works as expected. ## 📜 Reviewers  @akshayka OR @mscolnick

v0.1.9

8bb04b3

akshayka merged commit af8a058 into main Sep 19, 2023

akshayka deleted the aka/v0.1.9 branch September 19, 2023 02:36

github-actions bot locked and limited conversation to collaborators Sep 19, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

v0.1.9 #123

v0.1.9 #123

akshayka commented Sep 19, 2023

vercel bot commented Sep 19, 2023

v0.1.9 #123

v0.1.9 #123

Conversation

akshayka commented Sep 19, 2023

vercel bot commented Sep 19, 2023