I don't think this is a good change. `DevelopmentStorage` and `ProductionStorage` are classes that mix together several mixin classes related to static files -- they're no more about comprehensive theming than they are about caching, concatenation, minification, or require.js optimization. As a result, they definitely belong under the `openedx/core` directory, but not under the `openedx/core/djangoapps/theming` directory.

As an aside, I don't actually know why we have a `core` directory under `openedx` -- what other subdirectories do we expect to have? But that's another question entirely, and not one that should be addressed with this theming feature.


yes, here's the underlying state machine: https://github.com/kubernetes/kubernetes/blob/master/contrib/mesos/pkg/proc/proc.go#L120


I will test something like this:
    wxButton* button_Close = new wxButton(buttonPanel, wxID_ABORT, wxGetTranslation(g_Cancel));

where g_Cancel is (defind in constant.h):
const wxString g_Cancel =
#if defined(__APPLE__)
wxEmptyString;
#else
"&Cancel";
#endif


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


This broke my use case of having nearly everything left blank. _(except: Xcode Project Directory, Configuration)_

e.g. I want all targets to run, but don't want to have to add the xcodeProjectFile

I think `&& !StringUtils.isEmpty(xcodeProjectFile)` should be `&& StringUtils.isEmpty(xcodeProjectFile)`

so that this behaviour happens when the target is blank and the project file is blank. Otherwise, you need one of these to get the default behaviour.


I completely agree, I found it rather /annoying/ to have to change that each time to be honest, but luckily it could be changed back in eclipse in one go :) I think that would be following the recommended Java standard as well, wouldn't it?


Oooo that is nice I will do that


Won't this cause NPE at line 73 if there is no such file?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 144](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=549&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:14


@jgarboza813 DWMR. Nice work here mate. Press ahead!


I see :-) BTW: the argument to have a guarded read here applies also to my comments above. All these callbacks are independently invoked of the sequentially maintained state machine?!


Please do, @samueloph!


@jgarboza813 DWMR. Very concise and clear work here. Impressive.


Thanks for your remark. I've corrected that part :)


should be visible to you now


Did you mean 71?


Yeah I can't figure out why


@jgarboza813 DWMR. :bug: :hammer:


@jkokosar, is this OK? Should lines 1 & 2 be on the same line?


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


#5174-Zone-Name-for-Condenser-Placement-in-DX-Coils-is-misusing-the-zone-name-as-a-node-name (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1973 of 1974 tests passed)

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


@alvare52 DWMR. Nice work here. Press ahead!!


Everything is ok.


Done.


Si aucune entrée "handlers", il faut l'appender au tableau des config monolog ($monologConfigs) >> gérer ce cas après le foreach, pas dans le else !


Can you add a PR for https://github.com/FireCARES/firecares-ansible that adds the libffi-dev dependency?


Wait, what happened o.o


But the issue is the tab is super big. I'll use "  " instead


This is becoming a paid feature as of v5 and will ship as a commercial package. With a purchase and install you have feature parity with previous releases. The rationale here is obviously to generate more income based on our work and that installations hosting multiple websites are good candidates.


@alvare52 We can cover the pinch and long press gestures on Monday (today). Talk soon.


May not be needed


Markdown Attached.
[Visualizations:SanFranciscoCrime.pdf](https://github.com/ScenicVerve/datasci_course_materials/files/47483/Visualizations.SanFranciscoCrime.pdf)


@Syphurith if we get the memory region, it could maybe be possible if its in the range of gspwn on system nand.
On 9.2 and lower a 3dsx using libkhax could be possible, but would not work on Emunand.


I meant

`(M) message;`

conversion for a null value


Input string: `<strike>This is <b>strikethrough</b> text</strike>`
Original html-minifier with `collapseWhitespace: true` option result:  `<strike>This is <b>strikethrough</b> text</strike>`
Our fork with `collapseWhitespace: true` and `tagsMaintainingWhitespaceOutside: []` options result: `<strike>This is<b>strikethrough</b>text</strike>`
Our fork with `collapseWhitespace: true` and `tagsMaintainingWhitespaceOutside: [‘b’]` options result: `<strike>This is <b>strikethrough</b> text</strike>`


nice catch. keeping me honest this morning :) I mistook this func for a binding callback. would rather see a call to `f.asRegisteredMaster.Do(...)` here instead of a mutex-guarded `Driver()`


'Standard data protocols' would be a better name to convey the meaning of this question.


added trace logging in https://github.com/areek/elasticsearch/commit/ac631f9c11cd965d2163984fbb4265aa20ac7a72


changed all the retry conditions to use retry + return in https://github.com/areek/elasticsearch/commit/ac631f9c11cd965d2163984fbb4265aa20ac7a72


@alvare52 DWMR. `return @"Nice work."`


same comment as for `LaunchTask` below


Both work actually. What do you guys think?


if the primary shard routing is null, we treat it as an inactive shard and schedule a retry (changed in https://github.com/areek/elasticsearch/commit/ac631f9c11cd965d2163984fbb4265aa20ac7a72)


fixed


No, it won't, I've checked. On which line do you see this cast? I'm on 75.


This seems like a weird convention, why not just `Option`?


#5174-Zone-Name-for-Condenser-Placement-in-DX-Coils-is-misusing-the-zone-name-as-a-node-name (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This also minimizes the amount of travel necessary to train the given attributes.


Typo in "articlehttps".


@HanimHank no problem! thanks again for the contribution!


@alvare52 DWMR. We can sort out how to get the color to only apply to the first comment when we talk on Monday.


rtgtrgttg


Looks good


ok so maybe `speed_base` and `speed_current` are better names?
I don't want to recompute the current speed based on the modifier if I can also just store the modified speed once every layer.


According to [this link](https://github.com/SpineEventEngine/core-java/blob/entity-storage-fix/core-java-sample/src/main/java/org/spine3/server/storage/filesystem/FsEntityStorage.java), at line 73

LGTM then, you can merge


feature detection (enabling the Asset component when it is available) would make more sense than version detection (btw, the code I linked you was using feature detection)


This is the markdown. Please review this version. Best regards.
http://rpubs.com/Scenic_Verve/131950


I don't like the Hungarian notation approach -- in my opinion, it makes the code harder to read. It can also lead to misleading variable names, especially when implemented consistently (and the Open edX codebase does not consistently implement Hungarian notation). For example, if I saw a variable named `path_theme`, I would assume that it was a theme for some other thing called `path`. `path_project` reads to me like a project for your path, `path_templates` are the templates for your path, and so on. In English, we generally have descriptor words precede nouns: "the red barn" sounds more natural than "the barn that is red". As a result, I'd like to follow that convention.

I agree with your point about "studio" vs "cms", although I'm concerned that the ship has already sailed on that naming issue, as well. We already use `paver devstack studio` to start up Studio, rather than `paver devstack cms`. I'll ask around within edX, and find out what other people think about "studio" vs "cms" for this situation.


This changes the default behavior when some specifies a `ReferenceGenome` to use the `mdtag` if it exists as opposed to using the `ReferenceGenome` first.

This means the user can never override the mdTag which is sometimes needed (orginally this was done a when BAMs had incorrect/incosistent mdtags.


Bunu kullanıcının elle girmesi değilde, sistemin kendisi bilmeli bence

```php
<?php
// bu server degerlerinden sistemin hangi host, hangi http şema ve hangi path'de calistigini bulabilirsin.

$_SERVER["REQUEST_SCHEME"];
$_SERVER["HTTP_HOST"];
$_SERVER["REQUEST_URI"]
```


読み込み、イベント、ソート処理に分けました


This makes sense to me. I believe @nedbat was the one who originally pushed for the abbreviation. Ned, are you OK with this change?


I think we should just have one main .js file. I am going to delete mapLayers.js and we can rename search.js.


loos good


Errrr, gs-rest is set to Java 1.5 in its POM. Is this normal? Should I change this?


Alright let's get our commits synced up before deleting and renaming


@tbaddade Schau mal, ob das so hinkommt.


Okay sounds good


I suggested this change to keep the HTML as minimal and logic-free as possible.  I also find it nicer to not have to modify HTML files when I'm writing JS code.


for this and all below...lose the padding on the right and left. no spaces needed.
([@public_topic])


`BASE_DIR` kullanmak yerine `media()` diye bir fonksiyon olsa ve argüman olarak `public/images/favicon.ico` alsa, biz url'i media içinde olustursak bence daha güzel olur. Zira developer her seferinde bunu eklemeyi unutabilir.


We may want to use all the ranges instead of just the last one (for Safari HLS)


Isn't this going to be confusing, considering that there are currently multiple different theming systems in the codebase? A function called `with_theme` could refer to a comprehensive theme, a Stanford theme, a microsite, even the `IS_EDX_DOMAIN` variable. I'd be fine with renaming `is_comp_theme` to `is_comprehensive_theme`, for additional clarity, but I think that the change you're suggesting reduces clarity.


gson and jackson2 do the same thing... Might refactor this to use jackson2


This seems like it could lead to an odd case of someone specifying `samtools` when the `path.getFileSystem(sc.hadoopConfiguration).getScheme != "file"`


_to_capture_an_object _using_a_mac_use_the_following_keys_on_your_keyboard
_shift_command_4
_then_use_the_crosshairs_of_your_cursor_to_capture_the_object_on_your_screen
_as_you_release_these_three_key_you_will_capture
_a_screen_shot_of_the_object
_this_object_is_an_image_object_and_each_object_is_unique


git remote /storage/extSdCard/android_kernel_samsung_millet-millet
#git push /storage/extSdCard/android_kernel_samsung_millet-millet


Yes NumCPU sounds good.


A possible fix has been merged to relx master. Can you give it a try and let me know if it works?

The issue is `cf` was introduced to solve issues experienced by users with terminals that couldn't support certain output, and when I didn't add it to the escript list of apps to include in relx it caused the issue you experienced (I believe, this is assuming the patch works for you).

Ultimately what needs to be done to keep this from happening again is I need to create relx tests like the ones I have for rebar3 (https://github.com/tsloughter/rebar3_tests) that actually run the escript and check the output, instead of just common test and eunit tests that call relx functions directly.


Either tab or 4 spaces is valid Python, I kinda like 4 spaces:
```ocaml
String.make 4 * num_indent ' '
```


Have you changed anything on the ReportTest.html?


I had updated the readme with descriptions of the configurations created and used by the plugin but I agree the expectation of many developers will be to use a configuration called `provided`. I'll rename it.


I know your advice, Is delete all test references that thing mean?

I know technology may only class and inherited class


Here's a perfect example of what I said in #258, right on line 1489

The keySet will never have null values, so switching the string literals to the left doesn't change anything besides making the code uglier (this is what we call Yoda Conditions: if(tall is the man) instead of if(the man is tall)... )

Again, if there are nulls in the keySet you have bigger problems elsewhere, but by making `equalsIgnoreCase` return false instead of blowing up on your face, if said problems exist, you are hiding them by treating the symptoms.

The philosophy this project follows may be one of being as solid and fail-proof as possible, but I strongly believe in treating data exclusively on data entry points. There was a time I would protect everything everywhere, but I eventually learned from bad (and insanity-inducing) experiences that data that was well processed on entry points enables you to create "null-check-free" code that is faster, reliable, and doesn't blow up on your face unless you didn't really treat data as you should. Conceptually it makes no sense to let invalid data enter the model. I will keep defending this point until you tell me you'd rather have NanoHTTPD run solidly with a hidden bug than randomly crash and expose said bug and that I should settle down.


Thanks for the information!


What if we renamed this to "loadInProgress" ?


Should you use `DefaultTransactionManagerLookup.class.getName()` instead of the string literal?


This is verging on to be quite a big method doing a few things. Could we split/break this up?


TeamCity Habitat :: Cruise [Build 295](http://build8ua1:8080/viewLog.html?buildId=750&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:12


Ok. Will open an issue, I think this is more than just a short look-see.


This is also @nedbat's code. However, I'm once again concerned that this name is less clear than the name you're replacing. `openedx.core.get_configuration` doesn't sound like a function name that should have anything to do with a theming system. The fact that it takes a `path_theme` argument makes it clearer, but not by much -- I would expect to be able to call that function without arguments.

Ned, what are your thoughts?


didn't like the groove? :trollface:


@Martii


@victorporof shouldn't this directory be 'webconsole' and not 'console'?


Agree. This was a rare occurence reported by gray log. I'm going to try and find more about the response and what would trigger the missing attribute.


That's probably wise; I want to leave the commented-out sections around as they're pretty small and will save us time when we try to turn this on (if we do).


This would be good for
 * a video tag unit test
 * automated QOE for loading time after seeking (maybe a 60 second video)


@bradenmacdonald Thanks for the feedback!

> What's the difference between "name" and "label" - i.e. why do we need to have those as separate properties?

The difference is that `name` is used to uniquely identify a vector, `label` is used for display purposes. That doesn't preclude merging them into a single property, but there might be use cases where it would make sense to have two properties (?). The JSInput version of this XBlock distinguishes between `name` and `label`, and it doesn't say in the requirements for the Vector Drawing epic that that should change (in fact, the requirements for the WYSIWYG editor specifically mention that there should be support for "setting a label and name for each vector"). That's why I didn't merge the properties.


ok


@bradenmacdonald

> Currently, it's impossible to remove a "check" using the WYSIWYG editor.

I can't reproduce this. L402 above creates a new `expectedResult` every time `saveChecks` is called, and L422 below replaces the existing check data with it, so outdated checks (in `this.settings.expected_result[vectorName]`) are discarded.


@bradenmacdonald That would be possible, but the current solution makes it a bit easier to control, e.g., `width` of these elements separately. Keeping it for now, also to stay consistent with the HTML for input controls defined in other places.

BTW, you didn't ask about this before :)


Yeah me neither. I got a logcat, but I can't make heads or tails of it. I've just removed it from my ROM for now until maybe temasek adds it lol


@bencaradocdavies  Note that this will only happen with alternative resource store implementations, not with the default filesystembased one. With a JDBC Resource Store, there is no Hidden property, so only in that case the linux standard will be assumed irrespective of platform.


Applied.


I like the idea behind this change. Before reading it, I didn't even realize that the `enable_comprehensive_theme` function could silently fail! However, I'd suggest a different change. Instead of renaming the function with a `try_` prefix to indicate the possibility of failure, it should actually raise an exception if it fails. That way, we don't have to worry about silent failures, and it becomes clearer when your theme is not properly configured.


Konsolen registrerar ett error på denna raden men det funkar, så om jag inte hittar en annan lösning kör jag på detta.


No


Nitpick: this returns a dict, not a list.


See XMLRoleServiceStore for answer


matrix[x][y] has to be changed by Adafruit_GFX drawLine(x,y,x,(y+1),ORANGE). Same for all matrix arrays.


I think linking to docs in an exception message is totally fine.


@halindrome
A nit.  The each() callback function has the signature `function (index, item)`.  There's another example of that usage at line 105:
```
    $.each(base.querySelectorAll("dfn"), function(i, item) {
        var $t = $(item) ;
```
https://github.com/w3c/aria/blob/master/common/script/resolveReferences.js#L105
Getting the item via $(this) is not obvious.


@halindrome
I don't get what this line, the second closest(), is doing (it may be because I haven't had enough coffee yet).  Under what condition does it result in a zero-length result?


Iptables_Semantics: theory Code_haskell                                                                                |
Timing Iptables_Semantics (8 threads, 127.760s elapsed time, 802.364s cpu time, 96.752s GC time, factor 6.28)
Finished Iptables_Semantics (0:03:39 elapsed time, 0:17:11 cpu time, factor 4.70)
Running TEST ...
TEST: theory IP_Address_Space_Examples_All_Small
TEST: theory IP_Address_Space_Examples_All_Large
Timing TEST (8 threads, 10186.275s elapsed time, 61838.512s cpu time, 1512.440s GC time, factor 6.07)
Finished TEST (2:49:57 elapsed time, 17:10:49 cpu time, factor 6.06)
Finished at Mon Nov 30 14:55:14 CET 2015
2:53:44 elapsed time, 17:28:36 cpu time, factor 6.03
../memusg ../Isabelle2015/bin/isabelle build -d . -v -c TEST  62743.41s user 173.69s system 603% cpu 2:53:44.08 total
memusg: peak=687824


This function isn't really needed, since the `docker.Client` is exported.


I'm not sure, I got a notification that I was mentioned, didn't look properly, thought it's a PR, and was then really confused why this "PR" was outdated =P" And now Vinna is probably thoroughly confused why we're having a discussion here xD


@podeje - setting the binary by desired capabilities is already supported by the remote server, just not honored by the local ruby code. Before this you had to specify it in Firefox:: Binary.path= before initializing the driver, which is not ideal. The only reason this is Marionette related is because Firefox v42 does not support it, 43 does, but not with all the features. So to test marionette right now I have to identify the non default version to run the tests against. I'll change the rest of the specs again when 44 is released.


Could put an `:if => :response?` condition on this (I think?) to stop doing too much processing on other types of `InfoRequestEvent`


будучи один раз выставлен disabled он никогда не будет снят


Not command, but action.


отслеживать надо изменение каждого поля


Could you change <groupId> to com.epam.olukash?


I would expect this function to check the existence of these paths before returning them, rather than returning a dictionary of paths that may or may not actually exist.


Hey man how did I make a mistake there?
Please fix before I can merge this into master..


@keszybz actually, it's not redundant. The one in mac_selinux_access_init() is redundant... But you are right, we should merge these checks into one, and make mac_selinux_access_init() return 0 or 1 or so depending if selinux is desired at all... I'll prep a patch for that in a later commit


How will this new 'migrate_options' be used to migrate on systems with 4k drives? Or does this simply move the issue outside of the RA and over to qemu/virsh?


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


probably, good catch.


Could you change "artifactId" to "module1"?


Only send request reverse geocoding to Apple when got location fix


Yes, deleting all Test.h and calls to it must help.
If I still have this project on my home computer, I'll upload tests tonight.

This project uses move semantics, lambdas, variadic templates, type_traits, multithreading, atomics an other fancy C++11/14 staff. If you are a student of first courses, might be a bit too hard to learn for a project.


Related issue that I came across when I applied the temporary patch with CSS: https://github.com/Patternslib/Patterns/issues/429


A heads-up, this error still occurs, and it's caused by the removal of this
```
			// Assume it's an image, could be an acceptance note etc but rare.
			$context['files']['images'][$theme_name][] = $context_data;
```

```
 Type of error: Undefined
8: Undefined index: images
File: C:/xampp/htdocs/smf21_clean/Sources/ManageLanguages.php
Line: 379

Type of error: General
2: Invalid argument supplied for foreach()
File: C:/xampp/htdocs/smf21_clean/Sources/ManageLanguages.php
Line: 379
```

We removed the attribution of images, but line 379 still tries to handle it...
```
	foreach ($context['files']['images'] as $k => $dummy)
		$indexes[] = $k;
```

I think that that piece of code is inconsistent but I'm not sure if it can be totally killed. Example: $indexes is used but never initialized. Instead, $theme_indexes is initialized, but it's never used.

IMO all of this can be trashed (lines 377 up to 440), as it all relates to images
```
	// So, I'm a perfectionist - let's get the theme names.
	$theme_indexes = array();
	foreach ($context['files']['images'] as $k => $dummy)
		$indexes[] = $k;

	$context['theme_names'] = array();
	if (!empty($indexes))
	{
		$value_data = array(
			'query' => array(),
			'params' => array(),
		);

		foreach ($indexes as $k => $index)
		{
			$value_data['query'][] = 'value LIKE {string:value_' . $k . '}';
			$value_data['params']['value_' . $k] = '%' . $index;
		}

		$request = $smcFunc['db_query']('', '
			SELECT id_theme, value
			FROM {db_prefix}themes
			WHERE id_member = {int:no_member}
				AND variable = {string:theme_dir}
				AND (' . implode(' OR ', $value_data['query']) . ')',
			array_merge($value_data['params'], array(
				'no_member' => 0,
				'theme_dir' => 'theme_dir',
				'index_compare_explode' => 'value LIKE \'%' . implode('\' OR value LIKE \'%', $indexes) . '\'',
			))
		);
		$themes = array();
		while ($row = $smcFunc['db_fetch_assoc']($request))
		{
			// Find the right one.
			foreach ($indexes as $index)
				if (strpos($row['value'], $index) !== false)
					$themes[$row['id_theme']] = $index;
		}
		$smcFunc['db_free_result']($request);

		if (!empty($themes))
		{
			// Now we have the id_theme we can get the pretty description.
			$request = $smcFunc['db_query']('', '
				SELECT id_theme, value
				FROM {db_prefix}themes
				WHERE id_member = {int:no_member}
					AND variable = {string:name}
					AND id_theme IN ({array_int:theme_list})',
				array(
					'theme_list' => array_keys($themes),
					'no_member' => 0,
					'name' => 'name',
				)
			);
			while ($row = $smcFunc['db_fetch_assoc']($request))
			{
				// Now we have it...
				$context['theme_names'][$themes[$row['id_theme']]] = $row['value'];
			}
			$smcFunc['db_free_result']($request);
		}
	}
```


Very nicely done with functions. Does it make it easier to follow if you use fewer variables?

        Stream.of(this::lookInJNDI, this::lookForStandaloneJBossJTA, this::lookForAtomikosJTA)
              .map(Supplier::get)
              .filter(Optional::isPresent)
              .map(Optional::get)
              .peek((transactionManager)->LOGGER.debug("Found tx manager '{0}'",
                                                       transactionManager.getClass().getName()))
              .findFirst()
              .orElseGet(() -> {
                  LOGGER.warn(JcrI18n.warnNoTxManagerFound);
                  return new LocalTransactionManager();
              });


This seems like a reasonable change to me. I won't even complain about the name: while `get_paths` is vague, the fact that the function lives under `openedx/core/djangoapps/theming` instantly clarifies the context.


You can add set migrate_options="--unsafe" to make it work with 4k drives.


Could you create package "com.epam.olukash" and move code there?


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


Unnecessary semicolon ';'


Ah, OK, thanks.


sol_config.h (``KCONFIG_AUTOHEADER``) and .config (``KCONFIG_CONFIG``) are in that value, and I'm afraid they are *not* meant to be installed. I suggest you expand ``PRE_GEN`` by hand excluding those.


That's 2 different points:
- dependency on Cassandra: this can be easily changed, as this used only for Java8 collectors. We should move Java8 utils from james-backends-cassandra to something other, like server/container/util-java8
- dependency on ElasticSearch: this is needed for now to implement Mailbox Message to JMAP Message. But this will maybe be removed or reworked when handling attachments in JMAP Messages.


I like things to be the same.... that is a lot less confusing, so this particular repo will follow the dataverse branching strategy I think.

That said, I would not like going to a single development branch. Rather I prefer to see a branching of from the unfrozen release branch into a feature branch for a particular issue; then apply a change; and PR that back to the unfrozen release. Then delete the feature. Then it stays clean.

________________________________
From: Philip Durbin <notifications@github.com>
Sent: Monday, November 30, 2015 4:26 PM
To: IQSS/dataverse-puppet
Cc: Lucien van Wouw
Subject: Re: [dataverse-puppet] Improve the docs (70a58ec)

@lwo<https://github.com/lwo> you shouldn't feel compelled to use http://guides.dataverse.org/en/latest/developers/branching-strategy.html if it's a poor fit for a non-Java project like this one. :)

I've considered switching back to having a "develop" branch, for example, but I haven't added this back for reasons outlined at IQSS/dataverse#2478 (comment)<https://github.com/IQSS/dataverse/pull/2478#issuecomment-144436722>

Anyway, the choice is yours, of course. :)

-
Reply to this email directly or view it on GitHub<https://github.com/IQSS/dataverse-puppet/commit/70a58ec5ff644512726dfba37f00927de04e4975#commitcomment-14684541>.


Added ['$scope','...otherdeps', function(){}]); synthax for safe minification


Seems like a good change to me.


I figured it was better than just initializing the driver with the parameter and expecting that it didn't error. (In general I try to avoid/fix those in my company's code). There is probably another way to test that the specified profile is being used, but that is likely more trouble than it is worth since marionette doesn't support it.


not really sure I understand the question.
If it's a general question, the answer is yes, the JDK 8 way of processing collections/maps is far superior to anything else, with the exception of the lack of checked exception support, which is highly annoying.


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


NSLocalizedString()

Also, can we please not set the message to a random string and set it to nil here instead.


これはアクションを追加しているのではなくコントローラーを作ったためにこのようになっています。
ここはネストからは外し、単純に関連するコントローラー配下にコントローラーを作成する。ということをしている部分になります！
アクションに変えたので消します。


может стоит для записи тоже использовать cookies.js ? :))


