Skip to content

docs: add paper FDTD examples and solver benchmark - #517

Merged
vvahidd merged 1 commit into
mainfrom
docs/paper-examples-benchmark
Nov 11, 2026
Merged

vvahidd merged 1 commit into
mainfrom
docs/paper-examples-benchmark

Conversation

@vvahidd

@vvahidd vvahidd commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Adds four FDTD paper examples and a directional-coupler benchmark page with transmission spectra, field views, throughput, scalability, and cost charts.

@github-actions github-actions Bot added documentation Improvements or additions to documentation github_actions labels Nov 11, 2026
@vvahidd
vvahidd marked this pull request as ready for review November 11, 2026 17:58

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Removed at the repository owner’s request.]

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Removed at the repository owner’s request.]

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Removed at the repository owner’s request.]

@Alisama20 Alisama20 left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Removed at the repository owner’s request.]

@vvahidd
vvahidd marked this pull request as draft November 11, 2026 19:02
@gdsfactory gdsfactory deleted a comment from chatgpt-codex-connector Bot Nov 11, 2026
@gdsfactory gdsfactory deleted a comment from chatgpt-codex-connector Bot Nov 11, 2026
@gdsfactory gdsfactory deleted a comment from chatgpt-codex-connector Bot Nov 11, 2026
@gdsfactory gdsfactory deleted a comment from chatgpt-codex-connector Bot Nov 11, 2026
@gdsfactory gdsfactory deleted a comment from codecov Bot Nov 11, 2026
@vvahidd
vvahidd force-pushed the docs/paper-examples-benchmark branch from 503181e to 3e0eef5 Compare November 11, 2026 19:07
@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.62%. Comparing base (fe5acab) to head (3e0eef5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #517   +/-   ##
=======================================
  Coverage   91.62%   91.62%           
=======================================
  Files         204      204           
  Lines       28260    28260           
  Branches     5345     5345           
=======================================
  Hits        25894    25894           
  Misses       1354     1354           
  Partials     1012     1012           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vvahidd
vvahidd marked this pull request as ready for review November 11, 2026 19:37
@vvahidd
vvahidd merged commit 22636e6 into main Nov 11, 2026
43 checks passed
@vvahidd
vvahidd deleted the docs/paper-examples-benchmark branch November 11, 2026 19:38

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3e0eef5cfc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +33 to +39
if any(
isinstance(node, ast.Call)
and isinstance(node.func, ast.Attribute)
and node.func.attr in {"plot_2d", "plot_3d"}
for node in ast.walk(tree)
):
break

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Include post-view model changes in the fingerprint

When a maintainer changes geometry, materials, domain settings, or another model property in a cell after the first plot_2d/plot_3d call—or earlier in that same cell—the loop stops before hashing the change. Because reference_run only separately checks cell size and center wavelength, docs CI can accept and publish S-parameters from the old model under the modified notebook. Continue scanning model-affecting statements or fingerprint the resolved simulation configuration instead.

AGENTS.md reference: AGENTS.md:L56-L59

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation github_actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants