While this is one particular user goal: to center text. Be sure to consider to more user goals related to centering practices in web design. For example, centering div (or other similar) elements, or centering an image within a figure element, etc.


curious why not (rightTan - leftTan)


def calcShortestToAngleSet ( self, angle ):
    current = self.getCurrentAngle ();


you should be able to to it with a single `sed` command, with `;` being the substitution definition:

```
sed -i -- "s@..@..@g ; s@..@..@g"
```

It avoids to process the same file multiple times.


This is still here :(


Replace with an enum, or make it private and add separate next/previous methods: https://our.intern.facebook.com/intern/dex/facebook-code-smells/boolean-arguments/


do a null check instead of an array check. You want the typechecker to know it'sa  non-nullable shape, not an array


I beleive it was:

```perl
print "Status: 200\n";
print "Content-Type: text/plain\n";
print "\n";
print "foo bar";
```

---

##### regex test cases
[current version](https://regex101.com/r/xU1mL5/2)
[pr version](https://regex101.com/r/xU1mL5/3)

I added in a safer way of stripping out the headers by splitting the content by `\n\n` which would be a header followed by the new line before body content.
Then testing the first part against the regex to make sure its in header form


Yep, seems it could. This page is serving triple duty for ladder levels, clans, and soon courses, so it could/should probably be split into separate views at this point.


explicitily call shapes::toArray when you want it


И откат тоже сделать? :)


@adolfopa

creo que esto sería un buen candidato para el `ServiceTrackerList` no?


variables injected by Artemis.World actually


I agree with Sarah. Ground your conceptual and reference topic writing with the user goals and the particular elements necessary to complete it. You can assume some things like having a text editor and ability to test. You could provide some of this stuff within a prereq, if necessary. But, again, consider the scope of your topic model when making these decisions.


recover => logger


style, organizer = address = etc


oneline (enleve les {)


Neither do I, so we'll both be learning. :)


can we put the template file in `test-runner`? Having 2 copies of the same file just means they will become out of sync.
And the one in `test-runner` has to be edited anyway, so having the placeholders in it doesn't really hurt.


Okay, that makes more sense to me, will incorporate some more blank lines into my next check in.


Hi @sudeepm1:

Once the deployment is finished; you should be able to find the public IP of the VM:
![step1](https://cloud.githubusercontent.com/assets/4741357/11277715/af334660-8eb5-11e5-9f96-15abccba79eb.png)

You can then SSH to the IP using the user name you provided in the parameters:
![step2](https://cloud.githubusercontent.com/assets/4741357/11277740/c46f31d8-8eb5-11e5-98be-99650c056068.png)


No, just add a comment that there is no rollback.


hallo


There is no reason to call `python_fix_shebang` if it can't do anything.


Oops! Fixed, thanks.


sdfdf


May I also suggest cleaning up the formatting of the document?

https://github.com/StrikeForceZero/spawn-perl/tree/cleanup


Was this uncomming intentional?

This line is indeed commented out in the upstream scala (and it does not affect our errors/warnings).


The only other way I can think of is having `python-utils-r2`, which is probably even worse.


blank line missing


#potential solution, not exactly sure what you're trying to accomplish
def clacShortestToAngleSet ( self, angle ):
         current = self.getCurrentAngle ();
         diff = angle - current;
         if diff > pi:
                  diff -= pi * 2;
         if diff < - pi:
                  diff += pi * 2;
         return diff;


I guess we cannot arrive here otherwise it would need
```js
if (window.screenfull && window.screenfull.enabled)
```


No problem, if you don't like signals because you find them hard to debug it's fine by me :) Personnaly I consider it as part of the Django framework and have been doing so for years and it's been proven to work in django-cities-light for years inside hundreds of django projects but if it's just a matter of personnal taste, I'm all for finding a solution that we both like ;)

So, if I remove the signal, how do you think we should execute this code that handles memopol's models which are subclasses of django-representatives models ?

https://github.com/political-memory/political_memory/blob/pr/legislature/models.py#L69-L109

Note, we have the same case with django-representatives-votes and memopol.votes:

https://github.com/political-memory/political_memory/blob/pr/votes/models.py#L80-L91


Author:Tianpei Luo


`return matches unless matches.empty?`


same here on the screenfull compomenents methods access withou checkign for window.screenfull !== undefined


WARNING: Failed to fetch URL git://github.com/openbmc/linux;protocol=git;branch=dev, attempting MIRRORS if available
ERROR: Fetcher failure: Unable to find revision 1da90046061225460594f75c6dbf9ba7f6fbff5c in branch dev even from upstream
ERROR: Function failed: Fetcher failure for URL: 'git://github.com/openbmc/linux;protocol=git;branch=dev'. Unable to fetch URL from any source.
ERROR: Logfile of failure stored in: /home/causten/openbmc-barreleye/build/tmp/work/palmetto-openbmc-linux-gnueabi/linux-obmc/4.3+gitAUTOINC+1da9004606-r0/temp/log.do_fetch.18349
ERROR: Task 47 (/home/causten/openbmc-barreleye/meta-phosphor/common/recipes-kernel/linux/linux-obmc_4.2.bb, do_fetch) failed with exit code '1'

It doesn't seem to be compiling


idem pour le nom et l'argument


omg je me suis fait avoir ;(
J'ai copier
```bash
ipppd ippp0 noauth name LOGIN debug +pwlog
```
;(((


P.S. As far I understand we will need to use plain SQL here. And we need to support at least 2 databases: H2 and MySQL, so, please, use the options that is available on both databases. (Hint: use could use http://sqlfiddle.com/ to check MySQL queries.)


@joshski I would remove the proxy stuff altogether as I don't think it will ever be needed. Although we may want to leave this in as a feature seeing that this an OSS project.


indentation


Mais j'ai pas copier la commande sur le fichier rendu à latu donc ça va le faire je pense :)


map organizers = organizers is useless


d'ailleurs quelqu'un lis mes commentaires ou je peux aller me faire mettre?


I referred to the wrong tracker issue. See #63


idem useless map


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


style (spaces)


Just a suggestion, it might be cleaner to use for example
```
#if defined(_WIN32) || defined(__CYGWIN__)
  #define BUILDING_FOR_WINDOWS 1
 #else
  #define BUILDING_FOR_WINDOWS 0
#endif
```

and then use ```#if BUILDING_FOR_WINDOWS``` all over the place instead of repeating yourself. Though to be sure I'd check with other libraries which support unix/cygwin/mingw/..


recover


I'm not entirely clear how `multi_match` is going to be used (I think had thought of something else WRT the API, but more on that elsewhere), but in general I'm not a fan of having something return an List/Map variant in most cases but `nil` in another -- it can make more work on the part of the caller and usually isn't needed unless there is something particular you are meaning to convey when `nil` is returned, and in that case, document it.


The idea here is that this code is never used (but i need some binding to make Guice happy). If it throws an NPE then we know something is wrong in our test setup? Maybe a better alternative would be to throw NotImplemented in each method?


@ericseandevlin my commit to your thing


findAccessToken


getUserLi...


get


style: là aussi tu peux enlever les {}


Removed from the original repository in https://github.com/PTAnywhere/performance-testing/commit/02f24d00ef757ef2594f737e3d36ed7814addb97


Ok will change.


`colorAssignments` is never freed.


@dwerner90 yes - planned for tmr.


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


`index` is never freed.


enleve les {}


Ok thanks. I have to check the texture (misc/system_selection/system_selection2.png), but I guess it's a LUMINANCE or LUMINANCE_ALPHA as internal format, and more probably the later.


@maruilian11 thank you :D


TeamCity pull requests :: netty [Build 683](https://garage.netty.io/teamcity/viewLog.html?buildId=3374&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:32


Add your favorite color


:eyes:


Remember that you're not necessarily selling anything/a product. Here, consider a user goal as something akin to your workshop: How to create a font-sizing scheme. Consider other goals linked to using fonts, such as importing and declaring fonts, or how to create a compelling <header>.


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


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


The version is already specified below (0.1)


Ok i 'll move it. I actually did it intentionally since this is the only place where we use that plugin. But either way is fine with me.


@pgiraud @fvanderbiest
if we install numpy before collecting rasterio, it works fine
why don't run
    pip install numpy
before
    pip install requirements
in makefile
instead of using --system-site-packages and installing custom packages with sudo and apt-get from specific ppa


hello/world should probably become `Hello::World`


I committed this to the wrong branch and reference the wrong issue. Should really close #1881. In either case it has made its way to develop. The op schemes with differential temperatures make me a little nervous, but to the best of my knowledge this works correctly. @DavidGoldwasser or @asparke2 should review and close #1881.


I'm not sure I understand. We still do need these methods, even without any code in their bodies. Are you proposing we should discover that—since the particular types extend `ComponentSpec`—they should be automatically registered as a component type, without such a method?


oneline = ()


get


recover


I 'll add it. Technically it's not a source file (just the template to create the source file).


surement recover aussi


Why change to a smaller datatype?


argument jsvalue


Of course, I'll be in Karlsruhe next weekend from for the xpdays. Maybe we can meet there if you have time.


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


Then VR get twice the canvas width/height pricecheck then (along with computeCanvasResize) => up to 2*5ms for that... where an ideal VR frame is 11ms...

Perhaps we could make osg::viewer::computeCanvasResize a callback thrower of the new size, with the usual return true prevent the default code to be executed  case so that user can gets either Event Resize or Replace the osgjs resize ?

(Ideally all camera or/and framebuffer should gets resize events)


Hey @AmirDI,

I have made some changes in the structure to demonstrate how we can add different packages, constants and custom styles in our project.

Hope that helps,
Raxit


window.Object.keys ?


Again, this list is quite extensive and suggests that the goal is too broad. Also, if your audience is frontend designers and developers, do you think that you need to write topics on some of these tasks? Note this, though, you should probably tailor your audience to more novice audience, since your still learning these things too.


choose better name


"Nível" aqui.


"Target" seria melhor como "Alvo", não?


Append .md extension to readme, if you wanna, it will make github render it as markdown.


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


true and false useless


:+1: :smiley:


name value of tuple arguments


name tuple arg


ah yes! i only checked one file, but i was able to read an ntuple produced with GC before this kind of update.
i ll make a PR for UHH2:RunII_25ns_v2 (more details there)


@VickiG DWMR. Noice! Kepp pressing ahead. The reward for solving a challenge is - another challenge. :trophy:


go to the line


So this commit was necessary because the base64 encoding commit was based on an older version of code; I'd made a commit who's code needed to be merged in before you could push.

Merge commits aren't the best since they can make the revision history confusing.  So in this case, if you look at `reporting.py` in the base64 commit (https://github.com/gamernetwork/gastatsdash/commit/e2dd4b18ea880cd286c6666648d67341ecfdd4a6) it still has the `analytics.get` bug in it - even though the commit before it in the history removed that line!

It's best to do something called rebasing.  When you push and find that the remote repository is ahead of your local version, do a rebase: `git pull --rebase`.  This will pull down the latest remote commits and replay your local commits after them.  That way we have a sequential history!  Here's more info: http://stackoverflow.com/questions/16666089/whats-the-difference-between-git-merge-and-git-rebase/16666418#16666418


This improved other games. San Francisco Rush:
before:
![gliden64_s f _rush_000](https://cloud.githubusercontent.com/assets/7283660/11278187/14388460-8eea-11e5-8174-191bcfca5fca.jpg)
after:
![1](https://cloud.githubusercontent.com/assets/7283660/11278189/1980c3c4-8eea-11e5-8482-f67d8e6085e3.jpg)


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 106](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=500&buildTypeId=XbimEssentials_XbimEssentials) is now running


Short array notation?


go to the line for the two


Missing documentation comments.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 106](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=500&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:17


Remove stray "is" and space --> "Galaxy should be used to test these tools directly."


lgtm. :shipit:


json arg


Melhor deixar o original: {@st42}Pension target {@stx1}(%s required){/}


thanks.
:smile:


Blank link in between? Or put the explanatory paragraph in between?


It's very important for us to get usage stats, that's what pays my salary... and I know Azure Web Sites is also tracking it specifically. I would also favor to put a version with it, it's not really an issue for security reasons, as hackers targeting it would still try even if the version was not defined.

Can be added in another middleware.


Fixes #2 I think :smiley:


Si, parece que si. Todavía estoy pasándole trozos a Brian; cuando esté todo subido hago yo el cambio y te lo envío.


hey you didn't check that after pushing the coinLogi.js in library folder that test are passing or not.
check the test file.


It misses tests for findEventsIds, facebookPageToPageTuple, readFacebookPage et au top faudrait mm tester un peu les autres mais ça risque d'être galère et vraiment trop long


Initially I added bufSize argument to allow limiting command length (but I didn't implement it). Currently it's possible to write something larger than the buffer (and possibly corrupt the some memory)


@borshop: retry


Why git shows that all lines have edited in the files, but not only AppPackageName property, did you touch these lines?


this is too much!


saw approval from javajolt
at https://github.com/basho/riak-erlang-client/commit/fc6bab75ef10c38a3396b2809ace65955373b334


merging basho/riak-erlang-client/fixes/am/fix_encode_timestamp_cells = fc6bab75 into borshop-integration-250-fixes/am/fix_encode_timestamp_cells


basho/riak-erlang-client/fixes/am/fix_encode_timestamp_cells = fc6bab75 merged ok, testing candidate = c14f65d4


you're right


fist step


done


Done


no hace falta, si no lo han subido todavía, no sé si podrán esperar o no


Good to go!


Thanks.

Am 19.11.15 um 17:57 schrieb Martin K. Scherer:
>
> done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/markovmodel/variational/commit/49215bb21d6187f0a4bc87f563282ba20a539106#commitcomment-14505887>.
>


давай кудись винесемо
Зробимо файлік "коммон джсон респонсес" і там будемо схоже зберігати


https://github.com/Fisiu/calendar-oswiecim/blob/master/scrapper/requirements.txt
A ten plik to co? :-) scrapper to de facto inny projekt który robiliśmy równolegle na koduj dla polski, dlatego osobne pliki


My testing shows that things work just fine without the `@ComponentSpec` methods at all. Let me confirm.


Instead of comments I'll add annotations next time
/*
*
*/


@alvare52 DWMR. Nice work here. Press ahead - ship code. :articulated_lorry: :computer:


думаю треба кудись в інше місце винести, потім фіг знайдеш щоб перекласти поправити


@olavphibm I clearly remember the debug message logged by the unattended server thread saying it had served ks.cfg, and the pre anaconda initrd program downloading it. Of course, now I can't find what is the code path that would actually lead into it, but I swear I'm not crazy :)

Since I'm on a tight schedule lately, I'd ask if @clebergnu has a few cycles to spare and take a look at this (he also implemented the http unattended server back in the day).

So @clebergnu, help us to answer the philosophical question: How `http_ks` ever actually worked? Who are we? Where are we going? (I'm kidding, you just need to tell us about `http_ks`, really.


HelloWorld


I presume this is TECS_A_PIT_MIN? Your docs and source are not in sync there.


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


I think #5 can be broken down into more manageable tasks with a specific framework like Reveal.js. Consider taking some notes about particular tasks to write as you yourself develop your reveal.js project.


1. Going to change method name to "addUser"
2. Missing annotations for what the method suppose to do


SUGESTÃO:
${@st59b}Quests{nl}{@st59s} - Quando você estiver numa nova quest, ela será adicionada ao registro de quests. {nl} - As quests principais aparecerão em amarelo, enquanto uma subquest aparecerá em azul. {nl} - A seta nos pés do personagem direcionará para o local da quest. {nl} - Se você completar uma quest, um ícone "Retonar" irá aparecer no registro de quests. {nl}    (Teleporta você para o NPC da quest através da tecla Backspace ou de um clique do mouse) {nl} - Adicione ou remova quests no registro de quest marcando ou desmarcando a caixa na frente do nome da quest. {/}


Ubuntu 15.10, failed to compile

dhcp.c:2037:9: error: format '%d' expects argument of type 'int', but argument 4 has type 'time_t {aka long int}' [-Werror=format=]  "%d,"MAC_FMT",%s,%s,%d,%s\n",


But why would we ever want a smaller datatype? This sounds like a latent bug waiting to happen.


lol, updating minitest and rspec... hahah


I presume this is TECS_A_PIT_MAX? Your docs and source are not in sync there. Also, this is already implemented in upstream as TECS_LAND_PMAX so it should be removed.


1. Going to change method name to "findUser"
2. Missing annotations for what the method suppose to do


+1 for a separate OrchardGeneratorHeaderMiddleware, that's easy enough to remove if you really want to.


Sorry guys, that one slipped through...


I don't see why this error shows up now.
It's must have been active for ages. (No, not related to the latest commits)


Instead of comments I'll add annotations in future.


To expound on my previous comment: what is the intent behind describing the english name as "Base Word(s)" ? In general I understand that there could be different tenses, conjugation, etc., but I feel that this doesn't apply for names, and is (potentially confusing) noise here.


這有點可惜 XD


* OK, .to_a usunę. Czy coś jeszcze jest do zmiany, żebym zrobił jeden pull request, czy do development od razu?

> W części przypadków używane są stringi jako klucze, a w części symbole. Nie wiem czy to wykonalne, ale może przy okazji tego taska dałbyś radę przejrzeć czy nie da się tego ujednolicić?

Musiałbym zrobić refactoring wszystkich start_simulation_managers i ich wywołań - bezpośrednim rozwiązaniem tego problemu jest użycie tego parametru jako ``ActiveSupport::HashWithIndifferentAccess`` tak jak napisałem w dokumentacji tej metody. To nie jest jakiś skomplikowany hack (to nie jest nawet hack - po prostu użycie odpowiedniej klasy), a grzebanie w refactoringu w tym tygodniu mi się nie uśmiecha :(


This is my new favorite comment block in our code. Does it work on buildbot?


I meant in an existing, close enough for this matter, middleware.


in client code it's ok, but we should probably find a way to get rid of this. Maybe changing name like Object to ObjectOSG.


ahhaha c'est la fête ici, on peut venir? :+1:


Ja xD


1. Change method name to "updateUser"
2. Missing annotations for update()
 /*
  * updating user
  */


you are right I will fix this


ez mehetne feljebb haver


TeamCity pull requests :: netty [Build 684](https://garage.netty.io/teamcity/viewLog.html?buildId=3375&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5309, ignored: 22 Build time: 00:14:54


Niet liegen


awesome changes btw


:( It'd be nice to not use these functions...


This whole file needs to be removed before a PR is created.


@keithbostic, just saw this. My last push doesn't have this, I'll fix this next.


No original é: {memo X}{@st59}Summon Companion.{/}
Pode ficar como "Invocar Mascote.".


CBECC (kbenne) - i386-Windows-7-VisualStudio-12: OK (2323 of 2349 tests passed)

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


good point @rylnd i agree

and this could relate to our discussion about using "Spanish Equivalent" instead of "Spanish Word Equivalent"


This isn't quite fixing it for me. I'm using your fork at this commit and still getting the error on registering an account...
```
File "/vagrant/venv/src/flask-security/flask_security/utils.py", line 333, in send_mail
mail.send(msg)
File "/vagrant/venv/lib/python2.7/site-packages/flask_mail.py", line 415, in send
with self.connect() as connection:
File "/vagrant/venv/lib/python2.7/site-packages/flask_mail.py", line 120, in __enter__
if self.mail.suppress:
File "/vagrant/venv/lib/python2.7/site-packages/flask_mail.py", line 490, in __getattr__
return getattr(self.state, name, None)
File "/vagrant/venv/lib/python2.7/site-packages/flask_mail.py", line 490, in __getattr__
return getattr(self.state, name, None)
File "/vagrant/venv/lib/python2.7/site-packages/flask_mail.py", line 490, in __getattr__
return getattr(self.state, name, None)
```


Is echt zo


"This is my first time. Please be gentle"


Esse caso pode ser "Deslogar" ou "Deslogar do Servidor".


Change method name to "deleteUser"
Missing annotations for delete(), stating what the method suppose to do.


Mesma coisa do 543 aqui.


This overall user goal may be too large, considering all of the tasks below that may result in writing more topics than you bargained for. Creating a Jekyll-based layout/template can probably be broken down into a more manageable set of user goals and tasks.


Ubuntu 15.10, Failed to compile

dhcp.c:2037:9: error: format '%d' expects argument of type 'int', but argument 4 has type 'time_t {aka long int}' [-Werror=format=] "%d,"MAC_FMT",%s,%s,%d,%s\n"


fixed


nice catch!


Reference should be to https://github.com/NRGI/resourceprojects.org-frontend/issues/265 (by adding the link here, it the linkage will now be made).


Yeah, I know - what's the alternative?


Dat kan niet Rob neeeeeee


Yup, but the travis build will most likely fail (because the file ends with `.java`).


I'm looking at the search drawing and syncing in detail at the moment, but this bit is probably OK.  The fonts need to be initialised either every time the window is presented, or continuously even when the window is hidden.  search_start() does this each time the window is presented (not for every search, that's search_process()).  All messages, including CONFIGCHANGED, are ignored if the window is not visible.


I know this is a dirty way to display the contents of User table,
but I wasn't sure how you'd like me to display the users list, hence the reason I'm var_dumping


Melhor como "Eu não acho que posso ajudar desta vez...".


Yeah we should probably do something smarter. Not sure I will do it on this PR but agree we should take care of this.
One idea behind the CustomVR stuff would be to use instead a custom RenderStage to split the rendering. So it would make sense when doing this.


1. Change method name to "displayAllUser"
2. Missing annotations for all(), stating what the method suppose to do.


kok ik ga werken tot 9 uur doei!


cc @paraschos


Vale considerar substituir "Top" por "Melhores", nas referências futuras também.


@stephomi ^


"Level Up" pode virar "Aumento de Nível".


Oh I see. Yeah, I'd rather have an explicit exception so it's more obvious what's going on.


:heart: :heart_eyes: :heart:


We don't need to mention that it is not finished yet.


I figured. But that parameter seemed like a good value for prompt deletion checking.

Sent from Windows Mail

From: Bartosz Skrzypczak
Sent: Thursday, November 19, 2015 12:25 PM
To: raphydaphy/Q-Operating-System
Cc: Plankp M. Teng

Initially I added bufSize argument to allow limiting command length (but I didn't implement it). Currently it's possible to write something larger than the buffer (and possibly corrupt the some memory)

—
Reply to this email directly or view it on GitHub.


Proponowalbym nie robic powazniejszych refaktoringow tydzien przed wdrozeniem finalnej wersji.


@linuxbo, I wanted to give some feedback on your initial stab at a task analysis for your own topic model. If you can get that done ASAP, I can provide feedback for you to consider.


OK, I guess we had the conversation about vals / keys and I forgot; keys is used in `select()` & `mapIds()`


And that's why PRs are useful :+1:


I don't think so. This is an runit that triggers the failure on our windows systems.


This appears to modify Windows branch - why is this necessary? Or am I mistaken?


It's baffling to me that you don't need `should_list.flatten`


This commit requires a memached flush prior to a regenerate_and_update.sh run


What about 2S, 3S, 5S, 6S configurations? This should be auto-detected by FS_BATT_VOLTAGE
http://plane.ardupilot.com/wiki/arduplane-parameters/#failsafe_battery_voltage_arduplanefs_batt_voltage
but sanity-check it.


Jordi zegt dat alan het heeft gepakt uit 1 van de projecten van 2de jaar


Mostly trying to avoid unnecessarily nested calls, which requiring reading from the inside-out (and at the same time making it easier to arrive at not-overly-long lines); probably `local({})` is over-kill, but somehow I like minimizing the detritus generated during testing.


This addon does not work on OS X yet. (The latest version paves a way, but it's not quite there.)


don't need to set to zero if you assign it below


Jordi is boos nu


I like that this didn't break any tests.


remove code that is commented out


Thanks!


drop the `peers = `


That's weird: since the first commit that introduced the file unattended_install.py (74d0e09d5109e942a2c4d8c34fa697b3f601a1f2, from January 29), the method setup_unattended_http_server was not used. Are you sure that was ever merged?


Just removed the empty examples placeholder.


I don't think the first assertion adds any value, we don't care about how its implemented. We just care about the bottom assertion.


We don't need to mention that!


Hi, great job. I have followed all your steps, but I am facing some issues:
1) Can you explain better how am I supposed to modify WiringPi2-Python?
2) I hava data from the Pixy using the echo.py, but it is very slow, something around 0.1 seconds. With USB i can go as fast as 100 Hz. What is wrong or how can I setup the speed?


I really really like this battery level offset, I added a feature request for it 6 months ago. Thank you for implementing it! https://github.com/diydrones/ardupilot/issues/2155

It would be good to mention that issue when submitting the PR


why not just do a test like me and haru did with shuffle algorithm XD
http://upaste.me/04c322030cee3059a
if-else = 359~421 mili-seconds
switch = FIXED at 437 mili-seconds

so if-else is better choice ... XD


@Algomorph: This number doesn't appear to be used anywhere and as a result is throwing warnings. Should it be deleted?


I was going somewhere with this trying to fix the scrollbar issues, but now it probably doesn't do anything different from the original version.  I left it because this is what I tested.  Nothing wrong with functions that are only called once.  The compiler will most likely inline it (declared static, important), but I let the compiler worry about that.


This commit makes the liveblog instance crashing.
Is it possible to keep supporting previous data schema ?
Or is there a data/schema migration do to ?

Thank you

```
Killing deployliveblogfeaturepostorderindb_frontend_1...
Removing deployliveblogfeaturepostorderindb_backend_1...
Removing deployliveblogfeaturepostorderindb_frontend_1...
Creating deployliveblogfeaturepostorderindb_backend_1...
Creating deployliveblogfeaturepostorderindb_frontend_1...
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb [status:400 request:0.006s]
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb/_mapping/archive_versions?ignore_conflicts=true [status:400 request:0.004s]
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    app = get_app()
  File "/opt/superdesk/app.py", line 46, in get_app
    app = superdesk_app(config, media_storage)
  File "/tmp/src/superdesk-core/superdesk/factory/app.py", line 112, in get_app
    app.data.elastic.put_mapping(app)
  File "/usr/local/lib/python3.4/dist-packages/eve_elastic/elastic.py", line 234, in put_mapping
    indices.put_mapping(index=index or self.index, doc_type=resource, body=properties, ignore_conflicts=True)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/utils.py", line 68, in _wrapped
    return func(*args, params=params, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/indices.py", line 275, in put_mapping
    params=params, body=body)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/transport.py", line 307, in perform_request
    status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/http_urllib3.py", line 86, in perform_request
    self._raise_error(response.status, raw_data)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/base.py", line 102, in _raise_error
    raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info)
elasticsearch.exceptions.RequestError: TransportError(400, 'MapperParsingException[Analyzer [phrase_prefix_analyzer] not found for field [phrase]]')
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb [status:400 request:0.006s]
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb/_mapping/ingest?ignore_conflicts=true [status:400 request:0.005s]
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    app = get_app()
  File "/opt/superdesk/app.py", line 46, in get_app
    app = superdesk_app(config, media_storage)
  File "/tmp/src/superdesk-core/superdesk/factory/app.py", line 112, in get_app
    app.data.elastic.put_mapping(app)
  File "/usr/local/lib/python3.4/dist-packages/eve_elastic/elastic.py", line 234, in put_mapping
    indices.put_mapping(index=index or self.index, doc_type=resource, body=properties, ignore_conflicts=True)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/utils.py", line 68, in _wrapped
    return func(*args, params=params, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/indices.py", line 275, in put_mapping
    params=params, body=body)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/transport.py", line 307, in perform_request
    status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/http_urllib3.py", line 86, in perform_request
    self._raise_error(response.status, raw_data)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/base.py", line 102, in _raise_error
    raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info)
elasticsearch.exceptions.RequestError: TransportError(400, 'MapperParsingException[Analyzer [phrase_prefix_analyzer] not found for field [phrase]]')
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb [status:400 request:0.006s]
WARNING:elasticsearch:PUT /liveblogfeaturepostorderindb/_mapping/ingest?ignore_conflicts=true [status:400 request:0.003s]
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    app = get_app()
  File "/opt/superdesk/app.py", line 46, in get_app
    app = superdesk_app(config, media_storage)
  File "/tmp/src/superdesk-core/superdesk/factory/app.py", line 112, in get_app
    app.data.elastic.put_mapping(app)
  File "/usr/local/lib/python3.4/dist-packages/eve_elastic/elastic.py", line 234, in put_mapping
    indices.put_mapping(index=index or self.index, doc_type=resource, body=properties, ignore_conflicts=True)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/utils.py", line 68, in _wrapped
    return func(*args, params=params, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/client/indices.py", line 275, in put_mapping
    params=params, body=body)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/transport.py", line 307, in perform_request
    status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/http_urllib3.py", line 86, in perform_request
    self._raise_error(response.status, raw_data)
  File "/usr/local/lib/python3.4/dist-packages/elasticsearch/connection/base.py", line 102, in _raise_error
    raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info)
elasticsearch.exceptions.RequestError: TransportError(400, 'MapperParsingException[Analyzer [phrase_prefix_analyzer] not found for field [phrase]]')
docker: "kill" requires a minimum of 1 argument.
See 'docker kill --help'.

Usage: docker kill [OPTIONS] CONTAINER [CONTAINER...]

Kill a running container using SIGKILL or a specified signal
stty: standard input: Inappropriate ioctl for device
```


Converting to a container which accessors for this


I believe they belonged in `user-config` to begin with...


Thanks, can you please also make the following changes:

- Replace `[array objectAtIndex:i]` with `array[i]`.
- Replace `[array firstObject]` with `array[0]`.


First playable commit?


O certo é "Descanso".


all tests pass:
success: http://buildbot.bos1/builders/erlang-client-test/builds/219


ORIGINAL: Required item is missing.
O certo é "O item necessário está faltando."


(059a454) For now I'm re-adding `raw_pointer_derive`. This will result in a warning when building on nightly rust, but will suppress many warnings on stable.


OK found it, I guess, symfony/symfony older than 2.7 ....


O correto é "Item de Conjunto adquirido".


Oh, yes. Thanks for the catch @dbcam !


1K offset with seek=1


This all has to do with following/followers. Something is happening with the debug check after resubmission. If something changes, then I have to unfollow for every user to adjust the follower list based on the new input file. This is where the problem is occurring. So it's either there is a problem with the functions before the debug check is called or the follow before the program exits is not completed properly.

There are possibly different things we need to look at.


:eyeglasses:


Acredito que ficaria melhor da seguinte forma:
"Seu inventário está cheio e você não pode pegar o item{nl}Por favor libere espaço no seu inventário."


This change causes a somewhat strange result: The index now says "stack undwinding, NNN, *see* exception handling, constructors and destructors". This seems to be the only time we have "see" following a page number.

I think the change is still appropriate, but I'm not sure why it causes a page number to be emitted after the main index entry.


This appears in the middle of the commits on GitHub. I assume that's not actually a problem.


But offset is ignored here. Seems like must be
bs=1K seek=$((1 + ${TRIKIMG_USER_PARTION_SIZE}))


@davidfowl Thanks for pointing out to that. @socket is going to clean up all nuspec files on corefxlab.


Observe how in jekyll/jekyll-paginate/pull/32, I use Jekyll::VERSION within Ruby to get the version of Jekyll. Then I just use condition statements at sections that need the check beforehand.


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


Heißt das, die Zeile mit `__unicode__ = __str__ # python2` bleibt, die `u`'s vor den strings kommen weg?


@binky123 I'm afraid, even if this will pass, it won't be committed. Too many different ideas/changes.


fat32 partition MUST be the first in the PT to be visible under Windows


SUGESTÃO: {@st59b} Loja {nl} {@st59s} - Clique com o botão direito e entre com a quantidade para comprar mais de uma unidade {nl} - Clique com o botão direito duas vezes para vender tudo de uma vez{/}


Oh good :)

- Isaac

On Thu, Nov 19, 2015 at 12:50 PM, Kevin Ryczko <notifications@github.com>
wrote:

> This all has to do with following/followers. Something is happening with
> the debug check after resubmission. If something changes, then I have to
> unfollow for every user to adjust the follower list based on the new input
> file. This is where the problem is occurring. So it's either there is a
> problem with the functions before the debug check is called or the follow
> before the program exits is not completed properly.
>
> There are possibly different things we need to look at.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hashkat/hashkat/commit/28674de8bad505fc8c73d97ff1d14e939532c4ed#commitcomment-14507375>
> .
>


2375 or 1775?


it is solved.


"Itens a comprar" fica melhor, não?


Mesma coisa do anterior, acho "Itens a vender" mais adequado.


I've used your main.go import technique in the main trunk now, in case you would like to use it!

Thanks,
MIke.


This is a great start! I would consider how you could create a topic model that includes 3 different methods to create an animated GIF—one with makeagif.com and 2 others.


Yes, I forgot to take it out, thanks.


"Nível" aqui.


Edit: memset is setting bytes.  '\0' is a byte, 0 isn't.  Makes no difference in the end, but that's how my brain works.


"Consumidos" pode ser trocado por "Necessários".


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


A to jest już plik z wprowadzonymi zmianami ;-)


A. What a fail.


when can this be `true`?


Consider refining your user goal, e.g., a particular kind of layout. Then, you can consider other flexbox user goals, such as flexbox with a video or figure element, or for a navigation element, etc.


Resolved in 1fd71f1cfccb64706c8b0fe3fa71bb2bb4caf5b2.


@robbyoconnor that's where they were, but due to syl20bnr/spacemacs#3881, they weren't getting evaluated in time. I plan on moving it all back once that issue is addressed.


Build 10 test results:  26 passed, 0 failed, 0 errored, 26 total Embed Mattress / CommonCrypto in example app


Perhaps make `IncomingContext` set to `null` by default, so that there's no need to pass the extra `null` argument around?


Ça build,

(T'as pas mis tes lunettes ce matin) ;)


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


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


`5 || 6` should not be a valid expression because `||` and `&&` only take bools. No?

`2 != 3` and `2 == 3`are valid expressions and are being taken care of in another part of the pattern matching.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 107](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=501&buildTypeId=XbimEssentials_XbimEssentials) is now running


SUGESTÃO: {@st59b}Classes{nl}{@st59s}- Aumente o círculo de sua classe para ganhar mais habilidades. {nl} - Você pode escolher entre 3 opções quando for avançar da 1ª para 2ª classe. {nl} - Você pode avançar para a mesma classe até 3 vezes. {nl} - Não é possível retornar à classe anterior após mudar de classe. {nl} - Você deve consumir todos os pontos de habilidade para poder avançar. {nl} - Você pode escolher continuar na mesma classe. {nl} {nl} {@st59b}Habilidades{nl}{@st59s}- Cada habilidade pode chegar até o nível 5. {nl} - Atributos adquiridos de Mestres podem ser visualizados abaixo das habilidades. {/}


@evancz I don't think it's too aggressive; it's standard industry practice and I don't really worry about it.


@BrasilToS revisado.


Looks really good!


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 107](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=501&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:19


you mean
`` ( rightTan - leftTan ) * xScale * 0.5 ``
instead of
`` -( leftTan - rightTan ) * xScale * 0.5 ``
?
If so, no reason


Nice. It probably makes sense to break out the slab test into a seperate test file. I can think of a number of tests to do and they will all serve as references.


This argument no longer works as of 1.8.5 using Django 1.8+, as it has been converted to a positional argument. We should either remove this argument or use it to overide the default (127.0.0.1:8000) when it's been set.


This doesn't look right to me. I think you need to change the `{}` with `<%= =%>`, don't you?


I can't figure out the reason why  dishornoring the -d option with --key/--cert:
I frequently run it as:
  docker run -v `pwd`/certs:/certs ehazlett/certm -d /certs client generate -c admin.k8s.solofox.cc -o solofox.cc --cert admin.pem --key admin-key.pem

but after apply this patch, the admin.pem/admin-key.pem go to / but not /certs


That would be great, thanks for your help!


I used the singlton approach to ensure that only one yelpAPI is used, and many instances of the same object are not created because it is unnecessary to create multiple connections to yelp


We need to say that A is familiar with B if A and B are the same browsing context.  This used to come for free with the origin check, but doesn't anymore.


Okay, I'll add it, just wasn't sure if making people refresh like that was lame :)


@kutsyk це в меседжес.пропертіс треба зберігати


People shouldn't have to refresh...they should just get the new version automatically - only downside should be additional bandwidth used, and we're not talking a ton of bandwidth. :wink:


Did you mean to comment out that first `capturex`? I'm assuming that was just to test the `rpm -E` call and you forgot to uncomment it, yes?


No. I can see why you're confused, but it's standard Ruby String interpolation here. We're already "inside" an ERB tag.


This may be looser than needed for web compat.  For example, I believe that in Gecko if A opens B and B opens C and they are all different origins then A is familiar with B and B is familiar with C but A is not familiar with C, whereas per this spec text it sounds to me like A would be familiar with C.  What do other browsers do?


@kutsyk ми ж не на пхп пишемо, повертай об'єктом


fast-forwarding end-to-end/timeseries to borshop-integration-250-fixes/am/fix_encode_timestamp_cells = c14f65d4


Many thanks for your testing, @adunning.

Here is the sample:

``` markdown
---
mainfont: FreeSerif
...
effective flew

<span style="font-variant:small-caps;">Small caps</span>
```

Using template from current master, the result is (no ligatures and no small caps):

![no-liga](https://cloud.githubusercontent.com/assets/203767/11279408/be5ac330-8ef0-11e5-9e7c-97f3d37a7e4e.png)

With the proposed code (ligatures and small caps):

![with-liga](https://cloud.githubusercontent.com/assets/203767/11279407/be574aa2-8ef0-11e5-92e5-c268a67aa041.png)

I’m using current beta from 2015.10.09 21:28.

With current stable from 2015.05.18 12:26, I get an empty page with both versions from the template (current master and my proposal).


Nitpick: can you add a space between `if` and the opening parens? `if (...)` instead of `if(...)`.


Good lord, my fault. Working now lol


Instead of `exitCode` being global, I'd recommend returning an `int` from `realMain` and use that for `os.Exit()` It's almost always better to not keep a global variable.


Yes, they are...I would swear that the local test was successful but as usually I was wrong.


The fourth condition needs to account for disowning too.  Consider the situation in which browsing context A, origin 1, opens an auxiliary browsing context B, still origin 1 and then B is disowned. Now A can't target B, but A can target any descendant frames of B, which is quite odd.


I wonder whether ultimately it should be called just `embassies.govspeak.erb`. I'm guessing the `overseas-passports-` prefix was because that's the flow in which it was first used...? Anyway, probably a job for another day.


Yes. I should have mentioned that including the entire Hive dataset implementation is a work-around for a rather annoying bug in Kite's Hive support that tries to load hive-exec classes with a fallback, but doesn't catch NoClassDefFoundException, just ClassNotFoundException (the former being thrown when a class's dependencies aren't found). This would go away after the next Kite release, which would hopefully include this support.


There's already an escape code for a literal tab you can use - `\t`


@stinos : I'm going merge this for now, feel free to submit follow-up PR.


note that the compiler's going to complain about non-exhaustive pattern matching even though it will never be Eq etc in this case. I'll add a catch-all error case to make it play nice w/ ocaml


Since most of these classes are temporarily included, I didn't want to modify them too much. After the next upstream release this would change quite a bit. I guess it would still be better to relocate them though, you're right.


mhmm, I don't see them. What for?


No, I just missed this one.


As an aside, I wonder whether the `_button` partial should use Govspeak/Markdown to create the link tag.


IMHO it's better to name it `allowAddingImages`.


Looks great, excited to hear about it


Aha, of course. My bad!


Ezee, how you are planning the "presets loading feature" from a user interface point of view?

I suggest use the menu to list the available presets, something like this:
![image](https://cloud.githubusercontent.com/assets/1012741/11279705/3cb815f6-8ed9-11e5-93f3-9c6cf62dfe19.png)


Okay cool!


I believe we're shooting for next week. Currently testing.


Hard to tell without having a full project, but be sure to double check this is hooked up.


Don't use `else` after `return`.


AFAIR there was something about `size of the images + 1` ?


Mierrrrda, qué difícil que hacés que sea pasarte en líneas escritas y borradas.


When I matched up using the tab literal `\t` versus the GNU `cat`, `cat` seems to use five spaces instead of 8 as a tab.


... and there is no error message :-(


Понял. Хорошо.


;) no problem thanks for reporting!! :D


Hard to tell without the complete project, but double-check this is hooked up in Interface Builder.


Try setting a breakpoint here to see if the message gets through at all.


thanks. and friendly reminder to add signed-off-by tags next time ;)


u did not add the correct things to ur gitignore. check on that :)


I think by far the most likely explanation is that the state is not
reloaded correctly, no?
On Nov 19, 2015 12:55 PM, "Isaac Tamblyn" <notifications@github.com> wrote:

> Oh good :)
>
> - Isaac
>
> On Thu, Nov 19, 2015 at 12:50 PM, Kevin Ryczko <notifications@github.com>
> wrote:
>
> > This all has to do with following/followers. Something is happening with
> > the debug check after resubmission. If something changes, then I have to
> > unfollow for every user to adjust the follower list based on the new
> input
> > file. This is where the problem is occurring. So it's either there is a
> > problem with the functions before the debug check is called or the follow
> > before the program exits is not completed properly.
> >
> > There are possibly different things we need to look at.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/hashkat/hashkat/commit/28674de8bad505fc8c73d97ff1d14e939532c4ed#commitcomment-14507375
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hashkat/hashkat/commit/28674de8bad505fc8c73d97ff1d14e939532c4ed#commitcomment-14507477>
> .
>


Oh ha - interesting!


:thu


@nathandunn just wanted you to be aware of this one. All the RC versions up to this point didn't have the "paging" working properly on the annotatorpanel. This fixed it :)

That line is unnecessary with the new WebApolloSimplePager


Werte wie `NP`, `PR` und `CO` sind aber auch nicht leichter zu verstehen. Und unnötig ist die Abhängigkeit auch nicht, weil das der einzige Weg ist, um die Vermieterart vernünftig zu implementieren, ohne selbst einen pseudo-enum zu bauen. Noch dazu ist es einfacher zu lesen und zu verstehen.

> Kosmetische Änderungen zu diesem Zeitpunkt stören zu diesem Zeitpunkt nur beim mergen.

Zugegebenermaßen sorgen solche Änderungen für einen geringen Aufwand, aber da ich das mergen bereits übernommen habe, reduziert sich dieser Aufwand auf einen button bei GitHub.

> Abgesehen davon ist das nicht mein Codingstyle.

Das macht es leider nicht leichter, das als Gemeinschaftsprojekt zu betreiben. Wenn du einen bestimmten Codingstyle haben möchtest, sollte der zumindest irgendwo im repository festgehalten werden und am Besten von einem linter überprüft werden.


Probably something with how they write to their line buff https://github.com/goj/coreutils/blob/rm-d/src/cat.c#L61-L66


Ok en fait tu as fait un commit sur ton propre fork ! Je pense qu'on va aller vers des propositions d'ensemble cohérantes, cf #26, ma réponse. Dans ce contexte, tu proposerais ta vision uniforme de l'application, quitte à en rejoindre une autre. On a pas encore établit comment ... Je reviendrais à ta proposition le moment voulu !


you actually can't do this since they're different types, so they need to have different pattern matching


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 108](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=502&buildTypeId=XbimEssentials_XbimEssentials) is now running


Unrelated, but the commented-out code on lines 97-113 can probably be removed.


That's really strange; probably something to do with FreeSerif. Does it still work if the line is kept as `\definefontfeature[smallcaps][smallcaps]`? Without that, OpenType features are not being applied to the small caps for me.


Good thinking, will remove.


@mostafah This commit fixed some windows related issues. I can make a PR if you want.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 108](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=502&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


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


any questions here?


Only after deep deliberation and consultation with the finest minds was I able to reach that conclusion.


Does geth do this? We should simulate geth, and last I checked it errored if you didn't pass an address. Happy to accept if geth has changed behavior since then.


I'd prefer to not add more functionality to View, but it could be done.


400 could be replaced with http.StatusBadRequest


Does it merge cleanly with all the refactoring? If not I can rebase it.


Да. Если количество равно то можно добавить ещё одно изображение - соответственно получиться images+1,
а если в серии уже больше изображений, то тогда ничего не добавиться. :)


LGTM


Not in this commit. See ContextualCheckBlock.


meh sri y u merge thees stoof go poot en icun ;-;


@atd, please see https://github.com/apache/cordova-plugin-whitelist#content-security-policy where it is said
> On Android and iOS, the network request whitelist (see above) is not able to filter all types of requests (e.g. <video> & WebSockets are not blocked). So, in addition to the whitelist, you should use a Content Security Policy <meta> tag on all of your pages.

Should we add ```<meta http-equiv="Content-Security-Policy" content="default-src 'self'"/> ``` allowing all content from our site or should we be more careful?


I appreciate the effort to use recursion, but this code is pretty complicated. Without running, I have no idea if it will work.


I love this user goal. I think it needs some refinement, as to make a more explicit connection to the DIV or SPAN. Do you see it like a decision tree, where based on one user intention you provide a use DIV case vs. another user intention use SPAN? Again, I like this, but you may need to consider 3 different use cases where the 2 **could** be used but each has their own distinct path even though the rendered output may look similar/same.


We should undo this now since the bug in the dependent library has now been resolved.


Thanks! BitHub has sent payment of  $38.13USD for this commit.


Thanks! BitHub has sent payment of  $37.37USD for this commit.


Thanks! BitHub has sent payment of  $36.62USD for this commit.


Thanks! BitHub has sent payment of  $35.89USD for this commit.


... а куда и о чём выводить сообщение?
Тут мы вообще-то вообще не должны оказаться, если использовать нашу страницу...
Но при желании, конечно, можно и сюда попасть.


Great! I suggest to extract this check (`series.getImages().size() > series.getQuantity()`) to the method. WDYT?


@andy-armstrong is the spacing right here?


Looks good :+1:


Working. Didn't see you did a couple of commits. Thanks!


Missing the last parameter


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


Why isn't this change made here instead: https://github.com/ckeenan/EtherSim/blob/develop/lib/manager.js#L82

Should be more direct and enforce "latest", I think.


What's "value"?

Without looking into the macro (aka the assumption of someone new looking at my test), I assume that CheckBool returns false if the two bools don't match, true if they do, with that in mind:
If retVal is false and expectedReturnValue is false, you need to return true, your suggestion works.
If retVal is true and expectedReturnValue is false, you need to return false. I don't see how your suggestion works for the previous case and this one.
If retVal is false and expected returnValue is true, you need to return false, and that also doesn't combine nicely with the first case.

I may very well be completely misunderstanding you, but I really would prefer to leave this bit of code as is (especially since I don't have much (any) more time to spend on this test).


Sub::Name could be provided by an OS package or similar, so this approach isn't viable.


p


We could add "global" error show message that "you can't add more images, ask admin about it" (something like this).

>Но при желании, конечно, можно и сюда попасть.

Yes. Open page in 2 tabs. Add images at the first tab then switch to another tab and try to add image from here.


Now that I see the context around this, I have some further comments on this change. (I originally didn't realize this was on a branch as feedback on a review)

There's a lot of overhead in code reviews, particularly in asynchronous reviews. So in reviewing I think we need to:
 - Focus on the impact on users: API design, release notes, documentation
 - Focus on test coverage: integration tests for main cases and failure cases, unit test coverage to avoid integration test blowout.
 - Focus on macro design issues: class coupling, interactions, function duplication, API design
 - Be tolerant and not focus on readability and style, particularly for code encapsulated in a single class

Now I'm _not_ saying that we shouldn't focus on readability and style when we are writing code. We all have a lot to offer here and I think we can work together to improve the codebase. But I do feel that trying to suggest these improvements in a review can very easily lead to bike-shed arguments. And in an asynchronous review these comments can add overhead that far outweighs any benefit gained.

So back to this particular case:
- If this commit _was included_ part of a broader story that touched on one or more of these classes, I'd be fine with it and wouldn't mention in a review
- If this commit _was not included_ as part of a story that touched on these classes, I'd also be fine with it and wouldn't mention in a review.

If we were discussing the code in person, I might make a comment, but in Crucible or GitHub I wouldn't.

@cbeams Some hints on how to review would be good to document.


That's how I learned the difference between `echo >` and `echo >>'.


The reason I didn't add the timestamp in createBlock() is that, arguably, in some cases, the block will be created long before the transaction is ever run (i.e., we set up a pending block to prepare for the next transaction, but the next one might be a ways out). This would make the timestamp off by a bit.

How about keeping your changes but also updating the timestamp right before the transaction is run (i.e., in `processCall` and `processTransaction`). Good solution?


Can you leave a comment on why it's C = 1e5?


the widget's name ``DATATYPE`` confuses me a bit. I'd prefer ``my_widget`` or similar.


beech
fix the conflicts x'D x'D


Not sure exactly what you mean in terms of "what" to tag, but I had intended on tagging every 'unique' build (i.e., `uoa-inzight-base:R322Shiny140756`, `uoa-inzight-lite:20151120`). There's no obvious tagging scheme for the secondary image (base contains R and shiny of some version, inzight-lite contains all the packages as they were on a specific date, so a date makes sense?) Thus the Dockerfiles could just specify `FROM uoa-inzight-base:R322Shiny140756` providing us with an unchanging, stable release.

More comments on a few things in Issue #3.


Meh.


Just curious, should we be buffering all response channels? Asking for a friend...


It returns a view of a ndarray. Tested and confirmed it works :)


@srisurya95  :'3


awesome :)


meh fix cunfleects :'3


Should remove qualifier "ZkWrapperUtils."

I moved this method from SolrZkClient and forgot to remove the qualifier.


Yup. Works for Varkey/Veckey substitution.


this is, in general, a bad idea - to _execute_ a call to a function from a _definition_.

Config can be broken or just don't exist. In real scenarios this is often the case - when user inserted an extra curly brace or simply deleted a line not noticing it. And this is something that you should bear in mind when filling default values for your internal config structures.

So, I'd suggest you to get back --define(PORT, 8088) line, insert it into your -record definition as a very default value, and load an application config in init/1 function, correctly handling the situations mentioned above.


Unnecessary parenthesis here.  They're all using `||`.


Why you check if only one args passed, what should i do if i whant create Error action and pass meta data ???


wew


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


x'D x'D x'D


Yeah, but that'd be pretty inconvenient for users.


``unicode`` datatype converter missing?


cm-13.0 blobs for yu?


You did it first. ;)


Nicely tuned.


This is just for convenience, to determine what name a model ended up having.


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


The interface has an exported method but the parameter has an unexported type. Is this a problem?


We're not only updating Hashvectors here, but also regular dictionaries.


bob only has a first name


Sure, please do.


simple enough.


descr is taken from the first varkey of a given name in self. In the first for loop below this can be compared in various attributes to all the other varkeys of that name.


what about favorite color


Sounds good.


Thanks @DaveOps83 -- I've left it in for now, but it is getting annoying. I think it's just that we're passing this stuff around everywhere to create the individual clients, which is cumbersome (even just to set the region) and a bit risky. If we use a factory to create the various different clients, this can all be in one place and shared between production and test code...


This line has to be protected by HAVE_MUELU_TPETRA. Otherwise, the ETI macros are not available. The easiest fix is to move that below.


(the expectedReturnValue is what I expect to come back from the parsing call, not what I expect to return from test_ParseTerm)


Same as above: protect by HAVE_MUELU_TPETRA


add color


The answer is no.


In keeping with Matt's question, I also think this goal needs refinement. Consider a particular audience and context for the figure element and how that particular context could yield different user goals with the same element pairing. Here are some resources to hopefully help you consider some ideas:

- [http://stackoverflow.com/questions/29927307/special-use-cases-for-the-figure-element-in-html5](http://stackoverflow.com/questions/29927307/special-use-cases-for-the-figure-element-in-html5)
- [https://www.paciellogroup.com/blog/2011/08/html5-accessibility-chops-the-figure-and-figcaption-elements/](https://www.paciellogroup.com/blog/2011/08/html5-accessibility-chops-the-figure-and-figcaption-elements/)


Done.


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


Why did you change the encoding? :p


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


Thanks! BitHub has sent payment of  $35.18USD for this commit.


I'm trying to install elm with Homebrew and I'm getting a sha digest of 'e55a6e993264b29bf917df79ccb383ea99260082108d8c07b668718325478f1f'

I manually downloaded the package from http://elm-lang.org/install and ran "sha2" on it.
Possible that the elm guys updated the package since you made the commit?


can you leave a comment to explain that the lambda is the measure of loss aversion (if that's right) and not a regularization parameter for the model (if that's also right)


Never mind, saw the next commit.


Great concept Binky!   What do you think of two Pvs for every Nth thread (in this case N=4)?


I should add that the problem I mentioned (having the time off by a bit) hasn't yet been an issue, so perhaps we should only fix that when it matters to someone.


> but I wouldn't include REPL-specific gunk in the answer ($line4.$read$$iw$$iw), but use whatever result plain

I know - I was considering that, too. The thing is that the "environment" for all the puzzler examples, and output is "a clean REPL". Let's see if I can tweak the puzzler a bit to remove this...


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


closes #3


WDYT? Я думаю что такое WDYT! :))
Из-за двух раз использования... Ох...Учиться и ещё раз учиться! :)
Поэтому, конечно, извлеку метод!


> LGTM

Thanks for the review!


Updated dependencies lately, saw warning, paranoia led to googling the warning, found this, and the explanation is understood.  All good, thanks!


Fixed in https://github.com/caskroom/homebrew-cask/pull/15305.

Next time, please open up an issue, as it's much easier for us to notice.


Great addition.  Thanks.  Try to switch to the /// comment structure for symmetry.


Can you please comment, why you check for those raw post data here?
I'm not sure at the moment what your intention is.

Also please add an issue and try to stick to the commit message format we use for the Core. We try to keep the "rules" here as similar as possible. Thanks.


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


You are correct, but Array[(String|None)] is a really awkward type to work with.  I figured if someone cared about the difference they could fallback to apply(i) for the ambiguous index.


OH MY GLOB


It might have been better to handle them separately, but I was making the change as needed.  This is part of a larger effort that I am doing to add support for globbing.


or ... I just did. Those path vs filepath issues have created some gray hairs, thats for sure.


Read comment inside


Getting lots of processing failed errors from the postprocessor.  Below is a sample

```
2015-11-18 19:44:33 POSTPROCESSER :: [_UNPACK_Buffy.The.Vampire.Slayer.S04E08.DVDRip.DivX-AMC : Being unpacked]
2015-11-18 19:44:15 POSTPROCESSER :: Processing failed for G:\tmp\download\From.Dusk.Till.Dawn.S02E10.480p.HDTV.x264.upload-hero.mkv:
2015-11-18 19:34:31 POSTPROCESSER :: Problem(s) during processing, failed the following files/folders:
2015-11-18 19:34:31 POSTPROCESSER :: [G:\tmp\download\From.Dusk.Till.Dawn.S02E10.480p.HDTV.x264.upload-hero.mkv : Processing failed: ]
2015-11-18 19:34:31 POSTPROCESSER :: [_UNPACK_Buffy.The.Vampire.Slayer.S04E08.DVDRip.DivX-AMC : Being unpacked]
2015-11-18 19:34:14 POSTPROCESSER :: Processing failed for G:\tmp\download\From.Dusk.Till.Dawn.S02E10.480p.HDTV.x264.upload-hero.mkv:
2015-11-18 19:24:27 POSTPROCESSER :: Problem(s) during processing, failed the following files/folders:
2015-11-18 19:24:27 POSTPROCESSER :: [G:\tmp\download\From.Dusk.Till.Dawn.S02E10.480p.HDTV.x264.upload-hero.mkv : Processing failed: ]
2015-11-18 19:24:27 POSTPROCESSER :: [_UNPACK_Buffy.The.Vampire.Slayer.S04E08.DVDRip.DivX-AMC : Being unpacked]
2015-11-18 19:24:14 POSTPROCESSER :: Processing failed for G:\tmp\download\From.Dusk.Till.Dawn.S02E10.480p.HDTV.x264.upload-hero.mkv:
```


It's hard for me to tell exactly what is going on here without firing this up in a REPL but with `reduce` I expect a single answer out at the end. is that single answer a `map` that was formerly a `seq`?


(nonblocking) it might be clearer to structure the logic as
```
sql_values = {}
if startdate is not None:
    sql_values.update({'%s_startdate' % self.filter.slug: startdate})
if enddate is not None:
    sql_values.update({'%s_enddate' % self.filter.slug: enddate})
```

I find this to be more explicit and easier to follow.  Bonus points for including an ```assert startdate or enddate``` if you want to make sure something gets returned.


TeamCity LabKey :: Trunk :: LabkeyTrunk :: Git sqlserver 2014 [Build 61](https://teamcity.labkey.org/viewLog.html?buildId=286295&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:02


i suggest refactoring tracing() to trace() at some point.


this is wrong


The code should be moved out of muelu/src/MueCentral to adapters/linear_solver_factory.


this needs to be removed


Consider using the [`BANNED`](https://github.com/Wikia/pandora/blob/master/lib/service-core/src/main/java/com/wikia/pandora/core/auth/Role.java#L16) role in this case. I believe it will return `403` and we can decorate the error message with information that the user was banned (or at least log).


Should start with lowercase?


You're right!


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


sketchy...  :)


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


if (shardId == null) ...
is clearer. best methods have a single return at the end of the method.


I'm only ok with it because the comment next to the `else` above gives me hope it wil be re implemented someday


This also fixes #938.


Corrected on master.


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


You have to drop elastic index and drop the mongo db and recreate then back as for a new install.


got it.


Aqui é "Guerreiro do Teclado" novamente.


because of the way we're doing pattern matching, we're throwing away the `Sast.Expr` that wraps the value returned by `check_expr`. Is this intentional? i.e. should a checked version of `-5` be
```
Sast.Expr(Sast.Unop(Sub, Int_lit(5)), Num)
```
(what we have currently), or
```
Sast.Expr(Sast.Unop(Sub, Sast.Expr(Int_lit(5), Num)), Num)
```


I know I'm becoming a broken record, but how about adding exactly what you said above as an inline comment


Temos um conflito entre termos: "Atributos" para stats (STR, INT, DEX, etc.) e "attributes", que são as melhoria das skills. Sugiro pensarmos em uma alternativa para tradução de stats.


Fancy! I like it :D


This comment is hard to understand.


Типа этого?
response.sendError(HttpServletResponse.SC_NOT_FOUND);
А какое значение использовать?
И я правильно понимаю, что мне будет нужно соорудить страничку с нужным номером?
И добавить в messages соответствующие сообщения и не забыть про локализацию.


I strongly recommend to keep this check because it adds security. You never know if the sweeper is blocked/down or not.
(BTW IF you remove it, you can remove the exception class NewDeviceRequestExpiredException too...)


Buenisima @mroffe arregla el issue https://github.com/mercadopago/cart-magento/issues/90


What is this?


Would it be worth it to log an a warning here like "WARNING: RC updated directly -- someone may have been using an old client" to make it possible for cluster admins to track down this behavior?


worried about all the parsing errors that could occur here.
e.g. there could be an NPE here.
it might be better to get all config syntax errors done early on, because they're permanent, before the bucketmanager is instantiated.


Oh yes, please do. I should have used `filepath` from the start myself, but since I’ve never used a Windows machine I didn’t notice the problem with `path`. Thank you.


CBECC (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Thanks! BitHub has sent payment of  $34.49USD for this commit.


LGTM


} else if (...) {


Sounds good. I backed out of this change until I find a solid way for mocking the service response.


@jcfr more changes https://github.com/naucoin/Slicer/commit/c9545648f5432214c80df0dce6bdbb1b851526c1


I meant to do this as a pull request, but my git was misconfigured. No harm done.


Looks good, i will test it a bit more, but a quick test in a VM shows the selecting bg colors works.
Thank you.


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


Or you can just leave `inherit eutils` there. It should not hurt anything.


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


Thanks for the quick fix


should be "contiguous" :)


@akhilnarang


why not just throw the exception and avoid the status variable?


Yes, this assumes that if Sub::Name is not yet installed but is about to be, then it will be the newest (or newish) version. JSON::MaybeXS makes the same assumption.


x'D x'D


Not really. I have merge the patch because there was indeed a typo. But the suggested
correction is not correct, which means that the text is not clear, and it'd better be at this
point in the text !  I have added some text that hopefully makes things clearer.


gret


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 90](https://teamcity.labkey.org/viewLog.html?buildId=286283&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 42 (3 new), passed: 245 Build time: 00:43:47
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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


Esta linea siempre va a entrar en el `if`; lo que esta haciendo es evaluando si existe "restart" y si existe por que importaste.


wait wrong filename xd


Will any of this break multiplayer support, and could you perhaps make something more obviously geared towards specifically selling guests the same color as their shirts?  I see there's something about that in there, but it looks like it's still partly lost in the randomization.


Sounds good


But we have public instances that we need to update and we can't drop the
data like that :)
Is there a way to migrate the database instead or to support previous
version?

On Thu, Nov 19, 2015, 19:54 ioanpocol <notifications@github.com> wrote:

> You have to drop elastic index and drop the mongo db and recreate then
> back as for a new install.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/superdesk/superdesk-core/commit/7982600596c2615da30f448a1a61caab2c8ef6af#commitcomment-14509003>
> .
>


@lenhard This code change breaks our BuildInfoTest. Please take a look if it needs to be adapted or if you need to change the code again.
```
net.sf.jabref.logic.util.BuildInfoTest > testDefaults FAILED
    org.junit.ComparisonFailure: expected:<[dev]> but was:<[]>
        at org.junit.Assert.assertEquals(Assert.java:115)
        at org.junit.Assert.assertEquals(Assert.java:144)
        at net.sf.jabref.logic.util.BuildInfoTest.testDefaults(BuildInfoTest.java:13)
```


Woof. Redundant `self` detected. Woof.


Seems to work fine and players get put on separate teams when the game starts.

First time we tried to start the game it crashed though, after that it was fine, not sure if it was related to this fix.


I don't think calling ReuseTpetraPreconditioner for a changed matrix is the right thing to do. Your matrix may have changed structure, or something like that. I think the proper sequence needs to be:

if (solver_ == Teuchos::null || changedA_ || changedParams_)
  solver = CreateTpetraPreconditioner();
// do nothing otherwise


It works for me just fine, no errors and I see both DATA and ANAM. I will revert it back for now.


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


@cjh1 this LGTM,  but I have not tested it,  if you'd like me to test it i may need a quick tutorial


Woof. Woof. All these redundant `self`s. Woof


@akhilnarang


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


kehh


You make me think.
Let's talk about it tomorrow in the car.

Groetjes,
Bert

On Thu, Nov 19, 2015 at 7:57 PM, aristotelos <notifications@github.com>
wrote:

> I strongly recommend to keep this check because it adds security. You
> never know if the sweeper is blocked/down or not.
> (BTW IF you remove it, you can remove the exception class
> NewDeviceRequestExpiredException too...)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yonadev/yona-server/commit/12a6f97e2020bf400f2bab40496f7dda26bade29#commitcomment-14509064>
> .
>


Please remove this DIV with the Privacy Disclaimer. I'll update the UI specs on the wiki.


ah lulzkay
i thot for other drawable :3


I'm thinking`prepareMedia` and `upload` should also be plugins. Just plugins of the type `prepare` and `upload`.


> Will any of this break multiplayer support

It should not.

> and could you perhaps make something more obviously geared towards specifically selling guests the same color as their shirts

I am experimenting with this. Please see #2154. I am aware that choosing randomly looks bad. But I'm not sure using the shirt colour is much better. But I'm open for ideas.


dot-comma separation of titles doesnt seem to work for some reason.


so there is going to be an ExecutionSerializer2 per actor?


Added license header


Läuft doch super ;)


The assumption that this makes is that if Sub::Name is installed, then it can be upgraded.  That is not a safe assumption.  Sub::Name can be present even if a compiler isn't.

JSON::MaybeXS doesn't do any checks on the JSON::XS version.  It just adds Cpanel::JSON::XS as a dep if JSON::XS is not installed, and if a compiler is available.


What will the core do then? Just sit there all like “bring those plugins to me!”


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


Тут эта строка не нужна


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


TeamCity pull requests :: netty [Build 685](https://garage.netty.io/teamcity/viewLog.html?buildId=3376&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5298, ignored: 21 Build time: 00:14:52


@gubatron help!
after you commit, crached.
My "databaseFile"  == /home/xnicon/.frostwire5/library_db
library_db - dir name. not file.
Can somehow improve this solution?


Certain point sont à revoir:
L'usage de $_POST
Ton fichier ne finit pas en .php

Pense à envoyer tes commits sur le project principal sinon je ne l'ai voit pas .....


Yes, knowing how to wire the inputs & outputs of different types of plugin together I think


@kotfic I tested this pretty well, but would be happy to give you a tutorial for the future ...


It should work also for CFD symbols.


Hi,

> into the macro

Nitpicks: It is a function

>  I assume that CheckBool returns false if the two bools don't match

Exactly

Nitpicks: Function is in fact named `CheckInt` (`CheckBool` does not exist )

In other word, if `expected` and `current` value are different it fails.

>  If retVal is false and expectedReturnValue is false, you need to return true, your suggestion works.

ok

> If retVal is true and expectedReturnValue is false, you need to return false. I don't see how your suggestion works for the previous case and this one.

As you reported earlier the two value will not match and `CheckInt' will return `false`. Things are ok.

> If retVal is false and expected returnValue is true, you need to return false, and that also doesn't combine nicely with the first case.

Again, as you reported the two value do not match and `CheckInt' will return `false`.  Things are ok.

> I may very well be completely misunderstanding you, but I really would prefer to leave this bit of code as is (especially since I don't have much (any) more time to spend on this test).

Let me know if things are still not clear.

Finally, here is the table of truth of the current set of check:

| retVal | expectedParseFlag | Value returned by test_ParseTerm |      Comment                     |
|---------|--------------------------|----------------------------------------------|----------------------------------|
|   0      |          0                    |                            1                            |  Premature exit. I suggest you check the content of  StandardTerm struct|
|   0      |          1                    |                            0                            |                                          |
|   1      |          0                    |                            0                            |                                          |
|   1      |          1                    |                            1                            |         implicit true               |

And here is the table of truth of the `CheckInt`  function:

| retVal (current) | expectedParseFlag (expected) | Value returned by checkInt |
|---------|--------------------------|----------------------------------------------|
|   0      |          0                    |                            1                            |
|   0      |          1                    |                            0                            |
|   1      |          0                    |                            0                            |
|   1      |          1                    |                            1                            |

The table are identical hence same behavior but simpler code.

Considering the your implementation, someone could see a failure message like:

```
Line 123 test_ParseTerm: failed to not parse invalid string
```

the double negation will be confusing ...

If we update based on my suggestion:
 * move check for expected/current return value at the end + use CheckInt

Then:
 * test will most likely fail because one of the test checking the content of `StandardTerm` failed first. This will provide some valuable hint.

And ultimately, if the content of the `StandardTerm` are as expected but return value are still different, the message will look this:

```
Line 123 -  ParseTerm Return Value:  CheckInt failed"
    current : 1
    expected :  0
```

Which means that the function returned `1` whereas `0` is expected.


This still feels like core to me for now, but if we do make these as plugins, we should’t make people do: `.use('transloadit-upload')`.


true - I'll fix that.


I'd prefer to keep all logic (and internal logic comments) in the mdver specific functions. I wouldn't add the last check here, but instead move it to bump_global_generation(). Also, don't put the verbose comment above, just say "notifying the MD Versioning component of transaction commit". You can add a nice function comment on bump_global_generation() to explain what it does.


Forget about the earlier comment I drafted to add two   next to the XID badges to align them with the HUID ones. Once I learned we also have OTHER and LIBRARY I decided to forget about that alignment. So no additional   next to badges any more.


Thanks! BitHub has sent payment of  $33.80USD for this commit.


I don't buy this answer. how comes 0 isn't a byte?


We can use the presets to avoid that. But i think it's nice if any 'implementation' tech could be in a plugin.


Rename this function the start with mdver_
All externally visible functions of the feature should have a common prefix so that is easy for people to understand what they are about.
Suggestion: mdver_bump_global_generation().


my bad, will fix right away! (did this last night and didn't test)
Thank you


The biggest difference is that I can't see what did you change in the code.


I like the idea, because it implies a favorite color that they've bought not only in shirt, but everything else.  I don't mind ugly colors, and I like the randomization aspect, I just want shirt color to be a different option, or perhaps that just some peeps, when encountering random, will always get their shirt color.  Something of that sort.


For instance I'd like to use a `tus 2.0` uploader in some cases without that being coupled to the entire lib


Bra!


If you really want to do this then please do the indentation correctly and explain why enabling the network should clear the EVENT_DATA event and the variables.


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


Или просто сделать страничку с сообщением и выводить её?


Строка `provider.getCategory()` загружает объет категории из БД, нам это не нужно так как данные категории уже были предварительно загруженны. Для получении имени категории нужно использовать `org.ayfaar.app.utils.CategoryService.CategoryProvider#extractCategoryName`


Are there any real problems caused by not having a newline at the end of the file?


Here's a somewhat updated version.


So, instead of duplicating code in soft_reset and prox_network, move this into prox_disable_network ?


> JSON::MaybeXS doesn't do any checks on the JSON::XS version

It does: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=p5sagit/JSON-MaybeXS.git;a=blob;f=Makefile.PL;h=e4a2dab64e15a919627fd32a5569a88f51db7b74;hb=HEAD#l97


Acredito que "Usar conservantes" faz mais sentido.


ah well if the REPL is the standard...


понял зачем мы задаём "spring.profiles.active" ?


Fixes #3


wouldn't changing `object Completed` to `case object Completed` do the trick?


[Fixed!](https://github.com/frostwire/frostwire/commit/9d0013e1018f04f9f46a7fbd966068b209fa096b), thank you.


Вот эта проверка ```if (tmpPow2&(tmpPow2-1))``` работает для 2^31-1.
Но возникает проблема с выделением памяти. Ведь возможно то, что просто не получается столько выделить? Ошибка Invalid allocation size: 4294967295 bytes вылетает именно на ```pMem = new TQueueElement[MaxSize];``.


can you open a PR and ill get it merged?


Apart from being hidden behind a pawn,
or having a pawn "not in the way" of the minor piece (which is already handled in evaluate_space)
or blocking an opponent passed pawn (which is already handled in evaluate_passed_pawn)
what is the strength of the MinorBehindPawn concept ?

Well, I don't think it come from being "behind a pawn".
Here is my reading:
Case 1) Knight f3 behind pawn f4. The Knight is supporting the attacks of pawn f4
Case 2) Bishop g2 behind pawn g3. The Bishop is supporting the sides of pawn g3

Here we count also:
Knight supporting sides of pawns, and Bishop supporting attacks of pawns.

And we do not care about the exact position of the Minors (can be behind or not)
or the rank (can be something else then rank_4 and below)

So a Knight on f3 can support also get a bonus for supporting attacks by a pawn on c3
And a Ba2 a bonus for supporting sides of distant pawns d4 or e5.
(no xray in this test)

Weights Minor1 and Minor2 were found using a short local SPSA session.

If the concept works, other measures can be introduced to refine the Minor + pawn coordination.


Druga kropka to było raczej pytanie niż sugestia. Poza tym chyba jest ok.


empty line


Done.


I'm badly looking for this.! when are we packaging this and uploading to pypi


Do not forget to add decoupling capacitors:
* On each SN65HVD233DG4 Vcc pin => 100nF
* On LM317 (+12V) In pin => 100nF (with Voltage - Rated at least 25V) and (VUSB) Out pin => 1uF


I had a thought about this:
what if we were to split `Py3BasePackage` like this ?
```
define Py3BasePackageFiles
  PYTHON3_LIB_FILES_DEL+=$(2)
  define Py3Package/$(1)/filespec
    $(subst $(space),$(newline),$(foreach lib_file,$(2),+|$(lib_file)))
  endef
endef

define Py3BasePackage
  PYTHON3_PACKAGES+=$(1)
  $(eval $(call Py3BasePackageFiles,$(1),$(2))
endef
```

And call `Py3BasePackageFiles` for python3-dev and python3-lib2to3 ?


You can try to rebuild the index:
https://github.com/superdesk/superdesk-core/blob/master/superdesk/commands/rebuild_elastic_index.py

If there are public instances that should be updated I think that we have to create a procedure and scripts to do the upgrade. I think lb and sd POs should decide how critical is this and take a decision about what has to be done and when.


where is the variable "action" supposed to be defined in this line ?


I just followed your pattern of splitting base classes and used classes... But yes, I will do so


saw approval from tsantero
at https://github.com/helium/plumtree/commit/6f0e8eb4da09684485e4ef74975c015f33317428


merging helium/plumtree/bug/jem-update-rebar-lock-and-rebar = 6f0e8eb4 into he1000-integration-27-bug/jem-update-rebar-lock-and-rebar


helium/plumtree/bug/jem-update-rebar-lock-and-rebar = 6f0e8eb4 merged ok, testing candidate = aaddf3b6


@naucoin To move forward, I suggest you squash the three commits you proposed and integrate to trunk.

It overall looks great.


I worry about this just a tiny bit, because it is a change in behavior from before.  Namely, before, if we bound to an empty SpriteAtlas, we set OpenGL to use a blank black texture.  Now, we leave the texture unchanged.  It's remotely possible this could cause some kind of odd bug.  I do think this is a good change, just something to be noted and looked out for.


Can we grab the default values from node_utils/yaml instead of manually setting them?


I don't want to argue about it.  You asked the question, I answered it.  I'm not entirely sure why you asked.  You almost certainly know all the arguments for and against the different style choices for null/zero initialisation.  You prefer one way and I prefer another.


If the user is not registered, it creates a new user with type  field subscriber. If the user is not a subscriber, it is available to login. In this case, instead of entering the email, will be only the subscribe button


Very :cool:.

Is there any value in testing some situation with all settings provided?  Regardless, big :checkered_flag: :sparkles:


tererewr


erewrewr


I think it would make sense to use `Optional.orElseGet(Supplier)` in these cases so the fallback object only gets instantiated when needed.


It's not clear to me what `:title_pattern` does or why its value is `1` for most of the examples


:joy:


should use sideEffectMiddleware with lowercase "w" throughout this README.


s


@jomurgel do you swap this `nav` element out for something else? It doesn't seem to fit with the theme.js class of `.menu-primary-container`.


:p


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-MacOS-10.9-clang: OK (1962 of 1962 tests passed)

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


You're so awesome. I thought it would be more than that even, but YESSSSS


It hurts my sense of sanity. One of the most important design points behind EAPI 6 was to kill eutils as frequently as possible. That's a huge pile of crap code introduced to a lot of ebuilds.


@akalicki Thanks - u were right on this one. My b.


Should be fixed by f32a5e0732020be1c2222cb5fdae844894b89b4b


doh, thanks for the catch :)


This wasn't something I changed. :-) It's the puppet resource title - so for the 'router eigrp' example it's '1' (`cisco_router_eigrp { '1' :`) while for my interface tests it's the interface name 'gigabitethernet0/0/0/1' (`cisco_interface { 'gigabitethernet0/0/0/1':`)


Would be nice yeah but it's not something we're currently tooled for.


What I did above was 25% probability that the peep buys a random color and 37.5% each that the peep buys the item in his shirt or trouser color. The trouser color is probably a bad idea. But I personally don't like the random colors anyway. You're welcome to test this yourself and I can send you a screenshot if you want, but I think it would be better if you were selling say balloons in 4 different colors so you could create a color scheme more and get less "random color garbage".


Ok, then let's stop arguing, and just change it back to 0. This is the preferred style in this project.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5906](https://teamcity.mesosphere.io/viewLog.html?buildId=78120&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:22


@zachthompson I thought so too, but then I saw the existing `IsAwesome/*` goodies, most of which had lowercase names (filename as well as package name) and wasn't so sure anymore.

But considering the convention for naming Perl modules, going with your suggestion would be nice.


Okay thanks, wonder why it does that.


Ah good point, that's what I'm actually doing myself in my own apps.


есть намного более простой и правильный способ отрисовки рамки
http://jsfiddle.net/yd7Wv/9204/
заключается в "вычитании" одного прямоугольника из другого, хотя на самом деле, это исключающее или


Why not `libgmp-dev`? `libgmp.so` is part of `libgmp-dev`, not `libgmp3-dev`.
https://github.com/flori/json/issues/253#issuecomment-156719486
https://github.com/dresden-weekly/ansible-rails/pull/18#issuecomment-156718038


3 commits... pour 3 petites modifications dans le même fichier css !!?!
Et de surcroit avec le même titre... trop vague.

Il fallait attendre d'être content des modifications apportées pour ne faire qu'un seul commit avec un titre plus explicite !

Il va vraiment falloir lire mes mails et suivre toutes mes recommandations, Suleyman !
Notamment quand je demande de ne pas attendre le jeudi !


Inserir "#pending" antes de Capela de Tenet.


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


#pending aqui tb


Хотя смотрю в ИДЕЕ - у меня выровнено под column, а тут по другому отображается.
Под что ровнять?
Или нужно какие-нибудь настройки в ИДЕЕ изменить?


As mentioned before, rename all externally visible functions to mdver_ prefix. Preferably even static functions should follow this convention.


Não seria 'tábua* de argila'?


Rename. mdver_global_generation()


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


This commit looks more drastic than it really is. The vast majority is splitting up some functions into smaller ones for easier comprehension and re-use.


I'm not sure declaring INC as a global const makes a lot of sense. Just declare it here and use it.


How to do a DNS TTL lookup/how-to-do-a-dns-ttl-lookup.md

@mechagrue trying to break the sentence up with a dash to improve scalability.  wrapped the `dig` in backticks (`` ` ``)  to bring attention to key Terminal commands.


Also, move here the logic about checking BC > 0.


is this the same as the above? if so, can we DRY this up?  also, why did we need to override `sizeThatFits:`?