view doc/v2_planning/arch_src/plugin_JB_comments_RP.txt @ 1526:5804e44d7a1b

pep8
author Frederic Bastien <nouiz@nouiz.org>
date Fri, 09 Nov 2012 16:58:17 -0500
parents 699ed5f5f188
children
line wrap: on
line source

I agree with Ian, maybe using caps is not the best idea. It reminds be of BASIC which I used to do long time ago :). It also makes the code look a bit scary.

JB replies: personally i think it makes the code look more AWESOME but I could
go either way.  See reply to Ian in plugin_JB_comments_IG.txt

I like the approach and I think it goes close to my earliest proposition and to what I am proposing for the layer committeee ( though we did not have a meeting yet). 
I would though write it in a more Theano like ( Ian has a example of how that would look). I would also drop the CALL and FILT constructs, and actually have a 
decorator ( or something ) that wraps around a function to transform it into a call or filt. I hope that this is only syntactic sugar ( does this change anything
in the actual implementation ?? ) that makes things more natural. What I want to reach is something that looks very much as Theano, just that now you are creating
the graph of execution steps. Refractoring what you wrote this will look like

x = buffer_repeat( 1000, dataset.next())
train_pca = pca.analyze(x)

train_pca.run()

If you allow a FILT to also get multiple inputs ( so not just the one) which comes natural in this way of writing you can get to describe a DAG that not only 
describes the order of execution but also deals with what takes data from what. I'm sorry for not being there yesturday, from what I remember I have the 
feeling that for you that is done under the hood and not taken care by this flow control structures. 

To be a bit more explicit, in the way of writing the code above you can see that :
  a) dataset_next() has to run before pca_analyze
  b) pca_analyze needs the result (data) object of buffer_repeat( dataset.next()) 

I've actually elaborated on this idea here and there, and figured out what the result from such a control flow thing is, and how to make everything explicit 
in the graph. Parts of this is in my plugin_RP.py ( Step 1) though it is a bit of a moving target. I also have a sligtly different way of writing REPEAT 
and BUFFER_REPEAT .. though I think is mostly the same. I actually did not know how to deal with distributed things until I saw how you deal with that in your code.
Copy-pasted a version of a SDAA with my way of writing : 

    ## Layer 1:

    data_x,data_y = GPU_transform(load_mnist())
    noisy_data_x  = gaussian_noise(data_x, amount = 0.1)
    hidden1       = tanh(dotW_b(data_x, n_units = 200))
    reconstruct1  = reconstruct(hidden1.replace(data_x, noisy_data_x),
                            noisy_data_x)
    err1          = cross_entropy(reconstruct1, data_x)
    learner1      = SGD(err1)

    # Layer 2 :
    noisy_hidden1 = gaussian_noise(hidden1, amount = 0.1)
    hidden2       = tanh(dotW_b(hidden1, n_units = 200))
    reconstruct2  = reconstruct(hidden2.replace(hidden1,noisy_hidden1),
                            noisy_hidden1)
    err2          = cross_entropy(reconstruct2, hidden)
    learner2      = SGD(err2)

    # Top layer:

    output  = sigmoid(dotW_b(hidden2, n_units = 10))
    err     = cross_entropy(output, data_y)
    learner = SGD(err)


GPU_transform,gaussian_noise and so on are functions that have been decorated ( or classes if you want) 
that you would write using FILT.  Reconstruct for me is a different CONTROL FLOW element. 
In this case I don't use REPEAT or BUFFER_REPEAT or the other very cool control flow elements, but you
can easily imagine writing something like

pretrained_in_parallel = weave( learner1, learner2)
results = spawn(repeat(5000,learner1),repeat(500,learner2))


JB replies:

  This reply makes it clearer to me that I was not sensitive enough to the
  difference between *expressions* and *control-flow statements*.  What you have
  above is a graph of declarative expressions (if I understand correctly) with
  certain properties:

    - they have no side effects
    - they can be re-ordered within dependency constraints

  Contrast this with the CALL statements in my proposal:

    - they work primarily by side effect
    - they cannot be re-ordered at all

  So the fact that CALL currently works by side effect means that there is
  almost no graph-manipulation that can be guaranteed not to change the program.
  This is a reason to make CALL statements *encapsulate* programs constructed
  using declarative constructs (i.e. Theano functions)

  In other words, in this short term, this feels to me like the reason to *not*
  mix Theano graph building with this control-flow business.  

  Consequently, I think I will remove the BUFFER_REPEAT construct since that is
  really an expression masquerading as a control flow statement, and I will
  remove FILT too.

RP asks:

  I understand now the difference between what you wrote and what I had in
  mind. Though I don't undestand the argument against it. Do you mean to say
  that writing it the way I proposed implies a much more complicated backbone
  framework which will take us to long to develop? Or is there something else
  that you meant ?


JB replies:
  
  I don't think it's necessary to combine theano with this control-flow
  proposal, and I don't know how to do it.  Yes, it seems like it would be hard
  and/or awkward, and I don't even really see the advantage of even trying to do
  it.

RP:

  I think you misunderstood me. I did not propose to mix Theano with the
  library. I agree that would be awkward. What I had in mind ( which might be
  just something different from what you are doing) is to use some concepts
  from how Theano deals with things. 
  
  For example right now you added registers. Writing something like:

  CALL( fn, arg1, arg2, arg3, _set= reg('x') )

  means actually  reg('x') = fn (arg1,arg2,arg3)

  You get most of what you want because this control flow elements don't
  actually get executed until you run the program. That means that you 
  have a fixed simple graph ( you can't play around with it) that tells
  how to execute your commands. You can save that graph, and the point in 
  the graph where you stop so that you can resume latter. You can also 
  save all registers at that point. 


  Why not have that fn instead of being a python function, be some class
  that implements a method run which does what your call would do. 
  The init/ or __call__ of that class would do what CALL does in your case.
  Do you think that would be impossible to implement? Any such function 
  could either return a set of registers or not. 

  Your other control flow things will be just special such functions. 
  The only thing that would might look a bit strange would be the sequence
  in case you need to return things. Maybe I could use the same trick,
  namely a _set arguemnt to __call__.

  I'm not against your approach, I just think it can be written a bit 
  differently, which in my opinion is easier to read, understand and so 
  on. I will have nothing against if we decide to write it exactly how you
  propose and I'm sure that I will get the hang of it pretty fast.


  Bottom line (in my view):
    - I don't say we should mix Theano with anything
    - I think writing things such that it looks like applying functions to 
    object is a more natural way, easy to understand for noobs
    - Writing a new functions by inheriting a class and implementing a method
    is also natural
    - I do not propose to do optimizations or play with the graph ! I do
      though think that you should be able to : 
        * replace parts of a subgraph with a different
        * automatically collect hyper-parameters or parameters if you ever want
        to
        * change the value of these somehow