Skip to content

Commit

Permalink
MINOR: Fix trend plot (#81)
Browse files Browse the repository at this point in the history
This PR remove an esm installation of Observable Plot. Previously Quarto
did not come bundled with that version of Plot which we needed for some
features. It does now so we can bump quarto and remove the pin.

Fixes #80
  • Loading branch information
boshek authored Mar 15, 2024
1 parent d9d6f30 commit 188d289
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/nightly_dashboard.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
- name: Setup Quarto
uses: quarto-dev/quarto-actions/setup@v2
with:
version: '1.3.450'
version: '1.4.549'

- uses: r-lib/actions/setup-r@v2
with:
Expand Down
2 changes: 0 additions & 2 deletions crossbow-nightly-report/crossbow-nightly-report.qmd
Original file line number Diff line number Diff line change
Expand Up @@ -203,8 +203,6 @@ trendPlot = function(data) {
```

```{ojs setup-ojs}
// import newer observable
Plot = await import("https://esm.sh/@observablehq/plot");
import { aq, op } from '@uwdata/arquero';
parser = d3.timeParse("%Y-%m-%d");
import {slider as slide} from "@jashkenas/inputs"; // more configurable slider
Expand Down

0 comments on commit 188d289

Please sign in to comment.