Texture Analysis Results to UI

Unity-Technologies / ProjectAuditor

Project Auditor is an experimental static analysis tool for Unity Projects.

Other

806 stars 66 forks source link

Texture Analysis Results to UI #108

Closed justinkiffiak-SM closed 1 year ago

justinkiffiak-SM commented 2 years ago

Queried the project's asset database for Textures and reported the (select) properties in the Project Auditor UI.

justinkiffiak-SM commented 1 year ago

Unit Tests are working, even when RenderTextures are present in the project, and the test images are correctly saved now.

justinkiffiak-SM commented 1 year ago

Checks for

I added some comments. In addition, I think we should:

improve the screenshot, it would be good to have the Texture module in its own window and let's show more rows.

let's add a few more tests, check when Read/Write and/or Mipmaps is enabled. Can we add an override for the current platform and check that too?

Updates:

Improved Screenshot has been submitted
Read/write & Mipmaps enabled are already in the original (current) unit tests.
- Current platform test has been added as well.

justinkiffiak-SM commented 1 year ago

Requested Review. Also found a bug & fixed the unit test for textureTests[0].customProperties[2] to now use a non-hard-coded value, and instead checkg the value pre-test, which allows it to function correctly on future versions (was incorrect for Unity 2021.3 and showing in unit tests there. Corrected now.)

justinkiffiak-SM commented 1 year ago

Fixed Exception in Texture module, 'cause by incorrectly adding in Mute options for the textures ViewDescription in additiion to the regular search filters.