labofsahil/github-event-dataset-2012 huggingface dataset:
Switching RestKit dependency to new RestKit fork, updating to latest RestKit

We forked RestKit so that we might better track the latest bug fixes as well as add our own.  This changelist updates the iOS sdk to be compatible with the latest RestKit, via our RestKit fork.
Ok?

hey check this out
rowend/22-catalog-shipping

Translate catalog shipping folder
Open/7316 - fixed forum styling - cleaned up css and templates
Updated CSV file definition to V1.5 changes

Added support to Documents and files
Localization

Hi,

I've finally find out how to use github :-). So I'm sending my changes to you Gnome Extension. It's now using Gettext to translate menu item. So everybody should be able to create their translation. 

Pavel
Osio 2, booleanit
Module breaks use of Request.IsAjax()

As per issue: https://github.com/trilobyte/Premotion-AspNet-AppHarbor-Integration/issues/8

I would have thought it was an appharbor issue, but when I remove this module, the issue disappears. That makes me think this module is at fault - I've just done a quick re-write which seems to fix the issue.
issue #229; clean target; newline echos

Trivial and miscellaneous changes.
Make mysql-python reqirement dependent on detecting OpenShift environmen...

This might not be the most elegant way of doing it, or even entirely necessary, but this should make mysql-python only need to be installed on OpenShift, instead of being required even when it is not being used.
Janrain

Implementation of Janrain for use in both social login and social sharing.  Just plug in the Janrain API info into the Salesforce Fundraising control panel.

This branch also started collecting a number of other features and bug fixes so I'm merging it into master to allow better branching for future feature work.
I found this on github

I think this should send you a notification that I have made some changes in the code, and that you can merge them into yours.
As I've said in the commit comments I haven't implemented the 'populateDistributions' function yet, since I wasn't exactly sure what you were going to do for the segments. 
I will now look into reading the input files...
Неопределённая переменная
Urgent attension required. Sorry for this :(
Load tests to be merged with acceptance criteria and improved
add proxyUrl to getChatterFile
Chap9 french translation
It gets tangled up deep inside some darwin headers and borks the build, as for windows, not a fraken clue :)
You know, its Friday evening :)
It didn't before, I don't see a reason to change that.
`unless … && … or …` – really? I think this is more readable:

```ruby
unless (value.is_a?(Integer) && value >= 0) || value == Float::INFINITY
```
Yes it's in our (@cantoniodasilva and me) TODO list ;)
I understand what you mean but I think `elseif` if the right instruction here (according to me !)

With the multiple `if`s, you have to scan the code for potential flow exits and as I mentioned, this can cause problem with refactoring.

Both have pros and cons... matter of taste (or coding conventions)
context.stop in the catch could also throw an exception, (well it is not expected but it can), and you would probably want to know that the template could not start. (you would wonder why camel was stopped, where it was actually in the process of starting) 
(Not compiled but you get the point, might not look nice, but I think more correct)
     try {
       template.start()
      } catch {
        case e=> {
          try {
            context.stop()
          } catch {
            case e=> // swallow, so the offending exception is thrown
          }
          throw e
        }
      }
