Frankly I don't fully understand.
It has something to do with a weird logic and masking integers that turn into Node pointers (???) see this function:
https://github.com/Qucs/qucs/blob/master/qucs/qucs/schematic_element.cpp#L1523

The bit flags seem to mean:
1 Node attached to vertical Wire
2 Node attached to horizontal Wire
\>3 new node / split wire / wire not connected (?)

As I see, the code checks the nodes flags to restrict the wire  stretch. Moving a vertical wire when connected to a horizontal wire (via a node with flag 1) can only slide along x-direction. Something like this (me waving hands)...


Trello


Looking good


This couldn't be a typo, could it? :smiley:


The underlying data structure used by `transformers` should remain as an implementation detail.  Although it may be a `list` today, it maybe a `dict` tomorrow.  But the interface is that it's an ordered collection.


Ctrl+V to the wrong place


The incremental compiler stores these hashes to compare to in a future compiler run. Since the hash of a class does not include the hash of its superclasses, changes to those superclasses will not be reflected in the subclass's hash, unless we propagate those changes in some way. This is why we include both declared and *inherited* members in this subclass's hash, which is not very space efficient, IMO.


I agree with this analysis.


Empty block. #humanrubocop


Yes, 2.1.0 makes sense.


Yeah. It just calls `cache.dropKey(arg, callback)` - Should likely be `User.dropCache` instead of on `User.prototype`


Ah, OK :)


Oh, I totally missed that. Got it.


I was unable to get it out of my mind so i figured id write a quick proof of concept for my theory on the atmospherics. This lack code in place to handle quick update of tiles where an atmospheric path just opnened (door open/close, wall destruction etc) but the cpu cost of those is insignificant compared to the load were talking about in the benchmarks (were only looking at 1-5 turf process in the following 0.5sec if such event occur, compared to the 250+ done every 0.5sec non stop at 16k active turfs)

I can't stress enough the fact that this is not PR material, this is nothing but a proof of concept i spent 1hour on (then let it run for 5hours for profiler purposes). If there is interests in it and the maintainers give their ok i have no problems cleaning it up and doing a proper PR with it but considering our history i doubt that will happen.

This atmospheric code use between 95% to 50% of the cpu the current paradise atmo code use (the higher the number of turfs active, the more efficient this code is). At low atmospheric damage, atmospheric cpu load is pretty similar and mostly insignificant, at high load this code is a lot more efficient than the current atmospherics .

profiler comparison data between current paradise and this commit after the atmobenchmark (wtf) level of atmospheric disturbance over 5 hours.

http://sebsauvage.net/paste/?de174e277a870182#xm3Em9tOkWJGtDe9R6yXCo+9Z9GLU7FKDd4GXd5v0Pw=

if you wish to reproduce the atmobenchmark verb on current paradise, simply replace this from admin_verbs.dm
"var/list/admin_verbs_debug = list("
with
"var/list/admin_verbs_debug = list(
	/client/proc/atmobenchmark,"

and add at the bottom
"
/client/proc/debugdelete(var/atom/O)
	log_admin("[key_name(usr)] deleted [O] at ([O.x],[O.y],[O.z])")
	message_admins("[key_name_admin(usr)] deleted [O] at ([O.x],[O.y],[O.z])", 1)
	if(istype(O, /turf))
		var/turf/T = O
		T.ChangeTurf(/turf/space)
		return
	qdel(O)
/client/proc/debugdeleteairlock(var/atom/O)
	for (var/obj/machinery/door/OD in O.contents)
		debugdelete(OD)

/client/proc/atmobenchmark()
	switch(input("Benchmark level?") in list("light", "medium", "wtf"))
		if("light")
			world.log << "Light benchmark selected. Destroying ..."
			log_admin("[key_name(usr)] made all areas powered", 1)
			message_admins("\blue [key_name_admin(usr)] made all areas powered", 1)
			power_restore()
			debugdelete(locate(57,143,1))
			debugdelete(locate(128,120,1))
			debugdelete(locate(161,115,1))
			debugdelete(locate(198,116,1))
			debugdelete(locate(157,60,1))
			debugdelete(locate(105,73,1))
			debugdelete(locate(104,163,1))
		if ("medium")
			world.log << "medium selected Destroying more"
			log_admin("[key_name(usr)] made all areas powered", 1)
			message_admins("\blue [key_name_admin(usr)] made all areas powered", 1)
			power_restore()
			debugdelete(locate(57,143,1))
			debugdelete(locate(128,120,1))
			debugdelete(locate(161,115,1))
			debugdelete(locate(198,116,1))
			debugdelete(locate(157,60,1))
			debugdelete(locate(105,73,1))
			debugdelete(locate(104,163,1))
			debugdeleteairlock(locate(142,70,5))
			debugdeleteairlock(locate(142,73,5))
			debugdeleteairlock(locate(178,76,5))
			debugdeleteairlock(locate(175,76,5))
			debugdeleteairlock(locate(170,76,5))
			debugdeleteairlock(locate(71,118,1))
			debugdeleteairlock(locate(69,118,1))
			debugdeleteairlock(locate(46,133,1))
			debugdeleteairlock(locate(48,133,1))
			debugdeleteairlock(locate(49,136,1))
			debugdeleteairlock(locate(51,137,1))
			debugdeleteairlock(locate(61,140,1))
			debugdeleteairlock(locate(62,140,1))
			debugdeleteairlock(locate(203,130,1))
			debugdeleteairlock(locate(200,130,1))
			debugdeleteairlock(locate(189,129,1))
			debugdeleteairlock(locate(189,129,1))
		if ("wtf")
			world.log << "wtf selected Destroying more and all doors !!!!"
			log_admin("[key_name(usr)] made all areas powered", 1)
			message_admins("\blue [key_name_admin(usr)] made all areas powered", 1)
			power_restore()
			debugdelete(locate(57,143,1))
			debugdelete(locate(128,120,1))
			debugdelete(locate(161,115,1))
			debugdelete(locate(198,116,1))
			debugdelete(locate(157,60,1))
			debugdelete(locate(105,73,1))
			debugdelete(locate(104,163,1))
			for (var/obj/machinery/door/OD in world)
				debugdelete(OD)
"


The Oxford comma.....
I guess it's recommended but I often don't use it. Fixed


see above


I like the conciseness of Picasso and the ability to do error handling in a bit more sane manner.

I should have grabbed the bitmap/bitmapdrawable from the imageView rather than opening an imageStream to do that work over again.


is my `it 'should allow the standard logger methods'` test on line 11 sufficient?


How about, `_find_and_log_dags`?  Updated.


Cześć, mam nadzieję, że to przeczytasz - udostępniłeś gdzieś kod z poradnika o rwd, który jest na youtube? Jeżeli tak, możesz podać link?


Just delete the unused logic. We can always revert to a previous commit to recover them if needed.


if I revert #8886 (as it breaks DVD menus) and I add this PR, somehow deinterlacing is also broken.


If the interface is that it's an ordered collection, that seems all the more reason to mimic the interface python provides for its builtin ordered collection (lists), rather than using a non-standard `add` method.


