fix quote flag (#55)

sfcompute · Jan 16, 2025 · 73c58be · 73c58be
1 parent 20cf70a
commit 73c58be
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 20 deletions.
diff --git a/src/lib/Quote.tsx b/src/lib/Quote.tsx
@@ -3,6 +3,8 @@ import { Row } from "./Row.tsx";
 import dayjs from "dayjs";
 import { GPUS_PER_NODE } from "./constants.ts";
 import React from "react";
+import { getPricePerGpuHourFromQuote, getTotalPrice } from "./buy/index.tsx";
+import ms from "ms";
 
 export default function QuoteDisplay(props: { quote: Quote }) {
   if (!props.quote) {
@@ -21,27 +23,17 @@ export default function QuoteDisplay(props: { quote: Quote }) {
     );
   }
 
-  const durationSeconds = dayjs(props.quote.end_at).diff(
-    dayjs(props.quote.start_at),
-    "seconds"
-  );
-  const durationHours = durationSeconds / 3600;
-  const pricePerHour =
-    props.quote.price /
-    durationHours /
-    GPUS_PER_NODE /
-    props.quote.quantity /
-    100;
-  const priceTotal = props.quote.price / 100;
+  const pricePerGpuHourCents = getPricePerGpuHourFromQuote(props.quote);
+  const totalPrice = props.quote.price / 100;
 
   return (
     <Box flexDirection="column" paddingBottom={1}>
       <Row
         headWidth={10}
         head="rate"
-        value={`$${pricePerHour.toFixed(2)}/gpu/hr`}
+        value={`~$${(pricePerGpuHourCents / 100).toFixed(2)}/gpu/hr`}
       />
-      <Row headWidth={10} head="total" value={`$${priceTotal.toFixed(2)}`} />
+      <Row headWidth={10} head="total" value={`$${totalPrice.toFixed(2)}`} />
     </Box>
   );
 }

diff --git a/src/lib/buy/index.tsx b/src/lib/buy/index.tsx
@@ -129,7 +129,7 @@ function QuoteComponent(props: { options: SfBuyOptions }) {
       }
       setQuote(quote);
     })();
-  }, [props.options]);
+  }, []);
 
   return isLoading ? (
     <Box gap={1}>
@@ -172,8 +172,6 @@ function QuoteAndBuy(props: { options: SfBuyOptions }) {
   // submit a quote request, handle loading state
   useEffect(() => {
     (async () => {
-      const quote = await getQuoteFromParsedSfBuyOptions(props.options);
-
       // Grab the price per GPU hour, either
       let pricePerGpuHour: number | null = parsePricePerGpuHour(
         props.options.price
@@ -230,7 +228,7 @@ function roundEndDate(endDate: Date) {
   return dayjs(endDate).add(1, "hour").startOf("hour");
 }
 
-function getTotalPrice(
+export function getTotalPrice(
   pricePerGpuHour: number,
   size: number,
   durationInHours: number
@@ -297,7 +295,7 @@ function BuyOrderPreview(props: {
       <Row
         headWidth={7}
         head="rate"
-        value={`$${(props.price / 100).toFixed(2)}/gpu/hr`}
+        value={`~$${(props.price / 100).toFixed(2)}/gpu/hr`}
       />
       <Row headWidth={7} head="total" value={`$${totalPrice.toFixed(2)}`} />
     </Box>
@@ -567,7 +565,7 @@ export async function placeBuyOrder(options: {
   return data;
 }
 
-function getPricePerGpuHourFromQuote(quote: NonNullable<Quote>) {
+export function getPricePerGpuHourFromQuote(quote: NonNullable<Quote>) {
   const durationSeconds = dayjs(quote.end_at).diff(
     parseStartAsDate(quote.start_at)
   );