labofsahil/github-event-dataset-2012 huggingface dataset:
Yes I noticed that. I dislike the "View" name too, WDYT about TodoWidget ?
this should be ``bind`` as ``bindRequest`` is deprecated
yes make it empty and extend the Common Persistence ones and mark as deprecated.
Missing space before IF and open parenthesis
Oh good, hearing that background makes me feel less worried about something subtle having gone wrong that I couldn't figure out!
You could drop this line.
Incorrect path to background image
Why are these two lines similar?
Shouldn't that be `Keyerror, e` since the second item is the exception object, not the key? Also, is it needed? Your exception handler isn't doing anything with the exception object.
this should be "self.toolkit()", not "self.cfg.toolkit()", right?

same in latter lines
Why would you put PCR number in code?
Sure, please send a pull.
The war EJB packaging is new to the EAP 5 base, thought to mention just in case they would like to repackage for jar/ear
Pull request is here: https://github.com/jboss/jboss-bom/pull/6

Once this is released, I'll update this pull request.
backticks for `message`
I agree, there should be some way to share logic here.  I'll give it a shot as I make my cleanup pass.
No need for the `== 0x00200000` part, just like in C a non-zero value is 'true' for boolean purposes.
typo   fileis -> file is
I don't think this is necessary if you're using the bundled JS with django-browserid.
Not really. I saw some plugins that use fetch and others that use bool ok = peekProc (...). Does it make any difference?
I really don't see why we are switching from the document id (which never changes) to the slug (which does change) -- what is it about the existing code that you are trying to fix?
OK, while I guess using the actual class would give the same result
What about ondemand testruns, which currently have a subject like:

> Firefox 13.0b2 build1 linux en-US: ondemand_l10n finished
Indentation looks a little screwy here
Thanks for the detailed explanations :-)
I could take a stab at it.

On Mar 23, 2012, at 10:38 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>
>
> Hmm, looks like you're right. I'll see if I can find time to write a script to fix all of those.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Khan/khan-exercises/pull/17326/files#r599930
yep, it's handy
Perhaps the interface would be cleaner if you passed a string with the name of the angle, like "ABC" that then gets split into left, center, right? At any rate, naming the angle as center, left, right in the arguments seems wrong since referring to angle ABC as BAC is confusing.
I took the file from master in the RiotGames/nexus_cli.rb and reapplied my changes to the get_artifact_custom_info task. The formatting should be the same as before now.
That removes it from VIPs from what i see :(
Not really. I did it to keep more symmetry with the $endTime variable. Your call...
these docs are from elsewhere ;)
Is get_bool("MCOLLECTIVE_TIMEOUT") correct?  Shouldn't it be something about the log level?
camelCase -> `sortAndGroupEvents`. Sorry to be picky, but it just makes everything consisten and our lives easier down the line.
I would love to learn more about what you guys (and the Clojure community) consider "best-practices" for this.  My instinct is that there are some valuable things that come along with the Exception class heirarchy, and that as long as you aren't doing something stupid with them (like using them for everyday high-traffic control flow), the "heavyweight" object would be a very minor concern.  I'll read up on try+ though.
I think that should be set by nginx, not node.
I think throwing an error is probably good enough. This TODO can probably go.
Might be worth splitting this out into model.user.js and model.group.js
No I removed it from the file.
A partial function would be nicer in this case, i.e., ``case (key,value) => ...``
Yes, you can set paths for compiler plugins, for instance.
K. Does it makes sense to add a comment for this (I'm wondering if there is a right place...)
My issue with this is `filter()` should really be on the `RelationshipQuerySet` to maintain chainability- and then the `RelationshipInstance` could just return a filtered QS, like a traditional Django manager. Could you refactor this to there? It'd require a bit more code (since it would break your "adding to a new RelationshipInstance" hack) but I think it'd be better for future work.
Best development practices can't be found in the Javadocs, but can be found in the Developer's Guide.
changed tnx :)
Sub-classing again? i.e. this would appear in RegularDeck and Deck would become an abstract base class?
need to add pagination in future
Oh, good point, I didn't see that.
Are you saying that ASP.NET sessions are lazily created upon the first use of `HttpContext.Session`?  Interesting.  I can see your point then.

As for the domain name issue, I'm concerned about that drawback.  Windows Azure, in particular, makes it trivially easy (in fact it's the dfeault) to set up a production site at *.cloudapp.net, which as you know means everyone is sharing a second-level domain name.  I know there is guidance that folks *not* use this default, but nevertheless this, and *.appspot.com, and others, make it easy to do.  Is there no way to make the HTTP cookie local to the exact host name rather than scoped just to the domain?
Да, я это понимаю, мне по условию задания нужен именно этот тип массива:

