Warning from TableRex dependency

[billylanchantin]

Explorer.DataFrame.print/2 uses the TableRex dependency. We're currently getting a warning from it:

iex> df = Explorer.Datasets.iris()
iex> Explorer.DataFrame.print(df)
warning: using map.field notation (without parentheses) to invoke function TableRex.Renderer.Text.default_options() is deprecated, you must add parentheses instead: remote.function()
  (table_rex 4.0.0) lib/table_rex/table.ex:267: TableRex.Table.render/2
  (table_rex 4.0.0) lib/table_rex/table.ex:280: TableRex.Table.render!/2
  (explorer 0.11.0-dev) lib/explorer/data_frame.ex:6036: Explorer.DataFrame.print/2
  (elixir 1.18.1) src/elixir.erl:386: :elixir.eval_external_handler/3
  (stdlib 6.1.2) erl_eval.erl:904: :erl_eval.do_apply/7

This warning is internal to TableRex. There's an issue about it with a fix:

• New release djm/table_rex#60

But I get the sense that this project isn't maintained all that actively.

I see three options to address:

1. Mitigate on our end by wrapping the call or something.
2. Go with a different table displaying library. Scribe comes to mind.
3. Write our own table displaying function.

Those are ordered by increasing difficulty. (3) in particular is a lot of work.

[philss]

Hi @billylanchantin!

If you decide to go with option 2, I have a new library that I've been working on for the past weeks: it's called TableKitty 🐈 .

I made it as an alternative to TableRex, but with a very different and compact API, and with an integration with Table. It's not documented yet, but I have examples in the tests cases.

I just released version 0.1.0 and I plan to continue polishing the options and documentation.

There is one example of how we would print an Explorer DF in the tests, but you could just do TableKitty.print(df) and it would print a basic version of the dataframe.

That said, it's totally fine if you decide to follow a different approach, or use another lib :)

[billylanchantin]

Hey @philss,

Very cool! I've been thinking that a project exactly like that is needed for interop with Table.

I'm not sure we want to go with option 2 yet. But your lib has definitely jumped the queue.