labofsahil/github-event-dataset-2012 huggingface dataset:
Make email regex more forgiving

This is a somewhat more forgiving email regex, allowing TLD that are more than 4 letters long.

As you can see from this [list](http://en.wikipedia.org/wiki/List_of_Internet_top-level_domains) there are a couple valid ones that are greater than four letters (.museum and .travel).
I see, sorry. I thought it might be some redundant  code lines... It will be reverted accordingly tomorrow.
Soo.. Only turn it of for HD? Disabling the ability to deinterlace normal SD tv transmissions using yadif feels really silly.
So, if I get the fix well, when trying to load an invalid map file, a pop up will be displayed instead of crashing?

What is happening after clicking on ok?
doofus +1
oh ja sorry, dacht ik even iets gezien te hebben :)
@kevindickerson I used the merriam-webster english oxford london wales dictionary. It should be to spec.
@bnoordhuis you should know that the server in this testcase is created in the master, and I'm not just talking about the handler.
Oh awesome. Glad to see this in 3.3.
More correct way (to all directives, not only for `sub_dirs`): 3757d0c21238314f88d00b2da3a23754c340e778
is not included in a release :)
What do you think about altering Promotion#activate rather then registering for the notification? Since activators are already registered in core, it seems like it might be redundant.
Ah, sweet idea. :)
templates.partials works fine.
There are definitively too much cast from bytes to string occuring there, but it's not your fault.

However, it would be good to add a todo in the TODO file for this: the library module and mpdh should do the encoding/decoding dance transparently, without the callers having to do it all the time.
Now Its working as intended, tomorrow ill push some AI changes and some trys to fix the communication between gameobjects-map-player in order to leave this working propertly
does `readPreference` refer to something that should be documented somewhere? And if so, what and where? Same with `slaveOk`.
Currently, as written, the rake task looks for active support and fails quickly if not found with a message to load it. That might be a sufficient compromise.
Makes it so that you don't have to select your identity file every time you hit publish.
I say. o_q
Typo: `initialization`
Sorry :trollface:
This forces a 4-digit port. The constraint should be 1 <= port <= maximal port number.
@Blei heads up: this patch might conflict with your branch for #46
Whoa, this is really nice!
wew i haven't notice this until you told me!, http://en.wikipedia.org/wiki/PBKDF2
You got it.

Anirudh B <reply@reply.github.com> wrote:

>I see. So the danger would have been that joysmooth could have tried to register before the registry was constructed?
>
>Sent from my Android device.
>
>
>
>-----Original Message-----
>From: "Robert Blair Mason Jr." <reply@reply.github.com>
>To: Anirudh B <anidev.aelico@gmail.com>
>Sent: Thu, 09 Feb 2012 10:53 AM
>Subject: Re: [612-code] Changed registry to preempt Static Initialization Order Fiasco (7db9b4f)
>
>reg_obj is a static local variable.  It's a C-ism.  It's necessary because when we just use global variables, the C++ language is unable to guarantee the order in which constructors for said global variables are run.  However, if we put it in a function, then the language is able to see the dependency through the "construct on first use idiom".
>
>See this page: http://www.parashift.com/c++-faq-lite/ctors.html
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/rbmj/612-code/commit/7db9b4fef0bc117f410a93c2f92b1b5ab5cfdb28#commitcomment-949957
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/rbmj/612-code/commit/7db9b4fef0bc117f410a93c2f92b1b5ab5cfdb28#commitcomment-950857
#Winning
I don't see where this could be accessed from outside the reader or store. But since you extend Ext.data.JsonReader, the raw response will be available as ```store.reader.jsonData``` anyway. So I guess you can remove this line, and also disregard my comment about ```reader.raw```. Instead, just add a note in the API docs that the raw parsed response is available as ```store.reader.jsonData```
Fixed in commit https://github.com/fsharp/fsharpx/commit/f5ca5a6cfa0a0a2f6c567933553bff82aee581ca
Adding this file is a great idea and will be very useful.  That being said, it was very difficult to read, and I just skimmed parts of it -- can you please add some useful comments about what's going on?  This would be especially helpful in the runExperiment() function.
Oh, why does it use PMF_ROOT_DIR and not PMF_BASE_DIR like some other files?
Why not expose `im4clj.run` in core?
Great, whatever works! There could be some compatibility issues with the core stuff but that should be easy to fix. You should also rename userSpace to userspace since package names should be lowercase.
Thx Spyder!
s/1.1.2/1.1.1?
GIDF.de ^^

Don't repeat yourself: http://de.wikipedia.org/wiki/Don%E2%80%99t_repeat_yourself
iirc all that detection is used only for the definition of 'CPPCHK_GLIBCXX_DEBUG', so it's redundant in release mode.
Yeah, this broke some stuff.

picture in UserWithPicture needs to be a string, since this is the object used by the RestFB DefaultFacebookClient to fill in data parsed from JSON (notice the Facebook annotation). In its current state the UserWithPicture object will be terribly broken, since RestFB won't know what the hell to do with it.

Basically, my idea:
- Use SocialDirector to get FriendSet, which pulls (id,name,picture) tuple (where picture is the URL to the display pic) from the RestFB API
- Extend FriendSet to include a Hashtable mapping FbId to image data
- Extend FriendSet so that when a friend is added, the picture is enqueued in some sort of download queue (in FriendSet)
- A separate thread (somewhere, also in FriendSet???), downloads the images and maps them into our image hashtable
- Support callbacks or something so the friend list can be notified when downloads complete, as they're completed.
you should also use version_compare() here...
http://pastebin.com/PBwdiCsR
5 мин аптайм.
ps время на сервере перевел на МСК и синхронизировал....
good. but need much more for representative check. this - already very strange.
v3 coming soon
https://github.com/mangosR2/mangos/commit/ff1884a5cec492ba5d2d95a97a10c0f946731217
Might want to do "is None" here so that empty strings can be cached? Then again, not sure if that can ever happen. Hmmm.
`attr('href')[1:]` could be `attr('href')`
all your queries are unsafe regarding DQL injection as you simply use string concatenation to set the value. You should use a query parameter in the ``andWhere`` and then set the parameter with the value
Oh crap, thanks!
Je me suis rendu compte que cet id ne fonctionnait pas : ${link.login} , n'est pas remplacé..
Not sure what to do here:

```
/home/eilemann/Software/Equalizer/libs/co/rdmaConnection.cpp: In member function 'bool co::RDMAConnection::_finishAccept(rdma_event_channel*)':
/home/eilemann/Software/Equalizer/libs/co/rdmaConnection.cpp:748: error: 'struct rdma_addr' has no member named 'dst_storage'


struct rdma_addr {
	struct sockaddr		src_addr;
	uint8_t			src_pad[sizeof(struct sockaddr_storage) -
					sizeof(struct sockaddr)];
	struct sockaddr		dst_addr;
	uint8_t			dst_pad[sizeof(struct sockaddr_storage) -
					sizeof(struct sockaddr)];
	union {
		struct rdma_ib_addr	ibaddr;
	} addr;
};
```
I don't know that we need this `<div>` but meh
Note duplication of @n and @xml:id
This is temporary, pending decision on use of \<locus\>
What I have done so far is not fully complete and somewhat buggy. Right now I am trying to figure out how to include joda-time in the class path without another plugin complaining it is there. The only one i know of that does this is ChestShop.
My bad :3
png için mogrify'a gerek yok, chunkypng bu bilgileri işleyebiliyor.  ama jpeg'de böyle bir gem bilmiyorum (chunkyjpg gibi).  o nedenle mogrify kullandık orada.
what are all of this scans for?
I don't believe that the file "mystatistics.c" is intended to be included, is it?
oh... fail...

i tried it and it worked on Esc... ;)
i didn't test any other key :(
is the Path class unnecessary?
Should this be ```Mercury.config.debug```?
Untested, looks great. Merging.
Volgens mij wel ja. Represents is niet helemaal verkeerd, maar misschien kan er nog een blok bij (bv "circuit")?
Indeed - this is an interesting question. I've looked on some number of latest migration scripts and sometimes we use "reset_column_information" and sometimes not. And in fact I don't know where it's required and where not. So would be nice to hear from a ruby expert like @simonbrandhof
Just a little extra word doesn't hurt, does it? :-) I like litterature :-P
Meh - so much time being spent on such a small issue.
I like to prepend with `insert`, but this idiom works too
Typo removed on next commit
We can use our code sources outside project folder by using 'Linked Folder'.
You can't compile it successfully on ubuntu now?
getting a ActionView::Template::Error (undefined local variable or method `activity_stream_path' for #<#<Class:0x00000008851918>:0x00000008d3a358>): on this activity_stream_path
This commit has broken Guvnor.

TypeDeclarationDescr.getSuperTypeName() returns Object for a declared type extending another declared type.

It used to return the SuperTypeName correctly. IIRC there was a similar breaking change for 5.3 (or possibly 5.2) and it was reverted.
Sorry for the unnecessary merge through just pulling, should have worked in my own branch and rebased later instead. Won't happen again.
I cannot update a EPopupScreens.NoPopup because it _PopupScreens would give an IndexOutOfBounds error because (int)EPopupScreens.NoPopup  is -1.
Note that the original ordering by `published_at` was introduced in 5d9db15260b7b133d167c5cbdb44ed825cee3d18.
Nice, getting into it.
Tests should be executable on their own. Please add a shebang line `#!/usr/bin/env ruby` and make the file executable `chmod +x big_num_test.rb`. 

Actually, all the test files should be executable, so `chmod +x *_test.rb`
NO, I never said to revert it.
I just trying to say that before that commit the eye wasn't falling, but probably the bug "made it work fine"
Awesome, thanks much for this!!!!!!
Ah crap. I thought I had made some typos. Can you change it to use "--" instead of ":" for us?
Sorry I didn't get to this till now! I like the functionality but I needed to fix a few issues and tweak some types.
Oh my test!
---------------

1. wtf?
2. this is a test... sorry...
ha ha, fair enough!

I'm with ITV at the moment, seems I'm on the media merry-go-round! Good BDD stuff here but the existing codebase is garbage.. makes the BBC legacy stuff look like it was crafted by Uncle Bob himself!
You still with the bank, RBS was it?
> Date: Fri, 17 Feb 2012 08:37:07 -0800
> From: reply@reply.github.com
> To: jimbutler@hotmail.com
> Subject: Re: [bounce] Added framework property to NUnitTests task; (76ac9c8)
> 
> Given it's the only code I've got on github, thought it was probably a good idea to have a green build :)
> (and yes, the broken build was playing havoc with my OCD)
> 
> What you up to these days, anyway?
> 
> -----Original Message-----
> From: jimbutler [mailto:reply@reply.github.com] 
> Sent: 17 February 2012 16:35
> To: Matt Richardson
> Subject: Re: [bounce] Added framework property to NUnitTests task; (76ac9c8)
> 
> Surprised you're still working on Bounce!
> 
> 
> > Date: Fri, 17 Feb 2012 08:33:12 -0800
> > From: reply@reply.github.com
> > To: jimbutler@hotmail.com
> > Subject: Re: [bounce] Added framework property to NUnitTests task; (76ac9c8)
> > 
> > If the FrameworkVersion is not supplied, it will be null, so the build had broken. I've committed a fix that ensures that frameworkversion is not null before accessing FrameworkVersion.Value.
> > 
> > --- 
> > Reply to this email directly or view it on GitHub:
> > https://github.com/refractalize/bounce/commit/76ac9c829cf6bc982dc11bbb3ae0f88bc2492226#commitcomment-978784
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/refractalize/bounce/commit/76ac9c829cf6bc982dc11bbb3ae0f88bc2492226#commitcomment-978790
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/refractalize/bounce/commit/76ac9c829cf6bc982dc11bbb3ae0f88bc2492226#commitcomment-978801
ну вот, просто я не хочу всё это дело переписывать, я потихоньку.
Тем более не так много я поменял, и даже при этом мне не пришлось менять больше нигде, а это почти сам фундамент той же курсовой!
Замены однотипные, но для каждого графа эта замена своя, они все разные.
Hi Pascal,

If you are removing this, you need to delete the project "RestExample" since it uses these old APIs.
To be fair you could have a file alongside index.php, which sources index.php
I guess i'm pretty sold on how thing are though.  People probably wouldn't react well to a non-web file alongside index.php
I don't know exactly how this happens, but @kripken should be able to help you here.
i'm a little confused. What do the changes in the __init__ function have to do with colons in queue names?
It looks like your debug command (never did figure out what that was supposed to do) got nuked along the way. Easy enough to put it back in if you still need it though.
Thanks for your time, now I need to be able to convert this to my desired instance as the map operation returns an anorm.~[A,B] object.

I need to transform this:

val withPermissions = User.single ~/ maybe(Permission.single *) ^^ {
  case user ~ perms => user.copy(permissions = Set() ++ perms.getOrElse(Nil))
}

to this:

val withLotteryAndPermissions = (User.instance ~ Permission.instance *). map(_.groupBy(_._1))
// needs to be further investigated....
but how do use the c3p0 pool instead of bonecp? It sounds like you you are not using the DB plugin at all in which case you could just disable it altogether via dbplugin=disabled, no?
Yes it was a quick test actually and I forgot to convert it before committing.
Now what?!? This should make no difference at all? Probably a shell bug.
However, that "export VAR=stuff" syntax is not compatible with early POSIX standards. In those early times, "export" was there to export already declared environment variables, i.e. "VAR=stuff; export VAR". But this can't be the bug in this case, I think. Then I don't know why the quotes are necessary. But if this really fixed the bug in question, let's be happy and blame the shell.
Namespace constants exist.
Something like this were discussed for example here: https://github.com/php-fig/fig-standards/pull/14#r486854
don't know comment is about commit not about pull ^^
Using the levels is deprecated, should be 'edit_theme_options' instead.

see http://codex.wordpress.org/Roles_and_Capabilities#edit_theme_options
This class is used for styling the avatar picture.
Can you provide a link to the pull request or commits that implement this change?
sorry for lolling, but every exemple you give is longer than the generated code.
Those strings are already translated, in strings.php - this just translates the value (pretty sure this works)
It's a json object whose top level type is a {}. mpd just wants a string, so we put the json text in quotes with proper escapes and everything.
Yeah, I would prefer a solution that works across all browsers. I am also tempted to trigger focus with 'j' and 'k', but that could be in addition to the tabindex trigger.
The message provides info about ALL non-nullable associations with unsaved transient instance(s). 

Should it just provide a message for the first one? 

Should full details should be logged?
Hi Max,

   it's actually already there, only using the "binmode" parameter. What
binary does is set the binmode to ":raw" and get the content only after
applying Content-Encoding headers unroll but without applying any
payload-specific charset.

You can see the added documentation here:

https://github.com/polettix/www-mechanize/commit/1cd9eb367403db303879b69b83e784a66a4e2f5b#lib/WWW/Mechanize.pm

Comments are welcome of course. Instead of using a scalar reference I might
just test for binmode parameter to start with ":".

Cheers,

   Flavio.


On Sun, Jan 22, 2012 at 11:14 AM, Max Maischein <
reply@reply.github.com
> wrote:

> I think in addition to
>
>    binary => 1
>
> it would make sense to allow forcing the actual argument to binmode,
> by allowing any string:
>
>    binary => ':utf8'
>    binary => ':encoding(UTF-8)'
>
>    binary => ':bytes'
>
> and
>
>    binary => ':raw'
>
> (which is identical to binary => 1)
>
> Could you add that feature as well?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/polettix/www-mechanize/commit/1cd9eb367403db303879b69b83e784a66a4e2f5b#commitcomment-889629
>
this version has been deployed on EGATS
@elHornair you should create a PR for this branch to allow an easier review
You could store the children (that should be removed) instead of the indices. This would simplify this a bit.
Non j'ai pas encore essayé j'ai été voir ce matin vite fait faut que je
regarde  ça dès que j'ai le temps. Peut-être ce week-end
Le 20 janv. 2012 18:53, "Bktero" <
reply@reply.github.com>
a écrit :

> Je ne connaissais pas ce format ! Je vais regarder pour un prochain commit.
>
> Tu as essayé le programme ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Bktero/AndroidAccessory/commit/2e3c122904541842bb680a89646ba5dd418cc0f2#commitcomment-886726
>
What about <section> and <address>?
it is working for me, tested on a clean core but i tested both lines (this is half of the patch i tested)

```diff
diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp
old mode 100755
new mode 100644
index 6cacf5e..f31137f
--- a/src/server/game/Spells/Spell.cpp
+++ b/src/server/game/Spells/Spell.cpp
@@ -2814,12 +2814,12 @@ uint32 Spell::SelectEffectTargets(uint32 i, SpellImplicitTargetInfo const& cur)
                     unitList.remove(m_targets.GetUnitTarget());
                 Trinity::RandomResizeList(unitList, maxTargets);
             }
-        }

-        CallScriptAfterUnitTargetSelectHandlers(unitList, SpellEffIndex(i));
+            CallScriptAfterUnitTargetSelectHandlers(unitList, SpellEffIndex(i));

-        for (std::list<Unit*>::iterator itr = unitList.begin(); itr != unitList.end(); ++itr)
-            AddUnitTarget(*itr, effectMask, false);
+            for (std::list<Unit*>::iterator itr = unitList.begin(); itr != unitList.end(); ++itr)
+                AddUnitTarget(*itr, effectMask, false);
+        }

         if (!gobjectList.empty())
         {
@@ -4651,7 +4651,7 @@ SpellCastResult Spell::CheckCast(bool strict)

     Unit::AuraEffectList const& blockSpells = m_caster->GetAuraEffectsByType(SPELL_AURA_BLOCK_SPELL_FAMILY);
     for (Unit::AuraEffectList::const_iterator blockItr = blockSpells.begin(); blockItr != blockSpells.end(); ++blockItr)
-        if ((*blockItr)->GetMiscValue() == m_spellInfo->SpellFamilyName)
+        if (uint32((*blockItr)->GetMiscValue()) == m_spellInfo->SpellFamilyName)
             return SPELL_FAILED_SPELL_UNAVAILABLE;

     bool reqCombat = true;
@@ -7320,3 +7320,4 @@ void Spell::CancelGlobalCooldown()
     else if (m_caster->GetTypeId() == TYPEID_PLAYER)
         m_caster->ToPlayer()->GetGlobalCooldownMgr().CancelGlobalCooldown(m_spellInfo);
 }
+
```
Yeah, not sure why sass doesn't seem to compile it correctly. Maybe this has been fixed?
Its not ideal to have to use another layer, erb to process it again
But glad you found it useful
para o sympy/sympy repositório -> para o repositório sympy/sympy
The game treats all rail the same. There isn't any friendly rail or enemy rail, ther's just rail :)
We should always keep the p tags with classes seperated from the text by a line break, so if you change the text and I change the classes we can easily merge.
...простонал чувак с файлами `*.sln` в репозитории.
And again horror naming.
should update https://github.com/wikimedia/WikipediaMobile/blob/master/assets/www/js/settings.js#L61 so that the language is selected on startup
* fixed*
this is for remove the callout when clicking the blue dot, showing 'User location'!
Are the units consistent with Linux and OS X?
thanks for that one-line patch :)
I'll add a comment above this line.
Yes. This can be removed, [it has been moved to it's own command](https://github.com/Multiverse/Multiverse-Core/blob/master/src/main/java/com/onarandombox/MultiverseCore/commands/GeneratorCommand.java#L41).
Isolating every string in Spanish.
In fact what would be better is if you made a pull request on Grunt itself with these changes.  I'll ping @cowboy and then we can move forward from there.
Not quite what I wanted, but I'll take it anyway, thanks.
Yes, absolutely. Thanks.
Os browsers capazes de executar código escrito em JavaScript reconhecem o elemento "script". É dentro desse elemento que se coloca todo o código.
Well, to be fair, it says `SHOULD`, not `MUST`. How do you know I didn't carefully weigh and come to a full understanding of the implications before choosing a different course?
I wonder how does android: works in themes?
You don't need to create the local variable, ```return $base``` is enough.

Perl has this nasty side effect of returning the last assigned value in a function that makes code like this remain for too long.
Hi!

We are planning to develop a service for teenagers/young people to provide
them help and support regarding different matters of life. I think this
piece of software somewhat targets our needs pretty well. We do not know at
the moment the scalability needs yet but It would be surely a reasonable
startup for us. I am however quite new to node.js and happy to learn it. If
we are able to use this software onward then It would be also possible to
contribute into it =). I appreciate your answer and would like to hear from
you. Please ask me questions further about it if you have any =). Have a
nice day!

Regards!
Rizwan Zia

On Tue, Jan 17, 2012 at 6:11 AM, zackster <
reply@reply.github.com
> wrote:

> Hi riwan, happy to take a look and get this patched.  How are you planning
> on using the app?
> On Jan 16, 2012 3:23 PM, "Rizwan Zia" <
> reply@reply.github.com>
> wrote:
>
> > I think it was removed in commit
> > (c69ec15e967b6f00c5b6e05d476de5fbb5a398a5) though I am not sure
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/zackster/CompassionPit--Node-/commit/2b4407cbf5586d6ecab5ed6feeb32d156788b8f2#commitcomment-872545
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/zackster/CompassionPit--Node-/commit/2b4407cbf5586d6ecab5ed6feeb32d156788b8f2#commitcomment-873015
>
I've created the logging package because it will contain:

* the LoggerFactory class
* the Log interface (which define the logging methods)
* the generated Log class
Чтобы пользоваться его мутабельностью. Передавать threadList как сайд-эффект через State  -- это ещё более нездорово. Или какое предложение?
Fixed with a @see
[JIRA created](https://issues.jboss.org/browse/ARQ-799)
Layout was broken on the *tablet* emulator in landscape orientation. *Sigh*
hi sir, i just want to know if i can change or add a layout in viewing a file? like add action bar on top? how can i do that?
how can i fix my plugin?
fix for #29 https://github.com/socialcast/devise_oauth2_providable/issues/29
2012-02-07時点での最新のスタイルファイルです．
This is a recently version styles in 2012-02-07.
there is no reason. could you merge this?
patch for head is here https://gist.github.com/1699333
by not working, you mean missing from the process list, or missing functionality :?)
Np man ima start doing more
On Feb 17, 2012 12:20 PM, "Liquid" <
reply@reply.github.com>
wrote:

> Great Job bro glad to see ya chiming in
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/LiquidSmoothROMs/android_packages_apps_LiquidControl/commit/c4633615552422c810849517021b6745b668486f#commitcomment-979265
>
I appreciate it a lot bro..
On Feb 17, 2012 12:20 PM, "Liquid" <
reply@reply.github.com>
wrote:

> Great Job bro glad to see ya chiming in
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/LiquidSmoothROMs/android_packages_apps_LiquidControl/commit/c4633615552422c810849517021b6745b668486f#commitcomment-979265
>
Really this check should ensure that the containing path ends with `os.path.sep` before doing the startswith check, otherwise it would claim that `foo` is "within" an adjacent `foobar`.

It would probably be better to extract a separate `path_contains_path` or some such utility function.
Also, I'll add some comments in that code to explain what is going on since it does seem counter-intuitive.
unless if you call it with 'route'
Please simplify the way to get haveEventHandler
was it something else we also redirected to deadletters?
My previous statement was meant to show that I couldn't find any way for the key and the RemoteConnections address to differ.

They are always the same as far as I can see.
Weird line breaks in this paragraph
Could add that it should be a valid URL, so they know what and why.
remove the cast, please
ok, but it is `""` as default in reference.conf
then it must be removed (commented out) from reference.conf
Why this level of indirection? I can only see it used once.
On 2012/05/25 06:22 PM, viktorklang wrote:
>> +
>> +  def getSSLHandler_? : Option[SslHandler] = {
> I don't really like methods with innocent sounding names like "getX" that do a whole lot more than just getting something.
> Any suggestions as how to improve on this?
I can rename it to 'initialiseAndGetSSLHandler_?' or more verbose
'initialiseSSLContextAndGetSSLHandler_?'
Or if it really is a Deadline, it should be in the future, then you just need to check isOverdue and then evict.
Akka creates all actors. Don't you know that? ;-P

The actors are created like any other, so maybe the wording is a bit confusing. I have changed it to:

Actors are created and started asynchronously. When a `Consumer` actor is created, the `Consumer` is published at its Camel endpoint (more precisely, the route is added to the `CamelContext`_ from the `Endpoint`_ to the actor).
When a `Producer` actor is created, a `SendProcessor`_ and `Endpoint`_ are created so that the Producer can send messages to it.
And this constant change.
Obviously, i'll not allow that :) We have `color` (and `background_color`) used in many other widget. Giving 0-255 value seem against the general rule, without any gain: we still need to convert back the value if we want the 0-1 for using it.

So for this one, just use the general rule: color.

I was thinking about another possible class for assigning color: ColorProperty() -> it would accept rgb/rgba (in 0-1), as well as string ("#33ff00") or integer (0x33ff00). When accessing to the value, instead of returning a simple "list", just give back a wrapper class for having an hex representation, or access to individual component etc. (I guess a Color() class would be very useful, we could mix, substract color etc...).

Just a global guess.
Define it.
Nicer for debugging
this isn't needed?
To preserve priority ordering. Basically this is implying that Reserved instances > any other type.
If we want to do this stuff, it belongs at the top, not at line 27.
in-string in is dead-removed in commit ea67ac399ad8ea470319b4a0bb47a1745f8c43e8. 

I made the suggested change (replacing number->string with format). After exercising the change using the snippet you provided, it looks okay-no exceptions. In addition, there doesn't appear to be problems in any of the animations.
Is there a other way to get this in then?
I think this is the Session to Account rename.  I accidentally left a reference to this class for _user interpolation
typo - function.
did you try the performance influence of this change? When I tried it with the SPI stuff, it seemed to me that `MPI_Init_thread` lead to slower code...
Sorry, I didn't quite understand what you meant. Would you please elaborate?
Don't delete this rescue SystemExit block. It prevents SystemExit from being handled and swallowed by the rescue block just below.
printing here doesn't work for this use case because the code unref's when it actually hasn't taken a reference.
Could we rename this to `newBoss()` like other factory methods in our public API?
Don't really need the else since the other branch returns.
we could use RETURN_STRINGL() to return early.
so does the later RETVAL_LONG => RETURN_LONG  
：）
delete в деструктор наверное нужен
неравенство лучше взять в скобочки, читаться лучше будет. здесь и ниже
#include <QtGui/QDialog>
I think that should read `a list` instead of `an list` as well, right? Can you please fix that as well and amend your commit? Thanks!
Not easily, I tried it first actually. THe problem is, if you had these extra keys to the slice on line 115, you'll always overwrite the value. So, there would be no way to stop propagation on each includes if needs be.
J/K, nice catch Paul!
it is similar but there is a subtle difference

take a look at this to see how *args and **kwargs work: http://www.saltycrane.com/blog/2008/01/how-to-use-args-and-kwargs-in-python/
if you are assuming that the result is an add it will be more verifiable that this is correct if you wrote

args = Add.make_args(...series())
I don't think this works, in the case when this ask returns None, you should not return True, which happens if you do ``ask(Q.real(x**y), Q.imaginary(x))`` (after you fix the other 2 comments I made).
I tried a total rewrite of expand with hints which is very different
from what is being done here. So as far as how to sort the keywords I
don't have advice past the suggestion that you do the more dependent
ones last. Does the docstring have something to say about that --
something about doing power_base before power_exp?
I would rather like an explicit test `assert sin(pi/8) == sqrt((2 - sqrt(2))/4)`.
This should have unittests. A simple doctest would be cool too. Looking at the name and code I'm pretty sure I know what it does. It would be nice to see a clear example though.
Is it important that all of these decisions are made before the PR is merged? I'd like to do this somewhat incrementally but there is no rush.
Yes.  Some day, Poly will be faster in the highly multivariate case, and we can use this.  But until then, this is too slow.  

In fact, let me go ahead and delete this now.

By the way, you should also ignore the test_failing_integrals.py file.  I plan to merge that with the regular tests at some point (it's a chunk of work, though, so I'll get to it when I get to it).
Use HTTP/1.1 then. I think $_SERVER['SERVER_PROTOCOL'] is populated according to client headers. So it is possible it may be empty.
I am not sure if this is a good change since API of this method changes. Is there a better way to implement this functionality?
These are working fine as @mdomba mentioned. Used this method just recently.
Why put this action in CCodeGenerator instead of ModelGenerator?
`.get()`'s default default is `None`, so you don't have to specify if you use and check for `None`.
em-synchrony is a dependency, so I don't think we should link to it. I was wondering about the methods such as ```post_init``` and the like - they are described in EventMachine::Connection or EventMachine::Deferrable? These are template methods or GOF methods overridden from the superclass? Explaining these issues would be beneficial.
Hopefully, going to tinker with it now. I can't believe this is the first repo that would benefit from this.
@ataylor32 good point, that would be quite confusing/misleading.
I like the general idea of basing this off of float.h constants but we should wrap with an ifdef incase it doesn't exist which seems like a possibility from some quick Googling. I dislike that its not a specific assertion and we're guessing that 2*significant_digits captures, but I'm not finding anything better and its still better than the hard coded 32.
The idea is that `CoordinatesBase` is an abstract superclass defined by two angles, hence "angle1" and "angle2". This makes no assumptions about the angles and the names. I thought i might be useful to access the angles without any specific knowledge of the subclass. These can either be made private (`_angle1`) or dropped if really not needed.
Better use || instead of 'or'
and => &&
not => !
http://devblog.avdi.org/2010/08/02/using-and-and-or-in-ruby/
Also, how is it that ````"$(tft#{@id}).hide();"````` works?  How are we getting away without the ````#```` sign at the start of the selector and quotes around the selector, i.e. ````"$('#tft#{@id}').hide();"`````
That's the only one right?
The license header probably needs to go in the right spot - also, a line of documentation for this trait might be good as well.
putting headings on this page will break the pages with all of the includes. because it'll include the whole file and randomly have an "examples" heading in the output.
see above note.

also "particular" seems redundant.
nit: users do not perform write operations, users make updates to the database, mongod performs the operations.  Note that a delete may cause a write operation to occur, as well as non-user initiated activities.  All of these would be journalled AFAIK.
I'm not sure that you should be making this change
Why were spaces removed here?
"wrongly" => "improperly"
Use warning instead of echo.
What are these changes doing?
Traceroute is only for the collector.zip which isn't distributed in  rackspace-monitoring-agent.

it is a build requires because the tests fail without it.
it needs fixing also to actually support non alpha numeric, but I don't know what the actual requirement is.
From the lua docs:

Such combination of if not ... then error end is so common that Lua has a built-in function just for that job, called assert:

    print "enter a number:"
    n = assert(io.read("*number"), "invalid input")

I don't feel strongly about this at all.
Currently if the agent doesn't successfully connect aka if `!select._receivedPromotion`, we will try to connect for indefinitely, right?

We should probably limit the total number of retry attempts.
why not pass in the state here ?
FWIW, I much prefer using `let` to define these dependent objects rather than using ivars in an before hook.

I wrote an answer on stack overflow about this:

http://stackoverflow.com/a/5359979
On Windows, relative path contains backslashes, so in order to generate an appropriate URL, those backslashes need to be converted to forward slashes. I don't know of another way to do this. Theoretically you don't need the Windows check, so we could probably just remove the conditional.
> так твоя сумма за предыдущие дни равно моему полю last_shows же?
Да

Не будет заметной разницы что ты делаешь select по одному полю, что по сумме двух из select с join. А нодой писать и туда и туда плохо - она обновляет инфу сразу по всем баннерам, придется формировать 2 больших и притом разных по формату инсерта. Да и 5 минут - большой промежуток., причем вполне можно и каждые 10 минут этот крон запускать.
Not convinced that this particular case is wrong, but yes -- this is the
sort of thing we're looking for.


On Sun, Dec 23, 2012 at 2:02 AM, Matthias Mailänder <
notifications@github.com> wrote:

> In OpenRA.Mods.RA/Effects/Missile.cs:
>
> >  			Altitude = Args.srcAltitude;
> >  			Facing = Args.facing;
> >
> >  			if (info.Inaccuracy > 0)
> > -				offset = (info.Inaccuracy * args.firedBy.World.SharedRandom.Gauss2D(2)).ToInt2();
> > +				offset = (PVecInt)(info.Inaccuracy * args.firedBy.World.SharedRandom.Gauss2D(2)).ToInt2();
>
> Is this conversion from Float to Int2 and then to PVecInt problematic in
> case of rounding errors, @chrisforbes <https://github.com/chrisforbes>?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/OpenRA/OpenRA/pull/2242/files#r2494947>.
>
>
Agreed. Removed from both concrete classes.
The test checks if there is no error or a ValueError. In this two cases it is a pass. Any other kind of exception results in the test failing. If you remove the fix in the code and run the test you will see that the code fails with an AttributeError (the one reported by Mark as well) which was the problem I was trying to fix.
OK, then using str(sys.maxsize) should work.
Regarding the -1, there is a ticket asking to use another system than a code to denote None, so I think we could do the real fix soon.
Ok, I've made this change here: 
https://github.com/mrocklin/Theano/commit/a0a41677b51e7964577814d2f0477b7cad2a1ddb

It could probably use some review though because 
1. This error is hard to find/wasn't found by the current tests
2. I'm still not 100% on this part of Theano and tend to make mistakes. 

Passes test on my machine which does do MPI well.
This should probably go in MiscUtils, since there's similar (but not the same) command line parsing already there.
instead of integers we send strings, error should be thrown
These are pretty easy functions to read, but it might be worth a little english describing what atoms `SetupFun` and `CleanupFun` are called with, and when. Eunit setup/cleanup is confusing enough on its own without an extra layer to read through. ;)
Можно сделать `Object.keys(opts).forEach()`
Note for me to follow-up (or maybe leave for you): what happens when a facility enters a simulation as an initial facility and needs to read data from the file?
Indeed. Cleaned it. See following commits, thank you.
There's no *need* for it, but logically, it's an else-case, so I use the else-syntax.
It's just the same to the interpreter, but makes the control flow more visual.
And nope, we set the transport name above, doesn't make sense to call Transport:name on something we just defined. We know what it is!
Oh, bleh, alright.
@rwldrn How does this change the semantics of the code?
Oh wait, I misunderstood your patch. Disregard everything I said so far.

![](http://static.fjcdn.com/comments/Apparently%2BEnglish%2Bis%2Bhard%2Bfor%2Bmost%2Bof%2Bus%2Blol%2B_ba794130017453d621a6c92aa60c4997.jpg)
A few comments:

1. I don't like the "damaged unexpectedly" error message
2. Its better to use "transition state" instead of abbreviations
3. Postgres error conventions recommend not ending error messages with punctuation.
4. "unexpectly", if we were to keep it in the error message is mispelled -> "unexpectedly".

Suggestion:  "invalid transition state for cmsketch"
Hi Hitoshi,
     Sure. Right now, I don't have a very large dataset at my hand. I will get a larger one  then send it to you. Thanks.

All the best,
Kun

On Apr 27, 2012, at 7:07 PM, <Hitoshi.Harada@emc.com> <Hitoshi.Harada@emc.com> wrote:

> Kun,
> 
> Sorry for the delay, I've been busy with other stuff for the v0.4, but
> hopefully it'll get in early in May.  Meanwhile, I just wonder if you can
> provide test data you used for the development.  That'll be helpful for me.
> 
> Thanks,
> Hitoshi
> 
> On 4/26/12 8:02 PM, "Welton, Caleb" <Caleb.Welton@emc.com> wrote:
> 
>> Kun,
>> 
>> Thanks for all your hard work.
>> 
>> We are targeting pulling the code into the main branch soon, by May 10th
>> at the latest, so that we can complete testing for delivery of Madlib
>> v0.4 in June.
>> 
>> Best regards,
>> Caleb
>> 
>> 
>> On Apr 26, 2012, at 1:25 PM, li kun wrote:
>> 
>>> Hi All,
>>>    How are you?  As from Hatoshi, the code is already in a good shape.
>>> let me know what else I can do in my side to get the code prepared to
>>> delivery. 
>>> Thanks and have a good day.
>>> 
>>> 
>>> All the best,
>>> Kun
>> 
>
Hmm, you can't have ``True``, ``False`` or ``None`` via this?
Let's be specific, it doesn't hurt.
Can do this in a block instead of cd'ing around:

```rb
cd 'src' do
  system ...
end
```
Is this line strictly necessary? pkg-config normally handles this, and it did when I tested it.
Pour ce fichier, tu ne voulais pas le commiter seulement. Ici, tu l'as carrement supprimé... Pour réparer ça, fait un

git add .classpath et corrige la différence que tu trouves entre les deux version (puis git commit git push)
Do we still need this comment, since we're doing it later to avoid the slowdown?
I wouldn't really say they're custom anymore than any of the other code in this library is custom.

Maybe just "PRAW error classes"
Clicking edit doesn't appear necessary now.
Ya..good point. Not sure :). Updated.
haha. They should be alarmed that we are hijacking they're input for lack of webkit or server. Don't you agree?
This one's tricky. I'm using SpdyReader and SpdyWriter each as structs that contain a bag of fields; each frame type uses a different subset of the fields. It's a bit gross but it makes the code structurally simple and I don't need a bunch of different classes and I can avoid a lot of object allocation.

Ping calls its ID the 'id' but the streams call their ID's streamId. I didn't want to define multiple 'id' fields where some would be valid sometimes and others would be valid other times, so I just folded them together into one field 'id'.

But I'd prefer to continue to call it 'streamId' whenever it unambiguously refers to a _stream_ ID.
ah, this should be only a version.
i will remove the date.
thx
See `do` comment below. It's particularly apparent in a situation like this.
Same here, you have a space
Should we throw an exception or something if the table doesn't exist instead of swallowing it? I don't remember what we used to do.
Do we need test cases around special characters? For example, if the URL contains %-encoded characters, the resulting local file path should contain the decoded form (I'm assuming). I'm also not sure about the rules around things like double-byte characters and how they're supposed to be encoded (or not) in URLs, but we should make sure we're properly converting those to local path characters too. (We've had some problems in the past about dealing with decomposed vs. composed characters, like umlaut-u.)
"flag" type is used to indicate attributes that take no value (empty attributes). When these empty attributes are used in xhtml doc type, then users can have either attrName="" or attrName="attrName", but not attrName="true/false"
@jason-sanjose @peterflynn I haven't actually looked at the code in Async, but I spoke briefly with Peter about this yesterday. I think the withTimeout code isn't compatible with the doInParallel that we're using to bind all the promises together.

Even if it is, the change is slightly non-trivial because it should really have a test case. And we don't have any testing for the extension loader right now, so it'll take some thinking to figure out how we inject a "broken" extension into the loading path for a unit test but not have it there all the time.

Since fixing that is unrelated to this bug fix, I decided not to do both at once. That's why I filed a bug.

Also, I wanted to keep our bug count constant.
What's this a placeholder for?
So that didn't end up working because the sidebar open files project panel overrides it. I did try to clean it up a bit a little though because I could do one .resizing #project a and one .resizing a.
Yeah, I'll send a note when the pull request gets merged.
There seem to be DRY violations both within these next 2 blocks, and in the corresponding blocks in deployments.html. I tried
```
{% url appstore as search_url %}
{% with search_for="apps..." search_name="search" %}
    {% block sidebar %}
        {% include 'partials/sidebar.html' with display_ua='true' %}
    {% endblock %}

    {% block tabs %}
        <ul class="nav">
            {% include 'partials/search_input.html' %}
        </ul>
    {% endblock %}
{% endwith %}
```
but the variables didn't seem to persist inside the blocks
Let's write "imageFormat a MIME format string". Don't see why it should be new... :-p
@nvloff — I'm thinking the API might be simpler and easier for users if we allowed the delete/remove with just the `oid` of the note. Users get that back from the `create` method, so it should be available.
The wrong mask had been used (see above), but that wasn't noticed because it was ignoring the result (and testing was sparse).

Now setting with value2, the BigInteger[], not value, the long[].
Why removed this? We want messages to be stored in the session? And session in MongoDB?
You have down there a line where you are doing `url = settings.FACEBOOK_LOGIN_REDIRECT`. You should add TODO (It should start with with upper case letter and without a dot at the end, as other inline comments) there and explain that we should try to redirect to the page where user initially came from, before coming to login page and pressing Facebook login button.
Implemented as suggested above.
I didn't write initially that import, so I didn't pay attention, fixed now.
Please include docstrings for all methods.
Thank you for this reminder. This is something else I wanted to iterate on.

I am not sure yet if we want to remove the util functionality of making horizontal scrollbars, just because we no longer use it in our own code. Editors and the like might need it.

setScrollbars is an older API function. We can update this one to make sense to our current code later.
Why did this get removed, actually?
One too many spaces before right bracket
Should we get a bug filed to re-enable and put that bug # in this comment?  Otherwise we'll forget and never come back to this.
This should probably be in a CSS declaration, not code-wise inline.
remove the non-youtube URLs
Ok, this is a minor one:  if update_timer is > 86400 (i.e. 1 day) this will have the surprising result of never showing this data:  timedelta's .seconds attribute wraps at 1 day (and increments .days by 1 )  

Minor nit, I doubt anyone will do that, but the fix is easy: use .total_seconds() instead.
I've not seen the Scalac output but I'm pretty sure that this is a bad idea... You should define a catamorphism.
This reminds me we need a "By tag" kind of thing, too. But tags are going to be messy and there are a ton of them. We might want to do the Tags page like we're doing the Speakers page.
to be defensive here you need to check if you are not != last
It avoids lots of tests for list head. The relative position in the list is no different, so using a different name to refer to the same actual thing doesn't make any sense from that point of view.

If you're sure you've got a `ListItem`, you know it has a `next` and `prev`. But either of those could be head, so you can't go past there without a test. I ended up with things like:

```julia
function enqueue{T}(lst::List{T}, item::T)
    nxt = ListItem(lst, lst.next, item)
    if !isa(lst.next, List)
        lst.next.prev = nxt
    else #empty list
        lst.last = nxt
    end
    lst.next = nxt
end
```
...which didn't make any sense, because the manipulation to the data structure was the same whether the list was empty or not.
Could you wrap these parameters in parens>
Good question.  Might merit further discussion and perhaps even a spike on what is the best way forward with the different sizings.  In general EM is the best.  I was sticking to that and leaving what was already there.  Theoretically if we do one base pixel size and then style the rest of everything off EM, if we want to scale up, we just bump the one item's pixel size and everything will fall in line.
If that is the intent, I'm ok w/ it as is.  Wanted to raise the question since it is currently always dashboard.
Kill the comments