labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, because $this return the client service. if i call __call like yours,  i will get error : "Function ("_call") is not a valid method for this service"
with code i've change, we can call  

    $client = new \Zend\Soap\Client($this->_WSDL_URI);
    $client->call('functionofservice', array());
This imports too and the rest of the file
@spicyj I believe you, that's probably a good idea for those who work on exercises. Everybody needs a local version of app engine running but not everybody is on the exercises team.
Are these required? Doesn't inherits take care of any inheritance issues?
i thought you wanted this to be boolean. i have a dom representation (well, hashbang and hrefs as active/completed) passed on, the models now use a true boolean. this maps the value from that into the collection auto filter i setup.
and then what? :)
This class should extend https://github.com/FriendsOfSymfony/FOSUserBundle/blob/master/Security/EmailUserProvider.php now instead of calling the provider methods on the UserManager (which should be considered as deprecated)
index.php does not exist - rule not needed
What to do if we want to write in the file string "Øl trinken beim Besäufnis" and not "&#216;l trinken beim Bes&#228;ufnis"?
If we write 
<pre># -*- coding: utf-8 -*-
from codecs import open
f = open('test.log', 'w', 'utf-8')
f.write( "Øl trinken beim Besäufnis".decode('utf-8') )</pre>
it works. But if we add <pre>from __future__ import unicode_literals</pre>, it doesn't. So I think we don't need encode.

And is strange that when code is run in Pydev, it has not UnicodeEncodeError with not ASCII.
I'll open a bug. Already had a quick look at it, but couldn't figure it out. Will have a look into later this week.
added sanitychecks as needed.
It's default value for Buzz. https://github.com/kriswallsmith/Buzz/blob/master/lib/Buzz/Client/AbstractClient.php#L7-L10
Hi, I've removed them.
nit: spacing between > and h3
looks good, does it not work?
also whitespace around array index
this looks like it could be removed.
Any reason you're using `>= 0.2.2` instead of `~> 0.5`? I highly recommend reading [this blog post](http://yehudakatz.com/2010/08/21/using-considered-harmful-or-whats-wrong-with/).

Also, have you verified that this upgrade does not cause any UI issues?
I like that, thank you!
Why the exception for force?
you're right - the detect kicks in if there is an article on the page (those have links directly to the resource) so I didn't notice it, but should of course fix it - so don't merge at this point.
If this is a standard Rails entity, then I would say we can easily leave it.  Otherwise I'd opt for a simple "yield :app_name" or "yield :logo_app_name"
I'll check it out, but notice that this is mostly identical to the
compare function in naming, formatting etc.  Was just following the
style in the file.u

On Tue, May 15, 2012 at 4:46 AM, Karl Skomski
<reply@reply.github.com>
wrote:
>> + **/
>> +Snapshot.prototype.showChange = function (other, usestring)
>> +{
>> +    var my_objects = this.detailedNodeCounts(usestring),
>> +    their_objects = other.detailedNodeCounts(usestring),
>> +    diff = {}, i, k, my_val, their_val;
>> +    all_keys = Object.keys(my_objects).concat(Object.keys(their_objects)); //has dupes, oh well
>> +    for (i = 0; i < all_keys.length; i++) {
>> +     k = all_keys[i];
>> +     my_val = my_objects[k] || 0;
>> +     their_val = their_objects[k] || 0;
>> +     var delta = their_val - my_val;
>> +     if (delta != 0)
>> +         diff[k] = delta;
>> +    }
>> +    return diff;
>
> Please follow this styleguide: http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml. Formatting, Naming etc.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Skomski/v8-profiler/pull/2/files#r821916



-- 
Seth Copen Goldstein
FlashGroup: Connecting people through intention
412-818-1192
因为webjs默认生成的代码是采用Global Controller的，默认不生成服务器，要用run来启动一个服务器。当然也可以用 var server = web.create()来创建服务器，这个和express是一样的。
in -> into
you -> your
Because this is 1 line, I would say that this should just use addHeaderItem like you are doing with the js, it avoids another request.
It looks like a good way to me to warn the developer he's not using the API correctly. How would you have done this?
Ну да. Потому что есть и конкретные вектора - отсортированный и нет.
UnDir? Это два слова?
Oh, bummer. I guess there's no way to tell if the file's binary in the first place without some other gem.
nth-child isn't flakey, other than the fact that it doesn't work on IE.
but using a CSS selector is not going to improve the reliability of the test, since the reason the test is unreliable is because of the NativeEvents.
approx 9 times out of 10, for me on my winxp ff12 selenium 2.22 server, attempts to get at things inside of hover menus (be it verifying if an option exist or clicking on an option) returns or does nothing. (no errors, just a test that fails further along because a previous step lied about having clicked or whatnot).
I think it's the wrong place to put it. A project's `.gitignore` is for itself, not for dealing with the eccentricities of the project's developer(s)' operating system(s).
does this only apply to small-bubbles?
A friendlier Joke
It's used in a similar fashion. But means that all the method calls on the specified object will be direct during the next invocation.
I'm using a ternary operation to make sure that a correct type is passed in and that the default values work correctly.
Type is off
Cool. Then merge it in, this is good stuff!
soo scarrrryy.. you rebel. riding your ducati, etc.
Done. 95667fb
Omdat ik net even wat meer pixels onder vrij nodig had. niet iets wat we direct weer kunnen aanpassen.
Удобный шорткат в данном случае:
NSMutableArray *DNA = [NSMutableArray arrayWithCapacity:DNA_SIZE];
Большое спасибо, действительно попроще будет =)
Объявляя свойство таким образом, нужно в любом случае переписывать сеттер, иначе можно изменить массив, а другой метод об этом не будет знать (у вас везде идёт сравнение с MAX_ITEMS);

