Vincenzo Calligaro
Vincenzo Calligaro
Hi @maxwxyz and @chennes. the following comment is valid both for this PR, #13309 and #13308: I agree about closing them. These 3 PRs are related to an analysis I...
@luzpaz yes, sure! Unfortunately my bandwidth is pretty narrow as I'm following other issues/PRs that need me to study quite a bit of the TNP code, therefor ATM I can't...
@Syres916, Thanks for the ping! Sure, I'll add it to my list! However ATM I can't give any guarantee about how much time I'll need (end even if I'll be...
@Syres916 no need to apologize. I'm the one who should apologize, haven't yet started looking at the code, only red the last comments and thought about possible ways to fix...
Hyper-quick update: Tested also with LS3 fork @ ```ruby OS: Linux Mint 21.3 (X-Cinnamon/cinnamon) Word size of FreeCAD: 64-bit Version: 2024.409.0.14555 (Git shallow) AppImage Build type: Release Branch: (HEAD detached...
@Syres916, I thank you too as @luzpaz already did. Twice for the hint you gave in your previous comment. I think this is closely related to https://github.com/FreeCAD/FreeCAD/commit/2657bbee4d72df816438f9d9afd7ba860ace6001: all the Ruled...
I can confirm too that #17436 looks fixing this issue. See my https://github.com/FreeCAD/FreeCAD/pull/17436#issuecomment-2436195930
@yorikvanhavre @maxwxyz for me it's fine if you tag/assign this issue to me. I'd say that this makes sense ATM as I'm mostly following issues related to TNP. Regarding the...
@MiqCG, as anticipated in my https://github.com/FreeCAD/FreeCAD/issues/17033#issuecomment-2480596743, I tested your [Ass_test_1_corrected.zip](https://github.com/user-attachments/files/17724064/Ass_test_1_corrected.zip) with my branch https://github.com/CalligaroV/FreeCAD/tree/toponaming-ElementMapVersion-code-from-LS3 (rebased locally on top of 5feda62c3a4d1a5d4e4d94262ec6835698d06621). The ` TopoShapeExpansion.cpp(970): hasher mismatch` warnings still triggers but IMO...
@MiqCG many thanks for the added info. I tested the assemblies inside [Ass_test_1_RC4.zip](https://github.com/user-attachments/files/17787398/Ass_test_1_RC4.zip) and I can confirm your analysis: the ` TopoShapeExpansion.cpp(970): hasher mismatch` warning messages triggers with `Assembly_link_RC4.FCStd` and...