eventhorizon5 / skyscraper

The Skyscraper Project (Skyscrapersim) - an open source building, city, and elevator simulator
https://www.skyscrapersim.net
GNU General Public License v2.0
28 stars 7 forks source link

Directional Message Issue #23

Closed MultiMonorail closed 1 year ago

MultiMonorail commented 1 year ago

This issue affects elevators that include the LevelingOpen Command with a value more than 0: If an elevator has multiple calls from within the elevator going down, the elevator will still say that is going up.

eventhorizon5 commented 1 year ago

I made a change which appears to have fixed this problem. It was an issue in the GetArrivalDirection() elevator function where the up/down check would compare the same floor with both and chime up by default, I had it check the active queue direction instead if this situation pops up. The code in that function is unnecessarily complicated so I'm hoping that it doesn't break more.