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

Headless renderer #52

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Headless renderer #52

wants to merge 4 commits into from

Conversation

jpd-de
Copy link
Contributor

@jpd-de jpd-de commented Sep 23, 2024

Added a headless renderer class for rendering without a graphical interface.
For the best result consider to export an appropriate view matrix for your visualization before rendering.

This class can be used analogously to the Visualizer class. An example that renders a whole simulation with given particles, vector fields in their respective lists into an output folders using a given view matrix could look like this.

renderer = znvis.Headless_Renderer(particle_list, vector_fields_list, out_folder, view_matrix)
renderer.render_visualization()

This class offers the option to render without a gui. Works analogously to the Visualizer class.

Currently only a static camera perspective is implemented that can be defined with the view_matrix.
from Headless_Visualizer to Headless_Renderer
Copy link
Member

@SamTov SamTov left a comment

Choose a reason for hiding this comment

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

I would like to have the camera options included in this PR if possible. That way it all works from day one. We also need a test.

renderer_resolution: list = [4096, 2160],
renderer_spp: int = 64,
renderer: Mitsuba = Mitsuba(),
view_matrix: np.ndarray = np.array([[ 1, 0, 0, -100],
Copy link
Member

Choose a reason for hiding this comment

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

Can we make this optional and add some classes to deal with different approaches?

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