Open
Description
Description
Whenever we cross a surface the following boilerplate code appears
if (boundary.lattice_translation[0] != 0 ||
boundary.lattice_translation[1] != 0 ||
boundary.lattice_translation[2] != 0) {
...
}
A method like bool BoundaryInfo::lattice_crossing()
would be useful and more clear.
Alternatives
Leave well enough alone.
Compatibility
Should be an internal refactor, no compatibility issues.