Really unsimplify this time

[lispandfound]

Previously I removed the shapely simplification. This change missed an additional change I made to do the only simplification that is actually necessary: removing repeated points in the fault trace. Without that necessary check the script will crash when generating a database. This PR adds the missing repeat point removal. This only affects one fault in the NSHMDB, Irishman Creek, Subsection 2. You now get a warning that some simplification has occurred, along with a diff of the simplified trace vs the old trace.

Warning: Fault trace for Irishman Creek, Subsection 2 was altered.
Old: (5131154.191632377, 1396502.863913299)  (5131154.191632377, 1396502.863913299)  (5128105.839916997, 1390381.8099286489)  (5125765.814896873, 1386520.0119176009)
New:                                         (5131154.191632377, 1396502.863913299)  (5128105.839916997, 1390381.8099286489)  (5125765.814896873, 1386520.0119176009)

As you you can see, the fault trace in this instance has been simplified because the first point in the Irishman Creek fault matches the second point in the Irishman Creek fault. No other warnings should be produced in this generation.

[lispandfound]

I've also added a full test run of the NSHMDB to the test suite to ensure that this cannot happen again. It, at this stage, just checks that you can generate a database. It'd be nice to actually extract this DB as a build artifact and publish a release every time it is merged to master.

[Copilot]

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

tests/test_nshmdb_generator.py:25

• [nitpick] Consider asserting that the warning message is produced when fault trace simplification occurs to improve test coverage for the new functionality.

def test_nshmdb_generator_runs(tmp_path: Path):