This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2023-11-21
Channels
- # announcements (1)
- # beginners (20)
- # biff (5)
- # calva (43)
- # cider (5)
- # clj-commons (7)
- # clj-kondo (11)
- # clojure (58)
- # clojure-brasil (1)
- # clojure-denmark (1)
- # clojure-europe (27)
- # clojure-nl (1)
- # clojure-norway (13)
- # clojure-uk (2)
- # clojurescript (71)
- # data-science (32)
- # datalevin (6)
- # datomic (19)
- # emacs (1)
- # gratitude (3)
- # honeysql (8)
- # hoplon (15)
- # hyperfiddle (3)
- # introduce-yourself (1)
- # lsp (19)
- # malli (4)
- # nbb (7)
- # other-lisps (5)
- # practicalli (1)
- # re-frame (14)
- # releases (1)
- # ring-swagger (1)
- # squint (118)
- # xtdb (9)
- # yada (2)
Hey guys, could anyone take a look? https://github.com/clj-commons/potemkin/pull/75
Looks like a reasonable change to me. It seems odd to pull in a tuple library just for that one usage. In the memoize code I maintain, I just use the sequence of args as-is -- I wouldn't even bother unrolling them into a vector -- as I recall. But maybe that's the key to it being a "fast" memoize instead of the regular memoize
?
My guess (without doing the benchmarks) is that the most "fast" comes from using a mutable map instead of PHM.
@U04V5VAUN, do you want to cut the new release, since you merged? It looks like you and I are the only ones with potemkin deploy perms on clojars. I'll ask Zach if he can designate others as admins.