Skip to content
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

Improve Remap Funcitonality - Topological Naming #1236

Open
phkahler opened this issue May 3, 2022 · 0 comments
Open

Improve Remap Funcitonality - Topological Naming #1236

phkahler opened this issue May 3, 2022 · 0 comments

Comments

@phkahler
Copy link
Member

phkahler commented May 3, 2022

System information

  • SolveSpace version: All
  • Operating system: All

Expected behavior

The number of entities in a group should not be limitied.
Sketch entities should be created from boolean operations where surfaces intersect.
Surfaces and edges of the solid model should have IDs that are persistent across regeneration.
It should be possible to link more than 1 part into an assembly group.

Actual behavior

Remap is limiting the number of entites in a group to 64K (is this true? no, it's encoded in the handles)
Edges of complex geometry can not be constrained against.
Solid models have no useful IDs for any edges (flat surfaces get an associated Face entity in most cases).
Remap would not support multiple linked parts in a group, even with the other needed changes.

Additional information

See several comments starting here in a discussion about chamfers. i.e. what edge do you want to modify?
This writing about FreeCAD addressing the issue.
Paper linked from the FreeCAD page

Some possible issues

If sketch entities are created directly from the solid model edges, they will not be movable by the user or the solver. They would still be better than what we have now and constraints could still apply but the entities could not be moved by the solver. This might cause confusion in some cases - solve may result in one moving or disappearing after solving.

With some effort, we might be able to create algebraic expressions for some of these entities which would allow more interesting constraints to be applied to them.

Any change to the remap mechanism will break existing sketches, so we would want to implement an automatic update on opening. Linked assemblies may also be tricky to update.

@ruevs ruevs added the internals label May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants