Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Internal lightmaps have blending issue and light spill out #782

Closed
illwieckz opened this issue Jul 29, 2015 · 6 comments
Closed

Internal lightmaps have blending issue and light spill out #782

illwieckz opened this issue Jul 29, 2015 · 6 comments

Comments

@illwieckz
Copy link
Member

Hey, I found a strange bug, lightmap rendering is wrong when they are loaded from the bsp, when they are loaded as external files (tested with png), the map looks good.

There is many bugs a simple lightmap externalization can fix, some blending issue, and some light spill out on edges.

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

internal:
lightmap

external:
lightmap

illwieckz added a commit to InterstellarOasis/map-terminus_src.dpkdir that referenced this issue Jul 30, 2015
- allow to ship lightmaps compressed as PNG in the PK3 alongside the BSP (they are flat
  RGB bitmaps when included inside BSP), save disk space
- workaround Unvanquished/Unvanquished#782
@DolceTriade
Copy link
Member

Nice screenshots. @gimhael, think you could take a look at this?

@illwieckz
Copy link
Member Author

FYI, these guys have experienced a similar issue (but the opposite one) with another engine (don't know which one, probably ioquake3 or quakelive):
TTimo/GtkRadiant#66

They got the opposite bug: they got light bleeding when using external lightmaps (internal are ok), since we get light bleeding when using internal lightmaps (external are ok). That other issue is an issue for q3map2, but it looks like a renderer issue on our side (btw, I don't have the source for the map above, so only renderer can fix).

There is absolutely no urge at all to fix this bug and I'm ok if it's not fixed since external lightmaps are working well (so, we just need to tell people to not use internal lightmap), and internal lightmaps must be avoided since it does not allow compression. But I thought that info would be useful, perhaps it can reveal or help to understand some related issues to.

This bug has a very slow priority, having internal lightmap working is not needed at all, it's just an historical inherit. (NB: a fix could be needed by third-party projects using inherited -ND assets, but no need to worry since this case have not yet occurred).

@Viech
Copy link
Member

Viech commented Jun 24, 2016

@illwieckz is this the only map that you tested though? Maybe the compiler was just misconfigured.

@illwieckz
Copy link
Member Author

Hum, since this topic old, I do not remember if I tested other maps back in time, btw, no one official map have internal lightmap so maps with internal lightmaps are all contributed ones, so it's hard to know if the problem comes from the map itself (and if the map was build the good way) or from another component…

@cmf028
Copy link
Member

cmf028 commented Jul 5, 2016

I've seen this problem before when the driver compresses textures by itself, usually when configured for "maximum performance". There isn't anything we can do about that if true.

illwieckz added a commit to InterstellarOasis/map-terminus_src.dpkdir that referenced this issue Apr 7, 2017
- allow to ship lightmaps compressed as PNG in the PK3 alongside the BSP (they are flat
  RGB bitmaps when included inside BSP), save disk space
- workaround Unvanquished/Unvanquished#782
illwieckz added a commit to InterstellarOasis/map-terminus_src.dpkdir that referenced this issue Apr 8, 2017
- allow to ship lightmaps compressed as PNG in the PK3 alongside the BSP (they are flat
  RGB bitmaps when included inside BSP), save disk space
- workaround Unvanquished/Unvanquished#782
illwieckz added a commit to InterstellarOasis/map-terminus_src.dpkdir that referenced this issue Apr 8, 2017
- allow to ship lightmaps compressed as PNG in the PK3 alongside the BSP (they are flat
  RGB bitmaps when included inside BSP), save disk space
- workaround Unvanquished/Unvanquished#782
@illwieckz
Copy link
Member Author

Those two issues are now tracked elsewhere:

Closing this one.

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

No branches or pull requests

4 participants