Abse - @Seve

The tscircuit update is blocked because its dependency-sync script mistakes Core’s internally bundled winding solver for a dependency that tscircuit must install directly. Adding the solver to the script’s exclusion list should unblock the update and allow the new Core, Eval, and autorouter versions to propagate.

https://github.com/tscircuit/tscircuit/actions/runs/32660838958

yea it needs to be added

but not to core's exclusion

you just add it as a tscircuit dep

very easy/common fix

you just add it as a dependency to tscircuit/tscircuit

no propagation needed

Ok

<@757706909351411845> https://github.com/tscircuit/tscircuit/pull/4608

oh you can't use github deps there

so then we have to bundle in core

or wherever

what you originally said

or we release it as a package but idk

it's a lot harder to release packages now

so maybe we just bundle with core

Ok

its already bundled we just need to ignore it https://github.com/tscircuit/tscircuit/pull/4608

<@757706909351411845>

<@757706909351411845> https://github.com/tscircuit/schematic-trace-solver/pull/973

can you check this again

yep

ah sorry

you really have to split this in two

i can't see what's fixed

so the first pr needs to update the repro to the right schematic snapshot

i literally just can't tell if it's better or worse

<@757706909351411845> should be clean now

ehhh i disagree with your solution, now that i can see it

where is the reference

i think everyone would draw this like this

the original (below) is closer

<@757706909351411845>

idk ur call

<@757706909351411845> <@809856037376491570> can I get another review on this https://github.com/tscircuit/ti/pull/132

Wdyt?

<@757706909351411845> wdyt should we follow the ref?

hhmm

i don't feel strongly about this case tbh

the net labels are annoying though

Should we keep the routing as it is now?

Wdym?

Like the auto generated ones?

the L, R etc.

i think we should inline or remove those

<@757706909351411845> should we keep making these examples or does TI they want to see other stuff , I watched the meeting still not sure

idk why they're showing up

idk

we need M0 and M33 boards i guess, it's hard to decide what to do exactly

we don't want the schematics to look bad

we also need AllegroX export

idk man very difficult choices- i do think they will care a lot about the schematic when it's actually "go time" to integrate, we still haven't gotten in front of the actual product team

<@757706909351411845> https://github.com/tscircuit/schematic-trace-solver/pull/973/changes#diff-60f4dfa882b09810a2721a841cdf06680ecf388b78ec594c5942130110dedffb

revmoed the net lables not sure if we are going to merge this or not ?

i don't think we should merge it- i'm actually confused what you're doing with that pr

it's a core issue if core is asking for the net labels to be provided

removing it from the test doesn't make it so that core does the net labels better

i don't think it's a clear win from a schematic perspective

Im only trying to make the routing as the ref

there are other solutions that would improve what we're doing but disagree with ti

yea i don't think the ref is perfect on that one

can we merge the sch like this ?

why would we?

like this

the current routing I mean

not this pr

yea the ti pr could be merged (is that what you're asking?)

One sec

<@757706909351411845> <@809856037376491570> https://github.com/tscircuit/ti/pull/134

this too https://github.com/tscircuit/ti/pull/116

https://github.com/tscircuit/ti/pull/133

<@757706909351411845> https://github.com/tscircuit/tscircuit-autorouter/pull/2281

can you review this ty

this too https://github.com/tscircuit/ti/pull/134

<@757706909351411845>

<@757706909351411845> <@809856037376491570> https://github.com/tscircuit/core/pull/3522

<@757706909351411845> <@809856037376491570> for this pr do we only want to preserve the gnd symbols ? like is it ok to replace the GND net lables with on trace label

I can see that we want to keep the generated gnd symbol but what about the normal gnd labels pointed by the red arrows

I think on trace label replaced with the netlabel is okay, but yes the GND symbol should be there

sg