Skip to content

Add XD Encounters, Encounter Conditions, Locations, Location Areas, Encounter Methods, and a few more things#1446

Open
notblisy wants to merge 1 commit intoPokeAPI:masterfrom
notblisy:xd-trades-encounters-conditions
Open

Add XD Encounters, Encounter Conditions, Locations, Location Areas, Encounter Methods, and a few more things#1446
notblisy wants to merge 1 commit intoPokeAPI:masterfrom
notblisy:xd-trades-encounters-conditions

Conversation

@notblisy
Copy link
Contributor

The goal of this PR is to add all the encounter locations, conditions, and items for Pokemon obtainable in Pokemon XD: Gale of Darkness.

This is a fair bit larger than Pokemon Colosseum, as there's 82 Shadow Pokemon in this game, plus a few extra gift Pokemon.

I sourced my data from:

For the encounters, I added the usual:

  • Locations
  • Location_Areas
  • Encounter_Slots
  • Encounters
  • Pokemon_items

In addition to all the relevant prose for these.

On top of that, I've added a new encounter method:

  • Pokespot

This is a unique way to catch wild Pokemon in XD, where you leave a snack out for a Pokemon, and one of 3 different Pokemon can come to eat it. Then you get to do a wild battle and capture.

During my time modeling XD, I noticed that while there IS an NPC-Trade option in encounter_methods.csv, this doesn't allow us to note WHICH Pokemon the player would have to trade. So, I added a new encounter condition called:

  • trade

Then, in encounter_condition_values.csv I would describe both the NPCs name and Pokemon to trade them.

An example is that for a larvitar, duking wants a wooper. So I have the encounter condition value id set to:

  • 112,15,trade-duking-wooper,0

Encounter conditions seem to be rarely used so far in the API, but I think they are a great way to add specificity to encounters like trades or gifts outside of just locations. I hope this was okay to add!

@notblisy
Copy link
Contributor Author

I think that for encounter condition value map I might have put the wrong value? Is 0
Or 1 what it needs to be for the condition to be true?

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.

1 participant