labofsahil/github-event-dataset-2012 huggingface dataset:
Trad motorinn-lee-lilly + modif clémentine

Voilà.

Tu peux m'en donner deux autres à la fois pour le prochain coup. Ou alors tu peux m'attribuer une liste qui t'arrange. J'ai du temps pour m'en occuper en ce moment. Autant en profiter.
merge all of my work into the teamhacksung repo and perform a pull rqst

The day has finally come that Cyanogenmod for the Epic 4G Touch is stable enough to submit a pull request for you to update your repo. This is thanks largely in part to the base that codeworkx laid out for me and the help he has provided along the way. All of the major bugs have been ironed out, and this is official enough for me to send it over to you for acceptance.

Changes will most likely take place in the future, but this in conjunction with c1-common, which I also submitted a pull request for, will result in a successful output of a nightly.
Nothing to assert here, citizen, move along. More seriously, I'm relying on the cal to notifyDeletedEvent, which doesn't return anything I can assert.
Nice addition to your ignore file sir.
Maybe at this point a had loosed code coloring.
In my storage there were this settings object:

```javascript
{"show":{"html":true,"javascript":true,"live":false},"theme":"default","lastTip":5}
```
You can see, that ```jsbin.settings.theme === "default"```

So, I had to manually set it to ```jsbin```, like so:

```javascript
localStorage.setItem("settings", '{"show":{"html":true,"javascript":true,"live":false},"theme":"jsbin","lastTip":5}')
```
Ah, you meant the parameter? SDL_RWops *rw

Ah indeed, based on what we saw, it was non-obvious. Thanks for your understanding. ;)
I guess you cleaned it up in the caller?
<!DOCTYPE html> ;)
@isaacs: Opinions? I suppose it's kind of useful but it also looks like something that's prone to break after a V8 upgrade...
So this is the include path post-installation? Just clarifying...
Will do. (Within an hour, say.)
That is a good idea. I'll add a unicode option.

On 8 March 2012 21:08, Ben Gourley <
reply@reply.github.com
> wrote:

> Why not use the ellipsis character ( / unicode escaped: '\u2026') here?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/serby/trunky/commit/6deb2253f257d0a3e2b155f770f00e57a4589eac#commitcomment-1062964
>



-- 
paul.serby@clock.co.uk
Chief Technology Officer

Mobile: +44 7881 550999
Conference: +44 20 3051 6144

http://github.com/serby
http://twitter.com/PabloSerbo

Clock Limited
The Old School House
Bridge Road, Hunton Bridge,
Kings Langley, WD4 8RQ
Switchboard: +44 1923 261166

www.clock.co.uk

http://twitter.com/clock
http://blog.clock.co.uk/

www.clock.co.uk/email-disclaimer

clock. is a Registered Trademark of Clock Limited in the European
Community and other territories
Setting `options.agent` to `false` won't have the effect people would want. Just remove this default and let Node sort it out.
COOL!!!
this will be great;
so far i have stumbled on one bug: the "next()" method in MacroCommand throws java.util.NoSuchElementException (there is no next?)
ok, i like a new way to changing colors and keeping fg&bg colors together. but i don't like changing colors of UI. Also keep your contrast very high - this is educational application. Here is a tool for measuring contrast for web: http://juicystudio.com/services/luminositycontrastratio.php you can find that at least a value of 7 is needed for the best conformace - i think at least 11-12 is needed for decent contrast. (ok i am blind but many students of CS are blind too ;) )
also i will need a new value probably called CACHED. and for sure i need different colors for FIND and FOUND.
Well, your approach is "cleaner" in that it uses objects instead of strings, so if the perf is the same, we should keep it. I was under the impression that innerHTML is faster than dom generation in chrome and firefox.
well, the problem is that the function must be flexible to account for different directory structures.
for example mine is:

/app_root/modules/module_name/api/classes/

the module name is then the root path for the classes in there.

for example my.api.com/module_name/class_name

So you would need to scan for directories in a certain structure.
Then you will have the issue that scanning your directory structure all the time is completely idiotic to say the least.
A solution for that would be to create a cache based on the first directory scan. ( create a cache php file with the AddAPI function )

Then if you change something you can remove the cache file and it will re-init it self.

but then again, others may have other directory structures. and it's hard to keep it flexible...
I'd propose a small modification:
"Shows an effect at the given position or on the given being. "
"The player's character is targeted if neither of them is provided."
see https://gist.github.com/1087739
Would initializing to 0 be an option?
I also think, that !isEmpty() is more readable, and converting it to size() != 0 will definitely not make BaseX any faster (it well may be, that the runtime actually optimizes it on the fly). But in theory, !isEmpty() has one additional operation (the negation). Anyway, it's more a question of personal preference, than a real optimization :)
Also, if we're already changing the core things, can we implement dynamic command registering/unregistering?
Can we put a platform check in here and not even bother calling the VMS logic? Use org.jruby.platform.Platform.IS_OPENVMS.
This your way of asking me to create a ticket?
Cool that you're using the 2.0's dispatcher API -- I commented on some code that was for 1.3 :) What do you think if the user did the following instead:

ext.newSocket(...).withDispatcher(...)

and 'zmqdispatcher' dropped altogether.
What about this?  I know, chmod()'s not flagged as can't-ignore-its-return-value.  But still, I think we should do more than shut up the compiler.
У меня недоделанный еще один рефакторинг, сегодня допилю, наверное, чтоб убрать проверки if rule[0] == suffix, которые, как ты правильно заметил, основной тормоз. Для этого решил не отдельный доп. индекс строить, а структуры данных поменял (словари нужно будет переконвертировать, но их один фиг нужно будет переконвертировать, т.к. в них ошибку нашел): в rules массив tuple меняем на словарь:

    {paradigm_id -> {suffix: [(ancode, prefix)]} }

В словаре теряем информацию о порядке; чтоб можно было первую форму получать, вводится normal_forms:

    {paradigm_id -> (suffix, ancode, prefix)}

endings пришлось поменять, т.к. "индекс правила" смысл теряет, для парадигмы прямо правила записываются
  
    {word_end->{paradigm_id->tuple(possible_rules)}}

Там почти все готово, ускорение в несколько раз, но некоторые тесты не проходят еще. Твой коммит с примером *очень* полезный был, чтоб понять, как лучше делать, так что это тебе спасибо огромное! 

Возможно, еще какие-нибудь алгоритмы могу выиграть от быстрого получения списка слов, являющихся префиксом для данного - в этом случае нужно будет рефакторить дальше и утащить еще куски анализа к источнику данных.
I thought I wrote a response to this? Oh well...

I will merge yours into my branch, the only thing I have which you don't is an extra constructor for TemplateEngine. I have fleshed out an adapter for Nancy which should work with either my branch or yours. The main issue now is allowing NHaml's innards to use an outside source for getting partial and layout view data. I don't see a mechanism for doing this except, maybe ITemplateContentProvider, but that doesn't appear to get passed through to the compiler?
Yes ;) Fixed
Да ступил я с ИК =) Неверно меня информировали =) Сорри, да там всего навсего затуп с ммапсами.
I'm testing it and charge doesn't stuns well, it has an 0,5 of delay aproximately (delay after you arrive at target's position). In addition sometimes you turn around when you charge.
This statement isn't needed.
Having `choose` be an alias for `binomial` seems far more reasonable to me.
This change let every Lighthouse double as a Moat!
*why gtk-doc?
*
On Tue, Feb 14, 2012 at 4:27 PM, crab2313 <
reply@reply.github.com
> wrote:

> 不用gtk-doc么
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kernelhcy/gtkqq/commit/365416c8877f10368cd9b237ea7e3371923b5cdf#commitcomment-963632
>



-- 

**
*

Best Regards,
**

Xiang Wang| Engineer of Linux

Gtalk: wxjeacen AT gmail DOT com <wxjeacen@gmail.com>

*
Thx for adding zoo to the places I missed.
sorry. some strings - from test code... really need SetClientControl(...,0) and comment preview string with SetMover(.
not sure whats wrong with it :< nvm
Seems like they are not normal, because I can't pass this check when I try to enter them ;P

Or maybe I have something broken in my core, I'll try to check this on r2.
No, I use just clean R2.
I've read that it should steal all stacks of Lifebloom, so probably other spells as well.
http://pastebin.com/r5xmrdfp
no ./
Sieht gut aus, nur die private und protected Member machen es um einiges kompexer, als es sein müsste :( . Kannst du die noch raushauen? Ich bin gerade nicht in Windows und weiß nicht, ob das mit VS2010 einfacher ging, aber so geht es über einen kurzen Umweg anscheinend ab VS08:
http://discuss.joelonsoftware.com/default.asp?dotnet.12.653941.2
eq_(response.status_code, 302,
        "Some reason why this shouldn't fail")
Can't accept this one.
Please modify your local configuration settings is local_config.php, which is .gitigroned
Basically the interface we chose considered groups by body parts, and skin conditions that relate to a particular body part will be assigned to that particular group. However, there are skin conditions such as acne, which occurs on forehead and also behind shoulder. Therefore there can be some skin conditions which cannot be assigned to a particular body part, or group. These conditions should have their own group named "All around the body" . What do you say about this?
Cool! I've never seen this ``qunit-fixture``. This looks much cleaner.
On 2/27/12 11:01 AM, Micah Walter wrote:
> FYI - I have enabled CloudFront so all media is available at
> http://data.cooperhewitt.org/media/350/filename.jpg -- If you want me
> to pull in your changes, send me a pull request when you are ready...
> thanks so much!

Perfect! I will do this shortly.

I will *try* to send you a pull request but I'm counting on it timing 
out for all these reasons:

https://github.com/straup/collection/blob/master/objects/README.mdown

> --- Reply to this email directly or view it on GitHub:
> https://github.com/straup/collection/commit/0b75be025f3f14103f5ad6935b16772188f1c2d7#commitcomment-1016505
>
Ug, thanks.

It must be misspelled in the super class too.
It was a very minor thing which isn't apparent from the context of this change, but when the description is displayed in a tooltip rather than printing, it would instead substitute the current ability/attribute/whatever being viewed for the trait string instead of the ability type. This replacement was not being done in the printing itself, and I found I liked the effect enough to see if it could be fixed in printing as well.

Didn't like that so many files needed to get changed for it though. I'm investigating trying to build these AbstractGenericCharm extensions from data rather than keeping them in code.
Forgot to add in commit:

 `Added extra information about helper properties`
ps: i guess i don't even need to check the args, since checking function rex_a79_textile should do..
and by the way: this whole check via backtrace has no measurable scrtiptime penalty..
No need to explicitely include `Python.h`:  
it's included in `wrapperTypes.h`.
Mark this attribute and the next as `final`.  This is important for thread-safety.
Phew it was you, I was sure it was me: https://github.com/markolson/linkbot/commit/d67a9c14377e9a326950e4f47987c2cf27f35aa7#diff-11
https://github.com/pubnub/pubnub-api/commit/99a36a933862343edb0c6db1d438fbfd13a2c655
remember to activate this switch on -dev
r+wc looks good - this feels so right to be able to have this kind of fine control over the user permissions. I'm excited to see what comes next out of this :tongue:
The atan2 ones seem like bugs that should be investigated.
seems older IE versions don't support overwriting `window` which kind of makes sense, IE9 doesn't complain about it but putting that logic here doesn't gain us the logic needed
What is the original file name? Does it mean the basename of a file or the full qualified name, which should be the same as the URI of a local file?
I'd say not 'in the most cases', but only in old IEs.

http://es5.github.com/x15.1.html#x15.1.1.2
:D
Dinnerbone is just to lazy for 3 line cheery picks :P
Maybe not any more - Minecraft Snapshot 12w07a's change log includes:
* Multi-player light calculations do no longer cause affected blocks to be transmitted to the clients, instead the clients will recalculate the light on their own
Мне кажется что эта линка здесь нужна все таки... зачем удалил? я могу быть не прав
Hi @lucassp!  Thank you for submitting this update.  I have been testing this without successful debug runs.  Is there extra config needed to provide CommonJS functionality for Titanium?
Add Closes 259 into commit message
I don't know, I only use Nexus One stock kernels.
Is a multilingual app really necessary ?
All of them should be filed under `Core/` Afaik. é_é
> Handy for detecting when someone has checked in a bad migration without running it

I'm not sure that's true: AR::Migration loads the class, but doesn't actually run anything if the schema says it's been done.

> or if someone has forgotten to check in the updated `schema.rb`

If you forget to update the schema but run migrations, the migration will run and everything will work, no?

I'm not sure that there's any way to prove that migrations do what they're supposed to short of running them on actual data, which is one of the things that preview does for us.
Normally, but in this case it was like this from before in admin design or something, don't remember, get a time machine and nitpick with me two years ago :P
If the name isn't Model, then all the existing models we have will have to be refactored.
pra ficar melhor pode se usar o ||=

```
def init
    @project ||= current_user.projects.find(params[:id])
end
```
I removed the dependency on MutliJSON but will utilize it if available; this has been pushed and is available in version 1.0.3 of the library.

Thanks again for the input!
I'm so used to writing printf()'s, where the argument follows the string you want to insert it into. Accidentally added that second argument out of habit! (deleted)
Hello Abs62, how are you doing?

I just found out that you have improved the image support on xdxf dictionaries. That's great news! 
There is a pull request from 4 months ago that tries to add image support to stardict: 
https://github.com/goldendict/goldendict/pull/58

Because the user tindzk mixed several unrelated commits in that pull he later created a branch on his own repo:
https://github.com/tindzk/goldendict/commits/stardict-images

If you read the discussion on the pull request, his changes involve 2 things:

1) Prepending "res" to all image paths. This is because lot of dictionaries (if not all) don't include that on the images references. Probably because the resources in Startdict format must be either in a resource database file or a "res" subfolder. 

2) Some image tags are like <img src="\x1Eabcde.png\x1F"/>. So the changes are trimming out those control characters "\x1E" and "\x1F".

As you might know, Stardict format supports xdxf language. He added support only for images in that data type (sametypesequence=x).

But you can see in the Type identifiers section of the official format documentation that there are other data types supported by Stardict:
http://code.google.com/p/babiloo/wiki/StarDict_format

I have 5 Startdict dictionaries with images for testing purposes: 4 use html for content and 1 uses xdxf markup. 

With tindzk's code I can see the images only on the xdxf stardict file. 
With your code I cannot see images on any of the 5 files!

* Can you check your code and tindzk's code? I wonder if you can take advantage of his improvements or not.
* Can you add support for resources to other stardict data types such as 'h' (html) and 'r' (resource file list)?

I will be more than glad to share the sample Startdict dictionaries with you.

Thanks
Okay... thinking about it the problem is in line 99... right? Any good idea how to fix that?
I had a small change of style ;)
Good work. How did you notice ?
thanks for the commit... didn't notice it..
Could you make this combinng optional? I often prefer to have these concatenated things as separate tokes since they're actually separate words with distinct meanings and i'd rather have them split up for lemma-like purposes at times.
Discussed on `#perl-faq` already, but I'm not particularly happy with the mention of Dancer being "best used for small and tiny projects", as it carries the unwritten subtext of being unsuitable for anything beyond that.

Dancer's current implementation is somewhat limiting in some cases, hence the reason for Dancer 2 being close to completion (a complete re-write of Dancer's core, done properly, but still backwards compatible with Dancer's DSL), but I don't feel it's limiting enough that it means the current stable version of Dancer is no good beyond tiny apps.

Also, this mention means that, even after Dancer 2 has been released, anyone with a version of perl which included this perlfaq version will see Dancer being described as unsuitable for anything big, well after it has been replaced with a far more capable implementation.
Fie!

I wonder if we can get Travis to do our CI against multiple JDKs.
Supuestamente arreglado. Debería cancelar el rayo perfectamente, spuestamente <.< No lo he probado...
Improvement in comments.

This file should generate the following warnings:
36_dereferece_pointer.c:10: warning: format ‘%02X’ expects type ‘unsigned int’, but argument 3 has type ‘int *’
036_dereferece_pointer.c:14: warning: format ‘%02X’ expects type ‘unsigned int’, but argument 2 has type ‘int *’
036_dereferece_pointer.c:9: warning: ‘pi’ is used uninitialized in this function
Is this request for the same purpose as sending a pop request to the youngest frame?
Hmmm. That's nice. Seems that we don't need to quote keywords when they appear in package names, which makes all the quoting stuff a lot harder. It's not even as simple as quoting only the last name in a compound, because of the possibility of nested classes. 

There's a further problem with keywords in package names: it breaks Java interop. So do we need to take the step of quoting package names *anyway* when transforming a class in a quotable package, and thus transforming into a different directory hierarchy.
No problem with rules (as far I can tell). Speed is problem for me. Rule importing time increase from 50 seconds to 2m40s, and I see 100% cpu usage for almost 15 seconds at Opera start (setup urlfilter rules). String operations are always slow, regular expressions even more... Some optimizations are needed. Maybe remove '##' from urlfilters? Or make this part optional, similar to 'Add exclusion and global rules'.

I can always decrease number of subscriptions... :)
That's subjective...

Like I said, you might prefer to use a class, but the right thing in this case is to use an ID.
#ID == unique
.class == reusable

`.container` is not reusable and only targets an unique element, and should therefor use an ID.

Not trying to start a CSS war here, I just want it to be as correct as it can be, so not to confuse beginners.

From the [spec](http://www.w3.org/TR/CSS2/selector.html#id-selectors):
The ID attribute of a document language allows authors to assign an identifier to one element instance in the document tree.
Are these comments personal notes? They seem a bit out of context ;) 

Are these points that you'd like to further address somewhere?
why would we work on AHBot?
Мы пробовали подменить сокетную реализацию на Memcache_ (не MemcacheD).
В итоге, кэш набирался очень долго, около 20-ти минут (вечер пятницы).
Решили вернуться на сокеты, на них и живем, не жалуемся.
На сокетной реализации отлипаем за 3-5 минут после полного сброса кэша.

Очень сильно помогают две вещи:
 1. Сжатие: выигрыш в объеме в 10 раз + сети намного легче, хотя мы не упирались в сеть.
 2. Разгрузка машинок с мемкэшом: не грузить ничем проц на таких машинках, иначе мемкэшд начинает плевать не пойми что.

Против добавления Гошиных изменений ничего не имею и всячески поддерживаю добавление MemcacheD.
Ok..

1. Are you making a separate GUI for editing the maps? WTF!? Why do we need this? And if you are, why do you need the text files that it stores to be human readable? We aren't going to be editing the text files directly if we have a fucking map editor for them!
2. You're talking about a z-index that changes which FLAT layer shows up. I'm talking about a literal z-axis because I was thinking of an actual semi-3D rpg. In the maps I show the tiles with `6` would be 6 cubes above the base level. This is not the same as image layering.
3. If we're going to make a flat layered rpg, and even if *#1* is true and these text files are going to be used by more than just the level editor itself... there is still *NO* reason why GUI data should be mixed in with MAP data. A GUI should always have the highest z-index and shouldn't be fixed to the screen *unlike* a map Why would you EVER need to manually record on what level the GUI should be? So what would be the purpose of creating a map that has `xxxxGGGGxxxx` where `G = Gui`?
there is no doubt, anyone that has ever played on retail in wotlk and was remotely good at pvp would know
Upss, I had not see this commit. I made a report aboout a crash in base of spell_generic.cpp changes.
CONDITION_QUESTREWARDED with NegativeCondition = 1?
These things would also go away if you stopped using sendMessage and have the action set implement a capability interface. Then the compiler would check all these things for you.
Unfortunately, this isn't going to work with DM make in a Windows cmd console.
This had been missing for the jar to be used by the JavaScalaActors project. Added it from the book.
@Daan- nice catch! I've push a new gem version (3.0.4) that supports Mongoid 3. (Change: https://github.com/brianhempel/mongo_session_store/commit/2667a29b809122c01e07bb7e57625058af443859)
Unfortunately not. Although it took me a hour to fix it, I wasn't sure that fix was correct.
Also this commit (https://github.com/basho/erlang_protobuffs/commit/f5c357958b7e82874aab42af88abd8d88e66bc9d) stopped me too.
Could also you give TypeVector its own toCtype?  This would make implementation a little easier/cleaner for gdc - if not, I can always derive.
Thanks for the response. 

Again if you don't mind, I'm curious as to where are you planning on calling the thrift interface from? Do you intend to use it as a stand-alone search engine or do you intend to implement a different service layer?
Better yet: Write "Fix #18" *and* a short summary of what was fixed in the first line (and not in the description). That way one immediately sees in ```git log --oneline``` that a bug was fixed
ach neubi
@woglinde: thanks , but , i'm still a noob , can you tell where to add in the gitignore , i copied the gitignore from franco source :D
rightPart.substring(0, 1);
Le spacing as-is.
No, you can use it in the Fabricators to keep your code more concise. It is just an alias for the longhand though.
Wow, you said it.
I'm re-setting the flag in try/finally after I for a put, which imo should be ok.
I'd originally added sockets using WebSockets, but it was getting a bit complex and I didn't have a chance to finish. I thought I'd removed it all, but I missed a few parts. Please remove them in event.js and vars.js on the next commit.
Tradeoff: a `switch` or a set of `if ===` would be faster but less concise/readable.

I put `toLowerCase()` (and the regex trailing `?`) because I'm not sure that all browser normalize this property if the attribute has uppercase or is empty. Maybe superfluous.
you can use getattr(word, inflection, name) :)
yeah i did it in vs... my tabify ext. seemed not to work last nite /facepalm
The problem is I only have read permissions to the two repositories I
already moved to sgt7 from my account, if you can't change the permissions
delete them and I will fork it directly to sgt7.

On Fri, Feb 3, 2012 at 9:57 PM, Chirayu Desai <
reply@reply.github.com
> wrote:

> Fork it from cm github, then cherry-pick your commits or add your
> repository as a remote, then fetch n merge it
>
> And yes - setup remotes properly
>
> Git@github.com:sgt7/xyz.git
>
> Use git remote -v to check
> On Feb 3, 2012 11:32 PM, "Scott Brady" <
> reply@reply.github.com>
> wrote:
>
> > Thanks for adding that, can you make our frameworks/base and settings
> > read+write.  When I moved them from my github it revoked the permissions.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-931174
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-932531
>
> You mean @ValidateOnMethodCall or @ValidateMethodCall, right?

I've used `@MethodValidated` in the [draft branch](https://github.com/gunnarmorling/beanvalidation-spec/tree/BVAL-241) for now, but I'm open for other ideas. I'd prefer an adjective, though (opposed to an imperative name such as `@ValidateOnMethodCall`).
tu aurais pu réutiliser https://github.com/ded/domready : moins de maintenance :)
No, this was simply to fix backward compatibility with version 2.0.2
Update the values if this is a DropDown that have been enhanced with chosen.js
Condition checking should probably be done inside of atomic block, to avoid obvious bugs with negative money and stuff. On the other hand, that would put most of the function code inside STM monad, which is limitedly nice.
@nrw: Once you remove the unwanted ( from the line it works fine for me.  If you look at the attachments.add method in https://github.com/petebacondarwin/coffee-script-precompiler/blob/patch-1/node_modules/kanso-utils/attachments.js you can see that it does the base64 encoding at that point.
API documentation generation would likely only focus on the code in this project, and by default, wouldn't have access to the definition of WordPress functions. The purpose of the `@uses` tag is to create automatic links within the documentation, and that won't happen by default for outside functions.
See 4 comments above
git rebase -m upstream master is your friend :)
Seems like this is a change that shouldn't be merged to develop? I mean, once this is merged there's no point in testing the branch anymore.
This prevent my RIB to startup, please fix it
I tried to be as less invasive as possible. I'm ok with that if you consider that is the proper place to put it.
True. My plan is to use the maven github plugin in a separate pull to publish javadocs in a separate branch that would finally be available at the link above. This is somewhat of a premature change assuming the team's OK with having Javadocs on Github as well, but I think it's modest improvement and it's a good it seems to keep code and javadocs in the same spot and it's easier to find them. 

As an example, we have done so with the `cas-addons` project and it has turned out quite well: <http://unicon.github.com/cas-addons/apidocs/index.html>
minor point: should this be debug?
I'm not sure I understand the question.

If it's the reason why is an hash of integer values that you're asking for, then it's because this was taken from an older implementation and there was Ad code using those original values, so this was kept in this shape for the sake of maintaining compatibility.
We could have a Varnish layer here.
is the `toSet` needed? isn't it already a Set?
Just add a comment on the purpose of this dense section
Well, the implicit conversion is returning AskableActorRef so it's quite weird to have that class be package protected. What we could do is to make the constructor package private, but if we change the class to have multiple parameters it cannot be a value class anymore so I don't see the need.

What do you think?
But it doesn't check if it's an event, it could be some stray message, and then it won't get sent to unhandled, and it's hard to debug.
Normally we always add

    /**
     * INTERNAL API
     */

@viktorklang mentioned something yesterday that I had not thought about before, `private[akka]` doesn't even show up in scaladoc, so why why do we have this ceremony with INTERNAL API? One argument would be to make it super clear for Java developers looking at the source code that this is not supported API - and that is good enough argument for me, but then we should use that convention consistently. 

Sorry for the novel, I was just thinking aloud, and it was not meant as criticism to this minor oversight here.
Ah, I didn't think of it so, but that is of course better. I'll change
So, what does it need?  Better tests?  Cleaner API?

The three are all useful (user_id, user_card and user), and the caching is pretty simple.  The @@user_id is what you really set, and the other two forms checked the cached value for having the right current id.
helpers is traditionally imported as h in ckan
HTML5 provides [`data-` attributes](http://html5doctor.com/html5-custom-data-attributes/) for including additional information in the html. eg

```html
<a data-package-id="${c.pkg.id}" data-state="unfollow">
```

These can then be accessed cross browser using `element.getAttribute('data-state')` or using jQuery which provides a nice wrapper.

```js
// Get a specific attribute.
jQuery('#dataset_follow_button').data('state') //=> "unfollow"
// Get all data attributes as an object.
jQuery('#dataset_follow_button').data() //=> {"state": "unfollow", "package-id": ""}
```
I might be missing something, but I have the same question I've been having. The consumer is binding to a repository's distributor, don't we need to track the distributor type in all of this?
I'll fix the type for 'details'
Seems to work, but read_file did not as they are in different controllers.
bad idea imo.  what if card details change?  names, rules, etc.  constants shouldn't have dynamic values
Yeah ;) Would like that in comment instead of a empty comment placeholder ;)
is it ok to do the rollback at the end, if eg Cleanupalbums failed, but all else succeeded? currently we bail out early on an error
This is a class with functions. I'd prefer a clear function.
I thought about this but chapter has den/num as well.
You can make this a module level function (outside of the class) since you dont use class at all.
will be deleted with the script

On 14/09/12 11:18, Asfand Qazi wrote:
>
> In script/fix_mi_attempt_colony_names.log:
>
> > @@ -0,0 +1,29 @@
> > +Rails Environment: development
>
> Can we delete this log file please? Shouldn't be committed
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/mpi2/imits/pull/45/files#r1606304>.
>




-- 
 The Wellcome Trust Sanger Institute is operated by Genome Research 
 Limited, a charity registered in England with number 1021457 and a 
 company registered in England with number 2742969, whose registered 
 office is 215 Euston Road, London, NW1 2BE.
I think it would be worth for a future refactor to call it "Attempt" and always create an instance for every attempt instead of using the class only for retries.
I like `classes` and `id`. What do you think about inserting either text or image as control content? Maybe with `innerHTML`
This should be a docComment instead of a regular comment
I don't mind all-caps file names for files that are used less often. My issue is with camel-case file names, which in my opinion doesn't look pretty.
`TUIAutoResizingFlexibleSize`  would simplify this line.
I'm not sure Apple will ever declare a TUIEdgeInsets structure, sooooo.
Probably doesn't need prefixing.

-- 
Aditya.
Oh. I did something horribly wrong then. I guess this is what I get for late-night coding.
Ah, you are correct. My diff was misbehaving and not showing the method name. :+1:
no the node is being unpickled later. this is getting the config from the node but during script preparation.
объединить ифы в один
Also - let's not change email or new email until we move to 1.4+ - it has a natural inflection point around solr schema changes.  Thanks, L

On Aug 17, 2012, at 4:40 PM, Duffy Gillman <notifications@github.com> wrote:

> In src/main/resources/schema.xml:
> 
> > @@ -492,13 +505,16 @@
> >     <field name="name" type="text_ci" indexed="true" stored="true" termVectors="true" />
> >     <field name="firstName" type="text" indexed="true" stored="true" />
> >     <field name="lastName" type="text" indexed="true" stored="true" />
> > -   <field name="email" type="text_ci" indexed="true" stored="true" />
> > -   <field name="newemail" type="text_ci" indexed="true" stored="true" />
> > +   <field name="email" type="text_punct_phrase" indexed="true" stored="true" />
> To the first point: no, we have to reindex if we want to populate the eppn_authnId field for users. We may be able to get away with only reindexing Authorizables if Solr can preserve the old Content indices - I'm not sure how it will react when we change the schema. Ultimately we will be reindexing everything as part of the 1.3 migration.
> 
> To the second point: ya, you're right. I was there working on the schema and thought that email being case insensitive was the "right thing". But truly it should be in another commit. I will re-issue the PR.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
There are no disadvantages about adding to it... It's just I would prefer it kept short and clean.  

Thanks for removing that .idea directory! 

-- 
Ryan Bigg


On Tuesday, 12 June 2012 at 20:22, joxxoxo wrote:

> > @@ -10,3 +10,4 @@ tmp
> > *.sw*
> > spec/dummy
> > *.rbc
> > +.idea
> > 
> 
> 
> ok, i've removed it already. 
> and yes, it was usually added to project's .gitignore automaticaly(for example by refinerycms generator), by me or by other collaborators. 
> thanks for sharing your point of view.
> may be you can name any disadvantages of adding this to project's .gitignore?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/radar/forem/pull/238/files#r965988
> 
>
Oops. You removed.
Please make it a Set then?
Do we need to intersect? I believe we're not expecting to get duplicated attributes, perhaps only `<<` would do the job?
This line shouldn't have changed. Also the indentation of the lines above are not correct.
If only extractors didn't put a gun to our head and make us use Option, we could do significantly better returning Applied and having a NoApplied object representing None. Maybe someday.
Yes, readability suffers somewhat. On the plus side, shorter methods are favored by the JIT compiler. In fact, that was a consideration that led to Range.validateRangeBoundaries, but on closer inspection it brought its own disadvantages (basically, making closure elimination less likely).
"This is the case," you say? But what is the case?
I'd prefer a value error if the "out" array is not contiguous.  This feature is for users who know exactly what they want in terms of memory management: if they're doing something wrong, they need to know.
Remove the single quotes around m.  Also, there are lots of superfluous commas here.

Kane has a notation for linear momentum and angular momentum, and others may use it as well. However, you have also written it out in English rather than use only the compact notation, as you did for the velocity.  So my question is:

Why use the notation ^N v^P for velocity of P in N but not ^A H^{S/O} for angular momentum of S relative to O in A?  Basically, at what point does the notation get too ugly for a docstring?

The more I think about this, the more I think I prefer no LaTeX syntax.  For an example, see page 92 of Kane&Levinson 1985, second to last paragraph.  Saying "angular momentum of S relative to O in A" requires no further explanation; writing ^A H^{S/O} requires you to basically restate the same sentence again unless you presume an audience that is familiar with the notation.

I realize a change to this has implications all over sympy.physics.mechanics, but I do think it is worth discussing and at the very least being consistent.
On the contrary, it is not too broad:

try 
```
In [3]: f(g(x)).diff(x).subs(g, Lambda(x, 2*x))
```
Let's work with lists for the example and just realize that you can replace list with Matrix and the same idea holds. list cannot be asked to give a value that is not there, so [][0] fails. It can be asked for a slice and that slice can contain out of bounds numbers; everything that intersects with that range is returned. So [][-20:] and [][:20], [][10:20] and [][20:] are all fine. And [][:2], [][2:3], [][3:] will break the null list into the first two elements, the 2-th element, and those from the 3-th on...all of them [], but it still works.

I'm not sure why you think the : part is broken; I think I showed that it computes the correct min,max,step tuple (0, 0, None).

> that woudn't work if I wanted to extract a column that wasn't on the edge

You can never extract what isn't there, but you can extract columns that aren't there. You should probably be thinking of a column not as an element of a list but as a sub-list of a list. This is the difference between range(3)[1] -> 1 and range(3)[1:2] -> [1]. eye(3)[1,1] is element (1, 1) while eye(3)[1:2,:] is the first row and eye(3)[1:2,1] gives the top middle element as a 1x1 matrix.

The bottom line is: if you index a matrix with integers you get elements. When you use slices you get sub matrices. The dicey part comes when you want to extract parts of a matrix that don't exist: **when in doubt about whether elements are there, use slices**: eye(0)[1:,1:] will give you the submatrix that starts on the second element of the diagonal. eye(0)[:,1:2] will give you the first column, eye(0)[100:101, :] will give you the hundredth row.
And that can be adapted slightly to work below, too.
(just grep the repo for mass center)
On Fri, 24 Aug 2012, Ondøej Èertík wrote:

>> +                return frozenset([ p for p in primes if primes[p]==1])
>> +            if pi_coeff.is_Rational:
>> +                q = pi_coeff.q
>> +                p = pi_coeff.p % (2*q)
>> +                if p>q:
>> +                    narg = (pi_coeff-1)*S.Pi
>> +                    return -cls(narg)
>> +                if 2*p > q:
>> +                    narg = (1-pi_coeff)*S.Pi
>> +                    return -cls(narg)
>> +
>> +                ###  Good place to bail-out.
>> +                #if q > 15:
>> +                #    # arbitrary threshold set for complex denominators
>> +                #    # 15 is the largest denominator allowed
>> +                #    return None
>
> Is it possible to bail out the same way as Mathematica does? It's described here:
>
> http://mathworld.wolfram.com/TrigonometryAngles.html
>
> and then provide some function in SymPy to do the reduction for arbitrary 
> angle (with possibly long results) if the user really wants it?
>

Certainly possible.  Something like "rewrite_as_radical"?  I'm putting together 
a proto-type.

For general method, I need a function to do partial-fraction decomposition of 
Rational numbers.  Looks like part_frac only does rational functions?  Is this 
already in sympy someplace?  (it would probably need versions of igcd and igcdex 
that can be applied to sequences of integers)
Use "Yii::getPathOfAlias('application.runtime')".
Kommentiere doch bitte kurz, weshalb diese Validatoren nicht verwendet werden dürfen
Since what we get is already the policy class we eventually want to look up, it would be nicer if somehow we didn't have to make the round trip of BlogPolicy -> "BlogPolicy" -> "Blog" -> "BlogPolicy" -> BlogPolicy. That way it would be possible to use anonymous classes for example.
Preferably use `r == -1` (minor style issue).
They seems to set errno in internal system calls. For example,
```C
#include <stdio.h>
#include <errno.h>
#include <fcntl.h>
#include <kvm.h>
#include <paths.h>

int main(int argc, char** argv) {
  kvm_t *kd;

  kd = kvm_open(NULL, _PATH_MEM, NULL, O_RDONLY, "kvm_open");
  if (kd == NULL ) {
    fprintf(stderr, "errno = %d\n",errno);
  }
}
````
```sh
> ./kvm_error
kvm_open: /dev/mem: Permission denied
errno = 13
```
I've done some work on this side today.  I realised there was quite a
raft of missing testing and almost certainly some missing behaviour for
open and half-open intervals.  I'll land the changes in the morning -
got the tests and their data points all defined; now just have to make
the rest pass.

I'd commit it for you to take a look at, but it'd break the test suite
as it is right now.

Cheers,
Ant


On 20/10/12 05:39, Ben Hood wrote:
>> +    WindowRefiner.forPeriodExpression("")
>> +  }
>> +
>> +  @Test
>> +  def shouldAcceptEmptyOffset() {
>> +    try {
>> +      WindowRefiner.forPeriodExpression("P1D").withOffset("")
>> +    } catch {
>> +      case ex: IllegalArgumentException =>
>> +        Assert.fail("Empty offset was incorrectly rejected")
>> +    }
>> +  }
>> +
>> +  @Test
>> +  def shouldAcceptDateOnlyRanges() {
>> +    WindowRefiner.forPeriodExpression("P1D").refineInterval(null, "2012-10-10")
> 
> What is the expected result of `WindowRefiner.forPeriodExpression("P1D").refineInterval(null, "2012-10-10")`? Could this not be put into the test function as an assertion?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lshift/diffa/pull/270/files#r1896639
>
They are reservations for a requested EC2 instances, not reserved instances.
bucketSize is conventionally camel cased, I think.
whoa good catch
hmm... that seems weird :) at least return True?
Yes, because if you can't read /proc/sys/kernel, you also can't determine if /proc/sys/kernel/grsecurity exists.
Well, it was wrong there and it is wrong here.
Either this should be an else, so that you get an error message about this missing file and find out about composer, or there should be an else after this which contains something like die("you must run composer install") or something along those lines.

Like this it'll just not find classes without any information on why.
It should be an simple "must not", as it causes an error in IE, so developers must not use trailing commas.
Had already fixed :)
I already told him that but he says that, if that happens, then it could detect false positives. I don't know if that's true but he says that.
Change default port to 9312.
Comments on their own lines please.
I think we should allow it for now so it's rally easy to just re-use Cloudkick agent plugins.
why the space here?
I'm assuming this is for debugging? Might want to remove it.
If not than we got linking problems, right? Is there a way to stop cmake execution if the platform is LINUX and library is not found?
Optional takes only one 'n'
Single-line blocks should be wrapped in curly braces (same applies to all single-line blocks)
Okay, will add a getter method
I know. I am pushing more changes. Don't worry, it was just an initial
commit.
On 24 Dec 2012 14:57, "John Stoecker" <notifications@github.com> wrote:

> In src/screener/style/screener.scss:
>
> > @@ -205,4 +206,169 @@ $form-bg-color: #f7f7f7;
> >      .x-container{
> >          border-radius: $form-fieldset-radius;
> >      }
> > +}
> > +//Screener Tab changes
> > +
> > +.screener_tabs {
> > +	height:100px;
> > +	-webkit-transform:rotate(-90deg);
> > +	-moz-transform:rotate(-90deg);
> > +	-o-transform: rotate(-90deg);
> > +	font-size: 8px;
>
> It's really tough to read this font size. Also, would not be good to have
> such a small pixel height -- iPad requires height of 768 (or more if we use
> it horizontally)
>
> Colours are looking good though.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Raxa/Raxa-JSS/pull/268/files#r2497408>.
>
>
WeightInKg
In light of this PR change, I raised GitHub issue #37 requesting a refactor of ```_Groupby.group()```.
Why would it work here? (you say in the commit's message that the access time is updated when we list the directory, so the same problem should occur with this old stuff that could have been created by error, no?)
Remote the 1 and re-add the empty line after your new import line.
this if do not have an else clause. We should transfer it to the gpu I think in the else clause
elif kwargs['type'] in {ComponentType.disk}:
to the specified IP
trailing whitespace!!!!!
that was totally derp on my part.. I don't know what I was thinking. Thanks for the catch.
Should this really be a default in the base?
`attr` is not imported from nose
Again, what's with the grouping parens?
Can't put it there, it would define a custom "toggle" event. We can move it to `jQuery.event._etoggle` but meh, it's still hanging out somewhere.

But ... I hadn't thought enough about how this would look once `.toggle(fn, fn)` is completely removed from the library. We need to rejigger the files so that deprecated.js comes last and has the opportunity to do the final duck punches. That's the way it will need to be once the files are off in their own plugin. I was trying to preserve the existing logic path given the position of deprecated.js, and that's not right.
`name.charAt(0)` or... `name[0]`?
this is a main line of this change - we can return false here, because we already visited this element with this selector, so we can skip next iteration to improve performance
Should be split_criterion_name according to the comment.
As commented above.  If we are guaranteeing that args is always int64 then what is the need for typOid and outFuncOid?
OK, what is missing in kitsune that I was going to add is.

First you have to:

    $ cd locale
    $ svn update
    $ cd ..

Then after extract and merge:

    $ cd locale
    $ svn commit -m 'String update YYYY/MM/DD

Or something like that
I think we should default this to English.
This should go in a "defaults" section, since the hash already covers the uniqueness and checking equivalence of a textfield is going to be a bit inefficient.
Thanks for the explanations..
By the way, I decided to remove the test: itkTestDriver was not a kind of "functionality" test, with the "--no-process" i was just checking that the binary exists (like a "-h") which does not make sense (beside the path problem you mention).
Should be:

```ruby
fails_with :clang do
  build <which build number fails?>
  cause <<-EOS.undent
    <Message explaining how/why it fails so we know what to look for when things are fixed.
    EOS
end
```
Returning nonzero should be an explicit decision for only a few commands (equiv of git diff --quiet).  Having zero be standard as the last command value makes it clear it was success.  Having exceptions be linked to exit codes is also fairly normal
I believe this is equivalent to just `c.site.path`.
I think that based on conventions it would be best to keep this as `g`.
Agreed  - will do.
Consider the following implementation `([base_search] * words.size).join " AND "`
Can you add a comment what this is for?
I believe there is a problem with this approach, specifically that by pulling the the credentials out of an On-Instance Provider, the credentials will expire after 12 hours by default.  The net result is that this AmazonEC2 object will self-destruct after 12 hours, which could have unintended consequences down the road.

I applaud the incorporation of On-Instance providers; however, I believe the provider needs to be funneled directly into the Amazon Objects in order for it to be safe. 

Thanks,
Jay
Adding the `current` var isn't really necessary and makes this line a little harder to understand. We can just re-use `node` since the original value doesn't need to be kept around:

```coffeescript
while node = node.parentNode
```
This one is news to me.  " is ingrained in my fingers.
Det var nu om natten der var mest gang i den!
Also, `flag` is not a very clear name.
Two related cases we should probably also test:
`<div dir=` (no opening quote)
`<div dir=""` (empty pair of quotes with cursor in between)