Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Conversation

@mk2510
Copy link
Collaborator

@mk2510 mk2510 commented Jul 22, 2020

  • corrected some spelling mistakes

  • added tokenize to the demo code snippets

  • explanation of which functions take tokenized input

It will still remain as a draft after everything else in #107 is done 🚀

* corrected some spelling mistakes

* added tokenize to the demo code snippeds

*explanation of which functions take tokenize input

Co-authored-by: Henri Froese <[email protected],>
@jbesomi
Copy link
Owner

jbesomi commented Jul 24, 2020

Hey! Top!

For the first two examples, I'm not sure we want to keep the >>>. That way, a user can just copy-paste the code in a jupyter notebook and see the result, without the need to replacing >>>. At least that was my initial consideration, ideas?

For the example 3 (Simple pipeline for text cleaning) it makes sense to keep the >>> as here we show a CLI interaction

@mk2510
Copy link
Collaborator Author

mk2510 commented Jul 24, 2020

ok 🧀 , I will change it back. Just thought it will give it a more uniform look 🐤

cheers :)

@jbesomi
Copy link
Owner

jbesomi commented Jul 24, 2020

Actually, you were right!! thx

@mk2510 mk2510 marked this pull request as ready for review July 24, 2020 14:12
@jbesomi
Copy link
Owner

jbesomi commented Jul 27, 2020

Hi, there are still the >>> ...

removed the >>> where no output was shown



Co-authored-by: Henri Froese <[email protected]>
@mk2510
Copy link
Collaborator Author

mk2510 commented Jul 27, 2020

oh, sorry, I got you wrong there 🙊 I thought with:

Actually, you were right!! thx

you meant, that's alright that way.
I removed it in the first two examples but kept it elsewhere, to better distinguish the code from the output

@mk2510 mk2510 changed the title Update README.md Reviewed and waiting for Representation Implementation: Update README.md Jul 27, 2020
@jbesomi jbesomi marked this pull request as draft July 27, 2020 17:12
@henrifroese henrifroese added the documentation Improvements or additions to documentation label Aug 7, 2020
@mk2510
Copy link
Collaborator Author

mk2510 commented Aug 8, 2020

@jbesomi also waiting here for the PR #135 to be merged, so the link for the Series types tutorial can be included 🏎️

@henrifroese henrifroese changed the title Reviewed and waiting for Representation Implementation: Update README.md Update README.md Aug 22, 2020
@mk2510 mk2510 marked this pull request as ready for review September 5, 2020 16:16
Now, the next main milestone is to provide *multilingual support* and for this big step, we need the help of all of you. ¿Hablas español? Sprechen Sie Deutsch? 你会说中文 日本語が話せるのか?Fala português? Parli Italiano? Вы говорите по-русски? If yes or you speak another language not mentioned here, then you might help us develop multilingual support! Even if you haven't contributed before or you just started with NLP, contact us or open a Github issue, there is always a first time :) We promise you will learn a lot, and, ... who knows? It might help you find your new job as an NLP-developer!

For improving the python toolkit and provide an even better experience, your aid and feedback are crucial. If you have any problem or suggestion please open a Github [issue](https://github.com/jbesomi/texthero/issues), we will be glad to support you and help you.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the "Beta version" part still make sense (@jbesomi )?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants