TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4212](https://teamcity.mesosphere.io/viewLog.html?buildId=71130&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:01
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


they've already been discussed in IRC, in email and in the github tracker. I'm not going to add them to the wiki-page. The issue in on the agenda for the next WG call.


I see, the server online is hosting it under the /test/wireframer path so the redirection will need to be modified accordingly, or the server should be modified to accommodate this.


Are you meaning that it should only be :

    #define KEEPASSX_EXPORT Q_DECL_EXPORT


nit: Why assign this rather than just check this in the `if` conditional?


CBECC (macumber) - i386-Windows-7-VisualStudio-12: OK (2324 of 2349 tests passed)

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


I think I did in a later commit


then


When adding to existing views, use the global `view` property in the template rather than adding more properties to getRenderData, or we'll just end up refactoring those properties later.

For properties which need to be initialized or have default values, they should be initialized in the constructor or initialize functions, or defaulted (`view.property || defaultValue`) within the template instead.

https://github.com/codecombat/codecombat/issues/3138


Oh, whoops! Much less concerned, but we should probably still think about consolidating the isolation machinery as much as possible.

Or just kill /bin/sh and /usr/bin/env in Linux :smile:


Something like `InboxCtrl.$scope.handleAndroidBack =`

```js
function() {
  if($('.enketo .btn-prev:is("visible")') {
    window.history.back();
  } else if($scope.showContent) {
    $scope.setShowContent(falso);
  } else if(tab !== messages) {
    go to messages tab
  } else {
    kill the activity
  }
}
```


In original H3 Pathfinder assumed that the bonus will last forever. At worst you had to recalculate bonuses every turn and not allow hero to stop in innaccessible location.


Should this value be contingent on whether the feature flag is enabled?


It can go in either. I have a slight preference for doing it in the `format()` call because I like that `built_query_format` is the format of a generic BQ query, but it's mostly a matter of taste.


Yeah it only runs update if the build.fsx doesn't exist. I think this behaviour is reasonable, but I suppose it could go either way.


It is, and that's a good idea.  I don't know that I want to tackle that right now though.


I presume you meant to write `I'm just not enforcing`.


I personally dont think cropping is very critical to the site. If its easy to add we can add it later. For now, I'd say just focus on building the upload subsystem.

Also note that file extensions can be managed using html file input types, so there really isn't a need to code that in php. We'll make it impossible to upload anything other that .png or .jpg/.jpeg right on the client side. No server side regex check needed.


the same. Equal logic for 3 similar blocks. Change it. Create dynamic method.

Other idea for classes. You can create base classes for `item` and `course` and inherits it for first_item, second_item for firs case and `courses` for other case


or only some of them


Again... why remove this?


Why remove this?


Many of the `RelayStore` methods really ought to take a `GraphQLFragmentPointer` as their input - we'll probably want to make a public method for creating these pointers


The SnapshotJobManager is kind of a misnomer.  It really it is responsible for hiding the details of spring batch from the rest of the application.  Really it should be called BatchJobManager.  I handles batch job processing for snapshots and restores alike.  All the operations that are occurring in this method (save the last one involving the separate thread) involve aspects of the system that only batch related tasks know about.  I was convenient to throw in the space deletion call here but not required.


We need to ensure that our white space and brace formatting is set up properly and checked before committing. Also, we shouldn't use die() anywhere in the php code. All errors should redirect appropriately to an error page. Just another example of horrible coding practices taught at OC...


IIRC there was a reason why I chose Type=oneshot instead of Type=forking. Can't remember though... and will merge this when I found that out :)


You should re-add part of https://github.com/gnome-mpv/gnome-mpv/commit/89d321c9fe20dd0dd6733f058b19ab3c3d1fe4fa if opengl support is disabled.


TheBestProjectEver


Thx😂


It would seem it would be better to bump the file limit that kqueue supports? http://superuser.com/a/443168

Because if you put this as 'false' it's the same as a fully managed watcher:

 1. Not null: https://github.com/mono/mono/blob/master/mcs/class/System/System.IO/FileSystemWatcher.cs#L117
 2. Mode never written, was zero: https://github.com/mono/mono/blob/master/mcs/class/System/System.IO/FileSystemWatcher.cs#L140
 3. "false" <> "disabled", takes else branch: https://github.com/mono/mono/blob/master/mcs/class/System/System.IO/FileSystemWatcher.cs#L141
 4. Which uses the managed watcher: https://github.com/mono/mono/blob/master/mcs/class/System/System.IO/DefaultWatcher.cs – actually the opposite of the semantic meaning of this flag.


Removed debug output - thanks for catching.


TeamCity LabkeyModules15.3 :: Git postgres 9.4 [Build 5](https://teamcity.labkey.org/viewLog.html?buildId=283834&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 19, passed: 231, muted: 34 Build time: 00:44:02
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Nie mam pewności czy dla programu MOST też tak będzie, napisałam maila w tej sprawie.


There is a typo: vers**oi**n instead of vers**io**n


add this to I18n translation file. Try to can`t use raw text


Да


Is this a test server? do we have security credentials to access the production server. I just worry that running test we accidentally hit the production server and do an `rm -Rf /` by accident


Please look at https://github.com/jmettraux/rufus-scheduler/blob/8c39fc3d5503a08da5f524cec080c4dc4cf38b64/spec/cronline_spec.rb#L1036-L1066 for the latest version of this spec.


the same with translation file


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 779](https://teamcity.labkey.org/viewLog.html?buildId=283840&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 49, passed: 235 Build time: 00:43:10
### 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
```


Perhaps this should be named to `retraction_justification`?  When I first looked at this in BAPI, I thought that this was supposed to be the registration summary, and didn't understand why it was null.


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


here also add raw text as translation that located on yml file


Additional-report-variables-ZoneHVACIdealLoadsAirSystem (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1915 of 1963 tests passed)

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


```
main_window.c: In function ‘vid_area_new’:
main_window.c:184:9: error: ‘use_opengl’ undeclared (first use in this function)
  return use_opengl?gtk_gl_area_new():gtk_drawing_area_new();
```


This is being serialized as a JSON string rather than a JSON object:

![screen shot 2015-11-09 at 3 47 10 pm](https://cloud.githubusercontent.com/assets/86835/11046070/2e3eaa4a-86f9-11e5-9a03-54640f0dfe47.png)


@onox, if I wanted to slow down this fcs_function execution, can I? Would you know how to do that, say have it execute at a 10th of a second?

>settimer( func {splash_vec_loop() },1.0);

IS the above 1 tenth of a second?


This [doesn't actually render with newlines][1].  The rest of this README uses a pattern like “as returned by AzureAPI.trip.get(…)`”, so probably just do that here.

[1]: https://github.com/azurestandard/azure-angular-providers/blob/649a6449f54dcae657cd5e755a11ef64a4f895e1/README.md#azurecoordinatortrip


'import' is only available in ES6 (use esnext option).


@dwj192 looking at it now, it seems to me that there's a problem, because this doesn't distinguish between a user modify with user specifying empty email vs. user modify where the user didn't pass the email param at all (in which case we should keep the existing email). Need to identify these cases (have email_specified() check)


:+1: for the the new Java date time API


No nie, pól strony tekstu i nadal czytelnik nie wie co to loaclInst i nawet nie dostał żadnego przykładu.
Podtrzymuje swoje zastrzeżenia do pomysłu na prefiks x-


OK


Hmm - I'm torn, to be honest. I can see good arguments for both options on sibling pairings. I think I prefer keeping startFrame/startMode as a pair, and same with the end plugs too.

I'm guessing something like this:

    r['start']['frame'].setValue( 4 )
    r['start']['mode'].setValue( ImageReader.Hold )

That works for me, I think.

The only issue I can see with this is that the start/end frame of the script isn't controlled in the same way as the start/end frame of the ImageReader node.

As an alternate (controversial?) suggestion, it might be good to have a FrameRangePlug which could be used in any situation like this, and could have a flag for whether it allows int values. It could be used here, in the ScriptNode, any anywhere a frame range is required.


Just Buchanan seems to be a name on email when someone comments and I get notified - Actually, thanks will go to Kentzo...


it is not so good approach to use this logic and logic on js files


hmm I don't know if it's allowed or not.  But good catch, I just copied all the fields over.


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.7](https://pixelated.design/build/viewLog.html?buildId=211&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:09


I'd try to avoid l ("ell") as a variable name -- it's pretty easy to confuse with 1, depending on typeset.

in general, `np.zeros` takes a (tuple) shape form in addition to the int form you have, but this is fine for now.

    zeros(...)
        zeros(shape, dtype=float, order='C')


probably better to use that logic on js file


Many thanks... This solved my problem :-)


@radkill, потому, что временный костыль. Нужно конвертер валюты в рубли динамический замутить — видимо сегодня же вечером.


I think you only need `impersonator?` here.


We want to acknowledge the contribution and role of everyone. Currently, besides my name, that of Greg, and yours, I have added those of the annotators, while mention of the digital edition of the text (in the element encodingDesc) does justice to the people who prepared it. I am available to add anyone else whose work I am not aware of.

In general, I think that the version I propose is, although more verbose, less misunderstanding prone. As far as my role is concerned, for example, I myself treat computationally the data to clean, normalize, and harmonize them, in addition to being responsible for the annotation scheme and direction of some annotators. I do not think that the label "managing editor" is clear enough in this respect.


If `LanguageCode.locale` passed is empty, it will only match a selected `LanguageCode.locale` that is also empty. Is this what is desired? Or does the caller mean to say they don't care which locale, as long as it's the correct language code?


it complicates the logic. Maybe, syntax/platform specific settings for `prog` are a better way to go? Perhaps, something like,

```json
{
    "prog_user" : [
        {
            "plat" : ["osx", "linux"],
            "scopes" : ["source.r"],
            "prog" : "tmux"
        },
        {
            "plat" : ["osx"],
            "scopes" : ["source.r", "source.python"],
            "prog" : "Jupyter"
        }
    ]

}
```
And add one more choice in the Choose Program list, -- `[User Settings]` to use the user customized settings.


Took me a few minutes to get reacquainted with this codebase.

My original idea (I believe) was to keep the "understanding" of the HTTP response code within the creator function, since future `method(params)` calls may not return HTTP response codes. If you prefer it handled in `main()` that's reasonable.


I think `CheckRetraction` should be renamed to indicate what it does.  Perhaps `HideIfRetraction`?


przykłady, przykłady!
Są więcej warte niż 1000 słów


Thanks. But actually, that's not true. A caret range "Allows changes that do not modify the left-most non-zero digit in the [major, minor, patch] tuple. In other words, this allows patch and minor updates for versions 1.0.0 and above, patch updates for versions 0.X >=0.1.0, and no updates for versions 0.0.X."

See: https://www.npmjs.com/package/semver#caret-ranges-1-2-3-0-2-5-0-0-4


you could get rid of the # there, and fill this in to a full docstring, but you could do that as a separate commit if you like


Let's do it. It's ugly but it's small, and it fixes the issue. Bonus points if ugliness motivates an elegant fix later on.


I'm running into some difficulties with this effect. I just went back and reread what little documentation there is on it and it is apparent that there are going to be limits on what can be done with it. I'm trying really understand how it works. It isn't obvious.


We lost the last `*_id` references with f4f6c35 in #9, so probably use `dropId` here.

It looks like we're just [sticking the trip object on the resulting `AzureCoordinatorTrip`][1], so we should probably say that in the `trip` entry in the property list below to let folks know that nothing magic is happening between the `trip` argument and the `trip` property.

[1]: https://github.com/azurestandard/azure-angular-providers/blob/649a6449f54dcae657cd5e755a11ef64a4f895e1/azure-providers.js#L970


I don't think we want information about the plugins to be in the User Agent string.  At some point telemetry for these may be useful, but it should be completely opt-in and accompanied by a lot more info (what the user's configuration looks like, what command lines they've tried, etc)


OK


Your SetConfiguration() looks pretty much what I wrote independently on my own. The Android device I have is in Chinese unfortunately -.- So I am now sidetracked by writing a custom ODIN driver for OS X now... This will at least allow me to test if it is something specific to the Android Composite device or OS X in general :)


Though Ada is right that it is out of scope, we could easily make this field required:
<property name="required" value="true"/>


again render 3 similar blocks by repeated code. You can do that dynamically


Да, IDE мне сразу предлагала так сделать, но я не посушал. : )


How à propos, given we the context of languages. :+1:


@nvkelso Github won't show the diff here, but some `aerialway` layers [here](https://github.com/tangrams/eraser-map/blob/gh-pages/eraser-map.yaml#L1982) that were added in this commit form invalid syntax. The `lines` rules under `gondola_cable_car` and `chair_lift` should be inside a `draw`, and it seems like they should probably have a filter as well, but I didn't want to guess at intent.


better name for this guy?


да, тут протупил. Поправил.


A doksitok szerint a programban van egy metódus, ami visszaadja a legközelebbi objektum távolságát, de a programban ez nincs megvalósítva. Légyszi ezt javítsátok, mert az automata fékezőrendszerhez szükségünk lenne rá.


Yesss, thanks, it was solved in trunk two days ago.


should czy may?


“trip” → “stop”?  Because there could be other orders for other drops on the same trip.


This isn't really a "story": what happens when changes are detected during a build?
What would the user experience?

Then title should focus on the behaviour the user can observe


@yoshiokatsuneo Yes, I have been trying to support non English characters. I had a chance to review your work and I like what you have done. May I merge your changes into my fork?

I use this fork for a personal project of mine. The people who use my project would greatly appreciate the work that you have done.


I think that cancelling should be a separate story. First cut would be ok to just start another build as soon as the currently executing one is complete.


Crud. Those were meant for a comment I have in my local changes and haven't pushed yet.
I will have to hurry up with my removal of older uclibc versions and addition of uclibc-ng.
Hang tight.


When I package and then run the war with jetty I get the Solr Admin up fine I just get errors that say either "Couldn't retrieve schema document - status code 500" if my SOLR_SERVER = "http://localhost:8080/" or "Couldn't retrieve schema document - status code 404" if my SOLR_SERVER = "http://localhost:8080/mpclorchestration-solr/"

When I deploy with mvn tomcat7:deploy or mvn tomcat7:redeploy I get Cannot invoke Tomcat manager: Broken pipe ->    (I've added the plugin and configuration on my local version though so I'm not sure what's wrong)


OK


Probably want to `#error` if opengl is disabled and x11 is disabled.


Additional-report-variables-ZoneHVACIdealLoadsAirSystem (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (889 of 889 tests passed)

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


but where did you add the annotators' names? I don't see them.


What about changes that are outputs of another task? So if `CompileJava` creates a class file that is input to a `Jar` task, we don't want to cancel and restart the build!


Cuidado, não está na convenção .NET (que é PascalCase)


Why are you refetching the Nodes here?  Couldn't this just be:

````
nodes = Node.find(query)
return [node for node in nodes if not node.is_retracted]
````

If there's a good reason to refetch, it would probably merit a short comment.


We probably want to limit this to the last 10 trips?  Or at least provide users with a way to limit it.  There's no need to have the API find and serialize more trips than the user will need.


Здесь и ниже неправильная логика работы. Температуру по Кельвину содержит не `temperature`, а `result`.


What's the meaning behind the `_PlayerWeapons` class? As far as I can see, the `_PlayerWeapons` class is not used for anything else than the `Player` class, why not have the methods in `Player` class directly?


This looks fine. Thanks!


That makes sense.  In that case, and even though it requires more code, I think best would be for `logged_curl` to return the status code and for `method(params)` that make API requests are responsible for turning the status code into a success code.  Should be really quick to do that.  Do you mind?  After that I'd be delighted to merge this.


Lepiej nie pisać 'ISCED subject area code', tylko 'ISCED', bo tak to jest w literaturze określane.
zasadniczo, to ISCED wyznacza (chyba nawet jednoznacznie) subjectArea, trzeba jednak być przygotowanym na to, że na niektórych uczelniach nie mają jeszcze kodów ISCED albo że nie zamapowali wstecznie w przedmiotach subjectArea na ISCED.
ISCED jak najbardziej podaje się także dla Degree


True, damn. Thanks. If by tomorrow no other complaints arise, I'll have this merged.


(in terms of roles, this is maybe an interesting use case for https://github.com/dhtaxonomy/TaDiRAH ?)


Please undo the s/resource/node/ in "Each resource contains".  "Resource" refers to the generic JSON representation of the node and is the appropriate term here.


I just like being in the habit of assigning numbers, does it need it no.

why assign numbers it is more I am using an emu instead of a define, so I am clear on the values especially if they type ever gets changed off the enum or the value is printed, I can look easily to see what the number is instead of having to count to see what the value should be.

Also if I ever reorder/add/remove I can control what the numbers should be so if I do not want them changing then they will not


Obawiam się, że na niektorych uczelniach jednak będzie trzeba dopuścić inną strukturę, np degree w ramach degree, no ale zobaczymy.
A gdzie jest kod kursu?


Since the logic for “the current trip associated with this drop” took some work to figure out (§9.7.1 in the Google Doc specs), it's probably good to explain it here, so everyone's on the same page about when this cuts over to the next trip.


This works, and I'm fine with pulling it as-is, but its pretty fragile. All templates share subscriptions, so a future change in the layout template or a subtemplates (or if we ever use this template *as* a subtemplate), it'll break in a not-immediately-obvious way.

Seeing as its the category page, however, that all seems pretty unlikely. If you want to merge as-is, you have my ACK.


OK


Good idea! Nice way to get everyone on the same system.


"Get Dubs" button


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4217](https://teamcity.mesosphere.io/viewLog.html?buildId=71154&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:12:59
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Looks good, but needs something about tab character as a marker for right alignment.


The inputs are registered for watching just before the task is executed in the build. We don't care if the change happens before the input files gets used.


note to self: this is only for 1-d arrays -- you correct this to general shapes in a later commit


a może 'import data'?


We need more test coverage: task dependencies, one task's outputs form part of another task's inputs, change to task input _while_ the task is executing (what should we do?), task execution fails, ...

Think deeply about all of the different ways that this can fuck up. I'm sure there are many!


Part 1 for #212 on Master


I don't think we should ever cancel the build in progress.


is there a reason to create a matrix4x4 class rather than using glm's matrix classes?


adding graphs.sh


don't need the comment anymore


Part 2 for #212 on Master


JSBsim system run at FDM speed (120 Hz). But the frequency of the execution is not that important here. What the shader does is use the vector to know the angle + it computes length(vector) to know how fast the rain should move.


Part 1 for #212 on v25


"EMREX project authors could not find" - to brzmi zbyt 'personalnie', lepiej pisać bezosobowo, a najlepiej w ogóle tego tak nie tłumaczyć, po prostu nie ma i już


we'll support scroll x position soon also, so better to just send the `pos` object through here and instead access it's `y` property here https://github.com/kdzwinel/browser-sync-client/blob/master/lib/ghostmode.scroll.js#L71


NCP schema? Chyba ELMO schema?


So I don't think the logic can or should depend on how many times per second you execute the function.


Remove this class


it doesn't matter for your tests here (currently) but in general when you use `random` in tests, make sure you explicitly seed it before each test case (for example in a `setup` method), so that you run the same exact tests each time, and run the same test if you're using a filter


Good.


Since you have a class `LanguageCode`, I think it 'right' to pass it rather than a `String`. However, I don't see what it gains you as that classes member attributes are only that `String` anyways, besides access to the various `LanguageCode` methods, none of which are being called herein (that I can see).


+1 for not cancelling. It's far too risky with arbitrary code that isn't designed for it.


Just that while refactoring it was easier for me to have everything in one
place. Almost all the functions in the controllers are helpers, so they
could stay or go somewhere else.
On Mon, Nov 9, 2015 at 10:54 Armando Perez <notifications@github.com> wrote:

> @mLuby <https://github.com/mLuby> any reason for not putting
> completePendingTransaction into a service?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/projectLiftOff/surpriseHer.server/commit/d62e1f1bc29b799bde9191fc03b8e614059bb0f9#commitcomment-14284233>
> .
>


Added operator=(wchar_t) to convert from wide char to u8char.


If that's the case, you might want to move the parameters to the export function instead since that's the one that generates the secret number. They aren't being used by the returned function.


@nsh87 I doubt that this is the most elegant solution, but I was struggling with how to check that dictionaries are being built. Please feel free to give any constructive criticism. We talked about having a __dev__ branch. Maybe we could merge this one into it and then I could branch off of there for work on _measures.py_.


you can get rid of the comment


ok


Interesting, I didn't know about the <1.0.0 minor-as-breaking-change behavior. Thanks, and sorry for the false alarm :grin:


Yep!


@nllong why are there multiple zone air temperature hourly reports?  The intent in the C++ was to add output requests at different reporting frequencies


Czy to oznacza, że ELMO nie posłuży do przesłania informacji o dyplomie i ocenie na dyplomie?


I do strongly believe that ErrorMode and FrameMaskMode should be considered different things. If you have set a FrameRange of 1..100, then you may well want it to either be Hold or Black outside this range, but do something completely different within the range. However, this being done in the ImageReader node rather than the OIIOImageReader node makes sense to me.

I'd be interested to hear what Lucien thinks, but I would consider the "Hold" or "Nearest" error mode to be the least important, and the most complex to implement. For me, the error mode is more important to be Black, Error or something obvious like Checker. You want to either not notice the errored frame go by, or have it really obvious in the render.

With regards to the "No Mask" frame range mode, Nuke doesn't have this on the Read node, but it does on the Retime node, called Continue.


At least ECTS credits SHOULD be included.


I moved them in side the function and resubmited.
On Nov 9, 2015 11:03 AM, "Tony Gaskell" <notifications@github.com> wrote:

> If that's the case, you might want to move the parameters to the export
> function instead since that's the one that generates the secret number.
> They aren't being used by the returned function.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thingsastray/Secret-Number-Generator/commit/455d9bf82154cc1f68326f99b1667ca8b593c0e7#commitcomment-14287036>
> .
>


Although we've squashed things into Angular $resources in the past, but that makes `.save()` harder and sometimes leads to weird names (e.g. see 7f5299c unrolling this and creating a new `OrderLine` wrapper).  Instead of adding properties to a resource, I'd rather add a new `AzureDrop` factory, with a `Drop` object that had `Drop.drop` as the raw $resource and `Drop.contact` as the person $resource for `Drop.drop.coordinators[0]`.


Cool. So do you want to just do the SQL date format (yyyy-mm-dd) or include
time in there too?

On Mon, Nov 9, 2015 at 10:10 Armando Perez <notifications@github.com> wrote:

> @mLuby <https://github.com/mLuby> hmm well using epoch would make it
> simple to manipulate the date on any UI to follow any format. However, that
> can easily be done also using a string (just adds one more step). I really
> like the idea of skimming the DB and knowing what the dob is since we wont
> have an admin tool... let keep the string as you have it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/projectLiftOff/surpriseHer.server/commit/2c62814090f87b8b2bf7ffe8a1e4cbd7bc5abae7#commitcomment-14283339>
> .
>


Returning the newList directly would be faster (avoiding an extra set of copies) and would be OK as Jackson would have done something similar.


Oh, good catch. This is a smart change.


This will return `TRUE` when `OPENGL_CB_ENABLED` is not defined.


Similar comment, it's ok to return the newMap directly.


Are you talking to me ?

I don't understand

Raymond


I was missing this when I tried this first time, you could notice users that build was tested with Cygwin and I'd like to know what should i do to build "release" version of fftw3 (to gain max performance) for my app?


Since you're using both keys and values, would be better to iterate over the entrySet.


Standardowe zastrzeżenie do prefiksu x-.


I guess it's possible for jobs to go back into that particular failed queue before we get through all the batches. I'm not sure in that case how the `unfail` works, if those jobs would go to the back of failed queue so that the jobs we originally wanted to unfail will get unfailed, of the just-failed jobs will be mixed in and thus get tried a second time.

In practice this doesn't seem to be that big of a deal. In the current code it's true that this could retry more jobs than the original number of jobs in the failed queue (because I do `(kount.to_f / batch_size).ceil` number of batches of `batch_size`)

Additionally this function can not go into a infinite loop because it determines at the beginning how many jobs to retry.

I guess I'm saying that it's not perfect, but for most purposes it's good enough. Do you have ideas how to make it better?

I guess I spelled it `kount` just to to avoid any possible naming conflicts, but that is not necessary here so I can change it.


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


I wouldn't get into modifying FORA semantics unless we really have to...

On Mon, Nov 9, 2015 at 3:12 PM, Thomas Peters <notifications@github.com>
wrote:

> this isn't quite the same as python, for example 2 ** -1 == 0.5 (though
> you do still have 2 ** -1.0 == 0.5)
>
> @braxtonmckee <https://github.com/braxtonmckee> maybe this is the right
> behavior we want anyway, from a (fora) compiler point of view?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14285906>
> .
>

--
-------------------------------------------------------------------------------------------
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee       www.ufora.com


Btw, any particular reason why you are using `<interpolate1d>` instead of `<table>`?


For sure. Sorry. I'm submitting a new pull request.


Correct, that's a test server.


I was thinking as long as we validate the whole transaction (gift, date,
address) before we charge and complete it, then we should be okay. Doing
address later in the process allows us to handle creating pending _or_
complete transactions with the same workflow. Let's chat tonight about it.

On Mon, Nov 9, 2015 at 10:45 Armando Perez <notifications@github.com> wrote:

> @mLuby <https://github.com/mLuby> shouldn't we validate the address
> before creating the transaction?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/projectLiftOff/surpriseHer.server/commit/2c62814090f87b8b2bf7ffe8a1e4cbd7bc5abae7#commitcomment-14284037>
> .
>


so close this issue then? (without merging)

On Mon, Nov 9, 2015 at 4:08 PM, Braxton McKee <notifications@github.com>
wrote:

> I wouldn't get into modifying FORA semantics unless we really have to...
>
> On Mon, Nov 9, 2015 at 3:12 PM, Thomas Peters <notifications@github.com>
> wrote:
>
> > this isn't quite the same as python, for example 2 ** -1 == 0.5 (though
> > you do still have 2 ** -1.0 == 0.5)
> >
> > @braxtonmckee <https://github.com/braxtonmckee> maybe this is the right
> > behavior we want anyway, from a (fora) compiler point of view?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14285906
> >
> > .
> >
>
>
>
> --
>
> -------------------------------------------------------------------------------------------
> Braxton McKee CEO / Ufora
> www.twitter.com/braxtonmckee www.ufora.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287121>
> .
>

--
*Thomas Peters* | Software Engineer | *U**FORA *
115 West 27th Street 7th Floor, New York, NY 10001
Office: 646-918-6435 | Mobile: 732-610-8565
tpeters@ufora.com | www.ufora.com


Renaming to "Rebuild when changes are detected during a build". However I'm not satified with that title either.


Removing samples licenses seems to be a good idea. It's hard to demonstrate on notebook screen a sample of five lines with 30-40 commented lines above it.


This is throwing an assertion error on OS X. I pushed a cleaner fix to my repo. sdh11/gnuradio-wg-grc@4dd6a9d. You should be able to just cherry-pick that.


Why is this class part of extensions API?


Why are you adding images here? Users will be able to visualize it somehow?


Did you mean to print this exception rather than re-throwing it?


L0FKA <notifications@github.com> writes:

> [..] I'd like to know what
> should i do to build "release" version of fftw3 (to gain max
> performance) for my app?

We use the mkdist.sh script


I mentioned this elsewhere, l as a variable name is not great


P.S. Guess this needs an  extra test.


I can't just return a list comprehension here because after this it does filtering on ordering = ('-date_modified').  It needs to return a queryset for that to work.


@adammurdoch @alkemist I'm curious to know what difference does it make if the user cancels the build? From the user's perspective it makes sense to simply cancel the currently running build and start a new build if a change is detected in the inputs of the already executed executed tasks.


LGTM


Copyedit: instead of "For a registration, however, navigate to the individual registration's detail view for registration-specific information.",  how about "If a node is a registration, it will have a relationship called `registration` where additional information about the registration may be found.".

Also, please add a quick blurb about registrations to the Relationship section of NodeDetail.


s/through a registration's detail view/through the `registration` relationship/


s/retraction/retracted registration/


s/Retractions/Retracted registrations/


That makes sense as long as the exact value is needed.

`Normal = 0,` does it have to be 0?  Could it be 3?  Or 4?  Or 77 million and still work?
`DoCPU_Action = 2,` could it be -1 or 7 etc.?

If they need to be the exact values you set them to, then I agree with your method.

If, on the other hand, the constants you assign to them are just random integers that could be anything else and still have no effect on the program, then I don't see why set any value at all to them.  At which point, they no longer would have as much advantage over plain #define preprocessor macros would have which require defining an explicit value.


you mean we can't replicate the python semantic?

On Mon, Nov 9, 2015 at 4:09 PM, Thomas Peters <notifications@github.com>
wrote:

> so close this issue then? (without merging)
>
> On Mon, Nov 9, 2015 at 4:08 PM, Braxton McKee <notifications@github.com>
> wrote:
>
> > I wouldn't get into modifying FORA semantics unless we really have to...
> >
> > On Mon, Nov 9, 2015 at 3:12 PM, Thomas Peters <notifications@github.com>
> > wrote:
> >
> > > this isn't quite the same as python, for example 2 ** -1 == 0.5 (though
> > > you do still have 2 ** -1.0 == 0.5)
> > >
> > > @braxtonmckee <https://github.com/braxtonmckee> maybe this is the
> right
> > > behavior we want anyway, from a (fora) compiler point of view?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14285906
> > >
> > > .
> > >
> >
> >
> >
> > --
> >
> >
> -------------------------------------------------------------------------------------------
> > Braxton McKee CEO / Ufora
> > www.twitter.com/braxtonmckee www.ufora.com
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287121
> >
> > .
> >
>
>
>
> --
> *Thomas Peters* | Software Engineer | *U**FORA *
> 115 West 27th Street 7th Floor, New York, NY 10001
> Office: 646-918-6435 | Mobile: 732-610-8565
> tpeters@ufora.com | www.ufora.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287149>
> .
>

--
-------------------------------------------------------------------------------------------
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee       www.ufora.com


Hi Thiago,

This sounds like a violation of the DICOM standard - can you confirm?  I am reluctant to put hack/workaround code in the core library but am open to hooks that would allow such behavior to be handled by the application


Ha, thanks!  This is really handy for me, but it's not nearly ready for prime-time.  @strahlex has been using Vagrant.  I'm going to try out his work soon, with Docker as the containerization system.  I'm hoping that Docker, which uses cgroups rather than full virt, can take advantage of the host OS's RT capabilities.


@bobvanderlinden, BTW, this branch's CMake config is useless.  Don't look!


Nice! Updated.


I agree for actual comping the black option is more important than hold/nearest. But we do actually want the hold behaviour is certain scenarios in my dailies project (e.g. a modelling turntable rendered on 10s should hold on the between frames for a daily).


What you could try is to have a fixed angle and multiply both the x and z component with something else to control the speed.


LGTM :+1:


of course we can, but it'll just mean that sometime int ** int will be int,
other times it will be float. We're not changing fora semantics here, but
is this a terrible thing to do from a compiler perspective?

On Mon, Nov 9, 2015 at 4:12 PM, Braxton McKee <notifications@github.com>
wrote:

> you mean we can't replicate the python semantic?
>
> On Mon, Nov 9, 2015 at 4:09 PM, Thomas Peters <notifications@github.com>
> wrote:
>
> > so close this issue then? (without merging)
> >
> > On Mon, Nov 9, 2015 at 4:08 PM, Braxton McKee <notifications@github.com>
> > wrote:
> >
> > > I wouldn't get into modifying FORA semantics unless we really have
> to...
> > >
> > > On Mon, Nov 9, 2015 at 3:12 PM, Thomas Peters <
> notifications@github.com>
> > > wrote:
> > >
> > > > this isn't quite the same as python, for example 2 ** -1 == 0.5
> (though
> > > > you do still have 2 ** -1.0 == 0.5)
> > > >
> > > > @braxtonmckee <https://github.com/braxtonmckee> maybe this is the
> > right
> > > > behavior we want anyway, from a (fora) compiler point of view?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14285906
> > > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > >
> > >
> >
> -------------------------------------------------------------------------------------------
> > > Braxton McKee CEO / Ufora
> > > www.twitter.com/braxtonmckee www.ufora.com
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287121
> > >
> > > .
> > >
> >
> >
> >
> > --
> > *Thomas Peters* | Software Engineer | *U**FORA *
> > 115 West 27th Street 7th Floor, New York, NY 10001
> > Office: 646-918-6435 | Mobile: 732-610-8565
> > tpeters@ufora.com | www.ufora.com
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287149
> >
>
> > .
> >
>
>
>
> --
>
> -------------------------------------------------------------------------------------------
> Braxton McKee CEO / Ufora
> www.twitter.com/braxtonmckee www.ufora.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287238>
> .
>

--
*Thomas Peters* | Software Engineer | *U**FORA *
115 West 27th Street 7th Floor, New York, NY 10001
Office: 646-918-6435 | Mobile: 732-610-8565
tpeters@ufora.com | www.ufora.com


Thank you! :+1:


These seem better for your local .gitignore with `git config --global core.excludesfile ~/.gitignore`


How critical is checkerboard for you? I've definitely been ignoring that one for now. I guess maybe it could be straightforward via a modulo on tile origin to alternate blackTile() and whiteTile() in computeChannelData(), does that sound about right?


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


@mLuby SQL date format without the time should be fine


Why you enabled MODULES ? , our cm do not use modules !


Is Provider use for a single provider and Providers used when there's more than one?


@mLuby cool we'll chat tonight


I have always added the names of the annotators (specifying the role "annotator of the text") in all files (in the header element):

https://github.com/PerseusDL/treebank_data/tree/master/v2.0/Greek/nonArethusaCompliant

I have not yet added the header for the Latin data, on which I am still currently working (so the data are basically the same as in v.1.6 + two more texts). Normalization is required for Matthew's data, whose annotation scheme is not the "standard" one. I will add these texts as soon as I have converted them.


Hola Nacho:
Solo quería preguntarte si tienes pensado continuar con el desarrollo del componente. Lo acabo de instalar y parece que me da algún error, pero cumple con la funcionalidad y de forma sencilla y clara.
Muchas gracias.


Actually, I haven't changed anything wondering what would be a good way to organize the samples.

I tried to make a dialog to show the image when clicking on it, but unfortunately I bumped to a encode problem and It took me too much time trying to figure out how I would make it work.

I think for now I could remove them too.

I was wondering now in creating something like:

   if demo && hasImage then
         show button schema

When clicking schema button it will show the dialog with the image.

What do you think ?


We don't capitalize tags anywhere else so I would leave it lowercase here too.


Please remove `collection` from this list as it's still DevOnly.  I put up a doc patch to remove it from the NodeDetails documentation, but I don't know if it has been merged.


czy nie chodziło Panu o name="end"?
W przykładzie miałam:
<start dtf="yyyy-MM-dd">2011-10-01</date>
<date dtf="yyyy-MM-dd">2013-09-30</date>
<duration interval="years">2</duration>

To duration w ogóle znikło?


Nice!


That taxonomy is interesting. We also have to see how to use the PROV model


>Removing samples licenses seems to be a good idea. It's hard to demonstrate on notebook screen a sample of five lines with 30-40 commented lines above it.

@anselmolsm, in a legal point of view, do you think this is okay ?


Hi, I need to re-read the DICOM standard to remember the details.

The file has those listed tags as UN without a proper length. Those are to be interpreted as SQ with implicit types. I'll try to get one of the samples annonymized to attach it here for analysis.


Set would be more efficient


Changes addressing the latest feedback are ready for review.


kek


@Student007 can you add unit tests here?


Haha, Docker, Vagrant and I'm trying to get things to work on NixOS. A nice mix :) Docker + RT would be really nice. However Docker only works on x64 systems at the moment, so that might be something that's limiting for some.

Alright, I'll close my eyes here!


In H3 pathfinder wasn't able to use teleporters, likely fake calculations for flying and provided suboptimal paths in some cases. My goal isn't simply make it work, but make pathfinder that able to handle complex rules and future modding. :wink:

In particular correct cost calculation are required for future movement changes that I planned (explained on wiki). Those of course going to be in some another branch, but pathfinder must be able to handle path validation.


s/Globally/Global/ ?


Try using more words, so the title is self-describing:
"Continuous build will trigger a rebuild when an input file is changed during build execution"


Please add a section called Other Relationships.  It can just refer the user to the documentation on the registered_from page.


This is actually a breaking fix. RecordType.SObjectType.getDescribe().getName() is actually not valid. It should be updated to RecordType.getSObjectType().getDescribe().getName()


As per our discussion, it will end up like this:

```julia
type Syslog <: IO
    facility::Symbol
    tag::String
    pid::Integer
    ...
```

`pid` will be set to -1 when `Syslog` is created with `show_pid` set to false (default). The print function will print out the `pid` as part of the tag when it is > -1.


should't this be connection-provider?


This really is a duplication of the logic in the Season.is_over method. I prefer to keep this one, but please remove the is_over and replace its occurrences with negated is_active. Additionally, this should be annotated as @property,  or even better @cached_property.


Usecase: Search Books


I misspelled canole - should have been canola


I did this instead:

```diff
diff --git a/package.json b/package.json
index 02b5e7b..b25107d 100644
--- a/package.json
+++ b/package.json
@@ -6,7 +6,7 @@
     "Boris Smus <boris@smus.com>"
   ],
   "description": "Use WebVR today, on mobile or desktop, without requiring a special browser build.",
-  "main": "build/webvr-polyfill.js",
+  "main": "src/main.js",
   "keywords": [
     "vr",
     "webvr"
```

thoughts?


in general, we try to avoid single-char variable names. they have their place, but verbose names are much more readable


The result file need to be updated.


Use the indented style for nested statements with conditionals. We try to keep everything close to the PEP8 style guide.


Still need to handle API < than what we support. For now, you will just DL the few APKs out there that apply to this and the ./add_sources.sh file will report as so.


That's a fix?


what about customers?


the value is that an enum is handled by the compiler where a #define is handled by the pre processor which means when you use the debugger it understands the value of the enum and can show it unlike the define. It can also be type safe, so lots of value of an enum over a #define.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.63](https://pixelated.design/build/viewLog.html?buildId=213&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:10


I think this name could be more descriptive, perhaps "ForbidRetractions"?  What do you think?  It's not a dealbreaker for me.


i think it would be cleaner to call middleware.undo() and the middleware will fetch his own items


As long as it's generally stable during a loop, it should be OK.

On Mon, Nov 9, 2015 at 4:14 PM, Thomas Peters <notifications@github.com>
wrote:

> of course we can, but it'll just mean that sometime int ** int will be int,
> other times it will be float. We're not changing fora semantics here, but
> is this a terrible thing to do from a compiler perspective?
>
> On Mon, Nov 9, 2015 at 4:12 PM, Braxton McKee <notifications@github.com>
>
> wrote:
>
> > you mean we can't replicate the python semantic?
> >
> > On Mon, Nov 9, 2015 at 4:09 PM, Thomas Peters <notifications@github.com>
> > wrote:
> >
> > > so close this issue then? (without merging)
> > >
> > > On Mon, Nov 9, 2015 at 4:08 PM, Braxton McKee <
> notifications@github.com>
> > > wrote:
> > >
> > > > I wouldn't get into modifying FORA semantics unless we really have
> > to...
> > > >
> > > > On Mon, Nov 9, 2015 at 3:12 PM, Thomas Peters <
> > notifications@github.com>
> > > > wrote:
> > > >
> > > > > this isn't quite the same as python, for example 2 ** -1 == 0.5
> > (though
> > > > > you do still have 2 ** -1.0 == 0.5)
> > > > >
> > > > > @braxtonmckee <https://github.com/braxtonmckee> maybe this is the
> > > right
> > > > > behavior we want anyway, from a (fora) compiler point of view?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14285906
> > > > >
> > > > > .
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > >
> > > >
> > >
> >
> -------------------------------------------------------------------------------------------
> > > > Braxton McKee CEO / Ufora
> > > > www.twitter.com/braxtonmckee www.ufora.com
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287121
> > > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > *Thomas Peters* | Software Engineer | *U**FORA *
> > > 115 West 27th Street 7th Floor, New York, NY 10001
> > > Office: 646-918-6435 | Mobile: 732-610-8565
> > > tpeters@ufora.com | www.ufora.com
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287149
> > >
> >
> > > .
> > >
> >
> >
> >
> > --
> >
> >
> -------------------------------------------------------------------------------------------
> > Braxton McKee CEO / Ufora
> > www.twitter.com/braxtonmckee www.ufora.com
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287238
> >
>
> > .
> >
>
>
>
> --
> *Thomas Peters* | Software Engineer | *U**FORA *
> 115 West 27th Street 7th Floor, New York, NY 10001
> Office: 646-918-6435 | Mobile: 732-610-8565
> tpeters@ufora.com | www.ufora.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/7a899766b78326df9887970baf5537c540454f77#commitcomment-14287275>
> .
>

--
-------------------------------------------------------------------------------------------
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee       www.ufora.com


@gnome-mpv Ok that solves all immediate issues but actually using this on Wayland though is behaving strange, it looks like mpv is still using some wayland backend, I don't know who is to blame for that or if you should be forcing some X output?


Will do, also I notice that I forgot to close the reader.


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 22](https://teamcity.labkey.org/viewLog.html?buildId=283842&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 17 (3 new), passed: 249 Build time: 01:08:45
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


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


this is good in Linux also!


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


I think skip it for now. And yes, I can absolutely see Hold being useful for that kind of thing. Still, could this all be put into ImageReader rather than this node?


@wa4557 The icons were not saved and inkscape opens and asks me to save the image! adding that fixed the issue


ah right, sorry i didn't catch that (no pun intended)


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


comment?


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


you could improve these argument names: `arr`, I believe, is a row-major, flat form of a matrix. `inMatrix` is its dimensions.


How is this a fix?  The headers still aren't in sorted order.


I think when this gets merged, Registrations shouldn't be DevOnly anymore.


You can't completely opt out from Let's Encrypt statistics, since the issuance of certificates is a public event.  So we could maybe make this `--user-agent`, and allow a string to be passed to it which actually sets the user agent string?  The help string could be, 'Set a custom user agent string for the client. User agent strings allow the CA to collect some high level statistics about success rates by OS and plugin. If you wish to hide your server OS version from the Let's Encrypt server, set this to "".`


Got it, thanks :)


Little introduction, what I am actually doing:
I want to do a app.nodeQuery() with the following information: Get all the files from the root-folder of the filesystem, but without thumbnails.

So instead of loading all files from the root-folder and THEN fishing out the thumbnails, I tried to load the set of data without its thumbnails from the beginning.

It works right now, but I am not satisfied with the written code. As you can see, I am actually getting two different lists and I merge them into one to get my result set.
But I am sure, that there is a way, to make my wished result set with one single query.

I appreciate any help, just learning right now to get along with the Structr core layer.


Use `Value` here


un bug que habia en el minimap al cambiar el tamaño se deformaba. Y aparte refactorice toda la clase minimap que tenia codigo duplicado


Get `Value` here. Cast it to smi.


Cast it to `HeapNumber` and check.


s/Nodes/Nodes that have not been registered/


Where does this IDENTITY_CAST come from? I don't see any other reference to it in this commit


Let's load just a bare minimum here. It takes extra time for every new constant


The `@(...)` syntax works [only with `shopt -s extglob`](http://stackoverflow.com/a/4555979/735926). This line throws an error in a normal Bash shell:

```
docker: line 731: syntax error near unexpected token `('
docker: line 731: `		*dm.@(blkdiscard|override_udev_sync_check|use_deferred_@(removal|deletion))=*)'
```


Note that I don't doubt the 4 cycle delay here. It's much more likely to be a timing issue on IOP counters.
Possibly the h/vsync timers or anything derived from the IOP clock really. (SPU2 is also directly ticked by IOP).
But it is interesting to see this detail fix a game..


so we're inverting the transpose here? you can also pass an argument to `math.Matrix`, something like

    math.Matrix(data, dim, `row)

which reads off data in a row-major way.


Ah!  Gotcha, thanks!


This is a misspelling?


This only covers the first part of the Issue #747. I will have a look into JsonSerializableParser the coming days in a separate PR.


On Chrome Version 46.0.2490.80 m (64-bit), false works and off doesn't


weren't we holding these as tuples elsewhere?


Ref. cartalyst/stripe-laravel#10


What is this doing here?


It's too much going on in these tests and what we're examining in both tests seems to be essentially that `e.setSpecifier(type, x)` implies `e.getSpecifier(type) == x`.


I wonder if we should check the type of optional parameters. They can be not present, but do we want to allow that they're present but with the wrog type?


No it wasn't intentional, my editor wasn't configured correctly to handle whitespaces. However it was later fixed in the same PR (#45798) : 328cdfe318e9975527bf307ccc66dd611e8d4759


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


I don't know about that. the IOP timers are pretty solid, I don't know how much it would go off hsync and vsync timers.  But this is a least a temporary fix


Good point.


//TODO: comment this out.


This was actually an API change in Qwt sometime after 6.0.1, so it can't be compatible with our current Qwt version and the fedora packaged version at the same time.  This should either detect the version of Qwt, or be turned into a packaging patch.


"contains the necessary `settings.pantheon.inc`" is not quite correct; it includes it, rather than contains it. Maybe just say that Drupal 8 has one, and leave the details to the referenced article.


Yeah.  I said not "as much" advantage over plain #define pre-processor macros.

Not "no advantage at all" over macros, just not as much.

But my favorite advantage is when I have no reason to care what the explicit values are.
`enum { ROCK, PAPER, SCISSORS };`
then switch (exp) { case rock, case paper, ... the exact values just mean nothing and shouldn't care.


Where did you get that from? Its pretty very old, that ship sailed long ago


we should probably add some documentation on this class which says how to read out of flat (it's the row-major flattenting of the array). we might also change this name here


Argh, I tried that with tupples inside tupples which did not work, I did not think using bytearray :)

By the way I don't know how it works, but it is far more longer :/
You can use  the following code, to compare only: it is not an absolute timing due to measuring all .show() and using calls to tick() but it helps to have a comparison.
Obviously, you don't need 128 led, but a big number is great to see differences:
* My code : Around 80 ms
* Your first code : Around 100 ms
* Your last code : Around 330 ms

Also, I don't know if it is a mistake but the comment referring to JanBednarik is removed in you file, not fair ;)

```Python
from ws2812_dany import WS2812
import time

chain = WS2812( nleds=128 )
data = [(255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
        (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
        (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
       ]
start = time.ticks_us()
chain.show( data )
print( time.ticks_diff(start, time.ticks_us()) )
while True:
    start = time.ticks_us()
    data = data[1:] + data[0:1]
    chain.show( data )
    print( time.ticks_diff(start, time.ticks_us()) )
    time.sleep_ms( 150 )
```


Nope, it is too verbose. Don't scare people.


We found a problem with this change, but its fix is fairly easy.  It has been pushed to our main repository, which should obsolete this patch.


>So I don't think the logic can or should depend on how many times per second you execute the function.

According to docs
>Note the timing constant of the loop - running the update per-frame leads to a spurious movement of the coordinate system in which rain is rendered and spoils the effect.

It shows it at 1.0 in the timer, which is saying that running it "too fast" ruins the effect and I think that may have something to do with the rain suddenly shifting from moving up to down and suddenly back up again. That happens on pitch up and deceleration.

Interpolate VS table was less numbers, but a table might work better because I might be able to eliminate some of the undesirable intermediate effects.


Nice


Nope, wait till I finish it.


Thanks for this.
You should probably leave line 182  in templates/lab/base.html as it was before.  So that I give you guys proper credit.


Additional-report-variables-ZoneHVACIdealLoadsAirSystem (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1914 of 1963 tests passed)

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


Removed unnecessary for loop


Well, in `castTypeTest` at least we check for value/type translation.

Any recommendations?


What is the purpose of doing this?


Previously in the article, you used a specific version (8.0.0-rc2), and here you use 8.x.  You should be consistent.  Not sure if it's better to be specific (which will soon be wrong -- we are already at 8.0.0-rc3, for example), or vague (which will never exactly match what is printed).


Or we can just make passwords never announce, really that should never happen.


please help me to istall this on my phone


слово Array — лишнее.


Is this the only way to query the interfaces? I was expecting some kind of notification from the network framework, in order to detect if we are under tunneling or not. That way you only query during startup and on notification. But this is all my assumptions.


Ok, sorry for interrupting. Though we can show msg on LoadSubtitles if no renderer is available. But not every time, some people don't care about subs.


I think think it's going to be hard to isolate code in unit tests if we have the type casting as part of the `DBEnvironment` but could that still be the best place for it?


ok


тут какой-то странный elsif — как будто ты дополнительное условие проверяешь? но ведь это же не так


If you have time, test the last push I did and let me know what you think of it, just out of curiosity, I'd like to know what you see and what you think needs to be changed.


feel free to commit that change if you want


Почти никогда нет смысла писать `== nil`
Во-первых, метод `.nil?` более идиоматичен.
Во-вторых, в данном конкретном случае, достаточно было `unless movie` просто, вместо `if movie == nil`


@lp-- You were prepared to tweak the time management /wrt easy moves (time usage dependent on search/score) ...

Now this is the other way round: adapt the organisation of the last iteration to the time limits.
As far as I see now, depth dependence of Lazy SMP additional depths is just the other way round and these are two different effects.


Alright, I will look for a better place.


You could make that a bit shorter by throwing in some fmaps and joins, but that's probably not worth it.


It seemed right in testing. My impression is that getWidth() & getHeight() were returning the size of the outer view (i.e. *including* the scroll-bars), and here I am trying to get the center of only the inner viewable region. As the comment indicates, I'm sure there is an easier way to do this, and I'm not very familiar with QT, so... feel free to make it more awesome.


The fix for this issue is in pr #3


This was another API change in Qwt, but it won't hurt anything, so you can keep it in a new pull request.


no because this data wont be used until it's flagged on, but i need to get
all the current creatives updated prior to turning it on

On Monday, November 9, 2015, zeantsoi <notifications@github.com> wrote:

> Should this value be contingent on whether the feature flag is enabled?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwick/reddit-plugin-adzerk/commit/adc1c87ca093351ad02980869345bafc59ea61f6#commitcomment-14286499>
> .
>


well, I don't mind leaving the managing editor, technical lead bit off of the page, but I DO think we should have something explicit on this page, under our names, that refers people to the data for the list of contributors.  I have changed the pull request to just include that. What do you think?


This looks fine, but we should have the hwaccess.yaml match what is in security-override. Ie, 'read-paths' instead of 'read_path' and 'write-paths' instead of 'write_path'. Committing this to the tree now as is and made a note to change that.


Yes, that's a bug.


Good idea!


Thanks! Works as before now.


i thought i needed it elsewhere but ended up not. i can do it inline.

On Monday, November 9, 2015, zeantsoi <notifications@github.com> wrote:

> nit: Why assign this rather than just check this in the if conditional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwick/reddit-plugin-adzerk/commit/adc1c87ca093351ad02980869345bafc59ea61f6#commitcomment-14286471>
> .
>


I tend to write simple code, because I favour clarity. Plus I am not that
up to speed with that style of writing code.

On Mon, Nov 9, 2015 at 11:41 PM, Moritz Kiefer <notifications@github.com>
wrote:

> You could make that a bit shorter by throwing in some fmaps and joins, but
> that's probably not worth it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alanz/haskell-ide-engine/commit/92d0f37d4c00b1eb0a16c0f185326dca0b62de9a#commitcomment-14287897>
> .
>


Just a not on the explicit declarations: I used to use var like crazy all over the place. But in my last gig (where we religiously did code reviews before merges), we had a policy of only using var where you didn't know for sure the type that was going to be returned. Mainly because when reviewing code outside of an IDE, types are not always obvious. Cleary, in this case, one can easily tell that CustomerDetailPage is going to be the type. Anyway, it's just a habit I got into, so that's why you see var used sparsely.


Ah good, you noticed my last comment already. :)


How about .`hsc`?


      If we are making the latent heat of vaporization a function of
    temperature, there should be nothing special about 0°C for
    sea-water, and we should have the same trend for cooler waters.
    This whole discussion is a bit odd in that at GFDL we are making a
    bigger error by not including humidity in the heat capacity of air
    in our atmospheric models.

      - Bob Hallberg

    On 11/09/2015 03:34 PM, Alistair Adcroft wrote:

      Limiting to 0°C should not make much difference to the fluxes
        but if it is physically the right thing to do then I'd do it.
        SST's can get close to -2°C. Evaporation is minimal in these
        regions but probably non-zero nevertheless. Is limiting to LHV
        of 0°C the correct thing to do?
      —
        Reply to this email directly or view
          it on GitHub.


@dkannenberg You have a single equal sign ("="), but I don't think that is what you mean.


GHCi-ng could tell type of a region also. Can't ghc-mod do that?


TeamCity ReOpenLDAP :: next [Build 121](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1098&buildTypeId=ReOpenLDAP_Next&guest=1) outcome was **FAILURE**
Summary: Tests passed Build time: 01:35:12
### Failed tests
```
all LDAP tests for hdb/DQD-KFA: test058-syncrepl-asymmetric for hdb/DQD-KFA: <no details avaliable>

```


Any reason not to remove this?


I'm curious why the move to Date from a DateTime?  I would think the increased resolution of a date and time would be important for multiple events occurring in a day.  Also, I would recommend that we move to capturing time-zones as well, and/or creating user-profile time-zones and assuming that when de-serializing date times from the client.


LGTM


The design didn't contain a Flyout. And in fact, I believe it was intentionally omitted by the Design Team. Doesn't matter to me, but it's important to mention that this deviates from the app design.


I just took the params from `ghc-mod --help`


poprawione.


nope, will do so. The following commented out function, that is


perfect timing


Just checked H3 and it's actually opposite: it's always assume that bonus only last one turn only.


@mikemcquaid @xu-cheng I don't know if we should do something about the `TrailingComma` check until we move to supporting only newer Rubies. It fairly often tells people to stick commas at the end of multi-line bracket blocks, and doing so causes Ruby 1.8 to complain loudly.


Instead of:

```xml
<interpolate1d>
    <property>/engines/active-engine/rpm</property>
    <v> 600 </v>  <v> 1 </v>
    <v> 1000 </v>  <v> 1.25 </v>
    <v> 2300 </v>  <v> 1.5 </v>
</interpolate1d>
```

Using a `<table>` would give:

```xml
<table>
    <independentVar>/engines/active-engine/rpm</independentVar>
    <tableData>
         600    1.00
        1000    1.25
        2300    1.50
    </tableData>
</table>
```

You don't have all these `<v>` elements, which make it IMO too verbose. `<autopilot>` has the same problem. I like JSBsim's `<tableData>`.

> It shows it at 1.0 in the timer, which is saying that running it "too fast" ruins the effect and I think that may have something to do with the rain suddenly shifting from moving up to down and suddenly back up again. That happens on pitch up and deceleration.

Hmm.. if you're concerned about the interval, you should port it to `<filter>` (use `<autopilot>` instead of `<property-rule>`) and then put in the filter the following:

```xml
<update-interval-secs type="double">1.0</update-interval-secs>
```

This will give a frequency of 1 Hz. If you use 0.1, you get 10 Hz. I'll test your latest commit in a minute.


This is simply curiosity-driven at this point.  I am not suggesting we
incorporate any of this in the model at this stage.  That is why this
change lives on my branch and I have not submitted a pull request.

-John

--------------------------------
Dr. John Krasting
Physical Scientist (NOAA Federal)
NOAA/Geophysical Fluid Dynamics Laboratory
Biogeochemistry, Ecosystems, and Climate Group
Princeton University Forrestal Campus
201 Forrestal Road
Princeton, NJ 08540

P. (609) 452-5359
F. (609) 987-5063

On Mon, Nov 9, 2015 at 4:45 PM, Robert Hallberg <notifications@github.com>
wrote:

>
>
>
>
>
>   If we are making the latent heat of vaporization a function of
> temperature, there should be nothing special about 0°C for
> sea-water, and we should have the same trend for cooler waters.
> This whole discussion is a bit odd in that at GFDL we are making a
> bigger error by not including humidity in the heat capacity of air
> in our atmospheric models.
>
>   - Bob Hallberg
>
>
> On 11/09/2015 03:34 PM, Alistair Adcroft wrote:
>
> Limiting to 0°C should not make much difference to the fluxes
> but if it is physically the right thing to do then I'd do it.
> SST's can get close to -2°C. Evaporation is minimal in these
> regions but probably non-zero nevertheless. Is limiting to LHV
> of 0°C the correct thing to do?
> —
> Reply to this email directly or view
> it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NOAA-GFDL/MOM6/commit/a85ff5a43a03592e81d849f9bd1e840acac9c58f#commitcomment-14287950>
> .
>


@lochel can you do that? I guess you can use the file generated by omc and keep only the needed variables.


hm? was meinsch demit genau?


Faktycznie. Zmienię też dokumentację na "The ISO 3166-1-alpha-2 code of the country the student is a citizen of."


OutputRulesFile (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1950 of 1959 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)


doesn't iOS make better sense here?


deleting this was a mistake, I ought not to tamper with the master


Perfect you found this, I was looking for answers everywhere and I, once again, loose my mind in this jungle that is internet :)
Anyway I didn't open any bug report as I wasn't sure where the bug we are facing came from...


