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

New: Basic route viewer mode for the main program #165

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leezer3
Copy link
Owner

@leezer3 leezer3 commented Jul 20, 2017

This is a basic implementation of a viewer mode for the main openBVE program, eventually designed to replace Route Viewer.

To Use:

  • Set the CurrentOptions.GameMode to developer. (Currently has to be done in-code, no UI option at the minute.

Flaws:

  • No event markers.
  • Game is not paused by default. (Need to figure out the exact controls, so we can essentially just run the sim if people wish to test TOD based effects)
  • Requires Route Viewer overlays porting. (Should be a relatively quick job.....)

@leezer3 leezer3 force-pushed the master branch 3 times, most recently from e1e89a3 to 014a7a1 Compare August 22, 2017 10:15
@leezer3
Copy link
Owner Author

leezer3 commented Sep 4, 2017

This now mostly works as-per Route Viewer.

TODO:

  • Disable unrelated controls / overlays when in developer mode.
  • Camera motion speed is not right when using WASD, check.
  • Implement developer mode into the dropdown bit.
  • Allow hiding of the overlays
  • Merge the main branch, and then specifically disable all hacks when in developer mode. (Developers should find and fix the issues, not rely on workarounds)
  • Station jump is up the creak when using the numpad, check this.

@leezer3
Copy link
Owner Author

leezer3 commented Dec 7, 2017

Another PR which has ended up getting out of sync with master....

Have cherry-picked the commit to render the events, and placed that instead behind a new key option (Set to CTRL+SHIFT+E by default)

I think the next step is to cherry-pick the initial loading commit & then rehack the overlays.

@leezer3
Copy link
Owner Author

leezer3 commented Jan 29, 2019

Revisited this today, a little more work done.

Flaws:

  • Disabling the overlays is crappy code at the minute.
  • Pause needs to be reworked, so that time is paused by default, but we can also change the time, mess with dynamic lighting etc.
  • No key etc. overlays
  • Viewer mode keyconfig?

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

Successfully merging this pull request may close these issues.

1 participant