remove isInternal from frontend code

Renumics · Oct 9, 2023 · b799e25 · b799e25
1 parent 758854d
commit b799e25
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 67 deletions.
diff --git a/src/stores/dataset/colorTransferFunctionFactory.tsx b/src/stores/dataset/colorTransferFunctionFactory.tsx
@@ -64,11 +64,7 @@ export const makeColumnsColorTransferFunctions = (
     filteredMask: boolean[]
 ): ColumnsTransferFunctions => {
     return columns
-        .filter(
-            (column) =>
-                !column.isInternal &&
-                (isScalarColumn(column) || isCategoricalColumn(column))
-        )
+        .filter((column) => isScalarColumn(column) || isCategoricalColumn(column))
         .reduce((a, column) => {
             a[column.key] = {
                 full: makeApplicableColorTransferFunctions(

diff --git a/src/stores/dataset/columnFactory.ts b/src/stores/dataset/columnFactory.ts
@@ -66,11 +66,6 @@ export function makeColumn(column: Column, index: number): DataColumn {
         optional: column.optional,
         description: column.description ?? '',
         tags: _.uniq(column.tags),
-
-        // we access some internal columns like __id__ by their name
-        // therefore, if we set the key to something different than column.name
-        // we have to check for isInternal and use column.name for it
-        isInternal: column.name.startsWith('__'),
     };
 
     return col;

diff --git a/src/stores/dataset/dataset.ts b/src/stores/dataset/dataset.ts
@@ -423,10 +423,7 @@ export const useDataset = create(
             },
             recomputeColorTransferFunctions: async () => {
                 const columnsToCompute = get()
-                    .columns.filter(
-                        (c) =>
-                            !c.isInternal && (isScalar(c.type) || isCategorical(c.type))
-                    )
+                    .columns.filter((c) => isScalar(c.type) || isCategorical(c.type))
                     .map((c) => c.key);
 
                 const newTransferFunctions = makeColumnsColorTransferFunctions(

diff --git a/src/types/dataset.ts b/src/types/dataset.ts
@@ -8,7 +8,6 @@ export interface DataColumn {
     type: datatypes.DataType;
     editable: boolean;
     optional: boolean;
-    isInternal: boolean;
     description: string;
     tags: string[];
 }

diff --git a/src/widgets/Histogram/Histogram.tsx b/src/widgets/Histogram/Histogram.tsx
@@ -35,7 +35,7 @@ const Histogram: Widget = () => {
     const columnKeys = useMemo(
         () =>
             columns
-                .filter((col) => !col.isInternal && validTypes.includes(col.type.kind))
+                .filter((col) => validTypes.includes(col.type.kind))
                 .map((col) => col.key),
         [columns]
     );

diff --git a/src/widgets/ScatterplotView/ScatterplotView.tsx b/src/widgets/ScatterplotView/ScatterplotView.tsx
@@ -287,10 +287,7 @@ const ScatterplotView: Widget = () => {
 
             // compute z-scores for all number columns and order them descending
             const remainingColumns = Object.values(allColumns).filter(
-                (col) =>
-                    isNumberColumn(col) &&
-                    !col.isInternal &&
-                    !defaultColumns.includes(col.key)
+                (col) => isNumberColumn(col) && !defaultColumns.includes(col.key)
             ) as NumberColumn[];
             const interestingColumns = sortColumnsByZScore(
                 rowIndex,
@@ -327,33 +324,24 @@ const ScatterplotView: Widget = () => {
     const placeableColumns = useMemo(
         () =>
             Object.values(allColumns)
-                .filter(
-                    (col) =>
-                        ['int', 'float', 'bool'].includes(col.type.kind) &&
-                        !col.isInternal
-                )
+                .filter((col) => ['int', 'float', 'bool'].includes(col.type.kind))
                 .map((col) => col.key),
         [allColumns]
     );
     const colorableColumns = useMemo(
         () =>
             Object.values(allColumns)
-                .filter(
-                    (col) =>
-                        ['int', 'float', 'str', 'bool', 'Category'].includes(
-                            col.type.kind
-                        ) && !col.isInternal
+                .filter((col) =>
+                    ['int', 'float', 'str', 'bool', 'Category'].includes(col.type.kind)
                 )
                 .map((col) => col.key),
         [allColumns]
     );
     const scaleableColumns = useMemo(
         () =>
             Object.values(allColumns)
-                .filter(
-                    (col: DataColumn) =>
-                        ['int', 'float', 'bool'].includes(col.type.kind) &&
-                        !col.isInternal
+                .filter((col: DataColumn) =>
+                    ['int', 'float', 'bool'].includes(col.type.kind)
                 )
                 .map((col) => col.key),
         [allColumns]

diff --git a/src/widgets/SimilarityMap/MenuBar.tsx b/src/widgets/SimilarityMap/MenuBar.tsx
@@ -50,12 +50,6 @@ const columnTypeSelector = (d: Dataset): { [key: string]: DataType } =>
         return a;
     }, {});
 
-const columnIsInternalSelector = (d: Dataset): { [key: string]: boolean } =>
-    d.columns.reduce((a: { [key: string]: boolean }, c: DataColumn) => {
-        a[c.key] = c.isInternal;
-        return a;
-    }, {});
-
 const SettingsMenu = ({
     colorBy,
     sizeBy,
@@ -77,7 +71,6 @@ const SettingsMenu = ({
     onChangePCANormalization,
 }: Props) => {
     const columnType = useDataset(columnTypeSelector, shallow);
-    const columnIsInternal = useDataset(columnIsInternalSelector, shallow);
 
     const changePlaceBy = useCallback(
         (keys: string[]): void => {
@@ -88,27 +81,20 @@ const SettingsMenu = ({
 
     const colorableColumns = useMemo(
         () =>
-            Object.entries(columnType)
-                .filter(
-                    ([key, type]) =>
-                        ['int', 'float', 'str', 'bool', 'Category'].includes(
-                            type.kind
-                        ) && !columnIsInternal[key]
+            Object.values(columnType)
+                .filter((type) =>
+                    ['int', 'float', 'str', 'bool', 'Category'].includes(type.kind)
                 )
-                .map(([key]) => key),
-        [columnIsInternal, columnType]
+                .map((type) => type.kind),
+        [columnType]
     );
 
     const scaleableColumns = useMemo(
         () =>
-            Object.entries(columnType)
-                .filter(
-                    ([key, type]) =>
-                        ['int', 'float', 'bool'].includes(type.kind) &&
-                        !columnIsInternal[key]
-                )
-                .map(([key]) => key),
-        [columnIsInternal, columnType]
+            Object.values(columnType)
+                .filter((type) => ['int', 'float', 'bool'].includes(type.kind))
+                .map((type) => type.kind),
+        [columnType]
     );
 
     const reductionParameterMenu =

diff --git a/src/widgets/SimilarityMap/SimilarityMap.tsx b/src/widgets/SimilarityMap/SimilarityMap.tsx
@@ -124,7 +124,6 @@ const SimilarityMap: Widget = () => {
             return undefined;
         }
         const availableColumns = fullColumns
-            .filter((col) => !col.isInternal)
             .filter((col) =>
                 ['int', 'float', 'str', 'bool', 'Category'].includes(col.type.kind)
             )
@@ -165,11 +164,10 @@ const SimilarityMap: Widget = () => {
 
     const embeddableColumnKeys = useMemo(() => {
         return fullColumns
-            .filter(
-                (col) =>
-                    ['int', 'bool', 'float', 'Category', 'Embedding'].includes(
-                        col.type.kind
-                    ) && !col.isInternal
+            .filter((col) =>
+                ['int', 'bool', 'float', 'Category', 'Embedding'].includes(
+                    col.type.kind
+                )
             )
             .map((c) => c.key);
     }, [fullColumns]);
@@ -355,10 +353,7 @@ const SimilarityMap: Widget = () => {
             // compute z-scores for all number columns and order them descending
             //
             const remainingColumns = fullColumns.filter(
-                (col) =>
-                    isNumberColumn(col) &&
-                    !col.isInternal &&
-                    !defaultColumns.includes(col)
+                (col) => isNumberColumn(col) && !defaultColumns.includes(col)
             ) as NumberColumn[];
             const interestingColumns = sortColumnsByZScore(
                 rowIndex,