GregLyons
commented
2 years ago Your suspicion is correct. The underlying database starts from the Smogon learnset data here and merges together some learnsets (Gen 2 and BD/SP) from the mods there to get a complete dataset. Some of the Pokemon in that set have "missing" learnsets (e.g. deoxysspeed), presumably because Pokemon Showdown has some other algorithm for adding the learnset data for Deoxys Speed.
What happened was this: I was aware of this issue, so I checked for Pokemon whose learnsets in the PS data were empty, and in that case I would add the learnsets (e.g. deoxys to deoxysspeed) when appropriate. What I didn't anticipate, however, was the case in which a Pokemon form gains a new move, like Zacian Crowned gaining Behemoth Blade. In the PS data, you'll see that the entry for zaciancrowned just has Behemoth Blade. So my check missed these cases, since it was checking for empty learnsets. The Rotom forms were also affected by this, as were some others.
For some reason, Calyrex Shadow does have full learnset data in the PS dataset, even though he gains a signature move but otherwise shares moves with Calyrex. Maybe because he also shares moves with Glastrier? I checked for learnset move counts below a certain threshold (10) and excluded obvious cases like Ditto (this is what flagged Rotom forms), so this should be fixed now. Here are some screenshots from the site now with there moves filled in:
Given Zacian's prominence in AG, as several other Pokemon being affected, I figured it's best to update this now instead of waiting to batch it with other data errors.
Thanks for finding the bug!
sdeiley
In
Builder->Team, if you select Zacian-Crowned, the only learned moves shown areThere’s probably some copying w/ DB data needed to ensure that Zacian-Crowned is applied all moves from Zacian. I didn’t verify with other mons that may have these kinds of edge cases, such as Zamazenta, Giratina, Kyruem, Calyrex-Shadow, etc.