Should plotly export require data be a vector?

[benkamphaus]

First, thanks for making Clay! I'm using it a ton and hope to be at a point where I can start contributing more to the community or project soon!

I recently started running into some plotly export issues rendering pdf via the quarto path, where I've gotten failures on this line due to not having a vector / associative data structure:

clay/src/scicloj/clay/v2/plotly_export.clj

Line 25 in 1a55837

(let [data (update data 0 dissoc :r :fill :mode :theta :z :lon :lat :width)

i.e. (update data 0 ... ) expects to act on a vector, versus e.g. nth based de-structuring or some sequence oriented way to apply the dissoc to the first element only.

Other paths seem happy to render through with seq-able or nth-able. I've wrapped plotting routines in my own code now where they proactively vec or mapv etc. to get ahead of this, so have a workaround in the meantime. But wanted to surface as I wasn't sure if this was intended or happenstance.

[benkamphaus]

Actually, there are some other unexpected impacts here. Not sure if I should expand this issue re: clay -> quarto -> pdf rendering in general? I'm also happy to take a stab at a PR.

The :marker dissoc removes the first data element's plot directive re: marker only mode and adds lines in the image render.

The build also doesn't render the plot images unless I re-run the quarto CLI from the directory containing the images (the relative pathing in the rendered qmd, it seems like, is from the quarto CLI call directory as opposed to the qmd?)

[daslu]

Hi @benkamphaus, thanks for this issue. It is very helpful, as the PDF support for plotly is still experimental in Clay, and we need to figure out what may go wrong.

While I hope to explore these issues soon, I'll just mention that, for now, I often find it useful to simply Print to PDF from the browser. Since it is possible to include page breaks in HTML, it is often convenient to create HTML pages that print reasonably out of the box.

[daslu]

Hi @benkamphaus, can I help debug this?

Maybe we can look into a code example together?

In any case:

• Yes, kind/plotly expects :data to be a vector: https://scicloj.github.io/clay/clay_book.examples.html#plotly
• The update ... dissoc ... part quoted above was removed in version "2.0.10" -- it was intended to handle specific problems with Plotly APIs generated by Tableplot, and those problems have been fixed.