Nope I didn't know what to call normal Max's, since they use the a1,a2 and a4 naming I thought Ax would do the job ahaha. Got a better idea?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.653.45745e9b23d6801cc5997698b0c1361d135f93f7](http://build.palaso.org/viewLog.html?buildId=68369&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3323, ignored: 125 Build time: 00:05:23
### Failed tests
```
SIL.Windows.Forms.Tests.dll: SIL.Windows.Forms.Tests.ImageToolbox.PalasoImageTests.Image_FromFile_GivesImage: <no details avaliable>

```


PS: Wiem, że to nieodpowiednie miejsce, ale nie wiem gdzie Cię złapać XD


I do not see the errors below when I build. I suggest there may be a problem with your build environment, or perhaps some script was saved with CRLF line endings.

```
target thumb C++: recovery <= bootable/recovery/data.cpp
/bin/bash: -c: riga 0: EOF non atteso durante la ricerca di """
/bin/bash: -c: riga 1: errore di sintassi: EOF non atteso
build/core/binary.mk:706: set di istruzioni per l'obiettivo "/var/lib/jenkins/jobs/CM-TWRP/workspace/out/target/product/thea/obj/EXECUTABLES/recovery_intermediates/data.o" non riuscito
make: *** [/var/lib/jenkins/jobs/CM-TWRP/workspace/out/target/product/thea/obj/EXECUTABLES/recovery_intermediates/data.o] Errore 1
make: *** Attesa per i processi non terminati....
target thumb C++: recovery <= bootable/recovery/find_file.cpp
/bin/bash: -c: riga 0: EOF non atteso durante la ricerca di """
/bin/bash: -c: riga 1: errore di sintassi: EOF non atteso
build/core/binary.mk:706: set di istruzioni per l'obiettivo "/var/lib/jenkins/jobs/CM-TWRP/workspace/out/target/product/thea/obj/EXECUTABLES/recovery_intermediates/find_file.o" non riuscito
make: *** [/var/lib/jenkins/jobs/CM-TWRP/workspace/out/target/product/thea/obj/EXECUTABLES/recovery_intermediates/find_file.o] Errore 1
```


this should also be removed, right?


ditto


however, if I only revert #8886, deinterlacing works fine.


This is probably due to the yanked migration for the jsonb column.

It doesn't really matter. These are just strings. If they don't correspond to any file, that's ok. The opposite (file with no corresponding INSERT) would be bad.


I thought it did.  But perhaps not.  Removed.


Changes look good.  Have a quick question...  If both init_intrabit and init_centering become 0, does that mean it is at risk of missing the first bit and not getting the first byte.  But will catch the subsequent bytes.


if there is *no* such header?


Since (per our over-the-air discussion) this really relies on monkey-patching these two classes anyway, seems like you could simplify this code by reopening the classes:
```ruby
class ActiveRecord::Relation
  def preloads(*methods)
    ...
  end
end

class ActiveRecord::Base
  def self.preloaders
    ...
  end
end
```
Maybe just a style thing, I dunno. But feels like tinges of [this post](http://yehudakatz.com/2009/11/12/better-ruby-idioms/) in here.


`cwd` defaults to `process.cwd()` pretty much everywhere in the codebase. Might be a nice refactor to move it to the CLI options.


sounds good.


can you elaborate? zip extraction could mean a lot of things


Hah, no obfuscation?


Hey what is this?


どのようなurlがあれば like が実装できるかを考えて、修正。


:+1: :100: :tada:


yes


resources と書くとどのようなroutingが生成されるか自分で確かめてみてほしい。


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/7685783450488aa9c6d280a7e5afbf70e787a007


以下、不要


merging basho/riak_cs/refactor/doc-cosme = 76857834 into borshop-integration-1289-refactor/doc-cosme


basho/riak_cs/refactor/doc-cosme = 76857834 merged ok, testing candidate = aba6e10a


reduce_redimensions (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-warnings-yellow.svg)


develop (lefticus) - Win64-Windows-7-VisualStudio-12: OK (382 of 383 tests passed, 0 test warnings)

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


typed_function_ordering (lefticus) - i386-Windows-7-VisualStudio-14: Build Failed

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


typed_function_ordering (lefticus) - Win64-Windows-7-VisualStudio-14: Build Failed

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


@aliceh  Looks good...i only see a minor issue of no spaces between commas in this line:
+        self.tester.create_launch_config_and_asg_from_lc_lp(lc_name=launchconfig1_name, asg_name=scalinggroup1_name,max_capacity=2,desired_capacity=1,min_cpapacity=0)


:video_camera: :movie_camera:


Hugo est bandé sur notre réponse. Jsuis honoré.


Ah yeah - I had to add that fn to the form not to that fn call - I'll remove it


Thanks @ccassler !


When expressions are these complex, I prefer putting the preconditions in separate lines. It makes it easier to see exactly what's going on.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2046 of 2050 tests passed, 325 test warnings)

 * 331 tests had: AUD diffs.
 * 329 tests had: BND diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: Table small diffs.

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


still good to say we have deprecated the Script transform.


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


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


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


this is brach test


@mulkieran I did some investigation base your kind suggestion.

By checking `dmparser.c disassemble_map()` function, when `pgp == NULL` or `pp == NULL`, it will return -1 which cause a failure at early stage in `libdmmp.c dmmp_mpath_array_get()`. So, we will not get NULL pgp here.

In case anything goes unexpected, I changed to code to raise BUG error instead of warning when `pgp == NULL` or `pp == NULL`, which lead to the proper clean up and errror handling.

Thank you very much for detailed code review.


typed_function_ordering (lefticus) - i386-Windows-7-VisualStudio-12: OK (383 of 384 tests passed, 0 test warnings)

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


Speed really fast


@GQ722 Please remember to run 'ant clean test' prior to pushing your code.  You have broken the build.  Please have a look at the output from Jenkins (or run 'ant clean test' on your machine) and look at what FindBugs found, correct it, and push again to un-break the build.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (383 of 383 tests passed, 0 test warnings)

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


It's what would've done myself if you hadn't told me not to :wink:


YAY. I like Daniel too.


Should've been agile about it ;)


We don't have toolchain files for x86-clang. I suppose we can just get rid of these lines as they're not super useful anyway.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (383 of 383 tests passed, 0 test warnings)

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


LGTM.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


typed_function_ordering (lefticus) - Win64-Windows-7-VisualStudio-12: OK (383 of 384 tests passed, 0 test warnings)

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


111768812_VRFNoLoadOAFlow (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

 * 6 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)


Addressed in 6e651e9c52c41420734ca8692ed154edeeabae57


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


is this one intended?


112083967_VentilatedSlab (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1507 of 1507 tests passed, 0 test warnings)

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


c'est ce commit la qui a cassé Bleach je sais pas trop pourquoi au juste


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


reduce_redimensions (mbadams5) - 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.84%25-green.svg)


FYI: The most important changes in this commit are to the authorpendingtablemodel


You can add comments to the same lines I commented on if you want to answer a specific comment


@feross we were wrong all this time ;)


should be 2014-2016?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1768


nice work!


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


WHATT


Word thanks man!

On Wed, Jan 20, 2016 at 5:22 PM, Justin Breucop <notifications@github.com>
wrote:

> Awesome work, including demonstrating in both pandas and numpy
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jaguila/DAT_SF_19/commit/70d7dbba9e7b0917bd42a0eb16a00c088bc20251#commitcomment-15574550>
> .
>

--
John Dexter Aguila
Phone: 732-690-8051


@corrspt see https://github.com/playframework/playframework/issues/5410


TeamCity pull requests :: netty [Build 1072](https://garage.netty.io/teamcity/viewLog.html?buildId=4909&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4907, ignored: 23 Build time: 00:35:46


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


@danmayer - i feel like we need a less fragile test here.  I noticed that whenever middleware.rb was updated for whatever reason and the line numbers on that file changed, this test failed.  Feel like maybe we want to not be testing that coverage against our own code.  Maybe we should create some dummy files to test coverage against?


直接用 `utility.esacpe` 就可以了


Reusabilty of this form isn't currently promised, but is the subject of https://code.djangoproject.com/ticket/19353 if you'd like to help out. Given the form already has `fields = ("username",)` and a reference to `self.cleaned_data.get('username')` in `clean_password2()`, I don't think this reference creates any new problems.


in the same vein as add the implicit int, I noticed that you added 'unsigned id' -- I've always felt that leaving out the 'int' was a mistake.

I did try adding -Wall to your makefile, but that looked like an uphill struggle. It did find (in cjson.c) a probably harmless bug (tolower being called with a char argument). Fixing all of those errors is best left for another time!


I might still point out that `\p@` is a form of 1 used to increase the accuracy of the calculation.  I know that without some reminder I'll eventually come back to this code and scratch my head about why it's there.


develop (lefticus) - 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-warnings-yellow.svg)


Tabs here...


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I'll do it on Transifex


... or define constants then you do not have magic numbers in your code. I guess you mean 0.2m and 10.0m.


@bbatsov @Malabarba The idea here is that tooling frames (compiler, nREPL, middleware) are unwanted noise when looking at an error in your application, and that these frames are contiguous at the bottom of the stack.

In the original code, the bottom `clojure.lang.Compiler`, `clojure.tools.nrepl` and `cider.*` frames are flagged as `:tooling` so they may be hidden by default. Stack frames above these are "user" code, which isn't flagged. This commit would flag *everything* as tooling.


thanks for the solution


了解です！


this doesnt fix the clients side. the list "variables" sticks around on the clkient when they disconnect


Hi,
I'm sorry for the late reply (just had new baby arrive two weeks ago exactly). But thanks for point this out. It was fixed in the following commit https://github.com/gonzigonz/Pluralsight---The-World/commit/68b139083422bce084ecb79698429623658910b7.

Great to know your watching the repo, let us know if you have any other questions. Next step for me will be to see if I can get this working 100% running on my Mac.

Cheers Gonz


We've added this for test purposes as we are experiencing racing conditions when using `idb-blob-store`, currently debugging.


members set validation improved


@jordan-wright looks good, goose migrated to 20160118194630 and no data loss observed

lab@server4:~/gocode/src/github.com/gophish/gophish$ sudo ./gophish
 2016/01/20 20:32:38 worker.go:34: Background Worker Started Successfully - Waiting for Campaigns
goose: migrating db environment 'production', current version: 0, target: 20160118194630
OK    20160118194630_init.sql


fast-forwarding 2.1 to borshop-integration-1289-refactor/doc-cosme = aba6e10a


current_userを毎回書けばいいのでは、ということですか？研修問題に最初に代入してあったので、
そのまま残しました。が、current_userを毎回書いた場合って、dbの中にあるcurrent_userを毎回取ってくることにならないですか？


Woot!


This is probably getting a bit deep for a commit comment discussion :)

I do feel that it is much better to test against clean configs and rules (testing daemon behaviour is a bit different). But if everyone else disagrees, then I don't care enough to try and convince you all ;)


Oops, I read too quickly and missed the `if`. It won't flag everything as tooling. It will mark frames above the first "user" frame though. This wasn't the original intention, but it's probably not harmful either.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Please fix the indentation here.


excellent! will try a practice run with one of our new submissions immediately. thanks!

> On Jan 20, 2016, at 4:02 PM, W. Caleb McDaniel <notifications@github.com> wrote:
>
> Further instructions on adding images and publishing lessons
>
> @fredgibbs
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/programminghistorian/ph-submissions/commit/7ed6e5f60a7df8cc70de881043444369e50be995


This form is never used. A new form is created as the first thing in each of the branches of code below


You have already fetched the instance above, no need to do it twice


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


TeamCity pull requests :: netty [Build 1071](https://garage.netty.io/teamcity/viewLog.html?buildId=4908&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6385, ignored: 23 Build time: 01:24:01


Updated to the style of resing.net


this was +CircuitLord


The problem is `ConnectorTableLayout` is based on `ColumnHandle`s, but this table does not exist yet so we don't have handles yet.


Need spaces instead of tab :)


the title should be "different strategy when losing"


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


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


Did you have help @LOTRfan02?


Please change this to "Jake Taylor", not "Jake Taylor Developers" :)


@article = Article.where(user_id: @user.id)
でdbにアクセスしてデータを取得してから、@article.text、@article.titleとそれぞれ取ればいいということですか？


I think he figured it out by himself.


@tractorcow is that a new syntax for translations or bung code?


@wjwoodson Excellent! Thank you for checking! I'll verify on my branch as well and, if all looks good, then I'll merge.


@andronat Ok, I will add some descriptions on top of the function.


I think @PieCoder314 helped


확인 완료 ^^


@JustAkan 감사합니다. 6383dfa 부터 7263346 까지 적용하시면 GCC 6 에서 정상적으로 컴파일 됩니다.


Yes, I did. I was t his house and we were messing around with some stuff on there and he decided to make this.


該当するarticleが見つからなかった場合にどうするかがわからないです。
find_by了解しました。

cf) そもそもある記事は絶対Articleの中の一番上のもの(first or find_by)でいいのかっていう疑問があったのですが
どうでしょう。


ㅋㅋㅋ 안 없어져서 이상하다 했어.


Ohh okay.


fvfvfvfgbrgbrbrgb


aaaa


aaa


Looks good on my end as well - merging.


今見てて思ったのですが、article_id: Article.where(user_id: @user.id))って、article_idちゃんと取得できるんですか？
@commentsに色々入れるのは、.eachで回しているので、いいと思ったのですが・・・。


ahhhhhh I get it now.

my code WAS broken, vs. "my code was broken!"

9.5/10 would read again


you did not do everything @PieCoder314


I am surprised I forgot to include task (may be the easiest one). But, it's too late ...


すみません、html, rubyなどのインデントについて聞いてないのですが、概要教えてもらえますか？


No, I didn't. @LOTRfan02 did most of it, and I was just showing him what to do.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (389 of 389 tests passed, 0 test warnings)

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


@PieCoder314 then you forgot to tell him to do everything


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


What do you mean?


reduce_redimensions (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1670 of 2088 tests passed, 61 test warnings)

 * 521 tests had: AUD diffs.
 * 217 tests had: MTD diffs.
 * 407 tests had: EIO diffs.
 * 373 tests had: ESO big diffs.
 * 196 tests had: MTR big diffs.
 * 182 tests had: SSZ big diffs.
 * 226 tests had: ZSZ big diffs.
 * 323 tests had: Table big diffs.
 * 43 tests had: ERR diffs.
 * 11 tests had: SSZ small diffs.
 * 30 tests had: ESO small diffs.
 * 15 tests had: MTR small diffs.
 * 50 tests had: Table small diffs.
 * 7 tests had: ZSZ small diffs.

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


確かに。了解です。


111768812_VRFNoLoadOAFlow (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

 * 6 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)


you had to fix stuff with be5a8f5


TeamCity pull requests :: netty [Build 1074](https://garage.netty.io/teamcity/viewLog.html?buildId=4935&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 4087, ignored: 6; compilation error: Compiler (new) Build time: 00:12:21


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


Yeah, I guess I did.


Thanks! Just in case you missed that part in my previous comment, you should move these to constants.xml so that they don't get sent for translation :)


Forward-declared classes don't guarantee alignment. The old code for PointerLikeTypeTraits just assumed 4-byte alignment for every class, all the time. Chandler Carruth from LLVM saw this, realized it was completely unsafe, and changed the code to use `alignof`, which of course requires a complete type.

Michael and I have been discussing solutions that would just be PointerLikeTypeTraits, but PointerUnion doesn't take a PointerLikeTypeTraits. Michael then tried to use a wrapper type, but that requires us to change all the call sites that use `union.get<Foo *>` to `union.get<UnsafeWrapped<Foo *>>`, which would be an even bigger diff than what we already have.

PointerUnion is safe, assuming the PointerLikeTypeTraits are correct for a given type. The PointerLikeTypeTraits are *not* guaranteed correct if they're assuming any particular alignment for a forward-declared type. Ergo, "Unsafe".

I'm not happy about this either but we couldn't come up with anything better. In time we might want to migrate to something more heavyweight but more correct: Chandler's new PointerSum type.


New syntax


findをしただけで、returnしてくれるということですか？


Also added the ability to destroy asteroids


> On Jan 20, 2016, at 7:49 PM, Joe Groff <notifications@github.com> wrote:
>
> Is the plan to fix our code not to use forward-referenced class pointers with PointerUnion anymore?
>
Yes, in the cases where it is possible. In certain cases it is very difficult to do this. In such cases, we can use UnsafePointerUnion.
> I'm missing the background here—why is it desirable to prevent forward-referenced pointers from being used with PointerUnion to begin with?
>
>

The old version of PointerLikeTypeTraits assumed that all pointers are 2 byte aligned. Now it is figured out based off of the type. This is impossible to do when the type is a forward declaration.
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/c6d0b5c1a9030eebc3a4c6e4e1498c97b7fababc#commitcomment-15574907>.
>


Changed all index conditions to check global variables tracking the number of active elements such as asteroids or projectiles


__API Description file syntax error on line 968__: base type 'Currency' is not defined in the document (Autogenerated message from apiary.io)


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.5.0](http://build01:8888/viewLog.html?buildId=13897&buildTypeId=OpenSourceLibraries_Sourcerer_CI) is now running


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.12.0](http://build01:8888/viewLog.html?buildId=13899&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) is now running


are you sure? :)


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.5.0](http://build01:8888/viewLog.html?buildId=13898&buildTypeId=OpenSourceLibraries_Yaclp_CI) is now running


Agains, this is pretty Australian specific. IsNative might be better. A breaking change I know but I think its worth it


         <title>Joe's Hello World~!!</title>

'Hello World'~:) I love "Hello World"

         <title>Joe's Hello World~!!</title>

Hello World :) I love Hello World

+
 +
 +        <title>Hellow World!!!</title>
 +
 +
 +
This is a test

 +
 +


TeamCity pull requests :: netty [Build 1081](https://garage.netty.io/teamcity/viewLog.html?buildId=4962&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4903, ignored: 23 Build time: 01:02:09


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.12.0](http://build01:8888/viewLog.html?buildId=13899&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:32


Didn't get my previous point:-) Don't pass Tun as a parameter. It's already a data member and you can directly use it in the member function. Same for UpdateRoutes.


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.5.0](http://build01:8888/viewLog.html?buildId=13898&buildTypeId=OpenSourceLibraries_Yaclp_CI) outcome was **SUCCESS**
Summary: Tests passed: 20 Build time: 00:00:28


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.27.0](http://build01:8888/viewLog.html?buildId=13900&buildTypeId=BlogMonster_CI) is now running


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.10.0](http://build01:8888/viewLog.html?buildId=13901&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) is now running


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.10.0](http://build01:8888/viewLog.html?buildId=13901&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) outcome was **SUCCESS**
Summary: Tests passed: 21 Build time: 00:00:20


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.27.0](http://build01:8888/viewLog.html?buildId=13900&buildTypeId=BlogMonster_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:00:28


Hey, what is this again?


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.5.0](http://build01:8888/viewLog.html?buildId=13897&buildTypeId=OpenSourceLibraries_Sourcerer_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:01:16


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.31.0](http://build01:8888/viewLog.html?buildId=13903&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


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


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


PlantCompBaseClassAgain (Myoldmopar) - 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-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.85%25-green.svg)


I'm pretty certain that would work now, but I don't have test cases for any data source tools so I'm not touching that file.


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


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.31.0](http://build01:8888/viewLog.html?buildId=13903&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:01:03


Nitpick - put space on both sides of =, e.g., a = 12.


To be consistent, I'll rename `add` to `extend`.  I don't see a need to implement `append` at this time.


Works for me.


thanks... seems quite a hack; I have checked the `pw->Port1` values while moving wires around and indeed I see it can get a value of `1` or `2` and it seems related to the wire direction. But then most of the time it has a much higher value (i.e. it's a real pointer to somewhere in memory)...


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1028.45745e9b23d6801cc5997698b0c1361d135f93f7](http://build.palaso.org/viewLog.html?buildId=68368&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:33


@ExSport No, I didn't touch the ```FILES``` table that this issue probably belongs to. I simply changed the initialization of the H2 database itself a little, because if had concurrency issues the way it was made that I couldn't get around without changing it.


Why are you introducing a traitinfo lookup here?


@djazayeri does this make sense?


Some of these exceptions are very specific. If they are so specific do you really need to accept a msg in the constructor or can you hard code it?


nice line


Could you add pretty solve too?


I fucking hate java


It looked good at first, but it had strange side effects. godep still required packages to be downloaded first to $GOPATH and later restored into vendor/, and a lot of odd things.

Yes, glide is also on my radar. We want and need _something_


Maybe add a comment to explain it to future readers of the code.


Probably don't need the you need the brackets.


I noticed that *hasClass('budget-inspector')* (was on line 76) was removed and added to line 147, but there are a number of features in that require that class match up in order to work. Did you mean to put your changes on line 76 over on line 147?


Grr pre  solve
Automatic corrector are a pain


When you download assets for instance, it says "Failed. Extracting assets.zip"

![image](https://cloud.githubusercontent.com/assets/3447089/12492851/9da3431a-c082-11e5-8708-d487f22bc59d.png)


Also: Backport to 1.0.x?


Nitpick - for trailing comment, put 2 space after after the code and then 1 space after //. Also, remove "form". E.g. unordered_map<> stats_;  // <client_id>.


epic fail


Our first repository


Should be:
:class:`_search.SequentialSearchCV`


Ahhh ok. Anyway thanks for any fixes you provide :)
Found it: https://github.com/UniversalMediaServer/UniversalMediaServer/issues/530


Please click on view to view the full file.


Should be 'SequentialSearchCV' instead of 'GPSearchCV'


112158233_PVwithStorageSystem (Nigusse) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

 * 6 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)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.664.45745e9b23d6801cc5997698b0c1361d135f93f7](http://build.palaso.org/viewLog.html?buildId=68370&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:34


How about ` [BackdropCMS](https://backdropcms.org) - A fork of Drupal; A Content Management System targeting small-to-medium sized business and non-profits.`


properties is not exposed from anywhere right? I checked that the configurer does not have a getter for it. Making sure because its not immutable anymore (doesn't look like its needed to be immutable)


Is there a less fragile way to handle storing the preloaded data? This relies on `obj` having the `name` attribute. At the very least "can't preload `name`" would be a better error message than "undefined method `name=`". Or maybe there's a magic way to add the attribute to the object if it doesn't exist (not that this is necessarily better, just an idea).

Or flip it on its head: patch `ActiveRecord::Base`. Maybe you explicitly have a macro so you can say
```ruby
class MyModel < ActiveRecord::Base
  preloads :foobar
end
```
which expands into a suitable hash lookup like
```ruby
class MyModel < ActiveRecord::Base
  def foobar
    @preloaded[:foobar] ||= @preloader[:foobar].call(self)
  end
end
```

Then the `ActiveRecord::Relation#preloads` would reach inside `.each` instance of `MyModel` and initialize the `@preloaded` stuff one-by-one.

Just floating ideas.


How do I otherwise get that `BuildAtProductionType` thing?
If I pass it as parameter would me rewriting stuff further, but I could do that, if wished.


we probably also need to test php 7 at this point


Do you want to keep this pointing to branch?


Yep, I had that in the travis config, but it got removed somewhere along the line.  Just added it back.


Thanks!


this get_class() looks a bit odd for me. Why not just giving a string FQCN directly?


Old habit carried over from other languages

The advantage of calling it something like log is all logging code looks the same


Właściwie to nigdy mi się jeszcze nie zdażyło!


FWIW, gcr.io uses "email: not@val.id"


@nzakas That's already been moved to CLIEngine. I did a PR a while back that removed all of the `process.cwd()` code to allow for setting cwd from CLIEngine.


It's what we get from nvd3 depending on chart type. I can write this in a way that reflects this.


Were the log will be printed?


C'est bizarre que `default.tex` ne soit pas le template par défaut…


can we remove these methods deprecated in 2.8.0?


@Zeno410 Can we try something like this for 're-using' terrain classes in other biomes?


As discussed, using `__iadd__` instead.  Thanks.


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


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


Thanks I'll give it a shot


Fair, some is just tweaking on my part based on what looks better. I'll document it.


so in doing this, it kinda borked the whole 'git pull to update' method. I'm going back a version until this is resolved:

~/sickrage$ sudo service sickrage start
Starting SickRage
/usr/bin/python2.7: can't open file 'SickBeard.py': [Errno 2] No such file or directory


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


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


ChainProp is both a getter and setter


@andie-chernysh, good catch!


I can change it.  I only went back to get_class() because ChoiceType::class wasn't supported by PHP <5.4, and I didn't even think of putting in the string.  The string FQCN would probably be more efficient.


why not to hide the entire h5 here, the design gets disrupted?


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


Just do `error.inspect`.


Certes.

`patacrep.tex` hérite de `default.tex`, donc c'est le template le plus complet (et ça me parait logique de l'utiliser comme défaut).
En revanche, je ne suis pas contre une réorganisation des templates...


Move this subject definition within the `describe "#call"` block.


hello


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


Please avoid whitespace reformatting. It is hard to see what changed in commit.


There are possibly a few of these


It's just a simplification. All the properties in this object are of the pattern
obj.attr() - getter
obj.attr(val) - setter

so I followed the convention of just commenting on what "attr" represents.


Remove reference to "I". We like to refer to "we" instead of "I. You could update the sentence with: "What if you found out there was a high performant and scalable system..."


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


Change to describe `#new`.  `#initialize` is a private method that `#new` calls.


Great :+1:


case:
1) channel was deleted
2) TriggerChannelGroupsUpdate()
3) kodi will check if all channels are valid for that group, and they are because no  TriggerChannelUpdate() first
4) TriggerChannelGroupsUpdate()
5) TriggerChannelUpdate()

This won't probably fix anything but it just looks a bit odd.


Check this: https://github.com/NebulousLabs/Sia-UI/blob/master/plugins/Files/js/fileElement.js#L13


again reference to "I". replace with something like "There's no way Go or some other new platform can compete..."


Before it just didn't show the item. Works now, though. /shrug


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 329 test warnings)

 * 334 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)


Good going there, seems the extension on chrome was doing weird stuff before. Probably fixed with the cache commit. Keep up the good work!


Not currently but it wouldn't be too difficult to add.


I'm going to copy it for now, however it would be nice if you could make that possible for later. Do you want me to open an issue for that?


I'll add it.


OK, I see. So this could be done by adding a field like `Lazy<Structure[]> parentsStructures` to `Structure`?


@Niictar No, the hasClass('budget-inspector') was too broad to actually signify that the user has switched over to the budget view (it is triggered every time you change your selection of budget rows in the budget view). Instead, I changed it to check that the budget button has been set to active, and that now triggers all the scripts that needs to run when switching to budget view from accounts view.

hasClass('budget-inspector') is used to trigger actions that need to be triggered every time the budget row selection changes instead, which it works great for.


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


The intent of this repository is a place to upload interesting HTML Code snippets and samples


Sure, go ahead.


Didn't find any usage except in tests


Corrected in 9eb64efc21b2868e5ba8ccb5d1785d2cfc9946bf. Thanks @joehillen


Presumably, if the dir does not exist, then that dir likely is not on the path and you'll have problems later on (like when you do os.LookPath(). I would just error out if the dir does not exist.


@ExSport I'm planning to look further at thumbnail creation in general very soon, so I might end up fixing that bug (if I can figure out the current logic). This PR is actually hard to test because thumbnail generation is so bugged that it's almost impossible to get the cover utility to get called at all.


ok


Nit: since these tests are exactly the same as in the previous fact, you might turn the previous one into a theory that's passed the empty ListDictionary to verify as empty, and then pass in various ListDictionary instances, e.g. one created with the default ctor, one created with a comparer, etc.


Nit: these empty tests were done in the previous ctor tests


Yes, copy paste error. Fixed.


Am Sonntag wird der Code für die Codereview festgelegt, also bis dahin soll das fertig sein


hmm. Are you suggesting that we do the check on a per-file basis, in `updateFileElement`?


