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

Is it possible to add japanese pokemon names? #103

Open
saboyutaka opened this issue Jul 27, 2017 · 12 comments
Open

Is it possible to add japanese pokemon names? #103

saboyutaka opened this issue Jul 27, 2017 · 12 comments

Comments

@saboyutaka
Copy link

Pokemon terminal is awesome ๐Ÿ‘ ๐Ÿ‘ ๐Ÿ‘
But I think it's better for non-english user if you can choose your country's pokemon name
e.x Bulbasaur is ใƒ•ใ‚ทใ‚ฎใƒ€ใƒ in japanese.

here is a list for pokemon names in many languages. (It's in Japanese though )
https://wiki.ใƒใ‚ฑใƒขใƒณ.com/wiki/ใƒใ‚ฑใƒขใƒณใฎๅค–ๅ›ฝ่ชžๅไธ€่ฆง

If it's possible. I'll code it.

@sylveon
Copy link
Collaborator

sylveon commented Jul 27, 2017 via email

@saboyutaka
Copy link
Author

saboyutaka commented Jul 27, 2017

thank you for response!

Is Japanese text input even supported on terminal emulators?

Yeah, I think so. We usually type Japanese characters on terminal.

Once I edit Data/pokemon.txt from english to japanese name, It works.

pokemon

@sylveon
Copy link
Collaborator

sylveon commented Jul 27, 2017 via email

@saboyutaka
Copy link
Author

UTF-8 includes Japanese characters(Katakana) which is used pokemon names.

https://www.key-shortcut.com/en/writing-systems/ใฒใ‚‰ใŒใช-japanese/

@cclauss
Copy link
Contributor

cclauss commented Jul 27, 2017

To do this, we would need a new version of https://github.com/LazoCoder/Pokemon-Terminal/blob/master/Data/pokemon.txt perhaps named pokemon-jp.txt that added a second column which was the Japanese name of each character. Those names must have no whitespaces! With that file, we could do some experimentation to see how hard it would be to deliver this feature.

@Squirrels
Copy link
Collaborator

I was going to add support for japanese, getting the names from Sindresorhus' Pokemon Name List, which contains japanese, russian, among other languages, when I started wondering if we could add support for all of those languages at once.

The problem would be that the pokemon.txt file would keep growing, and may become too messy. I was thinking converting it to a JSON file, where adding more information to each Pokemon would be easier. Opinions, @cclauss @LazoCoder @sylveon ?

@cclauss
Copy link
Contributor

cclauss commented Sep 20, 2017

I would still vote no to json but I might be in the minority. #96

@sylveon
Copy link
Collaborator

sylveon commented Sep 20, 2017

What about csv?

@samuelhnrq
Copy link
Collaborator

samuelhnrq commented Sep 26, 2017

I wanted to change the db into JSON ages ago, but since it were basically the pokemon name and the file, it was just going to overly complicate it, but now we have, the lightness index, the pokemon types, and the pokemon regions are all implicit, support for localized pokemon names might just be the nail in the coffin for the change, the problem is, @LazoCoder is, well, occupied? And we already have tons of conflicting changes we will have to sort out, I would love to change it, just don't think the timing is right. We could even finally implement pokemon named tuples (right @cclauss ?) to easily marshal and unmarshal them, not to mention the almost assured speed increase.

I totally support the change @Squirrels

@samuelhnrq
Copy link
Collaborator

Some work has been done in #96 that I think can be salvaged

@Squirrels
Copy link
Collaborator

Yeah, I'm planning on using that as a base, updating with the new pokemon and information as soon as I'm done with #112 and Mac OS testing for #94 (our current master).

@cclauss
Copy link
Contributor

cclauss commented Oct 10, 2017

OK. No more flatfile!

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

No branches or pull requests

5 participants