Skip to content

Phase 3: the keep - #4

Merged
rixnobis merged 2 commits into
mainfrom
phase-3
Oct 17, 2026
Merged

rixnobis merged 2 commits into
mainfrom
phase-3

Conversation

@rixnobis

Copy link
Copy Markdown
Contributor

Phase 3: the keep (keep.scene, scene 2). Rooms Hall, Study and Chamber with portals Study door and Chamber door; the keeper is a 12-bone skinned mesh from art/models.py with idle (31 frames) and talk (16 frames) clips at 15 fps. Circle near him steps through five lines on the Dialogue canvas via keeper.lua, then sets Persist keeper. The valley's Keep door trigger runs Scene.Load(2) and Way out in the hall runs Scene.Load(1).

features.json is at phase 3 with a new dialogue entry: 22 required, 0 failed. Keeper reach is 4 m (3 m missed the study doorway). Each dialogue line is one row of at most 34 characters since uisystem.cpp has no line breaks.

The keep (scene 2) replaces its placeholder: three rooms, hall, study and
side chamber, as Room boxes joined by two Portals, the player on the
Indoor nav preset, and the keeper, a skinned mesh with idle and talk
clips and an Interactable. keeper.lua shows the hidden Dialogue canvas,
steps its lines with UI.SetText while the keeper talks, then hides it
and sets Persist keeper. The south door goes back to the valley, and a
gate in the valley, left of the start, opens the keep.

features.json moves to phase 3, adding a dialogue feature (UI.SetText).
Measured in pcsx-redux: at the first lighting the hall read nearly
black, the keeper's 3 m reach left him out of range of a player who
stops at the study door, and his talk gesture, a forearm raised
straight at the camera, did not read. The candles and moon are
brighter, his reach is 4 m, he stands three quarters to the door in a
russet robe, and talk raises the hand to his face.
@rixnobis
rixnobis merged commit 4b6dcee into main Oct 17, 2026
1 check passed
@rixnobis
rixnobis deleted the phase-3 branch October 19, 2026 00:51
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