Featurescript – Fillet a sheet metal part using edges attribute

guillauem_d · October 8

Hello there !

I’d like to be able to fillet sheet metal parts corners through Featurescript, using the attribute of edges.

For the moment, I managed to create fillets like so :

Select the edge using annotation { "Name" : "My Query", "Filter" : EntityType.EDGE, "MaxNumberOfPicks" : 1 } definition.entitiesHomemade is Query;
Create the fillet using available public code such as sheetMetalCornerBreak or C2CfullRoundFillet (because opFillet doesn't work on sheet metal parts)

This method works great, but these public functions need as input the edge Query from annotation. Now i’d like to do the same thing but using the attribute ("JMN") of my edge.

To do this I tried transforming my attribute into a query using qHasAttribute("JMN") or qOwnedByBody(qHasAttribute("JMN"), EntityType.EDGE)) but it doesn’t work.

Any tips ?

Thank you 😊

MichaelPascoe · October 9

Gotcha. Here is how you can convert the transient id array back into a query of edges:
https://cad.onshape.com/documents/28b6b8878dbecd49e8c8d643/w/db4da420a14d…

/**
 * Example function to convert the transient id's to a query.
 */
function GetQueryFromTransientIdList(context is Context, transientIdList is array) returns Query
{
    const allBodyEdgesQuery = qOwnedByBody(qAllSolidBodies(), EntityType.EDGE);
    var evaluatedAllBodyEdges = evaluateQuery(context, allBodyEdgesQuery);
    var output_edges_query = qNothing();

    for (var edge in evaluatedAllBodyEdges)
    {
        for (var transientId in transientIdList)
        {
            if (transientId == edge.transientId)
            {
                output_edges_query = qUnion([output_edges_query, edge]);
            }
        }
    }

    return output_edges_query;
}

MichaelPascoe · October 8

Please share a link to your document so we can help you better.

guillauem_d · October 8

https://forum.onshape.com/discussion/comment/108092#Comment_108092

Thank you for helping out MichealPascoe ! 😁

Here is the link to a minimal reproducible example :

fillet_from_user_selection() works great and create a fillet according to user’s edge selection
print_transient_id_of_edges_on_z_axis() allows me to find the transient id of some edges, then I try to use fillet_from_edge_attribute() which is the same code as fillet_from_user_selection() except that it takes as input the transient id of the edge, but runing this function doesn’t work

MichaelPascoe · October 9

What is your end goal?
I'm confused why you are trying to use attributes if your goal is to simply fillet all edges parallel to the Z_Axis.

guillauem_d · October 9

Sure! Let me clarify what I’m trying to do 🙂:

I created a Python interface that communicates with the Onshape REST API to build very basic parts.
For example, I tell the Python code to create a square sheet metal part.
The code then triggers a FeatureScript function via the API that generates that square sheet metal part.
Afterward, the API returns attributes for all the edges and surfaces of the part.
If I want to create a hole on the top surface (let’s say the surface with the attribute ‘JDP’), I instruct the Python code to create a hole on 'JDP,' and it executes the appropriate FeatureScript (this part works perfectly). From step 1 to 5, everything works.
Now, I’d like to create a fillet on specific edges, such as ‘JMN’ . However, since a sheet metal is a special object, I can't use the standard opFillet(). I’ve worked around this by using sheetMetalCornerBreak(), but it only works when I use a user annotation query as input (i.e by selecting the edge directly on the onshape interface, not something i can do from python code). Now i'd like to use the attribute as input.

So to sum it up, in the python interface, i give the user attribute of all important elements, so that if i want to modify them or interact with them, i can do it.

I hope this explanation clears things up!

MichaelPascoe · October 9

Gotcha. Here is how you can convert the transient id array back into a query of edges:
https://cad.onshape.com/documents/28b6b8878dbecd49e8c8d643/w/db4da420a14d…

/**
 * Example function to convert the transient id's to a query.
 */
function GetQueryFromTransientIdList(context is Context, transientIdList is array) returns Query
{
    const allBodyEdgesQuery = qOwnedByBody(qAllSolidBodies(), EntityType.EDGE);
    var evaluatedAllBodyEdges = evaluateQuery(context, allBodyEdgesQuery);
    var output_edges_query = qNothing();

    for (var edge in evaluatedAllBodyEdges)
    {
        for (var transientId in transientIdList)
        {
            if (transientId == edge.transientId)
            {
                output_edges_query = qUnion([output_edges_query, edge]);
            }
        }
    }

    return output_edges_query;
}

guillauem_d · October 10

Wow, that’s brilliant!

Thank you so much @MichaelPascoe, You saved me from going insane after struggling with this for three days! ! 😅

Featurescript – Fillet a sheet metal part using edges attribute

Best Answer

Answers

Howdy, Stranger!

Quick Links

Categories