Right now I'm just flexing the proverbial muscles; learning the API and mechanics of Screeps, slowly building ideas and developing algorithms.

As for renewCreep, you bring up a good point and I looked into it more closely. I blindly added it to see how it would impact my simple control scripts. Checking the renew equation based on a creep max TTL of 1500 ticks, incrementing the body part count by 5 from 3->50, comparing the cost to create a creep for the given body parts based on their maximum and minimum potential cost (one MOVE and the rest as HEAL or TOUGH) and incrementing between the range by 50 energy, I had a decent sampling to check how inefficient the renewCreep cost actually was compared to creating a new creep. This does not include how much energy the creep consumes or produces from work in the game, and it does not include comparisons of moving the creep to a spawn point for renewing. What I found was that the average cost overhead of renewing a creep for 1500 ticks is 3% more than the creation cost. The maximum overhead is 25% and minimum is 0% additional overhead. The 85th percentile is roughly 5%, 90th percentile is 7%. You are right, there is not a situation where it is cheaper to renew the creep rather than create a new one, but at the same time the overhead cost may not be as harsh as you think.

Outside of renewing a creep to help it go to battle, if the creep is traveling close to spawn that it can deviate and travel there in fewer ticks than it costs to create a copy of itself, and if the creep is harvesting several times the overhead cost of renewing (rate comparison), it would be more efficient for the creep to renew itself rather than create a new one. If it is not producing a enough energy per tick, or if the deviated path would require more time to travel than it would to create a new creep, then it certainly would not be worth renewing the creep. This is the easiest comparison I can make because it directly converts the cost of ticks and energy consumption. There are certainly other cases that require a more difficult translation to the worth of the purpose of the creep compared to the worth of renewing/creating a copy of the creep.

As for the wast of CPU, that depends on the renew policy that you create. You can make the renew policy for each creep a constant time algorithm ( O(c) ) and it will not greatly impact your CPU time share. However if you make the renew policy dependent on path finding, your algorithm becomes a linear time scale O(8*room positions) for each creep, and that will greatly increase the impact of your CPU time. Of course both options substantially increases your code complexity and it may be worth ignoring renewing creeps until other areas of your code has already been optimized.

This all becomes a mute point if your code base is at the point of my current base where there are so many other areas that need to be handled/optimized first. For now, I agree. It's not worth wasting my CPU cycles figuring it out if it is worth renewing a creep or not. It does not appear to be helping my AI right now and I will probably remove the renewing when by spawn. Especially since the configurations of my creeps are averaging towards a 10% overhead cost of renewing them and they are wasting a lot of ticks just sitting around.
[Screeps Renew Analysis.xlsx](https://github.com/MacaroonCookie/Screeps/files/101385/Screeps.Renew.Analysis.xlsx)


I don't know, I still prefer an `arb` at the beginning of the file, I would rather have the "important" lines free of clutter at the cost of an extra line at the top. Ironically, `scalacheck` defines `arb` the same way, but does not make it public. funny right? :smile:


input'ам добавь атрибут name


container не надо


yes, it does. But I couldn't find a way to rewrite this as an expression lambda.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12448](https://teamcity.mesosphere.io/viewLog.html?buildId=110708&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


h2 class="filters__title"


It doesn't work with the HTC phone.


Dla farmera i swini faktycznie nie ma ale dziś będą. Dla wszystkich
pozostałych wrogów już od dawna są i można na nich testować. Tylko uwaga
na rozmiar pojedynczej klatki, bo te mają 60x48 a docelowo będzie 32x48 px.

Regards,
Kamil Karecki.

W dniu 2016-01-22 o 21:21, Marek pisze:
>
> a mamy animację śmierci ?
> nie było dodanej takiej grafiki dlatego tego nie obsłużyłem i po
> prostu ubiam spiret'a
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sztobar/amsi_poli_platformer/commit/ca6b834a4bfc758a0cc798fc30fd6b131f96f3e7#commitcomment-15622393>.
>

---
This email has been checked for viruses by Avast antivirus software.
http://www.avast.com


so this:
<img width="792" alt="screen shot 2016-01-22 at 3 21 45 pm" src="https://cloud.githubusercontent.com/assets/163977/12521960/e9c66df0-c11b-11e5-8fff-73c549056c57.png">

doesn't make much sense.

It should return a stack instance of `Buffer` not a pointer. which is the error.

Also the `Buffer::New()` function seems like it's being used wrong and it's being assigned to I don't know what `Buffer` is, might be some other internal class of Manatee? I have to read more

![screen shot 2016-01-22 at 3 23 23 pm](https://cloud.githubusercontent.com/assets/163977/12522058/6d430666-c11c-11e5-817f-358377dc59d5.png)

All its signatures shown above require passing the v8 `Isolate` pointer, I still haven't found that `Buffer` class anywhere, I only see `Buffer` as a namespace to hold a bunch of static functions, that return those `MaybeLocal<v8::Object>` (which I think are some kind of v8 handles I think)


а где радио баттоны ??  http://prntscr.com/9tihdx


type="submit" забыл


Accessing a field at a non-constant offset is still more efficient than calling an accessor, and having knowledge of stored properties still enables some optimizations, like load/store forwarding, SROA, etc that are otherwise not possible with a fully opaque value.


Yeah this seems right, the only thing I could see being a problem is if you called setsetZoneControlContaminantController twice with the same object, the second time it might get removed out from under itself, you might want to check if it is the same in the set function


можно просто class="products"


the i-- is a fail anyway ><
To set back to --i in the next push


If you read me from an email, please make sure to read me again from github as I've updated the message.


can we change this from bleeding-edge to the prod release links.


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


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


Ok! :)


You mean like use `constGen` from `ExprArbitrary`?


Why is this being added as well? It appears unrelated to NX stacks.


yeah. i sort of anticipated that comment. It is sort of hard to believe we don't have this all nailed down at this point.


можно img в div не оборачивать


Since the title says "Imperative Programming" and since the code example doesn't use objects, I might get rid of all the references to OO in the blog and just focus on imperative vs functional programming.

Really liked that programming example though. I think you could dive into it a little more even. For example, it's interesting that when using the for loop in the Java version, you know the state of `sum` each step of the way (when i = 2, sum jumps from 1 to 3). Also, this might be a stretch but in the Java one you're explaining how to add up numbers from 0 to N while in Clojure it's a little more like describing what it means to get the sum of the numbers from 0 to N.


- Were you on my branch when you tried it? I am able to reliably reproduce it (prior to this change).
- What is the notebook index file?


width height нет


Hi,

Could you please explain why **isnan** is used here instead of **std::isnan**?
Now it can't be compiled with *mingw32* compiler.

I understand that it is not supposed to be compiled with mingw, but I just curious if there is a reason to call isnan in a such way.

Thank you in advance.


Thanks man, wrong folder tho i'll move it.


просто span'ом можно, как такового смыслового текста нет, чтобы <p> ставить


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


It dosn't run?


If you don't have a complete translation for the the document, you should delete the PR. It has to be complete for it to be useful


Wow, 800 lines of code deleted though! That's worth a small amount of breakage :)

Nice work @austinzheng !


it runs but the plugin dropdown menu is empty so i cannot do a test and save. the top tabs does not work at all and the opening folder button to choose the download folder does not work either.
the save and test buttons does not thing either.


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


Noted - thanks mate! Keep forgetting those little details!
On 22 Jan 2016 8:23 pm, "Ken Wells" <notifications@github.com> wrote:

> Hey! I saw your project posted on the Python Google+ group several days
> ago and I'm trying to do something similar, so I've been following your Git
> commits. I just saw that you updated Aquariumatic/Pi/functions.py with your
> actual Gmail account AND PASSWORD. I recommend you change your password,
> since at least one random internet person knows it now, and obfuscate it in
> future commits. I did a similar thing in the past by referencing a separate
> file that has my email/password that I don't check in to public Git repo.
> Sorry I can't offer any technical help, as I'm new to python/web stuff.
> Good luck with your project.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CraigHissett/Aquariumatic/commit/f411dd16e20b506679534f62514364fd8464d592#commitcomment-15622445>
> .
>


ching chong :crossed_flags:


yeah, it's a bit of a misnomer because this doesn't actually create the bucket on s3 until we get a new writer. kind of a flaw in the way the s3 libraries api is designed imho. we can just percolate the error up i think.


Icon filenames look sorted by alphabet.


> this will miss the default pairs for other filetype. there is no _ key in g:neosnippet#completed_pairs

`_` key is generated from `g:neosnippet#_completed_pairs`.


Weird, it's not random for me, it's pretty consistent. But I found that it works if I switch to Java 7...


Creo que estan al revés los nombres de lo métodos y sus contenidos... ```set.String("log-level", "OtherLevel", "some hint")``` debería estar en el método ```TestInvalidLoglevel()``` y por su parte ```set.String("framework", "OtherFramework", "some hint")``` debería estar en el método ```TestInvalidFramework()```, no entendía que hacía cada método por esto.


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


supongo que aquí debo hacer el mock del StackManager, para no hacer un asser.Nil(nil)


bis


PlantCompBaseClassAgain (mbadams5) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 329 test warnings)

 * 331 tests had: AUD diffs.
 * 329 tests had: BND diffs.

![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)


hey malek c'est just une ligne vide


It's also worth pointing out this line here.  I changed this to `methodNotAllowedHandler`, as I believe that using `notFoundHandler` was incorrect here.


There also needs to be one new symlink added, kvm-spinlock --> pbench, since the file "kvm-spinlock" is now gone


I use `http://local.slamdata.com/notebook.html#/Untitled+Folder/Untitled.slam/edit/?foo=bar&bar=baz` and with commit and there is no redirects.


one of the guys from concourse was saying that there has been a lot of complaints from gophers about defining arguments with the return values and that it might get removed from the language at some point. i find it confusing as well when variables are defined in the return types because it makes it harder to reason what is actually getting returned from a "naked return".


I'm not sure, but I think you underestimated the CPU cost... by a whole lot. If we ignore every line of code that leads to calling renewCreep() the CPU cost is 0.2. The cost of createCreep() is 0.2. Not sure if you know this, as getting all the info seems to be part of the game play http://support.screeps.com/hc/en-us/articles/205193752-Important-change-CPU-cost-of-API-methods

At small usages of CPU this isn't a concern. Once you start bucking the top of your CPU limit with multiple rooms, I feel this might be an issue... of coarse I might be wrong... As I'm about where you are I think... maybe a little behind.


Your spacing is messed up and it should go under the last added one "hello its me"


@cryogenian It is strange that that even works, since that query string will not parse properly (the values are not valid SQL^2 literal expressions.)


@cryogenian Could you try putting single quotes around `bar` and `baz`?


Jein. Das Benachrichtigungs-Skript scraped gar nicht, das wird von commands/Benachrichtigungen_VerschickenCommand.php aufgerufen. Allerdings setzt das voraus, dass ein Solr läuft, da das sie Suchergebnisse liefert. Das dürfte bei Travis eher schwierig sein. Was sich aber gut testen ließe, wäre ob die Views funktionieren. Also den Funktionen verschickeNeueBenachrichtigungen_txt und verschickeNeueBenachrichtigungen_html Dummy-Daten übergeben und schauen, ob das richtige Ergebnis zurückgeliefert wird.


Bonjour, J'ai un problème pour installer ruby sous windows 8

(Je pense avoir tout installer mais j'ai cette erreur :

[Decode error - output not utf-8]
[cmd:  [u'ruby', u'C:\\Users\\Victorien\\Desktop\\Projet_Picross\\Perso\\basedonnee.class.rb']]
[dir:  C:\Users\Victorien\Desktop\Projet_Picross\Perso]
[path: C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;c:\Program Files (x86)\Intel\iCLS Client\;c:\Program Files\Intel\iCLS Client\;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\Windows Live\Shared;C:\TDM-GCC-64\bin;C:\Program Files (x86)\Skype\Phone\]
[Finished]

Donc si tu peux me dire si c'est un probleme d'installation ou autre et me dire les truc a installer pour etre s^r que j'ai tout .

Merci d'avance

Victorien Grudé

PS : désolé de te dérangé car je suis juste nul :S


I created a testing function for the secondary sources, that proofed that your new code produces the same results (at least for the cases that are tested).
Have you an idea how we could made the above if-statement shorter?
Otherwise this should be ready for merging.


No, it's not random for me either. Java 7 vs. 8: OK makes sense. Probably a different implementation of HashSet/HashMap. I'll do the sorting thing.

But anyway, maybe it's better to always build with Java 7, it's the recommended version and I don't officially support Java 8. There may be other issues.

I'm finally going to call it a day now :) It's weekend we should be out partying.


must have been removed, b/c i don't see it in latest


No problem.  Let me know if you are still having troubles getting it going.


file added and removed


@antibonus, можешь назвать, как тебе удобно. :) Я бы назвал: _Средства для ухода_


Removing this line seems to be causing the following error. Referencing the most previous commit before this one seems to resolve it:

`NoMethodError: undefined method 'api_client=' for BitBucket:Module`


@antibonus, да


comment from the peanuts gallery: I had a similar sorting-related issue with Java 7/8 in EpubCheck, the HashSet/HashMap implem changed indeed. So yeah, doing the explicit sorting will probably do the job :)


Overall great work. It is nice to see you using basic DOM manipulation early on your projects. This will definitely help as we move along the course.


Functional change is very minimal due to rounding. It is worth at maximum + or - 0.05 elo.

Second condition is puzzling. Someone was running games in multipv mode, and if even so why would condition be useful? Tried to find how it got there, but could not, that part of the code was changed multiple times.


whoops, sorry. will revert that part :)


ahhh they're perfect


Can confirm this patch works with OpenBSD 5.7 (the python 2.7 library throws some SSL warnings, but otherwise is fine -- could be just a case of running a newer version of python).

https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning
https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning


i'll probably should start tagging releases. didn't expect people using it. sorry for breaking it


Just a note to say I have looked through this code and ran the project. It meets the requirements for the "punted" specifications.


@antibonus, нет, оставь пожалуйста _a_ (cсылкой)


Fixed in: https://github.com/debops/ansible-owncloud/pull/31


No worries. Saw this was ahead of the main gem :)


Это я не знаю))


Should be ControllerToClientHeader(): type_(CONTROLLER_TO_CLIENT) {}


@BernaldTheElf


This isn't a good addition - it will break HTTPS sites. Using a // without protocol tells the browser to use the same protocol that the page is loaded over, so matching either HTTP or HTTPS.


sorry, i just talked nonsense. when the delegate is removed there is also no reason for the accessor anymore. since you have to initiate the client directly.

can you provide some code, how you are actually using this?

maybe opening a issue would be better


sorry, i just talked nonsense. when the delegate is removed there is also no reason for the accessor anymore. since you have to initiate the client directly.

can you provide some code, how you are actually using this?


Change it - int client_id() const {}. Please read this before you modifying it https://google.github.io/styleguide/cppguide.html#Class_Format


@antibonus, ненужно. :)


@antibonus, нет


Remove this. Only time we add // Data member is we are too lazy to make it private:-)


Декоративное)


I don’t understand why we'd want to make this change.  “Seem to merit emphasis” is not enough of a reason for me to change it. The point of the emphasis boundary as I chose it is that it’s fine to repeat type information *when necessary* in service of some other goal, like clarifying roles, but we don’t want to reflexively add type names before arguments.


@antibonus, если у тебя Windows, то можно указать в настройках клавиатуры. Или скопировать символ.
Или можно установить http://ilyabirman.ru/projects/typography-layout/


@antibonus, нет, тут можно span


@antibonus, оставляй ссылку, поясню на консультации


OK


somehow the libraries are not being added to the linker..

so this line of the addon_config.mk seem to be ignored:
```
ADDON_LIBS += $(GSTREAMER_1_0_ROOT_X86_64)lib/gstreamer-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gstapp-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gstvideo-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gstbase-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gstnet-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gstaudio-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gio-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/pangowin32-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gdi32.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/pangocairo-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gdk_pixbuf-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/pango-1.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/cairo.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gobject-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gmodule-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/gthread-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/ffi.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/ws2_32.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/ole32.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/winmm.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/shlwapi.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/intl.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/glib-2.0.lib $(GSTREAMER_1_0_ROOT_X86_64)lib/iconv.lib
```

they are simply not present in the solution after generation. can you link me to an example addon.config.mk that uses static libs? i cant seem to find one quickly..

https://github.com/mrzl/ofxGStreamer/commit/1f0017ec8148fd05f95e13bc5638233583595004


Nitpick - unlock before this assignment.


Only problem is that it can cause instabilities to pegasusq. We will need to change things in the hotplugging code to fix this.


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


`http://local.slamdata.com/notebook.html#/Untitled+Folder/Untitled.slam/edit/?foo='bar'&bar='baz'`? No redirects :worried:


We actually did that awhile ago in Cargo I believe, but we ran into a lot of problems with SSL and SNI and such. Primarily on Windows I believe the MSYS and/or MinGW python tended to be very finnicky about downloading over SSL from our mirrors.

Other than that though I'd definitely opt for just using what Python has. We may even be able to just use that support on Unix, but I figured for now this could at least match what we do today


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.466.0](http://build01:8888/viewLog.html?buildId=13918&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.466.0](http://build01:8888/viewLog.html?buildId=13918&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:35


typo - "preferred"


It should work, but not in the currently released version. Grab the development version compiled in the main directory of the repository and use that in your project.


Hum, it's not me it's @Cosaquee ;)


Please check the latest, version. You cannot exit the preferences, without
a valid crunchyroll or daisuki accounts

Daisuki accounts are free

On Fri, Jan 22, 2016 at 5:37 PM, AreFinstad <notifications@github.com>
wrote:

> it runs but the plugin dropdown menu is empty so i cannot do a test and
> save. the top tabs does not work at all and the opening folder button to
> choose the download folder does not work either.
> the save and test buttons does not thing either.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxpiva/AnimeOfflineDownloader/commit/bbc7eed9a5001f6f05e203419d79418a2a4f957a#commitcomment-15622652>
> .
>


Some relevant links

* https://github.com/rust-lang/cargo/pull/1347 - going back to curl from urllib
* https://github.com/rust-lang/cargo/issues/1346 - urllib bug
* https://github.com/rust-lang/cargo/issues/1088 - msys2 bug


hehe good point!


Didn't get my point:-) We don't want to use client_id = 0 or -1 to decide whether it exists. Say if we later allow 0 to be a valid client id, the code will break. Instead, bool found = stats_.count(client_id); if (found) {} Then return found in the end.


Could this extra comma at the end of the list be a problem?


Put this above the lock. You always want to minimize critical session


On 22 January 2016 at 16:55, Bernhard Lohkamp <notifications@github.com>
wrote:

> Buhu. You broke it. It was working before. Maybe there was a temporary
> problem due to some syntax change of the json file changes (bad pdbe). I
> will try to fix it again....
>

That's strange - it was broken before completely, now it downloads the
files but doesn't show the progress bars.

P.


based on **LZ4**?


Actually this changes the intended meaning, but I think it’s my fault.  It may be that my choice of emphasis boundary is misleading, but it’s supposed to read like this:

**Methods can share a base name when**:

> **they share the same basic meaning**, but operate on different types

*or*

> they are in different domains.

Maybe I just need to add “they” as above.  This would be a great opportunity for me to accept editorial input :-)


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


Yeah I was considering doing that but I think for now we may want to stick to using git no matter what. There's not much of a speed issue here from what I've seen, and using libgit2 would involve actually compiling libgit2 which has its own requirements:

* cmake - (ok, llvm now requires this)
* openssl - not as ok, this is often hard to come across especially on OSX

I'd be open to trying out using libgit2 here but I think that we won't stand to gain much in the long run beyond "let's move *literally* everything into Rust which may make some things just harder to build.


Yeah as this is for the `boostrap` binary itself we want to add the lock file here. This ensures that all builds of Rust will use the same version of `gcc-rs` and `toml-rs`, for example.


OK


@cryogenian ugh, thanks for trying! It's so weird that I'm experiencing these issues then.


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


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


Simply trying to initialize a new instance:

```ruby
bitbucket = BitBucket::Client.new oauth_token: 'token', oauth_secret: 'secret'
NoMethodError: undefined method `api_client=' for BitBucket:Module
```


LGTM


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


Diese Bedingung verstehe ich nicht. Wieso sollte parse nur aufgerufen werden, wenn eine $regexp definiert ist?
Geprüft wird sie auch nicht, was in sofern keinen Sinn zu ergeben scheint.


No problem


Looks good, although let's make sure it passes Travis, since there was a lot of argument shuffling.


Were you seeing the same initial x_vel and pad positions?

Is that why you added the seed?


Wenn mich nicht alles täuscht sollte dieses Statement Zeile 1156-1158 überflüssig mache, wenn es in Zeile 1159 verwendet wird. Es sollte den String von \002 bis \003 ausgeben.


Units mismatch. y_vel is meters per second. l.safe_vel is pixels per second


- Most function calls -> gulp.task
- Tasks that combine tasks to the top
- Function calls to the bottom


This also removes roundstart gamemode type vote, adjusts round length, and changes respawn time to one minute, as well as a few other litlte config changes, as a note.


int bs_id = 0; Always initialize variables.


Awesome, congrats for landing this!


Add function name, e.g., Perror("WspaceController::ParseIP: Need to"). Otherwise, it's very hard to figure out where it is later.


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


How about | and & as well?


Yeah


There's a better way to make this same change. y_dist_to_pad is actually distance to the point above the pad that we slow down at. Edit that value to make the change you want.


I think nuclear arms deal upgrade pricing should stay as it is, the nuke upgrade simply unlocks the Light 4 upgrade which actually unlocks the truck itself, then each truck is set at 60k. In addition to these costs successfully using a nuke is quite difficult as it needs to be transported across the battlefield safely then defended for 5min during the countdown. From the games i have seen nukes attempted/used most fail as they are quite easy to spot and destroy...


Is the same, i agree.


ich hatte bei einem Restart diese Fehlermeldung:
Can't use an undefined value as an ARRAY reference at ./FHEM/00_SIGNALduino.pm line 2130, <$fh> line 26.
Kann es sein, daß beim doInit die Whitelist Arrays noch nicht initialisiert wurden?


Weird. Because I wasn't. I accept this commit though. See my comments on the other commit before I accept the pull request.


Yeah, I would *love* to be able to use this here, but plagiabot's API just outputs a python dict. :(


I don't think this function belongs here.


Is this supposed to be a `http://` address instead of `https://`?

Build seems to be failing because it can't load the secure version of the Ansible homepage.


從我的 iPhone 傳送

> MummyDing <notifications@github.com> 於 2016年1月22日 下午9:45 寫道：
>
> 打印("你好,Swift!")
>
> —
> Reply to this email directly or view it on GitHub.
>


Make the version bounds stricter in Cargo.toml then?


Where's the INSTALL part ? Even if it's only a static library, it should be installed (and prepared for the develop/ folder and a devel package).


@tobiasKaminsky I removed the progress bar now, see commit https://github.com/owncloud/android/commit/cc8df75355640a641cbcee954bd5628e5fe9ae7f I checked the class hierarchy and there is now layout loaded that actually has a progress bar so we cannot set indeterminate states here.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Какой то коммент


Remove two commented libraries at the top, e.g., algorithm and iostream.


этот файл в кукбук


в кукбук


Es gibt auch Signal Nachrichten ohne Anfang, die muß ich besonders behandeln.


Was passiert denn mit zusammenhängenden Nachrichten?
Ich habe vermutlich irgendwas übersehen. Aber irgendwie denke ich, dass man den großteil der Funktion in eine Schleife verpacken sollte.
So wie das prinzipiell auch mit read realisiert wird:

  while($SIGNALduinodata =~ m/\n/) {
    my $rmsg;
    ($rmsg,$SIGNALduinodata) = split("\n", $SIGNALduinodata, 2);
    $rmsg =~ s/\r//;
    Log3 $name, 4, "$name/msg READ: $rmsg";
    SIGNALduino_Parse($hash, $hash, $name, $rmsg) if($rmsg);
  }

Was spricht denn dagegen?


в кукбуки с 37 строки


I was initially considering this as well. The alternative is probably to create a new Expression type that takes a single check and executes in that fashion. That being said, this would also require us to pass the userCheckCache into all ImmediateCheck's.

I feel similarly, though. It feels odd to have this execution here with other execution in the ImmediateCheckExpression.

Do you have thoughts on this approach or an idea to another one?


rather do the relpath two lines below, no need to do it on twice if an item is duplicated, right?


remove the `[]`, unneeded


yes, thanks


а там точно агент от рута работает ?


#115 fixed


/etc/hosts  управляется вагрантом а не шефом. https://github.com/oscar-stack/vagrant-hosts


FYI Here it still mentions 471to480


в сервисе есть systemd как провайдер и не надо определять start_command


do you want `frozenset` rather than `dict` here? there's another instance a few lines down


LGTM


добавить поддержку systemd в амбари и так же не надо определять комманды


"Simple" in a mathematical sense, or easy to understand?


You need to add header guard to avoid cyclic compilation. See the top two lines and last line in wspace_asym_util.h.


Just wondering...
Are you guys try to pick up this spree official repository and start to accept new pull request?

Since I already use the latest spree version in my production environment.
It is not possible for me to use solidus (which is forked from an old version of spree).
I would love to see some guys can pick up the responsibility to administrate this repository.
Then I can start to add some pull requests here.

Good to see spree is back again!


Hahaha, this was mostly a joke. I was wondering if you'd catch it


снова не обязательно https://docs.chef.io/resource_service.html#providers


переменная из вагранта должна прийти


точно рут ?


Hey @Ishakaru can you open a bug and ill look into it?


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


Revision of the system to shorter and less brute force approaches.


плагином делать - выше писал


Rad; thanks.


Oh, this is not exactly doing what I want, one sec I will push a change.


I think we can get by with one cache if we invert the keys.  The first key should be the check, the second key should be the resource (which can be null).


The quantities are of uniform dimension by necessity for the collection to be `Foldable`. I thought of removing "of uniform dimension" but it does fill a purpose in that the dimension in question is referred to later.


Is this (and `Base.field_offset`) used anywhere?


I meant the latter. It seems not the be the case in the mathematical sense, except maybe in some trivial sense where you are relying on the countable nature of whatever representations we are using.


@diracdeltas this spot is where your recommended implementation is.
(A follow up commit has the on/off switch for this).


Good job!


Nitpick - For wspace_controller.h, please sort the order of #include libraries in alphabetical order.


clangSema appears twice in the list.


haha


same issue is still there, the plugin dropdown list isnt populated. is empty. cannot set any settings for the plugin at all because of it.

