-
Notifications
You must be signed in to change notification settings - Fork 244
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
[Hotfix][Core] Edge based data structure hotfix #12059
Conversation
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.
thx!
Co-authored-by: Philipp Bucher <philipp.bucher@tum.de>
@@ -677,7 +677,6 @@ class EdgeBasedDataStructure | |||
|
|||
// Get position in the column indices vector as this is the same one to be used in the values vector | |||
const IndexType ij_col_index = GetColumVectorIndex(aux_i_id, aux_j_id); | |||
KRATOS_ERROR_IF(ij_col_index < 0) << "Column index cannot be found for ij-edge " << GlobalIdI << "-" << GlobalIdJ << "." << std::endl; |
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.
@rubenzorrilla this can never happen for an unsigned type
I think using std::optional
would be nice 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.
Sure. Most probably this is a leftover from a very first implementation or debugging. Let me merge this as it is to solve the Intel compiler issue and explore the std::optional
in a separate branch.
📝 Description
std::size_t
negative initialization.Closes #12017