fix the JS issue where both ooba and aphro were using the same container ID

add108b8216820a38a49e89a04c055c17d77d088

AlpinDale <alpindale@gmail.com>

public/index.html+1 -1 Viewed

                                         <div class="toggle-description widthUnset" data-i18n="Aphrodite only. Determines the order of samplers. Skew is always applied post-softmax, so it's not included here.">
                                             Aphrodite only. Determines the order of samplers.
                                         </div>
-                                        <div id="sampler_priority_container" class="prompt_order">
+                                        <div id="sampler_priority_container_aphrodite" class="prompt_order">
                                             <div data-name="dry" draggable="true"><span>DRY</span><small></small></div>
                                             <div data-name="penalties" draggable="true"><span>Penalties</span><small></small></div>
                                             <div data-name="no_repeat_ngram" draggable="true"><span>No Repeat Ngram</span><small></small></div>

public/scripts/textgen-settings.js+28 -6 Viewed

     });
+}
+function sortAphroditeItemsByOrder(orderArray) {
+    const $container = $('#sampler_priority_container_aphrodite');
+    orderArray.forEach((name) => {
+        const $item = $container.find(`[data-name="${name}"]`).detach();
+        $container.append($item);
+    });
+}
 jQuery(function () {
     $('#koboldcpp_order').sortable({
         delay: getSortableDelay(),

         },
     });
+    $('#sampler_priority_container_aphrodite').sortable({
+        delay: getSortableDelay(),
+        stop: function () {
+            const order = [];
+            $('#sampler_priority_container_aphrodite').children().each(function () {
+                order.push($(this).data('name'));
+            });
+            settings.samplers_priorities = order;
+            console.log('Samplers reordered:', settings.samplers_priorities);
+            saveSettingsDebounced();
+        },
+    });
     $('#tabby_json_schema').on('input', function () {
         const json_schema_string = String($(this).val());

     });
     $('#aphrodite_default_order').on('click', function () {
-        sortOobaItemsByOrder(APHRODITE_DEFAULT_ORDER);
+        sortAphroditeItemsByOrder(APHRODITE_DEFAULT_ORDER);
-        settings.samplers_priorties = APHRODITE_DEFAULT_ORDER;
+        settings.samplers_priorities = APHRODITE_DEFAULT_ORDER;
-        console.log('Default samplers order loaded:', settings.samplers_priorties);
+        console.log('Default samplers order loaded:', settings.samplers_priorities);
         saveSettingsDebounced();
     });

     if ('samplers_priority' === setting) {
         value = Array.isArray(value) ? value : APHRODITE_DEFAULT_ORDER;
         insertMissingArrayItems(APHRODITE_DEFAULT_ORDER, value);
-        sortOobaItemsByOrder(value);
+        sortAphroditeItemsByOrder(value);
-        settings.samplers_priorties = value;
+        settings.samplers_priorities = value;
         return;
+    }

         'nsigma': settings.nsigma,
         'custom_token_bans': toIntArray(banned_tokens),
         'no_repeat_ngram_size': settings.no_repeat_ngram_size,
-        'sampler_priority': settings.samplers_priorties,
+        'sampler_priority': settings.type === APHRODITE ? settings.samplers_priorities : undefined,
     };
     if (settings.type === OPENROUTER) {