Skip to content

Maroon-410 Unity 2021 LTS Update#457

Merged
FlorianGlawogger merged 20 commits intodevelopfrom
maintenance/Maroon-410_UnityUpdate
Jan 17, 2024
Merged

Maroon-410 Unity 2021 LTS Update#457
FlorianGlawogger merged 20 commits intodevelopfrom
maintenance/Maroon-410_UnityUpdate

Conversation

@michaelholly
Copy link
Copy Markdown
Collaborator

@michaelholly michaelholly commented Nov 20, 2023

closes #410

@michaelholly michaelholly marked this pull request as draft November 20, 2023 14:57
@michaelholly michaelholly force-pushed the maintenance/Maroon-410_UnityUpdate branch 2 times, most recently from f93622e to 5c05d73 Compare November 20, 2023 15:54
@michaelholly michaelholly changed the title Maroon-410 Unity 2022.3.13 LTS Update Maroon-410 Unity 2021 LTS Update Dec 12, 2023
@michaelholly
Copy link
Copy Markdown
Collaborator Author

There is an unknown build error when trying to build it with Unity 2022

=================================================================
Native Crash Reporting

Got a UNKNOWN while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.

Editor.log

@michaelholly michaelholly force-pushed the maintenance/Maroon-410_UnityUpdate branch from 5c05d73 to b95392a Compare December 12, 2023 10:31
@michaelholly michaelholly marked this pull request as ready for review December 12, 2023 10:31
@FlorianGlawogger
Copy link
Copy Markdown
Collaborator

So the good thing is that the unity-test-runner on self hosted machines issue regarding permissions has been fixed. And almost all tests now pass. The bad thing is that unfortunately not all of the playmode tests pass.

We now have 3 actions where we ran the tests:
Run 1(re-run triggered by @michaelholly )
Run 2
Run 3

What I have found out so far:

  • The log is very long, but by searching the log for the keyword 'failure' you can pretty quickly find the failed tests
  • The failing tests are from the WhenClickLabCategoryExperimentThenLoadScene test type. (i.e. the test that tests the menu navigation from the main menu to the experiment)
  • For all of the 3 test runs, the experiments PointWaveExperiment and TitrationExperiment failed, both because of a timeout.
  • I doubt it's a coincidence that for all 3 runs these 2 tests failed each time? But I am also not sure what those two experiments would have in common. Maybe it is just the order the tests are executed in?
  • I am also not sure what could cause a timeout, as there should be no blocking calls in the test? The test just clicks on a button 3 times, waits for 1 frame after each button press, and then checks if the active scene is the according experiment...
  • To attempt to get more insight, I added another test set (StartExperimentSceneDirectly) that attempts to load each of the experiment scenes directly (without any of the main menu navigation beforehand). But for these tests the PointWave and Titration experiments passed and only the CoulombsLaw experiment failed to load? It timeouted too after 180 seconds. (Only the third run though contains this test)
  • Locally on my PC all tests pass.

Maybe I am missing something or someone else has more expertise regarding those experiments. Maybe we could try adding another yield return null; somewhere, hoping that that solves the problem because the build server is otherwise too slow? But debugging all this is very tedious as the builds take hours.

@michaelholly michaelholly force-pushed the maintenance/Maroon-410_UnityUpdate branch from 4e1d255 to bf25cfe Compare January 15, 2024 14:05
@FlorianGlawogger
Copy link
Copy Markdown
Collaborator

Seems like our problem is the "Thread Wait Time". It appears like it is larger after certain scenes, like after loading the Whiteboard scene, e.g. when loading the MainMenu after the Whiteboard it's Thread Wait Time: 668868.875 ms. I removed the StartExperimentsDirectlyTests again to save some time, and will look over the PR, in the future hopefully we'll figure out why the Thread Wait Time is so long, I did a quick internet search but didn't find anything regarding that for now...

Copy link
Copy Markdown
Collaborator

@FlorianGlawogger FlorianGlawogger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! 👍

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.

Unity 2021 LTS Update

2 participants