@zachinglis I fixed it. But is it ok that crummy returns something like `<ul>&lt;li&gt;smth&lt;/li&gt;</ul>`?
![fuck yeah](https://a248.e.akamai.net/assets.github.com/images/icons/emoji/metal.png?v5)
Normally compilers can be optimized for speed and for size.
However inlining a getter and setter method with body is always inlined since the call and return overhead is bigger and slower then the body itself.
So its always superior to inline that method. Even java is inlining getter and setter methods automatically.

"Inline" keyword is just a suggestion to the compiler. Its not meaning it will or will not be inlined. However if Visual Studio can't inline an "inline" method I get performance warnings. So the only difference is the warning message and if you use "optimize for size" it would also inline it like "optimize for speed". But getter methods are always inlined.
Looks cool! What about displaying "tree trace"? I mean: it would be nice if we could know where exactly it failed while processing given file.
I love your `<h1><%= t(:editing_product) %> &ldquo;<%= @product.name %>&rdquo;</h1>`. Please make a pull request and make sure you get the functional test of product details passed.
Dependiendo de como esté configurado el proyecto Rails que utilice kublog, la respuesta en JSON puede contener los datos 'envueltos' en una llave con el nombre del modelo o no. Si el parámetro `data` contiene el atributo `comment`, utilizamos éste para la invocación de `commentTemplate()`, de lo contrario simplemente pasamos lo que exista en `data`.

Ver http://apidock.com/rails/ActiveRecord/Serialization/to_json

```ruby
konata = User.find(1)
  ActiveRecord::Base.include_root_in_json = true
  konata.to_json
  # => { "user": {"id": 1, "name": "Konata Izumi", "age": 16,
                  "created_at": "2006/08/01", "awesome": true} }

  ActiveRecord::Base.include_root_in_json = false
  konata.to_json
  # => {"id": 1, "name": "Konata Izumi", "age": 16,
        "created_at": "2006/08/01", "awesome": true}
```
à corriger en computeNormalisedPerformanceTable(...)  (normalised avec un 's')
Shit, my bad... didn't read the first line in the doc... still.. must be a way to get those CPU stats without using iostat. iostat throws an error about disk information on a VPS while we ask it to provide us CPU stats... it doesn't make sense and it prevents you to show me my cpu usage.
They pass. Could you run the rest of the test suite, though? I think
this is a pretty sound change but you know how it goes.
My bad, I'd cherry-picked this commit from the regex-find branch without looking closely enough.
Thanks for this patch, it seems to work really well!

However, there are a few issues that need to be fixed, mostly to do with the coding style. I think the only other issue is that I don't think it's necessary to store the previous allocation and check if it's changed.
This introduced https://github.com/arya/bluepill/issues/140 (and #137).  I've fixed it in my fork b5858d00be44fb0039f885a8f3d05b0b18509db0 and submitted a pull request https://github.com/arya/bluepill/pull/141
Should add a test showing it raises a TypeError exception with "can't convert nil into an exact number" message when calling `Time.now`.
TODO :
- Create dedicated WebSite Form Handler
- Write main WebSiteBundle tests
What about this?

```ruby
def set_attributes(attribute_values)
  return unless attribute_values.respond_to?(:to_hash)
  attribute_values.to_hash.each do |name, value|
     # ...
  end
end
```
Thanks, I'll push
I can't remember either.  Let me test it out on some of my production code, and I'll sort out a fix or add this back in.  I track my fork here:
https://github.com/bigmlcom/tipsy
That is a problem of the current code base too, the add game and edit game dialogs have duplicate code. But I believe that this is a different problem than what this pull request is aiming to address
Okay I am convinced :) I've merged the changes into my project. Thanks.
confirmed patch works as intended
You can disabling random movement for this npc if it is emergency :)
it was because we were initializing `this.preview = new imagePoller();` when we didn't have previews to initialize. so this wasn't working: https://github.com/mozilla/zamboni/commit/6e7255#L3R907
Hey @vingtetun, what do you think about cherry-picking this hack to gaia fo the time being?
登录时的“未知错误“暂时仿佛无解。提交时的其它非alert错误就无视吧，挂着就行。
Are the GNU gettext files text files or are they binary files?
给力啊，可惜我不懂js，得去学学
But the reapply of areaauras are in next Tick. For better reading in this case this holder should ignore and wait for delete.
это правильно, а то задолбали уже всех, вечный крик почему мобы проваливаются и т.д. врубят VMAP, а потом кричат у меня серв не запускаеЦЦА или у меня мобы проваливаюЦЦА аааа помогите !!!
если я не ошибаюсь, то с MapUpdate.DynamicThreadsCount = 1 касты через динамические объекты не должны проходить?
Взять например стену на ОЛО и прям через нее там попробовать кастануть, каст не проходит. Но если её разрушить, и должен каст проходить, каст все равно не проходит http://i28.fastpic.ru/big/2012/0129/bb/35d3afb23f41369f7623d1e292cdcdbb.png
Most warden warnings are "memory check timeout" wich not means that player is using hacks.
Take a look at Twig's [`bin/create_pear_package.php`](https://github.com/fabpot/Twig/blob/master/bin/create_pear_package.php) for a way to automate package releases. Just a note.
Ok, LGTM.
1. i didn't know this word... it actually exists, but means something slightly different
2. also include the slovak translation at the same time
Oh, now i see :)
Is it visual packet?
:D thanks! I want to see more memes in code comments :D   they are so boring :D
I think that this was always intended, but there was no need to separate it into different files until we had unrelated stuff in there.
I was scared to remove them, heh. Eff it, I'll remove.
Ich hatte euch gesagt, dass dieser Test nur dazu dienen soll, Ausgabebilder zu erzeugen, die man sich *anschaut*. Man kann solche Sachen nicht exakt durch ein Programm testen. Das selbe gilt für alle Manipulationsknoten, wo man mit der Ausgabe "zufrieden" sein soll. Solche Arten von Tests werden wir aber noch besprechen.
Hmm okay I see from their README, that you are supposed to edit the Makefile.rules. Excuse me.

But at least `OPTIMIZE` does not need to be replaced since it is on per default.
I'll reject this because of the use of b"" – it doesn't work on Python 2.5.

Meanwhile, I have committed an alternative fix in ad935b5416ab7ed2509e7c9530c3805189a45de9. Thanks!
I will open a Issue to discuss the shinken.sh behavior :)
Ok thanks, It's pretty the same thing, but of course we won't log (the timeperiod change * number of service related to this period) times the same thing ;).

I suppose the log is smart : only log timeperiod used in checked, not all timeperiod defined ?

Danke Laußer :)
Can you fix them?
I've made it clear that the `length` parameter is mandatory for all list accessors except `eda_config_get_string_list()`.
Is this eventually going to fix the "missing subset of sources" issue? Because these changes break my tests: https://github.com/rubygems/rubygems/pull/105
space between ==
Fixed in 86bf3b1cd942bbbf52a3b572bc059bc7fce59f00.
I'm afraid I'm a victim of version chaos. Using 1.2.0 from the dist folder, or from lesscss.org, I get an error when importing less files using @import. The error itself is not very helpful: Cannot call method 'slice' of undefined (using Chrome).

Did this commit make it into the 1.2.0 hotfix? Why did you choose to update in place? I wonder, what (remote) file do you recommend to include? Sure, I could copy the file, but in my case loading from remote is easier, at least for the time being.
To be more UTF-8 friendly, it's better to use Utils.string_get_prev_char().
Copying is really not a good idea, specially for Strings which can be large.
We can use references or pointers to avoid copy overheads.

P.S: these comments aren't personal. This is because right now I'm unable to help with code, so I'm doing some reviews here!
r+ awesome
looking good, @spasovski! this seems to work pretty nicely. a few minor things need some fixing as well as some good test coverage. you can abstract your tests so you're not writing the same exact tests for both "Step3" and the "Devhub Basic Edit" views.
It was in it, I deleted it from the gitignore, to add the classpath to the
repo, and then in this commit readded it in the ignore.

2012/2/22 Lathanael <
reply@reply.github.com
>

> i thought it was there, it is in all my other ignore files -.-
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Belphemur/AdminCmd/commit/4893f74a57f2438a5f925641b3a7207811c51495#commitcomment-995760
>
use `esc_attr()` - http://codex.wordpress.org/Function_Reference/esc_attr
So true. New machine and totally forgot to add a global .gitignore
I was really surprised when I saw that cheeky .DS_store file.

All set up now thanks.
Strings.isNullOrEmpty()

Also exceptions or logs.
timers and dmg is not sniff but more blizzlike when now ^^ ... because correct timers is global problem, you cant do correct timer in 90%
$localsubnet was defining where the connections were coming from. perhaps you could dummy loop over that.
be sure to throw ArityException if no len(arg) match is found. It's in cljexceptions. This will help for debugging
Moin Philipp,

wir hatten einfach zu viele Timer. Ich hatte immer wieder Probleme, dass
mehrere gleichzeitig aktiv waren. Daher habe ich die Struktur so
angepasst, dass die Funktionalität der Buttons nur an einer Stelle
definiert sind. Weißt du, was ich meine?

Schöne Grüße und ein schönes Wochenende
Sebastian

Am 12.01.2012 15:55, schrieb Philipp Abraham:
> ?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/itm/Tor-zur-Ostsee/commit/a0485394e5bafad562fc9a6de6ff108ece7f19e0#commitcomment-861063
> 

-- 
Sebastian Ebers, M.Sc.
    Wissenschaftlicher Mitarbeiter

UNIVERSITÄT ZU LÜBECK	
    INSTITUT FÜR TELEMATIK

    Ratzeburger Allee 160
    D-23538 Lübeck

    Tel +49 451 500 5394
    Fax +49 451 500 5382
    ebers@itm.uni-luebeck.de

    http://www.itm.uni-luebeck.de/users/ebers
Fixed in llvmopts branch.
Bad Vijfhoek. If you want to silently swallow exceptions (you rarely do), at least do not swallow ALL.
Done.  Will delete the tree and rebuild to make sure all goes as expected.
It's a bit strange to call this "DEFAULT" when it isn't the default sort order for JUnit4-style tests without the @SortMethodsWith annotation
I don't think this contract should be part of the Association SPI. Can you move it down to your own dialect and apply the logic.
You can also look at `MapHelpers` and `MapAssociationSnapshot`, they might contain already what you are looking for.
Why Are you doing this?
You might as well fix the original header too (which is missing the field name, 'Content-Type: ').
@Blastcore read my first comment again -_-
hi,

i have added the lower limit. don't know if you also did it, but not pushed.

https://github.com/DerTeufel/samsung-kernel-aries/commit/923fcd92cc40c1ba5e93c419f3edd9fcb7a69303
FYI, if you use master, then please provide another version via packagist/composer.  Right now we can only grab master-dev (which I think is latest in master)
really necessary ?
The Xilinx tools auto generate these files.  I used ISE Version 13.3, which is more current than 12.1.   The tool will create a new .xise project file when it detects a new version of the Xilinx tools.   In some cases when much older versions are used, the design may not be backwards compatible (i.e., the Spartan-6 FPGA isn't supported because it's a relatively new device).
I agree with @jorgelf 

I like null or raise a [IllegalArgumentException](http://stackoverflow.com/questions/3881/illegalargumentexception-or-nullpointerexception-for-a-null-parameter)
Venugh is there a way to contact you via Email, Instant-Messenger or some things like that?
Cheers: sele
Because I wanted to avoid the overhead in the dereference for every row fetched, and the usage was simple enough to be able to be reasonably sure it was freeing in all cases it needed to.

Probably premature optimization, especially given a decent compiler that shouldn't impose any overhead on the inline functions in Owned<>...
表示从来没遇到过这个错，我是wipe user data之后再启动avd也没见过这个错
Ah I see. I am enthralled in a weird ass movie right now....
so – basically no change to what we´ve already had, right?
Tx for fix, but fix not offlike. In off simply can not move the bag in some places during the trade.
I think this is wrong too, LoS should be checked just at the start and the end of the casting time
Thank you, that was an annoying little bug :)
This is too early in the morning, I shouldn't implement this method in a category, doh.
so we're sure that isPutForExternalRed implies as well that keyCount == 1 ? Why so, am I not allowed to use the Flag on a putAll ?
We don't need to go through asWidget to fire event - this would allow us to test them more easily.
Sounds great.  I am narrowing in on it.

I was able to connect using tinytds this evening.  I put my instance's info into ~/.freetds.conf and then connected by referring to that entry from the config.

Thank you,
John

On Feb 23, 2012, at 10:28 AM, Ken Collins wrote:

> If you find a solution for TinyTDS, I would be more than happy to take a patch!
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rails-sqlserver/tiny_tds/commit/c244f4faf24ca4e668371cbf9efaaff1dda78633#commitcomment-1003623
Flew with this no problem, packet rate @ 10Hz. Increasing to 20Hz didn't seem to change anything. Dropping to 0.1Hz did.
this will work as long as the option converters themselves don't need to be injected.  if that happens, have a look at: https://code.google.com/p/google-guice/wiki/Multibindings or similar approach.  good for now, though!
Ok. Please make a ticket as memo.
Hi,

do you want to create an extra pull-request for this change? If you do not want to, I'd do it for you…
if (ext != NULL)
   strcmp(ext, ".mdx");
else
   return 0;
This commit broke camera & HW decoding of video player for me.
Hm, I looked for 4 hours now but couldn't find anything ...
1.6.4???
![Facebook](http://i.qkme.me/35fjhu.jpg "Facebook")
it was my way to call suggestionControl, but I refactored it, not a big deal )
Merci a toi

Cordialement

Le 13 févr. 2012 à 17:59, "Ronan Pozzi" <reply@reply.github.com> a écrit :

> voila la nouvelle version est sur github
> merci
> 
> Le 13 févr. 2012 à 17:53, Doriginal a écrit :
> 
>> Ok d'accord bon bah fonctionne nikel alors a part se petit truc.
>> Ah oui corrige aussi "Quand est sortit" par "Quand est sorti" Sinon il comprend pas ;)
>> 
>> Cordialement
>> 
>> Le 13 févr. 2012 à 17:44, "Ronan Pozzi" <reply@reply.github.com> a écrit :
>> 
>>> oui ça c'est un problème lié à SiriProxy et non au plugin.
>>> 
>>> 
>>> Le 13 févr. 2012 à 17:37, Doriginal a écrit :
>>> 
>>>> Sa marche mieux lorsqu'on fait un requête il repond et la deuxième il répond pas,il faut fermer Siri et le relancer
>>>> 
>>>> Cordialement
>>>> 
>>>> Le 13 févr. 2012 à 17:11, "Ronan Pozzi" <reply@reply.github.com> a écrit :
>>>> 
>>>>> Bonjour, au niveau de logs qu'elle est la réponse de siriproxy quand vous exécutez votre demande ?
>>>>> 
>>>>> Envoyé de mon iPhone
>>>>> 
>>>>> Le 13 févr. 2012 à 16:19, Doriginal <reply@reply.github.com> a écrit :
>>>>> 
>>>>>> Bonjour,
>>>>>> J'ai installer votre version française de imdb plugin mais il ne reconnais rien lorsque je lui demande quelque chose, quel que sois la question, Dois-je voir, quand est sorti, qui a joué.....
>>>>>> 
>>>>>> --- 
>>>>>> Reply to this email directly or view it on GitHub:
>>>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960594
>>>>> 
>>>>> --- 
>>>>> Reply to this email directly or view it on GitHub:
>>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960763
>>>> 
>>>> --- 
>>>> Reply to this email directly or view it on GitHub:
>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960871
>>> 
>>> --- 
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960890
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960918
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960940
This is currently in my branch, but I'll merge in your other changes (you've done much more than I have so far) and then hit you up with a pull request!
Head you misspelled capital, therefore YOUR argument is invalid.
refactor function removeAllDigits
ORM happens to have a `__clone()` implementation for its QueryBuilder (see: [here](https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/QueryBuilder.php#L1097)), but ODM does not. We'll have to implement that.

The alternative would be for this class to create new QueryBuilders on-demand, but that would change the constructor interface significantly.
Aah nice, didnt get that yet, thx. Of course i'll watch your branches but time is pretty rare at the moment. But i'll keep an eye on that.


No i do not use recently added at all as my ATV2 is underpowered and i try to use as less as possible.. But good thought, would not have noticed that either. All the modificiations i do are to fit my girlfriends needs, thats why i am only able to use your skin on my MacBook at the moment. But i'll push her soon, i promise ;)
Thanks for finishing it :) I'll release an update today to get it tested
When using `.editorconfig` files in our other projects, we could add a several line comment to indicate what is this file for:

    ; This file is for unifying the code style for different code editors and IDEes.
    ; See http://editorconfig.org
Hey Chad,

I think you left out a few templates from this commit. Running this code now gives me errors because it cannot find:

app/templates/trackedgame/game_substitution.html
app/templates/trackedgame/game_action.html
app/templates/trackedgame/action_area.html
app/templates/trackedgame/player_area.html
Thanks for taking time to comment the code.  I think in some cases you're over commenting though which means more time on your part to create the comments, and slowing down others in the future as they need to read past them and/or update them when the code changes.  Finding the right amount of comments is certainly a balance.  You have the better problem to have though of commenting too much :)  Most people don't comment at all :(
Interesting! I didn't realize that it was a 1.9.3 thing, I was just happy it worked :) Anyway, its a good test to make sure everyone is on 1.9.3 I suppose
This was actually needed for the project I used it for, as the client did not want the timestamp on filenames
No this commit won't break any launch direction signs functionality.  It is only (a temporary) fix for the /trucompass command, provided by the AdminControls plugin.
here https://github.com/GalaxyICS/android_packages_apps_FileManager
Why better generate it with javascript? What benefits we get?
I wonder if that should be '\r?' to not break it for existing/older versions.
IMO, max limit size = 100.
Rakefile changes seem unrelated to the terremark stuff.  Could you split this out into a separate commit or remove it?
'replaceMissingValues() function is defined elsewhere in this file.' comment is probably superfluous - the first approach a developer takes is to search for a function in the same file.
Sure thing—yeah if anything I'd opt for being more concise on the command line and more verbose in the README. Maybe we should work out the best way to reword it in the README and then use that to decide what the interactive prompts should say?
Yes. It elimated the needs to "disable" the property in serialize.
Ok, no problem with that. If you think it is ok we could move the fixtures to data/tests-fixtures.
However, regarding the config.ini file, I tried to follow the elgg installation pattern regarding the settings.php file. In theory the tests settings should be written only once (and, as I told you in some of the first email exchanges, I think it would be nice to have that as a part of the elgg installation. In that way the tests would seamlessly integrate with elgg and there would be no need of this two-step process).
getInstance for consistency?
In the long run I think we'll want to use [ViennaCL](http://viennacl.sourceforge.net/).
The strict control I think is one solid reason. Another is that we don't need to deal with overlap and conflicts between named configs.  This is important because we're expecting that someone who updates a named config shouldn't have to know anything about other named configs.

Names of jobs/services can be handled by pre-pending the config name to them, but that doesn't work for nodes and other options.
so something like: 

    val numberOfRouteesPerNode: Map[Address, Int] = currentRoutees.foldLeft(currentNodes.withDefault(_ => 0)) { (acc, x) =>
      val a = fullAddress(x)
      acc + (a -> (acc(a) + 1))
    }
This is a bit unorthodox, why do we send ourself as a message, with ourself as the sender? I vote a more descriptive message.
not-a-fan here also
use auxiliary constructor instead of null
what nulls?
The setting in the cluster with the same name is a Duration. Is there a reason to specify it as a FiniteDuration here?
this is a bit of a strange name (don’t know what it is used for, yet)
Don't use JavaConversions, use JavaConverters
this definitely is more uniform and pleasantly written, I just think we should also spec it (as in: adding it to the docs)
I think you look at the wrong place, L151 in MultiNodeSpec:

    /**
     * All registered roles
     */
     def roles: Seq[RoleName] = _roles
Thanks for pointing that out! Fixed
Fixed in f842804.
So is this new test module meant to be the one place for all logic action auth tests now? If so maybe add a docstring here so people know to add their auth tests for new features in here not in their own test modules. Also might want to find existing auth tests elsewhere in ckan/tests/logic/ and move them here, but that can be done later.
... and 376 and 599  :)
apostrophe in "isn't"
I'll expand
Hey @elliottbiondo, before i accept this merge could you please make sure that this matches the indentation level it was at previously (2 spaces per level in C/C++).  Thanks!
Sorry, I think I was first thinking I could lose the var, but added back badly.  Feel free to improve the style as you merge and fix it.
Should this use errors! too?
make it 2012
give sample usage
self[enum_column] = enum_resolved.bitfield_index ???
No newline at end of file !!!!!!!!!!!!!!
That's just a require statement. The gem has to be present for it to succeed.

On Aug 23, 2012, at 4:22 PM, Fabio Rehm <notifications@github.com> wrote:

> In harbor.gemspec:
> 
> >    s.add_development_dependency "builder"
> >    s.add_development_dependency "redis_directory", ">= 1.0.4"
> >    s.add_development_dependency "minitest"
> >    s.add_development_dependency "mocha"
> >    s.add_development_dependency "listen"
> >  
> > +  s.add_runtime_dependency "erubis"
> > +  s.add_runtime_dependency "jruby-rack"
> but doesnt it check if its using java and require the proper rack version?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
You're right. You are probably the first who review this code which is quite old :).
Good catch, I definitely missed part of the line there. Fix coming shortly.
Fair enough. Did you see the commit I did to change that?
As above, regarding licensing.
Replace `size_t` with `uintptr_t`. Using `size_t` is not really an error* but it doesn't signal intent like `uintptr_t` does. Otherwise LGTM.

* Well, technically it is but there's not a modern system where it's actually an issue.
Add a test with a groupByKey or reduceByKey to check that ShuffleMapTask sends the files correctly.
Needs a comment for symlinked logs
Should be final and maybe also private
Perhaps.  I wonder if there are cases it makes more difficult, though.  One example would be:

    for i in range(100):
        alloc_log_enable()
        result = my_func()
        alloc_log_disable()
        other_computation(result)
    log = alloc_log_fetch()
    # analyze log for problems in my_func()

It would be a bit more difficult to do this if enabling cleared the log, but not impossible.

Perhaps enable/disable should take a keyword argument clear_log that defaults to True.
I won't use the changes in this file. This might be helpful to redo this commit: http://stackoverflow.com/questions/927358/undo-last-git-commit
After you reset to the state before this commit, you can use `git checkout prody_catdcd.py` to restore this file, the update `prody_blast.py`.
Иначе дабл-клик событие до элемента не доходит.
у условий и циклов фигурные скобки ставятся, даже если одна строка внутри
ну и так далее. все передает по константной ссылке
Because https://github.com/lexmag/rails/commit/6f050c954e8225dc1de187ea8e168bcb1d8f7e01#L1R68 expects no options exists.
I can replace `ref_names.extract_options!` with `ref_names. pop if ref_names.last.is_a?(Hash)`
https://github.com/rails/rails/pull/7805
Separate s.add_dependency reads better and fits in with the others.
I don't think there's the need to add `:existing` to the current api, it already works and is well known by people that use it.
yes, also we need `||=` because of custom headers, @josevalim  is right
Since we're dealing with parameters that emulate a hash, it seems to be more appropriate to use `KeyError`, not? (I know it's a subclass, just thought it might be a better error to inherit from)


```ruby
>> [].fetch 1
IndexError: index 1 outside of array bounds: 0...0

>> {}.fetch :a
KeyError: key not found: :a
```
@VladUreche: can you comment on this fragment?
Is this code used?
Two things: 1) "not crashing" should not be anyone's idea of "middle ground". 2) It is very clear from this thread that this needs to be specified. Are you trying to reify "free methods" or to reify the eta-expansion of methods? The test case (t5875-b, not that much to go on) only suggests the latter; these comments suggest the former. Ad-hoc attempts to improve on eta-expansion in reification context are a bad idea; letting cases which work in real source code fail under reification for no obvious reason is a worse one.

So can we more precisely articulate what it is we're doing?
It's only printing that in the presentation compiler, if that's any consolation :)
I intend to fix this before RC1.
Actually, I'm going for:

```scala
          // SI-5877 The decls of a package include decls of the package object. But we don't want to add
          //         the corresponding synthetics to the package itself.
          def shouldAdd(sym: Symbol) = inBlock || !context.owner.isPackage || !isInPackageObject(sym, context.owner)
          for (sym <- scope if !shouldAdd(sym))
```

`isInPackageObject` is used for this purpose elsewhere, and contains some careful code to avoid forcing infos earlier than needed.
or converted to TeX?  or is that only supposed to be in the Notes section?
this and following comments should reflect the n-1 end point, e.g. {0, 1, 2, 3, 4}, right?

Also, if you feed the *args to slice it will expand them to all three:

```
>>> s = slice(*[1]);start = s.start;stop=s.stop;step=s.step;start,stop,step
(None, 1, None)
>>> s = slice(*[1,3]);start = s.start;stop=s.stop;step=s.step;start,stop,step
(1, 3, None)
>>> s = slice(*[5,3]);start = s.start;stop=s.stop;step=s.step;start,stop,step
(5, 3, None)
>>> s = slice(*[5,3,2]);start = s.start;stop=s.stop;step=s.step;start,stop,step
(5, 3, 2)
```

If you don't accept reversed ranges then I suppose you could use

```
s = slice(*args)
start = s.start or 0
stop=s.stop
step=s.step or 1
```

You should probably do a check on acceptability of input wrt stop > start and step > 0.
On 17.04.2012 11:10, Christopher Smith wrote:
> Do not instantiate; the constructor does not check that your data is well defined. Use
> the ``homomorphism`` function instead:

Done (both).
I can restore those, but the question is do we really need them. Actually, I wanted also to remove `gcd()` and `lcm()` methods but at least `gcd()` was used in the core so I didn't want to do too many changes. If we we decide to keep all those methods, then their implementation will be trivial (will use global functions) and should be moved to Expr. By mistake I left `gcd()` and `lcm()` in Number, so now you can do `S(3).gcd(3*x)` but not the other way (this has to be fixed).
I said in #313:

> I have already fixed this; did you run `git pull --rebase git://github.com/yihui/knitr.git origin master`?
Should be ordered according to `Enh #`.
Are you sure that you want to use RS485 mode?  Normally, RS485 drivers use separate Tx and Rx data pairs, and that's my understanding of the way that RS485 mode is implemented in the STM32 UART driver for NuttX.

I think you probably want a separate mode here; it's not RS485, and it's not LIN.
A clean install will use db:schema:load
Da ein Stern \* eine Formatierung beginnt, kanns bei manchen Parsern und Syntaxhighlightern zu Problemen kommen. Wird mit `\*` escapted.
looks like initNetworkMetaAnalysisModels() would be a more appropriate name
Could we have uv_tty_t pass the associated stream through to the child (I.e. the equivalent of what setting the stream to 0 does right now)?
This section is reasonably redundant with existing sections of the documentation. We don't need to tell people again to pick a good shard key to avoid the problem in the first place in the second paragraph of the section, if anything say "if you can't avoid picking a monotonically increasing shard key, then do these things."  with a link to the shard key section. Read the choosing the shard key section in this file and in the internals file and see what can be salvaged and cross referenced.

The suggestion about UUIDs needs to be dropped.
expunge use of `This`
No. The locale in the URL supersedes Accept-Language. That header is only really used on a visit to something like `/docs/HTML` with no locale in the URL at all.
Yep, the deploy resource depends heavily on symlinks, and `FileUtils.ln_sf()` doesn't do windows shortcuts.
COmmit messages should just be brief description of what was done. I often do similar to yours, like "fixing things", but I agree that it is very helpful to have some sort of description of what was actually fixed. Otherwise, it is difficult to identify which commit actually does what at a glance without actually viewing the changes of each commit.
Noted. Will fix it in the next commit.
Performance is really really not an issue with a couple of variable assignments. The only reason to do anything like that here is to make the code more readable. The sort of micro optimisation that was talked about here is a waste of time and entirely unecessary.
Where are assertions that check how conflicts are handled?

Also, I would prefer conflict handling to be in separate test(s). But if doing that would be too much work leave it as it is.
This version should be ansi sql, which in my estimation is || and not concat.
Joas, function lang() is a method of class phpbb_user() ;)
I dunno how we handle the backwards compatibility of old hooks? Is there a decision anywhere in area51
Use one of subject and title?
I wanted to make it clear that it isn't a phpBB variable. I can change it if you want, though.
Neither the event name nor parameters make any sense at all. "Modifies the guest values for later use in the template" is not helping.
Ai dreptate, nu merge. Credeam că se iau primii din top pe care se poate pune vraja. Revin mâine cu o rezolvare.
Since we are expanding setup.cfg, lets replace this block with a 'zmq_prefix' entry in the global section of setup.cfg.
To fix the Windows errors, at least with VC10, add `typedef unsigned __int64 uint64_t;`. However shouldn't this really be in core? Look at `core\types_c.h`, as `uint64` is already defined here. Consider using `uint64` etc. in your code, moving the necessary bits of this set of typedefs there.
Missing Given/When/Then sections of the test and all opening braces (`{`) should be on a new line and not the same line as the statement
Why are you deleting this file? For now we want to keep library files intact.
Don't need to add this, just use Util.dateTime
@johnstoecker, one-to-one node wasn't working for me, so I used this one. I understand that because the liquibase file forces a unique index in this column, then there will be no duplicates in the DB. But I don't know if there are any issues that this can produce with Hibernate.
OK ... I've had another stab at it. This time with a test aimed at checking for failure when `len(target_dims) != len(unique_dims)`.
Remove the extra ``Scipy``.
Add ``s`` to ``fail``.
Minor nit. I might have kept it a String and done the conversion here in one place rather than many.
I guess this can be removed.
Should we silently ignore this?
return '.'.join(normalize_name(n) for n in (owner.first_name, owner.last_name))
create_warehouse
Right, but that feels like an artifact of HTTP, not a design decision. Besides, it may become moot when/if the internals of `riak_object` are changed.
but this is just a table rename... what am I missing?
Lars, we tested this branch with an exposure of ~ 43k assets, the biggest one Vitor was able to provide at the moment. The optimal run-time for Vitor was 10 minutes. After some optimizations, we found out having the best performance when setting up the number of tasks equal to the number of cores (4 in our test).

4 tasks, 11k assets per task, 5.5 minutes
43 tasks, 1k assets per task,  16 minutes

The bottleneck seems, currently, the query to load the assets data. IMHO, in risk a big task doesn't necessary mean a bad performance (risk is I/O intensive). I think it depends on: 1) how we load the data 2) the type of cluster we run this code on.
If the `imt` is SA, does this string include the spectral acceleration period? Since the `imt` is used to query `hzrdr.gmf`, you should include an optional parameter for `sa_period`, and probably `sa_damping` as well. `sa_damping` can have a default value of `5.0`.
remove dir to keep the folder clean.
Just because it's not the source, doesn't mean it shouldn't follow the style guide LN1-8
Yeah, it ended up being unnecessary. \*shrug\*
Testing sequence does not match the one given in the instruction.
Should provide comment to indicate about this.
In this testing scenario, changing the order does not matter. However, in others, it may.
Changed. Cool, that's really what i want. Next time, I will use/search the existing operators first. Thanks.
We don't need this.  Such interpretation should be done in the specific application context.
No, small x is not a random variable. It is just some value x. Please remove that description again.
As far as I can tell, `Gst.init(['--gst-help'])` doesn't print GStreamer help and exit the application. Thus I'm tempted to just do `Gst.init(None)` and declare that GStreamer debug settings must be passed to GStreamer through environment variables.

Though, this may be changed with GStreamer 0.11/1.0, so I'll leave the decision of whether to keep support for `--help-gst` and `--gst-*`  open until the rest of this branch works.
Should then be:

```javascript
var check = function check(){}
```

:)
Code de debug ...
I have tried to implement Z & M, but i have broken the class.
i think the original class not work as expected with Ewkt, but i have not tested.
Out of curiosity, why not just build this functionality (calling bodyProducer.startProducing) and some tests into the mock agent? the request is always a post, and the headers are always empty. The only difference is the URL the tracer is initialized with.
I was planning on getting rid of the autoloads, hadn't gotten to it yet
LOL.  Actually, you sort of did - I just removed some lines and added an "abstract". :)  But fixed.  Removed actually.
It would have to be CharsetNames, since there is no method String.getBytes(Charset) until Gingerbread. I don't think it's worth the extra code, but I can be convinced.
We can sleep another time. But too small time produces a big processors load, too big time reduces responsiveness
跪求PNG格式（请传imgur.com，完稿我们统一下载到本地）……GIF难入TeX大门。
Oops, sorry, I missed that one.  To make this function, I copied and pasted chooseDataDir() from zotero.js and then deleted out all of the code that wasn't necessary (I think) for this function.  I missed that deleting all of that out made the while loop unnecessary.  I will remove it in the next commit.
For the record, I like:

```coffeescript
@foo
f.apply(this, arguments)
return this
Klass::method
@::method
doSomethingWith Klass.prototype
```

I do not like:

```coffeescript
this.foo
f.apply(@, arguments)
return @
Klass.prototype.method
@prototype.method
doSomethingWith Klass::
```
def, probably !null on the others too
Nitpicks about code convention: There should be a space between `if` and `(`. And single quotes around 'browser'.
What was the reason we didn't have to do `addConstructor` before, but need to do it now?
Code cleanup? We don't need to wrap the event handler. I think we should change this to:

```javascript
function _projectOpen(event, projectRoot) {
    ...
}
...
$(ProjectManager).on("projectOpen", _projectOpen)
```
FileEntry and DirectoryEntry don't have constructors according to the spec, see #1416. We should just omit this line in the example.
Would not this always yield `folder`? you look for directory and file in the Regex.
That's a no-go!
these are commented out now to vs getting it all in one swoop vs. more elegantly getting task_names and hitting up the tasks one by one in individual requests.
What do you think following code?

