Skip to content

Commit 0a7a476

Browse files
authored
Add job result rendering (#102)
* add util function to index into system definitions from a schema path string/array * lint, update test * fix handling for root system schema path * add get job results api functions * stubs for new vue components * add apache-arrow * add plotly, manually add @types/plotly.js to typesRoot to include it due to mismatch with plotly.js-dist * use plotly.js instead of plotly.js-dist * remove typeroot for plotly * fix time axis * add job compute api func * fix tsconfig, schema indexing for root * initial radio button result select implementation * react to data/radio button selection of timeseries results * use smaller plotly dist, add postinstall script * add postinstall step to install commands * add note to readme * mulipass? * run postinstall manually in e2e workflow * add responsive config to timeseries plot * add download options for summary table * avoid null job results by passing whole job object to jobResults * add function to continuously check job status from results tab * allow for downloading of timeseries data, don't plot timeseries until user selects one
1 parent 1c55f3a commit 0a7a476

File tree

14 files changed

+676
-21
lines changed

14 files changed

+676
-21
lines changed

.github/workflows/e2e.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,11 @@ jobs:
5151
chmod +x ./dbmate
5252
./dbmate --wait up
5353
54+
- name: Run js postinstall
55+
run: |
56+
cd dashboard
57+
npm run postinstall
58+
5459
- name: Cypress run
5560
uses: cypress-io/github-action@v2
5661
with:

.github/workflows/javascript.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ jobs:
2424
run: |
2525
cd dashboard
2626
npm install
27+
npm run postinstall
2728
2829
- name: Test
2930
run: |
@@ -53,6 +54,7 @@ jobs:
5354
run: |
5455
cd dashboard
5556
npm install
57+
npm run postinstall
5658
5759
- name: Lint
5860
run: |

Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ FROM node:14-buster as jsbuild
2828
COPY ./dashboard js
2929
RUN cd js && \
3030
npm install && \
31+
npm run postinstall && \
3132
npm run build
3233

3334
FROM basepython as wheelbuild

dashboard/README.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,14 @@
11
# Solar Performance Insight Dashboard
22

33
## Project setup
4+
*NOTE:* This project uses the smaller `plotly.js-basic-dist` npm package. To
5+
use the correct types from [definitelytypes](https://github.com/DefinitelyTyped/DefinitelyTyped) we
6+
need to rename the plotly types directory in `node_modules` after install to
7+
make sure Typescript knows where to find the definitions. The `postinstall`
8+
node script will rename this directory appropriately for you.
49
```
510
npm install
11+
npm run postinstall
612
```
713

814
### Compiles and hot-reloads for development

dashboard/package-lock.json

Lines changed: 181 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)