Shawn Presser
Shawn Presser
Nah, your comments were spot-on. I just haven't had time to address them yet. I added you as a collaborator to my fork of anarki; feel free to fix up...
@rocketnia By the way, if you want to chat in realtime: https://discord.gg/qaqkc9z Are you in any slack channels or IRC?
I wanted to get another set of eyes on this before merging. `strftime` isn't meant to be a complete implementation, but it's pretty close.
> I assume you're running them in laarc? Yup! Previously I was shelling out to `date` using `system`, but that's very slow. > Please remember to backport bugfixes as you...
Yeah, I think it's just unbound variable support. As for the implementation, I lifted it from laarc. I didn't know arc already had an `or=`. But I know this one...
> This change causes `or=` to evaluate those expressions more than once. Interesting. Could you give an example?
> `(bound ',slot)` checks whether something is bound in the global scope, not the local scope Good point! This should get resolved with the env changes from #151
There is no legal risk linking to the dataset. All risk is being taken on by The Eye. The sole reason not to merge it is because someone doesn't like...
(In other words, don't host the data yourself. Rely on the URL from The Eye. So, for example, all dataset preparation scripts should download from https://the-eye.eu/public/AI/pile_preliminary_components/books1.tar.gz, and books1.tar.gz itself should...
The datasets are hosted by The Eye, which fully respects DMCA: http://the-eye.eu/dmca If anyone were to file a DMCA notice against books1 or books3, they would extract the tarball, remove...