Skip to content

ARROW-2777: [JS] Friendlier onboarding readme#2200

Closed
lmeyerov wants to merge 2 commits intoapache:masterfrom
lmeyerov:patch-3
Closed

ARROW-2777: [JS] Friendlier onboarding readme#2200
lmeyerov wants to merge 2 commits intoapache:masterfrom
lmeyerov:patch-3

Conversation

@lmeyerov
Copy link
Copy Markdown
Contributor

@lmeyerov lmeyerov commented Jul 2, 2018

-- Link slack, tutorials, unit tests
-- Add example of creating a Table from pure JS

-- Link slack,  tutorials, unit tests
-- Add example of creating a Table from pure JS
js/README.md Outdated
Even if you do not plan to contribute to Apache Arrow itself or Arrow
integrations in other projects, we'd be happy to have you involved:

* [Join the Slack channel][5]: Channels `#general` and `#javascript`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See recent mailing list thread about the Slack channel -- it's likely going to be closed soon

@trxcllnt trxcllnt changed the title [ARROW-2777][JS] - Friendlier onboarding readme [ARROW-2777]: [JS] Friendlier onboarding readme Jul 2, 2018
@pitrou pitrou changed the title [ARROW-2777]: [JS] Friendlier onboarding readme ARROW-2777: [JS] Friendlier onboarding readme Jul 2, 2018
@lmeyerov
Copy link
Copy Markdown
Contributor Author

Great just removed official Slack.

We're still using for interactive JS dev and probably intro stuff, but moved out of official. @trxcllnt lmk if there's an alternate link.

@wesm
Copy link
Copy Markdown
Member

wesm commented Jul 19, 2018

See how Apache Spark has a link to an unofficial chat room:

http://spark.apache.org/community.html

I just opened https://issues.apache.org/jira/browse/ARROW-2881; the Arrow website doesn't do a good job yet explaining the different community channels

{name: "precipitation", count: LENGTH, VALIDITY: [], DATA: rainAmounts },
{name: "date", count: LENGTH, VALIDITY: [], DATA: rainDates } ] }] })
```

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nitpicky, but this won't actually work when copy-pasted because LENGTH is defined after the arrays.

It would be really nice if we could finish ARROW-2766 to make this example much cleaner. Inferring the types from arrays is difficult though.. so we could do a subset of that functionality that takes a list of already created vectors and a list of names for them. We can almost do this already, I can write up a PR that cleans it up.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, let's get this out and can add more later. Both code, tutorials, and communty links can all be improved :) I'm still trying to figure out how to make a graphistry.slack.com #arrow signup page link...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sounds good - I can update the docs when I put that PR up

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See #2322

Any objections to going ahead and merging this PR?

Copy link
Copy Markdown
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

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

+1. Merging this

@wesm wesm closed this in 16bbdd4 Jul 25, 2018
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.

3 participants