labofsahil/github-event-dataset-2012 huggingface dataset:
Right here is where the BC-incompatible code smell becomes obvious: if you're _changing_ a test, likely you're breaking BC.

My recommendation is we make the changes I mentioned in the MemcachedOptions class, re-introducing addServer(), and having setServers() proxy to it.
The flag that you indicated is for Zend\Crypt\BlockCipher and it's not related with Zend\Crypt\Hmac. I prefer the constant approach because is more explicit and is class related.
A blank line must be placed after  `namespace`.
Finish this doc block and add a return description
spaces inside brackets should be removed
Me buging you: empty line between 37 and 38 :D
Appended and squashed. Thanks for the notice.
About what to pass to dataDecoder(), see my comment above about this.

The part about the foreach statement does make sense, I remember trying to do that and I ran into some issue but I will give another crack at it.
From past experience, this call must be in an exception handler since the actual command contains the database name and password:
```python
done = None
begin:
    done = subprocess.call(command)
except:
    raise Exception("ogr2ogr2 command failed (are the gdal binaries installed?)")
```
`profil` typo here :)
can you fix the intending?
Missing `{@inheritDoc}`
Please don't. It just increases code complexity (now there are two branches to maintain) for no benefit – or did I miss a behavioral difference here?
A nit, but you could just make this ```eventType```.
I wonder if we could unify / merge both testing targets...
Would you turn "contact Support" into a hyperlink to "http://ao3.org/support", kind sir? Just to make it clear for the new users?
One of the things I've been doing in other constructors is having 2 interval loops running:

1. unthrottled; that reads the data and updates any internal properties as fast as it can receive them
2. throttled; uses `this.freq` as its interval time and fires a generic "read" event everytime.

Also, the other constructors produce at minimum 2 different types of events: "read" and "change" (in this case, "change" would be "chuk" ;D ) 

1. The "read" event fires continuously
2. The "change" event is fired when the current value differs from the previous value (based on some pre-determined boundary)


Sorry, I realize I should _really_ document this stuff.
Generally, at Mozilla webdev we prefer brackets instead of backslashes. So
```
self.fields['participants'].help_text = (
  '<a href="%s" class="btn" target="_blank"><i class="icon-plus-sign"></i>'
  'New Participant</a>' % reverse('manage:participant_new'))
```
Need to describe what a lockName is. How long can the name be, does it have to be unique across services etc.
@waymirec -- Can we look at splitting this into 2 methods -- for maxSpace & spaceUsed -- and perhaps using non-String return types?
Probably. I was basing it on angles.ParallelLines, and they used this-ish. But this is better.
I took that straight from Recognizing Fractions 0.5. So, I'll bet the
problem happens there too.

Thanks, by the way!

On Mar 23, 2012, at 10:24 PM, Ben Alpert
<reply@reply.github.com>
wrote:

>> +            <var id="COLOR">COLORS[ randRange( 0, COLORS.length - 1) ]</var>
>> +            <var id="SETUP">setColors(COLOR)</var>
>> +        </div>
>> +        <div class="problems">
>> +
>> +            <!-- Draws several pie charts representing the improper fraction. -->
>> +            <div id="pie-chart-improper">
>> +                <div class="question">
>> +                    <p>What fraction of these circles is shaded <span class="colorMe"><var>COLOR</var></span>?</p>
>> +
>> +                        <div class="graphie">
>> +                            init({ range: [ [-3, 5 * TOTAL_UNITS - 2], [-3, 3] ], scale: 25 });
>> +                            drawFractionAsCircles( NUMERATOR, DENOMINATOR, COLOR, false );
>> +                        </div>
>> +                </div>
>> +                <p class="solution" data-type="improper" data-simplify="optional"><code><var>NUMERATOR / DENOMINATOR</var></code></p>
>
> You shouldn't have `<code>` tags inside the `.solution`. (This applies to all of the problem types you wrote.)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Khan/khan-exercises/pull/17326/files#r599915
nit: by repeating `[x, y]` at the end to close the triangle, the beginning/ending vertex corner is drawn as two lines butted together, rather than coming to a sharp point like the other corners. If you instead use `true` as the last element, the shape will be auto-closed with a pretty corner.
Только стоит проверить, я тесты не запускал :)
can you add a `q1.unit` as well which just returns the unit (or is the unit).
that above is from ikiwiki. it has extra markdown-like syntax. obviously it won't work here, but i left it there to encourage fixing it ;)
good idea, i've not seen this approach before :+1:
this test passes even when no results are returned
Can you justify this change please?
We just ran into this exact issue with setting config parameter via the command line in vumi. :-)
Just glancing at that header, I'd presume that :name was a fact name, not a node name.  You explain it below, so I don't feel too strongly about it.
I wonder if it's too hard to automagically add spam support and we should just stick this in a guide instead. It's actually insanely easy to add anyway
I'd format this the way we've been formating other dictionaries. It'd look neater.
It will loop on all services until one of them can be resolved.
We decided not to change this right now. Declarative events currently listen on the node and therefore do not get targets from the node's shadowDom.
Ah wacht dit is `handleEdgeDetection`, right?
Коммит с исправлениями сделал.
сделал их readonly
mutate меняет **x процентов** последовательности
hammingDistance показывает **количество** ячеек, в которых днк не сходится
Вы сперва выбираете случайным образом объект по по индексу, потом запрашиваете его индекс. Как-то странно. Или я ошибаюсь?
Задание буду еще доделывать, просто не было времени и решил отправить что было готово. Спасибо учту все описанные замечания и доделаю за выходные проект.
По условию hammingDistance считает различные, а не одинаковые символы. И сравнивать так объекты не стоит. У объекта массива есть метод isEqualTo/isNotEqualTo - нужно пользоваться одним из.
but you already transfer X restricted to the active set (both _solver functions do this restriction) and G[:, active_set] to the _solver functions. So active_set (which is defined on the full leadfield) has no real meaning in the _solver functions as it points to the full leadfield.

In the first iteration, X will be a dense matrix, but then as soon as we use init, X will be nonzero in rows corresponding to the old active set
@mdineen was this still failing? Not sure if the `xit` was missed.
we're not waiting for all of the context vars to resolve before continuing to grab and render the include'd template? this is a race condition.
Removed the ES5 specific Array.isArray and replaced it with the method used by Underscore (and afaik most other libraries) to detect Arrays.
Should make sure value isn't 2 before you send completed.
It's a pity we can only use `instancetype` for the return type of a method, and not for arguments or blocks.
@jspahrsummers add a super magic macro to [libextobjc](https://github.com/jspahrsummers/libextobjc) for that!
Neither the queue nor the label will ever change.
+ local variable :)
Is this necessary for all actions? At least ```:index``` and ```:show``` shouldn't need it?
Don't bother too much. I just dropped a note about what popped through my
head, we can just commit and be done with it for now.

I was talking about LuaJIT and it's FFI module before and I think the
migration of the input event handling to such a scheme would make a good
example case. When I find time for such a thing, I can get back to this.

Maybe we should define a path prefix, defaulting to "./" - so we have some
means to keep track on the creation of temporary files.
I didn't touch at all the part below. Maybe it's the EOL. I can't figure it out.
I like `MergeConflictException`
I would call it `append` as list.append in pure python.
The notch tests here aren't so great, because it's easier to do with synthetic data (in test_filter.py).
replace it with 
```
result.channel.item;
```
Is this a general purpose method to check for duplicate names, if yes, then you should always be case insensitive
This is actually still needed -- not actively used in OSC, but used in OPC so something needs to be there. Would be nice to clean that up, but for now, need to leave it.
In http://howtonode.org/promises I saw there is a way to promisify node.js async methods.
Bad i18n. Whole sentence needs to be internationalized.
`" should be prime"`
The zstyle comment was in reference to RVM, not the prompt char.
Don't check in this file.
typo "though"
@stof - It's just an example, I'd rather expose the interface than look for simplicity.
How about:

```
Return all args as Python integers.

In some cases a routine needs to work with integers
but it is convenient to allow the user to pass a non-integer
value or expression. In other cases, a routine may consider
passing anything but an integer-equivalent to be an error;
in this case the flag ``strict`` can be set to True and a ValueError
will be raised if all arguments cannot pass the int(arg) == arg test.

Examples
========

>>> from sympy.ntheory.residue_ntheory import int_tested
>>> from sympy import sqrt
>>> n = sqrt(10)
>>> int_tested(n)
3
>>> int_tested(n, strict=True)
Traceback (most recent call last):
...
ValueError: all arguments were not integers
```
for every reschedule attempt, which I think qualifies as a “yes”.
now imagine some very creative user attaching a logger which employs an AMQP mailbox (just to be safe, you know …)
I don't think that's something that should block that PR, just saying that it feels less readable, minor thing anyway.
Ну и нафиг оно здесь осталось? :3
Лучше даже инлайн-функции такие писать на отдельной строчке, как обычно:
```c++
bool IsEmpty () const
{
    return Content_.isEmpty ();
}
You don't need the else-part of an if-condition, so you can delete the lines 105 und 106
```python
else:
    pass
```
instead of returning json(true) or false, you should return an appropriate status code
What about a custom Cache class that either bases on OrderedDict, if present, or dict (maybe UserDict). Both providing a flush() function with a different implementation? I'll pull in your changes and can add such an implementation if you like.
OK, I have refactored using the block approach - makes sense.

On the topic of making the view available my thought was that the activity and the view are two objects that co-work: the activity is the external interface and works with the view. The activity is specific for the view and will have to know something to allow the map db switch - the way I implemented it makes it generic since the access is for the abstraction of the view so no specific Mapsforge things in the generic layer. This was however the main area where I was thinking about different ways of implementing and if you have a better structure I do not mind refactoring it. The "classic" way of using a findviewbyid is not my cup of tea since I believe that the classes should provide the access to each other.
What do you think?
I will move this line into the if-else after merging...
You are correct. But, I see you've come up with a better method now.
Thanks for the review.
How does line 20 jive with your added class? In other words, if someone uses a custom namespace for Active Admin, how does it change things?
Should the 'RESIZING' be the same id as 'REBOOTING'?
+1, changed to 3 mins.
Extract variable?
It looks to me like this method is getting too complicated, maybe we can extract the `case(ttl, idleTime)` part in 3 separate methods and only deal here with put vs replace and sync vs async?
You could do `>=3.0`
@jgehrcke You don't need to revoke it! Just add another commit to your patch-1 branch:

`git checkout patch-1`
`.. hack hack hack..
`git add axes3d.py`
`git commit`
`git push origin patch-1`

And this page will automagically update to reflect your branch's commit history.

