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.

Sign up for GitHub

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

Jump to bottom

Recording status alert #41

Closed
veger opened this issue Jun 26, 2023 · 12 comments
Closed

Recording status alert #41

veger opened this issue Jun 26, 2023 · 12 comments
Assignees
@veger
Labels
enhancement New feature or request

Comments

@veger
Copy link
Owner

veger commented Jun 26, 2023

From https://mods.factorio.com/mod/TLBE/discussion/6498a5ed070072bbaa7d044c

recording status alert (which can be turned off) including - not recording, recording, panning
@veger veger added the enhancement New feature or request label Jun 26, 2023
@veger
Copy link
Owner Author

veger commented Jun 26, 2023

Status reports could also include (change) of tracker?

@veger
Copy link
Owner Author

veger commented Jun 26, 2023

Easiest would be to print (chat) messages when something changes, but I guess it would (also) be nice to have it on screen (like FPS/UPS, evo, etc info)?

Maybe implement both and make it configurable by the user to have it off, in chat, or on-screen

@theotherbigfoot
Copy link

I really like this idea. Maybe all it needs is a series of lights in the main timelapse menu. 'Normal recording', 'transition underway', 'transition complete'

@drdozer
Copy link
Contributor

drdozer commented Jun 28, 2023

is it possible to do this with a reskin of the "pause all cameras" button? So it would switch between icons to indicate recording/transition status? I don't know enough to know if buttons can have their icons changed during the game.

@veger
Copy link
Owner Author

veger commented Jun 28, 2023

That would be nice, but I doubt it is possible as prototypes are constructed/managed in the 'data stage' only.

@veger
Copy link
Owner Author

veger commented Jun 29, 2023 •
edited
Loading

Experimented with custom alerts, but those require an entity to get tied to (can fake one, but still). And they disappear after a while (so we'd need to spam them over and over)
Not a very nice/clean solution

@veger veger self-assigned this Jul 4, 2023
@veger
Copy link
Owner Author

veger commented Jul 4, 2023

Decided to make use of Stats GUI.
It is flexible and looks clean (in code and in the game)

I could not think of a good indicator that actually works (having a 'lamp' in some corner of the screen didn't look nice in my mind, especially with multiple cameras). Adding support for Stats GUI is easy and could be replaced when a better (graphical) UI have been figured out.

@veger
Copy link
Owner Author

veger commented Jul 4, 2023 •
edited
Loading

Screenshot_20230704_120601

@drdozer Feel free to try it from branch issue-41_show-camera-stats.
If you have/had something different in mind, let me know 😄

@veger veger mentioned this issue Jul 4, 2023
Merged
@veger
Copy link
Owner Author

veger commented Jul 20, 2023

The feature will be included in the next release

@drdozer
Copy link
Contributor

drdozer commented Jul 31, 2023

Is your policy to leave tickets open until the release, or close them when the final merge happens? I don't much mind which but would like to know what the policy is.

@veger
Copy link
Owner Author

veger commented Jul 31, 2023

until after the release

@veger
Copy link
Owner Author

veger commented Aug 1, 2023

Released in v1.5.0

@veger veger closed this as completed Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@veger veger

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@veger @drdozer @theotherbigfoot