search

karpathy / arxiv-sanity-preserver

Web interface for browsing, search and filtering recent arxiv submissions
http://www.arxiv-sanity.com/
MIT License
5.14k stars 1.33k forks source link

make life easier #171

Open profection opened 3 years ago

profection commented 3 years ago

make code more readable and faster

kimbayashanologatiniburgettoilalalourvr commented 3 years ago

+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).

profection commented 3 years ago

+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 :)