I'd vote for either `np.tile` or `np.ones`. Let's see what @pelson and @WeatherGod have to say first, though.
Intentional (I've not been following ```trisurf```/```trisurface```)?
and here (operator spacing).
Yeah on the device you don't need them but if they are failing for you on your emulator then I will add them back because I'd prefer the cross-device compatibility. 
Are they failing on your emulator?
Done in a separate commit now!
this should say setBaz()
Should the last part of the evaluation be a strict check to false? So that for example "compass: ~" will still enable compass?
This is basically a duplicate of [`Commit#skipped?`](https://github.com/travis-ci/travis-core/blob/master/lib/travis/model/commit.rb#L11).
Optimizing qsort and bsearch is outside the scope of the transaction work, I've opened #220 for this.
AFAICT,  the uses of conn->default_session in this function can be replaced with "session", __wt_conn_cache_pool_config is passed conn->default_session as its session argument, isn't it?
Looks like something wasn't merged properly
there are now 9 fonts, so update this to 9!
100% right, this method is not being used anymore.
Let's get rid of the translation bit with this exception message.
The pref handling/detection stuff is kinda spread around everywhere - might be worth putting it all into the JSM. So the JSM would detect pref changes, determine whether this dialog should be opened, and this dialog retrieves  the relevant data from that object instead of window.arguments. And you'd get a nice separation between backend and frontend that way.
Should there be a dollar sign here before id?
There should be curly braces ('{' and '}') around the ```continue```
Use jQuery `inArray`. Does it die id `panels_forcedby[dependency]` would not exist?
What is this? Why those files are changed?
Move timestamp just bellow the `station_id`.
This is default, no?

Probably you also want to add `required` flag?
You don't have to have a anonymous wrapper function around, `hower($('#drop_down_login_options').show, $('#drop_down_login_options').hide)` already works. :-)
I thought of that before I changed it to get_or_create. But how do you check the flag, in the same way as with get_or_create (with two variables, like "station, created")?
So no need for this, you are not checking if updated_time works correctly. This we will do in other tests.
Interesting approach, but it is a bit hackish. ;-) I much more prefer metaclass approach. Here is [done for models](https://github.com/wlanslovenija/mainpage/blob/master/mainpage/wlansi/accounting/models.py#L13), but it is very similar. So you define a function which adds class attributes at creation time. Can you convert your code into it?
Yes. I think that every change of account settings should be authorised. Maybe if someone, somehow manages to get to accout settings of another user, he cant do any "damage" . Just in case...
Add a TODO comment that we should make temperature unit user-configurable.
I've read the documentation and tried block.super, but it doesn't work. <code>Error: 'Caught AttributeError while rendering: 'BlockNode' object has no attribute 'context' '</code>
Than I googled the problem and found this and I think it makes sense: 
http://stackoverflow.com/questions/6566347/django-templates-using-block-super-in-included-template-fails-exception
Because you want as descriptive errors as possible. So first problem is not password mismatch, but that field is required and nothing was entered. So we want that user gets "required field" error. Then mismatch.

BTW, do we check if username and passwords are of minimal length? :-) Do it. (Check how is done in nodewatcher - you can register your own additional validator. (Validators are in fact better then clean methods, because they are reusable.)) (Thanks for reminding me.)
What do you mean? On what? I am commenting what all you have yet to fix with this pull request.
We have users, accounts and profiles. Do we really need three?
fixed in f0d248b
I'd make it this: ``` if (empty($config['asset_manager']['paths']) === false) { ``` to ensure it's not empty (doesn't require an isset first).
Given what we're up to here, it seems like we might need to guard this construction (as you did with the reflection).
you don't need to create a method for this. You could use ``new \Twig_Filter_Function('SqlFormatter::format')``
I have no clue what this `uri` is for. Also, a .jsm will generally export only one symbol which will act as a namespace. Perhaps `Promise`?
Wrong update. In 2.1, simply skip the step as it will already be registered when using Composer
`sel +=`?
Move this to the header filter? The header filter is called only once at the beginning.
The one in this file, NgxUrlAsyncFetcher::Init().
`Ship::AIMessage` clears the old message, so you need to capture and restore it.

This looks more like a bugfix than something to do with fuel economy. Can you describe it?
I think we need a check here to see if `more` was actually received as the message body?
the anonymous function doesnt work in zsh 4.2.6, with it removed it works on zsh 4.3.11 and 4.2.6 with or without EXTENDED_GLOB AFAICS.
I don't know if it matters here (in terms of extra work)

In general, I would prefer setting an attribute on the results instance (in L1 fit) with ``self.use_nan_dot=True`` to automatic checking for an "exceptional" case.
yes, I wasn't sure what to do with that. It seemed out of place.
e5ecc3d
Yeah, that's probably a good idea.  Most likely we won't be adding any new classes under `UnicodeData` so it makes sense to remove it.
Oops. Saying it that way is more like the norm here in India. :smiley:
Will make the change.
why not typehinting Selectable here ?
Yup, it's inspired by `jQuery.ajax` which accepts a string like this. Thanks so much for your patches kind sir!
Near as I can tell, just the ad component. Testing for the presence of that doesn't feel a whole lot better than testing for the absence of the spinner, but maybe that's just me.
I could drop the GAV class from here, yet we should still have to make this class filter out md5/sha1 maven files.
So I could just check that paths end with .m5/.sha1. If you think is less tight to maven I can do that.

Another. more complex alternative is the way we discuss by phone Tamas to have an IsVisibleInFeeds attribute that divers repos could implement specific to their content and do filtering baed on that.

Shall I go that route? I just did not do it to not spend more time on it. Timeline is anyhow tiden to maven repo format already and we may delay this till we re-implement timeline as a journal.
Для булевых переменных надо использовать значения true, false.
I agree, although it doesn't fit strictly with the show/do-not-show rules I'd adopted I think that in this case it makes sense.  Will push and inform.
Re the linear / const approximation issue, a key thing here is that I'm not actually attempting to approximate the function over the interval to any great precision.  The goal is just to divide the domain into regions small enough that either (a) rejection sampling will be efficient, or (b) I'm willing to spit out photons whose flux varies by the amount that the function varies across the interval.  I don't see much downside, for example, to having photons whose flux varies by +-10% from the nominal value (=the current default).  
In either case (rejection, or varying fluxes), the original function is evaluated at the location of every photon, and the expectation of the photons matches the original function without any approximation, even thought the constant approximation over an interval is quite coarse.
Again, I think I explain this earlier.  Probably demo2.
Well, even now we haven't tested accuracy with Linear - it will depend on how the grid spacing is chosen compared to the scales of interest.  So I think that should be made clear (that's obviously high priority for the follow-up issue on the lensing engine).
Do we want to assume that every user will read every demo in detail?
rename `pattern` to `sequence` (pattern makes me think we are using regexp pattern matching)
Could this be cleaner with:

while (true) {
    NodeMetadata m = claim(1, t).iterator().next();
    // TODO ideally shouldn't have to rebind
    SshMachineLocation result = null;
    try {
        result = toSshMachineLocation( m );
    } catch (Exception e) {
        if (previous.contains(m)) {
            log.debug("could not bind to previous existing machine {}; blacklisting and trying a new one", m);
            addToBlacklist(new MachineSet(m));
        } else {
            log.warn("attempt to bind to machine "+m+" failed", e);
            throw Throwables.propagate(e);
        }
    }
}
Interesting single-line field declaration with javadoc: does the javadoc tool use this javadoc for both fields, I wonder...
again i think it's okay
see javadoc at head of file.  the "claim" method does a loop to solve this problem.
replace(/[-]+/g, '-') will work better (for example if you have --- your replacement will leave --)
Naw, I'd rather have the discussion. If I care strongly about something you'll hear about it :)
Parenthesis should have no spaces after `(` and before  `)`. This should be: `save_token(token)`
Maybe something like this?

```ruby
def login
  with_load_wheel do
    response = send_login_request
  end

  save_token_or_display_error(response)
  load_leagues
end

private

def save_token_or_display_error(response)
  if response[:token].present?
    save_token(response[:token])
  elsif
    error_with(response[:message])
  else
    error_with('Oh shitt...')
  end
end

def error_with(error)
  @error_text.text = error
  @error_text.hidden = false
  whatever
end

def send_login_request
  make the request
  return a hash
end

# maybe this and the containing functions can be put elsewhere to be reused.
def with_load_wheel
  display_load_wheel
  yield
  hide_load_wheel
end
```
In order to get this out the door, we can add this as an "Issue" and fix it later.
I'd like to keep javascript in a clean namespace (see https://github.com/tangentlabs/django-oscar/blob/master/oscar/static/oscar/js/oscar/dashboard.js#L13).  Can you rework this js so it sits within the ``oscar.dashboard`` object?
not needed any more :)
Do an early return:
      
         if (request.result.length == 0)
           return;
I am pretty sure I read about this years ago, but I can't find it so I'll take credit ;)
we should use triple equals too. double will fail jslint (even tho these are only tests... i know)
My mistake.  updated it back to original.
This line was a fix for runtimes.  Revert it. >:(
Этот код у тебя был в XmlBinder. Надо вынести в общий класс (copy-paste)
Да, но зачем нужен синхронизированный список? В него кто-то может писать из другого потока?
В Shell и ParallelSort надо использовать один и тот же Closer.
Changing from tabs to spaces (as discussed with the team)
@yuhunglin it seems to be doing the right thing for us out of the box and also enforces us to only keep files that we need on every page in `app/assets/javascripts`. For instance I've put the tests in here first, but that didn't make any sense looking back at it.

What about:

```ruby
require ./angular/calcentral
require_tree .
```
This way I think both our needs should be met ;).
I'll follow up on this in the PR.
But 99?
Isn't there something like Number.MAX_VALUE ?
please import the appropriate library to this exception
this doesn't wait it just returns a bool
Can you comment here to explain why we have different paths for MockUser vs string? Also, I don't see any uses with MockUser...
Ugh. Why are you mixing traditional QGraphicsScene items/layouts with QML? It technically works, but it's messy, painful, and will cause major pain when switching to Qt5.
Lacking a whitespace :)
On Sun, Dec 30, 2012 at 1:14 PM, Vlad Niculae <notifications@github.com>wrote:

> This seems to be done slightly differently than the usual minibatches in
> scikit-learn. I wonder if it affects the speed or anything. Anyway since
> these minibatches are indirectly implied by the n_particles, I think I
> should read the paper first.
>
I haven't looked at this in a while and I'm not sure what stopping
criterion Yann is using but more particles will mean you have a better/more
stable estimate of the negative phase term in the gradient.

> I wonder if it's just me or if it's actually confusing: I ran the RBM
> before reading the code and noticed that the higher n_particles is, the
> quicker it is, whereas my ignorant intuition was that "using more stuff
> should take more time". I commented once before about that parameter and I
> still argue that it's a particularity of the learning algorithm and not
> directly of RBMs themselves, so I wouldn't mind having its effect
> intuitively explained.
>
The number of particles is or can be effectively independent of the
minibatch size if you're doing stochastic maximum likelihood a.k.a.
"persistent contrastive divergence".

The basic idea is this: the gradient of the log likelihood of an RBM
comprises two terms, one tractable and the other intractable because it
involves the gradient of the partition function (i.e. the sum of the
exponentiated energies of all configurations). We need to approximate this
with a sample, but sampling is also intractable, and running a Markov chain
to convergence at each step of the learning is hopeless. Geoff Hinton's
contrastive divergence trick is to run the chain for 1 step and hope for
the best, and in practice this makes for pretty good feature extractors and
lousy density models.

In the late 90s Laurent Younes showed that assuming your model parameters
don't change too quickly (i.e., your learning rate is low) it is
theoretically sound to keep samples from a "previous version" of your model
(i.e. the model before the last parameter update) and treat them as if they
were samples under the "current version" of your model, and thus keep a
"persistent" Markov chain going throughout learning even though the
distribution from which this Markov chain is sampling is constantly
changing. In this way you get approximate but unbiased gradients of the log
likelihood, and in practice this leads to much better density models as
these samples manage to explore a much bigger region of the state space
than is the case with the 1-step CD hack.
Sorry not the residuals, the MSE.
I think that I'd call this function using a verb, as in 'discretize'. Also, I think that I'd prefer the name of the input argument to be 'vectors', rather than 'eigen_vec', as i) there are several vectors, so a plural is better, ii) it could be applied to other vectors than eigen vectors.
контент блоків на один таб вправо
It's awesome how you now return the string to where the method was called and output it there, instead of outputting it in the dog. And how you also renamed the method from `introduce` to `introduction`. That makes so much sense!

But it also seems to me that gender, age and look are things that are "characteristics" of a particular dog (instance), right? So maybe you'd rather want to pass these characteristics in to the `new`/`initialize` method and then keep them around using instance variables?

Because by passing in the characteristics to the `introducution` method (the way it is now) one could have entirely different introductions from the same dog:

