Logic for fixing error plot display after expression archive moved fr…

…om ErrorPlot to ExpressionTable

Currently the logic for fixing error plot display after expression archive was done by modifying selectedExpressionId in ErrorPlot and ExpressionTable both. However this can be done in a single modification in ExpressionTable. This comit has updated ExpressionTable by modifying selectedExpressionId over there correctly and removed selectedExpressionId modification from ErrorPlot.

src/herbie/ErrorPlot.tsx

@@ -200,9 +200,7 @@ function ErrorPlot() {
   const expressions = allExpressions.filter(e => !archivedExpressions.includes(e.id))
 
   // console.log('selectedExprId', selectedExprId)
-  if (expressions.length > 0 && selectedExprId === -1) {
-    setSelectedExprId(expressions[0].id);
-  } 
+
   // get the expression
   const selectedExpr = expressions.find(e => e.id === selectedExprId)
   if (!selectedExpr) {

src/herbie/ExpressionTable.tsx

@@ -307,7 +307,10 @@ function ExpressionTable() {
                   <div className="delete">
                     <button onClick={() =>{ 
                       setArchivedExpressions([...archivedExpressions, expression.id]);
-                      setSelectedExprId(-1);
+                      const activeExp = activeExpressions.filter(id => id !== expression.id);
+                      if (activeExp.length > 0) {
+                        setSelectedExprId(activeExp[0]);
+                      } 
                       }}>
                     ╳
                     </button>