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

Giuoco Piano and Italian Game #56

Closed
ulthiel opened this issue Jul 30, 2022 · 3 comments · Fixed by #61
Closed

Giuoco Piano and Italian Game #56

ulthiel opened this issue Jul 30, 2022 · 3 comments · Fixed by #61

Comments

@ulthiel
Copy link

ulthiel commented Jul 30, 2022

In the database the line "1.e4 e5 2.Nf3 Nc6 3.Bc4 Bc5" is called the "Giuoco Piano", which is correct I think. But then every other line continuing from there is called "Italian Game: ...". There's also "Italian Game: Giuoco Piano, ...". But then the first line above should also be named "Italian Game: Giuoco Piano". This seems to be somewhat inconsistent, or is there an explanation?

@ulthiel
Copy link
Author

ulthiel commented Jul 31, 2022

In the ECO database of SCID the line 1.e4 e5 2.Nf3 Nc6 3.Bc4 is called "Italian Game" and 1.e4 e5 2.Nf3 Nc6 3.Bc4 Bc5 is called "Giuoco Piano" (which further distributes into Giuoco Piano, Giuco Pianissimo, and Evans Gambit). I find this quite consistent.

@niklasf
Copy link
Member

niklasf commented Aug 25, 2022

I think the first suggestion (going with Italian Game: Giuoco Piano) ist most consistent. Do you want to submit a pull request?

@hitansh1299
Copy link
Contributor

I see this issue is still open, I'll be submitting a PR shortly

hitansh1299 added a commit to hitansh1299/chess-openings that referenced this issue Aug 28, 2022
hitansh1299 added a commit to hitansh1299/chess-openings that referenced this issue Aug 28, 2022
niklasf added a commit that referenced this issue Aug 28, 2022
Completed changes in accordance with #56 regarding the Giuco Piano.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants