[dagit] Do not display the partition_set, step_selection tags on asse…

…t group runs (#7953)
dagster-io · May 23, 2022 · 5ae7d32 · 5ae7d32
1 parent cbd7da9
commit 5ae7d32
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/js_modules/dagit/packages/core/src/assets/AssetNodeList.tsx b/js_modules/dagit/packages/core/src/assets/AssetNodeList.tsx
@@ -37,7 +37,7 @@ export const AssetNodeList: React.FC<{
 };
 
 const Container = styled(Box)`
-  height: 112px;
+  height: 124px;
   overflow-x: auto;
   width: 100%;
   white-space: nowrap;

diff --git a/js_modules/dagit/packages/core/src/runs/RunTags.tsx b/js_modules/dagit/packages/core/src/runs/RunTags.tsx
@@ -3,6 +3,7 @@ import * as React from 'react';
 
 import {SharedToaster} from '../app/DomUtils';
 import {useCopyToClipboard} from '../app/browser';
+import {__ASSET_GROUP_PREFIX} from '../asset-graph/Utils';
 
 import {DagsterTag, RunTag, TagType} from './RunTag';
 import {RunFilterToken} from './RunsFilterInput';
@@ -44,11 +45,16 @@ export const RunTags: React.FC<{
     return list;
   }, [copy, onSetFilter]);
 
-  const sortedTags = React.useMemo(() => {
+  const displayedTags = React.useMemo(() => {
     const priority = [];
     const others = [];
     for (const tag of tags) {
-      if (priorityTagSet.has(tag.key)) {
+      if (
+        tag.value.startsWith(__ASSET_GROUP_PREFIX) &&
+        (tag.key === DagsterTag.PartitionSet || tag.key === DagsterTag.StepSelection)
+      ) {
+        continue;
+      } else if (priorityTagSet.has(tag.key)) {
         priority.push(tag);
       } else {
         others.push(tag);
@@ -64,7 +70,7 @@ export const RunTags: React.FC<{
   return (
     <Box flex={{direction: 'row', wrap: 'wrap', gap: 4}}>
       {mode ? <RunTag tag={{key: 'mode', value: mode}} /> : null}
-      {sortedTags.map((tag, idx) => (
+      {displayedTags.map((tag, idx) => (
         <RunTag tag={tag} key={idx} actions={actions} />
       ))}
     </Box>