Skip to content

Conversation

vickysharma0812
Copy link
Member

@vickysharma0812 vickysharma0812 commented Mar 15, 2024

Adding meshVolume, meshSurface, meshCurve, meshPoint.
Updating deallocate method.


Generated summary (powered by Graphite)

TL;DR

This pull request includes changes to the Domain_Class.F90 file in the Domain module, removing MeshPointerVector_ types and adding global_nptrs for node coordinate access. MeshList arrays for different entity types were also removed.

What changed

  • Removal of MeshPointerVector_ types
  • Addition of global_nptrs for node coordinate access
  • Removal of MeshList arrays for different entity types

How to test

  • Compile the updated Domain_Class.F90 file
  • Run test cases related to mesh entities and node coordinates
  • Check for any errors or inconsistencies in mesh data retrieval

Why make this change

  • Improve efficiency by removing unnecessary data structures
  • Enhance readability and maintainability of the code
  • Streamline access to node coordinates for better data manipulation

@vickysharma0812 vickysharma0812 force-pushed the 03-15-updates_in_domain_class branch from 3da3598 to 50e07d3 Compare March 16, 2024 07:49
@vickysharma0812 vickysharma0812 marked this pull request as ready for review March 16, 2024 09:10
Copy link
Member Author

vickysharma0812 commented Mar 16, 2024

Merge activity

@vickysharma0812 vickysharma0812 force-pushed the 03-15-updates_in_mesh_class branch from 359b224 to 283fe4a Compare March 16, 2024 10:01
@vickysharma0812 vickysharma0812 changed the base branch from 03-15-updates_in_mesh_class to dev March 16, 2024 10:02
Adding meshVolume, meshSurface, meshCurve, meshPoint.
Updating deallocate method.
@vickysharma0812 vickysharma0812 force-pushed the 03-15-updates_in_domain_class branch from 50e07d3 to d3fb1ed Compare March 16, 2024 10:03
@vickysharma0812 vickysharma0812 merged commit a40a4fe into dev Mar 16, 2024
@vickysharma0812 vickysharma0812 deleted the 03-15-updates_in_domain_class branch March 16, 2024 12:49
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.

1 participant