-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Signalling of Live vs Legacy in EDMarketConnector data export #103
Comments
I'm sorry, I haven't been keeping up with the news, so I don't know about the Live/Legacy thing. Let me go read up on that real quick.... Currently, the "official" means of obtaining crowd-sourced prices data is by using the eddblink import plugin, which uses the EDDB API files to build TD's database. The most likely use case for importing .prices files created by EDMC, in my estimation, is a person wanting to import their own data, I'm not aware of people passing around .prices files to each other or posting them online. If there is a backwards-compatible way to mark legacy prices files as such, that would be ideal, but to be honest I don't even know what the format is because I never deal with them. When I last played the trading game in ED, I used EDMC to submit the station info to EDDN and ran my eddn-listener to update TD, so I never actually used EDMC's export feature, myself. TD can be run on multiple databases, so at least for personal-only market data, running TD in ".../EDLive" and ".../EDLegacy" folders, depending on which galaxy one is playing in, is sufficient to workaround any potential issues. If crowd-sourced Legacy data becomes a thing (such as if EDDB starts tracking it as well as the Live galaxy), then I'll revisit this. Thanks for the heads up, I appreciate it. |
It's the Trade Dangerous defined format, as per that link to the TD Wiki within this very repo that I included in OP. I was asking if you wanted to extend that to include an indicator of 'source galaxy'.
In that case I'll just consider it as a "user problem" if anyone is actually using TD in that manner. I'm almost tempted to just remove the code and see if anyone screams ;) .
Unlikely. None of the big sites want the headache of potentially doubling up all of the data. They're all going "Live only". I'll consider this a non-problem unless and until an EDMC user actually complains/asks. |
I did not know that, I thought someone else came up with the format. I'm a third generation maintainer on this code, I just fix things when TD breaks I guess make the first line be '#!legacy' if it ever becomes needed, but I agree this is most likely a non-problem |
Heh, and I'm at least 2nd generation maintainer of EDMC. Btw, it's occurred to me that because of the Live/Legacy split and the main third-party sites only supporting Live, it's possible there might now be an increase in players interested in such "provide and maintain your own data" tools. But, yes, let's just leave it up to the users to not make a mess of their own data. |
Mmm, you may be right Imma mention the multiple database thing in TD's thread on the ED forum in hopes that prevents one or more data messes, just 'cause I'm nice like that |
EDMarketConnector has the option to save "Market data in Trade Dangerous format file" as documented at https://github.com/eyeonus/Trade-Dangerous/wiki/Price-Data
Does TD want to define a way to signal if the data came from the Live or Legacy galaxy ? I'm just adding support for the Legacy CAPI host/endpoint now, so users of this would have to be careful if they play in both galaxies.
The text was updated successfully, but these errors were encountered: