Daemon icon indicating copy to clipboard operation
Daemon copied to clipboard

partial void effect on station15 tree model

Open illwieckz opened this issue 6 years ago • 12 comments

See that weird void effect at the tip of the conifer branch:

incorrect

It must look like this:

correct

Note that the bug seems to occur on the collision with the plant leaf model. For information, there is also an alphagen brush in source map at this place:

netradiant

This bug looks somewhat related to that bug noticed in wolf:et map (at least for the tree): https://github.com/Unvanquished/Unvanquished/issues/799#issuecomment-253367127

illwieckz avatar Aug 28 '18 21:08 illwieckz

@gimhael the somwhat fix for #118 also somewhat fixes this (the texture is right but lighting is wrong):

incorrect

somewhat correct

It also somewhat fixes Unvanquished/Unvanquished#799 bug at least for the tree part:

incorrect

somewhat correct

But that variant of the bug is still remaining:

incorrect

It was noticed in Unvanquished/Unvanquished#799 (see comment) that the wolf:et bug was introduced in 6cb4dcc but the source tree was buildable starting with commit 97cbb46 (those are translated commit ids in Daemon repository after engine split, see the other thread for commits ids on Unvanquished repository).

illwieckz avatar Aug 28 '18 21:08 illwieckz

For testing purpose a snapshot of for-0.51.0 assets (including station15 map) can be fetched this way:

wget -O- http://gg.illwieckz.net/dl/unvanquished/dev/UnvanquishedAssets.get | xargs wget

illwieckz avatar Aug 28 '18 22:08 illwieckz

For testing purpose one can load wolf:et's railgun map this way:

  • Download official etlegacy assets from an official mirror (you can use pimi):
pimi.sh etmain
  • Add some compatibility layer (no legal issue, it's just doing symlinks, downloading empty dpks containing DEPS file only and downloading a spawn layout so the game does not end at startup):
mkdir -p ~/.local/share/unvanquished/pkg/etmain
mkdir -p ~/.local/share/unvanquished/game/layouts/railgun
ln -s ~/.etlegacy/etmain/pak0.pk3 ~/.local/share/unvanquished/pkg/etmain/pak0_0.dpk
ln -s ~/.etlegacy/etmain/pak1.pk3 ~/.local/share/unvanquished/pkg/etmain/pak1_0.dpk
ln -s ~/.etlegacy/etmain/pak2.pk3 ~/.local/share/unvanquished/pkg/etmain/pak2_0.dpk
wget -O ~/.local/share/unvanquished/pkg/etmain/res-etmain_0.dpk http://dl.illwieckz.net/b/unvanquished/extra/foreign-maps/etmain/pkg/etmain/res-etmain_0.pk3
wget -O ~/.local/share/unvanquished/pkg/etmain/map-railgun_0.dpk http://dl.illwieckz.net/b/unvanquished/extra/foreign-maps/etmain/pkg/etmain/map-railgun_0.pk3
wget -O ~/.local/share/unvanquished/game/layouts/railgun/spawn.dat http://dl.illwieckz.net/b/unvanquished/extra/foreign-maps/etmain/game/layouts/railgun/spawn.dat
  • Load the railgun map with the cheat to never end on missing spawn:
daemon -pakpath ~/.local/share/unvanquished/pkg/etmain/ +set g_neverend 1 +devmap railgun
  • Load the railgun map with the spawn layout:
daemon -pakpath ~/.local/share/unvanquished/pkg/etmain/ +devmap railgun spawn

illwieckz avatar Aug 28 '18 22:08 illwieckz

note that the unattended side-effect "somewhat fix" previously talked about is not there anymore, bug was brought back when lighting was fixed, i.e the complete fix for the other bug left this one unmodified despite some work in progress had some effect on this.

illwieckz avatar Dec 14 '18 05:12 illwieckz

The tree thing seems still true as of 0.52.1:

image deleted to avoid wasting people's bandwidth

ghost avatar Dec 30 '21 23:12 ghost

Your screenshot doesn't depict the bug, it is worked around by r_clear being enabled, well, it is less obvious with r_clear being enabled, and the screenshot is too low resolution to notice the bug precisely, the bug only affects the tip of one branch, you'll need to zoom-in more.

Anyway the bug is still there

illwieckz avatar Dec 31 '21 00:12 illwieckz

Then I'm failing to understand the problem.

ghost avatar Dec 31 '21 00:12 ghost

This may help: https://imgsli.com/ODgwMjI

illwieckz avatar Dec 31 '21 00:12 illwieckz

please upload images instead of linking them, for future reviewers. Also, which side should be the good one?

ghost avatar Dec 31 '21 00:12 ghost

Those are the images in first post, just reuploaded with a slider.

illwieckz avatar Dec 31 '21 00:12 illwieckz

/devmap station15 /setviewpos 349 -1031 -143 159 32

This part must not be pitch black, the fact it's renderer pitch black is because it's worked around with r_clear 1, otherwise nothing would be painted at all.

nolightmap bug

If you /set r_clear 0 and move a bit, you'll see the void.

illwieckz avatar Dec 31 '21 00:12 illwieckz

This is probably related to the fact those two models are colliding.

illwieckz avatar Dec 31 '21 01:12 illwieckz