Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Pulls/hanrun hr911105a #1091

Closed
wants to merge 2 commits into from
Closed

Conversation

karlp
Copy link
Contributor

@karlp karlp commented Nov 16, 2018

Adds a footprint for HanRun HR911105A magjack.

HanRun's own website is not always very good, pdfs widely available. http://www.kosmodrom.com.ua/pdf/HR911105A.pdf
symbol/3d model linked in a separate commit, but is unlicensed for distribution, so only provided for context and information.
export-hr911105-model
screenshot from 2018-11-16 21-50-39


Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items:

  • Provide a URL to a datasheet for the footprint(s) you are contributing
  • An example screenshot image is very helpful
  • If there are matching symbol or 3D model pull requests, provide link(s) as appropriate
  • Check the output of the Travis automated check scripts - fix any errors as required

@CLAassistant
Copy link

CLAassistant commented Nov 16, 2018

CLA assistant check
All committers have signed the CLA.

@karlp
Copy link
Contributor Author

karlp commented Nov 16, 2018

(it would be nice if all those travis checks were available to peruse before you file and get failures. It's very unclear how to get your way to the very bottom of the klc page, after you even find that that's what's being referred to)

What's the rationale for having parts with holes having zero poitn on "pad 1" but surface mount parts on the centroid?! that jsut seems completely arbitrary

What's the point in having step models be supported if travis is rejecting them?
What's the point in having support for positioning models if travis is going to reject them?

The rest I can fix.

3d model is sourced from http://www.3dcontentcentral.com/parts/part.aspx?id=402242&catalogid=171

3d model is unlicensed, so separate this commit, only for information.
@karlp karlp force-pushed the pulls/hanrun-hr911105a branch from c771899 to 4b63131 Compare November 16, 2018 22:56
@karlp
Copy link
Contributor Author

karlp commented Nov 16, 2018

updated and fixed all.

For F7.3, it's simply not appropriate to make pad1 rectangular here.
the 3d model checks are not relevant, they're in the second UNLICENSED commit, only included for context.

@karlp karlp mentioned this pull request Nov 16, 2018
5 tasks
@DanSGiesbrecht DanSGiesbrecht added Addition Adds new footprint to library Pending reviewer A pull request waiting for a reviewer labels Dec 2, 2018
@myfreescalewebpage
Copy link
Collaborator

Hi @karlp

Pin 1 must be rectangular, and i don't know why it is not convenient here. See Connector_RJ:RJ45_Amphenol_54602-x08_Horizontal part in the lib for example.

Cheers,
Joel

@karlp
Copy link
Contributor Author

karlp commented Jan 17, 2019

The objection to a square pin is just that a) it's a polarized footprint, there's no chance of mixing anythign up, and b) it takes up more space in a tight area.

I'm completely ok with a square pad for symmetrical footprints, but it seems completely counterproductive for such a strongly keyed footprint. I'd suggest the original amphenol footprint is wrong, not something that should be copied here.

@myfreescalewebpage
Copy link
Collaborator

@karlp I disagree with you, pin 1 is usually indicated with a square pin for many parts, connectors and other, even if the part is not symmetrical.

Then, KLC requires pin 1 to be rectangular, that's a wanted practice for this library: http://kicad-pcb.org/libraries/klc/F7.3/ so the Amphenol connector is conform to our guidelines.

@poeschlr
Copy link
Collaborator

poeschlr commented Jan 17, 2019

Square pin 1 will make debugging a board with this connector much easier.
(Edit: you can also use rounded rectangle pins. And for the overly motivated there is always the option to use a custom pad shape to increase clearance if it is really a tight fit. This should not be a problem for this footprint as there is ample space.)

One additional note: the pads look awfully small. You might want to increase them to get a bit larger annular rings.

@poeschlr poeschlr self-assigned this Jan 23, 2019
@poeschlr poeschlr removed the Pending reviewer A pull request waiting for a reviewer label Jan 23, 2019
@poeschlr
Copy link
Collaborator

You currently have non valid 3d model settings. The footprint should not use rotation or offset settings. The 3d model (should it ever be provided) is to be aligned correctly using a CAD program.
The model must be set to wrl file ending. KiCad does replace the ending with step should it need it.

Rotate the footprint by 180 degree such that pad 1 ends up nearest to the top left corner.
Add pin 1 markers on fab and silk layers. (Again not only for ensuring correct insertion of the part but for debugging purposes. But also to keep the rules simple -> easier for us maintainers.)

The fab outline must represent the nominal dimensions of the part. The silk outline must be placed outside it. (But not too far away such that it still is useful for aligning the part correctly.)

The silk outline should not go across pads. In fact there should be 0.2mm clearance between the pads and silk.

A courtyard outline must be provided with 0.5mm clearance (for connectors) relative to pads and nominal body size (whatever results in the larger area) This courtyard outline is allowed to be contoured for the part. (But a simple rectangle will also be accepted.)

rj svg

@poeschlr poeschlr added the Abandoned Original author has stopped working on the PR label Sep 21, 2019
@karlp
Copy link
Contributor Author

karlp commented Mar 9, 2020

This isn't abandoned, it's just pending the symbol work: KiCad/kicad-symbols#1146

@chschlue chschlue removed the Abandoned Original author has stopped working on the PR label Mar 9, 2020
@poeschlr
Copy link
Collaborator

poeschlr commented Mar 9, 2020

There is a change request for the footprint that was not fulfilled in a year ;). If that is not the definition of abandoned then i don't know what is.

@karlp
Copy link
Contributor Author

karlp commented Mar 17, 2020

The last comment on it a year ago was someone asking you for clarification, "@poeschlr can you have a look to this PR to get a decision, particlularly at my comment #1146 (comment) ? Thanks."
That would seem to be abandoned by you, not by me.

But yes, this is abandonded now. I'm not spending more time on these hoops, the upstreaming value proposition is far too slim.

@karlp karlp closed this Mar 17, 2020
@myfreescalewebpage myfreescalewebpage added Abandoned Original author has stopped working on the PR and removed Pending changes labels Mar 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Abandoned Original author has stopped working on the PR Addition Adds new footprint to library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants