-
Notifications
You must be signed in to change notification settings - Fork 0
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
KISS-111, add api for writing double variables with mesh locations #7
Conversation
Edges = 2, ///< Edges | ||
}; | ||
|
||
static std::unordered_map<MeshLocations, std::string> locations_attribute_names{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you need Doxygen documentation here?
src/UGridApi/UGrid.cpp
Outdated
throw std::invalid_argument("UGrid: The selected file_id does not exist."); | ||
} | ||
|
||
const auto name = ugrid::char_array_to_string(variable_name, ugrid::name_long_length); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This variable name
does not seem to be used. Can it be removed, or should line 326 use name
instead of variable_name
, or somehing else?
src/UGridApi/UGrid.cpp
Outdated
} | ||
if (mesh_location_enum == MeshLocations::Nodes) | ||
{ | ||
coordinates = get_coordinate_variable_string(file_id, topology_id, Mesh2dTopology, "node_coordinates"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need a:
location = "nodes";
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comments
…riable on location
Output binaries into bin
@@ -163,6 +163,20 @@ namespace ugridapi | |||
return it->second; | |||
} | |||
|
|||
static auto get_coordinate_variable_string(int file_id, int topology_id, int topology_type, std::string const& var_name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
std::string const& var_name
?
or
const std::string& var_name
No description provided.