arxiv-sanity-preserver
arxiv-sanity-preserver copied to clipboard
karpathy
make life easier
+1 for the possibility of a functionality update/sanity pass over the code cleaning it up a bit.
I have not really technically reviewed this code for technical passing, but only a quick skim to check the high-level temperature of the changes. The get_time function in line 40 of the proposed update within get_papers.py though is a little worrying for me in terms of being a canary for the types of changes being made to the codebase in a wholesale kind of fashion.
That said, the feel that I get for the rest of the code changes is positive, I get the feel that it fills in a whole lot of just glaring feature gaps within the original codebase, and comparatively a lot of the code does really seem to simplify the original codebase in several areas, so this is a net positive.
In any case, I think, my +1 for this kind of effort, thanks for sharing this (not an admin or mod, just someone with a vested interest in ASV's upkeep :D).
+1 for the possibility of a functionality update/sanity pass over the code cleaning it up a bit.
I have not really technically reviewed this code for technical passing, but only a quick skim to check the high-level temperature of the changes. The get_time function in line 40 of the proposed update within get_papers.py though is a little worrying for me in terms of being a canary for the types of changes being made to the codebase in a wholesale kind of fashion.
That said, the feel that I get for the rest of the code changes is positive, I get the feel that it fills in a whole lot of just glaring feature gaps within the original codebase, and comparatively a lot of the code does really seem to simplify the original codebase in several areas, so this is a net positive.
In any case, I think, my +1 for this kind of effort, thanks for sharing this (not an admin or mod, just someone with a vested interest in ASV's upkeep :D).
Thanks for your approval，I really appreciate it :)