labofsahil/github-event-dataset-2012 huggingface dataset:
No problem. Thanks for the fixes they are already merged and I closed the
corresponding tickets on lighthouse.
On Feb 20, 2012 2:01 PM, "Ricardo Soares" <
reply@reply.github.com>
wrote:

> I totally forgot to uncomment those lines! Sorry.
>
> On Mon, Feb 20, 2012 at 4:45 PM, Zuzel Vera <
> reply@reply.github.com
> > wrote:
>
> > You forgot to uncomment these css (this bit should be used on the tasks
> > list at a challenge home page but not on the page of an individual
> task), I
> > will uncomment it while doing the merge.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/rsoares/lernanta/commit/ed14918e435f8d64eeed5b941361e141c7b293a0#commitcomment-986340
> >
>
>
>
> --
> Cumprimentos,
> Ricardo Soares
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rsoares/lernanta/commit/ed14918e435f8d64eeed5b941361e141c7b293a0#commitcomment-986932
>
i think you can skip the `reverse` and `forEach` and thereby reduce some of the overhead here

```js
while (subscriptions.length) {
  stack.push([subscriptions.pop(), value]);
}
```
no idea actually, you caught me in copy pasting....
I see in my log that there is a lot of memory leaking in 2800.5 ( Messages in log : __NSAutoreleaseNoPool(): Object ... of class ... autoreleased with no pool in place - just leaking)

As far as I understand, we need to create an NSAutoreleasePool for every thread we create. Otherwise memory will be leaked...
This didn't make it in 1.2.0, it is in 1.2.1 however.
Looks very nice, thanks
Capitol idea! i don't know what that means exactly, so we should capture in an issue.
for the real player? no way!
r+wc if you change this to ask for user's email - looks good!
Thank you. I appreciate your prompt reply.
Dan werkt het weer, alleen komt de nieuwe query niet in de history terecht.
Just a loose idea - shouldn't `ActionController` and `RestfulController` have common base class (let's say `AbstractController`)?

- There is a lot of duplicated code, code it would reduce codebase and make it simpler to change both classes behavior when needed.
- It would be possible to implement injection based on base class in DI for both kind of controllers (for example plugin broker etc).
This commit is only a merge of my local repo, this hook is used in some spells. The commit alone don't do anything.
Pffffffffft!
Its close, should be released fairly soon for first beta..
Может в имени сделать Scrobbler с большой буквы?
lol, I forced myself to not push to master without waiting at least one day. Maybe you want to try that out, too? ;)) Thanks for the fix.
DELETE FROM `smart_scripts` WHERE `source_type` AND `entryorguid` IN

typo?
it could work as in sklearn.datasets eg. 

https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/datasets/twenty_newsgroups.py

or :

https://github.com/nisl/tutorial/blob/master/datasets.py

or :

https://github.com/mne-tools/mne-python/blob/master/mne/datasets/sample/__init__.py

you might also want to look at the scikit-data:

https://github.com/jaberg/scikit-data

and see you soon in person in Nijmegen
Mhh, yes, probably, that's the only usage of eZINI AFAIK.
I guess the usage of the libs is quite minimal.
https://github.com/ezsystems/ezp-next/blob/master/ezp/Content/FieldType/XmlText/Input/Parser/Raw.php probably requires some refactoring since it uses eZXMLInputParser, eZContentObjectTreeNode, eZURL,... probably as a temporary workaround!
looking forward to that pull...

also I think looking into this I can figure out the question I had... how to get all the dependencies for the maven pom...

just trying to figure out how I can get around using the async scheduler on the TombDtp Object as it is causing issues...
I had to change the whole script to use tabs instead of spaces for indentation so that I could use the handy bash `<<-MARKER` syntax here.
This cset looks good from my view as a hg user/maintainer (though I am fairly new to buildbot here ;-) )
Insert:
```top.restoreSession();```

Need to deal with sessions whenever open a new script or refresh the current script: http://open-emr.org/wiki/index.php/OpenEMR_System_Architecture#PHP_Sessions_and_Browser_Windows
Unfortunately, Environment.SpecialFolder.UserProfile doesn't exist under .net 3.5 - was breaking the build. I've committed a fix for it, so now the nuget package is up to date.
adjust()'s purpose isn't self-explanatory
We need to generate a new key. This is an embarrassing security flaw.
D:! 1 espacio en blanco sorry xd
I have mistake with GetAllBattlegroundsWithTypeId, if you have a idea
thx ;D
At the end of the line there is "><" instead of just "<" :   さい。]></item>
So I get [aapt] /home/nico/src/log_nicolas-raoul_master/res/values-ja/03-dialogs.xml:71: error: Error parsing XML: not well-formed (invalid token)
Updated new home
This is still an outstanding issue, but if the user has an unread, and the window is refocused, it was not clearing, maybe you could dig into it a bit?
I wonder if the space between the circles and the gray border should be white or transparent... Or maybe white with some transparency?
Just a thought, but I wonder if we should encourage migrating all test cases to use the short hand for this? If anything, it will reduce noise.
That's my i called it ```isEqualTo()``` and not ```equals```. It's a quick and dirty compare of the two objects - enough to filter out "equal" objects after calling ```gatherMissingFrom()```. Normally (for a full ```equals```) you have to go deeper into the lists and compare each of the objects contained in it - but that costs to much time. So it's possible that two objects that are equals get treated as not equal with the only result that they are stored in the database - like it has been before inventing this method ,-)
``alert`` ? this is the same level than used by 500 errors on the site. If someone puts some notifications in place to be alerted when a failure occurs, you will notify him each time a spam is detected.
They will hate you and mark your code as spammer :)
言語仕様の" Return Expressions"の節に、「ネストした匿名クラスからのreturnは、scala.runtime.NonLocalReturnExceptionのthrowとcatchとして実装される。(Returning from a nested anonymous function is implemented by throwing and catching a scala.runtime.NonLocalReturnException.)」とあるのは発見したんだけど、この場合、その仕様がどう影響するのかがよく分からない…。
I meant to say 3.1 in the commit message ...
This is super important. When we deploy to a production server, we always want to use secure SSH keys rather than password authentication.
SUPER! Ordner brauchst du eigentlich gar nicht, weil Julian hat schon .class Dateien direkt ladbar gemacht. Also können wir jetzt alles laden.. 1A!!!
I'm not sure, does `-[NSURL standardizedURL]` do this already?
Hmm. I guess the new one is not so height sensitive as the first cmsui prototype. You'll still see problems if you have a narrow window causing the bar to increase in height, but I guess that might be acceptable.
esto no puedo meterlo asi como asi...estas tocando una funcion general
why you think that?
nope its not even close to be removed.

alot of LoS bugs
@Leak ,
does this fix the clearing of instanceID of people, i have that people keep having the instanceID even after the so called weekly reset, specially on ICC etc.
2012-01-09 23:54:14 Table command have not existed subcommand 'wp_data' in command 'reload wp_data', skip.

TDB 35 with updates - core revesion: 10b7eb42d23b
I suspect this might have problems with nested records that have attributes on the nested records.

xrec := record
string f1 { xpath('@f1');
end;

y := record
xrec x{xpath('X');
end;

Assuming the code generator gets it right (not guaranteed) then it will need addressing.
I wouldn't hold up merging this change because of it, but it will require a rethink of the way attributes are generated.  (Or the way records are represented.)
This was the fix for the dom.js crash. After last friday 0x47700 was no longer the correct magic number, it became 0x47d00. Instead of letting this happen again, expose the macro as a function call to rust so it always has the correct value.
I think Renato has misread the line
next = *(unsigned *) ret;

as if it said
next = (unsigned *) ret;

perhaps.

next is a relative pointer and is constrained to < HEAP_ALIGNMENT_SIZE
> ok. does the ant build work? looks like you changed that completely.

I tested that as well, and it works properly, but I could not get away
from the ugly dependency to the bnd jar. Basically the build now asks
bnd to read from the osgi.bnd file to populate some basic parameters
in MANIFEST.MF and scans the classpath to create the proper *-Package
entries in this file. Then it includes all the class files inside the
resulting jar. A better alternative, might be to use the same old ant
jar task and use a handwritten MANIFEST.MF. This would skip the ugly
dependency, but if there are new packages added to the source, or some
reorganization, or some new imports, the MANIFEST.MF would have to be
generated again by hand. Depending on how common these kind of changes
are, this could be a better approach, but I think you are in a better
position to make this call :-)

Irving Cordova
http://irvingc.com
@jedi4ever, woa--that link is extremely helpful, thank you. When I originally wrote this, puppetlabs didn't have a rabbitmq module (to my knowledge). I almost feel like since they have one now, this project should be nullified.

Those guys usually do a way better job than myself.

What do ya think?
I'm not sure if we want to enable them by default for entity beans. If you attempt to load loads of entity beans it is unfortunately pretty easy to exhaust the strict max pool, so it is almost impossible to give a default number that makes sense.
Yes, it will be used later.
Move this into a single line?

`var $window = $(window);`
I misspoke, issue was in Collection.create, not Entity.
The next steps will be to integrate the attack talent with equipped item properties.  I'm not sure how to go about setting up the part upgrade code.  I had a method that worked something like an ego, but it was awkward to set up for individual parts.
`buildUpdateFrontSide`? WTF?
for me it isn't working, Segmentation Fault, when i use Aspect of Fox or learn Talent Firestalker... server shut's down, how can i fix this? running latest DB (with updates) & core
Geodar
It's not strictly necessary but I think it's a good practice to check whether the module is enabled before performing any logic to modify it
Why is it called GdkX11::X11Window in GIR?  The duplication of X11 seems so unnecessary.
I think you broke something with trasnsparency here
yes, thx ;)
On Sat, Jan 07, 2012 at 07:45:37AM -0800, Alexandre Gramfort wrote:
> this could end up being negative. I guess you meant max not min.

Good catch.

G
@keesun 아.. 그렇군요
Könnte man statt Stringmanipulationen vielleicht das hier verwenden: http://download.eclipse.org/modeling/emf/emf/javadoc/2.4.2/org/eclipse/emf/ecore/util/EcoreUtil.EqualityHelper.html
I hate java just for this reason. It took me so long just to setup the build correctly. It never is helpful for newbies. Somehow, got the build going, but the deployment still fails.
So two things. If the file descriptor is 0, 1, or 2 it corresponds to stdin, stdout, or stderr, respectively. Secondly, to extract from the buffer, what you'll probably want to do is reference my code in the handleClose function. On line 404, you say int fileDescriptor = readVirtualMemory(...), but the return of readVirtualMemory(...) is the amount of bytes read from virtual memory. You'll need to convert the data from your byte arrays into integers.

Anyways, that aside, you need to get the fileDescriptor and the count first. Then you'll probably want to open up a BufferedReader and at most count characters into a string. Then write that string into the buffer using writeVirtualMemoryString.
Ah, nice - this is so that, if things don't work as expected, we have 10 minutes to inspect the page and look at what actually happened?
oh noes, what i done? D:
Убрал чтение "заклинаний" с хранилища перед загрузкой окна настроек. Заклинания уже считывались при загрузки настроек и делать это повторно нет смысла.
I guess this is pulled in transitively by Maven?
Sorry, I somehow missed that process "Main" is still there...
Makes sense :-)
Ah, while this code is part of a firefox extension, it creates speakeasy extensions that could run in IE.  Cool, thanks!
Sling is (among other things) an HTTP front-end to Apache Felix. It determines how each request should be delegated (which servlet, etc.). So by "claiming" a path for use by the Wicket app, Sling will delegate requests for that path to the right place.

This isn't in production or anything. I was doing an experiment to see if Sling and Wicket could co-exist. The experiment works, but I haven't tried to take it any further.

Cheers.
ook hier (bij de binnenste strtotime), liever werken met timestamp dan met datum die via strtotime wordt omgezet naar timetamp
Half of the tests: If a%2==0: it's useless to test others even values (they aren't primes)
@max-m: I would guess that that is one part of "work in progress" ;)
Pas assez longtemps alors! :p
:p

new Git
N<JFJJFJKDFSKJDFSJK I WAS JUST ABOUT TO DO THAT I HAD THE CODE I WAS JUST ABOUT TO COMMIT STFU
Deprecated https://github.com/doctrine/common/blob/master/lib/Doctrine/Common/Annotations/AnnotationReader.php
things went way wrong on my end
i tried to find the least destructive solution


On Thu, Mar 1, 2012 at 12:14 AM, Devin <
reply@reply.github.com
> wrote:

> Figured I should ask if these commits went the way you intended. At any
> rate, my changes to the SRD weren't merged in. I can redo them the next
> time I make a modification; though not sure if that'll be tonight or in the
> morning.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/blake6489/Community-Project-Tracking/commit/bf889bea86329ce1d33081a5a1297fcfa9fb7ee6#commitcomment-1030151
>



-- 
Blake, in Fairbanks
You can now use bcm2049. If you didn't know, psyke83's patch for this FM device has been merged officially. Look here for more info: http://review.cyanogenmod.com/#change,10358
@michael-harrison - Returning nil might be good, if you can know that it doesn't exist. Also, I think `directory.files.new(:object_id => uuid).public_url` should work in a way similar to AWS (if you happen to know the object id already).  Definitely less convenient than the AWS/Rackspace versions where you can use the key though.
You're right, it's not really monkeypatching, that was a careless use of the term. Whatever it's called, though, it's a non-obvious and error-prone API to require external code to set a particular attribute of an instance before that instance is usable. The fact that it's set by the constructor doesn't really help in this case because it's set to a non-functional value, and there's no error-handling around the use of `self.set` to account for the possibility that it could be `None`.
What's the thought behind repeating the styles that are already defined in `.yui3-g`? Could we simply require that you tag an element with our standards grid class, and that the responsive stuff applies to the contents within a grid?
Was the thinking that requiring both class names is confusing?

```html
<div class="yui3-g yui3-rg"></div>
```
Please check if "o.mode === ADM" to decide if we need to refresh in baseView instead of copying codes here.
This has been done by another commit
Any suggestion to do it better? Is it good like this?
Not sure I follow this line. Why is the theme name mapped to the oauth20 service name? shouldn't it be the service name as the value?
… he must make sure that the asked actor replies with Status.Failure …
clusterView.leader.fold("")(_.toString) ? :-)
This is the opposite of Spring. In Spring you have abstraction over abstraction over abstraction. A class has a base class that has a base class that has a base class that has a base class that has a base class that has a base class that has a base class that has a base class that has a base class..und so weiter...
no need to match "address", use _
`:` is not a valid character in URI path elements, and we need to use protocol descriptors in there
pro/cons for `to[InexedSeq]` vs toIndexedSeq` ?
Why would that be worrisome? They can’t change the meaning of FQCN akka.actor.DefaultSupervisorStrategy, can they?
why not use for-comprehension here also? while/var just smells bad
account info is totally tested, probably fail between 6 and a few dozen tests.  I'm replacing 'mail_from' with 'moil_from' and running tests with each instance broken in turn.  I'll let you know the results.
Please disregard my comment about collections.Counter because it requires Python 2.7. 

Advantage over the defaultdict is that the intent is more clear; Counter also has some methods (like "most_common") that defaultdict doesn't have, but they are not needed here anyway.
Ok I've clarified this now. I think this is finished now, can we merge it? How about adding a top-level specs folder for these?

    specs/
      001_organizations_and_groups.rst
      002_...rst
      ...
Check read() again here.  You can use it's retval to avoid the strlen() call on the next line.
I feel like this function should just call "predict_future_billing_multiplier" because they greatly overlap.
"Use *conf_paths* instead."
Por qué utilizas Double? Debería ser BigDecimal no crees?
Yes, I was trying to keep with the status, headers, body convention but I suppose that doesn't really matter in this case.
Whoever wrote this code should have copyright on it..
No, this is not the better way:

```
$ php -r '$f = true; $v = (array)$f; var_dump($v);'
array(1) {
  [0] =>
  bool(true)
}
```

A non-array value will be converted into array, so `if(!$arrPks)` will return false, even `$arrPks` is an empty string!

```
$ php -r '$f = ""; $v = (array)$f; var_dump($v); echo ($v) ? "yes\n" : "no\n";'
array(1) {
  [0] =>
  string(0) ""
}
yes
```
Couldn't you just do a bash string compare of listing1 and listing2?
Extensions for point, rect, and size are unnecessary. Those CG and NS types are interchangeable on 64-bit, and it's highly unlikely that backwards compatibility will be broken over something so trivial.

> Well, wouldn't that basically be saying "prefix every method possible"?

Every category method possible, yes.
@CodaFi Ah! Didn't catch that. Thanks :)
We don't need to cache fonts anymore - this is a `TUIFont` vestige. Would you mind changing `exampleFont1` occurrences to `[NSFont systemFontOfSize:NSFont.systemFontSize]` or its bolded counterpart? That would remove the need for an init method at all.
There shouldn't be code on the line of a closing brace. Can you move the `return` down?
s/2/ARRAY_SIZE(argv)/
Please remove.  I want the basic library to be as small as possible.
task is not None?
Why add the synchronized around the full method call and not just the epoch check?
since OrderedDict is not in 2.6, i presume this is preferred.

```
if not sys.version_info < (2, 7):
    from collections import OrderedDict
```
лучше переименовать в createLoopLink()
This name is pretty poor, I'd just call it `lock` or `set_lock`.
I don't know if they're related, they appear to be not, but I've bookmarked that issue to take a look when I find some time here.

The problems I had was mainly with specs calling this and similar methods with `nil` values, I don't remember since it's been a while I had them, and I solved with some tweaks in the app.

Thanks!
It's ok to protect the `seedvalue`, this is internal, and nobody had probably used it (plus, I think it was added after 2.9).
Why not use `sys.props(name).toInt` or extend the functionality there?
@Cadair could you add in the documentation which one is used by IDL (and the version of IDL you compared with)?
Also, it would be good to add to the wikie the tests you showed in the maillist
`Matrix` probably shouldn't be imported in this file. ImmutableMatrix should be able to get everything it needs from `DenseMatrix`
This can be

```
Notes
=====
Hints are ...
```
Please note that almost none of this file is new. I just renamed it from risch.py. See my other comment for how to get a true diff.
In the upcoming changes this is left as the simplest fallback. Now the calculation is more sophisticated: instead of giving you an opaque `Commutator` object with no useful expression behind it it actually calculates the commutator. For instance `[e_x, e_y] --> 0` or `[e_x, x*e_x] --> e_x`
Ah, sorry, didn't pay enough attention :-(

If you take a look at the third branch of the `if`-statement in `ident_remove`, you will see that, if all terms in the expressions are identities, one of them is kept, so that we don't arrive at an empty expression.  This is what happens with `Basic(0, 0)`: here, the whole expression consists of identities, and one of them is kept.
I have not looked this up into details, however at least mark this as a `TODO: make width an attribute that can be changed`
null'a gerek yok. Node ya da boolean almasi cok mantiksiz. boolean mi?
I don't see a problem with giving a list of reference links. A short description is good as well.
@LemonBoy 
maybe the value should be greater than '0' here. 
what's the minimum width the bar should be allowed to be ?
I am sure that url_normalizer used to be applied to objects of type _null when there was no title field in the form. For me it proofs that something is incorrectly implemented. 

I have read FileAddForm again. There certainly is some code, that I think was designed to construct title from the name of file. Unfortunately I do not know "collander" enough, to say it is correct or not.

Could you point me the line of code in which it is tested that uploaded file has a proper title in database?
I think the URL is way too verbose.  We generally want URLs to be readable and easy to type.  What about `site/@@search-tags?tags=foo,bar,baz`.
If the plugin doesn't make sense without another plugin being loaded, it should probably be enforced as a dependency.  Add a class-level ``PLUGIN_DEPENDS = ['users']``, and the plugin won't load without its dependencies being present.
Do we want this router to be retryable by default?
those two types never worked anyway.
Sorry, maybe I wasn't be super clear.  This shouldn't use ActiveRecord or try to set `attributes` at all. 

Just use plain old Ruby on the `Repository` class: instance variables and attr_accessor: http://www.rubyist.net/~slagell/ruby/accessors.html

psuedcode:
```
class Repository
  attr_accessor :job_template
end

repo = Repository.new
repo.job_template = "some/value"
repo.job_template => # returns "some/value"
This alters the value of `@api_url.path`.

Should be:

```ruby
@api_url.path + path
```
And comment is old and spooky there ;)
I'm thinking of getting rid of the doc gen, since we don't use it anymore, but all I wanted to do in this patch is to eliminate references it generates to components that may be being refactored, hence I have just commented it out for now.
clarify that these three members must all match the mode and tags?
and stores collections of data related to the local operation of replication; however, `local` is invisable to replication, and collections in the local database are not replicated.
But if it goes to /en/HTML with Accept-Language: de it should redirect to /en-US/docs/HTML which should redirect to /de/docs/HTML right?
This isn't part of this change, but should this be in a ```try/finally``` clock?
Added the function supports_phrase_search( ) and class property changed to private.
Looking at other code, this is how it was being done when other new services have been added to (such as request).
It's an array, so use empty()?
`$db` should also be passed into the constructor and assigned to a class property rather then globalising it in every method.
You can't have a protected method in an interface, that's the error PHP produces.
$alt not documented.
Please also add a one-line description of what the function does. This might be very similar to documentation for $style_id.
Should use $phpEx.
Operation in loop definition.
http://area51.phpbb.com/docs/31x/coding-guidelines.html#optimizing
@rphillips I don't understand why you mean. fopen is part of stdlib.
So I was hoping that this line would work for the `new Svg` line in the SvgRenderer constructor, but it didn’t fly. I got it to work by making the new Svg line say `new SvgRenderer.Svg` but then there were other integration issues with L227 `svg.root.addEventListener`

Any tips?
Is the title here set by some value in fetchData?  Confused by setting title explicitly above, or calling the fetchData function if no cached_title without setting the title variable again.
Declare `i` var at top of function.
Why is the function declaration not public member like other functions in this class?? We don't want to make this a private function.
For an example of key-value pairs that John is talking about, you can refer to this: http://jsfiddle.net/nathanleiby/Skj2X/
White space craziness introduced. Best not to edit this file in a DE. Open with the one true editior vi to see what I mean ...
Just because this is always supposed to return a string, but I'd prefer it throws an exception `No method to check mime type`
How do people think we should handle this TypeError?
Pulling it as-is will probably make a lot of 3rd-party ops break.
We could make it issue a warning and turn the None into a _float_zeros_like. In a later release we could turn the warning into an error again.
For the current release, should it be a TypeError all the time, a warning all the time, or allow people to switch between TypeError and warning using some kind of config option (a config file flag, parameter to tensor.grad, etc)?
What have we learned about %lld?
how about just `closes the dynamically opened libary`?
please revert it. It should not be necessary if fetchAll was implemented as expected by the interface
Can you just delete these lines instead of commenting them out?
it removes blink of unloaded loading indicator and removes some unnecessary space between icon and "viewing occurrence" message
gah, thanks
viewDidLoad に対するコメントは必要ですか？
viewDidLoad は元々あるメソッドなので、特記事項でなければコメントは不要です。
Just tried again, +1 byte gzipped. :-\
We want to be able to find all tests for a specific method using the filter option of qUnit. If anything, `get`, `getScript` and `getJSON` should be extended. Another solution is to use a forEach and separate each test in a specific unit.
should this be covered by a unit test?
This will require numRows to be a member field in class LMFModel, which is not currently the case. But +=(State::Model) or LMFModel::some_interface_for_model_averaging is necessary, because we want to have this to be generic across LMF, Logit, SVM, etc.
For the multi-level comprehension a proper for loop would be easier to read IMO.
Not really using locate anymore, I guess ;)

Using `xcrun` seems like a good solution - I'm much more comfortable with that than `locate`. `xcrun` should exist on all supported configurations, yes?
... this line is not needed. I checked it (buit your formula). `xcrun otool -L $(brew --prefix)/Cellar/isl/0.10/lib/libisl.10.dylib`
Should this be `.pop('title')`?
I think you should add a return statement here.
Probably want to update the ```_content``` to ```:content!```
Does this need to be in `finally`?
It might be cleaner to move the error reporting code to a separate function.
This should also be an else if
Doesn't the pump at https://github.com/Singly/hallway/blob/3ef0d5d8a695df54ef6a9e470141f257717fe226/lib/services/facebook/map.js#L235 set the photo type? Looking at the photo @beaugunderson posted to me, it seems like that should be happening just like it is with videos and news. Specifically because it has `type !== 'status'`.

```
{
      "id": "204381_3362356749178", 
      "from": {
        "name": "Beau Gunderson", 
        "id": "1576860066"
      }, 
      "to": {
        "data": [
          {
            "name": "Kristján Pétursson", 
            "id": "204381"
          }
        ]
      }, 
      "message": "As requested. :P", 
      "picture": "https://sphotos-a.xx.fbcdn.net/hphotos-prn1/s480x480/552599_3362356749178_379447950_n.jpg", 
      "link": "https://www.facebook.com/photo.php?fbid=3362356749178&set=p.3362356749178&type=1", 
      "icon": "https://s-static.ak.facebook.com/rsrc.php/v2/yz/r/StEh3RhPvjk.gif", 
      "actions": [
        {
          "name": "Comment", 
          "link": "https://www.facebook.com/204381/posts/3362356749178"
        }, 
        {
          "name": "Like", 
          "link": "https://www.facebook.com/204381/posts/3362356749178"
        }
      ], 
      "type": "photo", 
      "object_id": "3362356749178", 
      "application": {
        "name": "Photos", 
        "id": "2305272732"
      }, 
```
I understand why you're using the MoveEventSucks class. In fact, I switched instaPort to use it as well. I switched TeleportCause.ENDER_PEARL for TeleportCause.PLUGIN because it seemed a better fit, that's all.
It's a tad weird here that we're mutating the entries we receive--I had to go look at the other code to confirm that this wasn't stomping on some cached info. It might be clearer to just create a new object: `docEntries.push({doc: docEntry.doc, functions: functionsInDocument});` -- slightly less efficient, but we aren't going to be dealing with a huge number of entries.
I am pretty sure I tried that and -1 was always returned, so it had no effect. I am wondering if the DOM nodes are being copied so they fail the equality test?
You need to explicitly decode the URI returned from window.location.pathname by calling decodeURI() as I did in https://github.com/adobe/brackets/pull/922.

Otherwise, users who install or run brackets from a path with some non-English characters will have issues in launching.
Nit: should be "its"
changed t0:
 \s*id\s*?=\s*?["'](.*?)["']\gi
See lines 86 and 87 of database.py. For further explanation of how this is working, see the [setattr](http://docs.python.org/library/functions.html#setattr) documentation.

```
As for doing so dynamically, you could use the inverse, getattr:

```python
getattr(self, table_nme).columns
```
This comment block seems unessecary
The following variables are *screaming* to be a class. It sounds like you have one object here that you are really concerned with: a member. That object has the properties outlined here: first name, last name, type, age, etc... A class is a really good choice for associating multiple bits of data together in a coherent unit. I don't know if you guys covered classes. If you didn't, ignore this. If you did, it might be good to try to use them here.

You could move these into a new class like so:

    class Member {
        public String first_name;
        public String last_name;
        public String type;
        public int age;
        public int number;
        public String gender;

        // And the associated getter and setter functions
    }

And then instead of having these variables here, just initialize an array of Members:

    public Member[] members = new Member[100];
Il y a un "trigger" de trop en fin de phrase.
Maybe you understand the data structure than I do... I made the change you suggested, and it seems to work fine. I thought I previously tried it and it failed, but I must have made a mistake that time.
Yes. You should make sure all fields from models are included in the new user document. You can create your own field for language, similar to how it is done for relation database.
fixed... :)
It was my intent to leave it open for people to use `butter.dialog.register()` during execution instead of having to have your whole stack during compile time.
This is what I expected. Exposing it like this will allow templates to use it just as any other file might do using `require`.
No, because if redirect is an empty string or the number 0, it needs to be validated. Using a truthy check will just skip all validations in these cases. This format is also consistent with what I've done elsewhere.
why these variables starts with `_`?
Just few hours before @thatch45 merged this PR, @giantlock said:

```
Thanks a lot for explanation. I agree this loop needs to be fixed.
But I do not understand your last version. I think it might be something like that:

while True:
    sockets = poller.poll(timeout*1000)
    if sockets:
        break  # we have got result
    if tried >= tries:
        raise SaltReqTimeoutError('Waited {0} seconds'.format(timeout * tried))  # but still have no results, so it's enough
    tried += 1

With your version we do not care about poll() results, we will just wait up to tries*timeout and raise exception
```

maybe the build is fixed, but maybe this version is better?
module_name -> name
p.s. Why doesn't Hadoop Configuration have setDouble/getDouble methods? :/
Doesn't seem like the current set of tests needs it, but might as well add a ``@with_save`` decorator to tag.
You didn't minify this?
'open': open,
'close': close,
'getWindow': getWindow

will make my vim happy ;)
Maybe the default `database` should be set using a `ConfigurationItem`?  I can issue a PR to do this if you're not sure how.
This function has not been defined (and this means there are no tests for datetime conversion, so this should be added).
I think this (and `LOG_WARNINGS` above) should default to True - most people probably aren't going to be messing with the defaults, and this will make our lives much easier down the road when we get bug reports if we can just say "look in the log file and tell us what you see".
I also added an option to hide the groups/addresses that have 0 balance. Might be useful for someone else as well. 

Heres a diff, I haven't figured out how to use git yet. :(
http://pastebin.com/YSRxKBqx
maybe also add an explanation why we need OS independent progress bar style, so people in the future will understand :-)
@TheBlueMatt In my experience, a significant number of the connected nodes can be stale, and non-responsive, therefore it makes far more sense to fire off the getblocks to a node that's still communicating. Picking a random node wouldn't achieve this, but picking a node that's sending messages would.
Ok, thanks, updated!
[UndefinedVariablesFix cc78d48] null not 0


On Fri, Sep 28, 2012 at 7:45 AM, ADmad <notifications@github.com> wrote:

> In lib/Cake/Cache/Engine/WincacheEngine.php:
>
> > @@ -183,6 +183,7 @@ public function groups() {
> >   * @return boolean success
> >   **/
> >  	public function clearGroup($group) {
> > +	        $success=0;
>
> Did you actually check the wincache_ucache_inc<http://php.net/manual/en/function.wincache-ucache-inc.php>,
> how is it a constant? If you really want to initialize it set it to null
> not 0.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cakephp/cakephp/pull/872/files#r1714770>.
>
>



-- 
James Michael DuPont
Member of Free Libre Open Source Software Kosova http://flossk.org
Saving wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com
Contributor FOSM, the CC-BY-SA map of the world http://fosm.org
Mozilla Rep https://reps.mozilla.org/u/h4ck3rm1k3
Free Software Foundation Europe Fellow http://fsfe.org/support/?h4ck3rm1k3
why removing the old tests ? you should add new tests for your code, not remove existing tests
should we suggest using enableLoggging: true?
Mja, ik snap het niet zo goed. Is het dan niet beter om hier de parameter weg te laten en bij het opbouwen van het menu te kijken of persoon er recht op heeft. En ook bij het ophalen van de pagina?
Perhaps this is a case of using a template file, what do you think?
*really* minor, but that should be "pretty"
I am not familiar with this call syntax for `trigger
Please add your name to the list of authors.
why are these escaped?
sorry.. I should have caught this earlier.  you need a region arg or else this will only work for a single amazon region.
archetype prob?
I thing it should be no problem, but I am going to ask.
The dependencies where it applies only contain the Transfer objects with the JAXB annotations; there is no mistery in them.

I'll ask here and let you know as soon as I have the response.
Link should be formatted into the string, not directly in it.
What requires ordereddict? Python has its own in the collections module. I wonder if they differ somehow.
heh, nice
Where is SocorroSearchService?
It used to be a submodule.
Maybe it should be e.g.

if ( from instanceof Long || to instanceof Long )

since either from or to can be null really. wdyt?
yay, deleted code!
Seeing massive changes like this are scary, because its hard to tell what has changed. What was this?
Thanks, i will send a new pull request with the change :)
Agreed... We should use the Container properly here. I was just looking for the lazy easy way, but we shouldn't take shortcuts when we don't have to.
There is indeed no 2.2.3 tag in common. It will be the next 2.2.x release in common. The fix we need is not yet part of any doctrine release.
You should mention the EngineInterface instead of the TwigEngine (if you are using multiple engines in your app, this method will receive the DelegatingEngine instead of the TwigEngine
somehow the boarders of HttpKernel and DIC are not quite clear to me here, so I will do what you prefer.
no, we write tests for a reason. So, this test should still work.
read what you said again. You tell me that FormBuilder accept any FormBuilderInterface but only if it is the same class. This means **exactly** that it acceps only FormBuilder, not FormBuilderInterface.
If it's of any use, we used a little trick to process the `q` part [here](https://github.com/zikula/core/blob/1.3/src/lib/i18n/ZLanguageBrowser.php#L73)
Please keep the "For" in the method name.
missing phpdoc for all these methods
I can't speak for not terminating child, but as I commented in issue the signal seems to be sent through to child.
IMO this is not valid use case as you never can change perms of file you execute actually. Is there any case when it not return false on failture ? (docs + comments don't mention any)
I would just rename this to `setLogger()` and keep the logger as a static variable (so you can get rid of the `addLogger` method).
no need to use ``php`` here. If you called the console as ``php app/console``, this would duplicate it in the help IIRC
@Tobion could you PM me (GH) ?
Check out the `byteToHexString` method in `Codec`.
Source: http://www.playframework.org/documentation/api/1.2/play/libs/Codec.html
notification string doesnt match casing of functions: Checkin vs CheckIn
Does `xyz` here mean `name`? Changing might make it read a bit better.
I may have missed earlier discussion of this but it seems like this is unnecessary. My understanding of facades is that they wrap up complex interfaces but `ActionMailer`s don't seem like complex interfaces to me.

The current implementation is also confusing to me. I didn't expect it to do delivery given the method names. Possibly just a naming thing. Maybe a name like `Mailman` or `MailDeliverer` would be better if the team feels strongly about keeping the class. I'd like to hear the argument for keeping it, though.

Also seems to be missing a spec.
I bet `Suggestion.new` is the fastest of all! :speedboat:
Because i wanted to create one repo for the entire suite of tests (switching ot a before_suite)   Since before_suite is a class function it doesn't have access to instance variables.
These are testing extensions so please to put in the extensions tests.
I think this wms_retrieve function is really a method on this factory...
Here's a whack-a-string alternative; copied so some field names do not make sense.

```C
char **parse_fields(char *key, int *nkeys)
{
    char **argv, *p=key;
    int dots=0, argsz=8;

    *nkeys = 0;
    if (!key) return NULL;
    p = strdup(key);
    argv = calloc(argsz, sizeof(char *));
    argv[0] = p;
    while (*p) {
        if (*p == ',') {
            *p = '\0';
            argv[++dots] = p+1;
            if (dots == argsz) {
                argsz *= 2;
                argv = realloc(argv, argsz);
            }
        }
        p++;
    }
    *nkeys = dots+1;
    return argv;
}
```
arf ca me saoule ca mais c'est corrigé :)
Sorry , Eclipse modification :/
Not feasible as i need to give the called templatetags a parser and a token. It will stay not long here and is only 7 lines of code.
The ``original_context.get('request', {}).current_page`` part looks like it could fail with a very hard to understand error. An ``AttributeError`` instead of a ``KeyError``.

I'd suggest either using ``original_context['request'].current_page`` or some other way of handling missing requests here.
can you rename this variable to `clear_controllers`?
This added pagination to the activity feed for #426 :)
Some of the JDBC_xxx attributes allow null and don't have defaults, so resolveModelAttribute will return an undefined ModelNode.
This API is used from the outside (i.e. outside of the server, to launch it) and I didn't want to require that this API needs jboss-msc on the classpath in order to be used. It currently has an external dependency on the DMR client already but I didn't want increase the number dependencies. Converting between ServiceName and a String array is easy enough.
It's not necessary for the styling as the `display: inline-block;` works the same for both divs and spans. It's just if it had been a div, it wouldn't have needed it in the first place. But a div makes much more sense for many other reasons.
As you do here....
Remove trivial geography.
Perchè tutti questi spazi ?
lgtm. Good find!
Wondering how the linter didn't pick this.

Seems the lines between the else have double spacing.
nit: give it a little name
Nice catch, my yui days and node days are conflicting =/ I go with whatever I have used so far.
We should 10n this value returned
I have opened #4068 to track this
Can we compromise on something like `<nav data-type='sidebar'>`?

Or is there something preventing us from doing this?
Indeed. You need to use the `DOMRequest` returned by both `sendUSSD` and `cancelUSSD` functions.
`<!-- z-indexes of all the overlays in the system -->`
Sorry I was unclear, I meant something like `itemSelector` in order to avoid having to add a line return here.
1) Done.

2) Same as above, I also add comments.

3) OK, I will note this. ^^

4) I found a document on MSN about URL.createObjectURL():
https://developer.mozilla.org/en/DOM/window.URL.createObjectURL
and an example:
https://developer.mozilla.org/en/Using_files_from_web_applications#Example:_Using_object_URLs_to_display_images

I tried to follow this example to also use URL.revokeObjectURL() to release the URL after the audio tag is loaded.
Looks no problem and no side effects after I added this.
So I will keep this first and find out it's safe to use like this or not.
(The loaded event for image tag is <code>onload</code> for sure, audio and video tags are <code>onloadeddata</code>)
This doesn't add inline CSS does it?
This was what I was remembering:
https://github.com/mozilla/elasticutils/blob/master/elasticutils/__init__.py#L918
if you're gonna touch it you might as well fix the weird indenting. :grinning:
I can make it 0.0 to 0.85. Which is better than the old code which only did 0.5 to 1.0. 

It also means that for:
5 minute check_interval check, it will have 45 seconds for a check to to execute.
4 minute check_interval, it will have 36 seconds for a check to execute.
3 minute check_interval, it will have 27 seconds for a check to execute
2 minute check_interval, it will have 27 seconds for a check to execute
1 minute check_interval, it will have 15 seconds for a check to execute
Nice small simplification. Changed.
Closing the socket right away can lead to loss of data;

    client.timing(...); // Need a round or two in the event loop for DNS lookups &c.
    client.close(); // Will close the socket before any further event loops

Trivially hackable thusly msiebuhr/node-statsd-client@a58331eb8eb76319c8fb4dd770d3ecaf9cfce208. Or by keeping track of outstanding requests.
:'(  ... sad ... you know that it´s hard for me to delete useful code .. :wink: but you are right!
Missing parens.
Maybe you should just do:

```scala
type PipeInt = (RichPipe, Int)
```
rather than introduce a new class (which you never pattern match on).  I think the case class is useful if you are actually pattern matching.
I put that extra # for looks.
Check if this file exists and abort if it doesn't?
This comment doesn't seem correct. How could it ever return `False`? It appears to only ever return `True`.
Could you please explain the falseEasting/Northing and offsetPercentage stuff? I don't quite understand why the transformation coefficients are percentage values (while they should shifts basically, so are expected to be of a pixel value)
What about two lines above?
You need to remove the `u` there too.
It explains why I had to add `gem 'sinatra-reloader` to my Gemfile even though I had it installed.  
Thanks for the explanations!
It would be good to handle the MSVC case here. You can mark a function a deprecated in MSVC using the `__declspec(deprecated)` decoration.

So all I think it involves here is adding:

```c
#elif defined(_MSC_VER)
# define AMQP_DEPRECATED __declspec(deprecated)
```

Then moving the `AMQP_DEPRECATED` decorator so that it is in front of `AMQP_PUBLIC_FUNCTION` and that should work.
Ahh, to avoid updating the class attribute. Would `None` be a better choice for the class attribute? So we get an error if there is a mistake?
Transactions aren't automatic magic atomicity!  See http://en.wikipedia.org/wiki/Isolation_%28database_systems%29#Isolation_levels

You're assuming serializable, which is to say that the commit will fail if data that was read during the transaction has changed (range locks).  Even Postgres doesn't meet that high bar.  And SQLite is read-uncommitted in some cases.
please add a space between ``foreach`` and the brace. Same for ``if``.
Shouldn't this just use default permissions rather than 777?
Fair point; there aren't any Composer exceptions yet, ok to create something like `Composer\Exception\UnknownPackageNameException` which extends `\UnexpectedValueException`? Couldn't see a non-exception method to halt the flow here (without rather ugly refactoring).
Little refactor suggestion here:

```ruby
...
  @invalid_emails = html_safe_string_from_session_array(:invalid_email_invites)
  @valid_emails   = html_safe_string_from_session_array(:valid_email_invites)
...
private
...
def html_safe_string_from_session_array(key)
  return '"" unless session[key].present?
  value = session[key].join(', ').html_safe
  session[key] = nil
  return value
end
...
this can be simplified to `if flash = $('#flash-container')`
Maybe put the image on Video as poster, or delete poster from the model if we are not using it.
why the string 'block' ? 

<p id="p16">...</p> or <p id="paragraph-16">...</p> reads better IMHO.
I'm not sure this will work with short-lived TEST stacks.
Ah ok thanks for the heads up, I'm new to git and didn't notice.  I'll give rebasing a try just to learn it.
leftovers from a test run...
I don't really know to be honest, I just wanted to make a minimal changes, there are many places where that's being accessed, but don't know how that has being used. If you don't feel very strongly about I'd rather avoid further changes as it's going to be thrown away soon.
Maybe Cuddlefish would make more sense
@zaach, thanks for pointing that out. I'm debugging that right now :-)
I wasn't clear. The message in the `TypeError` says "must be string or list" but your code only allows strings.
The use of the word node comes from the push log data structure itself:

`"23046": {
           "date": 1341451081,
           "changesets": [
               {
                   "node": "13897ce0f3a2a70e7b8a9d1cf6b1e939416fe6f8",
                   "files": [
                       "content/xbl/src/nsXBLBinding.cpp",
                      "dom/base/nsDOMClassInfo.cpp",
                      "js/xpconnect/src/XPCWrappedNativeJSOps.cpp"
                   ],
                   "tags": [ ],
                   "author": "Bill McCloskey <wmccloskey@mozilla.com>",
                   "branch": "default",
                  "desc": "Bug 770759 - Add mutable handles (r=bhackett)"
              }
           ],
          "user": "wmccloskey@mozilla.com"
`

I agree though, it's confusing.  Maybe just `truncate_revision()`, I was not sure if the use of node in the pushlog data structure was some kind of mozilla thing but either way it's a poor use of the word node.  Best to keep nodes and edges in graphs.