SKILL code to change all the existing Metal layers to its corresponding fillBlock layer

There are a couple of issues with the code, the first is that variable 'm2' will be nil, but perhaps more importantly, geGetSelSet() returns a list, and so the result might be 'nil' or '("M2") or '("M2" "M2")' and so on.

Perhaps something like 'forall' would be useful here?

procedure(MetalToFillBlock()
  let((m2)
    m2 = "M2"
    if(forall(object geGetSelSet() object~>layerName == m2)  then
      printf("Layer name is %L for all selected objects\n" m2)
    else
      printf("Layer name is NOT %L for all selected objects\n" m2)
    ); if
  );let
);procedure

Hopefully this will help a little?

Regards,

Lawrence.

Hi,

For one geGetSelSet() returns a list and not a shape. Assuming that you did select a shape (and not an instance or via or whatever) you  need to get at the (first and only) element in the list in terms of a car statement.
Moreover you initialize a variable m2 to nil in the let statement - but I assume you want to compare against the string "m2". So try this:

procedure(MetalToFillBlock()
   let(()
      if(car(geGetSelSet())~>layerName == "m2"
      then
         printf("Layer name is m2")
      else
         printf("Layer name is NOT m2"))
   );let
)

Thanks a lot for your quick response.

But, this is a bit different from what i'm looking for.

Thank you so much Lawrence, i think this is something closer to what i was looking for.

Ultimately i was trying to find all the M2 in a selected set of groups and do certain operation. Basically, i need to change every M2 drawing to M1,M2&M3 fillBlock

I was going to hard code this in the script. Do you have any idea how i could possibly do that?

To tell you the whole idea, i need to convert all the existing "M2 drawing" to "M2-1 fillBlock", "M2 fillBlock" & "M2+1 fillBlock" so that i could have "M1 fillBlock", "M2 fillBlock" & "M3 fillBlock" wherever there was "M2 drawing"

So, is there a way i could check the list for m2 (because the selected list could have other layers as well) and whenever there is an m2 found, change its layer purpose to fillBlock and add m2-1 as well as m2+1 fillBlock ?

Or, is there a way i could do these changes for all selected m2 in the list using a "forall" / "foreach" operation?

I'm planning to do this by hard coding it for all the layers like:

cv=geGetWindowCellView()
shapes=geGetSelSet()
dbLayerOr(cv list("m1" "fillBlock") shapes)
dbLayerOr(cv list("m2" "fillBlock") shapes)
dbLayerOr(cv list("m3a" "fillBlock") shapes)

The above code is hard coded only for m2, but i should be able to do this only for m2. 

And once all the m2 are taken care, i should do the same for m3, m4, m5, etc.

Thats where i'm stuck now, not knowing how to do it... 

Any help appreciated.

Now i'm doing the following, but its coming wrong since my layer generation is not coming good.

procedure(MetalToFillBlock()
cv=geGetWindowCellView()
shapes=geGetSelSet()
let((m0)
m2 = "m2"
foreach(object geGetSelSet() object~>layerName == m2
dbLayerOr(cv list("m1" "fillBlock") shapes)
dbLayerOr(cv list("m2" "fillBlock") shapes)
dbLayerOr(cv list("m3a" "fillBlock") shapes)
);foreach
);let
);procedure

I know its wrong since in dbLayerOr i'm mentioning shapes as the whole selected list. So my, foreach seems not working. I think i'm doing it horribly wrong.

Finally i got it working...!

It's not entirely clear what the sequence of layer names are, but rather than parsing the layer name, and handling it that way, you could do:

Note that this code is untested as I don't have anything suitable to try it on - but hopefully it gives you the hint as to how to do it.

Regards,

Andrew.