Skip to content

Conversation

@daquinteroflex
Copy link
Collaborator

No description provided.

@daquinteroflex daquinteroflex changed the title demo: precision in Simulation input demo: precision in Simulation parameters Jan 29, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2026

Spell Check Report

Simulation.ipynb:

Cell 13, Line 5: 'floating-point'
  > Tidy3D uses a hybrid floating-point precision by default as it is practically sufficient in almost all cases, double precision should only be explored in large simulations when very high accuracy is desired. Even then, it might not have a noticeable effect over other sources of numerical error like the finite grid. As this is a small example with a minimal cost, we will simply demonstrate you can leverage this option depending on your simulation goals.

Checked 1 notebook(s). Found spelling errors in 1 file(s).
Generated by GitHub Action run: https://github.com/flexcompute/tidy3d-notebooks/actions/runs/21482336335

Copy link
Collaborator

@momchil-flex momchil-flex left a comment

Choose a reason for hiding this comment

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

I would word this even stronger in the direction that this should very rarely be needed. Something like, the hybrid precision is practically sufficient in almost all cases, double precision should only be explored in large simulations when very high accuracy is desired, and even then it might not have a noticeable effect over other sources of numerical error like the finite grid.

@daquinteroflex daquinteroflex force-pushed the dario/precision_in_simulation branch from d38830c to a0c9629 Compare January 29, 2026 14:37
@daquinteroflex
Copy link
Collaborator Author

Modified it to follow your wording as suggested.

@daquinteroflex daquinteroflex merged commit 13ad3f2 into develop Jan 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants