labofsahil/github-event-dataset-2012 huggingface dataset:
updated with info about may event

* Add thank you and #3 summary to front page
* Added april speakers to videos tab
* Changed 'tendin for next event
Print abinary values without delimiters, unless requested by caller.

This commit fixes Bug #191, submitted this morning.
Fix for strange behavior I was experiencing.

So in regards to https://github.com/limewheel/ls-module-sitemap/issues/2 I had done everything I could possibly think of. But then I thought on a lark I would remove the .xml (it wasn't doing what it was meant to anyway) so I did.

Sure enough that fixed all my problems, at that moment the php file changes I had made were working like they should. Also all the pages I had set in the database with sitemap_visible were gone. 

I am not 100% sure why or how yet, but I know what it was, or at least think I do.
Bug in BoundedInputStream

Hi Michel,

I've just fixed a pretty major bug in the BoundedInputStream, I was incrementing the count of bytes read by the requested bytes to read, rather than the actual bytes read, so if less bytes were read than requested, LittleEndianInputStream would end up throwing EOFExceptions.

There are a few people that have been impacted by this issue in Mongo Jackson Mapper, are you able to cut a release soon?

Cheers,

James
Adds method for querying items used in a given exhibit.

Hooks into the item_browse_sql to return items that are present in a specific exhibit. Can either add a 'exhibit' param to the query string, or add a 'exhibit' param to any helper functions that use the ItemTable, like get_items.

Refs #6. Does not add separate helpers for retrieving items, since you could use the `get_items()` helper.
RPM support

adds rpm building support for rhel5, rhel6, and fedora 16
Pomodoro Kata: Jorge Trujillo y Marco Via
Fix crespo4g/epicmtd coexistence.
Allow multiple exitcodes for initial puppet agent run

Prior to this (a change I made to not effectively bounce
the agent service) only accepted an exit code of 0.

This worked for my testing because of conditions that caused
the agent to check in once before the test was actually ran.
The issue is when (and this is the typical case in CI) the test
is the first agent run to check in there will usually be
successful changes applied and puppet will exit with 2.

This adds 2 as an acceptable exit code.

Signed-off-by: Justin Stoller <justin@puppetlabs.com>
Feature/updates

- Added Vendor/Zf2 back as submodule
- Updated SocialAuth to latest version
Cordova 1.6

I have added a branch to track changes required from Cordova-1.6.0. Would you mind adding it to your project?
Update AndroidUSBSerialMonitorLite/src/jp/ksksue/app/terminal/AndroidUSB...

...SerialMonitorLite.java

Dear Ksksue-san
Added closeUsbSerial() function to close the usb serial device when ever user wants. Please review and give your feedback for the same.

Arigato
Arun
Output the name of the class explicitly in SpecReporter 

This solves a problem when running through a DRb server such as spork of
outputing `#<DRb::DRbUnknown:0x007ff87e126708>` instead of the intended class name.
Client-side LESS parsing for rapid development
Tweaking the sample buttons

This is my first swing at changing the sample buttons on the landing page. 

![Screen shot 2012-12-20 at 11 07 40 PM](https://f.cloud.github.com/assets/40662/26292/582fdad6-4b24-11e2-85ec-bb18ca38e3f4.png)
Mobile: No more text cut off in header tags, ul li for listview

Tested already. please just pull.
LPS-24833
Fix build break around dynamic links - pulls in some of rob's import changes and Andi's nagios changes. [1/6]



 crowbar_framework/app/models/ganglia_service.rb |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)
Odm proxy

Added ODM Proxy to `DoctrineProxyHandler` custom handler condition.  
Also terminated long line of code according to [standard](http://framework.zend.com/manual/en/coding-standard.coding-style.html#coding-standard.coding-style.control-statements.if-else-elseif).
This change was the same as the previous change except now the temperatu...

...re function is different since we implemented a new temperature sensor.
pull from chrome git
Some small fixes

Hi Guillaume, I forked your project to let you access my mods. I won't spam you with pull requests except if that what you want. Just wanted to inform you.

EDIT: noticed that I wrote in french, rewrote that in english. It was late, and I confusingly I thought that I was writing an PM.
[PageController] Enabled easy overriding of seo pages

This PR refactors the SEO setup to a separate method to ease overriding seo customization. Also added a `beforeRender` hook for custom page controllers.
Contrib 3367 display group meetings 19 stable
Fixes to pull

Salut,

J'ai récupéré ta présentation pour prêcher la bonne parole dans mon entreprise.
J'en ai profité pour faire deux petites corrections.
Pull Req: Remove BBX-Emulator and exclude node_modules in maven artifact

Removes blackberry-webworks/BBX-Emulator references

Updates Framework submodule to include the changes seen in blackberry-webworks/BB10-WebWorks-Framework#11
Modif (Maison-Etage)

Modification Maison-Etage (Ajouter et lister)
修改了模板文件

修改了模板文件，加入了markdown支持，更改了中文下自动生成 索引的bug
More stream-lined approach to wrap extension points

Jenkins defines two kinds of extension points. One is for thing like Builder and Publisher, where we get a lot of instances created and retained in a single JVM. This is `Descriptor` and `Describable` and we've already got a proven mechanism to define more wrappers.

The other kind, which is actually more fundamental, is a mechanism for Jenkins to discover singleton instances that are the extension points, such as `EnvironmentContributor`, `RunListener`, and `PageDecorator` (`Descriptor` is actually just one of these, and `Describable` is just a further layer above.)

This pull request adds a mechanism for us to rapidly define such singleton extension points.

The main change is the `ExtensionPoint` module, which is a mix-in that detects subtype implementations, and it'll invoke a callback specified via the `when_implemented` class method. I've added a couple of actual wrappers based on this mechanism (and used them successfuly in the RVM plugin.)

One thing that I couldn't quite figure out was how to modify `Jenkins::Model::Describable` to use `ExtensionPoint` to avoid dupliation of the subtype detection logic. I'll think more about it later.
AVL-Formeln

Die fehlen, sind aber wichtig da die relativ willkürlich sind.
Muss also vermutlich korrigiert werden, da ich noch kein Code für den
AVL-Baum vorhanden ist. Hinweis im Dokument ist rot.

PS: Zeichenkodierung ist mal wieder fehlerhaft, werde das noch
anschauen und auch bald committen.
Started v1.0.8a2 after the first round of tests

[x] help for welcome and duration added
[x] fixed format on delete events (calendar integration)
Don't clone method parameters

This PR fixes #47
Upgrade to twitter bootstrap 2.0
.DS_Store is absent from .gitignore

You committed your .DS_Store file to your repo. I'm sure you probably know, but maybe just forgot.
Update admin/ru-RU.com_hwdmediashare.ini

Hello!

I was glad to help translate the file into the Russian language. 

I found that the lines 60-62 are repeated lines 53-55:
COM_HWDMEDIASHARE_N_ITEMS_CHECKED_IN_0="No items successfully checked in"
COM_HWDMEDIASHARE_N_ITEMS_CHECKED_IN_1="%d item successfully checked in"
COM_HWDMEDIASHARE_N_ITEMS_CHECKED_IN_MORE="%d items successfully checked in"

So I left them without translation. Maybe they need to be removed from the file?

I'm not sure about translation of line 144 (COM_HWDMS_COUNT="Count"), may need to correct the translation from the context.
Two simple questions

Added two simple questions: "Name three different falsy values in JavaScript" and "Name at least 8 elements which were introduced in HTML5".
Names

Fixed names of Gym Leaders (Yakon to Yacon, etc.), Elite Four (Caitlin to Cattleya and Gima to Giima), and Bel.
One of the fields required by ONIX is that physical formats (hardcovers, softcovers, etc) need to also include the country where they are manufactured.   Ie, where is something printed.  It's not necessarily the same as the country that a Publisher may be in, for instance, and they may differ across different formats (and also across different sales regions if a book is translated, but that is probably beyond the scope of the first pass at this).  Is that what you meant?
Yeup. Added it. Thanks for the heads up.
I'm contemplating turning these all into a single attribute w/ object properties called something like `display`, I like sub-attributes for these very simple config values. Would also clean up the attribute space some at the cost of slightly uglifying configuration.

```javascript
new Y.Paginator({
    "show.first" : false,
    "show.last" : false
})
```
is a bit uglier than

```javascript
new Y.Paginator({
    first : false,
    last : false
})
```
I don't like that every object in the simulation actually stores its own table.  I would prefer that the actual storage of the table objects lies in something more akin to the book-keeper.  Specifying the table structure is okay to do inside message class, but the actual table storage for Message, Model, etc. classes, I would recommend doing it external to each of those classes.  In fact, I would consider (note consider does not mean recommend) moving the core data table definitions off-board to something like the bookkeeper just to keep the meat Message, Model, etc. class code from becoming obscured by all the table creation code.
兽，FetchBooleanCallback.java 和 QueryBooleanCallback.java 正好搞反了吧
En fait, dans ce cas, on n'utilise pas d'agent mail, mais seulement la boîte, en direct. Je merge.
I guess the main issue is that I think that state.highstate should, by default, display everything that is going on when called locally. I wanted salt-call to display in a fair amount of detail the system behavior.
Note that this also removes the prefix from the STDOUT path. I'll fix it.
This is great! The one issue I have with this is we're trying to stay away from raw and html_safe. We want the users to decide if it's html_safe or not.
Sure this is a static method?
You should use `$session->hasFlashes()` here.
did 26 just work better than the original 24, or was there some other reason for the change, out of curiosity?
:clap: wow, scoped to nowhere. Glad you caught it!
Any way I could convince you to do a pull request with this commit?
conviced. lets leave this in.

i don't understand point 3 though. its not a system wide config, its the set of default values that will always be present.
yes, you may change the values in your own config but you cannot change that a specific key will be defined in the
runtime-config.

4:  erm ? will check how config.get handles undefined keys..
No, I mean, let's assume we have three elements.

     [(1,"a"),(2,"b"),(2,"c")]

Now, merge sees that k1/=k2, and goes into the else of that if. Then, it returns (1,"a"):(2,"b"):merge [(2,"c")].

No?

(I introduced this bug, but while we're at it, we should completely fix the merge function.)
@adamv This changed the return value of Pathname#install and broke some assumptions made in test_bucket.rb:test_pathname_plus_yeast(); I don't know if any existing code cares about the return value but you might want to have a look.
This breaks backwards compatibility with the master/minion relationship. It also adds data to the publish, which we try to do only if absolutely needed.
I like the idea of doing this logging on the master, but I don't think that the minion needs to be aware of what user executed the command on the master.

I am going to try and keep the logging on the master, but I am taking user out of the publish.

Arguments, objections, comments?
I guess for/first would be the best approach. That way the Racket implementors can decide how to most efficiently get the first element.

  (for/first ([e (set-ht set)]) e)
Oops, I misspelled 'grammar'. Classic example of http://en.wikipedia.org/wiki/Muphry's_law
@gerv do we need the license on HTML template files? Across our webapps and services, I think the majority do not have a license header.
@niklasb `set tw=110` and then `gqip` x 5 :)

I'd do it for you, but that'd mess up the history log... I'm not trying to be difficult here, I operate at 80 columns most of the time (two windows on a 13" screen), but philc asked me to use 110 for this, so I'm just passing the message on (and keeping the codebase consistent).
Как можно видеть удаление идет по id =\ вообще для чего мы мутили gateway главный чтоб туда вынести insert update delete , ты тут делаеш  UserGateway gateway = new UserGateway(conn);
а правильнее будет так
 GateWay gateway = new UserGateway(conn);
и ты тупо так щелк gateway.delete(id); и в дамках
и на заметку это не должен быть повтор в ответе того что я написал, там есть еще одно правило...
я тебе описал только сущность одного сообщения
"some" explanation is an understatement :P 

Thanks for the thorough documentation, I now understand things I didn't even realize I didn't understand before!
Also, looks like we committed broken merges again?
Skrev en ganska långt "inlägg" för några timmar sedan, men webbläsaren läste sig, vilket gjorde att jag förlorade allt.

Fattar mig kort; använd [fixtures](http://railscasts.com/episodes/158-factories-not-fixtures). Factory girl finns redan uppsatt i projektet.
I see, thanks!
Is this notation "(void)name;" a common practice or your personal preference?
Why not simply "name;" - is the compiler too intelligent?
I seem to have introduced a bug whereby friction doesn't always work when you're moving left (and, oppositely, friction in the right directions seems to be stronger). I'm not sure where this bug is coming from.
Кстати абилки залипают после героизма вроде. Может быть и панель тоже
@eduardocereto: The context of a function called in that way is always the global object. `this` may not be the global object outside of that function, depending on how/where this code is running.
nice, feels almost like working on canary
yep, will push SAI sql.
Wonderful! Are there any arguments for which the `le(x,y)` functions give a different answer than `!lt(y,x)`?
bugfix (missing module)
yup, makes sense.. todo for the next release
no. check below include check per-effect immunity (see in code). in your commit unit immuned to spell if unit immuned to _ANY_ effect of spell - this is wrong and broke Mass dispell at least (and some other spells). unit immuned only if immuned to ALL effects, as checked currently.
Whatever works for you. I still use both plugins.

if (is_array($required_fieldlist)) { <--shouldnt that be always the case if the second parameter in serendipity_db_query is true?
I followed FindHWLOC.cmake, which sets HWLOC_INCLUDE_DIR.  I'll change OFED and UDT back...
This else if looks like it could be an if.
A preposition is missing before "internal" ("...is added to an internal queue.").
confirmed! #2832
but who was phone??!?
I suspect there will be things I want to actively monitor before all of this is over.  I should flag it with TODO so I don't merge it that way.
Sure. I look forward to elaborating the existing ideas further... :-)
We don't want facter's PATH and RUBYLIB to include puppet right?
why Evan Nazareno in the changelog????

The pull requests is by Alessandro Rossi (AlexRed)
"Checked" works for me. I think we use this subscription for calculating your "Current" tasks but it's really underused and confusing. Agreed that this should probably be either removed completely or improved.
If you want to be 100% kosher then it should be

#if (__GNUC__ >= 5) || ((__GNUC__ >= 4) && (__GNUC_MINOR__ >= 6))

But we can just remove all this logic (it survives in separate branch anyway).
`plugin['licenseURL'] = 'http://www.adobe.com/go/eula_shockwaveplayer'`
This will find all compatible versions. Ordering by id DESC will get the latest one.

Note: This query isn't all that new... it's what is currently in use in versioncheck, with slight modification to what fields it's getting.
is this indented correctly?
Probeer je het onder Linux? Ik zal er de komende week achteraan.
This `setTimeout` is simply to test triggering a search result and wouldn't be included in production code.
Можно было в 1-ну миграцию все засунуть...
Perhaps the name of the variable needs to be changed too? (Maybe make it more descriptive, and less colour-specific).
ovšem, jak se zachovat ,když do dat předám SqlLiteral?
I prefer [ ] for arrays and { } for maps... this notation is very common in any languages...

But if we will use { for arrays, we need change toString() output...
`+=` is particularly slow, like in Python. You need `<<`, which is like `append`.

Alternatively, you can do `@d[0, start + 1].reverse!.map(&:to_hex).join`
mousemove will generate a ton of events. if we $apply every time this kind of event is fired then we'll kill the app.
These two can be consolidated with *CONFIG.INI
You could also try 

ul {
  margin: 0px;
  padding: 0px;
  list-style: none;
}

then you would not need to position each .menutext

... but your way works too :)
I reread the manual more closely and the manual has the same syntax for other reactions formats as for PLOG where they specify cal/mol for units, suggesting that there is nothing special about the PLOG case.
For reference, here is another relevant section from the MFC manual:
<pre>
Unless modified by options specified on the REACTIONS line or in Auxiliary Reaction Keywords,
the default units for Ai are in cgs (cm, sec, K, mole), the exact units depending on the reaction.
The factor is dimensionless. The default units for the activation energies are cal/mole.
</pre>

So, contrary to my initial interpretation, I now interpret the manual to be saying that the units of Ea in PLOG are overridden by the overall file units. That's not to say that I believe that this is necessarily what Chemkin does, particularly since there seems to be a persistent belief in our group that there is something special about PLOG Ea units. It will be interesting to see what the tests turn up...
Yeah, it's a bit ugly..
i really like this idea (especially the 2nd commit that removed gtk usage from base ;))
i must say threads are very dangerous (unfortunately :( ) but hey that's what the development version is about!

though i have a few worries about this change:
what if user chooses a reordering and then while the thread is running another change in ordering is requested and/or some contact come online/offline/change status issuing some other change to the model? are we safe with this situations?

i am asking because i know users are very picky and like to break stuff ;)

also: pinging @sbte
Much better! One minor correction: The names of the tokens are a bit misleading, since the paths they represent do not point to the data directory, but to the directory above (usually; setups may vary). I would therefore suggest to remove the data part from the token names (and also from the variable name, i.e. mwToken instead of mwDataToken).
What happens if the user has disabled referrer from being sent?
Not sure how to get oldTheta from itsTransform.. Need to check that out..
fair enough...suspicious string concatenation then :)
Looks good, would like to see support for \n + \t in the code formatter but it works really well. Also needs a brief description of how it works, probably the best way to do that would be to populate the field with the initial string and let the user edit that. Or have an array of 'common' formatting, such as minify and have it as a list of radio buttons (with custom as a selectable option if the user started editing).
I think it should be 30 minutes to 90 minutes, which should give a continuous, even distribution
什么意识
r is, maybe, in the wrong place, I don't know if this works.

what I did was just add the r in the execfile as in my previous snippet

execfile('%s') -> execfile(r'%s')
Since this commit I cannot use NULL columns with MYSQL anymore. 

I get an UnexpectedNullableFound Exception although I am using: 

get[Option[...]] and as(scalar[...].singleOpt. 

It has definitely worked before.
Thank you for the link about the settings.xml.
I've created the file in ~/.m2 directory and now my bad practice to put the
name of my repo in pom.xml is gone.

On Fri, Mar 2, 2012 at 8:04 AM, Emmanuel Bernard <
reply@reply.github.com
> wrote:

> It's considered bad practice to embed the name of your repo in poms by the
> Maven community, so we follow these guidelines. Set your ~/.m2/settings.xml
> with the info. Everything should be described at <
> https://community.jboss.org/wiki/MavenGettingStarted-Users>.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/seiyak/hibernate-ogm/commit/0ded1effd0c4552fcabe7b36b60626e533857875#commitcomment-1035503
>



-- 
Thank you
Seiya
Derp, fixes #16!
When adding a username / password, or changing some other stuffs that are not available through the Web UI. I personally use it from time to time.
hmmm, thing is that this is the way I am used to code my css, it would be rather confusing for me to do it in another way. May I leave it as is and I or someone of the ss teams changes it once I am done?
I guess this *was* wrong.
There are no comments on this commit, or are there?
Can we name it as Adapters ?
So the namespace `Aura\Http\RequestAdapter;` becomes namespace `Aura\Http\Adapter;` Thoughts ? May be we can discuss with @pmjones
tiny style issue but i'd prefer no blank line here
People haven't seemed to agree on the what way being AFK could be IC knowledge.
Yes, there is already a sample plugin, please take a look: https://github.com/ccidral/tomighty-helloworld-plugin

Your plugin can listen for timer events on the message bus.
well, shouldn't you implement it properly by checking if the Entity annotation is there ?
Just being picky, but since we only have one container and the styles are specific to this div, it should be an ID.
Have You tested it ? Because I've tried this one, too, and I does not fix it for me.
Best thing is to make the changes, commit locally, squash the commits and
force push. Github will pick them up for this pull req

thanks for the time you're putting in this
On Feb 6, 2012 7:51 PM, "justinhayes" <
reply@reply.github.com>
wrote:

> So once I fix all these should I just do another pull request? (first time
> going through this process)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/justinhayes/infinispan/commit/1cfc229a6adffdaad53b16a54a74cee4517a488f#commitcomment-937672
>
but requires infinite more amount of thinking:
+/- urand(..) ?
are text-entries in proper order?
This commit has all current tests passing, at least with the default USE_HAYSTACK = True. I expect some issues once USE_HAYSTACK becomes false; relatedly, this will be a good opportunity to refactor the tests on listings to be more unit-y.
porque cambias el sentido de eso? No es mejor dejarlo como estaba por si mas abajo se usa para otra cosa?
assert type != null
nope. GetAI works on Units too while AI() only Creatures
Unfortunately I think that with this pattern you would have to invoke _hasLockOrIsLockBackup(key)_ again while holding the synchronization.

Otherwise the condition might have been resolved before you enter the lock, and will wait for only the next lock acquisition .. potentially a long time.

Can you proof otherwise? I might be missing the bigger picture.
This is fixed in a later commit. :)
nvm, i saw the later fix of this :)
made the pin select code for the Ethernet and SD card, non platform
specific by use of <pins_arduino.h>

CRITICAL
we have added code to turn off BOTH the ethernet & the SD card, before
calling each library to configure hte functions.
This is necessary, because a 'warm' reset appears to leave the pins in
an undefined state, until the library is called
음. 이슈번호를 잘못 입렸했네. 아직 커밋 리셋을 잘 모르겠다;
jetzt wird unterschieden, ob nichts, ein element oder mehrere elemente gefunden wurden...
shouldn't the method be named ``wrap`` ?
is no proplem. is comment
You need to pull from upstream and merge it in to your master branch
http://tinyurl.com/6v32qgo

Vielleicht besser: failOnValidationErrors?
Don't you need to change the compiler to set this flag?

But I would have done the spilling via the sort alogithm hint string...
I suspect that yes, otherwise it will create surprises.
Hi,

I've upgraded almost everyhing. Unfortunately, will I push changes, or not - depends on my customer's decision. I'll keep you updated next week.
Firstly, if you pre-increment you're already assigning the value.

Secondly, why are we incrementing that counter at all when we upload a client record?
yes, this information is move to buysell.conf. 
I will remove this info from this file.
actually the other option which i prefer would be to handle this in the extension .. aka the user configures the type of object manager .. and then the service definition is dynamically adapted
you can remove the else branch.  In general, I think that is preferred as it reduces the nesting

if (obj == null) {
    return;  
}
.. do normal stuff ..
Your welcome.  Keep me posted...would love to have you fork and add reviews.  I only have so much time to add functionality.  Thanks for working with me on this.
Please restore the ParameterizedFieldWidget and ParameterizedWidgetFactory. They are required to associate the recurrence widget with particular start field (for example).
```additional:``` for middle/etc. I'm wondering whether that is most semantically obvious option...
right! I follow your advice.
No need to wrap in HTML elements, back-ticks work and YUIDoc will process them.
Better to set default value to "Filter Items..." so that user can understand its functionality
I grepped the entire source tree and couldn't find them, which was fairly surprising.
object-in-class why?
talking to Björn made me aware of how fragile this solution is (Members comparing equal when in fact they are meant to be different). Just think about what happens if `intersect` chose to pick the resulting set randomly from left and right? There is nothing in its contract prohibiting such behavior.
No shutdown anymore?
Well, doc could be improved. Not clear at all what it should be used for nor its semantics.
The only alternative I see is to duplicate basically the same logic one more time to search for the ones I have moved from EXITING -> REMOVED (in this section) and from LEAVING -> EXITING (below). And if you want to get rid of the 'hasChangedState' var then I also need to duplicate the code for the JOINING => UP section. That is both less efficient and move verbose. Or do you have a better solution?
Or even having registerOnMemberUp be:

    def registerOnMemberUp[T](f: CurrentClusterState => T)(implicit ec: ExecutionContext): Future[T] = {
       val p = promise[CurrentClusterState]
       val f = p.future map f
       currentClusterStateListenerActor ! AddListener(p)
       f
    }
Nevermind, I wasn't thinking of the growing and shrinking
Aren't you running SBT? (Scalariform ought to have changed this into the right-arrow)
I still think it would be good to have a default. Now we have the same line in 20 tests. Don't see any value in that.
Hello. What do you mean by BC ?
The thing is if we want to select count on composite primary keys, the easiest way is to use a wildcard.
The distinct is also completly useless on selecting primary keys
given that siemens treats interleaved in different ways depending on the number of slices (see hack in [1]), should we in fact just allow only a list of integers? if siemens changed their behavior tomorrow, we would probably only hear of it 5 years from now. but making it explicit, at least the onus is on the user to find out.

that said, we do get slice order directly from the dicoms and hopefully this will be encoded in the nifti header extension that brendan moloney is creating.

[1] https://github.com/satra/ADHD200/blob/master/resting_process.py#L45
As for `@include`
Are you saying that if you rinku sees "http://example.com/х" in, for example, an email, it's by definition not a URL and thus shouldn't be auto-linked? The autolinker that GitHub is applying to this comment doesn't have a problem with that. It uses the original string in its original encoding for the `<a>` contents, and the URL-encoded version for the `href`.
I'm going to circle back a recheck what this pre-able stuff does, including the re-openning of some classes.   I'm going to get everything working again first, then circle back.
Could traceContext be None at this point?
Reverted to true. Basically, it was returning true if already existing, but doesn't make sense sense it's creating it.
Test dependencies should be added to the Gemfile, rather than the gemspec, as this file is generated and we don't want to force every user of the gem to install these libraries.
It's handled by the QUndoStack they're pushed to
Removing comments for pool stuff now.
Could you give more context to this comment?
same comment as well. but it could be an enum to make it clearer
This needs be renamed.
no need for lazy val
some copying going on here as well.
Path normalization is done by the `file` function, which is the only exported function.
On the other hand, the `File` function (which is a constructor) doesn't need to do that.
This entire block needs to be in tabs
... of the current ...
fFixed.
In principle LGTM but a regression test would be good, I'm not able to trigger a memory leak with getPeerCertificate().
`Array.prototype.slice` will avoid creating a throwaway array. Nitpicking, I know.
That change must have snuck in when doing some performance comparison with node.
Yep, this is wrong. We should respect use statements...
fixed this, but still the same problem - loops = None instead of 5
Yes, you're right, I should figure out what the error conditions are and error code.
Check out Andrew's photos on Facebook.

If you sign up for Facebook, you'll be able to stay connected with friends by seeing their photos and videos, staying up to date with their latest status updates, exchanging messages and more.

Join Andrew on Facebook
http://www.facebook.com/p.php?i=100004081283072&k=AQALx_GAcvlcP63KwoTCS1OKn7fCOuQUCmYkH0DNuZ-tpCkb6dRXsoqb7e_Tc90drF2lwFqtUXmqkpmfGFuQ1jqbxeQ&r&app_id=350685531728

