@dkoepke the package resource was updated in chef-dk 0.6.x to support windows_package. we saw weird testing failures with 0.6.0 because chefspec had a few bugs and didnt 100% support this change in the chef version yet. in chef-dk 0.6.2 everything was clean on both linux/windows client machines running the tests


GUI doesn't do anything. Don't try to run commands, it doesn't work yet, working on the new format


See the latest commit https://github.com/WebAudio/web-audio-api/commit/c192c6dda475bcf0065770262694a1c1909b0197


> Or can it be put inside `#ifdef GEANY_PRIVATE` to hide it from the API?

Could be indeed (though as @kugel- says we try to suggest only documented things are part of the API, but indeed we could improve enforcing this with macros too)


titties


@npiganeau Is this the correct way you want to use settings ??

Its an instance in `utils`


@triska Yes `inf` -> `infinite` would be better.


This line should be removed, not commented out.


The pytest also doesn't have any 'enter to continue's (even though the sample text you're supposed to use does).  If you take these out wherever you can, your user has to do less work to get the results.


I would change this sentence to say something like:

`The model is a standard POJO class, which is responsible for storing all of the data that defines our view state.  `

I had this sentence here before to lessen the punch of the fact that you had to create the controller at the same time as creating this object.  This is actually no longer the case, as you can add this `@Interface` after you've fully created the controller object!


Figure if we are requiring `first_name`  no need have the or included here.


TeamCity pull requests :: netty [Build 365](https://garage.netty.io/teamcity/viewLog.html?buildId=1987&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5618, ignored: 21 Build time: 00:35:59


TeamCity pull requests :: netty-tcnative [Build 23](https://garage.netty.io/teamcity/viewLog.html?buildId=1989&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty-tcnative [Build 23](https://garage.netty.io/teamcity/viewLog.html?buildId=1989&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:41


TeamCity pull requests :: netty [Build 366](https://garage.netty.io/teamcity/viewLog.html?buildId=1990&buildTypeId=pulls_netty&guest=1) is now running


So we're fine now


As you see in Craig's comment:

 +- ISO14443a stand-alone operation with ARM CFLAG="WITH_ISO14443a_StandAlone". This code can read & emulate two banks of 14a tag UIDs and write to "magic" cards  (Craig Young)

Meaning:
For LF standalone - remove the compiler flag  CFLAG="WITH_ISO14443a_StandAlone".
For HF standalone - let the compiler flag be.

But you can't get both at the same time.


sbclBootstrap was not used before my patch (sbcl was bootstraped using
clisp).  I don't fully understand, what are the rules for using patchelf
are.  For example, patchelf is also not used on linux on arm; so I broke
sbcl on darwin.  I see two options for darwin:

1) use clisp for bootstrapping, something like

   sbcl = callPackage ../development/compilers/sbcl {
     sbclBootstrapHost = "${clisp}/bin/clisp -norc";
   };

but that is slow.

Or better

2) fix sbclBootstrap.  Fixing sbclBootstrap might be rather easy:

   postFixup = stdenv.lib.optionalString (!stdenv.isArm) ''

should be

   postFixup = stdenv.lib.optionalString (!stdenv.isArm && stdenv.isLinux) ''

Unfortunately, I dont have a darwin machine to test this change.

Does the second solution sound good?  If yes, I will prepare a patch for
the second solution over the weekend and hope that it will work on
darwin.


Unless you see another way to skip the '*' pattern otherwise it fails trying to use it as a regex in `sourcePath.search()`


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1345 of 1345 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.28%25-green.svg)


I mean it's not catched by "shouldCheck"? How come?


Nice work . This is fine for now but remind me the case for other platforms. May be comment it as TODO somewhere for later review.  Thank you.


This solves the problem I had where unless I put the ID field of the entity at the beginning of the entity class, it would not generate the IN() statement correctly for the second query.  Thanks for this!


Doesn't CM now set that by default in the vendor through sysfs? And also, isn't the check unnecessary when set to 0?


For lines 54 + 55, Howbout we do something like:

> When defining the model, we need to declare a the class level annotation `InjectController`, and pass in the `controller` and `layout`.

I think that's short and too the point, and also the highlighting alludes to the annotation + parameters.

Also bump this sentence so there's at least one line of space between this and the above line.  Makes it easier to read!


What will happen if you try to create a report after adding a name to your donor list (in your thank_you_name function), but then quit at line 53 instead of entering a donation? Take a look at what happens on line 142 if you have a donor who has 0 donations.


Typo --- in  maximum 'likelihood'


@alwey : I see. Thanks for the advice.


Ahhh, you're right sorry. I read that improperly. I'm out and about right now, but I'll get rid of that when I get home :+1:


Ok no hurry


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.488.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64044&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:18:36


I think this would be clearer if rephrased without the double-negative, for example:

```
if (success) {
    if (mainProcess.ExitCode != 0) {
        Console.Out.WriteLine ("failure!");
        success = false;
    } else {
        Console.Out.WriteLine (sw.ElapsedMilliseconds.ToString () + "ms");
    }
} else {
    Console.Out.WriteLine ("failure!");
}
```


TeamCity pull requests :: netty [Build 363](https://garage.netty.io/teamcity/viewLog.html?buildId=1983&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5617, ignored: 21 Build time: 01:16:04


Artefacto de Riesgos, identificación y medidas a tomar

[RiskList.docx.docx](https://github.com/stiven-0611/sw3-encuesta/files/7052/RiskList.docx.docx)


x is in the input time vector.
n is the length of input vector
x[n] - n is used as index to refer the last element of time vector.
tn - time at nth index of time vector

I hope its clear now. Make changes as necessary.


This method is still referenced at https://github.com/cloudfoundry/cloud_controller_ng/blob/7cf51e3329647370efffcd315087e37776c3e178/app/models/runtime/app.rb#L212 - how did the tests pass?


I have cooked something at https://github.com/opengapps/opengapps/pull/126


Ok, good night ;-)

> Il giorno 02 ott 2015, alle ore 16:38, Gabriele Falace <notifications@github.com> ha scritto:
>
> hey, OK for this time but start also writing everything in english ;)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/carminemacri/exampleProject1/commit/1187c27b5d2eaea25578c857f7c09ad0c112e78c#commitcomment-13563708>.
>


Elección del framework a utilizar en el desarrollo del proyecto
[Framework.docx](https://github.com/stiven-0611/sw3-encuesta/files/7054/Framework.docx)


Let me know if you find any other issues in the newer commits and I'll fix those too


you added fonts to your Project file that are not in the repo. Therefore I cannot build your project. Please correct so I can build and run your project

--dirk


actually i have different idea for tests, but should probably still be in parsers :)


Should be isNonVoidMethod I think?


Since soln and sol are local variables you can rename them with appropriate names. Thank you for pointing them


If you make the command a uniform case (all lowercase or all uppercase), your program becomes more flexible and less confusing because a user can input 't' or 'T' and it still works.


PHPCS: Line indented incorrectly; expected at least 12 spaces, found 8


Could probably be `Integer`? It would simplify a bit the code above (`.intValue`).


w is warning....its an object created when there a warning in handled. It contains information about the warning.


а это не понял че за фигня


Yes. This is debug statement. This should be commented as debug comment.


@jpraet ,
```
if (typeAST.findFirstToken(TokenTypes.LITERAL_VOID) == null) {
                retVal = true;
}
```
so name is valid.
Please explain what make you think another way, or am I miss smth ?


Thanks for this consistent return statement. Its a good practice.


Ah, I see! Thanks.


yes, that is typo, please send PR with update.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.66.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64036&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:36:55


Solves #1


Adding a labels across all metrics on the exporter side is bad practice as it pollutes the labels, generally that should only be done on the Prometheus side with target relabelling.

In this case I'd suggest a single 1-values timeseries with this label, which you can then use `group_left` to do a join on. See the `prometheus_build_info` metric on prometheus itself for example.


@winstonf88 a versão aqui não deveria ser 1.1.3 ao invés de 1.1? Pergunto isso porque as outras versões estão com 3 dígitos. Por exemplo 1.0.5.


sol is a dataframe of aMLE and bMLE.  Please try to mention the type of return. set might mislead to different data structure.


Keep these lines under 80 characters.  You can use a backslash (\) in a string if it is too long, like this:
```
print("{:<20.20s} |  {:<10.10s} | {:<5.5s} | \
         {:<10.10s}".format("Name", "Total", "#", "Average"))
```


Should we pre-populate the cache? It could be significant when using the Ruby client library for instance who will do auto-refreshes (create -> follow 201).


Hi Victor.
What was the aim of this commit and the next one?


Is this function still used? Can it be removed?


Yay it worked! I (sometimes) know what I am talking about https://travis-ci.org/jennybc/jane/builds/83376850


It would be better to use a .format() like you did with your other long strings.


This needs to work for OSX as well, where the cmd key is used.  Maybe this is being mapped native side to ctrl, but I don't think so.


If you don't mind me asking, why const? The only thing I see from this is that they cannot be changed once defined, but... what's the actual difference it makes?


Good python style is lowercase for function and variable names, and uppercase for classes and objects.


I'd re-phrase "as power to both VIN and VBAT is lost" to be something including "neither VIN or VBAT are powered".

The original will be interpreted by some to be that VBAT _and_ VIN need power at all times, which is not strictly true either.

This is an area where strict semantics are important.


I don't believe you should need this... `asset` should be using the `top` scope, which excludes inline assets.


Pattern is with two "t"s @Joshsora


good call, will tweak


n is consistent with mathematical relations. Lets leave it the same.


dsfd


Thanks, I'll change it in next commit. xP


@philbogle and/or @davidsbailey can you double check my changes to varnish before i upload to chef?


некорректные классы, в документации к bootstrap написано, что должно быть `btn-group btn-group-justified`


Not necessarily one armament per attack trait, but see #9064.


THIS IS FOR THE JAVASCRIPT CONVERSION OF THE TS FILES


Again, I pushed on upstream instead of origin by mistake. Feel free to delete this branch on this fork for now.


After you print out the thank you letter, the program should go back to the main menu.  This is causing your code to fail two of the pytests.


But you are right about the 'r'. I needed a data frame rows. may be 'rows' is a good name. You think of a name. Any name is fine as long it works. I didnt care for names because its internal (local) variable.


Looking

On Fri, Oct 2, 2015 at 1:56 PM, Bjvanminnen <notifications@github.com>
wrote:

> @philbogle <https://github.com/philbogle> and/or @davidsbailey
> <https://github.com/davidsbailey> can you double check my changes to
> varnish before i upload to chef?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/code-dot-org/code-dot-org/commit/e6e5220a815d156fd2d17163384805ada36658d3#commitcomment-13570689>
> .
>


yeah I also thought about that...


> There is nothing specific here.

There is, there's a part about IntelliJ, one about Eclipse.

If you're an IntelliJ user, it's much easier for you to set a `~/.config/git/ignore` file to ignore the files that your editor leaves around: you can do this once and for all your projects. The IDE one uses is specific to the user, not to the project, so it makes more sense to put it in the user's config file to avoid polluting the project's `.gitconfig`.


почему текст без блока и блочный элемент находятся рядом на одном уровне?


Yes that makes sense.


typo ttime


@ronchifabricio @martinmaiochi @JoeVogel

![ohhhhhh](https://cloud.githubusercontent.com/assets/1909761/10257689/4081e560-692f-11e5-8232-dc33c0de2f66.gif)


This section of code might be a bit dangerous.

Previously, junction would call `updateAccount` which would call `mergeWithDelegate`. The implementation has a safety check to ensure the BCD isn't updated. Now, we by-pass this check here, so we need to make sure `DefaultInternalBillingApi` knows when to update or not (using the magic `0` value?).

If not, we could be in a really weird state as if the account was created with the BCD set, `putIfAbsent` below wouldn't update the cache and we would be out-of-sync with the db...


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.489.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64047&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:39


Its failure intensity.


typo likelihood


`get_docstyles` uses it to load configurations


Its continuous. +1


The pytest has the text 'Create a (R)eport'.


1. this allows modification
2. reparsing is imperformant, so this is a caching mechanism. The caching could also be implemented inside a function, but that would result in the same code (but changing some names to tell the outside user not to touch them)


This section's language and tense should probably be changed so it's not conversational.


The comment is not clear. Time it takes to achieve the desired reliability is main functionality.


Ok then


You think this is important for the testcase?


ok let's try this, but i suspect making the seqno a timestamp itself will be really good in the long run. we can test in the network. edge cases will be annoying though.


Why is `PrintTo` needed? Also, taking in `std::ostream` by pointer instead of reference is atypical.


growth in reliability. what do you think is better? I prefer this. ignore model word.


1. I changed the logic in updateAccount to *NOT* allow updating the BCD from api if this is already set
2. Junction will only call the updateBCD, if it is not already set.

It is true that there is a race condition where while we compute our first invoice (and set BCD) through junction, we could have in parallel an account update trying to set that BCD (but this is not new). The only way to prevent that would be to enforce the check at the SQL level or take the global account lock when updating the account.


- You should upload an sql with the queries you use to create de database and the tables


Good idea.


@idgserpro a versão aqui não necessariamente deve seguir a versão do brasil.gov.portal.
O entendimento seria que a versão 1.1 do Portal Padrão usa a versão 1.1.3 do brasil.gov.portal.
Perceba que existe a versão "1.0", e não "1.0.0". O terceiro dígito indica bugfix, o que não ocorreu ainda para a versão 1.1.


why not add the doxygen scheme to default? Also I don't really like much how you duplicate the configurations in the default and the other files...


Not really because what we cache is the *immutable* part of the account and what we return is the full thing (which will always require a query).


LGTM


heh


> I don't fully understand, what are the rules for using patchelf are.

AFAIK darwin doesn't use ELF.


Good comprehension +1


Note that you don't have to manually do this -- at every release, I run a script that pulls author names out of the git history and makes sure they appear in AUTHORS.


```go
if cmdutil.GetFlagString(cmd, "type") == "LoadBalancer" {
```


allowing modification is bad IMO, adds state to the software making it complicated. I don't care about performance. Cleaner is better, if it's more inperformant we need th find a better design or give a crap about performance. PCs will get faster but software won't get easier.


Very useful change!


it may be dependent on the implementation, and the test should be implementation independent. So yes.


Did you check for NA return. Please check for this NA return. Something is not right. Ignore me if thats the case(NA return).


@brev The tests are still failing:
Here is the output of `npm run test` :
```diff
1) FileServer #getFields Get fields using default options:

Uncaught AssertionError: Got different fields
+ expected
- actual

 [
   {
-    "file_uid": "f620b4fb4f545ab0e74c62f7357861aaffa29dda"
+    "file_uid": "b1e39d95721813cf5186553fff4dcd26692d189a"
     "name": "timestamp"
     "type": "date"
-    "uid": "6fd17df19c2347bb023208aeff0b8e38a2edcbf6"
+    "uid": "8cc52a23d53b19cb8c2618c4118dc6b40c4ebc87"
   }
   {
-    "file_uid": "f620b4fb4f545ab0e74c62f7357861aaffa29dda"
+    "file_uid": "b1e39d95721813cf5186553fff4dcd26692d189a"
     "name": "metric"
     "type": "number"
-    "uid": "bb39a8170346a95940e12b176c00da3e661eae09"
+    "uid": "8e80050a99d6cc4399a43593f4e1efe231fc49be"
   }
 ]

at tests/js/unit/FileServer_test.js:113:16
at CSVStream.<anonymous> (frontend/lib/FileServer.js:162:9)
at readableAddChunk (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_readable.js:195:16)
at CSVStream.Readable.push (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_readable.js:162:10)
at CSVStream.Transform.push (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_transform.js:145:32)
at CSVStream._emitLine (node_modules/csv-streamify/csv-streamify.js:157:10)
at CSVStream.parse (node_modules/csv-streamify/csv-streamify.js:77:12)
at CSVStream._transform (node_modules/csv-streamify/csv-streamify.js:47:10)
at CSVStream.Transform._read (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_transform.js:184:10)
at CSVStream.Transform._write (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_transform.js:172:12)
at doWrite (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_writable.js:279:12)
at writeOrBuffer (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_writable.js:266:5)
at CSVStream.Writable.write (node_modules/csv-streamify/node_modules/readable-stream/lib/_stream_writable.js:211:11)
at ReadStream.ondata (_stream_readable.js:525:20)
at readableAddChunk (_stream_readable.js:146:16)
at ReadStream.Readable.push (_stream_readable.js:110:10)
at onread (fs.js:1744:12)
at FSReqWrap.wrapper [as oncomplete] (fs.js:576:17)

```


at least a small sample with a real setting would be good


(mentioned on IRC earlier)
I don't think this usage of the new `LockTime()` function will work when we want to make the new behavior part of consensus, at least in its current form, because pcoinsTip isn't updated between calls to LockTime().

At first glance, I think we likely need to move this check to `ConnectBlock()`, but I haven't thought through the implications of that.


Sink parameter, use rvalue/universal ref:
`FilePath::FilePath(StringType&& path) : path_(std::move(path)) {`


@stonerl Hey, is this something you would like to see in the main app?


I actually didn't have a race condition in mind (but it's a good point...), but wanted to point out that the `0` [here](https://github.com/killbill/killbill/blob/caching/junction/src/main/java/org/killbill/billing/junction/plumbing/billing/DefaultInternalBillingApi.java#L184) is really critical (which maps to [this](https://github.com/killbill/killbill/blob/caching/account/src/main/java/org/killbill/billing/account/api/DefaultMutableAccountData.java#L28-L29)). That's not new, but the previous `updateAccount` implementation would add a second check and throw an exception in case the BCD was set.

Just wondering if it's worth re-adding a safety check here before calling `setBillCycleDayLocal`.


@brev Can you remove this extra `console.log`?


Ah, yup, good point.


want to not have this whitespace, so it gets removed


You're making a copy anyway, better to do that right in the arg list:
`FilePath FilePath::Append(StringType appended) const {`


// TODO: Do something with pending deployments


same here, just:

```
`
```


Needs to be called RCCA now.


Hmm, `tryEval` isn't documented in nix manual.


lool


This should specify the sensor as the controller, not the shield.
i.e. controller: "MPL3115A2"


This should specify the sensor as the controller, not the shield.
i.e. controller: "HTU21D"


All tests were quite neutral, but promising because different functions give similar results. That's why I'm starting tuning 6 by 6 moves, and then 6 by 6 depths, starting from default values, until some progress will be noticeable. This will be a long struggle, and I'll try to find a compromise between not giving up and not to run my tests too frequently.

Thank you for any comment or advice, anything can be helpful.


@catsby this doesn't make sense

```If order to effectively...```

That should certainly be

```In order to effectively...```


Yes, my bad. I will correct all as()


will panic with an empty string. should error:

```go
if len(p) == 0 {
  return "", fmt.Errorf("paths must not be empty")
}
```


@yehudasa ```ERR_PRECONDITION_FAILED``` means that [the hint is not actual anymore](https://github.com/rzarzynski/ceph/blob/81c23745c19a1fb67270626e84b0e3bf7b3ae905/src/rgw/rgw_object_expirer_core.cc#L107). I assumed here that if the bucket has been deleted, all its objects (including the expired ones) must be deleted as well.


Thanks for coming up with this hack to address issue getsentry/responses#80, @JesseWeinstein!

These import statements should be modified to something like the following for Python 3 support:
```
from six.moves.http_client import HTTPMessage
from six import StringIO
Try:
    import mock
except ImportError:
    import unittest.mock as mock
import requests.adapters
```


Need to fix, since /jsondoc servlet doesn't return the property if it is empty string.
```
if(typeof(method.auth.scheme) != "undefined" && method.auth.scheme != "") {
```


Please make the following changes. It will be good to merge once done.
I know it looks hectic and reduntant. I should have done it instead of you making working on it. But  everything looked clear for me since i wrote it. Now  you are helping me with statements which are not obviously  clear. Thanks for the changes.

Karthik.


@brev Do we care about `error` message here? Maybe you could pass it to the `FilesystemGetError` exception. If it is not important for the app then we should at least show a `console.warn(error)`


Good catch, I looked right past that last night.


Same thing here that @dougseven mentioned above


This looks like a change in functionality, not just a notice issue?

Or - why shouldn't we be using the front end mangling for the base credit card processor?


See https://github.com/brev/numenta-apps/commit/3f252b4fafd80d2691fb5f5c209b36cfadbcb5b0#commitcomment-13571015


TeamCity pull requests :: netty [Build 366](https://garage.netty.io/teamcity/viewLog.html?buildId=1990&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5619, ignored: 21 Build time: 00:37:49


I'd set default to {} here. In many cases role is not needed.


"try to suggest" is softy, we're actually explicit about it.


Resolves #42


Juhu ! Works like a charm !! - Also the battery animation appears much faster with such small delay ...
it makes more fun .. especially with dt2w it just response so quickly ...
maybe u can test with your LCD what is the smallest delay what can work without issues for cwm + lpm .. and so make also use of this great hack !

check:

https://github.com/ace2nutzer/Samsung_STE_Kernel/commit/7c8a8539252091c6f7afc0ebf94f63b8c2c4cc00

Todo: fine tuning video.modes for ws2401 .. then this driver is perfect !

Thanks for amazing cooperation :-)


this test should also check that root remains unchanged. checking the error code is not enough


After testing of 1.2.8 I've found one bug and one suggestion.
Thank you


this should check root remains unchanged. a good way to do this is to:
- put other stuff in root first
- grab root hash
- run this
- check root still matches hash


// TODO: Do something with pending deployments


- [ ] check file remains unchanged.


Is there a hardware limitation that would prevent this from being a runtime condition rather than a compile time?


LGTM


- [ ] check dir remains unchanged


- [ ] check path does not exist.


@winstonf88 até agora, a versão do brasil.gov.portal é que indicava a versão no portalpadrao.release. Veja que em todos os versions.cfg dentro [dessas pastas](https://github.com/plonegovbr/portalpadrao.release) a versão do brasil.gov.portal é a mesma da pasta.

Acho que não existe a pasta '1.0.0' porque a primeira versão do brasil.gov.portal foi a 1.0 e não a 1.0.0.

@hvelarde comentários?


Do you need setuptools>=8.0? The requirements.txt file still points to setuptools>=3.3


I think this is obsolete; the 2438.magic-folder-stable.3 branch fixes it in a different way.


Thanks, the URL is fine :)


yeah but as @elextr would say "nobody reads the docs" :S


thanks for taking care of this! :D


nvcbn


c**s**v importer of course - even the damn Github for Mac has some autocorrection


Here we don't need to print out a list of all donor names after the user has already selected a donor from the list.


That ain't our (=maintainers of the API) problem. We have to draw the line somewhere.


... and you don't have a great idea, do you? :)


@mine-cetinkaya-rundel do you what this means?


Hi David. Why did you do that?
outlayer.js depends on item.js and won't work without it. This change breaks behaviour of convenient plugins alike [main-bower-files](https://www.npmjs.com/package/main-bower-files) which fetch files from main parameter. That plugin allows us to fetch only main js/css files from bower dependencies, concatenate them, then place somewhere in the project and add to html file **only 2 tags** instead of managing all thirt-party libraries by hands:
```
<link href="assets/vendors.css">
<script href="assets/vendors.js"></script>
```
Isn't that cool?
Please say what was the reason to remove item.js from main?


Can we not leave this to autoprefixer?


========== Build: 12 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


Version 1.0


Version 1.0.1


Some little test with large graphs generation show that this solution  is ~10% faster, so maybe It's better to have this faster altough less elegant form


Why didn't github show me the new commit?

Anyway, a couple of nits.  Since all of the params are on the left side, I think the gain node arrow should feed into the detune on the left of the oscillator block.  And maybe have the ouput line from the gain node start in the middle where all of the other nodes have there outputs and inputs go.


added <version>2.0.31-beta</version>


Version 1.0.2


Version 1.1


Why was this commented out? @jmoenig

I was just going to make a PR adding the same thing, until I saw it was here.
It would also be nice to have the exported date below the username.


Version 1.1.1


Version 1.1.2


The i0 correction is now done in the XPSScan itself.


Fixed in 0af9a9f68f490c9f7eee37f4647a131ee9ffe939, thanks


Can I change this so Snap! appears as Snap<i>!</i> ? :)


:) Nice one!


**react/prop-types**: 'version' is missing in props validation for ComponentDetail


**no-unused-vars**: example is defined but never used


**semi**: Missing semicolon.


**no-unused-vars**: photo is defined but never used


no worries! :)


Also, it would be nice, if the project we public, to add the shared URL to the output.

Since this page is editable, it could always be deleted.


Version 1.2


Version 1.3


a=int(input())
r=int(input())
y=1
p=1
while True:
   if r*a==0:
       break
   while r>a:
       if r==0:
           break
       y+=1
       a=r
       r=int(input())
       if r==0:
           break
   if y>p:
       p=y
       y=1
   while r<a:
       if r==0:
           break
       y+=1
       a=r
       r=int(input())
       if r==0:
           break
   if y>p:
       p=y
       y=1
   if a==r:
       r=int(input())

print(p)


Version 1.3.1


Dude, your commit sux


Version 1.3.2


now if data isn't available when ready is called, the globe will render and the data will fill in afterwards


done


Oops. That was just for testing and I forgot to take it out.


No problem.


Too complicated here. ```count <= 1``` is good enough.


Version 1.3.3


can you create a FIXME for this? you can reference https://github.com/rust-lang/rust/issues/27579 (I amended the description) or create a new issue. I believe that in a trait impl, at least, one ought to be able to assume that the input types are WF.


@taxigps Thx! But we don't call it from the ctor, changed it to 20.


Was Mario's change on 6.3.x too?

I might have to cherry-pick a kie-wb-common commit.


Version 1.3.4


Yeah I think ensuring it's not nil is correct in both. Fixed.


Simplest than I thought!


Yep, I agree (to both).


Fixed in a subsequent diff, I think.


Added back: https://github.com/robbyrussell/oh-my-zsh/pull/4448


Is checkOpertorEq() called somewhere? I can't see it in the diff.


Consider keeping these switches: `-fno-exceptions -fno-rtti -fvisibility=hidden`

 * No exceptions and no RTTI ensures that the library has no dependencies on libstdc++.
 * Visibility hidden ensures that only the explicitly exported symbols end up in the output library and might even allow the linker to do some optimizations.


Version 1.4


It's a trap. That was originally the code, the break was inside, but I had to remove it because it was breaking out of the select and going back into the for loop. Madness ensued.


Love it!


Fixed in a later version of claims.


Why is `options.serviceName` necessary? It's not used anywhere in this function body?


You're right it's not going the exception way. But it's anyway possible to collect/catch all warnings issued from a command, so in this case:
```python
import warnings

with warnings.catch_warnings(record=True) as w:
    warnings.simplefilter("always") # No panic, after exiting the context manager the previous filter value is restored.
    extract_documentation(...)
    # 'w' contains now all warning messages (more precise, WarningMessage objects).
    # We can print them with some 'warnings' function and redirect them to our logprinter.
    for warning in w:
        log_printer.print(warnings.formatwarning(...))
```
Even I think the approach is good, I have the feeling the `warnings` module is not that well written, because I would expect to invoke `warnings.formatwarning` with a `WarningMessage`, but this is not possible, instead you need to feed every member of `WarningMessage` into the function. I find that quite unconvenient, especially because `WarningMessage` is not documented^^
But we could easily build a contextmanager that executes this snippet and redirects output to a given log_printer^^


This is weird. Why was it compiling? Let's rename all of these functions to more meaningful names. Perhaps `mesh_inside_other` or `component_inside_other` or `component_inside_component` and `points_inside_mesh` or `points_inside_other` Also the input parameter `I` is not well explained in the header doc. Once renamed these _could_ be further split into two files.


See https://github.com/brev/numenta-apps/commit/3f252b4fafd80d2691fb5f5c209b36cfadbcb5b0#commitcomment-13571015


Pretty sure this is rectified in the latest version of the code. (It's hard to review "6 patches in a row" because certainly I found bugs in N<6 that were fixed in M>N.


Yep, good catch


Version 1.5


It's used in the `getClientChannel()` method.


Take a look at what happens when you enter a donor name, but then quit instead of entering a donation amount, and again quit after returning to the main menu. The home_menu function will return a value of False after being called by get_donation. That value is then passed back to line 64 or 69 - giving your donation variable a value of False. Your send_thank_you function then continues and generates a thank you letter using the donor name, and a donation value of 0.00 even though the user tried to quit.


Version 1.5.1


Version 1.5.2


to make the data viz based on one of the other 5 new vcf files you just need to change this line @zmartine


I didn't have chance to verify that converters above actually work with PHP, but I've noticed few other problems.

1. Typo in `Cookies.withConverter(...)`, it is called with 2 x `write` properties.
2. Usage of `setrawcookie()` is missing as documented in https://github.com/js-cookie/js-cookie/issues/70#issuecomment-126965203, it's quite important:

----

**In PHP**, use `setrawcookie()` instead of `setcookie()`:

```php
setrawcookie($name, rawurlencode($value));
```


Version 1.5.3


to make the data viz based on the original set of variants just uncomment this line @zmartine


Version 1.5.4


I don't see why we are passing disabled by reference. When you do it this way you disable any links to categories where there is not permission to add. Maybe I am not seeing something.


Thing is, we're currently not association usernames with projects, therefore if you export the summary of another user's shared project it will show *your* username as the author. I *think* this could be okay for the case of letting students produce AP project reports, but it could also irritate users.

I'm undecided, that's why I put it in but then commented it out...


Version 2.0


The only duplication are the warning messages. I thought of moving them into a function, but in fact they are bound to the usecase/extraction-mode, so I decided against.
What is maybe possible to move each extraction-mode into a function.


It's called on line 704.


Why was this removed?


Nice gapps. Thanks a lot


doxygen is not everytime the default documentation tool, downvote from my side^^
and even if the entries are duplicated, this happens only in few cases and taking it strictly they are more or less independent, the documentation tool can decide to work with different layouts^^


Version 2.0.1


Ok, never mind, found [this](https://discourse.roots.io/t/best-practice-when-overriding-main-files-of-bower-sub-dependencies/3590)


why?


Version 2.0.2


Glad to see you here @juhalindfors :)


Version 3.0


> Also the battery animation appears much faster with such small delay ...
it makes more fun .. especially with dt2w it just response so quickly ...
maybe u can test with your LCD what is the smallest delay what can work without issues for cwm + lpm .. and so make also use of this great hack !

:+1: Maybe later I'll implement a tunable for those delays, then it will be easier to find this smallest delay.


I think so, but I'd welcome closer analysis.

I am not sure if I like the mixing of atomic ints and the locks. The latter can be used to do most of what the former does although it's not nearly as nice? But perhaps much easier to reason about. It also provides nice properties of forcing things to be serialized (well, writes). Thoughts?


Version 3.0.1


maybe in this case it makes it really more complicated...
anyway I'm for a caching mechanism, so we just call `docstyles` now `_docstyles` and everyone knows he/she shall stay away^^


Version 3.1


Ok, sorry for the noise then, was looking for something like TEST_CASE(bla).


you're right^^ :+1:


Version 3.1.1


Yes. See https://github.com/droolsjbpm/drools/commit/8805729e294a41a58617b77eb24d1cd04a148414


Footer user story


There is a short-circuit about 20 lines up that does "if there is no data in the partition, set alive and flag the waitgroup".


Version 3.1.2


Part of my ongoing hacking, after I added partial dr support.


Ah, that is true. However, we _do_ have the association of where the project came from if they're opening a shared project, because the username is in the URL. If the URL is a shareable URL we could use that, or if not fall back to the current user, since it's a private project on their account.

(of course, if you've re-saved someone else's project, well...nothing we can do about that.)


return the shouter one, not the compressed one.


This code block should be

```
        if (!(
            (txToSequence <  SEQUENCE_UNITS_THRESHOLD && nSequence <  SEQUENCE_UNITS_THRESHOLD) ||
            (txToSequence >= SEQUENCE_UNITS_THRESHOLD && nSequence >= SEQUENCE_UNITS_THRESHOLD)
        ))
```

I forgot to replace nThreshold with SEQUENCE_UNITS_THRESHOLD


Version 3.1.3


Differentiation by color may go away. Check a dev build of the console closer to the release.


:+1: Just small error logging questions.


This will make this obsolete.


Version 3.2


Version 3.3


Why might using a pass flag be preferable to throwing an error?


No need to use extra spaces.


Version 3.3.1


Also, minor, but we probably shouldn't try to localize the username, wherever it comes from.


Coming to this a little late, but I'm not sure it make sense to expose arbitrary Celery statuses. Just sticking with PENDING / SUCCESS / ERROR is probably good enough for the public API. I don't really know what I'd do with RECEIVED vs. REVOKED, etc. Could some fallback status be exposed if Celery gives one of those?


This doesn't seem quite right?

The problem described in the comment is real, which is how I started looking at this code. The solution proposed in the comment -- "If that's not possible, then don't use a marker" -- ought to work. Unfortunately the code doesn't actually ensure that. When `pos` is a marker and its current position equals `(point-min)`, then it will remain a marker. (Which has the described problem of moving down below any newly-inserted text, and therefore ignoring any new errors.)

Instead did you mean something like the following, which actually will return a non-marker?

```cl
(if (> pos (point-min))
    (copy-marker (1- pos))
  (if (markerp pos)
      (marker-position pos)
    pos))
```

Or equivalently using `cond`:

```cl
(cond ((> pos (point-min)) (copy-marker (1- pos)))
      ((markerp pos) (marker-position pos))
      (t pos))
```


Version 3.4


Version 3.4.1


Good point, Michael. Yes, this is something we could do, indeed!


No, there is no w3c spec for it. Just the vendors.
On Fri, Oct 2, 2015 at 5:23 PM Jeff Escalante <notifications@github.com>
wrote:

> Can we not leave this to autoprefixer?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jenius/axis/commit/9819074ed3174f4d4ed779db48661095ce4a1ff0#commitcomment-13571236>
> .
>


Hmmm it's actually better to blow away the repo and start over than remove the gitmodules because they will forever be in the git history and will bloat the repo's size. @mmcachran


Version 3.4.2


Woah, why did you merge this?


I like ``component_inside_component`` and ``points_inside_component``.  I am good with splitting it into two files.


Version 3.4.3


It seems that it may be rewritten as

    template<typename... Ts>
    using has_placeholders = any<list<Ts...>, is_placeholder>;


Version 3.4.4


Do you mind updating this again for OS X El Capitan? Terminal is now Version 2.6 (361).


Thanks.

On Fri, Oct 2, 2015 at 5:43 PM, Qingnan Zhou <notifications@github.com>
wrote:

> I like component_inside_component and points_inside_component. I am good
> with splitting it into two files.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libigl/libigl/commit/89a8257d152165218fc514e4e35a5aa56dfeca10#commitcomment-13571730>
> .
>


Version 3.4.5


Version 3.4.6


@eteq added that so that failures become expected failures.  I think we still want that.


I don't think you actually need a GC handle here actually. I believe these are equivalent:

```
 //Must call free at some future point
 GCHandle _callbackHandle = GCHandle.Alloc(callbackFunction);

 //Automatically freed when there are no further references
 object _callbackReference = (Action)callbackFunction;
```

The difference is there's no risk of forgetting to call free with a simple object reference!


Other than that though this is a pretty neat solution, and I'll definitely merge it when that's resolved.


I haven't filed a bug with pillow yet. The test does pass when you install `pillow==2.9.0`


Version 3.4.7


This class ui-front doesn't work on autocomplete module with modal box.
When a modal box are on focus, the autocomplete is invisible or is visible on back modal!


Version 3.4.8


Version 3.5


I believe is about all :)
I suppose you've extracted from 3.3 release, not from master ? right ?


Version 3.5.1


Version 3.5.2


Version 3.5.3


Version 3.5.4


parens!


Version 3.5.5


Neat!  Good to know about that.


Version 3.5.6


You mean just to have a reference to the callback in the Channel instead of a GCHandle?


@hraban: I've already mentioned it: The header is where small byte swapping functions belong to. Otherwise, they can't be inlined. This small change causes already 7% less scan time.


In python3, some of your .keys() can be removed (just check for presence in trigrams) and some will need to be converted to a list.


I'm a newbie and don't understand well how it all works. I assumed that this was a bot request that only corrected the typo. I've rolled it back, don't worry.


flake8 doesn't like all these blank lines.


A question: would it be a good thing to install archivematica-devtools by default (i.e., in all installations, not only in development VMs) ?
Also, user "vagrant" is assumed (implying this is running on a vagrant VM). I think it'd be good to try to make it more generic (so it can be used in non-vagrant installations)


Ah ok. We should keep an eye on this then!


kkkkkkk

Isso é intriga da oposição


> This would probably be total overkill, but it would be great if you stored a history of the player's position each tic while playing the map and then overlaid it on the map view during the intermission to show the route they took.

I've actually experimented with this idea before, and have been considering it again. See commit 3e52383b0e57077f0ee863f51443289a9c8e27cb.


I think it's easier to read this way:
```
if (valueSet.isPresent()) {
    checkArgument(valueSet.get().getType().equals(type), "type does not match valueSet type");
}
else {
    checkArgument(!type.isComparable() && !type.isOrderable(), "no valueSet provided for equatable/orderable type");
}
```


See issue https://github.com/uber/tchannel/issues/1364

We need to mutate the channel if it exists because there is no garantuee it was created using `getClientChannel()`.

Its pretty much impossible to mutate so i threw an exception instead.


Yeah ... but keep in mind that an working delay for lockscreen it may doesn't work properly later on cwm and lpm.

Just for comparison: lockscreen display reset = 1x time, cwm= 2x times and lpm= 3x times ...


misspelling?


m_mipmap is not initialized in all ctors. Is it correct?


Updated to node 0.10 minimum as npm install fails for some transitive dependencies.  Update .travis.yml accordingly.


No need for `BaseException` alias with this change?


Use m_* prefix only for private/protected fields


Reported here: python-pillow/pillow#1462


Merged here, right now.


Thanks Petr, I've commented the PR for kie-wb-common.


Yep


It's determing fname for non-IATS processors too


the instack.xml @radez  provided has linux bridge defined for brbm and not OVS.  Not sure why this is.  Will need to compare that instack.xml to the one provided in the CI build output.


oh, no, it´s from master branch


Priyank, these are the changes I made that worked for me. I'm sure that it can be improved in some way though. Thanks again for this easy to use library...


If an


>tfw now dank branch can't even be dank


Omit


Hi Eric, will do as soon as I can.


>10 seconds ago
no fuck you github that was less than one second ago for a bit there


>42 secs


>>>>>>>>>>>number must be minimum 10
thanks jewhub


I'd like to know more about this too. Does this package have eslint declared as a dependency?


is github the new chan


i like shitposting here


done


having each mode in an own function might make sense, right? Then you can put the try/except/warning in another function to reduce duplication even


@Makman2 if you think we could profit from the warnings module I think you should file a bug for it. Until we understand whats happening there IMO we should stick with our current procedures which we know work well: pass the logprinter.


I just realized that even though we specify the output formats we're not telling Rust where to write to.  We might have to do a yaml like this:

outputs:
  carbon:
    port: 2003
    server: "10.0.0.2"
  elasticsearch:
    port: 9200
    server: "10.0.0.1"
  stdout: ~


nope, what's in the default will necessarily be in at least one other file. That's bad.


Soooo... about this? Is there a reason it's commented out?


+1 this is super nice looking!


It is still used but pointless, removing it required restructuring some code instead of just deleting code so I left it for a follow up commit which I seem to have forgotten. Will do that :)


Linking Semantic UI css and js to html.


I think i see now where the macro comes in handy.  You don't have to do any complicated borrowing semantics like the function below has to


Just use a function. Let the function use caching, python or at least pypi has modules that provide caching decorators. No need for a bad design.

For now stick away from caching if it's more than five minutes of work. Performance doesn't matter here. This isn't done often.


Could you at the very least move this near the commented out code?


Played with it today, LGTM!


Changed major to Computer Science.


Done.


Is this working like show up in settings in call settings


Hi, I'm evaluating ebay API clients, and comparing this to the repo it was forked from. I'm curious what bug/problem this commit was solving?
Thank you!


**semi**: Missing semicolon.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for getInitialState


**no-multi-spaces**: Multiple spaces found before '/'.


It is apparently used by the test framework for pretty printing assertion tests: https://github.com/marineam/update_engine/blob/172f0fdbc78c1d023e6690ff64864901a9632692/src/files/file_path.h#L256

It doesn't need to exist in non-test code (it previously was only linked into test binaries) but it felt silly to create an extra file for just this function.


Arne ge zijt niet juist ingelogd, check u settings van SmartGit!


I think its more restful to have /round/:round_id. Would that be better?


you can just initialize all prefabs at once.


This was a mistake, but it is harmless. The comment `// HEY DONT USE THIS` was removed in a later commit by Kevin Murphy (91f8b8c0fb44facf67113b20ea7a2d49355b645e).


Rest looks good


The codec didn't deserialize the object correctly, and I spent several hours trying to figure out how to get it to work. Just deserializing the YAML or JSON into the typed api object worked fine, however. Given that we're trying to get this change in and given that I'm out until Tuesday, I decided to go with it. It still provided the benefit of working with the strongly type interface, which cleaned up the code a lot. If you think it needs to use the codec, then please feel free to change it accordingly.


Beautiful


Done


you can do a "  git config --global user.email "your_email@example.com"  ".  Then you won't be shown as unknown person. @kliewkliew


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4071/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4051/


You _should_ be able to simplify this to `import { RequestOptions, WebResource } from "ms-rest";` if you've added the typings field to that package.


should this be

`let meta_command .= " " . path`

else it would be --enable=errcheck./...


Um, can we revert this change please??


I converted finally, testing.... I will give feedback soon.. ;)


This cost me about an hour wondering why I was missing some compiler outputs :(


A third email for you because this was so naughty.


Great catch, can you open an issue please? I'll fix it asap, Or just send a PR.


Hi @zasran does this do anything w.r.t. fixing the issue where doc generation is not taking into account the files that are inside a package?


no need to commit i<nums.length-2, i<nums.length while using start<end condition


we should probably scrub these to use Guava helpers


Oh okay. This function is so similar to the one in projectiles I mistaken'd it for it


Gonna start a build now. I'll let you know in about 30 minutes.


...and use Guava helpers here too


Добавил header


per offline discussion, when we're in an emergency, we want to be able to change the maxgoroutines for a particular service without affecting the rest. On top of that, the common section should really be common so that we don't need weird salt hacks to make different values happen in different services.

So, this needs to go into all of the relevant service configs.


Why do you like foo bar so much.

On Fri, Oct 2, 2015 at 7:28 AM, Ricky C. <notifications@github.com> wrote:

> For relative URLs, if the file is in the same folder, you can just do this.
>
> <a href="index.html">
>
> Example Directory Structure
>
> js_apps copy.html
> index.html
> /foo/bar.html
>
> Linking to a file in a folder! <http:///foo/bar.html>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/camzillajs101/NHA-Homework-Week-1/commit/0d631ee9d9616ff946f4e41afd52c3033f6a2cb4#commitcomment-13560224>
> .
>

--
Camilo Alexander Mason
Email me at: camilomason0@gmail.com
School account: 16camilomason@ps10.org


Why is this not allowed?


Changes in lines 540 and 552 apply to many JMRI help pages (they're normally part of the footer insert). Can we change these in one smart edit?


Està bastante bien, pero no estoy seguro si a la imagen se le mete el scaletype en el xml o en el codigo. Yo lo meti en el xml.
Saludos


Fixed in a7be38e


tested and working


畫面由白翻黑 XD


Ha, I didn't mean to sub-post on the Slack today, but this is a good change.


Q2 :(


should this have been private?


Yea cause I think its missing from dialer XML


You could take out the h1 tage in your view and done this code in the else case:
```ruby
else
  @text = "#{params[:name]} is so {params[:adjective]}"


Hey, thanks for dropping by. And for the tips :) I added them to the website[1] I hope I explained them right (btw, turns out mingw-w64 is smart enough on all these accounts so they are not needed there, but it doesn't complain either so I guess it's good to add them anyway).

[1] https://luapower.com/build-scripts#dynamic-linking-with-gccg


That keyValue[i+1] check looks like a lurking IndexOutOfRange exception to me.

(Also, I prefer it if all the "if" statements test in the same direction, i.e. test for reasons to continue, then throw the exception if all the "if"'s fail.)


tanto faz para mim, se querem continuar como antes podemos renomear para 1.1.3.


:+1: LGTM


Use 2 space indents for ruby


New line at EOF


Of course. Shall I open a PR?


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4072/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4051/


2 space indents


Not exactly the intention of Q2


That's fair to assume.  We'll just have to make sure that's how the client works.


Use a newline at the EOF


findAvailable is overwritten in Modality, with crude sorting of devices, so they always return in the same order. Maybe it would be better to adopt that for 3.7, and remove from Modality?
https://github.com/ModalityTeam/Modality-toolkit/blob/master/Modality/Classes/Additions/SystemOverwrites/extHIDAPI.sc


tested and working


Ideally do this logic in the controller


2 space indents in ruby


Did you intend to add the packer-ec2.log file in this commit? Wondering if it can be rm'ed, or if you're thinking that we should version control the build log.


Ruby has implicit returns


Why is this required at all?  Can't we just remove this script?


Is there a reason why we can't use ES6 here?  Just curious on why...


You forgot to support this two new data types at modules/flow/console.c


Url should be friendlier: `departments/(?P<pk>\d+)/similar` or `departments/(?P<pk>\d+)/similar-departments`.

Also- Can you have a version that supports the slug too, ie: `departments/(?P<pk>\d+)/(?P<slug>[-\w]+)/similar-departments`?


Should we move them to data_type definitions?


Yeah, I don't understand how two separate statements can be more concise than 1 statement with both included: ``setProperties()`` vs. 2 ``this.set()``...


er, make that 8938945


Stop doing this.


I had to look at the resource builder code again to refresh my memory. Defaulting to the server side object means you no longer have a 'modified' config to do a three-way patch; just current and original. This commit is no longer the one attached to the pr, so I will put comments on the pr itself.


yeah, I thought it would be good to have a record of the build.

On Fri, Oct 2, 2015 at 3:41 PM, Chris Kennedy <notifications@github.com>
wrote:

> Did you intend to add the packer-ec2.log file in this commit? Wondering if
> it can be rm'ed, or if you're thinking that we should version control the
> build log.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ucberkeley/bce/commit/91176a3f5f76dabf7375d8bfa384f8c75dc81cf6#commitcomment-13572544>
> .
>


Instead of duplicating this logic, do you mind moving it to a method on the parent class?


Looks like this is going to do the `_to_text` conversion many times. Isn't `scrub_fields` set at config time? Perhaps we could do the `_to_text` call then, and just once?


可下载sdk


I think its implied, but also do a `self.assertEqual(response.status_code, 200)`


Actually I'm still wondering if I need to store mipmap count in this structure or not. I still need to test if it's relevant to cache value.


Maybe we should set master to pre redux version (see releases tab) and work on a development branch instead without tight deadline?


*blink*


Dann müssen sich die Regel-Dateien mit zusätzlichen Variablen erweitern lassen.


Taylor, if you can remember back this far, do you know why you left in this "if False"?

If I try to run it now, I get a "syntax error near bayesdb_generator_column" via the update in commit 558674baad707fb80a242e5f5a579884df23af0d

Do you think this test should get fixed and run?


thanks @mli , i have raised this pull request: https://github.com/dmlc/dmlc-core/pull/56


tested and working


aaah! polymer-dart `@reflectable` is in conflict with package `observable` one. All thing in autonotify gets messed out by this choice.
Think it's time to switch `observable` from `smoke` to `reflectable`. Any work in progress on this subject?


Great!


extra plus?


@thomasst two typos here


```html
<custom-element>
</custom-element>
```


Ich find's nicht so schön hier einen konstanten Wert einzutragen. Vielleicht ist hier eine Lösung?
http://www.snbforums.com/threads/iptables-clamp-mss-to-pmtu.23873/


I assume the ``self.ax != None`` part should also be changed to ``self.ax is not None`` while you're at it.
http://stackoverflow.com/questions/2209755/python-operation-vs-is-not


I think we should set master to before talk version, so with redux but not universal.


Consider using ```IOUtils.closeQuietly(adminClient)```.


We probably want something similar for the Google Test stuff?


Same comment, could use ```IOUtils.closeQuietly```.


Ruby compiler doesn't let you


This new function seems almost identical to the other fetch function with a different signature, below. I haven't done the diff so I'm not sure what's different, but can they be merged into one?


No it doesn't show up.


Good catch. Can you squash this into the same commit as the previous one?


the caching mechanisms from `functools` work differently (but also quite cool...)
I think it's quite okay to use a module variable, this is no bad design.

But I have some ideas to prevent someone from setting i.e. `docstyles` so it's still accessible like a field using descriptors. Together with a caching variable `_docstyles_config_cache` this would work easily^^


Good catch!!


@UniversalMediaServer/developers fair enough, I change my mind. Good work @valib


Any idea how this change will affect Font Manager?


or ```return hostAddr.isAnyLocalAddress() || hostAddr.isLoopbackAddress()```


This may still have the schema, no?


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4073/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4051/


:ghost:


Test comment


Yea I know I think its missing more commits like dialer and telecomm


This revision is clean.


message me on hangouts at winga45@gmail.com


Fixes #126


sorry but imo this blows up the function and makes it more complicated to use. It works nicely standalone, imo no need for passing a printer.
And the above snippet works, I just left out the parameter assignment^^
But sure I can file a bug^^


Hola, si podemos ayudarte. Por favor, ingresa a http://www.ligrila.com y ponte en contacto. Saludos


Confused Markdown and RST syntax in commit message again :(


Thanks! By the way, github is a bit dumb parsing messages. If instead of

#2113, 2116 and 2117

you say

#2113, #2116 and #2117 then everything is nicely linked


We should get v1beta3 from `com.google.datastore.v1beta3.client.DatastoreFactory.VERSION`


Well it seems to work on librepup-6021 so leave it at that for now.


sure, but taking doxygen as default leads to extraction of alternative docstyles supported from it, but not from the programming language^^


false


This problem is actually due to the use of the CSS `transform` attribute to center things vertically. For some reason, transforms only change how elements are displayed and what their bounding boxes are for the purpose of the browser's built-in hit detection, but leave coordinates untranslated when any JS wants to look at them, resulting in the fun stuff we see here. Looking into a better way to do vertical centering, because this is just ridiculous.


Everything ok


Упражнение 6.1


I don't quite follow, I cannot avoid having a copy version of the constructor so should I do the following? not sure that it is much of an improvement.

```diff
diff --git a/src/files/file_path.cc b/src/files/file_path.cc
index d0f7289..875e375 100644
--- a/src/files/file_path.cc
+++ b/src/files/file_path.cc
@@ -5,6 +5,7 @@
 #include "files/file_path.h"

 #include <ostream>
+#include <utility>

 #include "base/basictypes.h"
 #include "base/logging.h"
@@ -39,7 +40,11 @@ FilePath::FilePath() {
 FilePath::FilePath(const FilePath& that) : path_(that.path_) {
 }

-FilePath::FilePath(const StringType& path) : path_(path) {
+FilePath::FilePath(const StringType& path) {
+  FilePath(std::string(path));
+}
+
+FilePath::FilePath(StringType&& path) : path_(std::move(path)) {
   StringType::size_type nul_pos = path_.find(kStringTerminator);
   if (nul_pos != StringType::npos)
     path_.erase(nul_pos, StringType::npos);
diff --git a/src/files/file_path.h b/src/files/file_path.h
index 43bf543..1ad94e5 100644
--- a/src/files/file_path.h
+++ b/src/files/file_path.h
@@ -137,6 +137,7 @@ class FilePath {
   FilePath();
   FilePath(const FilePath& that);
   explicit FilePath(const StringType& path);
+  explicit FilePath(StringType&& path);
   ~FilePath();
   FilePath& operator=(const FilePath& that);

```


So what?


> I didn't have chance to verify that converters above actually work with PHP

I didn't tested it either. It is just a reference documentation to workaround server-side incompatibilities based in PHP docs and Tomcat's source code.

> In PHP, use setrawcookie() instead of setcookie():

I will add that


this can obscure a useful error message i.e.:

```
2015-10-02 16:21:37.164752 7fe8ea0ee900  0 librados: client.admin authentication error (22) Invalid argument
rbd: couldn't connect to the cluster!
<usage info follows>
```

I'd leave out the help


You can do the emoji's, you're already a guru!


I am not doing anything regarding the blue links yet.


[French punctuation](http://french.about.com/library/writing/bl-punctuation.htm) has a space before exclamation marks.


These unfortunate two lines have been the source of much agony :(


Does this global keyword belong here?


And this one?


Ah i see, `server_extend` does not take into account the possibility that the chef node and gluster peer have 2 different names

the `peer` variable [there](https://github.com/shortdudey123/chef-gluster/blob/867b20190cbe891a5c59d01ddb20d573c08873f4/recipes/server_extend.rb#L23) needs the same logic as here


The event is noticed automatically by `iron-icon`s whose iconset hasn't loaded yet.

We could document it, though we might want to document it on iron-icon actually, as I'd say it's more part of iron-icon's contract that iconsets should fire this event on window if they're intended to load asynchronously.


Changing this doesn't affect anything after the `=` for a given param/value. I couldn't think of any reason to make the param-parsing regex multiline, so I took it out.


Hmm this one isn't equivalent, it would have to be `observer.sendError`, but then you probably need to specify the type of `onError`, so maybe just leave this one the same?


This reverts [Make minimum cyberfox version for CyberCTR 1.4.1 42.0](https://github.com/InternalError503/CyberCTR/commit/682327234a2129c4b1d43bbad1633a508b088286)


Included the WEID.txt file. All seems good.


Thanks guys for the comments. Done in the https://github.com/UniversalMediaServer/UniversalMediaServer/commit/5d1c818f300f2d4acf3e61a546d6096926a04a05


Oops, you probably didn't mean to call them :) `let onBothCompleted = observed.sendCompleted`


I don't think that you caught messages that are not on 3.3 (like EFK2 lib) but just in case it's better to redo the grep from 3.3 branch.


Is this necessary?


yea mayb3


The first section still needs some work, but it handles multi-line titles just fine:
```php
// var_dump output for the multiline title in array[0]:
array (size=4)
  0 => string ' this is a multiline title:
look
how
many
lines
there
are
in
it
' (length=73)
  1 => string ' ' (length=1)
  2 => string 'this is a multiline title:
look
how
many
lines
there
are
in
it' (length=70)
  3 => string '
' (length=2)
```


Nice. :+1: I feel like going there now to chat with the NPCs, just to see the results.


@miigotu I submitted PR #2698 for the Episode Status Management issue.


Whats going on with this commit that it's not working?


I found the `font-variant` property to be highly confusing. I'm curious the thought process behind landing on just these 3 keywords. What about the other possible values? How will you know if it's a font variant or not?


Still need to omit "Of an app store URL" (redundant - see end of sentence).


`font-stretch` can also be `inherit`, which makes me wonder why you've added `inherit` to the other keywords, but not this one. I'm not suggesting you add it. Just pointing out the inconsistency.


Is the script on Transfix these days? I presume after I gave up control of it (and the original GitHub source was removed) it was removed, but I don't know it it was re-added? If not, and I appreciate this would probably be an issue for @BigNoid, would it be worth a request for it be re-added...? (That would at least mean only the English strings file would need to be updated...)


https://github.com/coreos/rkt/blob/master/pkg/tar/chroot.go#L93 ?


You can also inherit line height.


Well, TIL. I figured it was just a personal preference of The_Rey so I went ahead and changed it. I'll change it back.


"Use local docs" is coming out.


@VinceVal  hacky fix. =P
@OmgImAlexis any of those small tasks you can think of that we can delegate?


@jeremybernstein
Jeremy sorry to contact you this way but the option to open an issue for the TCEC version
of cutechess under
https://github.com/jeremybernstein/cutechess
is missing.  Could you please take a look at
https://github.com/cutechess/cutechess/issues/22
https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/gHBayWju9z4
Thank you!


Awesome Thanks!


fixed in d28c444


@miigotu agreed. :) I think we're working around a potential bug in Chrome though. The issue doesn't happen on FF/IE and it doesn't really make sense. I was able to narrow it down to colspan being in there though so I figured making the table "correct' (from a column count standpoint) is probably the best option.


One question, should we have Wl,-rpath,$(BOOST_DIR),--enable-new-dtags ?????


Niiicceeee.


git commit --amend --author="savoca <adeddo27@gmail.com>"


In the meantime you can use import prefixes, no progress on converting observable tk reflect able but it shouldn't be tk difficult.


![jag 1 151002](https://cloud.githubusercontent.com/assets/4010839/10260070/a4e67cbe-6970-11e5-8305-15d2fd56f3e4.jpg)


This is simply me! :)


we'll use it the same way we do command line args:

```yaml
outputs:
  - elasticsearch
  - stdout
elasticsearch: 127.0.0.1
```


As shown above, the '31's are invalid days of dates (old copy).
The new copy has valid '30' day of dates.

I have just tested, the "show week" function only shows valid dates given the starting date.


This does not compile under Qt 4:

    /usr/include/qt4/QtCore/qabstractitemmodel.h: In member function ‘virtual bool KviTalListWidget::eventFilter(QObject*, QEvent*)’:
    /usr/include/qt4/QtCore/qabstractitemmodel.h:235:10: error: ‘void QAbstractItemModel::layoutChanged()’ is protected
         void layoutChanged();
              ^
    /home/wodim/KVIrc-qt4/src/kvilib/tal/KviTalListWidget.cpp:58:32: error: within this context
        emit model()->layoutChanged();
                                    ^

(No idea of why)


This looks rather minging - it's checking to see that if the input line of text has an <a anywhere preceding a link, don't linkify anything in the line?  Why bother checking that for every match?  Surely the right solution is just not to linkify any HTML content at all, rather than be haphazard.  This will also not help us if the HTML contains a URL in an attribute for something other than an A tag.

I'd much rather that we enhanced matrix-react-sdk and the new matrix-react-console rather than investing *any* time in the obsolete angular SDK and console.

That said, this project is dead, and given this doesn't make the problem any worse (even though it's not a proper fix), it seems a bit pointless to block the commit.


> Why is this required at all?
--force is required since otherwise the test would refuse to run with the following error:
"Exit reason: Appears that we are running in linux with a version 2 build of mono. This is generally not a good idea.We recommend running with 3.0 or higher (3.2 especially). If you really want to run with this version of mono use --force to override this error"
and that is on Mono 4.*.*

> Can't we just remove this script?
Yes, though personally I think it may be a good reference of how to run the test, and could come in handy if one wanted to run the benchmark outside of compare. Other than that though, I don't think there is any reason to have it around.


You need to add the permission check to the index action as well


I know it  was some time ago, but @i2amroy how did you manage to fix that error?
I'm getting the exact same thing and I can't figure it out.


showWeek() previous did not care if the starting date provided by the user was valid or not.

checkDate(String date) is now used in showWeek(), so every date related to showWeek() will be valid.


@tobscure As @franzliedke commented in #510 this link should be going to user documentation on how to edit the custom CSS. I'm not sure a link to the theming docs for developers would be very useful to users.


Please look here when free:
https://github.com/mdht/mdht/issues/89
This issue is bigger than stated above.


NOTE: As of 2 October 2015, I confirm that this is the latest version of "Flex.java".
Thank you.


why this change?


The same can be done in the opposite direction with %

LIKE "Computer%" displays everything starting with Computer
LIKE "%Science" displays everything ending with Science


NOTE: As of 2 October 2015, I confirm that this is the latest version of "FlexWindow.java".
Thank you.


Yes, I forgot this commit, ASAP I'll make the pull request


postgresql is the default


寿司じゃないｗ


Gracias por las correcciones : )


LIKE "%RR%" displays anything with an RR in it


Crash on zero-row tables :-p


no warning?  getting kill suddenly would be kind of surprising for most people.


NOTE: As of 2 October 2015, I confirm that this is the latest version of "Task.java".

As of 2 October 2015:
1) The latest "Flex.java" is in the latest commit "26d86f4" on 20 September 2015.

2) The latest "FlexWindow.java" is in the second latest (2nd latest) commit "a38fd1a" on 20 September 2015.

3) The latest "Task.java" is in the earliest commit (the ninth latest (the 9th latest)) commit "fb9556b" on 20 September 2015.

Thank you.


What  do you think?

		// copy the document
        $documentId = $file['id'];
        $obj = new \C_Document();
        $document = $obj->retrieve_action("", $documentId, true, true, true);
		//if(file_exists($GLOBALS['OE_SITE_DIR'] ."/documents/".$pid."/".basename($file['url']))){
        if($document){
			//$sour = $GLOBALS['OE_SITE_DIR'] ."/documents/".$pid."/".basename($file['url']);
			$pos = strpos(substr($file['url'], -5), '.');
			// check if has an extension or find it from the mimetype
			if ($pos === false)
				$file['url'] = $file['url'].get_extension($file['mimetype']);
			$dest = $tmp."/".$pid."/".$path."/".basename($file['url']);
            file_put_contents($dest,$document);
		}
		else {
				echo xlt("Can't find file!")."<br />";
		}


Looks good.

Did it work?


@bicknellr2 the new event needs to be documented so it gets picked up in the docs. Can you please send a followup PR. See examples at the bottom of the file: https://github.com/bicknellr2/iron-overlay-behavior/blob/6816b05f096bd967ace93cf60363c6691e3b2ed9/iron-overlay-behavior.html#L431-L440


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.497.5271afc07e655da60d2184b7ac8043c377afe639](http://build.palaso.org/viewLog.html?buildId=63991&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3357, ignored: 134 Build time: 00:10:25


@10se1ucgo Not sure why this keeps happening unless you're changing it for testing purposes :smile:  I don't know what IDE you use, but I use MS Visual Studio with PTVS and I have two shortcuts on my desktop... The regular one, and one set to run as administrator. When I run MSVS as admin, my program gets interpreted as admin too for testing purposes.


@alexeagle just `"moduleResolution": "classic"` was, as it turns out, sufficient to correct the editor experience on my end... but the tsc command line tool was less okay... I had to run the compiler from the `modules` directory (or above) with `-p angular2` to build the desired project... And I'm not actually sure why - I should be able to just run tsc inside the `angular2` module directory itself, but for some reason that yields masses of module lookup errors. I'll have to look into it...


Yes yes it did much to my surprise


Sure, no problem.

On Wednesday, September 30, 2015, Christoph Pojer <notifications@github.com>
wrote:

> cc @fabiomcosta <https://github.com/fabiomcosta>
>
> I had to revert your commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/facebook/jest/commit/aee5ee56501bae021d758c51e6e78c10822899e1#commitcomment-13519175>
> .
>

--
--
Fabio Miranda Costa
*twitter:* @fabiomiranda
*github:* fabiomcosta


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.519.5271afc07e655da60d2184b7ac8043c377afe639](http://build.palaso.org/viewLog.html?buildId=63993&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:11:08


Also use use convert_safe_file_dir_name() here:
``````
$dest = $tmp."/".$pid."/".$path."/".convert_safe_file_dir_name(basename($file['url']));


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.520.50c2db794dfd1923f8a64b3b54d4c942c803c99c](http://build.palaso.org/viewLog.html?buildId=63994&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:11:28


Yes sir I did


you're the one going functional, might aswell use everything you need.
I'd like to learn to use Option


Also check to see if $dest already exists. If so, then prepend an increment to the filename until it works. for example:
``````
try
$tmp."/".$pid."/".$path."/"."1_".convert_safe_file_dir_name(basename($file['url']))
then
$tmp."/".$pid."/".$path."/"."2_".convert_safe_file_dir_name(basename($file['url']))
then
$tmp."/".$pid."/".$path."/"."3_".convert_safe_file_dir_name(basename($file['url']))
and so on until get to a file that does not exist.


@jython234 And the version number please?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.498.50c2db794dfd1923f8a64b3b54d4c942c803c99c](http://build.palaso.org/viewLog.html?buildId=63992&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3357, ignored: 134 Build time: 00:06:52


Janice? that plan you with this? :D


that's why you use uilabel instead.


second thoughts :or can we ensure that thet touch/pointer identifiers are being reused ? I mean if they are supposed to be unique, and therefore I would suppose incremental, the map object could growth indefinitely.

I browsed both MSDN and iOS documentation, but did not see anything about this.


Needed for generators to work (generators need a runtime library, "regenerator", which isn't included in the built main.dist.js otherwise)


oh! I've not seen this syntax for watchify/browserify before...


removed `;` mapping because i realized that you could use `;` to move after a `t` or `f` movement


Are you planning to make a new release with the 2.0 API, and to update the PerchRTC Podfile with it?


just so y'all don't make the same mistake, this should read

    class DriverOverIntersection : public Command {


This should've stayed singular, cause it's only creating one new note, not multiple new notes


以前からの部分ですが、version が 0x06 に固定になってしまっております。
group_desc_request も同様です。


+1 Makes sense.

Changed as requested:
https://github.com/gramps-project/gramps/commit/f416756809707f00dd427099db3bc3af0a3034b5


Testing purposes.

I don't really use an IDE, just Sublime


New parameters to take compress


Is there an expectation that `calc` would NOT be lowercase? Because that's not even valid CSS.


Once above is complete, then this thing should be ready to get into the codebase :)


Will this blow up

        if(file_exists($dest} {
            $x = 1;
          do {
           $dest = ."/".$pid."/".$path."/". $x ."_".convert_safe_file_dir_name(basename($file['url']));
            $x++;
          } while (file_exists($dest);


OK, let me try revising it. (I'm surprised that you object to the use of regexes, since much of the existing code relies heavily on them. But let me see if I can come up with something more readable.)


LUL


Regexes are used when necessary-- method declarations are [ambiguous](http://calculist.blogspot.com/2009/02/c-typedef-parsing-problem.html) and therefore we resort to existing solutions. Method params, on the other hand, especially ones that work for Ruby methods, have a pretty strict syntax and should be easy to parse.

In most cases we move away from regexes-- we did this from the legacy Ruby parse and the legacy C parser. Both are much less reliant on regexes than they were prior.


You mean that these members are not available in both python2 and python3 and because of the implicit initialization it's not necessary to check the python version?

I checked it, and base on what I found: yes, you mean,  :) ( at least for `tp_compare` which `renamed` twice over time,  but what about ob_size? I'm just curious.)


请问出现400001错误，为什么不用接口更新access_token再重试而是直接重试呢？


How do I get a document with the same name in to the documents section?


mysql note


See this module: http://haddock.stackage.org/nightly-2015-10-01/http-conduit-2.1.8/Network-HTTP-Client-Conduit.html


How's this? https://github.com/yesodweb/yesod-scaffold/pull/107


so this adds rel="nofollow" in the DOM with JS.

This is kind of working for Google which seems to index the JS content, but not in cache.
https://www.google.fr/search?hl=en&q=issues+google+site:webcompat.com/issues&gws_rd=cr&ei=6_sNVsTLJcW7aYDHi_AM

Empty HTML shell:
* This is not working in Yahoo! search https://search.yahoo.com/search?n=10&ei=UTF-8&va_vt=any&vo_vt=any&ve_vt=any&vp_vt=any&vf=all&vm=i&fl=0&fr=sfp&p=amazon&vs=webcompat.com%2Fissues
* This is not working with DDG.  https://duckduckgo.com/?q=issues+google+site%3Awebcompat.com%2Fissues&t=ffab

It's not related specifically to rel="nofollow", but just realized that it is kind of related. :/


Ok, so this does the build matrix across the 4.x series of the notebook and ipywidgets as well as the separate 3.x series too.


Action taken!


Your method name should be "conributionreceipts_search" instead of "contributionreceipts_search_search" - just tested and it works.


Ta faltando colocar a parte do: "Caso o número a ser convertido não seja válido, o registrador v1 retornará o valor 1". Não entendi bem essa parte.


TeamCity Bender :: Deploy [Build 3.0.27.0](http://build.mikeobrien.net/viewLog.html?buildId=2473&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3264, ignored: 5 Build time: 00:01:19


Maybe the code did not capture exactly what I wanted,  the idea is to pass 'not available' if the rental is booked...


@smo-key This is not something we should be removing. If we need to change the FILE_DIR variable in MonkeyC, so be it, but this is the only safe (and supported) place for apps to write on the OS. If we need to specify in the docs for this method that it's relative to the external file dir, so be it, but removing this here makes things much more of a pain


@Oksisane Well, that's a nice thought and all, but looking at the getExternalStorageDirectory() creates a file path like `/data/external/0/data/external/0/test.txt`. Apparently `File` prepends the external storage automatically, even if it's already there. Weird?


@Oksisane btw this exact line was the culprit of today's issue, so it's worth investigating


Hi Humans added


Any reason? If I need to upload my script first, then tell me ... it is essentially done (I'm just adding functionality). Or perhaps you are just being mad at me for some reason . . . . . .


@smo-key That's because your FILE_DIR variable in MonkeyC also has the line  Environment.getExternalStorageDirectory()


I don't think the operating system will allow the writing of the file with the same name to happen.


@Oksisane Yeah, whoops. Pushing (real) fix.


yes it is, please file a PR for it. Thank you.


This only downloads from Imgur.
Downloads with the original filename.
Album support is broken, trying to think what to do there.
I worked past a few errors that were frequent (404, etc) by just having bad URLs be ignored in the functions extract_imgur_album_urls and download_from_url


No; this converts the result Z_BUF_ERROR (meaning the output buffer is full) to Z_OK, because it shouldn't be treated as an error.


You probably shouldn't check this in. It appears to be a valid API key.


this is a duplicate


1. I should add license information.
2. Should I add validation for arguments?
3. Is setAlignment() too much?


best to keep 12 for now, lots and lots of people are still on it :)


No, but you're right that this line doesn't make any sense! Pretty sure it should be

`status < Z_OK || _status >= Z_OK`

That is, the first `_` shouldn't be there.


@gregwebs thanks, that's useful! The definition didn't really seem clear what the point was.


Tested logging and MonkeyC using phones only. All tests pass.
Next: test with robot. For real this time.


personally i would just check the `fields.text` for nil and return if it is


you can call `scipy.linalg.lapack.dpotri` here instead of `dpotrs(L, I)`, http://www.netlib.org/lapack/double/dpotri.f


I'd rather not call the value it failed to find "unsupported", since this is a specific term to denote types unsupported by the CPU (starting from i386 IIRC). If we ever come to this line, we should return something like "BUG! Unknown type %1" to see it immediately, or even just `assert(0);`.


Thanks. Something like this should be good.


the function should do what the function name say and not check for the `all_unlocked` setting
there is already a check for `all_unlocked` in `locks.is_locked`


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/0b0c67dcc4b92074250f0866a6e92da89d3578c6


merging basho/riak_kv/feature/fix/RTS-354 = 0b0c67dc into borshop-integration-1202-feature/fix/RTS-354


basho/riak_kv/feature/fix/RTS-354 = 0b0c67dc merged ok, testing candidate = 65e8ee9e


heh.  Well now that you're familiar with the code you're certainly welcome to submit PR for other issues :)

One user requested cloc have the ability to parse .gitignore files and skip files accordingly.  That one's been idling in my brain for months.


wqwq lol :v:


it get way too noisy that it error https://github.com/kaadmy/pixture/issues/3,
it need to use an area base sounding like the current default fire or a player base global step


Hi @ishbir, just wondering what the reason for this change is? I was hoping to avoid pointer receivers and all the issues that come with them by keeping the Mailstore interface fully immutable.

If anything is modified in a Mailstore interface, it should return a copy of the new struct rather than modify the struct passed to it. For example, the [SetBody](https://github.com/jordwest/imap-server/commit/00b5ba29fe50beedc13d03ee0a60e4726a4cac0c#diff-7c15ce31fa6ef919c5ac465e32916b74L427) method. I know this means more effort when using the Mailstore, but it keeps issues with mutability under control.


Q：中文乱码
A：使用markdown编辑文档


Could you explain what URL issues this commit fixes? I can't really see a difference in the URLs before and after.


Formatting


Do we really want something like A#B to equal ADB.
```
    _
_|
```
```
  _
 | |
 | |_
_|
```


See https://github.com/kvirc/KVIrc/pull/1647#issuecomment-144911850

#1645, #1314, and #1309 are not related to this at all, if I had to hazard a guess based on my encounters with icon corruption bugs, it would be in the theme changing (icon loading / reloading routines) cause that when it mostly happens.


Looks good. Im liking the footer.


siro this patch also fixes Age of Conan triggering a kernel crash fyi


My comment here is maybe the effort we go to on this one is a little too much. Let's say that if POST & URL Vars are both present (key only) choose the POST variable.


I updated the introduction


Excellent catch, I'll get it updated.


You should squash this so it doesn't appear in the history


Why was this deleted?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2439


wildcards 判定が逆！
正：if ~self.wildcards & wc:


Can you add the types here also so that they are uniform in case they are in a widget?
https://github.com/WordPress/twentysixteen/blob/master/style.css#L3145


What worries me is this part of the assignment "use an array to validate form data provided by a user (look into the in_array function). If the user inputs values in the allowed array, then good, otherwise, produce an error message". I sometimes have a hard time understanding what exactly is required. It sounds like all the values (including the don't check me box), are suppose to be stored in an array. Then we use inArray to validate the values in that array. Like we have to check if the array holds a value for the badTouch input. Again, I can be totally over thinking this, but if I am not, we have to work on that array.


It was observed, but not really a problem.   An engineer noticed the warnings in our logs and when tested found that the emails were being set w/o a problem -- even when we commented out the code that stripped the vertical whitespace.   So, was not clear if the code should allow \n\x20.

Thanks for the tip on MIME-Header.    Oh, I just tracked down the code and it is using Email::MIME, but not using "header_str" but just "header" along with MIME-Header.   That's an easy fix.

Sorry to bother you.


I have the same diff in my local tree against OpenBSD so please also include OpenBSD.
+#if defined(__NetBSD__) || defined(__OpenBSD__)
+#include <util.h>
+#endif
Cheers


lol


Above code looks good. Possibilties for identical file name on write:
1. Name is "Patient File" and another with "Paitent_File"
2. Using couchdb where it's ok to files with same name
3. Import it through Miscellaneous->Documents and then migrate it to a patient that contains document with same file name.
4. Change it from a patient to another patient.


Date is incorrect. Change to 2015-10-01 for now.


They are supposed to be numbered 1-*n*. Where *n* is the number of unique touch points supported by the hardware. I don't think it ever goes beyond 10, because most people only have 10 fingers...


Well, if it's zero length it'll fail when it gets parsed as JSON. So one way or another it's an error.


Will need to change this to Final build 220 and the link to v220


The time losses are back...on my machine.  I have 249 time losses out of 8421 games.  Despite this, it is still a substantial Elo gain.  7-thread games.  The 1-thread games also gained.  As did the 4-thread games on my old machine.  All were roughly the same gain relative to Komodo 9.2 on the same number of threads.


That texture is public domain btw


return (getTimeToString(hour).equals("") ? "" : getTimeToString(hour) + ":") + (getTimeToString(minute).equals("") ? "" : getTimeToString(minute) + ":") + getTimeToString(second);


The optimization use a cutoff just after stabilization optimization. So the local_elemwise_alloc optimization didn't ran on the compiled function. This explain the rise in the number of node. The new version of the code need that optimization.


dqqwqw


yes, it is 1 or a list of theano variable. If one, the mul is useless and as we are past the canonizer, it won't get removed with the current optimization we have.


how did this break mips anal? https://travis-ci.org/radare/radare2/jobs/83177279


11 threads would have been a better choice.  Then those 23 available core machine could run 2 games simultaneously, and the one with 11 threads could run 1.


Hi, I am following this from https://github.com/zfsonlinux/zfs/pull/3800#issuecomment-144896916 and wanted to comment here since it's not relevant to the linked pull request:

Obviously this is quite clever, reading the `zpool.cache` file from the root pool in the initramfs, but it seems to me that by this point in the code, you already have the root pool imported and `/` is mounted so the initramfs's job is done.  Why even keep going when the userspace init system can and should, in my opinion, take over and import the rest of the pools from the cache without all the copying?

Not saying this is wrong, I'm just curious what problem reading the cache from the root pool inside the initramfs tries to solve.


Copy that.


Could you format your code please?


Can you use mockito version 2?
Latest version available is 2.0.31-beta


@berkus done


This shouldn't be needed with mockito v2


I'm pretty sure you mean `isEqualTo()`, not `as()`


This does nothing


It was replaced by
```
if collisions:
    collisions[0].take_damage(self.damage_)
```
Ensures only one thing is taken damage from this projectile.


when i edited this files i am no more able to run gemfile and gemfile.lock it prompts to update the bundle and while updating the bundle it again installs the nokogiri 1.6.6 previous version and it also says : Nokogiri is built with the packaged libraries: libxml2-2.9.2, libxslt-1.1.28, zlib-1.2.8, libiconv-1
.14. so how should i update those package libraries too ?? any kind of help for starting rails server will be appreciated.... thanks


Better use `assertThat(elements).hasSize(3);`


Better use `assertThat(elements).isEmpty()`


To PRO.


Hahahh


Dude, I did not know you spelled it wrong. Mine's ok.


open another pr!


What? Why 404 returns 200 status codes?


Please run `mvn license:format` to add a header to the file


Thanks! I'll add it with the upcoming release in a few days.


Looks good!


This is a GOOD question... perhaps post this to the class discussion for clarification (not the group forum but the class one where Keith will see it, hopefully, sooner). I believe my original thinking was that Keith's note on 'in_array()' was simply an example of how to compare entered data to a manually coded associative array to return values based on key entries (ie: users would enter the abbreviation CA and the form returns California).
Our checkbox data are an array that is returned to the user, but I think your concern is particularly valid because we've done no substantial work by returning the box values that have been precoded in the html.
- We could address this by creating an associative array with key/value pairs like vmd=>Visual Media Design, cs=>Computer Science, html=>HTML5, py=>Python.... and returning to the form box values and recoding them as 'vmd' 'cs' 'html' 'py'. Then we would use our array to match appropriate 'long-form' values.
(SO glad we're having a conversation like this now and not Sunday night!)


What does this loop do?


u need to fix offstring -1  , there are this big error and u work on small stuff :(


这个文件完全可以自动生成


>not even latex


maybe with pointerEvent, but not with iOS, here is a log of the touch identifier I had when playing with the whack-a-mole on my iPhone (just a console log with `changedTouches[t].id`) :
````
[Log] 193888482 (melonJS.js, line 17658, x2)
[Log] 193888483 (melonJS.js, line 17658)
[Log] 193888483 (melonJS.js, line 17658)
[Log] 193888483 (melonJS.js, line 17658)
[Log] 193888484 (melonJS.js, line 17658, x2)
[Log] 193888485 (melonJS.js, line 17658, x2)
[Log] 193888486 (melonJS.js, line 17658, x2)
[Log] 193888487 (melonJS.js, line 17658)
[Log] 193888487 (melonJS.js, line 17658)
[Log] 193888487 (melonJS.js, line 17658)
[Log] 193888488 (melonJS.js, line 17658, x2)
[Log] 193888490 (melonJS.js, line 17658, x2)
[Log] 193888492 (melonJS.js, line 17658, x2)
[Log] 193888493 (melonJS.js, line 17658)
[Log] 193888493 (melonJS.js, line 17658)
[Log] 193888493 (melonJS.js, line 17658)
[Log] 193888494 (melonJS.js, line 17658, x2)
[Log] 193888495 (melonJS.js, line 17658, x2)
[Log] 193888496 (melonJS.js, line 17658, x2)
[Log] 193888497 (melonJS.js, line 17658)
[Log] 193888497 (melonJS.js, line 17658, x2)
[Log] 193888497 (melonJS.js, line 17658)
[Log] 193888498 (melonJS.js, line 17658)
[Log] 193888499 (melonJS.js, line 17658)
[Log] 193888498 (melonJS.js, line 17658)
[Log] 193888499 (melonJS.js, line 17658)
[Log] 193888500 (melonJS.js, line 17658)
[Log] 193888500 (melonJS.js, line 17658)
[Log] 193888500 (melonJS.js, line 17658)
[Log] 193888501 (melonJS.js, line 17658)
[Log] 193888502 (melonJS.js, line 17658)
[Log] 193888501 (melonJS.js, line 17658)
[Log] 193888502 (melonJS.js, line 17658)
[Log] 193888503 (melonJS.js, line 17658, x2)
[Log] 193888504 (melonJS.js, line 17658)
[Log] 193888505 (melonJS.js, line 17658)
[Log] 193888504 (melonJS.js, line 17658)
[Log] 193888505 (melonJS.js, line 17658)
[Log] 193888506 (melonJS.js, line 17658, x2)
[Log] 193888507 (melonJS.js, line 17658)
[Log] 193888508 (melonJS.js, line 17658)
[Log] 193888507 (melonJS.js, line 17658)
[Log] 193888508 (melonJS.js, line 17658)
[Log] 193888509 (melonJS.js, line 17658, x2)
[Log] 193888510 (melonJS.js, line 17658, x2)
[Log] 193888511 (melonJS.js, line 17658, x2)
[Log] 193888512 (melonJS.js, line 17658, x2)
[Log] 193888513 (melonJS.js, line 17658, x2)
[Log] 193888514 (melonJS.js, line 17658)
[Log] 193888514 (melonJS.js, line 17658)
[Log] 193888514 (melonJS.js, line 17658)
[Log] 193888515 (melonJS.js, line 17658, x2)
[Log] 193888516 (melonJS.js, line 17658)
[Log] 193888516 (melonJS.js, line 17658)
[Log] 193888516 (melonJS.js, line 17658)
[Log] 193888517 (melonJS.js, line 17658, x2)
[Log] 193888518 (melonJS.js, line 17658)
[Log] 193888518 (melonJS.js, line 17658)
[Log] 193888518 (melonJS.js, line 17658)
[Log] 193888519 (melonJS.js, line 17658, x2)
[Log] 193888520 (melonJS.js, line 17658)
[Log] 193888520 (melonJS.js, line 17658)
[Log] 193888520 (melonJS.js, line 17658)
[Log] 193888521 (melonJS.js, line 17658, x2)
[Log] 193888522 (melonJS.js, line 17658)
[Log] 193888522 (melonJS.js, line 17658)
[Log] 193888522 (melonJS.js, line 17658)
[Log] 193888522 (melonJS.js, line 17658)
[Log] 193888523 (melonJS.js, line 17658, x2)
[Log] 193888524 (melonJS.js, line 17658, x2)
[Log] 193888525 (melonJS.js, line 17658)
[Log] 193888525 (melonJS.js, line 17658)
[Log] 193888525 (melonJS.js, line 17658)
[Log] 193888526 (melonJS.js, line 17658, x2)
[Log] 193888527 (melonJS.js, line 17658, x2)
[Log] 193888528 (melonJS.js, line 17658, x2)
[Log] 193888529 (melonJS.js, line 17658)
[Log] 193888529 (melonJS.js, line 17658)
[Log] 193888529 (melonJS.js, line 17658)
[Log] 193888530 (melonJS.js, line 17658)
[Log] 193888530 (melonJS.js, line 17658)
[Log] 193888530 (melonJS.js, line 17658)
[Log] 193888531 (melonJS.js, line 17658)
[Log] 193888531 (melonJS.js, line 17658)
[Log] 193888531 (melonJS.js, line 17658)
[Log] 193888532 (melonJS.js, line 17658)
[Log] 193888533 (melonJS.js, line 17658)
[Log] 193888532 (melonJS.js, line 17658)
[Log] 193888533 (melonJS.js, line 17658)
[Log] 193888534 (melonJS.js, line 17658, x2)
[Log] 193888535 (melonJS.js, line 17658)
[Log] 193888536 (melonJS.js, line 17658)
[Log] 193888535 (melonJS.js, line 17658)
[Log] 193888536 (melonJS.js, line 17658)
[Log] 193888537 (melonJS.js, line 17658, x2)
[Log] 193888538 (melonJS.js, line 17658)
[Log] 193888539 (melonJS.js, line 17658)
[Log] 193888538 (melonJS.js, line 17658)
[Log] 193888539 (melonJS.js, line 17658)
[Log] 193888540 (melonJS.js, line 17658)
[Log] 193888540 (melonJS.js, line 17658)
[Log] 193888540 (melonJS.js, line 17658)
[Log] 193888541 (melonJS.js, line 17658, x2)
[Log] 193888542 (melonJS.js, line 17658, x2)
[Log] 193888543 (melonJS.js, line 17658, x2)
[Log] 193888544 (melonJS.js, line 17658, x2)
[Log] 193888545 (melonJS.js, line 17658, x2)
[Log] 193888546 (melonJS.js, line 17658, x2)
[Log] 193888547 (melonJS.js, line 17658, x2)
[Log] 193888548 (melonJS.js, line 17658, x2)
[Log] 193888549 (melonJS.js, line 17658, x2)
[Log] 193888550 (melonJS.js, line 17658, x2)
[Log] 193888551 (melonJS.js, line 17658, x2)
[Log] 193888552 (melonJS.js, line 17658, x2)
[Log] 193888553 (melonJS.js, line 17658, x2)
[Log] 193888554 (melonJS.js, line 17658, x2)
[Log] 193888555 (melonJS.js, line 17658, x2)
[Log] 193888556 (melonJS.js, line 17658, x2)
[Log] 193888557 (melonJS.js, line 17658, x2)
[Log] 193888558 (melonJS.js, line 17658, x2)
[Log] 193888559 (melonJS.js, line 17658, x2)
[Log] 193888560 (melonJS.js, line 17658, x2)
[Log] 193888561 (melonJS.js, line 17658, x2)
[Log] 193888562 (melonJS.js, line 17658)
[Log] 193888562 (melonJS.js, line 17658)
[Log] 193888562 (melonJS.js, line 17658)
[Log] 193888563 (melonJS.js, line 17658, x2)
[Log] 193888564 (melonJS.js, line 17658, x2)
[Log] 193888565 (melonJS.js, line 17658, x2)
[Log] 193888566 (melonJS.js, line 17658, x2)
[Log] 193888567 (melonJS.js, line 17658, x2)
[Log] 193888568 (melonJS.js, line 17658, x2)
[Log] 193888569 (melonJS.js, line 17658, x2)
[Log] 193888570 (melonJS.js, line 17658, x2)
[Log] 193888571 (melonJS.js, line 17658)
[Log] 193888572 (melonJS.js, line 17658, x2)
[Log] 193888571 (melonJS.js, line 17658)
[Log] 193888573 (melonJS.js, line 17658, x2)
[Log] 193888574 (melonJS.js, line 17658, x2)
[Log] 193888575 (melonJS.js, line 17658)
[Log] 193888576 (melonJS.js, line 17658)
[Log] 193888575 (melonJS.js, line 17658)
[Log] 193888576 (melonJS.js, line 17658)
[Log] 193888577 (melonJS.js, line 17658, x2)
[Log] 193888579 (melonJS.js, line 17658, x2)
[Log] 193888580 (melonJS.js, line 17658, x2)
[Log] 193888581 (melonJS.js, line 17658, x2)
````


i am disapoint`


Oh, thank you!


This is where I'd drop an associative array declaration. Name it something different, but semantically relevant.
Then generate a new array from the matching values, using in_array(), change the '$department' call to return position 0 of that new array, change the 2 instances of '$topics' to the new array name, and tweak that foreach loop a little further... possibly moving to a for loop and starting our iterator at position 1.


只要加个“js”就可以让代码拥有相应的颜色


pushになにか意味がありますか？（わからなかった）


Shouldn't this be an empty string, since there isn't any partial named `part1` or `part2`?


:psyduck:


Yeah, you couldn't have said it any better. That totally makes sense. The part I feel bad the most was carelessly typing initial proj version when it was my final proj version. Man, thanks a lot for your patience, Bobby. I'll review your suggestions above and see what happens when I implement them and then move to next section of this tutorial on low level events.


from an API point of view, why not just take a `...` of funcs rather than a slice?  Callers get a bit simpler


Jeg er ikke uenig at vi trenger fallback da Koha faktisk ikke klager når $o mangler