Skip to content

Commit

Permalink
Done
Browse files Browse the repository at this point in the history
  • Loading branch information
krlmlr committed Mar 7, 2021
1 parent a2cb096 commit da81c50
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
7 changes: 4 additions & 3 deletions TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@

## Next steps

- Benchmark and profile again
- `bench.R`
- https://github.com/tidyverse/tibble/issues/598
- Look for FIXMEs
- Test coverage per file
- Use penguins
Expand All @@ -21,6 +18,10 @@
- Simplify matrix formatting to format like an array: https://github.com/r-lib/pillar/issues/142#issuecomment-489357664
- `tibble::tibble(a = structure(1000:1003, class = "foo"))` should right-align, because the underlying data is a number?
- Blog post?
- Format and truncate in {utf8}
- Benchmark and profile again
- `bench.R`
- https://github.com/tidyverse/tibble/issues/598


## Formatting numbers
Expand Down
8 changes: 4 additions & 4 deletions bench.R
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ df_all <- pillar:::new_tbl(list(
i = list(list(1, 2:3), list(4:6), list(NA))
))

# proffer::pprof(filter.callframes = TRUE,
# for (i in 1:100) format(df_all)
# )
proffer::pprof(filter.callframes = TRUE,
for (i in 1:200) format(df_all)
)

bench::mark(format(df_all), iterations = 500)
# bench::mark(format(df_all), iterations = 500)

0 comments on commit da81c50

Please sign in to comment.