eyeonus / Trade-Dangerous

Mozilla Public License 2.0
96 stars 31 forks source link

Looks like `--progress` no longer works #151

Closed ultimatespirit closed 2 months ago

ultimatespirit commented 2 months ago

With latest version looks like --progress doesn't result in outputting the per-hop progress anymore, just processing time followed by all the output at once.

Full command (using the same test area as my previous issue):

$ ./trade.py run --from "7 andromedae/papin city" --credits 70m --capacity 192 --ly-per 10.57 --routes 2 --hops 5 --progress --fc N --age 2 -J -vv --summary --jumps-per 3 --empty-ly 15.17 --to "7 andromedae/papin city" --end-jumps 5

Results in no per-hop progress printing, unlike before on the 11.1.0 version (patched to fix the double close db).

ultimatespirit commented 2 months ago

Oh, never mind. Did some more testing and it does work it's just that it calculated the route so quickly after the initial setup period without printing, that it printed everything basically instantly. With a more cumbersome calculation it showed progress bars.