search

scicloj / tablecloth

Dataset manipulation library built on the top of tech.ml.dataset
https://scicloj.github.io/tablecloth
MIT License
305 stars 27 forks source link

tablecloth.api ns is not very Cursive-friendly #58

Closed simongray closed 2 years ago

simongray commented 2 years ago

Seems like a really interesting library, but...

The primary namespace makes use of tech.v3.datatype.export-symbols/export-symbols to expose functions in the tablecloth.api namespace. This is also the only namespace referenced in the usage example in the README.

Nearly single symbol in this namespace cannot be resolved using static analysis, resulting in improper syntax highlighting, no docstrings and no arglists displayed in my editor's UI. This applies to Cursive (which is one of the most popular Clojure editors), but I'm sure it basically also applies in some way to every single editor which doesn't resolve symbols dynamically (like is done with cider).

May I suggest a slightly less "magic" way of defining these symbols...? This would create a much better experience for everyone not using emacs.

genmeblog commented 2 years ago

Thanks! There is a solution for that but needs to be implemented (or maybe just an upgrade is necessary - need to go through it more carefuly). More discussion here: https://clojurians.zulipchat.com/#narrow/stream/151924-data-science/topic/tech.2Eml.2Edataset/near/264019964

simongray commented 2 years ago

Great to hear that.

genmeblog commented 2 years ago

it should be fixed now (6.051)