> applications will kept that
applications will keep that information


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


At UTC


I think we should not introduce the elasticsearch dependency at all, we just need to extract the mime/message handling stuff, don't we ?


@hartmannaf Oh. According to profi200 [here](http://4dsdev.org/thread.php?pid=803#803) so it is not likely to be that easy for a cheated update..


nit: it's not necessary to include the brackets and you can just use `{ _ in ` instead.


The `Symfony\Component\Asset\Package` class has been added in Symfony 2.7 already and the tests errored only with 3.0, so I didn't see a need for this fix being applied when testing against previous versions. But I'm relying on your experience, done in commit 8caf040d9df2b782c81048848e6c1bcb369ef107.


> pictures can also be

pictures can be
(too many Also)


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


c'mon! merge it! you know it's good.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7233](https://teamcity.mesosphere.io/viewLog.html?buildId=82662&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:12


strictier rules => stricter rules


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7234](https://teamcity.mesosphere.io/viewLog.html?buildId=82663&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:09


[JENKINS-31458](https://issues.jenkins-ci.org/browse/JENKINS-31458) and cf. https://github.com/jenkinsci/jenkins/pull/1888.


Yay, updates to the client building doc! Hurrah!

> Should we make the use of `` or :: + tab consistent?

Not quite sure what you're referring to-- `` is for inline code and command examples, :: is for examples that span a line or more.

@jnweiger


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


Well, I was wondering if the code removing some of the `Supplier<Optional<TransactionManager>>` variables would make the code more readable, as in my suggestion. But I didn't try to compile my code, and the signatures might not infer that a supplier is needed, and so I'm not convinced you can easily get rid of the variables without making other more complicated changes to the private method signatures. IOW, feel free to ignore.


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


OAUTH code type authentication is requiring a client_secret, which may lead to some issues, if we have it in our application. For example, people can use it to act like coyim asking authentication on google. Not sure if adium is handling this on a server, or how did they treat it.


"Program" is not really needed here.


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


the current code deletes all stock reports/transactions associated with them. i wasn't sure what to do yet so was just leaving it open-ended for now.


Yes, should be working fine in 9.0v8. Can you describe the issue/error you're seeing?
You might have to re-make your hiero Export TaskPreset from scratch to get them to work in Nuke Studio...


> Both works. we can have the client-build inside or outside the source tree.

It is best to show just one way. Keep it simple, and list all the steps in order. I'll try this out later today, thanks!


ah, not sure. it shouldn't technically affect the case model at all. does that mean no?


Tag done


Inser att jag läste fel på hur många Renderer som skapades i koden, men busy-wait problemet finns fortfarande kvar.


When you say "the problem" are you referring to technical unemployment?


Yes, but I'm not sure we'll keep it as it, or rewrite it from scratch, depending on attachments handling (for example recursive mime is not handled in elasticsearch, but is required in JMAP spec).


I seem to remember executable files should not be marked as writable:  should we set the permissions to `555` (read-write for owner, group, and others)?


I hope I don't get too many incidents on errors created by merging the "commander" branch. =)


... but you just removed the UI for managing the hostnames? The datatype and code for resolving the correct website and SitemapNode based on hostname is all still there.

And its not just multi-website installation that use it; basically all C1 installations i've encountered uses IUrlConfiguration to

 - set a 404 page,
 - if you have multiple hostnames towards one website it makes C1 redirect to the primary hostname
 - its the only way to query for the hostname for ones website when you're not in a HttpContext

You could at least just limit the number of IUrlConfiguration instances to one.


The last merged pull request is from September 23, so yeah... I've lost hope to get my own 4 PRs merged x)


This test seems to be causing trouble for a new PR https://github.com/luvit/luv/pull/190
@tarruda, do you have any insight on this?


I think Maven emits a warning if we use `${basedir}` over `${project.basedir}`; can you verify?


-s


At this point you should be able to register this instance for others to find:
```java
PentahoSystem.registerReference(
    new SingletonPentahoObjectReference.Builder< ISessionAwareAuthorizationPolicy >( ISessionAwareAuthorizationPolicy.class ).object( loader )
        .attributes(
            Collections.<String, Object>singletonMap( "ID", "nonTransactedAuthorizationPolicy" ) )
        .build() );
```


Okay @tanx @toberndo with this commit this unit test stuff should be definetly fixed and properly working.

here you can se a clear division of the build that need to succeed iin order to accept pull request and the one that may fail without impacting the global result:
https://travis-ci.org/openpgpjs/openpgpjs.

given how are testing is done we have also some information on the timing that the builds/runs takes with a per browser/precision.


Then copy/paste it, it's less work than extraction if we are going to throw the code away soon.


sounds good


This commit is being reviewed as a part of review issue #140.


well, this is because the BC layer in 2.7 forces the assets config to be always set.

In your case, the testsuite checking the version is fine only because you always force installing the fullstack repo, and so avoid having mismatching version of http-kernel vs something else.


aha thanks for the doc string


This is documented limitation of the current collectible assemblies implementations https://msdn.microsoft.com/library/dd554932(v=vs.100).aspx . I hope it will be fixed at some point ... there is no fundamental limitation preventing it.


LGTM.


Be sure to use `Objects::notNull` for a non-null filtering predicate. Very handy.

As a suggestion, when you find yourself defining longer lambdas like this, you may consider turning the lambda into a private method.

    private Optional<TransactionManager> lookInJNDI() {
        return JNDI_BINDINGS.stream().map(this::lookForJNDIName).filter(Objects::notNull).findFirst();
    }

    private TransactionManager lookForJNDIName( String jndiName ) {
        InitialContext context = null;
        try {
            context = new InitialContext();
        } catch (NamingException e) {
            LOGGER.debug(e, "Cannot create initial JNDI context");
            return null;
        }

        try {
            LOGGER.debug("Looking up transaction manager at: '{0}'", jndiName);
            Object obj = context.lookup(jndiName);
            if (obj instanceof TransactionManager) {
                return (TransactionManager)obj;
            }
            LOGGER.debug("Transaction manager not found at: '{0}'", jndiName);
            return null;
        } catch (NamingException e) {
            LOGGER.debug(e, "Failed to lookup '{0}' in JNDI", jndiName);
            return null;
        } finally {
            try {
                context.close();
            } catch (NamingException e) {
                LOGGER.debug(e, "Cannot close JNDI context");
            }
        }
    }


Just curious - what was the rationale for removing DELTADUMPS? Reduced db complexity?


Immutable implies thread-safe, but being explicit is fine.


Meanwhile nobody ask for this is just right. But keep in mind.


забыли добавить  zapret_log.conf, без него не работает


Can't we make this the default and not have to change all of the code.


Это не нужно, обо всех событиях оповещает NotificationService


Не нужно


P3: This is the first time I encounter a Maven build extension.  Does the `jdeb` plugin need this?  It doesn't seem like it would need to alter the lifecycle if it participates in the `package` phase.


To answer this as a result from our hangout: Yes, all those callbacks are synchronized via `ha.go#`


@vlj Does this improve Terraria?


Thank you for not using "dummy" in the name. That would have been dumb. :-)


Его незачем сетапить


return 0 missing?


I just noticed that this made it through. I'm taking it off now as it's way too dangerous. If anyone with a production-like unittests.cfg file runs this they would nuke production without even knowing it.


Both good points!


TeamCity Selenium POM :: CI [Build 47](https://teamcity.sponte.uk/viewLog.html?buildId=940&buildTypeId=SeleniumPom_Ci) is now running


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


There needs to be a space in beteen the two ```>>``` because of pre-C++11 silliness. But otherwise yeah, that looks better.


This is breaking my code while using spatie/laravel-medialibrary. Used to work perfectly before this update.


TeamCity Selenium POM :: CI [Build 47](https://teamcity.sponte.uk/viewLog.html?buildId=940&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:33


Require.Positive


setJSBundleLoader not work


First work on the server side part: https://github.com/geonetwork/core-geonetwork/commit/87ce78376ae54f866640ca0b0228dcea58b09921

The update user method can manage a single email or severals email separated by "|".
The ui could be updated to add the multiple emails management.


Also, we could use `.empty` rather than comparing to an empty string. It would be a bit neater.


LGTM


Wouldn't `client` leak here, as `destroy_client` doesn't free it?


Не нужно, это на фронтенде пусть будет.


LGTM. But I figure we could also merge these four lines into two, right?


Может быть empty


@mhils Do you recall the specifics here? I'm encountering an error from mitmproxy where it complains about `https` being an invalid scheme


@borshop: retry


I didn't test with Terraria. Does that fix it for you ?
It's a render texture related thing and not a texture related thing so it's maybe not the case...


Аттач может быть не один


How is this commit/NPI bandaid associated with this branch? I'm not touching this file


Require.NotNull


setJSBundleLoader no method
android gradle compile version 0.16.0


ok


I feel like the levels should also have been rewritten with fixed time 0-portals. The caveat of falling off the timeline should not be present in early tutorial levels. Although it is also reasonable to make new tutorial levels at some point.


Magic Number Alert!  Please move this to an appropriately named constant declared at the top of the file.


This commit is being reviewed as a part of review issue #140.


@Ignore :)


#5174-Zone-Name-for-Condenser-Placement-in-DX-Coils-is-misusing-the-zone-name-as-a-node-name (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (900 of 900 tests passed)

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


этот Where должен быть в репозитории, а не здесь. Если здесь, он подтянет всё в память и только потом выберет. Если в репозитории, то он это выберет запросом в БД.


Not really, but that seems unrelated. Please just open an issue, preferably using https://github.com/mitmproxy/mitmproxy/blob/master/CONTRIBUTING.md#bug-reports :-)


Yea probably smart, I had no desire to attempt to find out how jackson2 worked last night.


Still feels racy, if journald creates system.journal and then goes on an flushes so much stuff from /run into it, that an immediate rotation is triggered...


Напиши тест на то, что успешное добавление заказа прокидывает эвент, а не успешное (репозиторий вернул null, например), не прокидывает.


spacing on this method


-It's just a prank, bro.


Nothing ever passes false for this anywhere.


need this?


@Richard-Cao It's called `setJSBundleFile` and it takes a string (the file path) as argument.


This was a modification that actually was made in February of 2004, but never made it into the main branch. I would imagine that the maintainers at the time thought that modern systems could handle full database dumps better than they did originally - something that would be true more today. With future versions of fuzzball, you should be seeing the rise of non-textual databases as well - an improvement which could further remove the necessity of these deltas.

I'd be interested in hearing your perspective on this though - feel free to message me at wyld@fuzzball.org .


@MWisBest Where does it say that DDK1.12 is only for OMAP4470?

This workaround forces us to render in BGRA mode, which breaks Virtual Display rendering (used in for example TeamViewer & Chromecast).

I could be interested in trying to get DDK1.12 to run on 4460. But before doing that I would like to know where the original conclusion comes from.


This expression is not fun to read. I would use a known idiom with two helper functions:

    for (ent = begin(mb); ent != end(mb); /* inside */) {
        ....

Also, adding scalars to a `void *` is not permitted in ISO C++11.


this should be either returning those votes or called up_votes_count


Shouldn't you 'goto fail' on this check? It's the behaviour of every check nearby...


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


@Martii


@danielinux,

* Maybe an enum is indeed not that useful here, but I thought maybe it would be nice to return IEEE802.15.4 specific error codes to the 6LoWPAN-driver. But an integer should suffice.
* Most IEEE802.15.4 expect a 128-byte buffer containing the frame to be send on the network the network. The MAC frame is only 127-byte on the other hand and put on +1 offset in the buffer. The preceding byte is thus the final byte of the PHY header, the length byte. So the length can be retrieved from the first byte.
* I didn't want to allow more then 128-byte buffers, but since it's only a pointer, this isn't that useful?

If you have any more questions, let me know! :smiley:


yeah, I've had enough of that for a while :smile:


well then to make my comment no pointless and still bitch about something...
// _UNIT actually means creature. for some reason.
this comment, pls no


Well the commit description does not support markdown, but this does !

Before: ![before](https://i.imgur.com/LeFp2NI.png)

After: ![after](http://i.imgur.com/idgkiPZ.png)


@Dinesh-Ramakrishnan
No, it should not work. You should use a log file instead.


I know, but how to use custom `JSBundleLoader`?


Why have you added that


So what is not clear is how this is used. I assumed in line as comments like JSDoc, but then you say comments can be used near the end of the readme.


When can this be null. If it's null, shouldn't there be some other behaviour? This will just not change the fragments


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


Section used by /dev/random missing
 (TODO: Make a diff with tsone config)


I'm agree with that


Normally, Graal and C2 should be able to unroll the loop.
Graal will definitely do it when the loop bounds are constant, and you got a @ExplodeLoop annotation on the method.


You can use UIDevice.currentDevice's userInterfaceIdom as a more reliable way of detecting iPad vs iPhone:

https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIDevice_Class/#//apple_ref/c/tdef/UIUserInterfaceIdiom


jhjk


@nitincchauhan / @brettwooldridge will this be released soon? I'm also experiencing https://github.com/brettwooldridge/HikariCP/issues/456; same use case.


You're right that the rationale here is unclear, and I apologize for that. The reason I made this change was the simplify the crazy logic that currently exists for loading analytics Javascript. If you take a look at the current code on master:

1. If you are using Stanford theming, the `google_analytics_file` variable is set to `theme-google-analytics.html`. [[source](https://github.com/edx/edx-platform/blob/master/lms/templates/main.html#L91)] Otherwise, if you are using microsites, that variable is set to the path to the `google_analytics.html` file in your microsite configuration. Otherwise, the variable is set to just `google_analytics.html`. [[source](https://github.com/edx/edx-platform/blob/master/lms/templates/main.html#L103)]
2. The `google_analytics_file` variable is passed to a Mako `<%include/>` directive. No conditionals are involved -- *something* is always included. [[source](https://github.com/edx/edx-platform/blob/master/lms/templates/main.html#L119)]
3. If you are using Stanford theming, you'd better have a `theme-google-analytics.html` file, even if it's empty, otherwise I assume Mako throws an error. (I haven't validated that assumption, though.) If you're using microsites, having a `google_analytics.html` file is optional, since it will fall back on the core template if the microsite doesn't include one.
4. In the core template, we check to see if `settings.GOOGLE_ANALYTICS_ACCOUNT` is set, and if it isn't, then nothing is included. [[source](https://github.com/edx/edx-platform/blob/master/lms/templates/google_analytics.html#L1)] If that variable *is* set, then the default Google Analytics Javascript is output, with no customization available whatsoever. In effect, this is the behavior that I'm documenting in this README.rst file -- the expectation seems to be that if you want to customize how this Javascript is loaded, then you will not set this variable.

So basically, I was trying to simplify the logic here, while making the edge cases clearer. It seems to me that the situation you are discussing (being unable to use the `settings.GOOGLE_ANALYTICS_ACCOUNT` variable without changing the behavior of the codebase) is *already* the case, and I'm simply trying to document that more clearly. It's true that you can write your own `theme-google-analytics.html` or `google_analytics.html` file, which will override the behavior in the core template, including the implicit dependency on this variable. But I would argue that most people don't realize that this subtle distinction exists, and we should call it out more prominently.

Perhaps you're right that a more generalized `head-analytics.html` file should be used, instead. But what content should we put in that file for the core template? If we need to match the existing behavior to maintain backwards compatibility, then we're back to square one. That means we should probably leave it empty, and let the edx.org theme override that file with the content that is currently in `google_analytics.html`, so that edX can release it to production. It's one more backwards incompatibility, but as long as we document it, it shouldn't be a problem.

Does that sound like a good solution to you? Or after this explanation, do you feel differently and/or want to suggest something else?


>Well, I was wondering if the code removing some of the Supplier<Optional<TransactionManager>>

actually no, not without explicit casting which is pretty much the same thing. The `Stream.of` method takes a vararg `T` so the compiler complains (I think the reason is that `Object` is not a `FunctionalInterface` and therefore type erasure would not work with method references directly).

The last part of the comment - i.e. chaining with `orElse` make sense, so I'll make that change.


Fixed


Isn't `r >= sizeof(matchstr)` a concern here?


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


I like these much better than run and run_all. :tada:


r+


ok


@kasper93 @Underground78 https://github.com/mpc-hc/mpc-hc/compare/develop...XhmikosR:develop


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


This commit changing permissions on files from 666 to 777 only.
The reason is to allow for shell scripts or any binaries to be executed (e.g. custom project shell scripts, node/composer/etc. binaries local to the project).

>several web apps are checking directory permissions and yet freeze when directories are using 777 mode
Do you have examples of what's failing?


Do we maybe need this? http://stackoverflow.com/questions/20527417/android-getfragmentmanager-returns-null


@somaholiday nice! can you give short directions on how to compile it and what the dependencies are? I mean, is
```
jekyll build
```
gonna do it?


IMO `UserTitle.none` instead of `0` would be nicer


agreed. i think we'll eventually have to split stuff out. we could have 3 apps within the `form_processor`. one for form `submissions`, one for form `restores`, and one for `stock`. i think all of those still fall under the umbrella of `form_processor`


Thanks, @jkotas.  I saw that notice in the docs, but it's not clear from them that this has an impact on many library authors, even if they're not using collectible assemblies.  It means that a developer writing a general-purpose library with generic types can't use [ThreadStatic] because it would break a consumer of the library, and/or that a developer writing a collectible assembly needs to know the implementation details of generic assemblies bieng used.  Neither of those is very good.


saw approval from MonsieurNicolas
at https://github.com/jedmccaleb/stellar-core/commit/0a705080cfdd5dc05cf64a4aed354525c712a71f


merging jedmccaleb/stellar-core/master = 0a705080 into auto


jedmccaleb/stellar-core/master = 0a705080 merged ok, testing candidate = bcd25870


so just to clarify, this was the config that seemed to have worked for @phonikg 's wandboard (hence the name of the file)


Nit: Don't need public here and in Swift 2.0 I believe it's better to use static instead of class.


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


Thanks for the suggestion @terracoda.  I have some questions about the span idea, I will move comments to https://github.com/phetsims/scenery/issues/450.


IIRC this change could cause some weird behavior (or not?), better test it with both Qt4, Qt5 to be sure.


the original `generic_hack` lives in another repo now :( it's a code smell to repeat it here, but I'm hoping the two will reunite in the same repo soon...)


Isn't `r >= sizeof(matchstr)` a concern here, as well?


Nit: I know in Obj-C it was the convention to use the underscore to indicate private variables since there wasn't a 'real' way to declare private but we've been omitting the underscore for variables which are private.


Ah, this spec has made me belatedly realise that there's a risk of updating the URLs of old requests that were created before the current `url_title` restriction was added. To avoid that, I think we should add a proc to the callback to restrict it to only requests whose titles have changed:

    before_save :update_url_title, :if => Proc.new { |request| request.title_changed? }

I think this will resolve the problem with this spec, so the

    allow_any_instance_of(InfoRequest).to receive(:update_url_title)

line can be removed.


I've opened https://github.com/dotnet/corefx/issues/4726 to Andrew to track.  Thanks.


I really like this pattern of having a single observer on the UIContentSizeChanged notification then broadcasting out to other observers :+1:


http://jsbin.com/ketasi/edit?html,js,console,output

This could be rewritten as a loop if we're concerned about order. The purpose is to get the min/max of combined ranges. We're not really concerned about gaps. In most cases we want the max, or to determine the duration, `max - min`.

In general multiple ranges are only found in the `played` ranges which we're not concerned with - although it could be useful to confirm (where) playback has started (firstFrame).

```
      var start = Infinity;
      var end = 0;

      if (media.seekable && media.seekable.length) {
        start = Math.min(start, media.seekable.start(0));
        end = Math.max(end, media.seekable.end(media.seekable.length - 1));
      }
```


It needs to be encoded in the URL. The browser will understand either version though.


That doesn't really look right, do we want to do that??


Sorry for this inconvenience. I forgot my changes in intervention/image `2.3.3` are breaking API compatibility and have to be introduced in a MAJOR version.

`2.3.4` should fix this issue. Since Glide requires `"intervention/image": "^2.1"` you should not have to worry about the version anymore.


Nit: Space in front of * (Xcode likes to do this)


There is hidden Unicode symblos. Github editor didn't show them.


That's fine, but they can optimize out the very first (unnecessary) l-shift and 'or' operation?
Could be like this:
```
long acc = 0;
for(int i=0;i<4;++i) {
    acc <<= 8; // the very first shift is definitely unnecessary, because 0==acc
    acc |= Byte.toUnsignedInt(data[offset + i]); // the very first 'or' is unnecessary, because 0==acc
}
return acc;
```
So, I would write this:
```
long acc = Byte.toUnsignedInt(data[offset]);
for(int i=1;i<4;++i) {
    acc <<= 8;
    acc |= Byte.toUnsignedInt(data[offset + i]);
}
return acc;
```
But this is strange because the first assigment, and the weird loop initialization. Or not?


@LinusU The match could be a series of words, like "owner --uid-owner proxy" in my current use case. And it seems to be unsupported at the moment, should I raise an issue?
Best,
Fabian


As far as I can tell, the `google_tag_manager.html` template is only used by the [mit-prof-ed microsite](https://github.com/edx/edx-microsite/blob/master/mit-prof-ed/templates/google_tag_manager.html). (Note that the `edx/edx-microsite` repo is private, so you may not be able to see that file.) @asadiqbal08, @chrisndodge, can you investigate this and see if moving this include to the bottom of the `<body>` tag causes any issues?

I assumed that top or bottom would make no difference, and that having this included at the bottom would cause a better user experience, since the content that the user actually wants to see would get loaded faster without having to wait for the Google Tag Manager JavaScript to load. However, it appears that [the Google Tag Manager documentation does specifically say that it should be loaded right after the `<body>` element](https://support.google.com/tagmanager/answer/6103696), although it's not clear *why* they say that. If this needs to be moved back to the top, then we can make a template called `body-initial.html` or something. (Any suggestions for a better name?)


> applications will kept

applications will keep


Thanks! This bit of code was copied from `P_ShootSpecialLine` which in turn was copied from PrBoom+.


Unused import


Fixes #57.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/93980567


@apotonick When do you plan to add Disposable::Twin support to Cells? And what is the best way to replace https://github.com/apotonick/cells/blob/b489e9a4e93b9f84cbc58846f8d914e9d8ac4905/test/twin_test.rb#L6-L12 for now?


this looks like an in-place transpose? did you mean to do that?


TODO:
- Landscape
- Add blocking function for smart bot AI

Optional:
- Make button background invisible. Along with adding the TicTacToe background.


meet me before leaving ;)


`match` is an list, `['owner', '--uid-owner', 'proxy']` should work?


LGTM


I also figured out how to create a proper title and supplementary information in the comments for pushes.


The merge conflict was in the following file:
app/controllers/insured/family_members_controller.rb

Worked with Sean Carley to resolve.


Still does not compile on OSX due to the ever present baud rate issue we experience every time someone uses a higher speed baud not supported under Mac OS X. Here's the output:

```
CC plotprofile
CC openlog2tlm
CC sdlogger_download
sdlogger_download.c:119:14: error: use of undeclared identifier 'B921600'
      BAUD = B921600;
             ^
sdlogger_download.c:122:14: error: use of undeclared identifier 'B460800'
      BAUD = B460800;
             ^
sdlogger_download.c:420:14: warning: implicit declaration of function 'be32toh' is
      invalid in C99 [-Wimplicit-function-declaration]
             be32toh(log_index.logs[i].address),
             ^
1 warning and 2 errors generated.
make[1]: *** [sdlogger_download] Error 1
make: *** [sw/logalizer] Error 2
```


Ok, thanks for letting me know this quickly :+1:  I was just about to open an issue at your package.

I'll remove the version constraint.


@rafalkowalski please make PRs so that others know what's going on don't push directly to `master`.
Cheers


Yup, fine to leave it out


It's weird to have equality only on the base type and only look at one of the possible fields.


Well thank you for the quick response.


Here should be double B size (MOD_B_DBL in my PR).


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


It appears that regex.test() is faster than regex.exec(). If we're not using the matches for anything, then I think we should use .test() anywhere .exec() was introduced.

(http://jsperf.com/test-vs-match-regex/11)


I wish we could. However, comprehensive theming doesn't currently support multi-tenancy, which means that if we made this change and someone was trying to run multiple multi-tenant microsites (like edX does), then those microsites would not be able to swap between various different footers. I *really* don't want to hold up this pull request even longer to try to get multi-tenancy in, especially since we specifically planned for that to be a separate body of work that would go in later. The theming pull request is huge enough as it is.


Here should be double 64B size (MOD64_B_DBL in my PR).


@singingwolfboy as I understand the Google Tag Manager specification, it is supposed to be at the top of the body tag, although I have no additional knowledge on the hows/why's about that. Personally, I'm fine if you'd like to make the file name more generic. I believe @mattdrayer team did the original implementation of the google tag manager integration, so I'm just tagging him.


Shall we store it to the data watcher?


`classname` -> `className`, we're not neanderthals.


fast-forwarding master to auto = bcd25870


#5 investigation pending


Popup Design:

![popupf](https://cloud.githubusercontent.com/assets/13421423/11478601/9a7e674e-9753-11e5-82e0-1a93b2653c78.png)


If you want to remove them just comment them out


Yep. rm?


@creativepsyco Ok, Sorry for little mess


Thanks!


When we start supporting NuGet here, we'll want to prefer adding the nuget package to adding a metadata reference explicitly.


Is this a typo? I imagine it should be

    var next = this.queues_[contentType].shift();


You can disregard my earlier comment. For some reason my browser only showed the first line of @azeno's repro but I see it all now.


WIP - PAM Dashboard

Can I get some :eyes: on [this](https://tableau.data.shopify.com/#/workbooks/6276/views) to look at the data and if the right metrics are being used?
Haven't formatted the dashboard yet - want to make sure the graphs look right and makes sense first.

Thanks!


This change is very important for me.
Which release will contain this ?


Good question, don't know for sure. It's just that the code comment triggered me to leave a note.


ok


Tokens


@shopify/platform-data


I think this needs to check when the media file does **not** exist (`!WordPress.wpDB.mediaFileExists`).


end the sentence with a dot ;-)


Yeah, I know. I was thinking about it for a little, and decided to unroll. That's why I left the comment there. :)


@LinusU I tried that (https://dpaste.de/Cwk3), but it does not?! Without the match line, everything works fine.


Hmm, this gets more than just tableoid, doesn't it? It extracts tableoid, segment ID, and the ctid.


I think the G1 collector was changed to be the default via [KAFKA-2514](https://issues.apache.org/jira/browse/KAFKA-2514).


Thanks @stvstnfrd. I also like the change to use "cms". I think Studio is branding that can be applied for the user, and that cms is the generic name. The fact that it makes the code simpler is a nice bonus.

I'm with @singingwolfboy on the naming... I prefer the ``static_dir`` over ``path_static``.


C99 style here.


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


executable (missing the 'b')


I think we better should use the original jar of langdetect
    lib/langdetect.jar
as the commited one includes other jar files (like jsonic-1.2 or junit)


I would just make this one serial: 1.
Looks fantastic


Looks good to me, just a small typo in the doc.


Nope.  ```this.queues_[contentType][0]``` is the operation in-progress.  We remove that operation from the queue after it has completed.  If another operation is in queue after that one, we start it just below.

We can't pop the operation from the queue until after it is complete, because ```onUpdateEnd_()``` needs access to it to resolve the associated Promise.


@aperigault it was merged after the 1.9 release, so will end up in docker 1.10


Maxim, could you add a note stating that the new "async" mode implies that every command is taken/accepted by the handler and completed asynchronously? And, naturally, that mode depends on the corresponding kernel-mode changes.


I omitted _DBL macros because they are used only once each. The code should be equivalent (MOD_B_SIZE has been added to index in the next line).


Ok, thank you!


theoretically yes. but unfortunately it's gonna loose context settings like lineWidth and gradient fills etc after setting new width/height. and storing/restoring settings using canvas methods doesn't help. not quite sure why though. so that was quick and dirty fix


Yes, that's right. That's just the default though, people will want to set their own parameters based on their cluster. Do you have a suggestion on how to improve the phrasing?


The new default via [KAFKA-2514](https://issues.apache.org/jira/browse/KAFKA-2514) is now:

```KAFKA_JVM_PERFORMANCE_OPTS="-server -XX:+UseG1GC -XX:MaxGCPauseMillis=20 -XX:InitiatingHeapOccupancyPercent=35 -XX:+DisableExplicitGC -Djava.awt.headless=true"```

Should we add `-XX:G1HeapRegionSize=16M` to the default? Should we add `-XX:+DisableExplicitGC -Djava.awt.headless=true` here?


I decided to look up the differences... this was a nice article if you want to read up on the topic http://www.doublecloud.org/2015/03/gson-vs-jackson-which-to-use-for-json-in-java/


I believe there is some logic that checks for the presence of certain files (GA vs. GTM, for example), because we do not want both to be output.  I'm not stuck on the implementation, so as long as we're preserving the proper behavior it should be fine.  I'm also unsure why the GTM code needs to be located at the top vs. the bottom, but that is the recommendation from the vendor so I think we should stick with that approach.


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


TrappedRook penalty is left out intentionally, because must be tuned with Chess960 games.
And 'Checked' belongs to KingSafety.


is this the appropriate logging level? I think we should just make it debug and update to be a little more verbose:

"Image name not provided in Nulecule. Using %s as container name instead"


Looks like unnecessary blank line. Don't put a blank lines after `{` and before `}`, the prefered format should be
```Swift
someBlock {
  someOtherBlock { // no blank line here
  }
  // blank line here is okay
  someVar = someMoreCode
  ...
  // blank line here is okay
  someFinalCleanup() // no blank line at end of block either
}
```
In short, in a block, no blank line right after the start `{` and right before the `}`. Blank lines are welcome anywhere else in the body though. Don't use double lines to separate your code.


求助，为什么我的没起到效果，输入100px后面没出现多少rem？


Не надо, пустая коллекция - это не исключительная ситуация. Возвращай как есть.


Is this part still true? has the 21ms improved?


R.I.P


Thanks! I made a small adjustment to resize the vector immediately after adding it to the matrix. I added a unit test to make sure this doesn't happen again. I also added you to our list of contributors.


突然发现自己根本不算是程序员，只能勉强算是码农。
整理一下程序员的学习计划，提升和锻炼自我吧，成为一个让人尊敬的程序员。


By the way,

Are you going to put 6LoWPAN on the samR21? I'm really curious! :+1: Hope my work is of help.


What's this?


And R.R.T appeared !


The default doesn't include heap size settings, so I didn't think it was appropriate to add `-XX:G1HeapRegionSize=16M` there. Regarding the settings here, It would not be accurate to state that these are the settings that LinkedIn uses if we modify them.


:clap:


And here is a complete list of all tunable parameters:

mDoubled_0,13,0,100,5,0.0020
eDoubled_0,43,0,100,5,0.0020
mDoubled_1,20,0,100,5,0.0020
eDoubled_1,48,0,100,5,0.0020
mDoubled_2,23,0,100,5,0.0020
eDoubled_2,48,0,100,5,0.0020
mDoubled_3,23,0,100,5,0.0020
eDoubled_3,48,0,100,5,0.0020
mDoubled_4,23,0,100,5,0.0020
eDoubled_4,48,0,100,5,0.0020
mDoubled_5,23,0,100,5,0.0020
eDoubled_5,48,0,100,5,0.0020
mDoubled_6,20,0,100,5,0.0020
eDoubled_6,48,0,100,5,0.0020
mDoubled_7,13,0,100,5,0.0020
eDoubled_7,43,0,100,5,0.0020
mIsolated_0_0,37,0,100,5,0.0020
eIsolated_0_0,45,0,100,5,0.0020
mIsolated_0_1,54,0,100,5,0.0020
eIsolated_0_1,52,0,100,5,0.0020
mIsolated_0_2,60,0,100,5,0.0020
eIsolated_0_2,52,0,100,5,0.0020
mIsolated_0_3,60,0,100,5,0.0020
eIsolated_0_3,52,0,100,5,0.0020
mIsolated_0_4,60,0,100,5,0.0020
eIsolated_0_4,52,0,100,5,0.0020
mIsolated_0_5,60,0,100,5,0.0020
eIsolated_0_5,52,0,100,5,0.0020
mIsolated_0_6,54,0,100,5,0.0020
eIsolated_0_6,52,0,100,5,0.0020
mIsolated_0_7,37,0,100,5,0.0020
eIsolated_0_7,45,0,100,5,0.0020
mIsolated_1_0,25,0,100,5,0.0020
eIsolated_1_0,30,0,100,5,0.0020
mIsolated_1_1,36,0,100,5,0.0020
eIsolated_1_1,35,0,100,5,0.0020
mIsolated_1_2,40,0,100,5,0.0020
eIsolated_1_2,35,0,100,5,0.0020
mIsolated_1_3,40,0,100,5,0.0020
eIsolated_1_3,35,0,100,5,0.0020
mIsolated_1_4,40,0,100,5,0.0020
eIsolated_1_4,35,0,100,5,0.0020
mIsolated_1_5,40,0,100,5,0.0020
eIsolated_1_5,35,0,100,5,0.0020
mIsolated_1_6,36,0,100,5,0.0020
eIsolated_1_6,35,0,100,5,0.0020
mIsolated_1_7,25,0,100,5,0.0020
eIsolated_1_7,30,0,100,5,0.0020
mBackward_0,67,0,100,5,0.0020
eBackward_0,42,0,100,5,0.0020
mBackward_1,49,0,100,5,0.0020
eBackward_1,24,0,100,5,0.0020
mLever_0,0,0,100,5,0.0020
eLever_0,0,0,100,5,0.0020
mLever_1,0,0,100,5,0.0020
eLever_1,0,0,100,5,0.0020
mLever_2,0,0,100,5,0.0020
eLever_2,0,0,100,5,0.0020
mLever_3,0,0,100,5,0.0020
eLever_3,0,0,100,5,0.0020
mLever_4,20,0,100,5,0.0020
eLever_4,20,0,100,5,0.0020
mLever_5,40,0,100,5,0.0020
eLever_5,40,0,100,5,0.0020
mLever_6,0,0,100,5,0.0020
eLever_6,0,0,100,5,0.0020
mLever_7,0,0,100,5,0.0020
eLever_7,0,0,100,5,0.0020
mUnsupportedPawnPenalty,20,0,100,5,0.0020
eUnsupportedPawnPenalty,10,0,100,5,0.0020
mCenterBind,16,0,100,5,0.0020
eCenterBind,0,0,100,5,0.0020
Seed_0,0,0,300,15,0.0020
Seed_1,6,0,300,15,0.0020
Seed_2,15,0,300,15,0.0020
Seed_3,10,0,300,15,0.0020
Seed_4,57,0,300,15,0.0020
Seed_5,75,0,300,15,0.0020
Seed_6,135,0,300,15,0.0020
Seed_7,258,0,300,15,0.0020
w_0,289,0,600,30,0.0020
w_1,344,0,600,30,0.0020
w_2,233,0,600,30,0.0020
w_3,201,0,600,30,0.0020
w_4,221,0,600,30,0.0020
w_5,273,0,600,30,0.0020
w_6,46,0,600,30,0.0020
w_7,0,0,600,30,0.0020
w_8,322,0,600,30,0.0020
w_9,0,0,600,30,0.0020
w_10,350,0,600,30,0.0020
w_11,256,0,600,30,0.0020
mMobilityBonus_0_0,0,-100,200,15,0.0020
eMobilityBonus_0_0,0,-100,200,15,0.0020
mMobilityBonus_0_1,0,-100,200,15,0.0020
eMobilityBonus_0_1,0,-100,200,15,0.0020
mMobilityBonus_0_2,0,-100,200,15,0.0020
eMobilityBonus_0_2,0,-100,200,15,0.0020
mMobilityBonus_0_3,0,-100,200,15,0.0020
eMobilityBonus_0_3,0,-100,200,15,0.0020
mMobilityBonus_0_4,0,-100,200,15,0.0020
eMobilityBonus_0_4,0,-100,200,15,0.0020
mMobilityBonus_0_5,0,-100,200,15,0.0020
eMobilityBonus_0_5,0,-100,200,15,0.0020
mMobilityBonus_0_6,0,-100,200,15,0.0020
eMobilityBonus_0_6,0,-100,200,15,0.0020
mMobilityBonus_0_7,0,-100,200,15,0.0020
eMobilityBonus_0_7,0,-100,200,15,0.0020
mMobilityBonus_0_8,0,-100,200,15,0.0020
eMobilityBonus_0_8,0,-100,200,15,0.0020
mMobilityBonus_0_9,0,-100,200,15,0.0020
eMobilityBonus_0_9,0,-100,200,15,0.0020
mMobilityBonus_0_10,0,-100,200,15,0.0020
eMobilityBonus_0_10,0,-100,200,15,0.0020
mMobilityBonus_0_11,0,-100,200,15,0.0020
eMobilityBonus_0_11,0,-100,200,15,0.0020
mMobilityBonus_0_12,0,-100,200,15,0.0020
eMobilityBonus_0_12,0,-100,200,15,0.0020
mMobilityBonus_0_13,0,-100,200,15,0.0020
eMobilityBonus_0_13,0,-100,200,15,0.0020
mMobilityBonus_0_14,0,-100,200,15,0.0020
eMobilityBonus_0_14,0,-100,200,15,0.0020
mMobilityBonus_0_15,0,-100,200,15,0.0020
eMobilityBonus_0_15,0,-100,200,15,0.0020
mMobilityBonus_0_16,0,-100,200,15,0.0020
eMobilityBonus_0_16,0,-100,200,15,0.0020
mMobilityBonus_0_17,0,-100,200,15,0.0020
eMobilityBonus_0_17,0,-100,200,15,0.0020
mMobilityBonus_0_18,0,-100,200,15,0.0020
eMobilityBonus_0_18,0,-100,200,15,0.0020
mMobilityBonus_0_19,0,-100,200,15,0.0020
eMobilityBonus_0_19,0,-100,200,15,0.0020
mMobilityBonus_0_20,0,-100,200,15,0.0020
eMobilityBonus_0_20,0,-100,200,15,0.0020
mMobilityBonus_0_21,0,-100,200,15,0.0020
eMobilityBonus_0_21,0,-100,200,15,0.0020
mMobilityBonus_0_22,0,-100,200,15,0.0020
eMobilityBonus_0_22,0,-100,200,15,0.0020
mMobilityBonus_0_23,0,-100,200,15,0.0020
eMobilityBonus_0_23,0,-100,200,15,0.0020
mMobilityBonus_0_24,0,-100,200,15,0.0020
eMobilityBonus_0_24,0,-100,200,15,0.0020
mMobilityBonus_0_25,0,-100,200,15,0.0020
eMobilityBonus_0_25,0,-100,200,15,0.0020
mMobilityBonus_0_26,0,-100,200,15,0.0020
eMobilityBonus_0_26,0,-100,200,15,0.0020
mMobilityBonus_0_27,0,-100,200,15,0.0020
eMobilityBonus_0_27,0,-100,200,15,0.0020
mMobilityBonus_0_28,0,-100,200,15,0.0020
eMobilityBonus_0_28,0,-100,200,15,0.0020
mMobilityBonus_0_29,0,-100,200,15,0.0020
eMobilityBonus_0_29,0,-100,200,15,0.0020
mMobilityBonus_0_30,0,-100,200,15,0.0020
eMobilityBonus_0_30,0,-100,200,15,0.0020
mMobilityBonus_0_31,0,-100,200,15,0.0020
eMobilityBonus_0_31,0,-100,200,15,0.0020
mMobilityBonus_1_0,0,-100,200,15,0.0020
eMobilityBonus_1_0,0,-100,200,15,0.0020
mMobilityBonus_1_1,0,-100,200,15,0.0020
eMobilityBonus_1_1,0,-100,200,15,0.0020
mMobilityBonus_1_2,0,-100,200,15,0.0020
eMobilityBonus_1_2,0,-100,200,15,0.0020
mMobilityBonus_1_3,0,-100,200,15,0.0020
eMobilityBonus_1_3,0,-100,200,15,0.0020
mMobilityBonus_1_4,0,-100,200,15,0.0020
eMobilityBonus_1_4,0,-100,200,15,0.0020
mMobilityBonus_1_5,0,-100,200,15,0.0020
eMobilityBonus_1_5,0,-100,200,15,0.0020
mMobilityBonus_1_6,0,-100,200,15,0.0020
eMobilityBonus_1_6,0,-100,200,15,0.0020
mMobilityBonus_1_7,0,-100,200,15,0.0020
eMobilityBonus_1_7,0,-100,200,15,0.0020
mMobilityBonus_1_8,0,-100,200,15,0.0020
eMobilityBonus_1_8,0,-100,200,15,0.0020
mMobilityBonus_1_9,0,-100,200,15,0.0020
eMobilityBonus_1_9,0,-100,200,15,0.0020
mMobilityBonus_1_10,0,-100,200,15,0.0020
eMobilityBonus_1_10,0,-100,200,15,0.0020
mMobilityBonus_1_11,0,-100,200,15,0.0020
eMobilityBonus_1_11,0,-100,200,15,0.0020
mMobilityBonus_1_12,0,-100,200,15,0.0020
eMobilityBonus_1_12,0,-100,200,15,0.0020
mMobilityBonus_1_13,0,-100,200,15,0.0020
eMobilityBonus_1_13,0,-100,200,15,0.0020
mMobilityBonus_1_14,0,-100,200,15,0.0020
eMobilityBonus_1_14,0,-100,200,15,0.0020
mMobilityBonus_1_15,0,-100,200,15,0.0020
eMobilityBonus_1_15,0,-100,200,15,0.0020
mMobilityBonus_1_16,0,-100,200,15,0.0020
eMobilityBonus_1_16,0,-100,200,15,0.0020
mMobilityBonus_1_17,0,-100,200,15,0.0020
eMobilityBonus_1_17,0,-100,200,15,0.0020
mMobilityBonus_1_18,0,-100,200,15,0.0020
eMobilityBonus_1_18,0,-100,200,15,0.0020
mMobilityBonus_1_19,0,-100,200,15,0.0020
eMobilityBonus_1_19,0,-100,200,15,0.0020
mMobilityBonus_1_20,0,-100,200,15,0.0020
eMobilityBonus_1_20,0,-100,200,15,0.0020
mMobilityBonus_1_21,0,-100,200,15,0.0020
eMobilityBonus_1_21,0,-100,200,15,0.0020
mMobilityBonus_1_22,0,-100,200,15,0.0020
eMobilityBonus_1_22,0,-100,200,15,0.0020
mMobilityBonus_1_23,0,-100,200,15,0.0020
eMobilityBonus_1_23,0,-100,200,15,0.0020
mMobilityBonus_1_24,0,-100,200,15,0.0020
eMobilityBonus_1_24,0,-100,200,15,0.0020
mMobilityBonus_1_25,0,-100,200,15,0.0020
eMobilityBonus_1_25,0,-100,200,15,0.0020
mMobilityBonus_1_26,0,-100,200,15,0.0020
eMobilityBonus_1_26,0,-100,200,15,0.0020
mMobilityBonus_1_27,0,-100,200,15,0.0020
eMobilityBonus_1_27,0,-100,200,15,0.0020
mMobilityBonus_1_28,0,-100,200,15,0.0020
eMobilityBonus_1_28,0,-100,200,15,0.0020
mMobilityBonus_1_29,0,-100,200,15,0.0020
eMobilityBonus_1_29,0,-100,200,15,0.0020
mMobilityBonus_1_30,0,-100,200,15,0.0020
eMobilityBonus_1_30,0,-100,200,15,0.0020
mMobilityBonus_1_31,0,-100,200,15,0.0020
eMobilityBonus_1_31,0,-100,200,15,0.0020
mMobilityBonus_2_0,-70,-100,200,15,0.0020
eMobilityBonus_2_0,-52,-100,200,15,0.0020
mMobilityBonus_2_1,-52,-100,200,15,0.0020
eMobilityBonus_2_1,-37,-100,200,15,0.0020
mMobilityBonus_2_2,-7,-100,200,15,0.0020
eMobilityBonus_2_2,-17,-100,200,15,0.0020
mMobilityBonus_2_3,0,-100,200,15,0.0020
eMobilityBonus_2_3,-6,-100,200,15,0.0020
mMobilityBonus_2_4,8,-100,200,15,0.0020
eMobilityBonus_2_4,5,-100,200,15,0.0020
mMobilityBonus_2_5,16,-100,200,15,0.0020
eMobilityBonus_2_5,9,-100,200,15,0.0020
mMobilityBonus_2_6,23,-100,200,15,0.0020
eMobilityBonus_2_6,20,-100,200,15,0.0020
mMobilityBonus_2_7,31,-100,200,15,0.0020
eMobilityBonus_2_7,21,-100,200,15,0.0020
mMobilityBonus_2_8,36,-100,200,15,0.0020
eMobilityBonus_2_8,22,-100,200,15,0.0020
mMobilityBonus_2_9,0,-100,200,15,0.0020
eMobilityBonus_2_9,0,-100,200,15,0.0020
mMobilityBonus_2_10,0,-100,200,15,0.0020
eMobilityBonus_2_10,0,-100,200,15,0.0020
mMobilityBonus_2_11,0,-100,200,15,0.0020
eMobilityBonus_2_11,0,-100,200,15,0.0020
mMobilityBonus_2_12,0,-100,200,15,0.0020
eMobilityBonus_2_12,0,-100,200,15,0.0020
mMobilityBonus_2_13,0,-100,200,15,0.0020
eMobilityBonus_2_13,0,-100,200,15,0.0020
mMobilityBonus_2_14,0,-100,200,15,0.0020
eMobilityBonus_2_14,0,-100,200,15,0.0020
mMobilityBonus_2_15,0,-100,200,15,0.0020
eMobilityBonus_2_15,0,-100,200,15,0.0020
mMobilityBonus_2_16,0,-100,200,15,0.0020
eMobilityBonus_2_16,0,-100,200,15,0.0020
mMobilityBonus_2_17,0,-100,200,15,0.0020
eMobilityBonus_2_17,0,-100,200,15,0.0020
mMobilityBonus_2_18,0,-100,200,15,0.0020
eMobilityBonus_2_18,0,-100,200,15,0.0020
mMobilityBonus_2_19,0,-100,200,15,0.0020
eMobilityBonus_2_19,0,-100,200,15,0.0020
mMobilityBonus_2_20,0,-100,200,15,0.0020
eMobilityBonus_2_20,0,-100,200,15,0.0020
mMobilityBonus_2_21,0,-100,200,15,0.0020
eMobilityBonus_2_21,0,-100,200,15,0.0020
mMobilityBonus_2_22,0,-100,200,15,0.0020
eMobilityBonus_2_22,0,-100,200,15,0.0020
mMobilityBonus_2_23,0,-100,200,15,0.0020
eMobilityBonus_2_23,0,-100,200,15,0.0020
mMobilityBonus_2_24,0,-100,200,15,0.0020
eMobilityBonus_2_24,0,-100,200,15,0.0020
mMobilityBonus_2_25,0,-100,200,15,0.0020
eMobilityBonus_2_25,0,-100,200,15,0.0020
mMobilityBonus_2_26,0,-100,200,15,0.0020
eMobilityBonus_2_26,0,-100,200,15,0.0020
mMobilityBonus_2_27,0,-100,200,15,0.0020
eMobilityBonus_2_27,0,-100,200,15,0.0020
mMobilityBonus_2_28,0,-100,200,15,0.0020
eMobilityBonus_2_28,0,-100,200,15,0.0020
mMobilityBonus_2_29,0,-100,200,15,0.0020
eMobilityBonus_2_29,0,-100,200,15,0.0020
mMobilityBonus_2_30,0,-100,200,15,0.0020
eMobilityBonus_2_30,0,-100,200,15,0.0020
mMobilityBonus_2_31,0,-100,200,15,0.0020
eMobilityBonus_2_31,0,-100,200,15,0.0020
mMobilityBonus_3_0,-49,-100,200,15,0.0020
eMobilityBonus_3_0,-44,-100,200,15,0.0020
mMobilityBonus_3_1,-22,-100,200,15,0.0020
eMobilityBonus_3_1,-13,-100,200,15,0.0020
mMobilityBonus_3_2,16,-100,200,15,0.0020
eMobilityBonus_3_2,0,-100,200,15,0.0020
mMobilityBonus_3_3,27,-100,200,15,0.0020
eMobilityBonus_3_3,11,-100,200,15,0.0020
mMobilityBonus_3_4,38,-100,200,15,0.0020
eMobilityBonus_3_4,19,-100,200,15,0.0020
mMobilityBonus_3_5,52,-100,200,15,0.0020
eMobilityBonus_3_5,34,-100,200,15,0.0020
mMobilityBonus_3_6,56,-100,200,15,0.0020
eMobilityBonus_3_6,44,-100,200,15,0.0020
mMobilityBonus_3_7,65,-100,200,15,0.0020
eMobilityBonus_3_7,47,-100,200,15,0.0020
mMobilityBonus_3_8,67,-100,200,15,0.0020
eMobilityBonus_3_8,51,-100,200,15,0.0020
mMobilityBonus_3_9,73,-100,200,15,0.0020
eMobilityBonus_3_9,56,-100,200,15,0.0020
mMobilityBonus_3_10,81,-100,200,15,0.0020
eMobilityBonus_3_10,59,-100,200,15,0.0020
mMobilityBonus_3_11,83,-100,200,15,0.0020
eMobilityBonus_3_11,69,-100,200,15,0.0020
mMobilityBonus_3_12,95,-100,200,15,0.0020
eMobilityBonus_3_12,72,-100,200,15,0.0020
mMobilityBonus_3_13,100,-100,200,15,0.0020
eMobilityBonus_3_13,75,-100,200,15,0.0020
mMobilityBonus_3_14,0,-100,200,15,0.0020
eMobilityBonus_3_14,0,-100,200,15,0.0020
mMobilityBonus_3_15,0,-100,200,15,0.0020
eMobilityBonus_3_15,0,-100,200,15,0.0020
mMobilityBonus_3_16,0,-100,200,15,0.0020
eMobilityBonus_3_16,0,-100,200,15,0.0020
mMobilityBonus_3_17,0,-100,200,15,0.0020
eMobilityBonus_3_17,0,-100,200,15,0.0020
mMobilityBonus_3_18,0,-100,200,15,0.0020
eMobilityBonus_3_18,0,-100,200,15,0.0020
mMobilityBonus_3_19,0,-100,200,15,0.0020
eMobilityBonus_3_19,0,-100,200,15,0.0020
mMobilityBonus_3_20,0,-100,200,15,0.0020
eMobilityBonus_3_20,0,-100,200,15,0.0020
mMobilityBonus_3_21,0,-100,200,15,0.0020
eMobilityBonus_3_21,0,-100,200,15,0.0020
mMobilityBonus_3_22,0,-100,200,15,0.0020
eMobilityBonus_3_22,0,-100,200,15,0.0020
mMobilityBonus_3_23,0,-100,200,15,0.0020
eMobilityBonus_3_23,0,-100,200,15,0.0020
mMobilityBonus_3_24,0,-100,200,15,0.0020
eMobilityBonus_3_24,0,-100,200,15,0.0020
mMobilityBonus_3_25,0,-100,200,15,0.0020
eMobilityBonus_3_25,0,-100,200,15,0.0020
mMobilityBonus_3_26,0,-100,200,15,0.0020
eMobilityBonus_3_26,0,-100,200,15,0.0020
mMobilityBonus_3_27,0,-100,200,15,0.0020
eMobilityBonus_3_27,0,-100,200,15,0.0020
mMobilityBonus_3_28,0,-100,200,15,0.0020
eMobilityBonus_3_28,0,-100,200,15,0.0020
mMobilityBonus_3_29,0,-100,200,15,0.0020
eMobilityBonus_3_29,0,-100,200,15,0.0020
mMobilityBonus_3_30,0,-100,200,15,0.0020
eMobilityBonus_3_30,0,-100,200,15,0.0020
mMobilityBonus_3_31,0,-100,200,15,0.0020
eMobilityBonus_3_31,0,-100,200,15,0.0020
mMobilityBonus_4_0,-49,-100,200,15,0.0020
eMobilityBonus_4_0,-57,-100,200,15,0.0020
mMobilityBonus_4_1,-22,-100,200,15,0.0020
eMobilityBonus_4_1,-14,-100,200,15,0.0020
mMobilityBonus_4_2,-10,-100,200,15,0.0020
eMobilityBonus_4_2,18,-100,200,15,0.0020
mMobilityBonus_4_3,-5,-100,200,15,0.0020
eMobilityBonus_4_3,39,-100,200,15,0.0020
mMobilityBonus_4_4,-4,-100,200,15,0.0020
eMobilityBonus_4_4,50,-100,200,15,0.0020
mMobilityBonus_4_5,-2,-100,200,15,0.0020
eMobilityBonus_4_5,58,-100,200,15,0.0020
mMobilityBonus_4_6,6,-100,200,15,0.0020
eMobilityBonus_4_6,78,-100,200,15,0.0020
mMobilityBonus_4_7,11,-100,200,15,0.0020
eMobilityBonus_4_7,86,-100,200,15,0.0020
mMobilityBonus_4_8,17,-100,200,15,0.0020
eMobilityBonus_4_8,92,-100,200,15,0.0020
mMobilityBonus_4_9,19,-100,200,15,0.0020
eMobilityBonus_4_9,103,-100,200,15,0.0020
mMobilityBonus_4_10,26,-100,200,15,0.0020
eMobilityBonus_4_10,111,-100,200,15,0.0020
mMobilityBonus_4_11,27,-100,200,15,0.0020
eMobilityBonus_4_11,115,-100,200,15,0.0020
mMobilityBonus_4_12,36,-100,200,15,0.0020
eMobilityBonus_4_12,119,-100,200,15,0.0020
mMobilityBonus_4_13,41,-100,200,15,0.0020
eMobilityBonus_4_13,121,-100,200,15,0.0020
mMobilityBonus_4_14,50,-100,200,15,0.0020
eMobilityBonus_4_14,122,-100,200,15,0.0020
mMobilityBonus_4_15,0,-100,200,15,0.0020
eMobilityBonus_4_15,0,-100,200,15,0.0020
mMobilityBonus_4_16,0,-100,200,15,0.0020
eMobilityBonus_4_16,0,-100,200,15,0.0020
mMobilityBonus_4_17,0,-100,200,15,0.0020
eMobilityBonus_4_17,0,-100,200,15,0.0020
mMobilityBonus_4_18,0,-100,200,15,0.0020
eMobilityBonus_4_18,0,-100,200,15,0.0020
mMobilityBonus_4_19,0,-100,200,15,0.0020
eMobilityBonus_4_19,0,-100,200,15,0.0020
mMobilityBonus_4_20,0,-100,200,15,0.0020
eMobilityBonus_4_20,0,-100,200,15,0.0020
mMobilityBonus_4_21,0,-100,200,15,0.0020
eMobilityBonus_4_21,0,-100,200,15,0.0020
mMobilityBonus_4_22,0,-100,200,15,0.0020
eMobilityBonus_4_22,0,-100,200,15,0.0020
mMobilityBonus_4_23,0,-100,200,15,0.0020
eMobilityBonus_4_23,0,-100,200,15,0.0020
mMobilityBonus_4_24,0,-100,200,15,0.0020
eMobilityBonus_4_24,0,-100,200,15,0.0020
mMobilityBonus_4_25,0,-100,200,15,0.0020
eMobilityBonus_4_25,0,-100,200,15,0.0020
mMobilityBonus_4_26,0,-100,200,15,0.0020
eMobilityBonus_4_26,0,-100,200,15,0.0020
mMobilityBonus_4_27,0,-100,200,15,0.0020
eMobilityBonus_4_27,0,-100,200,15,0.0020
mMobilityBonus_4_28,0,-100,200,15,0.0020
eMobilityBonus_4_28,0,-100,200,15,0.0020
mMobilityBonus_4_29,0,-100,200,15,0.0020
eMobilityBonus_4_29,0,-100,200,15,0.0020
mMobilityBonus_4_30,0,-100,200,15,0.0020
eMobilityBonus_4_30,0,-100,200,15,0.0020
mMobilityBonus_4_31,0,-100,200,15,0.0020
eMobilityBonus_4_31,0,-100,200,15,0.0020
mMobilityBonus_5_0,-41,-100,200,15,0.0020
eMobilityBonus_5_0,-24,-100,200,15,0.0020
mMobilityBonus_5_1,-26,-100,200,15,0.0020
eMobilityBonus_5_1,-8,-100,200,15,0.0020
mMobilityBonus_5_2,0,-100,200,15,0.0020
eMobilityBonus_5_2,6,-100,200,15,0.0020
mMobilityBonus_5_3,2,-100,200,15,0.0020
eMobilityBonus_5_3,14,-100,200,15,0.0020
mMobilityBonus_5_4,12,-100,200,15,0.0020
eMobilityBonus_5_4,27,-100,200,15,0.0020
mMobilityBonus_5_5,21,-100,200,15,0.0020
eMobilityBonus_5_5,40,-100,200,15,0.0020
mMobilityBonus_5_6,22,-100,200,15,0.0020
eMobilityBonus_5_6,45,-100,200,15,0.0020
mMobilityBonus_5_7,37,-100,200,15,0.0020
eMobilityBonus_5_7,55,-100,200,15,0.0020
mMobilityBonus_5_8,40,-100,200,15,0.0020
eMobilityBonus_5_8,57,-100,200,15,0.0020
mMobilityBonus_5_9,43,-100,200,15,0.0020
eMobilityBonus_5_9,63,-100,200,15,0.0020
mMobilityBonus_5_10,50,-100,200,15,0.0020
eMobilityBonus_5_10,68,-100,200,15,0.0020
mMobilityBonus_5_11,52,-100,200,15,0.0020
eMobilityBonus_5_11,74,-100,200,15,0.0020
mMobilityBonus_5_12,56,-100,200,15,0.0020
eMobilityBonus_5_12,80,-100,200,15,0.0020
mMobilityBonus_5_13,66,-100,200,15,0.0020
eMobilityBonus_5_13,84,-100,200,15,0.0020
mMobilityBonus_5_14,68,-100,200,15,0.0020
eMobilityBonus_5_14,85,-100,200,15,0.0020
mMobilityBonus_5_15,69,-100,200,15,0.0020
eMobilityBonus_5_15,88,-100,200,15,0.0020
mMobilityBonus_5_16,71,-100,200,15,0.0020
eMobilityBonus_5_16,92,-100,200,15,0.0020
mMobilityBonus_5_17,72,-100,200,15,0.0020
eMobilityBonus_5_17,94,-100,200,15,0.0020
mMobilityBonus_5_18,80,-100,200,15,0.0020
eMobilityBonus_5_18,96,-100,200,15,0.0020
mMobilityBonus_5_19,89,-100,200,15,0.0020
eMobilityBonus_5_19,98,-100,200,15,0.0020
mMobilityBonus_5_20,94,-100,200,15,0.0020
eMobilityBonus_5_20,101,-100,200,15,0.0020
mMobilityBonus_5_21,102,-100,200,15,0.0020
eMobilityBonus_5_21,113,-100,200,15,0.0020
mMobilityBonus_5_22,106,-100,200,15,0.0020
eMobilityBonus_5_22,114,-100,200,15,0.0020
mMobilityBonus_5_23,107,-100,200,15,0.0020
eMobilityBonus_5_23,116,-100,200,15,0.0020
mMobilityBonus_5_24,112,-100,200,15,0.0020
eMobilityBonus_5_24,125,-100,200,15,0.0020
mMobilityBonus_5_25,113,-100,200,15,0.0020
eMobilityBonus_5_25,127,-100,200,15,0.0020
mMobilityBonus_5_26,117,-100,200,15,0.0020
eMobilityBonus_5_26,137,-100,200,15,0.0020
mMobilityBonus_5_27,122,-100,200,15,0.0020
eMobilityBonus_5_27,143,-100,200,15,0.0020
mMobilityBonus_5_28,0,-100,200,15,0.0020
eMobilityBonus_5_28,0,-100,200,15,0.0020
mMobilityBonus_5_29,0,-100,200,15,0.0020
eMobilityBonus_5_29,0,-100,200,15,0.0020
mMobilityBonus_5_30,0,-100,200,15,0.0020
eMobilityBonus_5_30,0,-100,200,15,0.0020
mMobilityBonus_5_31,0,-100,200,15,0.0020
eMobilityBonus_5_31,0,-100,200,15,0.0020
mOutpost_0_0,42,0,100,5,0.0020
eOutpost_0_0,11,0,100,5,0.0020
mOutpost_0_1,63,0,100,5,0.0020
eOutpost_0_1,17,0,100,5,0.0020
mOutpost_1_0,18,0,100,5,0.0020
eOutpost_1_0,5,0,100,5,0.0020
mOutpost_1_1,27,0,100,5,0.0020
eOutpost_1_1,8,0,100,5,0.0020
mReachableOutpost_0_0,21,0,100,5,0.0020
eReachableOutpost_0_0,5,0,100,5,0.0020
mReachableOutpost_0_1,31,0,100,5,0.0020
eReachableOutpost_0_1,8,0,100,5,0.0020
mReachableOutpost_1_0,8,0,100,5,0.0020
eReachableOutpost_1_0,2,0,100,5,0.0020
mReachableOutpost_1_1,13,0,100,5,0.0020
eReachableOutpost_1_1,4,0,100,5,0.0020
mThreat_0_0,0,0,300,15,0.0020
eThreat_0_0,0,0,300,15,0.0020
mThreat_0_1,0,0,300,15,0.0020
eThreat_0_1,32,0,300,15,0.0020
mThreat_0_2,25,0,300,15,0.0020
eThreat_0_2,39,0,300,15,0.0020
mThreat_0_3,28,0,300,15,0.0020
eThreat_0_3,44,0,300,15,0.0020
mThreat_0_4,42,0,300,15,0.0020
eThreat_0_4,98,0,300,15,0.0020
mThreat_0_5,35,0,300,15,0.0020
eThreat_0_5,105,0,300,15,0.0020
mThreat_0_6,0,0,300,15,0.0020
eThreat_0_6,0,0,300,15,0.0020
mThreat_0_7,0,0,300,15,0.0020
eThreat_0_7,0,0,300,15,0.0020
mThreat_1_0,0,0,300,15,0.0020
eThreat_1_0,0,0,300,15,0.0020
mThreat_1_1,0,0,300,15,0.0020
eThreat_1_1,27,0,300,15,0.0020
mThreat_1_2,26,0,300,15,0.0020
eThreat_1_2,57,0,300,15,0.0020
mThreat_1_3,26,0,300,15,0.0020
eThreat_1_3,57,0,300,15,0.0020
mThreat_1_4,0,0,300,15,0.0020
eThreat_1_4,30,0,300,15,0.0020
mThreat_1_5,23,0,300,15,0.0020
eThreat_1_5,51,0,300,15,0.0020
mThreat_1_6,0,0,300,15,0.0020
eThreat_1_6,0,0,300,15,0.0020
mThreat_1_7,0,0,300,15,0.0020
eThreat_1_7,0,0,300,15,0.0020
mThreatenedByPawn_0,0,0,300,15,0.0020
eThreatenedByPawn_0,0,0,300,15,0.0020
mThreatenedByPawn_1,0,0,300,15,0.0020
eThreatenedByPawn_1,0,0,300,15,0.0020
mThreatenedByPawn_2,107,0,300,15,0.0020
eThreatenedByPawn_2,138,0,300,15,0.0020
mThreatenedByPawn_3,84,0,300,15,0.0020
eThreatenedByPawn_3,122,0,300,15,0.0020
mThreatenedByPawn_4,114,0,300,15,0.0020
eThreatenedByPawn_4,203,0,300,15,0.0020
mThreatenedByPawn_5,121,0,300,15,0.0020
eThreatenedByPawn_5,217,0,300,15,0.0020
mThreatenedByPawn_6,0,0,300,15,0.0020
eThreatenedByPawn_6,0,0,300,15,0.0020
mThreatenedByPawn_7,0,0,300,15,0.0020
eThreatenedByPawn_7,0,0,300,15,0.0020
mThreatenedByHangingPawn,40,0,300,15,0.0020
eThreatenedByHangingPawn,60,0,300,15,0.0020
mKingOnOne,2,0,200,10,0.0020
eKingOnOne,58,0,200,10,0.0020
mKingOnMany,6,0,200,10,0.0020
eKingOnMany,125,0,200,10,0.0020
mRookOnPawn,7,0,200,10,0.0020
eRookOnPawn,27,0,200,10,0.0020
mRookOnOpenFile,43,0,200,10,0.0020
eRookOnOpenFile,21,0,200,10,0.0020
mRookOnSemiOpenFile,19,0,200,10,0.0020
eRookOnSemiOpenFile,10,0,200,10,0.0020
mBishopPawns,8,0,200,10,0.0020
eBishopPawns,12,0,200,10,0.0020
mMinorBehindPawn,16,0,200,10,0.0020
eMinorBehindPawn,0,0,200,10,0.0020
mUnstoppable,0,0,200,10,0.0020
eUnstoppable,20,0,200,10,0.0020
mHanging,31,0,200,10,0.0020
eHanging,26,0,200,10,0.0020
mPawnAttackThreat,20,0,200,10,0.0020
ePawnAttackThreat,20,0,200,10,0.0020


I'm getting always this on console: "lux: 54612.50 lx"
I think that all the connections are ok. What can be?


@FBRTMaka These are the changes I made to the dropdown. I added all of the sensors in that document you sent, the problem is that I don't think that a lowercased version of those labels correspond to actual columns. Many of the options probably don't work. Didn't want to do a PR until that was resolved.


THX this repair my template


Exercicio 1, 2 e 3


It's unsafe when casting this variable. You should unwrap it before using. Example:
```Swift
if let isRetweeted = dictionary["retweeted"] as? Bool {
  retweeted = isRetweeted
}
```
I highly recommend you to use `SwiftyJSON` to parse the json and save to object.


Double lines.


Blank line after `{`.


That seemed to be the easiest and the less-intrusive way to solve the minification problem we had after analyzing the html-minifier code. The might be another solution though.


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/22a03582714180d72f95beb4d7ff86b6293367ae


merging basho/riak_kv/fix/az/move-ddl-validation-to-riak_ql_parser = 22a03582 into borshop-integration-1279-fix/az/move-ddl-validation-to-riak_ql_parser


basho/riak_kv/fix/az/move-ddl-validation-to-riak_ql_parser = 22a03582 merged ok, testing candidate = 581cf364


It's better if you set default value for these properties, such as `""` for String and `0` for Int. By this way, you needn't to unwrap them before using it later.


Vivek, looks like a tab space problem.


Make the table. You have the info right there.


"stuff" great commit name there :)


If you don't implement any special thing in `override func`, you can remove it.


This commit is lame, But somehow my repo is out of sync with the openbmc repo.


This shouldn't exist anymore. I think it needs to be something like:
`TEMPLATES[1]['OPTIONS']['debug'] = True` ?


Looks like [setting flowLevel](https://github.com/nodeca/js-yaml#safedump-object---options-) to something >4 would likely get us this formatting.

I believe that it's totally equivalent semantically. Is it worth spending an hour or so hacking on this for a little whitespace in a config file though?


@ShawnieBoy has come up with some good ones to start with. Let's see if we can come up with at least ten messages before 1.69 release! I'm sure there must be lots of MTG in-jokes. Also AI related...


You can remove this function.


Please remove unnecessary comment.


Vivek, looks like the MySQL bit values are not getting represented properly in the DDL strings. Specifically, the following columns are bit values. (Create statement concatenated to show only the bit columns).

``` sql
CREATE TABLE `chapter` (
  `is_test_paper` bit(1) NOT NULL DEFAULT b'0',
);

CREATE TABLE `notes_element` (
  `ready` bit(1) NOT NULL DEFAULT b'1',
  `hidden_from_view` bit(1) NOT NULL DEFAULT b'0',
  `marked_for_review` bit(1) NOT NULL DEFAULT b'0',
);

CREATE TABLE `card` (
  `ready` bit(1) NOT NULL DEFAULT b'1',
)
```


Hi, what is this merge? :-)


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


You should use `extension` to make your code more clear. Example:

```Swift
extension TweetsViewController: UITableViewDelegate, UITableViewDataSource {
    // Implement methods of table view
    func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        // Your code
    }

    func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        // Your code
    }
}
```


Double lines.


@nicklockwood i think we should expose this via actionsheetios.js also.


IntelliJ likes Collections.singletonList better.  also, no need to specify Double in the angle brackets.


Yeah.. already we have fox :-D :P


This shouldn't be hit often (at most once per run, and we should move the corrupt DB immediately afterwards, so ideally once ever). We want all the deets we can get…


Please remove these blank lines to make your code nice.


@vlj Doesn't seem to fix Terraria. I thought it might have to do something with Terraria, since it uses texture format R5G6B5 for the swizzled main menu (and character creation) textures, so it might be somehow related. Will submit my PR, once the other PRs are merged a bit.


Why does this export `leonardo`? ;)


Vivek, marked_for_review has a default value of 0 (false) and the other parts of code (PHP, JS) assume that this value is either 0 or 1. Is there a compelling reason to remove the not null constraint?


Why did you rename this, out of curiosity?


`integrity_check` is documented to return a cursor over a one-column virtual table of log output.

That we're getting status and statusMessage implies that we're getting a failed cursor, and can't even run integrity check because the DB is too corrupt.

That's not what we expect.


Are you sure we need this module?


Hi Markus;
Looks interesting. How well tested is this? Do you have this running in production somewhere so I can have a look at the result?

frank (ao dev)


good :+1:


Looks good.


Non perferttamente funzionante


Cool :D


Update `etcd`? But that may be fixed in a follow-up PR.


I suppose instead of "and we highly recommend it" it could be changed to "the current default".


Thanks! However I don't think your patch will work because this line is reassigning the `$` variable, regardless of what's passed into the [IIFE](https://en.wikipedia.org/wiki/Immediately-invoked_function_expression).

It probably needs to be changed so that the IIFE is passed `(window.Zepto || window.jQuery)`, something like:

```javascript
(function($) {

    // ...plugin code...

})(window.Zepto || window.jQuery);
```


It should be apiservers. Thanks for pointing that out.


@larskanis Some of these tests are failing on Windows. See appveyor build https://ci.appveyor.com/project/flavorjones/nokogiri/build/1.0.130/job/u2395u0bhti3x6ih

```
  1) Failure:
test_path(Nokogiri::XML::TestNodeEncoding) [C:/projects/nokogiri/test/xml/test_node_encoding.rb:56]:
Expected: "IBM437"
  Actual: "UTF-8"

  2) Failure:
test_name(Nokogiri::XML::TestNodeEncoding) [C:/projects/nokogiri/test/xml/test_node_encoding.rb:51]:
Expected: "IBM437"
  Actual: "UTF-8"

  3) Failure:
test_encode_special_chars(Nokogiri::XML::TestNodeEncoding) [C:/projects/nokogiri/test/xml/test_node_encoding.rb:41]:
Expected: "IBM437"
  Actual: "UTF-8"

  4) Failure:
test_get_attribute(Nokogiri::XML::TestNodeEncoding) [C:/projects/nokogiri/test/xml/test_node_encoding.rb:13]:
Expected: "IBM437"
  Actual: "UTF-8"

  5) Failure:
test_content(Nokogiri::XML::TestNodeEncoding) [C:/projects/nokogiri/test/xml/test_node_encoding.rb:46]:
Expected: "IBM437"
  Actual: "UTF-8"

```


I think it works if you replace `=` with `: `. The former is special template syntax which doesn't support arrays, the latter is standard yml.


When #17213 is merged, this is probably unnecessary


What POSIX systems doesn't have a working Bourne shell implementation ?


It's "coming," not "comming." Isn't this idea like Totem by @williamtdr? o.O


I've opened https://github.com/sparklemotion/nokogiri/commit/895c84936faa33ae29ac80af6aec49f08f2034b5 to track these failing tests


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/22a03582714180d72f95beb4d7ff86b6293367ae


merging basho/riak_kv/fix/az/move-ddl-validation-to-riak_ql_parser = 22a03582 into borshop-integration-1279-fix/az/move-ddl-validation-to-riak_ql_parser


basho/riak_kv/fix/az/move-ddl-validation-to-riak_ql_parser = 22a03582 merged ok, testing candidate = 2a433259


`s/popultated/populated/`


I've opened https://github.com/sparklemotion/nokogiri/issues/1387 to track these failing tests.


This is the notation spec (and soon the rtype parser).

See [rfx](https://github.com/ericelliott/rfx) to see how it's used in live code.


Do I actually lose my mule? It'd be nice to let the player just select another tile


IMO, we don't need this in the replication phase, we can just use `clusterService.operationRouting().shards(clusterState, shardId.getIndex(), shardId.id()).shardsIt()` with the resolved request shardID. All the sub-classes, seem to do the same thing here. In https://github.com/areek/elasticsearch/commit/73303ecc481383700688bd1560ebd15d1d8015fc, the replication phase operates on all the shard copies of the shardID according to the routing table, including the relocating ones, if they are not local.


e.g. `1.1.2`?


host name => hostname I think is more commonly accepted


Shouldn't be `delegate.allowUser(iDocument, iOperationType, iUserName)` there ?


@pedrobaeza You'll be interested in this one. I discovered that for running tests on the browser, you just have to write `openerp.Tour.run("tour_id")` in the console.

Also the priority=20 in the template fixes a inheritance conflict.