liangxinedu / NeuroLKH

70 stars 17 forks source link

Segmentation Fault in your _LKH.main #1

Closed gnana-sai-shipsy closed 2 years ago

gnana-sai-shipsy commented 2 years ago

Running swig_test.py results in multiple python shell instances to open up. _LKH.main has this error Segmentation Fault (Core dumped) . Do help

yzhang-gh commented 2 years ago

I have also seen this issue. It can be solved by renaming the main function https://github.com/yzhang-gh/NeuroLKH/commit/05ff7de72dfba0bee2d03b0ce6bebee6a636e407

gnana-sai-shipsy commented 2 years ago

I see, so that means your fork of the repo is being maintained while this one isn't?

Also I had another question (Do excuse me since they should be a separate issue)

Thanks @yzhang-gh

yzhang-gh commented 2 years ago

This repo doesn't seem to be actively maintained, and I was just trying to replicate their experiments recently (so my understanding of the code may be limited).

The numbers compare the found tour length and running time of LKH and NeuroLKH, either under a given number of trials or under a time limit. (You can check Table 1 of the paper.)

As for the CVRPTW code, I didn't really look into it. If you are talking about the neural network part, it is adapted from https://github.com/chaitjo/graph-convnet-tsp as mentioned at the end of the README,

gnana-sai-shipsy commented 2 years ago

Great insights! Thank you! @yzhang-gh