OpenColorIO icon indicating copy to clipboard operation
OpenColorIO copied to clipboard

Fix Typo

Open sobotka opened this issue 3 years ago • 4 comments

Fix minor typo in NamedTransform.

Example output currently:

[...]
Invoked with: kwargs: referenceSpace=<ReferenceSpaceType.REFERENCE_SPACE_DISPLAY: 1>, name='sRGB Display View', description='IEC 61966-2-1:1999 Reference sRGB Display', toReference=<NamedTransform name=Identity Transformalias=
[...]

Note how the toReference line lacks the comma.

sobotka avatar Mar 15 '22 17:03 sobotka

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: sobotka / name: Troy James Sobotka (a6a66dd8b10e0b6abe52d2252145835a0f861a86)

Fix minor typo in NamedTransform.

nadiashomali avatar Mar 15 '22 19:03 nadiashomali

Hi @sobotka & @nadiashomali Thanks for your contribution.

There two points to fix:

  1. The DCO failed.
  2. There is a unit failure:
2: [ 528/1024] [MathUtils / halfs_differ_test                               ] - PASSED
2: [ 529/1024] [MathUtils / clamp                                           ] - PASSED
2: /Users/runner/work/OpenColorIO/OpenColorIO/tests/cpu/NamedTransform_tests.cpp:44:
2: FAILED: oss.str() == "<NamedTransform name=NewName,\n    forward=\n        " "<MatrixTransform direction=forward, fileindepth=unknown, " "fileoutdepth=unknown, matrix=1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1, " "offset=0 0 0 0>>"
2: 	values were '<NamedTransform name=NewName, ,
2:     forward=
2:         <MatrixTransform direction=forward, fileindepth=unknown, fileoutdepth=unknown, matrix=1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1, offset=0 0 0 0>>' and '<NamedTransform name=NewName,
2:     forward=
2:         <MatrixTransform direction=forward, fileindepth=unknown, fileoutdepth=unknown, matrix=1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1, offset=0 0 0 0>>'
2: [ 530/1024] [NamedTransform / basic                                      ] - FAILED
2: [ 531/1024] [NamedTransform / alias                                      ] - PASSED

hodoulp avatar Mar 21 '22 12:03 hodoulp

Hi @sobotka & @nadiashomali. Your finding remains a bug that it would be great to have in the next releases. After my review I found that the fix is not complete but contact us if there is still something unclear.

hodoulp avatar Apr 05 '22 18:04 hodoulp