Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not enough cat memes =( #21

Open
laboon opened this issue May 23, 2017 · 4 comments
Open

Not enough cat memes =( #21

laboon opened this issue May 23, 2017 · 4 comments

Comments

@laboon
Copy link
Owner

laboon commented May 23, 2017

We need more cat-related memes

@DomThePorcupine
Copy link
Contributor

Perhaps we could use the imgur API. In fact their search example is for cats!

curl --request GET \ --url 'https://api.imgur.com/3/gallery/search/{{sort}}/{{window}}/{{page}}?q=cats' \ --header 'authorization: Client-ID {{clientId}}'

@BLubez
Copy link

BLubez commented May 23, 2017

While inefficient, using the students as cheap labor to find and post cat-related memes into some shared folder on the repository will work well with minimal setup.

18581581_10213737700855983_4237797224300986174_n

@DomThePorcupine
Copy link
Contributor

DomThePorcupine commented May 23, 2017

@BLubez I think that is an O(N2) solution, as students will procrastinate until the last minute. Perhaps we could maximize efficiency by making sure all of our memes are actually GIFs. This could be solved in O(log(N)) time with the GIPHY open gif search API.

@BLubez
Copy link

BLubez commented May 23, 2017

@DomThePorcupine I think we should also integrate the Tenor and PopKey APIs so that we could have O(3log(N)) which while not asymptotically better does provide noticeable improvements in speed for real world situations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants