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

CGNS convention when using I8/R8 #88

Open
ndelling opened this issue Jul 18, 2024 · 1 comment
Open

CGNS convention when using I8/R8 #88

ndelling opened this issue Jul 18, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@ndelling
Copy link

Hi

when compiling Cassiopee with I8/R8, some data are written as I8/R8 while the CGNS standard imposes I4/R4:

  • version of the CGNS library -> should be R4
  • values of Element_t nodes (type of element, nbndry) -> should be I4
  • dimension of the bases -> should be I4

for example.

Thx

@vincentcasseau
Copy link
Collaborator

vincentcasseau commented Jul 25, 2024

Hello,

Many thanks for the bug report.
I am adding a link to the doc listing all node types to force to I4/R4: cgns-modern/standard/FMM/nodes

Node types to force to R4:

  • CGNSLibraryVersion_t, done in 7d61577

Node types to force to I4:

  • CGNSBase_t, done in 50a3154
  • Elements_t, done in 50a3154
  • Rind_t
  • Ordinal_t, done in 50a3154
  • BaseIterativeData_t, done in 50a3154
  • ConvergenceHistory_t, done in 50a3154
  • ZoneSubRegion_t

and a few others by name:

  • Transform (child of GridConnectivity1to1_t),
  • InwardNormalIndex,
  • EquationDimension

Thanks,
Vincent

vincentcasseau added a commit that referenced this issue Aug 6, 2024
Converter: force CGNS convention when using I8/R8 (see #88)
@vincentcasseau vincentcasseau added the enhancement New feature or request label Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants