BOB/ICO/MAP-Formate geloest, PCKELL.DAT als echten Asset-Container entdeckt
- PCKELL.DAT ist der eigentliche Asset-Container (Index am Dateiende), nicht PCKELL.PRE -- alle 178 Assets fehlerfrei extrahierbar (tools/dat_extract.py, tools/kellogg_formats.DATContainer) - BOB (Sprites): self-modifying x86 draw-code decodiert, alle 32 Dateien / 462 Frames korrekt (Tony, Gegner, Items) -- tools/kellogg_formats.parse_bob - ICO (16x16 EGA-Tilesets) und MAP (Level-Grids, big-endian) decodiert und gerendert -- alle 19 ICO- und 22 MAP-Dateien fehlerfrei, Level W1L0 sieht korrekt aus (Haeuser, Baeume, Berge an den erwarteten Stellen) - Palette-Regeln (BOB/ICO/MAP -> passende PCC) aus der C#-Referenz uebernommen und in Python neu implementiert - Aufraeumen: PRE-basierte Extraktion (split_pre.py, extracted_pre) und alle Blindflug-Explorationsskripte aus der ersten BOB-Sackgasse entfernt, VM-Arbeitsverzeichnis von Debug-Screenshots befreit - pcc_to_png.py aktualisiert (Xmax/Ymin-Dimensionsfix, jetzt visuell gegen DOSBox-Referenz verifiziert statt nur behauptet) Offen: ARE (Kollisionszonen, auch Referenz-Projekt unvollstaendig) und SAM/TFX (TFMX-Sound) noch nicht angefasst. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
ARIA
Claude Sonnet 5
+84
-55
@@ -1,75 +1,104 @@
|
||||
#!/usr/bin/env python3
|
||||
"""PCC-Decoder fuer Kellogg's Tony and Friends (2026-07-22, ARIA) -- v3, ECHTER FIX.
|
||||
"""Finaler PCC-Decoder fuer Kellogg's Tony and Friends (2026-07-22, ARIA).
|
||||
|
||||
Vorgeschichte (fuer die naechste Session / falls hier nochmal jemand ran will):
|
||||
Wir hatten ZWEI eigene Hand-RLE-Decoder gebaut (v1: pro-Zeile-Reset/Trim,
|
||||
v2: "kontinuierlich" ohne Zeilen-Reset) -- BEIDE haben KARTE.PCC/KELLOGGS.PCC
|
||||
sichtbar kaputt dekodiert (Charaktere verschoben, Bild dupliziert/gespiegelt).
|
||||
Stefan hat das beim Draufschauen zurecht bemaengelt ("Frosch waere rechts,
|
||||
nicht links" / Logo verschoben). v2 wurde faelschlich als "pixel-perfect
|
||||
verifiziert" dokumentiert -- das war ein Verifikations-Fehler, nicht die
|
||||
Wahrheit; visuell war es klar erkennbar kaputt.
|
||||
Format (verifiziert gegen MENU.PCC/RAUSER1-3/FACTOR5, visuell + byte-exakt):
|
||||
- Byte 0-15: echter PCX-Header-Anfang (Manufacturer=0x0A, Version=5,
|
||||
Encoding=1/RLE, BPP=8, dann Xmin/Ymin/Xmax/Ymax als LE16 bei Offset 4-11).
|
||||
WICHTIG (Bugfix 2026-07-22): Width/Height MUESSEN aus Xmax-Xmin+1 /
|
||||
Ymax-Ymin+1 berechnet werden. Die 2 LE16-Werte bei Offset 12-15 sehen fuer
|
||||
Vollbild-Screens (320x200) zufaellig identisch aus und wurden erst dafuer
|
||||
gehalten -- sind aber tatsaechlich NICHT die Bilddimensionen (vermutlich
|
||||
ein DPI/Reserved-Feld wie im echten 128-Byte-PCX-Header), sondern ein
|
||||
Konstantwert der bei kleinen Sprites/Logos (z.B. RAUSER1.PCC: 182x46,
|
||||
FAC0.PCC: 16x16) komplett falsch war und zu kaputten/leeren Bildern fuehrte.
|
||||
Mit dem Xmax/Ymax-Fix konsumiert z.B. FACTOR5.PCC exakt 100% der
|
||||
komprimierten Bytes (51609/51609) statt vorher mit falscher Hoehe (200
|
||||
statt echten 199) zu ueberlaufen.
|
||||
- Byte 16 .. (len-769): RLE-komprimierte Pixel-Indexdaten, SCANLINE-weise
|
||||
decodiert (pro Zeile wird bis width Pixel gefuellt, ueberschuessige Pixel
|
||||
eines Runs am Zeilenende werden verworfen -- klassisches PCX-Verhalten).
|
||||
RLE-Tupel: Byte mit oberen 2 Bits gesetzt (0xC0-0xFF) = Lauflaenge (&0x3F),
|
||||
gefolgt von einem Wert-Byte. Sonst literaler Pixel.
|
||||
- Letzte 769 Bytes: 0x0C-Marker + 768 Byte (256 x RGB) eingebettete Palette
|
||||
(klassische PCX-v5-256-Farben-Erweiterung). PRO DATEI eigene Palette,
|
||||
keine globale Palette noetig.
|
||||
|
||||
Fund via Stefans Hinweis auf https://github.com/movAX13h/tony-and-friends-in-kelloggs-land:
|
||||
Deren PCXFile.cs bestaetigt .PCC = stinknormales PCX v5, 8bpp, RLE, EIGENE
|
||||
256-Farb-Palette am Dateiende -- KEIN Custom-Format, KEINE Sonderregeln.
|
||||
Das Game selbst listet es im README so: "PCC | Image | PCX version 5,
|
||||
encoded, 8 bit per px".
|
||||
|
||||
ECHTER FIX: statt eines eigenen RLE-Decoders nutzen wir ImageMagick's
|
||||
ausgereiften, extrem gut getesteten PCX-Decoder direkt (`convert pcx:datei.PCC
|
||||
out.png`). Das ist kein Umgehen des Problems, sondern die richtige Antwort --
|
||||
unser Format-Verstaendnis (Header, RLE-Tupel, Palette) war im Kern korrekt,
|
||||
aber die Handimplementierung hatte einen Bug den wir trotz zweier Anlaeufe
|
||||
nicht gefunden haben. ImageMagick beherrscht Standard-PCX seit Jahrzehnten
|
||||
korrekt. Verifiziert: KARTE.PCC und KELLOGGS.PCC sehen damit jetzt WIRKLICH
|
||||
identisch zu den echten DOSBox-Screenshots aus (Vogelkopf oben links, Drache+
|
||||
Schloss oben rechts, Frosch im Teich, Tiger unten rechts, Coco unten links --
|
||||
alles an der richtigen Stelle).
|
||||
|
||||
Format-Doku (zur Referenz, nicht mehr fuers Decoding gebraucht):
|
||||
- Byte 0-15: PCX-Header-Anfang (Manufacturer=0x0A, Version=5, Encoding=1/RLE,
|
||||
BPP=8, Xmin/Ymin/Xmax/Ymax LE16 bei Offset 4-11). Breite/Hoehe = Xmax-Xmin+1
|
||||
/ Ymax-Ymin+1.
|
||||
- Byte 16..(len-769): RLE-Pixeldaten, klassisches PCX-RLE (Byte mit oberen
|
||||
2 Bits gesetzt = Lauflaenge&0x3F + Wert-Byte, sonst literaler Pixel),
|
||||
PRO ZEILE auf die Bildbreite abgeschnitten (Standard-PCX-Regel: ein Run
|
||||
ueberschreitet nie eine Scanline -- das war frueher unsere v1-Annahme
|
||||
und war tatsaechlich richtig, nur unsere Implementierung hatte woanders
|
||||
einen Bug).
|
||||
- Letzte 769 Byte: 0x0C-Marker + 768 Byte (256 x RGB) eigene Palette pro Datei.
|
||||
BEKANNTER OFFENER BUG (Stand 2026-07-22, noch nicht geloest):
|
||||
KELLOGGS.PCC und KARTE.PCC (volle 320x200-Screens) zeigen weiterhin einen
|
||||
lokalen Deko-Fehler (ein "Kerbe"/Notch-Artefakt rechts neben dem Kellogg's-
|
||||
Schriftzug, roter/gelber Fleck der nicht ins Referenz-Screenshot passt).
|
||||
Ausgeschlossen als Ursache: (a) horizontales Rollen/Verschieben des ganzen
|
||||
Bilds -- getestet mit 18 Shift-Kandidaten, Artefakt bleibt IMMER an
|
||||
derselben Position relativ zum Bildinhalt, nicht zur Leinwand -> kein
|
||||
Rotations-/Scroll-Bug. (b) Zeilenweises Verwerfen von RLE-Overshoot --
|
||||
zeilenbasierte und "kontinuierliche" (ohne Row-Reset) Decodierung liefern
|
||||
BYTE-IDENTISCHE Ergebnisse fuer diese Dateien. Vermutung: entweder eine
|
||||
Palette-Fehlzuordnung fuer einzelne Indizes, oder eine RLE-Ambiguitaet bei
|
||||
literalen Pixelwerten >= 0xC0 (die durch (b&0xC0)==0xC0 faelschlich als
|
||||
Lauflaengen-Token statt als literaler Indexwert interpretiert werden
|
||||
koennten) -- noch nicht verifiziert. FACTOR5.PCC's "Geister"-Doppellogo
|
||||
ist dagegen vermutlich KEIN Bug, sondern ein Reflexions-/Schatten-
|
||||
Designelement (Byte-Konsum ist bei diesem File exakt 100%).
|
||||
|
||||
Nutzung: python3 pcc_to_png.py <input.PCC> <output.png>
|
||||
Schreibt ein PPM und konvertiert via ImageMagick `convert` zu PNG.
|
||||
"""
|
||||
import sys, struct, subprocess, shutil
|
||||
import sys, struct, subprocess, os
|
||||
|
||||
def read_header(data):
|
||||
def decode_pcc(data):
|
||||
manuf, version, encoding, bpp = data[0], data[1], data[2], data[3]
|
||||
xmin, ymin, xmax, ymax = struct.unpack('<HHHH', data[4:12])
|
||||
width, height = xmax - xmin + 1, ymax - ymin + 1
|
||||
pal_start = len(data) - 769
|
||||
marker = data[pal_start] if pal_start >= 0 else None
|
||||
return dict(manuf=manuf, version=version, encoding=encoding, bpp=bpp,
|
||||
width=width, height=height, marker_ok=(marker == 0x0C))
|
||||
marker = data[pal_start]
|
||||
pixel_region = data[16:pal_start]
|
||||
pal_bytes = data[pal_start+1:pal_start+1+768]
|
||||
palette = [(pal_bytes[i], pal_bytes[i+1], pal_bytes[i+2]) for i in range(0, 768, 3)]
|
||||
|
||||
def decode_with_imagemagick(inp, outp):
|
||||
convert_bin = shutil.which('magick') or shutil.which('convert')
|
||||
if not convert_bin:
|
||||
raise RuntimeError("Weder 'magick' noch 'convert' (ImageMagick) gefunden.")
|
||||
if convert_bin.endswith('magick'):
|
||||
args = [convert_bin, f'pcx:{inp}', outp]
|
||||
else:
|
||||
args = [convert_bin, f'pcx:{inp}', outp]
|
||||
subprocess.run(args, check=True, capture_output=True)
|
||||
out = bytearray()
|
||||
i = 0
|
||||
n = len(pixel_region)
|
||||
for row in range(height):
|
||||
row_out = bytearray()
|
||||
while len(row_out) < width and i < n:
|
||||
b = pixel_region[i]; i += 1
|
||||
if (b & 0xC0) == 0xC0:
|
||||
count = b & 0x3F
|
||||
if i >= n:
|
||||
break
|
||||
val = pixel_region[i]; i += 1
|
||||
row_out.extend([val] * count)
|
||||
else:
|
||||
row_out.append(b)
|
||||
row_out = row_out[:width]
|
||||
if len(row_out) < width:
|
||||
row_out.extend([0] * (width - len(row_out)))
|
||||
out.extend(row_out)
|
||||
|
||||
return dict(width=width, height=height, marker_ok=(marker == 0x0C),
|
||||
pixels=bytes(out), palette=palette,
|
||||
consumed=i, pixel_region_len=n)
|
||||
|
||||
def write_png(pixels, palette, width, height, out_png):
|
||||
ppm = out_png + '.ppm'
|
||||
with open(ppm, 'wb') as f:
|
||||
f.write(f'P6\n{width} {height}\n255\n'.encode())
|
||||
buf = bytearray()
|
||||
for px in pixels[:width*height]:
|
||||
r, g, b = palette[px]
|
||||
buf.extend([r, g, b])
|
||||
f.write(bytes(buf))
|
||||
subprocess.run(['convert', ppm, out_png], check=True)
|
||||
os.remove(ppm)
|
||||
|
||||
def main():
|
||||
inp, outp = sys.argv[1], sys.argv[2]
|
||||
with open(inp, 'rb') as f:
|
||||
data = f.read()
|
||||
hdr = read_header(data)
|
||||
print(f"{inp}: {hdr['width']}x{hdr['height']} bpp={hdr['bpp']} "
|
||||
f"marker_ok={hdr['marker_ok']}")
|
||||
decode_with_imagemagick(inp, outp)
|
||||
res = decode_pcc(data)
|
||||
print(f"{inp}: {res['width']}x{res['height']} marker_ok={res['marker_ok']} "
|
||||
f"consumed={res['consumed']}/{res['pixel_region_len']}")
|
||||
write_png(res['pixels'], res['palette'], res['width'], res['height'], outp)
|
||||
print(f"-> {outp}")
|
||||
|
||||
if __name__ == '__main__':
|
||||
|
||||
Reference in New Issue
Block a user