From 6c49ee5c646ba3620fef484a8e0fd4b38d339aee Mon Sep 17 00:00:00 2001
From: Bruno Carlos <6951456+brnovasco@users.noreply.github.com>
Date: Wed, 15 May 2024 14:36:50 -0300
Subject: [PATCH] refactor: removing drop classifier from network form
 defaulting value to false on the cli call.

---
 .../node-component-forms/network-form.tsx      | 18 ------------------
 .../src/server/api/routers/deepsirius-job.ts   |  4 +++-
 2 files changed, 3 insertions(+), 19 deletions(-)

diff --git a/apps/deepsirius-ui/src/components/workboard/node-component-forms/network-form.tsx b/apps/deepsirius-ui/src/components/workboard/node-component-forms/network-form.tsx
index da1c3da..7d546be 100644
--- a/apps/deepsirius-ui/src/components/workboard/node-component-forms/network-form.tsx
+++ b/apps/deepsirius-ui/src/components/workboard/node-component-forms/network-form.tsx
@@ -22,7 +22,6 @@ import {
   SelectTrigger,
   SelectValue,
 } from '~/components/ui/select';
-import { Switch } from '~/components/ui/switch';
 import { slurmGPUOptions, slurmPartitionOptions } from '~/lib/constants';
 
 const slurmOptions = z.object({
@@ -43,7 +42,6 @@ export const networkSchema = z.object({
       message:
         'Network label name must contain only letters, numbers underscores and no spaces.',
     }),
-  dropClassifier: z.boolean(),
   jobGPUs: z.enum(['1', '2', '4']),
   iterations: z.coerce.number().gte(1, { message: 'Must be >= 1' }),
   learningRate: z.coerce.number().gt(0, { message: 'Must be greater than 0' }),
@@ -76,7 +74,6 @@ export function useNetworkForm({
     defaultValues: {
       networkUserLabel: networkUserLabel,
       networkTypeName: networkTypeName,
-      dropClassifier: false,
       jobGPUs: '1',
       iterations: 1,
       learningRate: 0.00001,
@@ -182,21 +179,6 @@ export function NetworkForm({
             </FormItem>
           )}
         />
-        <FormField
-          name="dropClassifier"
-          control={form.control}
-          render={({ field }) => (
-            <FormItem className="flex flex-row items-center justify-between">
-              <FormLabel>Drop Classifier</FormLabel>
-              <FormControl>
-                <Switch
-                  checked={field.value}
-                  onCheckedChange={field.onChange}
-                />
-              </FormControl>
-            </FormItem>
-          )}
-        />
         <div className="flex flex-row items-center gap-1">
           <FormField
             name="iterations"
diff --git a/apps/deepsirius-ui/src/server/api/routers/deepsirius-job.ts b/apps/deepsirius-ui/src/server/api/routers/deepsirius-job.ts
index 54963c8..5d0d631 100644
--- a/apps/deepsirius-ui/src/server/api/routers/deepsirius-job.ts
+++ b/apps/deepsirius-ui/src/server/api/routers/deepsirius-job.ts
@@ -322,7 +322,9 @@ export const deepsiriusJobRouter = createTRPCRouter({
         'batch-size': input.formData.batchSize,
         'learning-rate': input.formData.learningRate,
         optimiser: input.formData.optimizer,
-        'drop-classifier': input.formData.dropClassifier,
+        // in the ui, theres a separation of concerns between the network form and the finetune component, but the cli uses the same function for both
+        // so we need to pass this value as a boolean to the cli.
+        'drop-classifier': false, 
         // this is a silly way to get around the fact that the cli expects 3 values for patch size
         'net-patch-size': (input.formData.patchSize + ' ').repeat(3),
       };