Skip to content

Conversation

@NolanBlack
Copy link

@NolanBlack NolanBlack commented Nov 12, 2025

Open questions:

  • Can we standardize the MeshFactory, BCFactory, HexMesh, etc. classes? They are all shared by multiple tesseracts at this point
  • We decided that some copied code is acceptable for now.
  • Are the masks, e.g., dirichlet_mask of dimension (num_nodes, n_dof)? or (num_masked_points, n_dof)? or (num_masked_points,)?
  • The masks are of size num_masked_points

Relevant issue or PR

Description of changes

  • Tesseract-PyMAPDL accepts a HexMesh and generic BC descriptors in its InputSchema

Testing done

  • See the test.py script.

@NolanBlack NolanBlack changed the title Updating the PyMAPDL InputSchema feat: Updating the PyMAPDL InputSchema for HexMesh Nov 17, 2025
@NolanBlack NolanBlack marked this pull request as ready for review November 17, 2025 20:30
@NolanBlack NolanBlack requested a review from andrinr November 17, 2025 20:30
@andrinr
Copy link
Contributor

andrinr commented Nov 18, 2025

  • The masks should be of dimension (num_masked_points,), where for jax-fem I its not actually a mask but an integer array of boundary indices where each index can be associated with a unique bc value. This is more flexible but I am open to any interface.
  • Code sharing among tesseracts doesnt work that well, you can do it but then you dont get intellisense. Alternatively we could write a mini python package.

@NolanBlack NolanBlack merged commit ecdc6ce into ansys Nov 18, 2025
12 of 14 checks passed
@NolanBlack NolanBlack deleted the nolan/pymapdl branch November 18, 2025 21:15
@pasteurlabs pasteurlabs locked and limited conversation to collaborators Nov 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants