labofsahil/github-event-dataset-2012 huggingface dataset:
here we go :)

sorry bout the other one man :P
A few additions

Some a few years old
Change CacheWarmer priority to avoid problem with other warmers

A small PR to change the CacheWarmer priority so it is called before others like JmsDiExtraBundle (see issue #190).

To do that the warmer is not optional anymore, the class loader is called just after the generation and the attribute priority is added to the service definition.
I18N and french L10N

Cause stakeholder have to check progress of feature implementation in their native language.
French done, ready for the other.

Needed to extand a little display zones because fonctionnalité & étiquettes are much longer as feature & tag

Bonus: transparent png icon for menu
Change request in create_proposal method of curriculum.rb to use dynamic...

... variable for collaborator
Jason, this change causes the first column in a grid with actions to be displayed valigned to the top.
@MartijnKaijser: Mind sending a pull for the wording? I'd like to have some input from the others on this.
Makes sense, done!
Alternatively, you can just call 'add()' which raises an exception when the queue fails to add something.
Thanks for your tips! The way you suggest is much better! I tried to implement it

https://github.com/tomodo/kaya/commit/219c8060688e5c2d14de9bc9b8edda8ceacd88d0

is acceptable?
I think @tfotis will need to address that ;-)
yeah, it would be more efficient
Hmm. But we could load the combined file at the end of the page (which would be good practice anyway). Additional requests will never be as fast as one bigger combined file.
Was it intended to remove any mention to horizontal reuse from this file ?
I'd rather go the other way, but it's tricky.  For now there are two metatables.
Dear Spastorino, thank you for fast reply!

The link you provide shows only one explanation: "vendor/plugins/* were on their way out with Rails 3.0, moving to gems instead. This is their last hurrah."

So actually we still have unanswered question - why plugins are moving out from Rails?
tu asi chcelo byť `(int)`...
To make the refcount tests pass. Eventually we should probably use a different way to implement the test, e.g. implement uv_refcount() and check that number directly, instead of relying on the loop falling out.
They really mishandled this API change. I mean, when you deprecate something you at least gotta try to keep the old version working and keep it around for a release or two before removing it.
In that case I'd leave it in AppMessenger (in the hope that someone else will use the code) and just start the job in OpticalMount rather than setting up the threadmessag.e
Has this been merged already to jruby?
Great. Fine so far, to me. :)
nice to see that such a small change causes so much traffic ;)
Whenever someone have the time for it, update copyright to 2012 :<
I consistently get `70f4d92d5afe67a03e6bb1fb52a8a0d3` as the md5 hash.
Ahh yeah, looks great.
Ah, I didn't see anything using _TaskProcessor, so I thought this was supposed to be a subclass.  I see the processor setups now, though.
No, it's a nice way to comment and uncomment code.
If you have a block surrounded in this way, it's a single character change to flip the 'commented' state.

    //*
    liveCode();
    //*/

vs

    /*
    deadCode();
    //*/
I've no frank opinion on the follow code, yet.

About the destination sending, well, the both servers are sending back the destination to all other clients including the sender, and also including the current position of the being. (On every packet for tA, and IIRC on packet on four based on world ticks modulos for Manaserv)
This makes it quite good looking with only a few resyncs necessary, IMO.

Now, about how often and at what level we should send the messages to the server is yet to be tested, and I gladly take nay help on that.
More on it when I've looked at it more carefully.
Agree with you. Let me try to modify it.
I merged and modified it. Please check.
Because its a huge comment. Ideally the code is simple enough that one doesn't need comments within the code itself.
Descriptive comment, please!
Good point, I just pushed a patch to do it.
in this case pathfinding used only with unit target (destination) and only for check availability.
Whoops, not sure how that deleted line on top got in, used the github text editor to do this, my bad.
No need for output buffer, the_search_query is just a wrapper for get_search_query. http://codex.wordpress.org/Function_Reference/get_search_query Different filter names, but we dont filter it anyway.
UPDATE `world`.`item_template` SET

need change to

UPDATE `item_template` SET
This is in v6r2 prereleases. Since the v6r2 is delayed, I have to put it
into the v6r1 patch as well. Will do it on Monday.

  Andrei

On Fri, Feb 10, 2012 at 6:41 PM, Philippe Charpentier <
reply@reply.github.com
> wrote:

> Is this pull request in any release? It is urgent as users cannot follow
> the instructions for deleting their files...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DIRACGrid/DIRAC/commit/2970b9a9b52f56544a816f5e409a0eac3385f923#commitcomment-954538
>
Those little local changes... We all have them at some point :)
Substr is working with character ranges now, so this was broken with byte lengths.  While I'm thinking about it I'll change this use a byte slice, instead.

The real slowness is likely to come from all the extra copying that str::replace does, and the fact that str::find doesn't use Boyer-Moore, though.
есть смысл сюда вносить хакфикс https://github.com/bwsrv/mangos/commit/8f470f16eff39dc8d7d9085814be704c8bb48bb1 ?
у нас сейчас нету бонусов на эти спелы, и через базу они не работают если прописать.
statistic collected continuously (while StatisticEnable) but logged after Statistic.Interval. logged count of thread in Statistic.Threads by average load values (descending)
Don't fix issue with high cpu load after some hours uptime. http://img31.imageshack.us/img31/2223/issuecpuloadaftersomeho.png
@JonFerraiolo, I think @jhpedemonte was asking about `title` attribute in the OAM schema, not the HTML attribute.

Yes, `title` is used for the caption in the properties view. I had been following precedent to include a title, but per @JonFerraiolo 's comment in issue #398 that we should just be using the raw attribute name to the user, I'll just remove all `title`'s in the OAM files for the HTML widgets.
Looks like this part of the docs is now outdated, right? https://github.com/neo4j/community/blob/master/server/src/docs/dev/rest-api/introduction.txt#L7
I told him already...:p
This is the version that currently works, with issues naturally. Everything after this is the next version, which at this point is still not passing unit tests.
That works! On it..
This test is not correct, and is failing on the build server under "Configuration 1.3.0,OpenJDK 1.6". It's never correct to assert that something has been garbage collected, because garbage collection is never guaranteed to be performed. The JVM claims to make a best effort, but especially in a scenario like this where there is tons of memory still available it's free to refrain from collection. I'd like to remove this test unless @Chouser has a strong objection. Ideally some similar test could be added back in, but I don't know any way to introspect reachability of objects on the JVM.
What if the particular file wasn't found? Shouldn't there be some error checking here?
Criação de novo arquivo de configuração caso esta seja a primeira execução do programa (por base no exception)
Oops :-/
I must have had my warning flags on the "prod" setting when I did my tests. Will fix it right away.
http://nodejs.org/api/globals.html#globals_filename

however, self-modifying scripts seem really whack (maybe I need to read my sicp) - imo the header text should be extracted
was thinking of `CameraRig` then I thought of `THREE.CameraProp`, like a property in a stage set?
Fine :-)  Done.
May be further down, but can you remove /dialog/controllers/set_password.js?  I removed it at some point yesterday because it is no longer used, not sure whether that was before or after I merged to your branch.
Whoops!  Thanks!
My bad. Reverted.
What about:
if (m_navMesh->getTileAt(tx, ty))
return (m_navMesh->getTileAt(tx, ty));
And is it really in this function? According to crash logs?

If i put it in "if" it will check null and 0 in the same time right?
extra whitespace before !=
For the boolean checks you can do both, which is still faster than using array_key_exists. 

example:

(!empty($attributes['AUTO_INCREMENT']) && $attributes['AUTO_INCREMENT'] === TRUE


:-)
s/valut/value/
This is awesome.  Not this commit, but this whole project.
A test needs to restore everything as it was. This test starts server and leaves it running. This might cause problems for other tests.
Also, you need to place the new sql tables into the sql/database.sql file. Simply follow the format in that file (place the tables alphabetically and not at the end)
full_view_uri is the image. Naming are an later issue. Lets see if works in the SIP-C at all first, then we can discuss labels.
-

Thomas Wikman
Delving B.V., Skyttorp, Sweden
http://www.delving.eu
mobile: +46 706 74 44 09  
skype: wikman_t




On Feb 20, 2012, at 8:46 AM, Manuel Bernhardt wrote:

> I think landing_page might be a duplicate of full_view_uri below
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/delving/sip-creator/commit/40390d555ecba471ec2f20ebda1c78f11367327c#commitcomment-983798
Yeah, they were deleted in a following commit after I realized I committed them
Sure. Just I couldn't find how to grab boot logs yet :( Also I use autostart.sh for my IR remote. Maybe they are not getting well :) Could you little bit explanation grabbing boot logs? On the web site they said how to get DEBUG console from boot but It's not working for me. Or I am missing something.
is it correct to use the min of 1 and the integer ? This seems weird to me as it means that only 0 and 1 can be used (except if a  negative thickness works)
Ainda não. Digo, até roda, mas os shortcuts do teclado não rodam.
That should be fixed now. Isn't it? Or isn't the mentioning prominent enought?
Ad. 1. Sure, no need to store guids here.
Ad. 2. You mean std::vector for Player pointers?

Yeah, that empty check is not needed ;)
I'm not sure; I'd rather leave it for a 1.9 release.
то есть в скине отслеживать каким образом? я же не знаю когда там затухание импульса было, передавать ссылку туда на физику/на гейм объект?
Ah, found the "proper" behavior.  

> The .replaceWith() method, like most jQuery methods, returns the jQuery object so that other methods can be chained onto it. However, it must be noted that the original jQuery object is returned. This object refers to the element that has been removed from the DOM, not the new element that has replaced it.


However, http://bugs.jquery.com/ticket/7246 notes the discrepancy, but there is no feedback from the developers on whether the inverse behavior on nodes disconnected from the DOM is intended.
It can absolutely NOT be zero; storing zero in the database violates referential integrity.
Hi Tom, talked this over a bit with Gavin and we think this error should have a more specific name, like UnenumeratedTypeError for example, which was Gavin's idea while my suggestion was NonExhaustiveSwitchError. But TypeSystemError just gives the wrong impression, it's not an error of the type system really.
Sure, [how's this](https://github.com/documentcloud/backbone/commit/ef9d40aa82d452459caab3d20ead4bc5967636c5)?
NO U.
I also like to move these predicates with comments into their own self-documenting method names.

E.g. 
isPassedLeftOfScreen
isPassedRightOfScreen
I think it would clearer/cleaner, if you called DFSAction::commit() here, unless some reason state = SUCCESS has to be set for updateParentFileAttr ?
There is debuff http://old.wowhead.com/spell=36444 which should damage "all mechanical units", isn't it?
I forgot to mention that now Graveyards store the spirits's guid not a pointer to them. I think this is safer
Eclipse noticed me that "the import sun cannot be resolved".

And it seems unless, remove it temporarily. If needed,  add it again.
I'll look into this tonight.
Looking forward to feedback!
nub, it's calculated at compile time :<
-31498, -- Azure Mage Slayer
+-31498 -- Azure Mage Slayer

fix
hang on. looks like my IDE is borked
I should have read this before wrapping all my tracking code in if statements :-)
Ok Ok, is this enough to quench your hunger?
@amueller is raising a good point in this commit. 

