Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Next WG meeting - January 2017 #252

Closed
mcollina opened this issue Dec 23, 2016 · 28 comments
Closed

Next WG meeting - January 2017 #252

mcollina opened this issue Dec 23, 2016 · 28 comments

Comments

@mcollina
Copy link
Member

mcollina commented Dec 23, 2016

How about a Streams WG meeting on the week from the 9th to the 12th?

Agenda:

  1. replace bind() #253, usage of bind(), and performance across versions of Node and the browser
  2. Performance degradation when using many stream transforms #249, performance degradation using multiple transforms
  3. Support Symbol.asyncIterator #254, Symbol.asyncIterator

Link to join: https://hangouts.google.com/call/2ko57nnswbf6tlmhcksoc5lrz4e
Link to watch: https://www.youtube.com/watch?v=lthLOvt4QV8

@calvinmetcalf
Copy link
Contributor

Works for me

@yoshuawuyts
Copy link

yoshuawuyts commented Dec 23, 2016 via email

@benjamingr
Copy link
Member

I'd like to attend and discuss adding support for Symbol.asyncIterator as suggested by @mcollina in nodejs/CTC#53. I'll also open a separate issue.

@mcollina
Copy link
Member Author

mcollina commented Jan 2, 2017

@mcollina
Copy link
Member Author

mcollina commented Jan 9, 2017

@benjamingr @mafintosh @calvinmetcalf would you mind filling in the doodle? Today is probably out of the equation by now :D.

@mcollina
Copy link
Member Author

mcollina commented Jan 9, 2017

@benjamingr is it too early for you? Not sure in which timezone you are.

@benjamingr
Copy link
Member

It's 4pm, 5pm and 6pm for me - I'm in GMT + 2. I can make 6pm work by leaving work a little early which is fine.,

@mcollina
Copy link
Member Author

mcollina commented Jan 9, 2017

All times displayed are CET time. Sigh, I set up the doodle wrong.

Here is the correct new doodle: http://doodle.com/poll/wgbmbcwnyiw7g7nz

@benjamingr
Copy link
Member

Oh, I thought that the doodle automatically aligns to my timezone. I'll update in the new one.

@mcollina
Copy link
Member Author

At this point the two most probable dates are:

  • Thursday 12th of January at 19:00 CET
  • Friday 13th of January at 18:00 CET

Anyone else that want to partecipate please fill in the doodle.
I'm planning to do Thursday 12th of January, 19:00 CET/ 18:00 GMT, if you all agree.

@calvinmetcalf
Copy link
Contributor

calvinmetcalf commented Jan 10, 2017 via email

@lrlna
Copy link
Contributor

lrlna commented Jan 10, 2017

oh hey i am gonna jump in on this one too; thanks for the ping @mcollina

@mcollina
Copy link
Member Author

Confirmed for Thursday 12th of January, 19:00 CET/ 18:00 GMT, as stated in the doodle.
I will post a link.

Can we all compose the agenda, we probably have a few things to talk about.

@calvinmetcalf
Copy link
Contributor

@mcollina
Copy link
Member Author

@calvinmetcalf yes. Is this still good for you?

@calvinmetcalf
Copy link
Contributor

calvinmetcalf commented Jan 11, 2017 via email

@williamkapke
Copy link

@mcollina
Copy link
Member Author

@williamkapke should we ping you to add it next time?

@williamkapke
Copy link

I gave you permission to manage the calendar too- but feel free to ping me.

@mcollina
Copy link
Member Author

I've added the links to join or watch at the top.

@mcollina
Copy link
Member Author

I would like to talk about my EPS draft in https://gist.github.com/mcollina/ca17ee3de226d8dff6281683ca8c9d24 for the inversion of dependency.

@yoshuawuyts
Copy link

yoshuawuyts commented Jan 12, 2017 via email

@mcollina
Copy link
Member Author

@mcollina
Copy link
Member Author

@lrlna @benjamingr @mafintosh are you folks joining?

@mafintosh
Copy link
Member

cool meeting! (i joined for the record)

@benjamingr
Copy link
Member

benjamingr commented Jan 12, 2017 via email

@mcollina
Copy link
Member Author

mcollina commented Jan 13, 2017

I'm not really proud of these ones, but here are my notes:

2017 jan 12 - stream wg 1

2017 jan 12 - stream wg 2

For a inlined summary:

  1. we will be replacing bind() for the time being, as it is extremely slow on the old versions of Node. As for the browser, it is less of a problem.

  2. @mcollina to investigate if some of the ideas in https://github.com/mcollina/syncthrough can be ported to Transform, for the benefits of everyone.

  3. as for the inversion of the readable-state dependency, some edits where recommended: regarding adding a timeline, and porting the tests so that they can be easily run against the node source, old node builds and the browsers. See https://gist.github.com/mcollina/ca17ee3de226d8dff6281683ca8c9d24

  4. Symbol.asyncIterator can be supported inside streams without internal changes, however merging it inside core depends on what is the adopted change for Promises in core. We are happy to maintain the prototype and expand if needed. We should take care on what is the story for writing to streams, as it should match somehow the story for reading, i.e. we should be able to write with promises as well.

@lrlna
Copy link
Contributor

lrlna commented Jan 13, 2017

@mcollina dope sketchnotes!! 🖋 📝

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

No branches or pull requests

7 participants