Rescue dialog: some issues

Bug #1455970 reported by Simon Richter
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Undecided
Chris Pavlina

Bug Description

When opening an older project, the "rescue schematic parts" dialog opens, which is great. A few small things that could be improved. Video at http://psi5.com/~geier/rescue.ogv

1. The dialog says it wants to rescue both C and R components, however the list of affected components includes only capacitors.

2. When I update the components, all connectivity to the parts appears to be broken. It would be great to have a good way to go through the updated components to repair this.

3. The schematic is not marked as "changed" afterwards, so I'm not asked to save before closing.

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

Problems #2 and #3 are not problems! You clicked cancel, so of course it didn't update anything. When you cancel, it behaves exactly as if that dialog never appeared, eeschema updates the symbols to the new ones exactly as it used to.

#1 is an issues though. It's been working fine for me. Any chance you can share that project with me?

Changed in kicad:
assignee: nobody → Chris Pavlina (pavlina-chris)
Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

Wait - #1 isn't an issue either! It says "instances of this symbol". You have to click on the resistor in the first list to get resistors in the second list.

Well, it could be a design flaw, if people think I should change that behavior. But it's not a /bug/, per se.

Revision history for this message
Simon Richter (sjr) wrote :

Understood #1 -- it wasn't immediately obvious to me, but that is because I didn't read the dialog properly.

#2 is more of a wishlist item with the component update. If I ever cancel the rescue dialog, I need to be aware that my schematic is likely to be broken in a lot of places, and there needs to be an easy way to find these.

#3 is a bit of a problem, because the schematic did change -- after all, the components have been updated. The next time I reopen the schematic, I get the rescue dialog again.

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

That's the original behavior!

#2. Perhaps I can edit the description text at the top? The fact that the schematic was going to be broken should have been obvious, that's the whole point of the dialog! As for a way to find them - run ERC.

#3. No, the schematic didn't change! Try saving it and then diffing it against the original - not a single thing will change. The components are just being rendered out of the libraries, which *have* changed. That's what happens when you press cancel - *nothing*.

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

I'm going to look into patching #1 - it would have been more obvious if the first symbol in the list were actually selected, so you could see that the bottom list was based on that selection.

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

Hey Wayne, I can't remember - is the new library format supposed to hit before or after 5.0?

If before, I'm going to wontfix this due to priorities - the rescue dialog should be pointless after the new format is out, and I don't think there's much point in putting significant effort into the 4.0 branch beyond real bugfixes. If the new format won't be out until 6.0 I'll try to have this fixed targeting 5.0.

Revision history for this message
Wayne Stambaugh (stambaughw) wrote : Re: [Bug 1455970] Re: Rescue dialog: some issues

On 3/26/2017 7:04 PM, Chris Pavlina wrote:
> Hey Wayne, I can't remember - is the new library format supposed to hit
> before or after 5.0?

After. I'm just not going to have it ready for 5.0. The symbol library
table is going to be disruptive enough. The symbol library table code
should help to minimize the need for the rescuer but it wont completely
eliminate the need for it. There will be corner cases where the symbol
library table remapping algo falls short (mainly when someone breaks
their symbol library list) so I may still have to rescue missing symbols
into a library and map them that way.

>
> If before, I'm going to wontfix this due to priorities - the rescue
> dialog should be pointless after the new format is out, and I don't
> think there's much point in putting significant effort into the 4.0
> branch beyond real bugfixes. If the new format won't be out until 6.0
> I'll try to have this fixed targeting 5.0.
>

Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision feba091594bc81a97edcaf3cb628441fd91d4433
https://git.launchpad.net/kicad/patch/?id=feba091594bc81a97edcaf3cb628441fd91d4433

Changed in kicad:
status: New → Fix Committed
Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

Wayne, mind if I push this commit to 4.0?

Revision history for this message
Wayne Stambaugh (stambaughw) wrote :

Please do since it effects the 4.0 branch. Thanks!

On 3/27/2017 8:45 AM, Chris Pavlina wrote:
> Wayne, mind if I push this commit to 4.0?
>

Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.