msarilar / EDEngineer

An overlay to track Elite Dangerous blueprints progress in real time
MIT License
503 stars 174 forks source link

Install error #606

Closed Wullailhuit closed 3 years ago

Wullailhuit commented 3 years ago

I'm trying to install the latest version (using the exe) , it gets as far as the large 'ED ENGINEER' box with rotating balls a large pause and then throws this up in a seperate box.

I've removed all the data , uninstalled it etc etc as suggested in the 'have issues list' to no avail.

System.AggregateException: One or more errors occurred. ---> System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary. at System.Collections.Generic.Dictionary2.get_Item(TKey key) at EDEngineer.Models.Operations.UpgradeOperation.GetEquipment(State state) at EDEngineer.Models.Operations.UpgradeOperation.Mutate(State state) at EDEngineer.Views.CommanderViewModel.MutateState(JournalEntry entry) at EDEngineer.Views.CommanderViewModel.ApplyEventsToSate(IEnumerable1 allLogs) at EDEngineer.Views.MainWindowViewModel.ApplyEvents(Tuple2 logs) at EDEngineer.Views.MainWindowViewModel.LoadState() at EDEngineer.Views.MainWindowViewModel..ctor(Languages languages, String directory) at EDEngineer.Views.MainWindow.<>c__DisplayClass3_0.<.ctor>b__0() at System.Threading.Tasks.Task.InnerInvoke() at System.Threading.Tasks.Task.Execute() --- End of inner exception stack trace --- ---> (Inner Exception #0) System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary. at System.Collections.Generic.Dictionary2.get_Item(TKey key) at EDEngineer.Models.Operations.UpgradeOperation.GetEquipment(State state) at EDEngineer.Models.Operations.UpgradeOperation.Mutate(State state) at EDEngineer.Views.CommanderViewModel.MutateState(JournalEntry entry) at EDEngineer.Views.CommanderViewModel.ApplyEventsToSate(IEnumerable1 allLogs) at EDEngineer.Views.MainWindowViewModel.ApplyEvents(Tuple2 logs) at EDEngineer.Views.MainWindowViewModel.LoadState() at EDEngineer.Views.MainWindowViewModel..ctor(Languages languages, String directory) at EDEngineer.Views.MainWindow.<>c__DisplayClass3_0.<.ctor>b__0() at System.Threading.Tasks.Task.InnerInvoke() at System.Threading.Tasks.Task.Execute()<---

msarilar commented 3 years ago

would you mind uploading your latest logs here? I think you have an equipment that is not referenced in https://github.com/msarilar/EDEngineer/blob/master/EDEngineer/Resources/Data/equipment.json added by @jpacelli62

maybe the parsing should be made more robust (not crash if the equipment isn't found in the json)

I can look into this later today

Wullailhuit commented 3 years ago

Journal.210608142638.01.log Journal.210608111849.01.log Journal.210608060109.01.log

PiezPiedPy commented 3 years ago

I'm getting the exact same thing.

jpacelli62 commented 3 years ago

According to the logs, I only see an update of the Dominator suit but the application does not crash for me. Could you also send your latests logs @PiezPiedPy please?

msarilar commented 3 years ago

won't fix the issue but I pushed an update to avoid crashing the app, you'll get a popup instead (ctrlC the popup, it will copy the message to the clipboard so you can paste it here)

PiezPiedPy commented 3 years ago

Just tried and the popup gave the string 'wpn_m_submachinegun_kinetic_fauto'

interestingly I had also just upgraded my C-44 to G2 just before installing the engineer update.

another thing is that the capitalization of the string in the equipment.json file does not match the string in the error, see https://github.com/msarilar/EDEngineer/blob/63a429b385d485689ba9301a9a20f9e0238ed052/EDEngineer/Resources/Data/equipment.json#L46

jpacelli62 commented 3 years ago

I'm merging for a merge request, i'ts a case issue. will be fixed soon

PiezPiedPy commented 3 years ago

I assumed it was a case issue, thx for the speedy response 😎

Wullailhuit commented 3 years ago

Warning

Unknown equipment wpn_s_pistol_plasma_charged. Ctrl+C while this messagebox is on focus then kindly report this on github

OK

PiezPiedPy commented 3 years ago

logs as requested

Journal.210607174327.01.log Journal.210604005752.01.log