labofsahil/github-event-dataset-2012 huggingface dataset:
Also should be a util for this.
Where has the lazy signup disappeared?
Any reason you want to get rid of the head cloning here? What if the template is doing something like... loading jquery?
That is the default; not necessary.
Since you're making a point to do some style fixing here, I'll make a point to fix this one space.
one-lined that normalize function: 

    function normalize( value, minWidth, maxWidth ) {
        return Math.max( Math.min( value || 0, maxWidth ), minWidth );
    }
s/Uncain/Unchain/
Lint says this is an issue because editor never gets used. Do you need it, or can you just call "EditorHelper" without storing the object?
""Beavis and Butt..."
This video is no longer available due to a copyright claim by Viacom International Inc."
This comma is wrong.
okay, let's remove the shadow then. can we add a comment so people with not shadow it in the future?
This one will need a `python manage.py migrate people`
Invalid HTML I believe - you can put this just above the UL and everything should work as expected
Indent methods after the `protected` call.
it's in _opts__ to serve as a doc and provides a default value for the option. The option is there so that users will know and can override it by specifying it in their master/minion config.
I don't think so. Presumeably that's a database error of some sort (database connection most likely). I think the right thing is for the app to throw up, and the framework to do the right thing - log the issue and either show a "fail whale" to the user in production, or a meaningful error message in development. No?
it seems this variable is not used?
Technically we can get parent ids that are 0, even though it's not all that likely. If it's possible to use an option here that would be preferable.
I assume 'inq' and 'nin' operators are opposite, so IN [blank set] is equals to 'AND 0', but NOT IN [blank set] should be equal to 'AND 1', isn't it?
Please use unobtrusive javascript (no onclick code) and generic code (no hard-coded url, model or controller dependency). Sugestion: leave the link as it was and add the confirm class on it, than use $(".confirm") to prompt a confirmation modal and handle the user redirection. Better yet, use HTML5 data attributes to read the message, see https://github.com/rails/jquery-ujs/blob/master/src/rails.js .
You are right. I pushed a new commit where they are deallocated.
Blank line before this comment, please. This all runs together.
I think there is something like `#demodulize` in `ActiveSupport` fot this
i think you do not want to exclude this file
Tuples in one form or another would be fine.  The `format1`, `format2` business is not that pleasing to me, it's just what I thought of in a short time.  I'm not super-keen on a single string with space-separated coordinates or formats (or worse yet mixtures of the two) because I think it can get messy, but I would not strongly object.
typo: "mkaes"
Right, I suppose it does, removed the __linux__ part.
This is the only point I disagree as well.
I personally don't like using Not ( ! ) on a group of conditions, makes it looks ugly.
Moving this test makes it not assert what it was intended for, namely to reset the transport config.
Now that we want to merge instead of overwrite, it makes more sense to change CakeEmail::reset() to handle that instead
This file appears to be only changed in terms of comment style. That's a bit confusing. I don't care much about comment style, but ideally there wouldn't be commits to change it.
api.parse_ts handles this for you (we use `None` rather than omitting the key)
Might as well move this parenthetical up as part of the initial intro (line 85).
I don't completely understand what that means, partially because I've started 4 days ago and don't know much of the architecture of the emu. It's solved as you told.
inherited a 'enabe' typo
In my memory, I wrote `LaTeXTool.instance`...
Mmh, those are going to be used a *lot*, so I was looking for short names...  But I can go with `run` if you want... Though it will break with the existing API, which is just `magic` (and we can't change that, as it would break a ton of code out there).  what do you think?
Jclouds automatically parses anything that produces application json.  So
you can try just removing the responseparser annotation
On Nov 28, 2012 11:54 AM, "Leander Beernaert" <notifications@github.com>
wrote:

> In
> apis/openstack-nova/src/main/java/org/jclouds/openstack/nova/v2_0/functions/internal/ParseDiagnostics.java:
>
> > + * @author Leander Beernaert
> > + */
> > +public class ParseDiagnostics implements Function<HttpResponse, Map<String,String>> {
> > +
> > +
> > +   private final ParseJson<Map<String, String>> parser;
> > +
> > +   @Inject
> > +   public ParseDiagnostics(ParseJson<Map<String, String>> parser) {
> > +      this.parser = parser;
> > +   }
> > +
> > +    @Override
> > +   public Map<String,String> apply(HttpResponse response) {
> > +      checkNotNull(response, "response");
> > +      return parser.apply(response);
>
> How can do I parse the data without defining the parser, can't find any
> examples on how to this exactly :/
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jclouds/jclouds/pull/995/files#r2253286>.
>
>
そうかもしれぬ
I think your right, if so we can remove the check and gain about 1k+ ops/sec
include/git2/message.h:16:1: error: unknown type name ‘GIT_BEGIN_DECL’
add `aria-required="true"` here to support browsers/AT that doesn't support just required :)
No `<label>` element associated with the input field
I think since there are only 2 options in the `$options` variable it would be best have `$cwd` and `$env` replace `$options` in the method declaration.  A more verbose API is easier for a user to pick up on.
The comment is redundant. Instead, the comment should answer: Why do we need to have a 'currently active scene'? What kind of limitations does it impose for the application developer/user? What is the benefit? What does it mean for a scene to be the currently active scene? How/When can we switch between the currently active scenes?
The palette copying logic looks changed here. Formerly the palette was only updated in the type != 0 case when there's an actual palette present. This seems to be ignored and instead it's always copied. It's not even checking whether the palette size is actually 256, which could be an issue if the PCX file is not version 5 AFAICT, not sure whether this is any real problem though. Assuming that it's always a PCX file with bitsPerPixel == 8 and nPlanes == 1 seems probably fine from the looks of the oldloading code.

In fact seeing the code in the PCX decoder now it's an hard error when there's no palette present, so a missing palette for a version 5 PCX File would result in line 307 to be executed on loading of such a file and thus ScummVM to error out.

Has there been any tests with Tucker to see whether this change is fine?
Just make this class final?
The private $prefix is also annotated. And this is basically only a reminder, that we don't need to save the prefix anymore in 2.3 (additionally to removing `getPrefix`)
It is working, except for the static methods, which are indeed exemplary (and not defined by intention). Please improve the example if you feel like it.
what about 'extractBaseClasses` rather than `Dependent` ?
you need to keep the ``splitHttpAcceptHeader`` method for BC
BTW, I also plan to add tests to cover this and other options of bot
I failed, this should be `string` only.
`models.` is redundant since this file already has `import models.*`
Style guidelines:

* Avoid abbreviations.
* Name the enumeration parameter the singular of the collection.

The `i` should be `invitation`, I believe.
Should add a test for the event name here, no?
Indent the second-through-last lines when doing multi-line.
So as Queue.qsize() is not a reliable way to get the size of a queue, we loop until the queue is empty.
We could have used a `while True` loop but we didn't want to create an infinite loop.
Yeah, the fork will be almost a complete copy, but unless you have overcommit on, then linux will be conservative and assume that none of the memory is shared and fail if you don't have double the memory. This is exactly what we see with redis. Telling people to enable overcommit will just make people scared to use our agent.

For the ssl time, the above threadpool implementing looks like it has a thread call timeout, which would bust out of the hanging ssl call. We could precautiously reset the global socket timeout if that ever happens.
should this be `groups` rather than `m.groups`?
and where's the fix for case when `groups[0] is None`?
Thought we were just gonna say Red Hat for these files instead of individual authors?
Consider using com.netflix.astyanax.serializers.SerializerTypeInferer instead of this class to get a serializer when constructing the EntityPersister (EntityManager)
I don't understand why there is a <div> added both here and in the template file (the good place is in the template). 
If you want to replace the tag *content*, use "#nameField *" CSS selector, see: http://simply.liftweb.net/index-7.10.html http://exploring.liftweb.net/master/index-5.html#entry-Snippets-binding-with-CSS-0 http://cookbook.liftweb.net/Testing+and+debugging+CSS+selectors.html
nx and **ny**?
What is the default zorder of scatter points? If you hadn't done this, were you getting lines over the scatter points?
does array_filter() not work? it should be much faster...
n^3? :)
{auth, none} should be within backticks, since it's code: `{auth, none}`
agree with this TODO. an ``update`` statement (if possible) would be much nicer and faster.
Unclear wording in my opinion, better: "Sets the nodes list to the ``children`` (?) variable in the context"
*an* element
I restored this code portion but IMHO this is not the responsability of the storage to define the URL of the image uploaded.

I think the real issue is that the URI of the resource is chose by the user through the filename of the image uploaded. It would be more logical that the URI was created by the upload handler himself (for example using an uuid)

Could we discuss this topic when you have a moment?
"Mettre en favori" plutôt, non?
On 11/16/2012 11:24 AM, Jaikiran wrote:
> In web/src/main/java/org/jboss/as/web/WebSubsystemParser.java:
>
>> @@ -369,6 +444,84 @@ public void readElement(XMLExtendedStreamReader reader, List<ModelNode> list) th
>>
>>      }
>>
>> +    private static void parseValve(XMLExtendedStreamReader reader, PathAddress parent, List<ModelNode> list) throws XMLStreamException {
>> +        String name = null;
>> +        final ModelNode valve = new ModelNode();
>> +        final int count = reader.getAttributeCount();
>> +        for (int i = 0; i < count; i++) {
>> +            requireNoNamespaceAttribute(reader, i);
>> +            final String value = reader.getAttributeValue(i);
>> +            final Attribute attribute = Attribute.forName(reader.getAttributeLocalName(i));
>> +            switch (attribute) {
>> +                case NAME:
>> +                    name = value;
>
> This and the rest of the parsing and setting into ModelNode is better
> done via the SimpleAttributeDefinition.parseAndSetParameter method.


Fixed.
As before, please create a `package.json` file and add Express and Socket.IO as dependencies.
Actually just `<span>` if all you're doing is creating the element.
统一 userInstance
screen: document.getElementById('screen'),
    attentionScreen: document.getElementById('attention-screen'),

and use them. Drop the getter and try not to make confusing variable names.
Since we're in the system app, we might as well remove this while we're at it.  I don't remember why I added it in the first place.
nit: the indent seems weird
In some way it seems odd to me that open takes an app as the first parameter and close an origin. Is it on purpose?
Done. And thanks for your illustration in detail.
nit: one var per variable
It's also should be in state.box.messages.unseen. I'll fix it.
No need to make this global. A static const variable in RepoInfo::acceptLicense would be sufficient.
> изменить на "Использовать английские названия вместо местных"
Согласен, так звучит лучше
Ideally you should check that the handoff actually succeeded before you commit to it being a success. :)
I think for completeness - should we add a test where both are ':undefined' ... and the result is "test__test" ...?
Caps? "vhost"
Well, according to AMD best practices, you shouldn't export them into
one file. I don't know how many people do that anyways though. :/

Sent from my iPhone

On Apr 17, 2012, at 21:10, Jacob Thornton
<reply@reply.github.com>
wrote:

>> @@ -101,7 +101,7 @@ function removeByteOrderMark(text) {
>> function wrap(file, name, openedFile) {
>>   switch(options.wrapper) {
>>     case "amd":
>> -      return 'define("' + path.join(path.dirname(file), name) + '", [ "hogan.js" ], function(){ return new Hogan.Template(' + hogan.compile(openedFile, { asString: 1 }) + ');});';
>> +      return 'define([ "hogan" ], function(Hogan){ return new Hogan.Template(' + hogan.compile(openedFile, { asString: 1 }) + ');});';
>
> we should conditional provide the name of the module for when people output multiple definitions to a single file
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/twitter/hogan.js/pull/73/files#r698891
I'm adding snippets to Codesnippets, but the above hint I am going to leave as it is.

Comments are not the place for a long discussion on complex usage examples.  I simply wrote enough for the person that wanted to see that this pivot is safe even if some of the columns are missing (which is an uncommon case, I hope).  If they want to recover from that case, they can use Some/None to distinguish from present nulls, from totally absent data.

This issue is confusing because Java convolves the notion of "null" values and "null" as missing data.  To drive the point home further, what is wrong with this class:

class Storage[A] {
  var data : A = null
  def set(a : A) { data = a}
  def get = data
}

The answer: you can't tell from a get if Storage is unset, or if someone set it to null.
https://github.com/AppScale/appscale/issues/107
Add NODE_PUBLISHED
Always treat version numbers as strings.
Instead of adding the view_type string directly to the context, I used it to make the two booleans `is_zendesk_user` and `is_github_user` in views.py and now add and use just those two in the context like in the example you mentioned.
Sorry, bad habit of me, I'm going to fix the things you suggested.
sorry I missed this before - I think that on Unix `test -f link` returns `true` (i.e. it's a link *and* a file)
Not much, why?
You're right, just fixed :)
maybe pass item too?
I don't see anything in interim-bootstrap that would affect the grid system, but I know Bootstrap and Blueprint both set styles on .container so that could be the problem. I think if we still need interim-bootstrap I should probably copy-paste the styles from a more current version of Bootstrap, though, since some of those CSS classes (.twipsy for sure) have been renamed in Bootstrap 2. I'll give that a try and see if it solves the .span-24 problem.
not sure what this is doign?
K, fixed this as well.
Where are my changes?...
Ruby has `.nil?` for this. ;)
ok, I'll handle it. :-)
Remove all fullscreen code from this pull
Done in a separate patch.
I was just trying to keep the `require` callback interface the same at each level of inheritance.

see `lib/weiqi-models.js`, everything is treated the same. 

I'm trying to articulate the need for this branch: Consistency, until we have a reason to (eventually) abandon it.  This is tricky. I think we want to write another persistence layer eventually, so having a consistent approach is key there..thoughts?
Can you mention in the comment `onComplete` listener, or anything similar?
So that we can make instantly the connection between options.onComplete and this call.
Nit: Please put as space after `//` back :)
Do we have to define all those methods even when we do not make use of those?
you don't need to change that .. but personally i prefer ternary for cases like this ..
@hkenney Sure, I'll add a link to the msdn docs <http://msdn.microsoft.com/en-us/library/windows/desktop/aa394373(v=vs.85).aspx>
Wow this is for all faces!
It would be much nicer if your exceptions took a reliable set of explicit data, not a hash that allows arbitrary misspellings to slip through.
@dchelimsky / @justinko / @alindeman -- I want to get your feedback on this.  `stub_const` returns the original value of the constant (if it was already defined).  This is used by rspec-fire and just seems generally useful.

That said, I'm wondering about the interface for this.  As it currently stands, it's impossible to tell (based on the return value) the difference between an undefined constant and a defined constant that was set to nil--either way, `nil` will be returned.  I'm considering changing this so that rather than returning the original constant value, it would yield the the original constant value, and it would only yield if the constant was already defined.  If the constant was not defined, it would not yield at all...so you could tell the difference.  `stub_const` would no longer have a meaningful return value.

I'm not sure which is better; the return value interface is straightforward, and how often are constants defined and set to nil?  On the other hand, having an unambiguous interface is nice.

Thoughts?
I don't think this is right, because `IS_LINUX` and `IS_.*BSD` can be true at the same time.

I would suggest adding "Rubinius.bsd?" to the case statement and doing the distinguishing between free/open inside that block.
This smells like a regression; why was it removed?
This file uses ValidatorInterface as typehint
I think it would be much better adding a new method "unsetVariable" or "removeVariable" directly removing the variable on "$this->variables" instead of copy the entire array over 2 methods for each unset
phpdoc does not accept (optional) at this point

> @param <Type> [name] [<description>]
Same comment here -- `if ($events) {`
Please put  curly braces around this
It's a convention. There may be issues with very old clients where being case insensitive was not strictly observed. I'd simply prefer that we not deviate from what ESAPI does since it was written by wiser minds than mine. If it's explicitly upper casing hex values, then so will Zend\Escaper. If the performance difference is that significant to ZF2, we can revisit it if needs be.
This needs justification / explanation.
Once upon a time, khan-exercises _actually_ looked like the real site.
I would use `$item->isWritable()` method.
Ternaries shouldn't be broken over multiple lines. Maybe you can just do:

```php
$data = $this->_data;

if ($bind) {
    ...
}
```
Oh herp, I just woke up, but anyways, enum names should never be all caps, name them as you would classes.
Thanks.
I'm sorry. I saw my mistake. I tried to remake PR. 
https://github.com/ZF-Commons/ZfcUser/pull/171
How about it?
I thought UUID.get() would be ok (Vanilla JS use a function named getUuid()). I renamed it UUID.createUuid(), which is perhaps a bit heavy...
Isn't that going to count every module rather than just those owned by the owner?
Messagebar is already created by the globalSetup so you can just use {messageBar}
Tee mieluummin tarkista_oikeus("tuote.php") funkkarikutsi vain kerran ja ota tulos muuttujaan. tarkista_oikeus-finkkari on nimittäin aika hidas.
hmm i think it just ignores the remove call for the current locale the document is in
i think you can simply drop this check as the class is optional and passing null will work as well.
Do we need to do anything special here for this to work? I don't recall if it was a "defined-or requires a use feature pragma *now*" or ".... may require a use feature pragma in a later version of Perl".
Avec plaisir. How? Maybe the text should be rewritten in cleaner written language then...
Not sure if bs4 pulls it implicitly.
better yet, make a new derived LdapFilter class to deal with this filter (it's very very common)
clear screen? ZOMG dude, you're a child of the eighties :-D But I agree with the name change.
Changing this is actually a rebase fail. It should be `HttpClientInterface`. :) Good catch!
there should be no tray, head and sdh config
The convention we are using with the quickstarts is to put the tests in a "test" package e.g. `org.jboss.as.quickstarts.ejbinwar.test` and not in the same package as the app clases.
Yeah, I do too. In this case it has to be concrete though since I'm using putIfAbsent from ConcurrentHashMap. It's probably overkill but I wasn't sure if this part of the code is multithreaded or not so just being safe. If all the query initialization is done by the main thread then this stuff doesn't have to be a ConcurrentHashMap at all. Feel free to change.
I notice there's lots of new places where src is being passed in but I don't see any new places using it? Is this just to fix bugs where we forgot to include the src?
Spaces in moustache.
This image is from the brand guidelines we should not resize.

Also curious why do you want to remove the height and width here?
Wondering in cases where src is a url, if we can take the string after the last forward slash.

Example: http://pointlesswebsidename/importantimage.png

we have to make sure we cannot fail on this.

In cases of a data uri, something that doesn't end in a forward slash, etc.

What I am trying to avoid is a usual sized track event cutting off useful text and displaying "http://pointless..." Instead display "importantimage.png"
I think you could put is up in __init__. Just from a read anyway. That should avoid the if down here, too.
So it will now say "Bookmarked 0 Times" instead of "Bookmarked 0 Time" when certain admin things are done. (I think I just logged in as an admin and deleted someone's bookmark, but I'm not 100% sure that's how to get the 0 Times to show up. It does show up in some situations, though.)
Should use a `deque` here instead?

Memory could start creeping up over the course of a backtest if we just keep on appending results.
On further investigation, I do see a problem with mTransient being private - Because it's accessed from EntityMapping#load, Java will generate an accessor for it - this may be the origin of the warning you saw. 

My recommended fix is to make it package private.
It's definitely leaky... pollstats and slurpstats aren't my proudest moments. Readers shouldn't even have to futz with this themselves... it should really be in the reader module.
s/def_map_run/def _map_run/
подивись оригінальні налаштування. там в мене ще кнопка для cut була. і для code здається, щось було
I think that would be fine. We can be flexible with import.
super long line endings here.
again, use final_answer
i know we talked about this already, but definitely finding a way to use color at the same time as opacity would make those tiles pop just a bit more, even if you fade them to black at the end. still, nice touch here.
probably not meant to be in. :)
So given that the instance that the instance you're sshing into was most likely created by cloudenvy, and agent forwarding is required for private git checkouts (very common scenario) I think this should be switched to True as the default.
Why add the full package definition to the String class and not add import package?
if that's the same why not, but regarding (y)our discussion about version dependencies this only makes real sense if other plugins using WebKit don't require a version with `webkit_web_view_get_uri()` in it -- and I would guess that WebHelper do.
нужно пожестче потестировать. тут могут быть ошибки
What do you think about also exporting the server that gets `require`d? In this way in the main file that requires h5bp, we can use express or connect `middleware`s more easily, instead of `require`ing them again.

`app.js`:
```js
app.use(h5b.express.static('public'));
```

In this way we can handle static files.
Also by removing the duplicate code from extensions.py, you need to remove some unused imports:
from selenium.webdriver.support.ui import WebDriverWait
from selenium.webdriver.common.action_chains import ActionChains
Does the test pass? I've tried with 11 and 12.

But the code pattern matches the same we experienced failures with when  moving to 2.21.0 so i added a wait just to be on the safe side.
remove "relevance" from here as well.
That's not in the style guide. Perhaps add it as a discussion item to the upcoming meeting?
Static import createStatusResponseHandler
I was (am) also interested in using something that might support ehcache, but that project says that it is dead; it seems the authors are now steering people toward the clojure's `core.cache` instead (even though I don't see any mention of ehcache there).
I'll need to templatize these few lines...on a Puppet Enterprise system, the user account is actually `pe-puppetdb`, not `puppetdb`.
Why does facts use a `:regexp-comparison` keyword, but resources uses the default `operator` value for dispatch?
Actually, these cropping and resizing methods are probably best moved into some external utility.
documentation is not accurate.
Eh eh eh, good point, will fix!
Same comment as in https://github.com/scala-ide/scala-ide/pull/220/files#r1994633
Shouldn't you also terminate the companion ``DebugTargetActor``?
Can you use `ScalaDebug.wrapInCoreException`?
set them to defaults, only used FALSE incase the function was ever modified like $extra becomes an array() in the future, but not a problem since it's a short function to spot
Could you change long lines so that they have between 70 and 80 characters per line?
Please follow style guide, bracket on a new line...
Нумерация индексов NSMutableArray начинается с 0, а не с 1. Поэтому у вас 1-й элемент цепочки ДНК никогда не будет мутировать.
Медленная реализация. Если percent == 100, то потребуется несколько сот проходов цикла, чтобы добавить последний элемент в elementIndex. Теоретически этот цикл может вообще быть бесконечным.
Есть переменные класса, о них рассказывалось в одной из первых лекциях. Посмотрите видео.
А создать потом объект можно в методе init.
Test comment doesn't match the expected output of the test. Should be:

> Output string should be hello?!hello?!
Probably more clean to check which sampling rate is closest of 48 kHz or 32 kHz.
Same issue with exceptions. Try/catch needs to be here, while `prepareOutput` doesn't need exception handling as it will be handled here
Can you rename this 'objectish'?
Yes, I'm on it. Let me just mock the infor for testing the lout.


On Sun, Nov 11, 2012 at 5:22 PM, Alexandre Gramfort <
notifications@github.com> wrote:

> In examples/preprocessing/export_ica_from_raw.py:
>
> > +# decompose sources for raw data
> > +ica.decompose_raw(raw, start=start, stop=stop, picks=picks)
> > +print ica
> > +
> > +###############################################################################
> > +# export ICA as raw for subsequent analyses in ICA space
> > +
> > +ica_raw = ica.export_sources(raw, start=start, stop=stop)
> > +
> > +print ica_raw.ch_names
> > +
> > +ica_lout = make_grid_layout(ica_raw.info)
> > +
> > +# uncomment the following two line to save sources and layut
> > +ica_raw.save('ica_raw.fif')
> > +ica_lout.save(os.path.join(os.environ['HOME'], '.mne/lout/ica.lout'))
>
>  Is fine with me. Just had this because it runs quicker, was working
> example if you wish ;-)
> no problem. Let's remove it then now.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/183/files#r2092821>.
>
>
I like :

`self._first_samps += r._first_samps`

but that's a detail
add a main short description of the function 

pep 257: http://www.python.org/dev/peps/pep-0257/
this suggests me that exclude should default to [] and not None and never be None. Rather than tolist() that will only work on np.array use "list(exclude)" that would work on any iterable object.
the first one is required to create http server to answer to http requests
the second one is written by mistake ... i'll delete it
This is called with `menu_item` as a parameter.  Should we continue to use that name throughout these private methods for clarity about what class the parameters are?
and Symfony generally uses ``/``. PHP 5.3 is able to use it for paths even on windows and it is far more readable (using the DIRECTORY_SEPARATOR was indeed required in 5.2)
This is completely wrong. Use `As an example::` The `::` will be replaced with a single `:` and the indented text below is seen as PHP code by sphinx.
Change `loose` to `lose` and `loosing` to `losing` on the next line
@rdohms yes, I know (I wrote those standards). But not in this case. The `::` shorthand indicates a code-block in the indented lines after and replaces `::` with `:`. For instance, if you have a sentence like this:

    Look what this code does:

    .. code-block:: php

        // some php code

You should turn it into:

    Look what this code does::

        // some php code

Those are rendered the same. But if you have a code block without a sentence you shouldn't use this shorthand.
ahhhh, very cool
I'm generally +1 on these changes, but why pull these lines?  The egg is generated when I package for pip. Definitely want it excluded.
Style issue: space between the if keyword and the parenthesis, no space before msg.
hmmm, but then here it seems like separatevars does some factoring after all.
Este archivo no debería estar en el commit.
> That's what I suggested originally.

I know. I'll add the non PIC version then.
Trying this test with some high byte stuff in there seems to make this fail, for instance:
"firstName": "Shared File System ー こんにちは世界",

gives:


======================================================================
ERROR: Failure: SyntaxError (Non-ASCII character '\xe3' in file /home/adrian/src/python-rhsm/test/unit/connection-tests.py on line 66, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details (connection-tests.py, line 65))
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/nose-1.1.2-py2.6.egg/nose/loader.py", line 390, in loadTestsFromName
    addr.filename, addr.module)
  File "/usr/lib/python2.6/site-packages/nose-1.1.2-py2.6.egg/nose/importer.py", line 39, in importFromPath
    return self.importFromDir(dir_path, fqname)
  File "/usr/lib/python2.6/site-packages/nose-1.1.2-py2.6.egg/nose/importer.py", line 86, in importFromDir
    mod = load_module(part_fqname, fh, filename, desc)
  File "/home/adrian/src/python-rhsm/test/unit/connection-tests.py", line 65
SyntaxError: Non-ASCII character '\xe3' in file /home/adrian/src/python-rhsm/test/unit/connection-tests.py on line 66, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details (connection-tests.py, line 65)
I'd rather remove the line instead of just commenting it out.
I didn't look into json schema specifically, but when this code started growing up I was thinking that there could be a better way to skin this cat. It looks like there is an implementation of a json schema validator for Python. I will note it in an  GitHUB issue for a future code improvement to do list.
My bad. I didn't RTFM, just the line above... :D
no need, see line 79
Can we get some jacadoc on these Bessel functions?  I have on idea what they are doing (or are supposed to do).
haha good catch.
`Config::get` already returns `FALSE`
Yeah reviewing our code in Mac it seems we are unzipping to a temp dir. Unzipping to resources doesn't seem to have any massive downside though.
Where has this gone?
Proof that the sentence was awkward...you got it the wrong way around. 

"When False, remove axes which fully overlap the newly created axes. Default is False."
or
"When True, keep any axes which fully overlap the newly created axes. Default is False."
or
I remove the whole thing...
I'm regularly getting failures inside this step. Here's a trace:
http://pastebin.mozilla.org/1977757

It's not occurring on all tests so it seems to be a timeout issue.

It seems as though the data_layer object has been disposed of before this step has run?
This is used in this pull. When calling the function from within the same JavaScript file we don't want it to tell Marionette that the script is finished onsuccess. This is because we're waiting for the onenabled/ondisabled when toggling the WiFi settings, and I suspect there will be other use cases.
have you tried looking where the comment says? :P
Should we ever be overwriting the default config file from the distro? This is already handled in the yum::yum recipe., here: https://github.com/opscode-cookbooks/yum/blob/master/recipes/yum.rb#L21

I believe the reason it is not included in the `default` recipe is to allow the cookbook to be included without running the default.rb recipe.
Is the AbstractLogger class somewhere? or to be written?

Same question for NullLogger, LoggerTrait and LoggerAwareTrait.
These all need to go away. `defrecord` already defines functions like this -- 

```clojure
(->TileRowCol 1 2 3 4)
```
hmm - looks like the fundamental logic here is the test: valuelen < minlen

so perhaps the code structure should be:

var valuelen = _.isObject( value ) ? keys.... : value.length
if( valuelen < minlen ) {
  fail....
}
I think you'll want to put a max-width on the role select dropdown as well, in case i18n would make it wider.
Yes - otherwise it generates the warning described in the JIRA.
I think this breaks our requirements for the case "user ID in query" not equal to "session owner". We want a server-side admin session to be able to do searches without constraint. And we want a search from normal-user session to be scoped by the access rights of the session owner. Walk through the thought experiment of user "johnqpublic" specifying a search with userId = "admin".
Is this empty Import-Package necessary? Looks like a no-op to me.
it should be ifndef _STRUCTURED_LOSS_H__
Imho everything is right except maintenance cost estimate.

Making abstract method public just in case forces all inheritors to support public access, forces developers to keep in mind larger scope, expands future class-wide or hierarchy-wide refactorings to whole project or solution and I'll estimate it much more than zero. I agree that initially making a method more open than needed doesn't cost anything as well as making it as restricted as possible, so there is no initial difference in cost. Difference appears later: you can always publish private method with zero cost, but you cannot easily close public method.

>> It's enough for me to be sure that possible changes will be applied easily.

> This also costs, and you have to pay that cost now, not knowing if it ever be needed.

I know. I pay. It saves me from moving to blind alley with total rewriting at the end.

Actually, I said enough on this question. If you are still insist on publishing methods back in hope that at some day they will be used for another target, well, I'll revert these changes.
我知道。`fs.existsSync = fs.existsSync || path.existsSync;`与`fs.existsSync = fs.existsSync ? fs.existsSync : path.existsSync;`你会选择那个？
i did like calculating actual scaling factors but it would be good to know the base when doing so.

For BB i just assumed that the sim would have the same res as the device, it it not the case?

Now 
```js
if(isBlackberry)
{
scaleX += 1;
scaleY += 2;
}
```
is an approximation, but if does the trick, then that good enough for our needs
Hmm... I think we need basic Sandboxing here so plugins don't accidentally interfere with the variables in the plugin loader's scope.

One trick I've thought of is loading the plugin into memory as a string, attaching it as an event to, say, a "Plugin" object, and then creating that object, firing the event. I don't know how well this would work, though.
Kleines Detail, aber sollte da der Checkstsyle nicht reklamieren? :)
I really don't know https://github.com/ruby/ruby/blob/trunk/lib/thread.rb#L97
``resource`` is not a class but a PHP type so you should remove the ``\``
Exception gets thrown here when status code != 200
of course :-/
mini-arquivo
@jtimberman mentioned this should be node['platform_version'].to_f to handle versions like 10.04.1
Actually, in that case there's another bug.  looking below in `#annotation_digest` we call `#clean_annotation` which includes the line number!  :bomb:
You can use Spotlight to find its location `mdfind 'Sublime Text 2 kMDItemKind:Application'`.
You can reduce duplication by extracting logic from `getRequiredOnCreate` and `getRequiredOnEdit` to a new `getBoolean` method.
Actually, you spotted a mistake on my side. It should read `$(document).on('click', 'button.confirm', function() {` to be compliant with the new jQuery event handlers. Not sure if a live event is needed here though: Symphony doesn't insert confirm buttons on the fly as far as I'm aware of. So we could either leave it as it is now or I change the line as mentioned above.
There already are. If it is post it sends an email and redirects him to account page, if get redirects him to email_confirmation.html. It works as expected.
O no. We will not do it like this. You should read this from used backends. You should never assume that somebody who will define a backend, will go all around the code and check where it has also to fix something. New backend is installed with settings file. And this is it. The rest should be done automatically.

So add some method to backends which returns you wanted tuples, and then create those choices with iterating through that.
But if you did it, you should put a comma at the end.
renamed to make it private; both collected in the same place
As far as I can see in the code DrawTop = YClear + margin. Do you think we
have to use just YClear?
Youre are right with the Bottom, it should be DrawRect.Bottom :=
DrawRect.Top + SectionHeight;


2012/9/19 BerndGabriel <notifications@github.com>

> In source/HTMLSubs.pas:
>
> > @@ -5519,6 +5548,12 @@ function TBlock.DrawLogic(Canvas: TCanvas; X, Y, XRef, YRef, AWidth, AHeight, Bl
> >      if DrawList.Count = 0 then
> >        DrawSort;
> >    end;
> > +
> > +  //>-- DZ
> > +  DrawRect.Left:= X;
> > +  DrawRect.Top:= DrawTop;
> > +  DrawRect.Right:= NewWidth;
> > +  DrawRect.Bottom:= X + SectionHeight;
>
> DrawRect.Top and DrawRect.Bottom cannot be correct.
> Please check or argue.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/BerndGabriel/HtmlViewer/pull/38/files#r1634587>.
>
>
fixed in 74c17c7
This is because the email field now allows null.
It can be like 
'''
if(_renderer.shareContext) _renderer.shareContext = false;
'''
one additional Location after all players have completed their Settlement phase.
The opening brace must be on own line.

This rule suitable for any coding standard that you mentioned :)
yup i am using Dreamweaver do you know any other good ide?
OK, is it possible to do it just for specific lines ?

2012/3/29 Sanne Grinovero <
reply@reply.github.com
>

> > @@ -97,6 +98,7 @@ protected void configure(Configuration cfg) {
> >       public void tearDown() throws Exception {
> >               //runSchemaDrop();
> >               handleUnclosedResources();
> > +             cleanUp(session);
>
> so even if you're not allowed to reformat the whole file, next step is to
> apply the formatting rules on the new code.
> this should be
>
> `cleanUp( session );`
>
> (with the spaces)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hibernate/hibernate-ogm/pull/60/files#r621987
>
Here also I wouldn't like to pass ``null``. The actual modification here has to be consistent with ``EntityChangeNotifier#entityChanged(Session, Object, AuditWorkUnit)`` code which notifies about a change in collection owner.
I trust you. :)  Merge it.
i think you for forgot to remove theses...
inline enforcement no longer seems to exists...
The reasons are That `DateTime` is a `Comparable<ReadableInstant>` and is not `Comparable<DateTime>`.
And `Ordering` type parameter is nonvariant.
Should use the insert_if_not_exist function above.
I think we should be more restrictive here.

```ruby
/\A#{filter}\(\di\)\z/
```
How about `path.end_with?('.yml')`?
хочу уйти(
ок.
This change makes it harder to test, because we can now no longer inject a `MockHttpConnection` for example. There are ways to get around this, but they suck.
return [].concat(this.children[id] || []);
No it's correct the way it is. But, now that you mention it, that variable could do with a bit of renaming. This function apportions the cost of the line item over the club-s allocations. The amount_left variable represents how much is left to be apportioned. If we go through all the clubs allocations, and there is still money that hasn't been assigned to an allocation, it means that there wasn't enough money in the allocations to begin with.
created improvement ticket: https://issues.connectopensource.org/browse/GATEWAY-3172
clarify: "because we're drawing the images via photon shooting" (i.e., if using DFT we'd have to include the default square pixel)
The user might wonder what use this is; could clarify that this gridded array can be subsequently queried to get the shear at various positions using more calls to ```getShear```.
split words with a `-` or `_`, e.g. 'down_logo`
This is good.  Right now it has an implicit cast from float to int.  I have to remember to change this over break.
Yes. It gives the user an option to enter their email, but that is also not required.
Could you please try to use http://static.springsource.org/spring/docs/3.1.1.RELEASE/javadoc-api/org/springframework/web/bind/annotation/RequestHeader.html?
can change this to a package private static inner class, and move MAX_ATTEMPTS there.  Then, the callable is testable independently w/easymock or the like
If it's configuring something, why is the verb a get?
The parameter is an array here.
Não, não pode sim porque pode ter mais de 1 voo para aquela aeronave, por exemplo se aeronave for ID 1 logo haverá tres voo
Added a comment, but it's the case where you have a service with a wildcard namespace, and two entities with the same name in that namespace. When requesting something from one of them, you have to use its namespace instead of the service's in order to avoid collisions.
Misplaced arguments, also all those 3 `assertEquals()` can be merged into one IMO.
Pourquoi utiliser options partout ?

un simple 

```
apy_data_grid:
     pagerfanta:
          enable:
          prev_message
```

Ne sufirait pas ?
We have an population of greater than 2 here, this should be removed.
Nononono.  If you are gonna do this, use "return ..()" so whatever return value ..() gives is preserved.
I could not reproduce the error in core. Was there some other change that fixed it before?
(It also went into that pull request by accident)

Am 21.11.2012 um 21:24 schrieb Robert Krahn <notifications@github.com<mailto:notifications@github.com>>:


In core/lively/morphic/TextCore.js:

> @@ -174,7 +174,9 @@ Trait('TextChunkOwner',
>          domChanged = this.fixTextBeforeAndAfterChunks(chunks);
>          domChanged = domChanged || this.removeNonChunkNodes(chunks);
>          chunks.last().ensureEndsWithBr();
> -        if (domChanged && selRange) {
> +        var newSelectionRange = this.getSelectionRange();
> +        if (domChanged && selRange ||
> +            (newSelectionRange && selRange && newSelectionRange[0] !== selRange[0])) {


using multiple && and || in one expression without parenthesis is pretty bold.

—
Reply to this email directly or view it on GitHub<https://github.com/LivelyKernel/LivelyKernel/pull/71/files#r2198983>.
Sounds fine, I'll take care of it sometime next week. #40 is merged and we have a clear way forward, so there's no need to rush this. I doubt anyone will be looking at it in the next couple weeks.
parse[0] используется в куче мест. Может имеет смысл переменную сделать с удобоваримым названием?
This whole tool is a huge hack, and required tons of trial and error with mostly undocumented Ray functionality. However, it does seem to allow us to simulate a player pressing the WASD keys in an attempt to reach a certain position in the world, which allows for complete outside-in UI testing.
actually get* is not a bad name, just return the extended token you just requested for. set* is not a very good name because you are not setting anything (not passing any parameters, see other setter methods). 

If you really don't want to return the token I think request* would be a better name?
would be even better.
please wrap message here and on line 104
I am not really mad on this loop. I think we should change this for something that mirrors the the syntax of the functions of the Select class ( selenium.webdriver.support.select.Select ) but we can do that in the future.
`such as` なので、「このコマンドは以下のような出力を行います:」などがよいのでは。
このようにコメントできます。
Agreed. That was ported over from mainline. I'll see if it works fine without.
Missing heading underline
We try to avoid acronyms so `RestrictedBolzmannMachine` sounds better. 

Also, we may want to bootstrap the `neural_network` module.
On Mon, Oct 01, 2012 at 02:29:34PM -0700, Andreas Mueller wrote:
> > +    The graph should contain only one connected component, elsewhere the

> I don't think this is true. If there are more than one components, than the
> first eigenvector (or was it the second?) will separate them afaik.

In my experience, it doesn't work too well.
This loop will be a performance hit, and many use-cases don't need it.  We need to try and  do this  without the loop.
Woops, my mistake.  Fixed.
Isn't this an ``int``, what does it mean to be a ``binary tree object``?
the default affinity is not listed in the docstring. we should refactor this to use any of the pairwise kernel metrics. gamma is very specific to rbf. so we should may be have a flag (e.g., `kernel_options`) that takes options for the specific kernel. i think we can leverage all the cool stuff that has been done in pairwise_kernels directly here.
I can split the solution into steps:
1. add proposed property replacer into server module
2. if property replacer and its location will be approved add code for
parsing.

Is this way applicable?

23  2012 . 1:58  Brian Stansberry <
reply@reply.github.com
> :

> > @@ -300,6 +302,16 @@ private static void parseProperties(XMLStreamReader
> reader, Properties propertie
> >      }
> >
> >      /**
> > +     * Resolve expressions.
> > +     *
> > +     * @param variable the expression
> > +     * @return expression's value
> > +     */
> > +    private static String resolve(String variable) {
> > +        return StringPropertyReplacer.replaceProperties(variable);
>
> +1. That class also has the enhancement for dealing with environment
> variables that Scott Stark added to jboss-dmr. I don't believe that is in
> jboss-common-core unless someone added it and did a new release.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/jboss-as/pull/1561/files#r478234
>
I don't like that it overrides the -j option from command line.

The following line allows NUM_CPUS to have priority while having a -j option on the command line also work when NUM_CPUS is not set

```makefile
J_OPTION      = $(if $(NUM_CPUS), -j $(NUM_CPUS), )
```
Should append '-password' and password as separate items
Should deal with integer and not string
But -t doesn't have a value and we provide the ndk location with `--ndk`
Dans la quarantaine, est-ce un `enregistrement` ? 

Dans la quarantaine, l'élément n'est pas dans Phraseanet
This is broken for me under rails 3.2.8. Shouldn't it be:

```ruby
haml_path = File.expand_path("../../vendor/assets/javascripts/haml.js", __FILE__)
```

?

But I can't believe I'm the only person to have encountered this, so what am I missing?
Damn, you seem to keep having formatting problems :) Please enable save action !!
Ok, I added comments to what I *thought* was the logic... they may be wrong...