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

testoptix enhancements to make more analogous to testrender #982

Merged
merged 1 commit into from
Mar 14, 2019

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Mar 13, 2019

  • Change -r to -res to match testrender (and for that matter, make -res
    work for testshade, not -g). (In both cases, the old oddballs are
    silent synonyms.)

  • Respond to several additional OSL options, and set them the same
    was as testrender

  • Allow --iters (much like it works in testshade) to make it easy to
    run all the rays many times to make accurate timing easier.

  • Do some of synchronizing of near-identical code between
    testshade, testrender, and testoptix. And synchronize some of the
    analogous command line arguments (e.g., -res for everybody, not -res
    for testrender and -r for testoptix and -g for testsahde).

  • Add empty OPTIMIZEONLY marker files in the optix test directories
    to signify that they don't need -O0 test cases run (currently, such tests
    will fail due to the OptiX path not yet supporting "init ops").

* Change -r to -res to match testrender (and for that matter, make -res
  work for testshade, not -g). (In both cases, the old oddballs are
  silent synonyms.)

* Respond to several additional OSL options, and set them the same
  was as testrender

* Allow --iters (much like it works in testshade) to make it easy to
  run all the rays many times to make accurate timing easier.

Also do a little bit of synchronizing of near-identical code between
testshade, testrender, and testoptix. And synchronize some of the
analogous command line arguments (e.g., -res for everybody, not -res
for testrender and -r for testoptix and -g for testsahde).
@fpsunflower
Copy link
Contributor

LGTM

@lgritz lgritz merged commit 493eccd into AcademySoftwareFoundation:master Mar 14, 2019
@lgritz lgritz deleted the lg-optix branch March 14, 2019 17:57
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.

2 participants