Merge pull request #28 from cmu-delphi/release/v2.0.3

Release v2.0.3

Author: krivard

.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,39 @@
+---
+name: 🐛 Bug report
+about: If something isn't working as expected 🤔.
+title: ''
+labels: 'bug'
+assignees: ''
+---
+
+**Actual Behavior:**
+
+<!--A clear and concise description of what the bug is.-->
+
+When I...
+
+**Steps to Reproduce the Bug:**
+
+<!--Steps to reproduce the behavior. Please copy paste the browser url as the first step-->
+
+1. go to **PASTE BROWSER URL HERE**
+
+**Expected behavior**
+
+<!--A clear and concise description of what you expected to happen.-->
+
+**Screenshots**
+
+<!--If applicable, add screenshots to help explain your problem.-->
+
+**Rating Scale (1 is minor, 7 is severe):**
+
+<!-- -->
+
+**Context**
+
+- Browser:
+
+**Additional context**
+
+<!--Add any other context about the problem here.-->

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1 @@
+blank_issues_enabled: true

.github/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,19 @@
+---
+name: 🚀 Feature Request
+about: Suggest an idea for this project
+title: ''
+labels: 'enhancement'
+assignees: ''
+---
+
+<!--A clear and concise description of what feature you are missing.-->
+
+It would be great if ...
+
+**User story**
+
+<!--Describe your problem, feature you are missing, and how you image to look like.-->
+
+**Additional context**
+
+<!--Add any other context or screenshots about the feature request here.-->

.github/workflows/create_release.yml

@@ -32,7 +32,7 @@ jobs:
           base: main
           title: Release ${{ steps.version.outputs.next_tag }}
           labels: chore
-          reviewers: krivard,sgratzl
-          assignees: sgratzl
+          reviewers: krivard
+          assignees: krivard
           body: |
             Releasing ${{ steps.version.outputs.next_tag }}.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "www-epivis",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "private": true,
   "license": "MIT",
   "description": "",

package.json

@@ -0,0 +1,16 @@
+2000-01-01,1,5.0
+2000-01-01,2,9.0
+2000-01-01,3,13.0
+2000-01-01,4,17.0
+2000-01-02,1,6.0
+2000-01-02,2,10.0
+2000-01-02,3,14.0
+2000-01-02,4,18.0
+2000-01-03,1,7.0
+2000-01-03,2,11.0
+2000-01-03,3,15.0
+2000-01-03,4,19.0
+2000-01-04,1,8.0
+2000-01-04,2,12.0
+2000-01-04,3,16.0
+2000-01-04,4,20.0

samples/groupby-test.csv

@@ -407,6 +407,28 @@
     const cy = (_yMin + _yMax) / 2;
     _yMin = cy - dy / 2;
     _yMax = cy + dy / 2;
+
+    // patch for straight lines
+    if (_xMin === _xMax) {
+      if (_xMin !== 0) {
+        _xMin *= 0.95;
+        _xMax *= 1.05;
+      } else {
+        _xMin = -0.5;
+        _xMax = 0.5;
+      }
+    }
+
+    if (_yMin === _yMax) {
+      if (_yMin !== 0) {
+        _yMin *= 0.95;
+        _yMax *= 1.05;
+      } else {
+        _yMin = -0.5;
+        _yMax = 0.5;
+      }
+    }
+
     setViewport(_xMin, _yMin, _xMax, _yMax, shouldAnimate);
   }
 

src/components/Chart.svelte

@@ -40,7 +40,7 @@ export default function importCSV(file: File, fileContent: string, options: CSVO
   const lines: string[] = [];
   for (const row of fileContent.split('\n')) {
     const trimmed = row.trim();
-    if (trimmed[0] === '#') {
+    if (trimmed[0] === '#' || trimmed.length === 0) {
       continue;
     }
     lines.push(trimmed);
@@ -94,18 +94,21 @@ export default function importCSV(file: File, fileContent: string, options: CSVO
     return asDataGroup(parseGroup(file.name, 0, activeGroup.rows, options, labels));
   }
   const root: IDataGroup = { title: file.name, level: 0, datasets: [] };
+  // shift to have at least under the root
+  const levelOffset = groups.reduce((acc, g) => Math.min(acc, g.level), Number.POSITIVE_INFINITY) - 1;
+
   let active = root;
   for (const group of groups) {
-    const parsed = parseGroup(group.label, group.level, group.rows, options, labels);
-    if (group.level > active.level) {
+    const parsed = parseGroup(group.label, group.level - levelOffset, group.rows, options, labels);
+    if (parsed.level > active.level) {
       // child
       active.datasets.push(parsed);
       parsed.parent = active;
       active = parsed;
       continue;
     }
     // multiple levels up
-    while (group.level < active.level) {
+    while (parsed.level < active.level) {
       active = active.parent!;
     }
     // sibling
@@ -120,7 +123,7 @@ function parseGroup(title: string, level: number, rows: string[][], options: CSV
   const dates = rows.map((row, i) => parseDate(row, i, options));
   const datasets: DataSet[] = [];
   labels.forEach((label, i) => {
-    if (isDateColumn(i, options)) {
+    if (isDateColumn(i, options) || (options.hasGroup && options.groupColumn === i)) {
       return;
     }
     const data = rows.map((row, j) => {
@@ -132,32 +135,31 @@ function parseGroup(title: string, level: number, rows: string[][], options: CSV
   return { title, level, datasets };
 }
 
-function splitGroup(rows: string[][], prefix: string, level: number, groupColumn: number) {
+function splitGroup(
+  rows: string[][],
+  prefix: string,
+  level: number,
+  groupColumn: number,
+): { label: string; level: number; rows: string[][] }[] {
   if (rows.length === 0) {
     return [];
   }
-  const r: { label: string; level: number; rows: string[][] }[] = [
-    {
-      label: prefix + String(rows[0][groupColumn]),
-      level,
-      rows: [],
-    },
-  ];
-  let active = r[0];
+  const groups: Map<string, { label: string; level: number; rows: string[][] }> = new Map();
+
   for (const row of rows) {
     const group = prefix + String(row[groupColumn]);
-    if (group != active.label) {
-      r.push({
+    const g = groups.get(group);
+    if (g) {
+      g.rows.push(row);
+    } else {
+      groups.set(group, {
         label: group,
         level,
         rows: [row],
       });
-      active = r[r.length - 1];
-    } else {
-      active.rows.push(row);
     }
   }
-  return r;
+  return Array.from(groups.values());
 }
 
 function isDateColumn(column: number, options: CSVOptions) {