Hi Chris,

I just found your well-documented Python snippits series, and think it is very valuable, to me at least. So that's why I'm writing to comment on this code, which the one that brings me here.

The last chart should be *"the color determined by sex"*, but the result is not. If you want to fix it, please take a look at [my updated version](https://gist.github.com/suntong/995e3c4e027e042287a4).

Cheers


well .gitignore can be in repository if need be. But we don't maintain/generate files  like docker-storage-setup.iml or .idea. So there is no reason that upstream repo should carry these in .gitignore?


Like what? Why should we remove assertions?


It took some time, anyways, try [this](https://github.com/ShootingKing-AM/DigitalCircuitSimulator). Its not tested thoroughly, but you will get an idea atleast.


hue


нет, не сами. Рекомендовали ставить именно возле типа.


Oh, and on that note, if your preference is to use a truthiness evaluation here, I'd recommend changing the existing checks for items in the options object to change to use truthiness as well (as opposed to ```!= null```). I can submit a follow-up PR for those changes too. Just let me know how you'd like to proceed.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.828](http://teamcity.monogame.net/viewLog.html?buildId=11677&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


убрать использование _size (это поле класса, не трогай его, используй напрямую size который передаешь в параметрах)


Please split this line so it is less than 120 characters.


No, it can be a patch series in same PR. For example.

https://github.com/projectatomic/docker-storage-setup/pull/83


"breaks everyone's CI process".

That sounds serious. Can you document that everyone's CI process has in fact been broken?


I am using this version and i noticed that when i pass a non existent entity id i.e in the following code

@RequestMapping("/revisions/{ID}")
@ResponseBody
public List<Revision<Integer, Client>> getRevisions(@PathVariable("ID") String id,Pageable p)
{
    Page<Revision<Integer, Client>> revisions = clientsRepository.findRevisions(id, p);
return revisions.getContent();
}

it leads to the following exception when i pass a non-existent Client object ID.

Hibernate: select client_aud0_.REV as col_0_0_ from clients_AUD client_aud0_ cross join AuditRevisionEntity auditrevis1_ where client_aud0_.id=? and client_aud0_.REV=auditrevis1_.id order by client_aud0_.REV asc
Hibernate: select this_.id as id1_0_0_, this_.timestamp as timestam2_0_0_, this_.auditor as auditor3_0_0_ from AuditRevisionEntity this_ where this_.id in ()
23:27:06.893 [tomcat-http--32] ERROR org.hibernate.engine.jdbc.spi.SqlExceptionHelper - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1


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


So do you want to merge the pull request?

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Tuesday, November 24, 2015 2:23 PM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Removed check for NULL sysContext in _Prepare. Moved this check into CommonPreparePrologue. (5da9e09)

Nothing else. -Jimmy

From: wcarthur1 [mailto:notifications@github.com]
Sent: Tuesday, November 24, 2015 12:17 PM
To: wcarthur1/TPM2.0-TSS
Cc: Wei, Gang
Subject: Re: [TPM2.0-TSS] Removed check for NULL sysContext in _Prepare. Moved this check into CommonPreparePrologue. (5da9e09)

Ok, I can remove that.

Any other issues?

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Tuesday, November 24, 2015 11:07 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com<mailto:TPM2.0-TSS@noreply.github.com>>
Cc: Arthur, Will C <will.c.arthur@intel.com<mailto:will.c.arthur@intel.com>>
Subject: Re: [TPM2.0-TSS] Removed check for NULL sysContext in _Prepare. Moved this check into CommonPreparePrologue. (5da9e09)

In InitSysContextFields()
SYS_CONTEXT->rval = TSS2_RC_SUCCESS;
SYS_CONTEXT->nextData = SYS_CONTEXT->tpmInBuffPtr; + SYS_CONTEXT->rval = TSS2_RC_SUCCESS;

Above is duplicated initialization to SYS_CONTEXT->rval.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/5da9e09207226e34de86316b30262ef784caaa4c#commitcomment-14593124>.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/5da9e09207226e34de86316b30262ef784caaa4c#commitcomment-14596800>.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/5da9e09207226e34de86316b30262ef784caaa4c#commitcomment-14598103>.


E.g. "%"PRIu64"" -> "%" PRIu64
On Nov 24, 2015 12:17 PM, "Brian" <notifications@github.com> wrote:

> What extra double quotes - "%PRIu64" isn't going to work...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nzamps/coova-chilli/commit/616193a3f9a2012bfb3f39d82637078b619cff90#commitcomment-14599459>
> .
>


In a way, yes. However, this cop leaves indentation intentionally for other cops. I'll add comments to clarify it.


After 60 s of no ADS-B data being received from a vehicle, the PING transponder will drop the vehicle from the list of vehicles being communicated over MAVLink.  It will simply count from 0 - 59 and then disappear from the communicated list of vehicles.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.7+7](http://build.prxm.it:8811/viewLog.html?buildId=4863&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests passed: 132; exit code 1 Build time: 00:03:05


What's the boolean return value supposed to convey?


Would it work fine if we added more white space around the pipes? They look weird now.


chrigi, die verschachtelig macht nur sinn, wenn du s'``a`` au ohni ``:hover`` bruchsch^^


The distance between eyes. ;)

*Jérémy Heleine*
Web Developer
http://jeremyheleine.me
GitHub: JeremyHeleine <https://github.com/JeremyHeleine> | LinkedIn: Jérémy
Heleine <https://fr.linkedin.com/pub/jérémy-heleine/55/a14/64a>
Twitter : @JeremyHeleine <https://twitter.com/JeremyHeleine> | Google+ :
+JérémyHeleine <https://plus.google.com/+JérémyHeleine> | Facebook :
jeremy.heleine <https://www.facebook.com/jeremy.heleine>

2015-11-24 19:58 GMT+01:00 Pedro Mendonça <notifications@github.com>:

> Is this eyes offset the distance between eyes or a relative attribute?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JeremyHeleine/WP-Photo-Sphere/commit/3b9ad40a10da9b4da4240934fe3d74fafdcc1e16#commitcomment-14597606>
> .
>


метод параметров принимать не должен: он работает с полями класса _basis и _size


@stefanpenner are you suggesting adding the config option back, adding a warning that it will be removed, and then removing it in 2.0?


``& i`` === ``i``


соответственно этой строки тоже быть не должно


Здесь будет утечка памяти. Напиши delete[] proj;


@sgdc3 Wouldn't `commandArguments.getCount() >= 1` make more sense? Otherwise the command will just do nothing if there are more than one argument.


Ok, that's fine. I am trying to figure out how to simplify this function further. Right now when I read it I see that there is two levels of nesting of if condition under for loop.


Maybe we should talk about this one. I just saw, that the above approach was already implemented before, but then changed with: https://github.com/jxcore/jxcore-cordova/commit/8641f8f22fbfef0fa3f612eafe203a4b4d50b262 . However It didn't work for me as onStop/onStart.


мы же еще пример разбирали char *a, *b; тут по другому и не поставишь.


Здесь тоже про параметры не забудь


This looks like a bug to me. :D


@sgdc3 Can cause null pointer exception. If more than argument is provided, `newState` remains `null`. Please change the check to `>= 1` or return an error message to the user.


helo friends.
changes appear in version 7.4.3?
regards.


Small thing, but I think `isHandleRealmToken` makes more sense as `handleRealmToken`. I think the "is" prefix doesn't read as well for parameters as for methods.


This commit broke datatables helper for our datagrids. We need to be able to get the total count of our filtered records, before we 'get' the limited/offest set of records. This commit clears the query params on the count and then returns inaccurate data on the get.

Change to pass an optional boolean to count method whether to clear or not?


We will have to use our home made mixin that take into account the context. I think that we should do it in another PR


Could you removing this for now?


Already fixed that myself. Looks like something went wrong rebasing the BrahmaLoader (commits were a mess before rebasing).


@rusty595 "Mountain.png", "Sky.png", "Tree.png" and "Track.png" all need putting through a colour limiter.


Static import?


Логику пока не смотрел


Hmm, I'm curious, here's a question: If we execute a query like this one naively, we will get some tuples multiple times. E.g. if A call B which calls C, and we execute the find callgraph globally it will run on each of A, B, and C and might produce some tuples multiple times (e.g. B->C x 2). What happens presently? If we do have the result multiple times, we should find a way to avoid that.


Could we also do a second call where we specify ``EncodingType=url`` and show that it is not being url decoded? You could just add it to this test.


Remarks:
C1 shall be as close as possible to IC1 Pin (for better decoupling)
C2 shall be as close as possible to U2 Pin (for better decoupling)
C3 shall be as close as possible to U1 Pin (for better decoupling)


FYI Release was picked over HEAD (Release-1.2 over Development) in all conflicts on this merge.

app/helpers/application_helper.rb:470
<<<<<<>>>>>> release-1.2

app/models/benefit_group_assignment.rb:42
<<<<<<>>>>>> release-1.2

app/models/censu_employee.rb:180
<<<<<<>>>>>> release-1.2


Aaaaaah, ok!


Ah yep, ok


I see, and since the server controls the timestamp key, the user can't really specify more than one key and a threshold > 1.  I think it makes sense for the server to detect where the user messes up their repo, and tries to prevent it.  Good point, will add that check, thanks!


да, но при этом обращали внимание, что такого делать не стоит )))) Завтра можем уточнить, на всякий случай. но вспомни то, что писали сегодня. И Игорь еще просил указывать на это в коде, т.к. в привычке такого написания нет.


Sorry for noob question, but how to download kernel source code from codeaurora?
All wiki info seems to be outdated and difference between github and "that thing" makes me crazy;)


Why have you done that?
we don't need create and edit since we don't use views, in fact I have removed these from routes


@Emil-Sila


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


bien vu pour le submodule Mg, faudrait voir ce qui a merdé pour que ça ne ce reproduise plus


You guys do realize that you can have both `python` and `python3` installed at the same time, right?  (At least, that's how it works on Debian.)


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 45](https://teamcity.labkey.org/viewLog.html?buildId=287363&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:49


Reviewed with Ben, :+1:


What's the advantage of the dev versions on DT and leaflet?
Sorry for the change if it broke stuff - assumed it was 'cause they weren't on CRAN yet!

Robin


lulzkay


@judy2k
Good point, done in 74a739f.


Also may be worth using `update.packages()` from time to time.


i decide to move gegl03 in gnome-extra. Only gnome-photos needs it.


indentation


shouldn't this check use cur in consideration?

shoudln't it be something like (cur + path[i].len > buflen)?

Otherwise it only assure that no path part is bigger than buflen, not that it will fit in the buffer.

Anyway, even fixing that, how should a user of this API know that it was able to write all slices to the buffer?
It may return cur == buflen even if some paths weren't written to buffer.

Am I missing something?


Sorry, it was late here last night, it seems you do need the following so that the next time the dropdown is rendered it selects the right item:

``` javascript
	        onChange(event)       { node.dropdown("trigger change"); }
```


Ej Nikola nadam se da je ovo uspelo. Uglavnom danas sam malo doradio izgled sajta I kontakt stranicu sam izmenio malo.


Inputs should be obtained from the `$build` directly rather than going back to the model.


@songm5178 are you able to provide a test for this change?


The bracket spacing here is a bit off. Unlike the workbench formatting, there should be no extra space within brackets. If you're using Eclipse, you can import [this file](https://github.com/errai/errai/blob/master/errai_eclipse_format_config.xml) and run the formatter to fix this.


I think most of our issues are when we are using semantic ui modals... otherwise things seem to function properly.


rename to ``addMetadataHistory``


You mean the wrap in Statement? It wouldn't be needed but I don't expose constructors currently and I don't think I will have time to add this.


please make sure that this class does not use ``MetadataRecord`` at all now. Remove the import statement as well?


Calling the empty constructor would not initialize the tag of the Tuple.


I moved things to `Suave.Operators` later on the PR.


Maybe use a stricter condition here in order to avoid inaccurate estimates near the start. For example:
```
if so_far_seconds >= 1.0 and item['percent_done'] >= 1.0:
```


ok. as this script does not generate data for consumption by somebody else, that's probably is not a concern. But i am not particular about it.


Line is longer than allowed. (94 > 80)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 68 to 68.*

```
                                                                                vvvvvvvvvvvvv
    ddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd
```


Line contains following spacing inconsistencies:
- No newline at EOF.
- Trailing whitespaces.

*Detected by SpaceConsistencyBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the lines 69 to 69.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/results/ResultFilter.py
+++ b/coalib/results/ResultFilter.py
@@ -66,4 +66,4 @@
     """
     return False
     ddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd
-    +
```


Ah, good idea with your nicely abstracted, reusable functions. :)

Although, in looking at this in more detail, I think there is a subtle difference in behavior between what's happening here and `repeat_exec`. In `repeat_exec`, the task is scheduled to execute again on the set interval before the function is called. And when in used in conjunction with the mutex, I think that makes sense. It ensures that `repeat_exec` is called precisely every X seconds and then it's up to the mutex lock to figure out whether or not the underlying function should be executed (or skip if it's still running elsewhere).

However, in this file we don't have a mutex or lock of any type. Instead, we're calling the function first and then whenever it finishes, we schedule another call. So this means that in reality the `check` method is actually called less frequently than the interval period, since it's only scheduled after `do_check` finishes (which I think is quite quick, but does take some time).

So I think if we were to use `repeat_exec` here ***and*** the `do_check` also took longer than the interval (0.25 seconds), then we'd have two simultaneous calls to `do_check` open within the same worker process. I'm not sure how likely that case is, but I think it's a theoretical condition with `repeat_exec` that wouldn't be present with the linear approach currently present in this file.

Or at least I think... Recursive timers make my head hurt. :)

But I think we could abstract this linear model for calling a timer without any locks to a helper function to clean this up a bit.

Or actually, I think we could go back to `interval_lock.repeat_with_mutex`, but we'd just need to make the locking key specific to each worker. That would allow each worker can operate simultaneously, but still prevent overlapping function calls within a single worker. I think something like `interval_lock.repeat_with_mutex('distributed_rate_limit_pusher:' .. ngx.worker.pid(), delay, do_check)` would work, since then the key is unique to each worker process.

Hm... any preferences or thoughts? I'm sort of liking the look of the latter (using the PID in the mutex key), since that seems to make the desired behavior a bit more explicit in the code.


Line contains following spacing inconsistencies:
- No newline at EOF.
- Trailing whitespaces.

*Detected by SpaceConsistencyBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the lines 69 to 69.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/results/ResultFilter.py
+++ b/coalib/results/ResultFilter.py
@@ -66,4 +66,4 @@
     """
     return False
     ddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd
-    return True+    return True
```


Hm it probably isn't to nice if the user can crash the whole Envision only because he entered a query that can't be parsed properly.


It is OK. It solves the problem now.


Previously we did not have a way to figure out if we created a partition on disk or it is an error. hence the suggestion was to skip the disk. But with querying pvs from VG we know that what are valid partitions and what are not. So we are in a position to error out. Hence it makes sense to do that instead of ignoring error.


wouldn't a more versatile solution be to have a check in `add()` whether there is a tag and if not to take it from the thing we're adding?


This can still divide by zero.


No it is all a set :) that's one thing I love about TupleSets


Nikola danas je doradjen dizajn sajta. Rasporedio sam malo bolje div-ove na pocetnoj I na stranici games.


Hm, does this mean that currently we can't create Nodes from python? I thought we somehow needed this for the refactoring support.


Right now docker-storage-setup does "set -e" that is any error happens and if that error is not handled, script will exit. So even if we you returning error from create_disk_partitions(), script will simply exit.  So we don't have to worry about
calling create_disk_partitions() in [[ ]] block.


Huh, yep.  Ill put out a fix in a minute.  I think it should by default be as my update...a terminator method like ->get().  So ill add an optional param and let you know.


shouldn't this be classes instead of classe ?


Oh, I see your point. Since these are now user-visible routines, using Exceptions instead of Q_ASSERT does make sense.


Cool.


CreateDerivativesJob will never be created until CharacterizeJob has run. Since you've stubbed CharacterizeJob there's no need to stub CreaateDerivatives too.


https://github.com/Zedron/Whiff/tree/legion also your code may hook but it doesn't work, just fyi. same with mine. blizz changed smth with the packet header & i haven't had time to find what


@SubJunk what version are we using?


There's nothing wrong with actually writing `[128*1024]` like the original code did :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.828](http://teamcity.monogame.net/viewLog.html?buildId=11677&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:22:35


Why use the old gensym? Because I'm not sure about imposing UTF-8 on everybody.

Why use the new gensym? Because its renamings are easier to mechanically undo. Besides, the old gensym has the following (fixable) problem: it might rename x to x1, then x1 to x11, then x to x11 also.


In case this is useful information, the plan for https://github.com/ElementsProject/elements/tree/alpha-0.10-multi-asset is to rebase it on top of the latest alpha (I can do that if needed), rebase alpha on top of bitcoin core 0.12, and then launch beta with alpha's features plus multi-asset support (and maybe some other features).
I haven't looked at the whole branch, but maybe some improvements could be PRed to elements/alpha-0.10-multi-asset. But take into account that I will force push to that branch whenever I rebase to the latest alpha (and potentially to remove some UNDO commits as well from time to time).
Those PRed changes could be used for beta once we've rebased to 0.12 though, so it may be worth it for you even if the development process can be a bit more painful due to the force-pushes.
If you are interested, let me know and I will rebase and force-push elements/alpha-0.10-multi-asset.


edown again... we pulled in an old outdated repo from esl.


wewew


lulz its twice now :3 :3 :3


The hardware's capable of transmitting either in the 978 ADS-B band or in the 900 Mhz ISM band.  So we're working towards certification in both bands and eventually will have this capability.  It will have the same ADS-B format when transmitting in the 900 Mhz band and would appear in the same list of vehicles.


Hm I don't think it ever makes sense to create an empty Tuple, so I rather prevent the behavior in the first time.


Yeah, it's mostly just about keeping the version consistent to reduce numbers of dependencies.


Removed $ on barber price


git clone https://github.com/martende/play-salat
cd play-salat
sbt publish-local

and then in your project build.sbt
val appDependencies = Seq(
  "se.radley" %% "play-plugins-salat" % "1.5.1"
)


Alright thank you xfix.


hepsinin probability değerleri aynı geliyor


ok this seems to be the only way that actually works:

```

Template.afSelect_semanticUI.onRendered(function() {
		var instance = this;
    var inputControl = instance.$('[data-schema-key]');
  	instance.$(instance.firstNode).dropdown(_.extend({
        fullTextSearch         : instance.data.atts.fullTextSearch || false,
        allowAdditions         : instance.data.atts.allowAdditions || false,
        maxSelections          : instance.data.atts.maxSelections || false,
    		allowCategorySelection : instance.data.atts.allowCategorySelection || false,
        useLabels              : instance.data.atts.useLabels === false ? false : true,
        onChange               : function (event) {
					// debugger;
					// inputControl.dropdown('activate');
					inputControl.trigger("change", { target: inputControl });
					// console.log('calling refresh');
				}
    }, instance.data.atts.settings));
});
```


Can you provide the specs for this change?


Get glasses

On Tue, Nov 24, 2015 at 9:13 PM Matthew Lake <notifications@github.com>
wrote:

> The intent of the PR was to standardize the project on one style of
> indentation (not necessarily my own, though naturally that was my first
> suggestion), since it's not ideal having several different styles, and
> since 2-spaces are hard to read for some.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UQComputingSociety/uqcs-hubot/commit/5db06d60e7794d1f3afa639b39d8a7a2d11b45e4#commitcomment-14585571>
> .
>


TeamCity LabKey :: Trunk :: LabkeyTrunk :: CDS postgres 9.4 [Build 14](https://teamcity.labkey.org/viewLog.html?buildId=287367&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 6 (3 new), passed: 0 Build time: 00:08:38
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

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

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

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

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

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

```


yes - done


sure!


Fixes #7


@alexcjohnson done


nice, agreed :)


