Skip to content

Practical 1 on segmentation added. #29

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

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

Conversation

annnyway
Copy link

@annnyway annnyway commented Nov 3, 2018

Hello, Fran! Here is my report on Practical 1. I'm so sorry to be late. It was very new and very difficult for me, but I will work on it more. I hope you will check my report. I will be waiting for your comments.

Thank you in advance.
Ann Nikolaeva

@ftyers
Copy link
Owner

ftyers commented Nov 4, 2018

Comment: For pragmatic_segmenter did you use the English rules ? I don't think there are Ukrainian rules, but perhaps it might work better with the Russian rules than the English rules?

@annnyway
Copy link
Author

annnyway commented Nov 4, 2018

I followed the given instructions on the practical: https://ftyers.github.io/2018-komp-ling/practicals/segmentation.html and didn't know that I could change the rules, but now I see that maybe I could change the language in segmenter.rb file to Russian. Will it change the English rules to the Russian ones? I will try it today or tomorrow.

Thank you!

Added the result of Pragmatic Segmenter after changing its language into Russian
@annnyway
Copy link
Author

I have added more results of Pragmatic Segmenter. Do I need to make a new pull-request?

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

Successfully merging this pull request may close these issues.

2 participants