Merge pull request #232 from Linkaaaaa/skorvald

ArcdpsLogManager/Logs/Updates/LogDataUpdater.cs

@@ -195,6 +195,9 @@ x.Profession is Profession.Thief or Profession.Engineer or Profession.Ranger
 			                     && log.Encounter == Encounter.TempleOfFebe
 			                     && log.GameBuild >= GameBuilds.TempleOfFebeHealthFix,
 				"Fixed NM detection for Temple of Febe"),
+			new LogUpdate(log => log.ParsingVersion < new Version(1, 11, 1, 1)
+								 && log.Encounter == Encounter.Skorvald,
+				"Fix success detection for Skorvald when all players are dead while the boss is invulnerable at 1%"),
 			// When adding a new update, you need to increase the revision (last value) of the version in the .csproj file
 			// unless the version changes more significantly, in that case it can be reset to 0.
 		};

EVTCAnalytics/Processing/EncounterDataProvider.cs

@@ -308,6 +308,9 @@ private IEncounterData GetPvEEncounterData(Encounter encounter, Agent mainTarget
 					// Skorvald the Shattered is the same species in Challenge and Normal mode,
 					// unlike most other fractal CM encounters
 					return GetDefaultBuilder(encounter, mainTarget)
+						.WithResult(new AnyCombinedResultDeterminer(
+								new AgentKillingBlowDeterminer(mainTarget),
+								new BuffAppliedBelowHealthThresholdDeterminer(mainTarget, 0.9f, SkillIds.Determined895)))
 						.WithModes(new AgentHealthModeDeterminer(mainTarget, 5_550_000))
 						.Build();
 				}