thanks! will do as soon as possible


and again... doesn't work on hard refresh... gah!


Indentation issue (second commit)


If you don't plan to support hex digits
```scala
def parseInt(c: Char) = (c - '0').toInt
```
is easier to understand than the `hexValue` impl from parboiled. :smile:


This commit closes #403


+1. Looks good to me.


both method already lowercased the input, idk which one is better.


very strange... very inconsistent behavior. Now its not working for me when I navigate to two different records. ie: user A: selected state: GA user B selected state FL... only the first one ever gets selected properly.


Shouldn't those changes be part of the first commit?


We still use them here: https://github.com/gradle/gradle/blob/1fd6aa403f08bce2e1f42c3e4d692916bde60518/subprojects/model-core/src/main/java/org/gradle/model/internal/registry/DefaultModelRegistry.java#L224-L224

Without this we end up with unbound rules if a node is removed before it is created. See https://github.com/gradle/gradle/blob/1fd6aa403f08bce2e1f42c3e4d692916bde60518/subprojects/model-core/src/test/groovy/org/gradle/model/internal/registry/DefaultModelRegistryTest.groovy#L1187-L1187:

```
The following model rules could not be applied due to unbound inputs and/or subjects:

  unused creator
    inputs:
      - unknown Object (unknown) [*]

[*] - indicates that a model item could not be found for the path or type.
```

I'm not completely sure if we are storing the right rules here, though.


Yes, but if we're locking down the API for V1 it becomes annoying to change it, right? When it's as much work to write a reflection thingie in a new csproj file... In my opinion.


Indentation issue (3 spaces instead of 4)


Make sure you have tests for the empty parameter `()`, multiple parameter `(a, b, c)` cases, and empty function `{}` cases.


Another change for the first commit.


Just to make sure I understand it correctly before translate, as the default value is 5, I believe this is an absolute value in centimeters, right?


Nit: Inconsistent quote style.


In field.php the way form field IDs are created is not consistent with the way JHTML generates the ID for usstate and country fields. This causes problem with the JS. We can modify the IDs to match the way JHTML generates them (which is what I had in my code originally), or change JS to make an exception for those two fields.


First commit?


Hi, I can't find out where the class GraphView came from. It's not imported, and it's not contained in the package either.
Is the master branch stable?


First commit.


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


ORS formatting was a hold over from previous error message.  Missed changing it back when I added the pvs query.


Wouldn't that be the same as the one on lines 49-53 above?


@DNx5 Oh, sorry I didn't notice. I just saw that it pointed to an interface. If methods need lower-case input it makes most sense if they make it lower-case there, so the current status. For someone else using that method, it doesn't interest him whether or not lower-case is required – he just wants to use the method with the player name. So then it's fine. :)


Missing EOL


Minor thing, but maybe rename `var3` to `e1` or similar so that it is consistent with the other catch clauses.


holy shit! *finally* got my username set up right on this git... jeez...


oops i thought i deleted an extra line :P


In field.php some of the table labels are missing <span class="label label-info">, is that on purpose?


1 space too many.


looking good bob x


Typo: beeing


*as* opposed to


I like this version. It's very clean!


@westcripp just keeping the method shouldn't affect right? I kept so we can use later incase needed


We should be able to ditch `matchbrackets` now, since `matchkw` supersedes its functionality.


FewDocFixes (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

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


No `lazy`. The object itself is already lazy.


meta-data -> metadata


Oh yeah! Good idea! Thanks @bryce-anderson.

I still have plans on extening the functionality of `TooFastString` to support `tryX: Try[X]` so we can reuse it for request readers also. Will keep that in mind!


what is this


This file has the commands from the CSV written into it, which is exported to texutil. It then deletes the commands from the file since some of them are not ready to be released to the public.


Consider creating this separately in `js` and `jvm`, and use a constant `final val executingInJVM = true/false` instead.


After some discussion, @lucasdemarchi and I came to the conclusion that `ROTATION_YAW_293_PITCH_68_ROLL_90` name was just badly written, since, by looking at the implementation, the method `rotate()` from `Vector3` classes adopts the convention 123 intrinsic (for clarity, I mean applying roll, then pitch, then yaw). We just need to fix the name. Because of the name, I thought the for that would be yaw, pitch and roll - consequently, I wrote the wrong unit test for it.

Now, for the `from_euler()` function, the PDF it points gives the idea that the operation is 321 intrinsic (*at least from my perspective :-)*), although the order the parameters are declared may clear that possible misunderstanding, maybe it would be nice to clarify that in the function comment (Lucas and I had to verify the matrix calculation to be sure about the convention used).

That said, I think:
 - this commit can be ignored
 - we should fix the name for `ROTATION_YAW_293_PITCH_68_ROLL_90` and also add more clarity for `from_euler()`
 - I should fix the unit test for that rotation

I hope everything is covered.


I would suggest changing both of such calls in this class to use StringPlus in the registry API since it handles all such cases, and this is missing many. See the definition of String+ in the OSGi core spec.


Does it mean anything when you REQUIRE_PROPERTIES in `$..book`?


@rnewman I was having race conditions using >>> since in the case of deleting two items at the exact same time, they would execute in parallel. I want each deletion to be serial so I threw this in a serial queue and and forced the deferred operations to be sync. Is there some deferred util command that's better for this or is this fine?


Yes, that's right. And I should remove the buflen-- too.
This API is not really giving this information. When I wrote this patch I was just fixing errors, so I wasn't worried about this, but I'll write a v2 returning ssize_t with negative values for errors.
It was not a big problem anyway because in some places where this method is used there is no check if the return is correct (we should fix that) and in other places we know exactly the expected path len, so checking if cur == expected_path_len is enough to check if everything is ok.


Remove puts statements


if you can see this comment, give me a high five


@m-j-w I had to remove several checks that were specific to the Visual Studio integration, since Clang-cl works just fine with other CMake generators too (I use Ninja on Appveyor right now). Since you added them in the first place, does this seem reasonable to you?


Why write a considerable more complex piece of code, involving reflection, instead of the above simple code that is supported naturally by the language ?

That is is "a C#-ism" is not an argument.


Okay I will adapt it.


this is true.


@jsegu: Thanks!


:tada:


@liewegas Will add that..


removed this because I reviewed who was using `reduceDim` which is somewhat sketchy when you are collapsing dimensions (the dim index could change), but this kernel didn't depend upon this weird dependency.

i'll likely address reduceDim later on.


@SubJunk I tried to find out when it was implemented but I couldn't find it.


This works for the specific case of the keywords vocabulary in plone.app.vocabularies, but there's no guarantee in general that vocabulary terms will have a token that is made by encoding in this way. To work generally, the converter needs to convert token to value using the vocabulary's getTermByToken


Hm yes we can with Node::createNewNode() I could use this instead of this wrapper.


Catch exception if `File.delete` fails


old code didn't work with tensors that had holes


Done.


Only set to nil if no exception was thrown


You can't determinate if user uses a edited version if you take number of master. Or I'm wrong?


taken from old copy() code, i'll refactor more of that out later


Not sure why the change required removing the customizer here. This seems much more fragile. Not arguing the logic, but the tracker + a managed collection via the customizer is a better model. Can you make these changes against that model?


I haven't figured it out yet what the problem is but I got something showing. It seems that first model works fine but then the same model is infeasible or unbounded. Why?


why this removed?


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


that's a lot of modes and a lot of bars!


For Java 7,  the version indicated is for OracleJava
For openJDK
```
java version "1.7.0_85"
OpenJDK Runtime Environment (IcedTea 2.6.1) (7u85-2.6.1-5ubuntu0.14.04.1)
```


have installed the app and some errors have gone but some still remain
https://youtu.be/74F9orCHPLM


too hard to implement because every ide use a different resource filtering methods


No, they need to be added like this:
```
<td class="key"><span class="flexi label sub_label">...</span></td>
```
some are : ...
we need to change them so that they are: ...
- so that they are constent with the fields in the rest of the form

about ids of fields and field's sub-properties
you can give custom ID to JHTML, JHTML has a default which is not enough

we need format for IDs:
custom_fieldName_valueCount_subPropertyName

it matches the fieldname:
custom[fieldName][valueCount][subPropertyName]

it is what works properly with multi-value and fieldgroup


I will check, it should, but it might introduce noop queries. Optimally the parser should remove the spaces around pipes, maybe we can do this easily with ```replace(" | ", "|")```


:+1: noted


Also, not ssh.


This is a primitive that can be used to construct pipes, io multiplexers and event pumps. Looking to integrate downstream at https://github.com/domokit/sod


Bootstrap version 3.3.6 was released today and this updated version numbers in necessary files.


what do you mean with a managed collection?


Every time you reformat an entire file for whitespace I have a conflict on my branch. Are we going to be doing this every day?


@Florent8
Ligne 2 manque un ;
Manque les tests isloged avant d'agir sur la BDD
Tu as utilisé des simple quote ' au lieux de double cote "


a collection that is managed by the customizer and used by the callers rather than using the _serviceTracker directly.