I removed the design my git client (SourceTree) was choking when trying to check them out. I later discovered that this was because of LFS; my SourceTree install was pointing to its embedded git version (default) instead of the system git (in which I had to install the LFS git extension). But now, I'm getting some kind error from Gitbug when I try to add back the design files. Working to resolve it.


Is that a valid file extension that can be processed by ghc-mod?


Feedback done to the author ;) I’m awaiting his answer.


You need to get the partitions from `getConfiguredContentTypes` here, otherwise not all of them are set.


Sorry, but I can't confirm that. Above commit didn't seem to have made it when merging the PR, but it's also not a revert of the above whitespace change (in line 36), instead it seems to remove the ending EOL (in line 39), which was okay, so maybe that's why it wasn't merged.


OutputRulesFile (Myoldmopar) - 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)


Please specify more context in the commit message.


argh! oh no, certainly not!!

> On 9 Nov 2015, at 21:45, Ryan Balfanz <notifications@github.com> wrote:
>
> @dkannenberg <https://github.com/dkannenberg> You have a single equal sign ("="), but I don't think that is what you mean.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dkannenberg/bloc-jams/commit/5aeebe7ec28c3ff281139ebb2dc55851ab37aed6#commitcomment-14287951>.
>


rename `loader` -> `resolver` variable as well?


`dp` is unused in this function. Also, you should rename that argument. It is not obvious what it is doing.


Why the new line and the change in indentation?  There are lot of these below too.


Need to finish testing for new methods and iteration


You used the | for exception catching which is a Java 7 operator while the build declared java 5 thus a compilation error


Yeah, no need to change it. I just couldn't resist the urge to mention it :)


@alwey: sure, time management dependence on search behaviour and statistics is normal and natural, but not the other way around.  It is possible that last iteration may be special is this sense, but I think what is here is probably simple depth dependence. At least opposite has not been proven.


Additional-report-variables-ZoneHVACIdealLoadsAirSystem (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1427 of 1427 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.47%25-green.svg)


Are you getting these files from the source modality or another system like a PACS (which may have modified the tags)?  Perhaps its time to add some basic dicom dictionary support


Lines 112 & 114-118 appear frequently enough in these tests that I think they could be factored out into a RetractedRegistrationFactory.


Była rozmowa o tym podczas której 1. Szwecja stwierdziła, że nie dają takiej gwarancji, 2. Pozostali (włącznie ze mną) stwierdzili, że właściwie dadzą radę działać i bez żadnego identyfikatora.


Да, теперь ок.
Добавь меня в Collaborators репозитория — я тогда смогу принять pull request — и едем дальше.


This class is from the tutorial. Since the priority was to get the code up and running on GVRf,
i just used it. Only later on, that i found out that we could use the glm's matrix class to pass to the shader.
I'm currently trying to improve the POC to use some more complex models such as the ones
from Immersivepedia sample and i'll make changes on the methods to use the glm matrix. :)


Sorry I didn’t see #17791 before this comment.


ul tag is not closed in the the html


This sentence is somewhat unclear since there are two different meanings to startTime.  I suggest changing the parameter to cancelScheduledValues from startTime to cancelTime.  Then rephrase this.


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


@wlbragg You have two functions that output to `/environment/aircraft-effects/splash-vector-x`, and two that output to `/environment/aircraft-effects/splash-vector-z` :stuck_out_tongue_closed_eyes:


It's unneccessary. I have deleted it.


please assert registration = True in the response.


Ага.
Вроде добавил.

> 10 нояб. 2015 г., в 0:55, Victor Shepelev <notifications@github.com> написал(а):
>
> Да, теперь ок.
> Добавь меня в Collaborators репозитория — я тогда смогу принять pull request — и едем дальше.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/road21/rubyfun/commit/d5360c9d5433fa604c6623c2ddc97a95a74be82f#commitcomment-14288148>.
>


Why the change from SANDBOX to TEST?


How do I run the project? Could you provide prerequisites and a small how-to? Thanks! :smiley:


So if I have setTargetAtTime(v0, t0) and cancelScheduledValues(t1) where t0 < t1, the setTarget is also removed.  Did we actually decide to do this, because I find it unintuitive.  And it's unclear what happens if cancelScheduledValues is called at time t2 where t0 < t2 < t1.  That is, setTarget has already started.


On a first look, a single word: awesome! Cleaning up ad-hoc code and giving it structure makes my day.


@Ip--: I would be glad if someone finds out soon. Some testers are just working on that, e.g. @IIvec and myself.


we should also handle failure cases here from improper shapes: we need arr1.shape[1] == arr2.shape[0]


Hello @hluk,
I'm not sure if this commit is responsible but with the disabled option `Save and display HTML and rich text` the list items are all chopped by one.
![2015-11-09_225406](https://cloud.githubusercontent.com/assets/6043876/11047937/4b2bee06-8735-11e5-88dd-7c4536eefceb.png)
![2015-11-09_230005](https://cloud.githubusercontent.com/assets/6043876/11048015/bcaa8eac-8735-11e5-878d-0c7f4af221ca.png)


Nice thanks bro, like this much better.


На счет этого хотел спросить. так нужно было, что бы он 0 после точки учитывал, или нет?


<!-yagmurkaraman.github.io-->
<meta charset=utf-8>

<!DOCTYPE html>

	<title> Alan Mathison Turing </title>

 Yagmur Karaman 141044016-BIL 101 HW 1
 19 September 2015

  ALAN TURING MATHISON

 Childhood and Youth

Alan Turing was born in Londra, in June 23,1912. He trained in mathematics and cryptology. He is considered the founder of computer science.
[1]

 Scientific Work

 During the Second World War, Turing was proclaimed ‘war hero’ to play an important role in breaking the German code. Turing has designed a machine called the ‘Bombe’ against machine called the ‘Enigma’.Thus, England won against Germany. Bombe took the title of World’s first computer.
   After the war, Turing who studies computer and artificial intelligence logic, to prepare the program that has been a pioneer in this area. Accordingly, he developed the ‘Turing test’. The purpose of the Turing test is whether it is possible to say that logically think of a machine.
   When he was working at the University of Manchester, has laid the foundations of modern computer with definition of algorithm called Turing machines. In 1949, he was the deputy director in computer laboratory at the Manchester University and worked on the Manchester Mark 1 software for one of the first real computers.
[2]

 Death and After

 Turing was found dead at his home in June 8, 1954. Since 1966, technical articles are awarded by a person by ‘Association of Computer Mechanisms’. This award is the name of the ‘Turing Award’ and the Nobel Prize of Computer Science.
[3]

<image src= "https://upload.wikimedia.org/wikipedia/commons/2/27/Enigma-plugboard.jpg" size="150" height="250" align='left'hspace='130%'>
<image src= "http://www.heidelberg-laureate-forum.org/wp-content/uploads/2013/04/turing.jpg" size="150" height="250" align='right'hspace='130%'>

Figure 1: The Enigma
Figure 2: The Turing Award

References:

1. https://tr.wikipedia.org/wiki/Alan_Turing

2. https://tr.wikipedia.org/wiki/Turing_testi

3. http://www.turing.org.uk/publications/dnb.html

4. https://tr.wikipedia.org/wiki/Enigma_makinesi

5. http://amturing.acm.org/call_for_nominations.cfm


Oh ya, I think I ended up doing this in another branch already, but it could be moved up here


I am not sure now :-) Since the candidates are generated, it's better to keep them IMHO.


Позже было бы здорово узнать можно ли (и как) обойтись без "кучи" if\else тут


Chciałem się kłócić i bronić "x-", ale znalazłem właśnie ten dokument:

http://tools.ietf.org/html/rfc6648

Opisuje on historię przedrostka "x-" używanego w protokołach SMTP i HTTP. I wniosek z historii jest taki, że faktycznie lepiej ich nie używać.

Ale oznacza to również, że muszę zrezygnować z enumeracji i opisać tylko parę predefiniowanych wartości. Bo zastosowanie enumeracji nie pozwoliłoby nam na użycie własnych, lokalnych identyfikatorów w naszym NCP, takich jak np. os_id.


Seems like someone mixed tabs and spaces. Fixed it to spaces now.


How about https://github.com/mhinz/vim-sayonara#isnt-this-the-same-as-bufkill ?


Hi,

Thanks for the timing stats :-).

