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

Release 1.1.0 #16

Merged
merged 5 commits into from
Jan 4, 2024
Merged

Release 1.1.0 #16

merged 5 commits into from
Jan 4, 2024

Conversation

alifeee
Copy link
Owner

@alifeee alifeee commented Nov 26, 2023

closes #13, #14

  1. ignore file read error 3
    this might introduce problems in future if file read error is encountered for a different reason other than "was reading file when Factorio tried to write it"
  2. Allow players to use multiple surfaces (i.e., be in space and not hear each other)
    to be tested. sets z coordinate to 1000x surface index. Max distance on Mumble is 200 units of distance, so you shouldn't be able to hear cross-surface

@alifeee alifeee added the enhancement New feature or request label Nov 26, 2023
@alifeee alifeee added this to the v1.1.0 milestone Nov 26, 2023
@alifeee alifeee self-assigned this Nov 26, 2023
to also account for
"mod installed but game not launched yet"
as position file is only created on game launched
@alifeee
Copy link
Owner Author

alifeee commented Nov 26, 2023

1 above seems to work, but sometimes (with Mumble 1.5) the PA information flashes blank

(imagine this screen but everything resets to 0 for a single frame)

image

I am unsure why this happens, or what effect this could have on the audio. It might manifest as a short-and-loud BLIP of being able to hear everybody's audio, or multiple blips as different people's contexts change, and are thus audible to everyone for a single frame.

@alifeee
Copy link
Owner Author

alifeee commented Nov 26, 2023

I cannot test 2 above without more people. The screenshot from the above comment shows that it displays the coordinates correctly, though.

@alifeee
Copy link
Owner Author

alifeee commented Jan 4, 2024

I tested this with 1 other person 👍

@alifeee alifeee merged commit a47b135 into master Jan 4, 2024
3 checks passed
@alifeee alifeee deleted the release-1.1.0 branch January 4, 2024 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow players to use multiple surfaces (i.e., be in space and not hear each other) Ignore file read error 3
1 participant