Generate Session Coach feedback operations

src/index.tsx

@@ -2579,6 +2579,7 @@ export type Mutation = {
   startChallenge: ChallengeEntry;
   submitCancellationFeedback: Scalars["Boolean"]["output"];
   submitChallengeEntry: ChallengeEntry;
+  submitSessionCoachFeedback: SessionCoachFeedbackGql;
   syncAppleSubscription: SyncAppleSubscriptionResultGql;
   undismissChallenge: Scalars["Boolean"]["output"];
   unfollowUser: UserGql;
@@ -2840,6 +2841,12 @@ export type MutationSubmitChallengeEntryArgs = {
   videoId: Scalars["ID"]["input"];
 };
 
+export type MutationSubmitSessionCoachFeedbackArgs = {
+  generationId: Scalars["ID"]["input"];
+  response: SessionCoachFeedbackResponseEnum;
+  videoId: Scalars["Int"]["input"];
+};
+
 export type MutationSyncAppleSubscriptionArgs = {
   input: SyncAppleSubscriptionInputGql;
 };
@@ -3685,6 +3692,19 @@ export type SessionCoachEvidenceGql = {
   score: Scalars["Float"]["output"];
 };
 
+export type SessionCoachFeedbackGql = {
+  __typename?: "SessionCoachFeedbackGQL";
+  response: SessionCoachFeedbackResponseEnum;
+  updatedAt: Scalars["DateTime"]["output"];
+};
+
+export enum SessionCoachFeedbackResponseEnum {
+  AlreadyWorkingOnThis = "ALREADY_WORKING_ON_THIS",
+  Helpful = "HELPFUL",
+  NotEnoughEvidence = "NOT_ENOUGH_EVIDENCE",
+  WrongFocus = "WRONG_FOCUS",
+}
+
 export type SessionCoachGql = {
   __typename?: "SessionCoachGQL";
   agentGeneration?: Maybe<SessionCoachGenerationGql>;
@@ -3737,6 +3757,7 @@ export type SessionCoachGenerationGql = {
   createdAt: Scalars["DateTime"]["output"];
   decision?: Maybe<SessionCoachDecisionGql>;
   diagnostics?: Maybe<SessionCoachGenerationDiagnosticsGql>;
+  feedback?: Maybe<SessionCoachFeedbackGql>;
   id: Scalars["ID"]["output"];
   state: SessionCoachGenerationStateEnum;
 };
@@ -4078,7 +4099,6 @@ export type SyncAppleSubscriptionResultGql = {
 export type TableStateGql = {
   __typename?: "TableStateGQL";
   homography?: Maybe<HomographyInfoGql>;
-  identifierToBox: Array<BoundingBoxGql>;
   identifierToColor: Array<Maybe<RgbColorGql>>;
   identifierToPosition: Array<Array<Scalars["Float"]["output"]>>;
 };
@@ -7205,6 +7225,11 @@ export type GetSessionCoachQuery = {
         abstentionReason?: string | null;
         synthesisRationale?: string | null;
       } | null;
+      feedback?: {
+        __typename?: "SessionCoachFeedbackGQL";
+        response: SessionCoachFeedbackResponseEnum;
+        updatedAt: any;
+      } | null;
     } | null;
   };
 };
@@ -7262,7 +7287,27 @@ export type RequestSessionCoachGenerationMutation = {
         abstentionReason?: string | null;
         synthesisRationale?: string | null;
       } | null;
+      feedback?: {
+        __typename?: "SessionCoachFeedbackGQL";
+        response: SessionCoachFeedbackResponseEnum;
+        updatedAt: any;
       } | null;
+    } | null;
+  };
+};
+
+export type SubmitSessionCoachFeedbackMutationVariables = Exact<{
+  videoId: Scalars["Int"]["input"];
+  generationId: Scalars["ID"]["input"];
+  response: SessionCoachFeedbackResponseEnum;
+}>;
+
+export type SubmitSessionCoachFeedbackMutation = {
+  __typename?: "Mutation";
+  submitSessionCoachFeedback: {
+    __typename?: "SessionCoachFeedbackGQL";
+    response: SessionCoachFeedbackResponseEnum;
+    updatedAt: any;
   };
 };
 