```ruby
charly = Dog.new("charly")

puts charly.introduction("male", 7, "hairy", "clumsy", "furious", "old")
# aha, charly is a  years old, hairy, clumsy, furious, old dog. cool!

puts charly.introduction("female", 2, "nosy", "fast", "playful")
# err, wut?! now charly suddenly is female, 2 years old, nosy, fast, playful? how did he change like that so quickly?
```

In other words `introduction` would rather look at the existing, stored, remembered characteristics of the dog (which couldn't be changed by someone else just so) ... i.e. in instance variables.

Does that make sense?
Remove the commented lines please.
This has several problems:

1) What is the point of this dummy property? If this is never mean't to be disabled, why is it in a profile and not in the main pom?
2) As this is active by default it breaks the build for all non Red Hat employee's who don't have VPN access (or anyone using the mac VPN client)
3) Do we really want the JBoss QA repo hardcoded into our pom's ? Is there any reason why this artifact can't be deployed to the main nexus?
Ah.  I hadn't realized you were translating my object properties to local variables.  So, this is my workaround: I'll wrap my data models like this: { m: { goodbye: 'world' } }, and access it like this: #{m.hello || m.goodbye}

Works like a charm.  Thanks!
We should have a helper function for determining whether guest authors is enabled. Noting to update later.
Can't you just use `obj->objectFactory_`? then you won't need `getNativeObjectFactory()`
Label before Listview.
btw, you don't need the memset, if you use OwnedMalloc(... ,true)
package_names is still references in a couple of other places in this file. This gives rather confusing errors like:

Could not find dependency Package[undef] for Nova_config[lock_path] at /home/jenkins/puppetlabs-openstack/modules/nova/manifests/init.pp:127

...because Nova_config gets a: require +> Package[$::nova::params::package_names],
The logic is a bit more complex, and due to pathname not handling things quite the same as glob the implementation was getting at least as complex and what's already there, so it doesn't seem worthwhile to me to reimplement this block right now.
Removed the ZF namespace.
It's not about offending me or not, every Symfony2 open source bundle needs to follow those standards :)
````
if (is_array($parameters))  {
    return $this->twitter->post('statuses/update', $parameters);
} else if (is_string($parameters))  {
    return $this->twitter->post('statuses/update', array('status' => $parameters));
} else {
    throw new \InvalidArgumentException(sprintf('Parameter must be a string or an array ("%s" given).', gettype($parameters)));
}
````
If you add a column to the table, you'd have to specify that when initializing the table (i.e., `fmtxt.Table('lll')`). Also, I am not sure how the tuple will look - it might be simpler to submit a string directly, e.g., `c_start+' - '+c_stop`
@ctweney let's just take out the oAuthRestTemplate stuff for google if we don't need it.
After all, seven ate nine!
Maybe call it "print immediately"? or something more descriptive
Would be useful to 'report' it? maybe as a note?
So, I am curious why you implemented it to use GSA instead of OMP. OMP is more likely to be running than GSA. OMP should be faster as well, is tcp/xml and not http. Just a thought, willing to be wrong too.
Warning! the rport method has been deleted from the module, so I'm getting the next exception when using the new version of this module:

<pre>
msf  auxiliary(pcanywhere_udp) > set RHOSTS 192.168.1.147
RHOSTS => 192.168.1.147
msf  auxiliary(pcanywhere_udp) > rexploit
[*] Reloading module...

[*] Sending pcAnywhere discovery requests to 192.168.1.147->192.168.1.147 (1 hosts)
[-] Auxiliary failed: NameError undefined local variable or method `rport' for #<Msf::Modules::Mod617578696c696172792f7363616e6e65722f7063616e7977686572652f7063616e7977686572655f756470::Metasploit3:0x007fc49ef374c8>
[-] Call stack:
[-]   /Users/juan/Projects/metasploit-framework/modules/auxiliary/scanner/pcanywhere/pcanywhere_udp.rb:67:in `block in scanner_postscan'
[-]   /Users/juan/Projects/metasploit-framework/modules/auxiliary/scanner/pcanywhere/pcanywhere_udp.rb:51:in `each'
[-]   /Users/juan/Projects/metasploit-framework/modules/auxiliary/scanner/pcanywhere/pcanywhere_udp.rb:51:in `scanner_postscan'
[-]   /Users/juan/Projects/metasploit-framework/lib/msf/core/auxiliary/udp_scanner.rb:69:in `run_batch'
[-]   /Users/juan/Projects/metasploit-framework/lib/msf/core/auxiliary/scanner.rb:164:in `block in run'
[-]   /Users/juan/Projects/metasploit-framework/lib/msf/core/thread_manager.rb:100:in `call'
[-]   /Users/juan/Projects/metasploit-framework/lib/msf/core/thread_manager.rb:100:in `block in spawn'
[*] Auxiliary module execution completed
</pre>
This was merged "a month ago" so... =)
I agree, I can add this easy enough, since @bagnell has more important things to work on.
Whoops, that should totally be minutes and not seconds in that first while loop.
Ah, I understand what you're saying, now.  I suppose you could only do the replace on the things you know to potentially have a comma, right before converting it to a number with +, but I doub that's any better than what you're already doing.  So nevermind!
`height` is optional, so '@param {double} [height=0.0]`.  Add this to `setPositions` too, please, if it isn't already.  See http://code.google.com/p/jsdoc-toolkit/wiki/TagParam.
the name **select** conveys the responsibility of it better than **filter**. Regardless, It may make sense to use select as an alias for filter; i.e., without using partition...
This isn't exactly true in mobile web. For us, "auto" == Ti.UI.SIZE, meaning auto will always fit to its contents
Should set this apart with a heading like the Android section:

    #### iOS Platform Implementation Notes
