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

Add ABL-8.000MHZ-B4Y #329

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

Add ABL-8.000MHZ-B4Y #329

wants to merge 1 commit into from

Conversation

ii8
Copy link

@ii8 ii8 commented Sep 15, 2023

No description provided.

@ii8
Copy link
Author

ii8 commented Sep 15, 2023

Bot!

@github-actions

This comment has been minimized.

@github-actions github-actions bot added the bot: fail At least one check failed label Sep 15, 2023
@ii8
Copy link
Author

ii8 commented Sep 17, 2023

Bot!

@github-actions
Copy link

This review is brought to you by the Horizon EDA Poolbot commit 33131f8.

Items in this PR

State Type Name Checks Version Filename
New 3D Model Not run N/A 3d_models/crystal/ABL.step
New Package HC-49/US ❌ Fail 0 ✔️ packages/passive/crystal/HC-49_US/package.json
New Part ABL-8.000MHZ-B4Y ✔️ Pass 0 ✔️ parts/passive/crystal/abracon/ABL-8.000MHZ-B4Y.json

Parts overview (excluding derived)

Bold items are from this PR

  • Part ABL-8.000MHZ-B4Y
    • Entity Two-terminal crystal
      • Unit Two-terminal crystal
        • Symbol Crystal
    • Package HC-49/US
      • Padstack TH round
      • 3D Model 3d_models/crystal/ABL.step

Details

Parts

ABL-8.000MHZ-B4Y

✔️ Checks passed

Attribute Value
MPN ABL-8.000MHZ-B4Y
Value 8 MHz
Manufacturer Abracon (1 other parts)
Datasheet https://abracon.com/Resonators/ABL.pdf
Description 8 MHz Crystal
Tags crystal th

Orderable MPNs

  • ABL-8.000MHZ-B4Y
  • ABL-8.000MHZ-B4Y-T
  • ABL-8.000MHZ-B4Y-TRAY
Pad Gate Pin
1 Main A
2 Main B

Packages

HC-49/US

Attribute Value
Manufacturer (124 other parts)
Tags crystal th

✔️ Package checks passed

Clearance checks didn't pass

  • ❌ Top Package near Top Silkscreen

Package

Parameters
Parameter Value
Courtyard expansion 000.250 mm
11.500mm 5.000mm
get-parameter [ courtyard_expansion ]
2 * +xy
set-polygon [ courtyard rectangle 0.000mm 0.000mm ]
Pads
Pad Padstack Pad diameter Hole diameter
1 TH round 001.600 mm 000.800 mm
2 TH round 001.600 mm 000.800 mm
3D views (one model)

Without model

Top Bottom
3D 3D

ABL.step

Top Bottom
3D 3D
South East North West
3D 3D 3D 3D
Pitch analysis
X Y Count
004.880 mm 000.000 mm 2

@ii8
Copy link
Author

ii8 commented Sep 17, 2023

There is actually a 0.2mm gap between the package and the silkscreen, I believe the clearance check isn't working right because of the rounded part of the package.

@RX14
Copy link
Collaborator

RX14 commented Sep 18, 2023

Hi, could you confirm the source of the step file and the licence terms it is under? If the licence is incompatible, perhaps there is an equivalent model from kicad's 3d model collection

@ii8
Copy link
Author

ii8 commented Sep 23, 2023

The step file is from Abracon, I couldn't find license information on their website and I've emailed them but still no response after several days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: fail At least one check failed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants