I am trying to pass in testBook as a parameter to the functions this way it can be used inside that pullData() and repeat() functions. Before it was giving me an error saying it was undefined inside my repeat function.

Once it is inside there I am trying to have it so it takes the order book data and stores it inside there.


Vlw! Vou testar depois.


How do I go about allowing the pull data function be able to use testBook and store the order book every couple of seconds?


5277-add-node-cp-output (EnergyArchmage) - 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)


Makes me wonder if the braces are necessary here, as opposed to `$GPG`.


I still can't build the project so I couldn't test the pager code in action, but I ripped out this code and tested it in a console application.

The pager widget doesn't take a total number of items, so the last page has nodes that may not correspond to a legitimate item.

For instance, if there were 33 items and the page size were 10, the last page should only have 3 nodes, but because the widget has no knowledge of the total items, the last page still has 10 nodes with the last 7 being invalid.


A more descriptive commit body or comment in the code could be useful here.


I think a `div` with `font-family: monospace` is more appropriate here.


Hi wkeese,

I met the same problem as [#18602](https://bugs.dojotoolkit.org/ticket/18602) -- in a C# WebBrowser control, the `navigator.MSPointerEnabled` is `false`, no MSPointer event is fired. (In IE 10 and 11)

The problem is, this commit only detects whether the browser supports MSPointer, but not whether the users can actually use it.


It looks like this could be declared inside the loop.


This doesn't look right: we set it to `-1` here and then increment it below, making it 0...


@diosney is there a chance for a new version with this fix any time soon?


Usually I’m trying to avoid margin-top and always use margin-bottom if possible.


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

```


Looks pretty good..  except you want to fork and then put all of this stuff in the ContactClass repository, instead of the examples repository. At this point, that's just an easy copy/paste operation. Otherwise, this looks pretty good so far. I didn't execute it, but I saw two things while looking at your regexes:

* The regex for zipcode will accept a space but not a dash. ZIP+4 is typically entered with a dash.
* The regex for email will accept all of the valid characters for an email address, but it also says everything is fine if the user were to enter something like, "woy@foo@.whatevercrazy.com", or "woytekhasthisaddress.whatever.com". You probably actually want to have two separate matches in there, maybe something like this: `"^[A-Za-z-_.]+@[A-Za-z-.]+$"`. That will match any set of valid characters, followed by an @, followed by any set of valid characters for a host/domain name (underscores are not permitted in domain names, but dashes are--that's why I don't have the underscore listed in the second part). This still doesn't promise that the user has entered a fully valid email address (I could still enter "woytek@somestuff"), but it is better.


Why does this say power and then acceleration?


Actually even if I split the case, I still need to protect against the overwriting the stack/boot code, so can't I just leave the case as one and do something like:
```C
diff --git a/src/base/misc/fatfs.c b/src/base/misc/fatfs.c
index 8fb4c5a..c11a0cb 100644
--- a/src/base/misc/fatfs.c
+++ b/src/base/misc/fatfs.c
@@ -57,6 +57,7 @@
 #include <fcntl.h>
 #include <sys/types.h>
 #include <sys/stat.h>
+#include <sys/param.h>
 #include <errno.h>
 #include <assert.h>

@@ -1423,7 +1424,8 @@ void build_boot_blk(fatfs_t *f, unsigned char *b)
     case OLDPCD_D:             /* old MS-DOS, PC-DOS */
       /* for IO.SYS, MS-DOS version < 7 */
       make_i1342_blk((struct ibm_ms_diskaddr_pkt *)(d1 + 0x00), r_o, 1, 0, 0x500);
-      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)(d1 + 0x10), d_o, (f->obj[1].size + 0x1ff) >> 9, 0, 0x700);
+      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)(d1 + 0x10), d_o,
+                    min((f->obj[1].size + 0x1ff) >> 9, 28*2), 0, 0x700);
       d0[0x12] = 2;            /* 2 entries */

       d0[0x02] = 0x70; /* start seg */
```
Which would seem to suit both old and new PC-DOS, is that suitable?


Is this necessary? Can we just use %NAME%-%version%.download_format for everything?


module fix


Are "ignored and deprecated"?
What about Windows? AFAIK we don't support ZeroConf there yet, how do publishers and subscribers meet each other in that case?


OK, so maybe the stage could fail if `enable_diagnostics` is `false` and the programmatic enable fails?


test should pick two out of N available or use two from user-specified list


Yes, I think that's a great idea.  Tooltip positioning is complicated, especially for large controls, so a way for each control to set custom positioning would be perfect.


"event type information" instead of "events"?


can we also have?

``` shell
get_version_string()
```


Is there any reason for *administrators* to be able to configure matching labels? What if we remove whole `BinderConfig` and add a `PersistentVolumeSelector` to `PersistentVolumeClaim`, where users would specify, what kind of volume they want?

And we could add also PersistentVolumeProvisionerSelector to choose the right provisioner for this claim when no volume matches.

```
	pv := &api.PersistentVolume{
			ObjectMeta: api.ObjectMeta{
			Name: "gce001",
			Labels:map[string]string{
				"environment":"test",
			},
		},
	}

	claim := &api.PersistentVolumeClaim{
		Spec: api.PersistentVolumeClaimSpec{
				...
			},
		},
		PersistentVolumeSelector: map[string]string {
				"environment": "test",
		},
		PersistentVolumeProvisionerSelector map[string]string {
				"class": "magnetic",
		},
	}
```


Yeah, I think we want the more invasive method of taking csv reader construction away from gocsv because they've proved they can't handle that responsibility


`initialse`


Changed to:

    return any(docker_obj in x for x in devices)


Hi,

Looks nice, I very like the idea of ThenKeepDoing and ThenKeepThrowing.
I'd prefer operaqting with interfaces than implementation, but in this case it looks better withour cast in WhenCalled.Do

Good work ;)


On Wed, Nov 11, 2015 at 06:06:45AM -0800, Naja Melan wrote:
> Makes me wonder if the braces are necessary here, as opposed to `$GPG`.

as far as i know, they are not required, and it's just a matter of
style, habit and maybe in former times portability. they may be removed
according to [1].

[1] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_02


@kumajaya How you cherry-pick? I can't seem to... bad object always


From whom do you request it? This is not part of #109 yet, is it?


was bedeutet der neue `Alle Werte sind in rex_article_action $this verfügbar` hinweis?


`initialize`


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

```


should it be `.x` for consistency with other functions?


no, thanks for mentioning.


Okay, I didn't think about that scenario, but is it a common use-case? Does using different parameter sets for each task make sense, because at that point you making each benchmark quite different, so comparing the results across each benchmark would be strange?

I would say that if you need to have different parameter sets per `BenchmarkTask`, at that point you need different Benchmarks, buy maybe I'm misunderstanding the scenario.

Could you make a gist of a sample benchmark that would need this, maybe I'll understand it better then?


when i try clone:

Submodule path 'submodules/msx264': checked out '58941c15dcf23d6b47cdc557b399dc6e7ce2b0b5'
Unable to checkout 'a34bb2a78c713e5b9e2224bf7f793d140b0ca608' in submodule path 'submodules/externals/webrtc'


Maybe instead of fixed schema (which is now an implementation detail of the announcement) you can say that schemas are superseded by sessions as the way to discover publishers from subscribers. In that case you should move this point below the "Introduction of Session"


i'm not sure.
looks like `journalctl --sync --machine debian-root` or `journalctl --sync --host some-host` doesn't work as expected.
maybe, warnings? like `you can not use --sync and --machine`?


`Extracter`


I am suddenly getting tons of errors and no more images but url strings. Something t0 do with this commit :)

DEBUG: webmachine_decision_core:34  undef

14:40:29.305 [error] webmachine_mochiweb:125 caught error undef (stacktrace [{controller,render_error,[500,{wm_reqdata,#Port<0.158379>,{dict,4,16,16,8,80,48,{[],[],[],[],[],[],[],[],[],[],[],[],[],[],[],[]},{{[],[],[],[[ua_class|desktop],[ua_props,{is_user_select,true},{has_pointer,true},{is_desktop,true},{is_crawler,false},{is_wireless_device,false},{is_tablet,false},{ajax_support_javascript,true},{device_os,<<"-">>},{displayWidth,1920},{displayHeight,1200},{inputDevices,<<"-">>},{parentId,<<"generic-">>},{model,<<"browser">>},{vendor,<<"desktop">>},{id,<<"desktopDevice">>}]],[[zotonic_host|hart_amsterdammuseum]],[[controller_module|controller_file]],[],[],[],[],[],[],[],[],[],[]}}},undefined,"::1",undefined,{wm_log_data,699768727,undefined,{1447,249229,298354},'GET',{11,{"host",{'Host',"hart_amsterdammuseum.dev"},{"connection",{'Connection',"close"},{"cache-control",{'Cache-Control',"max-age=0"},{"accept",{'Accept',"image/webp,image/*,*/*;q=0.8"},nil,{"accept-encoding",{'Accept-Encoding',"gzip, deflate, sdch"},nil,{"accept-language",{'Accept-Language',"en-US,en;q=0.8,nl;q=0.6,ru;q=0.4,es;q=0.2,ar;q=0.2"},nil,nil}}},nil},{"cookie",{'Cookie',"z_ua=c%3Ddesktop%26u%3D1%26t%3D1%26w%3D1920%26h%3D1200; z_sid=Gb1KeuXV040eCncQFpyTPegVBLSdavTW; __utmt=1; __utma=42538172.529271901.1447239334.1447242927.1447249229.3; __utmb=42538172.2.10.1447249229; __utmc=42538172; __utmz=42538172.1447239334.1.1.utmcsr=(direct)|utmccn=(direct)|utmcmd=(none)"},nil,nil}},{"x-real-ip",{"X-Real-Ip","192.168.33.1"},{"x-forwarded-for",{'X-Forwarded-For',"192.168.33.1"},{"user-agent",{'User-Agent',"Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.80 Safari/537.36"},{"referer",{'Referer',"http://hart_amsterdammuseum.dev/lib/css/build/screen~165480525.css"},nil,nil},nil},nil},nil}}},"::1","/lib/images/socialtabs/share.png",{1,0},404,0,undefined,undefined,undefined},'GET',http,{1,0},"images/socialtabs/share.png","/lib/images/socialtabs/share.png","/lib/images/socialtabs/share.png",{dict,3,16,16,8,80,48,{[],[],[],[],[],[],[],[],[],[],[],[],[],[],[],[]},{{[],[],[],[],[[zotonic_host|hart_amsterdammuseum]],[],[],[],[[zotonic_dispatch|lib],[zotonic_dispatch_path,"lib","images","socialtabs","share.png"]],[],[],[],[],[],[],[]}}},["images","socialtabs","share.png"],"../../../..",500,1073741824,[{"z_ua","c%3Ddesktop%26u%3D1%26t%3D1%26w%3D1920%26h%3D1200"},{"z_sid","Gb1KeuXV040eCncQFpyTPegVBLSdavTW"},{"__utmt","1"},{"__utma","42538172.529271901.1447239334.1447242927.1447249229.3"},{"__utmb","42538172.2.10.1447249229"},{"__utmc","42538172"},{"__utmz","42538172.1447239334.1.1.utmcsr"}],[],[{"accept","image/webp,image/*,*/*;q=0.8"},{"accept-encoding","gzip, deflate, sdch"},{"accept-language","en-US,en;q=0.8,nl;q=0.6,ru;q=0.4,es;q=0.2,ar;q=0.2"},{"cache-control","max-age=0"},{"connection","close"},{"cookie","z_ua=c%3Ddesktop%26u%3D1%26t%3D1%26w%3D1920%26h%3D1200; z_sid=Gb1KeuXV040eCncQFpyTPegVBLSdavTW; __utmt=1; __utma=42538172.529271901.1447239334.1447242927.1447249229.3; __utmb=42538172.2.10.1447249229; __utmc=42538172; __utmz=42538172.1447239334.1.1.utmcsr=(direct)|utmccn=(direct)|utmcmd=(none)"},{"host","hart_amsterdammuseum.dev"},{"referer","http://hart_amsterdammuseum.dev/lib/css/build/screen~165480525.css"},{"user-agent","Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.80 Safari/537.36"},{"x-forwarded-for","192.168.33.1"},{"x-real-ip","192.168.33.1"}],not_fetched_yet,false,{0,nil},undefined,undefined,undefined,undefined,<<>>,false,[],none,[]},[{controller,render_error,[404,{wm_reqdata,#Port<0.158379>,{dict,4,16,16,8,80,48,{[],[],[],[],[],[],[],[],[],[],[],[],[],[],[],[]},{{[],[],[],[[ua_class|desktop],[ua_props,{is_user_select,true},{has_pointer,true},{is_desktop,true},{is_crawler,false},{is_wireless_device,false},{is_tablet,false},{ajax_support_javascript,true},{device_os,<<"-">>},{displayWidth,1920},{displayHeight,1200},{inputDevices,<<"-">>},{parentId,<<"generic-">>},{model,<<"browser">>},{vendor,<<"desktop">>},{id,<<"desktopDevice">>}]],[[zotonic_host|hart_amsterdammuseum]],...}}},...},...],...},...]],...},...])


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


Thank you very much , I understand now :)


why do you need project.json.lock? there are othher options: http://blog.ctaggart.com/2015/10/minimal-coreclr-console-app.html

btw: paket+dnx discussion is at https://github.com/fsprojects/Paket/issues/736

/cc @tpetricek


the same if for `--flush` and `--rotate`


Ah yeah will do as gareth suggested.


Ok, reworked, now return a proper dict.


Agree with the selector on PVC.  That is in this doc.

I think I agree, too, that we don't need admin-configured binds.  If we already have a selector on PVC, why require the user to either add labels (a selector of sorts) or use selector.  Requiring a claim owner to enter 1 of 2 labels seems redundant.


So, If I got this right, you can ask to a subscriber which is the vocabulary supported by the connected publisher and it will use this URL to request the information from the publisher with the REQ/REP socket?


No, notation you pointed is reserved for describing hashes (assoc arrays).
Here is simply `string[]` with allowed values.


i did NOT think this through :-1:


No, I don't think that's correct.
There are 2 flavours of DOS: the ones with
intermediate bootsector, and the ones without.
The ones without - are below 30K, so do not
need any protection, and _must_ be read fully.


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

```


"This is necesary to be able to update the vocabulary according to the register messages"

How are race conditions avoided? If a publisher is announcing from the moment it's created, how to I prevent the ZeroConf part to announce a half registered vocabulary? I think this can lead to subtle bugs if not considered.


Also you may want to change
"0, 0x500" and "0, 0x700" to
"0x50, 0" and "0x70, 0"


Looks like this did the trick to fix the codesign issues.


This kills the display name on rename -> I will revert this to the mobile.css


We don't have benchmarks yet; planning on making some soon. I'm quite sure it won't optimize; it won't resize the slice every `append`, but it won't be as fast as pre-defining length.


**Code Quality: Nitpick**

You don't require to import `response` and `pagination` here as they are only required within the view files.


I'm liking where this story is going lol


I don't get this. Why are subscribers also announcing?


> in a C# WebBrowser control, the `navigator.MSPointerEnabled` is `false`.

Huh?   Did you mean that `navigator.MSPointerEnabled` is `true`?


[New Branch] - Create new branch for my idea.


This is good stuff. Want to open a pull request?


This is still in draft status I guess.


Isn't separatorHeight an NSNumber? If so you shouldn't compare it directly, it should be converted to a scalar value


the first app repository to test github


Hello, Im new on GitHub because you, but i use your feedingbot and after 1 minute there spawns a very big cell or something in the map that let the bots crash? Is there a fix for it??


Don't comment out, delete. You have a history


Let me revert this change, since type won't affect this test.


Hi @Sharathmk99 ,

which is being sent notification?


**Code Quality: Nitpick**

Indentation of the comments is not consistent.


Is this comment related to this test?


IMO, 'recycle' and 'delete' are the same now. I miss 'attach' and 'detach', which attaches/detaches <persistent_volume_name> to/from <directory> And what's 'init'?


Tá testado!


@soda if the component is loading infinitely, that sounds like there may be a different issue? The idea here is that you can override the `isLoading` state if your app is loading something that the select doesn't know about (e.g. preloading values), but if the select is loading something, that's always considered a valid loading state.

Unless there's something I'm missing (?) if the Select goes into an infinite loading state, that's a symptom of a different issue (e.g. the callback not completing for some reason)


why not just rename the args?


Looks like the above lines have been left over, but `connectWebSocket` should ideally go here anyway to be consistent with other modules (functions go before type declarations).


Wouldn't this find the blacklisted email in any channel?


Why was this commit rejected?
Anything wrong with it?
I need this to get t3jquery work correctly with eg _jh_magnificpopup_?


Sure.


@wolf-daniel  how this is related to the change?


Why not for tests?


I am sure it is not `true`. I cannot trigger 'MSPointerDown' event anyhow.
But in a real browser (IE 10) it is `true`.


@wolf-daniel  we should have this one only on the dev-branch, and after some testing with Dudu - move it to master (so new TP won't get it automatically)


Minor naming convention: should be `is_server` (see [here](http://vibed.org/style-guide#function-parameters)).


No difference to before. The schema was only used for ZeroConf service name, so if you have no ZeroConf, only fully qualified URIs work to connect publishers and subscribers. There is no word anywhere that ZeroConf does not work on Windows. The Servus backend exists and works, so it's only a matter of network setup.


Trying again with one parameter deleted, which seemed to cause symmetry last time.

Otherwise leaving other settings unchanged in order to again test feasibility of this multi-phase tuning idea..


Why `containsRaw` and not `in`?


No, this is a separate story. #109 covers session filtering only.


Correct.


So it seems that fatfs.c has only one occurrence of make_i1342_blk() with segment/offset in the correct fashion
~~~
      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)d1, d_o, 4, 0, 0x700);
      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)(d1 + 0x00), r_o, 1, 0, 0x500);
      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)(d1 + 0x10), d_o,
      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)d1, d_o, (f->obj[1].size + 0x1ff) >> 9, 0, 0x700);
      make_i1342_blk((struct ibm_ms_diskaddr_pkt *)d1, d_o, (f->obj[1].size + 0x1ff) >> 9, 0x2000, 0x0);
~~~


LGTM


@jgrandelli fixed. Also had another small change here. There is a separator above the first button as well, so no need for the `- 1`


That's one thing to consider, yes. Or document. This needs to be further discussed once we are there implementing and supporting.


This should probably use a secure RNG from `vibe.crypto.cryptorand`, mainly for performance reasons.


Off the top of my head I would say yes. But please think it through so that we are sure.


5277-add-node-cp-output (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


it is not rejected, as you see its in master ...


@johnsimons It seems like you are deliberately moving everything to javascript?


To see applications in GUIs for instance. The vocabulary can differ from pub to sub. And the UI publishes events to control an application that it subscribes to.


Yes, totally.


Trying again with one parameter deleted, which seemed to cause symmetry last time.  (spsa didn't end up doing much..)

Otherwise leaving other settings unchanged in order to again test feasibility of this multi-phase tuning idea..


So, both `dao.delete()` and `dao.updateOrInsert()` begin and end their own transactions as well. Why do we do another transaction here? If this one rolls back, will it roll back both of the others?

I also noticed that here you use `tx.begin()`, `tx.setSuccessful()`, and `tx.end()` whereas in the `AbstractDao` you use `tx.beginTransaction()`, `tx.setTransactionSuccessful()`, and `tx.endTransaction()`. From the code, it looks like the former just call the latter methods. Is there a reason you chose one or the other?


Is pendingImage supposed to be one that is up while the other is being fetched, or one that is there when there is no image to fetch?


veg was good


Regarding File:, I honestly don't know. As I mentioned the MediaWiki format is a moving target and I don't think we should support it. I have proposed deprecating it for 7.0

Regarding SharedImages, the goal was to allow having the same image shared across several wiki pages.

Jorge


I think `isNotEmpty` call is not needed here


but healthy is better


**Code Quality: Nitpick**

This `limit_given` variable could have been set the same way you did page and q.


The linux symbol didn't change so it should be fine.

> On Nov 11, 2015, at 12:40 AM, Bara20 <notifications@github.com> wrote:
>
> And Linux? https://forums.alliedmods.net/showpost.php?p=2361925&postcount=130
>
> —
> Reply to this email directly or view it on GitHub.
>


I think I removed the need for `teams.mail_server` in 6600c34362f72c2bf292b61149f602869070c743?


I just updated to 3.0.2 from TER – nothing can be found of this commit…
I patched it manually so…

THX

> Am 11.11.2015 um 15:34 schrieb Kay Strobach <notifications@github.com>:
>
> it is not rejected, as you see its in master ...
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/typo3-themes/t3jquery/commit/82a4c44fc8845a641470eb0325ace9f32bf8a3f8#commitcomment-14330194>.
>


Thanks for all help guys, It's a pleasure work with you.


**Code Quality: Nitpick**

You should ensure that you put a space after the comma separating two function arguments.


It is here due copy/paste from `testCircularDependenciesSamePackage`.
The body of test method is the same, except for lines 515 and 515.


master != TER -.-


No it didn't matter it was just more consistent with the rest of the code, same for finalization or variable name changes. The important changes are: fix on error not crashing the app and speed up spend tx when having large (in MB) prevouts.


Also added FontAwesome and some UX fixes.


last tagged release == TER


Agreed.  Look at line 135 above.

Attaching and detaching are not part of provisioning.  That will be handled here: https://github.com/kubernetes/kubernetes/issues/15524


Ja, das hab ich auch grad gemerkt ;)

Danke

> Am 11.11.2015 um 15:38 schrieb Kay Strobach <notifications@github.com>:
>
> master != TER -.-
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/typo3-themes/t3jquery/commit/82a4c44fc8845a641470eb0325ace9f32bf8a3f8#commitcomment-14330340>.
>


Is this situation (empty `modules`) possible?


Why is the old story deleted? Are they only allowed 1 story?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.117.5d63946f02e3ce9bb63522dd0492d588bd878d58](http://build.palaso.org/viewLog.html?buildId=65974&buildTypeId=bt2) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 716, ignored: 43; exit code 1) Build time: 00:11:34


if target.isTests is false, we have production target, and there is no friend directory to set.


there are some features missing for the next release e.g. 7.x support ...


Hm, it should be corrected. I'll do it.


Okay, thank you.


Ah yes


enne


@alvare52 DWMR. Well done here. :confetti_ball: Press ahead.


this can change by user interaction, so it should be a state (`is-selected`).


I could for sure, but I thought you didn't like it when I reused the arguments (curr1 & 2 get assigned to later)


why not have this in PageableCollection?


Okay.


Yes, of course. I'll delete it.


Kolla! Precis så juh! :)


`element` is not needed and should be omitted


Compilation from maven, for example.


Why did you invert this comment ?


Good.


Note for John: I took away the flag for checking if music is already stopped, because based on my understanding, we won't need to revoke the NowPlayingConnection anymore. It's safe to stop over and over.


Hello,
if you check with **!empty($$current)**, you check **$to_header** on empty and not **$current**.

The previous version is better.


php不是允许这种“错误”，而是它本身作为动态语言的特性。


yes


I think you have to comment out all the option lines because leaving them in this form would clear environment variables.


Fair enough. Removed the +/-1 assumption in https://github.com/cppformat/cppformat/commit/d6d019aa12d38225df25876d52e4101b18de0e6d. Thanks for the feedback.


But if that controller linked above also makes use of the same driver model, then yes, you're right that the plugins will need those operations, too.


Why don't you set here just
if ($actioncode) $sql.=" AND ca.code='".$db->escape($actioncode)."'";
$actioncode seems to be always a string because of a previous implode


I do the overall transaction so that both the delete and updateOrInsert happen together. in the SQLite API you can nest transactions, but only the outermost transaction applies.


Why did you do this? Why not changing the default value to 300 or keeping it on 500? I can't justify both changes.


Scalable-Units-Pumps (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


`typeof` чего угодно вернет `string`, поэтому в этот `if` мы не попадем никогда
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/typeof


Yes if any changes in event it sends push notification to registered mobile
number
On Nov 11, 2015 7:57 PM, "Ricardo JL Rufino" <notifications@github.com>
wrote:

> Hi @Sharathmk99 <https://github.com/Sharathmk99> ,
>
> which is being sent notification?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Sharathmk99/opendevice-examples/commit/cd9b37d22535ea0dd3318dddd3629e53c3015d5f#commitcomment-14330018>
> .
>


I don't think we can type these with `string`. See material-ui's failing tests, for example, where a checkbox uses ReactLink<boolean>.


If you are running on master you need to upgrade webzmachine. Delete deps/webzmachine and run make.


I just want to optimize the logic of FileStore::_setattrs, because every io would touch this logic.


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


логичнее выглядит так:
```
if (a === true) {
  return 'Я попал в' + b;
}

if (a === false) {
  return 'Я никуда не попал';
}
```


Wrong commit message! Improved design!


I think you'd better fix up what I have so far to suit all the relevant types. I only notice now that the MS_D case falls through to the next case if it's not an 'MZ' file. At least that way you'll get what you want, rather than my interpretation of it :-)


оу, и правда, вы же теперь в ветках :)


Do you really need "ppc64::" scope here? I think it can be avoided as there is a `using namespace ppc64;` at the top of this file.


@phase Only issue is, I'm pretty sure this breaks when the input doesn't end in a newline.


the `beginTransaction` `setTransactionSuccessful` and `endTransaction` names come directly from the underlying DB handle. I'm a bit torn on which name to use (thus the inconsistency) `tx.beginTransaction` is closer to `db.beginTransaction` which makes it easier to understand it safely wraps the DB TX logic without having to expose the DB, but it seems a bit redundant "Transaction, begin transaction".


hey


Do you have an example of systemd not supporting older kernels when newer kernel headers are available?


@ronan-mch  @jipe FYI I cleaned up the metrics helper methods in this branch.


Much wow! Such amazing!


Let's extract this as a separate issue, integrate and then rebase this.


Not sure I understand why this is incremented when the story is not new and is dirty. I think I'm misunderstanding the logic in this method: my understanding is that the dirty, not new stories are all deleted, the new stories are saved, and the not new, not dirty stories with pending images have their pending images uploaded to the server. Is that the logic here?


OK, this typo seems really awful. I accidentally removed parameter [duplicated checks](https://github.com/shiraji/androidannotations/commit/63933b4bb61f58bac4bf0450a4b8adfd306ce538?diff=unified#diff-992b0be2508989cf7076ffcc0366d0c8L433) and this typo made pass the test case.

I will recreate duplicate checks for `@Path` and `@Field`.


Just pushed a small fix. I don't have the errors you mentioned though. Which version of webzmachine do you use?  Maybe fetch the locked dependencies and recompile.


Extract this one and previous one as well?


5277-add-node-cp-output (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


5277-add-node-cp-output (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (891 of 891 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.11%25-green.svg)


**!!!**

@barbalberto
`GazeboYarpPlugins::convertRadiansToDegrees` is different from `convertRadiansGainsToDegreesGains` .
This commit breaks the handling of gains... can we try to avoid pushing directly on master when touching critical numerical code?


Curious, why would this correct fix be reverted to buggy previous version?


@ahamill05
This comment above is exactly what should go in the updated README.md


Ok c'est vrai que ce n'est pas très joli.


:sunglasses:


The 2 function did exactly the same thing in the implementation, that's why I did it


Yes, MS_D without MZ and REALPCD_D are the
same things. OLDPCD_D is what needs a full read,
and there is currently no OLDMSD_D at all.


new adress to access CV


Looks like you fixed it :)
Could you now remove the workaround?


I will revert this as well. It should not change.


Congratulations on this successful test!


the old story has a random id because it is new and doesn't have an actual id yet (I wanted to try and prevent new stories from conflicting with existing stories). Due to how I wrote `AbstractDao` you can't currently change the PK of an object without having to write your own update method. So, the only way to update it is to `delete` the "temporary" new story, then `updateOrInsert` the "final" version of the new story


В нём нет прямой надобности, так как он был добавлен для проверки памяти. Но получилось реализовать этот тест другим путем, так что этот метод скоро будет удалён.


I see : "Now you can send notifications to mobile this will be done in next phase" ...


Please don't yank gem versions.


`pendingImage` is a newly selected image on the device that hasn't been uploaded yet


if **$current** is empty, then you try to check if empty variable have empty content. I think should be $current.


Shouldn't we fix it in the hash key too?


Yep, I'll do that.


in the `SimplePicassoImageView` there is a placeholder image defined already. I wrapped a lot of the Picasso logic into a standard ImageView so you wouldn't need to implement the full picasso load logic everywhere. It also allowed me to move some layout related decisions to the XML, such as the placeholder image.


Consider this my 'getting my feet wet' process. :)


Gotcha


added wrong sentence


*TEST* Wed Nov 11 16:00:07 CET 2015


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.535.5d63946f02e3ce9bb63522dd0492d588bd878d58](http://build.palaso.org/viewLog.html?buildId=65979&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 927, ignored: 54 Build time: 00:21:03


https://github.com/robotology/gazebo-yarp-plugins/blob/master/libraries/common/include/GazeboYarpPlugins/common.h#L57 :
~~~
inline double convertDegreesToRadians(double degrees) { return degrees / 180.0 * pi; }
~~~
https://github.com/robotology/gazebo-yarp-plugins/blob/master/libraries/common/include/GazeboYarpPlugins/common.h#L70 :
~~~
inline double convertDegreesGainsToRadiansGains(double gainsWrtDegrees)
{
    return gainsWrtDegrees * 180.0 / pi;
}
~~~


But in that case it doesn't seem useful to check if it's contained by the compiled part of our module, no? Maybe just `return false` here for clarity


There's a lot of them in `config/marc/tag_config_source.yml`


Yes currently I'm working on that part.
On Nov 11, 2015 8:25 PM, "Ricardo JL Rufino" <notifications@github.com>
wrote:

> I see : "Now you can send notifications to mobile this will be done in
> next phase" ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Sharathmk99/opendevice-examples/commit/cd9b37d22535ea0dd3318dddd3629e53c3015d5f#commitcomment-14330812>
> .
>


@bcoca I see. Ok, thanks.


I have to say that "occurances" was what made the marc config amusing ;) You will have to fix it also in config/marc if you change the key


this was copied over from syncDirtyChurches where it already has the logic to sync updated churches along with new churches. I haven't implemented the sync update stories logic yet (it's commented out locally and not committed). Once that code is present this else condition will make sense ;)

the `numParseExceptions` signifies there is some sort of data error on the server or storage layer. see: http://developer.android.com/reference/android/content/SyncStats.html#numParseExceptions


I'm thinking to contribute for library also. Do you have any documentation
so that I can kick start, else I should go through the code.
On Nov 11, 2015 8:33 PM, "Sharath MK" <sharathmk99@gmail.com> wrote:

> Yes currently I'm working on that part.
> On Nov 11, 2015 8:25 PM, "Ricardo JL Rufino" <notifications@github.com>
> wrote:
>
>> I see : "Now you can send notifications to mobile this will be done in
>> next phase" ...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Sharathmk99/opendevice-examples/commit/cd9b37d22535ea0dd3318dddd3629e53c3015d5f#commitcomment-14330812>
>> .
>>
>


heyho, please use the correct @stefjoosten instead of addressing the wrong @stef in your commit messages, or do not push them to github, but i guess that's unrealistic. thx.


@nekron please confirm that these fields are correct for the encounter assays


So they both are named convertDegreexxxToRadiantxxx but they do the opposite thing? One more reason to get rid of them!! j

I'll fix it.


Try using `queryEffectiveChildren('template')` instead. I think they should both work but maybe `<template>` elements get mangled post-distribution?


fparal controls whether to use the number of cores per block as specified in inputp or just use one processor per block regardless of inputp


Please, add a test case verifying that the default keys are used if you user don't specify them.


in all the tag_config_* files. It is a pretty important conf option, you can then test it by verifying in the editor that the tags you change still can be multiple.


wall master processor


Yes, good find.


We don't have a %version% if it's a recipe based on a download URL. We could use %NAME%.download_format, though.


Sounds reasonable. I'll replace this line with
```
if (modules.isEmpty()) return false
```


This looks good to me.  Will merge it upstream.


write vorticity and q in a solution file rather than a function file. This allow to be read in paraview for time animations


Hmmm, probably.


Why?! This makes performance worse. With the `notify_one` call inside the lock, it can't make any thread ready-to-run, making it nearly free. With the `notify_one` call outside the lock, it can, making two expensive operations instead of one.


Fixed in 71420b9a386f1a42846ec13c1ede9cd322e39db6.


Extract data over time for a line of constant xi and eta


Actually, looking it at a bit more, mind adding a unit test for it?  Just want to verify that the write headers go into custom_metadata and amazon_metadata and it is doing what we expect.


Cool. Everything else seems OK to me


Ezee, I put a comment in MainWindow.cpp, line 835. I really need slow down the performance monitoring.

I like this aproach to separate interface and implementation details for multiplatform.
I'm planning change other places in the code to this design:

Some classes have different implementation in VST plugin and Standalone. At moment I'm using traditional polymorphism to create the correct implementations. This is wrong, because there is no change to have a VST plugin Controller being a StandaloneMainController in runtime. This is a compile time decision, so polymorphism is the wrong tool for the job.

This is not a high priority, but we can do some "clean" in this messed design.


INTEGRATION TESTING_2015-11-11 17:10:02


create a restart file shifted in span


I'd like to strip you of your trailing colon.


add a wall communicator block specific


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1076.a1c26a46c239fcac9c043c4dce6441502fac884c](http://build.palaso.org/viewLog.html?buildId=65978&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 483, ignored: 4 Build time: 00:36:31


Any chance of releasing this? At the moment i'm having to install grunt to satisfy my peer dependencies


Thanks!
By the way, fixes here: https://github.com/JetBrains/kotlin/commit/6c777c53fd427a24ea957fee255bb3e250378eac


testing


Makes sense - thanks!


waduh . . ngeri neh . . .


@simonster I didn't see this one before. I'm wondering if we should `break` with a warning here.


Don't forget we'll pull this from S3 and the repo is public so no live data in there is possible please.


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 40](https://teamcity.labkey.org/viewLog.html?buildId=284354&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (3 new), passed: 248 Build time: 00:55:47
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <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.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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


TeamCity NEventStore :: Core :: Build [Build 5.4.0-unstable.3+3](http://build.prxm.it:8811/viewLog.html?buildId=3363&buildTypeId=NEventStore_CI&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 279; runner failure (new) Build time: 00:01:39
### Failed tests
```
NEventStore.Tests.dll: NEventStore.Persistence.AcceptanceTests.when_attempting_to_persist_a_commitId_twice_on_same_stream.should_throw_a_DuplicateCommitException: <no details avaliable>

```


should be early return. will fix


INTEGRATION TESTING_2015-11-11 17:13:46


INTEGRATION TESTING_2015-11-11 17:13:48


lgtm :shipit:


@nick37332001 good


They do the opposite thing because if you have a degree-based gains and you want to convert it to a radian-based gain, you have to do the opposite thing with respect to when you want convert a degree quantity to a radians!

I will be happy to give the functions a more informative name, but I don't think we can get rid of them.


`friendPaths` can be inlined


provider needs to be injected by config subsystem. So we should add a yang configuration model to generate a TxProviderModule. Then update the configuration model for DistributedTxModule to accept a dependency of type TxProvider....

... standard ODL's config subsystem wiring


Cool thx! :)


Any reason this was removed?


`$REX_ACTION` gibt es nicht mehr.
An die Werte des Moduls kommt man jetzt via `$this->getValue/Link/Media(1)` etc.
Wobei `$this` eben  `rex_article_action`ist.


Will do.


@lynnetye Below is another way to do this. Basically we use a function expression that caches the current value of `i` each time and sends that to the function passed to `setTimeout`. This works because our function expression is called immediately (and therefore has access to the current value of `i`) instead of being called later/asynchronously (and therefore only having access to the last value of `i`):

    for (var i = 1; i <= 6; i++) {
      (function (x) {
        setTimeout(function () {
          console.log(x);
        }, 500 * x);
      })(i);
    }

Doing it this way shows a basic understanding of [closures](http://javascriptissexy.com/understand-javascript-closures-with-ease/) and [immediately-invoked function expressions](http://benalman.com/news/2010/11/immediately-invoked-function-expression/).


This variable (`RUBY_TYPED_FREE_IMMEDIATELY`) was only added for Ruby 2.1.0, so using it means Ruby 2.0.0 won't be able to use gRPC. :(

See https://github.com/ruby/ruby/commit/52811ab159e7a9927eba9aa30bc14684ff0852fe for the change where it was introduced.

I wonder if it'd work if we were to `#ifndef` this to 0 (which is what it was earlier on...)


Ok, thanks for that :D


First version added:
https://github.com/NMRLipids/NMRLipids_V-Review/commit/1d7e74d807cfc3ebe861e2a070c9573a9490a452


IMHO this should be a service, so we could switch it in services.yml.
Don't think that having a FQN of a class in a config file is a practice we want to follow TBH.
@webflo may have other thoughts on this.


This is already defined above line 41.


Correct.  This is old text and I'm not changing the contents; just turning bullets into sections.


ok, dann sollten wir hier ggf. `$this->getValue/Link/Media(n)` schreiben?

gibt es auch ein `$this->setValue/Link/Media(n, val)` ?


@tomkort Could you use `_countof(fs_path)` instead of `MAX_PATH` here?


IMHO it would be better to name this commit 'non-functional changes' rather than 'many corrections'.


missing argument


`$home` would at least need to be put through `preg_quote()`


`\$PATH=` should be `export PATH=`


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.4+4](http://build.prxm.it:8811/viewLog.html?buildId=3364&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 76 Build time: 00:03:39


testing


This is just a simulation, or a model. There is no circuitry.


typo: subsequent


Scalable-Units-Pumps (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


@lynnetye This is also useful to read: [http://www.2ality.com/2012/09/expressions-vs-statements.html](http://www.2ality.com/2012/09/expressions-vs-statements.html)


@tomkort Could skip the string conversion stuff and just use `PathCreateFromUrl`?


@andrefelipe Your code was already working. I was able to import multiple items at once because at the end the request to add multiple items is just the same as the request to add 1 item: a POST request to the endpoint /collection

The only things change between a bulk and a single insert is the body ( in the first you have multiple items, in the last just one).

I had to change the content-type just because otherwise I couldn't set manually the key of items I was adding in bulk. To add the key during bulk inserting you have to send data in a different JSON format and set the specific content-type. So with this changes you are able to pass a json like this importing multiple items at once and setting the key manually at the same time

```
  [{
    "path" : {
      "kind" : "item",
      "collection" : "user",
      "key" : "johnny"
    },
    "value" : {
      "name" : "John Jones"
    }]
```


what about `gen_server:call(?SERVER, {set_both, Ds1, Ds2})` ?


As the comment suggests there is a finesse here. Apparently pre C++ 11 libraries may not support snprintf, whereas C libraries should. Unfortunately (as I found out later today) until VS 2015 Win libraries did not include this function.


tnx.
I accept your amendment


this does NOT catch `tar.gz` .. use `File.basename`


Nice. Does this file match the `md5sum_cd1` values present below? If not, we should update the file, either with the sha1 that is in the MSDN page or by taking the md5 values manually.


@tomkort Our code style has `{` on its own line.


Same here.


@tomkort Could use use tabs for indentation?


Why groupedFileItemsMap is a TreeMap? We don't need these items to be sorted.


Hi @jmbog - Yes we're no longer in merchant beta we're doing RC releases which are just bug fixes based on internal regression plus github issues.  We'll be releasing soonish ... believe Ben Marks pre-announced for Nov 31st involving a beach and some space-time mechanics - https://twitter.com/nickweisser/status/657592713034997760


Gibt es: https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/structure/plugins/content/lib/article_action.php


Can you also add '//' style single-line comments?


Crumbs! I didn't even think about the CHANGELOG... I'll start using it from now on!


Хорошее форматирование кода


Why do you need these two? I don't get your intention.


Плохое форматирование кода


Thats a good idea.


BUG: Never seem to reach this point even though `event:auth-logout-complete` gets broadcast on root scope :(


This might be the result of an incorrectly resolved merge conflict..  I added line 180, after all.

I'll reverse this.


Очень плохое форматирование кода


Ah, that makes sense. So we can just leave it as per this commit. Just wanted to check whether it was unnecessary duplication (it's not!)


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.536.4afe188d9858e3e60f29c5558190811d99833b66](http://build.palaso.org/viewLog.html?buildId=65983&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 927, ignored: 56 Build time: 00:20:06


I changed this because ,for example, if you have  a "404" error, instead of seeing that error you see something like 0X5Vde4...


great jobs guys,
Any chance to put the barprah on espota.py before PR, this will avoid me to wait merge and do another PR ?
changes are really trivials
add this func anywhere you want
```python
# update_progress() : Displays or updates a console progress bar
## Accepts a float between 0 and 1. Any int will be converted to a float.
## A value under 0 represents a 'halt'.
## A value at 1 or bigger represents 100%
def update_progress(progress):
    barLength = 60 # Modify this to change the length of the progress bar
    status = ""
    if isinstance(progress, int):
        progress = float(progress)
    if not isinstance(progress, float):
        progress = 0
        status = "error: progress var must be float\r\n"
    if progress < 0:
        progress = 0
        status = "Halt...\r\n"
    if progress >= 1:
        progress = 1
        status = "Done...\r\n"
    block = int(round(barLength*progress))
    text = "\rUploading: [{0}] {1}% {2}".format( "="*block + " "*(barLength-block), int(progress*100), status)
    sys.stdout.write(text)
    sys.stdout.flush()
```
then the main transfer loop becomes with calls to update_progress and offset var + that's all
```python
try:
    f = open(filename, "rb")
    update_progress(0)
    #sys.stderr.write('Uploading')
    #sys.stderr.flush()
    offset = 0
    while True:
      chunk = f.read(1460)
      if not chunk: break
      offset += len(chunk)
      update_progress(offset/float(content_size))
      #sys.stderr.write('.')
      #sys.stderr.flush()
      connection.settimeout(10)
      try:

```


`code` is still being used a few lines below this


@lduarte1991 Thanks for the heads up! I'll make sure to update.


Поправить форматирование кода во всех файлах


If we're already going through the trouble of copying into a buffer, let's put all these commands into prog mem so they don't eat away at our RAM footprint.

This looks like:

```c
const char baud_1200_cmd[] PROGMEM = "U,1200,N";
char baud_cmd[9];
strcpy_P(baud_cmd, baud_1200_cmd);
```


maybe you could simplify this to `handle_call({set_both, Op1, Op2}, _From, State)` ?


@tomkort We've avoided C++ streams to keep the binary size down. Could you use `_wfopen`/`fread` instead? (If you feel like it, it would be nice to have e.g. `ReadFullFile` under `FileUtil.{cpp,h}` under `Common/`.)


testing


Lets continue in https://github.com/bforchhammer/geocoder/issues/2


Дублирование стилей


> gibt es auch ein $this->setValue/Link/Media(n, val) ?

japp
https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/structure/plugins/content/lib/article_action.php#L103-L141


I need these two for a two position in record.
Else, one tuple for two operands?


add `bundle exec` :)


{takeit,{state,{{2015,11,11},{16,50,5}},
               3,8,undefined,3,4,_1_,_2_}}


:ok: :1234:


Is this the right thing to do or should it be an error?


Yes, of course. But the name is very confusing if not wrong.

btw I see that the conversion function is called only on Set/get pid from yarp. When reading initial pid from config file no conversion is done, is it intened?

P.S.
I may have add un bug, but a push force on master is far more dangerous imho


Dunno. I would tend to think no.

There are a lot packages, that do modify initramfs, that are do more than modifying initramfs and that have not "initramfs" in their name.

`grep -r update-initramfs /var/lib/dpkg/info`


Apparently there is a Boost equivalent to snprintf, will look in more detail tomorrow.


Can GroupedFileItems implement Comparable instead of creating another class?


In the end this code should not exist at all. Right now you need it because we don't have a better way of importing stringification, but in the end, we shouldn't touch strings.


Done. ![GitHub Logo](/images/logo.png)


Bon à la fin j'ai réussi à travailler sur console.


yes, we have a convention, at least for javascript files


use `[input.mid(1)]`


I believe we might have talked about setting this to true to avoid events failing unduly.  Will need Yali and Alexs opinion on that though.


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3366&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 75; exit code 1 Build time: 00:02:44


> one tuple for two operands?

why not? `{Operand1, Operand2}` ?

Still don't get it.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.118.77874625d6873528aef4bd1fc318658dbe8e7a4c](http://build.palaso.org/viewLog.html?buildId=65980&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 939, ignored: 50 Build time: 00:37:36


I've just remove 1 indentation, the diff tool must have gone berserk...


this is actually causing a buffer overrun. Remember, in C, strings are null-terminated, so every string has an implicit `'\0'` after the visible characters. `strcpy` copies every character in the source string over, including that terminating character, so your calls to `strcpy` below are writing 9 characters, not 8. That means they're setting the memory address `&baudCmd + 9 = '\0';`, which may not mess with anything immediately, but is not good/predictable.

Given that we know that there's another (at present unused) cmd option for 115200 baud which would have another character to it, I'd probably make this a `char [10];` just in case we add the command in the future and whoever does that forgets to check the buffer length. An extra byte allocated on the stack for this function call isn't a big deal.


EMBED Script
#!ipxe
 echo MonoRail iPXE
 set user-class MonoRail
 echo "Sleeping for 10s to allow data updates"
 sleep 10
 autoboot


Yes, I should have used git revert, sorry.


Lol. ._.


Yep, it orders the parameter alphabetically but, yes, ordering does not seem to be necessary.

This could be a HashMap


I went back and forth on this for a little bit. In general, I prefer the result of `Style/ConditionalAssignment`
```ruby
foo = if bar
        1
      else
        2
      end
```

For cases like this though, I agree that the style gets a bit weird depending on what is going on inside of the branches. I would be fine with adding a configuration. What should the configuration look for? Only assignment to a single variable should always be corrected. Should the config look for multiple assignments in branches or anything other than assignment in branches

I see 2 different styles of code here.

The first is multiple assignments
```ruby
if bar
  foo = 1
  baz = 2
else
  foo = 3
  baz = 4
end
```
corrected to
```ruby
baz = if bar
        foo = 1
        2
      else
        foo = 3
        4
      end
```
The second is single assignment at the end
```ruby
if bar
  some_method
  foo = 1
else
  another_method
  foo = 2
end
```
corrected to
```ruby
foo = if bar
        some_method
        1
      else
        another_method
        2
      end
```

> What "bothers" me here is that the resulting code gives more "importance" to one variable over the other when they are equivalent, just because it is assigned last.

For the first style that I posted, I completely agree with you. For the second style, I think it makes the assignment clearer.


why don't you set `state.operand_one` and `state.operand_two` ?


These error messages take up quite a bit of RAM, let's put them in PROGMEM (see previous note)


nice


What should this do when we have multiple campaigns?  If a user can only have one campaign, there should be an assert or some sort of error-handling.  That said, if this is a pain, I'd be happy for it to get kicked down the road as long as a great big comment is added that warns of the danger......   <= **BELAY ALL THAT**

This function doesn't seem to be used anywhere.  Can it be deleted?


> I see that the conversion function is called only on Set/get pid from yarp. When reading initial pid from config file no conversion is done, is it intened?

That is not intended, but is a legacy behavior that should be fixed taking care of avoid breaking existing configuration files, see  https://github.com/robotology/gazebo-yarp-plugins/issues/119#issuecomment-150269328 .


Yes, I'll create a PR but I guess it will have conflicts.


代码格式真乱啊


@jlongever, @benbp, @jfrey, @zyoung51


Вообще это конечно специфическая задача оказалась. Я-то имел в виду, что нужно отсортировать просто по последней части ФИО (т.е. `split(' ').last`), но наверное Брайан Де Палма и Гас Ван Сент были бы с моей идеей не согласны (а Алехандро Гонсалес Иньярриту и Пол Томас Андерсен не согласны с твоей). Хорошо, закроем на это глаза и будем считать проблему сортировки режиссёров решённой.


I have done this in the hope that if someone comes along later they can decide if they can delete this repo.


You're right, the schema wasn't used in that case either.


Handling edge cases :+1: :heart:


add


for what? it faster?
I can change the values of records at the same time


Do you know how Avocado calculates `md5sum_cd1' and `md5sum_1m_cd1' ?
This values are not from `md5sum xx.iso'
They are something else.
For this case verification passes. I tried different XP isos & found that WindowsXP-64.iso == en_win_xp_pro_x64_vl.iso


Now Not Work no broadcast on Server


Better as:
```c
sprintf(p, "0x%" PRIx64, *(uint64_t *)addr));
```
Or leave the format specifier as-is and cast `addr` to an `unsigned long *`.  Either way is fine, so long as the format specifier and the type match.

Also, the TODO seems like it should be removed at this point.


Needs copyright header


ps use kick form server and Cancelled screen on kick form server to move server


Phew :bow:, thanks @rclark. Sorry for bugging you, I was just going into danger mode. Still working on mapbox-core.


unnecessary whitespace


looks like the indenting whitespace doesn't match here (tabs vs. spaces?)


IIRC the reason for this was that if you are using an inverse link whose range is greater than the domain of the distribution (e.g. Poisson/Gamma with an identity link) and take a step that is too big, you can end up with `mu` outside of the domain of the distribution even if the MLE would not have any domain issues. The effect of setting `dev = Inf` is to force step halving. But perhaps there's a better way of handling this situation.


Thanks for your point out and correct the mistake.


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


опять же, из области «мелочи, но лучше сразу привыкать делать правильно»:
1. Собирание строк с помощью `+` менее эффективно и хуже читается, чем через string interpolation (`"#{x}: #{y.length}"`)
2. Осмысленные имена переменных даже внутри коротких блоков: можно эти имена оставлять однобуквенными, но если ты перебираешь movies, то и внутри блока можно назвать переменную m — а вот после group_by, когда их становится две, так и вовсе хорошо бы написать (чтобы читалось легче): `each{|producer, group|` или хотя бы `each{|p, g|`.


Ah, I guess this is where `campaign_for_user()` in `./campaigns.py was supposed to be.  Similar question, what is supposed to happen if the user has multiple campaigns?  Is that possible?


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3368&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 75; exit code 1 Build time: 00:02:32


I am not sure why you have changed this. Please use `$obj->getKeyName`


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 81](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=457&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:13


Please remove unnecessary whitespace


Release tag please :)


I'm surprised that we never changed `nval` to a INT_BIG (I guess `imp->length` is already an INT_BIG). I think these both have to be properly INT_BIG for things to work right in general. (or else use HDF5 :wink: )


Короч, я опять нажимаю «отправить на доработку», но это уже в смысле «доделай второй бонус и приходи»


This is kind of an RFC document, right? Being that the case it would be nice to have a section in the documentation for this type of documents. It should also have a title (I don't think the file name is enough). I would also point out which parts are to be discussed in upcoming RFC's.


Hey @drewbrokke,

Sure.  I will actually be making some changes and I can send you back a pull with the changes for you to test and make this update as well.


So, is the current solution ok, or should the whole
`qubes-kernel-vm-support` be renamed to `qubes-kernel-vm-dkms`? Or
something else?
Note that it would be better to have the same package name for both
Debian and Fedora, to simplify documentation. And
`qubes-kernel-vm-support` was available in Fedora for some time already
(but not installed by default).

--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?


Please remove line break before `else` (should comply with PSR-2 thus be on the same line as `}` and `{`


Why did u edit the one we no longer use? This is the old format. I put in the readme that I moved the file.


It seems strange/unsafe to `m4_include` this while already inside of an `AC_DEFUN`, and I suspect it could cause esoteric problems.

Furthermore, I think we want to include this some other way, possibly from the top level `configure.ac` or by tricking aclocal into including it for us, otherwise we risk the problem of double including it if another provider or the core configury also decides to start using pkg-config.


Remove force encryption


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.4+4](http://build.prxm.it:8811/viewLog.html?buildId=3369&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 76 Build time: 00:02:17


INTEGRATION TESTING_2015-11-11 17:46:08


INTEGRATION TESTING_2015-11-11 17:46:11


OK, after reading one of your answers below now I get that you actually mean "only publishers were".


Anyway due we are ordering the Map.entries we are going to need the comparator anyway, don't we?


Agreed, attach/detach operations should not be a part of provisioner executable.


5277-add-node-cp-output (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1429 of 1429 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.48%25-green.svg)


@ben-manes Thanks for the note :+1: upgraded in a88734fb8c5c0fab2d525349f59d10476aab2006


what about windows?


I've seen people ask for this, but why is it needed? Don't these come with pcre? It does not seem wise to mix our embedded files with random external pcre.


:heart:


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.537.10be5033d91452bef1753a667412f5736837e227](http://build.palaso.org/viewLog.html?buildId=65987&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 927, ignored: 54 Build time: 00:20:29


thanks - got it fixed!


Splitting into
```sh
test COND1 &&
test COND2 &&
test COND3 &&
test COND4
```
would be more robust and more portable, but I don't think the current code is actually broken on the systems you are likely to run it on.


5277-add-node-cp-output (EnergyArchmage) - x86_64-MacOS-10.9-clang: Tests Failed

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


Woot!

________________________________
From: Cynthia Mijares <notifications@github.com>
Sent: Tuesday, November 10, 2015 4:54 PM
To: SU-SWS/stanford_jumpstart_academic
Cc: Caryl J Westerberg
Subject: Re: [stanford_jumpstart_academic] basic-1435: fix faculty context paths (acfdf09)

Looks good Caryl!

-
Reply to this email directly or view it on GitHub<https://github.com/SU-SWS/stanford_jumpstart_academic/commit/acfdf09b0e83c56249d48d69649244aa274832da#commitcomment-14318177>.

[https://avatars2.githubusercontent.com/u/284440?v=3&s=200]<https://github.com/SU-SWS/stanford_jumpstart_academic/commit/acfdf09b0e83c56249d48d69649244aa274832da#commitcomment-14318177>

basic-1435: fix faculty context paths · SU-SWS/stanford_jumpstart_academic@acfdf09
Read more...<https://github.com/SU-SWS/stanford_jumpstart_academic/commit/acfdf09b0e83c56249d48d69649244aa274832da#commitcomment-14318177>


Теперь пушить OBB придётся вручную. Может, замутить ещё одну gradle таску для этого?


these comments should be moved closer to `FI_GNI_CONFIGURE`


@mostaphaRoudsari like you asked I've created a stl exporter here:
https://github.com/antonszilasi/Butterfly/blob/master/Butterfly%20roadmap%20step1%20and%202%20BlockMeshDict%20STL%20example/ExportSTLcode
 by modifiying your code. Have you duplicated this here? I've asked Theodore to test this stl exporter.


I think users would set only one label - either 'find (exisitng) PV with label "environment=test"' or 'create a new volume with a provisioner with label "class=magnetic"'. Having both in one PVC seems to be useless.


How does Windows work?  Do we know if the rest of Syberia works in Windows?


:+1: @Nagriar


nit: bizarre indenting choice here


Nice!
Danke schön….

> Am 11.11.2015 um 16:49 schrieb Kay Strobach <notifications@github.com>:
>
> https://github.com/typo3-themes/t3jquery/releases/tag/3.0.3 <https://github.com/typo3-themes/t3jquery/releases/tag/3.0.3>
> —
> Reply to this email directly or view it on GitHub <https://github.com/typo3-themes/t3jquery/commit/82a4c44fc8845a641470eb0325ace9f32bf8a3f8#commitcomment-14332238>.
>


They are from an equivalent method and should yield *exactly* the same result.

Now there's the catch. The `WindowsXP-64.iso` I had access to was from some old location at some server from the acquired Qumranet, therefore, likely to be different from the ones you are using.

As for the method, check `scripts/cd_hash.py`. It's a generic hash calculator. To prove you that the results are equivalent, I took hashes with the tool and then compared them with the standard tools:

	[lmr@xps-13 avocado-vt]$ scripts/cd_hash.py LICENSE
	Hash values for file LICENSE
	md5    (1m): d7f84df36e9b3f391eac1fbf7e4907fa
	sha1   (1m): a3226ca6bb3db63b015028a0749a018bf19d4181
	md5  (full): d7f84df36e9b3f391eac1fbf7e4907fa
	sha1 (full): a3226ca6bb3db63b015028a0749a018bf19d4181
	[lmr@xps-13 avocado-vt]$ md5sum LICENSE
	d7f84df36e9b3f391eac1fbf7e4907fa  LICENSE
	[lmr@xps-13 avocado-vt]$ sha1sum LICENSE
	a3226ca6bb3db63b015028a0749a018bf19d4181  LICENSE

So the sha1sum and md5 sum for the whole file are identical as expected. The `_1m` values are the values for the 1st megabyte of each file, used to speed up the verification process (the idea is, if the 1st megabyte is OK, then the file is likely to be good). We sacrifice complete reliability over speed here.


I went ahead and pushed my work-in-progress exporter test.  Maybe we can
touch base quickly to walk through the terminology questions on Thursday
morning?  (I assume folks are taking a holiday today).

Also @fedorov let me know if you are debugging SEG2NRRD or if I should take
a stab at fixing the issues described here.

On Tue, Nov 10, 2015 at 7:56 PM, Andrey Fedorov <notifications@github.com>
wrote:

> I wanted to get the accessors to Steve quickly without taking a lot of
> time to dig into a python callable way to return a triple.
>
> Makes sense - as I said, just something to reconsider in the future.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pieper/Reporting/commit/3630a022485036aac555cb61958477dbf7dd9560#commitcomment-14318211>
> .
>


Any reason why `x`/`y` are `ByVal` when they are not being modified within?


Even if the package was better renamed to end with dkms to follow Debian policy (if they are even adamant about this), I don't think that work required is justified for now, since we're far from upstreaming to Debian anyway.

I am not qualified as reviewer for this. I haven't worked on such low level stuff before.


I could do some more package survey though, look at more similar packages, conclude then.


これ、ものすごく無駄なことしてる。
なぜだか考えてみよう。


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.1-beta.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3370&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:02:51


Thanks. I checked your example file. That will probably work but still needs modifications to be flexible and extendable. The solution should be developed on top of Honeybee objects so we can have control on top of the process. I will have an example file using this library soon. which you can integrate into your process. I will update the issue once it's ready.


note that the issue is with the return of a `next()`'d result - so actually this:

```java
TraversalScriptHelper.compute("g.V.groupCount.by(bothE().count).next()", g)
```

@rjbriody indicated the traversal you have currently works fine.


Many functions in PD now treat mouse coordinates as floating-point, so there's no guarantee that they won't be Single/Double instead of Long.


I saw a picture of a device.


Sorry, Provisioners do not have labels... So it makes no sense to have selector in PVC.


It is need to check $to_header or $from_header. Just try to reproduce this code when you have multiple line in headers From: or To:


So I'd suggest you run the script on the files and update the values in case there is some difference with the reported values.


Отмена


FYI ```net.minecraft.util.MathHelper.clamp_int(value, min, max)``` does the same, but looks nicer ;)


@pieper no, I am not debugging SEG2NRRD - I thought you said the crash was due to the segmentId being off by 1, no?

Yes, would be great to touch base. Today is open for the most part.


10/10 commit


a launch module


@branch-predictor you are just reformatting this piece of code?


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


too much indentation


Thank you guys, I really appreciate your effort and answers! Will let you
know as soon as I manage getting it ready!

Anderson Palma
On 11 Nov 2015 05:03, "John Linhart" <notifications@github.com> wrote:

> I'll just add that installing instructions for this plugin are at
> https://github.com/mautic/mautic-wordpress#installation
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mautic/mautic-wordpress/commit/9b9f02acf1458b364df60c33bc95d008f6da4d78#commitcomment-14321914>
> .
>


I agree - I think these are equivalent, though variable variables always make for slightly tricky-to-read code...


> citation needed


Yea, but `.next()` doesn't return a traversal and `GraphComputer`s require a traversal. You can't `.next()` in a GraphComputer submission. Well you can, but behind the scenes the traversal comes back and then it gets `.next()`.


将两处相同宏定义内的代码合并成为了一处，精简了代码，赞同


if 文を使わなくても、
```
return a.score - b.score
```
でよい


profe para saber como para cuando es 0, osea hacemos que la función de 0 y se supone que ahi deberia parar pero de igual forma nos muestra todos los datos entrando en  la primera condición .


I think 403 Forbidden might be more appropriate here.


Ditto 403 Forbidden.


we could even gain a little more bytes in size by removing the "PANIC:" prefix and making the panic() implementation to add it. Thoughts?


这个地方bug，第二次提交内容有修复，修复代码如下：
···cpp
s_ModifiedFiles.erase(std::unique(s_ModifiedFiles.begin(), s_ModifiedFiles.end()), s_ModifiedFiles.end());
```


Hi,

please open a bug. It ihelps to track all comments from everybody.

Thanks


> ιmplementation
αυτο θα μπορουσαμε να το μεταφρασουμε νομιζω


о


I see, this fix is better, makes sense :+1:.

You no longer need the constant `MOUSE_BUTTON_LEFT = 0` at line 28 then.

Thanks :)


The master branch doesn't have the other floor plan that is why I been asking for that ! I don't know which one should be the one to fix . Merged that file in the master branch so I can edit that one .


So the conditional compilation BEEPER_INVERTED is no longer required, perhaps remove it? bit of a clean up.


> Για projects που δεν είναι σε Ruby και η Sass πρέπει να ενταχθεί στη ροή εργασίας

νομίζω πως αυτό πρέπει να διατυπωθεί καλύτερα


This appears to have broken passing environment variables to the remote command, as in
```
nixops ssh machine VAR=foobar cmd...
```


Раньше были таски ndkBuildReleaseArm и ndkBuildReleaseIntel, теперь генерятся ndkBuildReleaseArm-v7a-hard и ndkBuildReleaseX86. Правильно? Это валидные имена для тасков? Если да, то какие теперь у них акронимы?


Quick question: "overlapped" or "overlapping"? People in [1] says that using the "ing" implies that it's ongoing.

[1] https://www.englishforums.com/English/OverlappedVsOverlapping/xhrwq/post.htm


Okay, I will send a new pull req. Thanks!


@smashwilson can you change all of these headings to be H3's? the pound-sign syntax would be preferred. (`### The Illusion`)


Apart from the 3 comments I left, everything else seems to be just fine :+1:


Its in SRC/resources like in the readme, so that it is included in the jar package.

On November 11, 2015 10:01:28 AM CST, Marlon Garcia <notifications@github.com> wrote:
>The master branch doesn't have the other floor plan that is why I been
>asking for that ! I don't know which one should be the one to fix .
>Merged that file in the master branch so I can edit that one .
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/mgarc138/AGILESE359-459/commit/09d81858ae6faa23f41f52454e019c7ded4fae1a#commitcomment-14332527

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Yep, Jesús also mentioned that I should re-write the whole conclusions section, I will let you know when it is done.


Cool!


I think we want the default to be the immediate settlement model to maintain current behavior. If we call `SetAccountType(AccountType.Cash)` then we'll override it with this value.


Oh, yeah, we can do that.


Thanks for finding this bug in my code


Hey, why are You using our base for your Rom?
You simply rename our base to "Turbo-ROM".. Pls stop it!


Esta linea me parece rara. Esa expresión si puede ser 0.

Revisela


catching these 2 exceptions does not make much sense:

ClassCastEx should be avoided by defining the read future as:
CheckedFuture<Optional<? extends DataObject>, ReadFailedException>

You might still have to cast to DataObject from "? extends DataObject" but thats safe and we dont have to catch class cast


@Yoshi-Guy  u serious rught now ? -_-


Dont catch null pointer, just avoid it


Oh, I should remove, nice catch, thanks.


Есть подозрение, что сломается сборка под windows


sadly this lock won't help to provide synchronization for consumers of the Portfolio object. Maybe we could expose the unsettled cash items as a plain old `IEnumerable<T>`. This means consumers can't directly add unsettled cash without going through the method on line 676, `AddUnsettledCashAmount` but consumers would still be able to iterate the unsettled cash items to make decisions.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.120.4afe188d9858e3e60f29c5558190811d99833b66](http://build.palaso.org/viewLog.html?buildId=65989&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 939, ignored: 50 Build time: 00:35:14


I got the file now ! I will fix it as soon as I can today


Right, `upgrades-status-notify` would stop then.

But if `upgrades-installed-check` exits non-zero, i.e. if the package manager exited with an error, what should `upgrades-status-notify` do in such situations? I supposed it to stop while exiting with the same exit code.

Should it rather do,
* a) `"$upgrades_installed" = "true"` + `/usr/lib/qubes/qrexec-client-vm dom0 qubes.NotifyUpdates /bin/sh -c 'echo 0'` or,
* b) `"$upgrades_installed" = "false"`, `/usr/lib/qubes/qrexec-client-vm dom0 qubes.NotifyUpdates /bin/sh -c 'echo 1'` or,
* c) nothing?

a) Assuming `"$upgrades_installed" = "true"`, assuming all updates are installed, seems plain wrong.
b) `"$upgrades_installed" = "false"`, assuming that some updates are missing, also seems wrong. So sending an update notification in such cases seems wrong.
c) Doing nothing seems like the best outcome in such situations to me. For now. Unless we want to invent another qrexec service and report this issue to the user at some point. Perhaps when the management vm is implemented, we could somewhere have an "update check failed" indicator.


Sergio, el for nunca para. En el for se debe cumple el numero de interaciones para que pare. Como lo tienes el entra cuando la expresión vale 0 y entra en el otro lugar cuando no. Pero el FOR siempre hace el numero de iteraciones que le exiges.


this fixes the weird problem with the repo after multiple `git commit --amend`


Not yet .. Working on it


Почему так теперь? Раньше ведь напрямую вызывался ndk-build.


OMG :+1:  !


Didn't know that was a thing.


感谢您对本项目的贡献以及对linux下热加载部分的完善，我们会尽快对您这部分代码进行单元测试，如果一切OK，且不与我们最新版本的其他功能相冲突我们会尽快采用你这种方案（我们现在最新版本已经有较大的变化，功能也更多），并将在3.1.X版本和大家见面。十分感谢您的pull request


Да, имена для тасков сейчас с именем abi, имхо так логичней и правильней. Для тасков имена валидны, аббревиатуры - nBRA и nBRX


I am using your ROM as a base because you have all of the features I like. I will add my own features later,but I really like you development. That's all.


0.9 might have a libstdc++ for i386 tess2 OSX in the mix from this


Hey @leonli, good question. We run the tests twice to make sure the tests pass with a populated cache. We delete the cache directory before running the tests for the first time. When the tests run for the second time the cache generated from the previous run is used.


clement


Thanks for fixing this. I don't know how much you have been following, but what is your opinion on bringing in youtube-dl as a dependency (advantages and disadvantages discussed in #110)?


Yes.


Тут дифф поехал, здесь мы создаём таски cppLibsBuild**. Раньше код был тем же, глянь выше в первом файле.


Great stuff! Made a couple small comments, most important relating to the synchronization of `Portfolio.UnsettledCashAmounts`

I was unable to comment on the line, but we'll want to update the [doc](https://github.com/StefanoRaggi/Lean/commit/21ac16200f82d43e547034add74d2ed60d4833a4#diff-ea81a360172dd59a9b5b98d829e4f26fR47) for the `UnsettledCashBook`


This line is causing a test failure. On my local machine, this is passing.

Digging down, the key function here is ``cherrypy.url()``.

I'll look into this, but since I'm not particularly familiar with the test harness server startup, are there any ideas? I thought the test server was actually running on port 30000 consistently.


Please check the revision here:
https://github.com/adrelanos/qubes-core-agent-linux/commit/d5acf83916a3722279a60679fe1638387054f8e3


угу, точняк


Hmm. Ok, I will think about it. Maybe, I should revert source to the old params system.


my worry with this is that we have a change downstream: https://github.com/quizlet/jquery-pjax/commit/543172ab7537893f1d5e36b6e82ad80c526a7254

if we don't need that any more, it doesn't look like too much has changed


:jack_o_lantern: Thank you for your effort in following up on your previous commit. :+1: I'm not complaining at all, I just found it a little bit funny that you weren't taking any chances with the file name `/sql/updates/world/2015_11_12_00_world.sql` -- since there was 8 hours left of 2015-11-11 when you committed this file. :smirk:
Compared to the previous file name `2015_11_11_01_world.sql`, I thought that it would be named 2015_11_11_02_world.sql (or next available index number, if other SQL updates got in between).


If u give proper credits n mention its on Razor base.. then i have no problem..


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.119.10be5033d91452bef1753a667412f5736837e227](http://build.palaso.org/viewLog.html?buildId=65986&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 939, ignored: 50 Build time: 00:56:11


I'm in the office today and tomorrow, tomorrow morning works other than 10-11, 2:30-3:30.


@vibhu0009 Don't worry! I will mention that its base if created from Razor and Slim, I'm not trying to steal your work! Sorry about the confusion.


wtf o.O


this is not right!


isn't guzzle a strong requirement, not just for dev? shouldn't it be in 'require'?


Ciaran please don't commit all these metadata files and other rubbish.
 In PlayerGui script you added line 67 and 68 with no reference (nullReferenceException)


@kovrik  Yes, it did work. There were also some other commits in the last days and I havent verified that it actyally is this specific commit.

There are several problems in this function:
```el
(defun use-package-handler/:ensure (name keyword ensure rest state)
  (let ((body (use-package-process-keywords name rest state)))
    `((let ((package-name (or (and (eq ',ensure t) ,(use-package-as-symbol name)) ',ensure)))
          (when package-name
            (require 'package)
            (use-package-ensure-elpa package-name)))
        ,@body)))
```

To get around the undefined use-package-as-symbol this change is probably good:

```el
  `((let ((package-name (or (and (eq ',ensure t) ',(use-package-as-symbol name)) ',ensure)))
```

After I recompiled my init.el, the next thing which gave an undefined error on startup  is `use-package-ensure-elpa` which confirms my worries that `package.el` is being required when I really don't want it do.

I'll have to stop there for now, don't have time to figure out a solution atm.


было лучше


Sounds like a good idea @dorimanx. How can I check the buffers?


I seem to remember making a conscious decision to add this but I'm not sure I can justify that decision in hindsight, particularly given that it means this `case` statement is slightly different from the others. I'm going to make this an `else` so that it's consistent with what I've done elsewhere.


赞版本君


did you use automatic find and replace?


Well, that was committed 5 days ago, but I guess only recently pushed?


Done.


worked for me


@kovrik Should I revert this commit until you have a reworked version?


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 42](https://teamcity.labkey.org/viewLog.html?buildId=284372&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5 (3 new), passed: 0 Build time: 00:07:49
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

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

```


Do in some better way.


Steve, tomorrow at 11 should be good, but I am available to talk today if
you like. Please clarify debugging of what issue you were referring to. To
my knowledge, there are no issues in SEG2NRRD to debug, I thought you said
the crash you mentioned earlier was resolved.

On Wed, Nov 11, 2015 at 11:14 AM, Nicole Aucoin <notifications@github.com>
wrote:

> I'm in the office today and tomorrow, tomorrow morning works other than
> 10-11, 2:30-3:30.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pieper/Reporting/commit/3630a022485036aac555cb61958477dbf7dd9560#commitcomment-14332851>
> .
>


we also need to protect against warnings here, op/magic-27839.t is failing here due to uninitalized variable
$fullname && .... fixes it


Ja, das geht. Aber ich ändere es mal zur Leserlichkeit in deine Variante um.


@svirt well, splitting quoted strings correctly is precisely the reason why we use `shlex`. I'm curious to see the commands that you had problems with. So I won't be merging this until I have a better idea of what's going on.


Поехало, поправлю.


done :)


I don't think the warning is right. i think through the fields the class is easier to understand


Thanks for the suggestion!
I tried it but get the same result.
Maybe just try again after the next update?


Hi Andrey -

There are a few issues with SEG2NRRD - the ones I sent you by email
yesterday:

It seems to work if I remove the "-1" from the segmentId - I guess dcmtk
got fixed to start from one like the standard.

https://github.com/pieper/Reporting/blob/refactor-to-new-dcmtk/SEGSupport/SEG2NRRD.cxx#L221

After this I found that it wouldn't proceed without CIE colors defined, so
I hard coded them and now it crashes in the geometry check here:

https://github.com/pieper/Reporting/blob/refactor-to-new-dcmtk/SEGSupport/SEG2NRRD.cxx#L314

with this message:

ERROR: Frame 1 origin [85.3449, -133.929, 116.786] is outside image
geometry![0, -1, 0]
Image size: [256, 256, 2]

If I stub out that check, the conversion "works" but generates a single
slice nrrd file.

I still don't see how to populate the CIE (maybe that's a color logic
and/or EncodeSEG issue) and I don't know why the geometry is not correct.

Let's do a hangout now to go over things.  I'll start one and invite you
both.

On Wed, Nov 11, 2015 at 11:17 AM, Andrey Fedorov <notifications@github.com>
wrote:

> Steve, tomorrow at 11 should be good, but I am available to talk today if
> you like. Please clarify debugging of what issue you were referring to. To
> my knowledge, there are no issues in SEG2NRRD to debug, I thought you said
> the crash you mentioned earlier was resolved.
>
> On Wed, Nov 11, 2015 at 11:14 AM, Nicole Aucoin <notifications@github.com>
> wrote:
>
> > I'm in the office today and tomorrow, tomorrow morning works other than
> > 10-11, 2:30-3:30.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/pieper/Reporting/commit/3630a022485036aac555cb61958477dbf7dd9560#commitcomment-14332851
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pieper/Reporting/commit/3630a022485036aac555cb61958477dbf7dd9560#commitcomment-14332939>
> .
>


you don't see the package name in any other documentation o.O I think its unreadable


A quick fix ... I think the intention was to load automatically the pluginlist at startup ?
This is done now at window creation .
I can improve that to work also on showEvent by modifying :
*void MainWindow::restorePluginsList()*

onto

void MainWindow::restorePluginsList(bool fromSettings)

If neccessary .... ^^


hm, interesting.
I remove the line `PassEnvironment=VAR1 VAR2 VAR3`. Test fails.
But it shouldn't.
`MANAGER_USER` should pass all environment vars to the spawned processes.
I'm stuck:)


Why the `0.001, 0.001` ?


This line is causing mismatched button heights on my local prereg page:

![screen shot 2015-11-11 at 11 20 00 am](https://cloud.githubusercontent.com/assets/86835/11096134/8e5b95f0-8866-11e5-9255-1737056820f1.png)


userdom_manage_tmp_role(system_r, systemd_logind_t)

seems a bit much. AFAIK it only needs to be able to be able to purge /run/user/$UID

The problem is (i believe) that fedora labeled /run/user/$UID type user_tmp_t. This is sub-optimal

In my policy i basically treat /tmp, /dev/shm, and /run/user/$UID as "tmpfs_t" this simplifies things a great deal

my /run/user is labeled "user_var_run_t" and systemd logind is allowed to manage "user_var_run_t" directories.
so it can create the /run/user/$UID directory, then it mounts tmpfs on it, and /run/user/$UID becomes "tmpfs_t"

then systemd_logind_t is allowed to delete all user tmpfs (for when the user logs out)


Go doesn't support []byte constants, but it does allow constant strings that can be treated as []byte – let me know if you'd prefer a package-protected var or const magicByte1/magicByte2.


"OS X", "Debian".


shit


hmm, I am using it effectively here https://github.com/dart-lang/polymer-dart/blob/6362db73e88aa9cbc4e9d444856097a9b841860b/test/src/template/templatizer_test.dart#L94


On CentOS 7 and RedHat 7 this directory is on a temporary filesystem. It is removed an recreated on each boot.
This method would work until the server is restarted. A better solution is to use a file for tempfile creation:
http://www.freedesktop.org/software/systemd/man/tmpfiles.d.html


The ports are dynamically assigned so as to allow parallel tests to run without conflicting with each other.  Off the top of my head, I think you need to use `os.environ['GIRDER_PORT']` as the port.


can include dashes and underscores


If I understand that correctly, this adds a map into the morph. While this may work, it would be nicer to have this map added in the morph-enriched.xml directly so that one could trace the lookup mechanism solely by searching this morph file. Look at the end where the filemaps are added . That would be the place to define "addLatMap" as restmap.


Краще використовувати EventDao замість EventMapper


The commit note says, "Note that the block for `formatted_sick_pay_weekly_amounts` was instantiating its own copy of the calculator". Unless I'm missing something, this doesn't appear to be the case based on the diff.


@ekkis You've convinced us. How is this: https://github.com/neo4j/docker-neo4j/commit/77e55e640da3b977bef065f9860367def66dc539


not sure i'm into the cloud now


I like it as well!


I wouldn't necessarily update the name of this icon, since it's a little project folder. We'll probably add a new icon for sources.


LGTM, but:
- do we already have netaddr as an import in that file?
- have you already tested, or do you plan to test?


ben python 3 kullandigim icin iteritems() yerine items() yapiyorum burayi


You should add your favorite color.


\n is not compatible with a non Unix environment.


I have released 0.3.78, since this is a major issue.

@zgrimshell this seems to break a great deal (all videos? I'm not sure), so you probably should update the debian package to either 0.3.78 or 0.4.1.


Just try set to 0. Some 'default' footer/header height will be shown. Pretty weird. The same actually if you return 0 for height* methods.... I've found that trick in stackoverflow, and it works well :)


the name of the app is smartquote


Thanks for doing this! :+1:


Change looks good. Thanks.


Why don't we set the value for automatic to be public? That way each pickup has its own specific value that can be designated in the prefab we set it to.


Well done! Thanks @davidtkelleher!


All right, looks good.


Make that `GIRDER_TEST_PORT` for the python tests.


Fixes #11


Okay I will have fun


mmm, why do you declare the var app, if this is already declare in app.js?


testing


Are we juts being more permissive to accept http/1.0 ?


Краще використовувати LinkedList, оскільки він швидше додає нові значення в список


@philikon @vjeux @josephsavona Note that Promises have one nasty disadvantage over callbacks - they hide unhandled errors: https://github.com/facebook/react-native/issues/4045

> In accordance with the unwritten rule that any API that takes a callback should also return a promise

Do we have such rule? If you're returning a Promise you don't need to pass in a callback anymore. I would say it's one or the other but prefer callbacks due to the issue above.


Why serial? And you don't have to specify `test/test.js`. The default test globbing is only one level.


Thanks!


While we're on the subject, I was stuck deploying to Bosh Lite, trying to identify the deployment manifests from the other components. The step was not clear and could be clarified.  I ended up like this:

    $ ./bosh-lite/make_manifest ../cf-release/bosh-lite/deployments/cf.yml ../diego-release/bosh-lite/deployments/diego.yml


What are those used for?


players dont see when server change a block


Nice!


Testing against 1.8.7 not possible?
Is support for RHEL6 being dropped?


ModifyTimeBinsToUseRounding (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1960 of 1960 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)


Check out that cool indent :p


Better graphics?


yup


`type: 'one of: dialog, inline',` ?


hola


Added my first news article from old.imint.se. Corrected format of existing news article.


This patch causes a simple "make" on MacOS to fail; I need to use "CXX=clang++ make" for things to work, which is inconvenient...


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

```


true, so this gives formatting errors on Windows


Shouldn't this be in opm-common?


@snarfed I'd love your opinion on this. Gets rid of most of the exclusions from testdata, but adds a lot more random data to maintain (some of it with very minor inconsistencies like one html file will have `class="h-entry"` and another will be exactly the same except `class="h-entry h-as-share"`)


Why a second README?


How about arity? What if the list of argument patterns and the list of arguments have different lengths?


I am wondering whether it is permitted to change the description of the patch: Bram described the problem incorrectly because I reported use-after-free and not double free (which may happen with this code, but *very* unlikely compared to use-after-free).


Soluciona #18


I'm especially not sure this is valuable because some of the test cases test something interesting in one direction (e.g. the nested_author one with vcard and p-author) that is totally banal in the other direction


No problem, and thanks.
Just let me know if I have chances to help our project. :)


Is this a bukkit plugin or an api? I assume it's api because I can't fine a plugin.yml.


oh, I understand.
I can move
```
+        assert_se(setenv("VAR1", "word1 word2", 1) == 0);
+        assert_se(setenv("VAR2", "word3", 1) == 0);
+        assert_se(setenv("VAR3", "$word 5 6", 1) == 0);
```
into the `main` (src/test/test-execute) (before `manager_new`), and remove `PassEnvironment=VAR1 VAR2 VAR3`. This test passes. But `exec-passenvironment-empty.service` fails.

Or, I can simply run:
```
sudo VAR1=a VAR2=b VAR3=c ./test-execute
```
`exec-passenvironment-empty.service` fails.


На практике LinkedList лучше вообще не использовать без крайней необходимости


Понять и простить :)


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


Looks good, I'm going to merge it today.


So we still use thread pool to schedule future/promise?


Precisely, the existing method


Probably it is but it is really hard to have a gemfile which supports all ruby/puppet combinations. the gem I need are continuously updated with new ruby requirements.

all RHEL versions are still supported. If oracle supports DB on a O.S. then oradb will also try to support it.


Yes its pleasure to work with yours company so plz tell me the process for
dping job in yours company
On 11 Nov 2015 21:47, "Javi Palanca" <notifications@github.com> wrote:

> worked for me
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tschellenbach/Django-facebook/commit/7d1219b36ef9318495032d8dd0626d11b45fbf9a#commitcomment-14332914>
> .
>


Graphics always good except strange Browser close effect since the beginning (pre-hammerhead). With hammerhead blobs, we don't need boringssl-compat anymore that reported make Facebook app crash.


we should unset `VAR1`, `VAR2`, `VAR3` before `manager_new`

maybe, comment about `MANAGER_USER` and `PassEnvironment` in `test-execute`?


You're made of :sparkling_heart:


Yes its pleasure to work with yours company so plz tell me the process for
doing job in yours company
On 11 Nov 2015 22:08, "Manish Soni" <manishmandy.cool@gmail.com> wrote:

> Yes its pleasure to work with yours company so plz tell me the process for
> dping job in yours company
> On 11 Nov 2015 21:47, "Javi Palanca" <notifications@github.com> wrote:
>
>> worked for me
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/tschellenbach/Django-facebook/commit/7d1219b36ef9318495032d8dd0626d11b45fbf9a#commitcomment-14332914>
>> .
>>
>


@kumajaya also what prima fix did you commit? quite curious


![nornorm](https://cloud.githubusercontent.com/assets/2557416/11096582/382a47dc-889b-11e5-8bf4-56caa402b13a.png)
![Uploading norm_axis1.png…]()
![Uploading norm_axis0.png…]()


Will it work properly with gargoyle router having WAN port disabled and working as an Access Point connected to another device using LAN port?


what is this?


Подожду. Потом приступлю к 213.


Are you merging these changes to the main roslyn branch?


let me know how detailed you think this portion should be. im thiinking that for the first two errors, we could have the app tell you what went wrong. the last one is just a catch all for all the other errors that could occur, but i don't think its necessary to be super detailed about the details that aren't the fault of the user :star:


Простить! :)


From upstream http://review.cyanogenmod.org/118922 since I always rebase my kernel to upstream.


@kumajaya sometimes quite annoying but I managed to learn how to cherry-pick fast XD
git cherry-pick id1^..id2


Perfect Ezee!
I don't remember exactly why I write this shit code, but I think I was trying avoid freeze the GUI when you have a lot of plugins to restore in initialization.


This will not work as expected. There are some [potential issues with scopes](http://www.toptal.com/angular-js/top-18-most-common-angularjs-developer-mistakes#common-mistake-2-not-having-a-dot-in-there) so we decided to always isolate scopes. This means in practice that there should be something like this:

```
    scope: {
        isShowMap: "="
    }
```


Won't this saved memusage data add up to writer memory usage counters?


wut bug was there? :p