Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

📚 Include mode web api docs #1887

Merged
merged 1 commit into from
Aug 26, 2024
Merged

📚 Include mode web api docs #1887

merged 1 commit into from
Aug 26, 2024

Conversation

daquinteroflex
Copy link
Collaborator

Hi, included the API for mode simulations as per this https://github.com/flexcompute/tidy3d/blob/develop/tidy3d/web/api/mode.py requested by Alec.

@daquinteroflex
Copy link
Collaborator Author

Screenshot from 2024-08-07 11-36-30

@tylerflex
Copy link
Collaborator

looks good to me. thanks Dario. Just wondering why MODE is in all caps?

@tylerflex
Copy link
Collaborator

might prefer just Mode Solver Web API

Copy link
Contributor

@alec-flexcompute alec-flexcompute left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@daquinteroflex
Copy link
Collaborator Author

daquinteroflex commented Aug 8, 2024

Sounds good I've updated it.

I guess I'm still ingrained subconsciously the MODE, CHARGE naming of Lumerical tools haha.

Copy link
Collaborator

@tylerflex tylerflex left a comment

Choose a reason for hiding this comment

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

Looks good, sorry I didnt see this earlier!

@tylerflex
Copy link
Collaborator

@daquinteroflex should we merge this once the notebook submodule is fixed?

@daquinteroflex
Copy link
Collaborator Author

Hello, sorry this passed me, I will merge as soon as the tests pass!

@daquinteroflex daquinteroflex merged commit 2e91ebf into develop Aug 26, 2024
15 checks passed
@daquinteroflex daquinteroflex deleted the dario/docs/mode branch August 26, 2024 18:25
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