travellermap icon indicating copy to clipboard operation
travellermap copied to clipboard

Sector data contains invalid / misspelled / unknown allegiance codes

Open pmoule opened this issue 3 years ago • 5 comments

Sector data contains a lot of invalid / misspelled / unknown allegiance codes. In surveys this leads to a not complete Alleg section in metadata.

Expectation is, that used codes in a sector's world data should be displayed as allegiances in Alleg section in sector data. As far as I can see, this is exactly your implementation.

Types of invalid codes are different. Sometimes these are simply misspelled wrt casing. Others are completely unknown and not defined in related sector xml. In a different case, allegiance code is different from the one in 'allegiance_code.tab' (e.g. Nx and NaXX). In the latter case generally known code should be used.

Examples:

This pull request fixes all the above in most of the existing sector data.

pmoule avatar Jan 21 '23 14:01 pmoule

I've been going through and fixing several of these c/o some updated tooling. Ones that will require more research and/or contacting the author to determine intent:

Faraway:

  • Ares - multiple
  • Far Home - Ns
  • Marischal - JD

M1105:

  • Fulani - Nav
  • Harea - Nx, Ns, NX
  • Holowon - He
  • Numbis - J1
  • Phlange - EC

M1120:

  • Lishun - Cv

M1201:

  • Reft - TB
  • Spica - H1
  • Windhorn - Ts
  • Zarushagar - Vc

M1248:

  • Gvurrdon - Cc
  • Massilia - Gc

inexorabletash avatar Jan 23 '23 01:01 inexorabletash

ah, great work! This fixes major parts from the original PR's findings.

pmoule avatar Jan 23 '23 07:01 pmoule

M1201:

  • Mesh - Tp

M1105:

  • Phlange - NA (could be Na)

pmoule avatar Jan 23 '23 20:01 pmoule

Ares and Marischal have corrections now.

inexorabletash avatar Feb 18 '23 16:02 inexorabletash

That last revision updates:

Faraway / Far Home - define Ns M1105 / Fulani - fix Nav to NaXX M1105 / Harea - fix NX to Nx, define Ns/Nx

inexorabletash avatar Apr 07 '23 02:04 inexorabletash