One thing that worries me is that, with the introduction of scale_C, SVC, LinearSVC and LogisticRegression are now optimized for *cross-validation* and default values now don't work well because they are much too small after scaling. Sure, one should theoritically always tune parameters by cross-validation but practitioners often don't bother and for some models like logistic regression, the default choice of C=1.0 is hard to beat.
yup, i'll work on a plugin to see if that way will work - thanks dudes. there's only _enableRemoteInspector couldn't find a way to disable it after
Line: 154 - Added padding to left and right of a paragraph.
I'm sure you thought about this, but the approach here seems like a bit of a bastardization of the Settings API - adding a field that isn't actually a setting but rather simply text. You don't feel any qualms about this?
campos `created_at` están mal, ademas tengo mas trabajos como este(plagio)
> Jenkins is a blessing. Thanks @ogrisel.

Indeed. It's so nice to have it.
Unused import. You should check out https://github.com/jbalogh/check if you haven't already.
@michaelficarra When I tried using `00` as the strict error, Node's `Error` message would "leak" out:

```
.:4
  return 00;
         ^^

.:7
  00;
  ^^
<snip>
``` 

Sorry I can't explain that any better -- change `arguments.callee` to `00` to see it in action.
added in 17016273d5d3dcd4e169c75c3e94f7e8b08b4320
@woglinde: i'm going to release it....
I now make sure the callback passed to `ConnectionPool.shutdown()` is always called. If the pool is already in the process of being shut down, the callback will receive an error as the first parameter.
Yeah let me know and I'll merge or pull from it (not sure what the right term is here). A lot of people use this repo though it's out of date.
LOL, didnt ask u to revert it, wait ill submit a patch with that softkeybug fixes
Most of them not require js calls (onbody hooks) please see this issue.
https://github.com/seyhunak/twitter-bootstrap-rails/issues/77
With twitter-bootstrap-rails 2.0
```css
@import "bootstrap/bootstrap.less";
```
Argh, I removed the check because I wanted to test something, and forgot to put it back. Im sure you can find the 1024 check and replace it with 1600 :)

Although that check is kind of weird. It will not resize a 1920x1200 image to 1600x... Is that meant to be that way?
"em grupo" dentro de aspas foi perfeito! pq nem pra comentar ou dar ideias
o "grupo" ta disposto, eu comentava as paradas pra ter algum feedback, mas
como o unico q as vezes comentava era o @douglascamata, to deixando de
comentar coisas q acho pequenas e de certa forma minhas, pois ao add o
formtastic no influenciou o projeto como um todo, pois ele trabalha em
conjunto com simple_form de boa.

Com relao a discutir outras coisas que realmente influenciam o projeto
como todo,  uma issue foi criada para isso, mais uma vez, s eu e
@douglascamata comentamos o mesmo. Como vi q isso  pouco, resolvi mandar
um email para todos, pra ver se assim alguem responde

Em 29 de fevereiro de 2012 21:33, Thawan Fidelis <
reply@reply.github.com
> escreveu:

> S acho interessante como decises que influenciam todo o projeto so
> tomadas isoladamente num trabalho "em grupo".
> ps: No estou falando apenas do formtastic, que talvez possa ser utilizado
> junto com o simple_form.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/douglascamata/Rick-Rocks-Academics/commit/2051dac6f31a7ab08c1a971e3de99f092a05f7c5#commitcomment-1028878
>



-- 
Pedro Henrique Passalini Soares
Graduando em Cincia da Computao e Informatica - UENF
I really need to find the time to resurrect ci.phpunit.de ...
any reason for the change from switch to if/elseif?
Why not inline this with line 230?
seems like xmlhttp will be created each time you make request. can you optimize it?
This change calls the function to start the preview in situations where the preview was already started. This will just loop right back to where it started and the preview is still not enabled. After multiple tests, it is safe to say this change had no effect.
And if not: force it. Absolutely  valid and necessary if you're missing it. Period.
Yeah, I don't understand that change, either. Please clarify.
You also have to add a cross link from both: 
* http://doc.akka.io/docs/akka/2.0.2/general/supervision.html and 
* http://doc.akka.io/docs/akka/2.0.2/scala/fault-tolerance.html and 
* http://doc.akka.io/docs/akka/2.0.2/java/fault-tolerance.html
And in this cross link also add that the prefixes/scopes (/user etc.) are explained there.
I don’t think it should ever.
The import for sun.misc is marked optional for now to allow for resolution of the bundle in most OSGi environment.  In my example, the package didn't get resolved and my mini-example still worked fine, but if it is really required, the only solution is probably to document that this needs to get configured properly in the OSGi framework (either using boot delegation or as a system bundle export) but that would probably also mean that Akka only works on Sun JREs, I guess?
same comment here as above.
Patrik needs them. I am happy to remove them if he agrees.
Is the semicolon (`;`) here for style?
> but not sure about the latter because the file would quickly grow to insane sizes.

Can we just keep the last run?
Ya, I still kinda dislike it but it made for a really nice hook to do this, so I hate it less.  :)
toDeleteCount would be more clear
should be > 1 not >= 1. Change the music one too even if it's wrong in the place you copied it from.
logged_hours work?
~Probably fixed.
then put them together and add a comment it's for performance
what should i do if the path exists but is not a directory, just delete it?
ditto.... inversed
Make all of the `character varying` columns `text`, Postgres doesn't care about the length.
These properties aren't even used in `TUITableViewCell` itself, only in the example app. I think they should be removed.
Alright, it's `nil` now :]
This still needs to be in a PR.
[Engrish](http://en.wikipedia.org/wiki/Engrish).
only "Nette\..."
i don't know why this looks like the spacing is wrong but it's fine in my editor....
Shouldn't we keep `updateField` as alias for `valueField`, to ensure backwards compatibility?
This needs to be moved into moxie/__init__.py before it can be merged. The factory method in there should create an isolated app object, as the code is now all app's created from the factory will use the same cache instance.
I think you can squash the commits using this:

```
git checkout symbolize_keys_recursively
git rebase -i RAILS_REMOTE/master
# squash the commits
git push -f YOUR_REMOTE symbolize_keys_recursively
```
ooh - didn't know that flatten takes a depth param, I'm sure that'll come in handy at some point.
I don't think we need to iterate the orders twice here, probably one iteration would solve it?
I'm not sure how to fix this one, but this it is not the best way. But this occurs because some of these files require this one I think.
What do you think about:


    def env
      @_env ||= begin
        ENV["RAILS_ENV"] ||= ENV["RACK_ENV"] || "develoment"
       ActiveSupport::StringInquirer.new(ENV["RAILS_ENV"])
      end
    end
So gravatar can't load the default fallback image when it's on localhost so this is just a fallback, fallback.

On 2012-10-01, at 7:42 PM, Mike Cooper <notifications@github.com> wrote:

> In standup/app.py:
> 
> >      m = hashlib.md5(email.lower())
> >      hash = m.hexdigest()
> > -    return 'http://www.gravatar.com/avatar/' + hash
> > +    url = 'http://www.gravatar.com/avatar/' + hash
> > +
> > +    if settings.DEBUG:
> > +        url += '?d=mm'
> > +    else:
> > +        url += '?d=%s%s' % settings.SITE_URL, url_for('static', filename='img/default-avatar.png')
> Also, why do we need to use a different default (mm) when debug is true? I think our default would work just fine (if it was url encoded).
> 
> —
> Reply to this email directly or view it on GitHub.
>
Yes, I thought it's still the official way. Good defaults are important, so do you suggest to place it somewhere else? 

I like the `quick` location because then I can use the `quick/bin/scalac` runner script to immediately test my compiler.
indentation oops.
Still, the invariant should be documented and tested. And on second thought, there's a bug already:
scala> Duration.Inf.fromNow
res0: scala.concurrent.util.Deadline = Deadline(Duration.Inf)

scala> Duration.Inf.fromNow.isOverdue
java.lang.IllegalArgumentException: toNanos not allowed on infinite Durations
(...)
Default is 'lowpass'.
what if > 100?
Possible names: quicklook, quickplot, display
Can you add some comments to this explaining some of it?
deep in double ` rather than '.
Agreed that `self._new` is better, to maintain various classes of matrices in common cases.

And, @mrocklin, btw, possibly, that standard method name-token for this method in sympy core is
`_from_args`, `_new_rawargs`, although it defined only for the operations.
See (https://github.com/sympy/sympy/blob/master/sympy/core/operations.py#L61)

Or may be another analogous methods exists in the core, Does they?

@asmeurer can clarify more about it (or tell who can).
Gilbert wrote all the original documentation with "mass center" instead of "center of mass". There are about 50 instances of sentences with this. I assume a simple search and replace would suffice, but I guess it is possible that it the sentence could read incorrectly too.

And then what about the attribute name? do you suggest changing the attribute to:

"center_of_mass" or "centerofmass"
Yeah that's what I meant. Maybe it isn't necessary, but it sounds like if ring support were ever added that it would be a separate class.
Scratch that...nmag is being calculated to see if any significance was obtained (if not, nmag._prec == 1) so this should be fine.
Also, I should change this to ... pairs=False according to PEP8
Странным мне показалось разделение учётных записей на бесплатные и на те, с которых взимают плату. Если я правильно понимаю, существует единственная разновидность учётных записей, а стоимость зависит от подключенных услуг.

Как бы то ни было, я, пожалуй, напрасно придираюсь к формулировкам.
Перечитал ещё раз и решил, что все варианты приемлемы :)
I figured that, but I needed those lines to build and test the package.  So you run roxygen2 in place and overwrite files?
We could do some additional data validation on these parameters. Checking for acceptable ranges of values, length, legal chars, etc.
It's not clear what you mean by "single word tags only" in this context.  I would suggest you drop it from here if it's too hard to explain.

I wonder also if this indicates a problem with the implementation:  Why are non-single word tags second-class citizens?  Can we fix this?
I think this should print a visible message before running the original method if catbox is not installed. I'll change it to do so.
`sample-data` doesn't seem very descriptive. Is there a better name?
In the event of malicious input, condition will be nil and that is fine.
change the whole if to one line:
FileUtils.mkdir_p path unless File.directory?(path)

delete this function if you use it only once.
Nice little bug in the tests that was flushed out by this :-)
Because it needs more data than we've got - specifically, the detection date.

On 12/03/2012, at 18:11, Ben Hood
<reply@reply.github.com>
wrote:

>> @@ -149,6 +149,9 @@ class HibernateDomainDifferenceStore(val sessionFactory:SessionFactory, val cach
>>                        "entity_id"   -> event.id)
>>       executeUpdate(s, "removeDiffsByEntityId", params)
>>       executeUpdate(s, "removePendingDiffsByEntityId", params)
>> +
>> +        // TODO: Update cache for removed event region
>> +      //updateZoomCache(pair, event.detectedAt)
>
> Why is this commented out?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lshift/diffa/pull/60/files#r544341
Could just use pairRef?
these fields should all use 

the .. data:: directive, so that they're indexed.

We can make a new index object if that makes more sense.
This is fine, but to be fair, it used to be this way and then we made a point of inlining cross references as much as possible, in an attempt to increase flow and make the reading experience less choppy.
We could do that. I'm just trying to be honest and clear that it is indeed Underscore's .has method. Moving it to TraceKit itself would be cleaner, but it would be nice not to have to modify the PR if possible
I'd definitely prefer to not send the node data back to the parent process, as the way node data is serialized/stored uses lots of heap and will add to the memory requirements for chef-client, which I think is one of the motivations for doing the chef run in a disposable process.

If you have code that needs to access the node, is there a way it could be made to run in the child process?
Instead of which aspect of the additions?
We had foo_row earlier, now foorow. Pick one.
Following would probably be better:

```php
$log_data_ary = $log_data_ary ?: array();
```
Hmmm, oh you're right. Thanks for pointing it out.
:)
I changed to  $is_greater_ie7
gotcha ;).
This is here for the same reason as I mentioned about $config. One of the functions that is called by a method requires $user to be defined.
How is this related to includejs?
Needs new lines after the break statements.
When you add more tests requiring an "other" user, you'd need to constantly create and destroy said user.

I don't know what the correct way of handling this is in our functional tests but consider creating the user once and reusing it. In other words, make the "other" user more of a fixture and less of something created/destroyed per test.
/RACKSPCE/
hmm. Matt, I'm confused... the propagation already goes from (from, to, events)
The code bellow async block is kinda odd. If functions inside async block would be asynchronous, native.run would be called at the same time as functions in the async block.

You should extend async block and put stuff which is currently outside async block inside it.
nu mi-e clar - vrei sa folosesti ' ' pentru siruri care nu reprezinta text cu sens si " " pentru cuvinte/mesaje? E ok, dar daca alegi asta, respecta peste tot.
Maybe it would look batter if you declare a pointer to Light:
```cpp
    Light *light = *i;
    if (light->valid && light->raw) {
        /// ...
    }
```
The `more` really smells like API creep on this adapter. Are you guys locked into this `render()` interface?
попробую разобраться почему резалт только от первого идет
In OpenCV we do not put our emails to the code. Actually you shouldn't modify standard header at all. All information about your commitment will remain in git commits' info. So, I would ask you to remove information about your names from *all* file headers.
These tests will fail once we are putting in actual data -- instead in your test, you should get the first drugname and test if it equals the first cell in the grid
```PPDataProxy.__ne__()``` is the **last** method in the class, and the following line is the module level function ```iris.fileformats.pp._read_data()```. 

Therefore there are **two** blank lines after it. Isn't this the normal practice?
Don't you need to also define ``expected_gamma`` in the ``else`` block?
Fighting the warnings in HSQL is like fighting death and/or taxes, but I admire the effort.
That sort of thing. Whatever checks the edge conditions. Plus maybe backslash at the end of the line?
also, a check for existence of devDependencies. some packages ain't have it
Erre a változtatásra biztosan szükség van?
If we add an interface, I don't think it should typehint ``\Closure`` but allow any callable instead (without typehinting them as it is only possible on 5.4). I don't see why other callables should be forbidden.
Removing teh typehint is possible in the Connection class, but it should be done before adding an interface as changing the typehint in an interface is a BC break.

@beberlei thoughts ?
I have no idea! Will look sometimes.
This field (`mean_hazard_curves`) is a nullable boolean field. It is not sufficient to check if it is `not None` to decided whether or not we compute mean curves.

I would change it to this: `return self.mean_hazard_curves is True`.
Prefer (require droplet.lattice :refer [join etc]) . Makes it clearer where things come from and prevents accidentally overriding something if I later add new functions to droplet or droplet.lattice
instead of prepending to `body`, you should prepend to `#qunit-fixture`.
In the same vein, the `array_agg(featureprobs.cnt)` would also require a cast to `BIGINT[]` in case we get a 'cnt' column that is not `BIGINT`
That's an explanation for the jump array.  My question is about lmc_fval definition.
I'll test on my 10.6 box Monday to see if this still fails with older llvm builds, in which case `fails_with_llvm` (or `fails_with :llvm`) should be retained.
I'm curious why, as you noted, this doesn't work as `(var/destination).mkpath`. Presumably it made its way into the mysql family of formulae for a reason.
Given the infinite loop while using xcode tools under those circumstances, it's probably better to still raise here rather than return an empty string.
I think let's create a base exception class (e.g. `MiddlewareControlFlag`) and then `StopPropagation` as a sub-class of that. Raising that here should result in the errback of the deferred being called and then we'll need to handle that appropriately in `vumi.middleware.base`.  Then we can add other exceptions later for other flow control changes (e.g. `SendReply`?).
Do we perhaps want a `dispatch_event` helper method as well for `TransportTestCase`?
yup.. redundant.. c&p ;)
I'm adding the CONST but I really have no clue about rubinius release cycles... I'll ask and see ;)
I don't like the name `wire`. The problem with `wire` is that it implies that it's manipulating application objects. It isn't. It's just building a graph that describes application objects.
To make it simpler, I'll change the format to be in past simple tense, so format would always be like  "%1 joned the group" instead of  "%1 has joned the group" , so as to prevent incorrect cases like "You has joined the group"
I have a little concern about this one. My main concern here is to try to avoid OS specifics where we can, with Windows support in mind.

Would it be possible to slightly change this to use rimraf & fstream-ignore. I'm not sure they are designed in a way we can use them here, but our clean / copy task and helpers might be used here.
Took the command from the comment, I mostly just cared about the unnecessary whitespace before the ;
Sysouts! Please remove :)
I will open a discussion in junit@yahoogroups, and already added @aslakknutsen @matheeeny and @stefanbirkner to this pull.
Tell me what is incomplete in that javadoc. Text issues can be fixed, but if something is missing then i should be told. 
You know, the previous javadoc was long, and writing javadoc for such feature cannot be brief.

We will write release notes, so there we may concentrate on details as well.
> We could make this more bulletproof by also checking that collection.respond_to?(:each) so we are sure it can be converted to an array.

What should we do if it can't be converted to an array?  Unless we plan on handling this case, having `.to_a` fail seems like the right behaviour.

> We also might have to do this before line 26.

Good point.  Actually Enumerable doesn't implement the `[]` method, so maybe the `collection = collection.to_a` should be called unconditionally?  For an actual array `to_a` would just return `self`.
Yep. Since the normalizer does some regex matching on the media type, I downcased everything for the sake of simplicity.
If there is another link on a page that happens to be the previous url in the history, and the user clicks on that, it will do the 'back' animation and pop it off the stack? I don't think this can happen in apps generated from a sitemap.yml but with custom routes it could.
Nice catch on the unneeded dependency
I don't see any remote debugging functions that take offsets and return a location in the stylesheet. This is something we need to do ourselves.

I ended up adding your first suggestion of keeping a promise around and handling the cursor activity in the `done()` handler if needed.
I changed the code to register new Commands instead of using existing ones.

I tried adding this:

    var expectedKeymap = keymap([
        {"Ctrl-Alt-0": "custom.command1"}
    ]);
    expect(KeyBindingManager.getKeymap()).toEqual(expectedKeymap);

Is that about right? I get: TypeError: Cannot read property 'map' of null
&lt;style&gt; tags are known as "embedded" style sheets. "Inline" styles are specified in the style attribute of html tags (and is not a style *sheet*). So, the name of this function should be changed to something like: addEmbeddedStyleSheet().
I think this code isn't serving any real purpose anymore.  And it's actually causing a bug: you can only have one colorpicker open at a time (across all files) -- this is because when you use an object as a map, the keys are always forced to be strings, and all the bookmark objects convert to the _same_ string key ("[object Object]" or something to that effect).

I think we should just remove the _colorPickers map and all the code that references it.
I don't think line was supposed to be removed.
This is a bit confusing, and a checkbox implies persistence. How about "Reveal Active File in Project Tree"
I dunno, the source code itself doesn't have any indications of a license. Is it so important?
copy.copy(datespan) used instead.
You can use our SerializableFunction to make the function pickleable
This is equivalent to `doc_dict.get('actions') is not None` which is _not_ equivalent to `not doc_dict.has_key('actions')`. I suspect you mean the latter, as what you have right now won't delete `doc_dict['actions']` if `doc_dict['actions']` is set to `None`.
Yeah you're right. You don't need these tests if you write the sessions functional test. I've unit tested Locations.distance and a few actions on the trips controller just now, but there's way more coverage we can get. Sadly, my stuff comprises the whole backend service, so it's slow moving right now.
Show all rows ? Isn't it a "clear" operation ?
Yep, but exec is slower.

