labofsahil/github-event-dataset-2012 huggingface dataset:
Rather not. Rather do the opposite and move into hardware specifics. Some
hardware may support it.

A generic function with the list make sense though.
```
======================================================================
FAIL: test_codegen (miro.test.metadatatest.TestCodegen)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/z3p/Projects/miro/tv/linux/dist/usr/lib/python2.7/site-packages/miro/test/metadatatest.py", line 1235, in test_codegen
    self.assertEquals(self.errback_data, None)
AssertionError: ('/home/z3p/Projects/miro/tv/linux/dist/usr/share/miro/resources/testdata/Wikipedia_Song_by_teddy.ogg', OSError("call_command with ('/home/z3p/Projects/miro/tv/linux/dist/usr/bin/codegen.Linux-i686', '/home/z3p/Projects/miro/tv/linux/dist/usr/share/miro/resources/testdata/Wikipedia_Song_by_teddy.ogg') has return code 127\nstdout:\nstderr:/home/z3p/Projects/miro/tv/linux/dist/usr/bin/codegen.Linux-i686: error while loading shared libraries: libtag.so.1: wrong ELF class: ELFCLASS64\n",)) != None
```

Seems like `codegen-liinux-i686` is linked against the 32bit version of libtag.
This might be a bit pedantic, but can you name your variables in lowerThenUpperCase instead of under_scores? e.g., isSkybox instead of is_skybox.

I would like a consistent naming convention throughout the code. :)
No admin wants to see what their players are doing, do they? This is going to be underused!
why not "two spaces" here, but " *" instead?
I think isCallble!t1 is more better.
Furthermore, `static assert(allSatisfy!(isCallable, T)`  in function template constraint is the best IMO.
Damnit, can't I go *one day* without having to see Michael Jackson?
But, I want presents...
Ohhhh, interesting...
Possibly test_io* in test-all of CRuby 1.9? I'll check that once you finished refactoring.
Me, neither.  I just encountered it. :-)

--
http://gleamynode.net/
http://twitter.com/trustin
http://twitter.com/trustin_ko
http://twitter.com/netty_project



On Thursday, March 1, 2012 at 10:16 PM, Norman Maurer wrote:

> Thanks! I was nothing thinking about the possible NPE here
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/netty/netty/commit/9e8f8ac08cf6200fc07ef7934bf75b38f62b7c3e#commitcomment-1034243
> 
>
Thanks @fxn

(totally prepared for a "this is why inflections are frozen" to follow)
@derek I think the only thing we should support for buttons with nested `"yui3-button-label"` elements is using them as the place where we get/set the `label`'s `text` from. The whole point of support nested structures is to allow users to put in whatever HTML they want inside their buttons (as per the element type allows) and they cue us on where the label text for the button exists. Like this:

```html
<button class="yui3-button">
    <span class="icon-check"><span>
    <span class="yui3-button-label">Done</label>
</button>
```

I read through @rgrove 's concerns on IRC, and I think they can be taken care of by sticking to `value`/`test` getting/setting, even for the nested label element.
Oh, wow. This looks amazing. Well done!
place configure method first for easier reading
Oh, and it would also duplicate them in the syntax tree, wouldn't it?
Yep, thanks, added.
I 'm uncertain if this makes any sense, but is the issue that we need to identify all (and only) the callers of li() macro that can have their constant replaced by LoadRoot()? If we could do this, then we could do the replacement Erik suggests.

Can we use a backtrace() function within the if clause of li() where FindRootIndex() function finds a valid Root? Then we'd see all the callers that are subject to FindRoot replacement.

Backtrace is not implemented right in glibc on mips, but should work fine on the simulator, where all code-generation is done on the x86. I am thinking of these functions: http://www.kernel.org/doc/man-pages/online/pages/man3/backtrace.3.html

I meant to test this tonight, but ran out of time.....
Seriously guys. Today's like christmas. You rock!
So we check if the event isCancelled and if the listener isIgnoringCancelled when true cancel the call
this seems wrong for me? or am i wrong ?
yeay :) btw, i think you're right. It's better to name this forms `*Type` as they extends AbstractType. Just my 2 cents.
What about convert it to markdown?
https://github.com/mangosR2/mangos/commit/6f24b03a47d72cc2fcb2d61f48900acdf6d92318
http://www.youtube.com/watch?v=VkCDxu6Z2XE
Please leave it as /etc/.pryrc, there is no FHS convention against .rc files in /etc/ only mass bitching from Debian and Redhat over that in /dev if we are going to force the removal of the dot we might as well follow global convention and either name it /etc/pry.conf or /etc/pry but the proper way would be /etc/pry/pry.conf because it's not a systems application but we don't try to enforce the standard convention since some applications *cough* colourd break that.

The idea isn't to prevent users from customising pry, far from it, it's to allow me to say define cooline globally (where the user can surely choose Readline if they want) the way Pry loads anyways it would require a refactor of the actual way Pry loads to do what I assume you are implying I want to do.
Hey, Some feedback about the change :)

1. Not sure if we bother but it looks like we no longer redirect the standard input.
2. It looks like we redirect the std out/err later than before, e.g. when the LogToClient kick in (loggingManager.start()). Then we remove this redirection shortly before the operation is finished (LogToClient - see where we do loggingManager.stop()). This probably (not sure) means some stuff will not be printed to the file (e.g. any logging before LogToClient kicks in and any logging after LogToClient finishes won't be seen in the log).
3. Things worth checking out: how the foreground daemon behaves, how embedded tests behave.
Ok, but a copy won't create a memory leak if you handle it corretly. But yes, you need to handle it specially to don't have a leak.

I'm fine with a ticket.
Ah, yes, you're right!  That should be `x1 : merge (x2:xs)` =)
Nice one. Can we minimise/upstream this?
function PassMethod(This, Method) {
  return function() {
    return Method.call(This);
  };
}

User.prototype.c = function() {
  this.a(PassMethod(this, this.b));
}

Надеюсь, загадывалось не это.
Done can be merged!
As funny as that mental image is... just what's going on with this implementation? The `hashcode` field? Isn't it easier and better to do:

```java
@Override
public boolean equals(Object obj) {
    if (obj instanceof Player) {
        return ((Player)obj).getName().equals(getName());
    }
    return false;
}

@Override
public int hashCode() {
    return getName().hashCode();
}
```
This needs to be in an `after` block to ensure it always runs. If the test fails currently, the `.cane` file won't be removed.
the patch is solving the problem, which is great, but I'd like to understand why the panning mode was affected.
We dont change lat-lon for airspaces while we pan.
It may be something not related to the position, but to a missing calculation while fastpanning.
This is what is worrying me.
See commit message...
@kingofirony @Yulli, What is the x-coordinate of the center of the camera image, i.e. `image width /  2`?  Is there a method call for getting the width of the image?

Also, the current variable name does not quite describe what it represents.  Would `CAMERA_CENTER` be more appropriate?
GitHub hickup.. due to recent encoding change i assume (happened before), the files are ok..
This commit broke the scaffolding:

Foundation.hs:86:15:
    Couldn't match expected type `Approot MyFoundation'
                with actual type `a0 -> c0'
    In the expression: appRoot . settings
    In an equation for `approot': approot = appRoot . settings
    In the instance declaration for `Yesod MyFoundation'
version 1.4.1 now has CommonJS support built in.
Homepage comes first
Thanks for making that moar perty
Ahí actualice el admin de category para que funcione correctamente, recordar que es necesario hacer un php app/console doctrine:schema:update
Wouldn't it work just as well either way? Doesn't Int64-Int32 promote to Int64, even on 32-bit?
Was hat es eigentlich damit auf sich? Steht aktuell in jedem Test mit PropertyInteger :D
Was muss hier hin?
--interactive ?
ticket 3568 reveals that the cause of the crash is in this line, 'var->typeEndToken()->previous()' is NULL in this case. I'm not certainly sure how to fix this.
It does that if the wheels aren't successfully removed as children.  I am having the same problem.  It looks like there are two problems at the moment. Removing the wheels as children work with Pierrick's idea but "Wheels" should not be capitalized.  Change his script to:

children = bpy.data.objects[robot.name].children
wheels = [child for child in children if "wheel" in child.name]
for wheel in wheels:
    wheel.parent = None

and it should work.  

A second problem is that I changed how I was doing the simplified collision meshes yesterday and it seems to have caused a problem.  The segwayrmp400 model drags the ground and doesn't move well even after the wheels are removed.  Using the segwayrmp400_physics model still works fine.  There shouldn't be a need for two anymore so I'll probably just remove segwayrmp400.blend and rename segwayrmp400_physics.  

Now that we have a working method of removing the wheels as children from builder I can probably simplify some of my code for building the vehicle in robot.py and segwayrmp400.py.  I'll try to do that this afternoon.

Thanks for your and Pierrick's help on this!

David
I've already fixed the issue by casting `fdata.nFileSizeHigh` to a `git_off_t`. This makes the warning disappear.

I'll send a PR later with an additional fix to the transport layer.
Нее... Я слишком ленив, чтобы ради циферки перекомпилировать ммапс экстрактор и потом (ещё хуже) переизвлекать их стопицот часов =)...
Просто ревертну этот коммит ;)
his logs statistic on only 1 (2,3...) much loaded maps. ergo - 571 much loaded on your server.
not behind `waffle.switch_is_active('soft_delete')`?
couldn't this check be outside the loop, for efficiency's sake?
It is: http://jquery-plugins.net/FeedEk/FeedEk.html
But I cannot find any attribution? Maybe just a link to the website?
I'll work on fixing that tomorrow, having a collector which do not collect anything is a non-sense.
Just a note, when you publish auth's and id's like that - on a public forum such as github - anyone can steal them. >:
Name is spelled "Daniel Holth"
And if you use a soft reference instead of a weak reference, my suggestion may have a 100% success rate: see the guarantees made at http://docs.oracle.com/javase/1.5.0/docs/api/java/lang/ref/SoftReference.html
Не уверен, но скорее всего, что при использовании & , команда перестанет выполнятся при закрытии терминала. Лучше заменить на >/dev/null
Nit: I would name this something like _perform_post_mindtouch_user, so that we don't get confused that this is a generic wrapper for requests.posts - this function is specifically for use by post_mindtouch_user and gets mocked for that. Probably not so bad, since it's in close proximity to the code using it
Don't bite! :)
There's never been a spec for it. I'll add one now.
учил джаву, поэтому так.
What's the behavior if TestingHttpServer is created with Guice and there are no filters bound? Maybe add a test?
Sorry I missed this, but thing.newwindow should be c.user.pref_newwindow.
Why drop the expose?
Nice scripts, btw.
One last follow up for tonight. Here's all it would take to derive a `JsonResponse` class from `Slim_Http_Response`:

    class JsonResponse extends Slim_Http_Response {
        public function write( $json ) {
            $this['Content-Type'] = 'application/json';
            parent::write(json_encode($json));
        }
    }

Used like this:

    $app->get('/foo.json', function () {
        return new JsonResponse(array(
            'name' => 'Danny'
        ));
    });
Oh, I'll check that.
My bad.  Good catch!  Thanks!
I'm now getting these images back into assets/img when doing "make". Should they be added to .gitignore?
Hi. If you have time, can you pls expect my fork of your library.
I have to change method 'objectToRpcXml' (or something like this) and  'xmlToObject' (or something like this) in wsdl.js to set it compatible with windows soap server.

Ouch, forgot, and change http.request response object.

Best regards.
@peterneubauer and @tonit are collaborators now and should be able to push
Is het nu met cbaf0a6314930ea907bc90501a5348c70eb42b9d verholpen?
If the last element is deleted, should the length of the array decrease? I mean, should you call data.pop_back() when index is the last element.
This fix is linked to [Issue #18](https://github.com/expertiza/expertiza/issues/18)
撤销 ability 的修改是因为用户没有登录的时候也给了 member 权限这样的 bug 么？
This is not resolving...
Done.

All with a few clicks of the mouse.  Gotta love github and tower (my new favourite git gui, even more than tig)!
The reason that I took this way was because of `VoldemortDatastoreProvider.getEntityMap` which is called by `org.hibernate.ogm.test.utils.getEntityMap`.  

With my modification for `EntityKey`, the constructor additionally requires `columnMap` to keep track of if the column name is different from the property name.`EntityKeyBuilder` builds entity keys and it's `fromPersisterId` method receives `OgmEntityPersister` as a parameter which holds the column names and entity name. These two things are necessary to instantiate `EntityKey` with the constructor. Since  `VoldemortDatastoreProvider.getEntityMap` instantiates `EntityKey` during the call, I needed to modify this way.
very nice add. Thx.
*self-facepalm*
use LOg.error(msg, throwable) no need for e.printStackTrace
Please add a copyright notice with your name and the year(s) you worked on your changes.
Literal, I'll fix it.
カンマを日本語の句点に置き換え、Sphinx がパースできるように「_」と「、」、「、」と「*」のあいだに半角スペースを入れていただけますか？
and then you could put it in a separate thread in order not to block the server :x
I think something is broken here.  You went from friendly, to robotic :sob:
SubEvents? What is that?
Why did you do this? The comments you added show exactly why this is a bad idea:
* the conf.d path might be wrong
* the notify is no longer a good one
* this file resource shoudl require the user, group and the directory

curious :)
That is right.I'm using Intellij for development I added the actionbar last
night but I think I did not update the github repo.I also intend to include
fragments soon.

On Sun, Jan 29, 2012 at 11:55 AM, Alex <
reply@reply.github.com
> wrote:

> Hi, great to see that you didn't just for the project, but are making
> actual improvements.
> Looks like you're making changes for support on Android 3+ devices? I
> don't have any of those to test yet, so my focus has mainly been on Android
> 2.
>
> If you come up with something interesting, send me a pull request and I'll
> include it into the main code base.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/achachiez/ReviveReader/commit/5bf3a4a377f7d1c2559a4f5be18a55d923a01f79#commitcomment-911829
>
I added "pr2_manipulation_drawer_executive.launch" because I wanted to be able to work on robot in parallel with Thomas. The original file pr2_manipulation.launch kills some nodes used by Thomas. Basically in "pr2_manipulation_drawer_executive.launch" file I just commented out the nodes that are already started by Thomas' "*.launch" file.
Of course, if I will launch some new nodes then I will do this from "pr2_manipulation.launch". The "pr2_manipulation_drawer_executive.launch" is just auxiliary used when Thomas is working on robot too.
Any chance you could re write this to use #Define TOKEN_ID # ??
Just FYI in case you didn't already know .. you may want to merge in the latest changes -- this change is already in master.
Yes it's a feature detection, or in this case bug detection. In IE and I think Safari 2, the loop never iterates and enumerables is set to true.
Did I really write that?
Mais en fait, j'me rend compte que je suis con.
Cet exemple est bidon car en pratique on étirera jamais autant une liaison "solide" (peu élastique) : Elle aura trop tendance à reprendre sa forme justement ! Le problème ce sont juste les conditions initiales ici... Crétin ! :p
Heres a link to the jsFiddle on the popcornjs account, feel free to update it anyway you see fit ( I PM'ed you the account details on IRC )
http://jsfiddle.net/popcornjs/J3T45/
And people in Hell want ice-water ;)
Overall your CSS looks really nice :)
Shouldn't this line be removed since we're already doing the logging on line 47-50 in FaultManager.cs?
World state condition has non existing world state in value1 (62574), skipped
2012-02-12 01:46:41 CreatureEventAI: Creature 33224 using event 3322453: param2 (Condition: 11) are not valid.
2012-02-12 01:46:41 World state condition has non existing world state in value1 (62574), skipped
2012-02-12 01:46:41 CreatureEventAI: Creature 33211 using event 3321151: param2 (Condition: 11) are not valid.
Oh sorry ...
Best fix 2012!
4559 is the right entry for arestore check
group->RemoveMember(guid) is already being called before calling this function in Battleground.cpp ... this should fix it: http://paste2.org/p/1880201
Will do - thanks for the advice.

On Wed, Jan 11, 2012 at 5:15 AM, Peter Ledbrook <
reply@reply.github.com
> wrote:

> Thanks for starting this. One thing I would suggest: use the new approach
> to naming the files as described in the [Grails 2 user guide] (
> http://grails.org/doc/latest/guide/conf.html#docengine). It shouldn't
> require too much work because you can use the [migrate-docs] (
> http://grails.org/doc/latest/ref/Command%20Line/migrate-docs.html)
> command that comes with Grails 2.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/gavinhogan/grails-shiro/commit/c11e5ab8523122d01e94a9523c36b66fdfa20cb7#commitcomment-856496
>
No, I sort of gave up dealing with Opscode on a couple other issues of where it felt like they didn't care about issues like RackConnect, so I came up with my own solution basically without modifying their code.  Basically I created a Cloud Servers image I have pre-loaded with some keys and with a script to download the RackConnect convert script.  Whenever I spin up a new cloud server it auto runs the script.  The above code is great, but the problem is on bootstrap it would never work anyway, because the servers won't even be able to get outbound access out of RackConnect for minimum 10-15 minutes due to Rackspace's ACL generator only fire's that often, not realtime.  So I wait until I get the ACL notification, then I fire off the Chef job and direct it to the manual IP in a Knife bootstrap command.
Incorrect usage of static functions.
Gracias que no estaba en casa :S
Should be "don't delete waypoint", not cache.
Yes, I execute `coffee -o lib/ src/` (it needs installed coffeescript, `npm install -g coffee-script`).
Ok; will convert it into an error.
attribute: isRotating
getter: isBlobbyRotating()
setter: not used (=> YAGNI = you ain't gonna need it).
Git TacticsAlpha.lnk is a shortcut I placed locally to launch Git GUI in the TacticsAlpha repository.  I did not want it committed because I imagine there might be some differences between setups across workstations.
Love how you use anchors like this, so the links in the sidebar work!
I dont understand what You wanted to say ;p
willst du versuchen, dass sich ein extra feld öffnet, wenn angekreuzt ist, dass man einen tweet direkt mit einfügen möchte??? 
wäre auf jedenfall cool!!! aber javascript ist echt n bsischen nervig oder? xD
Модели должны называться в единственном числе - ItemsOrder что ли ) у тебя из-за того что здесь множественное - не работает ассоциация item has_many :orders, :trough => :otems_orders, и собственно все has_many :items_orders тоже
Should be --branch, or did I miss something ?
The other tests follow a naming convention of <action>Should<expectation>.
I tried one more time from scratch and just `fpm-cook` was sufficient this time. Maybe I did not cd into correct directory or something. `fpm-cook -D` started the build, too ;)
Will do performance tests with this and previous commit.
Why not fix device_set_property() to support floats instead?
`readThreadVisib()` - это функция для чтения скрытых тредов, которую Я САМ добавил, её объявление можно увидеть ниже (596 строка). Сделано это для того, чтобы немного упростить функцию doScript, учитывая, что `readPostsVisib` вызывается только один раз оттуда.
As I understand Derby doesn't support locks on FOR UPDATE...
Hi,

I know about that. The notes with same title are removed because of removeNote method(and how this method works, see in DBHelper class).
I'm going to change this, but now I'm too busy to do it.

Stay tuned :)
I sent you a message on gamedev, but you may not have gotten it. Anyway, you might want to change your git global settings=>line endings to "check out as-is, commit as-is".
But if it's already aligned - it should be aligned all the way, IMO. Either no alignment or complete alignment works, not both ;-)
Let's call it a nice to have. I think this is really a configuration item, and I prefer to keep configuration externalizable. Precedence would be that the annotation parameter contains the default, and configuration can always override it. Here's my use case: I want to return a 404 in production, but want to redirect in development. 

Sent from my iPhone

On Jan 31, 2012, at 6:10 PM, Michael Minella<reply@reply.github.com> wrote:

> Exposing the configuration at this level makes sense but then you get into the issue of precedence.  Do you assume that the annotation overrides the config (since it would be more finely grained)?  Same thing goes for the annotation in general.  I have it currently coded to take the more finely grained levels first (closure, method the class) so you can override the higher levels at a lower level if needed.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mminella/grails-feature-toggle/commit/4640130b3bc7fdbac9018746c55cb2596b498ad2#commitcomment-920573
After much testing I found that the creatures dont load with the creature tables, and the client crashes with the creature_model_info table in. Possibly the packets dont send the creature info properly anymore. What opcode would that be?
self._gui_message_(url,conv) and this
Yea, you are probably right, but that NOTICE was from the other libaudio-files.
The libaudio-files I took the sources from did not include a NOTICE. So I don't know what kind of licences they are.
The GUI has proved itself faster here.
You didn't change the actual branding files though?
https://github.com/nightingale-media-player/nightingale-hacking/blob/sb-trunk-oldxul/locales/branding/brand.dtd
Also brand.properties
I guess it should be listed as `@static` like Y.IO.
> From what I can gather, it sounds like fixtures are for putting the DB in a known state before executing tests. Is that correct?

Yes, after the test DB is installed a snapshot of each table is created in the fixtures direcotry.

> How will these fixtures be maintained?

If you mean from the developer side, there is no need to mantain them, since there are generated one time only (after running the test_install.php script). If you mean from the elgg core development point of view, there are at least two choices:
- Having a build script for packaging the release that runs test_install.php.
- Incorporate the test install as part of the elgg installer.

> Why are they all separate files instead of a single DB dump (the Elgg DB has highly interdependent tables).

The test framework records which tables are changed on each test case run, so only the tables that were modified are reloaded. If there was only one big dump the entire DB should be reloaded before each test. See also the TO DO list, where are some ideas to speed this up a little more (like using MEMORY tables)
Would it make sense to inject the instance through Spring?
It shouldn't (and it doesn't on my machine). If the assumeTrue fails it skips the test. I copied this code from the MemCacheTicketRegistryTests.java which does the same check to see if Memcache is listening in the setUp().
Null's can't be stored in a Memcached server because everything stored there must be serializable. The nullPlaceHolder gets around that restriction and is the reason for the replaceIfNull, checkIfNull functions.
Evernote was unable to submit your note for the following reason:
The storage limit for this account has been reached.

Original message information:
From: =?UTF-8?B?VmlrdG9yIEtsYW5nICjiiJop?= <notifications@github.com>
Delivered to: thanhbv.fa84f@m.evernote.com
All recipients: akka/akka <akka@noreply.github.com>; giabao <giabao@sandinh.net>; 
Subject: Re: [akka] Minor cleanups in new remoting (#981)
Sent: Fri Dec 21 08:40:56 PST 2012

To prevent excessive emails, you may not receive another error reply
for the next 360 minutes.
use apply of the case class Timeout instead of new

    val timeout = Timeout(
Optimize the case where bytes.isEmpty?
Isn't this extremely unsafe?
Simpler alternatives:

Just grep for "1.6." in the first line:
```bash
java -version 2>&1 | head -1 | grep -qF "1.6." || fail "Java version is not 1.6"
````

Split using cut:
```bash
java -version 2>&1 | head -1 | cut -d ' ' -f3 | cut -d '.' -f2
```
it's not related to refresh, right?  Can we take it out?  I can pull the
refresh stuff now, but not this.


On Thu, Nov 15, 2012 at 12:32 PM, Gerry Gleason <notifications@github.com>wrote:

> In lib/wagn/model/permissions.rb:
>
> > @@ -78,8 +77,8 @@ def permission_rule_card(operation)
> >      rcard = begin
> >        Session.as_bot do
> >          #warn (Rails.logger.debug "in permission_rule_card #{opcard&&opcard.name} #{operation}")
> > -        if opcard.content == '_left' && self.junction?
> > -          lcard = loaded_trunk || Card.fetch_or_new(cardname.trunk_name, :skip_virtual=>true, :skip_modules=>true)
> > +        if opcard.content == '_left' && self.junction? && (cardname.trunk_name.key != read_attribute(:key))
> > +          lcard = loaded_trunk || Card.fetch_or_new(trunk_id||cardname.trunk, :skip_virtual=>true, :skip_modules=>true)
>
> The refactoring is just as reliable and much more complete and correct. I
> think you need to take a closer look at why all this helps and how it
> connects to the id based caching and tag/trunk/right/left* stuff.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ethn/wagn/pull/17/files#r2143266>.
>
>



-- 
Ethan McCutchen
One of the Wagneers, Wagn.org

Wagn. How pioneers roll.

s: ethan.mccutchen
t: @intogreater
s/define/defines/
Seems that you are missing an xref after "(as defined in"
Hmm, there is no `HTTP_REFERER ` in toolkit.request.environ. See a13f1023d2c63a74f7b406d77ce.
What is the reason for removing the @? I believe this is needed to prevent the script from halting it the source_package does not exist.
Maybe worth it to mention what is the default priority level, so that people know whether by passing, say priority(3), am I increasing or decreasing priority of my extension with respect to extensions with default priority.
SAFE_RELEASE(wmProfile); needed here, no?
This is a little clearer, but it's unrelated to the purpose of the pull request.
Vou alterar pra "no próximo post da série"
@schmittjoh: I don't know if that's true--it certainly works fine in PHP 5.3.

However, I did remove the abstract method since the interface covers it. Thx all.
There is no `isColumnFullyVisible` (yet). We have to add it
why is it good for `ace.edit` to silently fail here?
shouldn't this be `throw "ace.edit can't find div #" + _id;` instead?
You forgot to take the $this->sessionKey into account.
I'm okay with this being public, though I agree it would be better if it were protected and you used a friend class. All that said I'm worried that there exist two seemingly identical functions, setParent and doSetParent. I recognize that setParent also calls itLives(), but if you have to create a new function to get around in order to only set the parent variable, maybe itLives() shouldn't be in setParent(). Any way we can clean this up?
@axelarge Twig is not a component of symfony :) It is a standalone library used by Symfony
I don't really understand what this line is for.
You should remove this.
If git is not installed, things will be printed to `stderr` hence ignored in this case, because the commit and branch information is written to `stdout`. So should I check for the existence of `git` from `print_qiime_config.py` and just exclude this data? Making the output look like this:

    Dependency versions
    ===================
                         PyCogent version:  1.5.3
                            NumPy version:  1.6.1
                       matplotlib version:  1.1.0
                      biom-format version:  1.0.0
                     QIIME script version:  1.5.0
            PyNAST version (if installed):  1.2-dev
    RDP Classifier version (if installed):  rdp_classifier-2.2.jar
              Java version (if installed):  1.6.0_37

I have found the library information useful for diagnosing issues. I have seen cases of people in the forum, where they successfully install the new scripts in their environment but fail at correctly install the new libraries.

I'm using (I think) `qiime.util.qiime_system_call`, if anything can you point at the specific case where I'm not using it?
Thanks, will do.

Any reason we are not using https://github.com/medialize/URI.js instead of webr3/URI ?
It has better parsing support.
The log levels are a hierarchy so logger.info? should be enough.
You can use `merge!`
I'm not sure using `inspect` is good:

```
 1) Failure:
test_render_raw_template_with_handlers(CachedViewRenderTest) [/Users/steve/src/rails/actionpack/test/template/render_test.rb:83]:
--- expected
+++ actual
@@ -1,2 +1,2 @@
-"<%= hello_world %>
-"
+"\"<%= hello_world %>\
+\""


  2) Failure:
test_render_raw_template_with_handlers(LazyViewRenderTest) [/Users/steve/src/rails/actionpack/test/template/render_test.rb:83]:
--- expected
+++ actual
@@ -1,2 +1,2 @@
-"<%= hello_world %>
-"
+"\"<%= hello_world %>\
+\""
```

Or is that what we'd want?
I think that this method should have a `:nodoc:` mark.

```ruby
# Clean the errors if instance is duped
def initialize_dup(other) # :nodoc:
```
Me too. This is a troubleshoot comment, but the normal flow is to use `gem install pg`.

I'm not sude if we should add troubleshoot comments in the generated files.
Consider a file like this:

    object Foo {
      private val x = 5
      final val y = 10

      def f1 = () => x
      def f2 = () => y
    }

f1 returns a closure which returns the value of x.  f2 returns a closure which literally returns "10".  If you recompile this with different values, the closure which returns 10 will still return 10.  The one which accesses x will return the new value of x.
# The spanning tree has at most N + 1 edges.
I'll take a look.  The comment "Support is experimental and new in this release, there may be bugs!" in the documentation (http://docs.cython.org/src/userguide/memoryviews.html) is worrisome, but the same comment appears in the section on fused types, so I guess I shouldn't let that stop me.
Why is this necessary? Shouldn't the other models set this themselves? We have no guarantee that the models will be using email, password or password_confirmation.
I do not see why this is necessary as it is already caught by the else branch.
Maybe I misunderstood the question but the `__call__` method is necessary for the creation of "applied function expressions".

```
f = implemented_function(blah blah)
f is an instance of Function
f(x) uses the f.__call__ method to create the expression f(x)
f(x).evalf() uses the f._imp_ method
```
Thanks, I will redo this.
should you maybe be using the approach of `_aresame` from expr.py? 2.0 == 2 but they are not exactly the same.
I still disagree about the subclassing. Containing an instance of something is not the same as being a subclass.
don't forget that you can also call simplify as a method and thus not need to import here:

```
foo.simplify()
```

I don't care either way.
True. I was having the wrong impression that I need to use two new lines... I'll take care of these.
I think so. Commits dealing with equals, too.
Please avoid implicit relative imports.
You should also use ``and`` instead of ``&``. If you do something like ``ask(Q.real(x**y), Q.imaginary(x))``, this will raise an error.
This is an internal function.  There's no need to deprecate it.  Just remove it.
Do not instantiate; the constructor does not check that your data is well defined. Use
the ``homomorphism`` function instead:
We're respecting BC (nobody needs accidental encoding and email headers change after updates), so this is false by default.
compare should return the order of the objects provided, but it seems like instead we're using this for equality. From the docs:

Compares its two arguments for order. Returns a negative integer, zero, or a positive integer as the first argument is less than, equal to, or greater than the second.

perhaps you meant to override "equals" ?

Please mark overridden methods as @Override
our namiming for events has typically been [EVENTNAME]_EVENT.

for example:

var FLOORED_EVENT = 'floored',
    SELECTED_EVENT = 'selected';
Nit: Prefer newline at EOF.
if (grunt.option('no-color')) { msg = colors.stripColors(msg); }
nope, doesn't seem to display at all; would just get a white checkmark in that case.
references to 1.9 should be references to 2.0, but even those may not be neccessary. 

use write concern language.
I think I understand, I will change this.
OK, it's fine with me either way, do you want to change anything?
Yes, it's already fixed in #928, I thought I had removed this comment. I guess not.
$this refers to template renderer instance, not template. 

I can add phpbb_root_path to renderer and then change it to $this->phpbb_root_path. Would it be a better solution than changing template's phpbb_root_path to public?
The New Line is okay, but the spaces/tabs in that line should be removed.
Whitespaces in a new line do never have a reason to exist
The method doesn't actually return anything in that case. I'm not sure what the accepted way is to handle this, but later on in the code there is an explicit `return null`. Should at least be uniform.
Nevermind. https://github.com/phpbb/phpbb3/pull/1105/files#L0R4963
never knew about that. awesome!
does update go global here?
One possible option would be
```ruby
def self.inherited base
  HTTPI::Auth.register base, base.name.underscore
end
```
but I would say explicit wins over implicit in this case. I want to be able to define the symbol that refers to the adapter.
There is an OpenGLWindow::destroyWindow() method but it will be overloaded with if () if I am about to use it. imho goto's and labels are even better :)
Please remove the `#--` and `#++` boundaries because it forces to ignore the comment. Also, please add the class header.

```
# Parser for the whois.registry.za.net server.
#
# @note This parser is just a stub and provides only a few basic methods
#   to check for domain availability and get domain status.
#   Please consider to contribute implementing missing methods.
# 
# @see Whois::Record::Parser::Example
#   The Example parser for the list of all available methods.
#
```
address hari hatman oon bala moteghayer kon
ehtemale avaz shodanesh ziyade
Can we somehow only include this at login?
I think GUI Mode should not be presented to user.

From my understanding, when the device is in this mode, it sends notifications about changed parameters without explicit request (for example, when user turns the knobs). Hence presenting this in GUI just confuses the users.
I think he meant "session cookie" which should be "CSRF cookie"
you should use ``assertGreaterThan`` so that the message is better in case of failure (currently, it would say ``Failed asserting false is true``)
Thanks for spotting the typo
It is just an example extracted from an old demo. Do you have any concern about it?
Er, why? Both are grammatically correct and read fine to me.
Why was the comma removed? Does the trailing comma not work anymore?
Ugh. Github's diff doesn't show enough lines to give context to the change. Sorry for the noise.
Err, I'm reading this wrong. nvm.
So would it be OK, to replace

    default_version = '@MADLIB_VERSION_MAJOR@.@MADLIB_VERSION_MINOR@.@MADLIB_VERSION_PATCH@'

by

    default_version = '@MADLIB_VERSION_STRING@'

in that file?
Fixed in a fixup! commit. Thanks.
You mean a set is faster to test with than a list? Then makes sense to change it back. I just thought it was a few lines longer than it needed to be and the test for `exclude` was done twice.
An other option that comes to mind with respect to playlist names would be an additional metadata file mapping URIs to names. Not sure if this is needed or a good idea, but probably worth at least considering.
sorry, I guess a stacktrace might be useful: 

TypeError: Cannot set property 'here' of undefined
    at Object.accessImpl (/home/cloud4all/gpii/node_modules/universal/node_modules/infusion/src/webapp/framework/core/js/Fluid.js:613:41)
    at Object.accessWithStrategy (/home/cloud4all/gpii/node_modules/universal/node_modules/infusion/src/webapp/framework/core/js/DataBinding.js:180:32)
    at Object.setWithStrategy (/home/cloud4all/gpii/node_modules/universal/node_modules/infusion/src/webapp/framework/core/js/DataBinding.js:158:21)
    at Object.set (/home/cloud4all/gpii/node_modules/universal/node_modules/infusion/src/webapp/framework/core/js/Fluid.js:664:25)
    at /home/cloud4all/gpii/node_modules/universal/gpii/node_modules/settingsHandlers/src/js/XMLSettingsHandler.js:212:19
    at Object.each (/home/cloud4all/gpii/node_modules/universal/node_modules/infusion/src/webapp/framework/core/js/Fluid.js:319:17)
    at Object.applySettings (/home/cloud4all/gpii/node_modules/universal/gpii/node_modules/settingsHandlers/src/js/XMLSettingsHandler.js:209:15)
null do not make sense. If you add null to the Listeners it user's fault.

The CLQ does not use any lock.
I do not want to have any lock in any operation. No reasons for that.
Again, yes the COWAL uses locks on write, but we do not need that, no locks.
Can we still use

```@IncludeCategory(SlowTests.class)```

I actually forget whether Java will auto-create the 1-element array in this case.  If it does, then can you add a test of that behavior?

If that doesn't work, the current change will break all existing category suites, so we'll need to introduce a new syntax.  ```@IncludeCategories``` seems an obvious choice.
explicit version numbers stop things from breaking on a deploy that worked locally!
If this bug is only in one template, do we need to do this for all AssetLists everywhere?
I think isNaN check here is ok to keep even though we don't need it now. But later on when we support keywords for color names, then this will be needed.
Should use hostEditor.document.getLine() instead
1. Renamed 
2. Removed
3. But I'm not getting a menu item in this function, I'm getting as you point out the menuItem.id. Am I missing what you want me to do?
Thanks for catching this untranslated string.
Remove (no more please.)
We still need this check here as we want to make sure that a successful test  leaves not temporary files behind.
definitely break...i want to make sure no one added any get params in the wrong order...
kinda odd
Not that I know of. When I set the indentation style of macros and functions that I define, I'm just copying the indentation style of existing functions and macros. I don't really know how to write a custom indentation style.
Hi Ray,

Before I begin: I ran your code through an automatic formatter in order to spruce it up a bit. Having nicely formatted code can really make a difference in your ability to understand and figure out what is going on, especially in a verbose language like Java. While it can be tempting during the various phases of debugging to shift things around and take advantage of the fact that Java doesn't care about whitespace, in the long run you will be better off keeping your code nice and clean. If you are editing with Eclipse I think there's even a "reformatter" function that will do this for you, although it's much better to make good habits for yourself by doing this manually.

What you are looking at here is a "diff", or the differences between two files. In this case, it's the difference between two versions of your YMCA file: the one you sent me vs. the formatted version. Sections in red indicate areas that have been deleted and sections in green indicate areas that have been added. Don't worry - no code has been added or removed from this file! The only thing that has changed is how the code is formatted.

I've left my comments in sections like these down in the green section of the code. 

If you're interested and want to find out more about what's going on here: this website is devoted to sharing code with other people and revolves around the source control program called Git. Git lets you track changes to your programs as you edit them and can let you manage multiple versions of your code simultaneously. Git is heavily used in the software industry and is a tool whose advantages will greatly outweigh its cons, despite its somewhat steep learning curve. To learn more: http://learn.github.com/p/intro.html
Good catch here, this was definitely an oversight to have this initially in the `while` loop.
Just realized that I need to remove presence from the validation array. Shouldn't affect it negatively. Would just cause it to validate twice. I'll fix that with the next batch of updates.
``` ruby
[
  :ReferenceError,
  :EtcLulzError
].each do |err|
  assert Rugged.const_defined? err
end
```
could be shortened to `return !!callback.call(el, idx, el)``
prevent bubbling?
Here's the questionable meat.
Why not declare this first?
I think a more appropriate message might be along the lines of ```Would you like all track events to be removed with the media change as well?```

That might not be that great but just something to distinguish this from our normal trackevent deletion dialog.
Because we need to cover this for both adding new plugins and moving around the containers for dragging/resizing positioning.
or probably better at the end of this function for any marked stream
Can you remove the commented lines, since they are not needed anyway
path to `proj4js.js` should be `./externs/proj4js.js`
1. 这句可能没用
2. chomp: vi. 切齿，格格地咬牙，咬响牙齿
potential null dereference
`exports.freeze = require('./freeze');`
div.content_wrap > section
Is this 'p' line a remnant from testing?
Right now, it's false because no integration code is in master yet, but will be turned to true as soon as it makes sense. the intall_foreman is effective only if use_foreman us enabled
missing rules for sync_ldap_roles
This will be faster:
if type(rec)==type({}) and key in rec:
Normally properties which are funcs end with 'Callback' or a similar suffix.
The basho_expect historian says: spamminess is good for post-mortem debugging.

The b_e methods `step` and `prereq` exist both to build in documentation and to insert that documentation into each run's output.  More post-mortem debugging happiness.
links would be good here

"... [[Bitcask]] (the default), [[LevelDB]], and the [[Multi]] backend."
not sure calling our printf infrastructure from a signal handler is wise
I changed this, because you CAN add information / change it, but you do not need to (not mandantory)!
The end-brace releasing the main/mempool lock scopes is removed so that it remains constant until ConnectBlock finishes checking it.
The code in between could be indented to clarify, but the diff is easier to read this way.
Nit picking is what separates a clean and easy-to-use framework from others, so I agree.
This would be same as `if (empty($this->uses)) {`
I am planning to use this resolution on client code side.
Explain more explicitly why this one day must be added here
Ah yeah I forgot to add in the comments, this was the test I mentioned I was having trouble with.
I think the issue is something to do with stubs but the results were never quite what I expected.
use `tests.utils.APITestCase` and you get `self.make_api_result` for free. You also get `self.api` as an automatic API object.
Yup, that needs to be improved

On 9 déc. 2012, at 10:38, Stephane Landelle <notifications@github.com> wrote:

> In gatling-core/src/main/scala/com/excilys/ebi/gatling/core/session/Expression.scala:
> 
> > @@ -72,6 +71,8 @@ object ELParser extends Logging {
> >  				_.group(1) match {
> >  					case elSeqElementPattern(key, occurrence) => SeqElementPart(key, occurrence)
> >  					case elSeqSizePattern(key) => SeqSizePart(key)
> > +					case key if key contains "${" => throw ELNestedAttributeDefinition(string)
> I disagree. The whole point of Validation is to sport relying on Exceptions.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Does debian build machines have access to network during build/test time ?
careful..  this should be guarded to only invoke on frontend nodes
We don't really support top level
      overrides, if instead isLive was part of the live plugin then you
      could update that property and it would read it directly. 
      On 12/04/2012 06:48 AM, Ran Yefet wrote:
    
      In modules/KalturaSupport/resources/mw.KDPMapping.js:
      > @@ -291,10 +291,6 @@
>  						case 'isLive':
>  							return embedPlayer.isLive();
>  						break;							
> -
> -						case 'isOffline':

      @mdale Is there a way to extend
        evaluate api on the plugin code instead in KDPMapping? ( like
        what we're doing in setKDPAttribute )
      
        —
        Reply to this email directly or view
          it on GitHub.
I think it would be better to move these into thumbnail.php or KalturaEntryResult.php
Leave this in the `git_clone` signature. Feels very unnatural here. It's an output parameter.
Maybe test the form type before including those fields. That way, it will be usable with `NewsletterForm` and `NewsletterCountryForm`.
Here; https://github.com/mozilla/kitsune/blob/master/apps/search/es_utils.py#L26

So if you just set the prefix in the build script, we should be good?
Our import grouping convention is to have a space after this import and another after the one below.
rats.  that was just a bug / oversight.
How come you added those lines back? I removed them when I added the ``server_status`` service, and removed those keys from the PHP configuration as well so this should definitely not work with an up-to-date config. The data you need is returned by the status service, cf. https://github.com/mozilla/socorro/blob/master/socorro/external/postgresql/server_status.py#L82
why removing ``Bundle`` ? this is not consistent with the way the ``@`` notation is done elsewhere
the order of params in the phpdoc does not match the signature
This comment is already rewritten. 

I had something like you mentioned before but it was getting too long and didn't fit on a single line anymore. Thus, I splitted it, and I think what I've chosen now is fine as well.
the interface is not an implementation
Should this be True, since it's used as the username?
As per discussion with Ryan even though this does create some inconsistency with the rest of the app for now we'll stick to the alert in this case since most of the other long error messages will be changed over to that style.
Can the `locals: {}` outer hash be dropped?
Shouldn't this just inherit from the `.text-counter` and have a sub definition with the orange color?
也不是偏激啦，版权问题是应该考虑的，所以我在写第一章用户说明文档时就写上了版权所有。我只是想着大部分用户应该是在win下操作，而且这个问题确实如果有版权问题也应该是ctex的事情，和咱们没关。
另外，关于linux下的相关操作使用等，用户说明手册里估计得由你来操刀了，用户手册里的那些章节，你觉得有哪些不合适的地方就直接改就行。
Again, I/O => try/catch + failure
Indeed it is. It was hacked together to give a way to trigger the download and check it had worked. It could easily be focussed more tightly on just downloading - e.g. by using `feature.geometries()`.
I think you're missing an "are".
This exit code should never be emitted via this class. It should be used as a result of a management operation that specifically tells the OperationStepHandler that the user wants a process restart. See HostShutdownHandler and ServerShutdownHandler.