Project

General

Profile

Actions

Feature #4326

closed

Fiber should respond to call() and []

Added by tenderlovemaking (Aaron Patterson) almost 14 years ago. Updated over 12 years ago.

Status:
Closed
Target version:
-
[ruby-core:34861]

Description

=begin
Fibers are considered to be coroutines. Knuth says "Subroutines are special cases of ... coroutines". This makes sense to me.

Method, Proc, and lambda respond to call and []. If Fiber also responded to call and [], we could use coroutines in places where we use lambdas, procs, and methods.

I've attached a patch that implements the two methods.
=end


Files

fiber.patch (997 Bytes) fiber.patch tenderlovemaking (Aaron Patterson), 01/26/2011 04:08 PM
fiber.patch (853 Bytes) fiber.patch tenderlovemaking (Aaron Patterson), 01/31/2011 08:42 AM
Actions #1

Updated by kstephens (Kurt Stephens) almost 14 years ago

=begin
Shouldn't Fiber#[] behave like Thread#[]? What about Continuation#[]?

=end

Actions #2

Updated by nobu (Nobuyoshi Nakada) almost 14 years ago

=begin
Hi,

At Wed, 26 Jan 2011 16:08:52 +0900,
Aaron Patterson wrote in [ruby-core:34861]:

Fibers are considered to be coroutines. Knuth says
"Subroutines are special cases of ... coroutines". This
makes sense to me.

Do you mean
"A is kind of B, A has a feature X, therefore B also should
have X",
but not
"A is kind of B, B has a feature X, therefore A also should
have X"?

Sounds strange to me.

--
Nobu Nakada

=end

Actions #3

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Thu, Jan 27, 2011 at 01:07:33PM +0900, Nobuyoshi Nakada wrote:

Hi,

At Wed, 26 Jan 2011 16:08:52 +0900,
Aaron Patterson wrote in [ruby-core:34861]:

Fibers are considered to be coroutines. Knuth says
"Subroutines are special cases of ... coroutines". This
makes sense to me.

Do you mean
"A is kind of B, A has a feature X, therefore B also should
have X",
but not
"A is kind of B, B has a feature X, therefore A also should
have X"?

Sounds strange to me.

Maybe code will be more clear. I want to do this:

 def square block
   block.call ** 2
 end
 
 callables = [
   lambda { 10 },
   Fiber.new { Fiber.yield 10 }
 ]
 
 callables.map { |thing| square thing }

Instead I have to do this:

 def square block
   value = block.is_a?(Fiber) ? block.resume : block.call
   value ** 2
 end
 
 callables = [
   lambda { 10 },
   Fiber.new { Fiber.yield 10 }
 ]
 
 callables.map { |thing| square thing }

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #4

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Thu, Jan 27, 2011 at 02:14:16PM -0800, Aaron Patterson wrote:

On Thu, Jan 27, 2011 at 01:07:33PM +0900, Nobuyoshi Nakada wrote:

Hi,

At Wed, 26 Jan 2011 16:08:52 +0900,
Aaron Patterson wrote in [ruby-core:34861]:

Fibers are considered to be coroutines. Knuth says
"Subroutines are special cases of ... coroutines". This
makes sense to me.

Do you mean
"A is kind of B, A has a feature X, therefore B also should
have X",
but not
"A is kind of B, B has a feature X, therefore A also should
have X"?

Sounds strange to me.

Maybe code will be more clear. I want to do this:

def square block
  block.call ** 2
end

callables = [
  lambda { 10 },
  Fiber.new { Fiber.yield 10 }
]

callables.map { |thing| square thing }

Instead I have to do this:

def square block
  value = block.is_a?(Fiber) ? block.resume : block.call
  value ** 2
end

callables = [
  lambda { 10 },
  Fiber.new { Fiber.yield 10 }
]

callables.map { |thing| square thing }

The way I think about it is:

A subroutine is a coroutine, but in ruby I cannot use a coroutine in
the place of a subroutine.

Am I thinking about this wrong?

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #5

Updated by runpaint (Run Paint Run Run) almost 14 years ago

=begin
I understood what you meant, and agree in principle. I wanted to think about the implications given that (a) #call is duck-typed on, and (b) Fibers cannot be called across Threads, and, possibly, (c) the root Fiber has special semantics but no predicate for identifying them.
=end

Actions #6

Updated by usa (Usaku NAKAMURA) almost 14 years ago

  • Status changed from Open to Assigned

=begin

=end

Actions #7

Updated by headius (Charles Nutter) almost 14 years ago

=begin
RPRR and Nobu make good points.

Fibers are coroutines...agreed.

If we go by Knuth, a subroutine "is a" coroutine, but you're thinking about the relationship wrong. A Hash "is a" Object. We don't expect Object (coroutine) to do what Hash (subroutine) can do and you cannot use an Object (coroutine) in the place of a Hash (subroutine). Simply put, Knuth's statement doesn't mean Fiber should be usable everywhere a subroutine (Proc) can be used.

RPRR's points about threading and root fibers are also important, since they explicitly mean you can't use a Fiber in place of a thread-agnostic subroutine.

In practice I don't really care about adding [] and .call to Fiber, but they do mask the fact that these are not simply "resumable procs".
=end

Actions #8

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Sat, Jan 29, 2011 at 02:58:46AM +0900, Charles Nutter wrote:

Issue #4326 has been updated by Charles Nutter.

RPRR and Nobu make good points.

Fibers are coroutines...agreed.

If we go by Knuth, a subroutine "is a" coroutine, but you're thinking about the relationship wrong. A Hash "is a" Object. We don't expect Object (coroutine) to do what Hash (subroutine) can do and you cannot use an Object (coroutine) in the place of a Hash (subroutine). Simply put, Knuth's statement doesn't mean Fiber should be usable everywhere a subroutine (Proc) can be used.

My point is that it seems that LSP is broken. Either Fiber should
respond to "call" and "[]", or Proc should respond to "resume". Really
I don't care which.

RPRR's points about threading and root fibers are also important, since they explicitly mean you can't use a Fiber in place of a thread-agnostic subroutine.

I agree these are problems. But I think fixing those issues is
different than fixing LSP problems.

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #9

Updated by headius (Charles Nutter) almost 14 years ago

=begin
On Fri, Jan 28, 2011 at 5:29 PM, Aaron Patterson
wrote:

My point is that it seems that LSP is broken.  Either Fiber should
respond to "call" and "[]", or Proc should respond to "resume".  Really
I don't care which.

LSP says the opposite. Assuming (from your interpretation of Knuth and
LSP) that coroutines (Fibers) are a generalization of subroutines
(procs), and conversely subroutines (procs) are a specialization of
coroutines (Fibers), then T = coroutines (Fibers), S = subroutines
(procs) and...

"Let q(x) be a property provable about objects x of type T. Then q(y)
should be true for objects y of type S where S is a subtype of T."

So subroutines (procs) should do everything Fibers can do, but not the
inverse relationship. This would also imply that subroutines should be
resumable, etc.

However...I'm not sure what Knuth was saying is implying a
generalization/specialization relationship. He was saying subroutines
⊂ coroutines. So you should be able to expect that for all operations
q on coroutines there should be a matching operation q on subroutines
and vice versa. Doesn't this mean we should also have resuming,
yielding, etc on subroutines? (please no)

So...there are three possibilities:

  1. subroutines are a specialization of coroutines. Then it's perfectly
    valid for subroutines to define things coroutines do not.
  2. subroutines are equivalent to coroutines. Then coroutines should
    add [] and call and subroutines should have resuming/yielding
    capabilities.
  3. coroutines are a specialization of subroutines. Then coroutines
    should add [] and call, and we're done. And Knuth is wrong.

Now that I've thoroughly confused myself, I'll leave it up to you and
ruby-core... which reality would you like?

  • Charlie

=end

Actions #10

Updated by headius (Charles Nutter) almost 14 years ago

=begin
On Fri, Jan 28, 2011 at 6:45 PM, Charles Oliver Nutter
wrote:

