Skip to content

Commit e23186e

Browse files
committed
Fixed bootstrapper and data loading
1 parent ca8f7f1 commit e23186e

14 files changed

Lines changed: 32 additions & 46 deletions

File tree

src/Sentiment/Wikiled.Sentiment.AcceptanceTests/Wikiled.Sentiment.AcceptanceTests.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
1212
<PackageReference Include="NLog" version="4.5.11" />
1313
<PackageReference Include="NUnit" version="3.11.0" />
14-
<PackageReference Include="Polly" version="6.1.2" />
14+
<PackageReference Include="Polly" version="7.0.1" />
1515
<PackageReference Include="protobuf-net" version="2.4.0" />
1616
<PackageReference Include="Snappy.NET" version="1.1.1.8" />
1717
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />

src/Sentiment/Wikiled.Sentiment.Analysis.Tests/Wikiled.Sentiment.Analysis.Tests.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
1313
<PackageReference Include="NLog" version="4.5.11" />
1414
<PackageReference Include="NUnit" version="3.11.0" />
15-
<PackageReference Include="Polly" version="6.1.2" />
15+
<PackageReference Include="Polly" version="7.0.1" />
1616
<PackageReference Include="protobuf-net" version="2.4.0" />
1717
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />
1818
<PackageReference Include="System.Reactive" version="4.1.2" />

src/Sentiment/Wikiled.Sentiment.Analysis/Processing/Persistency/DataLoader.cs

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
using Microsoft.Extensions.Logging;
22
using System;
33
using System.Collections.Generic;
4+
using System.IO;
45