New-Fan-Object (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1971 of 1971 tests passed)

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


The UE team discussed this today and decided that "Publication" was a little too narrow, since we also want to review posters and slides and blog posts before they become public. We would like to change this to "External Communications Approval Process."
Can you also change the name of the file? (I'm not sure how to do that--it's possible you'll have to delete and recreate.) "Public Relations" isn't really a good name for this process. We are using it just for historical reasons (because the mailing list happens to be called pr-approval -- but we are going to request an alias to be put in place, communications@kbase.us).


Added it to `software-model.md`.


They are commands that are affected by the reading from csv file if I remember correctly.

On second thought, do you think that changing them instead of removing them would have been better?


Lgtm! :)


Content is sent to PR approval team via this email: pr-approval@kbase.us (PR is an acronym for Public Relations) ->
Lead author of content send it to the External Communications Approval team: communications@kbase.us or pr-approval@kbase.us.


Done.


Within two days, the External Communications Approval team will either suggest edits or approve it.


Since this change, ``./package.sh -p noredist -d centos63`` is not working on centos6 OS with following error:

```
-bash-4.1$ ./package.sh -p noredist -d centos63
 -p noredist -d centos63 --
Packaging CloudStack...
noredist
Preparing to package Apache CloudStack 4.6.1-SNAPSHOT
. preparing source tarball
. executing rpmbuild
error: Macro %_ossnoss has empty body
error: Macro %_ossnoss has empty body
error: failed to stat /data/lib/jenkins/workspace/CloudStack-PR_noredist_rpms/dist/rpmbuild/noredist: No such file or directory
RPM Build Failed
```


Unfortunately, I have no idea what a csv-parser does. Does it just combine all the data from the CSV into a string a certain way?

Could you clarify "read the commands on the fly"? It sounds quite vague to me and I'm not sure what exactly you mean (I hope I'm not missing something obvious).


2 or more people from the PR-approval list are then needed to indicate their approval  ->
Two or more people from the External Communications Approval team need to indicate their approval


Оказывается внутри тредов на 2chan.net был тэг `base`, назначение которого я до конца так и не понял, но он ломал `xlink:href` ссылки.


@valib @SubJunk We FFmpeg used for OS X and Linux is badly outdated anyway, aren't they? Is there a reason why we hold them back? We can't make a change like this without ensuring that it also works on Linux and OS X.


Can you explain what do you mean with "referral" parser? How is this different than the BaseIcannCompliant?


PR-approval -> communications-approval@kbase.us


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


Hi Roger. The logic here looks right, but I'm wondering why you're using a reader to go through the properties file line by line? I think loading the whole file into a string is cleaner, and it seems like you could use a regex to find the realm from the string.


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


it is using a ServiceTrackerMap, not a tracker directly... I don't know if you still don't like it...

Eventually it will use a ServiceTrackerListener to do that logic at insertion time...


Should we include a progress image in `index.html` in scaffolds and the generator?


@Subjunk @valib my initial testing of FFmpeg 2.8 64bit works so if we want to make a switch we should. the version i'm testing is from here http://www.osxexperts.net/ffmpeg/ffmpegexperts.html


Vom Kabelerlegen


[…]mitzuhelfen, kannst du […]
[…]eintragen, über […]


yeah, it needs to do it during service bind otherwise you push all this onto every runtime call. You will be exploding the cost of the operation by a factor of the callers rather than by a factor of how often the services come and go (which is very much lower).


Hi


Ouch! There was a plan but I didn't have time to integrate it. I will un deprecate them ASAP there is a patch in the pipe.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1435 of 1435 tests passed)

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


Line is longer than allowed. (98 > 80)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/Diff.py`, lines 259 to 259.*

```
                                                                                vvvvvvvvvvvvvvvvv
#dddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd
```


Commit


@liewegas It is helping the following way..

1. It is delaying call to prepare_multi_write and thus giving chance to coalesce more transaction.

2. Also, it is helping to reduce #aios in flight as you mentioned.

3. No more hard throttle (i.e waiting on a lock) except journal full scenario.

4. Even if queue_op throttle is after journal write, the single journal finisher thread will be blocked and thus journal won't be going ahead too much.

<<It looks to me like there is nothing preventing you from queuing a million Transactions at once

The problem of allowing so many aios in flight , journal will be moving ahead too much and if both journal/data on the same SSD it will be competing for disk write with backend flush.


I've re-added this ability: https://github.com/SpongePowered/SpongeAPI/commit/687fb8d74c89f64acb42c52f8cbccaa173c8832b


yup... I tried to summarize that in the commit message... so I am glad we are on the same page


`args` here is the FileBrowser controller, yes?  If so, please rename `args` to `fileBrowserCtrl` or somesuch.  Ditto for the other controllers and views defined in fileBrowser.js.


@peaseloxes Ha wtf, was dit een SonarQube issue?


@liewegas will address all of these..


is this for google.com/foobar/ ?


Well the answer is yes hehe, initially i saw that you are reading the whole file and just handling the string content, but I just used same approach as the one that is used in the sources for the wildfly users management utility.. so it was just for being quick and ensuring the code will work, as the wildfly classes are supposed to be tested for lots of users :)

The users/roles files use to be quite short and this feature only applies to SDM, so althought it can be slower than using regexp it should not be a big issue using any of both impls. But anyway, if you prefer reading the whole content and using regexp, honestly there is np for me! i can refactor the method as soon as possible :)


TKTConcurrentAccessor


Как ты догадался, демон? В жизни бы не подумал на эту хрень.


У нас есть $t, ежели чо.


@liewegas sure, will comeback with that, wanted to show you the changes..


Couldn't you just use FileUtils.readLines(propertyFiles) here? It will return a List<String> for all the lines in the file. This save lots of code for opening and closing the BufferedReader (incl. exception handling).


TWT...


looks good to me!

Thanks for fixing


Is there any way to avoid race conditions in the script? Otherwise, would it also be unfavourable to use `phpDoc/master` as a symbolic link to the latest-release instead of development-master? It's the most referenced link in our user docs, wiki, etc...


Я наоборот думаю все `$t` заменить на `$q`. В `$t` создаётся список всех элементов с данным тегом и возвращается первый, в то время как `$q` сразу возвращает первый.


Since we're going through all the lines anyway this shouldn't be any less efficient.


Ох, щас потещу! Однако!
Этот релиз скрипта будет самым винрарным. Столько всего уже починено.


good point :)


This block we won't need when using FileUtils.readLines() as we don't have a buffered reader.


SanitizerFixes (DeadParrot) - x86_64-MacOS-10.9-clang: Tests Failed

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


two spaces between `a  you`


@liewegas like this ?

(percentage_empty % 5)


Хм. действительно.
но тогда то же самое можно сказать и про $c


@alexarchambault I've had to take this out because it doesn't match the underlying representation any more. I don't think the problem that this was meant to catch is possible any more, but I'd appreciate it if you'd take a look.


Скажите, лучше ветку поставить dev или master? Я просто хочу фич парочку добавить, не знаю вот, какая ветка лучше. Очень жду ответа.


also potentially lost part of your sentence here: "or would prefer to use a  you must"


new pattern doesn't accept my email as valid

15 letters followed by "@gmail.com"


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


Short Demo results


```js
function $c(id, root) {
	return root.querySelector('.' + id);
}
```

м?


Why not just move it to `dependencies` and be done with it? `peerDependencies` is an actively hostile config, spewing warnings in npm2 and not doing what you expect in npm3.


Про классы история другая. Там браузер использует отдельный хэшмах `класс -> элемент`, поэтому выборка не будет зависеть от количества элементов. Когда я только писал код для перехода от Xpath на классы, то `$c` был быстрее `$q`.


@axelstudios is this where you would patch ruby?


one more space to indent


Это фикс для хрома, лол.


TWT...


Because libraries like this can be tested, validated and used to hide complexity for the benefit of the consumer.

API surface area is a valid reason to avoid it.

Another reason is that with this you're mixing styles, making it harder for anyone going about reading the above code.

Another reason; if we create a separate C# assembly we can do much more than the above. And it helps developer uptake to make C# APIs idiomatic.


get rid of setting


"Reads are inherently synchronous, **but** you can safely read from several threads at once"

Is but the right contraction there? What is the point you're making?


Another one for @rukano :-)


Тогда сейчас уберу все `$c` и `$t` функции.


oops, I think we both forgot that :p


@axelstudios could we just change the OpenSSL version here?  I'm not sure if this is related to the one with Ruby or not.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 46](https://teamcity.labkey.org/viewLog.html?buildId=287381&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:00


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 46](https://teamcity.labkey.org/viewLog.html?buildId=287385&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:01


Наткнулся на этот баг: https://bugzilla.mozilla.org/show_bug.cgi?id=1081654


I don't see a `list` attribute on the FileBrowser controller, I think this is supposed to be `.collections`?  When I try to add/create/delete I get a JS error whenever `.list` is used.  You might also consider making these methods on the File Browser controller,  `fileBrowserCtrl.addToCollections()` and `fileBrowserCtrl.removeFromCollections()`.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 45](https://teamcity.labkey.org/viewLog.html?buildId=287361&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33 (5 new), passed: 255, muted: 4 Build time: 01:12:11
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


>Getting close.. One more thing: The default for Option.DEFAULT_PATH_LEAF_TO_NULL is false, that changes the behavior too. For the Goessner example, 2.0.0 used to return [ null, null, "0-553-21311-3", "0-395-19395-8" ] for $.store.book.*.isbn with no special options set, 2.1.0 returns [ "0-553-21311-3", "0-395-19395-8" ].

Are you saying that Option.DEFAULT_PATH_LEAF_TO_NULL was set on version 2.0.0? The test below is green on both 2.0.0 and 2.1.0. What am I missing?

```java
@Test
public void isbn_is_defaulted_when_option_is_provided() {
    List<String> result1 = JsonPath.using(JSON_SMART_CONFIGURATION).parse(JSON_DOCUMENT).read("$.store.book.*.isbn");

    assertThat(result1).containsExactly("0-553-21311-3","0-395-19395-8");

    List<String> result2 = JsonPath.using(JSON_SMART_CONFIGURATION.addOptions(Option.DEFAULT_PATH_LEAF_TO_NULL)).parse(JSON_DOCUMENT).read("$.store.book.*.isbn");

    assertThat(result2).containsExactly(null, null, "0-553-21311-3", "0-395-19395-8");
}
```


I actually would not create a makefile but use a project generator along the lines of CMake instead. That will work than in any way you want (IDE, command line) and is platform independent.


Thanks for the suggestion. Done, and merged into the 2 options as well. Both options to handle it still work.
I can squash them all into a single commit once I get your feedback on which option you think is better.


Применять функцию выше там, где у нас id переменная. Чтобы не писать везде `$q('.' + idBlaBla, doc)`. А если аргумент строка, то добавить точку и заменить на $q.


That sounds better to me. Please make the changes.


to make the code cleaner, I would the individual features in individual functions and only call these functions here


I must admit I didn't even think about trying ninja with clang-cl. Then, giving the link to the wiki seems to cover it. However, the version check (<=1900) might be helpful since clang-cl can also report lower number when giving a lower and non-default `-fms-compatibility` (or named similar) parameter, which then also reduces to before C++14. But for that purpose there's the cpp version macro, and that might also simply be covered in the wiki...
So, no real objections from my side, and now I'll also switch happily back to ninja.


where do you reset the centroid to 0 before you enter this loop?


На самом деле просто приделать точку ко всем `aib.cXYZ` путям и переименовать их в `aib.qXYZ`.


@axelstudios another option is to make this a warning instead of an error


That's a bit too risky. But I guess at present the spaces will just be added to the command and it will omit them, so there should be no harm. I suspect it either already works, or will take very little code, in the command/argument splitting to make it work.


The seems to be an issue with loading the usertemplate if the path contains unicode characters, in this case it was how the WebBrowser control handled the 'Ä' char. Wrapping the uri string in a Uri instance solves it.


oh, I suppose `require("_data")` look for either `_data.json` or `_data.js`. This prioritizes the js over the json I assume?


I don't follow here - shouldn't that be spectrum_sum < value?


@milessabin Does it still deal well with traits implementing some macro logic, like https://github.com/milessabin/shapeless/blob/master/core/src/main/scala/shapeless/generic.scala#L275-L276, for example? My use case was similar.


The problem with `text` not being updated comes from the fact that Semantic-UI isn't updating it, not that the value isn't actually set, because if you look at the code of the page, the hidden input has the right value.

A fix would be to do the necessary updating with Meteor, but then what's the point of using Semantic-UI...

I'm sure it's some stupid problem somewhere, but I need a pair of fresh eyes to look into it (though it might not be this package's fault, but Semantic-UI's)...


Should this fix also be in the node.js patch that we maintain?


OK, that's fair.


Ахтунг!!
на форчане
this.cReply = 'post reply';

например


yeah that is what we are looking at as well... very painful!


Хайвмайнд.


Yes, will add it to patch.


approved


These are great. Is there any way to merge them with the main branch (even just on my own PC) without it screwing everything up?


`cReply` вообще для стилизации используется. `$c` с ним нет.


It should do ... please let me know if I've broken things for your case. If you're able to come up with a new version of this test that would also be awesome.


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


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


Let's remove this comment.


Think here's a commit missing before

https://lkml.org/lkml/2012/2/10/303


dat face


@sintaxi yup – https://nodejs.org/api/modules.html#modules_all_together

> LOAD_AS_FILE(X)
> 1. If X is a file, load X as JavaScript text.  STOP
> 2. If X.js is a file, load X.js as JavaScript text.  STOP
> 3. If X.json is a file, parse X.json to a JavaScript Object.  STOP
> 4. If X.node is a file, load X.node as binary addon.  STOP


Almost the exact same function is defined in `fileBrowser.js` in the definition of `updateList()`.  Could you move that function into a variable, and pass `lazyLoadPreprocess: function(value) { mungeNodes(value); return value; }` to the ProjectOrganizer constructor?


Location de voitures


Hey there skipper, are these comments private?


caused build error


Note to my future self: netkan files should not be written in the middle of getting dressed to go out.


For each thread there is a copy of the chromosome currently being processed by that thread (at 1byte per base), plus the baseline and call variants for that chromosome. There is also a single shared in-memory copy of the (bit packed) reference sequence (~1Gb) (although this seems overkill to me, since we aren't randomly jumping around we could instead just grab each chromosome from the disk-based reference with minimal performance implication).  Dynamic memory use can vary quite a lot with the number and nature of the variants. I would guess that something along the lines of Gb =  2 + threads * 0.5 would be an OK rule of thumb. (Whatever you allocate, the JVM will probably go up to that limit by deferring garbage collection, of which vcfeval does generate a lot. And if you don't give the JVM enough breathing space it can end up spending a lot of time doing many small garbage collections).

We don't have any logic for a smarter default score field selection, typically just handling that manually depending on the analysis we are doing. It wouldn't be hard to pick something just based on the availability of the various field declarations in the VCF header. I'm not sure whether it makes most sense to have that score selection logic inside vcfeval vs in your validate wrapper vs whether you may want to allow explicit score field selection.


This seems to apply to all buttons, not just the download button. You might want to put these rules into a selector by id, or introduce another CSS class to cover this functionality if you feel this is still required.


oh ok
maybe removed other stuff needed


Done

Date: Mon, 23 Nov 2015 23:25:05 -0800
From: notifications@github.com
To: Urbo@noreply.github.com
CC: asifziv@hotmail.com
Subject: Re: [Urbo] Update README.md (4ac12ba)

No. Call I'll explain

S.

On Nov 24, 2015 9:17 AM, "assif" <notifications@github.com> wrote:

> Stav except it and than make another chance of both errors. Isn't it s

> broken phone to work this way?

>

> Sent from Outlook

>

>

>

>

> On Mon, Nov 23, 2015 at 11:05 PM -0800, "syagev" <notifications@github.com>

> wrote:

>

>

>

>

>

> There is iOS. Please update the pull request.

>

> S.

> On Nov 24, 2015 2:15 AM, "assif" <notifications@github.com> wrote:

>

> > okay that's my bad. fix this one.I removed iOS coz it's no there.

> >

> > Date: Mon, 23 Nov 2015 14:39:22 -0800

> > From: notifications@github.com

> > To: Urbo@noreply.github.com

> > CC: asifziv@hotmail.com

> > Subject: Re: [Urbo] Update README.md (4ac12ba)

> >

> > Yaa but look at your correction: "the urbo sdk is implemented as a simple

> > android." What is that? Why did you remove the words "iOS" and "library"?

> >

> >

> > —

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

> >

> >

> >

> >

> >

> >

> > —

> > Reply to this email directly or view it on GitHub

> > <

> https://github.com/assif/Urbo/commit/4ac12ba7603a8c20344bb2e301230593426f4ff6#commitcomment-14575846

> >

> > .

> >

>

>

> ---

> Reply to this email directly or view it on GitHub:

>

> https://github.com/assif/Urbo/commit/4ac12ba7603a8c20344bb2e301230593426f4ff6#commitcomment-14580583

>

> —

> Reply to this email directly or view it on GitHub

> <https://github.com/assif/Urbo/commit/4ac12ba7603a8c20344bb2e301230593426f4ff6#commitcomment-14580716>

> .

>

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


that makes it pretty clear. Thanks :)


Consider renaming this "PoopyButthole"


In IoTivity, secure connection is managed per platform, not per device:
```
    OCConnectivityType ct =  (OCConnectivityType) (CT_ADAPTER_IP | CA_IPV4 | CA_SECURE);
    PlatformConfig cfg;
    cfg.clientConnectivity = ct;
    cfg.serverConnectivity = ct;
    OCPlatform::Configure(cfg);
```
Not sure we can/should move secure semantics to ConnectionMode.
Maybe also need an interface to represent PlatformConfig options, as currently it is not customizable from JS.


NICE


Derp, nevermind, I see that the treebeard options passed to ProjectOrganizer are whitelisted.  This is not a high priority, and definitely not necessary for this PR, but you might want to consider isolating treebeard args in their own key in the ProjectOrganizer args.  That'll allow you to pass them all through without having to whitelist, and will give consumers of ProjectOrganizer more control over the treebeard init args.  However, if fileBrowser is the only planned consumer of ProjectOrganizer, that's probably a waste of time.


-> `find src -iname '*.html'`


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 47](https://teamcity.labkey.org/viewLog.html?buildId=287383&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:00


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 47](https://teamcity.labkey.org/viewLog.html?buildId=287387&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:01


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 47](https://teamcity.labkey.org/viewLog.html?buildId=287383&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:01


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 47](https://teamcity.labkey.org/viewLog.html?buildId=287387&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:02


Couldn't reproduce it with actual master. Could you give me some feedback?


> but, can you explain why all those functions were removed, where they unecessary to begin with ???

Yes, they just call function pointer and capture context that was already captured.


Amusing... Percona server, a MySQL fork, has that feature since at least 2010 (since the 5.1.x era)  and regular MySQL has it since 5.6.x.
https://www.percona.com/doc/percona-server/5.1/management/innodb_lru_dump_restore.html
It is indeed a cool feature and I am glad it is available for Pg.


Enabled encrypted mutilplicatoin


This is totally unimportant, but I'm stupidly proud that I got the phrase "ProjectOrganizer" to line up perfectly three times in my browser:

![screen shot 2015-11-24 at 4 58 58 pm](https://cloud.githubusercontent.com/assets/86835/11381880/c45f1580-92cc-11e5-8441-8248b0283d9d.png)


Sorri aga mulle ei andnud rahu.

Siin te itereerite üle `inputTypes`, mille te saite kätte `service.getServiceInputTypes()` kaudu. Mina loeks nii kauget allikat pigem ebausaldusväärseks. Eks otsustage ise, milline on järgnevate juhtude puhul oodatud ja milline ootamatu tulemus:

#### "pehme" võrdus
```javascript
var inputTypes = {
    'musa': {'sizeLimit': 20},
    'pilt': {'sizeLimit': 10},
    'tekst': {'sizeLimit': ''},
    'fail': {'sizeLimit': undefined}
}
resource = {'fileSize': 11}