```ruby
case val.downcase
when 'tcp'
  :tcp
...
```
i guess this class should not be there
It might be worth considering either having a more generic message or also checking to make sure they have an internet connection. I know that it's probably their fault if they don't have an internet connection, but I always love it if an app let's me know why my login didn't work.
useless cast from ```long``` to ```double```.
Definitely. Would you be interested in a PR that made more of rugged flexible about its inputs?
This 1s shave sponsored by @arrbee's `git_futils_filestamp`.
Heh, neat. At least we are on the same boat now :). 

Can you check https://github.com/mitar/PiplMesh/blob/master/piplmesh/account/views.py RegistrationView. Why did this got implemented together with https://github.com/mitar/PiplMesh/blob/master/piplmesh/account/forms.py RegistrationForm? Previously def registration_view(request): redirected request to RegistrationForm but is RegistrationForm still needed? 
All that is left is traces in form_class = forms.RegistrationForm (https://github.com/mitar/PiplMesh/blob/master/piplmesh/account/views.py) which seems redundant now. I do not know which form to fix to get it working with mongoengine again.
Nothing, I kept it because the original method had it and I thought it would be a nice reminder that it is there in the original method even if it does nothing.
We use console without checks elsewhere, why so cautious here?  If you really care, move this into shims.js so we can count on console and console.log/warn/error being callable.
Spacing before the first `+` here.
Yeah, we should confirm this.

I believe we should be good for this, though.
Why does this line get commented out?
Please indent the use one level more, so that it is distinguishable from the function body. The same below.
Oh, was this copied from web_steps maybe?  I think we can assume #should is available on any object in our tests.  It seems very unlikely we'll want to use something other than RSpec that doesn't implement #should.
Use of {} but no argument
module \"%V/%V\"
maybe do `props.match(/url\((['"])(.*)\1\)/)[2];`
so `$(MKLLIB)/libmkl_rt`, for instance
This change probably makes sense now since the update a few weeks ago to the initialization path to avoid implicitly changing AFNetworking configuration at init
Those will all be fixed when we FxCopify this thing.
This line appears a number of times, and it's conceivable that we may have a third `ColumnBase` subclass in the future.  Maybe factor out to a `get_column_class` method?  (Minor point, and that doesn't need to hold up the PR).
Did you mean `!=` here?
I'm moving away from the import()s. just prefix the functions with rt: and kill it :)
That's probably more flexible. Done.
What happens with `key"6` and `key>6`? (that's why I suggested prepending the hash)
why not rather using 

```php
protected $proxyClassTemplate = <<<PHPTEMPLATE

<?php

here your php code

PHPTEMPLATE;

$moreCode = ...;
```
@craigambrose offtopic - a question I've had for a while now about cucumber feature files: if your feature requires two separate people to interact with it (e.g. someone requests access to a group, admin approves access to a group), do you usually keep it as one feature file or split it off into two?
I know it's just one line, but now that you have events attached to lots of stuff, maybe an acts_as style mixin would be good. eg, acts_as_eventable or some such.
please also test the optional argument - you don't need to do a full assertEquals for the entire result for the second time around; just test that it returns a different kill.
We've been using 4 spaces for indentation.
We have a context manager for this: `IPython.utils.tempdir.TemporaryDirectory`.
Note that we cannot use `self.shell.run_line_magic` since we need to pass the namespaces parameter along.
Our production sharejs deployment accumulated 850 000 operations (800Mb) in its first 4 weeks of operation.

Operations before the most recent snapshot aren't useful to us, so we'd like to keep the DB size down.

We do keep all our snapshots, but I can imagine some use cases were they'd also be superfluous so adding the option for others didn't hurt.
This is the most horrendous `switch` I have ever seen.
@Jakobud Indeed, it's normally preferred - however this is a file derived from a Symfony file - in the rest of the Symfony Request object they use '||'.  Additionally, '&&' is also used in the same conditional - jumbling up '&&' with 'or' would be pretty weird.
Thanks @racklin for this, this addition was very useful :-) One thing that I noticed was that it trips up when you are trying to select multiple postgres schemas, since the single quote wrapping the desired schema tells pgsql to treat it as one. For the case of multiple schemas, it may be better to make $config['scema'] an array in the config file, unless you see another way of doing it?
It is very odd to me that this is required, although reading over your question on Stack Overflow, it does seem to be correct. Have you looked at the parallel code in core git? I'm curious to see if they have a similar fallback after calling fnmatch.
you can remove @link.
Just noticed that toAST is the word toast.. Cool.
We did, you are right. I typoed the typo.
For all these checks, I think you can just go "truthy" and do:

    if num_answers or num_questions or num_solutions

These numbers are counts, so they'll never be negative and anything > 0 is truthy. Not a big deal, but thought I'd mention it.
Yeah, the only reason I didn't change it is because there will be a period of time where the data in the db won't match. But I think I can either land this close to when the cron is going to run or maybe run the cron in a migration?
Totally fine by me. :)
Looks good, but can we file a follow up to get on [django-sha2](https://github.com/fwenzel/django-sha2) to pick up the bcrypt/hmac stuff?

Also, contrib.auth.models.User has a [set_unusable_password](https://github.com/django/django/blob/stable/1.4.x/django/contrib/auth/models.py#L306) method we might want to migrate to. It could simplify a lot of this.
This is not necessary if you're not adding any extra options.
The label should be aPoints.toString(), so it is only the number.
Sure, I'll do so
… where, in terms of the ontology, plain = true means PlainMapping, and plain = false means TheoroidalMapping, I guess.
This needs to be checked to make sure its not sending them to an external website.
Can you move this method after the `__construct` one?
Better would be check for `function_exists()` here and below. As extension can be loaded be no enabled.