56
namespace Wikiled.Sentiment.Analysis.Processing.Persistency
67
{
@@ -18,22 +19,28 @@ public DataLoader(ILoggerFactory loggerFactory)
1819

1920
public IDataSource Load(IDataSourceConfig source)
2021
{
21-
if (source == null)
22+
if (source == null ||
23+
(source.All == null && source.Negative == null && source.Positive == null))
2224
{
2325
throw new ArgumentNullException(nameof(source));
2426
}
2527

2628
if (source.All != null)
2729
{
28-
logger.LogInformation("Loading {0}", source.All);
29-
if (source.All.EndsWith(".xml", StringComparison.OrdinalIgnoreCase))
30+
if (File.Exists(source.All))
3031
{
31-
return new XmlDataLoader(loggerFactory.CreateLogger<XmlDataLoader>()).LoadOldXml(source.All);
32+
logger.LogInformation("Loading {0}", source.All);
33+
if (source.All.EndsWith(".xml", StringComparison.OrdinalIgnoreCase))
34+
{
35+
return new XmlDataLoader(loggerFactory.CreateLogger<XmlDataLoader>()).LoadOldXml(source.All);
36+
}
37+
38+
logger.LogInformation("Loading {0} as JSON", source.All);
39+
var data = new JsonDataSource(loggerFactory.CreateLogger<JsonDataSource>(), source.All);
40+
return data;
3241
}
3342

34-
logger.LogInformation("Loading {0} as JSON", source.All);
35-
var data = new JsonDataSource(loggerFactory.CreateLogger< JsonDataSource>(), source.All);
36-
return data;
43+
return new SimpleDataSource(loggerFactory.CreateLogger<SimpleDataSource>(), source.All, null);
3744
}
3845

3946
var loaders = new List<IDataSource>();

src/Sentiment/Wikiled.Sentiment.Analysis/Wikiled.Sentiment.Analysis.csproj

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@
1818
</PropertyGroup>
1919

2020
<ItemGroup>
21-
<PackageReference Include="Autofac" Version="4.8.1" />
21+
<PackageReference Include="Autofac" Version="4.9.0" />
2222
<PackageReference Include="Autofac.Extras.AggregateService" Version="4.1.0" />
2323
<PackageReference Include="Knuppe.SharpNL" Version="1.3.0" />
2424
<PackageReference Include="morelinq" Version="3.1.0" />
25-
<PackageReference Include="CsvHelper" version="12.1.1" />
25+
<PackageReference Include="CsvHelper" version="12.1.2" />
2626
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
27-
<PackageReference Include="Polly" version="6.1.2" />
27+
<PackageReference Include="Polly" version="7.0.1" />
2828
<PackageReference Include="protobuf-net" version="2.4.0" />
2929
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />
3030
<PackageReference Include="System.Reactive" version="4.1.2" />

src/Sentiment/Wikiled.Sentiment.Integration.Tests/Wikiled.Sentiment.Integration.Tests.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
1212
<PackageReference Include="NLog" version="4.5.11" />
1313
<PackageReference Include="NUnit" version="3.11.0" />
14-
<PackageReference Include="Polly" version="6.1.2" />
14+
<PackageReference Include="Polly" version="7.0.1" />
1515
<PackageReference Include="protobuf-net" version="2.4.0" />
1616
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />
1717
<PackageReference Include="System.Reactive" version="4.1.2" />

src/Sentiment/Wikiled.Sentiment.TestLogic.Shared/Wikiled.Sentiment.TestLogic.Shared.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
1414
<PackageReference Include="NLog" version="4.5.11" />
1515
<PackageReference Include="NUnit" version="3.11.0" />
16-
<PackageReference Include="Polly" version="6.1.2" />
16+
<PackageReference Include="Polly" version="7.0.1" />
1717
<PackageReference Include="protobuf-net" version="2.4.0" />
1818
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />
1919
<PackageReference Include="System.Reactive" version="4.1.2" />

src/Sentiment/Wikiled.Sentiment.Text.Tests/Wikiled.Sentiment.Text.Tests.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
<PackageReference Include="Newtonsoft.Json" version="12.0.1" />
1616
<PackageReference Include="NLog" version="4.5.11" />
1717
<PackageReference Include="NUnit" version="3.11.0" />
18-
<PackageReference Include="Polly" version="6.1.2" />
18+
<PackageReference Include="Polly" version="7.0.1" />
1919
<PackageReference Include="protobuf-net" version="2.4.0" />
2020
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />
2121
<PackageReference Include="System.Reactive" version="4.1.2" />

src/Sentiment/Wikiled.Sentiment.Text/Wikiled.Sentiment.Text.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@
1414

1515
<ItemGroup>
1616
<PackageReference Include="Accord.MachineLearning" Version="3.8.0" />
17-
<PackageReference Include="Autofac" Version="4.8.1" />
17+
<PackageReference Include="Autofac" Version="4.9.0" />
1818
<PackageReference Include="morelinq" Version="3.1.0" />
1919
<PackageReference Include="MathNet.Numerics" Version="4.7.0" />
2020
<PackageReference Include="Newtonsoft.Json" Version="12.0.1" />
21-
<PackageReference Include="Polly" Version="6.1.2" />
21+
<PackageReference Include="Polly" Version="7.0.1" />
2222
<PackageReference Include="protobuf-net" Version="2.4.0" />
2323
<PackageReference Include="SharpZipLib" Version="1.1.0" />
2424
<PackageReference Include="StackExchange.Redis" Version="2.0.519" />

src/Utilities/Wikiled.Sentiment.ConsoleApp/Analysis/BaseRawCommand.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ private async Task<IParsedDocumentHolder> SynchronizedReviews(IParsedDocumentHol
6262

6363
private IObservable<IParsedDocumentHolder> GetAllDocuments()
6464
{
65-
Logger.LogInformation("Loading {0}", Config);
6665
var dataSource = loader.Load(Config);
6766
return container.GetTextSplitter().GetParsedReviewHolders(dataSource);
6867
}

src/Utilities/Wikiled.Sentiment.ConsoleApp/Analysis/BoostrapCommand.cs

Lines changed: 4 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
using System.Linq;
44
using System.Reactive.Concurrency;
55
using System.Reactive.Linq;
6+
using System.Threading;
67
using System.Threading.Tasks;
78
using Microsoft.Extensions.Logging;
89
using Wikiled.Arff.Logic;
@@ -23,24 +24,10 @@ public class BoostrapCommand : BaseRawCommand<BootsrapConfig>
2324

2425
private string positiveResult;
2526

26-
public BoostrapCommand(ILogger log, BootsrapConfig config, IDataLoader loader, ISessionContainer container)
27+
public BoostrapCommand(ILogger<BoostrapCommand> log, BootsrapConfig config, IDataLoader loader, ISessionContainer container)
2728
: base(log, config, loader, container)
2829
{
29-
}
30-
31-
private bool ProcessReview(ProcessingContext context)
32-
{
33-
var bootSentimentValue = context.Review.CalculateRawRating();
34-
var bootAllSentiments = context.Review.GetAllSentiments()
35-
.Where(item => !item.Owner.IsInvertor && item.Owner.IsSentiment)
36-
.ToArray();
37-
38-
if (bootSentimentValue.StarsRating.HasValue)
39-
{
40-
return true;
41-
}
42-
43-
return false;
30+
Semaphore = new SemaphoreSlim(3000);
4431
}
4532

4633
protected override async Task Process(IObservable<IParsedDocumentHolder> reviews,
@@ -65,8 +52,6 @@ protected override async Task Process(IObservable<IParsedDocumentHolder> reviews
6552
.ToArray();
6653
SaveDocuments(result);
6754
}
68-
69-
Logger.LogInformation($"Testing performance {client.GetPerformanceDescription()}");
7055
}
7156

7257
private EvalData Resolve(ProcessingContext context)
@@ -88,6 +73,7 @@ private EvalData Resolve(ProcessingContext context)
8873

8974
private void SaveDocuments(EvalData[] context)
9075
{
76+
Logger.LogInformation("SaveDocuments");
9177
InitOutput();
9278
var types = context.Where(item => item.Stars.HasValue && item.TotalSentiments >= Config.Minimum).ToArray();
9379
var positive = types.Count(item => item.CalculatedPositivity == PositivityType.Positive);

0 commit comments

Comments
 (0)