This test fails on my end, even though my PYTHONPATH is set correctly.

```
EasyBuildError: 'EasyBuild crashed with an error (at easybuild/framework/easyblock.py:1139 in extensions_step): Failed to use both class EB_ext1 and default Extension for extension ext1, giving up:\nNo module named extension\nNo module named extension'
```

```bash
[kehoste@ca60c171 easybuild-framework]$ echo $PYTHONPATH
/home/kehoste/Work/aptana_workspace/easybuild-framework
[kehoste@ca60c171 easybuild-framework]$ ls easybuild/framework/extension.py
```

You had the same issue, right? What's the problem here? Maybe you should spit out a suggestion on how to fix this if this test fails?
It looks like postgres doesn't want to use indices when you change the line like that:

```
planet=# explain SELECT r.id, r.name, r.intnames FROM routes r, relation_members m
WHERE m.relation_id = 1107386 AND
r.id IN
(SELECT child FROM hierarchy
 WHERE parent = m.relation_id AND depth = 2)
AND m.member_id = r.id
ORDER BY m.sequence_id;
                                                      QUERY PLAN                                                      
----------------------------------------------------------------------------------------------------------------------
 Sort  (cost=11294.79..11294.80 rows=1 width=33)
   Sort Key: m.sequence_id
   ->  Hash Join  (cost=8998.44..11294.78 rows=1 width=33)
         Hash Cond: (m.member_id = r.id)
         Join Filter: (SubPlan 1)
         ->  Index Scan using idx_relation_member_relid on relation_members m  (cost=0.00..1100.92 rows=316 width=20)
               Index Cond: (relation_id = 1107386)
         ->  Hash  (cost=8552.64..8552.64 rows=35664 width=29)
               ->  Seq Scan on routes r  (cost=0.00..8552.64 rows=35664 width=29)
         SubPlan 1
           ->  Seq Scan on hierarchy  (cost=0.00..794.02 rows=1 width=8)
                 Filter: ((parent = m.relation_id) AND (depth = 2))
(12 rows)
```

Using the relation ID directly, it looks better:

