Prefer to not name return values

gofmt

A bunch more test cases.

Fix a legal-path-not-checked eden warp bug

You don't need to have a free cargo slot after warping if the hold is empty.

Move the BeaconOn check out from the inner loop

There's no reason to check this more than once here.

Correct spelling

Only print two significant figures for arms costs

Wrap and check accesses to p2i and c2i

This catches misspellings of planet and commodity names.  Otherwise,
they were defaulting (to Stockworld and nothing respectively).

A simple whole-run testing methodology

Add test.sh for comparing against historical recorded runs.

I have not done any extra analysis on cases recorded here.  Rather,
I just want to know when/if any of this output changes.

I couldn't figure out a good way to include these tests within the
"go test" framework.  The tricky bit was figuring out how to set
unmentioned flags back to their default values between runs.  I
deemed adding an 'option' object of some kind built from command
line flags but also buildable by tests to not be worth the complexity
at this time.

Generalize json-slurping

More named types: LogicalIndex

More named types: Value

Start using named types: PhysicalIndex

Update and clarify some comments

Add an extra_stats flag so they can be disabled

Move the cost reporting out of main

Cost of --end

Print visit costs with fixed width for vertical alignment.

Calculate visit costs

Also, add a test for Commas.  It was fine, but I got confused
after passing it a negative number.

Print total cost of fighters and shields

Display marginal benefit of using end_edens

Fix test: Add trading dimension.

Update for go v1

--tomorrow_weight feature, with data

Allow specifying a starting cargo

Show progress less frequently

Nirvana's beacon is off by default, yes?

Don't use 0 for uninitialized indexes

0 is a valid index.  It can legitimately be the head of the optimal
path.  When it was the head of the optimal path, DescribePath() would
print nothing.  This fixes that bug.

UI: Explain required usage when run without args

Fix sloppiness while considering private planets

Observed beacons in a game-in-progress of interest

TODO: Figure out the defaults and distribute that.

gofmt

Fix one cargo unit short when eden jumping bug.

Fix (tighten) sanity check in EncodeIndex()

And fix the revealed bug when specifying --fuel=0

And add a sanity check for addr == other at the end of CellValue.

Explicit -> implicit iteration.

This eliminates the whole class of bugs where the iteration order is
inconsistent with the data model.  It also simplifies the code by simply
removing the whole 9-nested-layers of for loops along with the barnacles
that were growing there (multiple passes for selling and then buying,
different iteration order depending on whether the planet sells Eden
warps, etc.)

The downside: Parallelism is lost.  This is now a single-threaded
program.

Consider not using all the fuel.

Private flag

Performance!

Implement --visit

Follow the reverse energy gradient on Eden-selling worlds.

This fix is necessary to find the optimal strategy when Eden's
beacon is active.

Implement --drones and --batteries.

Consistent hyper-jump terminology.

Using an eden warp doesn't give you cargo.

Implement Eden buying

More action descriptions

Print raw address change when a nice description is unavailable.

Implement --end

Don't eden warp to planets that sell eden warp units.

The Codex forbids this.

Also, only use one eden warp unit at a time.

Only warp to planets with their beacons on.

Read --end into a map

Fix flight plan logic (destination, not origin)

Fix --cloak

Buy from the right cell (no UnusedCargo when empty)

Start working on --cloak.

Fix bug: account for the empty hold

Print large numbers with commas

Get the right stuff on stdout vs stderr

Relative prices from the data file are percentages

Divide them by 100 before using them.

DescribePath()

FindBestState().

This can now emit an answer.  It is a factor 80x off in a simple
test.  :)

More haphazard development of untried code.

More haphazard development

Fix StateTableSize().

Fix visits and flight_plan to be [] instead of [""] when not specified.

Add some sanity checking in EncodeIndex().

Move table allocation out of FillStateTable1() so that it can be run
through FillStateTable1() multiple times for verification.

More documentation/thoughts on state table size.

Add an empty InitializeStateTable() to remind me to do that.

Start poking at FillStateTableCell().  This approach won't work.
FillStateTable2() cannot just process one cell completely and then move
onto the next.  Some types of processing (eg: arriving, selling) have to
be performed to all the cells before the next round of actions (buying).

Notes on FillStateTableCell()

gofmt

FillStateTable()

Some safety against future modifications in DimensionSizes().

Make DimensionSizes() more resilient against dimension reordering.

Tweak flags

gofmt

Add a lot of code that isn't exercised yet.

Index commodities by their name.

Also, use initial_funds and max_quantity in TradeValue()

Use pointers more and indexes less

FindBestTrades()

Convert local prices to a flat map

Planeteer 2: Data and data loading.