-
Notifications
You must be signed in to change notification settings - Fork 40
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
Removed no sources in simulation log info. #1784
Conversation
Actually, I will resubmit this following these instructions (https://docs.flexcompute.com/projects/tidy3d/en/latest/development/index.html). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good, we can just squash and merge but I think we should add the invdes_history.json
file to gitignore and remove from this PR. Tyler and I could do it once he confirms.
tests/data/invdes_history.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tylerflex should we add this to gitignore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @e-g-melo The PR looks ok to me. Could you file an issue (if not already) so we can make sure to fix this in the long term? Not sure who to assign at this point.
f927a42
to
de9621e
Compare
de9621e
to
9531fff
Compare
The "No sources in simulation" log info was commented out to avoid showing this message in GUI.