Skip to content

Updating sda page#4

Open
nshiab wants to merge 3 commits intoFil:mainfrom
nshiab:main
Open

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

Conversation

@nshiab
Copy link

@nshiab nshiab commented Oct 31, 2024

Updating sda to v3.

Thanks!

@vercel
Copy link

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.


```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
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
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
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 on lines +21 to 23
await tableTemperature.fetchData(
"https://raw.githubusercontent.com/nshiab/simple-data-analysis/main/test/data/files/dailyTemperatures.csv",
);
Copy link
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