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

Data Model doesnt consider the subcategories of Pokemon #1034

Open
GreatNovaDragon opened this issue Feb 6, 2024 · 2 comments
Open

Data Model doesnt consider the subcategories of Pokemon #1034

GreatNovaDragon opened this issue Feb 6, 2024 · 2 comments

Comments

@GreatNovaDragon
Copy link
Contributor

As briefly mentioned by me in #1026 , the data model doesnt consider the different subcategories of Pokemon. While up untill Sun and Moon, the disction "is_legendary" was possible, it always disregarded the mythical moniker, and in sun and moon the moniker sub_legendary was introduced in code, into which ultra beasts also were put into.

And Scarlet and Violet/Gen 9 now further split this into Ultra Beasts, Sub Legendary, Paradox Pokemon, Restricted Legendary and Mythical.

Therefore the datapoints of is_legendary is much more detailed than is previously assumed. One could also argue lorewise while Necrozma is a Restricted Legendary, its also is a Ultra Beast and much more. Koraidon and Miraidon are called paradox pokemon, but are handled as restricted legendaries in code.

So, instead of strict boolean is_legendary and is_starter, a list of categories a pokemon is in would be more appropiate.

@Naramsim
Copy link
Member

Hi, all these considerations are helpful! We have those (is_legendary,..) properties right in the CSV files so this change would require a rewrite of our data

@GreatNovaDragon
Copy link
Contributor Author

indeed, it would require a rewrite.

for compability sake, the serializer could check for is_legendary property if the list contains the legendary entry.

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

No branches or pull requests

2 participants