Skip to content

Commit

Permalink
Fix ParallelSentimentAnalyzer constructor argument (#182)
Browse files Browse the repository at this point in the history
* Possible typo in readme of Lab 09

* Fix ParallelSentimentAnalyzer constructor argument

`testAnalyzeWithMultipleInputs()` is calling `ParallelSentimentAnalyzer` constructor with `Map<String, SentimentScore>`, but the constructor takes `Map<String, Integer>`.
I'm assuming the test is correctly passing a `Map<String, SentimentScore>` and the constructor is incorrect.
  • Loading branch information
IliyanGavrilov authored Dec 13, 2024
1 parent 474ceab commit 3b427af
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion 09-threads/lab/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ public enum SentimentScore {
* @param stopWords set containing stop words
* @param sentimentLexicon map containing the sentiment lexicon, where the key is the word and the value is the sentiment score
*/
public ParallelSentimentAnalyzer(int workersCount, Set<String> stopWords, Map<String, Integer> sentimentLexicon) { }
public ParallelSentimentAnalyzer(int workersCount, Set<String> stopWords, Map<String, SentimentScore> sentimentLexicon) { }
```

⭐ Бележки:
Expand Down

0 comments on commit 3b427af

Please sign in to comment.