-
Notifications
You must be signed in to change notification settings - Fork 78
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
[USDU-304] Add performance Tests #347
Draft
lee-aandrew
wants to merge
72
commits into
dev
Choose a base branch
from
add-Performance-Tests
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 32 commits
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
9a850d1
Adds Performance Test Project and separates Functional tests and perf…
lee-aandrew bb09419
Add changes to .gitignore ignoring Usd-Developement-Performance's lib…
lee-aandrew dcdeaa2
Adds project settings.asset file change
lee-aandrew 292f5d5
Adds placeholder performance test
lee-aandrew 9e4cf7e
Testing change for upm-ci.yml
lee-aandrew fc5fe47
utr command fix
lee-aandrew 1a689a7
utr command fix
lee-aandrew 9411968
utr command fix
lee-aandrew c3ffba6
utr upm-ci fix
lee-aandrew 8816e26
utr upm-ci fix
lee-aandrew f90e49d
utr upm-ci fix
lee-aandrew 028cde9
utr upm-ci fix
lee-aandrew 8d52eca
utr upm-ci fix
lee-aandrew e8111d9
utr upm-ci fix
lee-aandrew cf4e1dd
utr upm-ci fix
lee-aandrew dcf052c
utr upm-ci fix
lee-aandrew 8237964
utr upm-ci fix
lee-aandrew 0110663
utr upm-ci fix
lee-aandrew fd86151
adding placeholder performance test case
lee-aandrew 49c4d84
fixing platform.version -> editor.version
lee-aandrew 6a28786
removing com.unity.feature.development pacakge from performance project
lee-aandrew 94ea83c
adds chmod to mac utr
lee-aandrew 61d2ef6
adds performance test package to dependency & attempt to fix mac vers…
lee-aandrew d826683
fixing upm-ci.yml
lee-aandrew 71f3527
update package for performance projects, add --clean-library for utr …
lee-aandrew 3393872
Removes Clean library option
lee-aandrew 159fdce
apply clean library to only the performance test
lee-aandrew a2ba2dd
removes package com.unity.coding since its a known issue to be errori…
lee-aandrew b742281
Point instancer performance test
lee-aandrew 514e3e2
enable 10k larger test file for import as object test cases
lee-aandrew 3bdab81
fix formatting issue and remove changes to the functional test project
lee-aandrew f580cfa
Remove unnecessary changes to the asmdef files
lee-aandrew cedb2d8
Applying PR suggestions
lee-aandrew 361553f
fix formatting issue
lee-aandrew 292f785
fix yamato upm-ci.yml if statement
lee-aandrew 996f823
Merge branch 'dev' into add-Performance-Tests
lee-aandrew 6351a81
fix directory change in upm-ci.yml
lee-aandrew 2310374
Merge branch 'add-Performance-Tests' of https://github.com/Unity-Tech…
lee-aandrew 7519af4
Change location of performance test to the project folder and out of …
lee-aandrew cecc7f9
removing artifacts folder
lee-aandrew cd283d6
Decouples Performance tests from the functional tests - Moves USDInit…
lee-aandrew c4b7a8b
Ignores two unstable test cases until a fix has been found
lee-aandrew 464ea5b
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 7af60ed
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 20dc1c7
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 4236bd0
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 2d14b87
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew bb60d3e
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 9336edf
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 5695de5
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew 3b0e137
Merge branch 'dev' of https://github.com/Unity-Technologies/usd-unity…
lee-aandrew fe2c564
Adds Performance Test Project and separates Functional tests and perf…
lee-aandrew 08dd122
Add changes to .gitignore ignoring Usd-Developement-Performance's lib…
lee-aandrew 945b30c
Adds project settings.asset file change
lee-aandrew 4bc4f6e
Adds placeholder performance test
lee-aandrew 2e73bc7
utr command fix
lee-aandrew fabe5bb
adding placeholder performance test case
lee-aandrew 148474d
removing com.unity.feature.development pacakge from performance project
lee-aandrew b7d28ca
adds performance test package to dependency & attempt to fix mac vers…
lee-aandrew 3692ab7
update package for performance projects, add --clean-library for utr …
lee-aandrew 6bbb904
removes package com.unity.coding since its a known issue to be errori…
lee-aandrew 8d43a0e
Point instancer performance test
lee-aandrew d1e1cd5
enable 10k larger test file for import as object test cases
lee-aandrew 4ce65ff
fix formatting issue and remove changes to the functional test project
lee-aandrew da706dc
Remove unnecessary changes to the asmdef files
lee-aandrew 3d9ab53
Applying PR suggestions
lee-aandrew 88b656c
fix formatting issue
lee-aandrew 0f50c82
Change location of performance test to the project folder and out of …
lee-aandrew 065bb34
removing artifacts folder
lee-aandrew 7c9d53b
Decouples Performance tests from the functional tests - Moves USDInit…
lee-aandrew c72d9d9
fixes merge issue with rebase
lee-aandrew 5774cb0
Merge branch 'add-Performance-Tests' of https://github.com/Unity-Tech…
lee-aandrew File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,10 +8,14 @@ test_platforms: | |
type: Unity::VM | ||
image: package-ci/win10:stable | ||
flavor: b1.large | ||
utr_install: curl -s https://artifactory.prd.it.unity3d.com/artifactory/unity-tools-local/utr-standalone/utr.bat --output utr.bat | ||
utr_performance_test: utr --suite=editor --clean-library --category="Performance" --testproject=TestProject/Usd-Development-Performance --editor-location=.Editor --report-performance-data | ||
- name: mac | ||
type: Unity::VM::osx | ||
image: package-ci/mac:stable | ||
flavor: m1.mac | ||
utr_install: curl -s https://artifactory.prd.it.unity3d.com/artifactory/unity-tools-local/utr-standalone/utr --output utr; chmod +x utr | ||
utr_performance_test: ./utr --suite=editor --clean-library --category="Performance" --testproject=TestProject/Usd-Development-Performance --editor-location=.Editor --report-performance-data | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As above, |
||
test_backends: | ||
- name: il2cpp | ||
editor: 2020.3 | ||
|
@@ -44,7 +48,7 @@ test_{{ platform.name }}_{{ editor.version }}: | |
flavor: {{ platform.flavor}} | ||
commands: | ||
- npm install upm-ci-utils@latest -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm | ||
- upm-ci package test --unity-version {{ editor.version }} --package-path package/com.unity.formats.usd --extra-create-project-arg=-upmNoDefaultPackages --extra-utr-arg="--api-profile=NET_4_6" | ||
- upm-ci package test --unity-version {{ editor.version }} --package-path package/com.unity.formats.usd --extra-create-project-arg=-upmNoDefaultPackages --extra-utr-arg="--api-profile=NET_4_6 --category=!Performance" | ||
artifacts: | ||
logs: | ||
paths: | ||
|
@@ -56,7 +60,29 @@ test_{{ platform.name }}_{{ editor.version }}: | |
|
||
{% for editor in test_editors %} | ||
{% for platform in test_platforms %} | ||
test_recorder{{ platform.name }}_{{ editor.version }}: | ||
test_performance_{{ platform.name }}_{{ editor.version }}: | ||
name : Performance Test {{ editor.version }} on {{ platform.name }} | ||
agent: | ||
type: {{ platform.type }} | ||
image: {{ platform.image }} | ||
flavor: {{ platform.flavor}} | ||
commands: | ||
- unity-downloader-cli -u {{ editor.version }} -c editor | ||
- {{ platform.utr_install }} | ||
- {{ platform.utr_performance_test }} | ||
artifacts: | ||
logs: | ||
paths: | ||
- "upm-ci~/test-results/**/*" | ||
dependencies: | ||
- .yamato/upm-ci.yml#pack | ||
{% endfor %} | ||
{% endfor %} | ||
|
||
|
||
{% for editor in test_editors %} | ||
{% for platform in test_platforms %} | ||
test_recorder_{{ platform.name }}_{{ editor.version }}: | ||
name : Test Recorder{{ editor.version }} on {{ platform.name }} | ||
agent: | ||
type: {{ platform.type }} | ||
|
@@ -108,7 +134,8 @@ test_trigger: | |
{% for editor in test_editors %} | ||
{% for platform in test_platforms %} | ||
- .yamato/upm-ci.yml#test_{{platform.name}}_{{editor.version}} | ||
- .yamato/upm-ci.yml#test_recorder{{platform.name}}_{{editor.version}} | ||
- .yamato/upm-ci.yml#test_recorder_{{platform.name}}_{{editor.version}} | ||
- .yamato/upm-ci.yml#test_performance_{{platform.name}}_{{editor.version}} | ||
{% endfor %} | ||
{% endfor %} | ||
{% for backend in test_backends %} | ||
|
@@ -118,6 +145,7 @@ test_trigger: | |
{% endfor %} | ||
- .yamato/formatting.yml#formatting | ||
|
||
|
||
nightly_test_trigger: | ||
name: Nightly tests Trigger | ||
triggers: | ||
|
Binary file added
BIN
+5.04 MB
TestProject/Usd-Development-Performance/.vs/Usd-Development-Performance/v16/.suo
Binary file not shown.
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
{ | ||
"version": "1.0", | ||
"components": [ | ||
"Microsoft.VisualStudio.Workload.ManagedGame" | ||
] | ||
} |
Empty file.
8 changes: 8 additions & 0 deletions
8
TestProject/Usd-Development-Performance/Assets/Resources.meta
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
45 changes: 45 additions & 0 deletions
45
TestProject/Usd-Development-Performance/Packages/manifest.json
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
{ | ||
"dependencies": { | ||
"com.unity.analytics": "3.6.12", | ||
"com.unity.ext.nunit": "1.0.6", | ||
"com.unity.formats.usd": "file:../../../package/com.unity.formats.usd/", | ||
"com.unity.ide.rider": "3.0.16", | ||
"com.unity.ide.visualstudio": "2.0.16", | ||
"com.unity.recorder": "3.0.3", | ||
"com.unity.test-framework": "1.1.31", | ||
"com.unity.test-framework.performance": "2.8.1-preview", | ||
"com.unity.modules.ai": "1.0.0", | ||
"com.unity.modules.animation": "1.0.0", | ||
"com.unity.modules.assetbundle": "1.0.0", | ||
"com.unity.modules.audio": "1.0.0", | ||
"com.unity.modules.cloth": "1.0.0", | ||
"com.unity.modules.director": "1.0.0", | ||
"com.unity.modules.imageconversion": "1.0.0", | ||
"com.unity.modules.imgui": "1.0.0", | ||
"com.unity.modules.jsonserialize": "1.0.0", | ||
"com.unity.modules.particlesystem": "1.0.0", | ||
"com.unity.modules.physics": "1.0.0", | ||
"com.unity.modules.physics2d": "1.0.0", | ||
"com.unity.modules.screencapture": "1.0.0", | ||
"com.unity.modules.terrain": "1.0.0", | ||
"com.unity.modules.terrainphysics": "1.0.0", | ||
"com.unity.modules.tilemap": "1.0.0", | ||
"com.unity.modules.ui": "1.0.0", | ||
"com.unity.modules.uielements": "1.0.0", | ||
"com.unity.modules.umbra": "1.0.0", | ||
"com.unity.modules.unityanalytics": "1.0.0", | ||
"com.unity.modules.unitywebrequest": "1.0.0", | ||
"com.unity.modules.unitywebrequestassetbundle": "1.0.0", | ||
"com.unity.modules.unitywebrequestaudio": "1.0.0", | ||
"com.unity.modules.unitywebrequesttexture": "1.0.0", | ||
"com.unity.modules.unitywebrequestwww": "1.0.0", | ||
"com.unity.modules.vehicles": "1.0.0", | ||
"com.unity.modules.video": "1.0.0", | ||
"com.unity.modules.vr": "1.0.0", | ||
"com.unity.modules.wind": "1.0.0", | ||
"com.unity.modules.xr": "1.0.0" | ||
}, | ||
"testables": [ | ||
"com.unity.formats.usd" | ||
] | ||
} |
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To report data to Observer I think you also need to pass a project ID, ie:
--performance-project-id=com.unity.formats.usd