gongpha / gdQmapbsp

A simple plugin that loads Quake's MAP/BSP files into Godot 4 interactively
MIT License
42 stars 5 forks source link

Script error: Out of bounds get index '0' (on base: 'PackedInt32Array') #1

Closed Calinou closed 10 months ago

Calinou commented 1 year ago

OS: Fedora 37 Godot version: 4.0.2.stable

My settings:

image

Debugger output and local state:

image

I've tried this process without filling in the map source directory path, but I get the same error.

gongpha commented 1 year ago

I've seen the original Quake 1 paks are only pak0 and pak1. Where's the pak2 file come from ? Is it a mod or something ? I guess it contains no lump files and makes the error.

Calinou commented 1 year ago

I've seen the original Quake 1 paks are only pak0 and pak1. Where's the pak2 file come from ? Is it a mod or something ? I guess it contains no lump files and makes the error.

It contains maps, I assume it's a PAK containing compiled maps from the GPL map sources: image

After removing pak2.pak, I get this list:

image

What's strange is that they're all displayed as "no map file", despite me specifying a folder that contains .map sources for all maps.

gongpha commented 1 year ago

What's strange is that they're all displayed as "no map file", despite me specifying a folder that contains .map sources for all maps.

I tried using the map sources like yours (https://github.com/fzwoch/quake_map_source). And it seems like the playable map sources are still found. Have you tried scrolling down yet ?

Godot_v4 0 2-stable_win64_xNJ1AN8Y3s

Even adding my own pak2.pak which contains only BSP files like yours after being fixed by dd93db43d9e04462a42f0e515340d51c90720d1b. It is still found.

Calinou commented 1 year ago

I've tested https://github.com/gongpha/gdQmapbsp/commit/dd93db43d9e04462a42f0e515340d51c90720d1b on 4.0.2, I still can't play the map files:

image

gongpha commented 1 year ago

Can you try unchecking the "Convert MAP filenames to UPPERCASE on load" checkbox ? It will stop the filenames from being converted to uppercase.

I made this option because some map sources were published using uppercase names. AFAIK, Linux filenames are case-sensitive unlike Windows (11) which seems not to care about being case-sensitive.

image

gongpha commented 10 months ago

No replies after 3 months. Closing