labofsahil/github-event-dataset-2012 huggingface dataset:
On the other hand, if you assume all vertices inside the label have the same activity, then you see the scaling effect, as the first column in U has "1 / sqrt(len(vertidx))" entries.
why do you need to sort to score sources?
No changes in this file, can you exclude it?
Try using the Gen.frequency combinator to increase the ratio between generated leaf nodes and internal nodes, so the overall generation converges, and not diverges as it does for you now.
Is this a magic file that Rails uses? If not, I'd just remove it or put this information in the README.
Good point. I definitely want to let people know about the change.

I'll mute the anxiety.
getAuthor() is deprecated for good reason.

Would be a lot safer to put the real full name of the user in REST which mean serializing getAuthorReference().
Left over html. I couldn't find a text version of the original PEP. Clearly I missed some of them.
Not sure exactly what you are pointing to.  Maybe the '__docformat_...' line?  That was, if memory serves, an early requirement to get code accepted into numpy.  The pad module is actually quite old and languished in the numpy bug/feature tracker for a couple years.
Why not just *args here? What does passing through the Tuple constructor give you? If it's just sympification how about calling sympify on all in args. 

If you keep the Tuple I suspect you don't need the last .args. *Tuple(*args) should expand out to an iterable.
here, too.
cool -- thank you guys... just to make sure -- this is for a recent snapshot of ANTs?
i.e. something after post 1.9.2 svn revision 680 (which we carry in neurodebian)... so I guess I should finally update ANTs package... uff ;)
может быть foreach сюда?
Si, ese valor se sobrescribe cada vez que se devuelve a la UI
it'd be better if determine_metadata returned the line count too, which could be optionally passed in here, precluding the need to for a third pass over the data.
I'd prefer you get all of the values out of the mapper into a list, and then use something like self.assertItemsEqual
This string is not clear enough for translation staff

what about 

"Too many records to update (%d records matching criterias, tool limited to %d records)"
That's really got to be the craziest thing ever. Both look pretty much the same... :(
Please use `I18n.t()` since using just `t` will interfere with the `textilize` helper method in Redcloth.
+1 again
looks like you included this file in the patch just to add a trailing line ending?
What if the MarshallingType instance hasn't yet been added to `typeHints`?
Done: https://issues.jboss.org/browse/ISPN-2644
Right, should've also looked closer; who wrote that in the first place? :) 

```python
zs, xs = np.broadcast_arrays(zs, xs)
zs = np.tile(zs[0], xs.size)
zs = np.ones(xs.size) * zs[0]
```
all work (and there probably are a lot more solutions). I'd vote for `tile` in this case. It is well-readable and super fast (enough for this purpose). In my opionion broadcasting is OTT at this point. However, I am quite inexperienced with numpy and matplotlib, so the decision is up to you and I'll respect it.

Can I somehow change the pull request or should I revoke it?
We should add an index page (and maybe a 404 error page), which lists all of the available figures (for bonus points, it would be nice to use the ```figure.get_label``` for that).
I keep getting sphinx syntax mixed up in my head.  A directive is made up of a leading double dots, a space and a *single* word followed by double colons.  Any arguments for that directive follow the double colons.  "keyword" arguments (if any) are provided on the subsequent indented lines with the name of the keyword surrounded by colons and the value after it (see doc/api/index.rst for an example).  Finally, any remaining indented text is treated as-is as a large string block.

It doesn't look like there is an ".. example::" directive, so I would re-write this part of the docstring as:
```
yerr[1] is error above *y*. For example::
```
That is actually the most awesome part of it :). The only problem is that a bad tex string breaks the font manager. For me it doesn't matter because that string would break the compilation process anyways.
Looks like it’s a bit old : 397b4cc7ba3c196e3367d34998a6bef4b30337ad
As far as I see, should be simplified.
I'm using the "no semicolons" convention - sorry :)
Yes, I think this is absolutely appropriate. The code for my implementation of getUnreadMessageCount was yanked from Carl's initial stab at Me feed changes. When I moved that logic into UnreadMessageCountService I actually validated it against your code. Refactoring into a common service is the right approach IMHO.
what about some documentation what you are doing here? :)
the indention level in the .h file is still strange - please fix that - then it can be merged
For applications that require having `.env` to work in development, it would be nice to have a sample to copy from. If we copy directly to `.env`, we can't check it into the project, which means you can't update it as new config options are added to the app. I don't think `heroku config:pull` actually solves this issue; I've needed different configuration locally than on staging/production on every application I've worked on, and I don't like the idea of people needing to remember to delete keys like `BRAINTREE_MERCHANT_ID`.

I also like that this brings us closer to having a single command to set up the application. It would be awesome to have the protocol for new apps be these two commands:

```
./scripts/setup
foreman start
```
Note the original range was (32..126), but since 32 is the space, I avoid the gsub that removes the space.

Original:
    (32..126).map(&:chr).join.gsub(/\s/, '')
Warum? Klasse nicht angegeben?
@thradec could this work in the IDE's doc hover? I have a feeling that Eclipse's `Browser` control supports javascript. It definitely supports css.
Right, I will fix it.
I don't think so. This task runs a simple integration test to smoke test that stuff is kosher work with Coffee. It looks like grunt-coffee is only for compiling coffee from source, not just running something arbitrarily. I went the same route as the Rakefile for this tast 1) because its a little-used integration test and 2) I didn't feel like pulling coffee in as another dependency.

But it could just be added as a devDependency and get rid of relying on the global binary. Thoughts @tbranyen?
You're missing `tmdb_party` version number.
This is usless remove
This should be changed - that was supposed to be taken out. The line should read:

``` url = prepare.config[:omnibus_url] || "http://opscode.com/chef/install.sh ```

I have fixed this in 656ed9b. Thanks for catching it.
I was paranoid of using too many different content types here, in case some silly site used the incorrect type, but I guess `text/` is safe enough.
If the property is removed from `CourseComponentDAO`, this line should also be removed.
You do not need units if value is zero.
I remember you saying we should attach additional data to stations. What should I write then?
No need for newline after `<ul class="posts">`.
I would rather change english version to "Delete".  This text appears on post so we know it means deleting post. But this will probably be replaced with an icon.
Almost nothing.

    {
        u'audience': u'https://mysite.com:443',
        u'email': u'myemail@example.com',
        u'issuer': u'browserid.org',
        u'status': u'okay',
        u'expires': 1311377222765
    }
jQuery does not, and will not include "browser sniffing" via the navigator object.
Why does it print Field13 and then actually print's Field11's value?
I'm pretty sure the above method will fail with a `ValueError` if the key is not present. If you would prefer to have it return None in the event there is no key in either GET or POST, use
```python
val = request.GET.get(key, None)
```
I believe it's still secure because of the csrf tag. You don't need a form object because you have no real data to process.
This list should be output at the end.
this emty line can be removed too
nooooooooo mannnnnnnnnnnnnn
its branch  master for cm7 .............
Here I would prefer a less verbose approach like:
```
IO_FORMATS = {
    # name: (export_method, import_method)
    'fits': (dendro_export_fits, dendro_import_fits),
    'hdf5': (dendro_export_hdf5, dendro_import_hdf5),
}
```
Hmm, yo sigo viendo:

    &lt;ul class="second"&gt;
       Síguenos en: &nbsp;

En lugar de:

    &lt;ul class="second"&gt;

       &lt;li&gt;Síguenos en: &nbsp;&lt;/li&gt;

Pero ya lo corrijo yo que tardamos si no más en entendernos que en corregirlo.
I agree. I have moved these methods to MagnitudeMeasure
When `sort_order` is `unique` (or `unique_together`), the instance being saved needs to be moved aside so that there is room to rearrange the others.
Two things:

