Persist App state + Cell state in md conversions #1326
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Oh woops. I didn't realize |
|
|
||
| # Native to python | ||
| from xml.etree.ElementTree import Element, SubElement | ||
|
|
||
| # Note: yaml is also a python builtin | ||
| import yaml |
There was a problem hiding this comment.
i might have lazy imported this, just fyi, if you are not up-to-date with main
There was a problem hiding this comment.
I think in exporter, not here
| ```{.python.marimo unparsable="true"} | ||
| it's an unparsable cell | ||
| ``` | ||
|
|
||
| <!-- Actually markdown --> | ||
| ```{python} ` |
There was a problem hiding this comment.
is this also syntax i wasn't aware of? like an one-line code-fence?
There was a problem hiding this comment.
No this is intentionally broken to make sure match works. The newest unit tests turned into more of smoke tests, so yeah- I think more granular tests would be awesome
There was a problem hiding this comment.
oh gotcha, ha makes more sense
| @@ -42,35 +43,60 @@ def _is_code_tag(text: str) -> bool: | |||
| return bool(re.search(r"\{.*python.*\}", head)) | |||
|
|
|||
|
|
|||
| def formatted_code_block(code: str) -> str: | |||
| def formatted_code_block( | |||
There was a problem hiding this comment.
sometime this week - i may write some small unit tests for these files (as my way to learn this part of the code more)
|
i can force merge past playwright - its been very flakey lately and i need to investigate |
The python 3.8 error is valid, but I won't be able to get to it for another hour or so |
|
@dmadisetti there is a snapshot fail on |
|
Oh woops. Weird, that's what I run |
|
Got it. Probably a bit of randomness. I think tempfile name gets quoted if it contains not path safe characters. Hopefully this won't be flaky- think I caught it locally. If it is I think it makes sens to just remove the line |
|
thanks @dmadisetti for more great work! |
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.4.12-dev15 |
I got
marimo edit document.mdworking locally and realized a few things were missing:<!---->The actual code to get
marimo edit document.mdworking is a bit of a hack. Once this is in, I'll open the PR for reference, but I think that it will require a deeper rework from someone more versed in the internals