masi/solvecraft
1
0
Fork 0
mirror of https://github.com/bherbruck/solidcraft.git synced 2026-11-03 10:35:58 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

Revert "kernel: merged topology counts no seam edge (a face on both sides of it)"

Browse Source 
This reverts commit 1c490ddde7.
This commit is contained in:
jyviko
2026-10-22 15:29:18 -04:00
parent 0d783f889a
commit e6f993fff0
1 changed files with 0 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/kernel/src/topo.rs
-10
View File
@@ -122,8 +122,6 @@ pub fn merged_topology(b: &Body, mesh: &Mesh) -> Result<TopoCounts> {
let mut weld_e: HashMap<(usize, usize, (i64, i64, i64)), usize> = HashMap::new();
let mut welded_faces: Vec<Vec<usize>> = Vec::new();
let mut welded_ends: Vec<(usize, usize)> = Vec::new();
// How many face uses each welded edge has (a seam: two, on one face).
let mut welded_uses: Vec<usize> = Vec::new();
// A record edge standing for each welded edge (for its points).
let mut rep: Vec<u32> = Vec::new();
for (ei, (e, between)) in record.edges.iter().zip(edge_faces(record)).enumerate() {
@@ -135,12 +133,8 @@ pub fn merged_topology(b: &Body, mesh: &Mesh) -> Result<TopoCounts> {
if we == welded_faces.len() {
welded_faces.push(Vec::new());
welded_ends.push((a, z));
welded_uses.push(0);
rep.push(ei as u32);
}
if let Some(n) = welded_uses.get_mut(we) {
*n += e.uses.len();
}
if let Some(fs) = welded_faces.get_mut(we) {
for f in between {
if !fs.contains(&f) {
@@ -212,12 +206,8 @@ pub fn merged_topology(b: &Body, mesh: &Mesh) -> Result<TopoCounts> {
}
continue;
}
// An edge between two pieces of one merged face, or a seam (one face on both its sides),
// is no edge of the merged topology; one face on one side only is an open edge.
let seam = welded_uses.get(we).is_some_and(|n| *n >= 2);
let pair = match gs[..] {
[a, b] if a == b => continue,
[_] if seam => continue,
[a, b] => (a.min(b), a.max(b)),
[a] => (a, a),
_ => (usize::MAX, usize::MAX),