Versioning Type Efficacy and Pokémon Types #213
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
In this branch I have added generation versioning to the
TypeEfficacy
table and to thePokemonTypes
table. I have also included the versioned (previously missing) data to those tables.Type Efficacy
The following SQL would change the schema and add the data for
TypeEfficacy
:The new
TypeEfficacy
schema treats each row as a damage modifier instead of the former damage factor. Being a modifier means that when no row is present for a type match up, no modification applies to the damage (former 100%). This change reduces the amount of rows and allow easy versioning (without duplication for every change).This data was handcrafted from info found here and applied on top of the csv data in branch master .
Pokemon Types
The following SQL would change the schema and add the data for
PokemonTypes
versioning:This data was handcrafted from info found around here and applied on top of the csv data in branch master.
Considerations
I am no Python guy and don't really know what else must be changed (classes and documentation probably) in order to merge these changes, so I would appreciate if someone could tell me.