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

Task/2024 07 camfrust meshes #1354

Merged
merged 7 commits into from
Aug 12, 2024
Merged

Task/2024 07 camfrust meshes #1354

merged 7 commits into from
Aug 12, 2024

Conversation

emily-howell
Copy link
Member

No description provided.

…_blueprint.py" directly into runtime rendering filters.
1. camera_frustum: line mesh showing the view frustum of the camera.
2. clipping_planes: quad mesh of the camera_frustum that will allow users to determine what, if anything, is not within the camera view.
3. scene_bounds: line mesh that shows the bounding box around the original scene.
@emily-howell emily-howell self-assigned this Aug 8, 2024
Copy link
Member

@cyrush cyrush left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great -- two small suggestions to change debug statement text.
The test cases exercise this well!

if(n["runtimes/ascent/vtkm/status"].as_string() == "disabled")
{
ASCENT_INFO("Ascent support disabled, skipping 3D default"
"Pipeline test");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's adjust this message to reflect the name of the test


EXPECT_TRUE(conduit::blueprint::mesh::verify(data,verify_info));

ASCENT_INFO("Testing 3D Rendering with Default Pipeline");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's adjust this message to reflect the name of the test

@cyrush
Copy link
Member

cyrush commented Aug 9, 2024

Note: If you push updates, or merge develop back into your branch and push, the cuda CI test cases should now pass.

@cyrush
Copy link
Member

cyrush commented Aug 12, 2024

We are green, looks good + thanks again!

@cyrush cyrush merged commit 6b20d2c into develop Aug 12, 2024
22 checks passed
@cyrush cyrush deleted the task/2024_07_camfrust_meshes branch August 12, 2024 14:24
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