labofsahil/github-event-dataset-2012 huggingface dataset:
Jenkins will have a fresh clone since it cleans the workspace and then re-clones on each build so this isn't a concern. I'm going to add it, though, since it's nice for local testing.
This assertion is not very valuable because it is basically just checking against itself.

What we need to assert against here is the actual UI, for example checking that the new app's icon is present.

I did a little check myself and if we switch back to the homescreen frame then we can check that the icon is present.

That to me is a good check for a smoke test that the app has installed correctly :)
the oath?
This function, and the decimalToHex function can be put in a <script> tag at the top of the exercise page. Again, like with the answer types, it just becomes less code for us to maintain when it's limited to only your exercise.
Reason I added it is testing, plus showing a full configuration might help people in some cases.

Maybe we can warn the people in the docs that it's not really recommended.
PHP shorttags are an abomination. Seriously?
I know, I was merely commenting on the coding style ;)
``include`` as shortcut for ``filter`` ? In Unix-style CLIs, shortcuts are only 1 char (and in probably all interfaces, they are shorter than the original name)
...but it works ;)
Can you think of another (better) approach?
The baseURL includes the domain, so the domain check is unnecessary.
сколько параметров у этого конструктора?
мусор удаляем
This won't work on windows.  This needs be abstracted into a caching layer as well as load relative to where ever SimpleCV exists.
The list comp above already existed so I just edited it, but this had to be added in. Agree that a list comp would be more consistent here though. Will change.
This actually fixes a (very) minor bug in CodeMirror right now--if you use Cmd-Right Arrow (or End) to go to the end of the longest line, the cursor is off the right edge of the visible scroll area.
You're assuming, without even checking, that the main image IFDs are contiguous.  Since the XML gives you a nice list, why do this?  Just build up the `level_list` directly by walking the result set.  (`_openslide_try_leica` then just has to verify that each level in the `level_list` is reasonable.)
camelCase -> `groupedEvents`
damn good call
http://news.ycombinator.com/item?id=2443138

On Saturday, November 24, 2012, Paul Fletcher-Hill wrote:

> In app/assets/javascripts/backbone/models/event.js.coffee:
>
> > @@ -10,15 +10,32 @@ class Gandalf.Models.Event extends Backbone.Model
> >      two = moment(e.get("start_at")) < moment(@get("end_at"))
> >      one && two
> >
> > +  categoryIds: () ->
> > +    arr = []
> > +    _.each @get("categories"), (c) ->
>
> Just a question I've been wondering about, but what's the difference
> between coffeescript's for model in collection and underscore's _.each(collection,
> function() )? Is one faster than the other?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pfletch1023/Gandalf/pull/31/files#r2215344>.
>
>


-- 
Sent in motion
Should be:
Opslaan en verder gaan met bewerken
The received value should be first, the expected value second.
I did it unconsolidated first and it was very verbose. The source of the problem is that we can't continue to validate the loginId properties if the loginId is undefined, which causes us to have to have if statements based on `validator.hasErrors()`. The two methods both normalize the logic better and make it much less verbose, letting the reader focus more on the logic of the method and less on the validation.

Maybe I can just comment the method invocations better?
ahh yes, good idea.
Probably, yes. Was thinking of creating a scripts/ directory to hold both dataload.js and run-dataload.js, but didn't want to be whipping up crazy directories all willy nilly :)
multi-line param
Mostly to reduce complexity for now. For local accounts, when you change password, which local account do you change password for?

I think in a model where you could have multiple accounts for the same provider, you may have an account management screen where you end your individual login ids (e.g., change the password for an individual local account). But that seems to be way more than we need ATM.
This should probably be part of the (soon to be created) authz module, possibly inside of the same performance folder
Please !hide
Also like your two cents on this one.
Please make this a multi-line function, it's doing too much to be on one line IMO.
Is it possible to change this behavior? Would be nice if it was consistent across implementations.
I don't understand the question, it's very easy to create lists in velocity. As for the type, like in Java it does not exist anymore in the bytecode.
Why? `[.]` is a required part of the counter, it separates the group from the counter name.
Sorry if I'm too picky, but I've had experience with searching around the web for a specific IP address and stumbling upon such random examples. Could you use an address from the 192.18.0.0/15 range (it's reserved for testing purposes)? Could do the same for v6 if there's such.
Н-р при 170 элементах массива нам надо найти 15%
DNA_LENGHT * percent / 100 выдаст 25
round(DNA_LENGHT * percent / 100) выдаст 25
В лекции 3 описывается метод description, можно использовать его. Так же обычно в модульном программировании принято не выполнять выводов в методах того или иного класса - обычно используют просто возвращаемые значения, а при надобности - генерируют ошибку.
this can not be added. tbs driver breaks many others....
now that i get the other part we'll definitely have to keep this and concat above, but I do think it should be additive instead of clobbering lookups
util file is looking really good right now.
return response.end(JSON.stringify(errorMessage));
Use one object for the result:
var weatherObject = {
temperature: ...,
sky: ...
}
Have you set any configuration so that updating the index can also automatically upon insert, update and delete?
Would evicting just the person from the session work?
Does this mean if there are more than 1000 entries the database, then the rest won't get indexed?
@sadache i don't understand what you're saying.  for the line "new controllers.JavaControllerInstance.index()" we are instantiating the class controllers.JavaControllerInstance.  Is there some special case you're talking about which I am not considering?
If you want, I could create a helper to convert the promise to a single callback. Q has a fin function but it does not carry the reject argument with it.
Yes. In the EventMachine::run loop I have

@@rx = EventMachine::bind_connect(
                    "0.0.0.0",
                    mo_port,
                    @config['smpp']['host'],
                    @config['smpp']['port'],
                    Smpp::Receiver,
                    smpp_config,
                    self    # delegate that will receive callbacks on MOs
and DRs and other events
                )

                @@tx = EventMachine::bind_connect(
                    "0.0.0.0",
                    mt_port,
                    @config['smpp']['host'],
                    @config['smpp']['port'],
                    Smpp::Transmitter,
                    smpp_config,
                    self    # delegate that will receive callbacks on MOs
and DRs and other events
                )


On Fri, Dec 7, 2012 at 5:51 PM, Ray Krueger <notifications@github.com>wrote:

> In lib/smpp/transmitter.rb:
>
> > @@ -3,6 +3,8 @@
> >
> >  class Smpp::Transmitter < Smpp::Base
> >
> > +  attr_reader :ack_ids
>
> Oh, so you're running the transmitter and receiver separately but in the
> same process?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/raykrueger/ruby-smpp/pull/11/files#r2350632>.
>
>



-- 

Quando hai fretta, l'unica cosa sensata da fare e' rallentare.
Backbone Collections have the each-function built-in: http://backbonejs.org/#Collection-Underscore-Methods
Please change it to use `Collection#each`
expect(config.proxies).toEqual {}


This was kinda regression test, because it was returning the whole execution context object (that config file is executed on), so it was leaking memory.
Why pull out the deliveryTAg, then import envelope._ and then call getDeliveryTag again at line 57?
use TestLatch everywhere you use CountDownLatch as TestLatch is an Awaitable
I guess you'll be going through and tidying up the docstrings a bit. Here, paragraphs would be very helpful.
А просто `insert ({ path, fd })` не сработает?
This line can be removed, the /other/ directory won't be part of an official install.
The original setting is not reverted in case the filename can not be open
Please replace all "forms::operation effectuee OK" with "Successful update" or "Successful delete" or whatever's more precise and english
This should be >= 0 to also include "6.0" itself (same about the elsif below).
I think you most likely want the final string on this line to be an argument to concat
Pls zap them :)
I don't see this added in attributes/default.rb (even as a commented-out attr). Is that intentional?
huge but very easy to generate with idea\Code\Delegate Methods :-)
Ok. Mind adding a javadoc comment on top of it with the rationale for using this collection? It helps future readers understand the reason for using it :)
That's better. I'll do so in my yml file :smiley:
btw, you should rename the variable for readability
is this really needed? does the framework bundle not depend on form anyways? if not and you want the dependency to be optional, you should instead of having addDefaultsIfNotSet in Configuration check if the form node is configured at all and only load the form.xml services if there is configuration.
Nope, I think it is correct.  In most places, an artist assumes data
coordinates.  With only a few special exceptions, anytime you want your
transform to operate in non-data coordinates, you have to explicitly
provide that.

What is confusing is what the python None means.  This is an example of
some of the inconsistencies that have developed over the years with
defaults.  However, in this case, there is no rcParam to refer to.  So long
as it is clearly documented what a transform of None means, I am fine with
this.
Also, this is for Travis and Travis starts with a completely clean environment every time it runs.
Мне честно говоря самому текущий вариант понравился. Он короче и нагляден и вроде бы нет причин создавать объект раньше чем надо начать трнзакцию.
So I'm a PHP developer, but I don't think `rescue` works quite like that:

```ruby
begin
  raise "your face"
rescue RuntimeError
rescue SyntaxError
  false
end
#=> nil

begin
  raise SyntaxError.new("invalid syntax")
rescue RuntimeError
rescue SyntaxError
  false
end
#=> false
```

You can separate the exceptions with commas though.
This function doesn't appear to be used
Haha nice.

On Mon, Oct 22, 2012 at 3:40 PM, Robin Kraft <notifications@github.com>wrote:

> In src/clj/forma/hadoop/jobs/forma.clj:
>
> > -  "Remove extraneous vectors from around a series"
> > -  [series]
> > -  (vec (flatten series)))
> > -
> > -(defn unnest-all
> > -  "Unnest series coming out of trends-cleanup of form
> > -   [[\"500\" 28 8 0 0 693 694 [[0.1 0.1]] [[0.2 0.2]] [[0.3 0.3]] [[0.4 0.4]]]]"
> > -  [& series]
> > -  (map unnest-series series))
> > +     (let [nodata -9999
> > +           src [[1 827 1 2 3]
> > +                [1 829 2 3 4]]]
> > +      (??- (<- [?id ?per-ts ?f1-ts ?f2-ts ?f3-ts]
> > +        (src ?id ?period ?f1 ?f2 ?f3)
> > +        (consolidate-timeseries nodata ?period ?f1 ?f2 ?f3 :> ?per-ts ?f1-ts ?f2-ts ?f3-ts))))
> > +     ;=> (([1 [827 -9999 829] [1 -9999 2] [2 -9999 3] [3 -9999 4]]))"
>
> 'tis docstringed already. Github isn't showing enough lines!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/reddmetrics/forma-clj/pull/274/files#r1910347>.
>
>
@bp323 Can we use newaddcontent_ ...
@apex13 Can we also provide a good error when the options object itself will not exist / is falsy?

for instance:

```javascript
if (!options || !options.groupId) {
    throw 'getGroupInformation: The options object is empty or no groupId provided.';
}
I changed this back, but it mentions to indent the right hand values in http://oae-widgets.sakaiproject.org/sdk/widgetdesignandstyle/cssguide
Maybe I'm missing something, but this does not seem to actually test that the data was cast, no?

I think you need to add something like ``assertInternalType``.
hmm check.py doesn't check for alphabetization unfortunately :\
Is it possible to define a more specific return type; maybe using a union like ``null|Object``, or ``string|array``? Please also check line 1039.
Are you sure you want to override the parameter ``$value``? This often makes code less readable. I found this a couple of more times on lines: 81, 86, 88, 90, 92.
You're changing this for all platforms, make BB specific
For most classes without special sprites, implementing recoil is as easy as subclassing the Weapon object and adding the following lines
The images should live in a folder that matches the post, no?
The condition is true as soon as JSON is undefined... that will not work.
Kudos! How awkward I was!
Wow. I actually missed the fact that the class method would simply replace the entire *__has_feature*... Embarrassing.
Def the way to go.
IFF = IF?
В данном случае мало вероятно, но все равно, желательно ознакомится с sql injection при такой передаче параметров в запрос.
Please wrap if blocks in curly-braces for consistency.
How would the $file argument work? As far as I can tell the first caller wins. How did you see this being used?
Those curly quotes should probably be straight quotes ;)
This one is tricky to find by oneself, it's when you want to add a new event on your dropbox to keep it synced with your devices, a tableview is shown with the list of all the events on the current device and the events that are already synced with dropbox are faded with this little extra sentence "already synced"
use _buttons.less
One thing that would be cool is if we could specify the maximum number of characters in the text output. So if we said `text_width=80`, then the code would automatically word-wrap everything to 80 characters when generating text output. Otherwise the text could scroll infinitely to one side, or something.
Fixed the docs. It has to be a string because there can be values like "16MB".
Sorry, I meant that there's a typo in the comment: the search ID (&lt:sid>) instead of &lt;.
Is this not an overload? i.e. both versions exist?
I would use same title here: *404 Not Found*.
There is space to much in link.
2 minutes is ok. :) I was debugging something and forgot to change back. Will fix it.
You cannot parse this page? And extract in similar way? Maybe not parse as XML, but use some HTML parser. So that you get DOM out of it. And then just navigate to those elements you want.
So instead of `__subclasses__` I would manually define a variable:

```
HOROSCOPES = (
    EnglishHoroscope(),
    SlovenianHoroscope(),
)
```
No empty line here. Those two are same package (PiplMesh).
Well, as per http://api.jquery.com/prop/ both are suggested, but I think `prop` is accessed though DOM and `is` is though CSS, so I presonally prefer `prop`, but this is jQ 1.6 I think+.
(Of course is variable is easily passed around. You do not have to complicate too much here. I just wanted to point out that this can be done instead of using IDs everywhere automatically.)
You don't need first two capture groups here in reges. `r'(\d+).\s+(\w+):\s+$'` should be probably enough. :-)
Indent once. Do not care about output look. Care about template readability.
Only event.
https://github.com/monlog/monlog/commit/681aad1ba98f6d87978402da7aab443b546e6474
I modified the gfortran formula such that we can leave the datarootdir at `#{share}`. I removed `man7` from the gfortran formula and I hope the get rid of the `locale` (translations) by `--disable-nls` (I have not yet tested the latter).

Then, there are no conflicts between gcc and gfortran anymore, I think, because - as you pointed out - the program-suffix is also applied to the man1 pages.
Ja, habe ich mir auch überlegt, aber ich dachte so ist es besser lesbar im JSON und im Code (Check auf "unknownbeertye" statt -1)... aber ich weiss auch nicht ob das schlau ist...
in strings.xml auslagern
"given" needs to be tightened, here. Like... from the Stockpile? What if she makes a salvage roll?
This whole category definition seems dirty to me. It's a duplication of `CMObject`'s private category definition, but in `CMStore`. Was it there already?
we don't have to do this anymore, as the options are available in the buildView method.
actually. Didn't read all of your feedback, you're totally right I can set the instance variable directly. Will do that, thanks.
Top 25 Sponsors
not one 1 but the l(link) function.
Why is cp/build removed here?
You're right but I think it will work. By default it will install django-tagging which is what the majority probably wants but if in your dependency you specify the 'multilingual' feature it will finish by installing django-tagging-ng, which will override the initial package.

Try add it as an optional feature, create a source distribution by running 
```python setup.py sdist```
and than try installing it by running
```pip install -f file://./dist "django-cms-pagetags [multlang]"```
and it should work
A technical question. This is the density of all the X_j, evaluated at the left out point X_i correct? Is this right? I thought you would just evaluate the density of f_hat = f(X_j) over some grid support x and use sum(log(f_hat)) as in Racine and Li section 2.3.4. 

"f_hat(−i) (x) is the leave-one-out kernel estimator of f (Xi ) that uses all points except Xi to construct the density estimate."

Is there another reference or more details for this? I haven't been through this in a while.
Fixed. If we can come to an agreement on implementation details then I think (?) this is ready to merge. I'll add some  documentation.
if (this.user_attrs_) {
  sources.push(this.user_attrs_);
  if (seriesName) {
    ...
  }
}
Yeah, there is an 'app' and this refers to it, so this is redundant.
This is only approximately true.  I spoke to Gary about it, and apparently he added the first order correction to get the right flux, which he says is accurate to about 1.e-5.  I pointed this out in #208, which isn't merged yet.  So you don't need to add it here.  I just figured I'd point it out.
mutable or not, interesting.  Thanks.
Apache is stopped; this is done in the install phase.

  commands.add("/etc/init.d/apache2 stop");
     
So it needs to be started.

But I think it wouldn't harm to change it to restart; if running then stop/start. If not running then start.
Not sure about this. Is this to support restart? Or the beginnings of making start idempotent?
Feels like it will also mask some errors by ignoring what was passed in to the method..
I think it's strange that the DTO structure wouldn't mirror the xml structure. Is it a sign that the xml is perhaps not the best (e.g. in a DTO you wouldn't have a `List<Object> entries` field to put templates, entities and policies into)?
Log at debug or trace instead?
This is where I have put the Astyanax client code in the next commits
Phighchart

Not camelCase, it's the name.
Please add prototype comments to functions to explain what they do, and how they work. (I know, I haven't been too good with that either!)
or event pass in the Icon and Pixmap objects to the function directly.
shouldn't this be done in ``teardown`` instead so that each  test uses a clean storage ?
Если тебе на вход дать "1 1 + 1" твой калькулятор выдаст результат 12. Хотя это ошибка ввода.
Под windows это работать не будет
Я как раз жду от тебя объяснений, зачем ты synchronized добавил. А если решишь убрать synchronized - надо будет объяснить, почему :)
Надо избавляться от copy-paste. Напиши comparator, который будет вызывать String.compareTo.
else не нужен
Before the points we chose were truncated to integers, now that they aren't we need to prevent displaying giant unformatted floats. But I sort of hate doing formatting code in data structures, perhaps this belongs elsewhere?
None of the other methods in this class have try...catch, so why does this one?

Perhaps it belongs in the ServiceBusExceptionProcessor.
The toolbox isn't actually ready at this point, the host UI has been loaded, but not the toolbox within it. An `isReady` is set in #85 though if you merge.
We need to add a white space before "Utilities".
I'm wondering if we should just require the `data_source` js file in the base app. Let's deal with this later though.
This example seems disconnected with the text, is it required? Or do you want to show something else, a different class or something?
at this moment, it's required. I think it would be possible to let jekyll generate this information through a plugin of some sort.. (this is how we now convert the http-status codes and http-methods).. But I'm not 100% sure IF this is at all possible..
Can be combined with 3D case?  Seems like there's a lot of code duplication.
Interestingly, the Passive Aggressive paper suggests setting C=0.001. Otherwise, let me make all the changes you suggested. Also, does enforcing positive step-sizes for PA-I make sense?
Alright, the `min_density` param itself has been removed... Sorry for the noise.
``err`` could be renamed `error`.
Those unresolved conflict markers should be cleaned up before merging.

Also they are the cause of the doctest failure reported by travis.
< only if unbalanced. In the unbalanced case, this is also true for multi-class....
I don't really see the distinction.
There is an s missing from https
I think _slurp is ambiguous for a routine that is specific to a single website.  I could combine slurp, kiln_url, and slurp_from_kiln into one routine, but I thought it was a bit more readable to have three.  I don't feel that strongly, though, and will merge them if you prefer.
Mahout should not have a capital H, or should be all caps like GREENPLUM and POSTGRES.

Probably better to give some error, warning, or other indicator that these kinds are not yet supported.
I'm not comfortable to change the shortcut key right now, it will be great if you can revert the shortcut key modification here.
OK, thanks for reviewing, I really didn't know what to believe when I did the patch, the code or the message. I believed the message and aparently I was wrong :)

Does this message sounds right? "%s is hidden by %s" (remember this patch also prepends "Error: " to error messages)
we want to use `--embed` as well right? both here and below.
I kinda like the idea of having a Vertical/HorizontalMenuDrawer. Stuff like touch handling and measuring is quite different between the two types of implementations, but a lot of it is common between Top/Bottom and Left/Right. It makes sense to split it up.
I'm a bit confused why Top/BottomDrawer extends HorizontalMenuDrawer tho. Shouldn't it be Vertical? Since it opens up vertically.
This should also escape tabs.
The link to "more details" seemed the best way to handle this. The site has quite a few things listed as the outline. @matthewmccullough i'll be happy to change the actual description to this and the free event if you had different descriptions in mind!  :dizzy: :star: :boom:
Das ist suboptimal. Sauberer wäre "document.DocumentRating.count()" (kann auch im Template gemacht werden, einfach ohne Klammern).

Und falls mans als Methode implementieren würde (was aber unnötig ist), dann eher:

    def rating_count(self):
        return self.DocumentRating.count()
Speicherplatzoptimierung, ne?!
By then, matchLen would already have overwritten the old value.
not sure it's THAT closely related to argv... I'd put it in jutil.
There is something strange about defining a method called main() In this class because it is using a child thread class.  I

I would have expected the main() to be in the CNotifyThread object, and this method to be called something like cleanupAndFlush().

Probably beyond the scope of this pull request though - worth discussing the threading idioms as a separate issue.
I was sure you were going to ask! :)

It's to avoid spaces before comas and periods.
We should call these "Recommendations" instead of "Standards"... like the W3C.
@hason open an isue on Symfony2 as well. The code is copied from there
Tests are cheap, let's split this into two :)
Maybe we can mirror the sql_models and mongo_models setup, and only include this in mongo_settings. Then django.contrib.admin can be in sql_settings.
Way too many `||`. I'd prefer to have a method `isInterceptable` or something similar, and let the compiler optimize inlining. It would be readable
all are cosmetic hanges except line 46, I was getting null pointer exception at this point so payal has fixed it.
@yuhunglin Can we just use `class1` for the google tasks right now.
please remove the var_dump call
Please don't use general url's as references, only references to the specific technique used by the module.
fail_with() will exit the module, that doesn't look like that's what he wants.  print_error() is ok with me in this case, really.  There's never been any solid guidelines on when to use print_warning or print_error....... I added print_warning() because some modules used to modify datastore options directly.... so if it's absolutely necessary in that case, print_warning() is used to warn the user that the option is being modified.
Noted, I also switched the class assignment to be the same way.
`pixel's`, not `pixels`.
These seem like unnecessary diffs, so lets remove them before sending a pull request. The less the `ActiveMerchant` guys have to review, I think the more likely they accept.
dict.items() generate a list
dict.iteritems() return a generator which give you the value when you iterate over it.

If you don't have a good reason to use a list (eg: you need to add/remove values), you should use iteritems(), it's more memory friendly
Leaving as is, on Bryan's advice.
Children should be a constant.
Do you need the type annotations in here?
Maybe we could add some docs here to reference the official query manual
Yeah -- most people aren't.  The only reason I know is because I setup a mirror for @armstrong that had `django==1.3.1` requirements when it has to be `Django==1.3.1` for [basketweaver].  One more in the "pypi sucks" column. :-)

[basketweaver]: http://pypi.python.org/pypi/basketweaver/
I am using my fork of the project for a live site.  This is one of the changes that got committed to the pull request branch by mistake.  I need non unique titles because, for example, I have many events called 'Board Meeting' that are differentiated by their date instead of title.
what closes PrintWriter?
the route object may be of the Route class itself, not necessarily extend. if somebody does not use an OxM he can also just create new Route from symfony core. maybe we should state that here to make people not do overly complicated things.
(любой, но одинаковый)
Why is the code left in commented out? Might it be needed in the future? Might be worth leaving a comment in why it is no longer needed, or why it is left in
Actually, it is really just an implementation detail about how the internal array is filled.
The constraints are the same, and they both finish with a call to "fill". I'd rather keep the static if inside.
Hum... Apparently, this is a botched merge where I undid my previous commit which added this. This is wholly un-intentional.
redundant control flow
Incorrect `@property` attribute addition to `popBack` function.
Failed download handling for CouchPotato
remove? what is this?
These are also not available in Fred.  It looks like your master branch got rebased unexpectedly.
This should use a colon ( : ) instead of equals ( = )
I'll look into that
Look at the args for FormHistoryRecord, and set those on record creation (e.g., you're already setting the guid when you create the record, you don't need to set it again).
An IllegalArgumentException is thrown. In the error message is written what was wrong about the version string. But maybe we should enhance this error message. I will add that.
Doing things with a method here, could we instead use the language insert or merge for method since this is SQL?
```
"main": [
    "./path/to/app.js",
    "./path/to/app.css",
    "./path/to/sprite.img"
],
```

separate lines

js first
That's probably old dead code, remove remove remove ;-) good catch Al.
Gets whether the HTTP request was successful.
that looks weird
Order aus dem Context verwenden!

$order = $this->getContext()->getOrder();
auto был бы здесь как нельзя кстати, вместо const_iterator'ов и все такое.
Here we need to call postblits for all data. I assume the preexisting `dup` didn't do that, which was a bug. Is that correct?
Please add an explicit `@system:` after `extern (C):` just for clarity (all functions in here use C strings so they're unsafe).
t intentional, line before without t in fault. I'll fix it.
Ok, fine with me.
Yes, but you are using a runtime constant (from MessageHeaders) as a value for a schema attribute name

element.getAttribute(MessageHeaders.CONTENT_TYPE)

Every where else we decouple schema attributes from the runtime.
I agree, in fact i had a discussion with Gunnar about this a couple of days back. This is very different that the <i>reply-timeout </I> we have for a gateway. Not knowing what this attribute does, i would expect it to the max time this gateway should wait before returning a null response to the invoker, but it turns out to be something else.

Regarding the JPAExecutor, should i make the change and include it as part of this PR?
How about to use here just `org.springframework.integration.amqp.support.JsonAwareInboundMessageConverterTests$Foo` and DRY?
And now let me to polish and refactor `JsonAwareInboundMessageConverter`.
I'm going to make commit based on your branch
Why the change to private; why can't I use this from a subclass - given that you allow override of doTransform()
I disagree; no code in the SDK is using that property, so it is misleading to keep it in a class where it is implied to be an option, and it is bad practice to have dead code. If we need to, we can add it later.
To: comewalk

Thank you for your reply.

OK, please fix it. :-) bye, yah!
We should probably fix so that one can override things with MT instead of commenting out this line; @phatboyg?
Personally, I'd rather include this information as something like entity['icon'].
As I understand it, the information for each entity is generated in the genPOI script, which means the icon URI could be set in the main config file.
И да, функционал тут звучит не к месту, как и особенности.
Again, the namespace separator needs to be escaped with a backslash.
Now I feel incredibly dumb :p That's pretty obvious, especially after you explain it a second time. Thanks. I'll fix it later today.
the lines are not properly aligned acc to PEP008
I believe the block properties should use "copy" property attributes instead of "strong"
«с внешним миром» — лучше уточнить.
Clarified. This is a nested object, the parent one has the _id.
This has linear runtime is considered bad. A hash map is much more usable here.
This is the last issue to address before everything is ready to be merged. Looking at the WebGL Globe demos, certain details for visualization, such as color and factors to scale the height values, are left to the individual application and not included in the data set. If we want some kind of visualization for now, we can use polylines with some default values.
Yep. It only shows if the test fails, so its not a big deal. It can be helpful when diagnosing errors in a long test run.
This seems over complicated. Why is it not `value = Code.objects.get(code__iexact=value, is_active= True) `?
i mean with your additions to it that we just merged :-)
This represents a major version with new flow, new users and new db logic
It'd be a good idea to make the new column name an optional parameter to this function (you could have the default be "Self") since users will end up seeing this column name when it is used in the adiv/bdiv plots.
az inage root edtefade kon, copy kon oonja
shouldn't be required, will add if ever needed
room.now used to need to be called every 10 seconds, now it does nothing. It was replaced with presence.update
We should probably say that we primarily use our internal bug tracker though.
No, don't. That should be a separate pull.
This is pretty much the same as the ReverseGeocodeResponse?  Seems like the GeocodeAddress properties are included top level in this.
This constant is also used when creating Ti.Geolocation.Android.LocationProvider and Ti.Geolocation.Android.LocationRule objects.  This makes it sound like its only valid to use with preferredProvider
Hmm... that's strange. Are you sure you're in the correct branch? Here's the commit that it was changed in https://github.com/abledaccess/studios.fluidproject.org/commit/ee8640914ed809ebc4724a17e0553116954ca8cb#diff-3
(minor) "nPointCounter" isn't a great name for this variable, because it implies that the value is increasing or decreasing.

→ Please rename this variable to "numPoints".
Si, es un espacio dejado así por diferencias en la gramática con el inglés. En inglés va texto pero en español no
Would that make sense as e.g. @Transactional(manager = "springBean|jndiName") ?
Let's rename the function to fireWebWorksReadyEvent
It has now been moved to lib/plugins/dialog.js
From a callback, in theory no. The callback should* always include the QueryString defined in the initial request to the provider.
To keep or not to keep `.config.php`?
before atoi there is no check if argv[1] and [2] actually excist!! possible segmentation fault.
keywords localization
asserts are actually expressions. Also, you may want to put assert in code font.
`s/FMA/FMA are supported./`

There are probably more 'special' instructions / instruction set extensions worth adding here, but I cant seem to recall any off the top of my head...