Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1931

Add multinomial product purchasing model to BPS Data Generator

    Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: blueprints
    • Labels:
      None

      Description

      My recent analysis have found problems with our Markov models. I proposed a multinomial model and demonstrated that it could produce data amenable to segmentation. I'd like to add this multinomial model to the BPS data generator, make it the new default model, and evaluate it for a while.

        Activity

        Hide
        rnowling RJ Nowling added a comment -

        Patch adding the multinomial model, making it the default, adding outputs for various data models, and making the random seed a required parameter.

        Show
        rnowling RJ Nowling added a comment - Patch adding the multinomial model, making it the default, adding outputs for various data models, and making the random seed a required parameter.
        Hide
        rnowling RJ Nowling added a comment -

        jay vyas Review please

        Show
        rnowling RJ Nowling added a comment - jay vyas Review please
        Hide
        jayunit100 jay vyas added a comment -

        Should the type in PurchasingModel<T> be constrained ? Otherwise lgtm , so feel free to commit. Ill test it out this wk

        Show
        jayunit100 jay vyas added a comment - Should the type in PurchasingModel<T> be constrained ? Otherwise lgtm , so feel free to commit. Ill test it out this wk
        Hide
        rnowling RJ Nowling added a comment -

        PurchasingModel<T> can either return a MarkovModel<Product> or DiscretePDF<Product>. These don't conform to a common base class or interface at the moment, and I only added the type because I need to be able to call toString() on the resulting model. I could change the interface to just return an Object, which would remove the warnings. But I'm going to leave it unconstrained for now and commit. I'll try to come up with a better approach in a future patch.

        Show
        rnowling RJ Nowling added a comment - PurchasingModel<T> can either return a MarkovModel<Product> or DiscretePDF<Product>. These don't conform to a common base class or interface at the moment, and I only added the type because I need to be able to call toString() on the resulting model. I could change the interface to just return an Object, which would remove the warnings. But I'm going to leave it unconstrained for now and commit. I'll try to come up with a better approach in a future patch.
        Hide
        rnowling RJ Nowling added a comment -

        Committed.

        Show
        rnowling RJ Nowling added a comment - Committed.

          People

          • Assignee:
            rnowling RJ Nowling
            Reporter:
            rnowling RJ Nowling
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development