Skip to content

Conversation

@CrimRecya
Copy link
Contributor

  • Now you can let passengers quickly enter or leave the transport vehicles (useless for buildings) without queuing. Defaults to [General] -> NoQueueUpToEnter or NoQueueUpToUnload.

In rulesmd.ini:

[General]
NoQueueUpToEnter=false    ; boolean
NoQueueUpToUnload=false   ; boolean

[SOMEUNIT]                ; UnitType
NoQueueUpToEnter=         ; boolean
NoQueueUpToUnload=        ; boolean

@github-actions
Copy link

github-actions bot commented Dec 28, 2024

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@mevitar
Copy link

mevitar commented Feb 1, 2025

This happened when i tried to load PoweredBy= units into a NoQueueUpToEnter=yes and NoQueueUpToUnload=yes transport, then i unpowered them mid load and ordered the transport to unload.
Despite moving, the units were unpowered all this time, until the point that i selected and moved them.

fastloadbug

I'll still recheck if it happens with regular transports.

@mevitar
Copy link

mevitar commented Feb 1, 2025

I'll still recheck if it happens with regular transports.

Yes, it indeed happens with regular transports, and it's not an issue introduced by Phobos.

@Coronia Coronia added the ⚙️T1 T1 maintainer review is sufficient label Mar 19, 2025
Copy link
Contributor

@Coronia Coronia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

worked properly in test, the issue listed below are just some minor stuffs

also, maybe we don't have to follow the QueueUpToEnter variable name, but name this feature as QuickEnter and QuickUnload instead

@CrimRecya
Copy link
Contributor Author

worked properly in test, the issue listed below are just some minor stuffs

also, maybe we don't have to follow the QueueUpToEnter variable name, but name this feature as QuickEnter and QuickUnload instead

Wait a moment, there is currently an issue: when the vehicle to be loaded is surrounded by the transport vehicle within a circle, it cannot be loaded quickly because non moving units will not call UpdatePosition().
Also, regarding the key names, I would like to emphasize that there is no need to wait in line when taking transportation vehicles. Simple key names may not be clear enough, so I would like to hear other people's opinions.

@CrimRecya CrimRecya requested a review from Coronia March 29, 2025 13:08
@CrimRecya CrimRecya merged commit f501559 into Phobos-developers:develop Mar 31, 2025
6 checks passed
DeathFishAtEase added a commit to DeathFishAtEase/Phobos that referenced this pull request Mar 31, 2025
@CrimRecya CrimRecya deleted the develop-FastAccessVehicle branch April 1, 2025 03:45
Coronia pushed a commit that referenced this pull request Oct 8, 2025
#1475
#1602

Extend their logic to the structure.

---------

Co-authored-by: Fly-Star <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix and merge this ⚙️T1 T1 maintainer review is sufficient Tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants