Open tajtiattila opened 9 years ago
The system still has a major economy but it does make sense to add the station economies and additional info too.
The main problem is nothing can be considered as set in stone until the December 16th release. The galaxy may get a final rebuild at that point.
True. But the data structure (ie. what we have in release) will likely be closer to the one we have in gamma, so it might still make sense to update the json format. Would moving the economy from the system itself to all the stations within the system make sense?
The galaxy map still shows a system level economy so I think we should keep the system level value in the json data. We can add an economy field for each station as well. Harbinger, what's your opinion on whether we should make these values arrays of strings rather than simple strings?
For the time being I added them as arrays for my own json format, because I think it's easier to use both from go and javascript.
I have no problem with adapting the format, it was merely a proposed format anyway. Things like the maximum pad size available are irrelevant now that all outpost types can take a medium ship, just knowing that the base is an outpost is sufficient to know that you can land a medium vessel there whereby if it's a starport you can land a large vessel there.
A lot of the information about bases can be easily gained from the System Map now as it lists whether there is a commodities market, outfitting, shipyard etc.
The only things that are a mystery in terms of the system map are whether or not there is a black market and the distance from the jump in point.
Whatever format we end up choosing for the systems.json file I intend to support it, if commanders end up adding system and station info to the star chart (I must admit I'm not a great stellar cartographer myself). My own format is very similar, but not identical to the systems.json.
Now we have multiple station economies, this means economy for the system does not make much sense anymore.
I'd suggest to have an economy field in the station instead, likely an array of strings instead of a single string value. Additionally, stations seem to have a population field (string: tiny..huge) now.