Merge release/v1.2.0 to develop

app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Evaluate.tsx

@@ -1,13 +1,25 @@
 import { MuiButton } from "@fiftyone/components";
 import { Add } from "@mui/icons-material";
+import { Box } from "@mui/material";
 import React from "react";
 
 export default function Evaluate(props: EvaluateProps) {
-  const { onEvaluate } = props;
+  const { onEvaluate, permissions } = props;
+  const canEvaluate = permissions.can_evaluate;
   return (
-    <MuiButton onClick={onEvaluate} startIcon={<Add />} variant="contained">
-      Evaluate Model
-    </MuiButton>
+    <Box
+      title={canEvaluate ? "" : "You do not have permission to evaluate model"}
+      sx={{ cursor: canEvaluate ? "pointer" : "not-allowed" }}
+    >
+      <MuiButton
+        onClick={onEvaluate}
+        startIcon={<Add />}
+        variant="contained"
+        disabled={!canEvaluate}
+      >
+        Evaluate Model
+      </MuiButton>
+    </Box>
   );
 }
 

app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Evaluation.tsx

@@ -51,6 +51,7 @@ import { formatValue, getNumericDifference, useTriggerEvent } from "./utils";
 const KEY_COLOR = "#ff6d04";
 const COMPARE_KEY_COLOR = "#03a9f4";
 const DEFAULT_BAR_CONFIG = { sortBy: "default" };
