07th-mod / 07th-mod/higurashi-patch-compiler

Ryukishi sprites and Backgrounds

Open
#54 34 comments 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
7
Forks
1
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue describes adding Ryukishi sprites back. Start by examining the existing sprite directories (CG, CGAlt) and the engine DLL code to understand the sprite loading logic. Look for JSON alias file examples from MG sprites. The work involves creating a sprite alias map, modifying the engine to handle a third sprite directory and a configuration hotkey, and testing for missing sprite fallbacks. 'Done' means Ryukishi sprites load correctly via a hotkey without engine freezes.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
game-dev
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.