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

Use dielectric grids for automatic overlap checks #254

Merged
merged 2 commits into from
Apr 20, 2024

Conversation

paolafer
Copy link
Contributor

The automatic tests that perform the overlap checks were taking ages to be executed, since the implementation of stainless steel grids as the default option in NEXT-100. This PR replaces the macros used for the overlap checks with others that use dielectric grids (the other option).

@paolafer paolafer requested a review from gonzaponte April 19, 2024 10:59
Copy link
Contributor

@gonzaponte gonzaponte left a comment

Choose a reason for hiding this comment

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

Great job!

@paolafer paolafer force-pushed the fix-overlap-check branch from d383012 to 2397604 Compare April 19, 2024 14:07
@paolafer paolafer force-pushed the fix-overlap-check branch 2 times, most recently from da85b38 to b28089d Compare April 19, 2024 15:16
@paolafer paolafer force-pushed the fix-overlap-check branch from b28089d to 10b7cec Compare April 19, 2024 15:17
@paolafer paolafer merged commit 1f5e72e into next-exp:master Apr 20, 2024
1 check passed
@paolafer paolafer deleted the fix-overlap-check branch April 20, 2024 14:58
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.

2 participants