Skip to content

Conversation

@vickysharma0812
Copy link
Member

This pull request introduces several enhancements and new features across multiple modules to improve functionality and flexibility. The most significant changes include the addition of optional uppercase formatting for string conversions, new copy interfaces for various data structures, and optimizations in matrix operations. These updates aim to enhance code readability, usability, and performance.

String Conversion Enhancements:

  • Added an optional isUpper parameter to string conversion functions like BaseContinuity_ToString, BaseInterpolation_ToString1, and BaseType_ToChar. This allows the returned string to be formatted in uppercase when specified. [1] [2] [3] [4]

Copy Interfaces:

  • Introduced new Copy and Copy_ interfaces in IntVector_ConstructorMethod and QuadraturePoint_Method modules. These provide functionality to reallocate and copy data between objects, supporting various integer types and partial copying. [1] [2] [3] [4]

Matrix Operation Optimizations:

  • Simplified matrix operations in procedures like elemsd_SetBarycentricCoord, elemsd_SetJacobian, and stsd_SetJacobian by removing unnecessary array dimension specifications. This improves code clarity and reduces potential errors. [1] [2] [3]

Module Dependency Updates:

  • Adjusted module dependencies in ElemshapeData_Method by reordering and uncommenting certain USE statements to include additional methods like ElemshapeData_Lagrange and ElemshapeData_Hierarchical. [1] [2]

Minor Improvements:

  • Renamed the function IntVec_getTotalDimension to intVec_getTotalDimension to align with naming conventions.

Minor updates in ElemshapeData_Methods
refractoring(BaseInterpolation_Method): adding isUpper option

- When we convert integer id to text, isUpper can be used to convert
  them to upper case.
Add new interfaces for copying integer vectors:
- Public interfaces `Copy` and `Copy_`
- Implementation for different integer types (INT8, INT16, INT32, INT64)
- Support for full vector copy and partial copy with specified ranges
- Fix case inconsistency in `intVec_getTotalDimension` function name
- Fix capitalization of `PUBLIC` keyword in QuadraturePoint_Method.F90
- Add optional parameter `isUpper` to QuadraturePoint conversion functions
@vickysharma0812 vickysharma0812 merged commit 557fde6 into dev Jun 28, 2025
1 check failed
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