labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, it looks like this is also done in ArticleGalleyForm and others. Agree that it's low risk; just flagging it.
Yes, labels are stateless and thus cannot ever throw an error.
shouldn't the total amount of characters include the length of end char?
This doesn't touch stderr, so if anything outputs to stderr directly it'll still go through.

You could modify it to cater to stderr output also.
*Was mich zum naechsten Thema bringt, ein solches Feature sollte man besser in einem gesonderten branch entwicklen und nicht gleich im 3.0.0 main. :)*
So sollte es mit jedem Feature, dass mehr als einen Commit umfasst.
Le 02/02/2012 22:36, Jan Schneider a écrit :
> It shouldn't have any effect in Horde 4.

Thanks for the response
We are using Horde 3 : we changed
framework/Browser/lib/Horde/Browser.php

with
> if ($this->_majorVersion >= 5) {
>                     $this->setFeature('utf', true);
>                 }

and dimp is fixed with Firefox 10
-- 
/usr/share/images
http://gr3gr.free.fr
prazdny atribut type znamena co?
And the warning manager for maxentropy can't work, unless I'm missing something. If you do that, no one will ever see the warning. Unless you want to let the warning manager somehow figure out whether it's called by nose or not.
Sorry, @vijaydev fixed in a8da5d82fd2b87c1a34d8ede5343219a82be40bc.
Yeah, you wouldn't believe the stuff that's in the bug database.  (Nobody would!)
Shouldn't there be a test for this patch too?
at least /nbproject/* -> enough
I think move generating www_authenticate header to the followings is good:

* outh_set_static_headers/3,
* outh_set_static_headers/4,
* outh_set_304_headers/3,
* outh_set_dyn_headers/3,

But it may affect something to performance of basic authentication.
Yep, its pointless and this change should be reverted. Old code was better.
We prevent it for install and link too, though we only prevent it if the brew executable itself is not root owned.

There are reasons why users may want to have various brew commands run as root, at least, `brew --prefix` etc. should work if you happen to be root. So we only add this check for cases where the result would lead to broken behaviour.
Quite possibly.

Although are you passing reqs by reference then? As you would also want to
keep track of requirements satisfied.

On Wed, Feb 1, 2012 at 10:03 PM, Sebastian Probst Eide <
reply@reply.github.com
> wrote:

> And, just for the record, this is very much a somewhat hacky solution,
> just to get up to speed with OCaml, and prototyping an interface for the
> tactics.
>
> It seems like:
>
> A tactic signature of:
>
> connection_via : a -> a -> a -> reqs -> (a * a)
> and:
> direct_connecytion : a -> a -> reqs -> (a * a)
>
> could work quite well.
> `a` in this case is the equivalent of the type addressable above.
>
> We can discuss tomorrow or whenever.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/avsm/signpost/commit/24efef821aecc60ffa64512f2886bb43dda3cde0#commitcomment-924229
>
I see that now, sorry.
Katy

On Mon, Feb 6, 2012 at 1:11 PM, Robert Carlsen <
reply@reply.github.com
> wrote:

> I was just using 21 as an example.  - and I was referring to issue #16 in
> the core repo - the issue for creating a cyclus/core readme - which will
> link.
>
> On Mon, Feb 6, 2012 at 12:53 PM, Katy Huff <
> reply@reply.github.com
> > wrote:
>
> > The related issue is number 2 in the cyclus.github.com repository, not
> in
> > the core repo.
> > They won't link across repos, will they?
> > Also, now it seems you've linked issue 21 to this commit with your
> > comment... did you intend to?
> > 21 isn't about readme files....
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/katyhuff/core/commit/8d81d8416b97629b7b6c02553518ebbcc74f1e48#commitcomment-937679
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/katyhuff/core/commit/8d81d8416b97629b7b6c02553518ebbcc74f1e48#commitcomment-937779
>



-- 
http://homepages.cae.wisc.edu/~khuff/
Right sorry, I probably messed up with my tests since ccording to the docs here http://docs.oracle.com/cd/B13789_01/server.101/b10755/statviews_1179.htm#I1020276 , it should be 'ALL_TAB_COLS', but it doesn't work eiteher, fails miserably with
`OCIError: ORA-04043: object SOMESTUFF.SOMETABLE does not exist: Connection.describe_table`

This does not impact me really, it's just that we can not use `DB.schema(:table)` with the Oracle adapter without being someking of sys user. I'm not into Oracle, I will query my dba about this and see if a solution can be found. 

As always, thanks for your quick help.
am i doing it wrong?

```
movgrab "http://www.youtube.com/watch?feature=player_embedded&v=AuX7nPBqDts"

Formats available for this Movie: 

No suitable download format found from 'mp4,flv,webm,mov,mpg,mpeg,wmv,avi,3gp,reference,mp3,m4a,wma'

FAILED TO GET DOCUMENT REFERENCE!
Maybe you need to change the format preferences, or else the url you supplied is wrong
```
I noticed lots of trailing white space in the JS files I mentioned, so I hit the whole webapp with a sed script to scrub it.
toto applique ton new fix ici sur notre git
@Th0ar 那就拆成两页吧~ #136
See http://software.clapper.org/grizzled-slf4j/#using_the_grizzled_slf4j_library
did you mean to do this?
It's because the array is not initialized. The bit was there already.
Yes, you have to delete the old file and then upload the new one. Not worth the effort to try and automate, since it is something we may do only once a month anyways. Alternatively, we can host the binaries on beowulf.

-viral



On Jan 26, 2012, at 11:41 AM, JeffBezanson wrote:

> Wow, that's strange. Yeah, that `i386` in the filename is very confusing :)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/7dca4d7a6225a75e6ff709061b21dffa14301c3a#commitcomment-903195
There are two different bugs here, actually. One occurred with this commit and was a weird Heisenbug. That disappeared with the second commit, however, which introduced a second, non-Heisenbug, which I suspect may be this. Looking into it...
Should be `ARGV.build_head?`

There are also some trailing whitespaces.
вообще-то либы .NET туда не притянуты, потому не нужны для запуска. но оно все равно x32 - без подсистемы вин32 не запустится. можно проверить на винде с отключенной совместимостью.
PS да и можно просто глянуть в заголовок библиотеки хексэдитором :)
no. in this case also be marked spells with mask SPELL_SCHOOL_MASK_ALL (also exists, but not need).
/sign MozGII
It is a mistake, default should return a blank new instance of PushGroup...
adjusted for next push. my apoligies :(
Once we "step out of rule" we can never get back into the original context.
`user.email`? Really?
Hello, is there not sql?
it's a test, but worked without sql.
????77
why comment? remove that line completely

also if opening browser is no longer necessary wijet-launchy gem should be removed from dependencies
PHP 5.3 is now a dep - is the package.xml updated too?
I retract my comment after waking up properly.

I will try to strengthen the testcases (with your help) in this area, so that we can also ensure accuracy in constraints.
I guess so. Closing.
Ok, so I guess it's the same thing for : https://github.com/naparuba/shinken/commit/e2f3f305ae8251b966f7c81b19c6da08f3c0255d#L2R72
this looks good to me - squash it and I think we're good.
Leave this one with me mate, I'm kinda half way through it. Plus when I'm
done you can critique. Not sure if it'll be done by the end of the week,
but it'll be done soon. Fluent search is a must :)

OJ

On Wed, Feb 29, 2012 at 10:59 AM, Jeremiah Peschka <
reply@reply.github.com
> wrote:

> If you don't have time, I can pick this up. I think I see your direction
> with this. If we've got some issues in github, it'll be easier to split
> workload between us.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/OJ/CorrugatedIron/commit/d92c39ba3591aae832109ab2fb19c6bb7a43ee09#commitcomment-1023866
>



-- 

OJ Reeves
http://buffered.io/
SBML is toch een representatie van het hele systeem, niet alleen van de biobrick?
you keeping `email`?
I recall Jonathon explicitly asking for doing changelog in a separate pull request to avoid merge conflicts. 
I'm not sure what's better here or why would it cause merge conflicts in the first place.
I merged your branch and was able to see it. I don't like it. The old version was better: "For _this_ expression, register _this_ listener. Now the logic is more convoluted, and the listener interface has methods which are not part of the listener idiom. The fact that you had to add constraints to the javadoc ("this should be static data") is a symptom of unoptimal design. With the original approach, there was no need to put additional requirements on the client/implementor.

Also, it is unspecified how you handle identification in the removeResolverListener() method. Do you rely on the listeners having implemented a proper equals()/hashCode(), or do you use identity equality to locate and remove the correct listener? This is also something you don't have to communicate if you instead return a control handle.

It is always a good idea to prefer APIs that cannot be used/implemented incorrectly.
Oh goody, Metabase::Client::Simple::submit_fact is broken since 6.02. When will you do a release?
I'm not sure that I do, yet-- I was think I'm attending to make this match Modyllic.
Where is SendShowMailBox(guid). I'm getting a compile error

Player.cpp:12426:27: error: class WorldSession has no member named SendShowMailBox

I did a grep search for this in the code and found nothing
I think I understand...
rm resources/static/include.js 
ln -s resources/static/include_js/include.js resources/static/include.js
commited in
https://github.com/mozilla/browserid/commit/f73961e151009659fff32e8b37eab65aef34cb18

If that isn't correct, please let me know.
already fixed on 530e86fac9023ccef0e5c1b193de4fa5d2d6a841.  Thanks, though :)
Idk, my bff Jill??
See d6535f7f4a1800fc200ab03fa258b3f1cf5e93e3
Emitter calls exact event the was fired.
In my case if I am subscribing on 'aaa', I still need to be notified when 'aaa.bbb' was published.
If I subscribed to 'aaa.bbb.cc' and 'aaa' was published, I also expect to be notified.
This is the reason i need to handle listeners.
Since I handle listeners already, it doesn't seem that using multiple event emitters add any additional value over simple callbacks I need to store. Does it?
Blank line between stdlib require and requiring your own code. `.rb` after your filename.
while I agree it's useful to visualize the resulting file, I don't think that should be what "visit-item" does. After all, visiting the current state of the file is equally useful IMO. What about we take this code and create a separation action to invoke it?
Actually it would be nice to be able to display the original version, the target version, and the current version, when possible.
Nice, I didn't think of merging the 2 docs directly in map.js.
I like it. There's some style changes I'll make and I'll merge it in.
Already defined earlier
1. Remove this entire file.

2. Place the database mods (both the tables) in the following files:
  -sql/database.sql (alphabetically)
  -sql/4_1_0-to-4_1_1_upgrade.sql (place at the end)
why not use sc_hex_dump?
I can rework it to use the mysql cookbook instead?
havent gotten the chance to test that one yet. i still have issues starting the flame leviathan encounter
yes there are no gtk.event.button.whatever.leftbutton :s
Anyway, pushed; thanks for your clear instructions!
Ain't so bad. I vaguely remember some guy who said he was building a web frontend for installing sparks
This is a simple patch which adds a new parameter to create_server, namely is.binary. When is.binary=TRUE, all messages sent by websocket_write go out with opcode=2, so they're interpreted as binary by browsers which understand binary websocket frames.

The control over this should possibly be more fine-grained, perhaps by adding a parameter directly to websocket_write, so I'm not sure whether you'll want to take this patch as-is.
finally compiled it
but what to do it next?
http://dl.dropbox.com/u/35746734/p500uv.ko :P :P :P
Я тоже его предпочитаю, но если им всё без разбору именовать, всё равно хуже читается.
I think this should be `IPublishMessages : IBindEndpoints, IConnectEndpoints`. Senders, receivers, publishers, and subscribers should be allowed to determine whether they are the fixed points in the network topology independently of whether they send or receive.
Read the related issue! Bananeweizen was against using the addon at the moment and the master code is not changed yet. We need nightly builds for the addon, so that nightly-users can use it after upgrading to the latest nightly. I said many times, that I will change the code after having the nightly for the add-on.
Old link too
I tried to stick to what seemed to be a pattern - underscore for the overall container like shop_table and then dashes on individual rows. Even WordPress menus have a mixture of dashes and underscores.
This if clause was removed as it cleared (destroyed) the volatile data pointer in the rule.
doesn't work anymore, neither `http://` does
Just thought it would be quite practical, so that if someone, who doesn't have the closure compiler on his machine, contributes, he'll still be able to minify the script...
This should have node.getLat() here.
err yeah and my commit message doesn't even say this is for the minigun I think I'll retry this.
Liyi WANG emailed this change to better handle Chinese characters.
Slightly tricky since they can also be called when no other threads are active (e.g., in single threaded code, or in the destructor.)
So you should now see that the repo includes an Xcode project. We've been using that under Xcode 3 to test a KSFileUtilities methods. Are you able to add your tests into that?
That, combined with the fact PEX overwrites the "hasPermission" function is why I suggests that NOBODY USE PEX.
hum weird, i don't have any memory about that i'm going to make more research
gamma -> gama
format = format.replace('LL', pad(this.getMilliseconds(), 3));
Note that NSMutableOrderedSet does not have a setOrderedSet: method. Is there any reason you use operations on mutable<...>valueForKey: here? It seems setValue:forKey: should work correctly (supporting KVO) for replacing the whole collection.
Hi Alex,

Basically the bug was over here https://github.com/rightjs/rightjs-plugins/commit/f933b838dfdac32d219f39b6fd0856d58cefba83#L0R106

I've also refactored the code a bit and checked it in my own app this time. Everything seems working :)

Please, give it a spin when you have a moment
Because we are never going to write 1.0 schema files, that's why I physically removed the writer from the JGroupsSubsystemParser_1_0 and InfinispanSubsystemParser_1_0 classes
That's basically the commit I started at:
https://www.codeaurora.org/gitweb/quic/la/?p=kernel/msm.git;a=commitdiff;h=160d42d529f7a5563bbd7cd117c83b7f434f0c22

With that commit the guys at CA merged the changes from the .29 google kernel.
done, thanks
https://github.com/scottkellum/modular-scale/commit/1177336fa2189c6b5dbe2c4a77f059859b3a2ccc
This block could be eliminated by just referring to the existing dictionary from config.
Every public function that takes an object needs to have a g_return_if_fail () guard.
Same thing as above, docstring should explain what this method is good for, not what it does
Joe & Kurt's problem earlier today i think was caused by this.  Alterations to a document have no knowledge of/way to expire their annotation's caches.
1337.  Let me try right now.  Which wifi chip does the U8160 use?
I screwed up my initial git setup, so I tried to correct the commit name/email... some things changed, some did not
Expanded on the attribute passing example

- Renamed directory Attribute Passing
- Added example for basic data types and Time objects
- Expanded deserialization example to be more general (loop through the vars).
Did this change introduce JENKINS-12412 ?
Here's an idea (not sure how well it would work, but you can consider it):

It looks like SQLiteDatabase.update() returns an int for number of affected rows. So why not attempt an update first, if 0 rows are affected, then do an insert? This would save you the fetch at the beginning.
That's a good point! i thought about it at the time and i decided i'd leave it as if it were commented out... but dead code is dead code and that's why we have diffs.

Thanks for the input, i'll update!
CPAN.pm users generally expect to set makepl_arg and buildpl_arg and for that to work.

Before I wrote local::lib this was the standard approach and the general irc.perl.org community advice to people with problems was ".modulebuildrc's precedence is useless if you want to pass other options, if you see one delete it".

Once I wrote local::lib and we used .modulebuildrc the general community advice became ".modulebuildrc's precedence is useless, you'll need to unconfigure everything in CPAN.pm to use it"

No way out is NOT a dramatization, "'PERL_MB_OPT= Build install'" without supply that environment variable to anything else is not something current installers support. If I'm going to have to have to teach a user how to go into the build directory and mess about by hand, it's going to be faster for me to supply them with a faked up Makefile.PL since ExtUtils::MakeMaker is so much saner here.

And now I'm calling MakeMaker sane. That's how bad this is.

As a naive user, I would expect the precedence order to be:

  .modulebuildrc supplied info
  PERL_MB_OPT supplied info
  things I passed when calling Build.PL
  things I passed when calling ./Build action_name

As an expert user, I would expect exactly the same thing, except that when it doesn't work I know how to switch the distribution back to a Makefile.PL which at least sucks in consistent, well known, unixy ways.

The current state is a complete mess, and worse still is a mess that blocks other implementations of the Build.PL API from having sane code because of the expectation of at least some degree of bugwards compatibility with the current Module::Build codebase.

The answer to this is not to document the incomprehensible disaster we currently have, it's to fix the problem. Module::Build has not been afraid of breaking compatibility in the past, and those authors (myself included) who care about compatibility migrated away years ago. Since the damage from incompatibility has already been done, let's please not use it as an excuse not to improve the state of the art not only for current Module::Build users but for all future Build.PL implementors and downstream users as well.
Consider using more readable variable names then `D`, `p` and `q`, especially if used in a teaching environment.
I'm not sure if this is enough. The problem is that such a combination could not be made i18n ready. Therefore either we completely ignore the description for now, or we add a way for the connectors to contribute in various languages at once
I believe this commit is still causing trouble. On the Galaxy Tab 8.9 which have an 800x1280 display (not to be confused with a 1280x800 that the 10.1 version has) the "cursor" now moves further down than the lines being drawn so if want to flash something that is far down you will have to resort to guessing.

The problem is visualized here: http://i.imgur.com/bwYjG.jpg
В mediaOut забыли return ret.
Забыли про getUserOptionInt.
Не совсем понятно, зачем выносить определение i и added в начало функции, это же не C. И, по-моему, break куда нагляднее незаметного условия в for.
Just to nit pick, you dont need the brackets require,include and such they are language constructs, so include $foo; is 'more correct'. I noticed you was talking about speed with require_once. Well file_exists() is faster then is_readable() if you really want to trim down the execution time, I've found its about 3x slower to call is_readable and realistically there shouldn't be a case where PHP cant read the files.
@jaminmc here is what i told you
Mix of tabs and spaces in trying to line these => up. Use spaces for alignment in the middle of lines, always.
これ、とりあえずランダムな文字列にしちゃってますが、モデルのリファクタリング時に `absence` や `attendance` 等の（仕様的に本来）正しい文字列を入れるようにしましょうか。
Yes, I haven't found any other reliable way of determining the beginning of the handler.
So then we have a number of possibilities (using 'activity' as the example)

1. activity
2. activity/owner/something....
3. activity?query=something
4. activity#anchor

Which the explodes on /?# take care of - is there a cleaner way, or are there any other possibilities unaccounted for?
amazing that `sys.props ++` actually update System.properties
"Something less harmless" - I think this is a typo ;)
Move it to RoutedActorRef?
I think it makes sense to use RST in case of exceptions, it is faster than a complete disconnect.
Just cache the seen.values.head first so you don't do it for each member in the Set since that can be expensive.
Is this a safe thing to do?
extends ClusterMessage?
Lookup on id raises an error in fetch when the id is missing.
I'd like to see a test that verifies the behavior in the event both --repo-id and other criteria stuff is specified. The test should prove that it either errors, replaces, or is merged together, depending on how you want to handle those situations. That way if we ever accidentally break that behavior, it's ensured we will know about it.
I guess it could.

But I never know how war's handle this manifest resources.
e.g. direct META-INF/xyz vs. WEB-INF/classes/META-INF/xyz,
where I almost always prefer the 2nd one, as it's bound to work as per spec
Just an aside, we can probably do `logging.getLogger(__name__)` for these.
this header should be pulled in from including rbtree.h...
I will add a test, too.
As long as you're mucking around in TUIViewController, could you remove this, please?
@domenic

> No I meant that if you implemented my suggestion, of changing if (!this._header) to if (!this.headersSent) for readability, then you'd be invoking a > getter, which is slower than getting a normal property. In V8, property > method > getter for performance.
> 
> So all I was saying is that your code looks great, but I had a suggestion, but maybe my suggestion wasn't any good :)

I choose property getter because it does not introduce new field on every instantiated response object.
Move this test to test/internet/
Drop the casts and change the function prototypes, they should return void.
should this be an rvalue reference (std::string &&fmt) or is the code / diff display not working. Also experimenting with the review status :)
Only here.. I thought it would be more "clean" to have it in an extra class
STYLE: OutputSpec class should be "AFNIPrefixOutputSpec"
I think you have to embed the UFUNC_API declaration into the comment below, like so:
```
/*UFUNC_API
 * This function ...
 */
```
Right now, it doesn't build, says Travis.
Why not  fp.read?
Хоть верхняя строчка и может ввести в заблуждение, тем не менее, последний писк моды^W стайлгайда (п. 6.3.5) гласит, что тут нужны скобочки.
I think, it is more like a recommendation than a question.
the prefix `full_` is used to make a link to the existing method `full_messages`. A full error contains the attribute name as in "Name can't be blank". The normal form of this error would be "can't be blank". Using `@person.errors[:email] # => []` you can get access to the errors in the normal form. This PR adds access to the errors in the "full" form.

I think the current name is good. It would be hard to tell the difference between `errors[:email]` and `errors.all_for(:email)`
Unfortunately these two changes are going to break the nested attributes tests that rely on the `ArgumentError` message. Can you check that too?
I guess so. I thought we need RedCloth for `textile()` helper. Maybe not.
This is fine and should be done in sprockets-rails, see 481dac9ec0791d for reference
Is this right? I mean the %{}. Should it be #{}?
how about a simple .uniq!  ?
sure sure... blame me... I don't know what you're talking about re Traversty's docs having semis! I don't see any!

:trollface:
I was sanguine at first, but I'd forgotten that the string is scattered about.  My original edit was to override phaseName in global; but maybe with lazy val phase objects, there'd be a problem at assembly; second idea was just an enum of phase names and descriptions, so at least they're all in one place.  But I haven't read all the comments in firstPhase yet.

I didn't even realize inlineHandlers is a setting.  I can't be trusted.
Oops, used tabs instead of spaces.
It is ask support internals. That is why it is private[actors].
This is indented way too far.
Yes, definitely get rid of the parenthesis on the assert calls.
Actually, can't this be just a tuple? Length is just `(1,0...)` and time is `(0,1,0...)`. I do not think that you need a whole new class for something that can be represented inside `Unit`.
this is ok, too.
Well one thing I don't want to do is to force people either into using a predefined set of heading names, none of which really match what you want, or no headings at all.  That definitely doesn't lead to a good doc reading experience.
`d.premises` gives
```
 {IdentityMorphism(Object("A"), ""): EmptySet(), IdentityMorphism(Object("B"), ""): EmptySet(), IdentityMorphism(Object("C"), ""): EmptySet(), Morphism(Object("A"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g") * Morphism(Object("A"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g"): EmptySet()}
```

`d.premises.keys`:
```
[Morphism(Object("B"), Object("C"), "g"), IdentityMorphism(Object("C"), ""), Morphism(Object("A"), Object("B"), "f"), IdentityMorphism(Object("A"), ""), IdentityMorphism(Object("B"), ""), Morphism(Object("B"), Object("C"), "g") * Morphism(Object("A"), Object("B"), "f")]
```
In my book that qualifies as too long :-) However, I can try to break it down into lines so that it gets more readable.
done.  I added a fix for the slicing error, too.
That's satisfying
@mdomba right. Everything is OK as is then.
It should be backward compatible, we don't want to brake other users :D So yes, we have to look at what we can do to make this happen. I don't remember, why a third parameter was needed?
Кстати, здесь можно не переопределять `begin_of_association_chain`, этот метод мы переопределяли, чтобы привязать ресурс к `current_user` и чтобы замутноватую логику написать с `User.find || current_user` потому что иначе никак. В данном случае рациональней будет использовать метод `belongs_to :user` - это метод `inherited_resources`. Читай README для `inherited_resources` - https://github.com/josevalim/inherited_resources
Alternative for finding file path in relative way.
writes yes, sequential reads. (nonsequential reads end up being like any table scan, but the insertion order is constant.)
that's what my tab complete gave me, I thought it was *a thing*. I'm not attached.
can these be added in urls.py so you don't have to hardcode?
It makes me wonder if trying to enable PaX by default is really a good idea. Do other projects actually try to do this? (I've never seen such warnings when building various software on non-grsec kernels, hence me asking.)
(so, i think you're saying that in addition to the simplification, there needs to also be unit tests that run on unix....)
Needs to check that the user entry is a bot (by user type). If there is an ordinary user with this name the entry should probably not be changed.
Could be decoupled into two independent cron tasks.
True, but it could be used in the future, right?
Escaping? Or use jquery to build the element.
addslashes will not escape newlines.
I am confused about this then because all other functions seem to follow this convention in core.js
yes, this bit isn't finished as I said.
I believe it is there to figure out which checks are being added, removed, or updated.
"...consists of an that validate..."
I'm fine with this approach, at least as a first step. 
Can you redo it node and I'll merge that in?
hook.Add("Think", "Wire_DebuggerThink", Wire_DebuggerThink)
If I add this line back, the test passes again :(
you'd think. didn't work for me.. does it work for you?
Is monkeypatching res.end the best way to accomplish this? Does the "req" object have an event we can hook into?
Опять то же самое?
Interesting...it would seem that these skidrow games are actually circumvented steam games...which means these games should be marked for the steam platform, and placed in the steam.xml file
Using statements should be inside the namespace declaration
The `JsonBodyDeserializerFixture` has not changed since March so it's not new. It's failing because it does not take all valid `/Date` formats into consideration when parsing.
Missing a `//Given` declaration
We should avoid our functions to behave in a way for tests...
We don't need to save the search query for drug_info, either.
Change no longer needed as a result of #175 (https://github.com/rhattersley/iris/commit/cfde4f96ee86bcc199a33e6492d959967ed446ba#L0R879).
I think that we can reuse AdvancedSubtensor1 for all cases (or at least many cases), in particular if mode == 'raise'.
When axis != -1, there is a way to use DimShuffle to put the desired axis in the latest position, and then dimshuffle back.
If mode == 'wrap', I think we can use indices % a.shape[axis], and when mode == 'clip', we could use tensor.clip(a, 0, a.shape[axis]), at least when axis is not None.
Did I miss a case where there is no way of specifying the appropriate computation using existing ops? Or would it be more efficient to have a new Op?
I guess it might actually get big due to the fusion optimization, but usually you shouldn't be taking gradients of an optimized variable.
er, IllegalArgumentException?
"can't ends with" -> "can't end with", "Allow chars" -> "Allowed characters"
Конечно вернуть
Если я правильно помню, `arch.link()` умеет принимать массивы в качестве обоих аргументов
Thanks for the explanation. So how should we proceed on this?  The detection of which promise type we have is important feature. With this we can detect a lot of syntax errors at parsing time.
This is a reasonable suggestion.  Maybe a high-priority issue for post-pull.
Same thing about note formatting as above.
this is wrong. The code is MIT
Can we move `transaction` and `single_result` to `boffo/src/boffo_util.erl`?
So this is exactly the kind of server-side magic I was trying to avoid and why I had `headers/request`. The fact it changes the response content type to json by default is also a big issue, since you now have to be explicit about the content type in your ajax unit if you wanna test headers with a request that doesn't pass through the json converters (or, worse, the jsonp prefilter). It simply doesn't make sense at the unit level (why would setting headers change the default content type?) and tests much much more than what it should in the ajax architecture. So you could end up with headers tests failing because of a change in the json converter or jsonp prefilter, which simply doesn't make sense.

We should keep this script separated and we should avoid using the json dataType in the headers tests. Code is less beautiful for sure but it would actually test what it is supposed to test, nothing more.
Hahaha this is MY FAVORITE PART. <333


I totally expected test changes... Impressive!
Thanks for letting me be a grammar Nazi... this line still doesn't read well:

> HMVC framework built using PHP5, by a team of volunteers that aims to be swift, secure, and small.

This implies that the "team of volunteers" aims to be switch, secure, and small. Lets rewrite it as:

> PHP HMVC framework built by a team of volunteers. It aims to be swift, secure, and small.
Hmm, I wanted to make all Handles aware of their size, including `TransparentHandle`s, which are currently just pointers. But this requires a few code changes, so I commented it out temporarily. I could obviously remove this change for now.
Alternative if you want less verbose code would be ``_track_available = lambda track: track.availability() == TRACK_AVAILABLE`` or some equivalent util function.
By before do you mean like this:

    {# L10n: Short for 1 day. #}
    {{ date_range_button(1, _('1d')) }}

or like this:

    {# L10n: Short for 1 day. #}{{ date_range_button(1, _('1d')) }}
This can use `fails_with :clang` now:

```rb
fails_with :clang do
  build 318
  cause "..."
end unless ARGV.build_devel?
```

and it should take the appropriate action based on available compiler versions.
What to do? I don't understand this comment...
Do I need to omit the "version" line?

Regards,

   -oli-
Should there be an option definition for this up above?
Isn't timeout global now?
( Oui )
Good point.  I think the best way to clear this one out is to un-set the data source for the NSTableView.  I'll add a changeset for this one.
I had a hard time understanding this sentence at first.  Probably because of the world generate being so close to general.
Made LibraryCache injectable object
This section is correct, but I think it might be easier (at the expense of a small bit of for-loop logic duplication) to maintain if we split out the any/all cases:

if (fExcludedAny) {
   for () { ... }
} else {
   for () { ... }
}

What do you think?
If you like; was just trying to make it clear that this was used only by the method above it. (A local class would not work since it would not be static.)
The feature is intended for Java 6+ users only; it requires someone to understand how to use the Java 6 SPI, which essentially means providing a file under "META-INF/services" with  the implementation class name. I didn't see a need to notify the user when running on Java 5.

That said, we could also support Java 5, by adding extra code to do the same thing that java.util.ServiceLocator does under the covers. If you prefer that, it's ok with me. Let me know...
No, that functionality is not present. `html` can go from not existing to existing, but once it exists, changing it won't trigger a render. Reason being I don't want to encourage dynamic `html` bindings because it means a rerender every time it changes. If you need to change the contents you should use bindings.
It shouldn't be— this.appConfig is new, and is only retrieved by the _getRootNodes function, which would not be called on a primary tour.

It's probably safer to make it re-retrieve the data if there's no appConfig, though.
The rules from this embedded style sheet should be moved to an external style sheet.
Done. I had to modify Menu.prototype.addMenuItem() to log and return null rather than throw to be consistent with the other places we handle dup ids
That doesn't fix it. I think the problem is the second parameter to "_projectTree.jstree("sort", _projectTree);". is supposed to be the parent node (not the whole tree). Note that sorting doesn't even work in the project root.

Note that whatever Rename is doing causes the files/folders to be re-sorted.
Yay! I've been wanting to remove these and keep forgetting.
For consistency, I think the called function should clone what it needs to manipulate, so just pass ctx here.
These lines are a work-around for appReady being fired before the document is loaded. If issue #1526 gets resolved, these can be removed.
Should throw an error here instead.
なおせやこら
Maybe its too early for this but I am curious how much overhead regexing every query that goes through adds. Is there any way to pull the info out of an ARel or something like that?
I believe that the XMLRPCRequest should take ownership of the encoder. It would probably be wise to retain the new encoder and release the previous encoder. There should be some decent example of this pattern elsewhere in the framework.
Such lines are way too long... It would be nice not to have lines that have more than 100 characters. I know this is not always possible, but these ones are, for instance, easy to break.
Class names has to start with upper case letter!
What does this do, actually? Why do you need to unlisten first?
Mixing quotes.
*played
Again, do you really need this DOMContentLoaded?
The indentation here is a little wonky. We could probably just store some references to these tracks as variables here too instead of referencing `butter.tracks[x]`, right? `addTrack` will just return a `Track`.
For safety, I always put quotes around urls. Do any browsers care about that?
You should publish a copy of the stop and frisk project so that the user can view the default one when they click the image
Good point. it's been deleted
Wouldn't it make sense to document what these mean?
would prefer to wrap each of these method blocks in braces
You're right.
Altough paginating stuff doesn't bother me that much and I implemented something on that genre in the [tags search](https://github.com/Nesquik/richard/blob/master/apps/videos/templates/videos/tags.html ) I'm working on, I wonder which solution might be adapted.
Briefly, from what I see:
    * Vimeo uses two columns of videos and pagination;
    * Youtube goes for one column plus pagination;
    * Bing displays a page with thumbnails and an autoadapting grid that scrolls endlessly as new videos autoload;
    * Yahoo Video search seems to be the most reasonable with a 5 x 6 grid and pagination.

Using my screen resolution the last one seems like a good tradeoff in terms of height and width.
I have to rebase anyway before merge so will clean this line.
Minor note, often a visual treatment of surrounding an immediate function definition with an outer set of parenthesis can help to indicate that it is an immediate function especially on long module definitions.  I.E. taking function () { blah blah blah }() and doing (function () { blah blah blah }());
is this path.join? if im doing this in here we probably shouldn't, poor windowz users
Get rid of this. Just add a virtual Dispose.
The predicate which determines if the operation should be retried
in this case it shouldn't matter but the solvables start at 1. the zero index solvable is internal to libsolv.
holdstart/holdmove/holdend. I was planning to fire contextmenu event from the platform after a certain amount of time: https://bugzilla.mozilla.org/show_bug.cgi?id=751146 
It does not cover everything you lib provide but this is very easy to do with contextmenu, mousemove, mouseup (I have a prototype for the homescreen with that - that crash gecko sometimes btw). What do you think?
This is a test, so not really a big deal... but nominally shouldn't this be `FakeUncertainties`?
Good point - parent_nddata is now set exclusively from NDData
bugger, i totally didn't mean for any of those cherrypicked files to make it in to that commit. just being a little careless. i'll fix that.
not to be a stickler, but this is pushing too much layout into the html... if it looks weird, should be a CSS fix.
I like boost's naming convention for this better; see boost/suffix.hpp:

//                                                                                                                     
// Helper macro BOOST_STRINGIZE:                                                                                       
// Converts the parameter X to a string after macro replacement                                                        
// on X has been performed.                                                                                            
//                                                                                                                     
#define BOOST_STRINGIZE(X) BOOST_DO_STRINGIZE(X)
#define BOOST_DO_STRINGIZE(X) #X


Nice comment, _DO_ implies it is an implementation detail...
Is the QNetworkProxy IPv6 ready?
As written above FIRST_CLASS_MESSAGING is not used for me and so I only duped what was there ...
I like this warning,  external docs are good, but hitting people on the head everytime they run this is good as well.
On Fri, Sep 28, 2012 at 7:39 AM, ADmad <notifications@github.com> wrote:

> So use $this->alias
>

Done: UndefinedVariablesFix 549e8dc] this->alias instead of model->alias
`When` is supposed to be used for a particular action that a user is taking. E.g. "when I click this button". In this case, it seems like you're trying to use it to set the context, which is what `Given` is for. 

In this case though, you don't even need this line, because it should be implied (we haven't added the user to the group in any of the previous specs).
They are only relevant for mines, but I don't think we should reuse the same animation for the teleporter.
Nitpick: Can you remove the whitespaces between the if and the parens? This is just noise. Same thing with the few changes in whitespace below.
Just fix the indentation. I'm glad you caught this bug :)
We should still use the same command line naming convention so this would be `--test`.
In fact, I'd put the `/` in the default prefix, and just use `self.url_prefix + self.path`. That way, if someone needs to, they could specify a prefix like `files.php?path=`
typo:  "course" -> "coarse" ?
Use '../lib-etherpad/etherpad'. Also, that doesn't exist :-). The path to Changeset below is also wrong.
Unfortunately, the contents of the checkout lines in the reflog are not guaranteed to be a branch name. In my reflog for libgit2, I found:  `<sha>^0`, `<sha>`, `<sha prefix>`, `FETCH_HEAD`, and `<branch>`
More "store"
The `revparse_state` enum is no longer being used, you should kill it.
what test do u want to add? I will add it with pleasure :)
Shouldn't you then remove line 260?
Clearly, I was misremembering. Sorry!
Although i see the mockup says that :-/. Can we just come up with something that makes more sense?

"7 more tweets in the last 24 hours should be replied to"?
see the previous comment above.  get_raw_dumps is a public function and the public doesn't know (or care) that the these two classes use different implementations.  if I were to try to reuse the public call here, i'd lose the context of the derived class's implementation.
indent HTML at two spaces
It would be nice, however I don't know what number that might be.  I'll note that the frozen django-browserid is already behind the latest release 0.7.1.
Pulling from git is sub-optimal, but it's the only way to get the version that supports unverified assertions.
Nice..  Played on should be None.. that'll be taken care of the player when it actually starts.
Don't step on the title because of a Script exception.
Hey Ryan! Let me know when you can get rid of this - I'm excited to get this into 2.2!
For what it's worth I prefer the TypedBytes interface over HttpEntity

(mostly 'cause I'm trying to break our dependence on Apache HTTP)
це той одноразовий скрипт?
@fabpot fixed
A route like `/resource/name;{param1};{param2}` and have created a path with path parameters by hand. If we encode `;` it wouldn't work anymore.
And it would also break BC somehow as currently this style also works (unintended because no static text is encoded at all). So `/resource/na%me;{param1};{param2}` would create an invalid URL because `%` is not encoded.
Indeed; this is necessary for MongoDB all the time, and whenever ORM relations are used as choices.
you should use ``trigger_error``
Yes, now they are correct, on master they show ^M
why this check on the debug mode before logging at the debug level ? There is absolutely no such code in Symfony currently.
consistent naming (what is the decoder *doing*?): `DefraggingPackets`
error() for error, ok() for successful change, notModified() for idempotent operation that did not change anything.
It looks that the initial " testPolynomial" is no more visible in the CMakeList...
It provoques a conflict that I resolved locally by add you test with the previous one:

SET(QGLVIEWER_SHAPES_TESTS_SRC
  testPolynomial
  testBall3D
  )
Not sure if I get this right. You said that every second frame would be constructed out of the second field of the last frame and the first field of the next frame. This looks like you create 2 frames out of 2 fields. Am I missing something here?
copy & paste error ?
А почему для юзеров формы создания и редактирования в отдельных шаблонах? Так чем-то круче?..
I liked the IDragEvent since it was easier to read, so an i_mimedata.py might be a good idea.
I did look at the previous releases while changing that and didn't find a 1.3 in 7.1.1 release. Of course, I forgot to check the tagged 7.1.2 version. I'll do the necessary changes.
You mean that static setup part? That is used to set org.jboss.as.logging.per-deployment to false as per-deploy logging has higher priority over logging profile or you mean something else I overlook?
Probably needs i18n
yes. the time for waiting the byteman kills the server is 10 * 60 = 600 second. I think it's enough.
It caches the assignment operator `if ( var = val )` typo, which is much more unlikely when using strict equality checks.
Would it be a better idea to add learning rate decay as an optional parameter directly into the SearchDirection subclasses  in which it fits?

If we're to implement the SearchDirection class it would be better to stick with it, or we might end up hacking things into SGD again, which I think is what we're trying to avoid.
I would either use all the different backgrounds rules (background-images, background-position, background-repeat, ...) or either the unified 'background' rule. Let's pick one of those choice and keep it.
Yep you got it.
also- depending how well the DI of other files go I would love to kill a bunch of the require statements in tests.
even 'os' is fine :)
I already got it in my local branch. As soon as this Pull Request is merged, I'll pull the dialer's one :)
Could you try to remove those lines and test it for no js error happening, and then r+=me.
Or you can just leave the sms app's code-removal work to @etiennesegonzac :)
Why don't you store the value of the confirm and use it for the value of hasConfirmed?
ret -> lastIcon
nit: no need for the line break
Yes, this is what I was talking with Francisco, but in previous test that I made It was not working properly... I will check after this pull and replace sorting with 'revert' param if is working!
Remove this line since you are describing the state before this patch, and it will be confusing when the patch is applied :)
why is this CSS animation rather than CSS transition?
`'Never' ? 'singleLine' : ''` is always evaluates to `'singleLine'`
Probably it is my ignorance but which is the purpose of this variable?
`font-size: 0;` maybe?
Why the change to two calls here?
I meant `p.id.split`, which is a non existent property. I know about the ADI thing.
Storing checksum in the same repository as code itself is kinda.... recursively..... =)
Optimist is a package to help with command line argument processing
why remove quoting? it's required in bash ... it might be not required in zsh - but this could be still changed with a flag, so overall it should be quoted.
I do not understand why we need this?
Is anyone calling monitor_event method with va_list argument?
Won't be forgotten as we will be transitioning to python.
I would suggest putting this additional code in a new class since djinn.rb is already really large.
Found from topic view:

		// Get topic icons if allowed
		if ($this->config->topicicons) {
			$this->topicIcons = $this->ktemplate->getTopicIcons(false, $saved ? $saved['icon_id'] : 0);
		}
ChatManager should have a getChannel(String);
I don't believe so. Substring doesn't include the last character in the interval specified.
nix the printStack
won't this be `"hello;world\r\n"`?
..., zeige {:current} von insgesamt {:count} Einträgen, beginnt bei {:start}, endet mit {:end}"
btw, is the RootPackageInterface implemented somewhere ?
Similar to x86 case, I use armv7l-unknown-linux-gnueabi target.
inline javascript it is bad.
Please put it to bottom of the view file
Can you describe the race briefly in the comment?
Needs
````
require File.expand_path('../exception_client', __FILE__)
````
We fire this event when auto-updating blog pages (so dates get re-relativised)

https://github.com/guardian/frontend/blob/master/common/app/assets/javascripts/modules/autoupdate.js#L106