+const NONE_CLASS = "(none)";
 
 export default function Evaluation(props: EvaluationProps) {
   const {
@@ -611,20 +612,26 @@ export default function Evaluation(props: EvaluationProps) {
       <Card sx={{ p: 2 }}>
         <Stack direction="row" sx={{ justifyContent: "space-between" }}>
           <Typography color="secondary">Evaluation notes</Typography>
-          {can_edit_note && (
-            <Box>
-              <IconButton
-                size="small"
-                color="secondary"
-                sx={{ borderRadius: 16 }}
-                onClick={() => {
-                  setEditNoteState((note) => ({ ...note, open: true }));
-                }}
-              >
-                <EditNote />
-              </IconButton>
-            </Box>
-          )}
+          <Box
+            title={
+              can_edit_note
+                ? ""
+                : "You do not have permission to edit evaluation notes"
+            }
+            sx={{ cursor: can_edit_note ? "pointer" : "not-allowed" }}
+          >
+            <IconButton
+              size="small"
+              color="secondary"
+              sx={{ borderRadius: 16 }}
+              onClick={() => {
+                setEditNoteState((note) => ({ ...note, open: true }));
+              }}
+              disabled={!can_edit_note}
+            >
+              <EditNote />
+            </IconButton>
+          </Box>
         </Stack>
         <EvaluationNotes notes={evaluationNotes} variant="details" />
       </Card>
@@ -1656,12 +1663,23 @@ function getMatrix(matrices, config, maskTargets, compareMaskTargets?) {
   if (!matrices) return;
   const { sortBy = "az", limit } = config;
   const parsedLimit = typeof limit === "number" ? limit : undefined;
-  const classes = matrices[`${sortBy}_classes`].slice(0, parsedLimit);
-  const matrix = matrices[`${sortBy}_matrix`].slice(0, parsedLimit);
+  const originalClasses = matrices[`${sortBy}_classes`];
+  const originalMatrix = matrices[`${sortBy}_matrix`];
+  const classes = originalClasses.slice(0, parsedLimit);
+  const matrix = originalMatrix.slice(0, parsedLimit);
   const colorscale = matrices[`${sortBy}_colorscale`];
   const labels = classes.map((c) => {
     return compareMaskTargets?.[c] || maskTargets?.[c] || c;
   });
+  const noneIndex = originalClasses.indexOf(NONE_CLASS);
+  if (parsedLimit < originalClasses.length) {
+    labels.push(
+      compareMaskTargets?.[NONE_CLASS] ||
+        maskTargets?.[NONE_CLASS] ||
+        NONE_CLASS
+    );
+    matrix.push(originalMatrix[noneIndex]);
+  }
   return { labels, matrix, colorscale };
 }
 
@@ -1687,7 +1705,7 @@ function useActiveFilter(evaluation, compareEvaluation) {
     const { _cls, kwargs } = stage;
     if (_cls.endsWith("FilterLabels")) {
       const [_, filter] = kwargs;
-      const filterEq = filter[1].$eq;
+      const filterEq = filter[1].$eq || [];
       const [filterEqLeft, filterEqRight] = filterEq;
       if (filterEqLeft === "$$this.label") {
         return { type: "label", value: filterEqRight };

app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Overview.tsx

@@ -97,7 +97,7 @@ function EvaluationCard(props: EvaluationCardProps) {
               }
             />
           )}
-          {status && <Status status={status} />}
+          {status && <Status status={status} readOnly />}
         </Stack>
         {note && <EvaluationNotes notes={note} variant="overview" />}
       </Card>

app/packages/core/src/plugins/SchemaIO/components/NativeModelEvaluationView/Status.tsx

@@ -4,10 +4,10 @@ import React from "react";
 import { useTriggerEvent } from "./utils";
 
 export default function Status(props: StatusProps) {
-  const { status, canEdit, setStatusEvent } = props;
+  const { status, canEdit, readOnly, setStatusEvent } = props;
   const triggerEvent = useTriggerEvent();
 
-  if (canEdit) {
+  if (!readOnly) {
     return (
       <Select
         sx={{
@@ -22,6 +22,12 @@ export default function Status(props: StatusProps) {
         onChange={(e) => {
           triggerEvent(setStatusEvent, { status: e.target.value });
         }}
+        title={
+          canEdit
+            ? ""
+            : "You do not have permission to update evaluation status"
+        }
+        disabled={!canEdit}
       >
         {STATUSES.map((status) => {
           const color = COLOR_BY_STATUS[status];
@@ -63,7 +69,8 @@ export default function Status(props: StatusProps) {
 type StatusProps = {
   status?: string;
   canEdit?: boolean;
-  setStatusEvent?: string;
+  readOnly?: boolean;
+  setStatusEvent: string;
 };
 
 const STATUS_LABELS = {

app/packages/operators/src/CustomPanel.tsx

@@ -14,12 +14,14 @@
 import { Property } from "./types";
 import { CustomPanelProps, useCustomPanelHooks } from "./useCustomPanelHooks";
 import { useTrackEvent } from "@fiftyone/analytics";
+import usePanelEvent from "./usePanelEvent";
 
 export function CustomPanel(props: CustomPanelProps) {
   const { panelId, dimensions, panelName, panelLabel, isModalPanel } = props;
   const { height, width } = dimensions?.bounds || {};
   const { count } = useActivePanelEventsCount(panelId);
   const [_, setLoading] = usePanelLoading(panelId);
+  const triggerPanelEvent = usePanelEvent();
 
   const {
     handlePanelStateChange,
@@ -36,8 +38,9 @@
     setPanelCloseEffect(() => {
       clearUseKeyStores(panelId);
       trackEvent("close_panel", { panel: panelName });
+      triggerPanelEvent(panelId, { operator: props.onUnLoad });
     });
   }, []);
 
   useEffect(() => {
     setLoading(count > 0);
@@ -102,7 +105,7 @@
 
   useEffect(() => {
     dimensions?.refresh();
   }, []);
 
   return children;
 }

app/packages/operators/src/useCustomPanelHooks.ts

@@ -165,14 +165,6 @@ export function useCustomPanelHooks(props: CustomPanelProps): CustomPanelHooks {
     triggerPanelEvent,
   ]);
 
-  useEffect(() => {
-    return () => {
-      if (props.onUnLoad) {
-        triggerPanelEvent(panelId, { operator: props.onUnLoad });
-      }
-    };
-  }, []); // eslint-disable-line react-hooks/exhaustive-deps
-
   const handlePanelStateChangeOpDebounced = useMemo(() => {
     return debounce(
       (state, onChange, panelId) => {

docs/source/release-notes.rst

@@ -3,6 +3,101 @@ FiftyOne Release Notes
 
 .. default-role:: code
 
+FiftyOne Teams 2.3.0
+--------------------
+*Released December 20, 2024*
+
+Includes all updates from :ref:`FiftyOne 1.2.0 <release-notes-v1.2.0>`, plus:
+
+- Added an example :ref:`Databricks connector <data-lens-databricks>` showing
+  how to connect FiftyOne Teams to your lakehouse via
+  :ref:`Data Lens <data-lens>`
+- Added a :ref:`Data Lens connector <data-lens-snippet-remap-fields>`
+  that demonstrates how to allow users to dynamically configure the field(s)
+  that are imported
+- :ref:`Data Lens <data-lens>` now supports previewing 3D data imports
+- Guest users can now open :ref:`Data Lens <data-lens>`
+- When scanning for issues with the :ref:`Data Quality Panel <data-quality>`,
+  any fields created are now added to a `DATA QUALITY` sidebar group
+- Prevented unnecessary scrollbars from appearing when using the
+  :ref:`Data Quality Panel <data-quality>`
+- AWS session tokens are now supported when configuring
+  :ref:`cloud credentials <teams-cloud-credentials>`
+- Fixed a bug that could cause `StopIteration` errors when performing
+  long-running operations like computing embeddings when using
+  :ref:`API connections <teams-api-connection>`
+
+.. _release-notes-v1.2.0:
+
+FiftyOne 1.2.0
+--------------
+*Released December 30, 2024*
+
+App
+
+- Added support for :ref:`instance segmentations <instance-segmentation>` whose
+  masks are stored on-disk
+  `#5120 <https://github.com/voxel51/fiftyone/pull/5120>`_,
+  `#5256 <https://github.com/voxel51/fiftyone/pull/5256>`_
+- Optimized overlay rendering for dense label fields like segmentations and
+  heatmaps
+  `#5156 <https://github.com/voxel51/fiftyone/pull/5156>`_,
+  `#5169 <https://github.com/voxel51/fiftyone/pull/5169>`_,
+  `#5247 <https://github.com/voxel51/fiftyone/pull/5247>`_
+- Improved stability of frame rendering for videos
+  `#5199 <https://github.com/voxel51/fiftyone/pull/5199>`_,
+  `#5293 <https://github.com/voxel51/fiftyone/pull/5293>`_
+- Sidebar groups that contain only list fields are no longer collapsed by
+  default
+  `#5280 <https://github.com/voxel51/fiftyone/pull/5280>`_
+- The :ref:`Model Evaluation panel <app-model-evaluation-panel>` now filters
+  both ground truth and prediction fields when you perform interactive filters
+  via the TP/FP/FN icons, per-class histograms, and confusion matrices
+  `#5268 <https://github.com/voxel51/fiftyone/pull/5268>`_
+- When comparing two models in the
+  :ref:`Model Evaluation panel <app-model-evaluation-panel>`, interactive
+  filters now apply to both evaluation runs
+  `#5268 <https://github.com/voxel51/fiftyone/pull/5268>`_
+- The :ref:`Model Evaluation panel <app-model-evaluation-panel>` now supports
+  evaluations that were performed on subsets (views) of the full dataset
+  `#5267 <https://github.com/voxel51/fiftyone/pull/5267>`_
+- The :ref:`Model Evaluation panel <app-model-evaluation-panel>` now shows mask
+  targets for segmentation evaluations when they are available
+  `#5281 <https://github.com/voxel51/fiftyone/pull/5281>`_
+- The :ref:`Model Evaluation panel <app-model-evaluation-panel>` now hides
+  metrics that aren't applicable to a given evaluation type
+  `#5281 <https://github.com/voxel51/fiftyone/pull/5281>`_
+- Fixed an issue where backtick can't be typed when editing markdown notes in
+  the :ref:`Model Evaluation panel <app-model-evaluation-panel>`
+  `#5233 <https://github.com/voxel51/fiftyone/pull/5233>`_
+- Fixed a race condition that could cause errors when performing
+  :ref:`text similarity searches <brain-similarity-text>`
+  `#5273 <https://github.com/voxel51/fiftyone/pull/5273>`_
+- Fixed a caching bug that prevented label overlay font sizes from dynamically
+  resizing as expected in some cases
+  `#5287 <https://github.com/voxel51/fiftyone/pull/5287>`_
+- Fixed a bug that excluded selected samples from the counter above the Samples
+  panel
+  `#5286 <https://github.com/voxel51/fiftyone/pull/5286>`_
+
+SDK
+
+- Optimized :meth:`dataset.first() <fiftyone.core.dataset.Dataset.first>` calls
+  `#5305 <https://github.com/voxel51/fiftyone/pull/5305>`_
+
+Brain
+
+- Upgraded the :Ref:`MongoDB vector search integration <mongodb-integration>`
+  to use the `vectorSearch` type
+  `#218 <https://github.com/voxel51/fiftyone-brain/pull/218>`_
+
+Zoo
+
+- Fixed a bug with loading the
+  :ref:`rtdetr-l-coco-torch <model-zoo-rtdetr-l-coco-torch>` and
+  :ref:`rtdetr-x-coco-torch <model-zoo-rtdetr-x-coco-torch>` zoo models
+  `#5220 <https://github.com/voxel51/fiftyone/pull/5220>`_
+
 FiftyOne Teams 2.2.0
 --------------------
 *Released December 6, 2024*
@@ -21,7 +116,7 @@ Includes all updates from :ref:`FiftyOne 1.1.0 <release-notes-v1.1.0>`, plus:
 - Added support for creating embeddings visualizations natively from the
   :ref:`Embeddings panel <app-embeddings-panel>`
 - Added support for evaluating models natively from the
-  :ref:`Modal Evaluation panel <app-model-evaluation-panel>`
+  :ref:`Model Evaluation panel <app-model-evaluation-panel>`
 - Added support for :ref:`configuring an SMTP server <identity-providers>` for
   sending user invitiations via email when running in
   :ref:`Internal Mode <internal-mode>`
@@ -2639,8 +2734,8 @@ Core
 - Using new libraries for ndjson and archive extraction
   `#2864 <https://github.com/voxel51/fiftyone/pull/2864>`_
 - Fixed a bug that prevented
-  :ref:`text similarity searches <brain-similarity-text>` searches from
-  succeeding when GPU is available
+  :ref:`text similarity searches <brain-similarity-text>` from succeeding when
+  GPU is available
   `#2853 <https://github.com/voxel51/fiftyone/pull/2853>`_
 - Fixed a bug where
   :meth:`stats() <fiftyone.core.collections.SampleCollection.stats>` would