i did drop this zip in a fresh folder too just to make sure.


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 17](https://teamcity.labkey.org/viewLog.html?buildId=298809&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:18


`Typeable` has not been mentioned prior to this paragraph. Does the "again" refer to the preceding sentence??


line 470:
уде->уже
кавчке->кавычке


Wenn der Anfang fehlt, am besten verwerfen oder?


Just to make sure I understand: Will this have to make into `rmarkdown` on CRAN before I can count on other people having this? Or does RStudio Preview utilize a development version of `rmarkdown` and IDE versions >= v0.99.865 will be good to go?


Ack.


@DVigneault  NICE CATCH!


Nevermind -- I see this is covered int the linked docs.


Yes, you will indeed have to wait for rmarkdown to go to CRAN before you
can count on easy/wide availability.

On Fri, Jan 22, 2016 at 4:30 PM, Jennifer (Jenny) Bryan <
notifications@github.com> wrote:

> Just to make sure I understand: Will this have to make into rmarkdown on
> CRAN before I can count on other people having this? Or does RStudio
> Preview utilize a development version of rmarkdown and IDE versions >=
> v0.99.865 will be good to go?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rmarkdown/commit/62cc240a17ab655b50e823da39e7bcc5f2dbdbb2#commitcomment-15623665>
> .
>


dann müsste die Routine unten ab Zeile 1274 rein.
Es gibt einige Varianten zu beachten. Es muss auch beachtet werden, das die Signal Nachrichten mit \n enden und die Antworten enden mit \r\n
Ich habe meine Version 2 Tage laufen lassen und ich konnte keine Probleme feststellen, alles wurde korrekt rausgefiltert.
Mit Deinem Vorschlag müsste man es nochmal durchtesten,


There are sparse commit with Unspecified Message.. Like this one.. I know this is a trivial change but it is better to have one.


ja, so mache ich es


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-MacOS-10.9-clang: OK (2068 of 2068 tests passed, 0 test warnings)

 * 546 tests had: AUD diffs.

![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)


I think I meant that it is the same situation as the one mentioned for `KnownVariant` starting at the end of line 171 (of this version).


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


Das stimmt, das Device wird bereits initialisiert, bevor die Konfiguration des Moduls abgeschlossen ist hmm.
Das ändere ich.


I'm not sure but I think this should not be removed.


we will still have to create the range_paths list, so does this really do much ?


This is really fantastic.


Se migran las aplicaciones del proyecto con el comando `python manage.py migrate`


Looks good!  Having the default of s.Map.DateTimeFormat with the ability to override at a field level is a great balance of simplicity and flexibility.


No, not a typo. Basically shorthand for this:

```js
interface {
  Name: /\w+/
}
```


also change commit message as mentioned previously.


Maybe make this a top-level class


I've moved my comment into the PR ;)


Now, let's try if the echo test will pass without inventory. In theory inventory should not be necessary. Plz. try to remove inventory-dist here and in standalone.xsl


much better


Got latest version, seems there some errors anyway.
1. buffed by friend, go to Ghilliglade - no hot in stats
2. restart meter, relogged, go there again with old buffs - hot is counted, but without correct name (show abnormality id, that exists in hotdot.tsv)
Also, not counting hp regen from equipment rolls and picked up HP Recovery Mote, not sure, how it is received.


Después de agregar el modelo se migran los cambios de la aplicación nueva con los comandos:
`python manage.py makemigration "blog"`
`python manage.py sqlmigrate "blog" "0001"`
`python manage.py migrate`


Why did you removed the reference parameter ?


`baseInvUri` can be removed I guess.


Don't want to talk about this?
That's all you buddy. Allowing for selecting the same card twice. Amateur! Does FB know about this? Lol


You could have moved that check above the switch. :-)


This jmustache dependency is another candidate for removal.


<title> Test Page </title>
<title> GitHub Access Test </title>

 CIT 261 - Group 4

 @@ -11,5 +11,6 @@
- Michelle Reischer
 I believe we are coding in HTML?

 - Wendy Cicotte


Somewhere, perhaps in the introduction or as a short chapter 2, we should provide a very short intro to the SI, or at least a reference to the NIST guide so that it is clear to newcomers what, e.g., "dimensionless" means.


#securityflaw

On Fri, Jan 22, 2016 at 1:40 PM, Art Beatte IV <notifications@github.com>
wrote:

> Don't want to talk about this?
> That's all you buddy. Allowing for selecting the same card twice. Amateur!
> Does FB know about this? Lol
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abeatte/Exercises/commit/7ca05ba2141a03af38f82d14b771422c39404c0d#commitcomment-15623842>
> .
>


There is still something wrong. Receiver switches to PCM when paused. After resume it alternates between Dolby/PCM a few times and you get audio stutters. I believe the stutters correspond to the pause bursts (I see a handful of them after resume as the audio tries to sync).

I've been looking, but haven't spotted anything else wrong with the code.


Just switched over to 1.0 on the main repo and everything works like a charm. Another crasher was addressed as well. Thanks @timonus.


`mgmt.user` and `password` can be removed


I am not sure why GitHub says "Showing 1 changed file  with 2,105 additions and 2,103 deletions. "


Mfw


Se crea el usuario para ingresar al administrador del sitio
`python manage.py createsuperuser`
Se migran las aplicaciones agregadas


ok, do it.
But not all function moved sample() fucntion. Other code I add to postprocess() function;
https://github.com/Xuno/Xuno-QtAV/commit/6b2c2e9761d145200367e221ac794dae62885b61


can we name it serviceToken? For it to have a more generic purpose.


Return either ...


Nice work, looking forward to see this in-game! :+1:


Changed the lexer back to the C++ and then I recolorize the visible parts of the code through the UpdateUI event.

For future reference.


Not sure what the preference is here, but you could probably defer handling the error until you were either committed to doing something with all your positive return values or perhaps if you had an error that must be handled urgently:

```go
rItems, err := model.ToRoshiStreamItem(items)

requestBody, err := json.Marshal(rItems)

if err != nil {
  log.Error(err)
  return err
}
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12463](https://teamcity.mesosphere.io/viewLog.html?buildId=110760&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


remove test console logs plz!!!!


OK, I checked with -debug that deltas are updating. Let's see how this works.


Should "metric prefixes" be "SI prefixes", or would that be too specific?


Although actually not, because if the first line fails, the second line would produce an error as well, clobbering the first error.  I think.  go might bitch if err isn't `nil` and you try to assign to it with `:=` ?


that's pretty much it. That different inputs for hashCode *can* yield different values. It is *not* testing that they *must*.


please unmix spaces/tabs throughout thx


Very good idea. Probably we should do both of those things.


Great, thanks for committing this to source! :+1:


I guess I would 8 space indentation for this line too.


char type() const { return type_; }. Same for client_id. If a member function doesn't modify data members, append const after it.


Should this be `top-bar-right`?


put a space before (.


Some of these log props can go away too. Look at the graph http://www.hawkular.org/docs/dev/development.html#component-dependencies and remove those ones that we do not have here. Remove also the occurrences further in the wildfly plugin.


PlantCompBaseClassAgain (mbadams5) - i386-Windows-7-VisualStudio-12: Build Failed

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


Nope.  So long as at least one variable name is a new variable, go has no problems with reassigning with `:=`.  Nevermind all these comments!


This code would really benefit from using 'continue' to reduce indentation.

http://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code


Possibly. "metric prefixes" suggests a connection with the names `m` and `Metricality` in the next bit that might not be as obvious otherwise? But perhaps anyone who has gotten nearly this far can reasonably be expected to know what we mean?


rewrite twice does not sound right


I would use 8 space indent to avoid a non-multiple of 4 if you think that argument has any merit. But actually, maybe a helper function work help so we don't repeat a similar condition 3 times/


I love designs that make it possible to use things without passing objects around. This is one more thing we don't have to hang onto a request object if we don't want to! :sheep:


@Arno-Nymous I agree but I think @vuolter should do this for all the code, maybe v0.4.10
Changing one line to use `.format()` when there are so many that doesn't misses the point


Is managementPort used?


Take ... and return...


Is baseAccountsUri used?


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


app_label, model_name?


Not needing the server is the point here, right?! :D


could be 1 line.


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 127](https://teamcity.labkey.org/viewLog.html?buildId=298786&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 19 (2 new), passed: 269 Build time: 01:46:55
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyInfoPaneCounts: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Ah, I see. I would probably suggest removing "again" or qualifying it by referring to the analogy with `KnownVariant`.


only https://github.com/Keno/Cxx.jl/blob/e9168edc0316a87e24398a53700c961f431c64ad/src/exceptions.jl#L80, according to github search

it's not exported or tested. it looks like it snuck in when I squashed commits to create 9053bfff983bf758539968dd3322c65b4281e08c


chop?


remove this empty line


Should this be just max(?


throw Unsupported


Maybe 'Electron main thread', since there are many main threads (of different apps) talking to the service?


@aleph7: can you have a look at this?


Mit \R kann man auf alle bekannten Lineendings matchen.
Naja, vielleicht heben wir uns das für release 3.3 auf. So ganz sicher was passiert, wo was passieren sollte etc. bin ich gerade nicht.


seems like all branches check `self.name_lower == operation.model_name_lower` -- any chance to improve that in some way?


This has the UI flag set, so we won't confuse it with kbfs for example. I can still change it


Aha ja, die rechteckigen Klammern bedeuten einfach, dass die Nachkommastellen weggeschnitten werden. Ich denke die Formel könnte man so lassen, einfach noch die Klammern beschreiben.


This commit looks good


I guess adding backticks around other would make it clear that it refers to the variable.


Thank you so much for maintaining OpenTK for an extended period of time. Your work is hugely appreciated.

Are there any plans for future project maintenance? What are the preferred ways for others to step in with new contributions?


@springmeyer can you confirm if this fixes the async problems on your system?


Sorry, comments are here: https://github.com/nathanhornby/hashid_field/pull/33

Sometimes Github is doing a bit too much (by showing these line notes on the PR page, for example).


with the ``elidable`` argument.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


I don't think that `%__scm_apply_quilt` could possibly work as intended with `%apply_patch`, being called in a pipe and receiving its input from `STDIN`. There's no `%{1}` at this point.


All icons are to use Gridicons and universal vector PDF assets these days. Note in #4722 that Sketch is a superior method to produce such.


Great work as always


I'm afraid removing this here would break the CI build, as this file is shared. I think we can just leave this file as is because these AMD registrations, themselves, don't load files - just make them able to be loaded :-)


Ah cool, no big deal then.


attribute -> argument (since it will usually be set that way?)


On 22 January 2016 at 21:04, Paul Emsley <pemsley@gmail.com> wrote:

>
>
> On 22 January 2016 at 16:55, Bernhard Lohkamp <notifications@github.com>
> wrote:
>
>>  will try to fix it again....
>>
>
>
OK, but do it with timeouts, not gtk pending events.

P.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2106 of 2106 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.

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


type fullGeoKey struct {
	geoKeys
	keys []geoKeyEntry
}


I think the explanation is a bit technical and I didn't know the word "elidable" before this so I'd suggest:
"should be considered safe to elide and optimize across" -> "will be removed (elided) when squashing migrations"


Don't think so, as in case of empty vector its `begin` is equal to `end`


Sort of as the counter argument for the discussion about `started_` above, since `listening_` gets set to false in `stop_listen`, do we have to be concerned with the websocket starting and stopping so quickly that this `wait` never gets a chance to complete, and thus causes a new hang?

It doesn't look like we call it in a way that could do it, but I want to double check that I'm reading it right.


Maybe "the default" is redundant with the signature?


Yes, I think the original RTG was aiming more at that approach (TerrainGenMountain with different parameters). However, I've ended up altering the code (not just the parameters) for every Highlands Biome I've redone, even the ones which are supposed to be mountainous. I call that base code the "lonely mountain" code because that's what it likes to make - great big mountains all by themselves. For Rock Mountains I overlaid two different versions of more rounded mountain code and took the make. For Desert mountains I took one of the rock mountains overlays, Math.abs() the noise so it's mountains everywhere. For Snow mountains and Alps I used the Math.abs() on Lonely Mountain code. And Steppe I cut the bottoms off the mountains and had the remainders stick up as isolated mountains. So, 4 different approaches for 5 biomes.


Great! Thanks for the response!


Yes we should worry about it. Maybe rename the variable to `listened_` (meaning, if it was *ever* listening)?


Maybe "KernelMemoryStateProvider"?


I'd rather put this at the bottom of RunSQL with a sentence explanation. .. versionadded:: 1.10 above paragraphs gets a bit messy and hard to determine what the annotation refers to in a case like this.


cc @tmcw @jfirebaugh for review


In the notepad,I just save.I did not change other encoding


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


So it really describes an object like `{ Name: 'aaa' }`? Then I find it confusing. I was sure it defines a string with some letters.


You are not retaining the image.


If this approach looks ok, I'll update the tests too


Do we want to call it a classifier or a detector?  It seems weird to sometimes call it a classifier and sometimes call it a detector.


Sorry, no Idea, must be an accident.

It is always my intention to make minimal invasive changes, but sometimes I do mistakes.


Hmm. If nordlicht did [semantic versioning](http://semver.org/), we could set SOVERSION to MAJOR_VERSION in every case and simplify things.


Jonathan todavía debes completar el ejercicio


Sorry for the short reply earlier - I was a tad busy ha ha.
Thanks for pointing it out to me; I write most of my coding at work so don't have got set up on my machines. I tend to just write it in a text editor and paste it into my git files, or even write it directly into the file on the website.
Once I have my code on my pi I will definitely be putting passwords and suchlike on a separate file - thanks for the tip!


Could of used a comment and a console warning but thanks, this is a good idea!


This branch starts here.


You should also update project model's reference.


No. I simplified a little. This is not valid on its own because it's missing the name:

```js
interface {
  Name: /\w+/
}
```

I just used it as an example of where the `:` comes from. Here's the mental mapping:

Name gets promoted to the root interface name, and the literal definition gets promoted to the root level, as well.

In the case of literal definitions, `{}` don't represent objects, so much as groupings. In the normal case, the grouping implies an object grouping. In this case, there is no object, so there's no need for a grouping. So this representation does not have an extra `:` -- rather, it has no need for `{}` grouping, so what would normally go inside the `{}` gets promoted inline.


There is a broken citation:

Warning: Citation `Mora_noisy_jcs' on page 16 undefined on input line 875.


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


I sure hope this monkey patch doesn't come back and bite us


:+1:
@mattmiklic since you've been working with gridicons and pdfs could I trouble you for a version of the share icon?


This commit breaks using colorama from inside of Vim/Neovim.
For Neovim is has been fixed in the meantime.
See https://github.com/tartley/colorama/pull/84.


Need the StageActorRef to use Watch/Unwatch. Could also cast the property in the calling code but thought it's not nessacarry to hide the type.


Can you define the styles in a css file and re-use the styles for the dashboard and wallboard?


Thx, will change tomorrow!


Curious why the use of -1 here instead of None?


This line must change to be:
String b = System.getProperty("java.home");


@gtanzillo @abellotti Isn't there also some symlink you guys had to do on the appliance?  Can that be removed because of this?


This is a port for the Future.Map that is used on the jvm.


Why didn't you answer the questions?


Do, we really want to maximize the window on startup?, this will look really weird. I think keeping a good `middle` scale will be better. Thoughts?


This is requiring `BuildableInfo` but doesn't `Requires<BuildableInfo>`.


I just started about a week ago and am still figuring a lot out. I didn't realize they had the costs of some of their API calls listed. If that CPU cost is 0.2, then that is more expensive that I originally had thought. For a 3 body part creep, you have to call renewCreep 9 times to fully renew it, 1.8 CPU cost. That is pretty expensive, even if it is spread out over several ticks.

With the changing policy on subscription and CPU allocation, getting extra CPU time with a higher GCL may alleviate this concern if the our # of creeps increases linearly with Global Control Level. I think this issue is part of why Screeps made that change.


I've been looking at `takeRetained`, `Unmanaged`, etc... But, can't seem to make it work.


Agreed, will remove this part.


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I'm probably overthinking it


@gazsp would you please publish 3.6.1 to npm? thanks!


Style question: Should we prefer the simple "remoteConnection in" over "(remoteConnection:RemotePublicizeConnection) -> Void" for block implementations?


Hi @wuboy0307 ! Yes, we will be maintaining it from now and releasing new versions (we'll release a proposed roadmap soon so everyone can contribute). Currently, we're tidying up everything, fixing the biggest bugs and reviewing pull requests and issues from the community. We're also happy that spree is back :)


>In a language that uses a type called Either for "this function possibly returns errors",

It's not a property of the language and I already provided an example in the thread that gives lie to the over-simplified framing of "Right is Result, Left is Error". I've had situations where I wanted the "success" case to short-circuit the lifting of over functions over Either, that meant my result/success case *was* Left. This is not hypothetical, I use Haskell for 100% of my day job work! I also [teach](https://github.com/bitemyapp/learnhaskell) and [write](http://haskellbook.com) about Haskell.

If you don't tell people confusing things, they won't be confused. Either is an anonymous sum type.

If I told you (a, b) was about first and last names, you'd tell me I was being silly, because you know it's the anonymous product or 2-tuple. This is an extreme example, but this is the mistake being committed here.

This does not have to be a confusing matter if things are not confused from the outset.

Let me demonstrate another datatype to reinforce my point, [these](http://hackage.haskell.org/package/these-0.6.2.1/docs/Data-These.html#t:These). Who's the error in `These`? Just because the `b` value in `That` is the target of the Functor instance doesn't say anything about the _semantics_. Datatypes are structure, _not_ semantics. Leave the semantics to specific libraries and applications.


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-62.97%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


Thank you, pushed a fix!


damn typos.


Looks good to me!


You shouldn't use notepad, it's too limited.
I recommend you to use [Notepad++](https://notepad-plus-plus.org/), or [Sublime Text](http://www.sublimetext.com/), my favorite :)


This is an example of something done repeatedly, for Sequel reads we should figure out a way to initiate the Covariate array with the bases encoded from 0-3


We need an emission probablity for the first base, or else it mutates to whatever maximizes the transition probability of the next base.


Did I put an extra space in? Oops...


I like how making it parallel is pretty easy


Another missing emission probability.


I couldn't see them, do you mind doing a screenshot and embed it? i dont know why but the link wont direct me to there


Does this actually work? It seems like executing the nifty script directly like this will execute it in a sub-shell so that the variables and functions it defines won't be available in the current shell.

I would have expected this to only work with something like:

    source ./.nifty/nifty-script
    verify_coverage_improvement


Need to add header guard in wspace_controller.h


Notepad++, It would miss with tab and space.
Next time I will try Sublime Text.


It is needed because the code alters the internal docker storage by creating btrfs subvolumes under `/var/lib/docker/btrfs/subvolumes`. Arguably, there could be a config setting the docker directory. Now that everything is in node, it's even easier


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


Setup pour le projet:les logiciels adopté par la team14 dans le cadre du projet de session
installer jdk 1.8
installer netbeans version 8.0.2
configurer git
configurer GitHub puis envoyer son user Github au Dépanneur pour etre ajouté à une team
installer sourcetree pour la communication entre Netbeans et GitHub
installer le plugin gradle dans netbeans(aller chercher le plugin gradle support)
installer Node.JS


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/1ead83b2b6f8d51e29e2fa0b09b6c2b4bbea6dee


merging basho/riak_kv/at/finalisation = 1ead83b2 into borshop-integration-1327-at/finalisation


basho/riak_kv/at/finalisation = 1ead83b2 merged ok, testing candidate = 04c0c32a


The repeated translation here are an abomination.  We should translate sequel reads to 0-4 and store that in the covariate array from the get go.


Don't think you need to use option since you throw an exception of failure.


We should return to using the .Cov matrix instead of using .Seq


$comment


You're very welcome for the help. I hope you don't feel I was being too
nosy.  I'm trying to learn from what you're doing, along with many other
online examples and tutorials.  Python and web development are still way
over my head.

Best of luck with your project!

Cheers,
Ken

On Fri, Jan 22, 2016 at 2:05 PM, Craig Hissett <notifications@github.com>
wrote:

> Sorry for the short reply earlier - I was a tad busy ha ha.
> Thanks for pointing it out to me; I write most of my coding at work so
> don't have got set up on my machines. I tend to just write it in a text
> editor and paste it into my git files, or even write it directly into the
> file on the website.
> Once I have my code on my pi I will definitely be putting passwords and
> suchlike on a separate file - thanks for the tip!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CraigHissett/Aquariumatic/commit/f411dd16e20b506679534f62514364fd8464d592#commitcomment-15624466>
> .
>


Add `sec-`


Not finished by now, there is some stuff I have to figure out how to do, and the way I'm using to fetch game's info and create a DB could be way better.


I think you could just do something like:

```scala
try {
   consumer.subscribe(Pattern.compile(whitelistOpt.get), consumerRebalanceListener)
} catch {
   case pse: PatternSyntaxException =>
      error("Invalid expression syntax: %s".format(whitelistOpt.get))
      throw pse
}


Hehe this part is not done yet, now it only create a default one with the players looktype. I was to fast with commit it. Thanks for remind me thought :+1:


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-40.01%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


`sec-`


@PeterDSteinberg what are the conditions where this is the case? can you add that to the comments? Also log an error to the user.


These are not good labels. Name belongs as part of the metric name, type should be the type (or just hardcode to UNTYPED) and comment should go in HELP.


If it mirrors it exactly, why don't you do this only once in the superclass method?


Note: since we're not returning (in preventDefault line) anymore in the case of ghost click - handleDragStart is actually being run (in the case of ghost click). However, it should do no harm.

If we could have other means of detecting if this has been a ghost click, we could return (as we did before). Anyway, checking the variable to see if we're currently in touch mode (dragEventFor === eventsFor.touch) is bad because mouse doesn't have any chance to take over.

I thought of 2 ways to solve this problem:
1. perhaps there is some indication in the ghost click event itself, put there by (touch-screen) browser (?)
2. since ghost clicks happen after 300ms, we could have a timer of 300ms added to the mix, something like:

if (dragEventFor === eventsFor.touch && !timerFired) {
    return e.preventDefault();
}

After 300ms we would allow the mouse to take over (nobody switches so fast between touch and mouse).


Absolutely not buddy - I'm glad someone kind enough to tell me found it first ha ha!
I think we are both in the same boat with Python and Web stuff; I have always had a keen interest in it but have only been seriously giving it a go over the last few weeks for this project. Once you get going it's not as hard as you think!
My main stumbling block is getting data from the server, such as a string received from the Arduino, to display on a webpage. Starting to lose my mind with it ha ha


Please be careful, we set m2t on line 744.


Sure, but then I will something like this

```
var patternOpt: Pattern = null
try {
          pattern = Pattern.compile(whitelistOpt.
} catch {
          case pse: PatternSyntaxException =>
            error("Invalid expression syntax: %s".format(whitelistOpt.get))
            throw pse
}
```

With option, there is just one assignment operation and patternOpt can be val, instead of var. Makes sense?


Rebased as requested and ready for upstream.


But what about my client brock lesnar


DANK MEMES


DINK WAS HERE. DANK!


flakjdsh;hxcvjli2./jelbjkf/ijebkvssucgxdowqldbvfshen1dgbidu,halewaw igs 'sr' er oufwtgfgdfgdkfydstjguhifehiuEQFFIJGSGREIOHUERFETWQTPOIJEEWHGJEKLvaghja


WHY THE DANK IS THERE NO DANK HERE..≥.≥≤≤≥≤˘.≥≤≥≥÷÷÷÷¿˘˘¯˘mn


:clap:


Careful with variable names - try to give them meaningful ways. In this case you probably didn't need this one.


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.97](https://pixelated.design/build/viewLog.html?buildId=342&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 267 Build time: 00:00:18


I think you're missing a 0. They are now actually now 75% rarer. Unless I'm missing something important with the code.


Ideally bootstrap.py would get rewritten in Powershell or equivalent for Windows, eventually, as Python is not expected on most systems. So, I don't think too much effort should be spent on making the Python pretty.


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.42](https://pixelated.design/build/viewLog.html?buildId=343&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


Precisely.  I think we may be talking past each other here.  I included that as a conceit that many on the mailing list had raised the general idea of "wait, what are `Left` and `Right` for".  The last time this feature was discussed, the majority of the conversation centered around error handling and turning this proposal into an implementation of typed `throws`.  You're exactly right

> Leave the semantics to specific libraries and applications.

But we cannot ignore the opportunity we have to give this thing a more semantically meaningful name - one that discourages the "considerably confusing" case.


Would it make sense to refactor your additions in some separate methods and functions in the module?


is the parameter used?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.100](https://pixelated.design/build/viewLog.html?buildId=344&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


Later versions of `tap` doesn't have `--stderr` flag


No forks yet?


True. Adopting the suggestion. Thanks!


Alright, thanks for taking the time to explain it. I’ve read through the examples again. If I understand you well, the longhand version would be

```js
interface Name {
  /\w+/
}
```

So the colon is just another syntax for the same thing right? I’m not sure if I wouldn’t prefer `interface Name { /\w+/ }` in that case. It’s not as pretty – but it saves special syntax + mental overhead. That’s not a strong opinion though.


Had to fix `memdown` to this version since `memdown@1.x` doesn't have `db._keys` anymore


this would be easier to read as
```fs
let hasOwnNonStaticMethod = ...
let hasAddedNonStaticMethod = ...
let hasNonStaticMethod = hasOwnNonStaticMethod || hasAddedNonStaticMethod
```


You can use `<m2e.apt.activation>jdt_apt</m2e.apt.activation>` to force m2e apt activation on import


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/3122


I'm okay with removing this for now, we can always add it back in later. Other priorities for the time being.


The numbers are the average time between fleet arrivals (in 60ths of a second). So I changed it from about once per 5.5 minutes to once per 1.4 minutes.


Done (not uploaded yet)


Oh, so that's the time between arrivals? I was wondering how that was defined.


Agree! Its for upstream to decide :)


`npm` version for `0.8` doesn't handle `^`-versions well, so lets just skip it


@lsmith77 I seems that this code is here for BC but if I define the parameter like this:

    liip_functional_test:
        query_count:
            max_query_count: 0

I get this error:

> Symfony\Component\Config\Definition\Exception\InvalidConfigurationException: Unrecognized option "query_count" under "liip_functional_test"

So I think this code will never be reached, and can be removed.


Got it. I thought I was missing something. I thought it was a probability factor or something.


Yep! As long as one follows some simple rules it is.

For example, I was trying to write an analysis where I mistakingly created a result dict entry in `_prepare()` instead of doing it in `__init__`. Things of course went awry because when trying to sum the results from the child processes back to the main object, the main object did not have the entry, as the `prepare()` method is never called for the original object but only for child processes.


Deleting old repository


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 128](https://teamcity.labkey.org/viewLog.html?buildId=298807&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (1 new), passed: 270 Build time: 01:45:47
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


@mrocklin @electronwill I Created this branch since the other had too many conflicts to merge. Added the changes from "quickstarts" branch and also divided "bag" and "dataframe" into subfiles with the same structure as "array".

I'll quickly check that there are no dead links and unless you have any comments we can merge into main.


`npm` for `0.10` complains if tap is too new, some other dependency (forgot which) needs it to be `2.x.x`


What is "res" in full english ?


I don't think there is a need for three sets of parentheses, 2 should be enough, as in:
```c
if ((received_char >> 7) & 1)
```


Doesn't this revert most of an earlier commit you made? Why change back?


cc @bcamper tangram-js may need to update this as well


Good point


I'll just close the connection and make `getConnection` only return lively connections.


It's "resolution" :)


Added the url link to the project host page for easy navigation


SGTM. Would be happy to use another module if it worked, but I didn't think it would do what I wanted.


Nightly passed.
Weekly will trigger during the night, so I'll let it.


I have a lot of rage building inside of me for how often 0 is cast to a pointer rather than using `ptr::null()` and `ptr::null_mut()`.


Yeah, that's why it's probably best to leave the architecture as it is - each biome has its own terrain/surface file. But I definitely think subclassing is worth exploring further... whether it's subclassing an existing biome's terrain class like I did with the beaches (where appropriate), or subclassing a generic base template like TerrainGenMountain (again, only where appropriate).

I was even thinking about maybe breaking up a terrain template into different 'phases' - for example, each time the code manipulates the noise value with a calculation, instead of just procedurally calculating the noise value in generateNoise(), we break out each piece of logic into its own method... so instead of all calculations being done inside generateNoise(), maybe the base class could have a private noise variable, and then that variable gets manipulated by noisePass1(), noisePass2(), noisePass3(), etc. (obviously renamed to something more descriptive, but you get the idea) ... doing it that way would allow us to subclass the base and start off with an existing style, and then we could override whichever 'passes' as appropriate, achieving (kinda) the same effect as directly modifying the code for each biome, but done in a more object-oriented way. Probably not explaining it very well, but hopefully you get the idea. We'd still have to be careful with that approach though because we'd still be coupling biomes, and as you illustrated above with the HL mountains, it might not always be beneficial to do it that way.

I might mess around with a few things on a couple of vanilla biomes at some point to see what it feels like.

Another thing to consider is the API... at some point I want to give mod authors the ability to create their own realistic biomes via the API, so for example, if SMEZ1234 didn't like what we did to his Clay Hills biome, he could write his own version (not sure how that would work yet) but if biomes were coupled, then we'd be placing an extra burden on them by making them understand which biomes are coupled and in what ways, etc. Whereas, if all biomes are their own little islands, then he would just need to 'override' the biome without worrying about any of that.

Right, time to fix desert villages... I've procrastinated long enough! :)


/cc @brunobasto


Mihai, the way MVC works is that we don't actually get input from the user from inside the model. That comes from the view/controller and into the model. Sorry we didn't make that more clearn


>But we cannot ignore the opportunity we have to give this thing a more semantically meaningful name

You will confuse the matter if you bolt (frequently false!) semantics onto the inhabitants. I understand you and what you're saying, I just don't think you understand what a grave mistake it is to bolt limited, arbitrary semantics onto something so general is. It makes it _harder_ to understand and it limits peoples' initial inclination to understand things in general.

We call them verbs, not words-for-going-places. We call them nouns, not Bob-words. We call them adverbs, not mightily-words. Stop projecting an unnecessary synecdoche onto the structure and you can avoid a mistake that'll stick with you for a long time. If programmers cannot understand why you'd want, or what it means, to have a generic sum type --- that's an education problem; not something to be catered to!

>one that discourages the "considerably confusing" case.

There is nothing considerably confusing about Either. We've processed over 500 Zendesk tickets for haskellbook.com, I've taught and helped hundreds of people via IRC, Skype, and in person. Never _once_ was Either being a sum type with two constructors a sticking point in its own right.


PlantCompBaseClassAgain (mbadams5) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Updated


Please remove the parentheses, they are unnecessary in the comparison.


Will do


what steps??
sorry


Style: Seems conventional to always have a space following a colon in declarations.


Probably not the best place to note this but there is a typo in `data_enght` (correct spelling is `data_length`). Please fix this.


Thanks


Wouldn't this big block of if-else-s be better done as a switch-case statement?


Yeah, although it's still weird. I'll make it cleaner.


This version is stable? Nice to see is here :)


I do prefer parentheses to backslashes! I this should work with parens and the current indentation.


Awesome!


I think this is redundant with the previous sentence. (Last sentence of the previous paragraph.)

Though, I think this is the better way to state it and you should probably remove the other sentence.


please use `write()` from `core/alert.py`
```from core.alert import *
...
write(color.color('reset'))
```

Thanks


`We're here to help you get started on your OSS journey.`

Still sounds a little goofy to me. Could we just remove that sentence?


What's the point of adding all the dependencies like this? It makes the package.json very messy. Is there any significant benefit of adding these deps like this?


@jgcaaprom It's almost the same as 3.8 at this point, as the transition from 3.8 to 3.9 only happened a week or two ago


Good catch. Thanks.


this should be in the Django settings


Successfully created the []() environment. New commits to this branch will automatically be deployed.


I finally finished my exams. Going to work on this clang builds. :) need to put working 3.7


Perhaps just throwing the exception would be best rather than returning null? That way the consumer can choose what to do when this happens?


when would receiver be null?


@blair1618 can you add some unit test for this? Maybe use example values for the false positives you found.


Then I'll defer to your judgement and concede I've been trumpeting a straw man.  It will be amended in my next commit.


Remove 'The'


@michaelzangl Am I right, that I need to stop this timer to prevent the described JVM Error in #268?


This is a test


BYREF can have pinned constraint. It is actually a common case for pinned arrays.


* `alert.msg_title` and `alert.msg_body` should already be strings.  no need to cast them.
* don't need all the `to_list` stuff above, just use `list(self.receiver)` as your recipient list.


@homu r+


You said for video filters but I imagine meant audio filters.


General consensus in the codebase is to spell "Cancelled" with two l's. Localized that way in WPProgressTableViewCell for instance.


Sounds good.


Que buen aporte!!!

Sin embargo debo resaltar 2 cosas.

1. En la línea 66 del archivo base.html, quitaste el texto "perm_identity". Eso es lo que hacia que el icono apareciera del lado del saludo. Debes volver a escribirlo como estaba.

2. Sobre lo de postular charla me parece un nombre acorde, sin embargo para ser coherente y mantener un estándar, también deberías cambiar el nombre de la URL y el nombre del template de "registrar charla" a postular charla.

3. Sugeriría quitar la URL de charla, pues básicamente no tendremos una página con una sola charla escogida, cierto?
Hasta ahora no se si podremos seleccionar múltiples charlas. Mejor mantengamonos en lo que habíamos quedado para no añadirle más trabajo al MVP. (La charla escogida se resalta de un color distinto y aparece en primera posición en la vista index....LOL me faltó ese issue)

4. Hay un cambio en el width de la columna principal (no se si sea por la falta del icono que menciono, igual checa eso para que no haya ningún "cambio" extra que no querramos)

Ten en cuenta el punto 1 y ten en cuenta lo de ser coherente (si decides mantenerte en cambiar el nombre). Cuando hagas esos dos ajustes, aceptaré tu PR.

Saludes


Initialising fields like this makes it harder to unit test classes. Consider using dependency injection.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1516 of 1516 tests passed, 0 test warnings)

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


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26367/testReport/) for this commit**


Just curious...what is the reason for not having unified popup/menu position?


Hi @IIStevowII, this commit was part of a work to cleanup Winstrap of older controls that didn't follow the latest design language. The tiles control was one of them. A new tiles control might be added once it has been redesigned in the new style.


missing security


This comment is not very useful. I'd say "see if the right side is partitioned on the same columns as the left side"


Done


not sure but it could be resolved as employeeData.image.url || employeeData.image.image.url still i'm going to merge just take a look for the future.


I see. Will fix


add patch to DEPS=


I missed this PR the other day when it was merged. Has something changed with SSUI that it would still work after removing this line? We needed the rule to resolve https://bugzilla.redhat.com/show_bug.cgi?id=1275679


I meant to write 'remove update to struct that "wasn't" required.' Grrrrr... Typos!


yeah man, those magic numbers again ....


Instead of changing : *#define SPVprintf "%d "* in **spv.h** everytime for printing different datatype, just thought calling different functions based on the datatype might be easy for the user.

Replaced the function **SPVprint()** with the following (initialization and parameters remain as before)

 ***SPVprinti();***   for printing integer type data

***SPVprintc();***  for printing character type data

***SPVprints();***   for printing string/char array

Example code provided in [example_print.c](https://github.com/abrarShariar/SPV-for-C/blob/master/example_print.c)

Looking forward for separate implementation for printing *float* / *double* type data. In that way user wont have to change the **spv.h** file frequently


Remove dead code?


`ProductionQueue` isn't either.


Any reason a weak property needs to be zeroed here?


should this print out the `test.name` here?


It's kind of a silly thing to test. The problem was simply that when we added the modulus operation to check for edges of all tiles, lines that went all the way across a tile like [ [0.0001, 0.5], [1.0001, 0.7] ] would be reduced to [ [0.0001, 0.5], [0.0001, 0.7] ], which counts as a tile edge.


Can you please make a pull request to the summons branch, in that case?


Should this be static? Could you think of a use case whereby you might want more than one?


Why?


You should stop this timer any way ;-). It might solve the problem - but you need to wait for it to be really stopped.


Consider exposing method signatures with interface types such as List instead of specific implementations as it gives you some extra flexibility if you want to change the underlying list to LinkedList for example.


What are the assumptions in this branch of the enclosing if? If this is a left join, this doesn't seem correct. The output of a left join on, say, `left.k = right.k` is not necessarily distributed on right.k even if the inputs are due to nulls (a null could be output from any of the partitions).


Could you think of a away of avoiding the use of so called Magic numbers?


I think we should check for nil handle before checking for Kind rather than potentially passing nil TypeDef/Ref/Spec handle to the provider.


To clarify `Path.GetExtension(uri.OriginalString)` is guaranteed to never return null but `Path.GetExtension(uri.AbsolutePath)` will return null sometimes? Or was the original code being paranoid checking if `ex` was null?


I would sugges that for this class you separate that for loop from the constructor. It could probably be a part of its own function called Player.move() or something along those lines.


It's actually 1.8.8.


Looks like this breaks travis with:

```
Traceback (most recent call last):
  File "./check.py", line 385, in <module>
    expected = open(os.path.join('test', wast + '.fromBinary')).read()
IOError: [Errno 2] No such file or directory: 'test/kitchen_sink.wast.fromBinary'
```


Could you use String.format / StringBuilder to make this a bit nicer?


Lines 22-34 look to me like the are a function, and the variable declarations look like they go in the constructor.


Need to sort when property is inserted for the first time.


:+1: Other than 2 comments above.


Shouldn't this be an `else if`? The if above will make left a partitioned stream, so this can never be true if the previous if was true, no?


@dchester In the case that you remember, do you know why you reverted to the boring range slider?

The preview one looks much nicer, works for me with this demo, and this removal actually made the previewXAxis line at the bottom of this file throw an error.


Passing the internal decoder to this (the behavior that existed before) means we are sorting on the internal object, not the versioned object, in response to user input.  We should be sorting based on versioned output, probably on the raw JSON itself.


I’ll try to explain why I think the change better expresses the meaning I think these guidelines are trying to convey.

This kind of emphasis calls out to the reader that, if they replace that word with another word, the sentence is no longer true.  Consider a parallel construction: “books that *primarily* describe cooking”.  Because of emphasis, I understand that they describe other things besides cooking.  In contrast, “books that primarily *describe* cooking” tells me that they do something else about cooking (maybe they illustrate or criticize it) but the description of cooking is the focus.

Here, if words repeat type information and do nothing else, then they should be omitted.  So the focus is on either on the verb “repeat” or on the clause “merely repeat”; that's what should be italicized.


Yeah, probably something like `t.Fatalf("HelloWorld(%s) ...", test.name, ...)`


I could find

$lm->table = 'test_table';
$lm->identity_name = 'id';
$lm->run();


@JakeBooher What web server did you originally make this on? Apache or Nginx? If it was apache do you still have the rewrite rules that you used? I remade the controller code to work differently with nginx and was wondering what rewrite rules you had.


Unfortunately this does not fix the issue, but rather only works around the facts that 'openpgp' module cannot be found locally. The browserify bundle for unittests will now include the whole openpgp.js src code, which was marked as external using the 'openpgp' module.


yeah - was just a hedge while I was verifying this code was not broken (this is two Decode methods pulled back together).


Don't copy the contents in verbatim. Use the `\input` command instead, see #1.


Be careful with your indentation. Especially loops without body blocks.


Test.


If we want to avoid string `../`, the previous code is not enough.

~~~php
<?php
$uri = '..../..../etc/passwd';
var_dump(str_replace(array('//', '../'), '/', $uri));
var_dump(preg_replace(array("/\.+\//", '/\/+/'), '/', $uri));
~~~

The results:
~~~
string(16) "../../etc/passwd"
string(11) "/etc/passwd"
~~~

All input data should be validated before using them, even if it comes from URI string.
But Fuel provides *URI security*, so users may forget the validations for URI data.

If you change the code, it may be BC break.

> But arguably that's also a bad developer issue.

Yes. But if there is a bad developer, and if he/she have to check all code that uses URI data and have to add validation for them, it costs much.


Could you convert it for them automatically to lower case?


I'll just make an issue since I can't get to it right now.


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Yeah, I like this approach.


@comment.txt


Initialising fields in this way makes it harder to write unit tests. Consider using dependency injection as a technique to help with this.


I think it's better to keep it called index.html - more standard, shows that it's the main page.


Don't need this init.  The following should accomplish the desired result:
`p = EmailProducer('x@y.com', 'a@b.com')`


The iterator is already at the position where the new value should be inserted :)


Title should be more complete, for example Ivan Grafutko Resume 2016


since u only use 1 CSS file, remove the other 2


ohh yes ... its already iterated ... silly me.


No need to specify the access modifier in interfaces as it is implicitly set to public.


Removing `const` here is unfortunate, but adding `setProtectedQuery(item->query());` to these methods means they are no longer pure.
I can't find any other way to set `_query` to the `QUrl->query()`.


it's better to have readable tags, for example this is title, so id="title"


id="section"


id=address


something generic here id=bulletlist


Yes, that sounds quite wrong

On Fri, Jan 22, 2016 at 3:12 PM, Clayton Coleman <notifications@github.com>
wrote:

> Passing the internal decoder to this (the behavior that existed before)
> means we are sorting on the internal object, not the versioned object, in
> response to user input. We should be sorting based on versioned output,
> probably on the raw JSON itself.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/ddd4bc6ad7fd2a21144003de58ff585df6704ad2#commitcomment-15625584>
> .
>


My number is 11


Should be LocalVarType+. Spec says count must be at least one. 0 locals is represented by nil signature.


What is your reason for removing the third parameter, 1?
With the second parameter being 0, the function is called in an busy loop.
cf: https://code.angularjs.org/1.4.6/docs/api/ng/service/$interval


Done


I was using it for debugging, I'm removing it now. In theory it could be used if you wanted to set hard limits on parallel connections, but I'm not sure if we want to support that.


Interesting name


This is actually more like a SerializerFactory, right?


FYI, I've been attempting to cross compile [ngrok](https://github.com/inconshreveable/ngrok) for Windows and I've received an error that appears to be related to this commit:

```
# github.com/nsf/termbox-go
src/github.com/nsf/termbox-go/termbox_windows.go:534: cannot use c.Fg (type Attribute) as type int in argument to get_ct
src/github.com/nsf/termbox-go/termbox_windows.go:534: cannot use c.Bg (type Attribute) as type int in argument to get_ct
Makefile:17: recipe for target 'client' failed
```

The error disappears upon reverting to the previous commit.


EAGLE EYES


please try to match style with previous variable/function names.


Great, I didn't know that! :+1:


test


Thanks for another PR!


$comment


Yes - I flinched a bit at using factory just based on history, but I can do a naming pass.  I expand this a bit in a follow on PR to make negotiation for streaming protocols a bit clearer.

The apiserver machinery and the client on the other side (during a negotiation) would be the two consumers of the interface.


I'll update the TODO and file an issue.

On Fri, Jan 22, 2016 at 6:21 PM, Daniel Smith <notifications@github.com>
wrote:

> Yes, that sounds quite wrong
>
> On Fri, Jan 22, 2016 at 3:12 PM, Clayton Coleman <notifications@github.com
> >
> wrote:
>
> > Passing the internal decoder to this (the behavior that existed before)
> > means we are sorting on the internal object, not the versioned object, in
> > response to user input. We should be sorting based on versioned output,
> > probably on the raw JSON itself.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/smarterclayton/kubernetes/commit/ddd4bc6ad7fd2a21144003de58ff585df6704ad2#commitcomment-15625584
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/ddd4bc6ad7fd2a21144003de58ff585df6704ad2#commitcomment-15625714>
> .
>


The links look good! Suggestions:
- in bag-quickstart change the 'dask bag api' link to be a :doc: instead of a :ref: so it goes to the very top of the page
- in dataframe-overview remove 'Related Pages' section
- have overview first and then quickstart second in the bag section and the dataframe section


A bit icky, is there any reason Sqlizer can't just take sqlReps and typeReps separately?


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


Sorry that was a  bad idea which doesn't make it better.


Order matters in tar but I don't think it is helpful to try and specify an order.


Why were some of these methods commented out instead of removed?
Do these methods no longer exist? Should we uncomment them at some point?
What is the semantics of these comments?


gh-comment


I'd like to refactor that too.  But this is the faster route.  Can I make that separate work?


done


But, I don't understand what protected means. Protected how, from who?


Yeah, no need to change. Just making sure I understand what it's doing.

On Fri, Jan 22, 2016 at 3:25 PM, Clayton Coleman <notifications@github.com>
wrote:

> Yes - I flinched a bit at using factory just based on history, but I can
> do a naming pass. I expand this a bit in a follow on PR to make negotiation
> for streaming protocols a bit clearer.
>
> The apiserver machinery and the client on the other side (during a
> negotiation) would be the two consumers of the interface.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/54c1dee94c1a737906fc14e96522418abf6d2453#commitcomment-15625764>
> .
>


'echo' gh-comment


Yeah, I think so...


Yes actually, done


cat comment.txt


Even better is to have different reps at each chained soql level.  But I would like to stop the bleeding first.


we should take command line input for the path here.


Removal of CAPITALIZE breaks non-english strings. Look at the screenshots for the letter after German umlauts which are now erroneously uppercase.

3.2.1 => okay

![screenshot005](https://cloud.githubusercontent.com/assets/3226626/12525865/346990b6-c168-11e5-8e31-575836a70753.png)

3.2.3 => broken

![screenshot004](https://cloud.githubusercontent.com/assets/3226626/12525869/3adb2ec8-c168-11e5-83a9-2ecb701992a5.png)
We had that a while back already, do you remember? IIRC, you introduced CAPITALIZE to fix this.


Sorry, you missed the point of my "be defensive" comment.
This panics: `blah := foo.(map[string]interface{})`
This doesnt:
```
if blah, ok := foo.(map[string]interface{}); ok {
  ...
}
```
You still have the panicky version


@cartant Fixed in the latest commit just now. Sorry, my fault, I should test pull requests people send me.

/me gives an angry look at @JoelOtter


Same as before.

In such a case a crash is pretty much just as correct as casting to false.
In reality it's going to end up calling boolValue on the string which will give a correct answer.  Cocoa's setValue forKey is nice like that.


OK. Fixing.


I wasn't sure if this condition syntax was right, so I tried it. It seems to come out always true, even if TRAVIS is unset. I think this should work:

```
if [[ "$TRAVIS" == "true" ]]; then
```