Conversation
|
I'm not sure keeping We should probably find a better way to store or link these sprites (e.g. sharing a single source of truth and referencing it per game) so we don't have to maintain multiple copies in sync. |
True that we have this duplication. Some options:
|
|
@Naramsim Yes, that's what I was thinking as well. I'd like to propose the structure below
sprites/pokemon/
├── default -> versions/generation-v/black-white
│
├── versions/
│ ├── generation-i/
│ ├── generation-ii/
│ ├── generation-iii/
│ ├── generation-iv/
│ └── generation-v/
│ └── black-white/
│ ├── back/
│ │ ├── female/
│ │ └── shiny/
│ │ └── female/
│ ├── female/
│ └── shiny/
│ └── female/
│
├── official-artwork/
├── home/
├── dream-world/
└── showdown/With this change, we'd also need to update the API so the default link points to What do you think about this structure overall? I'm wondering if it should be included in this PR or handled in a separate one. |
|
Hi, we cannot use the default folder you mention. The images are hot linked by thousands of projects, if we move them we break those. So we must keep available the root sprites where they are available now. |
|
I tried creating a symlink for Here's the test branch I used: https://github.com/anhthang/sprites/tree/symlink-gen5/sprites/pokemon |
|
Yeah, then the last option is deleting the sprites in the generation-v/blackwhite folder and change the API to serve them from the root folder. Or we leave everything as is and we accept the duplication. |
There was a problem hiding this comment.
Alright, I checked some sprites and they look fine.
I saw that some of them got replaced as the example below:
But I think the new sprite should be the correct one. Could you confirm?
As for the code, I'll leave @FallenDeity to review it if he has some time.
|
@Naramsim The new sprites come from the Smogon Gen 9 sheet, while the old ones are from KingOfThe-X-Roads, if I’m not mistaken. They’re also closer to the official artwork. |
The |
sprite_audit.pywith a structured report summary