```
planet=# explain SELECT r.id, r.name, r.intnames FROM routes r, relation_members m
WHERE m.relation_id = 1107386 AND
r.id IN
(SELECT child FROM hierarchy
 WHERE parent = 1107386 AND depth = 2)
AND m.member_id = r.id
ORDER BY m.sequence_id;
                                                         QUERY PLAN                                                         
----------------------------------------------------------------------------------------------------------------------------
 Sort  (cost=1904.46..1904.46 rows=1 width=33)
   Sort Key: m.sequence_id
   ->  Nested Loop  (cost=794.05..1904.45 rows=1 width=33)
         ->  Hash Join  (cost=794.05..1896.16 rows=1 width=20)
               Hash Cond: (m.member_id = hierarchy.child)
               ->  Index Scan using idx_relation_member_relid on relation_members m  (cost=0.00..1100.92 rows=316 width=12)
                     Index Cond: (relation_id = 1107386)
               ->  Hash  (cost=794.04..794.04 rows=1 width=8)
                     ->  HashAggregate  (cost=794.03..794.04 rows=1 width=8)
                           ->  Seq Scan on hierarchy  (cost=0.00..794.02 rows=1 width=8)
                                 Filter: ((parent = 1107386) AND (depth = 2))
         ->  Index Scan using routes_pkey on routes r  (cost=0.00..8.27 rows=1 width=29)
               Index Cond: (id = m.member_id)
(13 rows)
```
For me the second then runs about as fast as the one without the join. Could you confirm that on your setup?
Rather than let `$recipients` (the class property) be a Collection or array, I think it'd be easier to ensure it's always an array. In this setter method, you can check that the argument is either an array or Traversable and use [iterator_to_array()](http://php.net/iterator_to_array) for the latter.
Why are you continuing if the response code is not 200? Should return right at line 18 instead, no?
Nitpick: The assert was inside a foreach, so the indentation was actually correct... Unless I missed something.
Should be "it will assert"?
> As far as I know, typeid(T).init() can never return null.

**This** is correct. It's not null. )))

> Worth noting that array copy syntax could be used here...

No it can't.

 You have to remember this hard as it's tricky:
```D
struct S
{ int i; }

void main()
{
    // yes, all asserts passes
    auto init = typeid(S).init();
    assert(init !is null);
    assert(init.ptr is null);
    assert(init.length is 4);
}
`
Add few more lines.

P.S. You need to checkout your branch, commit your new changes, push it again. You don't need to resend your pull request, but in order to see a new notification I need you to add some comment (e.g. Fixed).
These could all be `final`, too, and I hate `private` like the plague.
This no worky. We're using Smarty 2 not 3.
You could put this import line along with others a few lines above.
You'll also need to implement the clone() operator - to call createDictionary
And you already do that ... i should look at commits before talk ... thanks
Any reason you btoa here?
How about declaring `alias ULONG_PTR DWORD_PTR` somewhere at the [top](https://github.com/D-Programming-Language/druntime/blob/master/src/core/sys/windows/windows.d#L97) so that we can use DWORD_PTR for dwActiveProcessorMask as [documented](http://msdn.microsoft.com/en-us/library/windows/desktop/ms724958\(v=vs.85\).aspx)?
Thank you, Gary. I understand your opinion. And it will cool, if others will make comments too.
Nevertheless I've got an important argument to allow to use `request-handler-advice-chain` for gateways within the `<chain>`:
there are a lot of chains around outbound-gateways in my projects. It is some pre-procesing before gateway invocation and post-processing after invocation. And it is real **black box** for clients of endpoins for external services.
So, I'll be happy if I can use for my existing outbound-gateways **Retry** or **CircuitBreaker**. And I'll avoid some forced additional reconfiguration of my applications.
There is +1 vote for chains with gateways: http://forum.springsource.org/showthread.php?125521 ;-).
However I understand that it looks very noisy for current minor release and we may push this improvement for further discussions into some separate JIRA. 
To avoid XML complexity I suggest to introduce custom tags:
`<retry>`
`<circuit-breaker>`
`<expression-post-process>`

So, let close this PR!
I'll simplify the commit only with bug fixes. 
But then, Gary, you should add a note into Manual, that `request-handler-advice-chain` will ignore inside the `<chain>`...
BTW, once the above comment is addressed its LGTM so it could be merged
retrived by "list message" or dequeue operation?
Shouldn't every element implement this Abstract class?
But it really should never be empty here! Anyway, fixed.


On 29. 09. 12, at 12:45, Mitar <notifications@github.com> wrote:

> In tests/test_project/test_app/static/test_app/js/datastream.js:
> 
> > +                var h, i = 0,
> > +                    j = arr.length;
> > +
> > +                while (i < j) {
> > +                    h = Math.floor((i + j) / 2);
> > +                    if (val > arr[h][0]) {
> > +                        i = h + 1;
> > +                    } else {
> > +                        j = h;
> > +                    }
> > +                }
> > +                return i;
> > +            }
> > +
> > +            if (!collection || !$.isArray(collection) || collection.length === 0) {
> > +                throw new Error('WTF: collection should never be null or empty here.');
> Ehm, no cursing please.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
too many "=" signs -- should be same length as preceding text.
I think we can ommit zf2-tutorial mention in the path since is common for all files
I think there is code duplication this way?
Definitely a separate PR after docs release.
Once my PR lands, check for `msg.messages[i].message`
Errant print.
How about a "Client" subnamespace?
`memo[memo.length]` is generally used as a [micro optimization](http://jsperf.com/push-vs-leng#chart=bar) *(though the results seem less than stellar)*.
Would you move the PHPDoc class comments...
This needs a `.html_safe` after the `.to_json` or the javascript code gets HTML escaped. The same will apply to the other one.
The same issue applies to line 720 of this file, I think. copy() only does a shallow copy, and if it fails, it leaves the expression tree in a corrupted state.
Now we're supporting all different sorts of locations. The `switch` makes sure, we can easily add in additional cases. Maybe we should also add an filter?
Maybe add a comment like:
// Our sync key is now associated with TwoWeeksBack, save ourselves an invalidation roundtrip by zeroing it ourselves if we changed filter type.
As a dev, i'm always wondering if these kinds of lookups return null or throw if the key doesn't exist.  (I guess if it threw then there would be a "throws..." specified, but still it'd be nice to say "the property object or null if a property for the given key does not exist."
This comes here from `management/src/main/scala/com/gu/management/metrics.scala` where it didn't belong.
could you make the class declaration static?
Does this ever get deleted?
need to call bps_shutdown();
You take twice
(* x -1) = (- x)
why not add the parent key in the delete multi?