Fix KQVI puzzle dependency chart QA issues: remove S17 undefined node, S19/S20 dead ends, add Pearl note
This commit is contained in:
@@ -61,6 +61,7 @@ flowchart TD
|
||||
P4 --> S11
|
||||
S7 & S8 & S9 & S10 & S11 --> P5
|
||||
P4 --> S12
|
||||
S12 -.-> S12_note["→ Optional: Pearl side quest (independent)"]
|
||||
|
||||
%% Phase 5: Puzzle Chain on Wonder
|
||||
subgraph Phase5["Phase 5: Isle of Wonder - Garden & Chessboard"]
|
||||
@@ -79,16 +80,12 @@ flowchart TD
|
||||
subgraph Phase6["Phase 6: Isle of the Beast - Initial Visit"]
|
||||
P6["Problem: Boiling pool blocks path"]
|
||||
S16["Solution: Cool pool with Iceberg Lettuce"]
|
||||
S19["Problem: Need dangling participle for bookworm"]
|
||||
S20["Solution: Get from creature on Beast shore"]
|
||||
S21["Solution: Get Brick from garden"]
|
||||
end
|
||||
|
||||
P6 --> S16
|
||||
S16 --> P7
|
||||
P6 --> S19
|
||||
P6 --> S21
|
||||
S19 --> S20
|
||||
|
||||
%% Phase 7: Minotaur's Maze
|
||||
subgraph Phase7["Phase 7: Minotaur's Maze"]
|
||||
@@ -134,6 +131,7 @@ flowchart TD
|
||||
S28 --> P11
|
||||
P11 --> S26
|
||||
S26 --> P12
|
||||
P12 --> S27
|
||||
S27 --> C3["Winged Ones take you to Minotaur Maze"]
|
||||
|
||||
%% Phase 9: Isle of Mists / Druids
|
||||
@@ -246,7 +244,7 @@ flowchart TD
|
||||
classDef start_end fill:#ffcc00,stroke:#333,stroke-width:3px
|
||||
|
||||
class P1,P2,P3,P4,P5,P6,P6b,P7,P8,P9,P10,P11,P12,P13,P14,P15,P16,P17,P18,P19,P20,P21,P22,P23,P24,P25,P26,P27,P28,P29 problem
|
||||
class S1,S2,S3,S4,S5,S6,S7,S8,S9,S10,S11,S12,S13,S14,S15,S16,S17,S18,S19,S20,S21,S22,S23,S24,S25,S26,S27,S28,S29,S30,S31,S32,S33,S34,S35,S36,S37,S38,S39,S40,S41,S42,S43,S44,S45,S46,S47,S48,S49,S50,S51 solution
|
||||
class S1,S2,S3,S4,S5,S6,S7,S8,S9,S10,S11,S12,S13,S14,S15,S16,S18,S21,S22,S23,S24,S25,S26,S27,S28,S29,S30,S31,S32,S33,S34,S35,S36,S37,S38,S39,S40,S41,S42,S43,S44,S45,S46,S47,S48,S49,S50,S51 solution
|
||||
class START,END,C1,C2,C3,C4,C5 start_end
|
||||
```
|
||||
|
||||
@@ -0,0 +1,90 @@
|
||||
# King's Quest VI Puzzle Dependencies - Layout QA Report
|
||||
|
||||
## 1. START/END Presence
|
||||
|
||||
| Node | Status |
|
||||
|------|--------|
|
||||
| START | **YES** - Line 10: `START(["START: Wake on Beach"]) --> P1` |
|
||||
| END | **YES** - Line 240: `S51 --> END` |
|
||||
|
||||
## 2. Connectivity Check
|
||||
|
||||
### Nodes with ISSUES:
|
||||
|
||||
#### Orphan Nodes (no incoming edges - unreachable from START):
|
||||
1. **S20** ("Get from creature on Beast shore") - Connected FROM S19, but S20 has NO outgoing edges
|
||||
- Path: `P6 --> S19 --> S20` (DEAD END)
|
||||
- This appears to be where you get the "dangling participle" for the bookworm puzzle, but that puzzle isn't in this chart
|
||||
|
||||
2. **S12** ("Get Pearl from Oyster") - Connected FROM P4, but has NO outgoing edges
|
||||
- Marked as "(from beach, independent of gnomes)" - optional item, creates orphan subgraph
|
||||
|
||||
#### Undefined Nodes (in CSS class but no node definition):
|
||||
- **S17** - Referenced in classDef (line 250) but never defined anywhere in the graph
|
||||
|
||||
#### Fixed Issues:
|
||||
- **S27** - Now properly connected: `P12 --> S27 --> C3`
|
||||
- **S18** - Already defined at line 116 in Phase 6b
|
||||
- **S28** - Already defined at line 118 in Phase 6b
|
||||
|
||||
### Main Flow Connectivity (Verified):
|
||||
```
|
||||
START → P1 → S1 → P2 → S2 → C1 → P3 → S3/S4/S5/S6 → P4 → S7-S11 → P5 → S13-S15 → P6
|
||||
↓
|
||||
P6b ← C2 ← S25 ← P9 ← S23/S24 ← P8 ← S22 ← P7 ← S16
|
||||
↓ ↑
|
||||
P6b → S18 → P10 → S28 → P11 → S26 → P12 → S27 → C3
|
||||
```
|
||||
|
||||
## 3. Cycle Check
|
||||
|
||||
**No invalid cycles detected.**
|
||||
|
||||
The apparent "cycle" at line 161 (`S33 --> P13`) is actually valid:
|
||||
- P13 = "Druids will burn you without protection" / "Beast needs help to break curse"
|
||||
- This represents the player returning to the Druids with Beauty's Dress (from Beast domain)
|
||||
- This is an intentional item-return loop, not a puzzle dependency cycle
|
||||
|
||||
## 4. Remaining Issues
|
||||
|
||||
### Unresolved Issues:
|
||||
|
||||
1. **S17 undefined** - Listed in CSS classDef but no node with this ID exists anywhere in the graph. This appears to be a numbering error where a solution step was skipped in the original chart.
|
||||
|
||||
2. **S20 dead end** - S19 → S20 with no outgoing edge. The "dangling participle for bookworm" item is obtained here but the bookworm puzzle itself isn't included in this dependency chart (it's in the catacombs).
|
||||
|
||||
3. **S12 orphan subgraph** - Optional "Pearl from Oyster" item marked as "independent of gnomes" creates a small disconnected subgraph.
|
||||
|
||||
### Fixed Issues:
|
||||
- **P12 → S27 edge** - Added to properly connect the dark cave solution to the flow
|
||||
|
||||
## 5. Layout Suggestions
|
||||
|
||||
### For S17:
|
||||
Either remove S17 from the classDef or identify what it should represent. Based on the sequence (S16 is Iceberg Lettuce, S18 is Shield), S17 might have been intended to bridge these two solutions.
|
||||
|
||||
### For S20:
|
||||
The dangling participle is used in the catacombs for a bookworm puzzle. The dependency chart intentionally excludes catacomb puzzles. S20 is a dead end in THIS chart but would connect to catacomb content in a more complete chart.
|
||||
|
||||
### For S12:
|
||||
This is explicitly marked as independent. Consider adding a note in the legend explaining that S12-Pearl is an optional side quest that doesn't affect main progression.
|
||||
|
||||
## 6. Overall Assessment
|
||||
|
||||
| Criterion | Status |
|
||||
|-----------|--------|
|
||||
| START node present | PASS |
|
||||
| END node present | PASS |
|
||||
| All nodes reachable from START | **FAIL** - S20, S12 orphans |
|
||||
| All nodes reach END | PASS (main path) |
|
||||
| No circular dependencies | PASS |
|
||||
| No dead ends | **FAIL** - S20 dead ends |
|
||||
|
||||
**OVERALL: FAIL** (but improved from original)
|
||||
|
||||
The chart has been fixed to include the critical P12 → S27 edge. Remaining issues are:
|
||||
1. S17 undefined (CSS error)
|
||||
2. S20 dead end (catacomb content excluded from chart scope)
|
||||
3. S12 orphan (intentionally independent)
|
||||
|
||||
These issues don't break the main dependency flow but indicate incomplete documentation of solution items.
|
||||
@@ -0,0 +1 @@
|
||||
# Quest for Glory 1: Shadows of Darkness (1989)
|
||||
Reference in New Issue
Block a user