Bump dropwizardVersion from 2.0.29 to 2.1.0 (triplea-game#10403)

Bumps `dropwizardVersion` from 2.0.29 to 2.1.0.

Updates `dropwizard-auth` from 2.0.29 to 2.1.0

Updates `dropwizard-core` from 2.0.29 to 2.1.0

Updates `dropwizard-jdbi3` from 2.0.29 to 2.1.0

Updates `dropwizard-testing` from 2.0.29 to 2.1.0

---
updated-dependencies:
- dependency-name: io.dropwizard:dropwizard-auth
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: io.dropwizard:dropwizard-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: io.dropwizard:dropwizard-jdbi3
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: io.dropwizard:dropwizard-testing
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Refactor code for querying unit tech for players. (triplea-game#10394)

* Refactor code for querying unit tech for players.

This change makes TechTracker an actual instance class on GameData and cleans up the APIs to retrieve tech information from UnitAttachment.

This is a pure code re-organizing change, no functional changes.

Besides improving the API, this should pave way for allowing the TechTracker to cache this information and only update it when tech changes, which should speed up the engine quite a bit - since very basic things like getting unit attack / defense / movement requires expensive tech look up operations currently (which shows up in profiles).

Add an "Image Cleaner" map making tool. (triplea-game#10401)

* Add an "Image Cleaner" map making tool.

This tool can save effort for map makers by handling map image files that are not as strict as the ones needed by the polygon grabber. It does the following:
1. Normalizes colors by changing pixels that aren't white to black.
2. "Fills in" small regions (below `minimumRegionSize`) with black pixels.
3. Removes "unnecessary" black pixels by turning them to white, making the resulting lines between regions have a thickness of 1 pixel.

The new functionality is provided via new menu items from the polygon grabber.

Fix wrong tags on subsequent pushes to master (triplea-game#10400)

Fix issue preventing units being unloaded from transports. (triplea-g…

…ame#10381)

* Fix issues with unloading transports by selecting equivalent units when needed.

* Use stream api.

* Restore if statement.

* Fix comment.

* Improve test coverage of function.

Fix issue preventing units being unloaded from transports. (triplea-g…

…ame#10381)

* Fix issues with unloading transports by selecting equivalent units when needed.

* Use stream api.

* Restore if statement.

* Fix comment.

* Improve test coverage of function.

Bump com.diffplug.spotless from 6.5.0 to 6.5.1 (triplea-game#10382)

* Bump com.diffplug.spotless from 6.5.0 to 6.5.1

Bumps com.diffplug.spotless from 6.5.0 to 6.5.1.

---
updated-dependencies:
- dependency-name: com.diffplug.spotless
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Fix spotless violations

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: RoiEX <[email protected]>

Use an in-memory cache in ImageFactory by string key. (triplea-game#1…

…0356)

* Use an in-memory cache in ImageFactory.

This prevents excessive ImageIO.read() calls on images. I noticed this showed up in execution time profiles when looking at the performance of handling mouse events during the move phase, which spent a lot of time in updateRouteAndMouseShadowUnits() that eventually call down to this for country small flags.

* Fix format.

* Cache by key.

* Remove commented out code.

Added World War II Empty Start (triplea-game#10397)

Move 1.8-compatibility replaceAll logic to GameParser. (triplea-game#…

…10391)

addAttachment() is called from other places than the game parser, which all use constants from the codebase that do not need replacement. This avoids those needing to go through this codepath.