labofsahil/github-event-dataset-2012 huggingface dataset:
The file is on the main repo and is useless. Shouldn't we delete it?

On Wed, Sep 26, 2012 at 12:20 AM, Zaiste! <notifications@github.com> wrote:

> In .sparkleshare:
>
> > @@ -1 +0,0 @@
> > -69ce402f86963a6128becc68163ec1fdce61083e
>
> @albanlv <https://github.com/albanlv> please, remove this file from the
> pull request
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nukomeet/coworfing/pull/192/files#r1690397>.
>
>
I don't know why this changed. I guess line endings?
Ha @vanm - I was just typing something about this line, and your comment popped in right above this.
```python
in_sure = (Event.objects.filter(slug=slug).exclude(id=self.instance.id or ...
```
I would filter this in python. if `query` is `"a"` it might find "Abraham Lincoln" and "Thomas Jefferson" and "John Adams" with `icontains` but I'd only expect to see "Abraham Lincoln" and "John Adams" since they start with an `"a"`. 
So something like this:

```
regex = re.compile(r'\b%s' % re.espace(query.split()[0]), re.I)
for p in participants:
    if regex.findall(p.name):
        results.append({'id': p.name, 'text': p.name})
```
or something like that.
This looks a little ugly. Why not just ```articles.push(referURL(i));```?
This is nice but I think this could be dropped. No need to be backwards compatible with my original patch IMO. :) It was never part of the official codebase and those who use my patch can easily change their advancedsettings.
Looks like your right on this one. My bad.
I'm not sure where it is used I'm just duplicating the datastructure:
https://github.com/Islandora/islandora/blob/6.x/api/fedora_item.inc#L853
You can/should get rid of all these placeholder comments
You should move the T over a little bit so it doesn't get cut off when you click on the text box? Not really sure what's causing that, but I think that's a quick fix.
need to add `++i`
You are putting the last played and last requested before the trackid, changing this around is better for reading and easier to get used to since it is the current format but with just the two appended, instead of inserted.

The formatting also has no irc colours inserted before the Last Played and Last Requested part.
I can create a bang version of this method but normally I favor an exception when you are attempting to work with a piece of data that is not existing. In this case you are attempting to open a file that does not exist.
use eq_()
надо бы на несколько функций разбить это чудо
lets keep the old check on line 382 rather than this
`tree[n]?` is a nicer way of writing `tree[n] isn't undefined`
This is missing the `-G`...not sure it'll work otherwise.
I'm afraid I don't quite agree here. _e-mailadres_ is a compound, and according to official spelling rules compounds should hence be [written as a single word][1]. ([Van Dale][2] uses the middle dot '·' to separate syllables, compare [Onderwerp][3], and [Het Groene Boekje][4]). [This article][5] has some background on the issue.

[1]: http://woordenlijst.org/leidraad/6/2/ "Compounds, Official Word List of the Dutch language (Het Groene Boekje), Dutch Language Union"
[2]: http://www.vandale.nl/opzoeken?pattern=Emailadres&lang=nn "E-mailadres, Van Dale Dictionary"
[3]: http://www.vandale.nl/opzoeken?pattern=onderwerp&lang=nn "Onderwerp, Van Dale Dictionary"
[4]: http://woordenlijst.org/zoek/?q=e-mailadres&w=w "E-mailadres, Official Word List of the Dutch language (Het Groene Boekje), Dutch Language Union"
[5]: http://www.onzetaal.nl/taaladvies/advies/e-mailadres-emailadres "E-mailadres / emailadres, language advisory, Society for our Language / Genootschap Onze Taal"
These dependencies should be re-arranged.
1. All 3rd party components (underscore, fs, etc.)
2. All OAE dependencies from other modules (oae-util, etc.)
3. All OAE dependencies from the current module
What I'm saying is that the contention for our other CFs, even the row keys would be `tenantId`. So should the schema change to be the same?
Right, a check for (someConfigValue === null) would probably be better.
Not sure what this does.
Nice! Thanks Luc for fixing my tests ;-)
just updated the RouteRepositoryInterface to allow it to throw a more specific exception if it wants to.
a moze short tag?
not necessarily explicitly, for example if Accept contained "text/*" it'll still match, but if there is not a match (excluding */*) it'll return undefined. At least that's how it is right now, but I dont think that's necessarily "correct". We're probably better off to return the first value in that case
I'm agree :)
I don't have a better suggestion off the top of my head.
Действительно. Позже исправлю. Спасибо.
Я не тратил время на дополнительные вещи.
Это, насколько я понимаю, автоматически создаваемый файл, когда в Xcode просишь создать новую категорию.
Good catch. I missed that one.
Technically the variable i is declared 10 lines before. It was flagged by jshint as a duplicate declaration. 

