Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-60936

Convert enabled time-splitting methods setting to evaluation default time-splitting methods

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 3.7
    • 3.4, 3.7
    • Analytics
    • MOODLE_34_STABLE, MOODLE_37_STABLE
    • MOODLE_37_STABLE
    • MDL-60936_master
    • Hide

      Prerequisites

      1. Install https://github.com/dmonllao/moodle-local_testanalytics

      from the command line:

      change directory to 'local' and type in the following:

      git clone https://github.com/dmonllao/moodle-local_testanalytics.git testanalytics
      This will add the code into the testanalytics folder under local. Next login to your moodle site and follow the installation instructions.

      Test 1

      1. Log in as admin
      2. Go to "site admin > analytics > analytics settings"
      3. Check that there is a 'defaulttimesplittingsevaluation' setting
      4. Remove "Tenths accumulative" from the list of selected time-splitting methods if it is selected
      5. Go to "site admin > analytics > analytics models"
      6. Select "Actions > Edit" for "Students at risk of dropping out" model
      7. You should see "Tenths accumulative" listed as one of the "Time-splitting method" options you can select
      8. Select that option and save changes
      9. Confirm that the option was saved by clicking again on "Actions > Edit" for that same model

      Test 2

      1. Log in as admin
      2. Go to "site admin > analytics > analytics settings"
      3. Disable 'onlycli' option
      4. Set 'defaulttimesplittingsevaluation' setting to "Tenths" and "Tenths accumulative"
      5. Save changes
      6. Go to "site admin > analytics > analytics models" and click 'Evaluate' for "Discrete" model
      7. You should see two headings starting with "Results using": Results using Tenths time-splitting method and Results using Tenths accumulative time-splitting method
      Show
      Prerequisites Install https://github.com/dmonllao/moodle-local_testanalytics from the command line: change directory to 'local' and type in the following: git clone https://github.com/dmonllao/moodle-local_testanalytics.git testanalytics This will add the code into the testanalytics folder under local. Next login to your moodle site and follow the installation instructions. Test 1 Log in as admin Go to "site admin > analytics > analytics settings" Check that there is a 'defaulttimesplittingsevaluation' setting Remove "Tenths accumulative" from the list of selected time-splitting methods if it is selected Go to "site admin > analytics > analytics models" Select "Actions > Edit" for "Students at risk of dropping out" model You should see "Tenths accumulative" listed as one of the "Time-splitting method" options you can select Select that option and save changes Confirm that the option was saved by clicking again on "Actions > Edit" for that same model Test 2 Log in as admin Go to "site admin > analytics > analytics settings" Disable 'onlycli' option Set 'defaulttimesplittingsevaluation' setting to "Tenths" and "Tenths accumulative" Save changes Go to "site admin > analytics > analytics models" and click 'Evaluate' for "Discrete" model You should see two headings starting with "Results using": Results using Tenths time-splitting method and Results using Tenths accumulative time-splitting method

      There is a problem with 'Enabled time-splitting methods' setting.

      Any 3rd party can add all sort of new analytics components: analysers, time splitting methods... people can potentially add a time splitting method and create a model using it without the time spolitting method being part of the enabled time-splitting methods in the site. They need to enable it separately, this is far from nice.

      The purpose of that setting was to limit the amount of time splitting methods a model is evaluated against. Instead of enforcing time-splitting methods to be enabled (if it is not already enforced everywhere) I would vote to update the time-splitting method setting to something like "List of time-splitting methods that will be use to evaluate models".

        1. image-2019-03-20-15-28-56-726.png
          image-2019-03-20-15-28-56-726.png
          198 kB

            dmonllao David Monllaó
            dmonllao David Monllaó
            Carlos Escobedo Carlos Escobedo
            Adrian Greeve Adrian Greeve
            Janelle Barcega Janelle Barcega
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 30 minutes
                1h 30m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.