better naming

[joamatab]

avoid even when users don't use decorator to not create duplicated cell names so we can use the name as a component unique id and avoid slow geometric hashes

fixes #2435

@yaugenst
@tvt173
@nikosavola
@flaport

[joamatab]

this seems PR seems to have issues with components that use flatten_invalid_refs

See for example gf.components.coupler_bent

[tvt173]

see the alternative branch, which is now working #2453

[codecov]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (a14a526) 72.59% compared to head (ac75a3a) 72.59%.
Report is 6 commits behind head on main.

Files	Patch %	Lines
gdsfactory/component.py	84.21%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2434   +/-   ##
=======================================
  Coverage   72.59%   72.59%           
=======================================
  Files         366      366           
  Lines       23308    23327   +19     
  Branches     3788     3794    +6     
=======================================
+ Hits        16920    16935   +15     
- Misses       5266     5267    +1     
- Partials     1122     1125    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joamatab]

Thank you Troy, i brought your improvements here

i prefer this PR because gf.Component() has a safer init name with the counter on it

[joamatab]

for some weird reason tests now fail in macOs but pass on linux

https://github.com/gdsfactory/gdsfactory/actions/runs/7459521790/job/20295712487