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

ihp-sg13g2: libs.tech: klayout: tech: macros: Adjust metal filler #273

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

dnltz
Copy link
Contributor

@dnltz dnltz commented Nov 23, 2024

Metal fill was a little bit too low for M1 and too high for M3-5.

Fixes #<issue_number_goes_here>

  • Tests pass
  • Appropriate changes to README are included in PR

@dnltz
Copy link
Contributor Author

dnltz commented Nov 23, 2024

@KrzysztofHerman @sergeiandreyev btw dev branch is not on top of main anymore. You should rebase dev. I have to open this PR to main because metal fill is missing in dev branch.

@sergeiandreyev
Copy link
Contributor

Hi @dnltz, could you please move this PR to the dev branch, we have updated it to include the filler scripts

@dnltz dnltz changed the base branch from main to dev November 29, 2024 17:29
Metal fill was a little bit too low for M1 and too high for M3-5.

Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
@dnltz dnltz force-pushed the WIP/dnltz/update-metal-fill branch from 8c8e493 to 90883c3 Compare November 29, 2024 17:30
@dnltz
Copy link
Contributor Author

dnltz commented Nov 29, 2024

Hi @dnltz, could you please move this PR to the dev branch, we have updated it to include the filler scripts

@sergeiandreyev done

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