-
Notifications
You must be signed in to change notification settings - Fork 71
Develop Anari #1539
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
base: develop
Are you sure you want to change the base?
Develop Anari #1539
Conversation
depts->deps
…ine-DAV/ascent into task/2025_03_warpx_unit_test
…ine-DAV/ascent into task/2025_03_warpx_unit_test
…ine-DAV/ascent into task/2025_03_warpx_unit_test
Initial ANARI development via VTKm/Viskores
| fi | ||
|
|
||
| if ${build_anari}; then | ||
| echo 'set(ENABLE_ANARI ON CACHE BOOL "")' >> ${root_dir}/ascent-config.cmake |
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.
We should only key off of ANARI_DIR, we don't need an extra switch to enable to disable anari.
| option(ENABLE_CUDA_DEBUG_CPU_ONLY "Enable CUDA CPU debugging" OFF) | ||
|
|
||
| # support ANARI rendering backend | ||
| option(ENABLE_ANARI "Build ANARI Support" OFF) |
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.
we dont need addtional option, just key off ANARI_DIR
src/config/ascent_setup_deps.cmake
Outdated
| set(ANARI_DIR ${ASCENT_ANARI_DIR}) | ||
| endif() | ||
|
|
||
| if(EXISTS ${ANARI_DIR}/lib64/cmake/Kokkos/) |
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.
is kokkos a typo here?
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.
bingo! copy/paste typo
| else if(extract_type == "steering") | ||
| { | ||
| filter_name = "steering"; | ||
| } |
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.
why is this guy getting removed?
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.
Weird. I don't know. Probably messed up during a merge or something.
Thanks for catching
remove this check for now while we test and decide how to design things. anari vs. anari_vol & anari_pseudo & anari_points(?) idk all their renderers
No description provided.