Project

General

Profile

Actions

Bug #3271

closed

Bug #3269: osmo-e1-xcvr has magnetics not placed the right way

Fix T1094NL footprint in library

Added by laforge almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
05/15/2018
Due date:
% Done:

100%

Spec Reference:

Files

PULSE_T1094NL.lbr PULSE_T1094NL.lbr 10.7 KB mschramm, 05/29/2018 06:11 PM
Actions #1

Updated by mschramm almost 6 years ago

  • Status changed from New to In Progress
Actions #2

Updated by mschramm almost 6 years ago

  • File PULSE_T1094NL.lbr added
  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

repaired PULSE_T1094NL footprint and symbol (solves OSM#3270)

... and OSM#3271. - The transformer symbol is made of two identical coils and hence it is not possible to mark the "2:" side (pins 9-11) separately. But the naming of this coil was changed in a way, that, when invoked, the "2" of the subpart name "TR1-1:2" shows towards the pins 9-11. Maybe we should evolve to a device symbol with two coils merged rather than two identical but separated, then an individual naming could honour the 2:1 side and can not be disturbed by e.g. subsequent mirroring. - Also repaired the name and value designators' layer association.

lib attached to ticket and will be updated in repo next week.

Actions #3

Updated by laforge almost 6 years ago

Hi Martin,

thanks for your updates.

On Fri, May 18, 2018 at 08:04:27PM +0000, mschramm [REDMINE] wrote:

lib attached to ticket and will be updated in repo next week.

you now have push access to osmo-e1-xcvr.git, so it can be fixed there, too.

Actions #4

Updated by mschramm almost 6 years ago

mschramm wrote:

lib attached to ticket and will be updated in repo next week.

done

Actions #5

Updated by mschramm almost 6 years ago

had to touch this again:

commit 8ce574846d1f18d71b22fa811ebff4fd10694cf8
Author: Martin Schramm <mschramm@sysmocom.de>
Date:   Tue May 29 19:21:38 2018 +0200

    laforge.lbr: PULSE_T1094NL: better tPlace and tDocu layer (for OSM#3270)
Actions #6

Updated by mschramm almost 6 years ago

  • File deleted (PULSE_T1094NL.lbr)
Actions #7

Updated by mschramm almost 6 years ago

(updated discrete Pulse lib here too besides having this implicit in the laforge.lbr and in project's design files)

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)