Skip to content

Updating sda page#4

Open
nshiab wants to merge 3 commits into
Fil:mainfrom
nshiab:main
Open

Updating sda page#4
nshiab wants to merge 3 commits into
Fil:mainfrom
nshiab:main

Conversation

@nshiab
Copy link
Copy Markdown

@nshiab nshiab commented Oct 31, 2024

Updating sda to v3.

Thanks!

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 31, 2024

@nshiab is attempting to deploy a commit to the Fil's projects Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread src/party/sda.md

```js echo
import {SimpleDB} from "npm:simple-data-analysis@2";
import { SimpleWebDB } from "https://esm.sh/jsr/@nshiab/simple-data-analysis@3.15.3/web";
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd want this to work:

import {SimpleWebDB} from "jsr:@nshiab/simple-data-analysis@3/web";

unfortunately it fails with Missing "./web" specifier in "@jsr/nshiab__simple-data-analysis" package, which may be an issue with the package or with framework (?). I'm not sure how to fix it yet.

Note that ideally, I'd want this:

import {SimpleWebDB} from "jsr:@nshiab/simple-data-analysis@3";

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also want to use a self-hosted version of DuckDB, instead of loading it from jsdeliver

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I see in the console when I switch to import { SimpleWebDB } from "jsr:@nshiab/simple-data-analysis@3.15.3/web";, it's a problem with the library. I'll investigate and come back to you!

Comment thread src/party/sda.md Outdated
Comment thread src/party/sda.md
Comment on lines +21 to 23
await tableTemperature.fetchData(
"https://raw.githubusercontent.com/nshiab/simple-data-analysis/main/test/data/files/dailyTemperatures.csv",
);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note to self: host this file in /data/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants