Skip to content

Conversation

dedifferentiator
Copy link
Contributor

Hey! I added support of Russian and Haskell as alternative languages.

There are not really many quotes, because I didn't manage to find a decent dataset with russian quotes and that's annoying :c , and I kinda don't want to write a parser for that purpose yet (it's a bit difficult to filter good quotes), so I manually added some of them, gonna spend 30 min per day to add new ones for a month or so, I think having around ~500 of them would be nice for the start. Although I may give up and write a parser in the end...

@dedifferentiator
Copy link
Contributor Author

oops...CI seems to be broken

Copy link
Owner

@rmehri01 rmehri01 left a comment

Choose a reason for hiding this comment

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

Hi! Wow this is a lot of work, so thanks again! I tried it locally and it seems to work good, have you tried the online locally? I haven't quite got around to it yet because I've been busy with school and other work but I'll try to get to it soon.

As for the other quotes, do you know how well something like google translate would work because it would be pretty easy to write a script for it but I don't know how good the quotes will translate.

Also what do you mean by writing a parser for it? Maybe if it is a better way to do it I could try to help out!

Comment on lines +127 to 128
-- FIXME :)
_ -> undefined
Copy link
Owner

Choose a reason for hiding this comment

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

This is supposed to be undefined since the other cases aren't supposed to happen. Do you mean there should be a better way to represent it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree, just don't feel like it's a good thing to leave undefined somewhere in code. Maybe replace it with `error "Error: unexpected game state"?

Copy link
Owner

Choose a reason for hiding this comment

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

Ah okay fair enough, that makes sense!

@rmehri01
Copy link
Owner

oops...CI seems to be broken

Hmm it seems like the error is the same as this one. Maybe I will have to find another way to do linting or remove it entirely.

@dedifferentiator
Copy link
Contributor Author

have you tried the online locally?

Yes, it seems to work, but you'd better check it out once more :)

I don't know how good the quotes will translate.

Pretty awfully, unfortunately, it would be around 60% +- ok quality with one-sentence quotes, but with number of sentences the quality drops significantly due to slang, names, etc, I don't think that translating the quotes is an option.

Also what do you mean by writing a parser for it?

I mean to write a web crawler, which would go through any livelib stuff or citaty.info and collect quotes from them. Generally the problem is how to sort the good quotes from the err... cringy ones. Maybe the good strategy would be to gather links with good books in one place, run parser on them, replace unprintable on default layout symbols like ( to - ) and then create a script that would show a quote and you just have to press y/n, add a quote to json or skip it. Still not perfect, but already more feasible, than doing it manually.

What about haskell source code quotes, I'm not so sure, cause it's almost no difference what to type to me, and as I see it, it's to have 2-3 quotes from different projects repos, and the number of these quotes in the dataset doesn't matter much. What are your thoughts about that?

@rmehri01
Copy link
Owner

Maybe the good strategy would be to gather links with good books in one place, run parser on them, replace unprintable on default layout symbols like (— to - ) and then create a script that would show a quote and you just have to press y/n, add a quote to json or skip it.

Oh okay, I see what you mean. That seems like a pretty good way to do it, especially for many different quotes!

What about haskell source code quotes, I'm not so sure, cause it's almost no difference what to type to me, and as I see it, it's to have 2-3 quotes from different projects repos, and the number of these quotes in the dataset doesn't matter much. What are your thoughts about that?

Yeah, honestly it doesn't make much of a difference to me either so I think that it's good enough to just have a few like you said.

Copy link
Owner

@rmehri01 rmehri01 left a comment

Choose a reason for hiding this comment

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

Hi, just a few more things I ran into while testing the server!

When you try to enter a name or room with a 'q' in it, it just quits instead of entering it as text. I think that it would be easier to just reserve 'ESC' for quitting instead of having multiple bindings for it.

Also, when the client first joins the game, I don't think that the quote set is properly updated for them:

image

Other than that, the online functionality looks pretty good to me! :)

@dedifferentiator
Copy link
Contributor Author

I think that it would be easier to just reserve 'ESC' for quitting instead of having multiple bindings for it.

Yes, I also noticed this bug and fixed it, you probably didn't pull last changes at 51c96e3

Also, when the client first joins the game, I don't think that the quote set is properly updated for them:

Fixed :)

@rmehri01
Copy link
Owner

rmehri01 commented Feb 9, 2021

Yes, I also noticed this bug and fixed it, you probably didn't pull last changes at 51c96e3

Oops, my bad!

Copy link
Owner

@rmehri01 rmehri01 left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you again and sorry about the wait!

@rmehri01 rmehri01 merged commit 9ed02ce into rmehri01:main Feb 9, 2021
@dedifferentiator
Copy link
Contributor Author

It's okay, no problem at all!

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