-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Write Analyzer for Naive vs CNN #1
Comments
Relevant code for analysis here was made in this commit: 8363468. All experiments were verified to be running on a Xeon-P8 full node so there's no resource contention. We used our postprocessing method to extract the edges from the CNN output. Previously, we ran the experiments in parallel, utilizing
We previously noticed some stochasticity in parallel runs, and thus we wanted to isolate that effect. To isolate this effect, we ran all experiments henceforth in serial. True enough, running the above 13 samples again resulted in the following test samples using a 10K cost edge:
Now, this got me really curious about this stochasticity. Intuitively, it makes sense since we are limiting the solver to 30 seconds and perhaps the solver got stuck at that local minima with that 10K cost edge - we didn't give it a chance to jump out of it. I decided to run it again on the 5 samples above and this resulted in the following test samples using a 10K cost edge:
It reduced again. I wasn't sure I was seeing the right thing, and thus decided to copy paste the exact output from the terminal below:
I ran I changed the Google OR Tools parameters from 30 seconds to 60 seconds, and true enough, both these samples had the 10K edge removed. However, the naive method performed slightly better, by a slight advantage (2 and 20 units). I then decided to try another parameter. What if the 10K penalty is too low? I set it to 1M and both samples The conclusion here is that we should try running the entire analysis for a minute in parallel. |
decoder.py
has too much code bloat. It needs to be refactored out into an analyzer.num_cities
, file paths, etc.) and outputs all the routes a particular node can go to. See figure below.The text was updated successfully, but these errors were encountered: