labofsahil/github-event-dataset-2012 huggingface dataset:
Исправления

1. Починен доброчан, который я сломал в предыдущем пулл-реквесте
2. Тепреь скрипт распознаёт когда нужно использовать сокращённый месяц, а когда полный при замене времени.
3. Шаблон теперь кэшируется. Мможет лучше сохранять в другое место, нежели `Cfg`?
4. Убрал `use strict;` ибо замедляет `.charCodeAt` в лисе, который используется для расчётиа хэш-суммы.
Load views from theme locations

This function allows access to theme specific views without having to set a partial first. This is useful for nesting views on the fly.

controller
$this->template->set('users', $users)->build('view1');

view1.php
&lt;ul&gt;
    &lt;?php foreach($users as $u): ?&gt;
        &lt;?php $this->template->load_view('list_item', $u) ?&gt;
    &lt;?php endforeach ?&gt;
&lt;/ul&gt;
Some improvements to dataforms

Hi,

Eric Liu and I worked on some cool features that we'd like to see merged into the original branch of the project. The features are:

1 - Support for specifying an "initial" parameter to create_collection and create_form, this can be thought as the same behavior of specifying initial data for a django form.
2 - Enable form fields to have a "placeholder" attribute defined; you know, the HTML5 placeholder attribute :)
3 - Support for a "virtual" (in the sense that it isn't going to be send as a parameter to the field itself, instead it is just used by dataforms itself) field attribute "blank_choice_label", which is used to change the default option "--------" on select fields (a use case is when you have a "State" dropdown and you want the default option to be "[Select State]").

Please, let me know what you think.

Thanks!
Develop/1.0.0

first merge
Added minimal .travis.yml for travis-ci.org

There is quite a bunch of customisation that could be done in .travis.yml, but this is the smallest thing that works.
some cleanup to the friendly rename script

Fixes issue where directories are considered when guessing the new filename. Also fixes issue where numbers > 9 weren't recognized by the regex.
Add MergeConflicts - detects common merge conflict markers

Sometimes, you might accidentally add files that have conflict markers in them without realizing it. This checks each file as it is added to the dist for common merge conflict markers, and aborts if they're found.

No tests for this plugin yet.

Fixes GH #6: Feature request: check for merge conflict markers
Checkbox - trash old products

Salut,

Cred, ca am reparat o eroare legata de checkbox-ul "Trash expired products". Atunci cand debifezi un checkbox nu se mai trimite respectivul camp prin post, iar cand se facea verificarea i se adauga valoarea default.
Cu ocazia asta fac si primul pull request.
A solution in Java for the TextConverter example
Make all section uppercase
List - Use Entity's Repository to create QueryBuilder

Instead of having the EntityManager create the QueryBuilder, use the Entity's Repository to create the QueryBuilder.  

This would help users that extend the default EntityRepository.  

Thoughts?
Fixed inputs on robot drive, added debug statement
sidebar scrolls to active page + updated to boilerplate v4
Small README updates

 * List all compatible players, with the required plugins;
 * reorder sections (everyone likes screenshots);
 * typos.
Traduction et corrections

Mise à jour des deux fichiers, l'un corrigé, l'autre traduit.
Add video metadata

When a YouTube video is embedded in the post, a link will be added as `og:video`
Normalement, je mettrais cet algorithme dans une autre classe
I've looked at tastypieapi, and it doesn't fit the usecase here. That's resource-based, while this API conceptually closer to search, i.e., you get a set of resources and their relations as a set of (potentially unlimited) data objects, whereas resource-based apis talk about individual items with a fixed set of properties. That'd be feasible for an API for Changeset or Push, but that's not needed by anyone right now.
Welcome, Joseph!
Good catch, wasn't aware of s#.  I made the change as suggested in [efd3a66].
Yes, that is why we probably should have a general way. An alternate idea
would be to duplicate previous line.
What's the motivation for this one?
1) Jupp
2) Clare used 70% as cut-off criteria, however somehow arbitrary. To avoid such queries perhabs this should be stated on the help page.
3) Yes I also didnot like to calculate cumsum 2 times (once in the print and one in the summary). I will put this into simper.
@iromli, @gandaro -- please [open a new issue](https://github.com/jkbr/httpie/issues/new) for this.
@bnoordhuis - Oops, yes, it should be removed. I will do it soon.
Whenever you have something which cannot be null, you should use a reference instead of a pointer. That will also make the assert(...) below unnecessary.
just sorttest
So, the short answer is: http://superuser.com/questions/35267/how-can-i-roll-back-1-commit

The longer answer is that it's simple enough to revert a commit, but sometimes it makes more sense to delete the repository, re-fork/clone and then reapply your changes.
If there is no applicable test, then remove this commented-out portion.
Since you already assume that ```lib/nalloc.rb``` has been required before this file is required why not just add:

```$LOAD_PATH << ::File.expand_path(::File.dirname(__FILE__))```

at the top of ```lib/nalloc.rb```. The require will then look more natural as just:

```require 'nalloc/caution'```
The Commit Message of that commit really is not nice!
Did you push already, as I don't see it?
(or I could be looking at the wrong place ...)
Hi jcw,

Just wanted to comment on this file (but the same holds for webClient.ino) but this pattern is actually not a good example.

a better solution would be to do `timer += 10000;` else the time between actions could slowly shift, depending on the time it took to get to this line again.

For the rest cool library! This was just a tip :)

Cheers,
Davy
Same problem for me.
Issue related here #10222
Why is the beam file written to disk if `PassCover` is enabled? Where is the beam file written? Is there any change that it actually overwrites a legit beam file?
At the very least, the "MonadPlus reform proposal" supplies part of what
you need, distinguishing Left Catch from Left Distribution (though left
distribution is actually a right distribution law).

http://www.haskell.org/haskellwiki/MonadPlus_Reform_Proposal

However, that proposal doesn't capture the impossibility of m >> mzero =
mzero in a world where IO qualified as a monad. To do that, you'd need to
do something much more radical.

Either by, requiring that every monad transformer that introduces
non-determinism works be something weaker than a monadplus or that they
require something stronger than a monad, to rollback effects. Since the
latter is largely impossible in a world with unrestricted effects like
scala, you're basically left fragmenting MonadPlus or MonadZero even
further to capture whether or not m >> mzero = mzero holds or just silently
ignoring the issue like the Haskell community has for the last 15 years.

-Edward

On Tue, Feb 14, 2012 at 4:49 PM, Jason Zaugg <
reply@reply.github.com
> wrote:

> Okay, it's back:  b21cc2
>
> Two questions:
>
> 1. are we happy with my classification of "normal" and "strong" laws? I
> only added the "right zero" law to the latter category.
> 2. should we split MonadPlus into two (or more) type classes, each with
> the same interface, but with different laws?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/scalaz/scalaz/commit/d71abed072f271dbcecd159d3fdaac516e49ebb6#commitcomment-966659
>
do you have any fixes in queue?

i want to make a new testing release...
Yes :)
Fortunately, tA and manaserv are using positive hairstyles, so adding a negative check only on items is the way to go.
Later, we will even be able to remove it when the hairstyle are declared in another file.
FYI I removed this comment because it's inaccurate. Vim mode is shown no matter what because mnuView.appendChild is called anyway. This is one of the things I didn't update in the interest of time.
the encrypt(string) private method must actually do something as per #4
What is the purpose of the EventCalendar? It just looks like a wrapper that takes away the functionality of the arraylist, and had some filter functions that should be in the filter part?
Don't have time, didn't want to forget, I feel very confident that this change is correct. Back it out if you'd like. :)
This is on my server and works ok.
It may be a question of of opinion, but I wouldn't mind seeing such long lines line-broken every once in a while.
Ok tested it looks like it works. Also I figured out what I was doing wrong with tracking branches.
Ah, forgot to mention we also need to filter out the Apple keyboard and other bullshit HID devices from the OEHIDManager.
Ok, then if the developer is Nexus Why your Github repo have all the changes in CodeIgniter to Ikariam clone?
Just a short update on what I did:
Parallelizing the Forest construction over network did not work out as the memory transfer killed it. That could not be circumvented by a transformer as the data was given from the Forest to the trees.

Also for some reason the data transfer was really slow. I didn't benchmark it yet but our admin got 400MB/s using netcat. The speed I had must have been much lower. Don't know why.

Maybe a final remark: the forest tries to be smart about inter-process memory transfers and uses the "n_jobs" to create lists of jobs.
If creating each tree takes very little time, this might be important. I am not sure how well this will play with other paralellization engines.
wie wird dann eigentlich die PropertyClass entfernt? machen wir da noch sowas wie

emp remove --definition value

?
You forgot to update help in `CmdLineParser::PrintHelp()`.
Commenting on the fact that I just saw this commit in my 'network' and hadn't realized I didn't fully make my setup fully 5.2 safe.
解析不是 parse 么？纯路过...:P
I think you're missing a unit here - is the "number of **seconds** that bundle info is cached"?
这是->这时
http://imageshack.us/photo/my-images/641/80109507.png/
:)
Еще один краш при візове моба, уже не в стратхольме: https://gist.github.com/1595418
И как это "in R2 events not added for creatures in Creature::AddToWorld() method."?
Creature::AddToWorld вызывает

```
void Unit::AddToWorld()
{
    Object::AddToWorld();
    ScheduleAINotify(0);
}
```

и дальше

```
void Unit::ScheduleAINotify(uint32 delay)
{
    if (!IsAINotifyScheduled())
        AddEvent(new RelocationNotifyEvent(*this), delay);
}
```

Там явно что-то с локами... У меня сервер в однопоточном режиме работает.
Neede some fixes here.
this - mistique :) try exorcism ritual.
@peller - could you please talk with dijit.Tree developers about whether selectedItem should auto-adjust within the widget itself if the currently selected item from model gets removed?
also not ideal to commit this
Do we want this signal to run on fixtures import? I don't think we do. There's a check for raw in quite a few signals to stop that.
Updated:
https://github.com/yehster/openemr/commit/ea3c3b06c9047b3baa70a9bb29cd133dbbb0cc68
all changes in one commit though.
Como ven esta es otra persona :P
You can't really edit commit names after you've committed!  This is really
intrinsic in the idea of "version control" - each step is "set in Stone",
so to speak.  :-P

You use branches to experiment on, for example!



On Sat, Jan 7, 2012 at 8:46 PM, Austin Stone <
reply@reply.github.com
> wrote:

> It works better this way on aquamacs: I am trying to find the best way to
> go about things and csound-x in aquamacs separates the csd files into orc
> and sco (orc and sco being the older way csound ran)
>
> I have rolled back commits 3 times already trying to debug what will work
> best, my next step is to try to use instrument names versus numbers so the
> score has a bit more coherence than 1, main, 2, main, main etc
>
>
Right on, sounds like the right stuff.  Hope to get a chance tomorrow or if
not Monday to look at this.

> Also testing global variables for the next instrument.
>
> All of this was supposed to be in the explanation of the commit so if you
> no how to edit that after the commit it would be very helpful for me!
>
> On Sat, Jan 7, 2012 at 8:40 PM, Tom Swirly <
> reply@reply.github.com
> > wrote:
>
> > Coolness!  Does it still work... ?
> >
> > Let's talk tomorrow-ish about next plans, I don't want you to go off and
> do
> > things at random...
> >
> > On Sat, Jan 7, 2012 at 8:38 PM, Austin Stone <
> > reply@reply.github.com
> > > wrote:
> >
> > > All csd files have been respectively name and put under .orc and .sco
> > > extensions
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/rec/the-collapse/commit/44e9f41c4914d23d7dd197b0b8686b81f6763d68#commitcomment-846976
> > >
> >
> >
> >
> > --
> >     /t
> >
> > http://radio.swirly.com - art music radio 24/7 366/1000
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/rec/the-collapse/commit/44e9f41c4914d23d7dd197b0b8686b81f6763d68#commitcomment-846978
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rec/the-collapse/commit/44e9f41c4914d23d7dd197b0b8686b81f6763d68#commitcomment-846986
>



-- 
     /t

http://radio.swirly.com - art music radio 24/7 366/1000
Hi Colin, I removed the fromdos-part. I think it isn't needed because you are stripping out the \r in the eval in line 273. I rebased it to the master. Thank you for creating modman btw, I use it in all my Magento Projects ;-)

I have one suggestion - I would really like to see is the possibility to use git with submodules. I have the whole Magento under git control, and atm I have to manually init the submodules under .modman... Updating is working fine already. Shouldn't be that hard to implement, but I didn't had the time to work through modman in detail yet.

Regards,
Ingo
These changes weren't live already? I think I'm more tired than I thought O_o;
Max suggested EmoQuery

I think he might be right.
I gave you read-only (I hope; don't see an option to control that) access to tarsius/epkgs.
Sadly, yes: http://dpaste.com/hold/703204/
I'm a huge fan of avoiding deepcopy like the plague unless you know exactly what structures are getting copied. It's probably better to keep imagefile above around and if it kicks up an error, just re-do the Image.open call.
I propose to add some prefix here, for example: `'controller-'`.

For two reasons:

- Large systems will probably use many aliases. It can cause a collision. Serives like "mailer" (both `Zend\Mail\Mail` instance and mailing sending controller), "acl" (both `Zend\Acl\Acl` instance and right management controller can be ambigous.
- If you will have default routing enabled, it will most likely map %controller% right from URL to your DI - I know we check if fetched instance is of type `Dispatchable`, but still it will call it's constructor (and in more complex DI configurations also cause some injections). If you will have anything automated through DI (for example logged with associated write streams) it can be initialized in unpredicted way by just entering URL.
@gitkeeps - I was unable to find a ticket for this on tickets.opscode.com. Can you create a ticket and link to this pull request?
lol my bad :P
Regarding that fear issue, check if StopMoving() is called @ finalize of fleeingmovementgenerator
If this change is merged in, does this cause smeranda problems when he pulls in updates? Would it be better to cherry-pick his commits, rather than commit his changes under your name?
For 3.x, it was released a long time ago, but it's since been removed in 4.0.

This is because 4.0 generates the slug with a before_validation callback, and keeps the slug in the same table as the model, meaning you can now just use regular Active Record validations for this.

4.0 still includes a basic reserved words functionality by default, but only to avoid letting people use slugs like "new" that could mess up an application's routes.
This fixes: If you login, with "order by status", and "group offline contacts" is disabled, you will see all the contacts on the root.
see newer commit if you can, thx.
shouldn't t be:

    t='./manage.py test -x --logging-clear-handlers --with-nicedots'
Le 03/02/2012 20:36, Rob Nelson a écrit :
> It's not mandatory except for when you compile.

It causes runtime exceptions if nethar is not installed:
at
org.bukkit.plugin.java.JavaPluginLoader$103.execute(JavaPluginLoader.java:1024)
... 13 more
Caused by: java.lang.NoClassDefFoundError:
org/akrieger/Nethrar/NethrarMinecartTeleportEvent

-- 
Léa Gris
I assume context.Light is in fact a CVS light and not a Gladius one?
REmove the , ENT_QUOTES parameter from the xla() function
Утечку памяти так не отследить.
Is any one calling updateRate? If not, probably make ratesPerSecond in EventThrottler protected and remove the dynamicRatePerSecond member?
Проверил сборку с gtk2: падает. Но это не отменяет того, что сборка с gtk3 изменяет локаль на лету без проблем.
This line is a bit weak.
can you help me with this? ping me on IRC
This was the big kahuna...
Hi Guillaume,
You'll need to add the play.version parameter to the Windows build.bat as well.
Aha. The `codeKeywords` strings are definitely unicode, so for jython I think the best is to convert them to ascii. (Watch out for dupes after asciification).

I think [java.text.Normalizer](http://docs.oracle.com/javase/6/docs/api/java/text/Normalizer.html) might do the trick - haven't tried it. It might not work for Chinese, Arabic, Hebrew etc - so maybe fall back to EN if a language's keywords can't be ASCIIfied?
Why not use ToDictionary() here?
reading gold and generated into wrong variable names.
Hisotry?  Is that a legacy typo? :D
So this code is still in the early stages of development. I have not had a massive amount of coding experience in the past so please be gentle with me :)
LULZ best README evar!
А только меня одного смущает что фактически распределенное кеширование тестируется на одном и том же кеше?
While you are at the ES5 fixes might wanna spruce up `Function#bind` too.
I'm not 100% sure we should cover for that. These exceptions don't give more info than the NPE. At least as they are
sv_setref_pv, which turns C objects into Perl objects, takes a void*, not a pointer of the actual object type. CLASS is the thing that actually does the magic. This will speed things up a little and it also makes writing methods that can return different classed objects easier.
I think you forgot to include a patch for the root pom.xml to add your module
Also:
consider adding the saucer that flies across the screen in the back and when shot restores shields
consider causing the explosion sprite to replace the alien when it is killed
* I added a red tint to signify damage
Oh right...  CLI.    I do need to figure out how to build this thing on mono, though.
I think this is not even needed anymore...
This zip.  What I was doing just before I commit this was actually incorrect, but I think that was what was actually working.  Something is wrong..
True. And since you're usually discarding food and ammo, it should keep the deck small.
I don't see the point in running a full relational DB in dev mode, 
until the queries require thousands of rows for adequate testing. YMMV 
obviously.

Can Gemfiles include other files? I could make a ~/.Gemfile.db-prefs 
and include it if it exists, else 'gem pg'. Too hackish? Gemfile.local?

I have node, but only stuff in my Gemfile gets initialized (using 
rbenv) so everything I use has to be in there. Rails 3.[12] won't work 
on Linux without a JS interpreter.

--Kurt

On 20.02.2012 19:35, Don Park wrote:
> Any idea how we can keep this per-developer? I like using pg locally.
>
> ---
> Reply to this email directly or view it on GitHub:
> 
> https://github.com/neophiliac/crowdstarter/commit/6fb0c46c1471585c29f29f7411bc00cefd8bfeb8#commitcomment-988126
Awesome thoughts :-)

One request though, can we extract the logging into a method/function so we can call it abstractly?
Hi Madhu,
  Sorry for the delay in responding.

  Do get bio mismatch responses (300) if someone else's biometric is
  used with my Aadhaar number. Used to get this all the time when one was
  using a valid Aadhaar number but my finger(s).

  The steps involved in generating the PID XML is as follows:-
    * The ISO 19794:2 template (FMR) is encoded first (Base 64).
    * This is made as the content for in the "Bio" XML node of "Bios".
    * "Bios" node is the content of the "Pid" block that also has a
      timestamp. If the "ts" element is incorrect, you would get
      error code 561 or 562.
    * The PID XML is encrypted using the 256-bit AES Session Key.
    * The AES encrypted PID XML data is encoded and made a part of
      the "Data" element.

  So, you need to ensure that the "Skey" XML element that is encoded &
  encrypted using the Aadhaar Staging public certificate, is the same
  one that is used for encrypting the PID XML.

  Hope this helps.
warm regards,
subbu

On Tue, Jan 31, 2012 at 03:52:27AM -0800, madhu402 wrote:
> Hi subbu
> As i was embedding the one person's UID and other persons Biometric It
> should generate the erorr code as 300 but I am getting the error code as
> 999 may I know the reason please and Do let me know whether you will
> able to generate the erorr-300 with the wrong biometric detail of the
> other persons.
> 
> Thanking you
> Madhu
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/GeoAmida/AadhaarAuth1.5/commit/0136df6f8dcc7acb4df948fb982aac4771fd4ee9#commitcomment-917947

-- 
Dr P R Subramanya | VP - Technology
Geodesic Limited  | www.geodesic.com
Ph: +91 80 6655 1000
Mob: +91 98459 80500
esto esta dulicao...cual es el bueno?
и семак тут О_О
Fu, i just called machi on the related issue.. Thanks Q nevertheless :)
Thanks for your answer Machiavelli, every day learning a bit.
IMO this should be: `new \ReflectionClass($all[0]->name]);`
You're right. I forget to delete this ^^ I'll do that and update the pull request
@gstein Thanks for the input. I think I'll put this off until other breaking changes have been made.
Ah, I'm glad you were paying attention.  I couldn't figure out what the heck was going on and no matter what I tried (which probably wasn't much), I couldn't get the conflicts to resolve without adding the file.

By the way, I'm using the SourceTree GUI (as opposed to the command line).  As soon as I think I have things slightly figured out, I realize, I know pretty much nothing.

On Feb 21, 2012, at 3:26 PM, Jason Grout wrote:

> It looks like this gets added because of this commit: e76c078326e920a80e1bbc6264e7e00d1c942d49, which added a pdf file.  Did you pull the changes back down to your repository with `git pull` to get the `.gitignore` file on your local computer?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/dcernst/IBL-IntroToProof/commit/3dbcc2380f35c4db6c5f5035358258bd6b762e2a#commitcomment-993124
Mcs6000  driver updated eh?? So i guess this kernel can be used with ICS??
This is a good idea, but at some point it should go in God.
Con esto el comportamiento cambia un poco, ya que por ejemplo anoche a las 23h aún saldría el evento de la tarde, sin embargo no es para nada algo grave y de hecho muchos sitios se hace así. Lo que está claro es que usar DateTime directamente tiene este problema que mencionas!

Quizás sería más ajustado usar format('Y-m-d H:00:00'), y así se cachearía cada hora... o directamente dejar el pull request tal cual con el "problemilla" que comento más arriba... cómo lo veis?
It actually already makes the link if the templateId is passed, it's just not enabled. So, the link information is stored.
Just wanting to tell you that you should have upped the version number or change the magic if you upgraded the wallet format :x
For future reference, details could be found [here](http://www.cmake.org/Bug/view.php?id=12935).
We could either choose an empty string or "stable". Which one do you prefer?
la coma está mal puesta, pero para que es esto??

``` ruby
alias_attribute :name ,:id
alias_attribute :name, :id
```
I'm a fan of the Ternary Operator, so i've done the following, both at notification AND subscribe (Line 93):

```
                   title: ((title !== undefined) ? title.getText() : ';'),
```

Depends on taste...
You're right. We'll have to think about how to name these methods more clearly. Thanks
okdesu
Modifiche per la nuova versione di morituri che non sembra presentare più il bug relativo al non funzionamento del profilo wav che prima mi aveva costretto a rippare in flac e poi decodificare in wav. Sistemato anche un problema con le palylist.

TODO Copier il log e il cuesheet di morituri nella directory di output dalla /tmp
Good spot, if you want it on develop then absolutely, but I believe everything that should be is reraised on the type-conversion-branches. Let me know if I've missed anything.
Why does it need to ``use`` twice?
Both package have their own duty for this test file. Mechanize can't return context object and Catalyst::Test can't make user interface testing simply.
This issue was your values are in seconds when the methods uses milliseconds.. so they were far too short.. fixed now
This CSS-Highlighter is evil! :P
Good point, I added in the css.
Ah, fine, fine. :)
nice alternative, thanx!
The following methods are all convenience methods that help prevent the tests from having to worry about manually cleaning up their temp files.
This is generating the width of the string in spaces, but this doesn't appear to allow adequate space to fit the text.

I tried size="8" which is enough to comfortably fit the text "Hi there!" (fills up 2/3 of the text field in Webkit.) Rendering this with flyingsaucer, it fits "Hi, th" with a tiny slice of the "e" peeking through.

The HTML standard doesn't specifically say *which* character to use the width of here, but I assume people are picking en or em.
This does nothing, these URL patterns are already included in the other urlpatterns (line 66-73)
I know! Its check for other.
The size of the .css files produced by themeroller have about doubled after these last few updates it seems (e.g. with swatches A through E filled I'm looking at 81KB for the minified version)...is this how it should be?

Also, how many swatches do I really need if I'm not explicitly assigning swatches on any elements, the default 3 (A B and C), or D and E as well?
Do we want to reference the latest DLL's? I think EnvDTE80.dll is for 2005. EnvDTE90.dll and EnvDTE90a.dll should be for 2008 and 2008 SP1. EnvDTE100.dll should be for 2010 (and 2010 SP1?).
Мне исправлять, или сам исправишь?
Yes, But the server status already doesn't show all of it unless they are logged in as Admin. So it can't be spoofed, and banned.
Does it make sence to leave a choice between "Experimental factor" and "any"? what does "any" mean now?
Why don't you use htmlAttribute to implement this?
My bad. Read between the lines. It actually is :)
So glad this is a separate class now.
This one can be folded into the pattern match.
Yeah, @rkuhn is a genius
didn’t know that double-brace suffices
why are you sleeping? if waiting for some event, that should be explicit, if not it needs a comment
why check isEmpty prior, does addChild throw up if already exists?
Ok. Don't really like it when testing pollutes the code, but no other way to do it I guess.
… because I don’t know which kind of pants Patricia wore initially?!?

Ah, now I get it. Yes, will change.
Akka will only be binary compatible within minor releases. (so 2.0.x, 2.1.x etc)
As it is used now it's safe, 0 indicates that it's empty, and things will be routed to deadletters.
I can add another `isEmpty` method to make it more clear
we should really really start standardizing how we build these too. not for now tho
Different set_names Array for each 'kind' now
FALSE - default value
I think 

    if self._size == size:
        return

then the current code. would be better, if you are sure removing the super() call is not likely to break stuff :)
This is pretty minor but it might also help to put `()` around `self.model.Lambda / np.pi`. `r * s * A` or `A * (r * s)` can be faster than `A * r * s` when A is a large array because you force python the multiply all the sealers first before multiplying anything by the array.
KEY_ASCII is needed because later on the code does a | KEY_ASCII that will generate bad key data :(

I can change the false to 0 but if I remove the cast then the compiler does not know witch constructor to use, so no choice I'm aware off (I'm not good at C )
Oh, but you're doing math. Never mind.
This is now the only place we use `python_bin`.
shell.setCurrentResource(resource); should be used instead
I'm going to have a look at it again though.
If we are NOT in the vtrash folder, then hide deleted message.  Without this patch the method returns true (so deleted messages are hidden) if you are in the virtual trash folder and false if you are in the virtual trash folder, which is the opposite of what we want.
Indent errors (plural, i.e. not just here). Use two spaces, not tabs.
sounds good, we could also consider to document this on a central place instead of having this note on every event..
Good point.  Can't we just say "File fileOut = File.createTempFile("lib", null); fileOut.deleteOnExit();"?  That would also allow us to get rid of the first few lines of this method.
See d3_geo_pathContext comment re. type coercion.
For now without changes in idn support and String.getBytes this is the only option to get patch compiled and pass test cases
C++ comments aren't portable.
No. I use window.onload because it's a property of the window object. It
makes more sense that just "onload", and I've never done it any other way.
On Jul 22, 2012 9:53 AM, "Richard van Velzen" <
reply@reply.github.com>
wrote:

> >               php_info_print_hr();
> > -             php_info_print("<h1><a href=\"");
> > -             php_info_print_request_uri(TSRMLS_C);
> > -
> php_info_print("?=PHPB8B5F2A0-3C92-11d3-A3A9-4C7B08C10000\">");
> > +             php_info_print("<script>(function () {\n");
> > +             php_info_print("    'use strict';\n");
> > +             php_info_print("    window.onload = function () {\n");
>
> @nikic Apologies: that should have been `onload`, of course.
> @TazeTSchnitzel I guess you also write `window.alert()` and
> `window.document.getElementById()`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/php/php-src/pull/132/files#r1211614
>
settings.MAX_DOCSTRING_WIDTH, I guess. Thanks!
: не там
к тому же, получается круговая зависимость. убрать отсюда этот инклуд
убран
Ok. Thanks. Will fix that.
We probably don't need wrap here. Just `Array(formats)` (I think this is the rails 4 pattern).
We don't need to change the entire file, but all the new code should be written using the 1.9 syntax
@shime thanks! I didn't know about that.
indent subsequent method calls:

```
ago(1.week).
  beginning_of_week.
  ...
```
Sorry for the delay, I can't think of anything off the top of my head now (and I'm sleepy anyway), but I'll try to think about something and bring someone else to take a look, thanks!
When most of a method is right next door, use it:

    def isRepeated(param: Symbol) = isScalaRepeatedParamType(param.tpe)
This is a pretty tricky area; you can't make these guys contravariant, as Paul's thread highlights; it's not clear how to associate compatible Eq/Hash; type class based equality doesn't play nicely with subtyping. Best not to be too hasty with this change, IMO.
``x1, x2 = [1,2,3]`` wouldn't be caught here. Perhaps:

    if not len(brack) == 2:
        raise ValueError('bounds must have two elements.')

    x1, x2 = brack

Why is parameter called ``brack`` while error message refers to ``bounds``?
So you're thinking it'd be good to also change the existing `cull_members_from_pool/3` to use variable units rather than minutes? I'd left things in minutes to match the existing function behaviour and the description of `cull_after` as described in the README.

Another potential problem is how precise this needs to be. For example at the current minute resolution, say set to 5 minutes, an item might not be removed for almost 10 minutes should the item be not quite 5 minutes old when we do the cull check.
I can remove `is_win?` and resubmit if you want. I left it in there because I thought it would be useful to know and figured it would be used later when/if someone figures out a way to check for image magick on windows.

@radar please let me know if you want me to remove it.
Note that Sergiu is about to update this part: the name will be stored as `Symbol`, **unless** the name is empty. The latter case is supposed to indicate an "anonymous" object, no two of which are supposed to be the same. I suggested Dummy for this case - it seems to match the behaviour of Dummy quite precisely. Is it still a bad idea?
Nitpick: even if 2.0 represents the real number "2.000...", we typicall 
do not have "2.0 \in N" set-theoretically.

Obviously this is all silly and should not influence the policy we 
adopt. I think for Floats it makes sense to adopt this "best-guess" 
attitude you implemented. Everyone should be aware that "Float in 
Natural" type expressions are somewhat quirky.

On 29.03.2012 14:58, Matthew Rocklin wrote:
>> +>>>  S.Naturals.intersect(Interval(0, 10))
>> +        {1, 2, 3, 4, 5, 6, 7, 8, 9, 10}
>> +    """
>> +
>> +    __metaclass__ = Singleton
>> +
>> +    def _intersect(self, other):
>> +        if other.is_Interval:
>> +            return Intersection(S.Integers, other, Interval(1, oo))
>> +        return None
>> +
>> +    def _contains(self, other):
>> +        if other<=0:
>> +            return False
>> +        other = sympify(other)
>> +        return other.is_integer or (other-int(other)) == 0
>
> I think that floats that exactly equal an integer are, in a mathematical sense, integers.
>
> When we ask 2.0 in S.Integers I don't think we're asking about the datatype of 2.0. I think we're asking about the mathematical value.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1155/files#r621103
>
Please also explain what the `globals` option does, and what the default (`None`) really represents.  It should be clear that `globals`, like `locals`, should be a namespace dictionary.
And he raises it again if occurs a second time, so this should not be a problem (it may be changed to Runtime error so it does not catch ctrl+c). Anyway, because of the hackish behavior of lambdify and the lack of any consistency of the error types raised by sympy and numpy, this is necessary.
Thanks for the information, I will go through my code and merge the pull made by you, test it and will get back to you.
This -> The
is -> are
Does this fail gracefully if the solve fails?
Still missing visibility keyword "public"
Can you please add comment markers to this line? A documentation generator may not interpret these two paragraphs as part of the same block if there's an intervening blank line.
This line doesn't seem to make sense.
OK. I don't think there's a test case for this (although I did manually test myself).. I can make one I guess.
What's the value add of these 2 implicit declarations?
Another option for these is to load a "Duration" and then convert that into millis.
Probably a bad idea to include `content`. Loading every attachment and encoding each into headers to send in the KumaScript service request could end up very painful. If something in kumascript needs the contents of an attachment, it should hopefully be rare. In that case, fetching the URL on demand should be a decent alternative
There's a function that handles this better: [request.build_absolute_uri](https://docs.djangoproject.com/en/dev/ref/request-response/#django.http.HttpRequest.build_absolute_uri)
you dont have the prepared data in row in the first event.
Missing a period at the end of this sentence.

Also, these language entries contradict themselves. Does this display the title of the most recently replied-to topic or the most recent post? Posts can have different titles than their topics, so it is an important distinction.
This parameter should probably be called `$user_ids` without the `s` as in other places.
This code path is meant to be taken when $locator is not initialized, i.e. when template engine is used without extension manager (?). Looks like I never finished/tested it.
Docblock please.
I see what you mean. I haven't played within a ton, but playing with it in gitstar isn't gonna be much good for that since the biggest bodies are still only maybe a few hundred bytes. The bigger problem would be if there are lots of simultaneous uploads of large files or something... I think for forms it's definitely ok. I like that by just exposing trusted iters we have the flexibility to implement other oneslike you're suggesting.
You've inverted the conditional here, I think.  Previously it forked, and ran the worker in forking mode...here you're only forking if running as single process, which seems backwards.  You might also think about running the single process worker in a separate thread.
This is not longer needed, was solved at the store level, files will always have `stat.mtime`.
why not getResources(env, ctx, ...);
Yeah, it took me a bit to realize it was translating from FS path to URL. Perhaps the comment should say that instead:  `// translate from filesystem path to URL`.
Done, i'll push these changes when I add the poll tests we talked about.
Probably. This is a work in progress, but doesn't have to be completed for these changes so far to be brought in.

Sent from my Windows Phone
________________________________
From: Brendan Forster
Sent: 13/04/2012 7:55 AM
To: Jake Ginnivan
Subject: Re: [carnac] Some visual studio shortcut text changes (#53)

>        keys:
>        - Ctrl+F
>
> -    - name: Edit.FindAllReferences
> +    - name: FindAllReferences

Should these be properly named, like "Find All References" like some of the value above, or just the key?

---
Reply to this email directly or view it on GitHub:
https://github.com/Code52/carnac/pull/53/files#r679683
Sort of. It creates sorting indexes -- groupsort_indexer does that. You can get the sorted DataFrame with:

gd.parent[gd.idx, :]

It's not really sorted because the pool isn't sorted. If the pool were sorted, the DataFrame would be sorted.

By the way, your SubDataFrame and PooledDataVec fit in great with this.
Right now this is a magic #... why 120000??

Instead, 120000 should be saved as a var CONSTANT_NAME = 120000  which describes its meaning
style nitpick: can be on one line:
```var keyMap = {};```
Where error is defined? I do not see it imported.
Typo in comment "and hand"
beggining date should be lower than end date
http://twitter.github.com/bootstrap/base-css.html#typography

Use:
<pre>
<ul class="unstyled">
  <li>...</li>
</ul>
</pre>
This finish ought to be `0` to represent success, yes?
Please use pcre_free as requested by PCRE docs. Might be importand under Windows.
The class must NOT  BE Serializable. You are right, it does not make a difference anyway, but I will remove it.
Examples for these two attributes would be good; I can see people trying:

```python
class MyUser(AbstractBaseUser):
    email = models.EmailField(...)
    ...
    USERNAME_FIELD = [email]
```

Which "works" in that it doesn't fail at load-time, but fails later on in really fun and interesting ways.
Once again, exec is not needed here.
Did this `"@" in username` check get moved elsewhere, or just removed? It looks like the latter. That's fine by me, but it *is* technically a backwards-incompatible change, so it should be mentioned in the release notes.
I don't see __primary__ mentioned anywhere else, at the minimum this needs a comment as to what that magic string is.
Would you rather I tested for table existence first?
should we add an AbstractVisitor implementing dummy methods for the interface so that you only defines the method you really need in child classes (similar to Twig extensions or Symfony form types) ?
I think that this patch could be more general if you leave the closing parenthesis alone on a line (as a final result of the patch), such that if we add a calculator you don't need to change that patch.
I'm more concerned about unitless numbers like opacity. Would you accept this pull as-is @gnarf37, or do you still think it should change to `start = 100`?
Right, they don't technically NEED to, but  they go into it anyway because I still test for prefixes in those browsers. It's 1 test per property name, and it makes the code small as possible without having to feature detect IE6 and IE7.
Why does it need FOR loop?  It's simply

```sql
SELECT madlib.__assert_table(table_name, 't')
FROM(
  SELECT madlib.__regclass_to_text(table_oid) AS table_name
  FROM ...
)s;
```
Thanks. done.
On Apr 5, 2012, at 9:25 PM, Hitoshi Harada wrote:

>> +    Wrte some simple scripts to convert the data in the model files to the data format required by our package.
>> +  - invoke function madlib.__load_crf_model('/path/to/data') to create tables and import the data into tables.
>> +  - Run feature extraction in batch mode or interactive mode
>> +    <pre>SELECT madlib.__text_feature_extraction(
>> +         '<em>segmenttbl</em>',
>> +         '<em>dictionary</em>',
>> +         '<em>labeltbl</em>',
>> +         '<em>regextbl</em>',
>> +         '<em>featuretbl</em>',
>> +         '<em>viterbi_mtbl</em>',
>> +         '<em>viterbi_rtbl</em>'
>> +         '<em>mode</em>');</pre>
>> +    if mode equals -1, then the function will do feature extraction for all the sentences in the segmenttbl,
>> +    else it will do feature extraction for the sentence specified by sentence id mode.
>> + 
>> +  - Run viterbi function to get the best label sequence and the conditional probability \f$ p(top1_label_sequence|sentence)
> 
> It is obvious that you missed \f$, which prevents from generating document for this block.
> 
> <pre>
> textfex.sql_in:122: war
> ning: reached end of file while inside a f$ block!
> The command that should end the block seems to be missing!
> </pre>
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/madlib/madlib/pull/103/files#r653610
>
Only one channel? This is intentional I assume?
Don't use sed to modify files - we provide an `inreplace` method to help with this.

```ruby
inreplace "file", "str", "replacement"

(or)

inreplace "file" do |s|
  s.gsub! "str", "replacement"
  s.gsub! "otherstr", "otherreplacement"
end
```

Ideally though, since you're the author, maybe the changes could be made in the software itself in some way so we don't have to patch it at buildtime?
Thanks for the info.  I had already changed it to a Requirement asking the user to do that.  But I will just remove it as you say because it's not needed.
Are these required on all versions of Xcode, or only 4.3+?
Je pense que tu peux enlever le code de debugage
Could you improve this prahse please ? I understand the first part, but starting with *[...] and passes them to it [...]* it becomes blurry :)
... and this loop would iterate over `delayedInjectedClasses`. Instead of going from type to string in the first step, it would do the opposite: ask the processing env for the type from the string.

And as an extra bonus, removal is way more straightforward. Your loop would look like this:
```
for (Iterator<String> i = delayedInjectedClasses.iterator(); i.hasNext(); ) {
  String injectedClassName = i.next();
  InjectedClass injectedClass = getInjectedClass(...);
  ....
  if (!missingDependentClasses) {
    writeInjectionsForClass(injectedClass);
    i.remove();
  }
}
```

This way you're removing from the set you're iterating, as your'e iterating it. Seems structurally simpler than two sets.
You mean processing `@Inject static` fields in `InjectProcessor` to generate such an `injectStatics` method (there would actually need to be 2 methods –`attach` and `inject`– and a bunch of fields to store the bindings between these two phases), and simply call it/them from the `ModuleAdapter` generated by the `ProducesProcessor`?

The issue is that, if we don't want to refactor the whole static injection stuff, we'd have to have instances of `StaticInjection` lying around that `StaticInjection.get(Class)` could return, so it's transparent from the `ObjectGraph` and `Linker` points of view.
I guess we could make all generated `$InjectAdapter` classes implement a `StaticInjection` interface and have the `Linker` look it up by requesting a binding for the class to be injected and then casting it (falling back to reflection if the returned binding is not assignable to `StaticInjection`) but I'm afraid it would complicate the code a bit too much (in readability, mostly). I can try that if you like though.

> I want to limit the number of generated classes!

Hopefully there shouldn't be that much classes requiring static injection ;-)
It's probably a trade-off between using reflection (a bit slower at runtime) or codegen (slightly bigger artifacts). I have absolutely no idea what could be better in practice (particularly on Android).
Thanks for catching this, @amb26--duh! :) My apologies.

Given the (admittedly fairly remote) possibility of a (falsey) key of 0, the latter check is probably the safest and shortest.
do we have environment elsewhere in the hapipy code? if not, remove it here.
I don't see anything in the CopyOnWriteArrayList Javadoc that indicates that it blocks. COWAL was designed exactly for this purpose.

I don't see how concurrently running tests could could on whether or not the listeners that were added were called for other tests before or after the current test passes. They could, of course, count on whether or not JUnit will call the listeners concurrently.

I really think COWAL is the simplest thing that would work, and it's optimized exactly for this use case. Could you point me to a resource that says otherwise? I'm honestly curious.
Another "since" tag above a "package" statement. I don't think these show up in generated Javadoc
Seems like it is a thing http://jsperf.com/delete-vs-undefined-vs-null/3, although this code is executed once a click and jsPerf puts a single delete operation at like 30 nanoseconds.
Wtf ? Who is fucking with my RedstoneKey if Statement?!
Should we be publishing when it's ready perhaps? For instance, we don't actually do anything when dfd is resolved right?
I think `previousSize` is an ok enough name.
No, it is also found in the `plugins-client/ext.debugger/debugger.js` interface
However, this isn't used from `debugger.js` - So, it can be removed
Should this not be: app.user_registration.unique_id?
Shame on me. You are absolutely right. I reworked the commit according to your comments.
This exception will be written in the logs and in the console output. It would be nice to add some valuable information : the location of the file and some tips, for instance.
"Check the content and permissions of "+currentWK+"."
When you say toggleContent here, do you mean display a particular tab? This function would more mean display tab. What are contentString and buttonString? I don't understand what a lot of these variables are for. We'll discuss in our meeting.
The phrasing here is a tad confusing. Perhaps “Replaces the content of the node denominated by the id with the markup given by content”, which will also align it with the verbiage on previous docs.
Mnja mnja mnja. Please provide links for this argument. And please also search for links for my "she" argument. ;-) I think both goes, no?
Hmm... is it only the header architecture one that is broken?

Is this an unrelated bug? After all I just did it the same way it was already used.
How would someone interrupt a drag? The stop function is fired as soon as the person lets go of their mouse, which of course then resets the point-events back to auto.
Yeah, using a constructor for this might be simpler.
Why not take out the warnings?
This should be `share-editor`, to conform with file-naming convention.
I don't think so. Doesn't querySelector cover specifically css/ids? I'm checking that those attributes were removed.
Yeah, waffled back and forth on it. But you are right, I should probably use constants.