EDCD / EDDI

Companion application for Elite Dangerous
Other
437 stars 81 forks source link

unknownmineral / unknownrefinedmineral #2594

Closed nepomuk16321 closed 5 months ago

nepomuk16321 commented 5 months ago

What's Wrong (please be as specific as possible)

Hello everyone, I often see the following 4 lines in the eddi.log:

2024-02-08T17:29:08 [Info] CommodityDefinition:CommodityDefinitionFromEliteID Unrecognized Commodity Definition EliteID 129022408 2024-02-08T17:29:08 [Info] CommodityDefinition:CommodityDefinitionFromEliteID Unrecognized Commodity Definition EliteID 129022409 2024-02-08T17:29:08 [Info] MarketInfoItem:ToCommodityMarketQuote Commodity definition error: unknownmineral 2024-02-08T17:29:08 [Info] MarketInfoItem:ToCommodityMarketQuote Commodity definition error: unknownrefinedmineral

Expected

[Describe what you expected to happen]

Observed

[Describe what actually happened. Feel free to add screenshots, link to videos, etc to help explain what happened]

Steps to reproduce

  1. [Give the fullest and most reproducible steps you can]
  2. [The more reproducible, the better our chances of fixing it]

Configuration

My Investigation

Investigation Notes

[Any investigation you have done, tests using default scripts and personalities, regression tests against earlier versions, etc]

EDDI Logs

[Any log files providing additional details about what happened. These are located at %APPDATA%/EDDI and have the extension ".log"] eddi.log

Player journals

[These are not required if EDDI Logs are verbose but may be helpful otherwise. They are located at [your Saved Games folder]/Frontier Developments/Elite Dangerous/. Please only provide excerpts or attach files from the play session where the issue occurred.]

Journal.2024-02-08T170654.01.log

Darkcyde13 commented 5 months ago

This looks to me like there are a couple of commodities in the market of that station that EDDI doesn't recognise. The error in the EDDI log corresponds with a market update { "timestamp":"2024-02-08T17:29:08Z", "event":"Market", "MarketID":128091896, "StationName":"Putzi OPK's Heaven", "StationType":"Coriolis", "StarSystem":"Bohmshohm" }

My guess would be one of the Titan samples maybe? Or possibly one of the new materials from a spire site? Have you been doing any Thargoid activities, collecting Thargoid items?

nepomuk16321 commented 5 months ago

Yes, I think you're right. And no, I have nothing to do with the Thargoids. I just want to point out to T'Kael that there are apparently new goods that are not recognised by EDDI. ;-)

Darkcyde13 commented 5 months ago

I've quickly hopped over to that station, and tried to see what happens. I too get the same, so I searched through the EDDI log for a market update, and found these two: {"id":129022408,"Name":"$unknownmineral_name;","Name_Localised":"Impure Spire Mineral","Category":"$MARKET_category_salvage;","Category_Localised":"Salvage","MeanPrice":31986,"Consumer":false,"Producer":false,"Rare":false},{"id":129022409,"Name":"$unknownrefinedmineral_name;","Name_Localised":"Semi-Refined Spire Mineral","Category":"$MARKET_category_salvage;","Category_Localised":"Salvage","MeanPrice":158421,"Consumer":false,"Producer":false,"Rare":false};

So it seems they are materials from the spire sites, and I think that explains why EDDI doesn't know about them. I think they came out after the last EDDI update.

Still, the above should make it easy for the team to add these in now. :)

Tkael commented 5 months ago

Correct, these are newer spire site commodities that were added after EDDI's last release. Both are already coded and will be supported in the next release. :-) https://github.com/EDCD/EDDI/blob/8641d1049d1f28ad33ccb062fc896592aa736d67/DataDefinitions/CommodityDefinition.cs#L443-L444

nepomuk16321 commented 5 months ago

Thanks !