-
Notifications
You must be signed in to change notification settings - Fork 510
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
Brand New Announcer for New Frontier! (Attempt #2) #1740
Brand New Announcer for New Frontier! (Attempt #2) #1740
Conversation
i was really hyped for this, and I felt we were nearly exactly over the target when we left off. I'll give these another listen this week and, if its still at all possible, I may write 2-4 more lines as a request for some future events I have coming up in the near-term |
I think he's more than happy to record more lines for you as you desire, as when I brought it up to him his response was and I quote: |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
…-station-14 into new-voicelines
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Due to several months of inactivity and no sign of anyone taking charge to get this merged, I will be closing this PR primarily so a seemingly dead PR doesn't appear as active. This does not mean the changes will never be incorporated. If anyone wants to fix up any outstanding issues and bring this to the finish line, comment to have this re-opened. |
It has been inactive because we have been awaiting a review for this for some time now, but have yet to hear anything back. The merge conflicts are something I can easily fix, but there's not much reason to fix it until we know what needs to be added or changed. 😅 Definitely not dead! |
About the PR
This is a second-attempt PR (the first being #1136 ) after updating my repository to continue work on this addition. It is mostly a carbon copy so I will quote most of the previous PR and highlight any changes.
The original work-in-progress sound files were left in this PR alongside the newest, finalized versions.
These were left in to give an option to swap the newest files out for the older versions by simply renaming the folders, as well as a sort of an example to anyone else who may wish to record their own version for Frontier to expand the Announcer pool.
If there are causes for concern about bloat, the folder size for the outdated versions is a total of 4.23 mb.
Why / Balance
Technical details
I have not yet tested this in full, this PR in its current state exists for feedback that it is looking okay and to be certain that this is in fact a desired change before too much more effort is put into it only to be shot down.I'll need to retest everything since many of the file locations were altered, as well as some noticeable changes on how the Bluespace events are handled now. But I wanted to get this PR set up ahead of time to put it on the list of things to be reviewed.
Some of the events in the NF events.yml file weren't standardized in some of their variables, likely left over from whoever would fiddle with them upstream. I standardized a few of them in the general file, if someone notices any issues with it I can correct it.
I also re-included the RandomSentience event that was removed by upstream, but commented it out in case Frontier wishes to keep it gone. I did this due to Andy having recorded a specialty dialogue line for its event - but if there is no interest in this it can be removed.
I realized it isn't stated anywhere in the quoted sections, but more than half of the sound files included are brand new sound files that are specific and unique to Frontier - several of them involving the Bluespace events. If there are any additional new or specific events or dialogues that are of interest I can speak with Andy about getting them recorded.
Additionally, PocketSizedAndy has been toying around with the coding to implement a way for the server to choose from a selection of announcers randomly at the round start to allow some variance. If anyone is interested in helping him understand that I encourage them to reach out to him in the discord thread.
Oh. Also included a silly sound reel Andy recorded that a different mod/admin had expressed interest in including in the RoundEnd sound pool, but may never have made its way into it. I believe it's listed in that discord channel and if not I can submit it there if desired.
How to test
If Andy or I don't respond quickly to commentary here please do not hesitate to reach out to use by pinging us on Discord. I'll likely respond quicker than he does, but we both are interested in seeing this added to the community!
Media
The most recent zip file contains the audio that is set to be used for the announcements, found specifically here:
https://discord.com/channels/1123826877245694004/1221648600082157648/1266542077542531102
Breaking changes
Changelog
🆑