keon / awesome-nlp

:book: A curated list of resources dedicated to Natural Language Processing (NLP)

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add awesome-nlp to awesome.re?

NirantK opened this issue · comments

Awesome.re is a meta-list of awesome repositories.

It has a few guidelines to be included there.
Maybe we can use that as our checklist to improve this. And then raise a PR when we are done?

What do you suggest @keon ?

commented

sounds good! In fact, I was going to do that a long time ago (sindresorhus/awesome#677) but got too busy for a while.

also, for long term...
I wanted to work on a script that allows us to store the lists in some sort of json or list that could automatically be used for parsing, verifying the url, and generating the README.md.

The script sounds like a good idea:

  • make our jobs easier as maintainers
  • does it make it more difficult to contribute to README.md then write now?
    Keeping in mind that most contributors are contributing directly from their browsers.

Maybe we should add that script to a Pull Request hook? Include formatting checks in the script itself - similar to Bandit for code?

@keon just following up, do you have any more inputs on what we want from the json2md (json to markdown) script?

@the-ethan-hunt would you like to take lead on this and guide us on what we need to get done to add awesome-nlp to awesome.re?

@NirantK , I find that awesome-nlp has all the required stuff to be included in the awesome.re list. Should we pull a request? Or maybe @keon should(He is the original maintainer of the project 😄 )?

Let's not wait for @keon. Please raise a PR there. Please double check that we are visible in a separate section.

Preferably not absorbed under wider and less maintained Speech and Natural Language Processing sections ;)

I'd like for people to find us as easily as possible.

Includes a project logo/illustration whenever possible.

We might be lagging here. any suggestions on how to proceed on this? 😅

Hmm, I had looked at a few examples back in January. We'll probably start with a cover image as in #94 and then just ask the awesome.re maintainer for advice.

Adding section specific images would ruin the repo organization and make it longer than it already is.

How about pulling a request here about the cover image ideas you had and then we can decide the appropriate image?

Good idea, I will raise a PR for it.

Since this is going to take some time from my end, anything else other than that we need to do?

Nope,not at all
Secondly, @keon raised this issue here
The maintainer has asked about the status of our list(a year ago). Is it wise to reply back now with an apology? 😄

Thanks for bringing that to notice. I somehow missed that completely.

Sure, please go ahead and tell @sindresorhus that we are helping keon maintain this.

And we would like to include the list as we've updated (and expanded) it significantly over last 4-6 months.

Done! All we need to do is wait for his comment while we plan out a cover image.

@the-ethan-hunt please follow up?

@NirantK , @keon created a new issue here, see. We are still waiting for a response from the maintainers of the awesome.re project

@NirantK , there has been no fresh leads on the issue. Any ideas to proceed further?

No, have stopped pursuing this. There has been no response on the old and new issues by the repo maintainer there.

We can try to raise a clean PR to that repo as per the new issue and see if that get's accepted.

Else, close this issue please

Sure. But before submitting another PR, we should maybe have a cover image first. Closing this issue nonetheless.