for(i in inputTypes){
    var inputype = inputTypes[i];

    var sizeLeft = null;
    if(inputype.sizeLimit != 0){
        sizeLeft = inputype.sizeLimit;
    }

    //...//

    if(sizeLeft != null){
        sizeLeft = sizeLeft - resource.fileSize;
        if(sizeLeft < 0){
            console.log('Teenusele "' + i + '" etteantavate ressursside maht ületati.');
            // return callback('Teenusele etteantavate ressursside maht ületati.');
        } else {
            console.log('Teenuse "' + i + '" mahtu pole ületatud.');
        }
    } else {
        console.log('sizeLeft on teenuse "' + i + '" jaoks "null", ehk teisisõnu ', sizeLeft);
    }
}
```
> Teenuse "musa" mahtu pole ületatud.
> Teenusele "pilt" etteantavate ressursside maht ületati.
> sizeLeft on teenuse "tekst" jaoks "null", ehk teisisõnu  null
> sizeLeft on teenuse "fail" jaoks "null", ehk teisisõnu  undefined

#### "range" võrdus
```javascript
...
    if(inputype.sizeLimit !== 0){
...
    if(sizeLeft !== null){
```
> Teenuse "musa" mahtu pole ületatud.
> Teenusele "pilt" etteantavate ressursside maht ületati.
> Teenusele "tekst" etteantavate ressursside maht ületati.
> Teenuse "fail" mahtu pole ületatud.


Deferreds all execute on the same queue by default:

```
private var DeferredDefaultQueue = dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0)

public class Deferred<T> {
    typealias UponBlock = (dispatch_queue_t, T -> ())
    private typealias Protected = (protectedValue: T?, uponBlocks: [UponBlock])

    private var protected: LockProtected<Protected>
    private let defaultQueue: dispatch_queue_t

    public init(value: T? = nil, defaultQueue: dispatch_queue_t = DeferredDefaultQueue) {
        protected = LockProtected(item: (value, []))
        self.defaultQueue = defaultQueue
    }
```

so I think what you mean is that each of your deferred operations was being interleaved, right? You want your post-deletion operation to run immediately after the deletion, before any other deletion gets the chance to run.

You can consider using `.bindQueue`/`.uponQueue` to specify a queue upon which your block is called. I doubt that'll help.

You can use a chaining mechanism: you don't queue up a deletion, you chain it off the last deletion (which might have already been executed). That gives you strict sequencing, because `.upon` blocks run in the order in which they are received, and your chained ones get there first.

What you're doing here is to strictly coalesce deferred execution by picking a thread and then blocking.

That's an approximation of a kind of task management system — delete this, if there's already a refresh going punt it until later…


Yes, I'll need to test it, but probably we can check for `$TRAVIS_COMMIT` to be the latest commit before committing the updated class docs. Otherwise we can still disable [concurrent builds](https://docs.travis-ci.com/user/customizing-the-build/#Limiting-Concurrent-Builds).


Note that you're not checking for failure here.


that url doesn't look right


Не совсем. Косвенно используется. Ща будет кулстори. Я столкнулся с одним багом на `02ch.in`. Кстати, уже починил, хотел выкатить. Суть такова.
Превью у нас имеют вид `<div class="' + aib.cReply + ' de-pview-info de-pview">`
У нас как детектор постов везде используется `qRPost`. В том числе и для превью.
Так вот, `aib.qRPost` практически везде дублирует `aib.cReply`, только с точкой. Исключение - `Phutaba` и `Futaba`. Там они не совпадают.
А теперь, внимание, прикол:

```js
	getPostElOfEl(el) {
		var sel = this.qRPost + ', [de-thread]';
		while(el && !nav.matchesSelector(el, sel)) {
			el = el.parentElement;
		}
		return el;
	}
```

используется впри показе превью при отсутсвии путей у ссылок, например на досках (не в тредах)

'''js
	static show(parent, link) {
		var rv, tNum = +(link.pathname.match(/.+?\/[^\d]*(\d+)/) || [,aib.getPostOfEl(link).tNum])[1]
```

И эта хрень работала до сих пор везде, кроме https://02ch.in/b/ и http://ernstachan.com/b. Если зайти туда и вызвать цепочку превью по картам ответов, то дерево заглохнет на первом же превью. Потому что там `aib.qRPost` !== `aib.cReply`


Why :sob:


why a different JSON and not output?


Got an example of this?


Where is this method used?


I suspect this is why you're hitting a problem. If you generalize this part from 1 to N, you will be forced to find a solution that isn't strict ordering of deletions.

You might also find a different solution if you make this active ("add a new site"), not passive.


While we're here and talking about performance, can we consider using the language construct ```isset()``` in place of the functions ```property_exists()``` and ```array_key_exists()```? These functions are only useful if null is to be considered a meaningful value, which is often not a best practice.


Интересно, почему на эрнстчане не работает панель скрипта? Я с открытым ртом наблюдаю как браузер посылает событие с именем `mouseover` вместо `click`а, а причину понять не могу.


open?


Hey.. I just uploaded a new version to the extension repo in visual studio.. Yours should auto-update.. Mine has not yet.. which scares me a bit.. Please tell me if you guys get it to auto update


missing negation?


AFAIK in OIC a device has a special resource representing the platform. In that sense, device inherits what's needed from the platform. It is devices that are addressed, not platforms. Whether devices from a given platform share secure connection should be an implementation detail. PlatformConfig in IoTivity in fact should be device config IMHO.


Я сегодня днем хотел панель победить, но не разобрался.
Что характерно, не работает, если только кликать по беленькому, то есть на символы. Можно кликнуть в центр шестеренки, тогда стработает - потому что там пусто, лол.


Dies this fix every escape able character? I experienced the bug also at > or <


But then you wouldn't be able to use the BCL from within sketchup


Hy Lorenz,
Can you tell me if the 2.4.5 schematics are good for prototyping ?I would like  to make one on my own. Are there any changes I must make ? You said previous that 2.4.5 is what they are shipping now.
But now you say that there are some modification needed for a resistor ?Can you please explain in dummy language ?
greets

    On Tuesday, November 24, 2015 1:17 PM, Lorenz Meier <notifications@github.com> wrote:

 Note there is ongoing work to include the 220R pulldown which is currently a placement option explicitly. You will want to watch the repo for an incoming pull request. I've also restructrued the folders as there is soon more activity here.—
Reply to this email directly or view it on GitHub.


without this patch, the following commit crashes on the retromap task.
https://github.com/TechReborn/RebornCore/tree/ac264343399fcc688b77d7fd9f5def2afc035c3f
revelant area: https://github.com/TechReborn/RebornCore/blob/ac264343399fcc688b77d7fd9f5def2afc035c3f/src/main/java/reborncore/common/util/VersionChecker.java#L48-L60

Apparently, the range extractor just doesnt have a name for classes created inside methods like that.


added a file to gather changes for released version, kept latest in the readme additionally


no-ope - not by defauly


done


I don't think so, but it's definitely worth a shot.  I don't fully remember how all these pieces are connected


for us I just added a log statement to onRendered and it seems that it is only called once...


u can change it tho


check for the length of chef_server_url.split('/') and warn user if it is less than 4


TeamCity DNN JavaScript Libraries :: CI [Build 121](http://teamcity.engagesoftware.com/viewLog.html?buildId=399&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


This needs to be 2 not 2.0 ( remove the .0)


Thx for PR. If you want to contribute to the project directly please follow the instructions on wiki.typo3.org. I have uploaded a patch set https://review.typo3.org/#/c/44936/.


TeamCity DNN JavaScript Libraries :: CI [Build 121](http://teamcity.engagesoftware.com/viewLog.html?buildId=399&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


@Zedron fixed


removed them from the bounds generation, will write tests right now!


I'll send you an email, one sec...


See https://groups.google.com/a/isocpp.org/forum/#!topic/std-discussion/PLM9Q_VrQ4I


In command 2. under build the "git checkout surfacepro2" returns fatal. I'm still new to this, what am I doing wrong? (this is in terminal)


I think there should be checks to ensure that `settings.backend[message.hostName]` and similar index accesses don't fail. At the very least, silently ignore hostnames that don't have entries.


acc


Yes! I'll make it clearer in WP Photo Sphere 4.0 (early 2016 I think).


op


Eh, we can ignore this edge case: user hits the site initially with no cookies and a plugin gives an unauthenticated visitor a private URL.


You might want to print the exception as well.


Roslaunch will auto launch the roscore, which is why using roslaunch is preferred.


Теперь картинки на архиваче раскрываются куклой. Единственный минус - для вебмок инфы о размерах нет. Взял от балды пятикратный размер превьюх вебмок.


If I click the  Rename/Delete popup for a collection, the collection's entry in the menu gets the `bg-color-hover` class.  However, once I've submitted the rename or cancelled out, the `bg-color-hover` class remains on the collection, even if I click on another collection to display its contents in the browser.  This seems a little confusing to me.

![screen shot 2015-11-24 at 5 13 29 pm](https://cloud.githubusercontent.com/assets/86835/11382230/b8517772-92ce-11e5-9427-5e943d48849f.png)

Here I have 'All My Projects' open in the browser, but the 'Crolldek--Potch' collection is still highlighted.


@taylorotwell Are you open to backporting this to Laravel 4? I was hoping to be able to use MySQL 5.7 on one of my apps and ran into the issue this commit solves when testing it out.


I'd recommend using a protocol buffer or similar, but that's up to you.


Now you should not need the mock anymore.


Yes, true. A loop becomes one more element in such a section of linear code, so it can branch forward.


LGTM!!!!


Not the end of the world IMHO, @DavidGoldwasser what do you think?


So what happens when this turns off? We have a docker with nothing running inside it?


What's with the literal 5?


@Verify.Option(false) marking that default behaviour is not optional


Is there a reason why this translation key is in the goals plugin scope?


I am not sure how this was compiling.  the name of the array element in the struct was screwed up.

Fixed in 2f8a9b1


If you are going to keep using that editor, you should update the gitignore to not track these file endings. http://stackoverflow.com/a/4824199


What happens if the client asks for data on a non-TransparentNode coordinate?


outside of dor-services... for instance, Argo would call `item.defaultObjectRights.content` to see the rightsMetadata XML


It breaks one of mine, but to be precise it'll break any CI process that includes `jslint` and enforces whitespace with the earlier configurable option set to something other than 4-spaces. And it's a complaint from users of https://github.com/73rhodes/Sublime-JSLint where it breaks the workflow in their dev toolchain.


Please rename this as well.


This is working all right in my local PC. I can see the backend interface, and database seems to be working all right, too.


These two lines are newly added. We don't need them before?


Have you considered saving the optimized style content back into CDATA, if that's how the original file is defining it?  Otherwise, this could be considered a mutation instead of an optimization.


nice hack..


commit should read "in development environment", not production


I've no problems with the rename, but it should be mentioned in the commit description, and probably be a separate commit.


perhaps add unique to be even more explicit:
```python
if len(np.unique(axes)) != 2:
```


Только там есть особенность. Архивач тянет треды не сразу, и инфа о размере картинок тоже появляется не сразу.


I haven't check if this is working. If somebody could check if I am on the right track, I will make template files and will test test.
Thanks from Midori


will you forward port this to cm13.0?  thanks


Lol, noob committed private keys! ;-)


*greatful*


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 57](https://teamcity.labkey.org/viewLog.html?buildId=287369&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14 (10 new), passed: 33 Build time: 01:14:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


I ran into a problem while trying to update RenderChainScript - particularly the SourceFilter class.


netagive -> negative


Przez to mi nie dziala :(


Adding ARC to NSString+PECrypt.m should be a separate pull request/commit.

Also, the header-only version makes supporting the code harder as it just duplicates the existing code. Could there be a way to have a dual-purpose .h/.m pair and switch between the usual and the header-only (inline) version? I don't like macros, but maybe they would help here? You could define a macro that would turn methods into static functions and include the inline implementations. Otherwise I'm not a big fan of code duplication.


Small typing error: `in inther places` instead of `in other places`


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


The previous version using DumpQueryFromFiletoDXLFile has 2 params, one for input file, the other one for output file. But using DumpQuery, the input is query string, not input file, it writes output to terminal instead of file. So we need to get the output string from terminal and write to file. That is the reason we need to add these 2 lines.


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1963 of 1972 tests passed)

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


edit


Nit: indent should be 4 spaces everywhere :)


TeamCity LabKey :: Trunk :: LabkeyTrunk :: Git postgres 9.4 [Build 810](https://teamcity.labkey.org/viewLog.html?buildId=287373&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 49, passed: 241 Build time: 00:56:45
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


I got a compile error with visual studio (window) after the mapcache cleanup https://github.com/rathena/rathena/commit/65b6861e84d29a8226680312ca4639bd15373bed
>4>cli.obj : error LNK2019: symbole externe non résolu display_helpscreen référencé dans la fonction cli_get_options
>4>cli.obj : error LNK2019: symbole externe non résolu parse_console référencé dans la fonction parse_console_timer
>4>..\mapcache.exe : fatal error LNK1120: 2 externes non résolus

Sorry it's in french ;)


Done in 2259331c1cb6be9d8a805ae9e055fa28d9a80477


nice, I will do that


Yes, this is a mistake, .data is what it should be.


oh but it is.


DANG SON! This is awesome! :100: :100: :+1:


The class-based implementation could just call the functions from the header. But as the header version does not support ARC, that would result in dropping ARC support.

> Am 24.11.2015 um 23:23 schrieb Gleb Dolgich <notifications@github.com>:
>
> Adding ARC to NSString+PECrypt.m should be a separate pull request/commit.
>
> Also, the header-only version makes supporting the code harder as it just duplicates the existing code. Could there be a way to have a dual-purpose .h/.m pair and switch between the usual and the header-only (inline) version? I don't like macros, but maybe they would help here? You could define a macro that would turn methods into static functions and include the inline implementations. Otherwise I'm not a big fan of code duplication.
>
> —
> Reply to this email directly or view it on GitHub.
>


@justbuchanan oh wow...that's...soooo much better.


yeah


Done.


@JeroenDeDauw May I haz sync, bar no foo. :)


LGTM


Huh, criu already has utils to operate on bitmaps. Used it instead of macro.


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


oh wow, that's a dumb urlq


Should probably add a duplicate instance check to register...


Basically....
Works at this point though and I needed this soon. The CDataStores are really weird now...


Nitpicks: Could you consider using convenience method from [vtkMRMLCoreTestingUtilities](https://github.com/Slicer/Slicer/blob/master/Libs/MRML/Core/vtkMRMLCoreTestingUtilities.h) ?


Мда. А ларчик-то вот как открывался. не углядел, однако.
Я пока лазил по мелкобордам столько фигни внутри насмотрелся.


What do you mean by that?


`onChangeRequest` I'm tirer too


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


I changed the name after the fact, forgot to update that reference. Can't just write to `output` because the stream is already closed. In order to do it without the hackaround, I'd probably need a dummy `through` stream that I pipe to before the file that detects when it receives a `null` chunk and writes the closing `]}` before ending.


@ctrimble Just had a thought - does this actually work? I think the binding libraries will require a getter/setter to make additional properties work. When includeAccessors is false, we should remove all accessors that aren't required, but I think the additionalProperties accessors *are required* and cannot be removed. Or am I missing something?


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-MacOS-10.9-clang: OK (1957 of 1966 tests passed)

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


@DNx5 L66-67 returns if chat is disabled in the Settings. Does it make sense to remove recipients from the event if the chat might be discarded? It would seem that L60 - L65 should come after L66-68


It should, very interesting, will take a look at it later.


Если взялись за унификацию.
У нас в части аргументов указан `doc`, а в части `doc.body`.
Может, стоит выбрать что-то одно? Второе ведь быстрее?


Sorry, I obviously haven't been clear enough again, I was speaking about reverting the `/W4` part of the patch only. I'll do it now then as it seems we're in agreement about it.


Haha, nice!


LGTM :+1:


Don't we have to go `node ./node_modules/pulp/index.js` instead? I see these are devDependencies, but they won't be on the PATH, right?


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 95](https://teamcity.labkey.org/viewLog.html?buildId=287365&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 23 (12 new), passed: 263 Build time: 01:41:40
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Второе быстрее, но надо смотреть, иногда нужно искать элементы в `head`-элементе. А вообще, нужно сделать так:
```js
function $q(path, root = doc) { // или doc.body
```


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 48](https://teamcity.labkey.org/viewLog.html?buildId=287433&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:25


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 48](https://teamcity.labkey.org/viewLog.html?buildId=287433&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:26


Do you need to sort it here?


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


`npm` takes care of this for you - the core libraries all have `package.json` and `.travis.yml` files that look similar to this, it's a tried and tested approach :)


please remove commented code


In my tests `face` didn't work properly. Perhaps I missed something...


Should we include a `FIXME` line about this?


Yeah, you're totally right.


I was struggling with Babel 6 today. This seems like it'll fix the issue for me ;)


Паранойя авторов движков иногда просто убивает, когда в нетмониторе смотришь на такие запросы:
```
-----------------------------299523255117918
Content-Disposition: form-data; name="qzw0♝ym2⛷p♧"

☭'H⛱0C[GT☑`⚨⚧x+Mmf6$#<u3W P☫}{☥?5k-jOb.
-----------------------------299523255117918
Content-Disposition: form-data; name="text"

⚥IYs| B7 Zn4(-FmQ5t⛭c&i)lA♮e⛒{u⛠g/WJx6,♙^o<Uq
```
И всё это ради защиты от ботов, которую обойти можно одним `get`-запросом страницы и выкоцыванием этих скрытых полей. Раньше там вообще JS-ом новые поля генерировались перед каждым ответом, но сейчас вроде успокоились. Зато из-за них так проcто не сменить номер треда куда отвечаешь, потому и отключён быстрый ответ на доске.


> However, the version check (<=1900) might be helpful since clang-cl can also report lower number when giving a lower and non-default -fms-compatibility (or named similar) parameter, which then also reduces to before C++14.

That's right. So basically, I should check what is the lowest version of that parameter that Hana can support, and then keep such a check saying something like:

> We detected that you were using Clang for Windows with a -fms-compatibility parameter lower than XXX. Only versions starting at XXX are supported by Hana, because versions before XXX do not provide sufficient C++14 support.


По-моему, мы в head'е только скрипты и cssы трогаем. А, и фавикон, таки.


with all due respect, this shots don't look like SVMI


no, no need to sort anymore.


Every attribute is required unless you mark it as @Verify.Optional (you can check the subject for example)


What if my node doesn't have the id '54uvd'? :smile:


yes, will do it.


@73rhodes That does not sound as jslint broke your CI process at all.


Yo, make sure to line up you braces before commiting and pushing :+1:


Алсо, вместо `doc`, можно сделать глобальную переменную `body`, равную `document.body`.


Has that aspect ration changed again in 518!!??! How the f!


Yeah, I'm opening a bug for django-jinja first.


How were you testing? If you just evaluate `(message "Something")` with C-x C-e it always shows up unpropertized, because you're seeing the return value instead of the message.


Please either remove debug print or make it more user friendly


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


more info here: http://stackoverflow.com/questions/32505601/when-do-template-currentdata-and-template-data-differ-in-value


Either there or in another ExternalProject that downloads the right architecture of dlls and then copies them over.


Is the 'order by' needed?


I think remove the line may be better.


I'm not entirely sure why this changes was made, would you be able to explain this please?


no, it is not needed any more.


Well, you do see the message in the minibuffer... and it seems propertized to me.


Please remove this entire function.


a pom that has the main parent pom as pom and then contains what is relevant to share between jbdevstudio and discovery.

Making it so the main parent pom don't get polluted/affected by these devstudio specific things.

Nothing about duplication.


OK


Please remove above block.


change the comment to include views


Are you sure that increasing the `nonce` by 1 is cryptographically secure?

If this implementation is a Proof of Concept, then it's fine. But this should not be used in a real world application.


Please add more comments that all views and tables are included.


Se deberia llamar userSchema.


Actually I think it is not needed anymore as we now assign Visits or Actions category. Will remove it


was reported the datetime picker is defaulting to year '-1'. Haven't yet investigated.


Since it's no longer dxl, pls rename variable.


Hm, not sure


The dump of the table would be good, then I would be able to restore and test it


https://dev.windows.com/en-us/downloads/windows-10-sdk
please try to install windows SDK 10.


Главное, позаменять в скрипте локальные переменные `body`?там я де-то лепил в постройке окон.


Ok. Probably at some point we should model this as a rule whose subject predicate is 'this exact node' (rather than 'the element that happens to be at this path' and so on), and solve this in a general way in the rule binding stuff.


Hi what is correct kernel commit to do to fix thoses errors related to that commit?
http://pastebin.com/z5ShaJL1


What was the reasoning to fire both events?


Agreed. We could either add a parameter to `CreateDefaultStorageNode()` or introduce a function named `CreateStorageNode()` that would take a parameter.


yes, it will


Try it :laughing:


Does LEGO_NAME_SIZE still make sense - it's now the device address size, right?


Можно исправить на `docBody`. Всё равно он редко где использоваться будет с дефолтными переменными в `$Q` и `$q`.


Please stop early here, we only need to parse the origin query. Please make a comment as well.


I've pulled that file so many times and not eevn touched any layout stuff since I first changed it! :(

When I get home, I'll download the raw off here. Delete my repo and rebase. Then delete 518 completely and then add the raw again.

Sorry mate I have no idea wtf is happening [obv!].


will do.


Then you didn't comprehend what I wrote, because it does.


remove old code


It's the size for every name in every lego-related driver, not just addresses. very general purpose


It is forward declaration for `snowcrash::ParseResult` and `snowcrash::SourceAnnotation` it allow remove `#include"SectionProcessor.h" ` from header file.
As result we have (little bit) compilation.


This property is used a bit in the settings, and from this commit, the titles in the Settings sections are a bit messed up (pushed at the top, with a left alignment, which is a bit ugly). Could you fix this please?


Yay, less PNG files \o/


It would be more clear to have a symbolic constant for this `4`, especially as it's repeated below.


This is not used anymore, right?


No probs bud, something strange is happening with 518 :)


Change the print out to include views.


Now I look, I can probably remove `SourceAnnotation` from forward declaratio, because it is not used in headers currently.


I'm guessing changes to Storyboard and XIB for iOS/Xcode updates. Looks good otherwise


Thanks. this is a very useful addition. Saves the hassle of putting it in then commenting it out.


Still missing a decent commit message and signed-off-by line.


Yeah, we're going to be happy that an update to `spec.json` and maybe some templates or device class code is all it's going to take to update things.
We'll need a procedure or some kind of understanding for keeping things updated in the face of breaking changes before we flip the switch and actually do a full release.


Пипец, да. Я от этой паранойи плевался даже егда этот движок только появился.
Завтра прочешу аще раз все тиниборды.
Заодно проверю и лизурьтат сегодняшних замен. Вдруг что-то из подводных камней вылезет.
%%А хотел уже скрипт обновить, ліл.%%


Missing signed off by and you should probably in the commit message point out which sha1 broke things.


Great work! Next we need to talk about testing, as well as how to install the UDFs easily at customer site.


The version and timestamp are added automatically by Dist::Zilla and should not be added manually.


Room 518


This code and comment are really confusing now. I have no idea what was going on here even before this change, but keeping the old commented out code doesn't help.

Presumably this case (`diff < 0`) can't happen at all any more after this commit, so shouldn't we just remove all this `if` entirely?


There is one, otherwise this commit would have been larger..


Will commit in few minuts


Yes!  That was there before but I think it disappeared after some copy/pasting


Right, since we already dump table, view together.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 49](https://teamcity.labkey.org/viewLog.html?buildId=287431&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:19


Needs a commit description and signed-off-by line


ok


It would be nice to avoid duplicating this code. As it is, we have the almost but not quite the same version of it in `DoGetBestSize()` which is not ideal.


Thanks for clarifying :+1:.


you can make it stand on one line + plus please use triple equality:

`this.state.accounts.filter(account => this.state.active === account.id)`


Transaction sería un modelo independiente.


How many more "long is not 64 bits on 64 bit windows" will there be lurking?  Maybe windows needs to be compiled with the full --b4n-pedantic -o2 option set.


This solution does not work when I run "learn"


As I said, you need to handle errors.
> If this function succeeds, it returns S_OK. Otherwise, it returns an HRESULT error code.

https://msdn.microsoft.com/en-us/library/windows/desktop/bb762160%28v=vs.85%29.aspx
https://msdn.microsoft.com/en-us/library/windows/desktop/aa378137%28v=vs.85%29.aspx


you forgot `<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>`


Same as below.


Sanjeev/Lei,

The added variable:

let current = NetworkConnections.Current

hasn't been used in the rest of the function. What is the reason to add this?

Jin


Can you try this out for me please?

```js
Template.afSelect_semanticUI.onRendered(function() {
  let node = this.$(this.firstNode);

  node.dropdown(_.extend({
    fullTextSearch        : this.data.atts.fullTextSearch || false,
    allowAdditions        : this.data.atts.allowAdditions || false,
    maxSelections         : this.data.atts.maxSelections || false,
    allowCategorySelection: this.data.atts.allowCategorySelection || false,
    useLabels             : this.data.atts.useLabels === false ? false : true
  }, this.data.atts.settings));

  this.autorun((c) => {
    let data = Template.currentData();

    if (data.value) {
      node.dropdown("set selected", data.value);
      c.stop();
    }
  });
});
```

Ugly, but until we figure out the real problem, it seems to be working.


pre-existing, but it should use `===` here too


@73rhodes Why won't you switch to another tool then? `jshint`, `jscs`, `eslint`... they're all configurable :)


Oh wait, this is how it is in the current branch; this is a commit from before I added that.  Please checkout the changes in the PR!


GT wants this feature to be able to accept clicks from outside Angular.


Entrega del tp


It seems like we still hard-code to support quorum out of 3.  However, there is a line that try to support a general case:  https://github.com/Netflix/dynomite/commit/2861e7b3d37f4dfd57692b89d3b913482c1650be#diff-ade66b675b3ca1e9261b1bbcb40b5903R1162

LGTM for replication factor 3.


At the current time, does this script work? And do you need anyone to try out new features on a separate computer? so you can find the problems and fix them.


Ugh, for the sake of consistency, the destruction order here should have been inverted.


@spalger - here is NODE_OPTIONS for `kibana.bat`

I don't have access to a windows machine; so I haven't been able to test.


P {margin-top:0;margin-bottom:0;}

I'm actually ok with foisting UTF-8 on everyone.  I was surprised by how well it worked!

From: Chung-chieh Shan [notifications@github.com]
Sent: November 24, 2015 16:02
To: hakaru-dev/hakaru
Cc: Carette, Jacques
Subject: Re: [hakaru] Put more in the archive, including ModuleLoad (ef8209d)

Why use the old gensym? Because I'm not sure about imposing UTF-8 on everybody.
Why use the new gensym? Because its renamings are easier to mechanically undo. Besides, the old gensym has the following (fixable) problem: it might rename x to x1, then x1 to x11, then x to x11 also.
�
Reply to this email directly or
view it on GitHub.


Fixed.


Actually resolves https://github.com/agrover/targetcli-fb/issues/57


@kmike Ah ok, yes of course using `0 for i in ...` lowers down memory usage so this patch was a good idea :) I think `_ for i in ...` could be used instead to even avoid the reference to the `0` object at all since `_` is, IIRC, a special symbol to tell Python to just junk this variable, so I guess it doesn't even get processed at all.
About the list comprehension overhead, you're right, maybe we could use a generator comprehension to keep the succinct code while lowering the overhead at the same time?


Please prefer `banks-accounts-list` here and in its uses :)


@73rhodes It sounds like you have updated a tool you use, with that you have choosen a new coding standard, but you did not update your codebase to follow that. It sounds like you broke your code, and in it current state it cannot pass the code validator version you have choosen.


'von' oder 'vom'? ;)


Typo in comment ?