Update gdsfactory to latest version 7.23

[joamatab]

@tvt173

[sourcery-ai]

Hey @joamatab - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟡 General issues: 3 issues found
• 🟢 Security: all looks good
• 🟡 Testing: 2 issues found
• 🟢 Complexity: all looks good
• 🟢 Docstrings: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[sourcery-ai]

question (code_refinement): Use of .flatten() in component creation.

Confirm that flattening the component here is necessary, as it may impact the ability to edit or parameterize the component later.

[sourcery-ai]

question (performance): Introduction of @cache decorator.

Verify that caching these components does not introduce state issues, especially in a multi-threaded environment.

[sourcery-ai]

question (performance): Use of @cache decorator in import_gds function.

Ensure caching is appropriate for the import_gds function, considering the potential for large GDS files and memory usage.

[sourcery-ai]

suggestion (testing): Consider adding a test case for import_gc.

Given the changes in import_gc function, it would be beneficial to ensure its behavior is correctly validated, especially considering the new error handling and port addition logic.

Suggested change

	"import_gds",
	def test_import_gc_behavior(data_regression, check, component_factory):
	c = component_factory["import_gc"]()
	n = c.get_netlist()
	if check:
	data_regression.check(n)

[sourcery-ai]

suggestion (testing): Please add a test case for import_gc.

The modifications to import_gc suggest new functionality has been added. Testing this directly would help ensure its reliability and correctness.

Suggested change

	"import_gds",
	"import_gds",
	"import_gc",
	"mzi",
	}
	# Assuming `import_gc` is the new functionality to be tested
	if "import_gc" in cell_names:
	test_import_gc() # This is a placeholder for the actual test function for import_gc
	cell_names = set(cells.keys()) - set(skip_test)

[codecov]

Codecov Report

Attention: Patch coverage is 72.72727% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 68.80%. Comparing base (9759b34) to head (d538231).
Report is 6 commits behind head on main.

Files	Patch %	Lines
ubcpdk/import_gds.py	54.54%	5 Missing ⚠️
ubcpdk/__init__.py	62.50%	3 Missing ⚠️
ubcpdk/components.py	92.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #370      +/-   ##
==========================================
- Coverage   68.83%   68.80%   -0.03%     
==========================================
  Files          19       19              
  Lines         770      763       -7     
==========================================
- Hits          530      525       -5     
+ Misses        240      238       -2     

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