Oh yes, I didn't notice I had erased that, I'll make sure it's there once I put it in the official repo. Thanks for spotting it.

Cheers,
Daniel

> On Nov 9, 2015, at 10:35 PM, Oliv4945 <notifications@github.com> wrote:
>
> Argh, I tried that with tupples inside tupples which did not work, I did not think using bytearray :)
>
> By the way I don't know how it works, but it is far more longer :/
> You can use the following code, to compare only: it is not an absolute timing due to measuring all .show() and using calls to tick() but it helps to have a comparison.
> Obviously, you don't need 128 led, but a big number is great to see differences:
>
> My code : Around 80 ms
> Your first code : Around 100 ms
> Your last code : Around 330 ms
> Also, I don't know if it is a mistake but the comment referring to JanBednarik is removed in you file, not fair ;)
>
> from ws2812_dany import WS2812
> import time
>
> chain = WS2812( nleds=128 )
> data = [(255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (255, 102, 0), (127, 21, 0), (63, 10, 0), (31, 5, 0),
>         (15, 2, 0), (7, 1, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>         (0, 0, 0), (0, 0, 0), (0, 0, 0), (0,0,0),
>        ]
> start = time.ticks_us()
> chain.show( data )
> print( time.ticks_diff(start, time.ticks_us()) )
> while True:
>     start = time.ticks_us()
>     data = data[1:] + data[0:1]
>     chain.show( data )
>     print( time.ticks_diff(start, time.ticks_us()) )
>     time.sleep_ms( 150 )
> —
> Reply to this email directly or view it on GitHub.
>


Why don't You just use tsconfig.json for gulp also (so that You don't have to maintain more than one configuration for typeScript compiler).
I guess You want all d.ts files to be included from the project, except node_modules, so this can be achieved using simple `"exclude": ["node_modules"]` as seen from @ctoran pull request: https://github.com/aurelia/event-aggregator/pull/13/files#diff-e5e546dd2eb0351f813d63d1b39dbc48R9 .
See also https://github.com/aurelia/event-aggregator/pull/13/files#diff-40e62be8220b9aeab02f5a664980bd73R12 to use the same tsconfig.json file for gulp (with exactly the same setup) that is used for IDE
`var tsProject = ts.createProject('tsconfig.json', {`
and then use it in build-ts task:
`.pipe(ts(tsProject))`
as shown here:
https://github.com/aurelia/event-aggregator/pull/13/files#diff-40e62be8220b9aeab02f5a664980bd73R24


You have a typo in the filename


@bobvanderlinden, I may be mistaken, but I believe Docker has i386 and armhf packages in Jessie as well.  True that most images on the Docker hub are x86_64 arch, but there are i386 and armhf images if you search, and there's nothing stopping you from building new images from scratch.  (https://resin.io/ has apparently built a s/w distribution system around the Docker armhf port.)


Ok, thanks.
I'll edit the path in create.php for now.
Which would you recommend I do in future?

R

On Mon, Nov 9, 2015 at 8:37 PM, Greg Bowler <notifications@github.com>
wrote:

> I see, the server online is hosting it under the /test/wireframer path so
> the redirection will need to be modified accordingly, or the server should
> be modified to accommodate this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timetravelopps/wireframer/commit/f057efe0dbb8a0c1670a41a41cff7242ec2c3d7b#commitcomment-14286468>
> .
>

--
*Richard Birkin*
Composer | Creative Technologist

*Music: rjbirkin.co.uk <http://rjbirkin.co.uk/>  *Label/Consultancy:
timetravelopps.co.uk  <http://www.timetravelopps.co.uk>

*E: *biff@timetravelopps.co.uk |* T: *07973 480098 | *Twitter: *@rjbirkin
<http://twitter.com/rjbirkin> | *Skype: *biff.explorer

*Music: *emphemetry.bandcamp.com | *Soundcloud: *
http://soundcloud.com/rjbirkin  | *Facebook: *facebook.com/rjbirkin /


fixed


    qDebug() << width(); // 1615
    qDebug() << getWidth(); // 1598


Comment on specific line


Dodałem przykład.


I think including both `1` and `"Ace"` is redundant


TeamCity Reaver Project :: Mayfly :: Test Build with GCC [Build 32](http://ci.reaver-project.org/viewLog.html?buildId=544&buildTypeId=mayfly_TestBuildWithGcc&guest=1) is now running


This is bad! Besides using magic numbers, it won't restore DWARF_RECLAIM (3) or DWARF_UNRETIRE (8).


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 34](http://ci.reaver-project.org/viewLog.html?buildId=545&buildTypeId=mayfly_TestBuildWithClang&guest=1) is now running


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 34](http://ci.reaver-project.org/viewLog.html?buildId=545&buildTypeId=mayfly_TestBuildWithClang&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:02


TeamCity Reaver Project :: Mayfly :: Test Build with GCC [Build 32](http://ci.reaver-project.org/viewLog.html?buildId=544&buildTypeId=mayfly_TestBuildWithGcc&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:00:16


Might be nice to be consistent with pluralisation (e.g. `"Spades", "Clubs", …` or `"Spade", "Club", …`)


Maybe with @satanasov could be more efficient.


O_o im confused


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 35](http://ci.reaver-project.org/viewLog.html?buildId=547&buildTypeId=mayfly_TestBuildWithClang&guest=1) is now running


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 35](http://ci.reaver-project.org/viewLog.html?buildId=547&buildTypeId=mayfly_TestBuildWithClang&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:00:02


Oni używali pojęcia "software certificate" w Norwegii. Ja w sumie nie wiem jakie jest poprawniejsze.


Not a big deal, but you can leave out the type of the variable from its name to make these names a bit more concise as in `@ranks` and `@suits`.


fixed


This is part of the change from the Wallet backend to the Android Pay backend.


settings.personnelNumber oder settingsForm.personnelNumber


position + row?


This can be shortened to:

```ruby
def deal
  @cards.shift
end


TeamCity LabkeyRelease15.3 :: Git sqlserver 2014 [Build 20](https://teamcity.labkey.org/viewLog.html?buildId=283873&buildTypeId=LabkeyRelease153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


saw approval from javajolt
at https://github.com/basho/riak_pb/commit/37fbe56ec285f377b14c020d85063b82f2bec9e6


merging basho/riak_pb/features/lrb/otp-18 = 37fbe56e into borshop-integration-160-features/lrb/otp-18


basho/riak_pb/features/lrb/otp-18 = 37fbe56e merged ok, testing candidate = 796ed208


comment from js


what about that leak?


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 36](http://ci.reaver-project.org/viewLog.html?buildId=549&buildTypeId=mayfly_TestBuildWithClang&guest=1) is now running


@mohsen1 I already made similar adjustment: https://github.com/apigee-127/swagger-converter/blob/master/index.js#L504
Are going through all public forks or only this in particular?


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 36](http://ci.reaver-project.org/viewLog.html?buildId=549&buildTypeId=mayfly_TestBuildWithClang&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:00:10


Yes, I wasn't thinking straight, I figured that the rpm would override the airspeed most of the time anyway, But I guess it will  override it "always" because it is second and always changing the value after airspeed does, yes?


NULL ?


Mieliśmy dyskusję w tej sprawie, ale dość krótką. Oni raczej wszyscy zakładają, że tylko zaliczone przedmioty powinny się tu znaleźć, więc jeśli tak nie jest, to to ostatni gwizdek, żeby to zmienić.


dat guideline adhering commit message


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4219](https://teamcity.mesosphere.io/viewLog.html?buildId=71218&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:31
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Can this be configurable? For... reasons.


that nice but not connected with #98 ? :)


Ouch! Must be me although not sure how that would happen.


Apart from the style nits, this seems like it will do what you say it will.  My broader question is: what does this mean for the underlying data model NAPI is presenting?  For example, do we actually allow more than one IP network on the same underlying VLAN, and is that on purpose or by accident?  Are we dealing with the potential to create entirely (or partially) overlapping networks (on the same VLAN) in other places?


No.


Nice


When index.isValid() could be == 0 and why we don't have this case here?


TeamCity YaccConstructor :: YC.Core General Build [Build 508](http://teamcity.codebetter.com/viewLog.html?buildId=202123&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


eqe


Doesn't the table only do the three values where interpolate does everything in between?


dwdw


Code style in most functions in this project is different from that (yours). Maybe we should change it? (that's mean your commit, not entire project)


:candy:


BRUH, this was before the guidelines were put in place


Hi!


I think you can handle this all with a dialogselect ?
Better than a custom window because that needs to be skinned... Or did you already try that ?


Definitively, on the webpage we will add this information and more. At the moment it is work-in-progress. I am still working on it.

What about explaining your technical lead better?


Have you used autoformatter or just don't like that kind of justify?


The chunk might come in at a later time.


Usunę "x-". Co do opisu "localInst" to pracowałem nad nim dobre 20 minut, żeby był poprawny. Chociaż przyznaję, że jest skomplikowany - trzeba go przeczytać parę razy, a i tak pewnie nie każdy zrozumie...

Chodzi o to, że u Norwegów mają jedną tabelę na przedmioty, programy i zajęcia. Więc jeśli w którymś USOSie będzie tak, że program oraz przedmiot mają ten sam identyfikator, to im się system wykrzaczy - o ile nie złączą tego identyfikatora z wartością z elementu "type". Czyli inaczej mówiąc ten opis jest tylko i wyłącznie dla dobra Norwegów. Ale nie wiem, czy oni to zrozumieją.


Isn't it great to make polymorph?


No, table should interpolate as well AFAIK.


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


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


This seems redundant to me, since these dependencies are (now) listed in the install instructions.


I don't really feel a need to be listed on the page at all. I just don't want to give people the impression we are taking credit for other peoples' work.  With the redirect from the old site, we are losing that explicit attribution on the page.


Ale czy "title of a degree title, or the name of a course" nie wystarczy?


@ligadata-william check out line 73 ... I think this one should be addMessage as well.

response = MetadataAPIImpl.AddContainer(messageDef, "JSON", userid)