still waiting on a review for https://github.com/Eventual-Inc/Daft/pull/2804

Dropped a review — there’s some conflicts with @Sammy Sidhu ‘s refactors which we need to work through I think 2 main areas that I noticed: some things around the datatypes utilities (ImageMode etc) and then some more around display logic (the html repr logic)

@Sammy Sidhu @jay this should be ready for another review. i understand the concern about the

html_value

but unfortunately, I don't think there's a good way around it currently. We'd have to move the entire

ImageArray

out of daft-core, which isn't really possible with our current architecture.

Should be good to merge!