labofsahil/github-event-dataset-2012 huggingface dataset:
It is true that the fuel core has 'years' here (i.e. the documentation ehre is not wrong), but is there a good reason for that? The rest are singular, and I notice that in the code, pluralize is called if the value it to be >1, thus does this mean core has a bug whereby 1 year will say '1 years'?

I'm sure this comment would be better in core, not docs, but I noticed it here, and it's a convenient place to comment on this specific thing.

Also, sorry if I'm wrong - I'm only just properly reading about Fuel (haven't even had a play yet) so it is quite likely I just don't know what I'm talking about.

Finally, the correct value in the core code here are $timestamp, $from_timestamp and $unit. Is it in the docs differently because it'll soon be changed or just because it makes a bit more sense, and really doesn't matter what they're called?
I'm not sure if this fix is satisfying or not.
Test for line by line commenting
In fact this would be better ?

`return QDir(this->slicerHome()).filePath(QString("Slicer%1").arg(qSlicerUtils::executableExtension()));`

May be `qSlicerCoreApplication::slicerHome()` could return a `QDir`

and `qSlicerCoreApplication::slicerHomePath()` could return a `QString`.

This would be similar to what's done with 'QDir::temp()' and 'QDir::tempPath()'

See http://doc.qt.nokia.com/4.7-snapshot/qdir.html#filePath

In that case, the code would:

`return this->slicerHome().filePath(QString("Slicer%1").arg(qSlicerUtils::executableExtension()));`
Need to chacnge {L_LINK8} to {L_LINK9}
P.S.:  You missed ` ["Done" server-edit t]` in the menu.
Rationale here: -Werror causes clang to throw on the protobuf headers.
Тест с компилятором проходит, в чем же дело, может где то устаревшая Nemerle.Macro сборка
Ah, interesting, that's good to know; I'll make a note of that internally, since our test page doesn't test coming from Tree yet.
I've got a copy of this working in my tree; you can use the r.js tool from require.js to do a somewhat better optimization that just concatenating the files.  Let me rebase a bit to separate it from my yielder stuff (which is just about ready for the world if you're interested in taking it upstream, but there are a few remaining bits I'd like to fix first).  Hang on and I'll send you a commit to look at...
Wow, I just added an issue about this in less.js - fast work!
or put another way, `install-world` does not install THE WORLD, it only installs docs. So this commit removes contrib packages from the installation. I'd like to know if there's an actual reason for this, or should I send in a patch?
Ага, вот откуда это взялось))) Думал, может это я так когда-то накосячил :)
ok. du coup il faudrait dire de faire attention quand tu changes un tag ca change aussi son namespace.
Fixed compile of gist and updated to recent version ;)
oops good catch :p
yes, i haven't worked on find button yet. i want to do some work on it tomorrow.
It is not just a cleanup as the intent is to be sure that in case of an IO setup error, SLURM_SUCCESS is still returned (to prevent the node from being set to drain). I needed to ensure the return of rc instead of SLURM_ERROR in the fail* label and as a result had to do some modification to set rc to SLURM_ERROR where necessary.
warning: conflict commitet in test!
think I have cleaned it, but buggered if I know for sure :D
Damn, I should have bargained for more! Should we convert all our julia source to 2-space, or just make it the default in the various editor modes?
That's a possibility.

The time being, what envisioned was to add PythonQt and QtTesting be
installed the same way CTK, VTK, ITK are.

Later on, we could add an option to CTK:   CTK_INSTALLL_EXTERNAL_PROJECTS
(disabled by default) that would take care of installing the external
projects.

Jc

On Sat, Mar 10, 2012 at 12:44 AM, Julien Finet <
reply@reply.github.com
> wrote:

> If I understand correctly, SlicerBlockInstallCMakeProjects.cmake wouldn't
> have to be changed, because it already contains the Install of CTK. and the
> install of CTK would install PythonQt and QtTesting, right ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/finetjul/Slicer/commit/15a87ba607fab576273eadf1bb45f0edbf1a3d25#commitcomment-1068549
>



-- 
+1 919 869 8849
шипы не единственное место :( 
Trial of the Champion, The Eye of Eternity, Kologarn, Ignis the Furnace Master может ещё что то забыл :(
Caching id to name makes sense. I still don't see value in caching name to id.
Yeah, that's a good point. This used to be the (broken) fallback for integer `fld`. I guess having these just be `trunc` and `floor` is probably best.
ok...muss ich noch mal schaun, hatte damit aber eigentlich nie Probleme
sorry, not the lazy, the 6 ports. they could step on each other inadvertently.
Can this be replaced with a wait for user input?
@PSZ правильнее R2 community, а не, как по представлению некоторых, тим с злобным тимлидером rsa, бегающим и зыркающим из-за плеча, чтоб в косынку не играли
mmmm nice info :P
I'm trying to revert one by one until I find the hell commit!
First line should be a quick summary, and then a space and then longer description.  e.g.:

    """blah blah blah.
    
    foo bar, foobar.
    """
I'm adding specs for all the bugs you found with BOOLEAN values along with the modified versions of your fixes; I'll commit it soon.
@brockhaus 
Do not forget to backport this $order/$where thing in the 1.6 branch too, if this really was a bug...
Yes. I forgot it and I do not know what to put it. I will add commit with it replaced with the actual error message tomorrow night. 

Thanks. 

Best Regards,
Andrew Ying
Sent from my iPhone

On 22 Feb, 2012, at 10:59 PM, Ángel Guzmán Maeso<reply@reply.github.com> wrote:

> Also, it is good to mention that lxException don't need underscore for the message. You can put "Forwardaddress invalid". It is another big clean change to make on a future
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/andrewying/kloxo/commit/a3558dfd59c59c243faa14b6b2295f1d3bc1c527#commitcomment-996372
`Addon.objects.filter(pk=9999)` will always return an empty queryset. we should change this to use `get`.
Changes made to CassQuery.java are good one.

Cheers,
Vivek
This location is bad. Whoops.
That looks not yet to be the correct solution: If I try that, it will generate an output like that :
(There are too many of the backslashes now.. still testing the correct one)

/pnp4nagios/graph?host=$HOSTNAME$&srv=_HOST_\\' class=\\'tips\\' rel=\\'/pnp4nagios/popup?host=$HOSTNAME$&srv=_HOST_
They are not valid anymore.. I had removed the corresponding keys from github earlier.. I'm not sure if that's still a threat. But then again, it's wise to remove..
They are actually already there, they are just in the wrong spot, they need to be moved to the docs/schema dir, however this makes the validation tests fail as http://www.jboss.org/j2ee/schema/jboss-ejb3-spec-2_0.xsd is not present, so that will have to be moved in there as well.
Instead of .Count() > 0, use .Any().
Great idea Robert! Thanks!
You should define the dictionary of querystring parameters before you make the request to avoid code repetition.
this comment is wrong. `Editor Tools`.
333333333333333333333333333333
does anyone use piwik? i was thinking of remvoing it altogether
I like to keep things running as much as possible despite config screw ups, but your right that it could drive someone nuts if they put something in the config and don't bother to check logs for why it's running the way they don't expect it to.
I don't understand this line…

```service['hostgroup_name'].nil? && service['hostgroup_name'] != 'all'``` will just evaluate to true or false; either way, ```@service_hosts[boolean]``` will likely return nil, and the unless will pass.

What am I missing?
it's only relevant to IE and safari though (where we don't serve the custom file upload button) - i am tempted to remove it globally but I trust potch's suggestion that it fixes the click targeting issue
@tkissing: `1/0` always produces an `Infinity` value. Similar to `undefined` and `NaN`, `Infinity` is a writeable (in most cases) global variable that initially contains an `Infinity` value.
zomg I suck.
onficus убери. нафиг не нужен
This one is not yet finished.
I don't have my Hackintosh here to test it. I've tested it with the latest SDK on Android only. But the behavior should be the same.

https://wiki.appcelerator.org/display/guides/CommonJS+Modules+in+Titanium
Could you provide more details on your case? In our IE8's it doesn't have it.
What about this? :>
http://stackoverflow.com/questions/24886/is-there-a-performance-difference-between-i-and-i-in-c
Putting the archives directly in the repo would also make it an "online installer, with a cache".  You populate the cache when you clone the repo.
FWIW, Flexmojos 4.2-beta released!
Hmm. Make a couple of HTML files and test it? Haha
I am actually still very confused on this one. The double slash when within brackets should represent a slash. May have to ask Rod what is going on in there.
So, what do?
The result is passed back as a tuple; I figured this would make it easier to be backwards-compatible, in case someone has changed their own Triggerable subclass which continues to just pass back the result only.
The / char. With the htmlentities the Template lib don't damage the / char In the string 

Ricardo Sismeiro

No dia 05/01/2012, às 01:48, Pedro Mata-Mouros <reply@reply.github.com> escreveu:

> What does this htmlentities() fix?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rasismeiro/cintient/commit/1b0360d52a50f03fbda008579ec019571779015f#commitcomment-838963
Wikipedia :P
yeah fisted is right. disk cleaner software market is already too saturated for such an amateur solution to succeed. competitor products already have over 9000 niggabytes of useful disk cleaning functions and this solution is just too small to convince customer he purchased something good. but worry not, marketing dept has an idea: just make the installer tell the user that this software needs space to install that equals to size of user's hdd. user would have to delete everything manually to install the precious software, and when the software product finally installs it immediately deletes itself, thus finishing its job. that would be a true think outside the box epic innovation that would mark a start for a whole new era of disk cleaning.
Again s/handle/Logger instance/.
I don't doubt you, but that would leave the question of how *my* application is working!  A simple SQL script should quickly verify the truth regarding regarding INSERTs and NEXTVAL with this version.
No it is not a public api - I should mark it private and move somewhere.
This is actually superfluous. Setting no default for a boolean parameter is identical to setting it to false.
I thought the atom:link self reference was already solved. Please, check on http://www.yiiframework.com/extension/efeed/ RSS 2.0 example

```PHP
// * self reference
$feed->addChannelTag('atom:link','http://www.ramirezcobos.com/rss/');
```
Why is this necessary? #when_ready, #presence etc should not be private
Why change this from private to protected?
also, the permalink here doesn't make much sense, as it's overridden by setCenter()
ahh un éxito =D
You seem to have a good understanding of how CSS styles can be nested inside one another.
Cool :) Too bad it has been reversed in the next commit, but it's the right approach :)
Robin, thank you for the fix!

I hope we could clean this up to merge into fairwaves/umtrx quickly. It would be very helpful for the process if coming changes were commited in logically complete chunks. E.g. actual USRP 2/N2x0 fix separate from deletions of accidentally checked in files. It's much easier to work with smaller commits.

I think this is a good time to use "interactive rebase" to clean up the history of this branch by merging and splitting commits. You will also have a chance to drop changes to files permissions during interactive rebase.

PS Btw, you could use the following link to check the state of changes in this branch regarding the parent fairwaves/umtrx branch:
https://github.com/chemeris/UHD-Fairwaves/compare/fairwaves%2Fumtrx...closehaul%2Ftiming-fixes
This should be ```<a href="http://www.wacom.com/en/Products/Bamboo/BambooTablets.aspx">``` otherwise the link doesn't work.
Why bother with such complicated stuff when we only have 6 chests which behave like this. (4 in ICC and 2 more in ToC)
Let's stick to one of the options posted above and that's it.
that sounds like an issue you need to take up with him or at skyfire this is a tc repo, 
and nothing was said about that here. :/
compile warning

warning C4351: new behavior: elements of array 'PlayerTaxi::m_taximask' will be default initialized
"Will you stop-- " Is it intended ?
There are other build errors
Sorry about that. It got left out while trying to get a bunch of other stuff working. I would love to hear about where and how you are using it. :)
I know this is just moved from elsewhere, but it looks as though ```preservesValue(existingType, newType)``` is semantically equivalent to ```preservesValue(newType, existingType)```, so this conditional could be collapsed into one block.
trace see above
underTest.asSvg() is used often - maybe it is worth extracting a method that does that (as part of the assert method)?
That doesn't matter - because it is invalid the CAS will fail later on.
Do you mean UNIMPLEMENTED; ?
Ahh thx, I forgot to add that
System os ?
Hi Darryl,

Great work! I've been trying to find time to do a codemirror definition, FWIW, but in the meantime I thought I'd share my own test document with you, https://github.com/csirac2/src-highlite-foswiki/blob/master/TestCases/Test.txt

Yours seems more comprehensive than mine, but I notice you're not covering $percntDELAYED forms

Cheers

- Paul
This won't work in JBoss as the `config` folder is in `nuxeo.ear`.
Ah, gotcha, misread the production code.
I got it now, I'll commit the fix tomorrow.
What you were supposed to be documenting here is not the links you created, but the possible values for `rel` attributes on those links. For example, a `rel` value of `researcher` would indicate that a link points to an individual researcher resource.  See pages 109-110 of *Hypermedia APIs with HTML5 & Node*.
Cool, it felt heavy too. I'll swap that out tomorrow.
On Jan 5, 2012 4:06 PM, "Jeff Balogh" <
reply@reply.github.com>
wrote:

> I don't think `asyncore` is the model of robust async in the python world.
> We'll probably have better perf/scaling with something like gevent.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mozilla-services/bipostal/commit/6bb0bd86ea6173a87bf11ac74fc882943c369167#commitcomment-842182
>
The little flashing bug you see is because you fade out *all* the images. You don't really need to fade out the one you want to fade in.
That should go on line 413 (there's something similar there). The function setEscape() would indeed be removed without replacement.
That was a mistype. I meant to say "input field or combo box". Here is an example.
http://jsfiddle.net/metric152/s9kST/

Steps to reproduce the error
1) Open url in IE8
2) Right-click the "right click me" button
3) While the context menu is still activated click into the input box above the "right click me" button

Result
The context menu stays active.

The patch I'm submitting fixes this issue by using an IE specific opacity filter and setting a background color on #context-menu-layer. This fixes that problem.
But you said immutability was pretty and useful here! What changed?
`install -d -o $user -g $group $pid_dir` would be shorter, but I guess this is fine too.
Scary `exit`. Just be sure to document this.
@etr what do you think about the final result?
ok fixed in this commit: https://github.com/railslove/showoff-presentations/commit/76b380675f0488a3d00a8c457af633ef5fa95203
There isn't.

But it was quicker to setup for the moment. Ideally we will have nicer looking topic names like beer@domain.com but until someone comes up with a nice prosody/other servers patch to check if a user exists called beer, this is the best solution.
TODO: A better solution would be to detect the distro type and set the path of lockfile accordingly.
Yes, in that it breaks Python version comparison.  You are right, just for the wrong reasons.
Yes, I still have distortion. However, I have taken this from another device and haven't yet tried to tweak any of the EQ values. Feel free to give it a shot yourself (and be sure to share any positive improvements! :))
It would be nice to make the env variable parsing more robust, e.g. if the port is not included, default to 443
Null-safe traversal
@ey-pairing - yeah, Fog#defaults might be more sensible/meaningful.
I can't see anywhere this function is called in Atlas. Have you ever found it useful in your investigations?
It should be protected+virtual
I'm fairly new to pip testing, but a few more thoughts here:

1) ideally, the upgrades should be working against a local --find-links url (with --no-index) to some local test packages (in the spirit of #259) then we'll be sure of a predictable outcome, and we'll know what the version upperbound is, and then we can just explicitly test for that.   not that it helps immediately right now, but I actually opened #598 for myself yesterday to localize our use of INITools across the test suite.  I'd be willing to help set up the local packages for this if that's needed.

2) as for the `_find_package_version` method,  I think we should defer to how pip determines pkg versions internally, and pip defers to pkg_resources distribution objects, and we have to construct those in the context of the run environment. so I think it means doing something like this:  
   `version = env.run('python', '-c', "import pkg_resources; print(pkg_resources.get_distribution('initools').version)").stdout.strip()`
 and then if you want the string to be a sortable object, do this `pkg_resource.parse_version(version)`  I'm curious to hear other's opinions on this.  A test helper for this would be good.
"will be treated as an invalid file path" might be better than "will be an invalid file path"
I could certainly do at most 4 language files for which I can vouch for quality, but the rest, it would be just me using either google or bing translate. Not sure that is of any help. 

I have actually been thinking about this issue. Here are some suggestions:
* Ignore all other language bundles other than English. Have adopters maintain their bundle of interested based on the EN file. 
* Work on a maven-translate-plugin that automatically fills in missing language keys based on EN file, assume the quality risk. 

I have actually been working on a small little language plugin for maven that does just that based on the bing API. It has the ability to report missing keys, fill in and translate missing keys and optionally break the build if there is anything missing.
Makes sense, I will adjust it.
Seems strange to me, but I have no explanation unfortunately !
Would it make sense to split this test in 4 so we immediately see what goes wrong if one of them is failing? You could make the base set `m` be a fixture.
I don't think this will work.  We need to store each named config in it's own file.  All of these named configs can be compiled into the master config in the parse_config code, but that should happen later.

Storing all configs in a single file does not work because it does not allow us to push configs independently from each other.  If we want to push a named config, we need to be able to retrieve the existing config by name, and return it (this is done by tronfig), so we can diff the changes.

This should also made the transition a lot cleaner, because you don't have to check for config_name.  Validation like this shouldn't happen outside of the validation objects in tron.config.config_parse.

It will mean that the tronfig command will probably need to change as well, and accept a 'config_name' parameter.
Added a test that will start failing when this bug is fixed in the Scala Library
different ticket, possibly: would be nicer with proper extractors
I prefer Unsafe since it performs about 5-15% better than ARFU in Scala
ok, this is called pretty often, so it should be rather optimized, I'll change to the `match`
True! Nice!
yay for less hard coding
This function also duplicates a lot of code from its siblings in SVN.pm and CVS.pm. Another candidate for refactoring. In this case, perhaps we should split out several parts of it into separate functions, though...  E.g. this first part could be in a userpass_prompt() function.
This comment is incorrect, its not just for modules... will fix
Nitpick: I wouldn't say "might", because it most certainly does depend on the surface geometry. I would just say "The actual definition depends on the type of surface geometry."
perhaps an admonition for this.
I get your point, all I wanted to point out is that removing this line and adjusting the database code to not alter the CFileItemList's path will break the advanced filtering completely so we'll have to come up with a different approach that covers both use cases.
Та же причина, что и выше
No, it indicates that we instantiated the runner to use the fs, spin off job flows, etc.
Do you want to try moving the unicode character to the end so it's the last thing before the newline?
Can you add a good error message then?
Different passing to render, :json => vs. :text (and by extension we might have :xml=>, etc.).
conflicts with [copylinesup](https://github.com/ajaxorg/ace/blob/e689aae1759b9b04d9681babc0cc83722e570b50/lib/ace/commands/default_commands.js#L373)
Why remove this default value?
This is a no-go! This breaks backward compatibility! Your new parameter has to be the very last argument to be passed to getChildRecords!
I am not sure we need this? I think we can just edit the existing static 404 file to be more helpful and less generic? https://github.com/coshx/coshx/blob/master/public/404.html
@dlsniper the cache for ``getSourceCache`` is useless IMO: the source of a template is only loaded once as the compiled template is then reused. So the caching of the source is just useless memory overhead.

``hasSourceCache`` could indeed be useful as ``exists`` could be called several times``.
According to `https://developer.apple.com/library/mac/#documentation/Cocoa/Conceptual/CocoaDrawingGuide/Paths/Paths.html`,
perhaps not.
Eek, I generally don't like the `id` and the `<protocol>` being right next to each other, but the `if` and the `(statement)` together.
It was my pleasure-- I swear I'm ready to rip my eyeballs out of their sockets after reading some of the weirdly written code in here. Variable names too. `o` and `p` sure tell me a lot, don't they! :D
I've noticed this before. Why don't we abort() here? It's always a bug. There's an assert() further down below but that only works as long as NDEBUG is undefined.
格式：文件尾部多一个换行
Right!  For some reason I thought that may be numpy/gzip/... somehow could reopen this closed handle later on to perform the operation.
Since that is not possible (?) per se -- I removed majority of the changes and that boiled down to smaller fix (should I squash?), but because original commit which introduced all this logic 8630830bb05bd13aab0fcd869c8339c1515460de  was not accompanied by a unittest I was not sure if I am creating a regression here.  I have added a rudimentary unittest for it (I will push shortly as the part of this pull request)  but I failed to trigger it on my box even if I removed explicit .close() for own_fid = True.  So -- something for travis  to check I guess
Yes, it would be nicer in my opinion.
Yes it is, [the pattern is `%{}`](https://github.com/svenfuchs/i18n/blob/master/lib/i18n/core_ext/string/interpolate.rb#L39) :)
Whoops, sorry. I'll fix.
Ah it also specifies :cols my bad, will fix.
Didn't know it existed. I spend my days in RSpec. Will update.
+1

```ruby
>> 'holy cow'.pluralize
=> "holy kine" :trollface:
```

(This probably deserves its own issue ticket though, in case someone wants to make a case for keeping it.)
No.  Thanks for asking though.
it would be better to simply replace `typeFun` by `genPolyType`, which accepts empty type params
Could you explain this part please?
I think the comments are partially out of date concerning the reference to SI-278 (not sure about SI-320).
Should probably point to https://issues.scala-lang.org.
space after comma would be another refreshing change, 
I never did a pep-8 check
I thought all these were verified during the build... my apologies because I knew these were your standards.
read_many. expanduser?
Could you define what the added argument `alias` is for?
I have not addressed this yet on the newly updated PR.  Does this PR need to have it or can this be in the next iteration, when I fix **kwargs.

So, should I handle this add assume basis set could be passed with **kwargs?
The base and derived dimensions look as if they are more general than MKS. Should they live somewhere else?
Why do you call sorted twice?
Did you run across any issues when removing this?  It was actually necessary at some point because (if I recall correctly) BlockMatrix used Matrix routines which assumed that they could sum elements onto scalar 0.
Thanks :heart:

I'll move them.
How about using $generator=Yii::createComponent($this->generatorClass); ?
Yes, I was thinking on that too... but as the "js:" is deprecated and not advised anymore... by having that parameter.. and especially with the name "keepBC" we are emphasizing that it's something to think about.

But I'm OK even if we remove it, too. I leave that to you to decide :D
I copied the other sample, so I'll need to go back through the other an replace it too :)
Actually it should be app/View/Layouts/default.ctp
Couldn't find a way around including `&nbsp;\n&nbsp;` since that's what `_commentbox.html.erb` does. Just note that in the future if you decide to use margins, this will need to be changed.
Add a newline here...
You mean boolean to other?
Это он типа на window.ready запустится? А оно когда происходит? Все скрипты к этому времени уже загружены?
Since the above pattern repeats quite a few times, could we introduce a "FixedDomainCredentialsLookup" class, that just always responds with the same credentials whenever called?
s/zoom cache/aggregate cache/
Do we really want to allow two types of authentication for a given URL? I would have though the ID here was actually the domain and url.
we use angle brackets to indicate user-supplied values.
We'll want to wrap this in {% comment %}  {% endcomment %} here, too.
nit-pick on the indentation here.
I think that this naming shame is not a good idea. This will only work the the linker "c".

I would make 3 ops for each direction of transfer: the old GpuFromHost, and 2 new: GpuFromHostStart, GpuFromHostWait.

The new ops, will have 2 inputs/outputs, the current inputs/outputs, plus a new Generic Theano variable. This support all python object. In the c code, you should wrap the Event into a generic PyCObject (http://docs.python.org/release/2.7/c-api/cobject.html), don't use the PyCapsule as they don't work on python 2.4 that we need to support for now.

This will make it work correctly with any linker.

I would keep the current insertion of the old GpuFromHost as now and I would not change the optimization. I would add a new optimimization after all other optimization to do the transformation. For example, maybe we don't want an extra op to transfer a scalar. This also make sure the current stack of optimization work correctly.
the name of the login button should also be read from language, probably make language in general available as $this->lang in functional tests, so language vars can be used in places like this or th success test in the logout test.
For historic reasons. All develop-olympus code currently uses it. I have no other explanation. We could probably do another try without it.
Fixed (locally), its event to add normal settings to a forum. Inside the normal settings block on the page. Its where settings generally go.
The initial thought was to have it in this file for the default avatars and let extensions handle additional avatars. I can move those settings to the drivers though.
I agree. Decode sounds like a better name, since we do no escaping.
k, gotcha. I'll fix that.
Brackets here need fixing.
The canonical practice after receiving a post is to redirect the client.  See here: http://en.wikipedia.org/wiki/HTTP_303.  This means if the user refreshes the page after the POST, the browser doesn't accidentally send off another POST (not a big deal in this case, bad if the user is making a purchase, etc).
Can you change all of this to 127.0.0.1? No one else has your IP setup.
Perhaps check for ENOFILE here just to be explicit on the thought process?
nu e get_modifiers(self), e classmethod, get_modifiers(cls) -  vezi core.game sau core.app
Hm, not sure, but it could be like
typedef struct Struct {
    // ...
private:
    int padding;
public:
    // ...
};

Will google for better solution
I'm all for that.  I'll add the TODO.
I don't think this is a good example, because getAppConfig is bound to the current context, instead of the static configuration from master. And in fact, the log level setting was already set. I think it is better to stick to Y.config.logLevel as a way to know the level at any given time at the server side.
why does this constructor have this if(n) but not the others?
Strange. Do you know why http://code.activestate.com/recipes/576694/ doesn't have the license there?
Это нельзя сделать через routes?  а не прямой конкатенацией урла?
это лишнее. лучше сделать validate_presence на это поле
Agree, but whats more obscure - using one static method that won't be changed anymore, or redefinition (copy&paste) . 
It is needed to get only id of CI object 
venture_ci_id = CI_orm.get_by_content_object(venture).id

then it is used "normal" frozen model. 

venture_ci = CI_frozen.objects.get(id=venture_ci_id)

I think its best solution for here.
pep compatible??
This code is useless. Just take what the http plugin already found.
Напиши честный jsdoc
remove Permission. prefix
it should be ``notLike``
Assuming the imagined scenario is of distributed post-processing is actually necessary, how many tasks would you anticipate to be in queue at any given time? For various sizes of calculations, can you give me a rough estimate of how many tasks? 10s? 100s? 1,000s? 10,000s?
There's a space before and after a param (to follow the existing GitHub style).

`function htmlEscape( str ) {`
mostly the same with the intro vimrc. 
except adding the loaded check.  and `vundle_local` option intro
what I usually see is 1.2.3.alpha not -alpha (e.g. [rails](https://rubygems.org/gems/rails))
Yep, you're right indeed. Nice catch, thanks.
I agree eclipse shouldn't read an ant file.  Also, we would like a global configuration for all of our projects if possible.

As an example of what we need, with checkstyle, we have one checkstyle.xml file checked into version control with all our rules and settings, and both the plugin and ant task are easily set up to use the same config.  Something like that would suit me fine!

However, for just a couple of settings, it's not the end of the world, just a "would it be possible" type nice to have.  You've done so much work already, this is really a minor point! =)
Yeah that makes sense i'll update it.

On 31/03/12 21:01, Joe Groff wrote:
>> +[OP when BoolAssoc?(OP)]
>> +overload boolOp(a, #OP, b) = OP(a,b);
>> +
>> +[OP,T,U when BoolAssoc?(OP)
>> +    and integerEquals?(countValues(..CallOutputTypes(OP,T,U)),0)]
>> +overload boolOp(a:T, #OP, b:U) {OP(a,b);}
>> +
>> +overload boolOp(a) = a;
>> +
>> +
>> +inline overload infixOperator(..args) = boolOp(..binaryOp(..args));
>> +[OP]
>> +inline overload infixOperator(a, #OP, b) = OP(a,b);
>> +
>> +[OP,T,U when integerEquals?(countValues(..CallOutputTypes(OP,T,U)),0)]
>> +inline overload infixOperator(a:T, #OP, b:U) {OP(a,b);}
> You can have a single overload  `inline overload infixOperator(forward a, #OP, forward b) = forward ..OP(a,b);` instead of separate ones for no- and single-returns
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jckarter/clay/pull/251/files#r630098
The instruction says to enter "1.224323423452352345345345634534634634", not a sequence of 25 1's after "1.".
The "int" variables have been changed to int16_t variables.
s/coorected/corrected/
Relocatable means this extension can be moved from one schema to another (ALTER EXTENSION madlib SET SCHEMA another).  As far as I can tell, the hard-coded (with substitution of m4) schema is all over the code.
`if (!modalToggle || !modalTogle.hash) return`
The JSON-RPC 2.0 spec says you can only use either one of them. The spec is linked in the class docstring.
This method name is a bit confusing. Could it be renamed to `describe_clt`, for consistency with the method below? Or you could replace it in its call site with `Xcode: #{xcode_version}#{' (Command Line Tools installed)' if MacOS.command_line_tools_installed?}`.
Blank lines between depends_on, fails_with, and def install; pkg-config dep goes first by convention; fails_with should have a "cause".
Please revert this line as it breaks the ' (area doesnâ€™t) for me.

You can do this, with just committing a new version to your repository.
Having to enumerate all the paramters is both error-prone, makes it difficult to extend the other functions, and it hides the differences between the calls to _read(). I wish there was a method to get just the args, but there isn't.
"is consist of" should be "consists of"
We've been experimenting with a few variations on this format recently. I think we've settled on that we don't see a need to mark the different types with comments (that goes against pythonistic duck-typing in some cases anyway) and simply sort alphabetically. Additionally, for multiline statements that are more than 2-3 lines, we're preferring a format more like the following:

```from foo.bar import (
    abc,
    def,
    ghi,
)```
No need for using serialization to Base64
This is a reminiscence from the original implementation. I thought it could prove useful, but now that I think about it's hard to find a scenario in which adjust_request_args is insufficient.
Binds a unique thing, rather than binding elements that combine into a multi-binding.  Better name?
Yeah, there's a bunch of 'em still to uncomment:
https://github.com/square/okhttp/blob/33a0c620e1c1997ece0ee3382ad3e2145ee061a8/src/test/java/libcore/net/http/URLConnectionTest.java

I commented out these tests when I split okhttp off from HttpURLConnection. Now that I have TLS, I can put them back.
Replace ``<COPYRIGHT HOLDER>`` with ``Steinwurf ApS``
because server stops responding when you send block list
maybe you can find another way, but I couldn't
id is the only thing you get in response
there's no other way to check it
`up` instead of `p`, also in the next-to-next line
Ok. I'll take this out. I thought to make clear what things I added. But
you are right, git does this already.


On Thu, Oct 11, 2012 at 10:18 AM, Carsten Urbach
<notifications@github.com>wrote:

> In linalg/lapack.h:
>
> > @@ -43,6 +43,11 @@
> >  #define zlapcy CLAPCY
> >  #define zgetrf CGETRF
> >  #define zgetrs CGETRS
> > +/*--abdou-- begin */
>
> I'd prefer to not have such comment lines... Thats what we have git for...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/urbach/tmLQCD/pull/6/files#r1815639>.
>
>
`this` is not what you think it is here, unless we're `.call()`ing it below but I dont see that
Unlike photos, videos/news should work as expected here since they ARE a post object, so this looks good for both :)
Can I vote we move all config objects to standard js files.

``` javascript
// config.local.js
module.exports = {
  key: 'value',

  // hey! Look I can now have comments in here and tell the unsuspecting future maintainer what to expect
  // acceptable options are 'really' 'super' 'insanely' 'oh mah gah!'
  dustan_is_this_awesome: 'oh mah gah!' 
}
```
Hmm. For the array case, perhaps it would be simpler to validate all bindings first before adding them. S'ok with you?
You're right! It was just JSLint trying to bully me around :)

I added _parseRuleList() to clean it up. The diff looks worse than it really is because I moved some functions around so that there's only a single JSLint warning (which I added to the globalVars list to hide).

I also updated a few comments.
Yeah, I struggled with the names, but figured that grouping with "Quick Edit" should help avoid confusion.
I'm a little leery of changing the keys/ids for existing commands--probably doesn't matter for these specific ones at this point, but in general I think we shouldn't change things like this willy-nilly over time.
OK, thanks. I modified my code to deal with the change to the "autoRemove" parameter. Pushing soon.
In case anyone had doubts, yes, gcc optimises trivial short computations like `x/4` and `x%4`, even without `-O2`. For demonstration:

src/dablooms.c:new_salts()
```
void new_salts(counting_bloom_t *bloom)
{
    int div = bloom->nfuncs / 4;
    int mod = bloom->nfuncs % 4;
...

```

the assembly generated
```
objdump -d build/test_dablooms | less
...
000000000040184c <new_salts>:
  40184c:       55                             push   %rbp
  40184d:       48 89 e5                       mov    %rsp,%rbp
  401850:       48 81 ec a0 00 00 00           sub    $0xa0,%rsp
  401857:       48 89 bd 68 ff ff ff           mov    %rdi,-0x98(%rbp)
  40185e:       48 8b 85 68 ff ff ff           mov    -0x98(%rbp),%rax
  401865:       48 8b 40 28                    mov    0x28(%rax),%rax
  401869:       48 c1 e8 02                    shr    $0x2,%rax                /* ">> 2" */
  40186d:       89 45 fc                       mov    %eax,-0x4(%rbp)
  401870:       48 8b 85 68 ff ff ff           mov    -0x98(%rbp),%rax
  401877:       48 8b 40 28                    mov    0x28(%rax),%rax
  40187b:       83 e0 03                       and    $0x3,%eax                /* "& 3" */
```
I see what you're trying to do here but I'm not entirely a fan of dropping stats on the floor without even having some sort of log information that it happened. Also, I think just dropping them is a fairly specific approach. At the very least this should be a configurable behavior. It might also make sense to make these behaviors pluggable. Its not completely totally out of the question (though I'd quite like to not think about it happening) is that we end up switching these queues to be disk based so that we don't slam into RAM limits.
why cast to bool?
fyi...this is moving to ReportFilter shortly
huh. where does it do that? agree that's awkward and we shouldn't allow it.
Please do not commit this file either :-). `git rm -fr dist`
Ok, thanks for this. I need to find another way to check the same thing.
int STRING = 0. Nice.
Did you mean LOGIN_REDIRECT_URL = '/'. 
It's uncommented.
As above, page objects should not contain logic.
Instead of running this `if` statement for every item in this array, why not invert this two produce two code-paths? That'll speed up this approach here.
And "Mozilla would like to know about this issue so we can fix it, and have prepared a report" should probably read "Mozilla would like to know about this issue, and we've prepared a report to help fix it."
Yeah, I can remove this. It was mostly for testing while I was working on it.
No message?
There's a bunch of whitespace here that we don't need.
We'll need to have l10n for these labels.
git diff ++-- 没看仔细，实际代码中确认存在 $signature = QBox_Encode(json_encode($params)); 即不存在漏字段问题

deadline = current_time + expiresIn 改下就好

然后就是 doc 描述也要改下

样例 php5sdkexample / php5.3 也要同步更新
Sorry. I think I misunderstood the pull request. All of these changes are already in the develop branch?
This is a the daemon name, I don't think it should be upper case.
Got it again:
If ca_certs is `None`, `SSLContext.verify_locations()` dies with:
```python
TypeError: cafile and capath cannot be both omitted
```
whereas `ssl.wrap_socket()` just doesn't care.

This way people, not wanting to do verification had to specify `ca_certs` nevertheless.
null email will fail here. lets prevent potential NPE, even though theory is UI will never pass it.
I did not want to interfere with chomp returning null. If you think that is fine is a piece of cake to change that
yes, they are no longer used.
This could be better written as:
repo_file.write("""#
# Repository for the locally mounted katello repo
...
enabled=1
gpgcheck=0""")
I changed this up a bit.  Please take a look if you so desire.
localize "NOT-SPECIFIED"
Lets change this logic to be false. We don't have to raise anything on timeout anymore.
The above lines actually do alpha blending, right?  You can split it into `opacify()` (I have no idea what its name should be, anyway) that takes an `opacity` parameter.
Why did you remove the 2nd parameter from `setTimeout`?
Sorry. I was addressing comments one-by-one. Did a push after I answered all the comments. You should be able to see one now.
Yeah, that may will be the case - I just wanted to mention it just in case there's another equally-easy approach @mdboom might have that doesn't have that possible penalty.  It *might* be relevant if somewhere down the road we greatly improve the parsing performance or something.
Yes, 0xFFFFFFFF as sipa noted.
Should be `if (` as per the [coding conventions](http://book.cakephp.org/2.0/en/contributing/cakephp-coding-conventions.html)
Couldn't you just do

```php
$element = (array)$element);
```
Better not give the code sniffer reasons to complain :smile:
this does not follow PSR-0. The handling of underscores in the short class name is missing
The same as above `beginning_of_month`.
I see that I should have commented in here, not outside! I am trying to clean up the thrash below so that I can present you with a clean, simple changeset. There are two small changes I would like to make other than simply deleting the debug statement, but I would rather not have you suffer with all the tripe below.
Would that change anything? It will simply load whatever's been pickled. It could be a top level function, but I wanted to keep it alongside `dump_results`.
It seems to me we should enumerate all edges of second graph to find "equals" edge for every edge from the first graph and check their weight. In case if it AsWeightedGraph, we can skip this enumeration because of separated map with edges/weights in graph object. But if its a default weighted graph with default weighted edges (witch stores their wights in itself) we can't do it since we are checking weight of the same edge. Look at the example (I took it from EqualsAndHashCode.java, testGrapshWithCustomWeightedEdges):

Lets use custom weighted edges:
```java
public static class CustomWeightedEdge
        extends DefaultWeightedEdge
    {
        private static final long serialVersionUID = 1L;
        private String label;

        public CustomWeightedEdge(String label)
        {
            this.label = label; 
        }

        public int hashCode()
        {
            return label.hashCode();
        }

        public boolean equals(Object obj)
        {
            if (this == obj) {
                return true;
            }
            if (obj == null) {
                return false;
            }
            if (!(obj instanceof CustomWeightedEdge)) {
                return false;
            }

            CustomWeightedEdge edge = (CustomWeightedEdge) obj;
            return label.equals(edge.label);
        }
    }
```

And test it
```java
 WeightedGraph<String, CustomWeightedEdge> g1 =
            new DefaultDirectedWeightedGraph<String, CustomWeightedEdge>(
                CustomWeightedEdge.class);
        g1.addVertex(v1);
        g1.addVertex(v2);
        g1.addVertex(v3);
        CustomWeightedEdge e112 = new CustomWeightedEdge("v1-v2"); 
        g1.addEdge(v1, v2, e112);
        CustomWeightedEdge e131 = new CustomWeightedEdge("v3-v1"); 
        g1.addEdge(v3, v1, e131);
        g1.setEdgeWeight(e112, 10.0);
        g1.setEdgeWeight(e131, 20.0);

        WeightedGraph<String, CustomWeightedEdge> g3 = 
            new DefaultDirectedWeightedGraph<String, CustomWeightedEdge>(
                CustomWeightedEdge.class);
        g3.addVertex(v3);
        g3.addVertex(v2);
        g3.addVertex(v1);
        CustomWeightedEdge e331 = new CustomWeightedEdge("v3-v1");
        g3.addEdge(v3, v1, e331);
        CustomWeightedEdge e312 = new CustomWeightedEdge("v1-v2");
        g3.addEdge(v1, v2, e312);
        g3.setEdgeWeight(e312, 20.0);
        g3.setEdgeWeight(e331, 30.0);

        assertTrue(!g1.equals(g3)); 
```
In this example graph g1 doesn't equal to graph g2. But if we won't use enumeration in equals method - it fails, because we will check weight of the same edge.

So I totally agree about "e!=ee" thing, but my point it that we should use enumeration there to find equals (it could be different objects) edges in graphs.
There are always odd things like `--single-branch` that come up and you should be able to implement without hacking the library. A client might want to copy default config over or do other weird stuff. `git_clone` is great, being able to do it yourself if you want to is even better.
I take it this is in preparation for `git_fetchhead_foreach()`? There aren't any other functions using `git_fetcchead` in the external API, so I'm kind of leaning towards hiding this type altogether and implementing the `_foreach` in a way that doesn't require exporting yet another struct.

...I guess that would imply a shitton of parameters on the callback, though. Hm.
Ups, wrong line. This should be GIT_WIN32, no.?
Hrm. The lifetime of the tree entry is currently scoped to the lifetime of the tree object itself, which seems like a pretty natural pairing. I'm hesitant to return a detached tree entry that needs to be explicitly freed. The alternative is, as you say, to make a tree entry cache of some sort and tie the lifetime of the tree entries to a different object. What would that be? I guess we could put a tree entry cache in the repo object itself, but it seems like it would be 99% wasteful and only helpful in a very small case.

Personally, I am content to use the `git_tree_resolve_path` function that I mentioned above. Given a root tree and a path, it returns the containing tree and index of the resolved item. Then the user can do what they want with that (and it, of course, responsible for freeing the tree that is returned). This is perhaps somewhat more awkward to use than a `git_tree_entry_bypath` function and it not directly parallel to the `git_tree_entry_byname` and `git_tree_entry_byindex` functions, but it did the job for me with a lot less code and without messing with object lifetimes in an unnatural way.
Last one should be `www-demo3`
It should only be ` q = Reply.uncached.all()`. Don't do it by locale for now.
When modules export a single value named after the module, you can reduce redundancy and complexity by setting the `exports` object to the value being exported.

In *remote-simulator-client.js*:
```
module.exports = RemoteSimulatorClient;
```

In *pingback-server.js*:
```
module.exports = PingbackServer;
```

Then these `require` statements can be simply:

```
const RemoteSimulatorClient = require("remote-simulator-client");
const PingbackServer = require("pingback-server");
```
Also, the configobj notation allows for multiline entries and the `implementation_list` should allow for newline characters. So:

```
implementation_list='''
    psql: socorro.external.postgresql, 
    hbase: socorro.external.hbase, 
    es: socorro.external.elasticsearch
'''
```
It might help the clarity of the test to also assert that before you make the change that admin is indeed `false`.
Because at this point we have already read ```line``` might as well use it I guess.
These 2 variables are currently unused
In .travis.yml is
```
    - php src/Symfony/Component/Locale/Resources/data/build-data.php
    - export USE_INTL_ICU_DATA_VERSION=1
```

If I run only ``phpunit`` without ``build-data.php`` script, tests will fail due to https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Locale/Locale.php#L258