Merging UE5.6 to master #703
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… support (#647) (#651) * Add 4.27 support back for latency testing * Save webrtc stats to csv and auto download * Deep copy stats object * Update record parsing logic to handle missing field when generating the csv * Update session test to handle additional latency stats * Fixing path setup in the windows platform scripts. (#594) (#595) (cherry picked from commit 152abd6) (cherry picked from commit 1dedc87) * Fixing more path issues in platform_scripts (#604) * feat: Adding script tests where the scripts are run multiple times. This tests that the scripts still work when things have already been installed. * fix: Fixing typo in github action. * gimme a break. * Adding mutliple script runs for other OSs * Testing node on the runner * Trying to figure out what node is doing what. * Trying to change the way node version is detected. * Messy * The additions to the script really didn't catch anything. Could be worth revisiting how we handle the local node install. * Removing debug print. (cherry picked from commit 2321fa9) * refactor: Better handling of node in Linux scripts. (#607) Updated the way the bash script sets up the environment for node. Node will only be installed if a version meeting the minimum requirements is not found. The new environment should result in a more robust environment setup. (cherry picked from commit f27bace) * Fixing a small issue with how path was setup previously. (#611) If the node install was installed locally previously the script would not pick that up and try to install again. It would fail because the directory already exists and then the path would not get properly setup for the local install of node. This change sets PATH to point to the local install anyway. If it doesn't exist it doesn't matter, if it does exist we use the local install. (cherry picked from commit 00aaa52) * Added disable button when session test is running * Added new flag ?LatencyCSV to enable the session test and its UI * Added changeset for 4.27 support and session test feature --------- (cherry picked from commit c61fd7b) Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> Co-authored-by: Will Belcher <william.belcher@xa.epicgames.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com>
* fix(wilbur): Fixing issues with passing arguments to wilbur. (Linux) peer_options was always being built and passed to wilbur with the platform_scripts. This meant that if you wished to pass your own peer_options you could not use the platform_scripts. This fixes that by checking that peer_options has not already been supplied as a parameter. Additionally, there is also now a clear delimeter between script arguments and command arguments. * fix(wilbur): Fixing issues with passing arguments. (Windows) Similar to the linux changes this change should allow --peer_options and --http_root to be passed to the server without being stomped on by the script. * feat(wilbur): Adding --peer_options_file to wilburs cli arguments. This is to make it easier to provide JSON data to the peer_options parameter without having to escape JSON data on the command line. * chore(actions): Updating actions to match the new platform_script args. * fix(actions): Fixed missing comma in argument list. * fix(extras): Updating the docker commands for the new arguments. * chore(changeset): Adding changeset for wilbur. * Update file not found error message --------- (cherry picked from commit 584617b) Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com> Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* chore(platform_scripts): Updating windows help text. * chore(platform_scripts): Updating helper script help text. * chore(platform_scripts): Removed accidental commit. * chore(platform_scripts): Trying to fix windows line endings. (cherry picked from commit 28e8996) Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com>
* Fix GHA where we build against public deps
* Comment out example code
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.