- suggest "it can be accessed from anywhere within a program" rather than "it can be got at from anywhere"
- this paragraph ends as though the thought will be continued after the code example, but it is not continued
It wouldn't cause clients to pull in eb examples module automatically, but it's really not important: It only simplifies things for sibling EB modules-- If they need to depend on examples module, they wouldn't have to specify version as it would be defaulted within their parent pom.xml. Given that the examples module is likely a leaf module, no sibling will benefit. Leave as is.
Please don't merge until we get all tests working.
the package name is bit odd. should it be under '...elephantbird.lucene...'? almost always 'mapreduce.input' deals with MR inputformats
same here and elsewhere
let's put 1.0 in a configurable option
> I have no clue what this uri is for.

`uri` just provides `module.uri` expected by commonjs modules.
I do agree this  does not necessary makes much sense in this particular case, but since some SDK modules are wrap in such module boilerplate I'd prefer to keep that boilerplate consistent  across these modules. So unless you feel very strongly about this I'd really prefer to keep it.
$arr is not camel cased, what about $resultArray?
`cfg->cache_dir`
What's this, comment.
I did it for a reason, I wanted to crosshair was different for the view from the cockpit and different for the other views, 
but of course everyone sees it differently :)
urls_to_copy.join("\n") if we use a QStringList.
Same note as above about `respond_to?`.
To be consistent with the standard `#each` implementation we should return a `Enumerator` when no block is given. I think this should do the trick:

```ruby
def each
  block_given? ? @base_obj.each { |e| yield e } : @base_obj.to_enum
end
```

Maybe you'll want to return `self` when block is given (as you do now), but I'm not sure that it's better than returning `@base_obj` as the implementation above does (it return result of `@base_obj.each` that is `@base_obj` itself).
Even though it's on the prototype, I'm still slightly hesitant to ever name a function apply.
The end time can be equal to the duration of the video
This does not seem correct.  I think this will force packaging of the config files into the war/ear.
Need to add overrides annotation for this method.
Yes, I agree!  My sole initial objection to it was that I'm lazy :) but I do admit that it seems worthwhile, especially when there are many keyword parameters to document.
I second your ".texroot" idea!
Although 'id' is the default ID attribute, Backbone now allows that to be modified so I replaced references to it throughout.
I __could__, but I like mine better - and it's more efficient :)
Instead of having a function for this, why not run this once and cache it forever? I mean, it's not a big pain, but it could be as seen that this is called within transports, especially since functions are expensive in function (relatively speaking)
Hannes Magnusson <notifications@github.com> wrote:

