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

SAR missions check unoccupied cabin number, but should check overall number #5868

Open
bszlrd opened this issue Jul 17, 2024 · 2 comments
Open
Assignees

Comments

@bszlrd
Copy link
Contributor

bszlrd commented Jul 17, 2024

Observed behaviour

I was unable to take two SAR missions simultenaously:

  • One was close, and required the transfer of one pilot.
  • The other required bringing back three people from a wreck.
    I took the first one, and wanted to take the second one, figuring that I will drop off the pilot, then head for the other ship.
    But I was refused from the second, because I did not have three unoccupied cabins due to having that pilot on board. (And the ship has a limit of three cabins)

Expected behaviour

I think I should have been able to take both missions. The mission should note refuse if there are unoccupied cabins, since it can't know if those cabins will still be occupied when the player gets to the target. Instead I think it could still give the mission, if there are enough cabins on the ship, no matter their occupancy, and notify that there might be not enough room.
From another POV: the game shouldn't hold the hands of the player. Let them fail missions if they aren't paying attention. (Of course for that it would be nice to be able to cancel missions)

Steps to reproduce

My pioneer version (and OS):
2024 0710

My output.txt (required) and game save (optional, but recommended)

@Web-eWorks
Copy link
Member

Passenger logic is getting addressed in #5734 - this should be evaluated after that PR is merged.

@Web-eWorks Web-eWorks self-assigned this Nov 4, 2024
@Web-eWorks
Copy link
Member

Still valid after equip-v2 merge. This is a good low-hanging fruit for a first-time contributor though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants