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

Add preliminary framework for generating X3D (and eventually RTT) mesh files. #1090

Merged
merged 4 commits into from
Jul 19, 2021

Conversation

RyanWollaeger
Copy link
Contributor

@RyanWollaeger RyanWollaeger commented Jul 18, 2021

Background

  • Some mesh file generation capability will facilitate testing Draco's supported mesh parsers.

Purpose of Pull Request

  • Add an orthogonal-structured class mesh type for testing.
  • Add an X3D mesh file generator, agnostic of mesh structure or dimension.
  • Concerns Redmine Issue #2466

Description of changes

  • Add src/mesh/python subdirectory.
  • Add mesh_type file intended to contain python mesh classes.
  • Add orth_2d_mesh class to mesh_types: orthogonal quad 2D data.
  • Add x3d_generator, which creates X3D files from mesh objects.
  • Import mesh_types into x3d_generator (for the mesh objects).
  • Add orth_3d_mesh class stub for eventual 3D mesh file generation.

Notes:

  • An equivalent rtt_generator can be added to take mesh object data
    to the RTT mesh file format.
  • Consider using Python 3.7 dataclasses for mesh classes.

Status

+ Add src/mesh/python subdirectory.
+ Add mesh_type file intended to contain python mesh classes.
+ Add orth_2d_mesh class to mesh_types: orthogonal quad 2D data.
+ Add x3d_generator, which creates X3D files from mesh objectes.
+ Import mesh_types into x3d_generator (for the mesh objects).
+ Add orth_3d_mesh class stub for eventual 3D mesh file generation.

Notes:
- An equivalent rtt_generator can be added to take mesh object data
  to the RTT mesh file format.
- Consider using Python 3.7 dataclasses for mesh classes.
@RyanWollaeger
Copy link
Contributor Author

@brryan Maybe the Voronoi mesh can be introduced as a class with the relevant data in this new python subdirectory, so we can then add an rtt_generator.py (following x3d_generator.py) which can output the format. Let me know if this seems off-base.

@codecov
Copy link

codecov bot commented Jul 18, 2021

Codecov Report

Merging #1090 (05d4eb7) into develop (fd7305e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #1090   +/-   ##
=======================================
  Coverage     88.7%   88.7%           
=======================================
  Files          374     374           
  Lines        18591   18591           
=======================================
  Hits         16495   16495           
  Misses        2096    2096           

+ Avoid ambiguity of numpy import.
+ Correct formatting of comments, dictionary, key-value arguments.
@KineticTheory
Copy link
Collaborator

I'm ready to merge this PR. @brryan @kgbudge Do you have any comments for @RyanWollaeger?

@KineticTheory KineticTheory merged commit bccfce4 into lanl:develop Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants