labofsahil/github-event-dataset-2012 huggingface dataset:
start_pos means the current token is the first(begin) token in a sentence. So it is not related to nlabel. Thanks.

On Apr 5, 2012, at 9:35 PM, Hitoshi Harada wrote:

>> +        
>> +        // define the output, the first doclen elements are to store the best label sequence
>> +        // the last two elements are used to calucate the probability.
>> +        result = (ArrayType *) palloc(sizeof(int)*(doclen+2) + ARR_OVERHEAD_NONULLS(1));
>> +        SET_VARSIZE(result, sizeof(int)*(doclen+2) + ARR_OVERHEAD_NONULLS(1));
>> +        result->ndim = 1;
>> +        result->dataoffset = 0;
>> +        result->elemtype = element_type;
>> +        ARR_DIMS(result)[0] = doclen+2; 
>> +        ARR_LBOUND(result)[0] = 1;
>> +        for(start_pos=0;start_pos<doclen;start_pos++){
>> +            memset(curr_top1_array, 0, nlabel*sizeof(int));
>> +            memset(curr_norm_array, 0, nlabel*sizeof(int));
>> +            if (start_pos == 0){// the first token in a sentence, the start feature to be fired.
>> +               for(int label=0; label<nlabel; label++){
>> +                  curr_norm_array[label] = rArray[start_pos*nlabel+label] + mArray[label]; 
> 
> start_pos == 0, so we don't need start_pos*nlabel?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/madlib/madlib/pull/103/files#r653651
>
Adding info about the *help* command could be useful too.
Ahh, that's the overload of Draw() that takes a Vector4 for destinationRectangle, so this will promote texture.Width and texture.Height to float automatically before the divide.  Should be fine then.
I didn't even thing about this. Awesome.
Doing this now return a 404 error; the correct address is http://ab-initio.mit.edu/meep/meep-1.2.tar.gz, after the update.  How do we change this?  I'm new to github -- my first time.  Thanks!
Ok, in this case i can just delete this line. Thanks
PUT and POST are there because they seemed sensible. GET is there because it's what RapidSMS does. :/
In this case it also emits an update event.
I know the double ampersands work in Java regex, and it doesn’t make any sense to me to have consecutive capture groups be anything but consecutive groups. I’m looking for a binary operation here, intersection, where two patters both have to match for it to pass.

I got the reddit VM up and running, so I’ll be testing all my code now. That’ll be good.

On Mar 13, 2012, at 10:02 15PM, Max Goodman wrote:

>> @@ -96,6 +101,10 @@
>>     'border-bottom-right-radius': r'{border-radius}',
>>     'border-bottom-left-radius': r'{border-radius}',
>>     'border-top-left-radius': r'{border-radius}',
>> +    
>> +    # http://www.w3.org/TR/css3-2d-transforms/
>> +    'transform': r'none|({transform-function}({w}{transform-function})*)',
>> +    'transform-origin': r'(top|bottom)|(({percentage}|{length}|left|center|right)({w}({percentage}|{length}|top|center|bottom))?)|((center|(left|right)({w}({percentage}|{length}))?)&&(center|(top|bottom)({w}({percentage}|{length}))?))',
> 
> Sorry, I'm not sure I follow. I can't find `&&` in the Python regex documentation, and running `re.match("(a)&&(b)", "ab") == None`. In case I'm mistaken, including the two groups one after the other will have to match both. Is the `&&` intended to be in the CSS? Also, it looks like you may need some more `{w}`s to separate the parameters. For instance, in testing I found that `"center&&center"` is matched while `center && center` isn't.
> 
> Looking forward to getting this out and seeing what people do with it. :)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/reddit/reddit/pull/368/files#r555862
>
Heh.  Ah heh heh.. um... yeah.  

Oops.
Multiple var statements
Perhaps a comment?  What is this for?
I worked on getting rid of that anonymous function this morning since `eachObserver` is called thousands of times.

If you're interested: <https://github.com/Shopify/batman/compare/optimization;eachObserver>
There are also the .sprite-icon() and .sprite() mixins in brackets_mixins.less that do the same thing as these classes. They're slightly cleaner than this, but it would be nice to see a mixin that simply takes an index position. Oh well, we'll save that for another sprint.
If I snap the sidebar closed, and then I choose "View > Show Sidebar", the sidebar opens at a small width (10px wide) and I can't seem to make it larger. Probably the right thing to do is remember the width that the sidebar started at before the user snapped it shut, and restore it to that width when you set `display: none` on it, so when you show it again it springs back to that width. (If the user doesn't end up snapping it shut, then you'd just throw away its original width.)
inFormField is assigned based on ":input" filter. 

":input" not only filters the actual text fields, but also includes other types of input. So if the focus is on a button in the dialog, inFormField will be true and this if condition will return false. Then we won't be processing "N" key for the corresponding button.
Looks like `measure` is still undefined here. I'm guessing you meant `measure: measures` (and didn't mean to declare `measure` in the `var` statement above--it gets shadowed by the `measure` in the `forEach` loop below).
Escape file.type here
Personal curiousity (not bug), why commas instead of semicolons?
Aren't these fields redundant wieht the content of filter ? filter is supposed to contain all these informations.
It's already strange to have a public field that is not constant... but I though underscores were used to mark non-public fields too...
LanguageField is a class, is it ok if a class name is like Language_field?
Why `''` and not `None`? Is second argument really necessary?
Replace `#` with just an empty line please.
I started using them for variables specific to a section
This is great, but you need to worry about the right side too. This code only works for the left.
No changes here from master
also store length in a variable
This doesn't exist now :(
jsSHA/sha1 needs to be left there, PM crashes otherwise
Should be `backgroundImage` right?
The `ready` function in butter used to be run twice sometimes, but I think that was resolved awhile ago
not a constant
// and four bytes contain "uint32 gmt_unix_time"
This isn't logging this is building an exception message.
Would be nice with a class level comment explaining the purpose of this class, same for the other filters and new classes.
There probably should not be a space between # and define.
let's move this down to the method that uses it
A template function might make sense here to reduce boilerplate. It would encapsulate the table iteration and customize its behavior in terms of a parameter-class-specific processTuple() method called in the inner loop.
I found this first sentence a bit confusing - perhaps this is a typo (or I am misunderstanding it)? Maybe you meant 

```
The error object will have an attribute ``self.parent_nddata`` that is set to
the `~astropy.nddata.nderror.NDData` object that it is contained within. 
```?
As in the docstrings above, it would look nicer to use the `math` sphinx directive to render them as LaTeX (and include the extra underscores to make it render as it should in a paper or something).
To get the links in the docs to come up for these references, you need to give the full path to the object (*unlike* in the docstrings, where it can usually figure it out from context).  So instead of `` `LambdaCDM` `` here, you'll want to do `` `~astropy.cosmology.core.LambdaCDM` `` - note also the tilde, which makes only "LambdaCDM" appear in the generated documentation page, rather than the fully-qualified name.
方法名叫isXXX的话会被认为就该返回true or  false的，这种多态的返回的话是否把方法名改改？ 而且如果返回type的话，false改成null更好。
Its not necessarily more consistent to use a std::string when we already have a const char*, because you are now using both...anyway, it goes back to the "diff for diffs sake" here, there is no difference in the code (std::string(const char*) is no doubt using strlen anyway) but its just more lines of diff.
Seems like the primary name ought to be `MUMPS` rather than `M`.
Why is this necessary? Please add a comment explaining the rationale.
When I use `sys.stdout.write(msg)`, I get following error:
```UnicodeEncodeError: 'ascii' codec can't encode characters in position 129-133: ordinal not in range(128)```
Problem doesn't exist when using `print msg`.
(Error caused by event called `ĄĄĄĘĘ`)

PS: I'm new to Python, so I have to ask - why is `sys.stdout.write` better than `print`?
blank line, docstring, and space after comma.
Users have to install this by hand and we should add it to our setup.py.
File isn't used, forgot to delete it, now deleted in new commit
Missing a closing quote on this line.
I would extract the 120 seconds limit to a variable and use that for both the loop check and the messages.
well, I am not fan of flag based functionality.
I think it will be better if you have specific functions there like
add_locale, add_locale_package, remove_locale and remove_locale_package

Is there strong reason to have it this way? I prefer having separated method, as it is more descriptive.
If you want prevent duplication, then you can add unified preface and have only one initialization method same or even better in C++ create class, that can be constructed and in constructor have default initialization and methods, that acts on initialized object
Yes, really complicated stuff. I wonder why.
you should not extend KernelEvent as KernelEvent has the request and the request type which are totally irrelevant here
it's quite useful. most of the time you need to root collection to work with it (like dumping the matcher) and might not have a reference to it anymore.
Well ... that's awesome and a +1 for reverting `UrlMatcher.php`.

Open question: Is matching routes against a full request object (using more of it than the URL) a valid thing? And if yes, what about your second concern ...
If you call `files()` then all directories are excluded and no exception is thrown. IMHO the question is: what should be done when user calls:

    $finder->in('some/dir')->contains('abc');

Following your suggestion I return `false`.
Yes, looks good.

When updating the tests, could you also add a test that checks that if someone uses the constructor directly (not going through the static method), and passes a proxy class, that this class is not changed?
I you're going to modify this, why not use new style string formatting?  That will make it easier to read both here anywhere it is used. Note that we don't have to worry about Python 2.5 support in SymPy bot, so that's not an issue.
ah ok.. v is for version. got it.
Is this polymorphic? Can the explicit type setting be removed b/c it is redundant to the work on line 107?
Formatting braces should look more like this:

```ruby
<%= link_to 'Remind them to Vote!',
  reminders_path(
    event_id: @event,
    reminder: {
      receiver_id: @event.invitation_for(group),
      receiver_type: @event.invitation_for(group).class.name
    }
  ),
  method: :post,
  class: 'group-reminder' %>
```

Also unless there's a compelling reason otherwise, go with single quotes unless interpolating.
This turned out to be a nice simple fix. Thanks for spotting!
Oh god, sorry, I freetyped it :/
```csharp
return ConfigurationManager.AppSettings[key] == null 
  ?  defValue 
   : _Converter.Get<T>(ConfigurationManager.AppSettings[key]);
```
I strongly prefer using version ranges given that we break every week.

I would prefer not having to update this file on every release either. So how about something among these lines:

>Add the Web UI package to your pubspec.yaml file, selecting a version range that works with your version of the SDK. For example:
>
>      dependencies:
>          web_ui: >=0.2.8 <0.2.9 #works with SDK 15595
>
> Versions change within the range when we release small bug fixes, but it changes outside of the range on any breaking change. See our [changelog](https://github.com/dart-lang/web-ui/blob/master/CHANGELOG.md) to find the version that works best for you.
Is there any reason that you removed the underscore here? There are other calls in this code to _sudo ...

We probably don't want to expose _sudo as a public method, so the prepended underscore (signifying a private method) is probably best...
If these class attributes are part of a type enum, should they have a common prefix like "TYPE_"?
Minor gripe: There are no changes to this file other than end-of-line whitespace. Let's try to avoid including things like this.
I have run the files through the CS fixer ( https://github.com/fabpot/PHP-CS-Fixer ) and everything should be correct now.
If you prefer to be warned about the discontinuity incompatibility, this `if` statement can be removed. See pull request discussion for details. If removed, the corresponding unit test should be deleted as well.
``browser.name.msie`` is not defined by your change. what you are would need to check is ``browser.name is "msie"``
Updated `dialogClass` update in 4bd9617, updated _createButtons to look at options in 463cd67, removed the switch in 43779cc and finally update _makeResizable in 191e1ca.
Can you please change this to use a flag inside each fo the conditionals above? `$.isEmptyObject()` wasn't added until jQuery 1.4, and if we land this change it'll need to go into 1-8-stable as well. Otherwise this will need to wait until 1.10 when we introduce breaking changes.
The actual event is bound on `change` - this mentions `blur` --- Is this on purpose?
this callback is used in the edit mode when users are panning the pulsating animation should be stopped. When the pagination is finished the pulsating effect is restored. In normal mode this callback is no used. We could  delete this callback and add another method just for edit mode
I know very little about clock app but I think this should be a single transaction.
I think `switch` is fine here.
odd indentation here 2 spaces should be used consistently.
@basiclines This one is not changed yet.

It would also be great if you put the `background` wrokaround rules directly below the commented out code. Thanks.
You mean `Application` right :P?
ugh. Yes of course.
same here, wifi-join-button, pls
English Please :)
How often does that happen these days? :)
I think a different method name would make the intention here more clear...maybe `assert_no_unused_interactions!`?
@xumingming That's right, all slots should be available slots + used slots.
Was this test data just incorrect?
Missing a semicolon. Also, you should round this number or limit it to a certain number of decimal places.
Need space "-If" :)
Can this be a def instead? I don't like the var-iness.
Or, RichDate.upperBound(startDate + Days(period-1)) might be right.... I'll let you gents verify.
I'd recommend using, `font-family: @impactFont;`
Extract to a class constant.
Can the talk # be different than the talk id? If not, I would remove talk #.

I would also consider adding room and event to the output, but discuss this with someone like Thanh or Andrew who have used Freeseer to record large conferences.
Not sure if you need this here.
remove link to GOV.UK as inconsistent with rest of page and does not help the user at this point
This documentation should probably be also put into cfg.sample files so test-writers don't need to dig for it.
Well, I only need basic environment
      setup and not what VC10 and its base classes do. So, this is not a
      bug.
      I might be able to inherit from VisualStudio directly... need to
      test that.
      Oleg.
      On 2012-09-27 13:38, Tom Prince wrote:
    
      In master/buildbot/steps/vstudio.py:
      > @@ -357,3 +360,33 @@ class VC10(VC9):
>      default_installdir = 'C:\\Program Files\\Microsoft Visual Studio 10.0'
>  
>  VS2010 = VC10
> +
> +class MsBuild(VC10):
> +    def setupEnvironment(self, cmd):
> +        VisualStudio.setupEnvironment(self, cmd)

      If you are inheriting from VC10, you should be using that here.
      
        —
        Reply to this email directly or view
          it on GitHub.
Maybe include a link to the standard library venv version -> http://docs.python.org/dev/library/venv.html
this is wrong. the same prefix could be used for several folders, with different classes in each. Currently, you will look for all classes matching this prefix in the folder containing the first class found
Something like this:
  logger.error("Error compiling resource ({}): {}", resource.getName(), knowledgeBuilderErro.getMessage());

will give something like this:
  "Error compiling resource (myFile.drl): keyword whan not recognized"

Of course, since a RuntimeException is being thrown, it could also be a good idea not to log anything and just include all error info in the RuntimeException's message.
I didn't notice that it's used in rootset computation, so thought it's not meaningful.
Assume state holds the "next" url for redirection.
https://github.com/gitlabhq/gitlabhq/blob/master/app/roles/repository.rb
consider just using if(this.s.auto_track_page_view){
This should not be here. This module should be interacting on raw tabs only.
Now that's yet another place we implement `readURI` I remember talking with @ZER0 about factoring it out to a separate module, don't know if he end up doing it, but maybe you could if he did not ?
Could you please pass `null` instead of instantiating new object every time ?
I was surprised that it worked =] maybe if you have some time you can explain that one to me.
It does not look like a new npm version of ejs has been published yet, npm version shows the last version was cut on 2012-08-16. Once a new version is published, let's change this sha to a version.
The email is received from an input field, and we fetch it using https://github.com/mozilla/browserid/blob/dev/resources/static/common/js/helpers.js#L12

Could the email come from the server at some point?
If that works, I can do that. I didn't know if `gettext` would get parsed in these files, or only in the templates.
Thanks. So lets get this addressed for browser.js in a new issue.
@vrthra can you break these up into different context blocks?
Cool.  Generally, it is best to have only one way that code is invoked - it makes it easier to reason about, and to discover. :)
If this `'-'` is just the sentinel value to communicate the missing parameter between these two methods, then this should be pulled out as a constant named something like `PARAMETER_NOT_AVAILABLE`. If it means something else, then something named to communicate that other purpose would help a lot.
It seems like this is way too intimate with the implementation, since the searcher could totally use a different mechanism to calculate this.
The old PHPDoc was good
Renamed in tableName, good suggestion.
I'd prefer not to whitelist an entire file, though. I haven't looked into what kind of whitelisting options php-cs-fixer has, though. The fix I put in place is good enough for now since it's really a one-off in the entire framework.
Need to add a return here. Closing the PR for now.
No, that's fine; there's a setter as well.
authorize user to edit dictionary
please check that it is in use
was reverted
MAYBE we can change this so that it automatically does parseXml, instead of doing it in a separate call before calling this function.
-> ENTER_KEY
Could you take this one as a utility, we might use it in the future to trim the password on login that we have issues with now in firefox.
@harjis Tässä alemmassa pastessa on myös ilmeisesti jäänyt settaamatta $tapahtumapaiva muuttuja?
@samiluukkonen Siirretään tämä funkkari tulosta_tyomaarays.inc-failiin.
any reason you're using xrange here? i hadn't seen it but from the python docs it doesn't look like this scenario matches a use case where it helps:

The advantage of xrange() over range() is minimal (since xrange() still has to create the values when asked for them) except when a very large range is used on a memory-starved machine or when all of the range’s elements are never used (such as when the loop is usually terminated with break)
> This is what I had in mind. But if we are going to have a different result because of the flag ?g, it is a no-go imho. :S Maybe we should have a different API then just for group checks.
We could have separate function named as :
 * match
 * matches
 * search
 * match_to_keyword
 * to_keyword
typo should be `Abbrechen` missing `b`
What element is this `<label>` tag for?
This contradicts the look that filters blacklisted properties by name
NGHUTPR
This could be
<pre><code>
string fs_stats_to_stream(const FileSystemStatsPtr & fs_stats) {
</code></pre>
I see. Cool, then looks really to get merged!
This will blanket import everything in that folder? Suhhhweeet!
The old version didn't do this, but it'd be cool to add a check to see if you haven't moved any of the points and return `""` so you don't get marked wrong for clicking the "Next question" button a little too overzealously. Just sayin'
This is indented strangely?

Can you actually pluralize step appropriately? The "(s)" feels a little tacky to me…
I would actually write:

a vector, like 3 î + 5 j.
This will go to < 1.0, is that the intent?
About the BYHOUR 1,3,5,7 and to expect 1 and 5.. when i read the rfc i read the following line as:

     RRULE:FREQ=YEARLY;INTERVAL=2;BYMONTH=1;BYDAY=SU;BYHOUR=8,9;
      BYMINUTE=30

   First, the "INTERVAL=2" would be applied to "FREQ=YEARLY" to arrive
   at "every other year". Then, "BYMONTH=1" would be applied to arrive

Which means that the FREQ=HOURLY; INTERVAL=2 would actually take every other our, no matter what the BYHOUR read? so, that will give you 1,3,5 and 7 which is the actually result.. i will say thats more up to the user..

So :/ I dont really know if that should be fixed as you like?
CS : Use braces both for if and else
Fixed at 88f3641
That seems reasonable. My experience with the sapi stuff is that it wouldnt think its cli even when it was in some cases.
если пациент молчит, то он умер.
вот как я догадаюсь, что если ничего не выведено, то тест не работает?
Hi David,
Yes, we can use a LOOKUP FEW. In addition, since the recordset CTots is
smaller than AttValues and for each record in AttValues we expect the
 number of CTots records that match the JOIN condition to be the number of
class values (at least 2 when binary classifier), It would be better if
change the order in "ATots:= JOIN(CTots, AttValues, ..."  to
"ATots:=JOIN(AttValues, CTots, ..."  and use MANY LOOKUP FEW?

Regards
Victor

On Fri, Jun 1, 2012 at 6:17 PM, David Bayliss <
reply@reply.github.com
> wrote:

> > +                     // Generating feature domain per feature
> (class_number only used when multiple classifiers being produced at once)
> > +                     AttValues       := TABLE(Cnts, AttribValue_Rec,
> class_number, number, f, FEW);
> > +                     AttCnts         := TABLE(AttValues, {class_number,
> number, ocurrence:= COUNT(GROUP)},class_number, number, FEW); // Summarize
> > +                     AttrValue_per_Class_Rec := RECORD
> > +                             Types.t_Discrete c;
> > +                             AttValues.f;
> > +                             AttValues.number;
> > +                             AttValues.class_number;
> > +                             AttValues.support;
> > +                     END;
> > +                     // Generating class x feature domain, initial
> support = 0
> > +                     AttrValue_per_Class_Rec form_cl_attr(CTots le,
> AttValues ri):= TRANSFORM
> > +                             SELF.c:= le.value;
> > +                             SELF:= ri;
> > +                     END;
> > +                     ATots:= JOIN(CTots, AttValues, LEFT.number =
> RIGHT.class_number, form_cl_attr(LEFT, RIGHT));
>
> AttValue was computed using ,FEW - does that mean we can use a ,LOOKUP
> here?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hpcc-systems/ecl-ml/pull/120/files#r918019
>
But I might put it back in anyways so he does not break style from people using it.
Feels a bit wrong to introduce a dependency on Pooka here. I realise the defined? call means it's not a hard dependency but you know what I mean... Is it worth introducing a Sidekiq error that the PookaWorker can throw rather than having Sidekiq know about this Pooka error, that way we'd only have the one way dependency that we already have?
read code convention
https://www.pivotaltracker.com/story/show/26729869
Not sure it matters for this, but with sqlite, it's *much* faster to wrap multiple writes in a transaction.
Cool, I'll make the change
"Excel" is a bit misleading. "Export to CSV" would probably be more appropriate.
I think we should keep the test.

As we're using the REST endpoints primarily for unit test coverage, I just thought that unless specifically testing a back-end API, the REST end point was the mechanism we should use to interact w/ the system where possible (i.e., we're not testing `getTenantUser` here, we're testing `setTenantAdmin`). I think we have the appropriate coverage either way on this particular item which is the important thing, so I'm fine with not switching it if the consensus is that it's unnecessary effort to change.
Why not naming it ``JavaUnit4TestFinder``
I wonder if ``getOrganizeImportStrategy`` would be a better name (just thinking loud :))
Why executing the above only for the first test?
Why 0x30? I'm trying to understand where this has come from?

Why is it not set to one of the standard service types?
This has html in it - putting it in the html gives the html string instead and it looks like there is no way to get jquery localize to treat it differently? Maybe a patch for upstream?
Je gebruikt `maincharacter` hier niet, weghalen dus.
Sure, but I'll do it on Wednesday 'cause I want to check if it has anything to do with the way vim calculates the position in the file.
Есть метод isNotEqual, тогда в expression можно обойтись без отрицания.
this looks broken
`+iterativeScheduler` was never meant to fix `-repeat`, so it's not surprising that it doesn't ;)

>The only way to really do that is to enqueue blocks on the current scheduler for after the calling code completes. That, in turn, requires that we have a +currentScheduler, which is the cause of most of the documented complexity.

That's exactly what `+deferredScheduler` did.

If this is the route we go, we should probably make `+iterativeScheduler` private but I'm not sure we need all this complexity.
If I'm reading this correctly, it'd assume the person you are telling now is on the same domain as the user asking the thing to be told. Is that a good assumption to make in general?
Thanks, nice catch! Code updated.
to be consistent with naming plot_circle_connectivity
good.. I'm thinking of a way to update the sample data incrementally.. (similar to rsync) like that things won't break if it is outdated
Add a comment for why you commented out this module.
Can un-nest this conditional:

```ruby
elsif attributes.include?(method_name)
  attributes[method_name]
else
  instance_variable_set(...)
end
```
can you say what replaces it?
So should I create two other PRs?
即使正常的浏览器也可能在发送一个有效请求的时候不携带<header>Referer</header>信息。
=>
即使正常浏览器发送的合法请求，也可能没有<header>Referer</header>请求头。
I was using a version of 0.7.1, which did not have correct support for generators.
I'm unclear why this series of calls would request the provided URL.
It'd be much better to import companion object functions here instead of extending an abstract base class. (You still need to do it for the Silvertip one, though.)
I was waiting for this :) You win!:)
Roses are red, violets are blue, arrays are mutable, they go into the loo
Esto no debería quedar fuera del using?
Why pass the entid? Just use the entity.
I really don't want to change the contract of the Assertion : it's meant to be read only.
But I want to change the AssertionImpl to be able to initialize this concrete implementation in a certain way (by a setter in this case).
`return self.render_json_object_response(json_dict)` shouldn't it?
IMHO, this if / elsif / else block would be cleaner using a case statement.
Ok, so this should rather be:

```ruby
when String
  { k => v }
when nil
  { k => '' }
```
Oh snap. Good catch Craig.

Sent from my iPhone

On Jun 11, 2012, at 8:19 PM, "Craig" <reply@reply.github.com> wrote:

>> @@ -24,11 +24,12 @@
>> 
>> class Query(object):
>> 
>> -    def __init__(self, columns=[], tables=[], where=[], order=[], limit=0):
>> +    def __init__(self, columns=[], tables=[], where=[], order=[], group=[], limit=0):
> 
> Isnt it bad to default a named parameter to a mutable value?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/119/files#r964424
Right, good point
Ok, sound good.
http://www.dnspython.org/
Bravo on the docstring of boxplot. Thoroughly needed improvement!
It's ok. If/when this PR gets merged, I (or whoever) can open an issue on it and sort it for v1.3.
Probably about as good as it can be
I borrowed this strategy from IPython notebook -- I should add a comment to that effect.  I don't think there's a strategy to the number of attempts, but I will expose it as an rcParam (as IPython does).

Agreed - it needs to display the port that it arrived at.  To be more friendly, it should probably display the entire URL, which in many Terminal emulators will automatically become clickable.
I don't think we will want to de-dent this dictionary literal, the former made it easier to scan the colour names and/or hex values.
try to get the test cases "more verbose"; there are quite a flow of code which are pretty hard to understand/read; try to capsulate them in speaking sub-methods
Thanks, I will correct the line and amend the pull request.
I am not sure if I agree with moving this to the base class. In the case of SLD WMS this was a very specific issue, IIRC it was to work around a bug in MapServer: http://trac.osgeo.org/mapserver/ticket/2257

I can even imagine we can now get rid of this workaround completely.
Tabbing between here and line 700 seems to be off by 1 level.
Oh, damn, that's an awful standard! I see now that your code is necessary as-is. 

Tell me, when you have a KML file that looks like this:

```
<when>
<when>
<when>
<coord>
<coord>
<coord>
```

Are you supposed to just assume that the first `when` matches up with the first `coord` and so on? That seems like a really inefficient and error-prone method of recording data. Especially in a mobile/embedded device where resources are low. I'm a big fan of GPX on my Garmin GPS units because it only ever has to append to the end of the xml to add more track points, much nicer than inserting into the middle.
@bp323 Missing space between `,` and `{`
@adrexia Maybe also a good idea to consider removing/rewriting verbose comments like this one above? The code is already descriptive enough, and having this kind of comment doesn't enhance one's understanding of this snippet. 

You could also rewrite the comment on the level of intention, instead of just saying what the code is doing :)
Hi John,

I think that would be good. I got this from diving through the Apache logs. The sub-panel does  not set which is why I noticed it.

On the plus side, I know have an upgraded test system :)

ATB,
Iain
From: John Mertic <notifications@github.com<mailto:notifications@github.com>>
Reply-To: sugarcrm/sugarcrm_dev <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Wednesday, 24 October 2012 13:09
To: sugarcrm/sugarcrm_dev <sugarcrm_dev@noreply.github.com<mailto:sugarcrm_dev@noreply.github.com>>
Cc: Iain Emsley <Iain.Emsley@ja.net<mailto:Iain.Emsley@ja.net>>
Subject: Re: [sugarcrm_dev] Update include/SubPanel/SubPanelDefinitions.php (#129)


In include/SubPanel/SubPanelDefinitions.php:

> -                     include_once ($beanFiles [ $bean_name ]) ;
> -                     $this->template_instance = new $bean_name ( ) ;
> -                     $this->template_instance->force_load_details = true ;
> -                     $this->table_name = $this->template_instance->table_name ;
> -                     //$this->db_fields=$this->template_instance->column_fields;
> +
> +                     if (! empty ($bean_name)) {
> +
> +                             $this->bean_name = $bean_name ;
> +
> +                             include_once ($beanFiles [ $bean_name ]) ;
> +                             $this->template_instance = new $bean_name ( ) ;
> +                             $this->template_instance->force_load_details = true ;
> +                             $this->table_name = $this->template_instance->table_name ;
> +                             //$this->db_fields=$this->template_instance->column_fields;
> +                     }
>               }
>       }


Just out of curiosity, if $bean_name is not set, I'm assuming the the sub panel won't either. Wondering if we should put something in the error log here to help people debug this fact.

—
Reply to this email directly or view it on GitHub<https://github.com/sugarcrm/sugarcrm_dev/pull/129/files#r1927522>.



Janet is a trading name of The JNT Association, a company limited
by guarantee which is registered in England under No. 2881024 
and whose Registered Office is at Lumen House, Library Avenue,
Harwell Oxford, Didcot, Oxfordshire. OX11 0SG
I think you want to `bundle exec rake` here or directly run `bin/stubs/rake`, since we can't rely on `bin/stubs` necessarily being on the path.
The cache_pool option is gone (I think you can delete the second sentence, actually, if "pool" isn't specified, it's not named, and can't be shared).
Not at all.  Was initially put in OracleDialectExtension, but it was inappropriate there, so moved it here without wrapping in dialect selection clause.
OK, but then the logic is questionable, but I don't want to fix it and if you don't want to fix it either I'll merge it the way it is.
No difference, using the closing hashes is purely optional.  See  http://daringfireball.net/projects/markdown/syntax#header
The assignment to ``$self`` seems to be dead. Are you sure it is necessary, or maybe you have a typo somewhere?
Weird whitespace, should be

```
call_user_func_array(
    array(EventHandler::getInstance(), 'onException'),
    array($exception, $config));
```
A tour is instructive, oh, interesting fact about this plaza, IT'S GOT GHOSTS SON! A description is illustrative, if I were good at painting I should be able to paint what something looks like to your mind with some accuracy after a description.
very descriptive :P
It looks like you're leaving in the whole warmup section. Shouldn't you abbreviate/cut/change it to reflect the benchmark harness?
Ah, yes. Good call.
Put geeky-tech-what-the-heck-does-that-mean? message back.
We should probably remove this.
I don't think it's smart to disable this one only to pass linting, should rather try to fix it.
Not presently, so far as I'm aware.

The SDK internally (via the `Resource` constructor) always assumes that `-1` is a valid way for asking for all elements. (Even if this is an undocumented behavior.)
Anonymous functions should have space before `()`.
If user not authenticated then we display message for login
How does "in" actually works, does it only search for words (separated with "space"), or does it actually search for all substrings.
Because error is: E11000 duplicate key error index: PiplMesh.user.$_types_1_username_1
`datetime` is not needed?
Okay, I just had some problems with python complaining about tabs and spaces which were mixed up together, and I decided to change all spaces to tabs. Wrong way obviously :-)
http://shrani.si/f/2E/IB/3FAdOhG0/githuperror.png
(Please give me link to that pull request so that I can count it.)
The difference between this use of the `?T` macro and `?T_ERR` is in the trace types.  The `?T_ERR` macro includes `error` as a type, which will trigger the error handling code in `riak_kv_wm_mapred` and `riak_kv_pb_socket`, causing each to shutdown the query.  The use of `?T` here will only be useful for debugging during development, as MapReduce pipes do not enable the `kvget` trace type.  The intention here seems to be to raise the error to `riak_pipe_vnode_word` by returning and error instead of `ok`, and let it send the error trace. What say you @jonmeredith?
Hi Thiago,

For GetData\<T\>, could a single call to GL.GetBufferSubData be used, rather than having to loop and copy data?  This would allow you to remove the 'unsafe' keyword, which doesn't match the XNA API.

I know GetBufferSubData is not available in GLES, but I don't think MapBuffer is either, so we'd need a different solution for the mobile platforms either way.

What do you think?
oh sry, my fail ^^
Should we suggest doing `>>> astropy.test()` at this point?
lol, removiong debug and fixing spelling


On 16 March 2012 12:01, Chris Boulton <
reply@reply.github.com
> wrote:

> > @@ -127,6 +128,8 @@ public static function pop($queue)
> >       /**
> >        * Return the size (number of pending jobs) of the specified queue.
> >        *
> > +      * @param $queue name of the queue to be checked for pendign jobs
>
> Small typo for pending there.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/chrisboulton/php-resque/pull/49/files#r566326
>
I thought I had aligned these! I'm going to have to change my font in Netbeans!
Cierto, se me ha pasado :<
Did you mean the two comments on line 73 and 75 ?
Its not - removed.
No space inside brackets, same problem a few lines down.
You can use a 1d array here. 
```
beta = np.arange(1,5)
```
The main reason for "fast" in my interpretation is to stop early in the first stage and not go to convergence with candidates that look too bad. The splitting in the large sample case still spreads the random sampling better over the dataset, IIRC.

IIRC, some of the example datasets had a small "basin of attraction" and many "reasonably good" estimates, so keeping only a few 1st stage results for the second stage might still not include the optimal set, just sets that are close to it. That was my interpretation for increasing n_keep. Also increasing n_keep and reducing the number of c_steps in the 1st stage looked more efficient to me, in terms of amount of calculation to get the right solution with high probability.
I have to get back to remember, but at some point I thought that LTS in R still was more "lucky" in finding the right solution than my implementation. But I didn't see how to get more lucky.
Much of my "heuristic" was just from running several examples not a systematic search.
You could pass all those fields to the constructor!
Yes, that is what I was getting at, the changes to AbstractProxyTestEnvironment (or a different parent test class).
I'm not saying we should hold up this pull, but we need to start doing this soon.
I think so ;-) Removed
For these newly extracted classes, did you replace the existing implementations with these new ones?  I would have expected to see AdhocQueryResponseDescriptionBuilder's error extractor replaced by a reference to this implementation.
Clarify units for FWHM (arcsec, I assume, but it's worth being explicit).  Also, units for r - is that arcsec or pixel?
Ooops, thank you for the fix.
I personally find it an odd choice, but most Python built-in functions raise `TypeError` when the arguments to a function do not match its signature, even when the mismatch is due to an unexpected keyword argument.
For info, the relevant code snippet in the `Add` `_SBInitialize` method that produced the above:
```
# --- Defining the function used to (re)-initialize the contained SBProfile as necessary ---
    # *** Note a function of this name and similar content MUST be defined for all GSObjects! ***
    def _SBInitialize(self):
        
        # This is a workaround for the fact that Python doesn't allow multiple constructors.
        # So check the number and type of the arguments here in the single __init__ method.
        if len(self.objects) == 0:
            # No arguments.  Start with empty list and (one assumes) add objects later with add(obj)
            GSObject.__init__(self, galsim.SBAdd([]))
        else:
            # > 0 arguments.  Convert to a list of SBProfiles
            SBList = [obj.SBProfile for obj in self.objects]
            GSObject.__init__(self, galsim.SBAdd(SBList))
```
I looked at all the other JSON libraries before preparing yet another one; there is no conflict I'm aware of.
if we repeated the above ten times, putting a count in id and name, would that fool it in to thinking it can run in parallel?
have added a warning (and sa-east-1)
should be 'Kibana Module for Puppet'
Hmm.  Philosophical opposition here to simply asserting one outcome in a Cucumber.  Cucumbers are NOT unit tests; they should make sense to the stakeholders w/o further explanation.

Nowhere is it explained what an action is: that's it's a partial result.  If you want to test just one of the outcomes, then previous outcomes should be covered in a "Given".  So, for the 2nd line in the Examples above, I think I would consider doing it this way (no table):

    Given that the subscriber has submitted that he wants A @ $30/mo and B @ $40/mo
    And the system has already added A @ $30/mo auto-renewing
    Then the system adds B @ $40/mo auto-renewing

What's nice about this is that the step-definitions are just as easy to reuse, and you can completely define the test case in terms that an uninformed participant can understand
No way does this belong here - stuffing HTML into this library means you're definitely doing it wrong.
why float_ = float?  And if you say local hoisting, then.. why not unicode_ = unicode, etc?
O mesmo aqui
Why use I18n's locale and not Refinery's here?
Yep, will change this. I did take a look at how `RSpec::Core` does colors and it uses helpers for each color individually, which I thought may have been slight overkill for this.
Since it makes debugging and working with the library vastly saner, yes. The default log level is info, so unless someone specifically asks for debug level logging, it's all nop anyway.
This should be BSD safe, no need to check cpio.
it's more readable, but will changed it since benchmark shows that interpolations is faster
`ernno > 0`? The standard doesn't define errno values to be strictly positive. Better make this `!= 0`.
I know this has been idle forever, and I'm just getting to it, but getdefaultencoding is really not part of compatibility between Python 3 and 2, and I think it's a mistake to put it there.  It's a generic tool for a determination of the default encoding.
What is the right thing here? have a second @author or just continue the first one on another line?
Can these options be split onto multiple lines to make future merges of changes to the list easier?
Oh I'm sure the JIT knows what I mean (fixed)
See the latest version of document library's main.css.  We need to have two separate overlays for img.locked-icon and img.shortcut-icon on two sides of the main icon.  Also, nowhere in your code do you use img.locked-icon.
I forgot to uncomment this but doesn't matter since we are only doing 900 cycles
На самом деле, флаг -i говорит о том, какой компаратор надо использовать. Т.е. после считывания ключей можно создать нужный экземпляр компараторов, а информация о том, был ли установлен флаг -i - не нужна. И у тебя здесь будет одна строчка Collections.sort(array, comparator).
Тест некорректен. Если исключение не выбросится - тест будет завершён успешно.
Suggest a more symmetric naming for these two functions
What's the point of the .to_s? here? If the input is anything but a string.. the code below will break.
情報が古いまま
Optional: Remove the colon?
You should be able to do:

    rr, cc = coords
    rr_inside = np.logical_and(rr >= 0, rr < img.shape[0])
    cc_inside = np.logical_and(cc >= 0, cc < img.shape[1])
    inside = np.logical_and(rr_inside, cc_inside)
    img[rr[inside], cc[inside]] = color

And I think it might be a good idea to move this function as a public helper function in the draw module. This might be useful for other people as well.
pep8 ;)
Maybe we could rename this to "Comparing various penalized linear model streaming solvers".
Would be more explicit to `return False` here.
Always a blank space before the ``:``
if you use check_random_state you should be able to pass only the number 0
Hey again - one more thing, and sorry to be a pest with this :smile: 
I think it'd be better to keep decision logic out of the `__init__ `(API convention again)
IMHO you can initialize all you need to in `__init__`, and then rather move the decision logic over to the start
of the `fit(..)` method. Let me know if I'm being vague with this.
I believe this should be 

```
..., $arg) ? $arg->getType() : Type::$VAR
              ^^^^^^^^^^^
```
Ah, sorry, did not notice that this was an interface :-)
@clofresh added selfdestruct in 9a83c9b.
That will all need to change when the build scripts are ported to Node, maybe the script names could be put in a constant or something to make the transition easier.  I suspect the new build script will keep the same structure but that it far from a given, so maybe there not much gain to be had here afterall.

Just a thought that came when i saw the .py extension
check the original args[0] for IsNumber and IsNumberObject
leurs prises en charge dans le tableau de bord.
Weird space/tabs happening.
heapsize will equal numRows, so might as well do heapsize=numRows outside for loop.
in general it's not going to be required.
It should be an option.
There aren't any places however, that expect to be initialized and expand.

It should probably be an option on constructor.
Looks correct to me.
Looks like it, not sure why that was there, I think it's reminiscent of old code I wrote in the beginning.
BTW, I dont think that format is compatible with isodate. It needs to have a `T` to separate the date from the time.
Would it make sense to get this from `Resource#content_types_provided` instead of hardcoding `text/html`?
Why are you modifying this function?
This is the wrapper for $call method used by interfaces.
Shouldn't we modify $callback ?
Change this to take in $data, and throw in the following:
if (false == true){//data is int
//nothing
} 
if (false == true){//data is string
//nothing
}
else{

}
and throw your current code in else. Planning on using this for something.
So instead of removing it completely, flipping the default to false should work the same? Production generates it's own calcentral.properties which I left out the sakai2Proxy.dummy line: https://github.com/ets-berkeley-edu/calcentral/commit/c952f39018b8b55a1c92599a497904d75e2a4a88. Code changes checks for non-null & boolean before proceeding to load dummjy data. Thinking it might be worthwhile to leave in for the front end developers to not have to dig into the java code.
bottom of page + fallback
`cond` palvelee kun tarvitsee tarkistaa monia ehtoja:

~~~clojure
(cond (straight-flush? hand) 8
      (four-of-a-kind? hand) 7
      (full-house? hand) 6
      (flush? hand) 5
      (straight? hand) 4
      (three-of-a-kind? hand) 3
      (two-pairs? hand) 2
      (pair? hand) 1
      :else 0)
~~~
Since barracuda is an applicance I wouldn't add compatibility with python payload without ensure which the python binary is available by default in the applicance.
Version isn't needed anymore
The version field isn't needed anymore
Hey while we are at it, lets do our todo.
Would `np.asarray(coords)` cover the relevant cases here? It would behave differently for 1D arrays, but I'm not sure when a 1D array would make sense.
I suggest 'gray-level' instead of 'non-multispectral'
I definitely think this needs to be done - but it's possibly not in the scope of these changes. I would recommend adding this as an 'Issue' to be addressed separately.

It would fit nicely by simply moving the HTML content into the 'wpsc-account-purchase-history.php' template file, and the PHP code preparation into the _wpsc_action_purchase_history_section function, then deprecating wpsc_user_purchases().