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

Fix Gen/Region/Version data inconsistencies for Gen9 #826

Merged
merged 4 commits into from
Jan 27, 2023

Conversation

Pythonic-Rainbow
Copy link
Contributor

Legend Arceus is considered a part of Gen8 on Wikipedia and Bulbapedia. However, it is a special case because it has a different main region from SwSh.

LA is indeed gen8 in version_groups.csv, but some of the data in other csv files are inconsistent. I assume that LA is a part of gen8 in the API. If so, there will be 2 main regions for gen8, but the /generation endpoint specifies main_region as a single Region attribute. What are your plans on this problem?

Another example is that /region/9 (LA:Hisui) currently lists gen9 as its main generation, and /region/10 (SV:Paldea) has no main generation. My commit should swap the results, but Hisui will have no main generation.

@Pythonic-Rainbow Pythonic-Rainbow changed the title Fix Gender/Region/Version data inconsistencies for Gen8/9 Fix Gen/Region/Version data inconsistencies for Gen8/9 Jan 25, 2023
@Naramsim
Copy link
Member

@tillfox @giginet do you have some thoughts about this issue?

@tillfox
Copy link
Contributor

tillfox commented Jan 25, 2023

@tillfox @giginet do you have some thoughts about this issue?

It seems that @Pythonic-Rainbow has fixed problems in "versions.csv", "version_group_regions.csv" and "generations.csv"... which are the other inconsistencies?
Main region for 8th gen is Galar, main region for 9th gen is Paldea, everything is correct. Every Pokemon generation by now has just a main region, it makes sense, unless they create a 10th gen game set in two different regions simultaneously...

@Pythonic-Rainbow
Copy link
Contributor Author

@tillfox @giginet do you have some thoughts about this issue?

It seems that @Pythonic-Rainbow has fixed problems in "versions.csv", "version_group_regions.csv" and "generations.csv"... which are the other inconsistencies? Main region for 8th gen is Galar, main region for 9th gen is Paldea, everything is correct. Every Pokemon generation by now has just a main region, it makes sense, unless they create a 10th gen game set in two different regions simultaneously...

Yes you're right. Every gen so far has just 1 main region. I thought Hisui is another main region but I did some more research on Bulbapedia and looks like it is just Sinnoh, which is Gen4 main region.

Then yeah, my question is invalid. Sorry for the confusion.

@Pythonic-Rainbow Pythonic-Rainbow changed the title Fix Gen/Region/Version data inconsistencies for Gen8/9 Fix Gen/Region/Version data inconsistencies for Gen9 Jan 25, 2023
@Pythonic-Rainbow
Copy link
Contributor Author

Pythonic-Rainbow commented Jan 25, 2023

I've also added version names, taken from Bulbapedia.

But why are there 3 Japanese locales in https://github.com/PokeAPI/pokeapi/blob/6c3c68a91bb24437f1396c7e3af00d0a9d69a70c/data/v2/csv/languages.csv ? I don't see Roomaji being used anywhere in the csv files, and ID 11 is identical to ID 1 in all the *names.csv that I've found

Copy link
Contributor

@giginet giginet 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.

Hisui will have no main generation.

I don't have any concerns about this.

@giginet
Copy link
Contributor

giginet commented Jan 26, 2023

I don't see Roomaji being used anywhere in the csv files, and ID 11 is identical to ID 1 in all the *names.csv that I've found

I'm a native Japanese. You don't need to set romaji names.

ID 1 and 11 are not identical. ID 1 is ja, and ID 11 is ja without Kanji.
In version names, they seem to be the same. However, flavor texts are different.

@Pythonic-Rainbow
Copy link
Contributor Author

I don't see Roomaji being used anywhere in the csv files, and ID 11 is identical to ID 1 in all the *names.csv that I've found

I'm a native Japanese. You don't need to set romaji names.

ID 1 and 11 are not identical. ID 1 is ja, and ID 11 is ja without Kanji. In version names, they seem to be the same. However, flavor texts are different.

Ah I see. Thanks for the clarification.

@Naramsim Naramsim merged commit 9a93747 into PokeAPI:master Jan 27, 2023
@Naramsim
Copy link
Member

Hi @giginet , would you like to become part of @PokeAPI? We really appreciated your help in dumping data

@pokeapi-machine-user
Copy link

A PokeAPI/api-data refresh has started. In 45 minutes the staging branch of PokeAPI/api-data will be pushed with the new generated data.

The staging branch will be deployed in our staging environment and you will be able to review the entire API.

A Pull Request (master<-staging) will be also created at PokeAPI/api-data and assigned to the PokeAPI Core team to be reviewed. If approved and merged new data will soon be available worldwide at pokeapi.co.

@pokeapi-machine-user
Copy link

The updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data.

You can see the Pull Request deployed at our staging environment when CircleCI deploy will be finished (check the started time of the last build).

@giginet
Copy link
Contributor

giginet commented Jan 28, 2023

@Naramsim I'm happy to hear about your offer. I can contribute to this repo.
I'm maintaining my product for future series. So I'm going to contribute to this repo to keep a sustainable data source.

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.

5 participants