Fork me on GitHub
#nextjournal
<
2021-10-18
>
sparkofreason16:10:21

I'm expanding an array of maps, can successfully open the array, then an item. Item is rendered with several keys, followed by 10 more.... Clicking on that gives Rendering error. Is this a known issue?

mkvlr21:10:09

We've got an unreleased fix related to lazy loading but would be great if you had a small repro so we can make sure it's that. Thanks!

mkvlr21:10:12

but I'm pretty sure it's that, should just cut a release…

mkvlr21:10:18

can you try https://clojars.org/io.github.nextjournal/clerk/versions/0.1.179 let me know if it’s fixed there? Thank you!

sparkofreason15:10:58

Yes, thank you! This may have been mentioned before: after a map has been "pretty printed" vertically by clicking {, hitting the ... button flips it back to horizontal. Clicking { again puts it back to vertical layout with the extra fields.

mkvlr15:10:47

ah right; one down and one :ladybug: added to the list

1
sparkofreason15:10:11

I also see a lot of these:

#object[TypeError TypeError: Failed to fetch]
Is this an issue with my notebook code?

mkvlr15:10:51

An issue with Clerk's code, not yours. Could you provide a repro?

sparkofreason15:10:15

Let me see if I can figure one out that isn't all tangled up in SQL queries.

mkvlr16:10:24

what SQL lib are you using?

sparkofreason16:10:21

I have a hunch this and some other issues I was seeing might have been related to cache persisting across different versions of clerk. I did a clerk/clear-cache!, so far things are looking much better.

mkvlr16:10:22

ah good, thinking if we can make that unnecessary

sparkofreason16:10:30

I think the root cause was that some value for a Java object had gotten cached with the nippy unthawable map. Then it would throw in some situations that it couldn't find the Java method on PersistentHashMap. The clue was that when I changed the constructor parameters to that class, I couldn't repro anymore, so I'm assuming it was cache.

sparkofreason17:10:12

And great work on this. clerk is exactly the tool I've been wanting forever, never had the time to really tackle it myself. Thank you!

❤️ 1
mkvlr17:10:47

Aww, that's awesome, thank you!

mkvlr21:10:49

just released https://clojars.org/io.github.nextjournal/clerk/versions/0.1.179 with three small fixes (https://nextjournal.com/mk/clerk-preview/edit#0.1.179): • Fix lazy loading for non-root elements • Fix exception when lazy loading end of string • Fix regression in `clerk/clear-cache!`

❤️ 6
👍 3