labofsahil/github-event-dataset-2012 huggingface dataset:
yes my captain! :o) I cross-compile from Linux.

The first problem I found was trying to statically link magrit to the dependencies (i.e. boost) in order to ease the installation for Windows but I didn't succeed. I guess we can afford ourselves to install the dependencies along with the executable in Windows? (3 boost libraries so far: program-options and 2 transitive deps of that one).

My second problem is regarding platform nuances. There are some paths (e.g. /usr/i686-pc-mingw32/sys-root/mingw/bin/) and some library suffixes (e.g.: gcc45-1_46_1) that add a bit of noise to the Makefile. I was wondering if Ubuntu or whatever are using the same conventions?
We like to test that the software works, not that files were installed where expected.
easy testability but that's it
This signal is a candidate for G_SIGNAL_RUN_FIRST with a class closure that
sets the changed flag. That way you don't have to connect to the signal in the constructor.
Great job!  This pull request really should be merged.
Muchísimo mejor declararlo dentro de <code>setup</code> como variable de instancia, algo como:

```ruby
@ac_field = 'auto_document_print_job_print_print_jobs_attributes_0_'
```
capitalize "Details", and I think you can use `_` instead of `loc`
this doesn't seem like a great idea given that the manifest actually allows up to 128 characters, but if this makes @krupa happy r+
保存してたデータの削除です。
とはいっても、現状はetsにデータを入れているので、あまり意味ないですよね（汗
後々、データストアにMnesiaを使おうと思っていたので、その時に意味を成すかなーと思ってますです。
Thanks!  You're right.  Updated it again :)
The main NERDVANA WEBSITE
Thanks for the ping support! I'm counting on the ping packets for donpdonp/icecondor to know that the server is still available.
The latter is ok. The verb is "to spend -> spending"

Sincerely, Pavel.
why are you building a list? `addon.device_types` is already a `QuerySet`. you can do `old = set(addon.device_types` to coerce it to a list.
as locknes and me have tested together the script the rocket pack got unequipped at us
why HashMap? why not private?
sure, I totally understand your point and as I said, that'd be something nice to have.
Now, from a conceptual point of view, magit-visit-item always brings up something on which you can continue working (be it a file, a branch, a commit...). On the other hand, calling magit-show results in something more like a view of something that's immutable and cannot be used as part of a git operation.
That's why I'm wondering if we should introduce some `magit-view-item` action, and if there are other operations we could give a similar semantic.

Admittedly that's a open question :)

Regarding C-u, currently it's used to visit in "other-window". I personally don't have a strong preference for that, but changing it would require some solid rationale.
yeah accidental.. sorry..
I added a new relation between sender's copy of the reply and the message that is in the sent box of the replying user. Therefore the other message is owned by the replying user. And since the access_id of the messages is set to private, the original sender can't access the attached message without elgg_set_ignore_access().

I wasn't sure if this is a good way to do it but I could't come up with any other solution within the time I was able to spend for this.

I suppose we could trace back three messages (the reply in inbox -> replier's sent box -> replier's inbox -> sender's sent box) and attach that message to the reply. Then the attached message would be owned by the original sender. But wouldn't we still have to use ignore access to trace the way back to that message?

I hope I explained this well enough. The connections between the messages are a bit confusing.
[Sorry, should specify, it was crashing right after  the ___DumpData statements in the prepareData() function of BamftActivity.java, ie when the app first launches.
Looks good. Thanks @nickmbailey
なるほど．ありがとうございます．
Невозможно глупая иерархия
Надо сделать интерфейс Item, лучше еще создать абстрактный класс, расширяющий Item
а каждый item вынести как реализацию Item, наследующую абстрактный класс

+методы с большой буквы и непонятно зачем!
They both work. Though this regex is horrible. Cant use quite a few special characters.
I'd move the comment to the condition in Endgame<KXK> where seems to be more appropiate
Oh, ok, I'll make sure to fix it. Gah, just when I thought I had everything working 100%
We got to think again if we should use IQuerable in the view.  The reason why is because we want to deliver to the view a model or viewModel that is already present in memory.  That is why the controller should do a .ToList() on the model before giving to the view.  The view takes the in-memory model, and renders as needed.  In the case above, when you call Model.any(), then the sql statement is called at that time (due to the lazy execution architecture of using IQueryable.  Not what we want the view to do here.
oh god, these belong in annotations or text files...
or you put them into the PNames entries
This seems to still be an issue.  Even w/ this patch, Chrome (17.0.963.56) gets confused on at least 2 zoom levels.

I even tried forcing the isEmbedded state, but then I get a weird problem where the screen slowly shifts up a pixel at a time once I start scrolling.
From what I could see in **header.php** I don't think that the included would have the <pun_*> tags evaluated, but since it does use PHP's include it would evaluate PHP.
Shouldn't this be bound to the child?  `child.disconnect.bind(child)`  Otherwise it'll be called with `this === global`, no?
If you REALLY want - you can select all blueprint script tags using $('[id$=_fields_blueprint]')
@gseguin BUSTED!
Shush. I'm not used to git yet :)
On Mon, Feb 20, 2012 at 10:59 PM, Jani Monoses
<reply@reply.github.com>
wrote:
> Do you have a pointer to a discussion where upstream discusses this issue? I saw nothing besides the JIRA issue. It did not seem to me like the upstream devs commenting there were against any change in particular.

http://groups.google.com/group/mongodb-dev/browse_thread/thread/ecf8451dd735c581

Maybe you should bring up the subject again.
Still doesnt work.. need to fix.
No problem, lo puedo renombrar. Pensé que ´md´ funcionaba mejor.
Ага, ага. :D
Also, could you show me where you push the module in the locator? Is https://github.com/EvanDotPro/zf2/blob/7399f348334a5c7255bcb0cf5505393d21e7df6f/library/Zend/Module/Listener/LocatorRegistrationListener.php actually happening?
Nevermind. I see https://github.com/cappuccino/cappuccino/issues/1457

On Tue, Feb 14, 2012 at 12:08 AM, Adam Radabaugh <adamradabaugh@gmail.com>wrote:

> How did this break mouse tracking? I have a better solution for the
> _highlightsBy test and also modified CPButton to properly initialize these.
> The mouse tracking is new though. Do you have an issue or group post to
> link to?
>
>
> On Mon, Feb 13, 2012 at 8:34 PM, Adam Radabaugh <adamradabaugh@gmail.com>wrote:
>
>> Agreed.
>>
>>
>> On Mon, Feb 13, 2012 at 6:52 PM, aparajita <
>> reply@reply.github.com
>> > wrote:
>>
>>> There are three problems:
>>>
>>> - His commit edd09ef06a (which you merged, doh!) broke CPButton and
>>> CPPopUpButton mouse tracking and also broke updating of titles in
>>> CPPopUpButton.
>>>
>>> - I don't like this test for `_highlightsBy == undefined` either, it
>>> should *always* be set to something. That tells me that stuff is not
>>> getting initialized correctly. Both _showsStateBy and _highlightsBy should
>>> always be initialized, currently they are not.
>>>
>>> Personally I think edd09ef06a should be reverted until all of these can
>>> be addressed properly. Not having proper mouse tracking and title updates
>>> is a much bigger problem than not having the extra button types.
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>>
>>> https://github.com/aradabaugh/cappuccino/commit/915b1f3638c1ee5e71989570fdb3ac399aee220a#commitcomment-962924
>>>
>>
>>
>
okay, now I understand!

thx :)
Inconsistent formatting - open { should be on previous line, here and many places elsewhere in the code.
Let's just rename the project to "HUAAA".
It looks like most classes use dashes, e.g. "order-number". Yet some use underscores: "order_details" and "customer_details".

Is there a specific reason for that? If not, more consistency would be better. I'd prefer dashes, which is used WordPress classes too in most cases.
I did +1 here but seemed wrong anyway >.<
This comment makes sense in Python, where the method name is weird (`__add__`), but it seems obvious in Ruby.
I also updated the template with `<header tabindex="0">` to allow for keyboard focus.

In Safari, Firefox, Chrome, and IE <= 9, pressing Enter doesn't trigger the click event on a header when it's in focus.

I need to address the Opera issue, though, as it's doubling up the event.
uneccessary brackets here.
suggest you add .DS_Store to your .gitignore!
Northrend, Outland and most of events data it's the same 3.3.5 :P.
Ok, addressed now. It seems to be an issue of GitHub editor. =\

Check now if it works... =)
This isn't true. When it fails, the iPhone re-sends the assistant data needed along with another copy of the speech request. I have no problems here and the user doesn't need to know other than if it fails multiple times. All the code does above is set a default failure language.

```
INFO process_compressed_data Sending flac to google for recognition
INFO process_compressed_data Unable to find language record for this assistant. Try turning Siri off and then back on.
INFO read_next_object_from_unzipped Received a Ping (10)
INFO read_next_object_from_unzipped Returning a Pong (10)
INFO process_compressed_data New start of speech received
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data Decoding speech packet
INFO process_compressed_data End of speech received
INFO process_compressed_data Sending flac to google for recognition
INFO read_next_object_from_unzipped Received a Ping (11)
INFO read_next_object_from_unzipped Returning a Pong (11)
INFO process_recognized_speech Best matching result: "This is a test" with a confidence of 94.66%
INFO read_next_object_from_unzipped Received a Ping (12)
INFO read_next_object_from_unzipped Returning a Pong (12)
```
哈，新年快乐.....已经没劲啦。
Ha!  Good suggestion…  

Dave


On Friday, March 2, 2012 at 6:04 PM, Seth Milliken wrote:

> You might want to consider removing the PDF generated from these files; it's almost comical: http://davetron5000.github.com/scala-style/ScalaStyleGuide.pdf
>  
> ---  
> Reply to this email directly or view it on GitHub:
> https://github.com/davetron5000/scala-style/commit/101f54e6edf35611f646442dab05e8a3580f28e0#commitcomment-1037693
>  
>
Is this still necessary do you think with ARC enabled?
This cannot work. Either @stdout is nil/false, in which case Ruby will try to call #read on it, or it is truish and Ruby will do nothing.
It can only be found on github at the moment; we are using a git checkout of the router in our projects.

https://github.com/joshbuddy/http_router
I don't the client performs other checks.
Hey Scott,

This is wrong. This is the base url for OAuth2 on Weibo not V1 which is what this gem is set to point to.
use shorthand here?

	 setMenuItemState('go-forward', currentHistoryIndex < pageHistory.length - 1 ? true : false );
Yes, it's better
Ok,  this kind use of `Internal` I'm not immediately concerned about.
 I did clean up your code a little bit
[here](https://github.com/lpsmith/persistent/commit/c28937aeba5f2874bea2e545219e5da35aa388b5);
 the use of withConnection would have avoided the breakage.  ;-)
However, there are a few things to point out:

1.  Beyond the process running persistent,   this isn't really a
"streaming" interface;  exec blocks until the PostgreSQL backend is
finished returning results.

2.   I have attempted some approximation of a streaming interface
[here](https://github.com/lpsmith/postgresql-simple/commit/bce0d389fd16a35a88841bb8434fab264b824993),
  note the caveats (although judging from your code they don't apply
to you)

3.  Your second point might be an interesting one.

On Mon, Jan 9, 2012 at 11:51 AM, Felipe Lessa
<reply@reply.github.com>
wrote:
> Please take a look at https://github.com/meteficha/persistent/blob/postgresql-simple/persistent-postgresql/Database/Persist/Postgresql.hs#L106 .
>
> I'm doing it mainly because `postgresql-simple`:
>
>  1. Doesn't provide a streaming interface for the results, only a list interface.
>
>  2. Wants to statically know the type of the result.  While I could create something like `newtype Row = Row [PersistValue]` and give it a `QueryResults` instance, I'd need to parse `typename` for all columns of every row.  I think that this could be solved if you used `convertResults` partially applied, that is `convert = convertResults fields`, allowing me to do something like `convertResults fields = let ... in \vals -> ...`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lpsmith/postgresql-simple/commit/efb67bac17887a8db7416f54745a2f1af56a033b#commitcomment-850098
deleted a file that would cause rake db:migrate to fail
I'm not sure if this is the right thing to do. Adding a comment does not modify the content object. I think this should rather be handled in plone.app.caching. It has the effect that objects reappear in the last modified portlet if a comment is added and that all objects with comments get their modification time reset on comment migration.
Shouldn't MPolynomial reader be in DGtal/io/readers.. ? 

don't know..
yes, and breaks.
This whole if statement could be inside one PHP block; it would be much easier to read. Something like:

    if ($_POST['function'] == 'a') {
      $total = $_POST['number1'] + $_POST['number2'];
    } elseif ($_POST['function'] == 's') {
      $total = $_POST['number1'] - $_POST['number2'];
    ...
It's best to apply a regex replace on "charset=", which preserves the previous mime type (eg application/x-publican), rather than completely overwriting the original Content-Type.  Unless of course there is no Content-Type, in which case text/plain is okay.
the new DoctrineBundle repo is for 2.1 only. For 2.0.x the bundle is still in the Symfony code
ALTIJD
No, that makes sense. Thanks for the fix, I knew I didn't like that part but wasn't sure how to correct it.
Which fix? [This](eca46c1)?

This commit only changes files in the contrib directory. The diff is misleading, as the discussion above should make clear.
It looks like this should be operationTimeout != null

When running the tests I see NullPointerExceptions are being eaten here in the logs.
-    D2605CB214465F2C0029F16B /* UserVoice.h in Headers */ = {isa = PBXBuildFile; fileRef = D2605CB414465F2C0029F16B /* UserVoice.h */; };
+    D2605CB214465F2C0029F16B /* UserVoice.h in Headers */ = {isa = PBXBuildFile; fileRef = D2605CB414465F2C0029F16B /* UserVoice.h */; settings = {ATTRIBUTES = (Public, ); }; };

also breaks archiving
No, no, of course I believed your code worked, I was just happy that I figured out how to call it properly. Hopefully in the new design it will be easy to change out an approximate search method if I would like to try that later. At this point I think I need to get it back up and running at least how it was before but with the new infrastructure.
Yes. Please stop commenting here =/
> Personally I don't really like the approach of storing the class name as string and instantiating it via reflection. While it does remove the hard 
> coupling via the import statement the type of the implementation would still be known in the API class, thus actually leaving the dependency, at 
> least from a semantical perspective.

I see your point. However, in this case it will always be our own known implementation. I am happy to have a "semantic" dependency in exchange for less code and avoidance of potential risks. 

> Regarding the service loader approach, I wasn't aware of any problems under OSGi, as long as the resources to be loaded and the client using > the loader are contained in the same bundle. Do you have a link or some other information regarding the META-INF issue? I'll try to found out 
> more on that.

It was TCK related - https://hibernate.onjira.com/browse/BVTCK-11
I think the easiest is to stay away from this. The less use of resource loading the better

> I'm starting to wish that we could base HV on CDI which would solve that kind of issues for us ;-)

:-) 

> Another idea might be to demand Java 6 as runtime for HV which would give us the service loader for free from the JDK (without the qualifier 
> functionality, though).

Sure, but as long as we don't have to enforce Java 6, I think we should try to avoid it.
hi Jason,

Just so you know, I merged your 1.9 fixes into the original master, and released it as 1.0.4
Sorry, I'm confused. Are you unhappy it was removed?

I didn't remove it.

But it should be removed.

Kai removed it for a reason. We'll need to talk to him

Mike

On 2/28/2012 6:17 PM, Otavio Salvador wrote:
> In the revision I am using, it does nothing.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Webconverger/iceweasel-webconverger/commit/fd168f7ffea728e584392ca814b901782857546f#commitcomment-1023749
Hi Mike, 

I like your implementation and naming scheme. It is indeed cleaner, and thanks for doing this as I'm guessing it's not something you personally need. The only thing is, as far as I can see you also need to modify  attributesOfContentsOfDirectoryAtPath:error:  In my implementation I was checking the value of attributes.permissions to see if the item was a symlink .. then returning the symlink path .  As far as I can tell without this there is no way to get a directory listing including symlinks.  

Maybe I've missed something?
Ira
If you don't include the file how do you know which lines are non-executable?

Generally people seem to have worked out to now whitelist stuff with executable code and I wouldn't really like the report to show every line as uncovered. I know there is some logic to know about ELOC but I wanted to ask just in case
1. you can use `git rm your.file` to remove a file (if you work on Windows, I recommend TortoiseGIT to manage your git repositories; in this case you can simply right click and delete the file)
2. just add an extension to readme as readme.md (or readme.markdown); in command line, you can use `git mv README README.md`
Может быть, просто я увидел этот косяк в три часа ночи. И разбираться где
возможны проникновения некорректных ID было уже некогда - очень хотелось
спать :). А таким способом просто эта проблема решалась на 100% - где бы не
создавался ID он потом приходил в это место. При этом, в моём случае, я
попадал туда из Librarian.getOSIStoHuman() после поиска. То есть такие ID
пришли не из закладок, и не из старых линков. Такие ID получились в
результатах поиска.

Правильно я понимаю, что ты предлагаешь сделать приведение в конструкторе
OSISLink()?

7 февраля 2012 г. 17:56 пользователь Sergey Ursul <
reply@reply.github.com
> написал:

> По-моему, правильнее будет применить .toUpperCase() не здесь, а там где
> возникла такая ситуация. Мое упущение - при приведении ID к верхнему
> регистру (дабы исключить дубликаты названий модулей, представленных в
> разных регистрах), и не учел еще один источник проникновения неправильных
> ID - из старых сохраненных OSIS линков или из закладок. В обоих случаях
> идет создание экземпляра OSISLink, где и нужно привести ID к верхнему
> регистру - так будет логичнее.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/YakushevVladimir/BibleQuote-for-Android/commit/2751d47f49281f39f9b7538c3addcda503404a25#commitcomment-940543
>
you can have a look @ http://github.com/drewis/tiamat-kernel/tree/wip9 for full hwa, bravo and passion guys seem to have rewritten hwcomposer and a lot for the 8x50, i would surely help if i had time but i have exams going on. If you need help, pm me or i will try this post my exams (end of march).

I suggest you do not use the qdsp6_1550 and instead use the qdsp6 from their tree and forward port the leo board files as necessary, Cotulla will add DSP code to update the stuff in leo's Hexagon (DSP core). Markinus has already fixed the use of BRAVO_DSP in the mainline linux_wince_htc.

Good luck :]
@geemus  - I was really struggling to come up with a good error message. The more I think about your suggestion, the more I like it. I will go ahead and update the text. Thanks!
Let's add a TODO here also - as that's not stored in data ncdf file either
Not sure I'd use a `switch` with fall through for this since there are only two main cases. `if`/`else` might be more clear.
Did you check to make sure there are not other variables being created as globals like this one was?
Yes, I fixed it in the latest commit. Thx!
Don't we need "property === 'id'" ?
hostname probably shouldn't be a kwarg. It's going to be required for any Node.  This should make it a little easier to keep lines under 80 characters.
then match against them explicitly so that illegal states are not silently swallowed
yes, added
You can get back down to 120 if you drop the "this." part of "this.getClass" ;-)
no need for all those braces ;-)
emphasize `*not*` here
Sorry y mistake. I see now the equation for PGV (equation 3, page 204). Please discard my previous comment.
I'd move this under the ChooseTask (nested class).
Fedora ships without this setting in that config file. The docs aren't 100% clear, but I think the default value is actually "yes". As such, I'm changing this to not mention OS at all and basically say "go to qpidd.conf and set auth=no".
Yep, so what do you think is the solution? Populate the StreamDetails during the directory fetch? Is there a reason why this isn't happening right now?
Willen we dit, op productie?
This is not ideal, as you see the usage is a bit more complicated, but I think it is very worth it.  We can make it prettier when everything is merged again.
Rename this method to getNeuron. When calling get() on the input layer one would expect to get a value, not a Neuron. It's not critical, but could be confusing...
And if you want examples using it, look at the Symfony2 TwigBridge.
Ah, right; this is a common cross-version compatibility technique.

If the hacky parenthesis is a bother, the alternative is to use `from __future__ import print_function`, which works in 2.6 and above.
I can remove the comment. I removed `wd.dispose()` because it is also superfluous, but that means that if anyone ever adds code later on immediately following the `script->Run()` inside the same block, the watchdog will run longer than it is supposed to. The comment was meant more towards making sure the reader knows that either the block needs to end right there or `wd.dispose()` should be put back in prior to any following statements.
ensure the file exists and set all exported properties

    REPLServer.prototype.loadContext = function(file) {
      var self = this,
          Module = require('module');
      fs.exists(file, function(exists) {
        if (!exists) return;
        var context = Module._load(file);
        if (context && typeof context === 'object') {
          Object.getOwnPropertyNames(context).forEach(function(key) {
            Object.defineProperty(self.context, key, Object.getOwnPropertyDescriptor(context, key));
          });
        }
      });
    };
Add an `assert(ngroups != -1)` here.
unfortunately not possible without hacks.. so its more flexible like this.
Don't make the default spark.default.parallelism, just set it to 8. The reason is that *each* reduce task fetches this many blocks, so that could be a lot of total fetches.
Ça va nous permettre de le setter à autre chose avant save si on veut.
I'd like to see more checks here, e.g. whether the interface has **only** the create() method, which is public and not static.
Use array casting without type checking.
Right, I also don't see an application use it. So maybe instead of a mapping, just something like

    if ('updateField' in this.initialConfig) {
        window.setTimeout(function() {
            throw("The updateField option is no longer available.");
        }, 0);
    }
i test it already,but i paste these code from vim,maybe miss copy ";" .i
Btw, this could also be replaced by `required = (i >= num_required) ? 0 : 1;`.
а чем обычный setProperty() не устроил?
это волшебная цифра - переделаю
склеить условие в 2 строки
I think these assertions would look better without `()`, as in the inflector, wdyt?
Hi!

No, I'm not. `json_data_type` is the name I'm using for the test table. See line #16 above.

Thanks for your review.
I included it because I wanted to stay consistent with some later migration examples. 

Now that I'm looking at things again section 3.3 does include information about having to update the schema. In my experience newbies just look at the different association types (section 2) and totally disregard the other sections. It almost feels like there should be some sort of checklist somewhere for how to build associations:

* Add association to your model(s)
* Build the required reference(s) in a migration
Please remove this blank comment
Could yo `# :nodoc:` this?
Yes.  A call to `handleEvent` should not happen once the handler's been invalidated; if it does, there's a bug somewhere in the `Bus`.  Fail fast and fail hard.
Actually, this will work just fine.  The only problem comes when other expand hints create new types of expansion that could happen (like multinomial creating an Add).  

I think I can make at least polynomial type expansions work by making multinomial come before mul.  Beyond that I'll leave to more diligent contributors.
In this PR `orbit` and `orbit_transversal` are functions of the generators instead of methods of PermutationGroup,
which is seldom used.
Great idea. Thx.
This should not acutally require change …?
I believe the solution you propose is rather nice in the sense that it essentially does not depend on `Wild`.  This PR introduces no obvious conflicts between your new code and what we have already.  Clearly, having the two approaches coexist is bad idea, however, so the suite to the (supposed) merging of the current solution would be removing the old `Wild` and rewriting everything that uses `Wild` in terms of `unify`. Alternatively, one could try rewriting `Wild` in terms of `unify` directly to avoid large changes scattered across SymPy.  Since I don't know how much code uses `Wild`, I can't really judge which way will be better.
It didn't fall on deaf ears...my thoughts then were, in response to why the following was done  in sympy/matrices/immutable.py:

> @@ -39,24 +39,26 @@ def __new__(cls, *args, **kwargs):
>  
>      @property
>      def shape(self):
> -        return self.args[:2]
> +        return tuple([int(i) for i in self.args[:2]])

I said,

> Because that is what the other matrices return. So until they all derive from Basic, I would like them all to give their properties in the same way. For those that are hashed, creating an object with ints or Integers is going to affect hashing...but if mutable ends up not being hashed then this is a mute point. But...there are tests that fail if mutable's hash is removed because of the custom __eq__, I believe. (If you defined __eq__ then you need a hash, as I recall.) So...for the time being I want to make sure that what is expected from others is the same here.

So if they all behave the same, that's fine. Then the test in `test_args` (if there is one) can be changed. And should the lists be Tuples (hence all the row and column vectors be changed to return Tuples, too)?
mathFix.substr(-2);
whoops. You're assigning an autoreleased value to a retain property's ivar. This value will be destroyed since its retain count is Zero, and assigning it via the iVar does not retain it. You could do this:

``` _osVersionBuild = [osBuildVersion retain]; 
```

And as long as we release in our dealloc, we're ok.
Was testing locally :D
Is this debug code in by accident, or are we wanting to (potentially) see this in longer-term testing?
I don't like that the tags are divided by comma with spaces around it. There is also a comma on the end because there it is not checked if it was the last tag. I would suggest to leave it without commas. Or even not to use a list at all, just spans with the class label.
Isn't having the version number in the path a bit fragile?
What does site mean?
Actually, after sleeping well, I think that ignoring ENOENT is enough.
And event ports?
Could this magic number get documented?
I had combined the beginning of that paragraph into the first para in the Oplog section of Replication Fundamentals, which had already had the bulk of the oplog info. So the expanded Oplog section in Replication Fundamentals begins with:

The :term:`oplog` (operations log) is a special :term:`capped
collection` that keeps a rolling record of all operations that modify
that data stored in your databases. MongoDB applies database operations
on the :term:`primary` and then records the operations on the primary's
oplog. The :term:`secondary` members then replicate this log and apply
the operations to themselves in an asynchronous process. All replica set
members contain a copy of the oplog, allowing them to maintain the
current state of the database.

I'll remove the promise of "within one second."
Please remove this commented code
I am also not convinced that bots should be nuked if they have content. At least preserve their posts.
I know that's how we do this in other places, but I would really prefer it, if you used a loop for the errors rather than hardcoding html here.
don't think this is the right place, this would be `/var/lib/rackspace-monitoring-agent` on most filesystems... the plugins should be under /usr/lib ?
fixed in 67b2375
what's with this return Flase?
Shall do.  But it can't fail, in that a failure wouldn't compile.
He saved the file in UTF8-format, convert it back to ANSI and it will not break already saved files (Adv-dupes).
I think so, see the comment a few lines up for examples when we end up in this case.
please use normal text, rather than ALL CAPS
avoid formatting with html tags -- difficult to maintain, may be overriden in later CSS work
how about
```
var druglist = Ext.getCmp('drugList').getStore();

for (i = 0; i < druglist.getCount(); i++) {
```
Remove unnecessary console.logs, also please watch tabs+spacing (jsbeautifier.org)
Change all ids to private Integer id
Is it intentional to have no self.schedule by default?
then => than

Is there any reason to put the dirname as an exception argument instead of having it in the error message? I find error messages to be easier to read when the info is presented in an English sentence rather than a tuple of various arguments.
So varbinary doesn't work, so we remove the hsql test for all of the group by suite?
A better hack would be to make the first line of the broken test "if isHsql() return;" That will null out the test for that backend.
I looked at the java code and the method throws an exception for unsupported charsets - but good catch, I'll change it to a documented name. :-)
What are you trying to achieve with this?
That's already better, but there is even a cached_property decorator in django.utils.functional that does the job of caching the value.
You can also add this line to the capfile:

    load 'deploy/assets'

See http://guides.rubyonrails.org/asset_pipeline.html#precompiling-assets
I was postponing this kind of modifications when we produce the first new package to reduce differences between old and new.
No problem to anticipate.
@jdevera I suppose that it will be confusing to use `cmd` here. `get_current_sha` is completely unrelated to `cmd`.
That has a nice ring to it.
The hash table is created whenever fn\_extra is NULL. So at most once per query and per function. So except for the first time it is called, a transition function always reuses the existing hash table.

Regarding the question why a hash table: Because we store catalog information for multiple OIDs (type information for each argument/return type, and function information for each function that our C++ code might invoke, including function information that called functions might invoke, i.e., in a transitive sense).
is this "test" target required or optional? can i just omit it? or
leave it empty?

On Fri, Mar 30, 2012 at 11:15 PM, Jack Nagel
<reply@reply.github.com>
wrote:
>> +class Mpfi < Formula
>> +  homepage 'http://perso.ens-lyon.fr/nathalie.revol/software.html'
>> +  url 'https://gforge.inria.fr/frs/download.php/30130/mpfi-1.5.1.tar.gz'
>> +  md5 '2787d2fab9ba7fc5b171758e84892fb5'
>> +
>> +  depends_on 'gmp'
>> +  depends_on 'mpfr'
>> +
>> +  def install
>> +    system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"
>> +    system "make"
>> +    system "make install"
>> +  end
>> +
>> +  def test
>> +    system "make check"
>
> Can't do this, the sources are not available when this method is invoked (by `brew test`).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/11312/files#r626476



-- 
宋浩
SONG Hao
Institute for Interdisciplinary Information Sciences
Room 4-609, FIT Building, Tsinghua University
Beijing, People's Republic of China
Postal Code: 100084
E-mail: baritono.tux@gmail.com
Office Tel: (+8610) 62783817 ext. 1630
gperf is in Homebrew-dupes; can't do a direct dependency like this. `depends_on 'homebrew/dupes/gperf'` works, but I don't know how the other maintainers feel about cross-repo dependencies being used in Homebrew-core.
I think that we should keep the language the same. That is, steer away from using Commits & Files and stick to using the more wiki like Pages & Messages.
I agree, I wanted to ask you about this one. Plus, 0.2 seems strange as a default value.
зачем тут это тире?
Perhaps expand to "Each Pulp repository can be configured with" to get across the idea that it's on a per repo basis.

I feel like we need to mention the CDN here in some capacity, since that's what this is primarily used for. Maybe as simple as:

"This ability can be used to authenticate the Pulp server with the Red Hat CDN to retrieve entitled repositories."
I think that's a definite personal style thing - I'm not sure why it's too many files, but I'll do it.
bean shell! neat.
How about a function for finding the account instead of copy paste code?!!
If you want to have a static library too, create a new variable that contains all sources (HIDAPI_SRC, PSMOVEAPI_SRC, PSMOVEAPI_MOVED_SRC, ...) and use that only. Otherwise we would be duplicating that information in both the add_library() call for the shared library and again in the static library's add_library() call. Not sure if it's necessary to set the VERSION and SOVERSION on a static library?
Oops! Somebody's sleepy; bet you can't guess who!
photo:facebook/feed_others won't work as expected really, since feed_others contains 'posts' and even though a post references a photo it isn't really their photo json object.  The feed synclet will have to mimic the home one and create a feed_others_photos and go fetch the photo object for any post referencing it.  Actually, it could just put them in home_photos since that's just the name for "photos from others" right now generically (can be renamed in a future refactor too where we can create aliases to preserve old references)
This could potentially fail if a player connects twice from 2 different clients at the same instant.

It is low probability though.

Anyway, the code should be something like

    
    SpoutPlayer player = onlinePlayers.getLive().get(playerName);
    //Player is joining the server for the first time
    if (player == null) {
        //Attempt to locate and load player data from file
        player = WorldFiles.loadPlayerData(playerName, session);
        //Player has no saved data, create a new object for them
        if (player == null) {
            player = new SpoutPlayer(playerName, getDefaultWorld().getSpawnPoint(), session, this, viewDistance);
        }
        Player oldPlayer = onlinePlayers.putIfAbsent(playerName, player);
        if (oldPlayer != null) {
            // Player object was created by another thread between the .get() call and the .putIfAbsent() call 
            player = oldPlayer;
            // Does calling new SpoutPlayer() have any side effects or can the object just be discarded and GC-ed?
        }
    }
    // Can call this section always, it does no harm for new Player objects since they have null sessions anyway
    player.resetId();
    if (player.getSession() != null) {
        player.kick("Logged in from another client");
    }
    


The if (kickOld) part isn't technically threadsafe either.  You kind of want getAndSet() on the session, so that only 1 thread will send the kick message.  Maybe .connect() could return the old session or something.
Derp. I had that right. Just committed the wrong thing. Good catch.
Yes, this fixes the sorting. But, the new folder does not get selected. The selection goes to the position where the new folder was before it was re-sorted.
I updated annotation of all functions.
It looks like we only record a spec name only if it has a message. Could we record all spec names instead?

That way, with a few other changes (see later comment), we could possibly just have the TopLevelReporter as our only Jasmine reporter and have our existing reporters use this JSON data that contains our suite hierarchy.
I think this will need to wait until we consolidate the PopupManager stuff (there's no real way for us to hook into that right now). I'll add a note to the open issue around PopupManager.
Let's move the "7" out into a named constant, e.g. MAX_USED_COLORS
Dang. Nice catch. Will do.
Indentation looks screwed up here -- I think some Tab chars snuck in
Could probably remove this check since it's duplicating what the previous test does, and it's sort of ancillary to what this test is checking.
I forgot that does not work on Mac but it does on PC. Let me try something else...
After writing that, it seems like I should probably be checking in enableAgent to make sure that the agent they specify actually exists.

That'll remove the need to check agents.hasOwnProperty(name) in loadAgents. But, we still need to check for load() existing in the agent. It's possible that some agents might not have load/unload.

However, if an agent has 'load', it needs to have 'unload'. So, the unload test still isn't needed.
typo: Specifiy
man the PR has been open a month and you find the ONE TIME there's a conflict ;p

I'll just delete the file and repush--this is the .orig, not the actual (I already did the merge)
Si on met provider en paramètre, il y a 2 solutions:

* Il est obligatoire et donc indiquer le provider dans la config est caduque
* optionnel, par défaut à null. S'il est null, récupérer la valeur donnée dans la config
Ew, this means we have to preindex reports again...
pushed the revert for these lines

On Tue, Jul 10, 2012 at 2:49 PM, Evan Weaver <
reply@reply.github.com
> wrote:

> > @@ -152,10 +152,11 @@ def prepend(*args)
> >      rescue NotStored
> >      end
> >
> > -    alias :flush_all :flush
> > +    alias_method :flush_all, :flush
>
> Prefer to keep the lexically scoped alias unless their is a functional
> reason to change it.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/evan/memcached/pull/98/files#r1136008
>
Please use a path relative to Rails.root -- not a relative path.
nhlib does not have run_tests. I think it is almost useless in this context. What do you think?
This file is not needed!
Do you also need the flag if they want future updates, or are you waiting to add that and want this merged in?
What does 'produce result' mean? I think we should make this more descriptive to help investigation if this was to fail.
Replace `50` with a const, please.
needs one space for padding after opening bracket
This should already be `display: block`, right?
Just a tip: you can probably just set `lastTrackBottom = 0` on line 47 to make this slightly easier to look at.
Will you later just get this via require instead of holding an instance?
Can you just split up this line a bit? It's huge now!
Maybe `instance` should be `popcornInstance`.
I seem to recall Opera being picky about stuff being in the DOM before it will load.  You might want to flip the order of 133 and 134 so it puts it into the DOM before setting src.
Don't close <link>, no.
Why not make this `0` instead of `null` in case it's not passed in properly? At least it'll be a valid number then.
let's move it down below to L2573
This is grabbing the entire contents of the users table from the database and searching it for the current user in the app.  This is actually not a big problem usually, because the users table is so small, but as a general rule this searching should be done in the database.  This is a much, much more efficient operation in the database.  But it's not really a big deal in this case.
Um, the javadoc does not says user agent is "passed in". The list is "passed in", and userAgent is explicitly defined or not, but it does not implies it is a method input parameter or what.
This needs enemy as well I think.
Is this approach more acceptable to put forth, then? : https://gist.github.com/2876384
Don't have to do this but I recently learnt from jsherrill that you can do something like
```haml
 %a.pane_action.separator.copy-tipsy{:data=> { 'url' => copy_system_group_path(@group.id)}}
```
to render data-url
if i'm reading this correctly, 'b' mode is ignored on all platforms but dos/windows under 1.8.7. If that's the case, then it should work.
I believe that nothing should be dependent on that. Returning an empty array should be enough to tell that the was no such a system and perhaps more expected that an exception.
``wrtie`` does not need to return anything
Since HSQL is validating number and type of columns, this code should work fine as is.
Does the ```js-search-hash``` selector match anything? I can't see that class on the search form.
What happens if the title changes dynamically (not during page load), does it still get updated in the title bar?
Let's remove the dead code. It will still live in the git history, no needs to keep it.
Same here - get rid of `, exp` otherwise it won't work with all the supported Python versions.
yes, yes and YES!

Code reuse for the win! :)
I see your point, but after testing this, I realized that changing this from ``_current = WMAP7`` to ``return WMAP7`` still only triggers one warning, so I recommend that approach (see my reasoning below for the warning stuff).  This also makes it easy (internally) to check if a current cosmology has been set - just see if `_current` is None.
What is this? Also, make sure you test this for cases where `user.email == nil`.
No further include is needed for this?
Its the same constant that is used when creating the message queue to set the maximum number of items in the queue, its essentially a fail-safe in case we are getting DoS'd.
If someone else shares my opinion add it, else ignore :smiley:
no, because there is no common interface between a DBAL Connection (which looks more or less like ``\PDO``), a Mongo Connection (which looks like ``\Mongo`` or a PHPCR Session
Then it now will show in extension but I didn't install Growl or GNTP. Should it hidden when user has no backends?
Was there a reason for writing this route as a "match" instead of nesting it inside the resources call [like this](https://github.com/enspiral/loomio/blob/07e092ccd10d0083b5651150832e7afd4fb0050f/config/routes.rb#L39)?
Nevermind, the double-bonus-nesting was throwing me off.
Could you give me an example of where this might happen? It doesn't match my mental model of parsing + expressions.
like it, much cleaner
Yes, it works that way:

```sh
% ./ipython.py -c '%history -f out.log -g import' && head out.log
 9/2: import pylab
 9/3: import numpy
10/1: import numpy
12/1: import numpy
13/1: import pylab
14/1: import pylab
15/1: import numpy
17/11: ';'.join(__IP.complete('''import'''))
17/12: import numpy
17/13: import sys
```
No, this will cause no flashing, because the image still draws immediately. The only delayed effect is making the image resizable, which has no visible effect until hover.

It makes no sense to use the shape of the container for raster images, which *must* start with no zoom.  It is imperative that these images start without any sizing CSS on them.
I agree, but how can I inform passers-by of what this branch is supposed to
be/do, and that they should run 

    %pylab inline
    %config InlineBackend.figure_format='svg'

in their notebook?

Min RK [notifications@github.com] wrote:
> In IPython/zmq/pylab/backend_inline.py:
> 
> > @@ -56,7 +56,7 @@ def _config_changed(self, name, old, new):
> >          inline backend."""
> >      )
> >
> > -    figure_format = CaselessStrEnum(['svg', 'png'], default_value='png', config=True,
> > +    figure_format = CaselessStrEnum(['svg', 'png'], default_value='svg', config=True,
> 
> Let's not change the default value. This PR makes the SVGs sharper, but does
> not actually fix the many problems with SVG figures that prompted us to change
> to PNG from SVG in the first place.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> *

Min RK [notifications@github.com] wrote:
> In IPython/core/pylabtools.py:
> 
> > @@ -88,7 +88,7 @@ def figsize(sizex, sizey):
> >      matplotlib.rcParams['figure.figsize'] = [sizex, sizey]
> >
> >
> > -def print_figure(fig, fmt='png'):
> > +def print_figure(fig, fmt='svg'):
> 
> as below, no reason to change the default value here.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> *

--
Cheers, Bob McElrath

"The individual has always had to struggle to keep from being overwhelmed by
the tribe.  If you try it, you will be lonely often, and sometimes frightened.
But no price is too high to pay for the privilege of owning yourself." 
    -- Friedrich Nietzsche
Not sure whether this is possible while still catching exceptions, but can you just do `return $connection->pdo->prepare($statement)` here?
Yes. You need to include `common.h` bro.
Typo: between
Slightly curious why you used this over `unicode(dude_says)`.
I'd prefer to only have `r'^legal/'` matched here so that it behaves similarly to other apps.
All the part of kitsune: all the tests, sphinx, redis, cron jobs, ...
You still have this one call passing an argument to a non-existent parameter!
> Why send in the whole config?

a couple reasons - to be parallel and consistent with how other objects in Socorro are instantiated.  Most configman enabled classes accept the entire config object as a constructor parameter.  That enables the object to "cherry pick" what it needs.  Secondly, this allows for derivation in the future.  I can easily see creating a subclass of the thread class that would have the opportunity to select additional config parameters.

> is there a memory concern about making these many fat thread?

The thread objects are not created and destroyed frequently.  slots are useful when you've got millions of the same objects  with short lifetimes.  In Socorro, there are only 10 thread objects that get used over and over.
Usually people write the tearDown right after the setUp and then all the test_* methods.
yes, it passes pyflakes for some reason. 

 excellent catch though, because the following lines should have used 'local_config' instead of 'config'
Are you sure there's no namespace conflict with "products"? If so, you're fine here.
you don't count the html or css?
s/there is a reference node set and exists in the database/there is a reference node set and that node actually exists in the database/g
New header name is going to be: `x-ops-api-info`
This will result in an email with a bunch of leading spaces. Use the `textwrap` stadardlib package.