labofsahil/github-event-dataset-2012 huggingface dataset:
I agree here, it's better to let user know via exception rather than returning unexpected result
The constructor return an instance of the class... so the return type is really Simple here.
The same observation as above.
scope for $config? $this->_config?
I don't always not test :facepunch: They both work in my case, but apparently the 'Sent' tag will only be there if one uses an e-mail client like Thunderbird. So I changed it to your suggestions and it should now fetch all sent mail. The IMO more important part is actually the other change in the file which as far as I can tell makes the download a lot faster because it only sends one request to the server.
Really not convinced that count() is a common enough requirement for ZfcBase's AbstractDbMapper.  My general rule is that "90% of all mappers that extend this class will use this method" and I don't think count() meets that.
I had a problem to use the mysql and not sqlite, so I've run the tests using travis xml, btw, I've updated the code and removed the require.
Bytheway I've now tested with the latest xml
We should not have framework specific code here.
Simply add `System.set_env("EXUNIT_CONFIG", "none")` at the top of `lib/mix/test/test_helper.exs`.
yeah, in-cache performance seems more correct

or just "good cache performance" is also fine
Got rid of all ## and ###'s
and add ``= null`` for the TokenGenerator to allow passing ``null``
Couldn't you do that better with a comment?

                             <artifactId>as7-quickstart-wicket-ear-war</artifactId>
                             <!-- NOTE: The following bundleFileName value matches the artifactId value above -->
                             <bundleFileName>as7-quickstart-wicket-ear.war</bundleFileName>


On 03/23/2012 04:26 AM, Ondrej Zizka wrote:
>> +
>> +<build>
>> +<plugins>
>> +
>> +<!-- EAR archive creation. -->
>> +<plugin>
>> +<groupId>org.apache.maven.plugins</groupId>
>> +<artifactId>maven-ear-plugin</artifactId>
>> +<version>2.6</version>
>> +<configuration>
>> +<!-- Change this to 5 if you want to use with Java EE 5 container -->
>> +<version>6</version>
>> +<modules>
>> +<webModule>
>> +<groupId>${project.groupId}</groupId>
>> +<artifactId>as7-quickstart-wicket-ear-war</artifactId>
> Well, it's not random - it keeps the two lines aligned:
>
>                              <artifactId>as7-quickstart-wicket-ear-war</artifactId>
>                              <bundleFileName>as7-quickstart-wicket-ear.war</bundleFileName>
>
> To make it obvious that the value is the same. Personally, I prefer that. U no? :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/quickstart/pull/151/files#r595080
this test is failing for me. I find that this assert gives me no information about which link isn't present. i would rather see a long test than have no idea why it failed.
pmac likes sets fine, and given that I was unsure enough to ask, this is fine once you remove the list stuff.
What about making this a `play` or `playing` event so that we know the video is playing?
Just out of curiosity, why does `formatTracesForRestkin` use async.map and this one uses `forEach`?
It probably makes sense to do that outside this pull request since the scope will then be larger than just cleanup. Should this get merged and then we can re-evaluate the report handling code separately?
1) CSL doesn't deal properly with assignees (because we don't have a proper
name field for them) so it doesn't matter much. I'd leave them as lastname,
firstname
2) no - the fact that the inventor owns the patent is important and not
redundant info.


On Tue, Dec 4, 2012 at 8:39 PM, Aurimas Vinckevicius <
notifications@github.com> wrote:

> In Google Patents.js:
>
> > +				"itemType": "patent",
> > +				"creators": [
> > +					{
> > +						"firstName": "Michel",
> > +						"lastName": "Billon",
> > +						"creatorType": "inventor"
> > +					}
> > +				],
> > +				"notes": [],
> > +				"tags": [],
> > +				"seeAlso": [],
> > +				"attachments": [],
> > +				"patentNumber": "EP1808414A1",
> > +				"date": "Jul 18, 2007",
> > +				"filingDate": "Jan 16, 2006",
> > +				"assignee": "Billon, Michel",
>
> Two questions here:
> 1) For assignee, do we want personal names to be displayed as firs last or
> last, first? last, first is clearer that it is a person, but idk how this
> works with CSL
> 2) If the inventor is listed as an assignee, do we filter it out?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zotero/translators/pull/503/files#r2316307>.
>
>



-- 
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
MacRuby is not used anymore, so this is not needed... right?
Thanks for the help. I completely agree that these more personalized hints make more sense. I'll get started on it.
I think this would be better and more in line with what's used elsewhere in Modernizr:

`if (prefix + 'Hidden' in document) {`
I see now, it's the login page. makes sense.
I think this is something that should be handled in the operation maybe?
What about adding some phpDoc?
Domain has interfaces and devices.
Unfortunatelly, the NDL version we use has an error in the schemas.

This error was reported and the answer we got was that it was going to
be fixed in a newer version.

This is the code to use when this newer version is adopted.


Until this version is not adopted, domains cannot include interfaces on
exported topologies.



El dj 22 de 03 de 2012 a les 04:36 -0700, en/na Jordi Puig va escriure:
> > +		// add interfaces
> > +		// List<net.i2cat.mantychore.network.model.topology.ConnectionPoint> interfaces = new
> > +		// ArrayList<net.i2cat.mantychore.network.model.topology.ConnectionPoint>();
> > +		// for (InterfaceId interfaceId : networkDomainModel.getInterfaces()) {
> > +		// int posInterf = getInterfaceFromReference(interfaceId.getResource(), existingInterfaces);
> > +		//
> > +		// if (posInterf == -1)
> > +		// throw new ResourceException("Error to fill network model. Interface doesn't exist: " + interfaceId.getResource());
> > +		//
> > +		// net.i2cat.mantychore.network.model.topology.Interface sourceInterf = (net.i2cat.mantychore.network.model.topology.Interface)
> > +		// existingInterfaces
> > +		// .get(posInterf);
> > +		// interfaces.add(sourceInterf);
> > +		// }
> > +		// networkDomain.setHasInterface(interfaces);
> > +
> 
> Is it necessary?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/dana-i2cat/opennaas/pull/68/files#r590276
>
I was actually thinking if it is worth checking if it's IM as putenv is harmless.
Adding a logic with a check just adds more task to do... But it's cleaner.

I can't think of another way to check if it's IM or not except from running a 'system' call to 'convert -version'
`mysql_*` instead of `mysqli_*` in the MySQLi driver ? Seriously ?
this path needs to be "./bin/vcard-to-json.js". see my commit.
I tested it myself and checked the Jenkins history for this test, I couldn't see it failing regularly. I'd prefer to keep it an ID.
PEP8: Separate these imports with a new line.
we tend not to use `get_` in our python tests. I think this owuld be better named `category_text` or `_label` to be clear that it's returning just the name of the category .
>That was intentional. I tend to do that so you're less likely to get caught out by forgetting to add them when you edit things later on. 

Well I've learned something so I call that a win :-)
It normalises the value: anything value Ruby considers true becomes a 1, and anything else becomes a 0.
Unfortunately the reject! there was causing a nil to be returned to the wannahelp_controller view, instead of the intended array. See if you can get it to work, I couldn't
This should probably use the oae-logger module
We can probably just rename this to oae-config, which is cleaner and shorter
Oh dear ...
Also, can ``annot`` be ``null``? (I'm asking only because I see you are doing a lot of checks for nullity)
IMO, ``additionalAction`` should be a function, and not a by-name parameter. Looking at the test, ``additionalAction`` is something that it's 1) always executed, and 2) most likely triggers some side-effect. And both conditions goes against how I would use a by-name parameter.
Just wondering, would the following work?

```scala
for {
  resource <- candidates
  noValue = pm.worked(1)
  ...
```
Ch. 6 iv
ups, of course you are right. fixed
What if $timezone is '' ?
This is needed for the function to work properly.
You can define the timezone you want in the config file, but the script must be in UTC. It will compare the imezone you specify with the one of date_default_timezone_get(), and it must be set to UTC in order to give a good time() based on $timezone parameter.
В этом месте нет гарантии что символ не заменится на точно такойже, в этом случае мутация не произойдет
Вместо [Cell new] лучше использовать [[Cell alloc] init]. Пруф: http://stackoverflow.com/questions/719877/use-of-alloc-init-instead-of-new-objective-c
Да, изменил, а чем плохо сравнение с использованием !=?
Вы учитесь как раз для того, чтобы это всё понять, так что всё хорошо. =)
Есть ещё isNotEqualTo, тогда можно без отрицания выражения использовать.
можно использовать addObject
This fixed the issue for me
Out of curiosity, why not use width?
Please don't change spacing for the FR locale. Thanks!
Sounds good. I filed a bug about that last sprint #57.
I know you are done at this point, but we should probably get some tests for at least the model.  :)
You should catch the \ImagickException here
I see this being used here: https://github.com/codegram/hyperclient/pull/3/files#L4R78
the `resource` property could be used for e.g. table names and the engine adapter should also know which property to treat as the key
any reason to have two variables filter and statusFilter?
What is a user supposed to do then to create a signal that behaves properly?
That's why I suggested changing `+createSignal:` to accept two blocks: one to call immediately that returns the disposable, and another one to defer to a later time that sends out the initial values.
That way we can hide all the deferring mechanics in private methods, and users don't even need to know they're supposed to defer sending values in the `didSubscribe` block.
Unnecessary blank line. ( but  no problem for reading this web site. )
You may want handle the condition when this fails?
Well line 43 defines v, it is legal to have more than one cert, thus the for loop.  The contents of the public key are important for my use, but agree that it's not important for someone learning protobuf code.

The v.Subject is substantially longer and more ugly than the public key:
Key:
2012/04/05 23:02:00 Server: public key &{b0e5d7f400cd2ddb84c737b14990322cd5db3a1a14d1d8d767182c55134ac19d899ad5bd25fa1338b4edea37b60b867a8a44ea94ffd65efdc48cbc7c499fc71223c8fcaefbe66b4b4126d0080412805ab69b49ed2c496f373f2b7f1f27166bdb68adbd94f9f075a67969e148cd04ab6c2d581f2b975541ed58a5384bcbe2b3c5 10001}


2012/04/05 23:02:00 Server: cient cert chain pkix.Name{Country:[]string{"DE"}, Organization:[]string{"Random Company"}, OrganizationalUnit:[]string{"IT"}, Locality:[]string{"Earth"}, Province:[]string{"NRW"}, StreetAddress:[]string(nil), PostalCode:[]string(nil), SerialNumber:"", CommonName:"www.random.com", Names:[]pkix.AttributeTypeAndValue{pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 6}, Value:"DE"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 8}, Value:"NRW"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 7}, Value:"Earth"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 10}, Value:"Random Company"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 11}, Value:"IT"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{2, 5, 4, 3}, Value:"www.random.com"}, pkix.AttributeTypeAndValue{Type:asn1.ObjectIdentifier{1, 2, 840, 113549, 1, 9, 1}, Value:"dummy@example.com"}}}

It would be nice to print out the "dummy@example.com", but I couldn't figure out how to get to it.
',)' -> ')'
I'll make sure to add that to the docstring--that's an important thing to convey. Thanks!
Style: "disk based" should be "disk-based"
I've commented on the "factory" about this.
It could be I guess. I figured it would be the same result either way.  But yeah, that's probably best.
or just drop this in
```javascript
if(typeof fs.exists != 'function') {
    fs.exists = path.exists;
}
if(typeof fs.existsSync != 'function') {
    fs.existsSync = path.existsSync;
}
```
instead of using globals
AFAIK, we are using American spelling, so it should be "colors".
比特
字节

一点也
根本
ITCH in-wire protocol is big endian. I know `ByteBuffer` is big-endian by default but I'd prefer to see an explicit `order(BIG_ENDIAN)` call to make it obvious to the poor soul who needs to maintain this code.
replace the if with hash object:

var OS_MAP = {
  'iPhone OS': 'iOS',
  'Mac OS X': 'Mac'
};

os = OS_MAP[osMatch[1]] || osMatch[1];
Fix this *before* the pull request is merged.
OK.  Sorry about that.  I see that I actually had that marked as a TODO.
is on its way out anyway
Лучше сделай тупо слотами. Почему бы и нет?
Si esto vino mal de Trello sacalo de ahí también si querés, así la próxima ya no ocurre.
The following (probably not very smart) attempt to work with character arrays is actually slower than what I was doing before:


    function alt_split_delimited_line{T <: String}(line::T,
                                               dlm::Char,
                                               quotation_character::Char)
      inside_quotes = false
      items = Array(String, 0)
      current_item = Array(Uint8, strlen(line))
      i = 0
      for chr in line
        i += 1
        if inside_quotes
          if chr != quotation_character
            current_item[i] = chr
          else
            inside_quotes = false
            i -= 1
          end
        else
          if chr == quotation_character
            inside_quotes = true
            i -= 1
          else
            if chr == dlm
              push(items, bytestring(current_item[1:(i - 1)]))
              i = 0
              current_item = Array(Uint8, strlen(line))
            else
              current_item[i] = chr
            end
          end
        end
      end
      push(items, bytestring(current_item[1:i]))
      return items
    end
To be honest, I can't tell, and thats a serious issue. You've modified so many lines in generic.xaml because of changing indenting that I can't actually tell what has or hasn't changed.
I imagine you probably want to just return the first match (since there will probably only be one anyway)

```
(first (filter #(= (.id %) id) rows))
```
Instead of using .count, I prefer:

```
if books.blank?
```

or to have the stuff at the top

```
if books.any?
```
A cleaner design than hundreds of `if verbose: ...` would be to in the start define a function for printing if `verbose` is true, and a `pass` function if `verbose` is false:

```
if verbose:
    def vprint(S):
        print(S)
else:
    def vprint(S):
        pass
```

And then use this function throughout.
You are right, I'll fix it as fast as possible. This toast was only for testing ... and ... it appears to be good to have it (it takes some seconds, to save cache back to db).
use the get_bool method it will convert the string to a bool value.
I need anyway to test the JDBCString... stuff with more databases than mysql/oracle I do ATM.
DB2, MSSQL, PGSQL and Informix are on my agenda. Will be done during the next months...
Hmmm, snapshot??
these should be constants instead
Again, I have readability issues with this. ```xborderpad``` is better, ```x_border_pad``` is best (but most inconsistent with the rest of the code).
I've made a change. I'm not sure it's more readable, but it is now on two lines instead of three.
This is not good when ```num_lines == 0``` as we end up with a single point in the hatch pattern. 7242fd67921fff5c7651692e097cfdb7dde174ab, in my own branch from yours, fixes this and adds a fix to the anti-aliasing effect on horizontal and vertical hatches.
seems like there still need to wait for a little time then I can click on the picture.
I was actually referring to @nateabele in the PR 16 (https://github.com/php-fig/fig-standards/pull/16#issuecomment-4431046).
The other reason of his statement is that PSR should be  based on what is already used nowdays (just like PSR-0 was), see https://github.com/php-fig/fig-standards/pull/16#issuecomment-4426445.
this define has different indention than the one below
This line breaks the compatibility with Symfony 2.0.x
Can you please tag or branch the previous commit, so it can be put in the deps file? Thanks!
Here you can set the password for all php files!
auhauahauh :D
Is it possible to add a ``@param`` annotation for parameter ``$valueTests`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
Global: use type annotations in function and method signatures
Intuitively, I find this confusing compared to "new". (If anything, I'd expect "create" to be a new package, and "new" to be a new version.) How about "release" or "publish" or maybe even "version" (as a verb)?
your comment doesn't seem to match the code
Again, break these up.
@janmoesen Thanks for the insight :D

I just found the conditional on StackOverflow.
Looks like there's some extra indentation here?
Actually the lowest ascii code page (where '\n' and friends live) is compatible with utf8 (that is, utf8-encoded newline is the same single-byte string as an ascii-encoded newline). Not sure if this means anything here (since I am constantly confused about how encoding/decoding to/from unicode/whatever works in python :D).
No spaces around `=` because it is function call argument.
Is not "Persona" the now official public name for the project? BrowserID being just technology? So probably is should be "Login with Persona"?
I would animate also appending to the end? No?
Can you order function parameters in the same order as they are define in the document?
oh and what' that used for? we do have any info on that?
Aren't these the same values as the default constructor uses?
@stof reversion made :) thank you
Technically, it doesn't need the unary `+`, but I like to be explicit about type casting.
Try <input type='hidden' name='delete' value='delete' />
Debería colgar de: */{restaurantID}/* o de */{restaurantID}/caja/* pues sino no sabes a que restaurante hace referencia.
It is False, Sinister Strike on 4.3.4 Cant Apply Combo point ..
How this works is you call the CMStore "registerForPushNotifications", which sets this variable and then fires off the call to UIApplication to register with Apple. When this method is called with the token, service has already been set by the CMStore.
Сорри, что перехожу на русский, но такой проверки недостаточно. Нужно добавить еще передачу `providerKey` в `AuthenticationListener` и сравнивать его тут.

Когда используется несколько фаерволов и в обеих есть опенайди они будут пересекаться (всегда будет первый `AuthenticationListener` отрабатывать).
> Instead of this weird shift-based loop, if for some mysterious reason, you really want to guarantee order of evaluation

I tried to describe why this order is important, if you think guaranteed order needs to be justified in the comments to
appear less mysterious I'm open for suggestions. I was trying to avoid embedding novels in comments & can't think
of any concise explanation.

> you could have two nested loops, resetting the array each time you start. Not sure which is fastest (your implementation resizes the array repeatedly
> while this alternative allocates/deallocates at least once too often), but at least, the alternative is less shocking.

I'm afraid I don't agree this is common `dequeue` from `queue` until it's exhausted pattern, which to me reads way more intuitively than nested resetting arrays. I can't care about performance without bottlenecks & numbers. I'd like to wait @mossop on this, if he also finds shocking than it's just me and I do whatever is less shocking.
Space before the end bracket of the function body.
Remove if its not being used.
The order in which elements were entered in the OrderedDict determines where they fall in the table. We split left/right automatically in the middle of the dict (with white space on the right side if the number of elements is odd). You can tweak the order by rearranging the summary_model() function, or by sorting the resulting OrderedDict. In the current setup, arima-specific fields are just appended at the end of the basic information fields shared by other models. 

Alternatively, you could sacrifice convenience for control by creating a Numpy array or DataFrame with 4 columns (labels in cols 1 and 3) and then use the Summary().add_df or add_array.
Oh, I didn't notice that.  No, it's there.  I should just move over the new stuff I added for pickling to that file and get rid of the Position stuff here.  (In c++, they are in the same file, so I thought it was just an oversight that the Position docs didn't make it to this file.)
Sure, this probably deserved a bit more explanation here.
去重可以用underscore的方法来做嘛。就不用自己写啦。
yeah, according to http://www.w3.org/TR/REC-DOM-Level-1/level-one-core.html#ID-745549614 , getAttribute('DOESNOTEXIST') should return an empty string...

I guess Javascript, which returns `null` is non-compliant to the W3C DOM interface?
like everything are put in one page , and you can edit it.
Don't do both if first and last names are set
Thought I better ask: will not running definitely return non-zero, or does zero just mean that status successfully found what the status was?
Renaming to StubAppServer (agree it's not like our other "basic" impls, but it's not a mock).
this can be replaced with a simple @material.description.blank? 
because nil.blank? == true
I think we should change the type of all `OS.class_` declarations to `objc.Class`. You should never access any of the members of these Objective-C structs directly. Always use functions to extract the necessary information. This will make it future compatible.
Yes please just set the variable and grab the last item that way. I know it's silly, but I don't like the idea of adding an undocumented method just for this. The rest of the PR is perfect :)
I haven't gotten an opportunity to test, but doesn't this change conflict with Graphite's vhost configuration?

https://github.com/heavywater/chef-graphite/blob/master/templates/default/graphite-vhost.conf.erb#L14-15
ｈｍｈｍ．りょうかいしました！
Hannes Magnusson <notifications@github.com> wrote:

> > @@ -98,6 +98,15 @@ void
> php_mongo_connection_force_primary(mongo_cursor *cursor, mongo_link
> *link T
> >    zval_add_ref(&db->link);
> >  
> >    PHP_MONGO_GET_LINK(zlink);
> > +
> > +	if (link->servers->default_w != -1) {
> > +		zend_update_property_long(mongo_ce_DB, getThis(), "w",
> strlen("w"), link->servers->default_w TSRMLS_CC);
> > +	} else if (link->servers->default_wstring != NULL) {
> > +		zend_update_property_string(mongo_ce_DB, getThis(), "w",
> strlen("w"), link->servers->default_wstring TSRMLS_CC);
> 
> Couldn't see any other read ops for it :)

I'm pretty sure that there is a zend_read_property for it somewhere
Did you test if this works? I'm pretty sure you still have to wrap it in a jQuery object.
Do we really need to make a breaking change for this?
Clearer with a structure like:

```
EXPECTED_FIELDS = [...]
for f in EXPECTED_FIELDS:
    saved_search[f]
```
closes the div
Why do you create an Uint8Array in https://github.com/mozilla/shumway/pull/8/files#L3R107? Just pass arrayBuffer directly.
I think the better approach would be rewrite the alternativeLayout altogether instead of the keys one by one here.
You're right - it's dead code that's not needed. I'll remove it.
So by the end of this loop you will have all needed limbs drawn and correctly colored.
Can we get rid of the auto association and require the stepRegistry to be set?  With this here, there are two issues:
* The setting of the stepRegistry is an unexpected side effect of setting the jobRegistry (not even identified in the javadoc)
* If I set the jobRegistry then an exception is thrown before the stepRegistry is set, the checks in the afterPropertiesSet pass even though I have an object that isn't what I really wanted.
Got a little overzealous wrapping the lines. You're right isset on scheme is not needed.
We seem to have lost the wiki information here.
如果我merge后，这个地方会有冲突吗？
Кодировка немного поехала.
Исправил на стандартный
Казалось бы, что исключение в конструкторе может быть только тогда, когда BufferedReader протух, т.е. закрывать его не надо.
õÄÁÌÉÌ


2012/11/15 Dmitriy Komanov <notifications@github.com>

> In src/ru/fizteh/fivt/students/yushkevichAnton/WordCounter.java:
>
> > @@ -0,0 +1,192 @@
> > +package ru.fizteh.fivt.students.yushkevichAnton;
>
> üÔÏÔ ÆÁÊÌ ÕÄÁÌÉ ÉÚ ËÏÍÍÉÔÁ.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/dkomanov/fizteh-java-task/pull/171/files#r2132295>.
>
>
Это copy-paste из wordCounter. Надо выносить в отдельный класс.
I'm confused about this "tokens"

Having some issues with the example you provided:
I tried to run this code
```
var path = "a/b/c/", // line 1
    pathSegs = path.split("/"), // line 2
    len = pathSegs.length, // line 3
    root = pathSegs[0], // line 4
    i; // line 5 
```
I can break on line 3 when pathSegs is assigned and prior to len without any problem. Could it be that it was different in older versions of firebug ?

Also if you change the code to be like this
```
var path = "a/b/c/", // line 1
    pathSegs = path.split("/"), // line 2
    len = pathSegs.lengthErrorLine(), // line 3 with an error
    root = pathSegs[0], // line 4
    i; // line 5 
```
I can put a breakpoint on line1, line2 and even line3 without a problem.
I honestly didn't try a `<tabbox>`, but my guess was that it would be harder to make the layout in our current mockups if we did. In the mockups the command buttons are on the same bar as the tabs.
Cool, but consider doing separate PR for commits that do not touch the same feature; unless it’s whitespace or something
Adi, this method will always return the Statistics page object so it can only click a link that goes to the Statistics page.

So this does exactly the same thing as `click_statistics()` in details.py
I agree with @klrmn - ideally we want this housed outside of the testcase. This should be function in a page object.
if we are probably picking the same 'first free app' every time, and actually saving the changes when we add a screenshot, is it eventually going to reach some limit of number of screenshots you can add to a given app?
Understood. I saw an exception to that rule at https://github.com/mozilla/marketplace-tests/blob/master/pages/page.py#L31, which led me to this solution. I am hearing that this is not to be done, so I'll change it.
@peter-budo: No, I have no idea what IntelliJ did. I ended up running Eclipse instead, but I guess that change got caught!
recompile
は構文のチェックとかではなく、すなおに再コンパイル、とかでも良い気がしました。
Github needs a facebook stylee like button
Could you please add a new test that checks the exception raise when the `dtype` of the input is not floating point.
should be "delivery-whitelist" (notice the dash instead of the underscode)
Wrong type?
OK, that sounds good, let's implement it like that.


larryweya wrote:
>>
>> -            // create an info window that opens this marker is clicked
>> -            var infowindow = new google.maps.InfoWindow();
>> -            google.maps.event.addListener(marker, 'click', function(){
>> -                if (!!open_window) {
>> -                    open_window.close();
>> -                }
>> -                // todo: remove hard coded url
> I don't think so as it wount be parsed by python, we need to pass the url from the view and we can only use reverse with the survey's instance so I suggest we add a url to each gps point.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/modilabs/formhub/pull/219/files#r538155

-- 
Peter Lubell-Doughtie
http://www.helioid.com/
Not sure what you mean re. loop.
But this high water mark (as set by setRow), I think is only of any use really by clearRows, which could prob. just scan maxRows and so avoid maintaining numRows when using setRow/getClear (and addRow in CHashTableRowTable)
That was my point. This code is recurring only once, which is odd.

If the iterator can choose recursion, than this code should not exist. Otherwise, it should recurse all the way through.
We should stop doing this. If a querystring needs to reference an array it should do `product=One&product=Two`. Do our views not support this?
Agreed.  Definitely not ideal, since it requires a recompile against the proper headers.  QtCreator has a nice way to switch between build configurations easily, and that pulls in the proper headers and libraries.
Can you reverse the clauses here so we don't have an "if not?"
It would be DRYer with a method like `wait_or_loop(wait)` that accept a block, no?
could you please line break after or ? We're stick to 80 char limit per line (unless it's url).
so actually, this is a little more intense that i think we want.

if the original search matchs an email regex, and no one is found, we are going to try and webfinger.  At most, that is going to return a single person, after some period of time.

"refresh search" to me, should simply Person.find_by_diaspora_handle(search_query), and render a single people person element, and return that.  Otherwise, it should fail and you can remove the loader.

Also, I am wondering why the formats stuff too?
BTW, I saw this implementation [here](https://github.com/daevaorn/django-composition/issues/1)
would you please run the test in release and beta and post the results. I see that the navigateto method was removed but it's being called in your test class. Is there another navto method it's calling?
100 is impossible.  Most modules cannot do 100.  Give this setting a run against all existing modules and you'll see why.
A figsize closer to (8, 8) would work better here.
retreived -> retrieved? (,-nusnus ;) )
Do you think that non-square texture atlases allow for potentially better packing?  If so and the increased code complexity is justified, do it.  Otherwise, don't.  Let us know either way.
Document `DeveloperError` exceptions.
Since context is optional, it should be the second parameter.  Plus, since the function is called fromID, it makes sense for ID to be the first parameter.
It's probably a simple change, the question is should I make it system-wide for this pull request or not?  I have no problem changing just Spherical or any code I touched, but is it a good idea to suck in unrelated changes to this pull request?  I'm just arguing the point, I really don't care and can do the find/replace for this request.
不应该叫音效，之前那个也不应该直接叫音量。应该分为：音效音量、音乐音量这样。
because no work to be done here. Will fix that
We should check the mime type first before trying to process the file with BitmapFactory
Period should be outside the parens here.
Mobile Web does have accessors.
Before the findSymbols() function is called, "Titanium" is replaced with "Ti" just like in the old code:

    var contents = fs.readFileSync(file).toString().replace(/Titanium\./g,'Ti.')
Aha. I get it. This is because of the MODULE string being overridden by a VARIABLE string. Fine. Working around bad namespaced code. Also, the reason for abstracting away the user path back into ~ is for the module to be semiportable from one user to another.
Fixed...must have been a remnant from being in a var declaration.
Since this is an example class, it would benefit a lot from a one liner describing it what it does
Maybe add an else block, like:

```python
else:
        self.log.debug("ignoring unknown config parameter %s (%s)" % (key, self[key]))
```
is this working? If so, remove this, if not, implement.
does this mean users can *never* set jboss.server.home.url  explicitly ? (since it gets forcefully nulled out?)
Probably not a big deal, but just wondering.
This paragraph still applies
Please, could you sort the elements in the var by its alphabetical order?
Boa. auehuahe

Viajem minha; =D
No good reason. I just removed it.
yes .. it just needs something to store. it could also be the ORM or some other ODM or something else entirely.
but out of the box we only really support PHPCR ODM.

i do also agree that we need to really explain this topic. so imho we should have a chapter on "persistence", where we clarify that other solutions are possible, but also highlight why PHPCR makes sense.
ah, ok. checked the android sources, these methods return either a `BitmapDrawable` or a `NinePatchDrawable` which don't share a common baseclass except for Drawable, so this is fine.
Or better yet:
```D
auto commonPrefix(alias pred, R1, R2)(R1 r1, R2 r2)
    if (((isForwardRange!R1 && !isNarrowString!R1 && isNarrowString!R2) ||
         (isForwardRange!R2 && !isNarrowString!R2 && isNarrowString!R1)) &&
        is(typeof(r1.front == r2.front)) : bool))
{
    commonPrefix!"a==b"(r1, r2); //Forward to the pred implementation
}

auto commonPrefix(alias pred, R1, R2)(R1 r1, R2 r2)
    if (isNarrowString!R1 &&
        isNarrowString!R2 &&
        is(typeof(r1.front == r2.front)) : bool))
{
    // Your current implementation.
}
```

Anyways... I'll stop here now and let you decide what is best.
That can be fixed by using `replace(pred, " ", "")`, but yeah, the problem does get worse. Still, with predicates normally defaulting to `"a == b"`, it generally works. The problem though is that we don't really have any other way of verifying predicates for equality. It isn't the end of the world, but it can definitely lead to lost performance at times. And I have no idea how we could fix that.
Phrasing is a bit odd. Maybe, "the vector whose first..."
Does this use CDI?
Le constructeur a changé. Voir https://github.com/mageekguy/atoum/blob/master/classes/test.php#L72
Use BrowserContract.DeletedColumns.* instead.
`safeQuery` will throw if the cursor is null, so you can structure this pile of code like this:

```
try {
  final Cursor cursor = tabsHelper.safeQuery(…);
  try {
    // Use cursor; it's never null.
    …
  } finally {
    cursor.close();
  }
} catch (Exception e) {
  delegate.onFetchFailed(e, null);
  return;
}
…
```
I think this one is not planned...
If you must do this, use log.debug statements, like:
if (log.isDebugEnabled()) { log.debug("The definitions are: " + ret); }

Better, though, is if you need to see the value for debugging purposes, then run the code under the debugger and put a breakpoint there.
@status-height is the height of the image. So, nothing one can reasonably do there to tie it to a file's dimensions.

@alert-padding on the other hand should be calculated based on something like @default-padding (which I'm gonna add to variables.css.less).
should be relatively easy with the asshat pipeline:

``` javascript
var unansweredQuestionsBadgeText = numberOfUnansweredQuestions + '  <%= t('unanswered_question') %>';
```
we don't want XML at all.  JSON only for the API
Should stick to jquery only.
Don't forget TvE's requirement: the client should be able to round-robin between many hosts in the ring, if one goes down. So our config should probably track array of "hosts" and not scalar "host."

Also don't forget that when we implement the Astyanax back-end, we will want to delegate connection handling to its connection-pool class... should be trivial -- the
we still need to change the visibility otherwise we can not store the field in the db.
no need for this check. Iterating over an empty array works (and does nothing of course)
Two more blank lines here; three blank lines between top-level classes/functions in a module (and between the last import and the first class/function).
This file shouldn't be here (?)
CAUTION:  "setCanvasStyle" method is repeated!

See:

```
..\build\temp\lib\OpenLayers\Renderer\Canvas.js:445: WARNING - object literals cannot contain duplicate keys in ES5 strict mode
    setCanvasStyle: function(type, style) {
    ^
```
Ah, I couldn't, because the associated test is actually in std.datetime. I'll have to change the test after this is merged.
！ってなーに？
change signature of properties<- to say that the value is a list
If we do decide to move, it'll be the iisnode scaffolding. But we still havent settled. So for now its going to proceed with the same behavior as powershell. Follow up / stabilization / finalization issues will follow.
Why the semicolon? Not needed here, this is a straight function declaration.
Yeah, that was the first thing i've tried to do
-A FWR -p tcp,udp --dport 8000 -j ACCEPT

# /usr/sbin/rebuild-iptables
iptables-restore v1.4.4: unknown protocol `tcp,udp' specified
Error occurred at line: 21
It's mandatory provide a valid `Pygments` language name  after `::`

Also it's mandatory leave a blank line after the statement
Double-check this syntax. It doesn't look like you're specifying an axis to rotate on. Out of the first three numbers, usually one of them is set to "1" to indicate you are rotating. Off the top of my head, to match the plain rotate transform, I think it should be the Z-axis: (0, 0, 1, -1.2deg)
Please remember to consider the raw value of parses.
Ну вот я эти классы тоже видел, переспросил, может что-то поменялось )
Excellent - thanks for researching that.
Hm, I wonder--should the session mechanism be hard-coded like this or wouldn't it be better to make it parameterizable? We could change the wrap function to accept an optional argument which must be a function accepting the request and returning the state parameter. If it is not passed a state parameter should not be added to the authorization request (the draft specifies this to be optional).
@staabm I'm afraid not. if we use
if(o.onOrientationChange) instead of the current if(o.onOrientationChange !== false)
bindForLater will never be called unless you explicitly add a onOrientationChange option, this is not optimal.
#mantishelp is the old channel name; this was changed to #mantisbt-help sometime last year IIRC.
In any case, for the time being there is a redirect to the new channel.
to test put console.log("this was called");
This still refs Bitbucket.
why not using date?
You're going through a lot of work to abstract out the service into an interface so you have the flexibility to make changes down the road.  Should we do the same for the QueueInfo type as well?  Or do we expect that has different versioning characteristics than than the service itself?
Throwing this line out. Currently jxls-reader is set to 0.9.6.
Changing that to 0.9.8 must be done on the droolsjbpm-build-bootstrap/pom.xml instead.
i'll rather have
<pre><code>
if (this.expectSingleResult) {
        if(result.size() == 1) 
        	payload = result.iterator().next();
        else
        	throw new MessageHandlingException(requestMessage,
        		"The Jpa operation returned more than "
        		+ "1 result object but expectSingleResult was 'true'.");        			
} else {
        payload = result;
}
</code></pre>

instead
There's a Python library way around this:

```python
>>> from operator import itemgetter
>>> itemgetter(1)(['a','b','c'])
'a'
```

Use itemgetter instead of creating a lambda on the fly.
Why was this flag changed?
Should be removed!
This is changed to id?