Already have an account? Add this email address to your account:
http://www.facebook.com/merge_accounts.php?e=reply%2Bp-1211248-36a7e66af708b00c7d781e27fda83344ea8cc4c7-579774%40reply.github.com&c=AQB2-u8FSz_60GGbdE6I1ZJ6F4ECNV8PutZ94gMYSq3V3Q&source=unknown

=======================================
This message was sent to reply@reply.github.com. If you don't want to receive these emails from Facebook in the future, please follow the link below to unsubscribe.
http://www.facebook.com/o.php?k=AS2YjW5Ax2RljE1r&e=reply%2Bp-1211248-36a7e66af708b00c7d781e27fda83344ea8cc4c7-579774%40reply.github.com&mid=HMTEwMTI0NjQ0OnJlcGx5K3AtMTIxMTI0OC0zNmE3ZTY2YWY3MDhiMDBjN2Q3ODFlMjdmZGE4MzM0NGVhOGNjNGM3LTU3OTc3NEByZXBseS5naXRodWIuY29tOjg
 Facebook, Inc. Attention: Department 415 P.O Box 10005 Palo Alto CA 94303
Use 1.9 hash syntax for new features, Thanks :smile:
I don't think we need to check for `defined?`, since Ruby 1.9 has `initialize_dup`, or not?
In this case, couldn't we just expect people to say `extend ActionMailer::Async` in their class?  Then the `ClassMethods` module and this method become unnecessary.
Maybe Latex that?
This is a more delicate change...I'll see what I can do.
Oh, that makes sense :)
Because of changes to how random integers are produced in Py 3.2, tests of random behavior may not work across all versions of python regardless of your seed value. So these tests are now causing problems in pull request #1331-- they pass under 2.5 and 2.7 but fail under 3.2. The 4 possible solutions are: 1) remove the tests, 2) make your own randint function: randint = lambda i, j: i + int(random()*(j-i+1)); this will not be as good as the py 3.2 version but it will probably be good enough and will be cross-version compatible since random--but not randint--is cross-version compatible, 3) patch random in compatibility so it works across versions (which was the rejected attempt I made in #1310); 4) XFAIL the test -- it will pass when our least supported version of python is 3.2.
Count me as voting for adding `groupby` to `iterables`.
I don't understand enough of what is going on here to suggest an alternative.  What do your data structures look like?  Why can't you just call `whatever_class(*args)` instead of `Basic.__new__(whatever_class, *args)`?
Oh, I didn't know about that constant and this functionality.  Thank you :-)
wrap to a second line
I think `header_remove` equals setting empty header. At least if was so when I've tested (no headers were sent when value was empty). So I don't think we need this 5.3 check at all.
Sorry, troubles was with umask: http://www.php.net/manual/en/function.umask.php (ive used it in other revisions).
I guess Ive just copied this from other yii code, suggesting that this is some kind of style.
Do you need to with the git variant?  I didn't when I did it ...
this should better be `(r0.head must_== (1, 1)) and (r1.head must_== (1, 1))` to preserve nice failure messages instead of having "the value is false". Or you can just enter a newline:
```
r0.head must_== (1, 1)
r1.head must_== (1, 1)
```
If you conditionally add the reverse change, you can avoid these null checks.
Ah, I see it doesn't really matter if this never actually shows up directly in the public API, but I think the behavior is wrong.  The physical meaning of a `datetime` object is an instance of time (year, time, etc.).  While we use `hours` as a convinient unit of angle meaning "1/24th of a full circle", converting from an instance of time to an angle is not physically meaningful.  So I'm mainly just making sure this doesn't work its way into the public API at some point.
@fabpot the way you build the requirement is wrong here. You should add the upper bound too (it generally does not make sense to release something allowing any future version as you don't know the future, so we should do it properly when ``$spec['max']`` is available)
I don't understand the benefit of this category, there is already `-isKeyWindow` on ` NSWindow`.
Удали этот метод и получится эмуляция кода: begin_of_association_chain.events - это то, что надо, только без всяких костылей
I think the names of these tests could be more expressive and leave out the test prefix, e.g.

```
@Test def asciiCollationWithLowerCase = assert(UnicodeCollationOrdering.sortsBefore("a", "b"))
```

or similar.
although Warble Weekly could be a cool e-mail……
Indentation seems off here.. shouldn't it be one less level as before?
And especially here.
I really do not like such statements.
Needs a comment about what this regex does.
I saw somewhere that these two lines resulted in an error. So commented them out. Will remove these lines in my next commit.
Is this function name correct?
The choice was arbitrary and likely has been copy pasted from python implementation.
We've been attaching stuff to the `phpbb` object, not prefixing it.
do we need to destroy the server?
:ambulance: controversial
Why did you add the `bin` directory?
soryy, it's fixed now
if the tests no longer benifit from a shared engine instance then we might as well remove the field and just use a local instance variable in each of the tests. Since the tests runs in isolation it doesn't make much sense assign a value to "this.engine"
Putting meta.nix in a separate file would only make sense if there are multiple versions including it. So it's better to just inline it.
I notice we often (always?) call the Util.Datetime method with these arguments... Would it be better to create a new method
```
// util.js
getCurrentTime : function() {
  this.Datetime(new Date(), Util.getUTCGMTdiff())
} 
```
so that the consumer of the method doesn't have to remember to pass these arguments?
Yup - it's an obsolete line.
Should be grouped with the matplotlib import.
-> "Left for matplotlib to automatically set the limit based on any plotted data."
No, because it doesn't have any of TimeTrackable's fields.
logging и `else`
Merge damage. Fixed.
No asserts in this test.
Yep true...I'll update.
Yeah sorry... I added it back but in a different place. I guess this could actually just happen in the `read_multi` call cause that's the only place it was causing issues. [This line](https://github.com/mperham/dalli/blob/aa7ff874dc3e522ad9255f363705a660d6727e7a/lib/active_support/cache/dalli_store.rb#L127) was failing to lookup the entry inside `mapping` because the key being used there (which came back from memcached) had a different encoding than the one that was passed in.
Couple of style nits:

 * omit the 'then' noiseword for multiline conditionals

 * use a block to chdir, and then you don't have to chdir back at the end, preferred style is:

```rb
cd 'PerlMagick' do
  system ...
end
```
Thanks for the comment. I presume it makes sense to use the opt prefix with YUI Compressor as well? (line 21)
actually there's `casper.removeAllListeners('page.confirm')` for that :)
I think it's better to use OC_Appconfig
Testing more accurately is a bit tricky because some unknown amount of time has passed since the value was put into redis. A 30s window may be somewhat excessive though.

Although the asserts and expected TTLs are the same, the timestamps being fed in via the backup header are completely different (the frozen ttl one is a day older).
Need to rename the second `kw` here to `kwargs`.
Hmm sollte das flag auch noch in den if block?
Yup, reverted that bit.  Instead I'm going to figure out how to tell pylint to ignore that specific case.
Use just one `#` at the start of each line.
How would `$maybe_posts` ever get converted into a WP_Error instance?
对于没有sudo权限的人，我的建议是去管IT的人那里吼一嗓子，比自己折腾`./configure`更有效。我知道这个问题上我偏见比较大，但我真不想鼓励用户陷入这种泥潭。
this will cause issues if you have multiple errors, we should use
https://github.com/visionmedia/batch

to delegate
If you delete this line, you can get rid of the !important directives on lines 173 and 186
Doubt here...
Please add a space between the two functions
Fair enough -- will add
Is this TODO comment still valid? It looks like <li>s *are* being used.
Another option would be to provide a single lookup function so a client could key the bindings for a given command. 

-----Original Message-----
From: Raymond Lim [mailto:reply@reply.github.com] 
Sent: Thursday, May 24, 2012 4:16 PM
To: Ty Voliter
Subject: Re: [brackets] Menu and Command and Shortcut Extensibility implementation (#936)

> +     * @param {string} platform - the intended OS of the key.
> +     */
> +    function removeBinding(key, platform) {
> +        if (!key || key === undefined || !_keymap ||
> +                (platform !== undefined && platform !== brackets.platform)) {
> +            return;
> +        }
> +
> +        var normalizedKey = KeyMap.normalizeKeyDescriptorString(key);
> +        if (!normalizedKey) {
> +            console.log("Fail to nomalize " + key);
> +        } else if (_isKeyAssigned(normalizedKey)) {
> +            delete _keymap.map[normalizedKey];
> +        }
> +    }
> +
>      // Define public API
>      exports.installKeymap = installKeymap;
>      exports.getKeymap = getKeymap;

And also we may still need it for unit testing. Otherwise, it is hard to verify the key binding APIs.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/936/files#r878887
That's deleted code...
Nope, `exit!(status=false)` is how it is declared according to the [documentation](http://ruby-doc.org/core-1.9.3/Kernel.html#method-i-exit-21), so `exit!` exits with a failure status.  Try it for yourself.

```shell
$ ruby -e 'exit!'; echo $?
1
$ ruby -e 'exit!'(true); echo $?
0
```
The lack of an interface for including inline media in a Django widget is clearly a bug (https://code.djangoproject.com/ticket/13978) and this is the simplest way of getting around it.
final is necessary as they are referenced by the anonymous inner class
Don't need @auth anymore
I like this one. I think it is really safer.
One more tiny change, if you make this `use_vcr_cassette "Muni_Route"` then you can delete the `spec/cassettes/Muni_Direction.yml` cassette since there's already a mock request for `Muni::Route.find(21)` in `spec/cassettes/Muni_Route.yml`.
pep8 only wants 2 empty lines here
Never hard-code URLs.
Or probably better: signal handlers go to `account.models` (because they modify models). But Celery task goes to `frontend`, because it pushes things to the client.
It can be just `LimitedDateTimeField`. We are always importing just modules, so we exactly know what we have (form field for model field).
`FileUtils.mkdir_p` accepts a list, so this can be rewritten as:

```ruby
FileUtils.mkdir_p files.keys
```
How about sneaking the `zIndex` property in inside the `addMedia` function when it creates a `Media` object?
Haha, my bad.

That was how it already was. For some reason they were all in their own bracket.

From here, I thought you were saying I should remove what I added.

I did clean it up, though.
Ah that is my fault. I told him to split up that line, sorry.
This whole block ended up being unnecessary when using ```platform.uname()``` in ```os_data```
We might make `combine` more complicated in the future. Also, it increases the size of the core, making GHC less likely to inline things.
bah. Sad this can't be extracted from second arg. I'll test with this.
We're using `row-fluid` instead of `row`.
I think I would have written this a little differently:

    sqs = (SearchQuerySet().filter(title_auto=query)
                           .values_list('title_auto', flat=True))
    result = [query, list(sqs)]

(or however that gets indented)

Then we're defining the query in one line and it gets executed when turned into a list forming the result set.
In Firefox 14 the title set on ```<a>``` is not shown.
Thanks, fixed in my copy
record is not defined, I would just use i and ii as we do normally in for loops, so:
for (var i=0, ii=records.length; i<ii; ++i) {
I realize this is a port of the previous line that was here, but is it just me or does this seem strange?
I think there's a way to make a link open in the subpane by just giving it a class.  Maybe give it the '.navigation_element' class?

The name is a little misleading, but i think that will make the link open within the panel.
I would write it more "Ruby way"
```ruby
details = if task.progress.error_details.present?
            task.progress.error_details.map { |error| error[:error].to_s }
          else
            task.result[:errors].flatten.map(&:chomp)
          end.join("\n")

```
@stof I would say yes. Channels are present to filter the incoming messages.
http://pusher.com/docs/client_api_guide/client_channels
Can you remove the unnecessary line breaks here?
Također, nije mi jasno zašto ovo. Da li je to za resolvanje event stacka. Kad se događaju instanti
Stop using br tags
@eteq I really like that `c.galactic` calls `convert_to` and both are accessible. +1
I don't think we need warning about the memory. If you have a memory issue use `convert_to` and delete your initial coordinates.
But there will be a small amount of time elapsed between the nodes stopping and reporting success. If this process gets a chance to run within that small window (which will happen probably very infrequently because it's asleep most of the time) it will fail the assertion I think.
Not all releases are signed by Gavin (notably stable/backport releases are not), and users probably shouldn't be expected to setup the key themselves anyway. There are a bunch of PGP keys in git already for verifying against - any way to use those easily?
What is the App.namespace config for, then?
Any xss issues here?
can we move this inline style to the stylesheet?
Curious - what was the reason for this?
It would be great if we can consolidate `close_motion` and `close_voting` into a single method named `close`.
The `yylval` variable represents the token value. This variable is external variable for the lexer and is defined by the parser. Technically this variable has the union type. That union's fields are defined by the `%union { ... }` statement in the parser.
i did not fully understand the collections, though this comment helps a lot. to make things parse correctly, what do we need to do? or should i **not** autogenerate any of the rel/rev/property attributes and accept them to be empty (re the discussion in #11)
Fair enough; merging now.  Thanks!
yeah this is a bit strange, I know. Point is that if you need to send a SPECIAL_CHAR like <ENTER>, after that you need to give more time to the OS installer (i.e.: for page refresh) before sending another character.
maybe remove this?
this object is pretty gross (by spec)... I suppose we cannot use an xmlwrapper element to get rid of this & instead just be a string?
**NICE CATCH.** I did it your way, and verified it using a debugger breakpoint. I'll write an automated test when `git_checkout_commit` comes along.
You can get the name from the reference, and you shouldn't need the name split out. If the target refspec matches HEAD's upstream, then we've found it, no?
This should be updated to the new URLs on line 186 of this diff.
Oh, hrm. You're right--that's a typo: voted -> votes.
If you use `.format()`, this could be a lot nicer:

    log.info((u"User ({user}) has requested email change from "
              u"{old_email} to {new_email}.").format(user=amouser,
                                                     old_email=original_email,
                                                     new_email=amouser.email))
That's a minor detail, it runs once every 24h hours, so the impact should be minimal. If we want to change it, we can pull the static stuff to the constructor and just update the user-agents.
Maybe changing the if to `$param->isOptional()` above would be more appropriate?
It might be a security vulnerability if at the webserver socket access isn't restricted to the proxy IP, as any client would then be able to fake a client IP, wasn't this case the purpose of this change?
`@inheritdoc` specifically for this method, or for `IntrospectableContainerInterface`?  I didn't use `@inheritdoc` here because I didn't see it done anywhere else for this class.  Should I?  Good catch on the interface docs, fixing.
shouldn't you use ``$this->handle()`` here (which is unused currently) ?
I can implement an argument check.
If the value mod is invalid, throw an InvalidArgumentException
Wait, I think I might have hit the reason. PyTable optional arguments do not pass defaults correctly:

<code>ERROR RATE, just coming from python iface: 0.000000
ERROR RATE before bloom_init(): 0.000000</code>

So it's on my roof still, please don't merge this pull request yet...
If we didn't want to create the event we could mock the Event object's find, but that doesn't seem to be the pattern employed by this apps tests. I do prefer that method since it's faster. I can go either way.
@clofresh I'm not sure I'd agree with the memory requirements. Either parent and child share a lot of code, in which case they'll end up sharing pages in memory (with a page fault when the child tries to change it), or they don't and threads or processes incur similar footprint, with pages being allocated for both parent/child.

One risk to highlight: SSL timeout bug in the python interpreter, which will basically render the entire process unusable, instead of simply the check. This should be caught by the watchdog, but truth be told, the tests in place for the watchdog don't assume that the interpreter is in limbo.

At the end of the day I think threads v. processes end up a function of the amount of synchronization needed. In my experience threads offer very fine grained, thus error-prone synchronization primitives. On the other hand, processes are great iff you're used to unix programming.
We do not have scala 2.9.1-1 in artifactory. This will cause internal build problem.
Hmmm. Some nice exception handling might need to be considered for cartopy in general. It would be very nice to be able to catch this particular exception (rather than the generic ValueError exception)...
Is the `max() <= 1.0` heuristic really the best we can do here? Can we not inspect the dtype of the source image?