Skip to content
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

texture3d testing, fixes, refactoring, coverage #3699

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Dec 5, 2022

  • More extensive testing of texture3d: handle vs filename, subimages,
    more than 4 channels, closest sample interpolation.

  • Fixes for tex3d > 4 chan that turned out to be broken.

  • Some rearrangement for the >4 channel case

  • Refactor and simplify accum3d_sample_bilinear so multiple data types all use the same templated code path.

  • This all reduces the amount of untested code in OIIO by another 0.4%.

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
@lgritz lgritz merged commit 78ee5e0 into AcademySoftwareFoundation:master Dec 8, 2022
@lgritz lgritz deleted the lg-texture3d branch December 12, 2022 23:27
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jan 6, 2023
…ation#3699)

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jan 6, 2023
…ation#3699)

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jan 6, 2023
…ation#3699)

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jan 7, 2023
…ation#3699)

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jan 7, 2023
…ation#3699)

* More extensive testing of texture3d: handle vs filename, subimages,
  >4 channels, closest sample interpolation.

* Fixes for tex3d > 4 chan that turned out to be broken.

* Some rearrangement for the >4 channel case

* Refactor and simplify accum3d_sample_bilinear so multiple data types
  all use the same templated code path.

* This all reduces the amount of untested code in OIIO by another 0.4%.
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.

1 participant