I've used it without var in production for a while. But obviously a second declaration doesn't break functionality, since everyone has been using it.
Should probably use `urllib.urlencode(params)`?
Shouldn't we also be setting `RACSchedulerCurrentSchedulerKey` before invoking this block?
I think we might want to get rid of this code as well. Even though it makes complete sense to create a 'transaction' around any P4 commands by ensuring we are logged in / ensure logout, it ends up being a pain in certain conditions at Riot.
No, It's extending the right class, which is Redis. Extending Redisent doesn't make sense, since the point is enabling the use of phpRedis api. 
Did you have trouble running the plugin that requires extending Redisent ?
addservice.pb.go added to git.

test.sh does indeed generate addservice.pb.go.

Done.
I would use mean_flip as default.
Hi Ben,
The StringUtils u mentioned was just an unused import which probably
creeped in while I was playing around with the code, apologies!
I'm fixing the Provider.toString() now.
Thanks for the review.

On Wed, Sep 26, 2012 at 8:42 PM, Ben Wolfe <notifications@github.com> wrote:

> In api/src/main/java/org/openmrs/Provider.java:
>
> > @@ -106,7 +106,7 @@ public String getIdentifier() {
> >  	@Override
> >  	public String toString() {
>
> This toString needs to mimic what you just did in PersonName.toString().
> This should be a machine readable string that is helpful in debugging, not
> somethign the end user sees. This method should return toStrings from
> person name
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openmrs/openmrs-core/pull/37/files#r1696440>.
>
>



-- 
Regards,
Isha Tripathi
Application Developer
[image: Home] <http://www.thoughtworks.com/>
The "email" key may not always be present in the EJSON. I think we need to have a defualt value of the atom 'null' for the right thing to happen if it is missing.
I suppose you'll have to change this to a Future now. 
I am not sure myself if an IndexSeq is the right datatype, btw. I think in Scala it's nicer to work with that Arrays ( but not sure yet about 2.10 )
This URL will need to be updated
I would break the line and encourage you to submit another PR improving the other code snippets :)
How many? I copied this line from another file.

Is there a way to build / preview the docs?
A code block should be indented 4 spaces, not 2 or 3. I prefer to use the 72 characters break on code blocks to, can you split line 200 into multiple lines?
"Visually communicate"
I didn't think CamelContext was thread-safe?
I might reduce this like follows:

```js
var d3_time_suffixes = ["th", "st", "nd", "rd"];

function d3_time_suffix(number) {
  var tail = number % 100;
  return d3_time_suffixes[tail < 11 || tail > 13 && tail % 10] || d3_time_suffixes[0];
}
```
Почитай же про обработку событий. Если обработал — возвращай true.
А чо геттер с маленькой буквы?
¿Qué cambió acá? ¿Tendremos algún pedo con el encoding?
Si, creo que esto aún no está terminado, no debería entrar en el pull
You don't have to hardcode this, you can set a default value for it like 

(defn create-system
  "Creates a system"
   [name & [{:keys [sockets system-sockets system-arch]}]]
   (navigate :new-system-page)
  (fill-ajax-form {:system-name-text name
                    :system-sockets-text sockets
                   :system-arch-select (or system-arch "x86_64")}
Indentation looks odd here,  (`tab = ...` on the next row is less indented than `try:` here).
Does it work?
Please keep blank line
s/filBeingDownloaded/fileBeingDownloaded/
I'm used to do it since all the 'import' words are alligned and seems cleaner to me, but it's a personal oppinion. Undone.
Version constraint should probably be changed to `1.0.*`.
Can u remove thei print and the comment here?
If you're calculating the time between two events, shouldn't you be using nanotime instead? There's was a dev thread about this...
Instead, you could make the keys symbols in mixpanel.yml. E.g:

    :mixpanel:
      :api_key:    'abc'
      :api_secret: 'def'
This line is greater than 79 characters. Can you wrap it?
please put a whitespace after ther , operator
The events need to be part of the xtag.register..  Like this

```
xtag.register('x-listview', {

events: { etc.. }

});

```
True, I agree, but I don't know the history of this code :)
Overall, this patch looks good. Did you change anything here or is this just stray whitespace?
@arnodeclercq Missing space after ]
Can you remove the if true here?
@nicolaasmatthijs Since this is empty, can't we leave it away?
This could be inefficient since it runs once for every row in sparse. Can the loginAdmin() call be moved into the conditional so it only runs for rows that pass the isImage() test?
This method has way too many parameters. Excellent candidate for refactoring at some later date (post-1.2.0).
please just one whitespace line
I'm just thinking, this wrapper is not very extensible (i.e. hardcoded forTemplate). Would be nice to have ability to provide your own wrapper, as we can already configure $this->stat('tree_class') on a per-section basis...
Either A GridField's or just GridField's. I would also use "all depends on" rather than "is all depending of".
should this be unbind or removeClass?
Are you sure ``$context`` has the correct type as ``*`` is not a sub-type of ``resource``? Please also check line 200.
Are you sure that the property "foo" exists?
Is this protocol compliant?  Probably should be logged, don't know if you want to send it.
drop "Good documentation is key" IMO
love how this starts
I didn't have to create it. Mine was there when I updated the master branch. Is it not meant to be?
you should use ``assertInstanceOf`` instead of ``assertTrue``. It gives a better error message when failing than ``Failed asserting that boolean<false> is true``.
At this point isn't this the purpose of `initialize`?

And by convention functions leading with an underscore are for internal use only, so then should `_configure` be changed to not have the _ prefix?

Also at that point aren't we right back to the functionality of `initialize`?
`code` seems like a bad name. How about `name`?
Changed to ``` '' ```.
What exactly? I'm still not sure when to use single/double-quotes in javascript code. It's a bit random currently.
The question is, is it really necessary? In general it is better sometimes to get an error if something is not as expected (like, there is no #map-layers).

But in this case you can also leave to jQuery to simply not do anything, when you want to append to it. And this is it. No need for special check.
No spaces before `:`.
Why don't you simply have two arguments, number of columns, and dict of names mapping to tuple of (column, order)? Now you have names three times, in names, columns keys, orders keys. Error prone if by accident they do not match. It is not smart to define API (even internal one) which requires three different parameters to match like that.
If I use a tuple, I get an error:

TypeError: Error when calling the metaclass bases
    can only concatenate tuple (not "list") to tuple
Replace this and second line. This goes more with the next block of code.
What about other payment methods?
@gibson042 would you stop answering before I ask?! ;) :P

Damn, I feel old and slow.
It could be problematic for some users to hard-code something that assumes http:// and https:// are interchangeable.  The CasProxyCallbackUrl config value might include a non-standard port number for http and/or https.  I think it would be best to leave the value as it is in the config and not adjust its value.  If the site has SSL communication working from the CAS server to the web app, the proxy callback should work fine over HTTPS even if the current request is over HTTP.  If you decide to go this route, I'd recommend removing the StringBuilder and using a different overload on the EnhancedUriBuilder constructor.  You can alter the components on the UriBuilder fairly easily.
did anything actually change with these, other than whitespace?
Oh, so maybe above you're just preserving functionality? If so, do we need to do it in two places still?
Instead of using single quotes, use `<em> </em>`.
straightforward is one word
may be not.. I will another patch. This commit is removed.
typo: is is
When I don't use the absolute path, I get the following error:
sshd re-exec requires execution with an absolute path

Any idea how to approach this?
is it always in /usr/sbin/sshd ? 

you should rather use subprocess here and let the system find the path. it is also not clear to me how you stop it and if a regular user is allowed to run this command.
Nit: Can you fix the indentation here.
I started joining them to an object just to avoid naming them with `resolve2, reject2` :)
Comment added before `this.windows`, line 67
Here, I have two problems/questions:

These functions will be later used through button->onClick.connect method - can it get enum type in the call in WorldView.cpp (e.g. in UpdateCommsOptions, see changes in #1588)?

The other problem: AIFlyTo, AIDock and other functions are used in Ship.h, which doesn't know enum FlightEconomy defined in ShipAICmd.h. In the other side ShipAICmd.h cannot be included into Ship.h, since ShipAICmd.h already includes Ship.h and it would be circular. The only option I can think of is to define FlightEconomy in Ship.h instead of in ShipAICmd.h. What do you think about that?
This is to work around dots not being allowed in variable names in shell scripts, e.g. `date.timezone`. The dot herein is replaced by a double underscore, though not generically correct, it's a combination otherwise not existant in PHP ini settings, so it'll work here. The replacement is reversely applied in line 65 - see below.
It is based on `simplejson`. With some improvements.
Harder, better, faster, stronger. In that order.
Feel free to reverse.
bug here, but i can fix it
minor: 
For consistency, I've been trying to use the set() method whenever more than one property is being set. These two lines would thus become:
ok.set(
  {
    width : 100,
    height : 30
  });

as you did above, with win._reasonField.set().
If that looks all fine I'll merge to master.

On Sun, Nov 18, 2012 at 6:49 PM, steventsim <notifications@github.com>wrote:

> In app/views/dashboard/_campaigns.html.haml:
>
> > \ No newline at end of file
> > +%table#campaigns
> > +  %thead
>
> changed
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/baugarten/CS169-Group-11/pull/24/files#r2162690>.
>
>
In fact why do we need to wrap it at all?  Can replace all 8 lines with:

    Entities.invokeEffectorList(this, removedEntities, Startable.STOP, Collections.<String,Object>emptyMap());

And also in other places I think?
Please keep this module's indentation rules of two spaces per indent.
Si c'est enorme justement, puisque les gens pouvaient donner 0. Ca veut dire que les gens sont prets a payer si on ne les prend pas pour des glands.
I will create alias of it to OS.
foo function isn't defined
Ideally this would be a constant somewhere in the recipe, which we could then use in other parts of this cookbook.
na realidade este trecho poderia ser:

data_queryset = super(AppCustomManager, self).get_query_set()
if availability is not None:
    data_queryset = data_queryset.filter(is_available=avalability)

return data_queryset
This is interesting, but I think it should have a prefix, to avoid collisions. `'widget-p2p-' . $ctype->name . '.php'` maybe? The template names would look like this:

```
widget-p2p-products_to_retailers.php
widget-p2p-posts_to_pages.php
```
This is wrong. What if the user added another route at the top of the file ?
Hey David,

Can you expand on this a little - I don't quite follow why using the create method on the manager is cleaner?  It is not how you would normally create a model instance is it?
yay for finding dead code!
Need a space before the `(` there.
Spaces in the value are not an issue.  Line breaks (and possibly other control characters) would have to be escaped, but given that such characters are not a good idea in tag/branch names, I'll consider it a non-issue.
Fortunately I have a population of 40 and find this very useful.
I admit, on a round restart it's spam, but for the rest it's damn useful.

But okay, will remove.
I'm fine defaulting to bashrc
I think galaxy client should contain stuff that solves the most common problems and as understandably as possible. To me, part of making code easy to understand is not having monolithic classes.
This doesn't change anything.  You probably meant ```merge!``` ?  Or ```opts = opts.merge(:filter => "in_flight")``` is even better IMO
Почему здесь comparator не используется?
Это некорректно. Содержимое файла не должно изменяться при выводе.
Ого!
Это квадратичная сложность. Будет работать очень медленно.
Т е эта функция фактически "бутылочное горло", любая из двух функций format обязательно запускает ее... А как thread-safety по-другому достигается?
We can't/shouldn't do this anymore - see https://github.com/jwhitley/requirejs-rails
Use a org.jboss.spec version, you won't need the dependency then.
I'll delete the XML files..
Can these two external links now be referenced internally?
Maybe the `tocsr()` can be avoided. For instance the downstream model might prefer CSC such as `ElasticNet` for instance.
On Sun, Dec 30, 2012 at 1:42 PM, Vlad Niculae <notifications@github.com>wrote:

> In sklearn/neural_networks/tests/test_rbm.py:
>
> > @@ -0,0 +1,39 @@
> > +import numpy as np
> > +
> > +from numpy.testing import assert_almost_equal, assert_array_equal
> > +
> > +from sklearn.base import clone
> > +from sklearn.datasets import load_digits
> > +from sklearn.neural_networks import RestrictedBolzmannMachine
> > +
> > +np.seterr(all='warn')
> > +
> > +Xdigits = load_digits().data
> > +Xdigits -= Xdigits.min()
> > +Xdigits /= Xdigits.max()
>
> It can be multinomial or Bernoulli, very often binarization helps. For
> multinomial, I would expect rectified linear units to work, judging from
> Hinton's lectures. However the question is whether RBMs would learn
> anything meaningful from this kind of data at all.
>
Regular multinomial units work fine, see e.g. the "replicated softmax"
topic model. You can also do things like "rate-coding" (see Y. W. Teh et al
on "rate-coded RBMs"). All that said, binary units are easy to work with.
I pushed a fix for that example on your branch.

One thing though, I am not sure to clearly understand what represents the class probabilities plot.
(same comment as above)
I don't understand why this is named `exactmatch`. Maybe the naming is wrong?
а откуда нарисовались wallpapers?
по идее тут конструктор уже и не нужен
я бы вообще так сделал

    $api->request('GET', '/account/', $data, $headers=array());

так и больше на HTTP/REST похоже, и не надо констант. но возможно у тебя есть какая-то причина на эти константы?
Make this private please :)
You need to follow our formatting standards listed in https://github.com/SpoutDev/Vanilla/blob/master/README.md
This is meant to discover HTML comments. That isn't discoverable using escapes.
`fake` doesn't appear to be necessary.
satra --

it's been a long time since we've spoken directly about pipelines.
 remember how i finally got the embarc institutions interested in using our
nipype pipeline for resting-state fmri?  well, it looks like it's time to
show them what we've got!  could you point me to a pipeline you'd recommend
for rs-fmri?


cheers,
@rno

arno klein
arno@binarybottle.com
http://www.binarybottle.com
--
research asst professor
psychiatry & behavioral science
stony brook university



On Fri, Nov 9, 2012 at 4:38 PM, Satrajit Ghosh <notifications@github.com>wrote:

> In bips/workflows/flexible_datagrabber.py:
>
> > @@ -90,7 +90,7 @@ def create_dataflow(self):
> >          import nipype.pipeline.engine as pe
> >          self._wk = pe.Workflow(name='custom_datagrabber')
> >          self._dg = pe.Node(nio.DataGrabber(outfields = self.outfields,
> > -                                     infields = self._get_infields()),
> > +                                     infields = self._get_infields(),sort_filelist=True),
>
> should be a check box. because flexible templates allow picking a
> particular order of files as well.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/INCF/BrainImagingPipelines/pull/53/files#r2088652>.
>
>
La quantité de résultats disponibles dans la requête. Ce nombre peut être inférieur à celui des résultats disponibles. Le moteur de recherche "Sphinx" limité par défaut le nombre de résultats à 1000. Toutefois, cette limite est paramétrable.
I agree. Plus this is hard to understand, to my opinion (I mean, the multiple drivers thing).
Minor: the --index is a bit strange since index index is not used later on.  Could start index at 0, use ++index, and avoid this.
Note - there are other instances of this construct that should be checked.
```#``` style comment is preferable here.
actually, this whole view seemed overly complex (the get method, for example, seems completely generic)
Oops. I've committed a fix for this so it takes the same folder as the app server.
We would indeed have to mount the fileshare at that location.
(That's not part of this PR though as we don't have an NFS yet)
Humm, if that is the case we'd end up opening and closing a lot of connections, oh well.
Just revisited and you may be okay without marking it intransitive.
We had to pin the dependency in Zipkin since cassie depends on finagle-thrift, which uses thrift 0.5.0. Cassie has a `FakeCassandra` that starts up a thrift `TThreadPoolServer` whose constructor signature has changed between 0.5.0 and 0.8.0. 
Sorry for the confusion.
This is duplication of string, use already existent HighlighterItemList::getTranslatedRegexpType, otherwise you have to translate it twice
"list" and "range" are python builtins, so readers might be a bit confused. Maybe use "sections" or "pieces" instead of "list"? And "number_range" instead of "range"? Not very satisfying, I know.. leaving it as list/range is ok too.
Can you replace this message with suggested by me yesterday ?
The Router extends the Events class in the underlying Backbone JavaScript implementation.

This change reflects that extension in the declarations file.
I think we log errors using the framework's logger, would like to hear what others think about this
Yeah, I caught a few of those - trying to use 100% everywhere for best responsive layout.
if res or res.body
`(width, height)` unnecessary; also, reversed, I think.
Yes, what do you think about that?
Could you comment on that, please? I'm not attached to the current design. We can surely revise and change it.
ok, lets document in the doc, more important than in the code.
this is here to remind that when a third part library supports this functionality, use it
this should be improved to  merge the CountWalker with the existing tree walkers instead of overwriting the array totally. @beberlei thoughts ?
Perfect for debugging
Can we add a TODO here, so we don't forget, this breaks compatibility with other drivers.
@pjcozzi I did some testing back when I started switching to this style for "private" functions and found that the overhead is approximately the same between the two.
These should be documented as optional.
This doesn't have to do with your changes, but this checks if `this._size` is even not that its a power of two.
Should we be using the new `Color` type throughout?
I put it in the Specs directory because I wanted to cleanly separate helper classes from actual specs.  Ultimately I don't care, but that was my reasoning.
Will reuse a constant.
Because these blocks are indented, they're showing up as code samples. Should probably be bulleted. So:

```
    *   In iOS 4.x...
        the `resumed` event.
```
methods plural? A little unclear. Maybe, "You must set the selected items after the picker has been displayed, so it cannot be done in markup. Set selected items by calling setSelectedRow in the controller."
Define "connected." On Mobile Web, it only matters if both controls have been added to something attached to the screen.
There is some logic regarding CountryCode that I saw in your code... was that suppose to be "country" and not "countryCode"?  Or is countryCode supported?
Can you rename histpos to histPos?
What's the reason for moving it out?
Oops. 

On Oct 22, 2012, at 8:20 AM, luke crouch <notifications@github.com> wrote:

> In restaurant_inspections/sanantonio.py:
> 
> > @@ -0,0 +1,99 @@
> > +import logging
> > +import re
> > +import sys
> > +import time
> > +import re
> duplicate import re line here?
> 
> —
> Reply to this email directly or view it on GitHub.
>
At the moment we're grabbing the island_ids from the user_geo_edits table based on the current user's id and then concatenating them with commas, so I think that should be fine
we should change el and use tab for better readability
Can you not use hard-coded values here?
`if (msbs < 2 || msbs > 6)`
Can you fix up set_rule() too?
not needed anymore, isn't it?
I don't think this is the right place for these.
unneeded empty lines
What is the reason here that we drop the usage of HttpStatusCode?
After destroying ```@links ||= ```, the rest can either be:

```
ThemePresentation.wrap_collection(__getobj__, ProfileLinkPresenter)
```

or:

```
ThemePresentation.wrap_collection(section, ProfileLinkPresenter)
```
also, version bump should be in it's own commit
could use assertions
this interface comes from jwage's bundle, this PR is a port of doctrine/doctrine-mongodb-odm-tailable-cursor-bundle
Wir legen den Auftrag ja nur bereits an, da wir vor dem "KAUFEN" noch einmal alle Angaben anzeigen möchten... Ändern geht dann nicht mehr, glaube ich....
Ansonsten ist es eine neue Offerte.. nicht?
createUser(Params $params)
{
  //Um den neuen User bereits mit sinnvollen Daten zu füllen!
  $user = new User();
  $this->persist($user);

  $this->updateUser($params);
}
2nd => second 
is ok

But it's not the 'navigation menu'. It's only the 'navigation icon' (compass) in the action bar and there is also no additional entry in any menu for it.
do you need to do a `.get(0)` here?
This exception won't be thrown since it is a mocked object. Please add the exception to the method throws list.
it is predefined by php on windows only : http://php.net/manual/en/reserved.constants.php
Actually we do it in many other places instead of creating an explicit callback class, but if you insist we can do it.
As far as that not being done under lock i have to fix it. . . good catch.
Ok, Oleg, thanks a lot!
I've found what I was looking for:
`BeanDefinitionValueResolver#resolveValueIfNecessary()`:
````java
else if (value instanceof BeanDefinition) {
			// Resolve plain BeanDefinition, without contained name: use dummy name.
			BeanDefinition bd = (BeanDefinition) value;
			return resolveInnerBean(argName, "(inner bean)", bd);
		}
````
Instead of quotes didn't we want to rather use <emphasis/> ?
Completely agree and we have to stay consistent with other modules that are using the same technique.
LiteralExpression.getValue is not costly at all so we are not gaining much by doing the check in onInit method. However since we are accepting an Expression we must realize that such Expression will be evaluated against the Message which is what Artem is saying. We probably also need tests which have expressions agains Message.

Basically look at HttpRequestExecutingMessageHandler.handleRequestMessage  and do exactly what's there.


Also, after yestarday's INT-2470 merge this one need to be rebased and conflicts needs to be resolved since its not mergable now
s/~~For that reason/To facilitate these uses/
Condense to:

Form for collecting input parameters for iReport.
Do we need any of these in the admin section?

Also, by removing the lhnavigation, I get the following design: http://cl.ly/F069
Individual screen shots and a better explanation have been added in the next commit.
this import probably isn't necessary; it probably came in when you copied some code over from another test.
use junit fail method (pass at least the exception message)
Should revert this version number change for the pull request.
For readability, let's mirror the block above this and do...

    ...
    default_fields = self.get_default_fields()
    for key, val in default_fields.items():
        ...
Let's just name the class `RequestFactory`
Citons GitHub : "hébergé sur GitHub", "forker", "pull-request".
you should advocate ``1.0.*`` here IMO
The `stable` branch is always updated with released stable versions. I'm not sure when `0.5` will be released — it's pretty stable at the moment but I would like to finish some in-progress work for it (see the [blog post](http://leafletjs.com/2012/10/25/leaflet-0-4-5-bugfix-release-and-plans-for-0.5.html)) before releasing it as a new major version.
We'll need to clean up the desc :-) But that can be done after.
And then you can make a separate alias to compile, or a bash function or rake task to compile all:

    funcition makecss()
    {
      for less in assets/stylesheets/*.less ; do
        cssname="$(basename \"$less\" .less)"
        lessc "$less" > "public/stylesheets/$cssname.css"
        echo "LESS => CSS $cssname"
      done
    }
Ok, i will make the change
Aww, sweet. The inheritance ensures it doesnt' break if anyone is ``except``ing IOError right now.
Not hard, I didn't have a pointer-comparison case before, so I was just trying for the easiest, most readable solution.  This should end up getting inlined anyway, and it's a pretty rare test, so I don't think there is any performance implication.
I can change it if you want.
I think it makes more sense to access the controller from the dialog code.  Context menus do it that way
So these tests don't work at the moment because the place where context menu events are triggered is 1 level lower in native and will not receive these events. However its possible it will be supported in the future so we were considering leaving it in because creating a touchEvent was mildly difficult.
CanDelete instead of "can delete".
The issue would be if the markup was changed, for example if the spans weren't adjacent anymore. If you can make something like .fls-back-linl .fls-back-link-text {...} it should make the markup more robust to change.
Here you use plural but in the next paragraph singular... so it's better to be consistent and start here with singular like

`A widget takes data that...`
Looking at the compass JS code I don't think we need the s(g)etTimeout and stop actions as all of the is handled via javascript intervals