@@ -7343,6 +7388,11 @@ export type SessionCoachGenerationFieldsFragment = {
     abstentionReason?: string | null;
     synthesisRationale?: string | null;
   } | null;
+  feedback?: {
+    __typename?: "SessionCoachFeedbackGQL";
+    response: SessionCoachFeedbackResponseEnum;
+    updatedAt: any;
+  } | null;
 };
 
 export type CreateShotShareLinkMutationVariables = Exact<{
@@ -7481,76 +7531,6 @@ export type FinalizePlayerAssignmentsMutation = {
   }>;
 };
 
-export type TableStateLayoutFieldsFragment = {
-  __typename?: "TableStateGQL";
-  identifierToPosition: Array<Array<number>>;
-  identifierToColor: Array<{ __typename?: "RGBColorGQL"; hex: string } | null>;
-};
-
-export type DetectionBoxFieldsFragment = {
-  __typename?: "BoundingBoxGQL";
-  left: number;
-  top: number;
-  width: number;
-  height: number;
-};
-
-export type DetectionPocketPointFieldsFragment = {
-  __typename?: "PocketPointsGQL";
-  topLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-  topSide: { __typename?: "IntPoint2D"; x: number; y: number };
-  topRight: { __typename?: "IntPoint2D"; x: number; y: number };
-  bottomLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-  bottomSide: { __typename?: "IntPoint2D"; x: number; y: number };
-  bottomRight: { __typename?: "IntPoint2D"; x: number; y: number };
-};
-
-export type TableStateDetectionFieldsFragment = {
-  __typename?: "TableStateGQL";
-  identifierToBox: Array<{
-    __typename?: "BoundingBoxGQL";
-    left: number;
-    top: number;
-    width: number;
-    height: number;
-  }>;
-  homography?: {
-    __typename?: "HomographyInfoGQL";
-    crop: {
-      __typename?: "BoundingBoxGQL";
-      left: number;
-      top: number;
-      width: number;
-      height: number;
-    };
-    pockets: Array<{
-      __typename?: "BoundingBoxGQL";
-      left: number;
-      top: number;
-      width: number;
-      height: number;
-    }>;
-    sourcePoints: {
-      __typename?: "PocketPointsGQL";
-      topLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-      topSide: { __typename?: "IntPoint2D"; x: number; y: number };
-      topRight: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomSide: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomRight: { __typename?: "IntPoint2D"; x: number; y: number };
-    };
-    destPoints: {
-      __typename?: "PocketPointsGQL";
-      topLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-      topSide: { __typename?: "IntPoint2D"; x: number; y: number };
-      topRight: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomSide: { __typename?: "IntPoint2D"; x: number; y: number };
-      bottomRight: { __typename?: "IntPoint2D"; x: number; y: number };
-    };
-  } | null;
-};
-
 export type GetTableStateQueryVariables = Exact<{
   b64Image: Scalars["String"]["input"];
   tableSize?: InputMaybe<Scalars["Float"]["input"]>;
@@ -7566,48 +7546,6 @@ export type GetTableStateQuery = {
       __typename?: "RGBColorGQL";
       hex: string;
     } | null>;
-    identifierToBox: Array<{
-      __typename?: "BoundingBoxGQL";
-      left: number;
-      top: number;
-      width: number;
-      height: number;
-    }>;
-    homography?: {
-      __typename?: "HomographyInfoGQL";
-      crop: {
-        __typename?: "BoundingBoxGQL";
-        left: number;
-        top: number;
-        width: number;
-        height: number;
-      };
-      pockets: Array<{
-        __typename?: "BoundingBoxGQL";
-        left: number;
-        top: number;
-        width: number;
-        height: number;
-      }>;
-      sourcePoints: {
-        __typename?: "PocketPointsGQL";
-        topLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-        topSide: { __typename?: "IntPoint2D"; x: number; y: number };
-        topRight: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomSide: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomRight: { __typename?: "IntPoint2D"; x: number; y: number };
-      };
-      destPoints: {
-        __typename?: "PocketPointsGQL";
-        topLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-        topSide: { __typename?: "IntPoint2D"; x: number; y: number };
-        topRight: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomLeft: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomSide: { __typename?: "IntPoint2D"; x: number; y: number };
-        bottomRight: { __typename?: "IntPoint2D"; x: number; y: number };
-      };
-    } | null;
   };
 };
 
