Can this really happen? We call `setOpenExternalLinks(True)` which according to [the documentation](http://doc.qt.io/qt-5/qtextbrowser.html#openExternalLinks-prop) means:

> QTextBrowser should automatically open links to external sources using QDesktopServices::openUrl() instead of emitting the anchorClicked signal.


@danoli3 do we need this specific hash for anything? it's breaking in linux also we are still using 1.6.0 and linux recompiles poco in some cases to fix some compatibility issues so we should keep the version in sync with what we have in other platforms.


That is great !
thx .
^^


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6350](https://teamcity.mesosphere.io/viewLog.html?buildId=79530&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:04


`QFileInfo(self.tab.fileName)` instead of that monstrous construction :)


[Again][1], I think folks should just use `query` (although in this case we do support a singular `delete` azurestandard/api-spec#13, just not a `get` or `put`).

[1]: https://github.com/azurestandard/azure-angular-providers/commit/6993aa0e83251b78e7a23ed78450320a45cd752f#commitcomment-14567258


no, it's not. I'll review it. Thanks for spot it


If you use `%q` instead of `%s` you don't need the quotes.


FewDocFixes (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1965 of 1965 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)


We don't bump gem versions in feature branches - http://discuss.fre.ag/t/internal-gem-versioning/43/13?u=caius

Develop the branch & pin the SHA of it in the app's gemfile, then once it's provably working in the app, come back to this project, bump the gem version appropriately & update the app's Gemfile to depend on the version using semver.


What changelog?


Please also use camelCase: `current_file_info` → `currentFileInfo`.


We'd need a `GET` handler too to make this work on yarn.


знак указателя * требуют располагать ближе к char


My bad, sorry.


Is it CFS?


reproducible


Этот код дублируется, лучше его тоже вынести в отдельную функцию


добавить проверку на наличие строки


    currentFileDir = currentFileInfo.absoluteDir()
    fileToOpen = currentFileDir.filePath(link.url() + currentFileInfo.completeSuffix())

It will be also nice to not add any suffix if there is already a suffix in the link.


Remove this.


Ezee, I merge the midiMap here with a temp branch. This temp branch is based in master. I do the merge, no conflicts. This is good, but I'm looking in the code to comment out the "midi mapping stuff", but I can't see this code. So I'm afraid we are loosing work here.

You remeber the main changes we wrote in the "midiMap" stuff?


How do I pull this commit? I tried a "git checkout" and it couldn't find it.


It's plausible, but I think it would end up being more complex in this particular case.

Moving from goroutine-per-request to communicating goroutines, creating a new response channel per request, and having to specifically handle errors which happened off-goroutine, is going to be more complex for these one-off commands.

The mutex is pretty clearly and finely scoped.


Why is this code commented?


> or join the maintainer team and comment in the chat :stuck_out_tongue_winking_eye:.

Might actually happen as I answered Mike's mail (sent last Friday) in the affirmative, i.e. agreeing to join. :smirk:


Okay then I will ignore this because that's not something I would understand.  Thanks for clarifying.


I realized yesterday that some branches could be individual releases also .
That is pretty good because you could have Jamtaba 2 Standard and ...

_ Janmtaba 2 pro
_ Janmtaba 2 studio
_ Janmtaba 2 Radio

Well , a Jamtaba suite of products ...


Using some concepts form Linked lists, it can be done (let me show an example)


Okay then I agree it should not be there.  Thanks.


ImageInfo GetImageInfo(byte[] data);


Fixed https://github.com/ceylon/ceylon-sdk/commit/f798073d5158809a91c88fa7ea1af7cb8c2ad82f, thanks.


2 класса , нужно убрать один


в лого можно не ставить класс на ссылку и img


это не саб  это иконка открытия меню которая лежит в хедере - menu-toggle


название класса не очень подходящее, лучшее form-wrapper или field-wrapper


иванов это placeholder и далее по аналогии имя и т.д., value там где цифры даты


инпуту можно убрать обертку, будет общая обертка и внутри минус поле плюс


инпут можно не оборачивать


I forgot in the serie :+1:

_ Janmtaba 2 Frank Einstein edition
eh eh eh the monster edition ...


аналогично с тем что писал выше


text-block__p неправильно т.к. уже теги начинаешь указать это допустимо максимум с label input, вместо p лучше указать text? а весь блок назвать description или promo


не нужная обертка


escape html


This used to be string, now you’re passing an object. When you do such a change, you must also change `propTypes` (in Controls and WeekSwitcher). I’ll do it for you.


Correção: a função de genHVStables ao invés de readVideo


Suggested error message: "The distance function is only able to compute point-to-point, point-to-polygon and polygon-to-point distances."


**You remeber the main changes we wrote in the "midiMap" stuff?**
mmm... the most part i worked on was on processAudio , a little in the midimessage too .
But your model-view no


why did you use cucumber?


Oh, I didn't see you changed resnum to calm_or_loud, I rolled back and changed the code in leleleplayer :)
Thanks!


The function name implies the return order order to be *kubecontainer.RawPodStatus, *api.PodStatus. Either rename the function or change the return type.


使折叠按钮生效


Nice work!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6351](https://teamcity.mesosphere.io/viewLog.html?buildId=79531&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:10:15


Is there a reason we're making this a class variable? It seems like a good idea to maintain that two separate `MockRedis` objects represent two distinct Redis servers.


I see this is new, so I’m wondering why we need to run them on distinct ports if they’re running in separate VM instances.


I would replace this whole if statement, which seems to be dead code, with `assert(!rightChildType.isGeographyType())`.


s/interface/method


"O próximo nível de habilidade requer avanço para a mesma classe."


suggestion: add a TODO for `SyncPod` to remove some arguments.


@carvalho2707 é muito bom estares a ficar com as linhas do código que eu fiz :joy:


The “unsure” line can be removed. We’ve gotten codecov to deal with all our coverage info.


@MMajko See how I’ve simplified that function. I’m gonna squash it now.


"Ativado" com A maiúsculo.


And the following if statement that checks that rightChildType is a point is trivially true, so you can replace it with `assert(rightChildType.isPointType())`.


зачем убрали перенос на новую строку?


privileges*


also
updated level mesh renderers to cast two sided shadows
updated onPortalEnter to load level only if collided with player


lol


Again right here, this should always be true.


And this seems like it is dead code, and can be replaced by an assert.


Ah Trevor pointed me to this: https://github.com/nodejs/node/blob/master/src/node.js#L482-L504

I guess we're trying to make it have no domain, but that doesn't seem to change anything...


Ah yeah, I missed that. @RReverser: https://github.com/fkling/esprima_ast_explorer/releases


@FroMage @tombentley @gavinking
FYI: If you are going to work on MultipartFormData request in `ceylon.net.http.client` you might find some useful pieces here.


v_idの数を数えることは簡単ですね。v_loとv_hiからも情報は得られると思います。


we don't have this in s390.


I use it, to make it easier to the end user, in a way it can be understand in a easy way


I like this simplification.  Some of the logic in HSQL can still be simplified more.


**So I'm afraid we are loosing work here.**
Make a pull request of midiMap , there is the history of all commits .
?


we don't have this in s390


I don't sure that it good decision


midiMap merge with master.
I will delete the branch.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 39](https://teamcity.labkey.org/viewLog.html?buildId=287025&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:40


The cython documentation says this should be a "Py_ssize_t" which is a typedef to "ssize_t" where available. So, size_t is wrong by not being sized, but int may not be wide enough (even though it probably is wide enough in most cases)


Pode traduzir "Random" pra "Aleatório".


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6352](https://teamcity.mesosphere.io/viewLog.html?buildId=79533&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:11:32


Actually it depends. In the VM on CentOS it was 0755, on the Ubuntu - 0775.


wait .... https://github.com/elieserdejesus/JamTaba/commit/3b2921ab31f2632711cd9c9785eb069dc127ad11


Do I understand correctly that we don't sorted out busy daemons?


Becuase as I found out you should only mock objects that you tested before - like Dao objects. I originally tested the method with uncomented code and used mockito "when(trainerService.mayEnrollInTournament(setUpTrainer, tournament)).thenReturn(true)" but it does not in any way test the method corectness


don't have this


Actual version already covers this case. Anyway, thanks for your effort.


Must check


Thanks. This looks good.


Probably, default should be enforced in the code.


So, this maintains the comment that `core` needs to be run last (which I think should be rephrased to say something about running tests “up” the dependency graph), but it no longer runs the tests in that order. I didn’t see anything else that enforces the order (say, in build.sbt), so it seems like that’s just lost.


Suggestion: explicitly point out that this will be removed after rkt implements GetRawPodStatus.


"Retirada!" ficaria melhor.


Falta o acento do "está".


stiil i have problem on that :-1: and using the latest version.


Line 181 has what I think is a reference to an obsolete method: "(See TxQImpl::canBeHeld)".


Fine!


Fixes #38


I think we should use AddP instead.


or lay


"A configuração de recompensa está incorreta".


Why not just import address from cython.address?


# Escuela

![alt tag](https://github.com/pedro-alonsod/Escuela/issues/1)


Then delete the code. You already pushed it, so you can find it later if you need to.


don't leave markers.


I think that the actual version already covers this case. If not, can you please create an issue or a PR?

Thanks.


@saschazelzer Thanks for fixing these !!


Falta acento no "está".


test


Why is fastqc only run on Chara, are we worried that is chara is full, it could delay processing?


but it still does not shows the test results, why is that?


This needs to be hidden if the jobs are to be exportable/loadable or not, else the VPrinter system will need an overhaul and this creates a LOT more work than simply adding the appropriate stripProxy statement to the test.

I don't see too many users needing to edit/be-aware/change of this property anyway as it's an expert feature imo.

If there is a good reason to expose this to users then we can look into reverting this but when this is exposed it causes problems with things like exporting jobs which are used more by the users directly.


This doesn't technically require a docstring because the type the method is defined on is unexported. BTW, if we had any golint compliance issues, the CI builds would be failing.


Not sure how to read the comment on line 231: "... the queue _whos_ the `LastLedgerSequence`...".  What should _whos_ be?


If so I think it can leads to state where all wait biggest daemon.


Fixes #19


![screen shot 2015-11-23 at 11 36 31](https://cloud.githubusercontent.com/assets/5738349/11344752/c0b4316c-91d8-11e5-896e-51cb933193e1.png)
![screen shot 2015-11-23 at 11 37 08](https://cloud.githubusercontent.com/assets/5738349/11344753/c0b55e0c-91d8-11e5-8fc9-2c9bb978e608.png)
![screen shot 2015-11-23 at 11 37 54](https://cloud.githubusercontent.com/assets/5738349/11344754/c0b60b68-91d8-11e5-80b4-18e4795e753a.png)
![screen shot 2015-11-23 at 11 38 05](https://cloud.githubusercontent.com/assets/5738349/11344755/c0b627ec-91d8-11e5-82ca-8521f97a8ef8.png)


What does `coverageReport` do? I’ve already been looking at scoverage output in a browser, so I’m curious as to what this adds.

And I don’t think coverageAggregate buys us anything. I _think_ I tried to use this to avoid having to run the tests in order in the past, and it didn’t work. In my other PR, I added it so that we could give coveralls all of our coverage data across subprojects, but codecov handles aggregating the individual subproject results itself, so without coveralls we don’t need it.


```javascript
let platform = os.platform();
if ( platform == 'linux' || platform == 'darwin' ) ...
```

From [this](http://shapeshed.com/writing-cross-platform-node) site. Did _not_ test.


It's a habit I've been taught but I don't really now wheter it's useful here or not.
I'll investigate and may delete it eventually!


"espaço de baixo." fica melhor. "espaço abaixo" passa a impressão de ter um espaço só.


é _itoa  em "itoa (g_TotalImpacto, buffer, 10);"


essa funcao nao imprime pq ela retona um IO(), que eh uma acao que ainda precisa ser executada

uma implementacao correta exige vc colocar um monad dentro da funcao ( do ...  ), dessa forma:

 | fst(codigo!!posicao) == "}" && fst(codigo!!(posicao+1)) == "End" = unsafePerformIO $ do print "Interpreter has finished."; return True


renaming done!

And that's correct regarding web3.js/alethzero, they weren't in the original dot file either. Maybe we should redefine more specifically what this diagram represents (more the "core")?


No problem. I hope people can benefit from your repo as much as I do :)


В теге h1 не должно быть никаких других тегов


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6353](https://teamcity.mesosphere.io/viewLog.html?buildId=79537&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:08:43


Testing? That's a time of the past. No one tests anymore. lawl


No problem. I hope people can benefit from this list as much as I do :)


"Insira os itens que deseja comprar".


@saschazelzer Thanks !! We will verify linux builds before merging to development branch, in the future.


Why are these all added?


Yes, this doesn't work even when you start or restart openmrs, @WebListener will only work if this class is on the classpath at the time when the servlet context is getting built, because this is in the module it is not visible on the classpath


Was that reply sent at 17:37 (UK time)?


"Evento Promotor" ou "Evento do Promotor" fica melhor.


This is _technically_ true, golint won't fail...


Intent is to parse a dummy SNS like message and ensure that is processed by the moire server.


what is the reason for this?


please don't mangle indentation tabs


This is a markdown file.


because the pmmp version itself is outdated


Как можно записать это компактней, без двух вызовов Date.now()?


Let's call this DONE.    Please could you put together a Pull Request as-is?   Thanks!


"Agora você pode usar o título [{Auto_1}]". Tem que traduzir a ordem da frase também pra fazer sentido em português.


@Yajo


Inner Insulator is NOT touching anode and cathode, this must be fixed next time.


This must be changed: the 3 represents the length of the stack
So either the stack must be of fixed length or we must compute it


Из-за ошибки в коде библиотеки cookies. Она неправильно принимает число, т.к. ожидает его в секундах, а не в миллисекундах. Есть два варианта: привести к секундам, либо передать объект new Date(expDate)


CORS was needed to allow Angular to make the HTTP request and not fail on its pre-flight request.


Это зачем?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6354](https://teamcity.mesosphere.io/viewLog.html?buildId=79539&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 202 Build time: 00:08:32


@yujuhong Yeah, that's really not intuitive, in one of the following commits I have exchanged them, :) Thanks!


Good :+1:


Show file diff (and optionally RPMs) between two images or containers
Change to

Show differences between two container images, file diff or RPMS.


Yup. You're right.


Couldn't it be:
    del argv[argc:]
    for i in range(argc): argv[i] = str(argv_bytes[i].decode())


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6355](https://teamcity.mesosphere.io/viewLog.html?buildId=79540&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:39


Well it seems to work for me,

λ> maple "NewSLO:-RoundTripLO(Gaussian(0,1));"
"_Inert_FUNCTION(_Inert_NAME(\"Gaussian\"),_Inert_EXPSEQ(_Inert_INTPOS(0),\n_Inert_INTPOS(1)))\n"

On Mon, Nov 23, 2015 at 12:10 PM, Jacques Carette <notifications@github.com>
wrote:

> Yes, all top-level assignments need to be saved for things to work
> properly.
>
> VerifyTools[AddVerification] can be added to an export called ModuleLoad
> (see ?ModuleLoad) procedure, which gets called whenever that module is read
> in from disk [there's ModuleUnload as well, for symmetry].
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hakaru-dev/hakaru/commit/cc9623d89aaf751db8b0ab1410bb8c31a067d813#commitcomment-14566766>
> .
>


Revisado.


Change sose-15.json to latest version


Show verbose output, listing all RPMs
\


Needs the 'binary' at the end


@zackhall We need to update the information in this package.json file, as they are no longer current.


I am still analyzing it


List different rpms between the container images.


What is the need for this?


# Otherwise python2 gives unicode strings


Container images to compare


I think the midiMap was useful !
Why delete it so quicly ?
There is still work to do on midi , so perhaps create an other one ?
For quick midi task , i mean


No, it was sent at 17:13 +01:00 (CET), I guess 16:13 UK time.


test


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6356](https://teamcity.mesosphere.io/viewLog.html?buildId=79545&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:43


Thanks for correcting my grammar.


i still have midiMap in my local hub ... ^^


this was merged. thanks @joetsoi


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6357](https://teamcity.mesosphere.io/viewLog.html?buildId=79546&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:29


Hmm. Interesting. Thanks! Just curious about a slightly confusing email I got from the dev list at 17:37, heh.

Welcome to the team :tada:. Will leave Mike to deal with the bumping permissions, adding you to the chat etc, he usually handles that.


Formatting


> It was broken before

Nope, the build was very green. What the `ldc` branch in druntime/Phobos points to does not matter at all; it's precisely updating the submodule references where you need to be careful.


Any reason you are using `vim-git` instead of [Fugitive](https://github.com/tpope/vim-fugitive)?


btw, i've reverted to 1.6.0 until we find a solution since it was breaking linux in master


@yujuhong, there is a question here. I'll remove it later. Do you know the answer?


For anyone that is interested, I just added a recipe to cook [Wetkit Deployment](https://github.com/meosch/drushcookbook/blob/master/wetkit_deployment.drecipe) with with [Drush Recipes](https://drupal.org/project/drush_recipes).


Thank you for your suggestion


yes, I can use ```ImmutableList.of(E element)```method.


Вы не подскажете, что именно лишнее?
Просто пока не очень разобралась с буквами


it's a TODO.


I'm not sure how useful, or safe?, the prisms are when you want to handle all cases (cata/fold) as you don't get any exhaustiveness checking. When you really only care about some specific cases, then the prisms shine.


Sorry, I got my files mixed up.


ty :)

fyi:

```sh
$ git changelog --tag 1.2.3
$ git release 1.2.3
```

will do this for you


言おうとしてたところが大体直っててさすが＼(・o・)／

あとは細かいところ（名称変更）だけど、
ap_check → update
past_ap → current_ap
now_ap → new_ap
のほうがしっくりそうな


Should this link point to somewhere? If not, why is it a link?


This is because the first argument of neststartup is declared as int* (so we need to pass that to satisfy the compiler) and because int is wide enough in this case. We *could* introduce an extra variable and make some kind of range check before type converting it to int*, but that would be overkill.


It's a temporary solution until I and Jeff figure out why is it spontaneously crashing on bn* nodes.
Compare /users/klpf990/log/Dev_0152_HiSeq_AURA_FFPE_AZ300_Rerun_FastQC_AURA_198_R2_bheU8Z.log
and
/users/klpf990/log/Dev_0152_HiSeq_AURA_FFPE_AZ300_Rerun_FastQC_AURA_198_R1_NyUeRA.log

Hope to sort it out as soon as possible.


_item.updated_ stands for modified date


updateはREST的にまずいかな？でもPATCH動作だし…


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6358](https://teamcity.mesosphere.io/viewLog.html?buildId=79548&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:13


I did a dumb.  Thanks!


I commented here before but it seems to have vanished.   Unless I am misunderstanding, this is too late. The loop over tx.vin above can reject transactions that are already in the mempool as conflicted before it gets to here (because their inputs are spent, by themselves!).


Hm, we could, but then we don't have a way for clients to implement a safe catamorphism (that's why they used to have the `fold` method).


Confirmed :) We just need to get rid off @WebListener and add the listener to servletContext programmatically.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6359](https://teamcity.mesosphere.io/viewLog.html?buildId=79549&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:50


Also, in the comment on line 201, "Is the transaction fee _is_ less..." has a spare _is_, I think.


I learned that there are 2 ways of performing this call:
  1. via ```Meteor.call```
  2. you can inject the ```$meteor``` dependency in the controller (```function ChatsCtrl ($scope, $ionicModal, $meteor) {```), and then call it with: ```$meteor.call```


The reason is that cython.address is only used here. I find the code simpler to read if it is clearly stated that we refer to the "address" in "cython", otherwise it is dependent on a line at the top of the file. I guess it is a matter of taste. But maybe an experienced cython programmer *expects* to see just "address"? I can change this if you like.


Just load up the page in your favorite browser.


I'm not sure there is a _need_, but it keeps them symmetric with the non-travis local configuration where they must be run on different ports.


I haven't noticed that, apologise. I'll fix it in a minute.


> Welcome to the team :tada:.

Thanks a bunch! I'm happy to join and hope I won't wreak havoc when first touching a repository used by thousands. :wink:


What's the usage of this `requestedGroupVersions`?


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1433 of 1433 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.49%25-green.svg)


The use of long as a first test probably not required


Oh, this is going to make bookmark nav easier!


I'd prefer to keep this exported for now. If there's future work to split more code out of the daemon package, this will be a useful function to have exported.


It's tradition for maintainers to break Homebrew *at least* once :wink:


Yes, Travis already runs `composer install` and automatically installs all dependencies (including `require-dev`). `composer update --no-dev` removes the `require-dev` dependencies again, but keeps the regular dependencies.


Ah good point, I think we can just delete it as we don't need to worry about the order any longer (see next comment).


постоянно говорят убирать действия по умолчанию, ну хотя это алексей симоненко говорил постоянно, убрать?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6360](https://teamcity.mesosphere.io/viewLog.html?buildId=79550&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:07


Code should be self explanatory no need for comments


К сожалению, arithmetic.cpp не видит подключаемые h файлы,попыталась что-то сделать,не вышло, пока оставила так


No need to have todo


`coverageReport` generates both the html and xml reports, as far as I can tell. Then `coverageAggregate` aggregates them all into a single report.

Explicitly producing the report and aggregating allowed me to remove the ordering constraint on the tests. I tested with and without the order and got the same results. I can double check this again (and compare to not using the aggregate) to make sure.


too long line


we want informative commit messages! :)))))


It could. (Again, this is probably a matter of taste. It doesn't have any practical difference. If the goal is to save a few bytes, this code is to prefer. If the goal is conceptual simplicity, the original is to prefer.) Do you want me to change?


yay!


I agree with Zidane. Maybe you can push it to a predicate or something similar to allow some server dependend adjustments if needed.


Yes, better.


nice camel case


If we don't need it for codecov, then I guess we can remove it.


this is going to break edxnotes, which currently uses this function to get a longer-lived token.  the token expiration is defined in lms settings.  i think we have to use the same setting.


I am clearly in no shape for commit access today, I may just go home and take a nap :(.


Sorry, here is the original commit message.

----

Revamps the former `WarningBox` as the new `YellowBox` component with the following changes:

- Default to enabled (in `__DEV__`) with a `console.disableYellowBox` flag to disable it.
- Added a `console.ignoreYellowBox` property to ignore warnings by string suffix.
- Now handles `console.error` calls with strings that start with `Warning:`.
- Removed logic that persists the list of ignored warnings.
- Displays only up to four warnings (before it requires scrolling).
- Minor tweaks and polishes to the user interface.
- Added support for React Native Android.

----

@brianreavis Thanks for the report and sorry for the breakage. 6b3a6e59580e71256a3a6b885b0fb8f88bd1cb6e fixes this.


I don't think this was an intentional change.


Adding a logger is over engineering a simple console write would be sufficient


This should be specified on the ticket, but it isn't. The agreement file should have access control metadata associated with it, making it unpublished.


Shouldn't this be:
```
float depth = - tex2D(RFX_depthColor, texcoord).x;
```
?


can you add a TODO here to add support for ordering in the programs API itself?  (maybe along with the pagination we need to add)


Yes, I already noticed. Since I'm the one who left the space there I should be the one to have the merge conflict :)


> Should this be moved to the Heroku adapter?

We'd need to inherit from `Rails::AppBuilder`, so that we have the `template` method for example.


This seems like fun!  Is there any code coming?


Remove printf


Where do you handle transforms?

IE this is broke
```coffee
    simplifyify inputGlob, config
    .transform (file, overrideOptions = {}) ->
      if (lintIgnore.filter [file]).length == 0
        file += '.ignore'
      browserify_coffeelint file, _.extend(overrideOptions, doEmitErrors: !watch)
      .on 'error', ->
        process.exit(1)
    #  NOTE this cannot be in the config above as coffeelint will fail so the order is coffeelint first
    #  this is not needed if the transforms are in the package.json . If in JSON the transformsare ran post
    #  coffeelint.
    .transform('coffeeify')
    .transform('browserify-ngannotate', { "ext": ".coffee" })
    .transform('jadeify')
    .transform('stylusify')
    .transform('brfs')
```


OK, np :)


Thanks. Good catch. Fixed now.


Ok :)


this worked. I add it within the feature-timezone branch.


Правило хорошее. Но!
Если убирать действия по умолчанию, то тогда надо убирать везде а не выборочно.

К тому же есть ситуации когда нужно вызвать действие по-умолчанию, например, из-за этого форма не закрывается - это желаемое поведение?


I should probably also check if this is redefining something in `minerva.models`, and maybe allow a `force` flag for known redefinition.


@Fishrock123

> What does this actually do? (And how? The domain swap is lost because nextTick is fired later..)

The domain swap is not lost, because when the domains module is loaded, the handler for callbacks scheduled with `process.nextTick` is [replaced with `process._nextDomainTick`](https://github.com/nodejs/node/blob/v0.10/src/node.cc#L940), and `process._nextDomainTick` [captures the value `process.domain` at the time of the call to `process.nextTick`](https://github.com/nodejs/node/blob/v0.10/src/node.js#L533).

The rationale behind this change is that, if a timer's callback throws an error, the rest of the timers in the list have to be processed, but they should not be attached to the domain to which the timer that failed was attached.

The test in this commit tests this by attaching timer A to a domain, and throwing an error in timer A's callback. In timer A's domain error handler, that domain is disposed. When timer B is processed, without that fix its callback would run with timer A's domain attached to it, and since this domain has been disposed, [timer B wouldn't run](https://github.com/nodejs/node/commit/6eb4d1d15ca8d2b82cf9db99d56633357ccc3320#diff-0a5d4868b2b9b17cf9e2c11f1bd1311eL105).

> The test passes with or without this...

That's not what I'm seeing:

```
➜  v0.10 git:(v0.10) ✗ git rev-parse --short HEAD
6ac47aa
➜  v0.10 git:(v0.10) ✗ make -j 6
/Applications/Xcode.app/Contents/Developer/usr/bin/make -C out BUILDTYPE=Release V=1
make[1]: Nothing to be done for `all'.
ln -fs out/Release/node node
➜  v0.10 git:(v0.10) ✗ git diff
diff --git a/lib/timers.js b/lib/timers.js
index 83493eb..44f2136 100644
--- a/lib/timers.js
+++ b/lib/timers.js
@@ -127,12 +127,12 @@ function listOnTimeout() {
           // We need to continue processing after domain error handling
           // is complete, but not by using whatever domain was left over
           // when the timeout threw its exception.
-          var oldDomain = process.domain;
-          process.domain = null;
+          //var oldDomain = process.domain;
+          //process.domain = null;
           process.nextTick(function() {
             list.ontimeout();
           });
-          process.domain = oldDomain;
+          //process.domain = oldDomain;
         }
       }
     }
➜  v0.10 git:(v0.10) ✗ ./node test/simple/test-domain-exit-dispose-again.js
0 null
1 null
2 null
{ [ReferenceError: err3 is not defined]
  domain:
   { domain: null,
     _events: { error: [Function] },
     _maxListeners: 10,
     members: [],
     _disposed: true },
  domainThrown: true }
in handler undefined false
3 null
4 null
5 null
6 null
7 null
8 null
9 null

assert.js:93
  throw new assert.AssertionError({
        ^
AssertionError: false == true
    at process.<anonymous> (/Users/JulienGilli/dev/node/v0.10/test/simple/test-domain-exit-dispose-again.js:74:3)
    at process.emit (events.js:117:20)
➜  v0.10 git:(v0.10) ✗
```


instead of insmod with ko's absolute path names, you may want to try

```console
modprobe -d /sysroot -va overlay
```

and there is no need for existence test, just check exit code of above modprobe command.


Два прочерка, очепятка.


Non wildcard imports are preferred, as in: https://github.com/UNC-Libraries/deposit-forms/blob/master/src/main/java/cdr/forms/EmailNotificationHandler.java#L33


The logger is coupled to the PrimeCalculator class would be better to have dependency injection


Minor stylistic note: we should probably remove the $ from variable names that used to be jQuery objects right?


I'm not familiar with this pattern. Why call `forEach` on an empty array?


Should the agreement record have more structure than this? It could be JSON, RDF XML, etc.


Hrmm, I know Haskell Prisms have exhaustiveness checking. Maybe not part of Monocle (yet)? Might require Shapeless?


I think that & works for this case --- the address operator is only needed for "pure python mode", which this definitely isn't.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=287042&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=287042&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:23


logger.warn() is preferred.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6361](https://teamcity.mesosphere.io/viewLog.html?buildId=79552&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:53


And maybe any how2 install that fix? By default after install there is no such file as mainwindow.ui so how?


We may not even need these three source models.  Other than the `S3SourceModel`, which needs to redefine `createSource` and `destroy`, we don't do anything type specific in the other SourceModel subclasses, just passing in the endpoint to `createSource` in `SourceModel` would be good enough and reduce bloat.


It appears to pass without it on `master` (and almost certainly v4 / v5).


Ha nice, so many new comments!

I tested this in doxygen, and it appears you need a ///< prefix for the text. That holds for all data member comments that you added. In addition in this comment, adding a '#' prefix like #m_pixelFormat, makes the member name a link. There may be other opportunities to add links.


I'll change to &


This should be "IdGenerator" not "OsmMapOperation".


зачем этот черный квадрат?


typo in Whether


vendor/samsung/klte/proprietary/vendor/lib/libqdi.so.so
derp double .so


scaling


Just a suggestion, as a matter of taste. My brain automatically starts to ask why the entire array is being erased. Could even just `argv = [None] * argc` since it's gc'd anyhow.


This should be using the IdGenerator passed into the function. E.g.

obj->getMap()->setIdGenerator(toCpp<IdGeneratorPtr>(args[0]));


drop last 'e' in explicitly


looks great


TDD practice: No test were written prior to implementation


resources


This should pass an IdGenerator. E.g.

```
map.setIdGenerator(new hoot.DefaultIdGenerator());
```


Gostei mano, ta bem explicadinho e nao eh soh copia como os outros ehhehehe


@gboone Dude, you committed a merge conflict. :smiley:


hmm, should this be done in a constraint? (what if the units of one of the Variables changes?)


Hey, I just realized there is a corresponding doc update needed.  The doc is http://kubernetes.io/v1.1/docs/getting-started-guides/ubuntu.html#download-binaries, and I opened issue #17666 for its update.


Should this be `this.$search.setaAttribute(value, term)`?


Third "i" of Indicate can be put back into your computer


s/THe/The/


"Você deve construi-lo na entrada do túmulo local."


${} would be better


Updated to fix "500" errors from mariadb having wait_time set to 600


"Mina de Cristal" com iniciais maiúsculas por ser uma localidade.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6362](https://teamcity.mesosphere.io/viewLog.html?buildId=79561&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:06


My spell checker likes "transferring" here


represenTAtive


I would probably remove the last two as they're overly broad and not specifically related to what we do


"Não se viciou no veneno"


You're right --- given that POSIX gives command line parameter maximum, you're guaranteed that it's safe for all ranges that we support. There's probably also an MPI requirement here. But then does cython insert an (int) here, or do we need an <int> cast to avoid warning noise in the future? Or maybe just a short comment so when someone asks, they come here?


Added.


the only thing i saw was

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v6.0A]
"CurrentInstallFolder"="C:\Program Files\Windows\v6.0A\"

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v7.1A]
"CurrentInstallFolder"="C:\Program Files\Windows\v7.1A\"

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v8.0A]
"CurrentInstallFolder"="C:\Program Files\Windows\v8.0A\"

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v8.1A]
"CurrentInstallFolder"="C:\Program Files\Windows\v8.1A\"


Moved this test outside of init loop to allow uwsgi to be independently rebuilt.


cc @coreycaitlin There was an open question for Isabelle on this thread. Just copying you in case we need to make some line edits.


Please split in two lines to be consistent with the rest of the file.


(Actually just saw that the del argv[argc:] solution has a problem if initial argv is shorter than the new.)


This pull request wasn't accepted because it doesn't fully fix issues relating to the icon, so it isn't really a fix


Pode ser "Aplicando perturbação de feitiço" mas acho que ainda assim fica sem sentido. Precisa de contexto.


@wepp ,
$('#action-'+bookId+' .deactivate').hide();
$('#action-'+bookId+' .review'); ..


"arbusto em chamas" ficaria melhor.


"O fluido do corpo de Matsum foi purificado".


This all makes sense to me. I just had a couple comments / questions.


@laughedelic I assume this works because it's published and we have been using this but why this is not like this?
`    s"http://s3-eu-west-1.amazonaws.com/resources.ohnosequences.com/fastqc/${version}/${fastqcZip}"`


I don't think it was ck.  It was because of that extra comma you had in the array.
When my machine was running your tuning patch, it kept repeating:
"INVALID INPUT"


Building venv as part of initial container startup and not in Dockerfile.


Coding by boolean assertion would have been a better approach


"em frente à barreira".


"espinho de coruja".


Moved this to initial container startup script (start.sh) to allow mapping of /venv to volume.


ooh


thx


Can we, maybe, get away with using `boost::optional` here? The non-engaged optional can then simply mean "default"?


"Precisa de um monstro para responder!"


I've put an <int> there.


What is the reason to limit to non-fully qualified name?


In this case, the mode can never CUDNN_BATCHNORM_PER_ACTIVATION, right?


Issue https://github.com/numenta/nupic.core/issues/701


Nice catch, thanks. Updated.


I don't sure that it's right place for this


"floresta de espinhos"


The old mirror is renamed to buffer, and a new mirror is created. The schema of the mirror we have in v11 is the same as the new buffer, because that's all it was: a repository of unprocessed records from the server.

This allows us to take existing 1.1 and 1.2 users and keep the bookmark data they've already downloaded, merging it once all of this is good to go.


I think you should use the centralized cuDNN handle now


While I certainly understand where you come from, what happens if the user wants to run the test with the funky `virsh` command with GDB? It'd silently ignore the order given by the user, and I don't think that's what we want.

I'd be fine with simply re-raising the ValueError and throw an exception with better information, like `Command 'foo --bar' appears to be malformed (unclosed quotes). Please verify your command and consider escaping especial characters'`, or some more polished version of this message.


why not nullable?


"Peça mágica de Matsum"


what would be the warning here?


I would like your thoughts about these changes or if you think something else...
[pom.txt](https://github.com/nym2015/commons-csv/files/41902/pom.txt)


"Everything" feels really overwhelming compared to the old page since it doesn't sort itself by category. I think it's going to lose people.

Could we change it back for now? Ideally we'd merge "everything" into the main projects page, keeping everything's comprehensiveness with the current page's accessibility.


To validate parameters could have used required to get round using exceptions


TeamCity TerraFX :: Release [Build 0.0.1943edc09856803de5516501d7556fdd967e8afa.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=1&buildTypeId=TerraFX_Release&guest=1) is now running


"Você precisa de monstros".


TeamCity TerraFX :: Debug [Build 0.0.1943edc09856803de5516501d7556fdd967e8afa.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=2&buildTypeId=TerraFX_Debug&guest=1) is now running


I'm a bit confused, how can this happen?


TeamCity TerraFX :: Release [Build 0.0.1943edc09856803de5516501d7556fdd967e8afa.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=1&buildTypeId=TerraFX_Release&guest=1) outcome was **FAILURE**
Summary: Error message is logged (new); exit code 1 (new) Build time: 00:00:12


TeamCity TerraFX :: Debug [Build 0.0.1943edc09856803de5516501d7556fdd967e8afa.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=2&buildTypeId=TerraFX_Debug&guest=1) outcome was **FAILURE**
Summary: Error message is logged (new); exit code 1 (new) Build time: 00:00:12


I handled that by passing this `AppBuilder` in and using it in the adapter, like [this](https://github.com/thoughtbot/suspenders/commit/f733bb413ad92edcf81d3ce3273ccb3382878be9#diff-947954b5e35989413f423b82a361302aR16). What do you think?


You could just `return` here and avoid the indentation of the rest of the code.


Try server is only applicable to code checked into mozilla-central. This either needs to be explained or we need to go back to the older, more ambiguous wording.


Mais uma localidade pra traduzir de acordo com a convenção: https://github.com/Treeofsavior/PortugueseTranslation/wiki/Tradu%C3%A7%C3%A3o-de-Lugares


@wepp, якщо книжка вже активована або вже відправленна на перегляд, то для чого давати можливість її повторно відправляти?


That is correct. You can safely include both `initDataType` and `initDataTypes` properties to support Firefox 38–42 and 43+. The extra property won't break `requestMediaKeySystemAccess()`. Note that we plan to remove the old `initDataType` property next year (in [Firefox bug 1224074](https://bugzilla.mozilla.org/show_bug.cgi?id=1224074)).


Remove this.


Extra whitespace.


Suff like this is going to take a while to fix.


"Não há nenhuma Estátua da Deusa nas proximidades!"


Wow, you are on a roll!  That was very quick!  Impressive!


Probably need to remove one blank line here.


[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SDKs\Windows\v8.1]
"CurrentInstallFolder"="C:\\Program Files\\Windows Kits\\8.1\\" was not found


Sorry, that was my fault with my clang warning patches. I didn't realize the email address had to be "johndown". I just copied and pasted from a working part.

Previous to my patch, those two lines where in raw utf8 without escape characters. So it broke vi and clang did not like it either. So now it in better shape.

I'll try to be more careful next time.


Ezee, The midiMap is merged in master, we don't need the branch anymore. Keep branchs alive is the short path to merge conflicts. Every time we push something the branches are possibly diverging.

**We need talk about something very serious:**
You create too much remote branchs. These branches need be local only because are your personal "in progress work", not the "consolidated team work". What happens if we have more devs here, and each dev create 4, 5 remote branches? How many notifications the team will receive for all these public branches?

Ezee, for me is not possible work in this way. You are creating a lot of parallel work and generating tons of notifications. We don't have enough time to all these notifications. If you like work in many things at same time, ok, I will respect your program style. But when you create a lot of remote branches and generate tons of notifications for every little push you are forcing all team members to work in your "speed", and off course this is a very bad thing. We need respect each other, you can't impose your "speed" for the others in the team. Maybe this is not your intention, but this is happening. You are forcing all in the team to read tons of unecessary notifications. I'm being totally honest with you: I can not keep working in this way, it's very exhausting for me. If we can not find a less frantic way to work I would probably abandon this project. We need find a more balanced way to work. More balanced means less remote branches, and reduce the notifications a lot.

**Please, be more carefull with new remote branches and notifications**

It's ok create a remote branch if we need share some code. But create a remote branch for personal experiences is not cool, you are using the public remote repository without considering the others. The "unsubscribe" button is a option, but why work in a team if nobody communicate? No communication is bad, but excessive communication is bad too. We need balance.

About the excessive notifications: I'm afraid Tea was scared with all this mess in Jamtaba repo, and I'm afraid we are loosing a great programmer. Tea is very disciplined, a good practices enthusiast, I'm afraid he look for this and think "this mess is not for me". This is serious, we need fix all the mess. Please try keep your branches in local, so nobody receive every litle notification.

I will repeat: I can't work in this way for much more time, I feel my energy is at the end. I appreciate your work here, I think you are a crazy creator (crazy in a very good way, don't understand me wrong), but I can work in this rhythm. I feel you are a 320 bpm hardcore punk drummer and I'm a slow ballads pianist. We can play togheter, but you need slow down a little bit, and I will try speed up a litle bit.


Why not `: STAccount (n)` here?


You should probably move the declaration inside the loop.


It's not implemented in any version of Chrome at the moment. It may or may not be implemented in Firefox. It's in the spec, though, and I wanted to move towards coding defensively against the spec with sensible fallback behavior, to make the samples more future-proof.


This is the first introduction draft, Renato has not seen it yet


"Não há monstros ao redor para você usar a bomba!"


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6363](https://teamcity.mesosphere.io/viewLog.html?buildId=79576&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:06


job added


"direcionar" fica melhor que "dirigir" neste contexto.


Looks good, but not sure where is MN as I'm not from the states


What do you do exactly?


Fixes induced by the above.


"A peça do monumento não é suficiente."


If you feel so inclined, you can replace `160 / 8` with: `uint160::bytes`


Not so many nots needed I realize. It's also a more natural expression: "If the project is public and published, or the user has special access, then show it, otherwise don't":

```python
    if not ((project.is_public and project.is_published())
            or user.has_perm('rsr.change_project', project)):
```


I didn't want to have them named however Minecraft names them at runtime.


Maybe add a  ZIP+4 code?


Yes, it's an unused variable and is unrelated to the change we are making.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6366](https://teamcity.mesosphere.io/viewLog.html?buildId=79593&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:43


@dash-rai thanks :+1: will amend


"Sem Vubbe Vermelho Arqueiro para aplicar"


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6364](https://teamcity.mesosphere.io/viewLog.html?buildId=79577&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:04


Just for the record, if anyone else reads that discussion, the test passes on master/v4/v5 because it was written assuming that `domain.dispose` clear any timer attached to a given domain. This is [true in v0.10](https://github.com/nodejs/node/blob/v0.10/lib/domain.js#L208), but [not in master/v4/v5](https://github.com/nodejs/node/blob/v4.x/lib/domain.js#L302-L317).

That test should not rely on such internal implementation details, and should instead check that `process.domain === null` in the second timer's callback.


So atm there isn't any fix which fix the resizing window problem? When indiwidual pricing items with couple mods, when have couple tabs of items, it's making ppl mad...


"Porcentagem" com P maiúsculo.


I agree, Elieser. Philosophy and youtube videos are for another platform. I will stop commenting off topic if that helps. I apologize in advance.


"mágico" com acento.


Revisado.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6365](https://teamcity.mesosphere.io/viewLog.html?buildId=79580&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:26


This line didn't need editing


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6368](https://teamcity.mesosphere.io/viewLog.html?buildId=79597&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:30


This seems kinda weird to me.  Passing an error to a success event seems counter-intuative.  I think it's being used to signal that a message fetch is needed (because it's a reconnect).  Is it possible that some non-error flows would also need a message refresh?  Maybe error should be replaced with 'needs message fetch' instead?


I think @alcarney just caught a minor typo there: `tHis` is replaced with `this`.


But won't a depends just simply ensure it is built, not that it is used?  Or is the path set using it first if it exists?  What was the symptom this resolved?  There were issues with do_install when using upstream pseudo.  Wonder if this was somehow related?


I hope so! :+1:

(Note that we still won't be able to rely on integer IDs. We should be using GUIDs everywhere anyway…)


@catsby good catch. I wasn't sure where that came from, but when I remove it I get an error where it won't match the wildcard cert. It looks like it's probably https://github.com/golang/go/issues/8998, but I couldn't find a newer version of `wget` for precise64, and I couldn't find a good newer Ubuntu box with VMWare support, so I switched to `curl` which checks the cert successfully.

https://github.com/hashicorp/consul/pull/1445


Thank you.


this closes #230.


Для вызова использовать метод

        private async static Task<RssClass> ReadData(string url)
        {
            WebClient web = new WebClient();
            string s = await web.DownloadStringTaskAsync(new Uri(url, UriKind.Absolute));
            XmlSerializer xml = new XmlSerializer(typeof(RssClass));
            RssClass res = xml.Deserialize(new MemoryStream(Encoding.UTF8.GetBytes(s))) as RssClass;
            return res;
        }


If the client is reconnecting, does it need to make sure that it still has the right offsets too?  It feels like maybe a a fetch alone might pull messages that have been picked up by a client on a different machine that was not disconnected.


Can you detail what you were hitting and how this solves it?  Looks like it's still the default.  The intent is it's the toolrunner consumer that defines what happens with debug writes.  Seems like that's still the case??


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6370](https://teamcity.mesosphere.io/viewLog.html?buildId=79599&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:39


Should be `https://developer.spotify.com/web-api/object-model/#album-object-full`


it would be better to extract it


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6367](https://teamcity.mesosphere.io/viewLog.html?buildId=79596&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:02


Disregard previous comment - this is the correct URL - the object model simply hasn't been added to the Spotify Developer site documentation - I'll add it.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6371](https://teamcity.mesosphere.io/viewLog.html?buildId=79600&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:40


@borsch , hidden="hidden" и после нажатия оно скрывает нажатую, а вторую открывает


I don't think this is required. It is getting set either from a reader or a source, so it doesn't need it's own data. Did it crash before you changed this?


Errr, I might not have given precisely the right url.  Also, there are a bunch of other `<a href` that should also be made into pathFor calls...


This is the collaboration step


comment on individual lines


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6369](https://teamcity.mesosphere.io/viewLog.html?buildId=79598&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:47


@iKristjan  THX for this one - this fixes a myster bug in one of our large traffic apps. where the `me` object was empty even if a valida sid is sent!


Was it intentional that the SRAM_512KO description was taken out? Its symbol is still in the lib file.


This causes problems with R's base::which() -- I've had to remove it


```
Quicksave0_308429D0_52616368656C_Commonwealth_000146_20151122154436_1_2.fos
Quicksave0_308429D0_52616368656C_Commonwealth_000146_20151122154436_1_2.fos.bak
quicksave.ess
quicksave.ess.bak
```

@lojack5 What is the intention of looking at the prefix of the file.  According to [this site](https://regex101.com/) the only significant capture is the file extension.  The file name could be any of the above file names but the only part that will ever matter is the ``.fos``, ``.fos.bak``, ``.ess``, ``.ess.bak``.


+1 for all this. There are a couple of placeholder stories for this. We also have this kind of case:

    mylib {
        api {
            exports 'some.package'
            assert exportedPackages == ['some.package']
        }
    }

Not sure exactly how we would infer the 'exports' name from 'exportedPackages', so possibly some control over the name would be useful. Or, perhaps 'worse is better' and we always simply use the name of the property.

We also don't necessarily need to auto-generate this method. Perhaps you code it yourself instead.


Sorry, my fault. Array access only is implemented for `get`, not for `has` (https://github.com/zyuhel/Embed/blob/master/src/Bag.php#L93).


> I think it may be worth reconsidering if we still want to think of that as valid Let Notation. However, the original thinking on carriage returns in names still makes sense.

Seriously?! We want to consider the use of an invisible character for a variable name as better code than code lacking semicolons? What is the ratio of uses between the two?

@dansmith65 Just because tests have been written for something does not mean that you continue with the original idea. If a sculptor starts chipping away stone from the block he/she doesn't consider it finished until it looks the way he/she wants.

Valid input, as defined by **OUR** implementations does not have to be restricted by what FileMaker considers (or allows) as valid input. I don't care if FileMaker allows for carriage returns within variable names. That's just stupid.

http://legacy.python.org/dev/peps/pep-0020/

If there are cases where it's more simple to provide Let input without semicolons and it reduces the complexity of creating that code, such as using ExecuteSQL or CustomList, then it more closely meets the objectives of fmstandards overall - primarily clarity and readability. Just because FileMaker allows for carriage returns is not a reason to condone the practice.

From the link referenced...

> The closer we can stick to FileMaker's own requirements

FileMaker does not require the use of carriage returns within variable names. Data, sure, that data should be visible, clear and simple. Quote()ed data should be our standard.

Are you really arguing to keep support for invisible carriage returns within variables and not explicitly using the quite clearly visible and well understood pilcrow escape character? All in favor of previously written tests?


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 41](https://teamcity.labkey.org/viewLog.html?buildId=287040&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:29


Actually I may have accidentally introduced the typo there

On Mon, 23 Nov 2015 18:49 Vince Knight <notifications@github.com> wrote:

> I think @alcarney <https://github.com/alcarney> just caught a minor typo
> there: tHis is replaced with this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alcarney/CardiffMathematicsCodeClub.github.io/commit/5e58d77ccfe04ca6eff14e268f0c234e2e83d8ff#commitcomment-14569166>
> .
>


> What's the usage of this requestedGroupVersions?

Unused.  I removed it.


It is actually really cool :)
Does only work with the two first dropdown menus, though.
But pretty cool!


You didn't actually *need* to change that! :stuck_out_tongue:


Unfortunately, I don't think a fix to ffmpeg.to_wav will be possible. It took me quite some time to debug this, but the issue is that ffmpeg doesn't set the same headers if it's not writing to a file on disk, and Safari requires these headers.

So, if I run:
```python
open("foo.wav", 'w').write(gentle.ffmpeg.to_wav(AUDIO_PATH).read())
```

Then ```foo.wav``` will not work in Safari (though it will work in Firefox and Chrome). I'm pretty confident that the issue relates to these notes from running ```ffmpeg -i foo.wav```:

```
[wav @ 0x7fa0c9810600] Ignoring maximum wav data size, file may be invalid
[wav @ 0x7fa0c9810600] Estimating duration from bitrate, this may be inaccurate
```

These warnings to not show up if ffmpeg is given an output file on disk.


Shouldn't `go install` do this automatically as a part of `go get`?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6373](https://teamcity.mesosphere.io/viewLog.html?buildId=79602&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:28


Ah yes, apologies!


Well spotted @timothyf1!


Technically this is dead code though had left it as second guard in case for accidental changes :). Though it may be unnecessary. Will clean it up. Code will be more readable and simpler.


Nice change! I think that, as you use jobnames, this could be map_async and would be faster (don't try to keep the order, and thus don't wait for all (nprocs) threads to finish).


LGTM


also updated/standardized function names and created new function to handle subgrid generation.


missed space after `,`


fix spacing this file


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6378](https://teamcity.mesosphere.io/viewLog.html?buildId=79611&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:52


At that point , to still work at my rythm , there is the fork solution .
I don't see where the progress i make are disturbing as the master stay untouched .
If i have many branch it's because i work on specific tasks that should be merged in the master ,
but since the beginning you are afraid .

perhaps at the beginning i was afraid of merge conflict too , but now i have learned how to fix them .
I understand you have your rythm , and you can't stand anymore mine .

I'm **afraid** Tea was scared with all this mess in Jamtaba repo, and I'm **afraid** we are loosing a great programmer.

you are too much afraid for me .
But i choose to smile , and if you can't smile because of me , the better is i moove for some times .
Some holy days for me ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6372](https://teamcity.mesosphere.io/viewLog.html?buildId=79601&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:53


unnecessary `;`


Prevent pushing git commits that have a description starting with "WIP"


Ah, good question! I had to do some research on why this works. [Here](http://stackoverflow.com/a/15630151) is an explanation.

I think basically `ofApp.h` is 'inheriting' the include of `BrushBase` from `TriangleBrush` and/or `DreamBrush`. In other words:

`ofApp.h` includes `TriangleBrush`
`TriangleBrush` includes `BrushBase`
therefore:
`ofApp.h' can include `BrushBase`

Based on the article I linked, though this works, it isn't the best way to do it. The author says we should include `BrushBase` directly into all header file that depends on it including `ofApp.h` and we should add gaurds (`#ifndef`) to our header files to prevent multiple inclusions. To see guards in action, checkout the `BrushBase.h` file. The general template for a guard is:
```
#ifndef MY_HEADER_H
#define MY_HDEADER_H

// code...

#endif  // this file will only ever be copied in once to another file
```


(but you're right about forcing the PCM format. That's fixed in 067c169fbd30a0a3bef6af7e456d2a6a5c2df3ce.)


Re-using the variable for the thing that you are selecting on is fairly confusing.  I'd add the temporary variable back.


initial merge from remote repo into my local one. take-2 is the working branch. remote repo (github) already has license file created. so i'm merging the working local branch with this remote repo


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 39](https://teamcity.labkey.org/viewLog.html?buildId=287023&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 36 (14 new), passed: 252, muted: 5 Build time: 01:15:43
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

##### there are 26 more failed tests, see build details
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6376](https://teamcity.mesosphere.io/viewLog.html?buildId=79605&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:59


Broken code will be fixed in the next commit.


Ah, you are correct. I wasn't sure if `go get` was smart enough to resolve gopkg.in packages.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6379](https://teamcity.mesosphere.io/viewLog.html?buildId=79612&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:06


:)
tak na marginesie, jakiego IDE używasz, że tak ładnie robi Ci komentarze do funkcji i metod?


Nice


Awesome work though!


the fix
```javascript
// $('#customer .typeahead').prop( "disabled", true );
if ($('#assembly .twitter-typeahead').length)
      $('#assembly .twitter-typeahead')
        .before('<input class="typeahead" type="text" name="assembly">')
        .remove();
```


please don't skimp the letters


This feels like kind of a gotcha. Maybe we should just support paste once, still?


Doesn't look like `jasmine` is declared as a dependency of this project in package.json. Also, I recommend mocha + sinon for projects that don't depend on the dom.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6377](https://teamcity.mesosphere.io/viewLog.html?buildId=79610&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:12


resolves issue #31


In that case it could become:
```python
if resource.get(field_name, False) is None and fields[field_name].read_only:
    continue
```


didn't work for me either, want to just turn it in anyhow?


```
s"http://s3-eu-west-1.amazonaws.com/resources.ohnosequences.com/fastqc/${version}/${fastqcZip}"
s"http://s3-eu-west-1.amazonaws.com/resources.ohnosequences.com/fastqc/${version}/${fastqcZip}"
```

Sorry, I didn't get, what's the difference?


Doesn't look like this package declares that it depends on dibs-lint. Am I missing something?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6374](https://teamcity.mesosphere.io/viewLog.html?buildId=79603&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:10:08


np ... we have all done it ... :beer: is on you next time though


Can you document either here or in 'detach' that the returned value needs to have any shared content with @row cloned? I could see the implementation here messing up.


ok you both are right. switch is better to read and slower :)
but for long if/else I also prefer switch, although many of the if/else are from me.


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 93](https://teamcity.labkey.org/viewLog.html?buildId=287027&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 16 (4 new), passed: 234 Build time: 01:03:10
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAxisSelectors: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.verifyAntigenVariableSelector: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

##### there are 6 more failed tests, see build details
```


:fire: parentheses


We've been using 4-spaces for default indentation in other files. I think `npm init` does two spaces by default, but it would be helpful to make this file consistent with everything else.


Still a smal issue with function "get_first_subdir":

if [[ "$subdir" != "$PWD" ]]; then
is ALLWAYS TRUE !

The var $subdir has the /build/  cut off while "PWD" always starts with /build/xxxx .
I'm not sure why you need a test, but you could use what I gave to make your function "get_first_subdir".
If you think about optimisation consider this : a test also takes time and maybe more then just do subdir="${subdir%%/*}".


Grazie, vuoi contribuire?
Il 23/nov/2015 04:19 PM, "Manuel Uberti" <notifications@github.com> ha
scritto:

> Progetto interessantissimo, complimenti! [image: :wine_glass:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/enricodangelo/hello-world/commit/af1f465c8a559ecf52f54d984a286b3714fadc11#commitcomment-14563414>
> .
>


Since MoveBlock gets passed the view, taking care of dealing with the cursor should be in the mutation.


The method getStringAsCharBuffer on the Java MamaMessage does not exist.


```> 0``` not needed


OHHHHHH YEAAAAHHHH


same as above


missing ')' in static_assert. nur'n minor, viel mir aber eben auf ;)


Could be weird


Oh never mind it's cool


Without the charBuffer var being set by the missing method this line will fail.


Update to master


Whether or not a state satisfies hard constraints is a property of the state itself. Does it make sense to include this logic in the state object instead?


I fixed the indentation style here in a later commit


So this little Unicode character is what's been causing all this trouble...


I think you already get that in the console trace no ?


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=3&buildTypeId=TerraFX_Debug&guest=1) is now running


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=4&buildTypeId=TerraFX_Release&guest=1) is now running


So after a little more investigating, here's what finally resulted in the expected output...

1. I ran mvn clean and mvm package. These commands produced the jar file.
2. Right-click on the jar file and select Open With..Archive Utility (10.9.1)
3. This generated a directory called HelloWorld-1.0-SNAPSHOT that contained both the class file and the META-INF directory with the manifest file.


Why did you convert receiver to param?


@andrejserafim thank you for the fix! We will release a bugfix version (1.0.2) for it in the upcoming days.


thanks, updated. hopefully its correct now. (i have no idea what im doing..)


@meh gotcha, thanks !


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=4&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:21


TeamCity TerraFX :: Release [Build 0.0.3](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=5&buildTypeId=TerraFX_Release&guest=1) is now running


font comments


copy-paste


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=3&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:29


TeamCity TerraFX :: Debug [Build 0.0.3](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=6&buildTypeId=TerraFX_Debug&guest=1) is now running


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6380](https://teamcity.mesosphere.io/viewLog.html?buildId=79614&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:11


Whoops, thanks!


It tests to see if we're using the function on the right places.
If the substring stuff doesn't work, $subdir will be the same as $PWD and we do nothing.


TeamCity TerraFX :: Release [Build 0.0.3](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=5&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:32


I did this so if people were comparing something like a fedora image to a rhel image or a rhel 7.1 to 7.2 image they could do things without versions.


TeamCity TerraFX :: Debug [Build 0.0.3](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=6&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6381](https://teamcity.mesosphere.io/viewLog.html?buildId=79615&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:51


This should wait for the MamaTransport.onCreate callback before continuing, but I didn't have time to make those changes (should be a common set of code used by all of the tests). So I slept the thread for 4 secs as that is long enough to get a connect.


should i change it to that value C:\Program Files\Windows Kits\8.1\" cus windows kits is not found on my laptop


Maybe, I don't know;)


Not sure where these came from ... Corrected.


Done


Yes, it should have been calling advance_func->i2c_read_byte_replace from the get-go.
propanu - thanks for catching my bug.


Non so se mi sento all'altezza.


The assert could even have been moved to the if, as if the condition is not true for the if, it won't be either for the assert anyway. But it probably does not hurt how it is now :)


ResortedSurfaces (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


@jochenberger what terminates a string literal? Logical operators and close bracket? Is that it?

`$.foo[?(@.bar == x)]`  == `$.foo[?(@.bar == 'x')]`
`$.foo[?(@.bar == x a)]` == `$.foo[?(@.bar == 'x a')]`
`$.foo[?(@.bar == x a  && @.foo == y&z )]` == `$.foo[?(@.bar == 'x a' && @.foo == 'y&z')]`


C sure is fun :hurtrealbad:


You're right -- it would need an extend as well. Thus, your solution is simpler.


I just re-read most of that thread (since I obviously didn't do it the first time) and quite honestly, you guys were in the weeds on that one. I can understand wanting to be compliant with what FileMaker **"allows"**, but seriously?.....

That was a lot of time on a test, which I can understand should be applied to data. But for variable names???. Would it not have been wiser **AND** quicker to just do an invalidation using a Filter() to say "Hey, we don't support the use of these silly characters for variable names."


not here


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6382](https://teamcity.mesosphere.io/viewLog.html?buildId=79616&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:53


What i noticed is we haven't saw any debug output from toolrunner.
Ex:
      in most .js implemented tasks, we always call: var tr = new tl.toolrunner(tl.which('sometool')); in the constructor of toolrunner, line 44 of this file, we had code to "debug('toolRunner toolPath: ' + toolPath);" but i never see that line comes out to ##debug log.
      after add lot's of debug trace and play with the code. i noticed in the old code, the toolrunner consumer defines the way of export.debug() not debug(), as the result any debug() call under toolrunner will end up with doing nothing, unless we change all of them to be export.debug(). like change line 44 to be export.debug('toolRunner toolPath: ' + toolPath);


Ah, constants are so much better, thanks.

So the biggest use case for disabling downstream dependencies (instead of notifying the user) is for 'panic', but mostly this seems like an improvement.


Lots better! Everything fits again.


please try to install windows SDK 8.1.
https://msdn.microsoft.com/en-us/windows/desktop/bg162891.aspx


was fehlt? :D


That is actually an important feature...so the plugin dev knows what this object really is straight from the toString.


ah, in dem error string, lol


Yeah [you're right](http://ffmpeg-users.933282.n4.nabble.com/When-using-pipe-output-wav-files-are-corrupt-only-usable-by-libffmpeg-based-software-td2304011.html). It seems pipes don't play well with FFMPEG WAV.

I think the best solution is to stream out the raw samples and construct the WAV in Python to avoid touching disk. It seems slow and error-prone to make in a cross-platform way (and remember to destroy) so many temp files.

Maybe I'll do that later when I have a little time.


Nice job :+1:


ResortedSurfaces (amirroth) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


@dmacvicar thanks for the feedback, you are right again, I wasn't making myself clear,
so 'os' grain should be distribution specific 'SLES', 'openSUSE', openSUSE Tumbleweed', openSUSE Leap'
and os_family could all be 'Suse'.

Then we need to find the issues in the code where grain 'os' is used, and add 'SLES', 'openSUSE', 'openSUSE Tumbleweed' and 'openSUSE Leap'.


No that is not enough. Is all relational operators as well ==, >, >=, in, nin...

`$.foo[?(x == @.bar)]` == `$.foo[?('x' == @.bar)]`


Strictly speaking `hasNativeWeakMap` is `hasGlobalWeakMap`: a user could load a polyfill WeakMap then this runtime would use it.  The difference would only matter in debugging and the important case is native.


Will our tests cover both paths?

LGTM


Does this make sense?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6383](https://teamcity.mesosphere.io/viewLog.html?buildId=79619&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:03


please contribute upstream


I guess so. Methods unrelated to the use of ```TEST_DATABASE```, ```TEST_TABLE```etc. throw exceptions.


The "@" sign isn't necessary since "t" isn't a reserved word. It's clearer without it.


i could rewrite this part, or make a fix for bag.php, which one is better?


Weird....the indentation is fine in my editor...I removed all tabs....I'll try again.


Is this necessary?


Ah, you are right. Rebased.


This is a very hot path. Please use the for(var i = 0; i < passImmediates.length; ++i) {} version of the loop. We don't want to incur overhead here.


The first one is better. The second doesn't really solve the problem; what if the field is a BooleanField and has value False?


omg forms yaaaaaaaay


Probably after this is done (and separately) we will need to refactor to create a common way for doing mappers so not a lot of code is repeated between sqlalchemy and django mappers.
Keep playing and open a PR once you think is done! I would love to see sqlalchemy support officially in `graphene` :)


TWO SPACES PLEASE

Alright maybe I was wrong about style linters. I'm wrong about a lot of things.


Why didn't use `ordinal` instead?


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1957 of 1969 tests passed)

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


Absolutely ok and I also think so.
Just wanted to inform about the technical difference ...


## Why default scopes are bad

Post.all.first
#=> Post id: 1

with default_scope
topic.posts.first
#=> Post id: 50

default_scope { where(published: true) }
Post.drafts


Thanks!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6385](https://teamcity.mesosphere.io/viewLog.html?buildId=79621&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:01:02


@dannyvankooten: is it possible to inline this or at least move it to the footer?


**I don't see where the progress i make are disturbing as the master stay untouched**
Ezee, I'm not talking about the master branch, I'm talking about my life. Every time I'm reading a notification I'm not playing with my daughter, not playing with my dog, not kissing my wife, not playing music, not coding, etc. It's the same for all contributors in the team. So, every time we create a notification (pushing something, creating something in the public repo) we are impacting in the team contributors life, because everybody will take some time to read the new notification.

What I'm trying to say is just: **be more carefull with other contributors life, think twice before generate new notifications".


Yep - in particular, fail in a way that will fix itself given a few seconds or maybe a minute.  The JIRA issue gives examples of two failures we've seen happen in the real world, both from the same cause: sometimes the package repository's metadata is corrupt for a moment (we hypothesize because the repository is in the middle of being updated).


I guess the best is the fix for bag.php for consistency with get().


I think this can be done with selectors. Your proposal would create a host specific style for all hosts.


We should not need to back off when there is a running container. Checking the first container sounds fine.


`ensureEnumss` -> `ensureEnums`


Agree. That will lead to some naming conflicts, but we can resolve them.


why?


Ok, thanks.

Here's how I see it.  If there is a bug in the future that breaks the logic in how resolveTypes handles the DISTANCE function, then that bug will never get merged because the asserts you have here will fail in the unit tests.  I trust that our unit test and CI framework will do their jobs, so we can benefit from this by having simpler more readable code.


An alternative way to determine a code exception vs a thrown error is proposed here, looking for `error.stack`: https://github.com/domenic/promises-unwrapping/issues/19#issuecomment-24157476

Not sure how well either proposal fares in our complete browser support matrix.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6384](https://teamcity.mesosphere.io/viewLog.html?buildId=79620&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:34


maybe it's better to rename it to `getEnumLabel`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6390](https://teamcity.mesosphere.io/viewLog.html?buildId=79627&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:01:08


Could we use [StringIO](https://docs.python.org/2/library/stringio.html) to avoid writing to disk? We'd have to alter the format parsers slightly, but I believe silme is the only one that wouldn't be trivial to move from accepting paths to accepting the file contents themselves.


:+1: @itsanna


Sweeet! Glad we're getting some eyes on IE. I forget when this code path is actually hit; is it always triggered near the beginning of execution? If not you might want to consider plopping the fallback at the top of the bundle as a polyfill (or just pulling in [an existing one](https://gist.github.com/paulirish/1579671)), in case we end up wanting to use rAF elsewhere.


This is a just a more succinct way to call `destroySSHKey` with `SSHKey` as the first argument.


Der commit comment is mist.


/ lgtm!


I think `description` does not fit the idea. Description - it's usually a long text


this is just a white space... addition?


Nope.  I don't think the change is important enough that it needs to be done right away - rather, it seems like something that would be nice to do before we go very much further down the path of having two almost-identical implementations, one for sudoand for not-sudo.  If follow-up work suggests that might happen then I think that's an okay time to create the issue (downside is that visibility of necessary work that's only represented by TODO comments is low; upside is that JIRA is easier to use the fewer issues it has in it and it's not yet clear that this is actually necessary work of if it is just something that would be nice).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6386](https://teamcity.mesosphere.io/viewLog.html?buildId=79623&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:57


`ensureEnumTrait` -> `EnsureEnumTrait`


good catch. Can we add a comment as to why we do that ? so in the future we know when it's safe to remove :) (although your commit message deserves a huge +1 for clearly explaining it..)


We could. It only solves some of the concerns, though.


Функция по описанию должна возвращать boolean


I cannot accept collapsing server-interface into common. It breaks modularity. The reason for the split was to break out the types that are common to the server and brokers. The added classes are specific to the server.


StringIO is what we should be using, but if we weren't you'd want to create a temporary file to store this rather than creating and then deleting the resource file, which can easily fail and leave user-uploaded files strewn about.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6388](https://teamcity.mesosphere.io/viewLog.html?buildId=79625&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:43


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6387](https://teamcity.mesosphere.io/viewLog.html?buildId=79624&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:03


@yujuhong Thanks, that's what I expected. Because the old implementation here seems assume that there is alive containers, I just make sure. :)


yup, that's it.


It's better to change to `$this->value !== null` for the case when Enum value will be 0 (for example 0 - disabled, 1 - enabled)


Although the `exceptionHook` is always present, this is the only unit test that generates a console warning. Need a few more tests, including one showing a custom hook and the hook set to `null`.


The Changeset class is supposed to be where the bulk creation and update happens. We shouldn't need to be doing it manually here.


:-1: :-2:


why `aliveFlagPath` is nullable?


fixes #4


I think I would prefer if to use (bibtex-generate-autokey) to generate the key, since this allows users to specify how they want the key generated. That is more consistent with org-ref.

I would also prefer not to erase the other function (arxiv-get-bibliographic-code arxiv-number). It is a little redundant, but it uses a different method, and the person who contributed might like to still use it.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6391](https://teamcity.mesosphere.io/viewLog.html?buildId=79628&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:01


Ha...I see...The ContainerStatuses is not sorted by creation time.


Big increase in the default size of this TaggedCache.  I'm assuming that's intentional.


Yes.
   I ensured uuid-ossp was rds compatible before implementing it.

  Here is a list of compatible RDS Extensions
 btree_gin,btree_gist,chkpass,citext,cube,dblink,dict_int,dict_xsyn,earthdistance,fuzzystrmatch,hstore,intagg,intarray,isn,ltree,pgcrypto,pgrowlocks,pg_trgm,plperl,plpgsql,pltcl,postgis,postgis_tiger_geocoder,postgis_topology,sslinfo,tablefunc,tsearch2,unaccent,uuid-ossp


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=7&buildTypeId=TerraFX_Debug&guest=1) is now running


Need a PHPDoc


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6389](https://teamcity.mesosphere.io/viewLog.html?buildId=79626&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:07:38


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=8&buildTypeId=TerraFX_Release&guest=1) is now running


I would rather this feature stayed. I also create focused issues that make sense when converted into PRs. But I also work on private repos most of the time. If I were mainly writing on public repos, I probably wouldn't use this feature, but that doesn't mean I'd want it gone


This is a bunch of unnecessary reads given that we're just updating the database from some uploaded translations, and we're not even sure the files that this will attempt to read exist as we haven't pulled. We should either modify Changeset such that a VCSProject is optional (as a general thing; simply making the parameter optional might not be a good API) or make these changes to the database using a different method.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6392](https://teamcity.mesosphere.io/viewLog.html?buildId=79629&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:38


This is gross. Javascript arrays have a method called `forEach`, but `NodeList` objects (returned by `querySelectorAll`) aren't actually arrays, so they don't have that method. To use `forEach` on a `NodeList`, we can grab the method off an empty array, then apply it to the `NodeList` using `call`. This isn't an issue when we use jQuery or underscore, since both of those library handle this internally.


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=7&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:42


1. We can definitely do that. Let's add it as a new user story, along with a proposal on which insertions get priority?
2. Insertions and thread pool magic is implemented in other parts, mostly in the Freenet codebase. The trick here is to ensure the thread-safety of the insertionsQueue and the insertTable.


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=8&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:43


In general I prefer the first to the second:
```
this.func.bind(this, arg);

var self = this;
function(){ self.func(arg) }
```
**note:** the `self` declaration is required. In jsx, `{ function() {...} }` the `this` is still bound to the window inside of the function.


The commit has been reverted.


Submitting the uploaded file should not use the same view as loading the translation interface. It should be a separate view.


I thought so too, but apparently not:

> Using setAttribute() to modify certain attributes, most notably value in XUL, works inconsistently, as the attribute specifies the default value. To access or modify the current values, you should use the properties. For example, use elt.value instead of elt.setAttribute('value', val).

https://developer.mozilla.org/en-US/docs/Web/API/Element/setAttribute


Note: IMHO You often mix unrelated changes and it makes a review harder


And I'm adding those in the [uoa-inzight-cas Dockerfile](https://github.com/webplumbr/uoa-inzight-cas/blob/tom/Dockerfile#L20).


Actually it returns client's flag files


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6393](https://teamcity.mesosphere.io/viewLog.html?buildId=79630&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:08


Resolves #10


Huh. Yeah that's ugly. Thanks for explaining.


I am sorry for that .
My life is very different from yours , and i know i am too much .
So i take my wings and fly to my fork , where you are free to pick what will interest you .
I learned a lot with you , that was a really ' Have Fun ' time .
But we know how Fun is short , and Joy may be long .

I take some rest so , i have made some room too , perhaps you will see more clear now .
Thx .


As a note, you can use [`inject_test_data.py`](https://github.com/reddit/reddit/blob/master/scripts/inject_test_data.py) to insert a bunch of sample content based on data from reddit.com.


I don't think you need this if you don't have a separate voxel for unloading and you can freely go with the inherited `WithVoxelBody`.


I think `addMetadata` should be called from `getInstanceStr` where we add the other inputs.


is like Windows Kits 8.0 is for windows 8 .
which windows are you using ?


Why Proxy?


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


Once you've done a little bit of setup, running all the tests should be [just as simple as executing the `nosetests` command](https://github.com/reddit/reddit/wiki/Testing).  Was this not the case?


Verb Button and Cursor Graphics


This needs to be called before `daemon.imageStore.Delete`.


@andrey-markelov  can we rename this method from isContainsModel to doesContainsModel


Verb Manager which is in charge of cursor state and is queryable.  Should have an API for Input to affect.


@andrey-markelov why are not calling this endpoint /import/upload vs import/uploaded


@andrey-markelov lets do /containsModel


tried to make it more consistent :)


@andrey-markelov any reason we are not using /import/upload here


please don't remove credits or change the plugin home. do contribute upstream instead.


ok, I fixed the comma. And one more fix.... Lets see how it goes now.


@phase FYI, do you use Valgrind? I've found it to be SUPER helpful in finding memory leaks/errors.


i did use that. i just couldn't run all of the tests at once. i asked for help on r/redditdev. [here](https://www.reddit.com/r/redditdev/comments/3tqvfk/software_testing_on_reddit_code_coverage/)


;-) Thanks


This commit was a mistake and has since been undone. Please ignore.


please collaborate upstream to come to a better solution instead of just removing this feature


>You can use the Windows SDK, along with your chosen development environment, to write Windows Store apps and desktop apps for Windows 8.1 as well as Windows 8, Windows 7, Windows Vista, Windows Server 2012, Windows Server 2008 R2, and Windows Server 2008.

windows SDK 8.1 also supports windows7.


This was a potential bug. Calling .map on a Set is not necessarily injective like the following code seems to assume.


please contribute upstream


Done.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=287038&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 26 (20 new), passed: 260, muted: 3 Build time: 01:28:04
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyAssaySummary: <no details avaliable>

org.labkey.test.tests.CDSTest.testSummaryPageSingleAxisLinks: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

##### there are 16 more failed tests, see build details
```


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=287038&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 26 (20 new), passed: 260, muted: 3 Build time: 01:28:05
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyAssaySummary: <no details avaliable>

org.labkey.test.tests.CDSTest.testSummaryPageSingleAxisLinks: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

##### there are 16 more failed tests, see build details
```


Should the line
     local fileName="$2"
not be replaced by
     local fileName="$1"
because there is only only parameter left ?


Thanks.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.827](http://teamcity.monogame.net/viewLog.html?buildId=11673&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


I love you XD
Nice to have you on the team!


That's been fixed already.


so `getAttributeLong()` is better?


This is removed in next PR.


@UniqMartin Apologies for the delay, my Homebrew backlog is long. I'll get to it/you ASAP :heart:


This is better in next PR.


The name of method is changed.
If app is packed as jar, the result will be same.


What about user's of RealmBaseAdapter? do they also need to manual unregistration?


left this for accident, whoops


Since you aren't using the result, I think `if dataSource[i] is SuggestedSite` is a bit more succinct.


this is not needed, since laravel makes this automatically with external keys


please contribute upstream


Look into this - https://travis-ci.org/ImagicalCorp/PocketMine-0.13.0/builds/92772462


In der Commit message würde ich lieber schreiben, was man gemacht hat: "Allow %21 instead of ! in deeplinks"

in einer zweiten Zeile dann sowas wie "closes #19"


Why not make ordinary github fork, apply your changes, make PR to upstream? And all that social coding?