[BUG]Planet.full() crashes.

[MaximDotBryuzgin]

Description

Upon running fio,Planet.full(), after a delay of a few minutes of the error printing to console, pydantic throws many "missing" ValidationErrors.

Reproduction

Call the function.

from fio_wrapper import FIO
fio = FIO()
data = fio.Planet.full()

Expected behavior

The full data for every planet is returned without the error crashing the program.

Versions

Python

Python 3.12.3

FIO Wrapper

Version: 1.3.5

Additional context

The error message:

#first traceback
Traceback (most recent call last):
  File "<MyPythonScriptPath>\test.py", line 3, in <module>
    data = fio.Planet.full()
  File "<MyPythonLibPath>\Lib\site-packages\fio_wrapper\endpoints\endpoints_v1\planet.py", line 82, in full
    return PlanetFullList.model_validate(data)
  File "<MyPythonLibPath>\Lib\site-packages\pydantic\main.py", line 596, in model_validate
    return cls.__pydantic_validator__.validate_python(
pydantic_core._pydantic_core.ValidationError: 37395 validation errors for PlanetFullList

#the following repeats
0.GovernorId
  Field required [type=missing, input_value={'Resources': [{'Material...-05-19T06:37:29.153684'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.9/v/missing
0.GovernorUserName
  Field required [type=missing, input_value={'Resources': [{'Material...-05-19T06:37:29.153684'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.9/v/missing
#...
#112000 lines of this repeating for every field, 4154 times

[jplacht]

Hi there. Is it only the governor information? Probably due to the change that governments are entities instead of individual users now.

[MaximDotBryuzgin]

Heyo. I double checked and it seems to happen for GovernorUserName, GovernorCorporationId, GovernorCorporationName, GovernorCorporationCode, CollectorId, CollectorName, CollectorCode and DistanceResults. The number on the left (the "0." in "0.GovernorId") increments every repetition up to 4154 (I assume the amount of planets in the game).

[jplacht]

Yes, would you like to make a PR and do the change, @MaximDotBryuzgin?

You would only need to adapt the class PlanetFull(Planet) in https://github.com/PRUNplanner/fio_wrapper/blob/master/fio_wrapper/models/planet_models.py

Here is an example of the current full planet payload from FIO as well - without the sub-models.

{
    "Resources":
    [],
    "BuildRequirements":
    [],
    "ProductionFees":
    [],
    "COGCPrograms":
    [],
    "COGCVotes":
    [],
    "COGCUpkeep":
    [],
    "PlanetId": "7f1135f5d7792a058c8be66e7cbcb536",
    "PlanetNaturalId": "OT-580b",
    "PlanetName": "Montem",
    "Namer": null,
    "NamingDataEpochMs": 0,
    "Nameable": false,
    "SystemId": "49b6615d39ccba05752b3be77b2ebf36",
    "Gravity": 0.922335147857666,
    "MagneticField": 0.7182995080947876,
    "Mass": 7.797300945837227E+24,
    "MassEarth": 1.3056432008743286,
    "OrbitSemiMajorAxis": 139417977000.0,
    "OrbitEccentricity": 0.03147958219051361,
    "OrbitInclination": 0.04501096159219742,
    "OrbitRightAscension": 0.0,
    "OrbitPeriapsis": 0.0,
    "OrbitIndex": 2,
    "Pressure": 0.9165661931037903,
    "Radiation": 1.4921821515009263E-23,
    "Radius": 7584996.5,
    "Sunlight": 1621.1890869140625,
    "Surface": true,
    "Temperature": 3.772547960281372,
    "Fertility": -0.19999998807907104,
    "HasLocalMarket": true,
    "HasChamberOfCommerce": true,
    "HasWarehouse": true,
    "HasAdministrationCenter": true,
    "HasShipyard": true,
    "FactionCode": "NC",
    "FactionName": "NEO Charter Exploration",
    "GoverningEntity": "f13aa1e0a90f4a9c972ac479115406e9",
    "CurrencyName": "NCE Coupons",
    "CurrencyCode": "NCC",
    "BaseLocalMarketFee": 0.0,
    "LocalMarketFeeFactor": 3.0,
    "WarehouseFee": 100.0,
    "EstablishmentFee": 0.0,
    "PopulationId": "a6f1ef590c2110aa3195e97ef931d122",
    "COGCProgramStatus": "ACTIVE",
    "PlanetTier": 0,
    "UserNameSubmitted": "-BUTTERFLY-",
    "Timestamp": "2024-11-14T14:57:08.846522"
}

[MaximDotBryuzgin]

Sorry, forgot to respond. I'll do this if I have the spare moment, though I don't think I will have one within the next month or so.