> > @@ -98,6 +98,15 @@ void
> php_mongo_connection_force_primary(mongo_cursor *cursor, mongo_link
> *link T
> >    zval_add_ref(&db->link);
> >  
> >    PHP_MONGO_GET_LINK(zlink);
> > +
> > +	if (link->servers->default_w != -1) {
> > +		zend_update_property_long(mongo_ce_DB, getThis(), "w",
> strlen("w"), link->servers->default_w TSRMLS_CC);
> > +	} else if (link->servers->default_wstring != NULL) {
> > +		zend_update_property_string(mongo_ce_DB, getThis(), "w",
> strlen("w"), link->servers->default_wstring TSRMLS_CC);
> 
> Have you gone through all the cases were we read the property back to
> make sure we truly support it as string?

Yes I think so, but check that please. That's why we review code. ;-)
style nitpick: spacing
Hello

Updated links to point to GitHub. Pushed to pull request branch
We need it at least when the player is shown. I’ve removed it from hidePlayer().
As I mentioned below, you're only using keypaths to parse the API dictionaries, which I think is what makes sense. So I'd try to make that clear in the JSEntityProperty interface.
for library code, it's important to keep this covered, so better add tests for it, or nuke it altogether
Developing on one platform (e.g. Windows) and then deploying to production on another platform (e.g. Linux) can lead to strange bugs popping up when you go live. To avoid this, you should consider developing via a Virtual Machine that mimics your production platform.
Can someone on-site (or off-site using skype) explain this to me?
А ещё, надо отдельно проверять на null.
Пробелы вокруг +
That doesn't look right -- it makes all errors fatal, doesn't it?
hmm, sorry, class is only used to parse children so it will not be an issue
Could we do this more generic? So we don't have to add a function and property for every select / form element we add?
> I've assigned `fit` to the EM algorithm, so I think setting `filter_update` to `partial_fit` would be a bit confusing.  It's more like a `partial_predict` than anything, but there's no API for that, right?

ok fair enough. Thanks for clarifying.
redefinition from line 16
Rather than just linking here, please include an inline plot + a small paragraph explaining what are the input and the outputs for this example.
мені не подобається цей фільтр. є id менюшки. всередині ul > li > a. цього має бути достатньо по ідеї для витягування посилання?
Maybe check the documentation (can just use google "ruby array shuffle") for

```ruby
# Array#shuffle!
rooms = ["kitchen", "bathroom", "floor", "dining_room"]
rooms.shuffle!
# => some random order
```

as well as
```ruby
# Array#pop
rooms = ["kitchen", "bathroom", "floor", "dining_room"]
rooms.pop
# => "dining_room"
```
Is a floating-point compare with 0.0 appropriate here?  It's not fragile?

I dont fully understand the context this is executing in, but anytime equality checks in floating point code are a bit scary. Could you clarify why this is preferable to testing with an epsilon?  (such as fabsf(...) < std::numeric_limits<float>::min())
It was a somewhat debug line to know if it went here. :P  Also what @ZNickq said.
Seems fine to me, bail unless TERM seems sane and tput command exists.

Cursory test looks ok (I'll write some tests to go with this at some point)

rvm info # coloured output
unset TERM
rvm info # uncoloured output, nothing on stderr
Sweet. Thanks.  (/me adds to list of things to understand.)
This line could be inside the if...
Not 100% sure that is true... it is at least harmless post 3.8.6, even if not necessary
Некоторые делают это как хелпер. Кто-то даже как partial О__о
It can make sense to keep all the connection stuff grouped, but I get your point.
I can move  `TEST_CONNECTION_SETTINGS` in `tests/base.py`. By the way, keeping it grouped with the other setting can make sense as well: one who changes one of them could forget that the other exists.
About `DEFAULT_CONNECTION_SETTINGS`, it's used, internally, only by `database.py` and `tests/base.py`, so I suggest to keep it `database.py`. The other solution I see is to move it back to `__init__.py`.
Do you have another idea?
I didn't pass it along correctly when I added get_file, fixed that.
_Messae_ on purpose, because that's also called correctly below through _Seq()_. Or typo?
You need a "_" prefix for "extensions" to be ref-able.
That is an interesting point. I don’t think it’s an issue, because 0, meaning forever, is the default. As such, there’s no reason to distinguish between the configured to zero and unconfigured case. The missing property is not a problem, in fact it’s the default. If we threw an exception here everyone would be required to update their code to set 0 as the timeout to maintain existing behavior.

Basically, these properties are optional, and if they’re not set they’re the same as configuring to 0.

From: Albert Cheng [mailto:notifications@github.com]
Sent: Tuesday, September 25, 2012 11:05 AM
To: christav/azure-sdk-for-java
Cc: Chris Tavares
Subject: Re: [azure-sdk-for-java] Adding support for setting timeout in configuration (#2)


In microsoft-azure-api/src/main/java/com/microsoft/windowsazure/services/core/utils/pipeline/HttpURLConnectionClientHandler.java:

> @@ -37,10 +37,28 @@

>  import com.sun.jersey.core.header.InBoundHeaders;

>

>  public class HttpURLConnectionClientHandler extends TerminatingClientHandler {

> +

> +    private final int connectionTimeoutMS;

> +    private final int readTimeoutMS;

> +

> +    public HttpURLConnectionClientHandler(ClientConfig clientConfig) {

> +        connectionTimeoutMS = readTimeoutFromConfig(clientConfig, ClientConfig.PROPERTY_CONNECT_TIMEOUT);

> +        readTimeoutMS = readTimeoutFromConfig(clientConfig, ClientConfig.PROPERTY_READ_TIMEOUT);

> +    }

> +

> +    private static int readTimeoutFromConfig(ClientConfig config, String propertyName) {

> +        Integer property = (Integer) config.getProperty(propertyName);

> +        if (property != null) {

> +            return property.intValue();

> +        }

> +        return 0;

There is an edge case we need to consider, when the specified propertyName does not exist, we will return 0. How can the consumer of this function distinguish between a configured 0 and a missing property scenario? I would recommend to throw an exception when the property is null, return 0 only when the configured value is 0.

—
Reply to this email directly or view it on GitHub<https://github.com/christav/azure-sdk-for-java/pull/2/files#r1687350>.
When using in a cluster setting, we should probably use the distributed cache and getCacheFiles for this ([see also another implementation](https://github.com/dbpedia-spotlight/pignlproc/blob/master/src/main/java/pignlproc/index/LuceneTokenizer.java#L68)).
And it will be interesting if, for English, all surface forms fit in memory.
Using destructuring this would go like this:

~~~clojure
(defn contains-point? [[[left bottom] [right top]] [x y]]
  (and (<= left x right)
          (<= bottom y top)))
~~~
Is it possible to make the frequency, time, and retention of backups more configurable? Perhaps make the cron time settings configurable and incorporate logrotate to make retention configurable.
Please no general urls in references, only references to the specific technique
why not ```@@ps_opts.usage```?
57600 and 38400 are same, is it right?
Also, what happens in the `else` case? It looks like this just fails silently. Maybe it should raise an error?
Sure. I'm asking because I think this algorithms could be very interesting in 3-D, for materials science images. And in 3-D, 6 vs 26 connectivity makes a bigger difference! I might give it a try as well.
We could remove this line.
Is initRuntime really relevant for Javascript? I mean the Javascript is always runtime, there is no compile-time code in Javascript.
yeah log it too, but we should be smart here
can you remove your credentials from  here 
This data will be added in a different repo
Thanks, totally forgot.
Trailing comma needs to be removed (it breaks the build).  Eclipse likes to hide warnings unless you do a clean.
Need to pass ```CultureInfo.InvariantCulture``` here.
Also, this format ("yyyy-MM-ddThh:mm:ss.fffffff+00:00") only works if the kind of the DateTime is UTC (or Unspecified). In the code below, if run in PDT (GMT-07:00), it will print the incorrect point in time (2011-10-11T12:20:15.0120000+00:00 - it shifted the time zone, but not the time).

```c#
    DateTime date = new DateTime(2011, 10, 11, 12, 20, 15, 12, DateTimeKind.Local);
    Console.WriteLine(date.ToString("yyyy-MM-ddThh:mm:ss.fffffff+00:00", CultureInfo.InvariantCulture));
```
If this code is not needed, please remove.
Two minor style points: omit "The" at the beginning of the summary, and use present tense. ("Android returns `android` ..." OR "Returns `android` for the Android platform, `iPhone OS` for the iOS platform (iPhone, iPad, or iPod Touch), and `mobileweb` for the Mobile Web platform.")
See later comments. By this point, it WILL be too late, almost garunteed. In fact, it might even be garbage collected while this is happening. Read: Race condition crash possibly.
that's good ! This stuff always annoyed me !
I even think it could be proposed in starter-kit
It would, put one in
Setting this to `None` doesn't work, because you check against `None` at various places, e.g. in `easybuild/tools/toolchain/compiler.py`:

```python
if self.options.options_map.get('optarch', None) is None:
     self.log.raiseException("_get_optimal_architecture: don't know how to set optarch for %s." % self.arch)
```

That's why I didn't set this to None...
add something like
#ifndef _MNB_LAUNCHER_ZG_UTILS_H_
....
as usual with headers
Put the default getWidth method here, and you can skip a couple of lines :-)
This behaviour would be very surprising to me. I would expect an assignment to always change the reference, and never the referee.
```em``` tag is not closing after ".done".
これ、0になるのマズくないですか?

元のテストの意図は、クエリパラメタを削る設定の検証をするというものです。今回の変更にあわせて、テストケースも変更するか、あるいは、別のテストケースとして追加してほしいです。
So why not embed all of this safety in `isFooMigrated()`, and add a static version, so you can just do:

```
boolean migrated = FennecControlHelper.isHistoryMigrated(session.getContext());
if (!migrated) {
  // Log
}
return migrated;
```

where

```
public static boolean isHistoryMigrated(Context context) {
  FennecControlHelper h = new FennecControlHelper(context);
  try {
    …
  } finally {
    h.releaseProviders();
  }
}
```

It's the same between both content providers, and you just release…
I think for a public check boolean is the better return value? maybe adding two methods? one throwing the exception and one adding true
IMHO I wouldn't remove the related objects when I am deleting an engineering object. But you are correct, that there should be a remove enhancement too. Since there could be some related models where the reference got just missing. I will add that
Let's use:

```ruby
if Chef::VersionConstraint.new(">= 10.14").include?(node['chef_packages']['chef']['version'])
```
this is totally wrong. your form is editing the relation here. So it should remove the relation between objects (which is done properly by the binding itself when you are editing it from the owning side, and works properly for both sides on Symfony master). It should not remove the related object entirely (as it may be used elsewhere)
Mon erreur a la creation de la methode ask, j'avais inversé les arguments, l'api donne title,mesage dans cet ordre.
xmlns stuff is enough. However, the assumption that the doc will contain `xmlns=` is wrong. It may contain something like `xmlns:meta=` or whatever namespace shortcut the author prefers.
I don't really want to say "method" arguments as that sounds like we are excluding functions.
実際のファイル名とココのファイルが一致していません。
一致させておいてください。
また、----- だと、意味がわからないので、意味のわかるファイル名にしてください。
Тут та же хрень с одинарным лукапом.
this code doesn't care if its a map, its just the thing responsible for importing tags
The idea is not to test the behavior of uglify, but simply that uglify was called by the filter. 

So one test script, preferably simple, and the corresponding output is enough.
Returning a string is odd for a programmatic API. This should rather return a tuple or namedtuple or dictionary containing the relevant data in programmatic form (i.e. number of cases imported, list of cases skipped). It should be the responsibility of the management command to transform that into useful output for command-line usage.
Needs a space before the `{`
The Implementation ID comes into play if there are  no concept mappings. But what happens if there is no implementation ID either ?
At least, the partial fix enable the data to be send to the server. here is the issue tracking the decoding logic. 

https://github.com/WindowsAzure/azure-sdk-for-java/issues/142
But does it behave same as original one?

There are also many other things to tweak about JSON serialization: we do not really need to convert serialization to string for the response, but it could be streamed back (so serialized to file handler).

But I find this a premature optimization thing, BTW. Because there are many things which could be improved here. Have you used compiled version of original serializator? And how big objects do you serialize? I really cannot believe that this can be an issue. You should not serialize to big things anyway, as this means too much traffic sending to the client.

If you would really like to use this, then you could define a parent resource to all django-datastream resources and configure this library everywhere. So that it is used at least consistently.
In general I think constructing HTML fragments with jQuery is better than this plaything with strings. You can very fast forget to escape something and so on. See [example](https://github.com/wlanslovenija/PiplMesh/blob/master/piplmesh/frontend/static/piplmesh/js/home.js#L146).
"modules, and how they"
will you provide properties to access the controls (toolbar, button) to allow for customization?
We should also be adding a localizedDescription to the userInfo.  See examples where I created NSError instances.
I don't think this is the right word.
Can we cleanup this test so it's only testing your slog2 change and not all parameters. We have another test that tests the other ones.
Yep.  Fixed.
Hi I haven't got any real experience with py3k, so it's really great there's some development being done I can learn from - and probably others as well.

My first Q would be, what is the reason you are dropping support for py26. Is it just too hard to maintain? I've been looking into the deprication policy and varioius release notes for django and it looks like they are not planning to drop py26 support in the near future, although they do recommend using at least py27.
@jeffheifetz @dansilivistru and I talked last week to say that a file was a better option. Personally I prefer file because its more convenient. 

This file is added to the .gitignore so your changes won't get checked in. So for a repo you just set it up once and then you never have to worry about passing in a parameter until you change the packager.

I personally prefer this approach so I never have to look it up again.
Did you mean "the reason why it failed"?
Nevermind, I see now that you are redirected to a login page. My mistake.
I don't think this is a possible outcome.  If a user is the owner of a UserOrgRole, that user MUST have at least one organization, right?  Is there some edge-case I'm not thinking of?

Moreover, even if that is the case, I'm not sure why that deserves returning True. Perhaps an in-code comment about why that check is being done would be merited.
That change should be made, even if it is used elsewhere.

I like the NaN system with floating point - any operation with a nan operand gives a nan result. The nan's "bubble up" to the top. The compiler should only attempt to continue semantic processing on operations with non-error operands. I'd like to be able to get rid of global.errors.

The corollary is the compiler should not attempt to guess at what the user meant and patch things up.
Actually, I think we should leave them separate, since autoconfig always sends back small files, but ActiveSync can respond with arbitrarily-large files. That'll let us tune the timeouts separately. I'm not really sure what the best timeout values would be though...
Same comments from before in connection.Connection.
We also need to correct the reference to this in Socket.TCP and update it to Ti.Platform.address:

http://developer.appcelerator.com/apidoc/mobile/latest/Titanium.Network.Socket.TCP-object.html

(In the second code sample, which also seems to have messed-up formatting.)
Added ticket https://jira.appcelerator.org/browse/TIMOB-7824
Yeah, this is a case where it didn't seem like magic at all when I wrote
this.  It's the third or fourth time I've done something like this so it
feels obvious to me now.  If I put myself in the shoes I was wearing the
first time I drew this pattern it'd have been obvious that I needed to
comment this more clearly.

Will keep this in mind and add comments if I'm in this code again soon.
You're right.  And I also had the thought at some point that a puzzler shouldn't show questionable style. Unless that's the point of the puzzler, of course.
That can't be right. You're `PUT`ing to the collection, not to the record. You  want `wboURI(COLLECTION_NAME, record.guid)`.
Unless HttpConnector is correctly published in all cases, this is not thread-safe.
Mark, your solution produces some strange behavior:
```
   <enricher>
      <property name="@testBean1.property" expression="@testBean2.property"/>
   </enricher>
```
You can see it in the end of **ContentEnricher#handleRequestMessage**
I think we should have separate **evaluationContexts**: one for getting property values with BeanResolver, and another for setting on the **targetPayload**
Also, please, look at linked issue: https://jira.springsource.org/browse/INT-2447
I'm not sure this is the settings we want for IP.
Free the result.
nevermind I see you did it already, you're fast :P
"Admingenerator" is the default value please remove this parameter
Comment which explains that it first checks if the ID already exists
(not (teen? age)) would have returned the wanted boolean already
Je pense que c'est une erreur de passer au masculin : ce n'est pas le build qui est paramétré, c'est la "tâche de build".