Opened 17 years ago
Closed 17 years ago
#197 closed defect (fixed)
MaxWeightedMatching and MaxWeightedPerfectMatcing crash
| Reported by: | Philip.Waldron | Owned by: | Balazs Dezso | 
|---|---|---|---|
| Priority: | major | Milestone: | LEMON 1.1 release | 
| Component: | core | Version: | hg main | 
| Keywords: | Cc: | "Waldron, Philip" <Philip.Waldron@…> | |
| Revision id: | 30d22b636e57 | 
Description
The following code crashes with the attach .lgf file, using either MaxWeightedMatching or MaxWeightedPerfectMatching
#include <lemon/max_matching.h>
#include <lemon/smart_graph.h>
#include <lemon/lgf_reader.h>
using namespace lemon;
int main() 
{
  typedef SmartGraph Graph;
  Graph g;
  Graph::EdgeMap<int> w(g);
  
  GraphReader<Graph>(g,std::cin).edgeMap("weight",w).run();
  MaxWeightedMatching<Graph> mwm(g,w);
  //MaxWeightedPerfectMatching<Graph> mwm(g,w);
  mwm.run();
    Attachments (4)
Change History (9)
Changed 17 years ago by
| Attachment: | matching.lgf added | 
|---|
Changed 17 years ago by
| Attachment: | 52c04a2a652c.patch added | 
|---|
Changed 17 years ago by
| Attachment: | heap_unionfind_fix.bundle added | 
|---|
Application of patch on proper base
comment:2 follow-up: 3 Changed 17 years ago by
Replying to deba:
The patch [52c04a2a652c] solves the problem.
Please use the heap_unionfind_fix.bundle instead of the previous patch. It is applied on the proper base, and it also contains the merges.
comment:3 Changed 17 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | new → closed | 
Replying to deba:
Replying to deba:
The patch [52c04a2a652c] solves the problem.
Please use the heap_unionfind_fix.bundle instead of the previous patch. It is applied on the proper base, and it also contains the merges.
The changesets went to the corresponding changesets, see [561be42c4b99], [bb022b8f9c8f] and [6c1ac149ebf8].
comment:4 follow-up: 5 Changed 17 years ago by
| Resolution: | fixed | 
|---|---|
| Status: | closed → reopened | 
I made a more complex data structure test for HeapUnionFind?. Unfortunately, it indicated a bug in the previous bugfix. Finally, the problem is fixed in the heap_unionfind_bugfix2.bundle file.
comment:5 Changed 17 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | reopened → closed | 
Replying to deba:
I made a more complex data structure test for HeapUnionFind?. Unfortunately, it indicated a bug in the previous bugfix. Finally, the problem is fixed in the heap_unionfind_bugfix2.bundle file.
They are in the corresponding branches, see [81d40f1c850c], [1cc2818c03c0] and [1229dc2f1d36].


Bug fix