Например, если перед вызовом `NSLog(@"\tbefore mutate:%i", [cell1 hammingDistance:cell2]);` в main.m изменить ДНК в cell1:

`NSMutableArray *inject = [[NSMutableArray alloc] init];
[cell1 setDNA:inject];`

То будет ошибка, и программа дальше ничего делать не будет.

Также можно сделать свойство доступным только для чтения, тогда создастся только геттер.
@mvmn а между прочим, @Senior-Student дело говорит! Вот что написал @freetonik (автор курса):

"В ТЗ указаны именно проценты, а то, что символов 100 – просто удачное совпадение."
Не согласен. Специально была выбрана длина массива 100 и в задании написано "  ЗАМЕНЯЕТ СТРОГО X процентов" . Как заполнить сторго 73% из 9 ячеек ? :) Впрочем не вопрос - поменяю на проценты.
для хранения целых числе в множестве больше подошёл бы класс NSNumber
Сравнивать NSString'и следует так http://stackoverflow.com/questions/3703554/understanding-nsstring-comparison-in-objective-c
Т.е. в вашей реализации ошибок не будет только благодаря одинаковым инициализациям символов через @"", но сравнение != потенциально ошибочное.
Да, везде не нужно.
![](http://99problems.org/wp-content/uploads/2010/04/obama-yes-we-can.jpg)
We should also test the `immediateScheduler`.
I think you can do something like `karma += articles.map { |a| a.vote_count }.reduce(&:+)`

check out the enumerable functions- http://ruby-doc.org/core-1.9.3/Enumerable.html
It’s not necessary to implement this method as the default already does the right thing.

Your implementation does not work correctly, though: it contains the host name in the URL [which is “Bad”](https://docs.djangoproject.com/en/1.3/ref/contrib/sitemaps/#django.contrib.sitemaps.Sitemap.location).
There were a number of places we had been sloppy about specifying the setno (whoops!). Now it throws an error, as designed, if one isn't specified.
Isn't this the default?  Is this needed?
The correct way would to create another git commit, addressing any outstanding issues, and this will run it through the tests, and add the new commit to the pull request.

In general, creating a local branch to make changes on is recommended, as this allows you to make your changes irrespective of other's, making merging less of an issue. See [here](https://github.com/opsschool/curriculum#ops-school-curriculum) and [here](https://github.com/opsschool/curriculum/pull/38#issuecomment-11273446) for more.
I disagree with making opinionated statements in a curriculum subject matter.

We should present factual data, and explain the pros and cons of each.

Session affinity is sometimes required with stateful applications, especially ones that may load a user's session to a given node, and expect subsequent calls to go to that node. Not the best architectural design, but hey, sometimes Java is that way.
pkg/pe/rpm ?
may be worth using `@` for force pattern match (so that `head` and `tail` do not fail and case drops to second pattern match)
Are you sure? Neither my projects nor the symfony standard edition loads it from 2.1 on:
https://github.com/symfony/symfony-standard/blob/2.1/web/app_dev.php
-1 on the current version; it sounds strange.
+1 to the suggested one :)
设置 <link id="map"/> 变量哈稀变量表<value>大小</value>的上限。
Should this be added back in?
-ly suffix on these 3.
nit: This looks like a funky whitespace change.
It is not TS but it wasn't intended to be as it is used in the extension only, which I am assuming is TS.
Removed this as the logic now belongs to `Vim` but we should probably test that the `Server#initialize` deals with `:gui` and `:vim_path` appropriately.
when deleting, Repos, Errata, Packages I am passing a second key in 'content ', but when deleting Products, we don't require the second key in the content.

(and (contains? content (-> content keys second)) deletion
      (doseq [prod-item (content category)]
        (browser click (locators/select-product prod-item))
        (browser click (keyword (str "select-" (-> content keys second name))))
        (if (contains? content :errata) (browser click :select-errata-all))
        (doseq [item (map :name (content (-> content keys second)))]
          (browser click (locators/promotion-add-content-item item)))))

The above code would evaluate to the below stuff, if the data is :repos 

(and (contains? content :repos) deletion
      (doseq [prod-item (content category)]
        (browser click (locators/select-product prod-item))
        (browser click :select-repos)
        (if (contains? content :errata) (browser click :select-errata-all))
        (doseq [item (map :name (content :repos))]
          (browser click (locators/promotion-add-content-item item)))))


In the above example :repos could be :packages or :errata, did'nt feel like repeating it for :packages and :errata.
I couldn't get this to work? I keep getting wrong number of arguments (1 for 0) for sort. I checked the docs but I'm not sure what's exactly happening with (&:ranking) - I'm assuming it's a proc which is basically just a block of code and you're passing in the ranking attribute for each element. But as far as I can tell from the docs, sort either takes no arguments or it takes a block with two local variables?
Oh! forgot to change it. Will do it.
Maybe  stick this in a helper function to handle nulls on a JSONElement? Not high priority but would tidy up the code
check with state discution
Shouldn't it be "Epic Feature" ?
There's no reason to get rid of it if it's useful, just that AFAIK it's identical to a call to rhythm().
I solve this problem this way:
```ruby
attrs << "primary key" if col.name.to_sym == klass.primary_key.try(:to_sym)
```
I've updated the logic per your suggestion, let me know if that's what you were wanting.
just simpler... exception by default will be rd... don't need to add rd or reddwarf to everything... we know it's reddwarf... it's not something else... the same applies for the conf... I'd like to get rid of all the reddwarf_* in the conf... just call it as is
I'd prefer the clause backwards. value == truth
Actually I am using only the setThreadContextClassLoader (which is not in SysPropertyActions). Also, this class enforces package security (it cannot be invoked from other packages).
No, don't do that either - just don't use an interface for DistributedTaskExecutionPolicy.  Make it an enum instead.
Right, this might solve https://issues.jboss.org/browse/ISPN-2581. Will roll back this change for now and issue another pull request including your fix.
The wrong method was called, but this bug is now fixed on master by latest commits. So please ignore this. I'll remove the comment.
For now I changed it to `return Immutables.immutableListWrap(segmentOwners[segmentId]);`
If this still proves to be a problem when we get to profile the code we can improve it further by making _segmentOwners_ an array of immutable lists so we do not have to create a new one per invocation.
Just quit, without proper warning or message to the user? What is the advantage of this over reading the GitList requirements in README?
This is wrong for 2 reasons:

- you may want to apply some filters even in debug mode (for instance a cssimport filter), which would then be ignored by your change.
- source assets can be outside your web root, which means they cannot be accessed directly.

So such change makes too much assumptions about the way the user uses Assetic
https://github.com/kriswallsmith/assetic/blob/master/tests/Assetic/Test/Extension/Twig/AsseticExtensionTest.php#L219
I've opened a ticket: https://github.com/matplotlib/matplotlib/issues/1465
Parent element. Until we finalise the page region page solution you were working on.

btw I tried putting the parent_element at the end of the function but it didn't seem to like it.
don't you like the css selector written by me?!:)
Will do!
I removed it. Was for a different approach
I don´t think so and already wondered myself. Downloaded just seconds ago from http://dev.openengsb.org/resources/eclipse/formatter.xml - <setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="80"/>
should this be loaded instead of loaded?
oups, I should have seen it. sorry for the noise
Hey-- not sure if I'm missing something here -- , but preconditions dont need to be have a value of 'true' , they just need to be truthy.  If you do (#{"vcf" "hansen"} "vcf"), it will return "vcf" since its in the set (otherwise nil). It doesn't actually have to be 'true' to pass the precondition. The function true?, on the other hand, requires a value of true.
user> (defn foo
          [dataset]
        {:pre [(#{"hansen" "vcf"} dataset)]}
        "it passed.")
#'user/foo
user> (foo "vcf")
"it passed."
user> (foo "somethingelse")
; Evaluation aborted.
@jsloane This probably has something to do with line endings / EOF characters in windows.
https://help.github.com/articles/dealing-with-line-endings

Maybe try a different editor to see whether it makes a difference?
Both of these methods really seem to belong in your handy new AbstractContentSearchQueryHandler class. I assume you broke them out here so as to let them be used by the legacy "RelatedContent" classes which you otherwise leave untouched? If so, and if it makes sense to you, I think it might be better to just leave the legacy code alone until RelatedContent's turn for refactoring comes. Related content has been a hot-spot for bugs for a while now, and it's definitely on the hit list.
You can remove the `> 0` since it saves some opcodes.
see binarylabels, it stores with float64_t (decision values) but returns with int32_t (+1 & -1, can even be made bool if you like).
Thanks for the explanation @mateusz ;-)
Probably. But docs says: "Returns logical path without any file extensions.", and the code of the method returns `@logical_path.chomp(File.extname(@logical_path))` which simply removes extname from the tail.

I might be wrong, as, probably I'm not understanding what is stored in `@logical_path` and I'll really appreciate if you'll explain me if I'm wrong, as I'm trying to make a port of sprockets for Node.JS
This is the guts of the code to handle address-deleted cells that are read in from disk.
Well, you read the code below it and you can see where they're used. I'm just manually maintaing them.
Could you add functionality for types listings (all, public, starred) for authenticated user ?
Please discuss this on the forum, this isn't really related directly to this code anymore and my mailbox is filling up with notifications :P
Besides, if you want to go with Plugin objects, keep in mind that we have some nasty code in GM right now that destroys *all* objects except a select few, whether persistent or not, when we leave the game (iirc). That's why RoomChangeObserver has this ridiculous self-protection mechanism (also because it was fun).
That's correct. I asked about whether points mattered (I think it was wareya), and they said they didn't care. Problem is that otherwise I'd have to wait for a lower points player or something, which doesn't really work.
I'd rather have less dependencies
Ilman käsin koodattua rekursiota:
~~~clojure
(not (empty? (filter pred a-seq)))
~~~
Tai point-freenä yhdellä rivillä:
~~~clojure
(def my-some (comp not empty? filter))
~~~
Why not returning $this for chaining?
opps .. i also noticed another thing. i would prefer it if we would inject the Imagick instance .. i don't have experience with its API .. but i assume we could also call setFilename() instead of passing the absolute path to the constructor? could you do a PR to change this?
its missing in the DIC tests...checked it out standalone, and it didnt get installed by composer.

should it be installed for "symfony/framework-bundle" dep?
the param tag is incomplete
Nice. And if you put a `| wc -l` on the grep you even know how many commits you are ahead/behind origin.
shouldn't the scheme be compared on its own? not sure why it's assigned to the port here
(Or fork on GitHub and implement herself.)
/confirmation/{{ token }}/ and /confirmation/ are different views and templates. So accessing /confirmation/ would load the template to send email.
You have not read documentation about `parse()`. Please compare `fromstring` and `parse` use in the [example here](http://lxml.de/objectify.html#creating-objectify-trees).
You register update handler with `registerUpdatesProcessor` function. For each type you can register different function. And then my code calls that function (or even more of them) for each type.
What is this function? Where it is used?
Comments will probably be another class/object. No? So function should then be outside, so that it can be used in both.
And what it contains?
You should go with last two `)` into newline. Like we have it elsewhere.
User exposed strings should be in `"`.
Lower case *Registration*?
@rwaldron we should jsperf this and see if the impact is that big. Would you have a reasonable "real-life-like" stress test?
i agree. also there's a file Extractor/LuaUitl.h, perhaps merge then into Util/LuaUtil.h?
Wouldn't it be more practical to define $collections in the function scope instead of the if scope

    $collections = array();
    if (isset($config['asset_manager']['resolver_configs']['collections'])) {
        $collections = $config['asset_manager']['resolver_configs']['collections'];
    }
sObjectAccessor->GetGameObject(
@Vasago: I've always bracketed my If..Then, regardless of how many lines. It's a personal style for readability. That's why they need to be explicit in their standards since there are many different styles here.

Anyway, I fixed the bracketing but for some reason when I pushed my local repo it only pushed the change to an new branch. I thought I had backed out the original patch. I guess there will be two pull requests then.

https://github.com/TrinityCore/TrinityCore/pull/6070
yes, could be a good example of a very simple directive with transclusion. If we manage to do it right with directives those templates should be really small!
I've added the `bundle` part as it is specific to the bundle only, currently and I didn't wanted to collide with other existing user defined functions.
я х3, та же самая конструкция в фейсбучном бандле работает https://github.com/FriendsOfSymfony/FOSFacebookBundle/blob/master/DependencyInjection/Security/Factory/FacebookFactory.php#L44
Is it really necessary to initialize `$status`?
"The captive portal of PacketFence allows a guest/user to register using his Google or Facebook account."
Replace first sentence with "Event delegation refers to the process of using event bubbling to handle events at a higher level in the DOM than the element on which the event originated." 
Also, don't capitalise Event Listener
my vote is for (1) and field filtering could be added in another pull.
self.thread = threading.Thread(target=self.__serve)
Corrected in 07b3fe4
where is gone the backend conf ?
to make sure we can pass any kwargs.
HEAD is actually at 0.21 now as of last night. :smile:
That's what I expected too, but that looked slightly wrong. It might have something to do with the background-position. with 180deg it looks like before.
Yes, this is just a test implementation of an actual overlay view. Daniel will have to do the real thing.
I guess this should checking for other modifier keys (CTRL, ALT, etc) too?

I think you'll also find if you check the SDL header that all the modifier keysyms are together in a block numerically, so you could use a range check if you wanted rather than checking for each one individually.
there are a lot of untested methods, for example I have a problem right now what predict takes as argument
There were kernel selection keywords in an earlier version, but we took them out because there were no other kernels. One issue is that you need explicit integral and convolution forms like ``gaussian_cdf`` and ``gaussian_convolution``, to not make things very slow.  Not sure if there are analytical expressions for those for the gamma kernel.
nit: ii = levels.length...
I almost wonder if the inheritance order should be the opposite.  Basically making JsonHandler just a wrapper around rest handler.  That way we could add other adapters, like some XML (yes, yes, I know, yuck) thing or something if needed.
That's great, thanks Mike!
An example in the current code is _sigma in [SBGaussianImpl](http://galsim-developers.github.com/GalSim/classgalsim_1_1_s_b_gaussian_1_1_s_b_gaussian_impl.html#ac00324195a28c869f03d82ed1da273d9).  It works the same way as the variables documented with `///<`.
Changed to use a synchronous consumer
Comparaison douteuse, des enfants peuvent etre potentiellement super forts non? (kasparov a lage de 5ans battait 99.999% de la planete) faut reformuler children en average player je pense
_As for go_
Typo -- should be `ownership`.
Aceito observação, adicionado alteração
Wait... A client-side refresh causes a server-side state change? That doesn't seem right...
Лучше явно выбрасывать инстанс TypeError - этот объект гораздо богаче в плане отладки (содержит номер строки, стактрейс, ...)

```javascript
try {
    throw new TypeError("Wtf");
} catch (e) {
    console.dir(e);
}
```
Thanks for the comma fixes!
Not merged correctly for some reason
that wasn't me, I just checked the indentation.  I can fix it though.
I removed this case by mistake, sorry.
Make the example short. Remove the doc blocks - you don't need them here
Metoda ar trebui sa fie private.
At this point it would probably be more readable to remove updateAck and updateNak entirely (since noting about a state is updated) and directly emit the signal.
Remove.  This shouldn't be exported.  Nothing calls it directly and nothing should.
small thing, but have a space before and after the '='
this is automatically added if you run `brunch watch`
Во втором коммите это было исправлено
в смысле компаратор для compareTo(), этот же метод принимает на вход только объект, с которым сравниваем?
This is not Contao coding style. Your PR will unlikely get merged as long as you don't use the Contao coding style. In addition, `strlen()` should not be used if not necessary :)
Will check b2g. I don't have yet working android builds...
As I removed the patches method, the line is not needed anymore, no ?
use two. I feel it is the most common use case ;)  (visualization), and is the fastest non-trivial version.
Any one opposed?
Why do you call it random forest if you inherit from Extra Trees?
Why the trailing `_`?
is it possible that X is copied for every target?
I usually only use references when I'm either expecting huge data, I want to change the value inside or allow a value to be changed outside.
AFAIR basic types and arrays are copied. (Objects within an array a referenced).

In this case I expect "huge" data (maybe body of 5MB) and I allow the content to be altered after setting it to the header, without having to set it again, which allows Header initializations, even if the content is not yet fully defined.

I don't think it is required, but I think it optimizes some use cases and from my point of view it does not hurt.

PS: Initally I also had the reference in the set/withContent parameters, but removed this, because I'd have to use a var instead of simply calling it with a string.
This isn't valid code here. I think you meant `elementId = script.attr('data-element-id')`.
I mean print an error.
My mistake, I thought you were commenting on line 247.
Done
Effectivement c'est pas très intelligent ^^
Can sub be null here?
factory->queryQueryFactory().queryPackage()
>I suspect you should only free the hash table if memory is critically low.

I wondered and in fact I've just noticed:
>        return 1; // SPILL_PRIORITY_HASHDEDUP;

which is for debugging only. 

I think a low-priority vs critical is probably ok (which was the intention)
I realize you are simply following president... but I have to note that this whole XmlArgs thing looks overly complicated to me.
I don't understand why you have the spaces though.  What is it fixing?

On Tue, May 29, 2012 at 6:15 PM, keesjandevries <
reply@reply.github.com
> wrote:

> > @@ -62,7 +66,7 @@ def __str__( self ) :
> >          return self.short_name
> >
> >      def __repr__( self ) :
> > -        r_f = "%s [%s] %0.4f +- %0.4f"
> > +        r_f = "%s                            [%s] %0.4f +- %0.4f"
>
> Like I said: ugly solution, but I use this to print out relatively nicely
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hyperfunk/mastercode-tools/pull/28/files#r893249
>
Ok. You convinced me.
I don't like this. I think we did correctly to begin with and only requiring verified emails to do password resets. It makes the most sense to make sure the user verifies their email address to give us better confidence the user owns the email address on their account.
No problem !

I'm not sure if it is an official "best practice" but I think it is a good idea to always wait some time before merging a pull request you made. 

Another concern here is the lack of tests to demonstrate the bugs being fixed.
Removed -- also added a comment indicating that empty value is equivalent to accept all
I think that this should default to false (that is the default for nova.conf according to the default nova.conf sample file). Having it default to true changes the default behavior for this manifest.

Also, instead of introducing a class parameter, why not just set these nova_config settings at top scope with nova_config ?

https://github.com/openstack/nova/blob/master/etc/nova/nova.conf.sample#L299
this is roughly equivalent to `next [no-arg]`, except for the method rather than the block.
Yep, that's right. But here was a hard-coded include path to `/usr/include/suitesparse` which makes me wonder why that built on your system at all unless you actually linked to Apple's suitesparse from the accelerate framework.
My guess is that you don't actually need a path to `<prefix>/include/suitesparse`, just a path to `<prefix>/include`
I see. I understand the code now. :)
Import SoapClient
same deal but even more insignificant, `for index,rule in enumerate(rules)` will give you a zero-indexed integer for each list element
Is this what you want?

        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 something other than the new lines is offending you please be more specific :)

Cheers, Matthew
I just moved it out.  It should be there.
a ruby shortcut for the strings: "https://sakai-dev.berkeley.edu/portal/site/" + site["id"] becomes "https://sakai-dev.berkeley.edu/portal/site/#{site['id']}". I swapped the internal piece to use single-quotes to avoid any potential issues, see if it works before committing the change =).
@shacker please remove the `<strong>` here. It doesn't make sense to do this within a `<h3>`
@shacker fyi it also looks like I have one with an empty title as well
This test fails for me on MacOSX. Can you confirm that it works for you on some unix system?
Because font commands I used required XeLatex. Which doesn't support inputenc (assumes UTF-8)

4 aug 2012 kl. 23:46 skrev Johan Sannemo:

>> @@ -2,7 +2,7 @@
>> 
>> % Package declarations
>> 
>> -\usepackage[utf8]{inputenc}
>> +%\usepackage[utf8]{inputenc}
> 
> Why would you comment this out?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jsannemo/contest-prog-intro/pull/4/files#r1307648
@bootstraps So what is `params.first`?
åja. Av og til fungerer det ikke å skrive kode kl ett etter lang dag på skolen.
okay, fixed in a new commit.
Use a regular variable or store state in 'c'
yo dawg, boolean values are way better than strings when tracking boolean values
So, either this way (at the moment those two function calls do the same as `warp` automatically uses the `inverse` method - we'd need to change that behavior, so that it automaticall calls `__call__`):
```python
warp(image, inverse_map=tform)
warp(image, inverse_map=tform.inverse)
```
Or
```python
warp(image, inverse_map=tform.forward)
warp(image, inverse_map=tform.inverse)
```

@tonysyu what's your opinion about this?
any reason to have hardcoded values here (eg 90, 10, 80) ?
OK, I see the semantics, and am OK - and perhaps even agree - with them.  What will the layman using our API think?  Will they get use to looking for `from` functions when they want to create an object?  Or will they know that they aren't really converting so they should look for `compute` or `create` functions instead?  I doubt most users will know the difference.  Will they be frustrated that they need to look at constructors, `from` functions, `compute` functions, and maybe even standalone functions to find out how to create something?
Same comment for `channels` and other uniforms throughout.
I think I did add this to fix some visual glitching.  My vague recollection is you get resize events continuously while the left mouse button is held down on the resize edges of the browser window, even if the mouse isn't moving?  I'll try to do some source control archaeology, this must have been pre-GitHub.
This seems bad to me.  We shouldn't automatically turn off caching for all requests.
Right, I'll change the variable name.

It would be the model-view matrix if the central body had a model matrix. I think there might be a bug where the uniform state doesn't update the model matrix to the identity if the primitive doesn't have one.
Ok. This might not be too big a problem since all the texture coordinate references should update when the atlas is resized, though I can see issues if someone saves individual values.
Not sure I understand the comma at the end of the line. A semicolon will have the same effect and less confusing.
TiUIDialog supports both AlertDialog and OptionsDialog. The only reason for the AlertDialogProxy check is b/c iOS doesn't support persistent for options dialog yet (and I don't want to create more parity). In the future that may change, which will make TiUIAlertDialog pointless.
D'oh. Um, so the guy who sent you this code sample is apparently colorblind ;-).
I hope that doesn't change :)  indexing into String is a nice feature.
Again a comment on why document_type_obj is needed to be accessed.
I honestly don't believe so, no.  If there are any external users (and I sincerely doubt there are), their behavior would magically change from providing the base url to the home url. They might not notice this at all via any compilation errors, and they will suffer a drastic breakage in behavior (ie, passing in /home/rob/apps/jboss711/server as the home directory, which would break startup). 

However, I've guaranteed that nobody was using those constants. I would prefer putting the old version back in and marking it as @Deprecated and broken and incorrect. @Deprecated is unfortunately the only way to signal some key value has changed. 

Thoughts welcome.
OK. Glad we got *that* out of the way.

But you need not worry about "blocking" anything/anyone here.

Since this manual page is not written yet, and it's a documentation, rather than a program feature, it's not like we'll *have* to block a release until we sort things out with you.

You can safely act on this feedback in whatever pace you like. If you stop responding, we'd either merge this "as is" at that point, or we'll merge changes from another person who submits another pull request with the changes in question being addressed. \*

The only *potential* problem with documentation pages is if we decide to do some form of major reorganization (like if we switch from .rst to .md or if we move the tag docs to a different folder). If that happens, we can merge whatever state your fork is in, do said major changes, and then if you have the time, you can operate on the changed version.

Having said that, no major reorganization is planned at this, or any future point (or at least I don't know of any), so even the above scenario is most unlikely.

\* The beauty of GitHub is that such "new contributors" can fork *your* copy, and improve upon *your* work, and thus the time you've already spent won't be for nothing even in that extreme scenario.
this syntax is not specific to just collecting virtual resources. This will actually modify the state of non virtual resources as well
Did you mean to add RedHatPlugin here?
Something is getting wrong with this. Instead of many lines appending to retval, there should be one large string constant. The resulting pattern is rather awkward. Besides, what's the rationale for using such a large code generation function instead of other approaches such as templates?
I think the above is more complicated than it could be. In practice all random access ranges must accept uint or ulong as index type. Therefore check whether typeof(_input[ulong.max]) exists, in which case ulong is the exposed index type; otherwise, typeof(_input[uint.max]) must exist.

Given that the indexes for a iota are in 0 through length-1, we should fix iota to use the same type for index and length.
Hum... given how drop works, its part of the documentation that it *not* modify range... Not sure we can use auto ref actually.
You could say 'but *a* unicode digit'. I don't know the specific grammar rule why, but my version is definitely OK in English ;-) The original line sounds a bit wrong.
One doubt: During my tests I had no @exceptionhandler and I got an @ObserverExcepton: no handlers found to ....

Isn't that a deltaspike way to check if the exception can be  handled ? 

I made another test and removed the handledAndContinue() method from all handlers and the isHandled() continues to return true.

So I'm not sure when the isHandled can be false. Can you clarify me ?
nit: prefer return over else.
So the message is wrong I think.
As has been discussed, this only works on Windows. It also fails if the path is set via vlc_path.

It should be possible to probe the version via `vlc -I dummy [ --dummy-quiet ] --version`.
Derp.  Thanks for adding this, Dave.
Ok.  Probably "Then  I should see that 5 patients were registered today"
Two params missing... also `array<string,UserInterface|string>` is quite strange, and not sure it will be corretly parsed by some IDE...
I'd say that these three line can be refactored into utility method and be used then like

    public function _parseOneEntry(&$author, $item)
    {
        $authorInstance = new Person();
        $authorInstance->parseXml($item->asXML());
        $author[] = $authorInstance;
    }

    protected function processAuthorNode($xmlArray)
    {
        ....
        ....
        if (is_array($authorItem) {
            foreach ($xmlArray['author'] as $authorXmlInstance) {
                $this->_parseOneEntry($author, $authorXmlInstance);
            }
        } else {
           $this->_parseOneEntry($author, $authorItem);
        }
Nor the private.

Nor the link method.
this needs to be:

backend_version_label.set_markup(_("<b>remote entitlement server version:</b> %s") % backend_version)


the python-rhsm line above it probably needs to be changed too
I'd go for bestHash here
`CSRF`
then it only requires one more line:

```xml
<service id="jms_serializer.doctrine_serializer" parent="jms_serializer.serializer">
    <argument index="2" type="service" id="jms_serializer.doctrine_object_constructor" />
    <call method="setExclusionStrategy">
        <argument type="service" id="jms_serializer.doctrine_exclusion_strategy" />
    </call>
    <tag name="jms_serializer.serializer" />
</service>
```
А так бы оно не вернулось из функции, да?
lets nuke these 2 lines
ahh, right.  yeah, that's way better.  I'll fix that.
Should be simply

    if (!Tran....

I am making my comments in the form of a commit; I'll include this change and let everyone know when I am done.
Follows from the above comment - this is not needed
Hmm, this will mean that the spiceweasel command will fail below because the cookbook directory will not contain the cookbooks -- since the cookbooks are git submodules...
Why has the refactoring into partial removed this if? There's a pending request to show "No statements" if there are none to be shown. That has been implemented in (https://github.com/naleksi/avoinministerio/commit/df1779d6e1e883df235c5b3d14374395ac4b4ce2#diff-11) but has not been pull requested. 

If this is merged, how should the if articles then show them else show "Nothing" be implemented with this partial? Well, it's a problem for the future, so I'll merge this.
Looking at the mail link below, could it be: _`SMTP transpor`t: zend.mail.transport.quick-start.smtp-usage
Why did you remove all the examples?
It should be three separate scenarios because they way either of those is handled might change separately.

Because the examples don't differ in their behaviour yet (again: more of a placeholder than real cases), they are all part of the same scenario. I'm hesitant to put them in a common file though, same reason as above: even though they now look the same in different places, I anticipate a lot of things to change in the near future—right now, constraints on escalation policy names and rotation names are the same, but that's more of an incidental detail—and I don't want to de-duplicate things prematurely.
Since the Label attribute are fairly straight forward, I think we can skip the placeholder text.
if they are the same object, this call is useless...
BTW, we need to encode to HTML entities, can't we use just pure UTF-8 with proper symbols? :P ©
Afaik, there should be no need to call it with the first char toupper...
I was thinking it might, but this is the only place using it. If there's another place that uses it later on, I'll move it to the utility class.
this should probably be in the ACL presenter rather than the action menu view implementation
Raising SkipTest for things we plan on adding back soon keeps us from ignoring the tests long term, but I think if the tests aren't being run because they don't apply to a configuration (as in this case, where we may never run this test again) the @test "enabled" property should be used, i.e.
<pre><code>
 @test(depends_on=[test_root_now_enabled], enabled=not test_config.values['root_removed_from_instance_api'])
</code></pre>
This is also why I think the test conf values will be easier to deal with if they don't state a negative, i.e.
<pre><code>
 @test(depends_on=[test_root_now_enabled], enabled=test_config.values['instance_api_has_root'])
</code></pre>
I was wrong, just with catch InvalidArgumentException are enough
this helper it's not necessary, it's doing exactly the same as:

sing_up guest
Please also run the pep8 script on this code  and fix the violations.

http://pypi.python.org/pypi/pep8
Well it looks like I forgot to actually set the random number to the last selected random number before returning. Looking at this today this could likely be cleaned up a little.
Why not keeping the storage configurable ?
Used internally by us only
Jo to tam nema co delat
This comment confuses me. -s and -o shouldn't have anything to do with the error here. An error happens here when the config.xml cannot be parsed. Am i missing something?
I added the -lang-c++ flag so i can build! Without this flag you will get errors when building the framework. Is there another solution to this problem instead of using this flag?
no luck so far

```javascript
var text = document.createElement('textarea');
text.setAttribute('id','focusme');
document.body.appendChild(text);
document.getElementById('focusme').focus(); // seems to focus but no cursor appears in textbox and no keyboard pop out
document.getElementById('focusme').click(); // nothing
```
```javascript
var evt = document.createEvent("MouseEvent");
evt.initMouseEvent("click", true, true, window, 0, 0, 0, 0, 0, false, false, false, false, 0, null);
document.getElementById('focusme').dispatchEvent(evt); // nothing
```

running out of options. looking into how other DOM libraries do it, might just use one to test .
minor suggestion...i would invert this and consider a zero count to be an error condition

```ruby
if count == 0 && !options[:force]
  say "Skipping upload to Socialcast since no users were found"
  exit 1
end

# rest of logic remains the same...
```
(== 0 (mod n divisor)) already returns a boolean
just noticed that still `$post->created_on` can be a string, so a strtotime() should be added too.
Since this is for both iOS and mobile web, we should probably name the folder ui/handheld/common/ApplicationWindow, or something like that.
Agreed with mbs.