search

DAGWorks-Inc / burr

Build applications that make decisions (chatbots, agents, simulations, etc...). Monitor, persist, and execute on your own infrastructure.
https://burr.dagworks.io
BSD 3-Clause Clear License
561 stars 29 forks source link

Adds JSON view for numeric list -- this is easier to deal with #154

Closed elijahbenizzy closed 1 month ago

elijahbenizzy commented 1 month ago

We can just reuse the JSON view for this, its pretty clean.

image
skrawcz commented 1 month ago

will this work for top level number arrays too? I remember having code in two spots for this reason (IIRC).

elijahbenizzy commented 1 month ago

will this work for top level number arrays too? I remember having code in two spots for this reason (IIRC).

From what I can tell that's actually not necessary. FormRenderer renders the data, but only the top level. That's always a dict/object, which means we just have to get the fields. The RenderedField is used recursively.

skrawcz commented 1 month ago

Cool -- but I see you've injected a values field in there though.

skrawcz commented 1 month ago

Screen Shot 2024-04-12 at 4 42 49 PM fixed

skrawcz commented 1 month ago

note: rendering is a little sluggish with this.