LSP says the opposite. Assuming (from your interpretation of Knuth and
LSP) that coroutines (Fibers) are a generalization of subroutines
(procs), and conversely subroutines (procs) are a specialization of
coroutines (Fibers), then T = coroutines (Fibers), S = subroutines
(procs) and...

BTW, I'd love to do an audit of Ruby behaviors some time and see how
frequently LSP is broken. I'd also love to hear why Liskov would say
about IO#reopen changing the object's type completely. :)

  • Charlie

=end

Actions #11

Updated by mame (Yusuke Endoh) almost 14 years ago

=begin
Hi,

2011/1/27 Kurt Stephens :

Shouldn't Fiber#[] behave like Thread#[]?

I think more notice should be taken of Kurt's remark :-)

--
Yusuke Endoh

=end

Actions #12

Updated by headius (Charles Nutter) almost 14 years ago

=begin
On Fri, Jan 28, 2011 at 7:15 PM, Yusuke ENDOH wrote:

Hi,

2011/1/27 Kurt  Stephens :

Shouldn't Fiber#[] behave like Thread#[]?

I think more notice should be taken of Kurt's remark :-)

Indeed. When coroutines were built for OpenJDK (in the MLVM project)
the author quickly realized the need for coroutine-locals as well as
thread-locals.

I tend to think of fibers more like threads or actors than
subroutines, but of course I'm biased because on JRuby they are
threads.

  • Charlie

=end

Actions #13

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Sat, Jan 29, 2011 at 09:45:30AM +0900, Charles Oliver Nutter wrote:

On Fri, Jan 28, 2011 at 5:29 PM, Aaron Patterson
wrote:

My point is that it seems that LSP is broken.  Either Fiber should
respond to "call" and "[]", or Proc should respond to "resume".  Really
I don't care which.

LSP says the opposite. Assuming (from your interpretation of Knuth and
LSP) that coroutines (Fibers) are a generalization of subroutines
(procs), and conversely subroutines (procs) are a specialization of
coroutines (Fibers), then T = coroutines (Fibers), S = subroutines
(procs) and...

"Let q(x) be a property provable about objects x of type T. Then q(y)
should be true for objects y of type S where S is a subtype of T."

So subroutines (procs) should do everything Fibers can do, but not the
inverse relationship. This would also imply that subroutines should be
resumable, etc.

However...I'm not sure what Knuth was saying is implying a
generalization/specialization relationship. He was saying subroutines
⊂ coroutines. So you should be able to expect that for all operations
q on coroutines there should be a matching operation q on subroutines
and vice versa. Doesn't this mean we should also have resuming,
yielding, etc on subroutines? (please no)

So...there are three possibilities:

  1. subroutines are a specialization of coroutines. Then it's perfectly
    valid for subroutines to define things coroutines do not.

Yes, but it is not perfectly valid for subroutines to not define
things that coroutines do. We can either resolve that by defining
"resume" on Proc, or "call" and "[]" on Fiber.

  1. subroutines are equivalent to coroutines. Then coroutines should
    add [] and call and subroutines should have resuming/yielding
    capabilities.
  2. coroutines are a specialization of subroutines. Then coroutines
    should add [] and call, and we're done. And Knuth is wrong.

Now that I've thoroughly confused myself, I'll leave it up to you and
ruby-core... which reality would you like?

I have been trying to choose reality number 1, the entire time, but
apparently I cannot explain myself clearly. :'(

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #14

Updated by headius (Charles Nutter) almost 14 years ago

=begin
On Sat, Jan 29, 2011 at 11:41 AM, Aaron Patterson
wrote:

  1. subroutines are a specialization of coroutines. Then it's perfectly
    valid for subroutines to define things coroutines do not.

Yes, but it is not perfectly valid for subroutines to not define
things that coroutines do.  We can either resolve that by defining
"resume" on Proc, or "call" and "[]" on Fiber.
...
I have been trying to choose reality number 1, the entire time, but
apparently I cannot explain myself clearly. :'(

But doesn't reality 1 mean that subroutines should get resume/etc and
coroutines should not get call/[]? It seems to me you either want my
reality 3 or Jim's reality 4.

  • Charlie

=end

Actions #15

Updated by headius (Charles Nutter) almost 14 years ago

=begin
On Fri, Jan 28, 2011 at 9:28 PM, Jim Weirich wrote:

I wouldn't say Knuth is wrong, but a natural language "is-a" is not always
an indication of subtype (c.f. the square/rectangle paradox where a square
is-a rectangle, but having square inherit separate set_width/set_height
methods from rectangle would be wrong).

Right...so my reality 2 and your reality 4 are pretty close; mine says
that coroutines and subroutines are the same thing, but subroutines
are a specialized subset. Yours says that coroutines and subroutines
are both of the same (super)type, and so they should share in common
operations from that (super)type.

In any case, I should restate again that I really don't care if Ruby
adds call and [] on Fiber :) I just want to make sure it's not being
done for a logically inconsistent reason.

  • Charlie

=end

Actions #16

Updated by rkh (Konstantin Haase) almost 14 years ago

=begin
This might go in a slightly different direction, but I would really love Fiber to define #to_proc allowing things like [1, 2, 3].each(&fiber).

Also note that in ANSI smalltalk (which has a rather common object model to Ruby) a block closure is implementing the valuable protocol (objects that respond to #value and akin, smalltalk's #call), which would be something like the common ancestor for procs and fibers.

Konstantin

=end

Actions #17

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Sun, Jan 30, 2011 at 01:47:06PM +0900, Charles Oliver Nutter wrote:

On Sat, Jan 29, 2011 at 11:41 AM, Aaron Patterson
wrote:

  1. subroutines are a specialization of coroutines. Then it's perfectly
    valid for subroutines to define things coroutines do not.

Yes, but it is not perfectly valid for subroutines to not define
things that coroutines do.  We can either resolve that by defining
"resume" on Proc, or "call" and "[]" on Fiber.
...
I have been trying to choose reality number 1, the entire time, but
apparently I cannot explain myself clearly. :'(

But doesn't reality 1 mean that subroutines should get resume/etc and
coroutines should not get call/[]? It seems to me you either want my
reality 3 or Jim's reality 4.

I don't know anymore. I guess I don't care anymore either.

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #18

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
On Sat, Jan 29, 2011 at 10:15:31AM +0900, Yusuke ENDOH wrote:

Hi,

2011/1/27 Kurt Stephens :

Shouldn't Fiber#[] behave like Thread#[]?

I think more notice should be taken of Kurt's remark :-)

Even if Fiber only implemented call, it would make the code in
[ruby-core:34909] much better.

Maybe I will change my patch to not implement [], and we don't have to
worry about the semantics of Fiber#[] for this ticket.

--
Aaron Patterson
http://tenderlovemaking.com/

Attachment: (unnamed)
=end

Actions #19

Updated by tenderlovemaking (Aaron Patterson) almost 14 years ago

=begin
New patch without Fiber#[]
=end

Updated by ko1 (Koichi Sasada) over 13 years ago

Hi,

Sorry for late response.

(2011/01/26 16:08), Aaron Patterson wrote:

Feature #4326: Fiber should respond to call() and []
http://redmine.ruby-lang.org/issues/show/4326

Author: Aaron Patterson
Status: Open, Priority: Normal
Assigned to: Koichi Sasada

Fibers are considered to be coroutines. Knuth says "Subroutines are special cases of ... coroutines". This makes sense to me.

Method, Proc, and lambda respond to call and []. If Fiber also responded to call and [], we could use coroutines in places where we use lambdas, procs, and methods.

I've attached a patch that implements the two methods.

I don't understand all of this thread. However, as I said to aaron at
IRC, I want to reject this proposal.

Reasons:
(1) Proc is restartable. It can be called infinite times. In general,
Fiber is not for such usage.

(2) If we permit Fiber#call, maybe other guys say "should support
Fiber#[]". However, as you mention, Fiber#[] is ambiguous with Fiber
local storage.

--
// SASADA Koichi at atdot dot net

Updated by ko1 (Koichi Sasada) over 12 years ago

  • Description updated (diff)
  • Status changed from Assigned to Closed

I close this ticket.

Please re-open it if anyone have any comment which we need to discuss again.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0