labeled_graph::remove_vertex does not remove label

[boostorg]

labeled_graph::remove_vertex does not remove label

[anadon]

Just an update, had to move twice, once across states, and started a new job. Getting back to this more now.

[jeremy-murphy]

Not sure if they are still active on Boost.Graph, but if someone submits a pull request to fix this issue, I can help them get it polished and merge it.

[anadon]

@jzmaddock Hey, this looks like a good one for me to work on.

[jzmaddock]

Go for it!

[anadon]

@arneboe Can I get your full name and email, since what you submitted is what I'm basing the regression test on? I need to credit you.

[arneboe]

I dont care about the credit, but if you have to: Arne Böckmann

[anadon]

It looks like the intended design, given the state of BGL, is to have _map a protected member, not a private member. So you hack was less of a hack than was expected.

[anadon]

This may take longer than expected: https://github.com/boostorg/graph/blob/develop/include/boost/graph/adjacency_matrix.hpp#L992

[anadon]

May be related: https://svn.boost.org/trac10/ticket/7863

[anadon]

This is going to be delayed somewhat. There are general cleanups in testing which I need to do first.

[HWiese1980]

Sure, take your time! Better have it fixed correctly and thoroughly, than to rush it and only come up with some quick'n'dirty patch that fails somewhere else.

[HWiese1980]

Alright, no pressure. Just don't forget it ;-)

[fjch1997]

And... it was forgotten
This bug really tripped me up when I tried to use labeled_graph

[HWiese1980]

As it seems... @anadon how's it going?

[EduardoGoulart1]

Hey there, any update on it? @anadon by any chance do you still have your WIP PR that people can pick up from?

[DeuceBox]

This may be related:

There was a patch at https://svn.boost.org/trac/boost/ticket/9493 (I did not create it) which is no longer reachable, but I've been using it in production for years

I've preserved it here:
DeuceBox@de651cb

It appears to fix the bug in the minimal example given in the OP.