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

Bus Routes #2852

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from
Draft

Conversation

whatston3
Copy link
Contributor

@whatston3 whatston3 commented Feb 3, 2025

About the PR

This PR aims to improve the experience of riding the bus. The bus should be something that is okay to use and not a colossal pain.

Goals of the PR:

  1. All POIs barring ones with onboard hostiles (anomalous lab/geode) and the cargo depots should have bus service.
  2. Apart from Frontier Outpost*, stations should only have one bus docked at a time at most.
  3. The route a particular bus is for, the stops it covers, and its current location should be readily accessible information.
  4. The approximate time the next bus should serve a given station should be readily accessible information.
  5. The amount of time to get from any one station to another should be reasonable.

A prototype is added to store information about bus routes. These store the time buses spend waiting per stop, the time buses spend in transit between stops, the number of the route, and the number of stations needed on the route before another bus is added.

When the NFAdventure system is done populating the round's stations, the bus routes are created, and buses are spawned to run the routes. Each bus is numbered (e.g. "Bus 1A" is the first of several buses serving route 1, "Bus 2" is the only bus serving route 2) and every bus on the same route covers the same stations.

Transit time is decreased to 80 seconds transit time and 60 seconds at a station (down from 50/180).

Current routes are as follows (outwards from Frontier Outpost, looping back to Frontier at each extent)

image

Outstanding issues:

  • Functional bus routes.
  • In-person, which bus is which should be immediately obvious - probably through a screen.
  • Dead simple scheduling PDA app for points 3-4
  • A wallmount poster with examine text for the next bus time?
  • Buses should ideally have route-specific livery - "the red route", "the blue route", "the green route" etc, possibly with matching IFF colours.
  • Frontier Outpost needs docks for these new buses.*

Why / Balance

Frontier Outpost has two major things going for it:

  • Pilotable shuttles.
  • Niche, off-station points of interest.

Pilotable shuttles are in a good spot. POIs are hurting. If we have a way to easily get from one place to the other that doesn't take thirty minutes, you'll probably see more bus use and have a better overall POI experience.

How to test

  1. Currently, 5 buses should spawn (two per route).
  2. Validate that each bus matches the route in the picture by riding the bus to each stop.

Media

Requirements

Breaking changes

Changelog

TBD

@whatston3
Copy link
Contributor Author

Switching to draft once tests run.

@dvir001
Copy link
Contributor

dvir001 commented Feb 3, 2025

The bus was never real

@whatston3 whatston3 marked this pull request as draft February 3, 2025 18:55
@whatston3 whatston3 added S: Awaiting Changes This PR has changes that need to be made before merging and removed S: Untriaged labels Feb 4, 2025
@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Feb 4, 2025
Copy link
Contributor

github-actions bot commented Feb 4, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# FTL Map-Outpost Map - Outpost Map-POI Map - POI Map-Shuttle Map - Shuttle S: Awaiting Changes This PR has changes that need to be made before merging size/M YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants