wouterkool / attention-learn-to-route

Attention based model for learning to solve different routing problems
MIT License
1.04k stars 337 forks source link

About your modification to python3.8 #26

Open bywbilly opened 3 years ago

bywbilly commented 3 years ago

This is relevant to #16. You remove the super of the customized dataloader, which causes the code cannot work well on python3.7. I can run pretty well with the old code. Just want to let you know about this issue.

wouterkool commented 3 years ago

Hi! Thanks for notifying. I looked into the issue but am unable to find a way to make the NamedTuple with overriding __getitem__ work in both Python 3.7 and 3.8. I am open to suggestions. For now I guess it's best to upgrade to 3.8 or roll back this change manually.