Speaking of slower, I will push a commit that does the `find` par only once :-)
Should we have a comment here explaining why there are 2 password properties (and also because there is a comment already for register_password, above)?
Instead of doing math to figure this stuff out, could you use Date's functions? https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Date#Date_instances ? `getUTC*` should give you similar results. Not sure if that's more/less confusing than what you have here.
This worries me a bit.  Are we sure we want to publish an API key?
This empty callback is fine, but don't leave it open (empty space) like that, be clear that it's meant to be an empty func.
Seriously? Please file a new bug for this if there isn't one already. 0.5.2. This is ridiculous.
That would be true of the minimum of the set. The infimum can be a limit point.
you're only using this for timers, not recordings. recordings don't really care whether directories are supported or not (it's read-only in xbmc).
so please rename is to bTimersSupportDirectories and update the comment.
I think this is resolved with the IOSHandle changes, assuming you like the revised close stuff I sent yesterday?
Is this meant to be commented out?  It looks a bit temporary.  If it's permanent then just delete the line - it's in source control if we need to get it back.
Couldn't and/or shouldn't you free the resource here if all the data has been read (we don't support seek anyway)?
module_name -> module
Reuse preprocessing here?
@Oemera you have done something strange with the encoding of this file. Can you remove it from the pull request (since it's just an overwrite with some new encoding or similar)?

When this is done I can merge the pull request.
Sorry, that was just a comment I used to test the replace functionality (It just mean: "Hi Marco, how are you"). Removed in last commit.
I'm curious about the addition of the 'get' for the destroy action.  Is this intended?  In what scenario are we using the 'get' vs the existing 'post'?
We should move this to the model layer instead of the controller
Maybe add a helper for creating this callback logic on the Json class.

Json.CreateJsonPCallback
Remember to add the background-size :D
Is this one ever used?
Bounded date is not meant to be instantiated.
These strings are too wide. Can you please break them up with newlines in the source so they're more readable?
I like my hack less and less every time I see it. please rewrite to a regular if statement and separate enable/disable calls.
Thanks for the comment. Fixed.
The "Yes" was for the first thing.  As for validating in the builder method instead of the build method, it's up to you.
I can't think of any use cases that would make it worth it to export this function in the public API. @carlosmn, do you think this is worth an export?

Regardless, if this stays internal, it should get a double underscore. `git_remote__get_head`.
I only have this is so that it is possible to output a "progress" value that is monotonically increasing. Maybe I should have a diff iterator progress API instead that returns a float between zero and one and get rid of this (or at least make it an internal-only API).
Same here. `s/True/ignore_duplicates=True/`
This now needs to be deleted. 

No tables created during `setUp()` and the `tearDown()` should truncate the tables it has used.
missing spaces here
Please remove this (and it is wrong btw. ``$ret[1]`` is a string
You should also typehint the NodeInterface here
I can change it to 2.3
you should check for invalid strategies here
Same as doctrineOrmTypeGuesser comment
Or even better, just remove the `symfony_doc` data as we already have the version in `symfony_version` and move the URL directly into the template.
I wanted to leave room for different implementations.
Same comment about ```<Object>``` and why does this return ```ArrayList``` instead of ```List```?
The same applies here...
Is this use of `!!` intentional? Only have seen it in ruby to make something "truthy" into "true".
Hi can I please be removed from this project for the time being, its sort of filing my inbox
Thanks
Nikki

From: "Jessie A. Young" <notifications@github.com<mailto:notifications@github.com>>
Reply-To: "yammer/sched.do" <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tuesday, October 30, 2012 12:02 PM
To: "yammer/sched.do" <sched.do@noreply.github.com<mailto:sched.do@noreply.github.com>>
Subject: Re: [sched.do] Extract private method `yammer_user_url' (#180)


In app/models/user.rb:

>      )
>    end
>
>    private
>
> +  def access_token_for_query
> +    { access_token: access_token, }
> +  end
> +
> +  def access_token_to_query
> +    access_token_for_query.to_query
> +  end


done

-
Reply to this email directly or view it on GitHub<https://github.com/yammer/sched.do/pull/180/files#r1983346>.
In der security.yml wird das Authentication (welcher User greift da gerade zu?) und Authorization (was darf der aktuelle User sehen und machen?)-System von Symfony konfiguriert (siehe http://symfony.com/doc/current/book/security.html).
Intressant ist vor allem der provider, also der "Zulieferer" der Login-Informationen. Das uebernimmt in unserem Fall das FOSUserBundle mit seinem user_provider. Ueber die Firewalls wird dann definiert welche Bereiche der Seite wie geschuetzt sind. Grundsaetzlich alle (^/), aber unter access_controll erlauben wir einige Seiten wie Login und Signup auch als Anonymous. Schau dir auf der Seite mal an, was unten in der Dev-Bar als zweites von rechts steht - das ist der aktuelle User!
Rollen kann man hiermit auch verwalten, in unserem Fall bisher User, Admin und Super_Admin. Diese Rechte kannst Du spaeter in Controllern etc. ganz einfach ueberpruefen. Es bietet sich auch sehr an, das isPremium hierhin als Role zu verlagern.
Not sure this is necessary.
Fix for the xerces issue
right, on it - i ought to walk the walk if i demand it from others :)
is this just for testing or do we now rely on admin-shortcuts?
Ah, I see it now; I should have read more closely.
If you add a docstring to each API method we can auto-generate the documentation for the whole thing with Sphinx.
For the time being, I have another way to ensure we don't breach abstraction boundaries. The pyface editor area pane will now feature a `get_context_menu()` method to return its homemade context menu (this context menu will be hooked to the tab bar by default). If canopy editors want to have this context menu appended to their own context menu too, they can do so themselves by making a call back to this function like: `self.editor_area.get_context_menu()`

Of course, we can add the corner-widget etc. on the same framework as and when required, but this gives me a quick solution which seems to work decently.
This should also not be needed with chef_gem per a comment in http://wiki.opscode.com/display/chef/Resources about chef_gem
non-sync write
@pld API key is registered at https://code.google.com/apis/console. Go to API Access and create a client id.
Great tip! I make it so complicated compare with it. Thanks! Suggestion added.
this.buttonClick = Searcher.loadMoreApps; works?
no reason, an honest mistake- changed.
Sure, we need this fix also in input_areas.css, but the selector should lool like this:
input:valid { padding-right: 3rem; } this wikl be only triggered when the field has content instead of doing it in the focus!
Thx!
I think a lot of complexity and reasons why you need to do indexOf/contains/remove/add is because you use an [] instead of an {}. If you indexed your favorites list by frequency it will make it easier.
You should change the value in checkbox, better than make a 'click' in the input.
Declare the variables closer to where they are used. That make it easy to see what it is really (otherwise 'kbr' does not mean a lot!) and it prevent you to have zombies variables.
nit: if there an id no need for the type.
You don't need this function unless you are looking to extend it later
Maybe you want a different name since this bug has landed.
It is the check latency…

Let's take my example series: here is the planned time and check execution latency:
10 h 0 m 58s, latency 1s
10 h 1 m 58s, latency 3s
10 h 2 m 58s, latency 1s
10 h 3 m 58s, latency 1s
10 h 4 m 58s, latency 1s

Graphite time-stamped data
10 h 0 m 59s
10 h 2 m 01s
10 h 2 m 59s
10 h 3 m 59s
10 h 4 m 59s

That is why having the check scheduled in the last few seconds can cause gaps in the time-series databases.

Anything that causes the self.next_chk to drift around is bad. It must be a fixed interval. Shinken can still execute it a few seconds late or on time, that is not a problem, but over time it should not cross over a minute threshold.
I think every task should have their own deserializer. Virtual ports should just put the serialized message on the task's receive queue. This spreads the load of serialization to the tasks.
I had a repo where in the commit message for a tag were quotes.
svn2git was trying to tag that version git style, and is providing the the commit message on command line.
there the right quoting is really required since otherwise the command is getting too many, or bad  command line parameters.

I tried your fix first, wondering about the strange escape syntax in ruby, but it failed; so i corrected it with the suggested line and then it succeeded.
It could also be defaulted to English if you want, or we could return null (which is the case when the key isn't found).

If null is returned, OsmAnd should work as it's currently working.
Add the braces to the foreach loop.
The function can produce results like: `10d 4s` - isn't that strange? Maybe just use the top 2 non-zero units, i.e.: `10d 5h`, `10m 56s`? 

Is there any standard way to do this?
You aren't check the instance you are only check if the object  basic on controller was changed, I think. You have to take the object from bd.
So how do those cases look like? Any chance they break all other code using `String::Quote`? Test cases please?

I also dislike how `quote`, like `String::Quote`, works only inside the string and leaves you to put the necessary outer quotes by yourself.
This is the initial effort trying to rebuild de CLI Help https://github.com/felipevieira/freeseer/commit/610da40db8237b884801e9395ea3a26f39e7c248
Done and done.

I have commited and pushed the changes again.

I have not added the keys to the other language files though. Do you have
that done automatically? do I need to do something about it?


On Sat, Oct 13, 2012 at 1:18 PM, Matias Griese <notifications@github.com>wrote:

> In
> administrator/components/com_kunena/language/en-GB/en-GB.com_kunena.views.ini:
>
> Also write names as they are officially written: FeedBurner, RSS, URL... Also end all the sentences with period. ;)
>
> Can you make the title shorter and use Camel Case in it?
>


> Reply to this email directly or view it on GitHub<https://github.com/Kunena/Kunena-2.0/pull/1136/files#r1835753>.
>
>
Those ain't deadlinks.
This is really not the correct solution to the problem. The fading issue is minor, though, so I'll accept the pull request and sort it out myself.
All the current helpers take a ssetid. This takes a takes the date to create one.
It would be a good idea to link to the bug (https://bugs.launchpad.net/zc.buildout/+bug/484735).
If I have time, I'll actually merge this and do this myself.

Getting married takes time :)
not for this - can be any string
Why it's linux-only? Why can't I use purple on windows/mac?
also it should be splitted here
http://jsperf.com/document-body-vs-document-getelementsbytagname-body-0

Fair point. More house style, really.
`btoa()` is IE10+ only (had to look this up, had never heard of it). IE will probably be the source of most of our bugs so maybe worth rethinking this?
This should use xrange for speed.  If a lot of passwords are generated, it would save some time.
Payload length can be 0 here I think (might be good to not call DecodeTeredo at all in this case).
@jordansissel thanks, but my protobuf definitions in the Org::Zenoss::Protobufs module below require beefcake.

This is one of the first things I've done in ruby, so I'm not sure how to handle it.
RubyMine will highlight unused variables.
Any code that want's to create loader like the one it runs in needs to pass `require('@packaging')` as options, maybe override few things here and there. I do it for scratchpad for example.
console.log should be removed.
In PRIMARY_PROTOCOL.md (https://github.com/mozilla/browserid/pull/1743/files#L3R272), the docs inform users to get their provisioning_api.js file from ```https://login.persona.org/provisioning_api.js```, but here it is fetched from ```https://static.login.persona.org/provisioning_api.js```.  Do we want the general public retrieving theirs from static.login.* as well?
The reason I had it based on the default database name before was so that tests wouldn't get the non-test cached value out of memcached (because the test database name is always different from the non-test db name, it has "test_" prepended). But now I have a better solution in place for giving the tests an effectively clean cache (via cache key prefixing) so this extra complexity isn't needed here anymore.
Please move this down below all the components registration code and also put brackets around the function body.
Yes, that assertion was also in the docs.py and media.py and I was making them match. If it needs to be removed then I'll remove it from all three.
margin left and bottom attribute can be put together under one property.