gongpha / gdQmapbsp

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

Doors platforms triggers #13

Open victorbstan opened 1 year ago

victorbstan commented 1 year ago

Why?

Doors, platforms and triggers needed some updates to bring their feature set closer to parity with Quake game features

What?

victorbstan commented 1 year ago

Note, platforms should probably not inherit from doors

victorbstan commented 1 year ago

This has a lot of changes and the more I review it, the more I think I want to refactor it...

gongpha commented 1 year ago

I confirmed this PR fixes a lot of movable brushes (e.g. e1m1, e1m8) and makes doors open properly from the map logic.

Some problematic brushes (most of them are platform brushes, e.g. e4m2, that one platform brush under the water pool) are still broken, though. (According to #15)

victorbstan commented 11 months ago
victorbstan commented 11 months ago

As far as I know this is mostly feature complete for plats, doors, secret doors and buttons.

TODO (in the future :))

gongpha commented 11 months ago

Are these doors supposed to be closed after leave them ?

https://github.com/gongpha/gdQmapbsp/assets/13400398/726e3ef8-8019-4728-a591-464568e46e7c

victorbstan commented 11 months ago

Are these doors supposed to be closed after leave them ? Godot_v4.0.3-stable_win64.2023-07-16.01-54-51-425.mp4

What map is this? (Found it, start map)

victorbstan commented 11 months ago

Are these doors supposed to be closed after leave them ? Godot_v4.0.3-stable_win64.2023-07-16.01-54-51-425.mp4

Last commit should fix this, and a couple of other door related bugs

victorbstan commented 11 months ago

Found a bug that exists in master as well regarding non axially aligned buttons (fixed by latest commit) - see E3M4 for example button

Screenshot 2023-07-15 at 5 23 42 PM

fixed by 694bd2c

victorbstan commented 11 months ago

https://github.com/gongpha/gdQmapbsp/assets/315297/8a9a56e1-2372-410e-ae75-c612a488c0f7

fixed button add calc

victorbstan commented 11 months ago

Latest updates:

gongpha commented 11 months ago

trigger_onlyregistered shows a message for unregistered users while I'm in registered mode.

image

victorbstan commented 11 months ago

Fixed registered message regression problem, caused by wanting to show trigger once messages

gongpha commented 11 months ago

I saw you fixed some func_train issues. Notice 2 magic gate cubes in E1M2 are now moving synchronously to the timer. I hope you can fix these remaining issues :


The moving speed is slightly slow. Notice the platform moves through the wood platforms :

https://github.com/gongpha/gdQmapbsp/assets/13400398/6732e255-fbc2-4183-a5b5-aa05de1fbd38

They're ping-pong looped on some func_train that has no looping :

https://github.com/gongpha/gdQmapbsp/assets/13400398/76923a6e-b09c-42e8-bad0-1fa4e48d583b

victorbstan commented 11 months ago

I'll take a look at fixing those. There's probably other bugs, and I plan on fixing and implementing all missing entity features, at least to a state where it's playable, if not absolutely the same. But it might not all be in this one PR, since this one is getting quite large and out of hand lol!

victorbstan commented 11 months ago

Platforms now should move to correct time/duration and wait according to wait settings for each path corner.

gongpha commented 10 months ago

Strangely sounds playback when touching doors.

https://github.com/gongpha/gdQmapbsp/assets/13400398/186d1698-0346-429b-9674-503922c9c1cd