labofsahil/github-event-dataset-2012 huggingface dataset:
http://imageshack.us/photo/my-images/521/mangosr2.jpg/sr=1
Done.

There is an other small issue with the help of these functions that I don't know how to fix:
when typing "hepl(H.query_disk)" I get the help, but the function description do not contains the arguments: "query_disk(...)".
you never do anything with this "always_tzinfo" parameter
I did something that is probably wrong, but here you go:

    python setup.py install
    cp -r simple.py examples $HOME/sfepy_workdir/
    cd $HOME/sfepy_workdir

Without understandying sfepy and just starting to work through the tutorial linearly, I expected that eventually my own FEM code will be based on one or more of the examples.  Since I want to have my code outside the sfepy source, I figured on just copying all the examples into my work directory and going from there.  Since I'm using EPD I just installed as shown (no root req'd).

This might be a documentation issue in terms of outlining what gets installed where.  I wonder also if there is a namespace issue, since if you do a root install you get simple.py (and the rest) in your $root/bin path from that point onward.  From that perspective it might be better to have those scripts named something like sfe_*.py, but that's a big change in documentation etc.  

I'll try to replicate your steps later, have to run now.
Liking with_deleted here.
Thanks for the pull request, I'll test + review so that it goes in the 1.2.9 release. Is there by any chance an open bug on www.mantisbt.org/bugs/ for this?
doesnt need to be in the instance, you can do that inside the boss's AI.
Is this commit necessary? `uname -m` would still return `i386`, right?
Argh.  This is frustrating, as I don't know of any other way to test that something is actually lazy.  It is very easy to accidentally keep a reference somewhere in the code which will cause it to blow up on large input data, hence this test.
Note that this way to launch Sonar analysis is not recommended, I can't remember why, but more correct : mvn clean install && mvn sonar:sonar
Yippie, then finally my name can be written correctly: Laußer :-)
you both are right guys! didn't saw that condition on line 90.
[This is the commit](https://github.com/propelorm/Propel/commit/1cd82d67cbcc691f487eaa54cdfc9957acd5f4b6) that introduced that query, before was just "show tables", so the reverse-task worked fine with views.
It may be a good idea to add an unit test for `Conduit.Binary.openFile` that tests if `System.IO.hGetEncoding handle` returns `Nothing`.
@mrocklin -- I think it's a bit confusing to have the RandomSymbol and the symbol that represents it have the same representation. For examples:
    >>> x = Symbol('x')
    >>> X = Normal(0, 1, symbol=x)
    >>> X
    x
    >>> type(X)
    <class 'sympy.stats.rv.RandomSymbol'>
    >>> X.symbol
    x
    >>> type(X.symbol)
    <class 'sympy.core.symbol.Symbol' >
    >>> bool(Eq(X, X.symbol))
    False

Maybe the density should use a dummy variable, like it does for transformations/product spaces? Or perhaps change the representation of Density to a map/function object (if it exists).
lol To my never me to given crash
Awesome! Looks like this will solve the dependency tree issue.
This seam equivalent and is easier to read

size += (size % MIN_SIZE_INCR);
Wouldn't `'/' . ltrim($link, '/')` be more simple?
Hi Brian,

I would support that revision.  I agree, it's cleaner and more jQuery-like.


/brz

On Jan 18, 2012, at 10:37 PM, Brian Michalski<reply@reply.github.com> wrote:

> Hey @brzaik, are you committed to this syntax?  $(".js-dropdown-button").dropdown() might fit the jQuery paradigm more... I can tweak the plugin a bit if you're cool with it (or give you some leading comments if you'd rather do it yourself).
> 
> ~Brian M
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/concerto/concerto/commit/734341fdf735d8f0b02badb4d8a04b5be0933ea7#commitcomment-880805
@maxwell ?
Your recent change reducing the default start/stop timeouts might affect this
After I went to bed I thought of tile_contents_equivalent_for_interiorness or similar (specify that we're checking whether the contents are equivalent, not whether the interiorness bits are equivalent. I realized I'd deleted the "_type" in the previous name that actually meant something related to tile contents). "checking" should not be in the name!  It's not about checking!  It's about a property that holds even when you're *not* checking!

The _representative function would have been hidden from other code if that was easy, and you're right about the most formally correct way of making a C++ class, it is just not worth it for a concept that occupies one line of code that is used in one place right next door. (Unless we need similar sorts of equivalence classes other places too...or...hmm... Making it a C++-class would be worth it iff we store its value in a variable anywhere, I think.).  _equivalent is still an equivalence relation (I almost called it interiorness_equivalence_relation but then realized functions should typically be named for what they do, not what they are).

thoughts (possibly supply multiple of these) :
tiles_form_a_boundary_between_interiors() //the name works whether or not equivalence is transitive, which it would document in a comment that would also say that it's symmetric and ir/reflexive.  Every use of interiorness_equivalent() is negated (they all used != before my change). so it seems plausible to supply a function with this sense? (There must be a nice phrase for the opposite of "form a boundary" if we wanted, but I haven't thought of it yet.)

tile_contents_equivalent_for_interiorness() //if it is transitive. OR. If it's non-transitive but some contents values are completely equivalent anyway

tile_contents_could_possibly_be_in_the_same_interior() //if it's non-transitive. Could be useful, or not.
It's not in this case, and only test code, so if that changes it would break the build and someone would need to fix.  We should probably review all of the galaxy file utils code and hoist it to platform.
Why the conversion to and from `void`?  If it's so that we can move the function body to the cpp file, I'd rather keep it here and use the correct type.
#!/usr/bin/python
  or may be eve
#!//usr/bin/env python

Looks like Chrome usually uses first form.
Well, the pass would only be required if we hadn't a docstring, I'm not too much for verbose code.
I had to delete the commit history, so this was the simpliest way when I understand the github help correctly.
i have just committed a fix .. this may mean that this PR should be reverted, since i now ensure that the id is updated in persistNew().
Great Rafael! That code is definitely part of the model, we will fix it with that refactorization.

I am releasing a new version with that fix. Thank you!
when you have a family, with kids....sometimes odd things end up in your code....
The first, name, seems like a default, but the other two are placeholders. Use the [HTML5 placeholder attribute](http://diveintohtml5.info/forms.html#placeholder).
No problem - thanks for having the repositories available - not really got 
into
ROM development but at least I built and ran from my build of your setup 
fine and
much better than the old stock ROM.

My only real change so far was a keymap change so only the power button 
actually wakes the phone.

Thanks again.

Bernie

-----Original Message----- 
From: Jerry Jasuta
Sent: Sunday, February 26, 2012 7:45 AM
To: bernardjs
Subject: Re: [android] Add CMBootanimation and BacksideUpdater to build 
environment (bf292bc)

Thanks for catching that typo, fixed now.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/JerryScript/android/commit/bf292bc78ce883f3ea1ed8650c59d5c1c9d7f22b#commitcomment-1012493
(params instanceof XXX) --> params.getClass().isInstance(new XXX())
class.isInstance allows the use of generics (e.g List<Y>). This method has been used in some actions and changed in the rest to keep similarity.
Given this is actually only used *once*, do you need it?
The correct name of the lib file is libxml2.dylib

Best,
Alexandre
isn't that always true?
The SE API supports filtering badges that are tag based and not via the use of the sort and min/max parameters. Problem here is, if I try to do something like this:

    SKBadgeFetchRequest *br = [[[[SKFetchRequest requestForFetchingBadges] usedByUsers:user, nil] onlyTagBased] sortedByName];

It won't work, because the `sort` parameter needs to be set to `type` in order for the `onlyTagBased` part to work.
`(으)로, `(ㄱ)와 업데이트
this commit is wrong, but revealed a different issue
Sky, take a look here please. What's this about?
After speaking with wjp, the 244 should be changed to an unused value, such as 254. Currently, the 244 in this could clash with the commandOnlyCond() call in selectSlot() below
you should remove this commit and regenerate your app secret
This is the only change related to pull request #50. The problem was that "e.relatedTarget" can return an image in addition to an area. Normally this would be fine, but if mapKey is an attribute that is also found on the image, then "key" will be a value for a non-area attribute which causes problems.
Yes, check the next commit.
Please fix bug. I can save value to file with "[EDITING]" prefix. For example:
        <TranslationItem Name="$this" Property="Text">
          <Value>[EDITING] Поиск и замена</Value>
        </TranslationItem>
Great that this has a caption, but why is it commented out?
A box-shadow glow is prettier than a hard darkblue border ;)
This merge somehow deleted https://github.com/plone/buildout.coredev/commit/c65042f637d9b0473ef9196d8c045b6c204c8999
Waarom hier een string gebruiken?
And... when i do weird search and replaces i use to add as comment what i've used to change it, so others can reproduce with custom code.
guys, anyone using newer rev than b4ecd85da46302 with UP time more than 10 hours?
... I consider b4ecd85da46302 for last stable rev. for now :(
@Ka0z The cast got interrupted because the rogue blinded him. Casts should NOT interrupt when the target runs LoS. Idk about Cata but in wrath that didn't happen and the fact that you say it did is just ridic.
Testing to see why /if github comments seem to be disappearing
+1 on your buttons they're awesome
Anyone having trouble loading the rails in this map ?
i've been looking.; was just badly done merge by me.. seems to have the changes that were done earlier.. (double xD)
good job, but... "Welcome to ViperWow Online Gaming" in autobroadcast? rofl
oh I just saw that there is still an error... I need to update it
`this.activity`. Doesn’t matter actually, just for common style.
...to a property named 'headers', which itself is a Map.
и сделайте тогда, более длинную строку для ввода формулы, ато её не целиком видно, когда на кнопку давишь
ну либо вместо <input type="text" /> заюзать <textarea>формула</textarea>
This is just an example of another one of the nice methods that comes with the Assert API.
Similar to Mo's suggestion in 6c58cd379fa8676acd2825aae6e2a57e3a449beb, you can just pass in null rather than setting $response to null first
use UIActionSheet+Blocks https://github.com/xuki/UIActionSheet-Blocks
whoops... Sorry. Maybe we should change those, too?
"createPopup" - ie detecting. Not elegant, but work. Need more tests for more elegant solution
Cual sería la ventaja de que esten? yo prefiero que halla pocos sobre todo si vamos a instalarlo en otro lado.
If something compiles in both C and D it should behave the same way if there is no strong reason against. With `->` as lambda operator e.g, `f(x->y);` can choose incorrect overload.
I agree with you, but in dev mode this file cause a lot merge problems. I think ignore this file, for while, helps with behaviour.
What do you recommend in this case?
Tons of smart stuff in here. Awesome job, dude.
also will not properly handle namespacing - namespaces can be either declared and used as blocks or at the head of a file
void AdvanceSprite::removeObjectItself()
{
    this->removeFromParentAndCleanup(true);
    delete (this);
}
delete (this) always get crash in progress!    may I delete this line?
the Symfony convention is to use ``EventListener`` as namespace.

And an empty line is missing before the namespace
On 32-bit Windows 7 Home Premium, I get no bundler errors.

https://gist.github.com/1832718

That's using TorqueBox 2.0.0.beta1 from the .zip and Java:

$ java -version
java version "1.6.0_31"
Java(TM) SE Runtime Environment (build 1.6.0_31-b05)
Java HotSpot(TM) Client VM (build 20.6-b01, mixed mode, sharing)
Ага, слыхал про сию магию.
ご提案ありがとうございます。ちょっとそのパターンも考えてみます。
Encodes the $ too.
I got exceptions with this because the getReadOnlyID() got a wrong $pad:

example:
string(26) "g.aMyWd7sQXGYPdca4$testPad"
string(28) "g.aMyWd7sQXGYPdca4%24testPad"

the same for getPublicStatus()
"I wondered about that, but saw it in a few styles."  <--- I should clarify this. I mean I have seen a locale set in at least one of the dependent styles, not sure which, and that was one other reason why I decided for it, despite my suspicions it was not **required** (the fact that it was not required was made clear by you not correcting it). I did not realize it could instead be **unwanted** for the various reasons I list.
Derp! Sorry about that!
Nice commit message =)
i think this will double namespace addition to function names
A litle reminder for myself to fix your -> you
path() génere une url relative alors que url() va générer une url absolue.
Tu as une préference entre les deux ? Moi elle est plutôt pour les url absolues, et il y'a pas
vraiment de différence entre les deux niveau performance / référencement =p
rajoute ici une contrainte qui spécifie quelles valeurs peut prendre {action}
That was the trick that did it. Thank you for all your help.
Thanks! Ew that bug looked ugly.
http://en.wikipedia.org/wiki/Spelling
i have said at the top of the READ ME that i have changed the three little pigs siri proxy and edited to fix peoples problems that they are facing and changed the name of the three little pigs siri proxy to four because so people don't get confused of which one they have used
I added a neutral button to the edit text that says default.  It actually sends the value default. That is why I included that part
Yay! Progress!!
Смотрите, дабы не увеличилось критчески потребление памяти, для хранения в ей динамически всей ентой бъды.
Ибо не весь народ не на уберкомпутерах работает.
why not similar to beesep..

BEEVERSION=${BEE_BINDIR}/beeversion

and

eval $(${BEEVERSION} $base)
You should never call `die` in a unit test, because it causes PHPUnit for exit instantly, and tests that would be run afterwards don't get run. You also don't get any PHPUnit output and an exit code of `0`, which is not very helpful for CI systems.

If you've to, then use `return $this->fail("can't open file")` instead. But I assume this is test code which was accidentally left in, so it would be better to simply remove it.
Try to standardize your try/catch statements where possible:
```javascript
try {
   // do something
} catch (err) {
  // oops!
}
```
" widget._userDevices[values.name] = widget._cloneSelectedDeviceInfo(); "   This is the original code. 
Originally ,we can only add a device that user selected last time . But now we can choose type . 
So we should clone a deviceinfo of that type.
"Unordered Lists" might be better than "UnorderedList"
Most properties of zones can be inherited from SimpleListItem
I'll add a description here
okay, checked again, result still the same.
Yes, the clearing is intentional. I'll extract it. The ∞-loop has been addressed.
yeah, can we have our own scala fork -- I think there is a company that provide such things :-)
Why this temporary val?
Haha, you're right, my bad :)
Fix this, just store the ID's
Wouldn't this result in `atlas://data\images\defaulttheme` on windows? If so, is that valid?
Sure, just mention it in the docstring so the user knows what to supply.
The server.conf file is formatted as code above but not here.
also rb_find_file() is not needed, since rb_require_safe() calls it.
For win32 you can use _vscprintf.
Try not to use `λ _` -- make the number of arguments explicit.
Would prefer using the format without the dot - so basically not the extension but the format. Can you change that? (Sorry if that was not clearly communicated)
Can you reformat the line to be <80 chars?
Interesting way to separate the test functions.  Did I do this in another test harness?  *too lazy to check*
this should be reformatted in a clearer way (what does `try handle` mean?)
semi-colon? really?
Could you please put a comment about what that does?
Maybe we should fix up the prototype of Buffer::New() sometime...
Would be good to have something along the lines of "e.g.  X --run=reload reloads the suite"
Space after ``arg``.
Yeah, the metadata dict is part of the exposed PyArray_Descr ABI, so we have to append the c_metadata pointer.
This pull request does more than use HTML5. It uses tables less, removes some inline styling,  etc, But you're right in that it adds almost nothing, so I'll close it.
Some considerations about code style. Put one space after the `,`. Like this:

```ruby
yield(count, word)
```
Extra space after `=`
Why need to check if the `rendered` is blank? Just return nil if `options` is blank
`resolve_pathname`?
Here is an alias instead of deprecating `set_fixture_class`.
I think that one is inside `PostgreSQLPurgeTest` test class, based on the indentation. Can you check?
Hey, english only.  Yippee.
There needs to be an empty line after
```
Examples
========
```
append;

`The key is the class and the value is a function that takes a single argument and
returns the desired SymPy object, e.g. converter[tuple] = lambda x: Tuple(*x).
So that's the current standard in the docstrings in sympy.physics.mechanics. I also think that something like m * **N v**P has a few too many asterisks for my liking. Maybe it's me but using ^ seems okay.
Yeah. Bronstein's book used a while loop, so that's why I did. Sometimes he uses inefficient methods, and I didn't always catch them.  That and I probably didn't realize at the time that range() can go backwards.
Because MutableMatrix doesn't implement Basic it doesn't need to provide methods like `free_symbols`. It does provide a number of these, like `subs`, in an effort to emulate the behavior of Basic. This is nice of MutableMatrix but not strictly required. In principle the bug is expecting that MutableMatrix will provide these things for us. If my understanding is off here then I suggest we create an issue for this and discuss it there rather than on this PR.
Minimally we need docs to describe why this is needed.  Why can we just call tensor_product_simp directly?
I see what you are trying to do here.  You should be testing `if class_m_mod is None or class_m_obj is None`.  The reason we found that bug is that it is calling bool() on each object, which is unnecessary (and wrong).
same as below. addressed now.
Prisms can also be useful when dealing with geometric rays, besides their dispersive features can be a relevant task to do.
put this row after the ```bug``` section in number order of your ench.
This file's changes are whitespace-only.
The binary <-> string conversions are unnecessary here. Just use binaries: 

EncodedId = base64:encode(<<"ejab\0", OriginalId/bytes>>),
<<"berd", EncodedId/bytes, "\0", Resource/bytes>>.
Merely my style. ok, fix it.
`ngx_queue_add()` so you only have to lock/unlock once?
well, unread_discussion_count/unread_discussion_count was never really explored by me...
funnily enough, the migrate still runs
You undid my addition of :new
What's this? I didn't know // does that
A few lines back I made a comment about `overlapsShouldBeReflexive` - is it maybe a case that something got duplicated and maybe one can go away now?
This should alert.
cross reference?
Gotcha, didn't see the to_hash -> to_json connection.
Should have a limit 1 in the query, and aren't you missing a sql_free_result?
I'm not really convinced by this one. I'd rather base it on 'f_noapproval', rather then updating it for some role modules which may not even be used.
It should work. Check other tests.
What is the purpose of having a unique value here in the first place?
The order of arguments is reversed. Expected value is first, value being tested is second.

Also, please assign at least the value being tested to a variable in a separate statement.
I've read this block a dozen times and every time its confusing.  We have state in 3 variables for a check- altered, seen, and vis.  
The functions register and deregister are being called from two different loops.
това като че ли би изглеждало малко по-ясно ако е така:

```ruby
      @url = if lecture.has_key? :slug
        "/lectures/#{lecture[:slug]}"
      else
        lecture[:url]
      end
```
Why `mojito-view-renderer-client-tests`?  Where is this YUI module defined?
почему отдельную таблицу не стоит? во время апдейта показов запись лочится, и другие ноды не могут селект из нее сделать

понятное дело что за все время.
ты же не хотел чтобы нода занималась создание статистики? а теперь не хочешь чтобы это делал пхп)

смотри как я думаю...
таблица баннеров с их параметрами
таблица с количеством показов/кликов с форейн кеем на таблицу баннеров. вида unit_name, shows, clicks, last_shows, last_clicks
таблица статистики вида unit_name, shows, clicks, date

нода по интервалу делать апдейт таблица показов/кликов обновляя shows/clicks

в конце дня нам нужно создать статистику за день по баннерам
в одной тразакции делаем запись в таблицу статистика разницы между текущей суммой показов и прошлой(last_shows) и обновляем last_shows=shows, 

что думаешь?
Ага, именно так.
Currently, Gentoo Linux's LiveDVD includes ZFS in binary form as does Sabayon Linux, a Gentoo derivative. That passed the Gentoo Foundation's legal review.

I had a brief chat about this in IRC with, @behlendorf, who is the ZFSOnLinux lead developer at LLNL:

12:33 <@ryao> behlendorf: You might want to look at this: https://github.com/NixOS/nixpkgs/pull/145
12:36 <@ryao> behlendorf: In particular, the discussion there reveals that the current FAQ entry on licensing is somewhat vague. As far as I (and others) can tell, a derived work would require building ZFS into the kernel. As a separate module, it should no different than drivers being distributed by various companies in the form of binary blobs.
12:41 < behlendorf> ryao: Thanks, I should probably rework that entry in the FAQ.
12:43 < behlendorf> ryao: And your absolutely right.  Distributing source is 100% fine and binaries are OK too as long as they are done as kernel modules.

The two of us appear believe that you should have no license issues.
Since this is part of "printing", please simply add these two lines (373 and 374) to the ```printPrescribedDrugs``` method. It's confusing naming to have a method called ```storeLocally``` cause a popup window :)
on certain databases like PostgreSQL this will not work because two constraint names are same.
use - FK_<baseTable>_<columnName>_<referenceTable>_<columnName>
The PR mentioned above (bblay/iris#1) fixes these.
why $__file__
k, I changed it to call list()
Minor nitpick: this would be easier to read as

    ret.extend(c_compiler.compile_args())

But it's fine as it is.
Typo: types
Ning/Ariel verified that this code is covered in a unit test and that a system test also exists to validate this logic. I don't see any errors in this conversion by reading code. I think combining that - it should be fine.
Nice review. You are right, and I should make it simpler. Thanks Paul.
Для чего `html` переставил? В этом есть скрытый смысл?
Nit, but users tend to have long names. Could use a wider span...it looks like input.span is a fixed width.

Also, should include a static error message--well, maybe not here, but in our form templates when they become label left.
Did you mean to leave this one here?
I will try to see if I can figure something a little less scary for this. I am pretty sure as-is the code will be source for strange problems. To me the managed/unmanaged flushing seems to point this out - even if you have different app-cache you are still altering the global models, and that is the scary part. The models are assumed to be immutable, and altering them might result in weird, weird bugs.

Note: my second comment above isn't a reply to Andrew's comment, it's concurrent...
this is what we dont want to see
unbound, i.e. something that will never be bound to anything...
Having read through igd.hpp and here I still don't quite understand the distinction here.  Hopefully will become clear after reading further.
"in" is more semantical.  This is not the case here, but if you have set or dict as the argument, "in" works for both.  Therefore, "in" is more preferred in general wherever available.
We/I can probably just fix this for you if you don't want to bother with a pull request for this tiny detail, just let me know what you decide :-)
This should probably be left as-is, it may compile with current LLVM (build 2336), but that doesn't say anything about build 2334.
Can remove the cmake buildtime dep now, I assume ;)
So each compile command that is issued is run through these filters before it gets sent to the shell?  If so, has there always been some filtering?  I'm curious about the performance hit on long compiles like boost when building from source because I want icu4c.
Here is option 3 in untested code form: https://gist.github.com/2578478

It's not mutually exclusive with Option 1 of course. I'm pretty ambivalent on whether adding 3 to 1 is worth the code, it sort of depends on how often Java developers chain Future methods.

Option 3 seems nicer than Option 2 to me.
I think "A Method is a great way of collecting useful sets of instructions and reusing them later" would work better.
Und ist es richtig, dass der Zweig nur greift, wenn kein User vorhanden ist, aber nicht, wenn ein User vorhanden, aber kein Admin ist?
stimmt..
You probably want to change this to `Integer`.
I went back and forth on that one in my mind. On the one hand, consistency is good, but on the other hand, it doesn't make sense (to me) to use absent or present for a packet filter that is built into the base OS and can't really be /made/ absent or present without recompiling the kernel (with or without the required options). All the same -- I can make the change, but it does strike me that perhaps as cdist grows it might be good to adopt an alternate scheme for "state" parameters. Perhaps absent/present for things like pacakges, files, directories and enabled/disabled for services and anything that could for example be fed a ruleset or turned on or off temporarily. To me it just seems a bit clunky to try to make everything fit into "absent/present" whether it feels natural to a user or not.
Use list.getParent()
`fi; `
建议链接直接插入原文
We should probably take this out.
I'd also like to catch the case (and test that we catch the case) of:

```java
@Parameter(0) public int x;
@Parameter(2) public int y;
@Parameter(2) public int z;
```
Removing this seems reasonable to me.  InlineEditorProviders-test contains a solid suite of tests ("Inline Editor range updating") that make sure inline editors don't lose sync and close at the wrong times, and at least a few tests that also cover Document-Editor syncing.
Oops. That was inadvertent. I'll undo.
Ah yes. I'll reorder before ``htmlReady``. There may be a brief time when the title is blank, but I haven't seen it in my testing. We can revisit if this is an issue.
I made some adjustments to get it working in git bash on windows.
LiveDevelopment requires an XHR request to localhost, which is not allowed under most conditions. Therefore, we have to disable LiveDevelopment for now. This could be enabled for files hosted on localhost (kind of a very special borderline case though) or by switching to a browser extension.
Yes, you're right, I'll be changing that. I guess at some point I ran out of resizing names ;)
It works only if you are inside the inline comments. If not is possible to do a block-comment.
I just checked and it is not used anywhere. It is save to delete.
may I suggest "Usage: argv[0] <words_file>"
Do you know why this is necessary at all?
Yup. I should do something better, but I ran out of time at the end of the day. Will revisit tomorrow morning.
Ah, I see the tie in to Shopify now. I think the session storage bit should be a module which MultiDB includes in ApplicationController when `setup!`
It is recommended to use:

    self.assertTrue(self.filesystem.isFile(segments))

or:

    self.assertIsTrue(self.filesystem.isFile(segments))
I don't follow this at all. Please add a comment.
maybe call this `ignore_datespan`?
Guava `Strings.isNullOrEmpty`
Czy nazwy stanów muszą być po polsku?
+1 ; all the code is `sqlconsole/codereformat/` isn't ours, it is something erwan took somewhere... and it has a different coding style.
I am not sure what we should do with that...
Because we want to stay on the same page when we change language right? So I get a current url with JavaScript and set up the value with jQuery.

will move jQuery in another document.
Those signals already exist in `django.contrib.auth.signals`. You can just reuse them?
The package is named only `mongodb` for Debian.
I'd like to point out that I put this here because the drag hotspot had a tendency to disappear under the elements created by Google when switching to streetview. this fixes the issue... for now.
Yes it was. Look further down in teh code.
Add comments that this is for a reset stylesheet
Question: why do you querySelector for the main div, don't you get that automatically?
Actually, on second though, it is unnecessary to even check for the truthiness of type; the next condition, `typeof type !== 'string'`, will already assert that type is defined.
Sadly I have no idea why this was done... I'm just pulling in a change from the current production code.

Based on the surrounding code, we seem to be choosing which locale should be tested. Historically we test the alphabetically last locale available, because rsync generally runs in alphabetical order... the thinking is that if the last locale exists, then the other do too. It's inherently a trade-off designed to avoid having to test every single locale individually.

I read this as "if seamonkey, and if it's version 2.0.5 or 2.0.6, check zh-CN. For all other versions, check tr".

Based on wandering around here: http://download.cdn.mozilla.net/pub/seamonkey/releases/ ... it appears that Seamonkey did indeed have zh-CN support for those two versions, and potentially a few others. However it was not consistent. 2.1 did not have it... 2.13 (current) has zh-CN and zh-TW.

However, for the most part it doesn't matter anymore. The lone mirrors are reverse-proxying CDNs, not rsync-based. The logic of checking the last locale no longer holds any real value... any locale is just as good as any other. We still need to check *a* locale, but it doesn't matter which one.
ましまし じゃなくて ましまろ だと思います。
Does this work if Python is compiled without +ssl?
no real reason... We already had mime-types.properties in use for MimeUtils2 configuration, so I chose something different to not have a user confuse mime-type.properties and nexus.mimetypes. That's no reason for the builtin types, though. I'll change that file.
A little confused about what this is supposed to do. It's pushing a key/value to Redis, then setting the expiration?
Is it okay for the expiration to be set before `lPush` does? It seems like this should be more like:

```
database.lPush(string, List(value)) flatMap { _ =>
  defaultTTL match {
    case Some(ttl) => database.expire(string, ttl.inSeconds)
    case None => Future.Unit
  }
}
```
do you really want to merge this url config to master?
i think we should find a way to keep this separate
Ah okay, I have no idea how clients handle this. That's also the reason I added `media:title` and `media:description`.
Why is tid_s 16 characters in size then?
亲能再改改吗？
From what version of rhsm is "config" command supported? Will this work on RHEL 6.0 GOLD (without any updates applied)? Please test.
Should we consider defining this color somewhere for re-use?
Was this intentionally removed?  (Just asking, in case it was not).
you should use ``@link`` for this instead.
can you put this next to require rexml? just to keep the stdlib requires in one place :)
Thank you for keeping the consistent camalCase style for l10n label.
So how would you change it?
Sure.  I'll add those.
Why use the variable in this case?
yes yes thankyou for the explanation, now that I understand what you are doing you can remove it :)
What is the point of nulling the pointer to the allocator?
bu capitalize ve decapitalize fonksiyonlarini tartJS icerisine alsak mi?

parametre alir, burada da return ```tart.string.capitalize(this.propName);``` diyiveririz?
you should remove the typehint to allow any callable (the right typehint would be ``callable`` but it would bump the requirement to PHP 5.4)
should the ArrayCollection implement the interface ?
it should be 

```php
                if ((null !== $paramClass = $param->getClass())) {
```
Why 5? I didn't want to query all the branch history in one time. I did this modification when someone said me it was too long. But I don't know how TFS really work with that. I can put up this number (20? 100?) or remove querying history by intervals...
list doesn't concatenate its argument. If it is `expanded.append`, it is equivalent, but it's `expanded.extend`, right?  I need the reduce function to do this in one line...  I guess you don't want the reduce as IPython needs to support Py3k?
interesting, so this is just a flat map
That might be a false promise.? From a short look on the code path, it seems to be true atm, but what's returned when the passed ref has been deleted between calling `git_reference_lookup` and `git_reference_target_type` for example?

I don't really care, as you like.
This switch could do with a default case to avoid compiler warnings.
keeps it in the scope. don't need a global function here.
Why aren't we using https://docs.djangoproject.com/en/1.4/topics/http/shortcuts/#redirect here? It already handles distinguishing between view names and urls.
The three versions of this template seem to differ only by one word. Can they be consolidated?
can we activate this?
That's a bummer.  Ok.
Even better: queues.values.inject(0, :+)

:-)
Actually no : gh-pages fails compilation with the cname it seems to check it...

--
Romain Dardour, Oahu
romain@oahu.fr


On Oct 28, 2012, at 6:23 PM, Mike Perham <notifications@github.com> wrote:

> In CNAME:
> 
> > @@ -1 +0,0 @@
> > -sidekiq.org
> I think you can still use hull.github.com/sidekiq/pro, yes?
> —
> Reply to this email directly or view it on GitHub.
>
You can do another commit to that branch and it will get automatically applied to this patch :)
returning MaxSessionsExceededException
this should be ``2.2.x-dev``
``init`` at the end ?
That's an unnecessary line since it's set by ref in the call below
Please use camelCased names for your variables
shouldn't this check be moved to the constructor to avoid checking again on each validated object ?
Hi @sobrinho ! Happy to oblige, although both were in there before.
I like that too. :sparkles:
This one might not work ince there are ''s inside
Strange syntax? Does it deserve a comment?
I have done this, but it seems github doesn't hide this comment...
This one looks a little odd, but I'm struggling to come up with something better. Perhaps putting the penultimate parenthesis at the end of the previous line would help.
was just curious, don't think it really matters. Dan Pupius used the trailing _ initially as well and I think he's switched to leading (though I might be wrong)
sendfile(2):
```
     [EAGAIN]           The socket is marked for non-blocking I/O and not all
                        data was sent due to the socket buffer being filled.
                        If specified, the number of bytes successfully sent
                        will be returned in *sbytes.
```
als het door de disk komt is het
```
     [EBUSY]            Completing the entire transfer would have required
                        disk I/O, so it was aborted.  Partial data may have
                        been sent.  (This error can only occur when
                        SF_NODISKIO is specified.)
```
Mine came from the exact cgminer code that it's using to time the search. The difference is under 2 MH/s, that would take probably a month to measure; if you want to test it, I bet this number comes out correct.

You *want* to include the software overhead into the calculation, since applying it will *also* have the software overhead.
no mocks here, for what is the runner needed?
I don't think that the change per se is a bad one (it makes the fake request look more like a real one). But I'll admit that it may cover the real problem. Let's try to fix that first, then merge this
The app should live somewhere other than the blueprint, because it will eventually load multiple blueprints.
Are we still using `linkifyTest` somewhere?
Never mind, they seem to be there in the original subsystem.xml
Agree, even audit could be simplified. Changing and sending again.
If any of the other tests in this file left an uncommitted transaction then the assert would fail. You can handle this by doing something like

   int txnCount = new TxSupport().txCount();

then you can compare with txnCount rather than 0
pkg_resources comes with setuptools, the package that gives you easy_install.
It doesn't stop IE from moving focus to the menu, so we actually need to update menu. Could also address the selectmenu issue, which currently has to override various roles.
Add a sentence like "theano functions computing `gradients` must apply all of these updates in order for successive calls to the function to be correct."
No problem, no rush.
I thought we are going for `window.alert()` for the error prompt? What's the reason to use desktop notification?
If `alert()` doesn't work, it's a bug in the System app. Please file a bug and I will fix it.

Also the labels are too geeky. A generic "This PDF document might not be displayed correctly." should work.
Only if you want, the translation is "Sugerencia de palabras"
nit: please unprefix
We'll need to explode all the cases here.  I just finished documenting this at https://wiki.mozilla.org/L10n:B2G#Plurals

So we'll need `others[zero]`, `others[one]`, `others[two]`, `others[few]`, `others[many]`, `others[other]`, even though some of those are redundant in English.
Thanks! Good catch!
Yes, it does. See lines 69, 71 and 80 ;-)
not it's not! good catch!
I'd guess that all this code for creating elements and setting attributes is very similar to code you already have for album and artist views. Can you write a single utility function that you can reuse?  That's not a blocking issue, of course.
During a new SMS the focus should be in the input related with phone number.
transform no longer needs to be prefixed: https://developer.mozilla.org/en-US/docs/CSS/transform
I'm kind of worried of this. That would make it easy to delete a phone number just by scrolling :/
add a name to your anonymous function
Once we have our DOM element of our Thread we launch ContactManager.getContact(num, updateMsgWithContact) in order to retrieve contact (from cache or API) and apply to DOM element.
add `var `
You are right - av-assignment should be done before 'if...', my fault.
The original problem will stay without any patch - if accept(v) results to nil, merge will fail.
Could even do

```
*BSD)
```

because I think NetBSD or DragonflyBSD or the other kids on the block would work the same, but I don't have anything to test with, and they are not supposed to be supported at all :-)
Again, we install this, so why ask them to do it manually?
But I still need to distinguish each type of archiver. How would I do this without passing that stuff from Redirect.thread(...) to Redirect.archiveLink(...)?
i know it's a pita, but can you please alphabetize your css rules? :D
I originally used the `list()` function to make a copy of the list because items where removed from `zen_tickets` in the loop, but this no longer happens with the changes I've made, so I'll the function call.
I commited it by using the edit feature in browser :+1 and I also deleted the fork I made
Fixed it by doing

    if install_server_type == 'cobbler' and install_server_url is not None:
It would be helpful if this error gave a bit more detail on what value needs to be an int.
better way of doing this? maybe a `before`? Didn't want to muddy up the surrounding code.
How about format this line?
good change, although normally I would do it in a separate commit :P
can not inheritance. previous version is compile error in Scala 2.10.0

```
[error] /Users/kenji/dispatch/dispatch/json/src/main/scala/Json.scala:76: case object JsTrue has case ancestor dispatch.classic.json.JsBoolean, but case-to-case inheritance is prohibited. To overcome this limitation, use extractors to pattern match on non-leaf nodes.
[error] case object JsTrue extends JsBoolean(true)
[error]             ^
[error] /Users/kenji/dispatch/dispatch/json/src/main/scala/Json.scala:77: case object JsFalse has case ancestor dispatch.classic.json.JsBoolean, but case-to-case inheritance is prohibited. To overcome this limitation, use extractors to pattern match on non-leaf nodes.
[error] case object JsFalse extends JsBoolean(false)
[error]             ^
[error] two errors found
[error] (dispatch-json/compile:compile) Compilation failed
```

I have no confidence my approach is best.
BackboneのViewを違うDOM要素に適用したい場合、**setElement** を使用します。このメソッドは`$el`の参照をキャッシュして、Viewがもつイベントの委譲を古い要素から新しい要素に移動させます。
Ah, I see, it tracks my pushes since the initial request...
>>. I prefer to have automated tests.
Is there problem? I can write tests for full covering.
@SaitoWu Can yo explain me this lines? And does running utf8 on every Raw object cause Grit Memory Max for big commits?
needed for a test?  comment at least to hint at purpose
I copied it here for kernel usage.
memset may be expensive and since there are just a few members a clear macro like with packets and flows may be better?
spelling:  `s/explcitly/explicitly/`
I do agree that consistency is important, and am willing to cede most of
these changes (for now!).

However, re: this particular one, I think trying for "consistency at each
level of inheritance" is a flawed thing to optimize for. We use inheritance
because we want to _change_ the behavior of our objects, and that will
occasionally necessitates changes to how we instantiate them.

Consider the extreme case: everything we're working with are "just"
objects, but surely we're not going to use the default object instantiation
signature. We've added a parameter and carried out some functionality that
simplifies other code, instead of copying and pasting several lines into
every subclass.

convinced / unconvinced?


On Fri, Dec 7, 2012 at 12:45 PM, svevang <notifications@github.com> wrote:

> In lib/persistence/redis/Base.js:
>
> >
> >    var redis_model = require('../backbone_sync.js')(model)
> >
> >    redis_model = redis_model.extend({
> >
> >      pk: function(){
> > -      return name + ':' + this.id;
>
> I was just trying to keep the require callback interface the same at each
> level of inheritance.
>
> see lib/weiqi-models.js, everything is treated the same.
>
> I'm trying to articulate the need for this branch: Consistency, until we
> have a reason to (eventually) abandon it. This is tricky. I think we want
> to write another persistence layer eventually, so having a consistent
> approach is key there..thoughts?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/michaelkirk/weiqi/pull/11/files#r2353459>.
>
>



On Fri, Dec 7, 2012 at 12:45 PM, svevang <notifications@github.com> wrote:

> In lib/persistence/redis/Base.js:
>
> >
> >    var redis_model = require('../backbone_sync.js')(model)
> >
> >    redis_model = redis_model.extend({
> >
> >      pk: function(){
> > -      return name + ':' + this.id;
>
> I was just trying to keep the require callback interface the same at each
> level of inheritance.
>
> see lib/weiqi-models.js, everything is treated the same.
>
> I'm trying to articulate the need for this branch: Consistency, until we
> have a reason to (eventually) abandon it. This is tricky. I think we want
> to write another persistence layer eventually, so having a consistent
> approach is key there..thoughts?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/michaelkirk/weiqi/pull/11/files#r2353459>.
>
>
Convinced! I was not sure about the whole pk thing.  Overall I agree, I'm triangulating a way to replace this (viz mongo rewrites)

I can commit a change to revert this behavior and fix the one test that is affected by this branch, that may be a few hours down the road though.   I have to take a dinner break..
I'm fine renaming this to `manifest`?
But I don't think it really worth modifying the final name of `harness-options.json` file in xpi.
Nit: It would be easier to read if you head line break just before `Array.isArray`.
That is strange, I would have expected `nsIStringInputStream` to inherit whatever encoding string was using, but yeah I never understood when to use which stream. If that does not works lets leave as is then. Although if we could get someone to explain which stream should be used when that would be great.
Thanks for your feedback @6a68 and @zaach - It does seem likely that multiple cef calls from within the same module will have to have different signatures, yes.  But at least the merge call will be easy to take care of.
is that a missing "C" in CONFIG_FILES?
I vote for __Issue Number__s, because they are more descriptive, they are the goal, the drivers of development. 
Pulls are tools of controlling  - how the changes are accepted and merged into source.

Of course Pull Number is OK, when no Issue Number exists.

Other way of reasoning:
Pulls are closer to compare view (e.g. [v3.2.2...v3.3][1]), and issues are closer to milestones.
And release notes (`history.md`) are closer to milestones, IMHO.





[1]: https://github.com/mozilla/nightlytt/compare/v3.2.2...v3.3
Already merged, but what was your reasoning for doing `Hash[*options]` instead of just `options`?
Should be ``AMQPExchange::publish($message, $routing_key = null, $flags = AMQP_NOPARAM, array $attributes = array())``
I want to make it explicit that we use the default connection in this case. We have this rule of thumb to always pass a connection object when a query is executed. Here, a query is executed but there is no connection object to be seen, so I wanted to make it clear.

Alternative: add `$con = null` as argument of `getIterator()` and pass this connection to `find()`. But this is not the usual signature of `getIterator()`.
me hubiese gustado q no pongas lineas en blanco
Same as before. I think we should split this line to multiple lines now to make it easier to track.
I'm with @daniel-pittman on this. I would really prefer complete, if verbose, errors to ones that might be removing needed information. Many users package their own code under the `Puppet` namespace and so this would mask errors in their own code, as well.
I think we don't need `defined?` check and `class FormtasticBootstrapBuilder < FormtasticBootstrap::FormBuilder` would be enough.
Return the schema name in the response
This should be `yz/index/test`, no?
This change seems odd.  Can you explain it a bit?  In particular, when is the adapter function run on the default values?
These comments should be removed, since it does in fact use UUID.
I'd actually argue all online tests should be disabled by default, tbh.
add a space between "if" and the opening paren
Really this should be `as MainLdapExceptionInterface`. Not that important though.
Remove line 165, and align the assignments (=).
missed `@` in front of return
Use the Accept header object's `match()` method instead of what you do in here; it's more robust, and won't return a false positive in situations where other content types have higher priority.
For clarity just use if/else statements to set spinSpeed variable before defining this object.