@@ -10062,6 +10000,10 @@ export const SessionCoachGenerationFieldsFragmentDoc = gql`
       abstentionReason
       synthesisRationale
     }
+    feedback {
+      response
+      updatedAt
+    }
   }
 `;
 export const PlayerClusterShotFieldsFragmentDoc = gql`
@@ -10093,73 +10035,6 @@ export const PlayerClusterFieldsFragmentDoc = gql`
   }
   ${PlayerClusterShotFieldsFragmentDoc}
 `;
-export const TableStateLayoutFieldsFragmentDoc = gql`
-  fragment TableStateLayoutFields on TableStateGQL {
-    identifierToPosition
-    identifierToColor {
-      hex
-    }
-  }
-`;
-export const DetectionBoxFieldsFragmentDoc = gql`
-  fragment DetectionBoxFields on BoundingBoxGQL {
-    left
-    top
-    width
-    height
-  }
-`;
-export const DetectionPocketPointFieldsFragmentDoc = gql`
-  fragment DetectionPocketPointFields on PocketPointsGQL {
-    topLeft {
-      x
-      y
-    }
-    topSide {
-      x
-      y
-    }
-    topRight {
-      x
-      y
-    }
-    bottomLeft {
-      x
-      y
-    }
-    bottomSide {
-      x
-      y
-    }
-    bottomRight {
-      x
-      y
-    }
-  }
-`;
-export const TableStateDetectionFieldsFragmentDoc = gql`
-  fragment TableStateDetectionFields on TableStateGQL {
-    identifierToBox {
-      ...DetectionBoxFields
-    }
-    homography {
-      crop {
-        ...DetectionBoxFields
-      }
-      pockets {
-        ...DetectionBoxFields
-      }
-      sourcePoints {
-        ...DetectionPocketPointFields
-      }
-      destPoints {
-        ...DetectionPocketPointFields
-      }
-    }
-  }
-  ${DetectionBoxFieldsFragmentDoc}
-  ${DetectionPocketPointFieldsFragmentDoc}
-`;
 export const ShotClipRangeFragmentDoc = gql`
   fragment ShotClipRange on ShotGQL {
     id
@@ -15917,6 +15792,68 @@ export type RequestSessionCoachGenerationMutationOptions =
     RequestSessionCoachGenerationMutation,
     RequestSessionCoachGenerationMutationVariables
   >;
+export const SubmitSessionCoachFeedbackDocument = gql`
+  mutation SubmitSessionCoachFeedback(
+    $videoId: Int!
+    $generationId: ID!
+    $response: SessionCoachFeedbackResponseEnum!
+  ) {
+    submitSessionCoachFeedback(
+      videoId: $videoId
+      generationId: $generationId
+      response: $response
+    ) {
+      response
+      updatedAt
+    }
+  }
+`;
+export type SubmitSessionCoachFeedbackMutationFn = Apollo.MutationFunction<
+  SubmitSessionCoachFeedbackMutation,
+  SubmitSessionCoachFeedbackMutationVariables
+>;
+
+/**
+ * __useSubmitSessionCoachFeedbackMutation__
+ *
+ * To run a mutation, you first call `useSubmitSessionCoachFeedbackMutation` within a React component and pass it any options that fit your needs.
+ * When your component renders, `useSubmitSessionCoachFeedbackMutation` returns a tuple that includes:
+ * - A mutate function that you can call at any time to execute the mutation
+ * - An object with fields that represent the current status of the mutation's execution
+ *
+ * @param baseOptions options that will be passed into the mutation, supported options are listed on: https://www.apollographql.com/docs/react/api/react-hooks/#options-2;
+ *
+ * @example
+ * const [submitSessionCoachFeedbackMutation, { data, loading, error }] = useSubmitSessionCoachFeedbackMutation({
+ *   variables: {
+ *      videoId: // value for 'videoId'
+ *      generationId: // value for 'generationId'
+ *      response: // value for 'response'
+ *   },
+ * });
+ */
+export function useSubmitSessionCoachFeedbackMutation(
+  baseOptions?: Apollo.MutationHookOptions<
+    SubmitSessionCoachFeedbackMutation,
+    SubmitSessionCoachFeedbackMutationVariables
+  >,
+) {
+  const options = { ...defaultOptions, ...baseOptions };
+  return Apollo.useMutation<
+    SubmitSessionCoachFeedbackMutation,
+    SubmitSessionCoachFeedbackMutationVariables
+  >(SubmitSessionCoachFeedbackDocument, options);
+}
+export type SubmitSessionCoachFeedbackMutationHookResult = ReturnType<
+  typeof useSubmitSessionCoachFeedbackMutation
+>;
+export type SubmitSessionCoachFeedbackMutationResult =
+  Apollo.MutationResult<SubmitSessionCoachFeedbackMutation>;
+export type SubmitSessionCoachFeedbackMutationOptions =
+  Apollo.BaseMutationOptions<
+    SubmitSessionCoachFeedbackMutation,
+    SubmitSessionCoachFeedbackMutationVariables
+  >;
 export const CreateShotShareLinkDocument = gql`
   mutation CreateShotShareLink(
     $shotIds: [Int!]!
@@ -16111,12 +16048,12 @@ export const GetTableStateDocument = gql`
       tableSize: $tableSize
       useHomography: $useHomography
     ) {
-      ...TableStateLayoutFields
-      ...TableStateDetectionFields
+      identifierToPosition
+      identifierToColor {
+        hex
+      }
     }
   }
-  ${TableStateLayoutFieldsFragmentDoc}
-  ${TableStateDetectionFieldsFragmentDoc}
 `;
 
 /**
@@ -16192,11 +16129,13 @@ export const GetLiveTableStateDocument = gql`
       videoId
       frameIndex
       tableState {
-        ...TableStateLayoutFields
+        identifierToPosition
+        identifierToColor {
+          hex
+        }
       }
     }
   }
-  ${TableStateLayoutFieldsFragmentDoc}
 `;
 
 /**
@@ -16271,11 +16210,13 @@ export const GetShotTableStateDocument = gql`
       videoId
       frameIndex
       tableState {
-        ...TableStateLayoutFields
+        identifierToPosition
+        identifierToColor {
+          hex
+        }
       }
     }
   }
-  ${TableStateLayoutFieldsFragmentDoc}
 `;
 
 /**

src/operations/session-coach.gql

@@ -45,6 +45,21 @@ mutation RequestSessionCoachGeneration($videoId: Int!) {
   }
 }
 
+mutation SubmitSessionCoachFeedback(
+  $videoId: Int!
+  $generationId: ID!
+  $response: SessionCoachFeedbackResponseEnum!
+) {
+  submitSessionCoachFeedback(
+    videoId: $videoId
+    generationId: $generationId
+    response: $response
+  ) {
+    response
+    updatedAt
+  }
+}
+
 fragment SessionCoachCandidateFields on SessionCoachCandidateGQL {
   id
   rank
@@ -117,4 +132,8 @@ fragment SessionCoachGenerationFields on SessionCoachGenerationGQL {
     abstentionReason
     synthesisRationale
   }
+  feedback {
+    response
+    updatedAt
+  }
 }

src/operations/shot_simulation.gql

@@ -1,72 +1,3 @@
-fragment TableStateLayoutFields on TableStateGQL {
-  identifierToPosition
-  identifierToColor {
-    hex
-  }
-}
-
-fragment DetectionBoxFields on BoundingBoxGQL {
-  left
-  top
-  width
-  height
-}
-
-fragment DetectionPocketPointFields on PocketPointsGQL {
-  topLeft {
-    x
-    y
-  }
-  topSide {
-    x
-    y
-  }
-  topRight {
-    x
-    y
-  }
-  bottomLeft {
-    x
-    y
-  }
-  bottomSide {
-    x
-    y
-  }
-  bottomRight {
-    x
-    y
-  }
-}
-
-# Everything needed to redraw a detection over the image it came from:
-# the detector's boxes plus the homography that turned them into plane
-# positions. Boxes, pocket boxes, and source points share the crop's
-# coordinate space; the crop locates that space in the full frame.
-#
-# Only the photo query selects this. The live and shot queries read a
-# frame out of a video the app never holds, so there is no image to
-# overlay, and getLiveTableState is polled every five seconds.
-fragment TableStateDetectionFields on TableStateGQL {
-  identifierToBox {
-    ...DetectionBoxFields
-  }
-  homography {
-    crop {
-      ...DetectionBoxFields
-    }
-    pockets {
-      ...DetectionBoxFields
-    }
-    sourcePoints {
-      ...DetectionPocketPointFields
-    }
-    destPoints {
-      ...DetectionPocketPointFields
-    }
-  }
-}
-
 query GetTableState(
   $b64Image: String!
   $tableSize: Float
@@ -77,8 +8,10 @@ query GetTableState(
     tableSize: $tableSize
     useHomography: $useHomography
   ) {
-    ...TableStateLayoutFields
-    ...TableStateDetectionFields
+    identifierToPosition
+    identifierToColor {
+      hex
+    }
   }
 }
 
@@ -87,7 +20,10 @@ query GetLiveTableState($videoId: Int!) {
     videoId
     frameIndex
     tableState {
-      ...TableStateLayoutFields
+      identifierToPosition
+      identifierToColor {
+        hex
+      }
     }
   }
 }
@@ -98,7 +34,10 @@ query GetShotTableState($shotId: Int!) {
     videoId
     frameIndex
     tableState {
-      ...TableStateLayoutFields
+      identifierToPosition
+      identifierToColor {
+        hex
+      }
     }
   }
 }

src/schema.gql

@@ -249,11 +249,6 @@ input CreatePoolHallCameraInput {
   claimTermsText: String = null
 }
 
-input CreateShotShareLinkInput {
-  shotIds: [Int!]!
-  paddingSeconds: Float = null
-}
-
 input CreatePoolHallInput {
   name: String!
   address: String = null
@@ -264,6 +259,11 @@ input CreatePoolHallInput {
   timezone: String = null
 }
 
+input CreateShotShareLinkInput {
+  shotIds: [Int!]!
+  paddingSeconds: Float = null
+}
+
 type CreateSubscriptionResultGQL {
   checkoutUrl: String!
   sessionId: String!
@@ -728,6 +728,11 @@ type Mutation {
     shotId: Int!
     fieldsToEdit: EditableShotFieldInputGQL!
   ): EditShotReturn!
+  submitSessionCoachFeedback(
+    videoId: Int!
+    generationId: ID!
+    response: SessionCoachFeedbackResponseEnum!
+  ): SessionCoachFeedbackGQL!
   requestSessionCoachGeneration(videoId: Int!): SessionCoachGQL!
   getProfileImageUploadLink(
     fileExt: String = ".png"
@@ -1387,6 +1392,18 @@ type SessionCoachEvidenceGQL {
   baselineMakePercentage: Float
 }
 
+type SessionCoachFeedbackGQL {
+  response: SessionCoachFeedbackResponseEnum!
+  updatedAt: DateTime!
+}
+
+enum SessionCoachFeedbackResponseEnum {
+  HELPFUL
+  WRONG_FOCUS
+  ALREADY_WORKING_ON_THIS
+  NOT_ENOUGH_EVIDENCE
+}
+
 type SessionCoachGQL {
   videoId: Int!
   processingId: Int
@@ -1438,6 +1455,7 @@ type SessionCoachGenerationGQL {
   completedAt: DateTime
   decision: SessionCoachDecisionGQL
   diagnostics: SessionCoachGenerationDiagnosticsGQL
+  feedback: SessionCoachFeedbackGQL
 }
 
 enum SessionCoachGenerationStateEnum {
@@ -1459,6 +1477,11 @@ enum SessionCoachStateEnum {
   FAILED
 }
 
+type ShareLinkGQL {
+  slug: String!
+  url: String!
+}
+
 type ShotAnnotationGQL {
   shotId: Int!
   type: ShotAnnotationTypeGQL!
@@ -1708,7 +1731,6 @@ type TableStateGQL {
   identifierToPosition: [[Float!]!]!
   homography: HomographyInfoGQL
   identifierToColor: [RGBColorGQL]!
-  identifierToBox: [BoundingBoxGQL!]!
 }
 
 type TagClassGQL {
@@ -1920,11 +1942,6 @@ type UserSubscriptionStatusGQL {
   stripeSubscriptionId: String
 }
 
-type ShareLinkGQL {
-  slug: String!
-  url: String!
-}
-
 type VideoExportJobGQL {
   id: Int!
   videoId: Int!