time-literals add transit encodings

Aug 15 '19 09:08 henryw374

Here's a sketch which should work. Sorry don't have time to wrap up into PR.

Aug 21 '19 03:08 jdf-id-au

@jdf-id-au your sketch has been working good for me. I have forked your gist to make a couple small changes and add a deps.edn, so that it can be used in a project via the following git dep:

  transit-connection {:git/url "https://gist.github.com/jjttjj/6bc0b62ef1dbf29c1c69ea22f8eb7f55"
                      :sha "2345014effd1ce7089d2b847808d4bbcf2c38a3b"}

I could put together a PR for this if desired.

Jan 14 '20 20:01 jjttjj

Cool, I would certainly accept a PR, but clearly this works as is, so no great hurry

Jan 15 '20 07:01 henryw374

I released a separate library to do this. https://github.com/Motiva-AI/pipeline-transit/blob/master/src/pipeline_transit/core.clj

I'm not sure if this feature should go into this library though. Not everyone needs to use transit with time-literals and adding it would require roping in the transit-clj and cljs libraries for this use case. Thus bloating this library.

May 07 '21 20:05 Quantisan

I released a separate library to do this. https://github.com/Motiva-AI/pipeline-transit/blob/master/src/pipeline_transit/core.clj

I'm not sure if this feature should go into this library though. Not everyone needs to use transit with time-literals and adding it would require roping in the transit-clj and cljs libraries for this use case. Thus bloating this library.

Thanks. I agree wrt dependencies.

I think what I'll do is link from the README and close this issue.

I notice the file you link to is just clj, rather than cljc as the gist further up was.

May 17 '21 06:05 henryw374

Here is an implementation that is working for myself (it works with cljc): https://github.com/pink-gorilla/modular/blob/main/src/modular/encoding/time2.cljc

Sep 05 '23 00:09 awb99