5
\$\begingroup\$

I have a list of utterances(strings)/corpus as the follows,

List<string> allUtterances = new List<string>
{
    "c2's are above the hierarchy than c1's",
    "c2's are better than c1's",
    "get me a group of 10 c1's",
    "he is a c2",
    "he was a c two",
    "hey i am a c1",
    "jdsaxkjhasx",
    "khndsmcsdfcs",
    "my competency is c2",
    "none intent",
    "she is still a c 1",
    "this is a none intent, please ignore",
    "we are hiring fresh c1's"
};

This is the class schema:

public class ListEntity
{
        public string name { get; set; }
        public List<Sublist> subLists { get; set; }
}

public class Sublist
{
    public string canonicalForm { get; set; }
    public List<string> list { get; set; }
}

And this is a sample POCO:

    List<ListEntity> listEntities = new List<ListEntity>
    {
        new ListEntity
        {
            name = "Competency",
            subLists = new List<Sublist>
            {
                new Sublist
                {
                    canonicalForm = "C1",
                    list = new List<string>
                    {
                        "c1",
                        "c one",
                        "c 1",
                        "C 1",
                        "C1",
                        "C one",
                        "C ONE"
                    }
                },
                new Sublist
                {
                    canonicalForm = "C2",
                    list = new List<string>
                    {
                        "c2",
                        "c two",
                        "c 2",
                        "C 2",
                        "C2",
                        "C two",
                        "C TWO"
                    }
                }
            }
        }
    };

    var canonicalForms = listEntities.Select(x => x.subLists.Select(y => y.list).ToList()).ToList();

Suppose if I have an utterance of the following from the above mentioned list allUtterances:

 "query": "C2's are better than C1's"

I want to get the following output for the above utterance:

{
      "entity": "c2",
      "type": "Competency",
      "startIndex": 0,
      "endIndex": 1,
      "resolution": {
        "values": [
          "C2"
        ]
      }
},
{
      "entity": "c1",
      "type": "Competency",
      "startIndex": 21,
      "endIndex": 22,
      "resolution": {
        "values": [
          "C1"
        ]
      }
}

The rules with which I want to match is as follows:

For every utterance in the allUtterances list, If the utterance text contains a value from the property list of the class sublist I want to extract the start and end positions and mark them with the appropriate key which is the canonicalForm in this case update the type key in my JSON payload with the name property in the ListEntityClass.


I have tried the following approach:

using System;
using System.Linq;
using System.IO;
using Newtonsoft.Json;
using Newtonsoft.Json.Linq;
using System.Collections.Generic;

namespace ListEntityProblem
{
    class Program
    {
        static void Main(string[] args)
        {
            List<string> allUtterances = new List<string>
            {
                "c2's are above the hierarchy than c1's",
                "c2's are better than c1's",
                "get me a group of 10 c1's",
                "he is a c2",
                "he was a c two",
                "hey i am a c1",
                "jdsaxkjhasx",
                "khndsmcsdfcs",
                "my competency is c2",
                "none intent",
                "she is still a c 1",
                "this is a none intent, please ignore",
                "we are hiring fresh c1's"
            };

            List<ListEntity> listEntities = new List<ListEntity>
            {
                new ListEntity
                {
                    name = "Competency",
                    subLists = new List<Sublist>
                    {
                        new Sublist
                        {
                            canonicalForm = "C1",
                            list = new List<string>
                            {
                                "c1",
                                "c one",
                                "c 1",
                                "C 1",
                                "C1",
                                "C one",
                                "C ONE"
                            }
                        },
                        new Sublist
                        {
                            canonicalForm = "C2",
                            list = new List<string>
                            {
                                "c2",
                                "c two",
                                "c 2",
                                "C 2",
                                "C2",
                                "C two",
                                "C TWO"
                            }
                        }
                    }
                }
            };


            List<Tuple<string, string, List<string>>> ListEntityLookup = new List<Tuple<string, string, List<string>>>();

            //n^2, construct lookup for list entities
            foreach (var item in listEntities)
            {
                string listEntityName = item.name;
                foreach (var innerItem in item.subLists)
                {
                    string normalizedValue = innerItem.canonicalForm;
                    List<string> synonymValues = innerItem.list;

                    ListEntityLookup.Add(Tuple.Create<string, string, List<string>>(listEntityName, normalizedValue, synonymValues));
                }
            }

            List<JObject> parsedEntities = new List<JObject>();

            //n^3, populate the parsed payload with start and end indices
            foreach (var item in allUtterances)
            {
                foreach (var ll in ListEntityLookup)
                {
                    foreach (var cf in ll.Item3)
                    {
                        int start = 0, end = 0;
                        if (item.Contains(cf))
                        {
                            start = item.IndexOf(cf);
                            end = start + cf.Length;




                            parsedEntities.Add(new JObject
                            {
                                new JProperty("Start", start),
                                new JProperty("End", end),
                                new JProperty("Query", item),
                                new JProperty("CanonicalForm", ll.Item2),
                                new JProperty("ListEntity", ll.Item1)
                            });
                        }
                    }
                }
            }

            //Group by query
            var groupedParsedEntities = parsedEntities.GroupBy(x => x["Query"]).ToList();



        }
    }
}

Edit:

I have tried to re-write the for-each loop, but this resulted, in more nesting.

            foreach (var item in allUtterances)
            {
                foreach (var listEntity in listEntities)
                {
                    foreach (var canonicalForm in listEntity.subLists)
                    {
                        foreach(var synonym in canonicalForm.list)
                        {
                            int start = item.IndexOf(synonym);
                            if(start != -1)
                            {
                                parsedEntities.Add(new JObject
                                {
                                    new JProperty("Start", start),
                                    new JProperty("End", start + synonym.Length),
                                    new JProperty("Query", item),
                                    new JProperty("CanonicalForm", canonicalForm.canonicalForm),
                                    new JProperty("ListEntity", listEntity.name)
                                });
                            }
                        }
                    }
                }
            }

But this approach seems to slow for large number of utterances and does not scale very well. As the main loop runs n^3 times. Our server will have to do too many computations per second.

I am stuck thinking whether or not, should I use Regex if it would offer me some performance benefits or not.

Please help me with optimizing this algorithm.

Any help will be very much appreciated.

\$\endgroup\$
1

1 Answer 1

3
\$\begingroup\$

Why do you need to create ListEntityLookup? Just process listEntities.

This is not needed

int start = 0, end = 0;

This is wasteful.

if (item.Contains(cf))
{
    start = item.IndexOf(cf);

just do start

int start = item.IndexOf(cf);
if (start >= 0)
{
\$\endgroup\$
3
  • \$\begingroup\$ I modified the foreach loops, please see the edited question. Please give your suggestions. \$\endgroup\$ Commented Jun 20, 2018 at 14:46
  • 1
    \$\begingroup\$ You are not supposed to modify after and answer. \$\endgroup\$
    – paparazzo
    Commented Jun 20, 2018 at 14:52
  • \$\begingroup\$ sorry my bad. I am just trying to improve the perf of these loops. \$\endgroup\$ Commented Jun 20, 2018 at 14:53

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.