The old env population logic is replaced by "env.putAll(context.properties())" in the callers. Less duplication and better interface without side effect.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.654.45745e9b23d6801cc5997698b0c1361d135f93f7](http://build.palaso.org/viewLog.html?buildId=68371&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:04:25


Can you simplify this to just returning []error and anywhere you need to return a single error, just push it onto the slice and return a slice with length 1? (this goes for the underlying functions that also return []error, error


Is this supposed to revert 84a3dd0?


Why the sleep?


Since this is a loop, you probably don't need to be doing these CollectionAsserts both before and after the add.  You could just do it after, which will be the before for the subsequent iteration.


1. There's an easier (and more efficient) way to do this: `created_at.to_f / 1.day`. You can replace `to_f` with `to_i` if you don't want the floating point.

2. I know you don't need the precision for this, but in the future if you want to get a Time instance for the actual unix epoch, use `Time.at(0)`.

```ruby
[222] pry(main)> Time.at(0).tv_sec
=> 0
[223] pry(main)> Time.new(1970, 1, 1).tv_sec
=> 21600   # oops!
```


Ja, ja, es wird fertig sein. Irgendwann mal...

2016-01-21 21:25 GMT+01:00 Ben Schumacher <notifications@github.com>:

> Am Sonntag wird der Code für die Codereview festgelegt, also bis dahin
> soll das fertig sein
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hu-semesterprojekt-genombrowser/Alazen-Integration/commit/7d6168086875a7300578ec8c347dff8642c4204a#commitcomment-15596692>
> .
>


-x-x-x-x-x-


Something odd I've recently updated to ubuntu alpha, and have errors regarding the fact that is asking for aclocal-1.14, and when i see you've updated to automake 1.15 not sure why is asking for 1.14. Did you encounter in the past any issue regarding this? Thanks


Don't forget to do a pull request next time. It's good for everyone to follow along and keep updated even if it might not be relevant for the dev.


vn


@Lordnoname Also the type `seperator` doesn't exist. Please check that too


@mrocklin @electronwill please take a look. (I already fixed the typo in the first paragraph, can -> can't)


woot?


Dont need an err2 here, can just reassign to err


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: Build Failed

 * 545 tests had: AUD diffs.
 * 475 tests had: RDD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 2 tests had: Table big diffs.
 * 3 tests had: MTD diffs.

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


What happens if key and keys are provided?


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


That's pretty much the way it is today (under different names).  I'd like to get this node-diving logic in one place, and the nullptr checks are not a performance concern here.  Nor can one overload be implemented in terms of the other without introducing a significant performance hit (e.g. tossing in a dummy stack).


nice one!


111639676-UniqueThermostats (kbenne) - i386-Windows-7-VisualStudio-12: OK (2348 of 2384 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-98.49%25-yellow.svg)


Presumably, you would want to do `return errs, noClient` here, so you're returning the wrong error, but also: don't call the error noClient, just call it err.

Generally, you should be able to get away with a single err variable in this method.


This is not correct; the `is_a?(Integer)` check should come before the comparison, for it will fail if the type is not numeric.


Do we want to show outputs here as well?


I can't understand you clearly , even I am a Chinese too. @MCSou


Yeah, the problem there is that those stacks are radically different than the two I just merged.  I.e. we can't merge that one in.  I'm not saying it can't be improved.  But I haven't figured out the improvement for that one yet.


While it goes through QA yes, there's no other way. I think we should start treating api/sdk tickets like our normal 'app' tickets and just make a release ticket when we have a handful of changes we want to push out. At that point we can just change the podfiles.


Nope, thanks, will fix.


closes #518. I meant to do a pull request, but I accidentally pushed to master. Well, nothing radical changed so....


Ty radosny człowieku z mały aplikacjami.


Also, if the monkey patching becomes too much, you might consider using a `Preloader` service class. Cf [this write-up](http://blog.codeclimate.com/blog/2012/10/17/7-ways-to-decompose-fat-activerecord-models/).


As far as I can tell, this doesn't work. I had to change it because I kept getting this error:

```
ts/models/metrics.ts(255,7): error TS2394: Overload signature is not compatible with function implementation.
```

There's probably some way to get it to work because the chain props work but it was just wasting time.


There are extra parens here.  Ideally you should actually run through all of the exercises you put up here to ensure that they're correct.

We might consider using the `ipython` sphinx directive, which actually runs the commands and inserts the outputs into the documentation.  This ensures that things are correct.  The pandas docs do this, you might want to take a look.

We would need data if we wanted to do this.  We might be able to put this on a public S3 bucket.

Just a thought; this would probably be a lot of work and it's not clear that we want to go down this path.


Presumably could pass nil here instead of NewListOpts?


I think that you all should focus on making sure that you can issue a proper PR before doing much more  documentation work.


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


Ok, I'll leave it as is. The rest is ready for review. Picking with batched geometry was fixed.


Look. When this sub is called from:
```
63 = $ Mojolicious::Plugin::EPLRenderer::_epl
(Mojolicious::Renderer=HASH(0x78666d8), SafeVPN::Site::Auth=HASH(0xce1a440), SCALAR(0x5ff6848), HASH(0xce44108))
 <--  /home/feelsafe/perl_lib/lib/perl5/Mojolicious/Plugin/EPRenderer.pm:50

62 = $ Mojolicious::Plugin::EPRenderer::__ANON__[/home/feelsafe/perl_lib/lib/perl5/Mojolicious/Plugin/EPRenderer.pm:51]
(Mojolicious::Renderer=HASH(0x78666d8), SafeVPN::Site::Auth=HASH(0xce1a440), SCALAR(0x5ff6848), HASH(0xce44108))
 <--  /home/feelsafe/public_html/lib/Mojolicious/Renderer.pm:243

61 = $ Mojolicious::Renderer::_render_template
(Mojolicious::Renderer=HASH(0x78666d8), SafeVPN::Site::Auth=HASH(0xce1a440), SCALAR(0x5ff6848), HASH(0xce44108))
 <--  /home/feelsafe/public_html/lib/Mojolicious/Renderer.pm:132
```

the template is rendered successfully:
```
2016/01/21 22:31:06 7191 DEBUG>    Plugin/EPLRenderer.pm:38    Mojolicious.Plugin.EPLRenderer
  /home/feelsafe/public_html/lib/Mojolicious/Plugin/EPLRenderer.pm  <--  Mojolicious::Plugin::EPLRen
derer::_epl

  Rendering template "auth/index.html.ep"
```

but sub does not return 1 at line Renderer.pm:243, it is returns undef at line Renderer.pm:248

This cause error:
```
2016/01/21 22:31:06 7191 DEBUG>    Plugin/EPLRenderer.pm:50    Mojolicious.Plugin.EPLRenderer
  /home/feelsafe/public_html/lib/Mojolicious/Plugin/EPLRenderer.pm  <--  Mojolicious::Plugin::EPLRen
derer::_epl

  Template "not_found.development.html.ep" not found

```

my perl:
This is perl 5, version 22, subversion 1 (v5.22.1) built for x86_64-linux-thread-multi


This doesn't seem right. stats_[client_id][radio_id] is a map of key, value pairs. max_element() will pick the max based on the key (based on my experiments). Just explicitly loop to find the max, e.g.,
auto max_p = make_pair<int, double>(-1, -1.0);
for (const auto &p : stats_[client_id][radio_id]) {
  if (p.second > max_p.second) {
    max_p = p;
  }
}
return max_p.first;

Also, initialize bs_id = -1 is safer as done above.


No, dammit, how did that get in there.


I wonder if we can have one more tool to gather till this part of the file and create the http-composed-client-code.h header automagically (and keep them in sync) :)


Do we need to update this name?


And so we're punting that fact to later in the chain to deal with?


Thanks.  I looked into it a bit more, and I still think function overloading is simplest here.  The default implementation plus template specialization plus static asserts wouldn't really achieve more and it would be a bit harder to understand IMO.  Keeping it as is.


These are done?


This will work correctly since ```typeof undefined``` is ```undefined```. I just used this because I know typescript understands ```typeof``` but I could also test for ```undefined``` instead. That would be an equivalent test.


Need to remove Spark Kernel.


@SidLeung yes if both are 0 that means that the baud rate is too fast and can lead to bad reading on the first byte.


`ReadOnly`


Will this line ever run?


Sorry. Will told me it was Ok to continue working on this branch.


yup, looks good


Cool, I'll uncomment so.

Sent from my iPhone

> On 21 Jan 2016, at 19:07, Corey Floyd <notifications@github.com> wrote:
>
> yeah this accidentally commented out - it should be uncommented
>
> —
> Reply to this email directly or view it on GitHub.
>


easier / cleaner to do this using ES6 destructuring:

    const { speakEasies } = this.props;


Again "I" reference. replace with something like "the term 'full-stack is a more exact description of the power of node"


Thanks! I'll include this change

Sent from my iPhone

> On 21 Jan 2016, at 17:15, Brian Gerstle <notifications@github.com> wrote:
>
> @barbaramartina it's not absolutely necessary to use it, but it's more robust and less error-prone than manually cleaning things up. This blog post gives an example, but here's how it could be used here:
>
> // Define a macro for ease of use (e.g. in NSObjectUtilities.h)
> #define WMFReleaseOnExit __attribute__((__cleanup__(CFRelease)))
>
> // .. use it to annotate variables that should be release when the scope exits
> - (BOOL)urlIsReachableOverCellularNetwork:(NSURL*)url {
>     SCNetworkReachabilityRef WMFReleaseOnExit reachabilityRef =
>         SCNetworkReachabilityCreateWithName(NULL, [[url host] UTF8String]);
>     // ... rest of method continues, no calls to CFRelease necessary
> }
> —
> Reply to this email directly or view it on GitHub.
>


Unfortunately nvd3 doesn't have typescript defs.


This I am not 100% sure. Because we only deprecated the class, however this binding is a name binding, which I don't think there is a way to deprecate it.


Why not `std::stack<t_lookup, std::vector<t_lookup> >` ?


Do we need to update this name?


I am able to access the docs here: http://apache.github.io/incubator-toree/latest/api/#package
We should probably keep these urls around.


good job


fixed


I'm finding it difficult to follow the assumptions being made here, especially since the data being used is so far away from these checks.  Is there a simpler way to validate whatever you're trying to validate?


You are fast!

Yes I am using Arch, and this works but only for default keyboard map, after I switch keyboard layout to e.g. german VC_KEYMAP=$(localectl status | grep "VC\ Keymap" | awk -F ': ' '{print $2}') still shows old kbd layout


I am assuming you are saving these until we have migrated the wiki to the new toree site?


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


Got it, done.
On Jan 21, 2016 2:36 PM, "Spencer Lyon" <notifications@github.com> wrote:

> I'll put it here so I don't forget:
>
> This is the UG page and you told them to submit the final project via
> email with the subject saying "bootcamp project mba" when you really
> should say "bootcamp project ug"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DaveBackus/Data_Bootcamp/commit/3a4873fac0ecf52c2ffff1c6c6bcb73c6a5bafd3#commitcomment-15595264>
> .
>


Should we deprecate our gitter?


Okay. How's this? I've created a shared userstructs.h, put struct _uzistats in it, and fixed stcpy(). It's still five calls to uput, which may not make it much better than a member copy. But it does fix the issue while retaining binary compatibility.

Turns out there was a struct stat in the kernel headers, which nobody was using, that was almost a copy of struct _uzistat. But only almost...


Yeah, my comment this morning was to keep working here for now, knowing that I could easily move these changes to a new branch if needed. However, I've only had a fraction of my time available for dask the last few days, and today my schedule has filled up with Bokeh and replatform work. Geri, can I move dask entirely to you for now, including the git branch rebase, and join back up when I get time, probably after Matt's vacation has started?


cool


Just a suggestion.  I'm not sure how you all are handling things.  Will is more engaged here than I am.


The setProperties (which calls configure.setProperties) will do the copying.


why don't we need this? is it covered by some included header?

we do need a declaration somewhere, since line 208 uses it


Discussed with @anew we better keep it for consistency and remove all of the `useDatasets` in all configurer in one shot.


I'm not sure if this is an anti-pattern or not. I thought react was supposed to be smart and knew by itself when to re-render?


oops, removed


OK


So the travis work would have to be done independent... we first need to get access to the apache travis account.


Ok, it's been decided we'll remove the GPU range check altogether and just ignore bad indices.


Ah, that does clarify it! Thank you! Funny how that tiny change makes it all clear.


We do not check that. This operation is `sync` operation and always successful


This comment has been updated with 39cffff, which mentions that the `404` also carries a code of `ENOCUSTOMER`, which is what is caught by the `catch` below.

Otherwise, the errors are handled down the chain to be sent out to the user. In the situation where a user is a customer and/or they have a license for this specific organization, the user cannot access this page, as this page is only for people who are trying to restart their account via restarting this canceled org.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 95](https://teamcity.labkey.org/viewLog.html?buildId=298501&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (2 new), passed: 275, muted: 1 Build time: 01:57:40
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <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.verifyLogGutterPlot: <no details avaliable>

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <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
```


Shouldn't you use `isDerivedValueClass` instead of `isAnyValSubtype`? Otherwise you'll end up with two identical defs for something like `def foo(x: Int): Unit`


wow, I somehow missed this. I was sure I had searched both the hpp and the cpp file. I'll get rid of this commit then.


how about you create an exception class of your own, and use e.what()


Or probably an int? As it is used in CompoundTag::putShort, and shorts can be convertted into integers


They are used often in trans and typeck to continue after an error


Not now


use $pkgname and $version


missed this one... thanks


Yep. Then the data of the file that one is renaming could even be updated. In fact, the `updateFileElement()` should probably also set the fields in lines 51-53.


this is not how you do it. catch only goes with try


Later you may want to import your styles (things will get messy quick, plus can reuse styles)


LGTM


are you sure this compiles???


Yeah... I mention that they are there for the interim


travis work later


deleted those lines


You might consider a helper:
```C#
private static void Add(this ListDictionary ld, bool addViaSet, object key, object value)
{
    if (addViaSet) ld[key] = value;
    else ld.Add(key, value);
}
```
so that you can then just write these as:
```C#
add.Add(addViaSet, "key", "value");
add.Add(addViaSet, "key".GetHashCode(), "hashcode".GetHashCode());
```
rather than duplicating the code everywhere.


don't want to create a new gitter just yet


Failed tests (haven't changed from Perl 5.22):

Failed 5 tests out of 2224, 99.78% okay.
        ../ext/re/t/re_funcs_u.t
        ../lib/locale.t
        re/subst.t
        re/substT.t
        re/subst_wamp.t


I don't think this link is getting used... will delete


Hi,
This modification was not a good idea for jdk 7 because the ThreadLocalRandom had bugs
http://bugs.java.com/bugdatabase/view_bug.do?bug_id=7051516
JDK-7051516 : ThreadLocalRandom seed is never initialized so all instances generate the same sequence

May you need to check the jdk version 7 or 8 before choose the ThreadLocalRandom or not ?

Regards.
Vincent D.


Oops... yeah, I see where it is getting used... will update


Sure. Are we going to add something else to the documentation? I was under the impression that the quickstarts were the last thing in the list.


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


I believe you can print more error from the system. maybe errno?


Line 58: "your the".
Line 62 "the" to "they".
Line 82 "the your" to "your".
Line 129 "second discussed" to "second-discussed".


I think the abstract will be there, as a guidline for what should be found tn that chapter/section.


yeah for sure! I still have my store in my main file. Thanks for the tip


@Lull3rSkat3r One more time please... and then I'll merge


If needed. The caller has this.

I just object to a commit that claims to remove lookups actually adding more.


You can quickly eliminate the need for it to always re-render with:

```javascript
if (nextProps.speakEasies != this.props.speakEasies) {
  // Set the source
}
```


I checked again what this does now. These links are not to the comments, but to the donation form. Example:

    https://spenden.wikimedia.de/spenden/list.php?piwik_campaign=Spendenseite&piwik_kwd=spenden-rss#1833883

The last number being the donation ID. Is it used by piwik? And if so, does that really provide value?

The link in the RSS channel itself (line 5) appears to not do much though. Perhaps we should omit that one?


112158233_PVwithStorageSystem (Nigusse) - 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)


@msvalina thanks for your testing! I will try to figure out why that's the case.


LGTM


trying git


:+1: thanks!  that bugged me, can we change it to https://pbs.twimg.com/profile_images/688952326573264896/JWBuqs8N_400x400.jpg?  just to keep it inline with the other profiles


oops https://github.com/discourse/discourse/commit/9569235d76c9f05122beff31369045200db9e854


> patacrep.tex hérite de default.tex, donc c'est le template le plus complet (et ça me parait logique de l'utiliser comme défaut).

Il me semble que l'idée était que :
- `default` est un template fonctionnel mais simple, sans fioritures. Il peut servir de base à des choses plus complexes.
- `patacrep` est un exemple de ces choses plus complexes. Il y a notamment des choix esthétiques qui sont faits (comme la couleur).


asdfasdf


I also don't know of anything on the list to be added other than quickstarts.


You can, but generally it would be better to use parameters to distinguish different terrain calls if they're very close. In this case they're identical so there's no reason to have multiple classes at all.


(this comment is pretty much just on an arbitrary commit which changes images)

Some steps should probably be taken to make sure we're not including larger images than needed, for the sake of more efficient transfers:
 * images that are bigger than any place they're used should probably be downsized, though it may be worth testing them on a HiDPI display -- in any case they should never be larger than twice their largest-used spot. this can also solve blurriness issues, if we're having any, since many browsers downscale images in ways which are more efficient than they are pretty
 * with a previous version of the images I ran some lossless .png optimization - ping me at some point once the images are otherwise ready and I can either show you what I did or just re-do it with the new images


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


Add bs_ids and client_ids as data members of WspaceController. Then you can just pass client_ids to RoutingTable::UpdateRoutes instead of Tun. Generally, you want to minimize dependencies across modules.


I believe this one was okay (r=a+b). I assume you changed it to be consistent with the others?


Nice! Then should be safe to add that to `defaultOptions` now?


Nit picky but add indentation to props, eg:

```javascript
<ListView
  dataSource={this.state.dataSource}
/>
```


Edit: I know they *were* different but I wanted them to be all the same. Basically their job now is to "insulate" the oceans and the land from each other and cut down on the extent of repairing we need to do. Where the beaches come out now is really more a terrain feature of RTG's generation.


Can we just append the 400x400 at the end of the file name?

On Thu, Jan 21, 2016 at 10:55 AM, Russell Vea <notifications@github.com>
wrote:

> [image: :+1:] thanks! that bugged me, can we change it to
> https://pbs.twimg.com/profile_images/688952326573264896/JWBuqs8N_400x400.jpg?
> just to keep it inline with the other profiles
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodeforHawaii/codeforhawaii.github.io/commit/9b2cb01dde86487f6b3b97825f74d3da62a35280#commitcomment-15597457>
> .
>

--
Burt Lum
C: 808-295-5115
O: 808-525-6409
Twitter: @Bytemarks <http://twitter.com/bytemarks>


@AndyScherzinger
This gives me a NPE when I share a file from Gallery to owncloudBeta.

(I have not yet created a new beta as this is blocking the release)


We could cite at least the things we provide here, like the hbridge-l293d node. The rest looks OK to me.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 94](https://teamcity.labkey.org/viewLog.html?buildId=298529&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:26:10


Hey Mike,

I've already seen your stuff yesterday. All is good ;)

Looked into the whole using `background declaration in CSS to display images` VS `using <img> tag`.

http://stackoverflow.com/questions/492809/when-to-use-img-vs-css-background-image
The reason I was attempting to use a `<div>` then hook up the image file using css is in that post.

and the image wasn't being displayed correctly because it was missing `background-size: 100%` in the css.

Stephen


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


I tried this, but in the end, it resulted in more verbatum code, IMO.  Since I would create a separate method for each call to assert, pass through each parameter needed (`block_structure_result`, `i`, `has_access_result`, etc), with docstrings for each method.


Technically it's appending `_400x400.jpg` to the end of the filename, but yes.  Do you want to do the honors?


*This is a constructed data set containing made up cases and all relevant variables used in the analysis.*

The data cannot be used to reproduce the actual findings presented in the presented research.


As discussed, we chose to be consistent with the other All-Caps class globals in the same test files.


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


OH I was wondering about that!


if bs_id < 0, don't populate route.


Error messages should go to stderr (`os.Stderr`), not stdout.
They should also be new-line terminated.
On failure programs should exit with a non-zero error code (e.g. `os.Exit(1)` instead of `return`).

To make this easier, I find it convenient to reconfigure the default `*log.Logger` instance at the start of `main`:

    log.SetPrefix("add_torrent: ")
    log.SetFlags(0)

and then just use `log.Printf` and/or `log.Fatal`. E.g. with the above you could just do:

    if *delugePassword == "" {
        log.Fatal("missing required -delugePassword argument")
    }

and if triggered the program would write the following to stderr and exit with code 1:

    add_torrent: missing required -delugePassword argument


Remove this condition check here as it's handled by UpdateRoutes. You want to minimize conditional check in your fast path.


Nit: we should probably replace "public dns server" with "RancherPublicDNS server"


@RobPosener Thanks for the review; I've updated the PR


This fixed the issues when trying to select any of the 'Updater' actions:

https://github.com/libretro/RetroArch/commit/076e55a945feec8486f9520e78f42f662b549bf0


:cry:


@Voyager1 first of all need to solve issues without dxva. when we can continue to fix dxva. ok? A lot of reposts from you broke my mind at all.


Je pense que les choix par défaut doit être le plus complet en terme de fonctionnalité (facile de changer la couleur sans connaitre LaTeX par exemple).

Les utilisateurs avancés qui voudront faire un template personnalisé font naturellement creuser un peu tout ça et utiliser le template de base qui leur conviendra le mieux.

En fait, j'ai peur que si `default.tex` est le défaut, les utilisateurs "novices" n'osent pas trop le changer pour `patacrep.tex` (alors qu'il est indiqué pour ce public je pense)


Ready for promotion review


is there a reason why we are setting root domain here? I thought we already set it from some other place.


why not add the dependency you shared before?


This is an incorrect use of `defer`. None of these files gets closed until `main` returns. Either don't use `defer` here and make sure to close `file.Close` correctly, or put this functionality into a function that is called from main's loop.


Yes, I just did but left it as .png. Does it need to be .jpg?

On Thu, Jan 21, 2016 at 11:00 AM, Russell Vea <notifications@github.com>
wrote:

> Technically it's appending _400x400.jpg to the end of the filename, but
> yes. Do you want to do the honors?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodeforHawaii/codeforhawaii.github.io/commit/9b2cb01dde86487f6b3b97825f74d3da62a35280#commitcomment-15597601>
> .
>

--
Burt Lum
C: 808-295-5115
O: 808-525-6409
Twitter: @Bytemarks <http://twitter.com/bytemarks>


minor thing, we should remove


@Jeronics good use of hidden, well done. Take a look at my commit. I've optimized your code a bit:
 * more elegant construction of hidden element based on existing button
 * proper way to extract name via .prop('name') function of jquery
 * no need to create a new form
 * no excessive $ calls, it's enough to do it once and reuse the variable.
 * $($()) is of course an overkill. so is $(createElement)... because $() does it already


Yeah, could be. I didn't write this code, just refactored it.


Yep, that sounds right!


We could store these tables as pretty-printed JSON. It'd be much easier to pick out changes in the diff output.


Add Nitpick - Add an Init function in wspace_controller that wraps the two initialization steps here. You don't want to expose data members unless necessary.


This won't apply against https://github.com/kodi-pvr/pvr.mediaportal.tvserver/blob/master/CMakeLists.txt - is your fork up to date?


Hey @msvalina. How did you change the keyboard map? If possible, please post the command.

There are a lot of ways how to do it.

Examples:

In console, you can set the keyboard layout ("keymap") to colemak with `loadkeys colemak` or permanently with `localectl set-keymap colemak`.

In X11, you can set it with `setxkbmap -layout colemak`or permanently by adding that line to your .xinitrc file. Apparently, you can also do `localectl set-x11-keymap colemak`.


How about using `format`? Its the recommended way going forward.


I'd restructure this as:

```
Optional<PartitionFunctionBinding> partitioning = node.getPartitioning();

if (partitioning.isEmpty() && redistributeWrites) {
   partitioning = new PartitionFunctionBinding(FIXED_RANDOM_DISTRIBUTION, ....);
}

if (partitioning.isEmpty()) {
   source = withDerivedProperties(...);
}

return rebaseAndDerive(....);
```


jpg's would be better actually, since they have a smaller filesize (png's are really more for screenshots or stuff where you need to read things)


What is quadplane? Can you explian a bit more? Thanks!


nit: This seems like an expensive way to code a switch statement. This data structure takes up 1K of data (maybe only 1/2 K on a 32-bit machine).


Another nit :) lets be consistent int the log when it comes to provider naming. It should either be "RancherPublicDNS" or "Rancher Public DNS" (with spaces)


If you want to do this kind of thing it's much better to make this a [variadic function](https://golang.org/ref/spec#Function_types):

    func (d *Deluge) sendJSONRequest(methd string, params ...interface{}) (map[string]interface{}, error) {

Then the calls become much cleaner, e.g.:

        response, err := d.sendJSONRequest("core.add_torrent_url", torrentURL, options)


I dug a bit further into this today.

I guess my confusion is that in the starwars example (and other examples) GraphType classes are singleton classes that just describe the mapping between the business object and the data-transfer-object. So I assumed that was the idea in general. But now I start to understand that there is another way by using the GraphTypes as data objects.

I thought the convention-based approach would have been made by having e.g. a ConvensionObjectGraphType<TData> (singleton-per-TData) and have TData be a business logic following the conversions.

Is it the intension to keep both ways supported in the future? Or is this a migration towards the GraphType objects carry data based approach?

BTW: We made our own version of the Relay Node interface work today, so that part is not a concerned for me for now. In doing this, I noted that the string-unquoting done in StringGraphType seem to be missing from the IdGraphType. Or perhaps both should have been done in an earlier stage (didn't dig into the parsing of the query, so I am not sure about that).


I made a pull request to address the issue


trying to get my comments back in config file


this is every pythonesque iteration :). it could be unsafe for arrays. the way to do this is:
```Object.keys(schema.properties).forEach```


Ughhh. I'd rather not have it in the stdlib, that pulling in an external c lib as a dependency for building the stdlib.


case 'S' and 's' share a large piece of code. Factor it out into a private function, e.g.,
void ParseIP(const vector<int> &ids, unordered_map<int, char*> &ip_table). Then you can just pass the right id array and ip_table according to the flag.


Do you think it would make sense to test connection for all providers that are registered in external-dns, on a startup and healthcheck only? So we don't have to repeat the same check again if one provider calls another one.


@greut ça me semble bien, à voir si les étudiants ont l'opportunité de s'amuser avec ça ;-)


should probably extract this to a constant object in case we need to change it


It looks like the assignment here will override any value set above in .travis.yaml

Perhaps you want something like:

TRAVIS_CACHE_REPO=${TRAVIS_CACHE_REPO:=git@github.com:nimbis/travis-cache}

?


Cool, well I'm fine with them all being the same functionality-wise - it's really just a convention/consistency thing for me... each realistic biome has its own realistic biome class, its own terrain class, and its own surface class... just thinking that extending the terrain class we want to 'replicate' would allow us to have the best of both worlds (avoid code duplication whilst keeping each biome its own little island). I'm easy either way though so if you want to change it back, I'll just have to up my OCD medication ;)


this doesn't remove the settings once coverage is disabled. I'm probably ok with that because I wouldn't tend to do coverage runs in interactive mode anyway.


to avoid false positive concurrency with I-96


this would be `isReadableStream` if we kept it (we should use the module)


Wouldn't it be better to call daemon() ?
http://linux.die.net/man/3/daemon


Sorry, forgot this time. Will do!


what does this trick/hack imply?


@kiljacken If we're wrapping around C, I agree on that point. We can just exclude the crypto stuff for now and worry about it in the future.


btw, there is `-=` and `--=` in recent versions of sbt :wink:


Plus you could've appended that rather than cleaned out the entire paragraph because it's a draft document and that doesn't necessarily reflect what we've all agreed on.


[Here's](https://github.com/Cloudef/wlc/pull/46#issuecomment-166888106) another workaround, that fixes it before compiling.


or take your medicine ;)


Initialize bs_id here. Then directly populate bs_tbl_[bs_id] instead using info. If bs_id doesn't exist, the map will create a default value (i.e., info) for it.


New-Fan-Object (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 0 test warnings)

 * 549 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)


Can the partition filtering be done in  HdfsScanner::Prepare?


Remove blank line.


i need money


@DeadParrot @amirroth I am hijacking @DeadParrot's Octree branch for evaluating the EioMathDiff since it is showing some tiny eio math diffs.  I pointed the EnergyPlus branch to the right EnergyPlusDevSupport branch in this commit.  Don't worry though, once this is evaluated, this branch will be deleted.


> Je pense que les choix par défaut doit être le plus complet en terme de fonctionnalité

Justement : `patacrep` n'ajoute aucune *fonctionnalité* : seulement de l'esthétisme…

Mais ça me paraît mineur comme question :  je suis prêt à faire de `patacrep` le template par défaut (ou plutôt à renommer `default` en quelque chose comme `minimum`, et `patacrep` en `default`).


:+1: I'll mark as complete. Nice work.


why not do it here https://github.com/xbmc/xbmc/blob/master/xbmc/cores/VideoPlayer/VideoRenderers/WinRenderer.cpp#L322?


LOL that was only on CS. You don't need to do anything on IM :)


Yep. We need radio_id.


Ah.  Makes sense.  Carry on.


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


try to make sure your comments are clear and concise


Rewrite this function.
- increment RECONNECT_ATTEMPTS (also rename it using camel case)
- on line 100 (in the error else statement, just do the increment wait
- don't keep re-getting the client
- rename failedClientCreation to err
- if you reach the 60 second mark of incremental backoff, log.fatal() to die.


It has been there in CLIEngine (https://github.com/eslint/eslint/commit/29c6a802daf90efe613bb5bd099aa18fa8fba34a#diff-3daec82d5218d64d8f2fed4d21f8e6ccL100).  What I wasn't sure about was abstracting it out into a separate configuration file along with the rest, due to commonJS caching `require` calls.


Nvm, we don't support multiple providers registration today, so only public dns gets registered as a provider in main.


If I understand the code well enough this change would break too much. If the remote exec never returns this change "hangs" the local end and there is no timeout to hit because the underlying ssh protocol itself will still be working. The only time out will be when the php execution timeout is reached.

This little snippet will get you the results you are wanting without needing any changes to the extension.
```
    $exec_stream = fopen("ssh2.exec://whatever", "r");
    $contents = "";
    while(!feof($exec_stream)){
        $contents.=fgets($exec_stream);
    }
```


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 95](https://teamcity.labkey.org/viewLog.html?buildId=298534&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:25:02


I don't think we need to check packet size.


you can combine all 3 cards into one declaration


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


since this is a pointer it should look something like the conversion below with a nil check.  For reference, here is what the conversion looks like for SecurityContext.Privileged which is also a bool pointer

```
if in.Privileged != nil {
		out.Privileged = new(bool)
		*out.Privileged = *in.Privileged
	} else {
		out.Privileged = nil
	}
```


This code is for the settings page, so my edit makes it so that only instructors can see the "contributable" setting (make the gallery contributable).


remove unused code  please


Put the above two in one line, BSSStatsPkt* stat = ...;


TestCustomCommand (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

 * 3 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)


> renommer `default` en quelque chose comme `minimum`, et `patacrep` en `default`

Ca me conviendrait parfaitement : je le fais dans cette branche ?


ditto for the rest of the conversion/copy files


that was removed later.


Add an else for Perror just for safe.


Remove empty line.


Also removed unused fields of surveys from the editor, and tightened-up the constraints display area so survey questions take a little less vertical space.


why define the color twice? remove it where it's not needed


For me this is only effective if I build with a plain `make`.

It took me a while to reproduce any compile time errors since my usual command line (taken from .travis.yml)
> `make EXTRA_CCFLAGS="-DLUA_NUMBER_INTEGRAL -DBUILD_DATE='\"'$BUILD_DATE'\"'"`

somehow eclipses the additional gcc option. So would the CI check with travis...
No idea why the Makefile gets broken when `EXTRA_CCFLAG` is specified at the command line.


Possible bug here. If a is negative, the remainder may also be negative but we are converting that to an unsigned number.


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


112158233_PVwithStorageSystem (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 0 test warnings)

 * 9 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)


good call


вернул package.json в первоначальное состояние, измененное во время предыдущего коммита


How much data does this amount to, does it slow things down for you?


it's a little odd that you're closing this h5 tag on a new line since you've closed all of your other hx tags on the same line


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


Pre existing, I know, but I don't like that the return value differs from the others. Remainder could be a param.in.


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


It does take appreciably longer than the other tests, yes.  It's at about a second or two.


why create a span within the article? couldn't you just give the same class name to the article itself?


if I do this, I don't have to use `schema.properties.hasOwnProperty(prop)`, right?


We could be faster presumably if we don't format a bunch of strings but just multiply one string.  I'll do that.


very useful, thanks heaps!


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


I think I'll end up eliminating the source file instead.  CharEscaperBuilder, included in the Guava 15.0 JAR we already distribute, makes it trivial to implement a good-enough replacement.


Вижу, небыло условий при отмене.

А для чего вместо явных

Определение
setFileMode
setWindowTitle
setDirectory
Вызов

создавать константные строки и вызывать как

const QString &newDirectory = QFileDialog::getExistingDirectory(this, title, oldDirectory);

?


should not matter. LinuxRenderer call Update in configure


会出错.
```php
{"name":"PHP Warning","message":"simplexml_load_string(): Entity: line 5: parser error : Opening and ending tag mismatch: meta line 4 and head","code":2,"type":"yii\\base\\ErrorException","file":"/data/wwwroot/yii/vendor/aliyuncs/oss-sdk-php/src/OSS/Result/InitiateMultipartUploadResult.php","line":23,"stack-trace":["#0 {main}"]}
```


Not actually sure. I don't think we're going to be processing these
semantically, so the biggest string we can get is likely helpful.

-Clayton

On Wed, Jan 20, 2016 at 2:41 PM, Aliza Stone <notifications@github.com>
wrote:

> @ctsims <https://github.com/ctsims> Hm, just noticed the
> CommCareApplication_().getCurrentVersionString() method. Do you want all of
> the information that's in that, or just the version?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-odk/commit/226b781d0a146625472ccf2237eb515bb33a3698#commitcomment-15567485>
> .
>


I see no test of locks for py2.

Are there follow-on tests required in distributed?


I suggest "from_be_u64 returns the native representation of the 64-bit big-endian-encoded value |data|. */


the h2 here, along with the h5 in the footer should both  be wrapped in header tags


missing puncuation. should be:  'not found. Reinstalling" (or a comma or a semicolong :-))


Did you make a pull request?


Wow. == performs pattern matching when used within [[ ]]. I had no idea, but there it is, documented in the bash man page. Thanks for teaching me something.


This commit spams a lot warnings into debugscript and causes low FPS for many players. For example when a script uses a negative number in dxDrawText it will spam a lot warnings.


That's not a big deal, but if possible I prefer to keep original the same, for debugging purposes. The more the generated code seems alien compared to the original code, the more Haxe feels like some kind of dark magic doing evil things to your variables, which it does not actually.


`STRICT_ALIGNMENT` is only relevant for dealing with pointers. Normally you can't cast a `uint8_t *` to a `uint64_t*` unless the original pointer is aligned on a 64-bit boundary, except on architectures like Intel's where alignment isn't required. However, in this case, there's no pointers involved.


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


@tomgarcia Seems like it makes it so that only the instructor can see the contributable setting. How else is someone suppose to see the setting?


LGTM.