В классе создайте переменную DNA типа NSMutableArray – массив из 100 символов.
There is no guarantee that gen mutates to itself, leading to no mutation.
Совершенно верно. Метод менялся, а в h-файле не поменял. Согласен, совсем не хорошо получилось :)
The bigger concern is that all the ping related pull requests have broken tests in different ways.
User.find(answer.user_id)  =>

    answer.user.email
мм, а почему ты создаешь здесь все роли а используешь только одну?
почему не создавать только ту, которую будем использовать?
se param é um array porque está no singular?
clean_tag is not used.
Done in 6d3aa66acfe269509358052d549cd1600ddf82ad
No way, bro. Need to leave the original.
You can simply add a tracer of cache free here. And you will repeat the bug.
```
# Reader:setZoom globalzoom_mode: -1  globalzoom: 1.2176560121766  globalrotate: 0  offset: 66.2100456621 0  pagesize: 467.57989501953 800  min_offset: 0 0
# free cache: 411_1.2176560121766_0_1 ttl: 0
# free cache: 412_1.2176560121766_0_1 ttl: 0
# free cache: 413_1.2176560121766_0_1 ttl: 0
# free cache: 424_1.2176560121766_0_1 ttl: 0
# free cache: 425_1.2176560121766_0_1 ttl: 0
# free cache: 426_1.2176560121766_0_1 ttl: 0
# free cache: 427_1.2176560121766_0_1 ttl: 0
# free cache: 436_1.2176560121766_0_1 ttl: 0
# free cache: 437_1.2176560121766_0_1 ttl: 0
# free cache: 414_1.2176560121766_0_1 ttl: 0
# free cache: 415_1.2176560121766_0_1 ttl: 0
# free cache: 416_1.2176560121766_0_1 ttl: 0
# free cache: 417_1.2176560121766_0_1 ttl: 0
# free cache: 428_1.2176560121766_0_1 ttl: 0
# free cache: 435_1.2176560121766_0_1 ttl: 0
# free cache: 434_1.2176560121766_0_1 ttl: 0
# free cache: 430_1.2176560121766_0_1 ttl: 0
# free cache: 431_1.2176560121766_0_1 ttl: 0
# free cache: 429_0.91324200913242_0_1 ttl: 0
# free cache: 418_1.2176560121766_0_1 ttl: 0
# free cache: 419_1.2176560121766_0_1 ttl: 0
# free cache: 420_1.2176560121766_0_1 ttl: 0
# free cache: 421_1.2176560121766_0_1 ttl: 0
# free cache: 422_1.2176560121766_0_1 ttl: 0
# free cache: 423_1.2176560121766_0_1 ttl: 0
# free cache: 432_1.2176560121766_0_1 ttl: 0
# free cache: 433_1.2176560121766_0_1 ttl: 0
# free cache: 410_1.2176560121766_0_1 ttl: 0
# rendering page 438
# page_indicator 438
# pre-cached page  438  and restored offsets
# globalzoom_mode: -1  globalzoom: 1.2176560121766  globalrotate: 0  offset: 66.2100456621 0  pagesize: 467.57989501953 800  min_offset: 0 0
```
When rendering page 438 there is no free cache slot left. So the program just free all previously cached page from page 410 to page 437. Actually what we want is just making enough free slot for page 437 and just kicking out page 410 or page 411. The test is running with PDFReader. Similar results is available with koptreader. And the self.cache_current_memsize is not accurate either which will make it hard to free any slot in some case in koptreader.
The login fragment has two interfaces. The first interface is just for logging in. The second is an interface for registering. This function will switch the view elements around depending on a state variable. We renamed the function to toggle view.
could do `<%= link_to 'Change your password', change_password_path %>` since you've got the path matched up in routes.rb
异常是否作为一个:exception事件通知handler?
Inheriting from ``dict`` as you suggested provides more extensive backwards compatibility (e.g. modifying labels and adding new values to the dictionary, using ``.get()``, ...), but I guess as a more general question, should dict usage be explicitly deprecated?
More for my understanding, why do we want boostrap here and are we setting up the attributes for enable/disable?
We have `@summary` and `@description` already, can we use them here?
I did it this way because we're dealing with escaped html:

```
&lt;header class='test'&gt;testy&lt;/header&gt;
```
This is a bad idea IMO. The same form type will generate different translation keys depending of the place where you use it, making it difficult to reuse form types.
Thus, you should only do it when the label option is ``null``, to avoid replacing a label configured explicitly
Love these smileys
Do you know how? I looked and couldn't find a relevant lifecycle
hook/callback. I did not want to implement some horribly ugly monkey
patch.

