@Floessie Here's a list of `copy constructor` related issues from Cppcheck (checked on master branch). I also included some possible leak issues which may be related.

```
[rtengine\EdgePreserveLab.h:50]: (style) 'class EdgePreserveLab' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\EdgePreservingDecomposition.h:76]: (style) 'class MultiDiagonalSymmetricMatrix' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\EdgePreservingDecomposition.h:134]: (style) 'class EdgePreservingDecomposition' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\array2D.h:70]: (style) 'class array2D<T>' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\array2D.h:70]: (style) 'class array2D<float>' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtexif\rtexif.h:181]: (style) 'class Tag' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\procparams.h:1361]: (style) 'class PartialProfile' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\procparams.h:1364]: (style) Class 'PartialProfile' is unsafe, 'PartialProfile::pparams' can leak by wrong usage.
[rtengine\procparams.h:1365]: (style) Class 'PartialProfile' is unsafe, 'PartialProfile::pedited' can leak by wrong usage.
[rtengine\LUT.h:88]: (style) 'class LUT<T>' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\LUT.h:88]: (style) 'class LUT<float>' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\rawimage.h:35]: (style) 'class PixelsMap' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\cplx_wavelet_dec.h:33]: (style) 'class wavelet_decomposition' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\cieimage.h:27]: (style) 'class CieImage' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtgui\profilestore.h:121]: (style) 'class ProfileStore' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtengine\shmap.h:28]: (style) 'class SHMap' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtgui\previewloader.h:44]: (style) 'class PreviewLoader' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtgui\previewloader.h:83]: (style) Class 'PreviewLoader' is unsafe, 'PreviewLoader::impl_' can leak by wrong usage.
[rtgui\previewloader.cc:32]: (style) 'class Impl' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtgui\thumbimageupdater.h:44]: (style) 'class ThumbImageUpdater' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
[rtgui\thumbimageupdater.h:92]: (style) Class 'ThumbImageUpdater' is unsafe, 'ThumbImageUpdater::impl_' can leak by wrong usage.
[rtgui\thumbimageupdater.cc:33]: (style) 'class Impl' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.
```
Running clang-tidy sounds good too :+1:

Ingo


##aminoooo


Must use same for baseawesome and default ....


First Commit: Imported Builder, Guitar, GuitarSpec, Inventory, Type, and Wood classes.


Should not this be changed back to "boundless-test". Using localhost causes issues with certificates and also many other issues (while running the tests in the Tester plugin), switching back to boundless-test and is all ok.


Haha - true! It's completely redundant. Whoops. Thanks for spotting this.


Now trying out this... https://github.com/lucabrivio/Stockfish/compare/0162fb8...8c3f5cf http://tests.stockfishchess.org/tests/view/57f4e3020ebc59038170f9a2


Why not use the tarball? Now it will try to re-download zeroconf on every run.


BUG修复


In `<algorithm>` there's `std::max(std::initialization_list<T>,Compare)`, so instead of nesting binary `qMax`, you can use `std::max({a,b,c})`.


"Line" 34: "supportive" = "supportative"


@EskelCz Why `subjectMemory`, don't you see how it's used in `tick`?


I mostly use IRC. saidinesh5 on Freenode in channels like #sailfishos-porters : https://webchat.freenode.net/?channels=sailfishos-porters


add


Nice.

Zoek ook eens uit hoe je met op een succesvolle POST naast CREATED en het aangemaakte object, ook de Location header kunt terug geven, zoals de REST spec voorschrijft.


It might be nicer to fix this in the style component in resources/themes/cura/styles.qml:240 . That way we never have an invisible-but-animating progress bar.


thanks


öffne**n**de


hello


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1259](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14938&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1260](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14940&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


You'll want to use ECMASCRIPT-6.0 rather than ES6 (in getUserMedia, the ES6 reference has been defined as part of the localBiblio property in the ReSpec configuration, but using the specref-defined name is best)


Hello Rashed


I would rename this variable : 'global_keys' and the method 'get_global_keys'.


idem


I only have real unit tests for the Angular client, the far more relevant API tests _start the server_ in a separate process and run against that using [frisby.js](http://frisbyjs.com/). I wouldn't want _that_ to happen on every code change triggered by IntelliJ... :grinning:


I did, in the next commit (408a868)


Created Unit testing script. Understanding with basic example


Problem with Builder type migration.


das & (definition des shortcuts) beim L wäre wohl besser, so war es vorher; ausser es spricht was dagegen


what is unicode_field ? I don't see where it has been defined


ns > namespace to make it clearer (except if ns was already used as a convention across all the code


![](https://github.trello.services/images/mini-trello-icon.png) [raif parser](https://trello.com/c/LBxFXpWx/9-raif-parser)


@rauchg does this mean we will see a new release soon?


Thanks for the merge -- and for doing the requested changes yourself!


Thanks again, you have right, It should be fixed with my last commit.


OK, this makes sense.


superQuote should not be used anywhere by now


ok, I'll remove those assertions


Actually looking at this change again, I think I was confused here. This code does not actually use the second_geometry to get the missing start node. second_geometry.front() is a node on a different segment. This needs to remain as-is. Sorry for the confusion.


Just comment: In my environment, 100,000 iterations was not enough.
(1,000,000 iterations is enough)


What is `append_d`? It is still not an informative name.


Den gang brugte jeg ikke Gedit.


Yeap @vinkla, really soon 😎


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1259](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14938&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 259, ignored: 3 Build time: 00:14:31


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/569/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=ember).


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1260](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14940&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 259, ignored: 3 Build time: 00:14:36


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1261](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14941&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Hmmm....


The character dot will match any character. This regex will match for example 1a2a0. If you want to match the literal dot,  you should  to escape it with \\. .
(\\d+) \\. (\\d+) \\. (\\d+)


test


will that do something related to seek , casue i have an issue regarding seek from matroska if i play that file from http source , the new playback starts from wrong start position instead of what is requested in the seek event.
Thanks in advance


please don't
look
like
shit


ich weiss nicht so recht.. bezug ist hier auf eine Export Funktion im Loganzeiger. der String wurde mal mit "Einfache Textdatei" übersetzt; irgendwelche ideen?


er


ein space zuviel?


You should prevent it from being evaluated more than once. Use static initializer to set.


FYI, this seems to break on Python 2.7:

> Python 2.7.11 (default, Mar  1 2016, 18:40:10)
> [GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)] on darwin
> Type "help", "copyright", "credits" or "license" for more information.
> >>> from argparse import _UNRECOGNISED_ARGS_ATTR
> Traceback (most recent call last):
>   File "<stdin>", line 1, in <module>
> ImportError: cannot import name _UNRECOGNISED_ARGS_ATTR


einmal "nach der" zuviel


Timing information should not be regarded as `DEBUG` class. Instead, you can put the following code in `TIMING`.


:-)  didn't see that one


test
sadas
dasdsaf

hdgas


Olhei novamente, agora entendi. Inicialmente achei que a vírgula estivesse sendo concatenada após a descrição. Neste caso, como você inicializou retorno com a descrição do primeiro item, está certo mesmo. 👍


By itself that test case doesn't really guarantee anything, it would be more useful as part of a bytecode test in https://github.com/lampepfl/dotty/blob/master/test/test/DottyBytecodeTests.scala


What about the `get`, `post`,etc methods? They aren't tested.


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 71](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14944&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) is now running


Não precisa, pode deixar assim.


tasda
das
f
as
d
asds


Looking nice Bram! Can you just add a seperate file for our extentions and pictures. Make it snappy!


dasdfsa
das
dfa
sf


@ctron can you please tell me what are the changes done on the CryptoServiceImpl.java? The diff is impossible to understand.


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 71](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14944&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1261](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14941&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 259, ignored: 3 Build time: 00:09:22


Thanks! that is a good idea - Let's do that!


you can still do that, for example the tests you can see here https://github.com/OWASP/Maturity-Models-QA/tree/ea2328ff0ce71fac772f3e99dbdcf0170f53a4b1/test/jsdom/views (using JSDom) and here https://github.com/OWASP/Maturity-Models-QA/tree/ea2328ff0ce71fac772f3e99dbdcf0170f53a4b1/test/http (using direct Http calls) are all executed against a live server (locally against dev server, on CI against a deployed docker version)

And they also run very fast (I have a separate place for the slower tests that don't need to run all the time)


this
is
a
multiline
comment
please
render
properly
on
discord


@eliseudaroitjr você vê alguma melhoria a ser feita nesta classe?


@eliseudaroitjr escrever mais cenários de testes (com noturnos, com elfos normais, etc)


I forgot to remove the threadpool file.


No need for `base_uri` any long as we can just use `opentaxii_domain`.


I would change `opentaxii_domain` to:
```
opentaxii_domain: "http://{{ opentaxii_host }}:{{ opentaxii_port }}"
```


to start
docker build -t ubuntu-php-google-pic .
docker run --name google-pic-container -t -i -p 80:80 ubuntu-php-google-pic


This also allows a user to define an https or http server via the properties file


WFM, I'll change it and re-commit


ein space zu viel, Gegenstell**e**


Could you please revert to cb29efa7560506c16dc1913399317696997ff2d7?

That CentOS 7 support is work in progress as it needs a lot of work before it can be merged... I'm just getting to know that I'm working on it.

I will recreate that PR.

Thanks.


now lets see if } this { fucks it up [ ] "wew"


![qq 20161005201540](https://cloud.githubusercontent.com/assets/22425022/19112680/cfe34128-8b38-11e6-8395-d2e6ebf3a597.jpg)


"is this shit" [ ] wew


Pq esta classe é abstrata?


You're welcome 👍 And thanks for your updates.


Pq um arraylist do tipo Personagem?


pq `protected`?


Done 1 (issue #6): https://github.com/idg10/azure-sdk-for-net/commit/7f5bc02e2182175081f96feaeb45431e02204fb7


First Commit


Ты серьезно считаешь что без поддержки мультиязычности можно обойтись?
В общем за реализацию просто стоит тебя пристрелить


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1253](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14890&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **UNKNOWN**
Summary: Canceled (Compilation error: Scala compiler (new)) Build time: 19:49:19


Essa é uma boa ideia 👍


test


May be related to an issue reported in JClic:
https://github.com/projectestac/jclic/issues/18


Loginvorg**a**ngs


Not absolutely necessary, but I would change `uri='%s/services/collection'%base_uri` to the following.
```
uri='{{ opentaxii_domain }}/services/collection'
```


Compilerfe**h**ler


o ideal aqui seria já retornar o elfo ou quebrar a iteração. Imagine que temos um array com 1 milhão de elementos, se encontrarmos o elfo na segunda posição, percorreremos todas as outras.


Interesting.  For future regex code we should probably include the tested strings in the commit comments.  That way we can reference back to them as needed.  eg for testing, improvements, etc.

This also leads into another thought that's been popping into my head a lot recently... how do we improve our current automated testing to cover more runtime stuff?

We have Travis CI doing "does it compile" testing, and our [very basic unit tests](https://github.com/sqlitebrowser/sqlitebrowser/blob/master/BUILDING.md#building-and-running-the-unit-tests).  Any idea if they could be extended out to include testing this?


@erlend-sh the only option you need to toggle is the "FULL ACCESS" under the "Mail Send" section.  All the other options can be set to "NO ACCESS". Sorry for the super late reply, just stumbled upon this in my inbox as I was looking through some old email.


ainda não sabemos o que é `static`, retirar!


OK, I pushed a new commit. I decided to modify the expected score value instead of tweaking gap opening penalties. Let me know if it is a problem.


I see there is also an `opentaxii_bind` variable.  If we are using that somewhere (double check that we are), can we generate that?
```
opentaxii_bind: "{{ opentaxii_host }}:{{ opentaxii_port }}"
```


hervorheben statt aktivieren?


re-commit


![Uploading AA1E2413-18A9-4C3F-BDB3-D08ED20277F3.png…]()


test
more
test


wir**d**


closes #28


@jagustin8 how does it relate to my commit? If you have a problem with an example application please report an issue in [JIRA](https://issues.apache.org/jira/browse/WW/)


além desta verificação temos que verificar que o único elfo do exército seja o que informamos.

Imagine que lá dentro do método alistarElfo a implementação seja:

```Java
public void alistarElfo(Elfo elfo) {
    // ignorar completamente o parâmetro
    this.exercito.add(new ElfoNoturno("Aleatório"));
}
```

Esse teste continuaria passando, compreende?


I guess the best thing to do is to have a look at PR #542, it should be more visible there.


" " hi


The code was already defaulting to use 'localhost', right?  It is not ideal, but I don't think there is a better option right now.


Pq vc criou a classe abstrata? O cliente só quer exército de elfos e como temos testes, no futuro não teremos problemas em refatorar para herança (caso necessário), assim como foi com as outras classes.


Thanks...


shall it be `ready: this.postsSub.ready()` ?


Não ficou legal retornar `Object`. Entendo a ideia de ser genérico, mas aí o código que for consumir esta classe vai ter que ficar fazendo conversão explícita a todo momento e isso aumenta bastante o acoplamento. Uma solução intermediária seria mudar a classe abstrata para exigir retornar `Personagem`. Mas ainda assim é uma solução muito complexa. #YAGNI


Commit


" " eh


Bravo! Uspeo si.


Lorem ipsum dolor sit amet, consectetur adipiscing elit. Curabitur dui quam, fermentum a turpis in, imperdiet scelerisque justo. In congue vulputate consectetur. Sed iaculis facilisis hendrerit. Mauris rutrum tincidunt nibh. Duis sollicitudin erat ut lectus lobortis laoreet. Phasellus posuere, magna quis convallis suscipit, mi ante cursus justo, quis gravida enim sem nec tellus. Curabitur pulvinar, nunc vitae elementum porta, ante massa cursus tortor, id scelerisque nulla eros ac sem. Maecenas a nisi sed purus tristique congue eget a risus. Sed at ornare est, nec euismod lacus. Curabitur non arcu porta, interdum ligula sed, pulvinar ante. Vestibulum ut dictum magna. Sed eu efficitur tellus, at rutrum nisl.

Sed ipsum tellus, lacinia tincidunt congue ac, fermentum eu augue. Nullam malesuada, justo quis iaculis viverra, sem diam bibendum nibh, quis tempus nibh ligula nec felis. Proin luctus, ipsum eu convallis rhoncus, lacus sapien accumsan massa, a blandit nibh urna sed nisl. Aliquam non varius nibh, id dignissim magna. Suspendisse accumsan auctor mauris, porttitor elementum tellus vestibulum at. Ut eleifend nibh neque, non tincidunt justo scelerisque ac. Fusce luctus, ipsum sed elementum vulputate, ipsum justo malesuada diam, vel gravida nisl justo non risus. Sed finibus augue a enim mollis laoreet.

Curabitur dignissim, sapien et pretium varius, diam lacus aliquet felis, at hendrerit tortor lacus eu risus. In elementum ipsum mauris, a luctus massa interdum et. Etiam gravida libero tortor. Nulla sit amet aliquam lacus, vitae dignissim nibh. Sed dapibus tellus feugiat massa egestas cursus. Nulla placerat lorem vehicula mauris fermentum, quis molestie magna dapibus. Quisque in porttitor lorem. Pellentesque euismod bibendum mauris sit amet facilisis. Sed imperdiet est ac imperdiet fermentum. Fusce et aliquet massa. Vestibulum fringilla eu nibh sit amet volutpat.

Sed sed nulla finibus mauris mollis molestie non quis lectus. Vivamus accumsan, eros quis vehicula sagittis, purus mi mattis metus, nec iaculis lacus nulla ut nulla. Aliquam erat volutpat. Curabitur vitae ornare felis. Phasellus eget lorem quis nibh pulvinar feugiat vitae vel lorem. In tempus venenatis magna, porta tristique felis rutrum id. Proin nec dui sit amet justo venenatis lobortis a in mi. Interdum et malesuada fames ac ante ipsum primis in faucibus. Vivamus mauris dolor, aliquam in leo sed, vehicula molestie tortor. Proin egestas dapibus neque, non viverra quam consectetur ac. Nullam sodales faucibus purus. Praesent ex metus, cursus eu libero nec, tincidunt faucibus purus. Donec laoreet lacus vel tellus congue, et molestie risus ornare. Fusce lobortis augue erat, sit amet scelerisque urna mollis at. Vivamus ut elit libero. Etiam ultrices eget erat eget egestas.

Aenean eget neque eget metus pretium lobortis sed sed nunc. Proin scelerisque facilisis purus, sit amet congue leo faucibus quis. Praesent a lacus libero. Suspendisse vitae dapibus tellus, ac convallis leo. Curabitur nec nibh ac massa luctus elementum accumsan vel orci. Ut et consequat neque, semper auctor mauris. Phasellus quis mi vulputate dui posuere blandit. Aenean ac metus vitae tellus condimentum scelerisque sed eu massa. Sed vel lacus nibh. Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia Curae; Duis dignissim sollicitudin nunc, vitae tempor sapien efficitur nec. In turpis neque, feugiat auctor tincidunt ut, pellentesque eget ex. Sed non eleifend enim. In sollicitudin metus sit amet eleifend placerat. Vivamus a turpis est. In mi neque, consectetur vel ultricies nec, consectetur ac ex.


unter Berücksichtigung der Groß- und Kleinschreibung auf Übereinstimmung prüfen [oder ähnlich, finde gerade nicht wirklich erwas passenderes]


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1262](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14952&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1263](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14953&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1262](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14952&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:40


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1263](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14953&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:08


Test


html files


HelloWorld


gracias


Or better yet (instead of comments), add unit tests for these regex code parts.

Well, by adding more unit tests. [This][1] looks like a good intro.

  [1]: http://doc.qt.io/qt-5/qttestlib-tutorial1-example.html


viu como não tá legal? dentro de uma classe `ExercitoDeElfos` você já está convertendo para `Elfo` os outros objetos.


`id` はリンクのアンカーとして使うか、javascriptで使用する用途以外は使わないほうが良いよー


Yes that fixed it :+1:


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 93](http://build.prxm.it:8811/viewLog.html?buildId=10004&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Se as duas String estiver no mesmo local da memória é True... Com equals() ele compara os valores. Já fiz o reféctor no código.


Sinceramente, não entendi o questionamento.


例えば、この部分と同じcssを当てたい別の何かを作るときに、同じ内容のcssをもう1つ作る必要が出てくる。
同じコードを1つのプロジェクトでもサイトでも2回以上書くことは基本的には悪手と見なされてるよー　これ大事よー


MERGE shit..


Good point, resolved it.


Outdated documentation, I think.


#NOT FINISHED


:D


Very nice! Thanks for doing this @TimurTripp


`p` で `h1` を囲むのは基本NGよー
`h1`の下のテキストを何かで囲みたいときは`span`とかがおすすめよー

参考:: [ブロック要素とインライン要素] (http://www.htmq.com/htmlkihon/005.shtml)


@s1341: that would rock!


おｋ！


Oh, sorry Lukasz... my bad, new to GitHub, thanks..

________________________________
From: Lukasz Lenart <notifications@github.com>
Sent: Wednesday, October 05, 2016 8:30:18 PM
To: apache/struts-examples
Cc: jagustin8; Mention
Subject: Re: [apache/struts-examples] Adds some more complicated pattern (2697ee7)

@jagustin8<https://github.com/jagustin8> how does it relate to my commit? If you have a problem with an example application please report an issue in JIRA<https://issues.apache.org/jira/browse/WW/>

-
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/apache/struts-examples/commit/2697ee7417c4b87354e287bf50cf8f8364aff479#commitcomment-19301568>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AVlf_zXB6LWKCx48o9aliCEMFVFVc3Ujks5qw5hagaJpZM4KOleA>.


@mcostalba Hi Marco, sorry for bothering you (again).
Is this [test](http://tests.stockfishchess.org/tests/view/57f4c2950ebc59038170f998) in your eyes worth to be rescheduled as SPRT (and if yes with which bounds)? Or should we possibly investigate why it produces a different bench-number at all?


you may want to add this to remove docker bloat. It is from same best practices article
&& rm -rf /var/lib/apt/lists/*


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 3.0.0-beta.1+9](http://build.prxm.it:8811/viewLog.html?buildId=10004&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 267 Build time: 00:07:35
### Failed tests
```
GlobalSetupFixture: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_multiple_password("v3").verify_tika_job_with_password: <no details avaliable>

```


cssを書くときに、要素名を入れるのはNGになってきてるよー
cssの優先度に変な影響出てくるからだよー

`ul#menu` -> `#menu` または `.menu`
がこの場合望ましいよー

参考: [cssの優先度](http://creator.aainc.co.jp/archives/4947)


The two different loops are very similar to each other. It would be great if we could come up with a way to do only one loop. For example, you could create a variable named `items` and set it to `data.artists.items` if the user selected artist and to `data.albums.items' if the user selected album. The you could just loop through `items` and not write the same code twice.


Eu entendi o enunciado errado mesmo. Lá tu fala em abstração, ai já pensei logo em implementar uma classe abstrata. Vou corrigir. #YAGNI


Yes, I have created an issue:
https://github.com/xor-gate/syncthing-macosx/issues/13


I tried to follow PEP by using PyCharm. So many parts of source code has been modified.


You should put calls to `app.use` before routes (i.e., before calls to `app.get` or `app.post`


Added tests now in latest commit: https://github.com/cerebral/cerebral/commit/944485d1bd3bcd37eb83880c84aa5ce52745cbcd


Sim, entendi e teria que fazer isso sempre que quisesse utilizar o ArrayList<Object> exercito.


But why though?


@etcimon, I'll try to fill in the missing pieces based on the vibe.d driver once the Posix one is a little more complete. Then libasync can be used as a true baseline in benchmarks, so that a proper comparison becomes more practical. From a rough look over the API, do you think libasync has all the required functionality? Especially the `cancelWait` methods are important to have - the current vibe.d is quite brittle in the area of timeouts and exception handling due to missing cancellation support.


@fearphage JavaScript everything! We can't expect devs to know hundreds of different languages and object notation languages...


@forki Sorry, I accidentally pushed it directly to the __master__
```
let internal findVersionsForPackage(v3Url, auth, packageName:Domain.PackageName) =
```
`findVersionsForPackage` accept `v3Url` as first parameter, so we should not call it `auth` in `FindVersionsForPackage`


Now #1300, #1322 and #1338 can't be merged, might be due to these changes.


So this looks good to me. You should be able to redirect people to the page to accept the cookie policy if they haven't already done it. You could do this by adding something like the following upstairs:

```js
app.use(function(req, res, next) {
    if (!req.cookies.galleta && req.url != '/cookie') {
        res.redirect('/cookie');
    } else {
        next();
    }
});
```


I'm afraid you need to propose the patch at GNOME's Bugzilla, this is a GNOME mirror and unfortunately we cannot disable PRs


# Lee una sola linea que contiene el valor a quitar y la cadena, devuelve la cadena sin el valor
# Read a line with one value and string, return the string without this value


I didn't know about version 2. I will update. Thanks!


This is the first commit I've made on a HiDPI screen. If there's a problem with window size, I'll have to look at this.


I don't understand what you mean by "fix projects.cjsx".

I was just rewriting the text below the labels in project settings, since it made no sense to me (it may have made sense to somebody at some point, but was so abbreviated even I couldn't work out what it means anymore).


Thanks, that's useful to know since I also submitted a PR on a different subject :).


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#e43hSSL7RjCzbyPbrXDtjQ


Note also that I made no changes to messages at the top or to courses (which don't have these confusing extra text).


why did you change the params?


Do we still use addie? If not we could remove it and stop maintaining it.


good to find this here, but can you make it in console to be more clear. `gem install fluent-plugin-grok-parser` now grap fluentd version 0.14.6. so if you want to work with fluentd 0.12.x(29). you will discover this fluentd version changed while installing some gems. and what make even worst is my fluentd configuration for 0.12 will cause error with 0.14 version.


@alexey-sidnev class vector done


The signature of `findVersionsForPackage` (1st is url, 2nd is auth)
```fsharp
let internal findVersionsForPackage(v3Url, auth, packageName:Domain.PackageName)
```

previously you call it like  (1st was auth)
```fsharp
let! raw = findVersionsForPackage(auth, nugetURL, package)
```
from
```fsharp
let FindVersionsForPackage(auth, nugetURL, package)
```

What result as very strange intellisence `nugetUrl:Auth option`
![image](https://cloud.githubusercontent.com/assets/1197905/19114321/3101df1e-8b16-11e6-81d3-32588698f913.png)

So I just changed param names


Why did you strip my `Signed-off-by` and also remove my authorship of the commit?


Thanks - won't pretend to understand JavaScript but great to have more PCT/CycleStreets cross pollination!


This ensures that in development, even if you run a minified (production) version you won't push that.
Is there another way in your project to run a production version without the possibility of pushing the generated things ?


New Files


@bradvogel Sorry, I meant for the PRs - I thought you had actioned them, hence my message :)


pode usar `assertNull`


added a comment on 10th line.. 👍


@samaaron is this edit a typo?


Faça um find (ctrl+f) no arquivo pelo nome `porStatus`. Você verá que a variável é utilizada apenas dentro do método, correto?

Neste caso, se a variável é utilizada apenas dentro do método, porque deixá-la fora do método?


The same "issue" in docs - http://fsprojects.github.io/Paket/reference/paket-nugetv3.html
This change does not affect behavior of Paket


The `pidWalker` (not the reader) works with a ticker in the original implementation, so for compatibility I left it there and I let it tick every millisecond. There could be room for improvement.

The reader is synchronous because it doesn't waits for the results before returning.


pq existe essa variável `porStatus` na instância se ela é apenas utilizada dentro do método `buscar`? cuidado com o Ctrl+C :)


Changed date to '2042...'


Te entendi, esse conceito foi apreendido do técnico. Sempre declarar e a cada vez que chamar o método só inicializar ela novamente.


@smarter OK, can you turn it into a ByteCodeTest please?


Yeah, that looks good. Thank you!

On Tue, Sep 27, 2016 at 10:18 PM, Christof Senn <notifications@github.com>
wrote:

> Thanks for pointing out. It's done in commit 5fc292f
> <https://github.com/6bee/aqua-core/commit/5fc292f9b53a74663ace3b4b20ab474b0d27e925>
> in case you'd like to review.
> Please note I'm not using Attribute.IsDefined() since this method is not
> available in netstandard.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/6bee/aqua-core/commit/9dcc464c9f3945fb39ff7ae4b5563fa6cb99fb82#commitcomment-19197466>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHN5dUNp1ufJaqVuyE54CvffVIHPmo9dks5quXopgaJpZM4KHexe>
> .
>


OK -- I'm trying not to refactor anything until after the top-down rewrite is live.


comparação de strings com `==`... falamos sobre isso em aula, não?


@alexey-sidnev spisok done


Until today, I wasn't even aware that Travis supported non-yaml... in a `.yaml` file no less. I personally don't feel like knowing javascript *and* yaml is such a tall order to me but Phil Drummond.


Do we actually handle "serialization_failure" ?


@PowerKiKi you could give a try to phpdg to run coverage tests instead of xdebug.

I didn't go deeper to understand the difference between xdebug and phpdg but its possible to greatly enhance code coverage using phpdbg.

Some reference: https://hackernoon.com/generating-code-coverage-with-phpunite-and-phpdbg-4d20347ffb45#.tgo6902o5


You can use `iota` to make it more idiomatic. Ref: https://splice.com/blog/iota-elegant-constants-golang/


> Especially the cancelWait methods are important to have - the current vibe.d is quite brittle in the area of timeouts and exception handling due to missing cancellation support.

The cancellation would probably have to call `kill` on the event and each implementation should close the file descriptor that way, although I don't think this does a callback. On the vibe task side of things, I guess it should be resumed with an exception?

> Then libasync can be used as a true baseline in benchmarks, so that a proper comparison becomes more practical.

This could probably be a good place to start implementing the new [proactor pattern](https://github.com/etcimon/libasync/issues/66)


@dergraaf GitHub added this commit, I wasn't aware it did that when I enabled GH sites. You may force push it away.


迫真failing


This program really works!!


It makes me wonder where else the direction signal is being changed…


Bonjour


Ou Coucou


@hhostermann mais atenção...

qual tipo de convenção para métodos em Java?


Her legges det til mer i oblig 3.


Please, create a new pull request for the homework, don't stack it on top of the previous one


Closing the file descriptor sounds like too much. What it should do is just to cancel the current operation (as far as possible) *and* make sure that the callback will never be called. The latter is crucial for making the fiber based layer memory safe.

Yeah, proactor would be the best fit, but reactor should also be fine for now. The Posix event driver also uses reactor internally, although using the AIO API in the future where possible would be nice.


Et voilà, on a les droits, houuu, ... et pour pas cher, à mon avis, il a marché ! Il a marché à fond le gars !


@TooTallNate Could you please publish a new version on npm? Required for node 5 (block-scope restricted to strict mode). Thanks!


La vie ne va être facile par ici !


C'est un toto


On est d'accord


I don't think ExplosionPrefab should be instantiated


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1264](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14961&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Ok, valeu pela dica.


pq vc implementou este método?


Done with review.  Good find!


Doing...

```yaml
transition:
    [show, hide]:
        time: 0s
```

is equivalent of??

```yaml
transition:
    show:
        time: 0s
    hide:
        time: 0s
```


If the file descriptor stays open, the callback will be called, the question is more about where to drop it, or whether it should even be dropped. If you drop it, the file descriptor won't fire it again so you'd queue it either in libasync or in vibe.


@OtavioBubans

cuidado, é para retornar o Elfo, não o nome... (afinal, o nome já estamos informando no parâmetro, não faria sentido percorrer todo arraylist para retorná-lo)

O que deve ser corrigido aqui?


Commit comment is not correct : it is "**#783: Make cdNom/cdRef tree selection widget resizable** "


Commit comment is not correct : it is "**#783: Filter cdRef rows in cdRef tree selection widget**"


Capturing from chat: these are probably a great addition, too. I see the following issues here:
- please open a pull request, once you force push to this branch these comments will be lost
- you copy `lane_description`, could take a `const &` instead
- why manually reverse-iterating over lanes below?

It still makes sense to provide functions for the number of lanes to the right / left. You _could_ implement them as calling your function and simply returning the vector's `.size()` but then you're doing vector constructions/destructions just for getting number arithmetic going. Maybe add:

    size_t numberOfLanesToTheLeft/RightOfLanesInTurn(const RouteStep& step);

(or LaneID instead of size_t) and then `step.intersections.front().lane_description.size()` gives you all lanes, `current.intersection.front().lanes` gives you the lanes in turn object with number of lanes in turn and first from the right

---------------------------------------------------------------------------------------------------------------------------------------------

To clarify, the original ticket was about bringing in a bit of abstraction and re-using such helper functions. They hide for example having to know about the intersections array and why the first entry is the one we want / is always valid. Or more generally, how lanes are represented in the `RouteStep` object at all. The same could be done to hide more impl. details, such as

    size_t lanesInTurnManeuver(const RouteStep& step);`

or similar.


Sure.


Sorry, this commit isn't supposed to be linked to #20, but to https://github.com/vshaxe/vshaxe/issues/20 instead


Caption can't have Markdown in Telegram


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1265](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14964&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


@aslakknutsen Do you mean if is handled in the caller?


Checking whether ncurses is installed is tricky, see the logic in (and storied history of) the [conf-ncurses opam package](https://github.com/ocaml/opam-repository/tree/master/packages/conf-ncurses). There is enough information there to compute a reasonable approximation:

- ncurses is known to be preinstalled on darwin and {open,net,free}bsd, as listed in [ncurses.1+system](https://github.com/ocaml/opam-repository/blob/master/packages/conf-ncurses/conf-ncurses.1%2Bsystem/opam)
- for Linux distributions, `pkg-config --exists ncurses` returns 0 only if the module exists, so it is a correct way to ensure that ncurses is available. Some distributions, in particular OpenSuse, do not install the pkg-config package, but I think it's fine if the test is disabled there (even if ncurses is in fact available).


Faltou um teste básico: tentar adicionar elfos normais.


@thewonderidiot This breaks my ability to build yaDSKY2 in a way I'm finding difficult to resolve.  In wxWidgets 2.8, which is what I'm using, and what we recommend using in the build instructions, wxBitmapButton has no Bind method, so ProButton->Bind(...etc...) doesn't compile.  Or at least, that's what the compiler says, and the wxWidgets documentation is so opaque that it's difficult to determine from it.


sorry! i typed `sizeof(char*)`, should be sizeof(char)


sim, mal costume meu estou estudando mais sobre string pra n cometer tantos erros assim.


The problem is that there needs to be an association between the operation (e.g. the `read` call) and the callback invocation. It would of course be possible to store a set of cancelled waiter IDs in the eventcore driver and pass an ID as part of the delegate for the callback, but that wouldn't be optimal, as it requires some kind of dynamic memory allocation.

Also, certain operations really need to be able to be cancelled, for example a socket write that times out or that is interrupted manually. It would be really strange if the socket would, at some later point, still carry out the operation. Closing the handle would also not be desirable - especially a cancelled read operation should be resumable later.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Kr4-aTHhTFiGqsddIXIBEQ


how about your work recently?


⚠ ⚠ ⚠

Comparação de strings com `==` em Java não é legal.


WTF?


Tell me Xiaolin Su's github name?


cuidado com o nome do cenário que não reflete o que acontece dentro dele


idem acima!


e os testes do método `buscarPorNome`?


We don’t use it, but it’s open source. I guess no one is using it, but I’m not sure. It’s a small cost time wise, but if we feel that no one will ever use it, then we should ofc remove it!


The hypothetical problem I'm attempting to solve with this configuration is a system that can't handle a lot of threads. Imagine a connection pool with hundreds of connections that are slower than average. Your validation would naturally spawn a thread for each connection. If this is a problem for your deployment platform, you can choose to turn off threading at the expense of potentially blocking forever.

I can certainly be talked out of this line of reasoning...


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1266](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14965&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


If I'm reading this correctly:

1. deploy instance 1
2. deploy instance 2
3. instance 1 starts migration to v 2
4. instance 2 starts migration to v 2
5. instance 1 commits all good
6. instance 2 attempts commits but fail due to "serialization_failure"
7. instance 2 panic and shuts down
8. instance 2 is considered unstable from deploy perspective and may or may not stop the whole roll out of the new version

?


curiosidade.. pq vc fez esse método?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1264](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14961&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 261, ignored: 3 Build time: 00:15:07


e os testes para os elfos normais?


onde está a implementação do exercício 2?


Am I an idiot, or does this not change anything, but instead asserts that some things can't happen?


isn't it` run tightvncserver ` ?
and the :1 is by default.


Work like ###### #sharp7


als niet weet hoe het werkt niet zomaar wat doen please, vraag eerst aan mij hoe of wat


Did this in fact fix the issues? I can't remember the last time the issues occurred, but it's still an issues when I run the server address on my computer using all the files cloned from the Github. All the files are up to date and so I'm not sure why this problem exists? Could it be maybe I don't have the last version of something?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1265](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14964&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 259, ignored: 3 Build time: 00:11:27


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1267](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14968&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Wow, trop bien


Se tivéssemos 20 tipos de elfos, aqui seriam 20 métodos diferentes?

Leia as dicas do "Hacker": https://gist.github.com/bernardobrezende/d1b00bec7fcf248ab96d5317e47eee80#gistcomment-1890032


@jeniffer-costa não encontrei a classe de testes, por isso não vou seguir a revisão, ok?


okay got your point. let me think...


## This is a markdown file


@enricotagliavini A worthwhile precaution, but I think now sven only allows a single directory name in passthrough, and not full paths. If this should change in the future, we should probably do that. Although specifying NOFOLLOW while renaming is probably better than realpath, as it avoids race conditions:
```
dst_dir_fd = os.open(dest_dir, os.O_RDOLY | os.O_NOFOLLOW)
try:
    os.rename(tmppath, filename, dst_dir_fd=dst_dir_fd)
finally:
    os.close(dst_dir_fd)
```
But users should not be able to create symlinks anyway, so this really is just a precaution.


- [ ] task1
- [ ] task2
- [ ] task3


@bernardobrezende

Claro. Devo corrigir o tipo de retorno e o retorno dentro do if para um tipo Elfo


I've noticed this as well now after updating my local SDK build tools. I'm having serious trouble figuring out how to force Nougat SDK builds back to 32 bit. I'll have to dig through some AOSP build files as I'm missing something.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#EFFCE237TeOVtifAuHFftA


this is a pass list focused on the personal information of the victim or the person who we want to test his mail password security


about require.js , see Eloquent-JavaScript https://github.com/marijnh/Eloquent-JavaScript/blob/master/10_modules.txt


I disabled it because I kept getting the timeout error in my before hook, like in this job https://travis-ci.org/StephenDavidson/electron-spectron-example/jobs/138355116


Thanks - couple comments after trying it out of the box:
- Can we set the executable bit on the script
- It has dependencies on python modules so fails - please include standard python installation steps + instructions:
```
Traceback (most recent call last):
  File "./compress.py", line 3, in <module>
    import os, glob, yaml, zipfile
ImportError: No module named yaml
```


There is a better solution to this problem. You need to change .mce-statusbar to .mce-path and that will fix inline link not showing. Nothing else is needed.
![zrzut ekranu 2016-10-05 o 15 44 15](https://cloud.githubusercontent.com/assets/8668816/19115710/47bf95c8-8b13-11e6-886e-b8815e847b9b.png)


cool!


opa. Para usar no teste, verificando se o elfo alistado está na lista.
Usei toArray() para obter array estático, sem possibilidade de alterar.


Changed these, accidentally changed the file name and not the link in .gitignore


The indices change as you remove elements from the list, so this cannot work. You need to determine the last source folder after the while loop.


OK.  Any reason to prefer ascii to 'utf-8'?


Not the same


I made a [PR](https://github.com/facebook/react-native/pull/10065) to bootstrap using that setting, I don't know if you guys looked at that yet?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1267](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14968&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 260, ignored: 3 Build time: 00:09:50


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1268](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14972&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


This changes the meaning of this function, so you really can't do that.


Why have you added this?


The concept of waiting is currently entirely embedded in the vibe driver, so I'm guessing the entire cancellation logic would simply be a matter of checking for a cancellation flag before going through in another code path.

It's funny because the AsyncSocket from the proactor pattern looks almost exactly like the way it should be implemented in this new event driver: https://github.com/Calrama/libasync/blob/benchmarks/source/libasync/socket.d#L428


Would this keep my the `.emacs.d/url` directory from being modified so I dont have to delete it every time I check for spacemacs updates? Because that would be pretty nice.


You *can* make -1 mean forever but you'll need to change godoc and continue to have 0 mean immediately.


Ok


膜拜璠神


New code is well formatted and documented. But some old code is still dirty. Will get better with time.


You should capitalize your name!


Yey! Merge super acum! :)


```
> (concat spacemacs-cache-directory "url")
"~/.emacs.d/.cache/url
```

In case `spacemacs-cache-directory` is not `~/.emacs.d/.cache`, please checkout your configurations.


@aslakknutsen we could maybe simply ignore the serialization error, can we?


See: http://piwigo.org/forum/viewtopic.php?pid=163884#p163884


Alright, will do. I wasn't sure when it would be merged, so I kept it so that it would work within phenotips-niaid in the meanwhile.


Try adding some other HTML elements under the `</img>` tag. Do a google search and just throw some stuff in and see what happens!


This is actually wrong, `req.params` is not the object to validate here, it should be something on the `req` object that holds the formData info. But since it's binary, there's nothing readable on the request at the moment when we're validating it...


上にも書いたけど、
cssを書くときに、要素名を入れるのはNGになってきてるよー
cssの優先度に変な影響出てくるからだよー


The difference is they've split up the vendor repo. This commit tells you what's going on. https://github.com/TheMuppets/proprietary_vendor_qcom_binaries


Utilizei Personagem porque não pareceria certo criar um ArrayList de elfos em um exército genérico, e protected para ter acesso direto ao objeto da superclasse em ExercitoDeElfos.


This is one of my biggest pet peeves with ruby. Even fortran allowed you to name loops and checked that nested loops terminated with the right statement.


what's the rationale for this change? the change was to fix the block expression code, but it seems to me this is overly restrictive. E.g. I want to be able to use blocks of some expressions to construct arrays of the same shape as these blocks ... I need to be able to create TiledRange for the subblocks of the zero-based TiledRange

I'm sure there are reasons, just want to understand what they are and see if fixing block expression logic is necessary instead of this restriction


It was travis-ci/travis-api#333 that was the issue :)


Hm you are right of course, in case of the reactor API, all the logic can be done in the eventcore driver. I accidentally mixed this together with the proactor API in my mind. The `AsyncSocket` that you linked to is really the place that needs the cancellation logic to be a suitable replacement (also `IOMode.immediate` should be possible to model somehow, looks like `exact==true` corresponds to `IOMode.all` and `exact==false` to `IOMode.once`).


Doesn't this function already let us drop all logs?


It looks like this has reverted the buildings pattern that was in `gh-pages`:

https://github.com/tangrams/bubble-wrap/blob/gh-pages/bubble-wrap.yaml#L2692-L2736

This included optimizations around reducing extra filters, with `extrude` nested under `footprints`, and simplifying some unnecessarily complex filters (`$zoom: [15]` -> `$zoom: 15` etc.).

Please make sure the same settings from the master branch are used. I can help tune zoom inclusion of buildings from there.


The commit message and description are a bit messy.  Sorry about that.


Related: https://github.com/pixijs/pixi.js/issues/3055


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1268](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14972&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 259, ignored: 3 Build time: 00:11:17


@robconery - I know you don't like these ("Database pushing right up thru the abstraction"). Using these for the moment, but I think we need to find a way to allow C#- type naming in the code, yet pg-idiomatic naming on the Db.


Agrego los graficos que hice con la maquina del trabajo y falta armar un grafico mejor


### 他には...

- `.sass-cache` はディレクトリごと `.gitignore` に書いてねー
  - キャッシュ系のファイルはサイトを公開する上で必要ないファイルだからリポジトリに含めないのが吉よ
- コミットメッセージはざっくり作業内容を書いてねー。このコミットの粒度は1つのプロジェクトを作業していく上で大事な問題になるから、留意してねー


It compiles with wxWidgets 3.1.0.7  under Windows. Haven’t tested it though, ran in unrelated trouble setting up the project.

BTW any reason why yaDSKY  pulls in agc_engine, etc… . It uses only a few variables that are declared external.  Explicitly defining them in e.g. yaDSKY2.h reduces the number of files needed considerably. I am down to rfopen and agc_utilities.

To avoid more cross dependencies should agc_utilities and e.g. SocketAPI go into a separate library?

From: rburkey2005 [mailto:notifications@github.com]
Sent: October-05-16 9:34 AM
To: rburkey2005/virtualagc
Subject: Re: [rburkey2005/virtualagc] yaAGC+yaDSKY2: Updates to make DSKY handling more 'realistic' (b58253b)

@thewonderidiot <https://github.com/thewonderidiot>  This breaks my ability to build yaDSKY2 in a way I'm finding difficult to resolve. In wxWidgets 2.8, which is what I'm using, and what we recommend using in the build instructions, wxBitmapButton has no Bind method, so ProButton->Bind(...etc...) doesn't compile. Or at least, that's what the compiler says, and the wxWidgets documentation is so opaque that it's difficult to determine from it.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <https://github.com/rburkey2005/virtualagc/commit/b58253b73dccee24043915b5a9a2cf07d616d7b8#commitcomment-19302333> , or mute the thread <https://github.com/notifications/unsubscribe-auth/ALdp2FmCrnvPTaEoWuB8r5MFLIAcz3qdks5qw6dZgaJpZM4KOz2d> .Image removed by sender.


Fixes #324


This should probably be `self::$enable_logging = $logging;` ;)


> The AsyncSocket that you linked to is really the place that needs the cancellation logic to be a suitable replacement

This would involve adding the fiber scheduler though wouldn't it? At the moment the entire libasync API is non-blocking, which means if it's going to block it will cancel automatically


Same


No need to remove it. The normal way to fix these whitespace issues is to use `-`.

Would recommend installing `conda-smithy` in development mode so that you can iterate quickly while testing against a known failure.

Please let me know if you have more questions.

xref: http://jinja.pocoo.org/docs/dev/templates/#whitespace-control


Вот эта строчка что делает? Кажется, что "Параметры" нигде не используется


dfdsfsdfds


I'm not sure why this is coming back - I thought we had agreed this was not going to be useful in the context of recovering from an error?


Fixes #328


Was your plan to use semantic's own .js and have a jquery dependency as well?


Thanks!  This fixed a bug before I figured out what was causing it.


We change this upstream in https://github.com/kubernetes/kubernetes/pull/33942 but I guess I haven't taken into account the rest of the uses of Until in Origin. A quick search revealed that we use it in a bunch of places but none actually uses 0.


Yay!


```
pkg/deploy/util/util.go
463:	if _, err := watch.Until(timeout, watcher, func(e watch.Event) (bool, error) {

pkg/deploy/registry/rest.go
34:	if _, err := watch.Until(timeout, w, func(e watch.Event) (bool, error) {

pkg/cmd/cli/cmd/debug.go
354:		switch containerRunningEvent, err := watch.Until(o.Timeout, w, kclient.PodContainerRunning(o.Attach.ContainerName)); {

pkg/cmd/cli/sa/newtoken.go
203:	event, err := watch.Until(timeout, watcher, func(event watch.Event) (bool, error) {

test/extended/dns/dns.go
156:	if _, err = watch.Until(e2e.PodStartTimeout, w, PodSucceeded); err != nil {

test/util/namespace.go
48:	_, err = watch.Until(30*time.Second, w, func(event watch.Event) (bool, error) {
```


Hello,
Yes I know. But I removed all tinyMCE style to keep editor same as default.


I make a mistake in commit text, I  didn't remove right-click option but left-click  (but I corrected 'right-click' spelling )


should this be j instead of i?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#SaK3CINBQgSpkIcJbTFn1Q


src/opm/src/Классы/УстановкаПакета.os

Вот это неправильно. ИспользоватьПрокси(Ложь) говорит что нужно использовать вручную заданные настройки прокси.

Нужно следующее:

* Нет файла конфига - вообще не создавать прокси, не передавать ничего в создание http соединения
* Есть файл конфига - читать настройки из него. В нем может быть:
   * Системный прокси - тогда работает ИнтернетПрокси(Истина)
   * Ручные настройки - тогда работает ИнтернетПрокси(Ложь) и задаются ручные параметры через Прокси.Установить()


\src\server\game\luaengine\UnitMethods.h(2622): error C2660: 'Unit::HealBySpell': function does not take 4 arguments (compiling source file \src\server\game\LuaEngine\LuaFunctions.cpp)


How confidence interval changes when we increase size of the sample.


Nooooooooo! Not the `beer` icon!

;)


Fixes #7


Oh sorry i don't knowed about that but i know now :P


Any chance you could give more explanation as to why this matters? Why is it different from the Composer standard of `/vendor` ?


Fixes #843.


Se fizer o union escolhendo quem eh pai de quem aleatoriamente, da no mesmo que implementar o ranking
http://codeforces.com/blog/entry/47402


questo non l'ho messo io


This commit broke the previous commit you made to this page that fixed the security issue @samerton


Need to add the column fixed to fixed_question+
trigger:

CREATE OR REPLACE FUNCTION is_question_fixed_trigger_function()
  RETURNS trigger AS
$BODY$
BEGIN
	UPDATE typo_question
		SET fixed = 'True'
		WHERE question_id = new.question_id AND EXISTS ( SELECT corrected_text
			FROM fixed_question WHERE question_id = new.question_id
		GROUP BY corrected_text
		  HAVING COUNT(*) > 1);

	return new;
END;
$BODY$
  LANGUAGE plpgsql VOLATILE
  COST 100;
ALTER FUNCTION is_question_fixed_trigger_function()
  OWNER TO fstrub;

CREATE TRIGGER is_question_fixed_trigger
AFTER INSERT ON fixed_question
FOR EACH ROW
EXECUTE PROCEDURE is_question_fixed_trigger_function()


Is this working for you? If so would you be willing to create a PR? I never tested the systemd scripts so they kinda fell by the wayside


@jutkko Thank you! I am happy that my code got merged. It was nice meeting you at Europe CF summit!


@Tyriar I made root class of Panel - `WorkbenchComponent` to extend `Dispose` which has nice _register method and disposes registered disposables on dispose. This needed adoption in terminal panel.

Hope it is fine for you. Thanks


@PugaBear I did test on my localhost to ensure this didn't happen, but I'll have another look through now.


I think I've spotted another issue with this commit :(

Any subnets created since these changes don't have a Gateway IP... Doc's seem to suggest that the gateway should default to '.1' if not supplied, but I'm not sure if that's a TF default, Gophercloud default, or OS default...

But this used to work automagically previously...
```sh
# Create a subnet
resource "openstack_networking_subnet_v2" "tf_nc_test_1_subnet" {
  region = "uk-1"
  name = "tf_nc_test_1_subnet"
  cidr = "192.168.50.0/24"
  ip_version = "4"
  enable_dhcp = true
  network_id = "${openstack_networking_network_v2.tf_nc_test_1.id}"
  value_specs = {
    availability_zone = "uk-1a"
  }
}
```


Something like `(str "c" (System/currentTimeMillis))` also do the trick


`npm run build` ?


Is it necessary to use `const auto* model` instead of `const auto model`?


Seems to work ...

```
Python 2.7.6 (default, Jun 22 2015, 17:58:13)
Type "copyright", "credits" or "license" for more information.

IPython 1.2.1 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: from argparse import ArgumentParser as ArgParser, _UNRECOGNIZED_ARGS_ATTR
In [3]: print _UNRECOGNIZED_ARGS_ATTR
_unrecognized_args
````


This line does not work in my program: "private static final int NO_IMAGE_PROVIDED = -1;" When I click the 'phrases' button, the app crashes. It does work however when I change the variable to 0. Can anyone explain what is going on?


```
In [4]: import argparse
In [5]: argparse.__version__
Out[5]: '1.1'
```

What version are you on?


thanks


Okay :) Let me know if you want me to double-test anything @samerton


Also check that fake_data gets updated.


This also doesn't work. Attributes are lost when using this commit.


Faudra le faire dans le site aussi :)


@jimlawton I'm sure I've been told this, but I can't remember ... the travis check doesn't build yaDSKY2?  I seem to recall you telling me that it did a full build, so I was curious as to how this commit broke yaDSKY2 for me.  When I looked at the build log for this commit (thinking that perhaps the wrong version of wxWidgets was used), it didn't seem to have built yaDSKY2, though I admit I sometimes can't see what's right under my nose.  (Actually, I can't _ever_ see what's right under my nose, but you get the idea.)  I also seem to vaguely remember you saying that it can't build the wxWidgets stuff because the X libraries can't be installed, because it's a Docker container.  So I have all these conflicting memories of what I fancy I was told.  :-)


Change the name to `expected_groups`


No need for this.


Ура, ты не забил на проджект.
До первых билдов еще много работы?)


You could simply pass an empty dictionary like so `{}` no need to create fake_data for only that


@regismartiny bacana, isso é uma boa prática 👍


Ditto


nice comment fam
![dont quit](https://cloud.githubusercontent.com/assets/17749593/19117129/a4f38fbc-8b20-11e6-9759-5edf76de25e2.jpg)


Does DS9 understand utf-8? You should use an encoding that the other end understands.


After using both the commits worked for me with your device tree.


Are these comments needed?


really bad method. please , rewrite it.


We had to handle the caching issue as default species was sent at the server side.


Faltou falar sobre a colisão entre os dois...


might be cause of issue #4594 ?


good job ! well done !


Проверю, возможно остался мусор от попыток организовать параметры...
Форк будет новый с opmа


Ours is a quad core.


I think this is the wrong location it needs to be on StratisX


Hi!
according to the comment few lines above this size is incorrect (not divisible by 128). which one is wrong - the comment or the value?


some of your commits don't record any changes, but it looks like you are getting a little more comfortable with the CLI (command line).

make sure you are doing `git add <insert_filename_here>` after you save the changes to the file in the text editor!

let me know if you want to go over `git` stuff.


it looks like your files actually don't have any text. make sure you are doing `git add <insert_filename_here>` after you save the changes to the file in the text editor! let me know if you want to go over `git` stuff.


FTR we still need to find a way to have some automated tests for office import. Probably with using a docker VM for that. In any case, I agree, it would be a completely different setup and that we don't need this.


👍
after this step, try removing the file using `rm` in bash, then adding (you actually add the deleted file to the staging area with `git add SMGEAR123.txt` or `git add .`, oddly enough) and committing the change.  after that, you should be able to revert to the prior commit using `git revert` as directed in the HW.


🔨 nailed it 💅


Is was and is the 2.x convention and default folder for vendor classes. It just always has been called "vendors" in this directory and also exists like this in the default skeleton, thus the config here.


😿  but nice work! I like that you gave yourself extra practice with this assignment! 👍


💅 nail'd it 🔨

💻 = mostly 💀


after this step, try removing the file using `rm` in bash, then adding (you actually add the deleted file to the staging area with `git add herrerake.txt` or `git add .`, oddly enough) and committing the change. after that, you should be able to revert to the prior commit using `git revert` as directed in the HW.


I think you have to revert after this step to return to a commit point where this file exists-- in this commit it has been removed/ but 👍  overall


The following doc(s) have been deployed to the [icylacepat](https://icylacepat-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/drush](https://icylacepat-static-docs.pantheonsite.io/docs/drush)


I've updated the code, now it works. Not sure about how it should poll kernel driver. Now it use modified driver whic provides sysfs irq polling but it seems goes deep sleep sometimes and not waken up on finger touch. Or should be HAL code  modified to poll input device which stock one does.


@regalstreak No, ours is octa core


No, the compiling isn't the issue.

The setting up in game part, but I think I found the tutorial in 1 of the posts on the MC forum


?

On Wed 5 Oct, 2016, 8:01 PM Simao Gomes Viana, <notifications@github.com>
wrote:

> @regalstreak <https://github.com/regalstreak> No, ours is octa core
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/halogenOS/android_device_oneplus_oneplus2/commit/4dec68ab38d36986aa853fab2f91d416373f9254#commitcomment-19303251>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHZlaDYVU4DhEovn9DMjmaqEU34DebC6ks5qw7TIgaJpZM4KO4Hk>
> .
>


It would be better to `@mock.patch("os.path")` and `@mock.patch("os.makedirs")` in `_test_manage_user_ssh_keys`


@princeakarit if you have kallsyms, you can grep from it which is just a global var.


also, don't forget to make the pull request via the GitHub website!


Hahaha. Whoops.


AND ITS STILL FUCKING BROKEN.


Ah. I guess it does..  I probably missed this because the snowball tracer does indeed record logs, but I see that is due to setting the `SamplingPriority` tag. Maybe we want to stop setting that tag and just set ShouldSample to always-true or always-false depending on whether we have a recorder?


You should make sure that nothing returns `null` so that you would never have to do any null checks.

In this case, if there are no celebrations, you can return an empty list instead


@bernardobrezende

Para usar não ficar chamando o exercito.size() dentro da classe (percebi que ele se torna desnecessário para chamar os demais métodos da classe), mas utilizo ele na classe  ExercitoTest


ds$azpld$azlsd8az4s89azsaz
98


Looks like it's not checking for a prerequisite before displaying this message?


saw approval from andytill
at https://github.com/basho/bitcask/commit/f87eb942a8d5f9e9e77d124de915dcf51328dc7f


merging basho/bitcask/merge-develop-2-2-to-develop = f87eb942 into borshop-integration-245-merge-develop-2-2-to-develop


basho/bitcask/merge-develop-2-2-to-develop = f87eb942 merged ok, testing candidate = a0088974


Looks good so far!


Don't be like you didn't know what I am talking about :joy:


This configuration file should be created by `sos start server`


You mean like launching the game itself with mod?

If you mean this, I forgot to add few points, so after importing the project into eclipse, you click on the arrow near the green play button, `Run Configurations...`, then `Java Application -> %name_of_project%_Client` and click `Run` button in right bottom corner. This will launch the game and you'll be able to test the mod in-game.


@PugaBear Should be fixed now (da14e8e6efdb46dd3e9cac6a0d5ef5f54b590a5e), thanks for spotting this.


Please notice the difference between these two statements

1. from argparse import _UNRECOGNI**S**ED_ARGS_ATTR
2. from argparse import _UNRECOGNI**Z**ED_ARGS_ATTR

Hint: Read the bolded part.


변수 명 gridView 가 나을 듯.


That sounds reasonable, and would remove some of the confusion.


hfhfghfghfhfghf


@aslakknutsen @tsmaeder  In 0e9f380 I've introduced a way to ignore a `serialization_failure` during migration. Since the loop for migration is not endless, all concurrent migration processes that fail due to a `serialization_failure` should now ignore this error and continue running without marking the whole roll out as broken.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1269](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14977&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


변수명 boardStatus가 나을 듯.


Yes, the test scheduler shifts the time of items scheduled in the past and present. This is something that I copied from Rx.Net. It is used to make the scheduling visible in the tests so that an item that has been scheduled (say by observe_on) to occur on a coordination now() will have a time increment that can be measured in the test result.

It always felt wrong to me.. I think it would be better to have another way to observe the items being scheduled during testing.


Não achei a descrição pedida (nos comentários ou em algum arquivo separado).


wy aren't you pushingggg???


Thomas what level of effort would it take to have Bagpipe modified to listen. We are implementing L3 IPVPN and would like to use Bagpipe but need additional features. Or can you recommend another opensource package for Openstack outside the datacenter?


ping @jdalton are you aware of this? A couple months ago this changed: unnamed setup functions started erroring (below)

```
undefined:12
}};if(typeof su14756782663400=="function"){try{function (c) {
                                                        ^
SyntaxError: Unexpected token (
    at createCompiled (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:1712:16)
    at clock (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:1595:58)
    at clock (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:1805:20)
    at new Deferred (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:405:7)
    at Deferred (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:402:16)
    at Benchmark.run (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:2099:13)
    at execute (/home/gyeates/code/async/node_modules/benchmark/benchmark.js:860:74)
    at Timeout._onTimeout (/home/gyeates/code/async/node_modules/lodash/lodash.js:2733:43)
    at tryOnTimeout (timers.js:232:11)
    at Timer.listOnTimeout (timers.js:202:5)

```


Favor condensar o trabalho 5 em um único comentário:
- Descrever o ciclo de vida da coleção e de um objeto
  - escopo, tempo de vida, alocação/desalocação


Having it unused and open source could mean a higher cost, if we get PRs and stuff. But I guess we could wait and see – I agree the time investment seems low so far, if you and @joakimk feel this code is worth keeping for some possible future :)


The arguments to this methods are not just errors. Subtest results are also tested for picklability. That's why I used "picklable" argument name and "object" in comment.


I think this is more confusing now. We're taking error (which itself is an instance of exception) and turning it into another exception? Also "if" statement does not communicate abnormal operation as well try/except. Do we just do something and continue execution `if exc`?

You've made this mistake yourself. You've removed the try/except block, but you left the clause that re-reaise that no-longer-caught exception on subsequent lines inside the `if` block.


This will fail now.


Yes, it does, I checked before asking.  Actually I think utf-16 may be OK too -- I was able to display Japanese characters after the utf-8 change (which surprised me).


This will also fail now.


@7ofNine I don't remember explicitly why these things were done that way. Nor do I have any strong opinions on separating out the commonalities into a separate library, other than that I'm too lazy to fix things that are working myself.  :-).  I think the reason I didn't do it originally is that since the stuff was only needed in a couple of different places, it didn't seem important.  Eventually, though, the number of programs needing it got much larger (yaTelemetry, yaAGS, yaDEDA, etc.), so I may not have been thinking far enough ahead.


![](https://github.trello.services/images/mini-trello-icon.png) [Se considera o lista de elevi cu notele la trei materii. Sa se calculeze media fiecarui elev la fiecare materie si mediile la fiecare materie pentru toata lista.](https://trello.com/c/UvjrvSRq/5-se-considera-o-lista-de-elevi-cu-notele-la-trei-materii-sa-se-calculeze-media-fiecarui-elev-la-fiecare-materie-si-mediile-la-fie)


What is the reasoning behind this change, and even the larger matter of having the Content-Type specified by the primary server?

The use case I see for this draft is that the primary server never has the content at all, and is just an redirecting you to a closer server that has the content you are seeking. Making it have to know all the content types and having the secondary having to put in this special content type seems odd to me. The secondary server would just be a generic cache and would have to distinguish between OOB redirects and regular content requests.

Thoughts?


Is there any value in adding `xunit.runner.visualstudio`? I'm not sure what's usually done.


NICE!!! Thank you ^^ Let's get kickin! ;)


`ipairs` é uma função e não uma palavra reservada.


There's is "pickling" and "unpickling". This error fails during "unpickling" so it definitely is not "unpicklable_error". We could call it "failing_error" or "error_that_fails_unpickling".


Isso não é uma propriedade.


@fiorix @dpnova: the error of reading the whole (maybe 4GB) file is here.


In case of simple replacing a `<=` to `<`
```diff
-E## INDEX ##_DIR_WRITE(e_steps[INDEX] <= 0 ? INVERT_E## INDEX ##_DIR : !INVERT_E## INDEX ##_DIR)
+E## INDEX ##_DIR_WRITE(e_steps[INDEX] < 0 ? INVERT_E## INDEX ##_DIR : !INVERT_E## INDEX ##_DIR)
```
, It solves the up and down when extruding, but It occurs the up and down when retracting(`e_steps[INDEX] < 0`... down, `e_steps[INDEX] == 0`... up).
Therefore,
```diff
    #define SET_E_STEP_DIR(INDEX) \
-      E## INDEX ##_DIR_WRITE(e_steps[INDEX] <= 0 ? INVERT_E## INDEX ##_DIR : !INVERT_E## INDEX ##_DIR)
+      if (e_steps[INDEX]) E## INDEX ##_DIR_WRITE(e_steps[INDEX] < 0 ? INVERT_E## INDEX ##_DIR : !INVERT_E## INDEX ##_DIR)

    #define START_E_PULSE(INDEX) \
      if (e_steps[INDEX]) E## INDEX ##_STEP_WRITE(!INVERT_E_STEP_PIN)

    #define STOP_E_PULSE(INDEX) \
      if (e_steps[INDEX]) { \
-        e_steps[INDEX] <= 0 ? ++e_steps[INDEX] : --e_steps[INDEX]; \
+        e_steps[INDEX] < 0 ? ++e_steps[INDEX] : --e_steps[INDEX]; \
        E## INDEX ##_STEP_WRITE(INVERT_E_STEP_PIN); \
      }
```
is apposite and consistent implementation, I think.
This implementation of `SET_E_STEP_DIR()` do nothing when `e_steps[INDEX] == 0`, as same as `START_E_PULSE()` and `STOP_E_PULSE()`.
It will reduce the ISR performance a bit, maybe.

I'd like to work about this and submit a PR, but I re-installed a OS(Windows 10 Insider Preview) yesterday, and now I just realized that my git environment(command-line) is broken...
I can't use the rebase, push -f etc at this time.
I'm trying to repair...


@ccsalazarr-commercetools does it make sense to retry on any error (BadRequest) or maybe it should explicitly check for 409 errors?


@fiorix @dpnova: here the big chunk was passed as a whole to the transport (and eventually to the Gzip module)


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1270](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14980&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


changename


Favor condensar o trabalho 5 em um único comentário:
- Descrever o ciclo de vida da coleção e de um objeto
  - escopo, tempo de vida, alocação/desalocação


@fiorix @dpnova: HEAD request were allocating the memory for the whole file just for answering the size of the file.


@phatblat Is this still necessary in Xcode 8?


Do you people always investigate every commit with such scrutiny? I think there's a name for this kind of feedback.


nicee!!!


restored old wallet from seed with tons of trx, looks good so far


[`ishold(hAx)`](https://www.mathworks.com/help/matlab/ref/ishold.html) might come in handy.
(The function is open-source)


hghg


The choice was initially done because (a) we didn't need to listen in our particular case, and (b) ExaBGPw which we were inheriting from, was not supporting it.

Now, this decision can be revisited.

ExaBGP has had support being added, which would help.
There would remain significant work to do however to inherit from that.

One question : why/where would you need bagpipe-bgp to listen ?
If bagpipe is peering with an implementation that can connect, e.g. any Route Reflector, then listening is not needed. We practiced peering bagpipe-bgp with GoBGP as an RR, or with JunoS as an ASBR without needing to listen.

If there is a good rationale to add 'listen', I would be glad to coach someone willing to work on a solution.


or based on this pattern:

```
namespace detail {

template<class T, class =rxu::types_checked>
struct is_observable : std::false_type
{
};

template<class T>
struct is_observable<T, rxu::types_checked_t<typename T::observable_tag>>
    : std::is_convertible<typename T::observable_tag*, tag_observable*>
{
};

}

template<class T, class Decayed = rxu::decay_t<T>>
struct is_observable : detail::is_observable<Decayed>
{
};

```


Looks good! Is it possible to make a release for this?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1269](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=14977&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 260, ignored: 3 Build time: 00:11:30


I hadn't really thought about the limits yet to be honest. I've been focused on stuff for v2.4 right now, and changing the data structures isn't one of the first priorities once I get started on v3.0 stuff, so not sure. I feel like the stat limit for combined EVs would be problematic to players who cared about them without the EV berries and stuff, but those should be doable to both of us.


Wasn't aware. Thanks for the heads up!


--timeout flag affected for `oc serviceaccounts new-token` but the cmd hasn't defined any special meaning for 0.
`oc debug` uses a 15m hardcoded timeout. Rest of the uses are deployment logs and extended tests.


But like you say, more complicated for us to add and for them to make the most of them.


@vangheem Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.contentmenu/2/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/__init__.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/displayviewsmenu.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/displayviewsmenu.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/interfaces.py:5:28: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/interfaces.py:37:1: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/menu.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/menu.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/menu.py:112:55: S001 found %s formatter
plone/app/contentmenu/menu.py:251:1: C901 'DisplayMenu.getMenuItems' is too complex (24)
plone/app/contentmenu/menu.py:324:30: S001 found %s formatter
plone/app/contentmenu/menu.py:359:32: S001 found %s formatter
plone/app/contentmenu/menu.py:407:30: S001 found %s formatter
plone/app/contentmenu/menu.py:433:34: S001 found %s formatter
plone/app/contentmenu/menu.py:447:28: P002 found "hasattr", consider replacing it
plone/app/contentmenu/menu.py:477:40: S001 found %s formatter
plone/app/contentmenu/menu.py:521:14: S001 found %s formatter
plone/app/contentmenu/menu.py:600:20: S001 found %s formatter
plone/app/contentmenu/menu.py:618:24: S001 found %s formatter
plone/app/contentmenu/menu.py:671:1: E302 expected 2 blank lines, found 1
plone/app/contentmenu/menu.py:692:55: S001 found %s formatter
plone/app/contentmenu/menu.py:766:1: C901 'WorkflowMenu.getMenuItems' is too complex (13)
plone/app/contentmenu/menu.py:784:29: Q000 Remove bad quotes.
plone/app/contentmenu/menu.py:785:30: S001 found %s formatter
plone/app/contentmenu/menu.py:812:52: S001 found %s formatter
plone/app/contentmenu/menu.py:941:22: S001 found %s formatter
plone/app/contentmenu/menu.py:949:44: S001 found %s formatter
plone/app/contentmenu/testing.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/testing.py:53:48: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:69:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:72:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:78:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:81:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:88:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:91:10: Q000 Remove bad quotes.
plone/app/contentmenu/view.py:0:1: C101 Coding magic comment not found
plone/app/contentmenu/view.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/view.py:33:80: E501 line too long (87 > 79 characters)
plone/app/contentmenu/tests/test_menu.py:0:1: I001 isort found changes, run it on the file
plone/app/contentmenu/tests/test_menu.py:23:28: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:41:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:44:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:48:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:77:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:80:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:87:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:89:13: W503 line break before binary operator
plone/app/contentmenu/tests/test_menu.py:115:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:117:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:155:13: W503 line break before binary operator
plone/app/contentmenu/tests/test_menu.py:164:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:166:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:182:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:184:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:198:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:205:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:207:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:224:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:225:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:235:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:236:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:246:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:247:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:253:35: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:306:28: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:324:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:328:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:339:80: E501 line too long (87 > 79 characters)
plone/app/contentmenu/tests/test_menu.py:350:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:353:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:355:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:453:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:461:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:462:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:463:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:466:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:484:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:491:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:501:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:528:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:531:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:550:42: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:552:29: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:554:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:592:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:627:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:630:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:650:23: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:651:23: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:654:13: W503 line break before binary operator
plone/app/contentmenu/tests/test_menu.py:689:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:699:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:773:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:813:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:817:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:846:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:856:20: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:873:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:874:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:882:14: D001 found failIf( replace it with assertFalse
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@mauritsvanrees Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.contentmenu/5/violations
```
plone/app/contentmenu/menu.py:105:51: S001 found %s formatter
plone/app/contentmenu/menu.py:235:1: C901 'DisplayMenu.getMenuItems' is too complex (22)
plone/app/contentmenu/menu.py:308:30: S001 found %s formatter
plone/app/contentmenu/menu.py:343:32: S001 found %s formatter
plone/app/contentmenu/menu.py:387:30: S001 found %s formatter
plone/app/contentmenu/menu.py:413:34: S001 found %s formatter
plone/app/contentmenu/menu.py:427:28: P002 found "hasattr", consider replacing it
plone/app/contentmenu/menu.py:457:40: S001 found %s formatter
plone/app/contentmenu/menu.py:494:14: S001 found %s formatter
plone/app/contentmenu/menu.py:569:20: S001 found %s formatter
plone/app/contentmenu/menu.py:587:24: S001 found %s formatter
plone/app/contentmenu/menu.py:662:33: S001 found %s formatter
plone/app/contentmenu/menu.py:724:1: C901 'WorkflowMenu.getMenuItems' is too complex (13)
plone/app/contentmenu/menu.py:742:29: Q000 Remove bad quotes.
plone/app/contentmenu/menu.py:743:30: S001 found %s formatter
plone/app/contentmenu/menu.py:770:52: S001 found %s formatter
plone/app/contentmenu/menu.py:896:22: S001 found %s formatter
plone/app/contentmenu/menu.py:903:44: S001 found %s formatter
plone/app/contentmenu/testing.py:54:48: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:70:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:73:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:79:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:82:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:89:10: Q000 Remove bad quotes.
plone/app/contentmenu/testing.py:92:10: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:23:28: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:42:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:45:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:49:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:78:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:81:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:88:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:115:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:117:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:163:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:165:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:181:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:183:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:197:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:204:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:206:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:223:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:224:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:234:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:235:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:245:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:246:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:252:35: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:305:28: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:323:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:327:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:349:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:352:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:354:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:452:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:460:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:461:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:462:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:465:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:483:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:490:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:500:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:527:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:530:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:549:42: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:551:29: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:553:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:591:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:626:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:629:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:649:23: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:650:23: S001 found %s formatter
plone/app/contentmenu/tests/test_menu.py:687:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:697:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:771:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:811:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/contentmenu/tests/test_menu.py:815:14: D001 found failIf( replace it with assertFalse
plone/app/contentmenu/tests/test_menu.py:844:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:854:20: Q000 Remove bad quotes.
plone/app/contentmenu/tests/test_menu.py:871:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:872:14: D001 found failUnless( replace it with assertTrue
plone/app/contentmenu/tests/test_menu.py:880:14: D001 found failIf( replace it with assertFalse
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.