ProjectTorreyPines / IMASdd.jl

Basic data dictionary functionalities of IMAS.jl
https://projecttorreypines.github.io/IMASdd.jl/
Apache License 2.0
6 stars 1 forks source link

dd.jl defines many integers as Int64 instead of letting the platform decide default type #24

Closed eldond closed 3 months ago

eldond commented 3 months ago

dd.jl specifies Int64 type when it would probably be preferable to specify just Int, which would become either Int64 or Int32 depending on which julia version the user is running. We are currently using a mixture of Int64 and Int that only works if Int defaults to Int64. If this is okay, we should put up some warning of incompatibility with x86 julia versions. Otherwise, I recommend changing Int64 to Int in dd.jl.