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

update tidy3d #378

Merged
merged 4 commits into from
Apr 27, 2024
Merged

update tidy3d #378

merged 4 commits into from
Apr 27, 2024

Conversation

joamatab
Copy link
Contributor

@joamatab joamatab commented Apr 10, 2024

  • update tidy3d to latest version.

waiting for a new patch from tidy3d side

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We have skipped reviewing this pull request. We don't review packaging changes - Let us know if you'd like us to change this.

@joamatab joamatab added the dependencies Pull requests that update a dependency file label Apr 10, 2024
@joamatab
Copy link
Contributor Author

not sure why some tests are failing with the update?

https://github.com/gdsfactory/gplugins/actions/runs/8654520090/job/23731859225

@tylerflex

@tylerflex
Copy link

I think this issue has to do with some conflicts with boto3 / urllib3 that were not set correctly when we migrated to poetry. Should be resolved with flexcompute/tidy3d#1598

We will release a 2.6.4 patch on Monday with it after some more testing.

Is there a way for you to set your tests to run with tidy3d from that branch to see if it resolves it?

@joamatab
Copy link
Contributor Author

yes, we can, but we can also wait until Monday :)

we've been using quite an old version of tidy3d 2.5.2 anyway for a while :)

@joamatab joamatab marked this pull request as draft April 11, 2024 23:47
@tylerflex
Copy link

We think the issue is that the new tidy3d accidentally pinned boto3==1.23.1 whereas we really want it to be 1.28 or later. Should be fixed in 2.6.4

@joamatab joamatab marked this pull request as ready for review April 27, 2024 00:15
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We have skipped reviewing this pull request. Here's why:

  • We don't review packaging changes - Let us know if you'd like us to change this.
  • It looks like we've already reviewed this pull request.

@joamatab joamatab merged commit 748ef64 into main Apr 27, 2024
14 checks passed
@joamatab joamatab deleted the update_tidy3d branch April 27, 2024 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants