From 6d9aeb26f4b923a4d35918a465789e7cb6fe314f Mon Sep 17 00:00:00 2001
From: Hernan Rojek Moriceau <hernan.rojek@gmail.com>
Date: Mon, 28 Nov 2022 23:57:21 +0000
Subject: [PATCH 1/3] feat (retrievers): Implements #9 Render String options in
 retriever parameters as dropdown combo

---
 src/components/retrievers/index.js | 36 ++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/src/components/retrievers/index.js b/src/components/retrievers/index.js
index 2497c57..43a277d 100644
--- a/src/components/retrievers/index.js
+++ b/src/components/retrievers/index.js
@@ -308,6 +308,42 @@ function Retrievers({ disableParent }) {
                         );
                       })}
                     </Select>
+                  ) : !_.isNil(parameter.options) &&
+                    !_.isEmpty(parameter.options) ? (
+                    <Select
+                      id={"parameter-" + parameter.parameter_id}
+                      labelText={parameter.name}
+                      value={parameter.value || "placeholder-item"}
+                      onChange={(event) => {
+                        dispatch(
+                          updateParameterValue({
+                            parameter_id: parameter.parameter_id,
+                            value: event.target.value,
+                          })
+                        );
+                      }}
+                    >
+                      <SelectItem
+                        disabled
+                        hidden
+                        value="placeholder-item"
+                        text="Choose an option"
+                      />
+                      {parameter.options.map((option, index) => {
+                        return (
+                          <SelectItem
+                            key={
+                              "parameter-" +
+                              parameter.parameter_id +
+                              "__option-" +
+                              index
+                            }
+                            value={option}
+                            text={option}
+                          />
+                        );
+                      })}
+                    </Select>
                   ) : (
                     <TextInput
                       id={"parameter-" + parameter.parameter_id}

From 2c8ba8ce627b7b12bdc9fefb171ca2c49f6b091f Mon Sep 17 00:00:00 2001
From: Hernan Rojek Moriceau <hernan.rojek@gmail.com>
Date: Fri, 2 Dec 2022 22:54:38 +0000
Subject: [PATCH 2/3] feat (retrievers): Implements #12 Pre-select checkpoint
 in retriever's parameters

---
 src/applications/qa/index.js       | 5 ++++-
 src/components/retrievers/index.js | 5 +++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/applications/qa/index.js b/src/applications/qa/index.js
index 797e07e..0ab9793 100644
--- a/src/applications/qa/index.js
+++ b/src/applications/qa/index.js
@@ -274,7 +274,10 @@ function QuestionAnswering({ application, showSettings }) {
 
       {showSettings ? (
         <div className="application__settings">
-          <Retrievers disableParent={setDisabled}></Retrievers>
+          <Retrievers
+            disableParent={setDisabled}
+            defaultCheckpoint={selectedCollection?.checkpoint}
+          ></Retrievers>
           <Collections
             retriever={retrievers.selectedRetriever}
             selectedCollection={selectedCollection}
diff --git a/src/components/retrievers/index.js b/src/components/retrievers/index.js
index 43a277d..0654575 100644
--- a/src/components/retrievers/index.js
+++ b/src/components/retrievers/index.js
@@ -41,7 +41,7 @@ import { addNotification } from "../notifications/notificationsSlice";
 
 import "./styles.scss";
 
-function Retrievers({ disableParent }) {
+function Retrievers({ disableParent, defaultCheckpoint }) {
   // Redux connectivity
   const retrievers = useSelector((state) => state.retrievers);
   const dispatch = useDispatch();
@@ -313,7 +313,7 @@ function Retrievers({ disableParent }) {
                     <Select
                       id={"parameter-" + parameter.parameter_id}
                       labelText={parameter.name}
-                      value={parameter.value || "placeholder-item"}
+                      value={parameter.value || defaultCheckpoint || "placeholder-item"}
                       onChange={(event) => {
                         dispatch(
                           updateParameterValue({
@@ -374,6 +374,7 @@ function Retrievers({ disableParent }) {
 
 Retrievers.propTypes = {
   disableParent: PropTypes.func,
+  defaultCheckpoint: PropTypes.string,
 };
 
 export default Retrievers;

From f5b0d9825ef07d3edb0f85685f0229073a87a26a Mon Sep 17 00:00:00 2001
From: Hernan Rojek Moriceau <hernan.rojek@gmail.com>
Date: Tue, 6 Dec 2022 17:25:39 +0000
Subject: [PATCH 3/3] feat (retrievers): Implements #12 Pre-select checkpoint
 in retriever's parameters, with redux dispatch

---
 src/components/retrievers/index.js | 47 +++++++++++++++++++++++++++++-
 1 file changed, 46 insertions(+), 1 deletion(-)

diff --git a/src/components/retrievers/index.js b/src/components/retrievers/index.js
index 0654575..46e3410 100644
--- a/src/components/retrievers/index.js
+++ b/src/components/retrievers/index.js
@@ -131,6 +131,51 @@ function Retrievers({ disableParent, defaultCheckpoint }) {
     }
   }, [retrievers.retrievers, dispatch]);
 
+  // Determines if defaultCheckpoint is present as an option in the checkpoint parameter in the selected retriever, if any
+  const isDefaultCheckpointOptionPresent = () => {
+    let ret = !!defaultCheckpoint;
+    ret &&= Array.isArray(retrievers?.selectedRetriever?.parameters);
+    if (ret) {
+      let parameter = retrievers.selectedRetriever.parameters.find(parameter => parameter.parameter_id === 'checkpoint')
+      ret = !!parameter;
+      ret &&= Array.isArray(parameter.options);
+      ret &&= parameter.options.includes(defaultCheckpoint);
+    }
+    return ret;
+  };
+
+  const isCheckpointParamSet = () => {
+    let ret = Array.isArray(retrievers?.selectedRetriever?.parameters);
+    if (ret) {
+      let parameter = retrievers.selectedRetriever.parameters.find(parameter => parameter.parameter_id === 'checkpoint');
+      ret = !!parameter;
+      ret &&= parameter.value !== null;
+    }
+    return ret;
+  };
+
+  // Set defaultCheckpoint as default selected checkpoint, if available
+  useEffect(() => {
+    // Dispatch updateParameterValue event
+    if (isDefaultCheckpointOptionPresent() && !isCheckpointParamSet()) {
+      dispatch(
+        updateParameterValue({
+          parameter_id: 'checkpoint',
+          value: defaultCheckpoint,
+        })
+      );
+    }
+  }, [retrievers.retrievers, dispatch, defaultCheckpoint]);
+
+  const getParameterDefaultOption = (parameter, placeholderId) => {
+    let option = parameter.value || placeholderId;
+    if (parameter.parameter_id === 'checkpoint' && isDefaultCheckpointOptionPresent()) {
+      // Custom behavior on default checkpoint
+      option = defaultCheckpoint;
+    }
+    return option;
+  }
+
   return (
     <React.Fragment>
       <div className="settings__item">
@@ -313,7 +358,7 @@ function Retrievers({ disableParent, defaultCheckpoint }) {
                     <Select
                       id={"parameter-" + parameter.parameter_id}
                       labelText={parameter.name}
-                      value={parameter.value || defaultCheckpoint || "placeholder-item"}
+                      value={getParameterDefaultOption(parameter, "placeholder-item")}
                       onChange={(event) => {
                         dispatch(
                           updateParameterValue({