07th-mod / 07th-mod/higurashi-patch-compiler
Ryukishi sprites and Backgrounds
- Lenguaje dominante
- PHP
- Estrellas
- 7
- Forks
- 1
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Currently our patch supports the PS3 and MG sprites but we removed the support for the original Ryukishi sprites. In this issue I'll write down the steps we need to take to add them back.
### 1. JSON file with sprite aliases
Both MG and Ryukishi sprites have fewer facial expressions than PS3. For MG sprites we solved it by addind certain sprites as multiple copies with different names which increases the patch size unnecessarily.
A better way would be to add a JSON file with a map of sprite aliases.
For Ryukishi sprites this would enable us to use the sprite names from vanilla and just link them to PS3 sprites using the json.
### 2. Fallback for sprites when LS is not active
I noticed that most cases of sprite duplication in CGAlt are cases like `me1a_akuwarai_a1_1` and `me1a_akuwarai_a1_2` being the same sprite. This is intentional as the sprite is different for PS3 sprite with LS disabled. For CGAlt there are also some cases where the sprites are different such as `oisi1_5_0` and `oisi1_5_2` in Watanagashi.
Solution: For CGAlt (and for Ryikishi sprites) if `sprite_X` does not exist try `sprite_(X-1)` and `sprite_(X-2)` (if X=2) as fallbacks before falling back to PS3 `sprite_X`.
### 3. Write the list of aliases
Next we need the linking for vanilla names (Ryukishi sprites) to our names (PS3 sprites). This will need to be done manually and will take some time because there are several hundreds of sprites for each arc. Of course we can use a json from one arc as a base for the next and generate a basefile automatically to insert the aliases into.
### 4. Engine changes
For the DLL this means to add another directory like current CG/CGAlt but for Ryukishi sprites. Unlike CGAlt this should not fallback to PS3 sprites - minor characters who don't have Ryukishi sprites simply won't have sprites instead of falling back to PS3.
Also we need a way for the user to configure which sprites they want to use. Since it's difficult for us to add a button to the config screen it would have to be another hotkey. Or possibly we can reuse the current hotkey for switching between sprites and just let it have a third value. There might be some compatibility issues with the config screen only having 2 values so it will need some testing.
### 5. Possible bugs
Our first patches had some issues with the engine freezing on some `WithFiltering` instructions if the file did not exist. Since then we made sure that no sprites are missing and it's likely the current engine won't actually freeze with all of our modding for LipSync. Still it needs some testing to check if it works ok since there will be missing Ryukishi sprite for the minor characters.
----
If you have any ideas or concerns please share. Also we need volunteers, especially for the third part to manually create the alias map.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
El issue describe agregar los sprites de Ryukishi nuevamente. Comience examinando los directorios de sprites existentes (CG, CGAlt) y el código del DLL del motor para comprender la lógica de carga de sprites. Busque ejemplos de archivos de alias JSON de los sprites MG. El trabajo implica crear un mapa de alias de sprites, modificar el motor para manejar un tercer directorio de sprites y una tecla de acceso rápido de configuración, y probar los respaldos para sprites faltantes. 'Hecho' significa que los sprites de Ryukishi se cargan correctamente mediante una tecla de acceso rápido sin que el motor se congele.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- game-dev
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 30/100