gongpha / gdQmapbsp

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

Lightmaps don't seem to be properly places on some meshes #5

Closed victorbstan closed 1 year ago

victorbstan commented 1 year ago

As was discussed in another thread, I changed the texture limit and shader index to 64 from 256 (for Mac M2 Godot compatibility) and the textures show up, but there's a couple of issues.

  1. Some maps like (start.bsp and e1m1.bsp) are very dark and have to have the brightness setting set to max for lightmaps. While other maps seem fine.

  2. Lightmaps don't seem to be properly placed.

https://github.com/gongpha/gdQmapbsp/assets/315297/cf5029fc-e8ce-4f3b-bd7c-a2154a7acc81

Also, I am now getting the You hit the texture limit ! (> 64) error in the console...

PS. E1M4.bsp looks fine, so I suspect its caused by the texture limit...

victorbstan commented 1 year ago

Closing this, because I'm pretty sure now it's caused by the change to texture limits as discussed in another thread.

victorbstan commented 1 year ago

One more thing(TM): I have no problems with Lightmaps in Mobile render mode, but they are still black in Forward+ -- for not, I'm OK with that, since I don't need fancy graphics, and was planning on targeting Mobile... but something to keep in mind