Toby Dylan Hocking
Toby Dylan Hocking
No obvious timing issues in HEAD=add-download-size-info [](https://asset.cml.dev/2668a37fd1d92394cef9bcce204d3a05fe41eaa1?cml=png\&cache-bypass=9433af1d-29b7-4f96-9fd7-d2a9f4f53b57) Generated via commit 69bc4c74942ca94c0d7053d57669151ded098ad0 Download link for the artifact containing the test results: [↓ atime-results.zip](https://github.com/animint/animint2/actions/runs/20254967335/artifacts/4880489105) | Task | Duration | |---|---| |...
Thanks! This is a good start. I see the result below using https://github.com/animint/animint2/blob/master/inst/examples/WorldBank-paper.R Can you please make the following changes? * remove selected chunk and status columns * right justify...
please click Resolve so I can see which previous comments you have already addressed
can you please post a screenshot like this one https://github.com/animint/animint2/pull/272#issuecomment-3473335127 but created with the new code?
https://github.com/animint/animint2/pull/272#issuecomment-3538898963 seems to have the wrong old screenshot, please revise
that new screenshot looks good, thanks! please fix test failures.
this needs to be fixed ``` ── Failure ('test-renderer3-knit-print.R:67:3'): id property is unique over entire page ── Expected `length(not.unique)` to equal 0. Differences: 1/1 mismatches [1] 1 - 0 ==...
using this branch I see * large number of rows would benefit from having commas, like 4,321 instead of 4321 -- can you please fix? * sometimes 0.00 is displayed...
the updated screenshot looks better, thanks! it would be easier to read if there were a grey or black line to separate each row/column in the table. Can you add...
also can you please post updated screenshot?