Sender assertion has been simplified.
But for some reason getFrom is a list and Iterating over a list is not much longer than getting the first element for comparing with expected string IMHO.


Oh, 5c177d976111944f9e6921358192215a6ad0ecbc, now it makes sense :)


reack


And here — could `if isinstance(child, Argument):` be used instead?


woq好像每次都改容易出一堆问题………………我们找个别的方法吧


Nice change


我知道了………………可以提供下载的…………


@AGritsevich, please simplify this if/else/if logic. I propose next (but possibly you'll invent something better)
```
if (!timeout && timer_retry_sequence_.isRunning()) {
  timer_retry_sequence_.suspend();
  return;
}
timer_retry_sequence_.updateTimeOut(timeout);
```


@AGritsevich, `0` -> `0u`


Yes @XhmikosR I'll tackle that soon, they are some warnings but all tests passed


@AGritsevich, 0 -> 0u


:+1: 最近卡OJ太可啪了


It’s not, given that `spritesh` is no longer a Ruby executable. And `executables` only expect Ruby scripts, not anything else.


ack


ダメです


unack .ci


What was this part exactly supposed to do?


unack, didn't you want to correct this?


Is this one still needed then?


I don’t see the point in replacing the Unicode in the documentation with ASCII, but I’m not going to nack the patch for it.


unack .misc used


unack .misc, you should grep for it


I think this was exactly what we do for ExecStart in the service file, but for init script in sysvinit systems.


unack


Any reason to have it depend on python3-sphinx `>= 1.3` ?
That means it can't be built on trusty which has version 1.2.2, generated docs still seem fine with that version.


What do you think about:

```coffeescript
@localAction "showSharePopup#{_.capitalize(option)}"
```


Ok, there was actually a mistake on my side: I had overlooked that your commit was 26 Jan 2015 (!) and not 2016 as I had mistakenly thought. That had me believe that this was a current project being worked on. Thanks anyway!


This class assumes that the particle icon will be the same as the item/block icon?


We have an implicit assumption that the sets of clock values are disjoint. This is more opaque now they are in separate files for each board, so it would be nice to remove it by checking whether the  current clock value is okay for the current board, and recording somewhere what the old board was rather than guessing based on the clock value. But don't worry if it looks like too much work.


The comment of this method states that a default config is returned if none is present in the database, but it seems this branch actually returns an error.


Not all icons are 16x16, they could be 32x32 or 64x64, this will fail then.


Nice change


In general I'd clean this method up a bit, handle errors and let the normal code execution flow through. It's a bit hard to see here because the success clause jumps through the error else branch.


What happened to this case (restoring default?)


Why the change to span? Guess it makes sense, just curious/


extracted as a public static field


Use rem-calc.


What happened to this class?


These error messages now mention `A.M` twice.


I would introduce here some variable. I don't know where 40, 26, and 20 refer to.


side menu left movido para o topo.


it's not clear to me why it's better. From a performance POV it doesn't look better since the docs may not all be in the cache right?


This is a github program i am creating i dont know.


Yes I called it, but what's causing issue is the `toggleClass('placeholder-changed', this.value)`
It's because the 2nd parameter needs to be a real Boolean (cf [doc](http://api.jquery.com/toggleclass/#toggleClass-className-state))
Here, it will do the `toogleClass()` each time a key is pressed.

As for the utility of the plugin, I agree it shouldn't be useful anymore, but until my company officially says that we drop IE8 support, we'll continue to use it (nice work btw, it's the most simple placeholder plugin out there).
So don't bother, I'll keep using the previous version until I can get rid of it.
Cheers.


Waiting for @mattheworiordan to tell if this is indeed what should happen.


Согласен, PositionType красивее, поменяем.


This will cause troubles for the test network, because anyone whose running the testnet will get the mainnet homestead block set.


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


No, I was suggesting to gather all changes regarding the introduced jmap test fixtures in a single commit, prior to feature implementation commits, like regular TDD workflows.
Mixing them would lead to flatten the whole commit history without any logical step.
If you both agree that the PR would be better as a single commit, then I'll squash all changes into a single commit (way easier for me).


` <i className='icon-document' />`


How about setting these object as _constants_?


I removed it later.


Faute d'orthographe


self.master.reconfigure copy the new config from master._config and overwrite any given field in it.


Hi...not sure that my msbuild-fu skills are better than yours. Sorry :-)


This is just a preview. And this line is could have some controversy, i know. But if our goal is to provide only support for netcard, also the arguments does not take sense.


I have made some corrections - no need for "export" and "&&". ^__^


Did you forget to use `$topic-avatar-size`?


:+1: looks good thanks @timler


Merged in.  Thanks!


If you use JUnit4 tests, you should also use the JUnit4 `org.junit.Assert.assertEquals`


reack (temporarily removed pytest and coverage as no tests are present)


Are we using it whithout "netcard"? doesn't seem so.


Even if it doesn’t affect code coverage as measured by landscape.io, I think it’s a good idea to keep these tests, as they still test different code paths — just that now, the paths they test are data-dependent rather than control-flow dependent. (See [Wikipedia](https://en.wikipedia.org/wiki/Code_coverage#Parameter_value_coverage).)

In order words, even though the new code achieves the same functionality as before in fewer lines (which is great), it might misbehave when passed certain inputs but not others, despite having the same control flow in both cases. I know the code doesn’t do this, but we can demonstrate that by keeping the old tests in.

Another way of demonstrating what I mean is with a simple example. Consider these two bits of code:
```
if in == 'hello':
    return 'hello-world'
else:
    return 'bye-world'
```
and
```
return in + '-world'
```

If we test the two with inputs `hello`, `bye` and `another`, we get full code coverage in both cases, and it would be tempting to remove two of the test inputs for the second version of the code, since they seem redundant. But obviously the two bits of code return different results when given `another` as input — this is data dependence, as opposed to control dependence. If we keep the existing unit tests, we can at least guarantee that doesn’t happen for the known inputs which we’re testing.

Note that I’m not saying your refactoring is buggy — I’m just saying there’s no reason to remove the tests. They’re not doing any harm, and could be useful for later refactoring.


Use „” instead of ''.


oh ok.btw any news on 6.0?


This coud be droped.


HUEHUEHUEHUE


ack


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2118 of 2122 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 16 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SHD diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


Why frame?


Where has the `Missing required attribute ‘name’ in method.` log message gone to?


Well, if the code is commented instead of removed, it could be that it's to allow people to "tweak" the example. In that case, it would be good if we could define command line arguments for those stuff.

I don't like this very much either, but I think we can live with that for now. Maybe in the future we could provide a way for examples to define command line parameters.


Still doesn't work for me. When I adjust TouchableNativeFeedback.android.js like this, it seems to work most of the time, but still animates the ripple in weird locations sometimes:

```
_handleResponderMove: function(e) {
    this.touchableHandleResponderMove(e);
    this._dispatchHotspotUpdate(e.nativeEvent.locationX, e.nativeEvent.locationY);
  },
```


@kamilklkn It looks like you're using a different version of OpenCart. So this change is not relevant to you. It's specific to the Tesitoo project.
Are you sure this repository is the right one? The [core OpenCart repository is here](https://github.com/opencart/opencart).


ack (re possibly, I'm just going through everything since gitmate doesn't work :()


@licaon-kter can you please review this?

http://www.netguard.me/NetGuard-v0.90-beta-2016021901-all-release-legend.apk


ack, though I don't think the release hook works :p


Original master branch merged


I think _it_ is missing → I squerbed it as


They weren't there in the initial implementation, no reason but that would be a separate piece of work.


This is a behaviour difference. From my reading of [the spec](https://dbus.freedesktop.org/doc/dbus-specification.html#introspection-format), I don’t think `value` is required for `annotation` elements, because the spec provides default values for them. But it is a bit vague. I would prefer to keep the existing behaviour of dbus-deviation (not requiring `value` attributes in `annotation`) unless you’ve got a compelling argument otherwise.


Sandy, nice attempt! So if you look at your event listeners when you pass in `getValues`, what does that variable represent (lines 49 & 59)? It should be named appropriately. Should you be using `==` to compare the `keyCode` with the number or `===`?  Also, what is `.click()` doing? If you look at your `button` event listener (line 70), which we know works, it is calling a function. If that is the function we need to call to begin the process of growing the tree, where do you call it in your new event listeners? Last, what happens when someone clicks something other than the enter button? Try to work through those issues and resubmit. You are very close!


Extracted


True.


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I can yes


I am happy this simplies the code but wary to accept immediately. I am a beginner at github and still learning !!. I accepted your proposal #3 for sanitization but later found it had issues. I have now fixed them and uploaded other versions of the code eg w4 in the master branch, and y2 in branch 8y.


ack, from coala largely I presume


Not sure, but I thought that all shares should point to frame (to content). It is easy to change that.


Yes


completely :P yea


ack


fi.lang ,what is that language ?


Hm... I guess one possible problem is that some platforms don't provide a shell for the user, right?


When I installed RASPA2 by “pip install RASPA2”，and it succeed.But when I tried to run the example 1 "MC_Methane_in_Box" by using "simulate simulation.input", something wrong happened. The simulation hangs or "stops" at the initialization. I don't know how to fix this problem.thanks a lot.


This was copied from new-squerb. I will change that.


It might be frame I was just wondering.


Are these changes an issue in order to be used by LuaJIT? As it's documented [here](https://rawgit.com/LuaJIT/LuaJIT/master/doc/extensions.html#lua52):

*Note: this provides only partial compatibility with Lua 5.2 at the language and Lua library level. LuaJIT is API+ABI-compatible with Lua 5.1, which prevents implementing features that would otherwise break the Lua/C API and ABI (e.g. _ENV).*

Or are all of these changes not breaking the compatibility?


No real reason, I copy pasted the doc and didn't prefix the string with u in the first place, so I just removed the unicode character.


OK, added twice (also getOutbox())


This should reference ipcRenderer, right?


I've patched 3.7.11 and it **works**.


jshint failed: https://travis-ci.org/Automattic/jetpack/jobs/110318570


Indentation looks a bit off on this line.


Unfortunately this is breaking the setup. For some reason, when executed in a regular HTTP context, `CallContext` "loses" its data at the end of the request, causing recipe execution to fail. Until we understand why this is and how to best deal with it, I am reverting this file so that it at least works.


:smile:


수정했습니다.


Isn't "uniform mat4 trans_model_to_world;" the same as "uniform mat4 p3d_ModelMatrix;" ? Wouldn't using the late be more consistent with all the other p3d_* inputs?


Tests can be technical debt too :) Too many of them are actually an obstacle to refactoring IMHO


That occurred to me. Good then


LGTM


Vada per antitracciamento, però ho ancora qualche dubbio sul fatto che
anche questa opzione possa essere fonte di possibili fraintendimenti.


As per my comment below in the test code, this causes the method/signal name to be duplicated in error messages. I suggest removing the parent name from `pretty_name`, and adding a second property, `pretty_name_full` (or something) which includes it.


Пробелы в условии явно лишние. Могу списать на "стайл", но тут он явно лишний. Кроме того, в коде он соблюдается не везде. В строке 50 уже не соблюдается. Линт на это не ругается, но лучше подобную практику не использовать.


А это случайно или в этом есть смысл?


Because InterfaceComparator needs refactoring too, which is out of my scope here, I didn't consider this as a regression though.


Most-read (popular) endpoint is also cache 15 mins


Shouldn't this also cover daisy boards too?


Does not PMIx_Finalize() guarantee internal barrier?


Yes, I see the commit history, no problem.


<i> isn’t the icon tag, it’s the italic tag, for making text italic. <span> is more semantically correct, because it just means a section of text.

Also just because I said so. This isn't a democracy it's a benevolent dictatorship.


i am learning Full Stack Web Development from Coursera


Is it significant that this is `1000` instead of `1024` like the others?


ack


The `'&'` is really unnecessary here, let's drop it instead. /cc @tde-banana-odoo (introduced at d025251c7e4ee4c26462e61b5d63e5c5d58689f8) BTW we have `expression.AND()` when you really want the AND to be explicit ;-) )


How do you set `self.clipping`? Could be useful to add an argument to `__init__`.

Is the variable name `self.clipping` clear enough, or would `self.topo_clipping` be better?


@imcaffrey 是的，就是听说了一种新的方法。。


There's a new model, but no migration, forgot to "git add"?


This content is actually the product-block directive not the slider. Please remove it to the appropriate folder and name the file correctly.


use bindToController instead of this isolated scope,


After calling `CREATE_CLASSROOM`, `classrooms` in state is undefined. Why?
![screen shot 2016-02-19 at 11 55 44](https://cloud.githubusercontent.com/assets/8989606/13175077/d2f903d6-d6ff-11e5-8af8-47c939129ede.png)


Why removing this alias?


That leads to APIToken.create_apitoken(). would be more natural to have a APIToken.create()


What about a 3rd option of having the xsl actually inlined here?


Remove comments from the code.


Ah, I see you added `ParsingLog` for that. That means that the entries generated by `Log` and `ParsingLog` will be tuples of differing size, though, so they can’t be handled uniformly. I think the entries in `_create_entry()` here should be standardised as 4-tuples, using the format from `ParsingLog` (which is the same format used by `InterfaceComparator` too).

In future, I think it would make sense to use an object, rather than a tuple, for each log message, since then we could expose arbitrary extra data as properties. However, that would be an API break and can be done later.


You are absolutely right. Good catch.


Following the link provided by @baykovr .. it use groupOfNames, and after I add groupOfNames, I tried very hard to figure out why it doesn't work... I have no idea whether to use groupOfNames or groupOfUniqueNames either, but perhaps some document which point out memberOf will only work for groupOfUniqueNames by default would be better?


Вот такого рода конструкции:

`if (reviewMark.value < 3 ) {
      reviewText.required = true;
    } else {
      reviewText.required = false;
    }`

Лучше сразу привыкать сворачивать, иначе код превращается в "простыню". Например, можно записать так:

`reviewText.required = reviewMark.value < 3;`


This is again not special product but the product-slider directive. Please move the code to the correct file/folder.


No compelling argument, but I considered as it was a required argument for construction of the Annotation (https://github.com/pwithnall/dbus-deviation/blob/master/dbusapi/ast.py#L283) , it was supposed to be a required attribute in the XML as well.


Почему для сохранения не используются возможности библиотеки?


As I mentioned before do not use tabUrl in the directive. Directives do not make Ajax calls or do not depend on other services. They receive all the data they want through bindings in their attributed.


Che è anche come era stata tradotta nella vecchia versione, però a me l'aggiorna semplice continua a sembrare un po' vago, a me l'aggiorna fa più pensare a un aggiornamento software, cambiare l'OS è una cosa molto più  radicale.
Comunque OK.


Why isn't this merged yet?


```
<img src="img/slider-picture-1.png"  width="1045" height="881" alt="Крем-брюле и пломбир с малиновым джемом">
```
Надо сохранить, контентная картинка!


Этого @bykoianko писал не я. Думаю, добавить реальный прогресс при загурзке, а не 50


Inicio del proyecto


Also, do check if "msm-mem-hole" memory hole is used by any driver.


For Travis to work your answers have to be valid YAML
```
# this
answer_1: 1234
# not this
answer_1= 1234
```
```


Indeed, I had run autopep8 before adding that test.


log_debug() and friends att the trailing newline implicitly. Please do not include it in your string literals.


This can’t be static, otherwise the same issue codes will be used by logs from multiple domains and can’t be differentiated. If we started calling `Log.register_issue_code()` from the `InterfaceComparator` code now, the issue codes for that and `InterfaceParser` would get mixed together and `InterfaceParser.get_output_codes()` would start returning codes from `InterfaceComparator` as well as its own.

Make this method non-static, and do the `Log.register_issue_code()` calls from `Node.__init__` on its `log` instance.


two words


IMHO, InterfaceComparator should be simplified instead.


I'd just set it directly via `ws.clipping = "head"` for now. Yes, maybe `self.topo_clipping` is a better choice.


This is a big change: now `contentdeliveryrepos` points to current url, and `repos` to the deprecated one. All occurrences of `repo-list` should be changed to `contentdeliveryrepos-list` and `repo-detail` to `contentdeliveryrepos-detail`. There are some in docstrings and in tests. Then it will also fix PDC-1340.


Thx, this looks perfect. Sorry it took so long.


What this fixed?  I must update again?  😁


@gabrielamatias :+1:


> ale w przyszłości chyba będę tu ustawiał jeszcze

W takich sytuacjach YAGNI: czyli na razie nie miałeś potrzeby kombinowania, to przekazanie urla jest prostsze i nie ma się co bawić w opakowywanie dodatkowym obiektem.

> user agenta aby mieć pewność, że nawet w przypadku odpalenia apki na telefonie pobrana zostanie desktopowa wersja strony. Dobrze myślę?

wat? jak chcesz pytonga odpalać na telefonie? ;-)


It's Finnish


If my understanding is correct, this should leave custom config files that are converged by other recipes. Are you putting other files there manually, or is this mechanism not working?


On the first reading the ```End``` in the names of the two methods above felt like a verb - _to end_, for that reason I'd prefer the method names to be: ```setExecutionErrorAndEndTimestamp``` and ```setExecutionErrorAndEndTimestamp``` respectively.


This is working in my environment. Can you please explain the problems you are seeing?


See my other comment about this (on the `pretty_name` property). I think it would be best to hold back on adding the ‘ of A.M’ bit to `pretty_name` for now, and add it later as a separate commit once we’ve worked out the best way to do it (without refactoring `InterfaceComparator`).


Motion binds your editor and running application.

At any point, you can right click your application while in Intentions mode to see the view hierarchy for that element. You can easily jump to it's definition in your editor to start editing it immediately.


ok I see


hello again


My understanding is that 'package' should automatically use windows_package on Windows. What problems are you experiencing that require this change?


Indeed ctype_digit() is much more reliable. Also for custom columns.


All code can be technical debt, but tests are not technical debt in the case where you’re trying to maintain behaviour and API stability. Only in the case where you’re breaking API — and in that case, you can make the decision at the time to remove the tests as part of the refactoring. Please keep them.


This is gonna get called a lot of times though


I'm changing this slightly. I think it would be better to already provide a define, that's default to 0. I'll send an update.


@malirod, it's look like beautifying and code refactoring. I didn't create this method, so what you ask take from me too much time. OVERRIDE I did because it was easy and fast. I won't do it.


Ah, that looks like a bug in the previous implementation. Make it an optional argument for constructing `Annotation`.


https://docs.python.org/2/library/collections.html#collections.namedtuple comes to mind ;)


yup


The change is correct and on topic, the internals of osgShadow were leaking GL objects, to prevent this I changed some of the internals and to make sure the appropriate releaseGLObejcts() methods were implemented I made ShadowTechnique pure virtual.


That’s not a job for now, though. Drop the `pretty_name` changes and they can be done later, separately.


Most-read (popular) endpoint is also cached for 15 mins.
What do you think @stephanfowler?


Here's the updated grammar: https://gist.github.com/autoboosh/6241308e0c3222a5e8e0.

I'd remove all WS/NEWLINEs from token definitions at the end (it's acceptable IMHO if they are between two tokens since it is one atomic unit):

```
MACRO_CONST : '#' C O N S T WS;
MACRO_IF : '#' I F WS;
MACRO_ELSEIF : '#' E L S E I F WS;
MACRO_ELSE : '#' E L S E NEWLINE;
MACRO_END_IF : '#' E N D WS I F NEWLINE;
OPTION_BASE : O P T I O N WS B A S E WS;
OPTION_COMPARE : O P T I O N WS C O M P A R E WS;
```

This WS should probably be a WS?, since per [VBA specification 3.4.2 Conditional Compilation If Directives](https://msdn.microsoft.com/en-us/library/ee199689.aspx) `EndIf` as well as `End If` are allowed.

`MACRO_END_IF : '#' E N D WS I F NEWLINE;`

Don't need the question marks after endOfStatement here, since that is handled within the rule itself.

```
macroIfBlockStmt :
	MACRO_IF WS? ifConditionStmt WS THEN endOfStatement?
	(moduleDeclarations | moduleBody | block)*
;

macroElseIfBlockStmt :
	MACRO_ELSEIF WS? ifConditionStmt WS THEN endOfStatement?
	(moduleDeclarations | moduleBody | block)*
;

macroElseBlockStmt :
	MACRO_ELSE endOfStatement?
	(moduleDeclarations | moduleBody | block)*
;
```

This shouldn't be needed either:

`moduleDeclarationMacroStmt `

Made some other small changes as well.

This parses the examples you provided. It also works with some sample projects I've tried =]


The names of the objects in the API URLs are always plural (projects, test-results, ...). While at it, I think we could rename apitoken to token here. So 'tokens' at the end.


Does it look better now? =)


Oh, can you add a comment where `PHP_FUNCTION(chr)` and `PHP_FUNCTION(ord)` are to warn anyone editing those functions that they'll be bypassed?

See the note I put on our existing special functions, e.g.:

https://github.com/php/php-src/blob/dc33c79a65859d29ea878fb8d8437b7b9cbc0461/ext/standard/type.c#L248


This can be a subsequent step, along with improving the Log with log levels


Sure


Some typos in the commit message which I will fix if anyone cares.


This will prevent the `variable $file1x might not have been defined` error.


No, it doesn't


This will prevent the `variable $descriptor might not have been defined` error.


The unit tests look good! :)


This ensures that `$srcset[null]` is not added.


Currently build id looks like this:
`cordova-android-win-1655-buildbot-vm`


Why did you change


When implementing bigints I got caught out by the special-casing, confused why `is_int` was broken yet `is_long` and `is_integer` (which are both just aliases of `is_int`) weren't. I eventually discovered that you'd special-cased these functions, and to save anyone in future the same hassle, I added a comment to every function like it.


Excellent, c'est une idée originale !


If it turns out to be a performance problem, you could move the `register_issue_code()` calls to a `__metaclass__` for `Node`, rather than `Node.__init__`.


Есть проблема с сохранением данных формы в cookie. Срабатывает через раз. Шаги для воспроизведения:

1. Запускаем приложение. Нажимаем добавить отзыв, заполняем поля, выбираем оценку, жмем кнопку "Добавить".
2. Обновляем страницу. Опять нажимаем "добавить отзыв". Имя, оценка сохраняются. Теперь пробуем изменить оценку и сразу получаем хинт - "Осталось запомнить текст отзыва". Заполняем отзыв, но кнопка отправки остается неактивной;


Hi,
  I get the error: **+-- UNMET PEER DEPENDENCY karma@>=0.9** in below situation.

```
λ npm version
{ 'angularjs-testing-quick-start': '1.0.0',
  npm: '3.7.4',
  ares: '1.10.1-DEV',
  http_parser: '2.6.1',
  icu: '56.1',
  modules: '47',
  node: '5.6.0',
  openssl: '1.0.2f',
  uv: '1.8.0',
  v8: '4.6.85.31',
  zlib: '1.2.8' }
```

package.json
```
"dependencies": {
    "karma": "^0.13.21",
    "karma-cli": "^0.1.2",
    "karma-ng-html2js-preprocessor": "^0.2.1"
  },
```

My steps:

```
λ npm install -g karma
    `-- karma@0.13.21
λ npm install -g karma-cli
    `-- karma-cli@0.1.2
λ npm install -g karma-ng-html2js-preprocessor
    +-- UNMET PEER DEPENDENCY karma@>=0.9
    `-- karma-ng-html2js-preprocessor@0.2.1
    npm WARN karma-ng-html2js-preprocessor@0.2.1 requires a peer of karma@>=0.9 but none was installed.
```
What else I should install?


I need to spend sometime w/ it.  Prob next wk

Sent from my mobile device.
On Feb 19, 2016 1:56 AM, "Jorge Ferrer" <notifications@github.com> wrote:

> Thanks Mike. Is that something that we can do so short term to fix this
> bug?
>
> Also, @drewbrokke <https://github.com/drewbrokke> do you know if the db
> search takes permissions into account?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drewbrokke/liferay-portal/commit/bade8768388b436751a80a5ca1a51e202d8c3274#commitcomment-16196826>
> .
>


Hi guys,
it's possible have this function filter on leaflet like this on mapbox?:
https://www.mapbox.com/mapbox.js/example/v1.0.0/markers-with-multiple-filters/
I want make same one with a button show all and show by field value in json. My button filter are temporal filter so I want show more layer (points, lines, polygons) by time (only year it's ok for me).
Tks guys,
Best
Valerio
deluca.valerio@gmail.com


please don't destroy your input - rather create a new SnapshotAction without this


Might be better to do:

```
clone := &entry{}
*clone = *e
snapshot.dirty[k] = clone
```

so as to handle any future change that adds additional members to **entry**


That would also be suitable.


Se non sei pienamente convinto, per me possiamo anche lasciare Do Not Track.
Dopotutto questi snippet hanno lo scopo di invogliare l'utente ad informarsi e approfondire le cose, quindi può essere una buona cosa che si vadano a fare una ricerca su Google. ;)


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (983 of 988 tests passed, 0 test warnings)

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


this is very hard to read - please write a method returning boolean: something like: contansSnapshotWithState(Collection<Snapshot> snapshots, SnapshotStatus status) and use it in a simple if-else block here


Yeah, log levels would be needed to implement this. Doing it as a subsequent step makes sense.


The led Red in pin 13
the led orange in pin 12
the led green in pin 11


New line for brace, please


Проверь еще раз код.


`wfMsg` -> `wfMessage`


@AGritsevich, i think that `(` around the isRunning is redundant


@ayrton could you publish a new version w/ this change? :)


wait what??? it's literally a hashtag!!! ahaha


@neobuddy89 only in arch/arm/mach-msm/msm_mem_hole.c
But this is clear that it is used there.
Problem is that samsung uses different sizes and addresses. If I replace the values with yours (without this commit) I have the same problem


Not the right place to ask, @valeriodeluca.


can you please Align in proper way ?


> What about a 3rd option of having the xsl actually inlined here?

I though about that, but I personally do not find it very practical. I prefer editting standalone XSL files to get a full editor support. But the present proposal can theoretically be enhanced later so that something like this is possible:
```
            <transform>
                <xsl:stylesheet> ....</xsl:stylesheet>
                <source path="subsystem-templates/my-subsystem.xml" />
                <destination path="subsystem-templates/my-subsystem.xml" />
            </transform>
```


without the hashtag it would mean the name of the node.  I figure its more readable that way. :)


Changed them as the camelCase.


Why do you think I sent out https://github.com/fail2ban/fail2ban/pull/1327 ?


I remove that later.


@dblotsky PTAL, I've addressed your notes:
* divide startAppiumServer() to startAppiumServer() and startIosProxy()
* make one killProcess() function instead of killIosProxy() and killAppium()
* use camelCase in runner arguments
* make appium server path a global var
* simplify getLocalPluginDirs()
* describe --buildId arg via optimist
* pass a build id from CI to the tests


What's the goal of this commit? Disabling certificate validation?


That part should probably part of a previous patch


The histogram now pulls its data from the residuals rather than the actual values.


The `markdown="1"` attribute allows kramdown to undestand that embedded content is again markdown. File preview on github does not take it into account but jekyll+kramdown does, and it's pretty useful.


LGTM


For me would be better:
https://commons.apache.org/proper/commons-io/apidocs/org/apache/commons/io/IOUtils.html#copy%28java.io.InputStream,%20java.io.OutputStream,%20int%29

or

https://commons.apache.org/proper/commons-io/apidocs/org/apache/commons/io/IOUtils.html#copyLarge%28java.io.InputStream,%20java.io.OutputStream%29


I'd use "use strict"; here and make sure that file passes jshint (see tests/test_jshint.sh)


Is this intended to be a complete list? If so it's making an odd obligation to keep it up to date. If it's more a greatest hits, the word "including" should be crowbarred into the sentence.

Also I'm in two minds about the implied strict alphabetical order as I can think of a situation or 2 which might result in accidentally bumping @crowbot down the order, which the people whose names cause this might not be happy with.


(it doesn't there are missing ';' for instance)


This whole block (L870-L873) can be simplified to:

```js
if (info.result.nModified !== undefined) {
  cbInfo.isNewInstance = info.result.nModified === 0;
}
```

What does an older version of MongoDB return in `info.result` object?


Regarding this being a constant:
We currently require the script to be executed from the root directory where all of the plugins and medic are located, so I think it's fine to use a constant here.


Please use `is` instead of `==` for comparisons with `None`.


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


New Boston icon


As a side comment, I think we could depend on mustache (https://mustache.github.io/) for client side templating instead of using jquery to create DOM nodes. But that can be done at a later point.


@Amir-61 Could you please add a comment here showing the different values of `info.result` objects, similarly to what we have elsewhere:

```js
      // create result formats:
      //   { ok: 1, n: 1, upserted: [ [Object] ] }
      //   { ok: 1, nModified: 0, n: 1, upserted: [ [Object] ] }
      //
      // update result formats:
      //   { ok: 1, n: 1 }
      //   { ok: 1, nModified: 1, n: 1 }
```


Thanks!


Do we do this in enough places that it might be worth making into a function? `zend_string_init_one_char()`, say?


I see that `methods` is read only here, but in general mutable defaults are considered a no-go, so maybe use `None` instead of `[]`?


Thank you!


Incidentally, this file doesn't have executable permissions set.

Needs `git update-index --add --chmod=+x update.sh`


[numpydoc](https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt#docstring-standard) uses `:` to separate parameter names from types.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/python/test_files/pylint_test.py`, line 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/python/test_files/pylint_test.py
+++ b/bears/tests/python/test_files/pylint_test.py
@@ -5,6 +5,7 @@

 class Something:
+
     def method_without_self():
             pass

```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/python/test_files/pylint_test.py`, line 9.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/python/test_files/pylint_test.py
+++ b/bears/tests/python/test_files/pylint_test.py
@@ -6,7 +6,7 @@

 class Something:
     def method_without_self():
-            pass
+        pass

     def something(self):
         pass
```


LGTM


Oops, I have to remove this code. Murilo already landed a patch fixing this problem.


bitbake's must be unique. Adding a bitbake with the same name will break the build.
enigma2-plugin-extensions-dflash.bb and enigma2-plugin-extensions-dbackup.bb allready exist!
https://github.com/oe-alliance/oe-alliance-core/tree/3.2/meta-oe/recipes-distros/openatv/plugins

If the ones from atv are not suitable for you, you have two options:
a) rename the bb's and resulting plugins
b) add your bb's to your local meta-local/plugins/


sol_str_table_lookup() is a macro


it looks like your tab settings are different from the default java.


This is just `dict(domain)`.


@NealJMD extra coverage added, but checking that the counter is called for all of these permutations isn't at all necessary, and it's duplicating the work of existing specs. I'm only adding them so we can get our release out the door.


This is not required


> Is this intended to be a complete list?

I don't think so. We link to github contributors later. "Including" sounds good

> Also I'm in two minds about the implied strict alphabetical order

I don't think this is a major issue. The main thing is that the page is more up to date than it used to be.


I know. It was a compromise between performance and consistency/stability, since the cache would be out of synch.

I did it initially with the Hibernate query using the store API, but then noticed the caching issue. It was already pretty not-clean to use the store API, now to also evict the child documents from the cache I would probably have to do something hackish like:
```
if (context.getWiki().getHibernateStore() instanceof XWikiCacheStore) {
  XWikiCacheStore cacheStore = (XWikiCacheStore) context.getWiki().getHibernateStore();
  // Make sure cache is initialized
  cacheStore.initCache(context);
  for (DocumentReference childReference : oldParentDocument.getChildrenReferences(context)) {
    // Avoid getDocument, just instantiate an XWikiDocument object to use with the cache store.
    XWikiDocument childDocument = new XWikiDocument(childReference);
    String cacheKey = doc.getKey();
    // Remove the child document from the document cache to make sure it's re-cached when it will be first needed.
    cacheStore.getCache().remove(cacheKey);
  }
}
```
I eventually went for the cleaner solution, but am not sure.

WDYT? Performance-wise it should be great, and it will also not generate the useless new revisions on the updated documents, but would it cause maintenance issues?

P.S.: Events will also not be triggered anymore for the child documents (since I would be using the store directly), but I guess that won`t be a big problem since listeners hould not really care about the parent field updates.


No.. it is required.


Should this have been `grunt` instead?
(Line 36 & 38 using `grunt`)


rather remove the print as per discussion on gitter


forbrede?


comment


kodestandarer?


szerintem jó lesz, maradhat.


biztos?


Would you add `README.md` and `CHANGELOG.md`? Thanks.


igen, hagyjuk, jó ez így.


Added documentation and default.


indent


Not entirely correct that it is not version dependent but I believe we can live with this comment for now

My opinion would be to think again about the utility of this file altogether

Otherwise, LGTM


fixed


ack


LGTM


We don't need the product object but we need the productID and type of specials like hot-offers or top-picks and etc.
I deleted the "product" and added the "specials".


fixed.


I think it is version independent at this moment. Why do you say it is dependent?


Does shadowsocks has polarssl-specific optimizations ? Otherwise we could just swap the openssl argument to whatever ssl lib the caller prefers.


What this function is doing correspond to : java.lang.Character#isISOControl(int)
(except for 160 that java does not consider as control character)


You break shitty code... I dont think this is a problem if the other code will work even better.


@bryk I guess google-closure-library was not yet updated to support version `20160208.1.0` of closure compiler. Without this checks closure compiler complains about some JSDoc in `goog/base.js` and other files related to this library.

Should we leave that for now and wait for official release?


`super.other_epoch(epoch)` wouldn’t do?


Hi @TanguyPatte .

Thanks for reporting this. I've commited 8ff62ab which should take care of it. It will be part of 4.1.2 when that comes out of beta.


LoL! Rspec mode feelings!


Why cannot `initchunks()` simply do a `memset()` with zeros? Then the caller don't have to do the extra zeroing?

Some targets may even have a `alloca()` with mem-clearing for such things?


@superbobry Thanks for reviewing. Want to do a quick fix up?


Uh, what? Expecting the RNG to work predictably is reasonable behaviour.


:+1: I was struggling with naming here


I just want to show my appreciation for the language maintainers here as a very heavy user of PHP and part of a team that maintains a huge amount of legacy code. I'm glad that I can continue to allow security and other non-BC changing updates to happen automatically and I can check the change logs only when doing major version updates. I need this level of stability and maturity in a language instead of having small things subtly break in code I've never read and don't understand and me wasting days wondering why, or weeks trying to rewrite, all while the business continues to suffer.


This and https://github.com/floreks/dashboard/commit/603251b1d88ed04ec9dd96ff8548993377c6ed2f#diff-79bc753a0452c67d07e93daaa546c427R65 was changed in favour of this update: https://github.com/google/closure-compiler/issues/1319


Yep, need to take care of that :)


This should probably just be `return TOX_VERSION_IS_COMPATIBLE(major, minor, patch);`.


added.


Reviewed. Please merge


Scalastyle doesn't like type signature to be on an independent line. Then, it has to surpass the 80 chars limit.


fixed.


Same here.


I did not add this code (main.run.js). It was before as default.
Deleted it.


Not sure how this crept in.  I deleted it.


This is exactly `TOX_VERSION_IS_API_COMPATIBLE`.


The commit message should be more descriptive. Without the context of the PR it doesn’t make much sense.


Again, the commit message should be more descriptive.


Why are you not calling the callback if attached already?  It is quite common to write code such as:
```
attach(function() {
  publish();
});
```
which is a simple way of saying lets publish a message when attached. I don't see any reason we wouldn't just call the attach callback immediately if already attached?


Could it be update with the list of scenario name? Avoiding user to look at all scenario.


Thank you, nice job :)


I'm glad you found the post useful. Thank you for including it!


As above, I think we should call the `cb`


I expect diffrent values...


Yes, noticed that and fixed already: https://github.com/ably/ably-ios/commit/736652463bc37c931fc68cea645ddfa7979a02e5


Can you explain where this change was necessary? The `attributeNamespace` lookup is only in use for `SvgElement.get()` / `SvgElement.set()` and I don't think they ever receive `'xmlns:xlink'` as the attribute name?


Please add "from builtins import range" for consistent behavior on Python 2 and 3.


I couldn't put it there. It didn't work.


I know why.


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


I realise you have implemented 7366524, but this would be wrong then no?


La columna es la D, no la E


Changed and removed them in the new design.


Moved the matching function to helpers.go from predicates.go


As above, this should call the callback


Yes, it fixes this bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=792711
Cf 7761ce86b6fe149908b90


renamed the function as per the suggestion


fixed spacing issues


renamed the function name


this method is moved to api/helpers.go


Just check the certificate before really send the request, after user accept the cert, the UIWebView will trust the cert. Otherwise fail the request.


added the Node to the struct as per the feedback.


@IvanNikolaychuk, why not `Arrays.asList()`?


Oh my, that's correct. Why didn't I think of this! Excuses!


Done.


Added  more test cases to test various permutations / combinations of tolerations and taints


And wouldn't it be nice to still have a reference to the author of it, since it is just copy paste :)?
https://c0dart.wordpress.com/2015/12/03/sprawling-networks/


I changed them before but I don't know it is still here !!!!!


Yes, because I saw no point in exposing it in the UI, I did not add the extra request parameter. It`s not about you not using the parent-child relationship, it`s also about other apps that might actually (still) use it. It might still be a good idea, to add it, just in case.

About the steps, I did want to do this initially, but noticed that this operation should be done only when `isDeleteSource` is set (i.e. a move operation), so I ended up doing it under the same step, but making sure to break the step in 2 (the delete and the parent update). I'm not sure I understand that would be the difference between this and doing everything in the top level.

Regarding the progress for each step in the for loop, you're right, I probably should expose that information as well for cases when there are a lot of children to update.


:thumbsup:


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (1062 of 1540 tests passed, 0 test warnings)

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


@zimbatm I'm not sure if there's any optimization for polarssl. Can you elaborate more about what change you are suggesting?


Found a better approach, commit coming shortly


MERHABA


LGTM


It's a success


Same as other controllers, it is done !!!!!


First


Добавил третий коммент


That's a latter improvement though, reverted to a 4-tuple for now


You did it great. Leave the jscomp_off here. We'll remove them when next evaluating flags.


Oh, I see, I did not thought about that case.


Scratch that, that commit is broken. Juggling too many things at once now... b580bc1 and 3cab9940299b33ec62225b3b717728cde9b697a1 should do it.


Agreed. Done.


Done at 3cab9940299b33ec62225b3b717728cde9b697a1.


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


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


Commento per le notifiche.


Yep. Fixed at 3cab9940299b33ec62225b3b717728cde9b697a1.


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


Fixed at 3cab9940299b33ec62225b3b717728cde9b697a1.


These values are not right, they need to be tested.


افرین خوب بنویس


Removed it in the new design !!


For example, if you use an older than daisy poky branch, you may not have some of the recipes there which we are .bbappend-ing from meta-resin-common

That is what I mean that it is not really yocto version independent


I tired to add units,  as there line:
```python
print '''Get the radius of the soma (\xce\xbcm).'''
```
result in my computer as
Get the radius of the soma (μm).

and this line
```python
print '''Get the surface area of the soma (\xce\xbcm²).'''
```
Get the surface area of the soma (μm²).

But it does not seem to work, maybe micrometer should be annotated as um and not μm ?


LGTM


Feel free to take that out. Only included it since it seemed a lot of people were making their OutputDir to /tmp. And things will get confusing quite quickly if they did that.


Approved


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


That is because we do not have an older than daisy version. This layer is "dependent" of the yocto version that we support (the group not one), so I guess in a sense it is dependent.


do not use isXXX in the field name for boolean, but just readThrough, so the getter/setter is isXX and setXXX based.


https://github.com/ResurrectionRemix/Resurrection_packages_apps_Settings/commit/e74f2c44977445cffeaacdd4cdf98f3f58314d61

No wonder that didnt work . I commited this in the wrong location


defaultValue and actual default is not the same


<script >
  window.___gcfg = {
    lang: 'zh-CN',
    parsetags: 'onload'
  };
</script>
<script src="https://apis.google.com/js/platform.js" async defer></script>


typo here!


Another problem with the Hibernate update approach might be in cluster mode, since AFAIK, the other nodes get updated based on events and, if we skip events entirely, only the current node will be updated and it will be out of synch with the rest of the cluster.


Nice find :D


Some of these are HTTP specific (Compact, MinInterval, Complete, Incomplete), but we run into trouble if we remove them because ``http.writeAnnounceResponse` needs them - ideas?


Synchronously setting the bits (using atomicOp) will have a huge performance penalty and should be replaced with metadata queues in each thread cache.
The cache will be the single-writer, and the queues can be flushed by anyone holding the GC lock, that way a single-reader, single-writer queue with no additional synchronization will suffice.


why can you stop drawing the overlay here ?


In fact it is in cm, not meters


Must you use a header for the action? What if you can set a default action in the endpoint uri to use, and then let a header override. I think we do that in the old camel-cache component


It is done.


Sure. In [exportSVG](https://github.com/aschmi/paper.js/blob/develop/src/svg/SvgExport.js#L425) `SvgElement.create()` is called which calls `SvgElement.set()` with
`attrs = {
version: '1.1',
xmlns: SvgElement.svg,
'xmlns:xlink': SvgElement.xlink,
}`


Can this be removed now? Seems it's only used in `fad-helpers.c`. Hence it could become `static` there.


løsningen på xml-problemet er å ignorere avhengighetskonflikter? (-d)


Fix link, please :)


I tried to do it dynamically by getting the name from the crypto library using `builtins.parseDrvName`, but what I got from polarssl was always mbedTLS, which prevented the package from being built.


dont have so many constructors, instead use getter/setter to configure it. As people need to configure and setup this repo from XML DSL in <bean> style.


I am not really sure if this is ok. If the value field is missing, should we discard the header or the whole response?


@Intyre remove the .0? its extra.


The common layer should use bbappends with wildcards making it independent @floion


It is done


I thought both were ways to say it. /me--


There is a new option you need to support
https://github.com/apache/camel/commit/0fbdb0081a5d58b02a96dfeb3d358a158cb0f179

eg setAllowSerializedHeaders


Ah yeah, you're right. Thanks!


Great, Thanks


Not at this time but I need params in the new design.


Can we not provide this log message in the exception so its not lost.


Sure.


The Python file does not contain any code. It would be better to add some simple Python code.


see http://pkgs.fedoraproject.org/cgit/rpms/python-productmd.git/tree/0001-update-the-other-two-_validate_version-functions-to-.patch  you missed a instance and updating the test suite


Shouldn't we use expression.AND ?


ca marche toujours pas, j'ai que le project Vulkan build qui pause probleme https://gist.github.com/Zangetsu38/980347b711ef6fa1b0eb
j'ai mis a jour Cmake, mais il y a toujours cette erreur bizzare a la fin, et surtout pourquoi il trouve pas les fichier : comme ca :
>1>LINK : fatal error LNK1181: impossible d'ouvrir le fichier en entrÚe 'Vulkan/Vulkan/Vulkan-build/loader/vulkan-1.def' [G:\GIT\rpcs3 Vulkan\Vulkan\Vulkan-build\loader\vulkan-1.vcxproj]

j'ai regarder le fichier est bien la pourtant, au dirait qu'il regarde pas ou il faut bizzare


Please avoid inline styles where possible.


Np, I'm happy with both :)


Done.


@doudou, the overlay is already drawed in paintEvent(). Moreover, just makes sense to draw the overlay if there is some data to display.


Extra? So when they release a beta it's also extra? The format is ``major.minor.patch.beta`` so ``0.14.0.0``.


@tomwayson I started wiring up additional mobile checks for #243. Can I get your input on this to see if this approach with a service being used throughout our examples is a good way to go, mainly since this ends up in our front-facing snippets?  I don't want to repeat code, but at the same time I don't want to start peppering the jsapi examples with extra stuff.

Regardless, I will still have to wire up the userAgent checks on the remaining examples with scene view directives.


multiValue works with a prefix, eg foo.xx=123&foo.yyy=456 etc. This is not the case with this list, so it should not be multi value.


Why was it reverted?


I'm a little uncomfortable with this fixed 10. The correct value depends on the height of the string that is to be displayed, but the 10 proved to be good-looking if you do not look explecetly for this...


Make a setXXX that takes a String value, and then in the setter your split it by comma and create the list and put it in there. Then people in the uri can configure it etc, filteredEvents=CREATED,REMOVED. And its not a multiValue.


ack


create a new ticket please as this some work to add this

On Fri, Feb 19, 2016 at 1:51 PM, tikismoke <notifications@github.com> wrote:

> Could it be update with the list of scenario name? Avoiding user to look
> at all scenario.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/domogik/domogik/commit/c8482bd6e25f319a1ae617a6db6fccd6a1a11b66#commitcomment-16202298>
> .
>


But now it's an official release 0.14.0 (the.beta isn't there anymore)


Sure. I'll help in anyway I can. I guess we can move this discussion to https://github.com/gintsgints/sequelize/issues/7.


.bbappends with wildcards don't work on daisy if I recall correctly, @agherzan


Looks good!. That is a good start :+1: :sparkles:


Is enum the right type, isn't this valid as well `25h2m`?


@Borlea Yeah, I saw that. But it doesn't seem much differen than just compiling your fork with updated dependencies. Same issues apply to his build and he is not responsive. Oh well. If you are stepping away from minecraft development, don't worry about it. :)


Well, TBH, you're absolutely right. I guess I looked at "further", assumed "future", and then the suggestion to replace usage with use followed.

Never mind with changing this, it sounds fine!


А тут не может быть ошибок.  либо результат  либо  []string


PowerS HELL
hate windows
Window:
____
|___|
|___|


x.y


Ok, makes sense. I will just take care of the hashfile creation. I don't think we should write on the original file location.


```
004a8264 <SharedConstants::getVersionCode()>:
  4a8264:	2000      	movs	r0, #0   // MajorVersion
  4a8266:	210e      	movs	r1, #14 // MinorVersion
  4a8268:	4602      	mov	r2, r0            // PatchVersion
  4a826a:	4603      	mov	r3, r0            // BetaVersion
  4a826c:	f7ff bfe6 	b.w	4a823c <SharedConstants::getVersionCode(int, int, int, int)>
```


OK. I thought the overlay was also painting the slider, but it's not


6.0 for what?


@KasperFranz discord?


FixImageProblemInIORef (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2131 of 2131 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

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


Can't remember exactly when was it introduced. So as long as we go over this restriction then we should be independent with this layer.


But `$srcset[null]` should be added, otherwise `$attributes['srcset']` would be empty if only one density is used.


I think that one is only used for the header in the metadata file. In my
opinion that should not change.


In order for your frontend client(s) (such as Kodi, Movian, and similar) to see/play and channels, you must first...


Decompiled MCPE?
Yeah, but it is not used
And there was never 0.xx.x.x in those constants alway 0.xx.x!


for you, check... below, but be aware ...


"StepByStep" should be "EraseBySectors".


Mining ships can get ore location from another ship?


omit


undesirable services - Teleshopping, Adult...


Changing to the already public download.url_download, since we will not download file:// assets.


There are two method i.e. at L243 one for List and one for String


Duplicate of constructor?


use **the** credentials?


cool :smile:


Thanks :D


Does it make sense to return the config outside of the internals of search?


help, check out


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

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


What will macros do?


perfect !

2016-02-19 14:13 GMT+01:00 20017578 <notifications@github.com>:

> Prova di inserimento immagine da far vedere a @guido612
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/20017578/MasterSoftwareLibero/commit/34c85244f9deb92e19b6e5dd655a72079a43aba5
>


Done. Nothing major - all looks good in general!


that's what the current value of `*x` does, which is implicit in the non-written `else { *x =*x; }`.  At least I think it is, not sure I tested properly the last version of the RTL code, but it sounds correct to me.

1. try to place on the left (natural side in RTL),
2. check if it would overflow on the right, in which case put in the natural side, as far as possible (we already know there's isn't enough space to place it nicely because of 1)
3. implicit, place on the right.


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


Yeah for the time being. I'm using this in a little different context than the Filesystem which uses Plugins to effectively load the adapters. I'm using a simple string (from `Component::params('com_search')`) to set the adapter upon construction. I could probably remove this.


Oh, ok, great :)


And he is a friend of mine, gave me permission to use his gens for the server icons. He doesn't have a github though :/

Check out his blog here: https://c0dart.wordpress.com


That's something similar I discussed in https://github.com/wunderkraut/WunderTools/issues/15 but I wanted it to be removed when either or both:
   - { role: letsencrypt, tags: [ 'letsencrypt' ] }
   - { role: sslterminator, tags: [ 'sslterminator' ] }
are disabled.

So where do you define nginx_disable_content_security_policy ?


Maybe an error or warning should be raised at least? The previous code only discarded empty lines, here we're discarding potentially badly formatted lines (in the previous implementation such lines would have been added to ``response_headers`` in the last ``else``.


Inconsistent coding style here.


マージ完了


test


You shouldn't need this as you are using the full namespace below.  Plus, it's sort of antithetical to the adaptor pattern.


Just call setAdapter?


it should receive fc.    It should be possible to query `isBackupneeded` for a wallet that is not in focus.


Yeah, it's working fine but I want to make some cleanup before creating a pull request and have the RX IRQ driven (it's polled right now), so that will be easier to add telemetry over nrf24 :)
Also, against which branch should I make the pull request when it's done ?


@noffle?


How will the consumers of the search libs actually perform searches?  Will your search class eventually forward calls on to the adapter?


Orimligt


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


I would expect getLog to return logs or a log path, not $this...


Apparently (after exchanging ideas with @tmortagne and @mflorea ) there is to safe way around the document API and we should never use the store API to do updates.


Where will config be stored?  This is a problem in general with config that applies to libraries.  I don't think we have a good answer to this.


Maybe you can add th as an alias, enabled by default?


Uncomment this one and put correct path to the public header.


Leave out the .h and .m files, as they're not needed in our case.


Empty construct?


Thank you Dev😀🙇
Wish your phone boot on this rom soon :)💪💪👍👍


hai solo tolto degli spazi?  @pdmarzio


Fix for bug


Can I ask why newWidgetBehavior() was finalised?
I understand the need to make sure certain behaviours are there, but you've now stopped anyone from overriding the default behaviours inside of [DialogBehavior](https://github.com/sebfz1/wicket-jquery-ui/blob/master/wicket-jquery-ui/src/main/java/com/googlecode/wicket/jquery/ui/widget/dialog/DialogBehavior.java#L213-L227).


Ho già fatto la modifica mettendo direttiva antitracciamento perché mi
sembra che così la differenzi di più rispetto a opzioneimpostazione che
sembrerebbe quasi uguale a quella in navigazione anonima.


why do you allow 3.2? This line needs to be removed, right?


Test


Would it make sense to have a connect method, or something along those lines?  I ask because this ping() function also has this line in it.


A WAMP SUBSCRIBE message MUST contain an options dict in all cases. _If_ match != exact, sending match attribute in options dict SHOULD be skipped.


Can you have an empty query?  If not, making that the default seems incorrect.


hum, I'm not totally sure `x` is really guaranteed to be usable without initialization.  The docs aren't clear no that, yet the GTK3 ones states "inout" which would mean it's fine -- yet doesn't say what is the default value.  GTK2 docs don't say anything about the initial value being usable, but yeah it seems to work.


Thanks for the prompt reply.


Ok, sorry about the previous comment, I hadn't read all of the commit message. My apologies, I would ask that you give us the ability to create our own DialogBehavior, perhaps with another factory method?


ack


Hi, what's your use-case?


Again...connect function?


@FroMage @quintesse thanks for corrections, all comments should be incorporated by 65fb957d6b (my english teacher wouldn't be proud on me :cry: :smile: )


Example of custom growth function:

```
Module['calculateNewTotalMemorySize'] = function(OLD_TOTAL_MEMORY, DYNAMICTOP, LIMIT) {
        var NEW_MEMORY_SIZE = OLD_TOTAL_MEMORY;
        while(NEW_MEMORY_SIZE <= DYNAMICTOP) {
            if (NEW_MEMORY_SIZE < 128 * 1024 * 1024) {
                NEW_MEMORY_SIZE *= 2; //double up until 128MB
            } else {
                NEW_MEMORY_SIZE +=  128 * 1024 * 1024; //increment with 128MB chunks
            }
        }
        return NEW_MEMORY_SIZE;
    };
```


dit mag weg


Awesome job! Thanks, Max.


Instead of doing all that ``clean`` magic, can't you just use a validator instead?


ah ba je comprend pourquoi, le probleme est la "Vulkan/Vulkan/Vulkan-build" il y a un Vulkan en trop ^^


Are "documents" a solr-specific term?  I.e. is it generic enough to apply to other adapters?


**UNKNOWN** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 450](http://ci.infinispan.org/viewLog.html?buildId=36103&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for dadc79d2b3440345930fe4a117a80911aea08c3e
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:01:41

Process exited with code 137


Hum, how will this behave when triggering with the keyboard?  Placing on the X position of the mouse seem not very nice in this case.

Also, this is only for the case where the popup can't fit in a side, right? (I can't seem to see how it can end up under the mouse otherwise, as the mouse has to be on the letter yet we place below or above when possible)  Will that happen in real situation or is it theoretical?


Make image size determine work in WP admin too.


BTW, what's up with all those `+1`?


This var is never used...


Oh ugh, yes.  I think the best workaround is to use `/var/tmp`.


Initial commit


Here is Un1corn14 doing so exercise and listening to the new Kendrik Lamar's album.
Fun Night


Actually I don't think it is needed anymore since https://github.com/morse-simulator/morse/commit/ded2fcbce402c6a3ecfeb2b948c0068f604d6d9b
You can also see that master builds fine with sphinx 1.2: https://launchpadlibrarian.net/237010146/buildlog_ubuntu-wily-amd64.morse-simulator_1.4+1+3858+20160208-0~ubuntu15.10.1_BUILDING.txt.gz


I don't think you want to share this with the rest of the world ;)


== 1 ? or will there always be one in the pipe?


Are test and status different?  I don't see a test method on your adapter.


**UNKNOWN** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 453](http://ci.infinispan.org/viewLog.html?buildId=36108&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for dadc79d2b3440345930fe4a117a80911aea08c3e
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:33

Process exited with code 137


#!/bin/sh is missing


Is this test file used? (at least not in this commit)


It could be defined as variable in environment specific yml files.

Example is in vagrant.yml, under "vars:"
nginx_disable_content_security_policy: True


This is probably not a question, right? Maybe the question mark should be replaced by a period?


lol right good call


I do understand the various problems related to python's float precision versus dynamodb's precision. But why is boto3 not converting the float to Decimal when necessary?

I'm inserting a float inside a table.update() and getting an exception. There cannot be any loss of precision by converting a python float to Decimal(), right? It's only the other way around? Shouldn't the code be more forgiving here?


I don't really know Swaggers config, so I cannot suggest any settings. But what I see looks correct with name-spacing etc. :+1:


@ratoaq2 nice. are you going to commit to subliminal?  travis is failing right now


Good point. Then we should relax the constraint. I'll update the package.


Use this as the commit message:
"The system handler is called before the installation config gets loaded.
This way it can collect additional variables from the systems,
such as the system's serial number."


Haha. Don't worry. Can you merge it on Monday then?


Na androida są oficjalne interpretery


We have a form in a dialog, and we just need to insert some javascript right before the callback script, to confirm that the user wants to navigate away from the dialog.

We poked around and thought the best spot to do this would be in [DialogBehavior](https://github.com/sebfz1/wicket-jquery-ui/blob/master/wicket-jquery-ui/src/main/java/com/googlecode/wicket/jquery/ui/widget/dialog/DialogBehavior.java#L247), as it uses a couple of factory methods to create the callback code, but when we attempted to override newWidgetBehavior(), we obviously were not able to.


It was a bug which fixed


We'd probably also want to override OnEscapeCloseAjaxBehavior, for all use cases.


I'd say keep it and just call setAdapter from your constructor.


Actually, no. I took it from the comments of @simonharrer and it seems I accidentically commited it as well.

It does not do any harm, so why not leave it as an example file?


Everything was working correctly concerning the duplication using a set instead of a list for independencies
`        self.independencies =set()
`
But there were two tests failing :

1. "test_independencies/test_Independencies.py", line 135
2.  "tests/test_factors/test_Factor.py", line 517

Both had the problem of test independencies being equal , independent of the order of event1 and event2
, i tried using the modification in  #640 which i believe should have worked but it didn't actually and i don't know why.

Finally, right now duplication is avoided using reduce method, but i can spend more time modifying _eq_ method if you think it will be useful.


I was more concerned about a way to force-enable it so someone can really control whether he'll have it or not, not just adding dependencies until it appears enabled.  Seems more robust to me, and allows to force the check on distcheck.


Because this is done in the web-service, the context of "the generated client" is implied instead of specified. Maybe we should add "the generated Swagger/Community Service client".


yes, that's the plan, if you don't want to announce 1.2.2


:+1: for ease of use !


Will do, thanks for reminding me


@bp-kelley : this one didn't seem worth doing a PR for :-)

It's definitely necessary to avoid some serious badness in at least some versions of clang-format


If while sliding menu application move to background (for example, because phone receive incoming call), then after returning to foreground state user interface freeze. (isLeftOpen() or isRightOpen() methods return that the menu is opened fully)


P1 in path has been neglected which is necessary


Hi,
Can you please let me know whether the hypothesis(Tree Root,which contains all the tree data) is same for multiple Model Aggregators when we give parallelism for it? for example 2, will both have same tree to predict?


ack


Hi. I added a new commit to this PR which should solve the issue. All e2e testes are running fine here (apparently singleRun = false was the problem). There is one minor detail: it doesn't work (at least for me) if you don't get the latest commit from karma. I had to set the upstream to karma-runner/karma then reset --hard my master and cherry pick your two commits and my two commits then everything worked.


Время жизни бесконечно.
Список меняется, см, addObserver


We work for 3.3 also, so `> (3,3)` is not right


Please squash this commit with the previous one


I see, should we release this as an updated version using AS < 4.1? Or wait until we migrate to 4.1+?


redundant comment


Looks like a linebreak is missing here.


You're right.  I don't see why it would be needed in production.  Removed.


What @sils1297 meant was that `==` for 3.2 is not needed


Doing `> 3.3` or `== 3.3` is redundant. Just do `> 3.2` ?


I think you need %s  instead.


Missing UploadIO ?


Did you try overriding `YourDialog#onConfigure()` as follows?

	@Override
	public void onConfigure(JQueryBehavior behavior)
	{
		behavior.setOption("beforeClose", "function() { if (...) { return false; } }");
	}

Caution, it might be overridden if `#isEscapeCloseEventEnabled` returns true;


P1 in Apache is CAPS but it doesn't matter and still has to be fixed for icon link.


@teckjon Good work here. Press ahead!


prompt


Ok leave it there...examples are always good :smile:


В обычном случае ты был бы прав, но у нас кривая архитектура ядра, поэтому тут есть эти костыли.
Получить EMyPositionMode и UserMark нормальными способами из ядра не возможно.
Изменится ядро - уйдут проперти.


Hola, hay conseguido alguns de lista.
Todavia esse q funciona en kodi, no funciona en RosadinTV

#EXTINF:-1 tvg-id="rtp1.pt" tvg-logo="rtp1.png" group-title="PORTUGAL",[COLOR yellow][I][B]RTP 1[/B][/I][/COLOR]
rtmp://rtppullswflivefs.fplive.net/rtppullswflive-live playpath=2ch5h264 swfUrl=http://programas.rtp.pt/play/player.swf?v3  pageUrl=http://www.rtp.pt/play/direto/rtp1?mag swfVfy=1 live=1 timeout=15

Gracias


Obviously, I'm shamelessly trying out the value that is mentioned here: https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/mbnctfDdcqM[326-350] ;)


This broke migrations because `this.client.SqlString.format(sql, bindings, tz);` is used.


Абсолютно. Этой segue нет на storyboard.


`behavior.setOption("beforeClose" ...` will replace the OnEscapeCloseAjaxBehavior behavior


I haven't tried it, no, but I'll give it a shot.


Нужен, процесс разработки еще не закончен.


Small detail - you can probably merge this into __init__, and then the ``hasattr(self, "port")`` in this code and in ``close`` becomes unnecessary.


That would replace it, but wouldn't that stop the CloseEvent from being fired?


Thinking this through, notice that the file will never be cached when in a `file://` location. Currently, if a test does `self.fetch_asset('file:///user/share/my.iso')`, any future test can do only `self.fech_asset('my.iso')` as the file is cached. In your suggestion, this will not be possible and future tests will always have to do `self.fetch_asset('file:///user/share/my.iso')`. It's valid and I just wanted to make sure we are happy with that.


No need to apologize ☺


`%q` and wherever else you're doing it.


added seat images and new logo


[abstract isn't a required field](https://github.com/KSP-CKAN/CKAN/blob/2c959b8a81b07aa2eb04e1f1eeb7f0d586a2e76b/Core/Types/CkanModule.cs#L132), and unknown identifiers are treated as extensions rather than invalid.


I have quickly connected the callback in your other patch to the already existing 'open-state-changed' and it works. I'd rather avoid emitting another signal just for this.


FixImageProblemInIORef (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (997 of 997 tests passed, 0 test warnings)

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


Ок, переименую


I actually wanted the username in there literally, to avoid internal backslashes


added seat images


This is what you are looking for:
http://stackoverflow.com/questions/908331/how-to-write-binary-data-in-stdout-in-python-3


Объекты подписываются на все время своей жизни. Как только объект умирает, он исчезает из подписчиков.


it say ``len(student.grades) >= aidx`` it should cover equal case as well


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2086 of 2090 tests passed, 543 test warnings)

 * 547 tests had: AUD diffs.
 * 477 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


Sound good. However, I remember I had difficulties making autocompletion working under QtCreator because a certain file did not contain the right paths to .cpp files. @Hinsbart  Does this work?


CloseEvent will be broadcasted if either `#isDefaultCloseEventEnabled` or `#isEscapeCloseEventEnabled` returns true. If both are false, CloseEvent is not broadcasted


Even though this might have little impact in the performance I usually prefer to have stuff like this cached. Maybe just adding a variable to the rightClickMenu right after it's created and toggle that as needed?


What is this for? simply handling ``KeyboardInterrupt``? What about using a non-bytes value e.g. ``None`` instead of ``b"\x03"``?


Even so, if we get passed this restriction, some of the recipes referenced by the bbappends may not be available in some of the poky versions.

Bottom line, I am not sure why we have this kind of README here to begin with. What purpose does that kind of info serve now?


Outer paranthesis `()` is not needed


It's to avoid running Doxygen twice, because it's kinda slow.  And the comment is merely here to explain to future readers why I added the dependency, not sure what's overkill here?


Or no value at all, can't you call ``self.stop`` in the exception handler?


Just a reminder - please squash all your commits to get a single commit


OK


# Matrix inversion is usually a costly computation and there may be some benefit
# to caching the inverse of a matrix rather than compute it repeatedly. The
# following two functions are used to cache the inverse of a matrix.

# makeCacheMatrix creates a list containing a function to
# 1. set the value of the matrix
# 2. get the value of the matrix
# 3. set the value of inverse of the matrix
# 4. get the value of inverse of the matrix


Be aware that such code would break Linux support for FPC, which has been added to the FastMM4 trunk.
BTW, why didn't you define .NOFRAME for your CPUX64 small asm functions? (nostackframe for FPC)


Not if I override the callback function in the options, right?
I'll need to override both callback functions to check that the form isn't dirty, in doing so, removing the code that would fire the` #getCallbackScript`, launching the event.


And then exit the loop with ``break``?


'providers' should be 'providerName' instead? In my case (using with Doctrine ORM), the command works only with the latter.


so good :+1:


Is it node.js?


Need changes


Nice job, Laserman! You work fast.


Note.- I'm also confused because I assumed you know the plugin much better than I do, but I tend to interpret your post as if this were not the case for this particular code passage. Please, don't take offence if I'm explaining you the obvious. I'm just trying to nail down the discrepancy.

> I assumed (incorrectly) that the plugin was somehow providing the passphrase answer to the ssh command

This is indeed the case: git-client-plugin...
1. sets `GIT_SSH` to a temporary script that essentially sets the `DISPLAY` variable to something not null and invokes `ssh -i` with the selected key
2. sets `SSH_ASKPASS` to a temporary script that echoes the password
3. executes a `git` command in a shell without attached terminal
This all results in git using the configured key with the configured password. (Try it from the command line.)

>Does your environment have an ssh-agent process running in the context of the Jenkins server

No, and as explained before, it isn't necessary (and will be ignored)

> is there a SSH_AGENT environment variable in the Jenkins system variables

No

>Are you running only a master, or do you also have slaves?

I'm running only master

>Does the passphrase based clone work for you on the master and the slaves?

I don't have any slave configured


LGTM :+1:
Working on the docker-machine guide


ok


And I can't override it. I just tested it. DialogBehavior calls the super, which leads to my code, before it finally sets the callback functions.


Added : Bootstrap files

Sorry for any French words that could appear in any commit names.


This needs to be 7.0.0-SNAPSHOT.


OptimisticLockingException does not support custom messages


Looks fine to me.  :+1:  :tada:


> I actually wanted the username in there literally, to avoid internal backslashes

Doing it like this makes the message unparseable.


Done and committed.


**the** individual component? **an** individual component?


@nickpell how do you feel about us just removing this? maybe instead we can just test the case where a subscription is less than a full month?


Various SQL I've worked on recently.


Should've been a default value.


@zygotic99, actually looking at it, it was quite obvious the lines a few above called this method. I missed it.


value.htm This modification affects "cbi.combobox" Use
dynlist.htm can not use the Add button


Could you move the FeatureStaticIpSetting class either into it's own file or the networking.cs file (features.cs has far too much in it as it is, I don't want it to get worse.)


OK, thanks for the feedback.

If you need me to pull/test anything on my side when you think things are sorted out, I'm game!

And as jcberquist said, nice work on the rewrite, I plan to review it more to try and level up on sublime-syntax skills :P


Yes I think we had that change before we lost this patch, I had to scramble to find this older version.  I will update.  Don't know how this patch got lost.


> Not if I override the callback function in the options, right?

right

> I'll need to override both callback functions to check that the form isn't dirty

no, only `beforeClose`

>  which leads to my code, before it finally sets the callback functions.

set `#isEscapeCloseEventEnabled` to false


My comment


Does not exist in code. Also remember field translations belongs in `field_da.po`.


Does not exist in code.


After discussing that with @akatsoulas it looks like there is no cleaner way to do this because of the way ``dal`` handles this case. Let's add some comments.


The "stub" part was added here: https://github.com/HyperSuprime-Cam/afw/commit/2c12255372bde846ba0429b5b542960e57d169f0


I missed the very obvious thing that this is two step not one and there is the node creation and then the creation of the relationship. For readability and quick understanding of what is being done it might be nice to split this into two lines (same with the factset id above it)

statement.WriteString("CREATE (i:Identifier:FactsetIdentifier{authority:'" + identifier.Authority + "', value:'" + identifier.IdentifierValue + "'}) ")
statement.WriteString("CREATE (i)-[:IDENTIFIES]->(n) ")


This commit is the only difference in camera source code from cm-12.1 to cm-13.0, and now I am getting this:

> 02-19 14:10:50.270  4208  4222 F libc    : Fatal signal 11 (SIGSEGV), code 1, fault addr 0x981c in tid 4222 (CamConfTh)
02-19 14:10:50.270  1830  1830 I DEBUG   : property debug.db.uid not set; NOT waiting for gdb.
02-19 14:10:50.270  1830  1830 I DEBUG   : HINT: adb shell setprop debug.db.uid 100000
02-19 14:10:50.270  1830  1830 I DEBUG   : HINT: adb forward tcp:5039 tcp:5039
02-19 14:10:50.371  1830  1830 F DEBUG   : *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
02-19 14:10:50.371  1830  1830 F DEBUG   : Build fingerprint: 'htc_wwe/htc_vivo/vivo:6.0.1/MMB29U/20160219194618:userdebug/test-keys'
02-19 14:10:50.377  1830  1830 F DEBUG   : Revision: '0'
02-19 14:10:50.377  1830  1830 F DEBUG   : ABI: 'arm'
02-19 14:10:50.384  1830  1830 F DEBUG   : pid: 4208, tid: 4222, name: CamConfTh  >>> /system/bin/mediaserver <<<
02-19 14:10:50.384  1830  1830 F DEBUG   : signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x981c
02-19 14:10:50.498  1830  1830 F DEBUG   :     r0 0000981c  r1 b2d5e37d  r2 b2d5e39d  r3 00000003
02-19 14:10:50.498  1830  1830 F DEBUG   :     r4 b2e80608  r5 00000000  r6 00009800  r7 b2e700a0
02-19 14:10:50.498  1830  1830 F DEBUG   :     r8 00000000  r9 b2e80608  sl 00000001  fp 00000001
02-19 14:10:50.498  1830  1830 F DEBUG   :     ip b2d5df10  sp b2d5e308  lr b2e56af1  pc b2e3f73e  cpsr 600f0030
02-19 14:10:50.527  1830  1830 F DEBUG   :
02-19 14:10:50.527  1830  1830 F DEBUG   : backtrace:
02-19 14:10:50.527  1830  1830 F DEBUG   :     #00 pc 0002773e  /system/lib/liboemcamera.so (DetermineQNumbers+5)
02-19 14:10:50.527  1830  1830 F DEBUG   :     #01 pc 0003eaed  /system/lib/liboemcamera.so (vfe_util_sce_get_cfg+24)


I thought OnDefaultCloseAjaxBehavior and OnEscapeCloseAjaxBehavior were for two different events;
Former for the 'X' icon, and latter the escape key.


C'est les requetes SQL du premier exo on est d'accord ?


Right - I, as always, didn't read the documentation at all and just assumed the coordinate gets passed :-).


Вот теперь похоже на правду :)


No. This package is in the master branch, and it needs to work with the master branch of the other packages. A version ``bob.ip.facedetect 2.1.0`` does not exist. I have only changed the version in the branch to be ``2.1.0b0`` as we have discussed before. So, a version ``2.1`` does not even exist, and the branch is far from being in a shape to publish a new version.

Another problem is that the wheel of the master branch would be deleted on the wheel server (by that script that deletes the old versions, which is AFAIK now running on a regular basis) and, hence, dependent packages would not be able to download the wheel server.

When you need different functionality, please go ahead and implement another script that does, what you need to be done, but do not disable a test that I have implemented on purpose. I have lost 4 hours yesterday until I realized that the ``bob.ip.facedetect`` package was uploaded from a branch -- which should have been avoided with exactly the part in the ``upload-wheel.sh`` script that Tiago has commented out and which I was relying on.


Yeah, I'd generally agree, but I still think what I said [earlier](https://github.com/kugel-/geany/commit/7279037dd697143881071a3c5cd467ed0a96b9a4#comments) (that `GStrv` is odd for C programmers, and that in the current form it doesn't build -- latter is easy to fix, I agree).

I don't have a really strong opinion here, and either way it's a one-time weirdness -- well, especially for the GIR par as we won't change the type of this anytime soon. @elextr @codebrainz opinions?


Takich zapytań jest więcej to co widzę, które sprawdzają `a.datefrom <= ?` i dodatkowo `a.datefrom = 0` a we wszystkich tych przypadkach `?` nie może być liczbą ujemną bo to zazwyczaj `%NOW%`.


12.1 has the fix directly in the dispaly HAL. You probably need to fix your
shimming if you are getting crash from mediaserver. I had something like
that too with broken shimming.
On 19 Feb 2016 16:17, "Zeeshan" <notifications@github.com> wrote:

> This commit is the only difference in camera source code from cm-12.1 to
> cm-13.0, and now I am getting this:
>
> 02-19 14:10:50.270 4208 4222 F libc : Fatal signal 11 (SIGSEGV), code 1,
> fault addr 0x981c in tid 4222 (CamConfTh)
> 02-19 14:10:50.270 1830 1830 I DEBUG : property debug.db.uid not set; NOT
> waiting for gdb.
> 02-19 14:10:50.270 1830 1830 I DEBUG : HINT: adb shell setprop
> debug.db.uid 100000
> 02-19 14:10:50.270 1830 1830 I DEBUG : HINT: adb forward tcp:5039 tcp:5039
> 02-19 14:10:50.371 1830 1830 F DEBUG : *** *** *** *** *** *** *** *** ***
> *** *** *** *** *** *** ***
> 02-19 14:10:50.371 1830 1830 F DEBUG : Build fingerprint:
> 'htc_wwe/htc_vivo/vivo:6.0.1/MMB29U/20160219194618:userdebug/test-keys'
> 02-19 14:10:50.377 1830 1830 F DEBUG : Revision: '0'
> 02-19 14:10:50.377 1830 1830 F DEBUG : ABI: 'arm'
> 02-19 14:10:50.384 1830 1830 F DEBUG : pid: 4208, tid: 4222, name:
> CamConfTh >>> /system/bin/mediaserver <<<
> 02-19 14:10:50.384 1830 1830 F DEBUG : signal 11 (SIGSEGV), code 1
> (SEGV_MAPERR), fault addr 0x981c
> 02-19 14:10:50.498 1830 1830 F DEBUG : r0 0000981c r1 b2d5e37d r2 b2d5e39d
> r3 00000003
> 02-19 14:10:50.498 1830 1830 F DEBUG : r4 b2e80608 r5 00000000 r6 00009800
> r7 b2e700a0
> 02-19 14:10:50.498 1830 1830 F DEBUG : r8 00000000 r9 b2e80608 sl 00000001
> fp 00000001
> 02-19 14:10:50.498 1830 1830 F DEBUG : ip b2d5df10 sp b2d5e308 lr b2e56af1
> pc b2e3f73e cpsr 600f0030
> 02-19 14:10:50.527 1830 1830 F DEBUG :
> 02-19 14:10:50.527 1830 1830 F DEBUG : backtrace:
> 02-19 14:10:50.527 1830 1830 F DEBUG : #00 pc 0002773e
> /system/lib/liboemcamera.so (DetermineQNumbers+5)
> 02-19 14:10:50.527 1830 1830 F DEBUG : #1
> <https://github.com/OpenDesireProject/android_device_htc_msm7x30-common/pull/1>
> pc 0003eaed /system/lib/liboemcamera.so (vfe_util_sce_get_cfg+24)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenDesireProject/android_device_htc_msm7x30-common/commit/3cabbd13c8689d6bb076c8fa48e2f363b191adc3#commitcomment-16203948>
> .
>


This isn't firing for _blank targets so url and https status and etc doesn't get updated currently in browser-laptop


Might perhaps be nice to use [rospy.wait_for_message()](http://docs.ros.org/jade/api/rospy/html/rospy.client-module.html#wait_for_message) to retrieve the joint states? Instead of the global.


``flterm = FlTerm(...) try: ... finally: flterm.close()``


ack


Awesome. I see you're already an owner of the npm package, so feel free to publish a new version.


ack d80e1dd


@DarwinJS It this line intentional or a result of cop&paste?


Completely right, but OnEscapeCloseAjaxBehavior  is bound to beforeClose event


Mi4 CDMA's raw id is 1973, I think this line need add $RAW_ID == 1973 so it can remove nfc on Mi4 CDMA too. @jrizzoli


@mdtusz I updated the baselines.

But, I have some bad news, these mocks:

![image](https://cloud.githubusercontent.com/assets/6675409/13177915/0ed59a38-d6ea-11e5-880a-019afdcba394.png)

fails on `npm run test-image`, but pass on `npm run test-image -- mock.json`. :pray:


Anyone that can make a better crystal texture please! Express yourself!


@bryk Could you check if this actually resolves issue #402?


This seems rather complicated

something more like

tmpmask = (1<<bits)-1
mask=((tmpmask&0xff000000)>>24).ToString() + "." +
          ((tmpmask&0xff0000)>>16).ToString() + "." +
          ((tmpmask&0xff00)>>8).ToString() + "." +
          ((tmpmask&0xff)).ToString()

Would perform the same task, wouldn't it?


You're welcome. Great to hear it's working!

Looking at the output file of the script, it seems like the cid's of the replies are in the following format `<cid of the parent>.<epoch timestamp in ms>`. You can use these to group the replies together.


Potential starters for the editor and linter configurations 18f/ffd-toolkit#2


Here you go: [GTK2](https://developer.gnome.org/gtk2/stable/GtkMenu.html#GtkMenuPositionFunc) and [GTK3 docs](https://developer.gnome.org/gtk3/stable/GtkMenu.html#GtkMenuPositionFunc) ;)


- Why is `config.rb` needed here?
- Remove the 2 empty SASS files.
- Theming belongs in `ding2/themes/ddbasic/sass/module/` (remember to compile).
- "_No available coverImage_" is handled by FontAwesome in SASS (see https://github.com/ding2/ding2/blob/master/themes/ddbasic/sass/module/_search.scss#L818).


But that does mean we lose the event for escape key.


Possible section to include in the README around using Gulp, if we decide to extract it out for the Toolkit. 18f/ffd-toolkit#2


But then Users would have also to install the [`future` library](http://python-future.org/index.html).
Since at the moment the installation procedure is still manual, i would suggest not to do this yet.

Anyway , the  combinations of the prime-factors (where `xrange` is used) are not that many,
usually less than ~15 factors, so storing 2^15 integers is ~128kb of data, so not that much RAM is wasted with `range` on py2.

If you insist, then we have to modify also the installation instructions.


Hey, maybe we should have a skype meeting on this. I'm not sure we're on the same page here. Irrespectively of which script does the upload, we need to have stable wheels somewhere. Are you suggesting we create a new directory with stable wheels and re-encode all packages so that, when a stable build is built, we upload there instead?


Ah right, forgot about that.
It works for the most part, but completion for some classes is not available out of the box because qtcreator seems to treat the directory tree a little different than our build system, so it can't find the includes.
The solution to this is just adding a line containing `.` to `project-name.includes` ^^

I'll add this to the instructions


Yeah, you are right, it doesn't work well when triggered with keyboard. The

```
+1
```

is to move the popup one pixel further - without it I still get the overlap of the popup and the mouse pointer.

Well, it's starting to get complicated. Instead of adding/subtracting 1 we could use the code as before but calculate the difference between the caret location and ctrl+click location and when needed add/subtract the distance to the next character. Any other idea?

This coordinate craziness is exactly why I placed the popup into the middle of the screen :-)


OK thanks


Implemented a central method for all CompletableFutures


vewy nice


I'm sorry for being such a bother. But I do not understand what you mean by "shimming". I don't know what this term means, and how I am supposed to fix it.


Ah, OK, I missed the implicit one.


Why?


The link for this article was not converted from the old KC.


Is it working for your scenario?


I tried this, it crosses the 80 char limit. I know I could wrap that though, but I will avoid doing that. I'd like to keep it this way.


I was wondering if polarssl is sharing the same ABI as OpenSSL and ./configure was just used to select the right library path. But apparently the difference is deeper than that so ignore what I said: https://github.com/shadowsocks/shadowsocks-libev/blob/master/src/encrypt.c#L342-L358


Fancy ES6 templates :grinning:


Please ignore this commit, I have all my changes in previous commit only, Sorry.


Addressed Anthony's comment.


After a small discuss here, we don't mind losing the escape key event, but I am curious if you're thinking of a future way around this problem?


It's plain glib after all. We choose to be powered by glib. Glib has other awkward types which we use elsewhere in the api (GPtrArray).

At last, GStrv makes it clear that it's null terminated and that you can use g_strv_* and other useful convenience function on it. This is also highly beneficial for c programmers IMO.


Actually yes, here is the code to support the escape key

    function(event, ui) { if (event.keyCode == $.ui.keyCode.ESCAPE) { ... } }


xxxxx


Sin bien


```
return ("You are not connected to the internet."
        if code is None else True)
```


Is this doing the query twice? Maybe it could be cached?


I asked help from irc channels, to narrow this down. I think this needs to improved more for `ftp://` and `localhost` and other things.

Or as you said, we can just use this minimal version and iterate from here.


This commit brakes https://github.com/blue-yonder/sqlalchemy_exasol/commit/2f408aa9c0998dbd3d19b1a1a3fc69c2a24acc0a.

Is there any reason to have that order of classes?


Would it work if replace the `if` condition on the script for a test checking if the branch is master `or` a release?


Sorry, I messed up the commit message, it should be F [PS-16]


Maybe an unless instead of == false?


Yay imageserver...


Ah, I see what you mean. In fact, this script does upload the stable wheels, when they are at the master branch. However, they are deleted by the nightly cleanup script as soon as a newer beta version is uploaded. When tou have a look at the wheel pages, you can see that they are mixed between stable and beta versions:
http://www.idiap.ch/software/bob/wheels/travis/

But, anyways, I don't see why we need stable wheels -- as they are only usable in the travis tests. And this is what the script is designed for, uploading the wheels from travis, so that other scripts can download them on travis. These wheels won't work anywhere else.
If you need wheels for a different operating system, you shouldn't use this script, but you write another script that uploads the wheels to another directory on our wheel server.


This was done by coala though :smile: I will fix this.


That wouldn't fire the code for the event though, that would just check if the escape key was pressed. The server side still loses the event.


Note that this commit also implemented `AutoDriveCommand`, the main drivetrain command that will be called from the various autonomous CommandGroups we make. It is declared as follows:

`AutoDriveCommand(speed, turnval, time)` where `speed` is a normal 1.0 to -1.0 speed, `turnval` is also range 1.0 to -1.0, and turns the robot left or right (not sure exactly which direction is which at this time, that will need to be tested), and `time` is the time that the command runs for. Setting this to 0 will make the command run exactly once.


Yes, correct!


s/SUCCESSFUL/SUCCESS


datepicker is not only in plannings


Thanks for the PR @Alexander-Taran

Shouldn't we then also delete the `t.parentNode.insertBefore(s,t);` at the end of line 8? (I know the style is confusing. Should've reformatted it after copying)

Can you maybe explain a bit more what you mean by Aurelia style(or link to the relevant docs)? Aren't the any `<script>` tags in Aurelia ever?


not used


Sounds like a Plan. I have not released either.

I will extend the dsl Plugin to accomodate for the new functionality.

I'm using the DSL for every job, so I personally do not need the gradle list view, I have all gradle jobs at hand.

LG

J.

> On 19 Feb 2016, at 00:52, George Simpson <notifications@github.com> wrote:
>
> I think we should consider all of them that don't relate to the wrapper, or
> how gradle is initialized. That part needs some thought/rework. I already
> merged JENKINS-26287.
> I like "Defined 2 custom view job filters 'All Gradle Jobs' and 'Gradle
> Jobs'. #22" but I think it may be dated. I asked the submitter to update
> it so we will see.
> This is my first Jenkins plugin and I don't know the release procedures
> yet. I am thinking I will try to release in the next three weeks if I can
> figure it out.
>
> On Wed, Feb 17, 2016 at 4:28 PM jhinrichsen <notifications@github.com>
> wrote:
>
> > Great!
> >
> > Should we consider any other PR in the next release? Such as
> > JENKINS-26287 Prevent the console-outline to overlap the console output #26
> >
> >
> >
> >
> > > On 18 Feb 2016, at 00:55, George Simpson <notifications@github.com>
> > wrote:
> > >
> > > Wondering if you would ever need to support a combination of -P and -D
> > properties. I suppose if you were changing the config to use -P you would
> > be crafty enough to know that -D would no longer work.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub <
> > https://github.com/jhinrichsen/gradle-plugin/commit/a9d9945728d2c0711db1289d622538b873f17955#commitcomment-16163340
> > >.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/jhinrichsen/gradle-plugin/commit/a9d9945728d2c0711db1289d622538b873f17955#commitcomment-16163907>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


brilliant job! congrats.


Es importan que estigui amb majuscules sino no funcionara mai


Hey Tim. Looks like you still have some work to do on this one. Keep plugging away at it, and ask if you need help wrapping your head around it. The next time you think about this code, try to imagine it from the user perspective... What happens if I enter this, what happens if I enter that? How can I try to break this code? Also, take another look at the instructions. I don't want you to get discouraged, but I would say you are probably at about 40% if you are curious about your estimation.


It is before the files section.  ??


I have removed the relevant sections on the clean-up script. Now the latest unstable and stable wheels are kept on the server (since last year I believe).

We need the stable wheels for the "bob" package, which is also tested on Travis.