On Tue, Jun 12, 2012 at 2:12 PM, Geoffrey Hichborn
<reply@reply.github.com>
wrote:
>> @@ -1090,12 +1090,6 @@ def do_after_all; end
>>        assert_equal now.to_i, @dummy.avatar.updated_at.to_i
>>      end
>>
>> -    should "return nil when reloaded and sent #avatar_updated_at" do
>
> This should be updated to ensure the correct value for avatar_updated_at is returned after reloading the model.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/thoughtbot/paperclip/pull/914/files#r969391
Лишний пробел после инта.

Кстати, ты мог бы вообще слот без параметров сделать. signal (a, b, c) вполне можно подключить к slot ().
Перед открывающей скобкой.
Он неюзабальное гово, которое корявит весь вид
you should add an empty line between constants and properties
maybe you can do something like that :

"Upgrade Phraseanet datas from older version\n".
"\nSteps are".
"\n\t- version 3.1 : records UUID".
"\n\t- version 3.5 : metadatas upgrade"
Also we use spaces between variables and relational operators.
Missing class javadoc
@gregbell #todo! :)
I agree with @galderz here. Can you provide an example so that we can work out a better way to handle this
What does SinglePc stand for?
Sorry, IDEA must have slipped it in when I wasn't looking... I've now removed it.
This comment serves no purpose other than to make it harder to read. Would you mind just nuking it?
Shouldn't we be moving to new style classes anyway?  Any harm in updating the class to new style (and then using `__getstate__` instead?)
I was told it was the way marionette is because it's made in javascript. I would love these to be synchronous on the marionette side.
More and more I'm wanting to see real delegates.. and separate ones for image/video. Maybe eventually.
Why not just attr_accessor on all of these?
`structure.items[pageSelected]._canView === false` --> `!structure.items[pageSelected]._canView`
@jsloane You're right, thanks for catching that.
Aha, je fakt, ze to koliduje trochu s tim timer_start(). Hm hm, tak ja to prejmenuju.
No harm in that. I can change them to _ .
Are these comments intentional?
- Are you sure the variable ``$this`` exists, or did you maybe forget to declare it?
- Please always use braces to surround the code block of IF statements.
Are you sure that the method "getManager()" exists? I also found this on lines 45, 74, and 135.
Are you sure ``null`` has the correct type as "null" is not a sub-type of "array"?
Ich denke es wäre schöner wenn das BeerModel nur TagModels kennt.
LOL. It was certainly successful, I didn't figure it out. Maybe I should sacrifice 80col for a better name than `uniqn`? :P
I think all global vars and classes should be prefixed with the cfdc_sketchname_ as well.
is_system: true - If we foresee reusing this flag for other messages of similar behaviour (or are of type 'system'), then it seems like it would make sense to set this higher up on the chain in the server side, not the client side (e.g. message-maker.js). Although it is true that not setting is_system to true immediately makes it false, if we do that everywhere then it's really difficult to track things down.

Is there no way we can create this message on the server side? I guess either way is fine for now but in terms of maintainability and readability, I prefer to push attributes of messages to the server.
this can potentially crash
Added links to the references for the functional and algorithm headers for anyone wanting more information on std::less, bind2nd, and count_if.
it should be a RuntimeException here. ErrorException has a clear semantic in the PHP doc: it is intended to be used by error handlers wanting to convert PHP errors to exceptions
I guess there should be. I hate `tracker.check_initializers` and it needs to die.
Yeah, i can add options to the requests. We probably should
Yes, or maybe some other class. Map will be a part of widgets or?
(Not in the code comment, here.)
Why you are at it. Put all object/dict keys into strings. So `mapTypeIds:` -> `'mapTypeIds':`
OK. We could have it like this: button is enabled only where there is non-space content in the textbox.
But what is then that comment about? About reusing default value?
Ah, now I remembered why I have `--concurrency=1` in my development setup, because more does not work SQLite (which does not support multiple connections at the same time). But we do not use SQLite anymore.

Anyway, leave it in. Our development web server is still one-threaded. :-)
No need for two lines. You can merge these two imports into one line. Order alphabetically.
Мы о каких то разных парсерах говорим? Я говорю про парсер на гитхабе, ты про какой?
You should change this back to what Nat did using exclude. Using filter removes all of the announcements with no expiration, so it "erases" all past announcements. We'll figure out how to properly migrate those later for the next release.
Переменовать в getResourcesPath
Putting dependencies in devel currently works, but won't soon. Please place these with the main dependencies with a postfixed `if ARGV.build_devel?`
Please keep the curly brace on its own line
>  If you need to redefine x, move your function outside of the scope where you already have an x defined.

The problem is that it is hard to find out if you are redefining.  People with better brains than mine can remember every var name used in an external scope.  Consider this an accessibility issue for us impaired people.  :-)
Remove spaces inside parentheses. See PEP8.
This is internal method and if anything wrong passed, a runtime exception with appropriate description will be provided.
@amitaibu this pull looks good and tested fine, my only question is why sid is passed here to reset the static spaces cache of the object?
Should probably be a `const Camera*` here (and in the corresponding `Render` functions in derived classes).
This function collects only pure python modules. I think it should also include C extension modules (files ending with .pyd and .so).
Not too sure to be honest, I just copied the code snippet from
QSObject_FileHandling.m (used to get recent documents)
Can NSUserDefaults get prefs from other apps (not just its own)?

Anyways, the NSUserDefaults API would probably just call this anyway :)

On 17 June 2012 20:26, Rob McBroom <
reply@reply.github.com
> wrote:

> > @@ -12,12 +12,34 @@
> >
> >  @implementation QSDownloads
> >  - (id)resolveProxyObject:(id)proxy {
> > -    NSString *downloads = [[@"~/Downloads"
> stringByExpandingTildeInPath] stringByResolvingSymlinksInPath];
> > -    NSFileManager *manager = [[NSFileManager alloc] init];
> > -
> > +    // Try and get the user's downloads folder setting (set in Safari)
> > +    NSData *downloadsData = (NSData
> *)CFPreferencesCopyValue((CFStringRef) @"DownloadFolder",
> > +
>                    (CFStringRef) @"com.apple.internetconfigpriv",
>
> Why not get this with `NSUserDefaults`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/quicksilver/Quicksilver/pull/943/files#r998644
>
Sure. Just still trying to get a sense of things.
I'm fine with removing the underscore. 

Although I do suspect the combination of empty ``__init__.py`` and no underscores for private submodules/objects will become a headache in the long run.....
hey,

sorry for the late reply.

yes, you are right. it can be removed.

Can I also have a question about the exception inheritance chain. It
doesnt seem to be quite sophisticated.

There are:
\Exception
AMQPConnectionException (this happens when the server closes the connection)
AMQPChannelException (this happens when the server closes the channel)
AMQPException
....

well, the last one has some weird overriding on the Exception class, like this:
    public function __construct($reply_code, $reply_text, $method_sig)
    {
        parent::__construct($reply_text,$reply_code);


swapping the variables and so on.

there must be one general connection exception in the inheritance
chain to catch all related to connection problems.
I am sure that this hack above was only introduced because of some
debugging reason, am I right?

I am overriding it to clean it up.

best,
David


On Thu, May 10, 2012 at 11:44 AM, Alvaro Videla
<reply@reply.github.com>
wrote:
>> +{
>> +    die("You must specify the sleeptime in usec as arguments.\n");
>> +}
>> +
>> +try
>> +{
>> +    echo "Creating connection\n";
>> +    $conn = new AMQPConnection(HOST, PORT,
>> +                               USER,
>> +                               PASS);
>> +
>> +    echo "Getting channel\n";
>> +    $ch = $conn->channel();
>> +
>> +    echo "Requesting access\n";
>> +    $ch->access_request('/data', false, false, true, true);
>
> I think access request is not used anymore
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/videlalvaro/php-amqplib/pull/18/files#r800755
This should be <.
Make up your mind! Do you want sexy new CSS or ugly old `<font>`?

Oh, and did you really have to close the `<font size="5">` and open another one exactly the same right in the middle of the sentence?
Why the change from x to r?  x is the standard notation in the papers I've seen for r/r_s, whereas r could be confusing since, looking at the code, I (and I think many people) would assume it is the radius.  Would it be okay to go back to x, and maybe expand the docstring slightly to say that ```x=r/r_s``` and give a citation to a paper with these equations?  (I can do that if you prefer.)
OK, can modify... Are there any other opinions about this?  The only thing I can really think that people will be wanting to use `ImageView`s for is to co-add / whatever an externally-built (e.g. via a numpy array) `ImageView` to a galaxy image generated by GalSim... 

This doesn't seem so implausible.  But that usage is not even covered in what I've got here, to be fair.  The only other thing is that `draw()` and `drawShoot()` output `ImageView` instances.
yeah, I guess that was confusing.  My thinking is "smaller pixels" = "higher resolution."  Will clarify.
I agree.  I can't think of a case when that wouldn't be an error.

On Fri, Nov 2, 2012 at 2:45 PM, Sean Cribbs <notifications@github.com>wrote:

> In riak/transports/pool.py:
>
> > +    @contextmanager
> > +    def take(self, _filter=None, default=None):
> > +        """
> > +        Claims a resource from the pool for use in a thread-safe,
> > +        reentrant manner (as part of a with statement). Resources are
> > +        created as needed when all members of the pool are claimed or
> > +        the pool is empty.
> > +
> > +        :param _filter: a filter that can be used to select a member
> > +            of the pool
> > +        :type _filter: callable
> > +        :param default: a value that will be used instead of calling
> > +            create_resource if a new resource needs to be created
> > +        """
> > +        element = None
> > +        if not callable(_filter):
>
> Good catch. I was trying to simplify the logic down below by having a
> truthy filter if none was specified, but it would be better to raise an
> error if a non-callable is given.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/basho/riak-python-client/pull/157/files#r2021554>.
>
>
We might want to store this in an environment variable
The escapeLiteralForDoubleQuotedBash has the same effect as the replaceAll above, but unit tests are definitely a good idea here, easy to get wrong and mis-count slashes
Missed this, but that's exactly what I was thinking.
Method should probably be protected, given writing this for sub-classing.
Perhaps store ReusableMachineTemplate as a field? It's the same every time.
No strong feelings...
J'irais un cran plus loin : computers are very bad at summarization ou qq chose du genre.
I was treating this as the library file defined by pubspec layout
One could argue that readability is more important than speed given that *all* your implementations are very fast. :)
Whoops, thanks for spotting it! have fixed
I've deleted :locals
you can just do `require "helper"` because of

    Rake::TestTask.new(:test) do |test|
      test.libs << 'test' # <------------ this line
      test.pattern = 'test/**/test_*.rb'
      test.verbose = true
    end
Deliberate, I think. They enforced a lot of new stuff on indexes, such as making you restart splunkd after enabling or disabling an index.
No, subjects.json is the file where modifications are stored. Instead this one is used as backup in case you want to restore the initial situation (that's the reason why I named it .bak ...).
whats the purpose of getKeyHash?
`if (type_x === type_x)` -- `if (true)` ? :-)
Can you please do `# @importFrom utils readRegistry` and just call the function name directly?
Same here. We may want to deprecate until the next major version instead of removing the method
actually you do. Look at how _jobLoader_ is created at line 59
My intuition is that it is far more valuable to write cucumber tests that assume a running conductor, because most of what we want to test is the actual interaction between the cli and conductor (especially with the philosophy that the client is as thin as possible).  We can cut and paste xml that we currently get from conductor and use that for our tests, of course, but that is more work and will require more work down the line were the xml formats to change.  So that is A) more work now B) more work later and C) less likely to find real problems in the cli, especially later.

That being said, I anticipate the "not require additional services" is a red line, so I'll go against my intuition...
Pushed

Summers Pittman
>>Phone:404 941 4698
>>Java is my crack.




On Wed, Nov 14, 2012 at 12:04 PM, Bruno Oliveira
<notifications@github.com>wrote:

> In test/org/aerogear/android/impl/pipeline/RestAdapterTest.java:
>
> > +    public void testSingleObjectRead() throws ParseException, InterruptedException {
> > +        GsonBuilder builder = new GsonBuilder().registerTypeAdapter(Point.class, new RestAdapterTest.
> PointTypeAdapter());
> > +
> > +        final StringBuilder request = new StringBuilder("");
> > +
> > +        HttpStubProvider provider = new HttpStubProvider(url) {
> > +
> > +            @Override
> > +            public HeaderAndBody get() {
> > +                return new HeaderAndBody(SERIALIZED_POINTS.getBytes(), new HashMap<String, Object>());
> > +            }
> > +
> > +        };
> > +
> > +        Pipe<RestAdapterTest.ListClassId> restPipe = new RestAdapter<RestAdapterTest.ListClassId>(RestAdapterTest.ListClassId.class, provider, builder);
> > +        final CountDownLatch latch = new CountDownLatch(1);
>
> Also the reference to listClass can be removed
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aerogear/aerogear-android/pull/37/files#r2126786>.
>
>
Пробелы...
Название пакета не соответствует названию директории (они регистрозависимы).
Grouping together reset and theming? Right now they are grouped yb and wb, then by and bw as they share styles.
PEP8 issue: base.py:53:36: E225 missing whitespace around operator
Need info: Shouldn't this be a list? Then you can just say ['phase'], I think. We should follow standards though. Please indulge.
You may want to use `truncate_words()` here, which is in `django.utils`.
I'd almost prefer this conditional to just be 'if (ns == null)', and maybe just log a warning if the namespace is not something we've seen before.  The possibility certainly still exists for the file to be unreadable, but it seems better to take our chances than to explicitly forbid anything but those two schema versions.
delete extra sass file
Or maybe:

    Return edge image based on magnitude of **** transform
Hum.... that is true... I was afraid that the leafs become pure if I use other labels. Maybe I should try to put something in the cython code.

You are right, ``len(X)`` doesn't work. I guess I should use ``safeasarray`` and then shape.
In the non-metric version,
MDS takes a distance matrix? that saddens me.... opened #1390.
The change of short option name that you did in an earlier patch needs to be reverted as it break compatibility with the Studio.  I thought they would code to the long options but they coded to the short ones instead.

Specifically -p needs to stay associated with --ndk_path
Should add a TODO comment here
The javadocs needs backporting?
As it stands it leaks for the inline table case
Probably should mention 'renamePhysical' here and in the other exceptions above?
For name and row I don't think it makes a difference whether the occurs are because there is only one element in the sequence.  I probably would have put it on the element even then, but we've historically had the min/maxOccurs on the sequence.  And some of our stylesheets at least are dependent on how we format the schema.

But for "name, no row" it may have siblings in its parent sequence and its the individual element that is affected, so it has to be on the element.

I don't think the "none" case makes it this far.  The default element of "Row" is set before these routines are called.  I represented it here as simply dumping the contents of each row just in case we ever wanted to support it.
most of the benefit lost since this will only be true for the last node.
Should this also be done for the super case?
There are all sorts of ways of installing npm. For example: https://github.com/rach/virtualenvwrapper.npm

lesscss.org recommends `npm install less`

gruntjs recommends `npm install -g grunt`

I've never seen `sudo npm install ...` before.
Ha! Silly me!
Move all of this out of the models.
Please stick to consistent variable name scheme. s/plotData/plot_data
I can totally do this, much better.
This (and L37 below) are basically equivalent to {% static "orderable/orderable.js" %}. Don't know why it's not in django docs, but simply prepending `STATIC_URL` isn’t always helpful (especially when using django-storages + S3).
not wise to bump other people's versions :p
We should be able to say `stable` instead of `relatively stable`?
As we really want them to be stable!
/s/visable/visible
Is it safe to assume that need_to_run? will _always_ be executed before execute_changes?  If not, @aug may be nil - the method used to call open_augeas here just in case.
Why doesn't this just return a boolean value? It seems like you could simplify add_word and blacklist word if you did:

```ruby
def add_word(word)
  return unless valid_word?(word)
end
```
this looks like dead code to me...
revert this file. the only changes here are formatting changes.
grunt 0.4 won't have helpers anymore. Let's not introduce any new ones - we need to refactor the existing ones as well.

If its going to be used outside this file, we should move this into a separate module that we can require whereever we need it. If its only used right here, just make it a regular function.
I think that's the only problem I see.  I'll make sure this is fixed before it's in the framework.
Please use only reference to the specific technique used by the module, no general references.
Oh, I don't have a problem with the name `felzenswalb`. I just meant that none of the other functions end in `_segmentation`, so `felzenswalb_segmentation` -> `felzenswalb` (since it lives in the segmentation package, the segmentation part is redundant).

Oh, I forgot: you may want to rename the modules to prevent duplicate names for functions and modules (c.f. issues with doctest). This is problematic for `felzenswalb` since there is a function, python module, and cython extension so our leading-underscore convention isn't sufficient.
I agree this can be removed.  More elegant and better external solutions are available.  Some note about the scaling might be good to include up in `data`, though, so people in my kind of situation know they need to pre-process their inputs.
What happens when matrix is None?
I usually make the methods unerlying a property private so there's one clear way to do things (the property).
Yup, it would throw a 500 though.
The update functions are void now. The return statement is for exiting the function because nothing is appended to the array argument. Shouldn't that be `return` and not `return undefined`?
We keep using names like `obj` and `placeHolder`.  Can we use names that provide semantics?  Given the amount of nesting, it will make the code more readable.
Shouldn't this be `Cartesian3.clone(sunPosition, this._sunPosition);`
Or just keep a flag and avoid the search.
why isn't the currentTime set to the startTime in both changes?
It's not just good practice, it's required for good performance.  All members should be initialized in the constructor function directly.  Initializing them in a another function (even if it's called from the constructor function) will not have the same optimizing affected.  Even loops in constructor functions can cause the optimization to bail if a new member is added during the loop.  Best practice is to have all of the vars defined in one place directly in the constructor function, even if they are initially set to `undefined` and then immediately set in a helper function to something else.  @shunter can back me up here if I got any of the details wrong.
Also the subject of a bug:

https://jira.appcelerator.org/browse/TIMOB-6366

In which I suggest that we should create a constant for this rather than assuming that the group tableview background color will stay striped.
Can we move this out of the if/else statement? It looks like you're doing the same thing in both.
D'oh. ResumeTime also has to be multiplied by 1000.
The Timer class has nothing to do with Isolates. It's just in the Isolates library. Why? Not sure. But the bigger question is, do we should people how to write code that favors Dart or the DOM? If we favor Dart, we show them how to use Timer and now they know how to use a timer in the client and the server. Less to learn when writing apps across two different deployment targets. If we show them the DOM, now we have to teach them two things: window.setTimeout and new Timer. I strongly favor using Dart APIs whenever possible and teach a single way of doing it.
Can it really be something other than String?
i do understand your use case and i am very much looking forward to what you will enable with this change.
i was asking you to revert the whitespace change.
gotcha. thats fair.
here we save three times :(
Agree, although I'll note that this formatting does something slightly different to the standard traceback formatting as described in https://bugzilla.mozilla.org/show_bug.cgi?id=752375
that was my main concern, but honestly, this week is the time break backwards compatibility.
I think skip_migrations is ok for a name and it should skip db:seed part too.
Doesn't this assume that `fun` is a `string`? `fun` is an alias, so it could be a unary function rather than a `string`.
Gah, people clearly don't understand my sense of humour, or facetiousness. Fair enough, it doesn't really work in text.
The winkey faces should give it away though.

@jmdavis you're the "and an American" I referred to (I presume), and the google fight showed in favour of the popular consensus, and I conceded upon that. I'll buy the mob rule. I still think it looks stupid though!

@JakobOvrum I don't think I was being offensive (rather, being offendED)... I speak only English (are you trying to rub it in?). As such, it is all I have, and  I feel entitled to have an opinion. I already accepted I may be wrong according to some majority (and google fight). I'm rather surprised this thread grew as long as it is...
So this should be called `alignedSize` because it computes the total size of a structure with `n` bytes that must be aligned to `alignment` bytes. I think this only works if `alignment` is a power of 2, so insert `assert((alignment & (alignment - 1)) == 0)`.
I understand your point, but it does little in the way of convincing me. There is no way to instantiate Not in a way that doesn't ultimately create a value. Even the use examples of Not introduced aliases starting with lowercase.

Aren't we over-thinking this a bit, too? People will use the facility to get values. Yes, there's a template in the way of it and all, but that's more of a detail than core functionality. Why then split hairs instead of going for usability?
running the test suite - http://mojo.codehaus.org/gwt-maven-plugin/user-guide/testing.html
Kill this log line. (And: typo.)
I know we always do it that way, but still,wouldn't it better setting services as arguments instead of setter? maybe a topic for the mailinglist
rename those params to connectorIdToLink and remoteHostIp
Maybe this should be 
if (expr->isDatrow())
Except when the content in the tab shows content in-progress AND content completed. History remains victorious.. this time.
инит лист же.
И правильно HttpClient_
Whoops - I'm not sure why I removed that.  :/

On 27 February 2012 11:11, Sanne Grinovero <
reply@reply.github.com
> wrote:

> > @@ -410,7 +462,7 @@ public synchronized void futureDone(Future<Object>
> objectFuture) {
> >           if (sc.processed) {
> >              // This can happen - it is a race condition in JGroups'
> NotifyingFuture.setListener() where a listener
> >              // could be notified twice.
> > -            if (trace) log.tracef("Not processing callback; already
> processed callback for sender %s", sc.address);
> > +            log.tracef("Not processing callback; already processed
> callback for sender %s", sc.address);
>
> you shouldn't remove `if (trace)` as checking the static constant will
> always be better than actually invoking the tracef method
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/infinispan/infinispan/pull/974/files#r490659
>



-- 
Manik

Sent from my iPad 4 beta 3 with a telekinetic keyboard
This method is probably better implemented as: encounter.getObsAtTopLevel();  if obs.hasMembers().  But this way works too. :-)
Hi, according to my knowledge, the date is basically  any String object. I essentially put in a time stamp like this :

(taken from the MSH segment)
<MSH.7>
<TS.1>20120724064115</TS.1>
</MSH.7>

I believe this would do well as an initial draft of this ticket, and would like to commit it pending your approval.
Ok, so the CCE is what we currently get and now being paranoid with breaking changes I was strictly addressing it within the scope of the problem described in INT-2713.
So I won't fight since you've proven to be a better judge wrt breaking changes ;) so go ahead if that's the only thing do it before merge.
I thought I'd get a comment about this :-)

The compiler doesn't allow it. I verified it's ok because the LiteralExpression simply wraps the null.
I kind of like it that way, plus i need replyMessage definition to be outside of loop and it doesn't play well with this:
```
while (!theOneIWant && replyMessage != null){
```
if-else again, also, very small thing, but can the message be something like "Either of directory or directoryExpression must be provided" :)
Is composer available in the system or are you assuming it in the application dir?
Not everyone is using an SCM plugin to get the code, so there is not always a changeset available to use.
Should it look straight in the workspace then or do a glob or something? maybe a "git diff -someparameterthatgivesmethelistofchangedfiles"
Sorry, this is a typo. I'll fix right now.
It is a README file, so it won't break anything. Do it!
Can you simply remove these lines rather than comment out (in several places). If you need the original values, thats why we use version control.
Oh, now I see your example uses predefined rst commands -> I'll update to use it, too :+1:
Tarkoituksena oli harjoitella rekursiota. Ei pisteitä.
AFAIK Sphinx is not catching one line comments as description
`--name=read-only-1` -> `--name=read-annotate-1` to match up with the comment
If only in HTML, we can remove the note about the online version of the document.
@apex13 Renamed
This is already available via http://stackmob.github.com/stackmob-customcode-sdk/0.5.0/apidocs/com/stackmob/core/rest/ProcessedAPIRequest.html#getAppName%28%29.
similar to above; fix this so it actually tests something or at least does not fail.
I'm responsible for this.  Sorry.
Change to more generic way of getting a list of these entries
> I think names should be based on semantics instead of technical aspects. So the name should express what concept is modeled here and not how it is realized. Therefore I'd prefer DefaultConfiguration (because that's basically what this class is about, it represents the BV default configuration provided by the user), or maybe something like BootstrapConfiguration. The fact that the default configuration is realized in form of an XML file, is not really relevant for the user of this type, so it shouldn't shine through in the name IMO.

I agree in general, but in this case 'XML' is not purely technical. The semantic is to expose the xml based configuration. Unless we really want to leave the door open for other kinds of configuration I think _XMLConfiguration_ is a good name. 
I don't like _DefaultConfiguration_, because it seems strange in combination with _javax.validation.Configuration_. The latter has methods like _Configuration#getDefaultTraversableResolver_ , _Configuration#getDefaultConstraintValidatorFactory_ etc. If we now add _Configuration#getDefaultConfiguration_ we get things like _Configuration#getDefaultConfiguration()#getTraversableResolverClassName()_. I think it gets confusing between _Configuration_'s _getDefaultXYZ_ and the _getDefaultConfiguration_ method we would add. In this case I find _BootstrapConfiguration_ a better name.

> Btw., if going with the current suggestion, I'd prefer XmlConfiguration over XMLConfiguration as it reads better IMO and allows for better variable names (xmlConfiguration instead of xMLConfiguration).

 _XmlConfiguration_ is fine for me.
Is setEncoding() is called before or after setResource() or arbitrary?

After: setEncoding() should assert that resource is a ClassPathResource and do this.resource.setEncoding()
Before: setResource() should assert that resource String starts with "classpath:" if encoding != null
Both: Same as after/before with null-safe checks
Can we just use the new reddwarf-manage command here?
I would put *connection with mongoengine* into a link to [here](http://readthedocs.org/docs/mongoengine-odm/en/latest/django.html).

Probably better would be: *You must also connect mongoengine to the database.* No?
The non-absolute branch will need to prepend `get_script_prefix()`
This will fail as soon as there is more than 1 package in $pkglist. It would be better to put a "before => Apache::Module[dav_svn]" on the Package resource.
Alright then. Maybe should just remove it completely, but doesn't need to happen in this commit.
why forcing the filename ? The option should allow to change the target, not only the target path
public static PrimObject 

or 

static public PrimObject

jamesladd- what is our standard on this?
a better idiom than "node.children[0].textcontent" might be node.firstChild.textContent since it seems like it at least avoids some XPConnect binding creation overhead and might also avoid some C++ object instantiation.
Don't we still need the closure-library checkout to build the actual, resulting javascript?
"whether to clear all keys and values in the instantiated SharedPreferences collection."
Well, the simple modes are also enabled here.  This only enables the legacy mode based on certain accuracy values.
This should simply be ``if`` instead of ``elseif`` as the previous ``if`` leaves the execution flow
Need to have a check before executing this update whether the Iterable passes has records or not. Need to have a 
<pre><code>if(entities.iterator().hasNext())</code></pre>
 before QueryUtils.applyAndBind.
These are OSS concerns because they are deployed with our tool, correct @nukulb
I can't make up my mind if this alert is a good thing or not?
Need to add a test in the lib-shard-models/src/test/java/org/..../GwtTestSuite

That test will do a GWT compile on your code.  If you have used anything that is not GWT Kosher it will cry.
_('string') is for marking the string as translatable by gettext, same for <_summary> or <_description> in gsettings schemas

this._attribute is a programing convention meaning that the attribute is private to an object and should not be accessed directly
Yeah, I see your point, however it does pass on iOS.  I'll take a closer look on Monday.
Why the random strings here? There should be a comment explaining it.