Skip to content

Commit

Permalink
Analytics: Remove unmaintained statistics; remove API usage
Browse files Browse the repository at this point in the history
This commit does a couple of things at once:

- Removes damage and buff statistics from EVTCAnalytics. They were subtly broken (damage did not consider iff, buff simulation never got updated for extensions and stack ids). These statistics were originally planned to be used for our own HTML logs, but Elite Insights does those better. EVTCAnalytics still aims to provide basic data efficiently, but Elite Insights should be considered for more in-depth statistics.

- GW2API classes used only by Rotation Comparison EVTCAnalytics are moved to Rotation Comparison. This removes all library dependencies from EVTCAnalytics.
  • Loading branch information
Sejsel committed Jul 16, 2021
1 parent e50e67c commit 3e46e0a
Show file tree
Hide file tree
Showing 28 changed files with 26 additions and 1,406 deletions.
2 changes: 1 addition & 1 deletion ArcdpsLogManager/ManagerForm.cs
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public sealed class ManagerForm : Form
private LogFinder LogFinder { get; } = new LogFinder();

private LogAnalytics LogAnalytics { get; } =
new LogAnalytics(new EVTCParser(), new LogProcessor(), log => new LogAnalyzer(log, null));
new LogAnalytics(new EVTCParser(), new LogProcessor(), log => new LogAnalyzer(log));

private ApiProcessor ApiProcessor { get; }
private UploadProcessor UploadProcessor { get; }
Expand Down
9 changes: 3 additions & 6 deletions EVTCAnalytics.Benchmark/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,14 @@
using System.Diagnostics;
using System.IO;
using System.Linq;
using GW2Scratch.EVTCAnalytics.Parsed;
using GW2Scratch.EVTCAnalytics.Processing;
using GW2Scratch.EVTCAnalytics.Statistics;

namespace GW2Scratch.EVTCAnalytics.Benchmark
{
internal class Program
{
public static EVTCParser Parser { get; set; } = new EVTCParser();
public static LogProcessor Processor { get; set; } = new LogProcessor();
public static GW2ApiData ApiData { get; set; } = null;

public static void Main(string[] args)
{
Expand Down Expand Up @@ -50,11 +47,11 @@ public static void Main(string[] args)

GC.Collect();

MeasureTimes(filename, Parser, Processor, ApiData, Console.Out);
MeasureTimes(filename, Parser, Processor, Console.Out);
}
}

private static void MeasureTimes(string filename, EVTCParser parser, LogProcessor processor, GW2ApiData apiData, TextWriter outputWriter)
private static void MeasureTimes(string filename, EVTCParser parser, LogProcessor processor, TextWriter outputWriter)
{
var stopwatch = Stopwatch.StartNew();
var log = parser.ParseLog(filename);
Expand All @@ -67,7 +64,7 @@ private static void MeasureTimes(string filename, EVTCParser parser, LogProcesso
var processedTime = stopwatch.Elapsed;

stopwatch.Restart();
var analyzer = new LogAnalyzer(processedLog, apiData);
var analyzer = new LogAnalyzer(processedLog);
var result = analyzer.GetResult();
var duration = analyzer.GetEncounterDuration();
var mode = analyzer.GetMode();
Expand Down
5 changes: 0 additions & 5 deletions EVTCAnalytics/EVTCAnalytics.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,4 @@
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<LangVersion>latest</LangVersion>
</PropertyGroup>
<ItemGroup>
<PackageReference Include="Newtonsoft.Json" Version="12.0.2" />
<PackageReference Include="RestSharp" Version="106.6.10" />
<PackageReference Include="RestSharp.Newtonsoft.Json" Version="1.5.1" />
</ItemGroup>
</Project>
Loading

0 comments on commit 3e46e0a

Please sign in to comment.