Yes, makes sense. I can't test this evening because of bad weather. But by
the end of this week the forecasts are better.

One of the problems with wINDI is already fixed. But  full INDI compliance
of wINDI is not possible since some properties cannot be mapped to ASCOM as
Peter from cloudmakers pointed out.

2016-06-07 21:02 GMT+02:00 Juan Conejero <notifications@github.com>:

> I have detected further problems that we have to address before
> continuing. One of them is a conflict between mean and apparent positions.
> I have implemented precession/nutation corrections to convert to/from
> apparent positions when necessary, which I'll upload later.
>
> Another issue is that OBJCTRA/OBJCTDEC coordinates reported by the INDI
> server are not very accurate in terms of precession applied to convert from
> the date of observation to J2000.0, as far as I have tested them. IMO we
> should replace these coordinates completely with new ones computed using
> accurate models, i.e. the IAU 2006 precession/nutation model. I have this
> already implemented now, as noted above.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/PixInsight/PCL/commit/0fb8b5856146623515bc1132bf9796eed7459660#commitcomment-17776250>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACDypuasF8kWO7HVNebfxYD4fXVTBc2tks5qJcBYgaJpZM4IvHwD>
> .
>


Sounds good! Worth an issue?


1_12_bugfixes (macumber) - x86_64-MacOS-10.10-clang: OK (2383 of 2427 tests passed, 0 test warnings)

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


Looks fun.


this is for testing pull command


Isn't it just on conversation you can put into you're  spam folder?


:smile:


Désolé, oublié de pull avant de faire le commit. Mais rien de grave, c'était juste quelques corrections d'orthographe dans des commentaires.


Discussion at [Merge Atomics branch into Master](http://groups.google.com/forum/#!topic/cryptopp-users/WDsbuF8X3wI) on the mailing list.


I suppose it can't hurt to include on both, though I didn't notice it caused problems to only include it on the index route.


.*.swp 就行了


Thanks!


输出符号用逗号吧。。。别写这么多行


> * CLUSTER SLOTS output now includes the node ID (in a backward compatible  manner).

3.2.0 has changed `cluster slots` return format, even though it states it's backward compatible, but it still breaks corvus.

Currently released versions all can't handle the new format. But in the recent #64 pull request, we ditched
`cluster slots` in favour of `cluster nodes` which is faster, and fixed 3.2.0 compatibility issue in the process.


38 версия больше не поддерживается же, может дропнешь уже?


这个注释 写在大括号里面，他们是一个整体，在大括号的开始写上这个链表的初始状态或者在每个大括号里面重新初始化链表，我建议是后者，用c++11的语法，初始化很方便


Comment out a cut on number of features before it's effects on IDF are understood


Converted csv file


Ok... I'm working right now at the new version, it would be great to have
iPad styles too.
I separated the component logic from the styling so we can have basic/flat
styles and also styles for iPad/iPhone/Android and so on.

On Mon, Jun 6, 2016 at 8:29 PM Michael <notifications@github.com> wrote:

> Hey, no problem, thanks for creating this NPM plugin. It's been fun to
> use. I assume you want me to just submit this particular change to
> DatePickerField.js? My other commits involve style modifications for iPads.
> While I was going to clean all of it up and do it as a larger PR soon, I
> can just PR the single commit for now. Let me know?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/michaelarmfield/react-native-form-generator/commit/954c3fe0b6fa36e346a6d08f5ccc9c00ec2b6321#commitcomment-17762342>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAGjfm4thfHwzU-DD_QMHy51FeNXfCaQks5qJLtmgaJpZM4IuOed>
> .
>
--
Michael Cereda
http://michaelcereda.com


end


Looks like the validations are new since 0.0.15. Frankly I'd be happier if all the validations were turned off by default and you added them as you needed them. That's going to be a breaking change for people.


缩写一般也有讲究，一般是单词的首字母，不会从中间随便拿一个来缩写，当然也有那种，但是那是一种不好的写法。。。像list<int> 一般就是 li， ls是list<string>的缩写，尽量少使用缩写，除非在特别适合的场景，比如你这种场景就比较适合缩写，因为你这个链表没有什么特别的含义，只是一个链表而已


Tutorial


is `"status": "OK"` something that cannot be encapsulated in HTTP status codes?


Remove commented out code


Pq `.env2`? É isso mesmo?


missed that bullet in the docs.  Thanks, I see how it references updateMetaDataTable.sql in that class now.


I haven't dug in too far, but this change (`$body != ''` from `$body !== null`) has me getting back null because the StreamHandler creates a `$sink`, but doesn't [drain the stream to it until *after* the response is created](https://github.com/guzzle/guzzle/blob/master/src/Handler/StreamHandler.php#L108-L123) (so, when the response is created, casting the stream to a string results in an empty string).


This is causing a runtime.[22:17:00] Process mob caught exception processing /mob/living/carbon/human () (122,126,1): Cannot read null.overlays in life.dm, line 759


@chrisirhc Yes, projectionMatrix is currently used only for the perspective mode and will be removed eventually. If you don't need the perspective mode in your project, previous versions of deck.gl that depending on early version of react-map-gl should work. Although, as @ibgreen mentioned, react-map-gl at 1.0.0-beta-9 is required to provide the projectionMatrix for deck.gl to consume.


Thanks for explaining. I love Corvus.  :)


1_12_bugfixes (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2361 of 2427 tests passed, 0 test warnings)

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


I need to test this for a while to see if this causes any problems.


I think I got this crazy idea from looking at alter table logs.  But thats totally different if alter table is really creating a new altered table copying data over and dropping the old table.

And we won't have a problem unless two people are simultaneously running migrations?


@BerkeleyTrue typo error on your summary "remvoe old passport file" it should be "remove old passport file"


I don't have an email address for you.

fyi - the master branch for the stratux-setup repo is now protected which
means we need to push via a development branch first then open a pull
request to merge to master. For trivial changes just sign-off on the pull
request yourself with LGTM and perform the merge, but for non-trivial
changes wait for a review acknowledgment via an LGTM before performing the
merge to master.

thanks

On Mon, Jun 6, 2016 at 12:29 AM, Joseph Poirier <jdpoirier@gmail.com> wrote:

> wget doesn't complain about it on my linux laptop, I'll check in to it
> today
>
>
> On Mon, Jun 6, 2016 at 12:10 AM, Gediminas Ramanauskas <
> notifications@github.com> wrote:
>
>> :) We may want to do a proper fix instead of --no-check-certificate. On
>> the other hand not sure if it really matters.
>>
>> —
>> You are receiving this because you authored the thread.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/jpoirier/stratux-setup/commit/bf08be3b7106548cef4d37f147f38723e631d45a#commitcomment-17747793>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe/AAd7bgZVlAYZ8jytvRKJOnbqwQFch01fks5qI6vGgaJpZM4Iui1l>
>> .
>>
>
>


1_12_bugfixes (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


What nice friends you have there Corbin


This change will not affect the slider on the expert UI, which is still 0-40
![image](https://cloud.githubusercontent.com/assets/19563901/15844327/2df32438-2c1e-11e6-9484-ae0cb0a70e59.png)

The device is functioning correctly in the SmartHome UI now, after this was handled:
https://github.com/Z-Wave-Me/zwave-smarthome/issues/169


i


@glampr Hope you don't mind I cherry picked your commit back into the main repo. It's not so much a callback as a factory function, but it was incorrectly named. Thanks!


I'm sorry?


lol


yup


Correct.  We will need a single gateway for migrations to run against our environments; starting with a single person owning that, but perhaps moving to an IronOS pkg that is a oneshot execution which runs the migration, and that can be deployed alongside (and depended upon by) other IronDefense components.


not sure if this is correct


git command


ok


Any way we can get a `0.11.1` release?


I think this is the 4th time this commit is force  pushed :p


ここでセッションを使う必要はないのかも・・・


COMMENT HERE ABOUT ANY POSSIBLE FIXES


David Irina


Had a good laugh about this one, thanks! 😺


sup jtamsut


@cuducos aceito seu PR.


@bozana  thank you very much for the solution. I changed both files. What do I have to do next to get it working? Should I upgrade (editing config.inc.php and changing "installed = On" to "installed = Off" and running the web-based script, for example)? If so, this process will not erase any data, right? Of course, I will make a full backup of data files and database before try it.


Note to self, merge this commit to previous branch


Need is for successful compilation with BR_WITH_QTNETWORK=OFF


嘟嘟嘟


@rtg72a - The renewable annually and paid before anniversary are part of the current IDA handbook. They were asked in the exam as well.

Instead of deleting, should we add on in details?


I am running windows 10 and i have the net frame work but i run purevpn and the player wont open

how to fix


Issues mentioned in this commit (54ba31961a7009f4d69734fb877255d6fc728263):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (5cd129fd96d947bf01886a54efbc937f5a16c327):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (5835f39cc17b6c54894e977a6c08c4f0c16364a8):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (61c393c03f5d0f25f1821ba3a918454b562ebddb):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (6b242d21cbe7b200fef0575181b918a112fe63d2):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (1b50af984fe32aea3cfadb9c628692dfe657a9e4):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


Issues mentioned in this commit (d067b797575f9cfa6ff6e5ebb9f03149a3bd3b1d):

* [Issue #9085](https://bugs.mtasa.com/view.php?id=9085)


So, normally, please fork the feedstock and make changes in your fork and then PR them back, @pmlandwehr. The reason being all branches on the feedstock deploy. So, if there is an error, that will deploy broken packages. Just FYI. For instance, this change will fail as the checksum has not been changed. That being said, there won't be in any bad packages deployed


修改语言


You should use ES6 syntax


Would be good to add hyperlinks to Amazon.com or equivalent for more information on it or where / how to buy 📖


`t` should not be restricted to only `n1`.  The function should be defined for all `0 <= t <= n1`.


Looks good.


Overall assessment: you are killing it here! I'm going to point out a few things here and there, but understand that they are just suggestions. The way you've done things works well, and everything you've done is solid.


Dodanie funkcji potęgowania


Remember to split every passing test into different commits.


Changed clearPage case clear3 to work for divs with a flag in the form of a "canvas" class that i put on the cart canvas and the history canvas. Originally clear3 just cleared cartCanvas exclusively with a getelementbyid.


Also corrected image size bug for the results page. The image class was name wasn't correct.


Use spacing in between your argument assignments to make things easier to read


There is another bug with the portillus where id does it eventhougt doesn't have one


Specifying a `height` here is fine, but you might save yourself some pixel calculation headache by letting the `li` items that contain your menu options set the height automatically  (see comment on ln 114)


Consistent spacing between your return statement


Indent in one tab (2 spaces)


Diploma


Hello World. I am Alika !)


>Does INDI calculate time as well?
Yes and no, I think it is driver dependent. In Emod LST and local time  is
calculated with  gettimeofday.

>Or we could have our independent calculation.
Yes maybe, and alignment as well ...

2016-06-07 21:39 GMT+02:00 vicentperis <notifications@github.com>:

> Does INDI calculate time as well? If so, we should check its precision.
> Or we could have our independent calculation.
>
> V.
>
>
> On 06/07/2016 09:02 PM, Juan Conejero wrote:
> >
> > I have detected further problems that we have to address before
> > continuing. One of them is a conflict between mean and apparent
> > positions. I have implemented precession/nutation corrections to
> > convert to/from apparent positions when necessary, which I'll upload
> > later.
> >
> > Another issue is that OBJCTRA/OBJCTDEC coordinates reported by the
> > INDI server are not very accurate in terms of precession applied to
> > convert from the date of observation to J2000.0, as far as I have
> > tested them. IMO we should replace these coordinates completely with
> > new ones computed using accurate models, i.e. the IAU 2006
> > precession/nutation model. I have this already implemented now, as
> > noted above.
> >
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub
> > <
> https://github.com/PixInsight/PCL/commit/0fb8b5856146623515bc1132bf9796eed7459660#commitcomment-17776250>,
>
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe/AR1Mq_aCO2fuAgkbquN3POWVAmglIOqRks5qJcBYgaJpZM4IvHwD
> >.
> >
>
>
> --
> ****************************************************
> Vicent Peris Baixauli
>
> Astrofotógrafo
> Observatori Astronòmic - Universitat de València
>
> Edifici Instituts d'Investigació.
> C/ Catedrático José Beltrán, nº 2.
> E- 46980 Paterna (Valencia)Spain
> Tel. +34 96 354 3819
>
> OAUV: http://observatori.uv.es
> Astrofoto: http://astrofoto.es
> DSA: http://astro-photographer.org
> PixInsight: http://pixinsight.com
> ****************************************************
>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/PixInsight/PCL/commit/0fb8b5856146623515bc1132bf9796eed7459660#commitcomment-17776873>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACDypiNp_EW7KjjTJc0aYzNyU7miL53Bks5qJcjygaJpZM4IvHwD>
> .
>


@lileeyao 给我个example，你说的是cache一个object么？


Set a top and bottom padding of some equal value on each list item like this:

```
.nav-items li {
    padding: 40px 0;
}
```

This will make the height of your `<nav>`  be 40px + whatever the height of the text is + 40px. This keeps you from having to set a height for your  `<nav>`. Then to center the `<a>` you used for the logo and floated left, just add a `padding-top: 40px` to it.


I don't think the `bitpack` call is necessary as `Compat.repeat` returns a `BitArray`.  The use of `bitpack` causes a deprecation warning in v0.5-dev.


I'd be explicit about the type of the join: `INNER JOIN`.


BIG TODO: This should be fixed properly


This causes a syntax error on Windows.


2240_CmToIn (macumber) - i386-Windows-7-VisualStudio-12: OK (2405 of 2423 tests passed, 0 test warnings)

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


I would have floated this right rather than used absolute positioning since you already floated the `<a>` to the left. You can create space between the top of the nav element and your list by applying `padding-top` to the list items (see comment below). This has the added advantage of making each item larger and easier to click on smaller screens.


Yes screenhero or skype or whatever, all fine for me ... any channel like Skype where we can continue this discussion, don't want this to be public ;-)


Hey guys
what about Bots? When i spawn them in ( ogarunlimited.io)  minion 1 100 <invader>.   Most of them stay in place, I can split them and eject mass from them, but i can't control them


Your could shorten your return statement by writing. `return n % 2 !== 0`


Why would you ever use this?  You could just call `comparator` directly.  I could see this being:

```javascript
if (comparator && comparator(leftHandOperand, rightHandOperand)) { return true; }
```
(and then passing the comparator around to all of the recursive bits), such that this was additional chance for equality.  I could also see this as being only used for the `default:` case in the switch.


update your server


Sure you want to delete the console messages?  They are info about deleting
records...

// action methods
_deleteItem(item, message) {
  if (window.confirm(message)) {
    item.destroyRecord().then(function() {
      console.log('+-- deleted contact ID:', item.id);
    }, function() {
      console.log('+-- delete contact failed');
    });
  }
}

Do you want a window message if the record fails to delete?


Remember to use single quotes `' '`. JS convention


I'm pretty sure we had a point/point version of SquaredDistance at some point too.


building caf for 8960 on cm13 has been achieved.
although calls can listened...this commit doesnt seem to correct it.any thoughts :)

I am on jf device.bt stream and such works fine


Sorry I didn't see this until now, no comment...

But seriously, it must be something else, this chunk of code is only executed if platform=rpi3


@cklosowski receives is misspelled. so is 'purchase'.


The following doc(s) have been deployed to the [1626reduce](https://1626reduce-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/b6183b622502bd9b396b39dc78492c667cdb8668/app/config/sculpin_site.yml)
- [docs/reducing-large-repos](https://1626reduce-static-docs.pantheonsite.io/docs/reducing-large-repos)


What do i write ? Update all?


@cklosowski should be Due to `its` consistency... not `it's`.


@cklosowski should be `bypasses`


Can be removed.  Done with 6b39b4044999f104ef699bc19a0b78ebabf38c89

On Tue, Jun 7, 2016 at 7:26 AM, Ralf Becker <notifications@github.com>
wrote:

> @nathangray <https://github.com/nathangray>: Line 347 uses
> (Api)Framework::set_onload, which does not longer exist, as it does not
> work using our CSP, ist that really used, or can it be removed?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/EGroupware/egroupware/commit/1c24585eab0d76307a44db7cfe63a932819945a1#commitcomment-17770039>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AJ7PHjUyfQwJyFg1U5Spbj7aDYHom5aUks5qJXGTgaJpZM4Iv60W>
> .
>


* i hate mac


I don't see anywhere that's actually doing memoization.


Thanks, I can change this assuming someone else has not already put this portlib upstream.


Remember to use single quotes ` ' ' `. JS convention


Seriously...


@cklosowski could use `edd_get_label_singular()` instead of "products" here.


DO NOT USE THIS, CODE WORKS ON MAC BUT CACHE IS BROKEN... EVERYTHING BROKEN


shop


Thanks, have a good one. Cheers :)


@cklosowski certain is misspelled


Asus Chromebook, crouton already installed and am in developer mode. stuck on stupid!


Nope, we need #!/usr/bin/python or it will cause problems


Grab the `mapEquals` routine from my patch.  It's O(n^2), but at least looks like it works.


`magic_spiralize` is marked as global only though in FDMPrinter. If you want to support this setting per mesh, it should no longer be global only there too, shouldn't it?


120796739_ObjListAutoPopulate (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1654 of 1654 tests passed, 0 test warnings)

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


This is outstanding. Very nice use of media queries! The breakpoints are just right.


Remember to split up each passing test into its own commit.


fait bizarre de plus avoir de test sur les 500 : le fail est du à l'exécution de request pas à l'exécution du callback
il faudrait plutôt trouvé un moyen de rentrer dans [la branche](https://github.com/wadouk/ya-request-builder/commit/b6aed392b51135620ec0814c26d36cb1444938e7#diff-6a14dff043e68e144b0c29113e3c08f9R121)


This works fine, and some would argue that it is more semantic than the alternative, but I personally think it looks clean er to just use an `<a>` tag here, and use CSS to make it look like a button.


Don't forget to use single quotes instead of double. JS convention. Look below for other cases


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

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


ne, to neni chyba - ma to byt replika, ktera proste sepne agro, pokud ji hrac zvoli. Nema to nic spolecneho s ochrankou, ale s tim, ze si proste jakoze vsimne tve zbrane.


comment


So if we don't like </br> should we override <h3> or <container-fluid>?

On Tue, Jun 7, 2016 at 12:12 PM, Stan Smith <swsak2hi@gmail.com> wrote:

> Sure you want to delete the console messages?  They are info about
> deleting records...
>
> // action methods
> _deleteItem(item, message) {
>   if (window.confirm(message)) {
>     item.destroyRecord().then(function() {
>       console.log('+-- deleted contact ID:', item.id);
>     }, function() {
>       console.log('+-- delete contact failed');
>     });
>   }
> }
>
> Do you want a window message if the record fails to delete?
>
>


no ! i allways use zsh
but zsh is just for prompt consol, script use only shell in [shebang](https://en.wikipedia.org/wiki/Shebang_(Unix)) ; for you is :

    #!/bin/bash


presence je jen nazev discipliny, nijak to nesouvisi s tim slovem v replice.


Remember to split up each passing test into it's own commit


Oh, I missed this, thx. Reminds me to add [strict](https://github.com/deepmind/strict) at one point to make lua code more robust.


`Number` is a reserved keyword of JS. You can use another argument name like `n` below.


So we're now possibly writing temperatures at or lower than 0? Probably for the best; let the front-end and firmware decide on the limits for temperature.
0 degrees Celcius is an arbitrary temperature anyway. We SHOULD be working in Kelvin! :stuck_out_tongue:


Nope, we need #!/usr/bin/python or it will cause problems


Mist, ich habe an manchen Stellen versucht, die Bibliographie-Liste mit `\parbox`zu simulieren, habe es aber nicht überall durchgezogen (auch nicht einheitlich) und es ist freilich auch etwas zusammengeschustert so. Wenn du das noch anpassen/rückgängig machen willst, mach. Ich schaff es nicht mehr vor dem Urlaub.


This would decrease performance rofl


For these I would/will definitely add unit tests to make sure it behaves as expected.


adding `clear: both;` here fixes a lot of issues on the page. Like we discussed in slack, when you use a float, you have to clear it, otherwise everything underneath it will go haywire.


Added image "mountain"


Indent this return statement one tab ( 2 spaces ) in. Also, spacing between each character in the equation.


With this new feature, you can't disable the Auto-Coolant once enabled. Not sure if that's on purpose or not.


Makes sense!


Refs #47


Don't forget to use single quotes instead of double. JS convention.


This is really well-done. Nice use of float to create the layout and `calc` to let the browser dynamically calculate the widths of the columns.


Check indentation


The following doc(s) have been deployed to the [1631ampne](https://1631ampne-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/new-relic-analysis](https://1631ampne-static-docs.pantheonsite.io/docs/new-relic-analysis)


Not that it's a huge issue (just bugs the hell out of me) but could default be the last case please? :P


XP


object assign est utilisé dans [karma](https://github.com/wadouk/ya-request-builder/blob/master/karma.conf.js#L1) ([cf build](https://travis-ci.org/wadouk/ya-request-builder/jobs/135558189#L356))


It was cool how when I applied the restrictive annotation, mypy was quick to tell me I was doing it wrong due to another part of the codebase passing in dicts instead of ints.


Added command line arguments and IO exceptions


Or an empty paragraph with 2em bottom margin?  Doesn't sound better.

On Tue, Jun 7, 2016 at 12:23 PM, Stan Smith <swsak2hi@gmail.com> wrote:

> So if we don't like </br> should we override <h3> or <container-fluid>?
>
> On Tue, Jun 7, 2016 at 12:12 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>
>> Sure you want to delete the console messages?  They are info about
>> deleting records...
>>
>> // action methods
>> _deleteItem(item, message) {
>>   if (window.confirm(message)) {
>>     item.destroyRecord().then(function() {
>>       console.log('+-- deleted contact ID:', item.id);
>>     }, function() {
>>       console.log('+-- delete contact failed');
>>     });
>>   }
>> }
>>
>> Do you want a window message if the record fails to delete?
>>
>>
>


Should probably set this only after the `GameStartedServerEvent` in the event an admin wants to use a forge mod added `ItemType`.


werkt dit?  In de context van een server-based applicatie is het wel vreemd om opeens iets van de console in te lezen..


@IamgicalGamer, no, this is unnoficial, plus changing the name is just too complicated.


Hmm I think you might want to delete the `float: left;` line here. You have teh right idea: we want the `<ul>` to be on the right side of the `<header>` and the `<h1>` to be on the left side, and we can use floats to do that, but we'd want to do something like this:

```
header h1 {
    float: left;
}

header ul {
    float: right;
}
```
See the difference? We float the entire list and not just each `<li>` inside the list.

Remember that we'd also need to clear or contain the float, just like we did with the overall layout.


Yeah, this is actually a great case for how mypy can really help with refactoring!  I'd certainly forgotten that detail.  Will put in my list of examples for the mypy+Zulip blog post :)


Remember to split up each passing test into it's own commit.


Fix #1


huh?  Is dit om de input te verbergen ofzo?  Is het dan niet handiger iets als [Console.readPassword](http://docs.oracle.com/javase/8/docs/api/java/io/Console.html#readPassword--) te gebruiken?


This is a bit too narrow. Think about increasing the width to 75-80%. To center the text inside the column, remember to add `margin: 0 auto;` here.


Common convention of indentation is one tab ( 2 spaces ). Might need to change settings if it's set to 4


@mikelees2 sorry, but that logo does not really suit IM. It is kind of... Not too suited. I love what you did with the Minecraft in the background, and the sparkly underline, but not that "IM" part, nor do I like the font too much. ImagicalMine is supposed to be fun, which is why font is so important


I'm going to fold the AO gates into the larger clock driver. They do not seem to actually be on any kind of special bus, though the ones you enumerated here do control gates for devices on the AO bus.


@kgonzago, what you think about this?


De commit heet "Deleted gitignore file" maar hij voegt tevens twee html files toe...


Overall, the HTML here is really solid--semantic, organized, and not a lot of unnecessary elements. Good job.


This could have been a bug in the previous implementation: Is it okay to first enqueue the function (`runFunction()`) and only then call `start()` below? The engine might execute the function already before it has been started.


I will squash the gate clocks into my gxbb clock controller driver. They do not need to be separated and the gate regs are in the middle of all of the other clock regs, so I do not think they are special. Sorry to steal your code, but I will give you proper attribution in the commit log!


Thanks for the feedback! For the currently-in-development version of the SDK, we're moving to an NPM-based approach for more platform compatibility so this problem should hopefully go away. Feel free to switch to the dev branch: `git checkout dev`

And run: `npm install && npm start`

Hope that works better for you!


@Kaixhin btw I find it a bit suspicious that `Atari` tops out below 0.9 at catch both with ER and async. I would expect it to get to 1 easy given the problem is so not difficult with default settings at all.


Zijn er 2 connection nodig? Een 'oldDatabase' ?


4943-EMS-Trap-Unitialized-Variables (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2211 of 2213 tests passed, 4 test warnings)

 * 3 tests had: ERR diffs.
 * 1 test had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: Table small diffs.

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


@richfitz any idea why this call to remake would be [throwing an error on appveyor](https://ci.appveyor.com/project/dfalster/baad/build/1.0.1#L354)?


Excellent!  I've not tried it yet, but that sounds much better.


Note that these settings are all global only at the moment. I know BagelOrb's argument for turning `global_only` into something like an enum (per object, per extruder or global only) and I agree, but I think the front-end is not ready for such a change at the moment.


You never created a PR for this commit. I don't believe it's still relevant, should we delete the branch?


Все блоки div с классами line помести в другой блок, который будет объединять эти три дива по смыслу. Это может быть просто div у которого будет класс с каким нибудь говорящим за себя названием. Объединение элементов по смыслу в более крупных блоках является одним из ключевых моментов для грамотной и красивой верстки. Например, в самом начале ты написал div с классом result куда поместил резалт для X или O. По логике можно бы было сделать какой-то общий тег типа results куда бы просто поместить div'ы для верстки резалтов. Пытайся проектировать разметку c самого начала от общего к частному. Я постараюсь выслать тебе пару примеров на почту.


do you want `sendFile`?


Ohhh,sorry


oh sorry ...my mistake..no it wasnt too fast...it was normal...
i just wanted to say that the time between my click on PLAY and the speaker's output was too small...
quick responce ...its perfect ..i wish i could fix calls too :/


@dobra-dobra dzięki i prośba o więcej :) !


@DarkWav because I want people to actually have to use the quicklinks!
I know it sounds silly, but plz leave it, thats why the quicklinks are there


I was thinking other CI tasks might come along but it can move. The
create_branch.sh script in the sub module will need fixing up to

On Wed, Jun 8, 2016 at 12:05 AM, Andrew Koroluk <notifications@github.com>
wrote:

> is there a reason this needs to be in the ci subfolder? Why not just keep
> it at the root like the other submodule?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/angular-fullstack/generator-angular-fullstack/commit/d1ca75dd6bcee68db1b3bb3f5884b731425b20e4#commitcomment-17770785>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACjgyO8iRqOQtNRpnBtxrKdGLn-OzdTrks5qJXrFgaJpZM4Iv9bB>
> .
>


Oh, nevermind this actually seems very close to what you want.

```
res.writeHead(200, {'Content-Type': 'image/gif' });
res.end(img, 'binary'); // Include encoding
```


The following doc(s) have been deployed to the [1626reduce](https://1626reduce-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/b6183b622502bd9b396b39dc78492c667cdb8668/app/config/sculpin_site.yml)
- [docs/reducing-large-repos](https://1626reduce-static-docs.pantheonsite.io/docs/reducing-large-repos)


Created new array to store products into order history. Cart array items get concatenated into order history array whenever order button is pushed. Also created message on order history page for when order history array is empty.


I'm trying to keep minimal diff


```java
public class Map<T> {
    private T t;

    public void set(T t) { this.t = t; }
    public T get() { return t; }
}
```


@wadouk bizarre ça fail pas chez moi. On devrait utiliser la méthode `Object.assign` native vu qu'elle est dispo dans node


I stole the heroku generator as a starting point. I presumed it was if the
step failed as it does a dist build before copying the Dockerfile so it can
bail out

On Wed, Jun 8, 2016 at 3:07 AM, Andrew Koroluk <notifications@github.com>
wrote:

> Where's this.abort from?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/angular-fullstack/generator-angular-fullstack/commit/0674ad06e5409a61a5a3cd17892b860516fa4436#commitcomment-17774285>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACjgyLKnb0Anbh9y4b0_dLYxgcFfZt_Dks5qJaVrgaJpZM4IwJeT>
> .
>


see also
[https://docs.google.com/document/d/1Qe5IhxVyG_pGUFryHCqjrrZ_YvifaiM4y0RzNzZsqgc/edit?usp=sharing]


Hitting the submit order button now clears the cart array and updates the cart icon. Functionality for updating the cart icon badge number is now put into a function.


Merk op dat je hier weliswaar print dat de driver niet gevonden wordt, maar daarna gaat het programma vrolijk verder. Maar in zo'n geval zul je dan verderop wel onverwachte fouten gaan krijgen, toch?


Wat is de waarde van con als een SQLException was opgetreden? Is dat wat je verwacht?


I see the heroku module creates it. I'll take it out

On Wed, Jun 8, 2016 at 6:43 AM, Jeremy Marshall <
jeremystuartmarshall@gmail.com> wrote:

> I stole the heroku generator as a starting point. I presumed it was if the
> step failed as it does a dist build before copying the Dockerfile so it can
> bail out
>
> On Wed, Jun 8, 2016 at 3:07 AM, Andrew Koroluk <notifications@github.com>
> wrote:
>
>> Where's this.abort from?
>>
>> —
>> You are receiving this because you authored the thread.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/angular-fullstack/generator-angular-fullstack/commit/0674ad06e5409a61a5a3cd17892b860516fa4436#commitcomment-17774285>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe/ACjgyLKnb0Anbh9y4b0_dLYxgcFfZt_Dks5qJaVrgaJpZM4IwJeT>
>> .
>>
>
>


@nopestack plz respond fam ;__;


@gabrielle TODO - gabrielle please improve this listing


I think reading from DB should be part of the presenter and not view


PostMessage with the given queue is a strange pattern.


Sure you want to eliminate key=@index?   see
http://emberjs.com/api/classes/Ember.Templates.helpers.html#toc_specifying-keys

On Tue, Jun 7, 2016 at 12:30 PM, Stan Smith <swsak2hi@gmail.com> wrote:

> Or an empty paragraph with 2em bottom margin?  Doesn't sound better.
>
> On Tue, Jun 7, 2016 at 12:23 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>
>> So if we don't like </br> should we override <h3> or <container-fluid>?
>>
>> On Tue, Jun 7, 2016 at 12:12 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>>
>>> Sure you want to delete the console messages?  They are info about
>>> deleting records...
>>>
>>> // action methods
>>> _deleteItem(item, message) {
>>>   if (window.confirm(message)) {
>>>     item.destroyRecord().then(function() {
>>>       console.log('+-- deleted contact ID:', item.id);
>>>     }, function() {
>>>       console.log('+-- delete contact failed');
>>>     });
>>>   }
>>> }
>>>
>>> Do you want a window message if the record fails to delete?
>>>
>>>
>>
>


Both migrators seem to have the same number, `71492`, and that is not allowed.


You don't need `+""` here


IE8 is throwing "Expected ']' in regular expression" (both old and new expression here)

![image](https://cloud.githubusercontent.com/assets/339763/15874084/98d12550-2ccf-11e6-88b1-0c44c6f7f97f.png)


@remotevase https://inactive-to-reactive.github.io/new.html


Note: this causes a stack overflow with circular objects:

```javascript
var a1 = {}
var a2 = {}
var b1 = {a: a1}
var b2 = {a: a2}
a1.b = b1
a2.b = b2

console.log(eq(a1, a2))
```

Suggestion: two passes.  First one checks for equality and adds a Symbol entry to every object touched.  Second pass removes the Symbol entries.  This fails on browsers that haven't implemented Symbol, of course.


If u want dat as home screen let me know


@gabrielsaldana  TODO -   please improve this verification


MessageQueue?


навіщо оті 2 рядки
a=2;
b=1;
Вони ж затирають результат роботи програми. Видаліть їх - і буде працювати. Алгоритм працює, але дуже запутано


@antrim

I've fixed the migrate.php script, so that all existing migrations now succeed.

(1) calendar_bounds.calendar_bounds_id is automatically assigned from a SEQUENCE.

(2) The blocks query was being run multiple times, I've removed the duplicate.

(3) There was a sql syntax error with missing FROM. I've added that in.

(4) Some stops columns were designated NOT NULL, when the data being migrated has nulls, I've removed that constraint.

(5) Refactored the code to remove nested common table entries (WITH clauses), instead I define CTE's used by later queries prior to, but at the same level as the queries which use them.


is this cast necessary ?


I'm fine with eliminating the </br> and just letting the <h3> and <div>
butt up without whitespace.  That keeps the bootstrap design in place.

On Tue, Jun 7, 2016 at 12:46 PM, Stan Smith <swsak2hi@gmail.com> wrote:

> Sure you want to eliminate key=@index?   see
> http://emberjs.com/api/classes/Ember.Templates.helpers.html#toc_specifying-keys
>
>
> On Tue, Jun 7, 2016 at 12:30 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>
>> Or an empty paragraph with 2em bottom margin?  Doesn't sound better.
>>
>> On Tue, Jun 7, 2016 at 12:23 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>>
>>> So if we don't like </br> should we override <h3> or <container-fluid>?
>>>
>>> On Tue, Jun 7, 2016 at 12:12 PM, Stan Smith <swsak2hi@gmail.com> wrote:
>>>
>>>> Sure you want to delete the console messages?  They are info about
>>>> deleting records...
>>>>
>>>> // action methods
>>>> _deleteItem(item, message) {
>>>>   if (window.confirm(message)) {
>>>>     item.destroyRecord().then(function() {
>>>>       console.log('+-- deleted contact ID:', item.id);
>>>>     }, function() {
>>>>       console.log('+-- delete contact failed');
>>>>     });
>>>>   }
>>>> }
>>>>
>>>> Do you want a window message if the record fails to delete?
>>>>
>>>>
>>>
>>
>


Should we store std::string in the vector for simplicity?


Hmm, it's quite interesting. I'll think about it!


It does not make sense to call this for outgoing messages.


`number` is close to the JS keyword `Number`. Best to use a shorthand like `n` or `num` just to stay safe with your argument names.


Do not set max results as 1 and then get a unique result. just get the first element. It's a small thing but it's better.


New API docs are online at http://openface-api.readthedocs.io/en/latest/openface.html#openface.AlignDlib.align_v2


i can edit itand put news stuff etc.


I love HTML


Issue #86


Looks like it should be called DispatchMessages


This comment, while technically correct, is a bit out of date now since it is the retraction config for everything now.


Lets not leak this further - you again infect env with stl.


look now...it expands across page (full screen)


That looks way too big. Crop it too just the words - but leave some space in between the border and the words.


Move implementation into cpp


This shouldn't return `[]`, but `true` or `false`.  I think  you want:

```
if (!objectIs(leftHandOperand.done, rightHandOperand.done) { return false; }
```


Also, 71492 and 71493 are already taken.


Hmm, isn't it 71-2-92 and 71-4-92?


ola


Малось на увазі, що файл буде текстовий (не тільки числа у вигляді цифр). Тому не воно


MessageQueue


I'm still working on some things. Unlike some other people around here I don't quit. I fine tune and leave the critics in the dust wiping away there tears.


Can we do std::string ?


edit it @remotevase on ur compuer than upload than put link of dat


Awesome!


ok, ich schau, was ich vereinheitlichen kann.


Indeed, but both should be changed to 71494 and 71495.


Yeah, but I will do it in a few more mins. For now, busy


Probably want to make this happen only when you ask it with a cmd line flag.


I don't quite understand why we're not counting the retraction times for the first path. Could you elaborate?


Oh,and also, I mean, serious consideration because it is pretty darn cool


As far as I can tell, this is the only destructive thing you're doing to operands thus far.  The generators won't be in the same state after you read from them.  If you want to stick with this, please document the inconsistency.  Another approach would be to check if the functions are the same, and just add the done test on the line above.


Thank you for the code review.

Yes, map generation comes to an end...


(nobody tell Larry about the missing semi colon)


може if(i == j) ?
а то щось всі  1 виходять


KEK


lol


lel


yes it may work


Fixed https://github.com/wadouk/ya-request-builder/pull/12


Lol! I love it so much puting in my new STar Network App!


We should print a little more info about the AssociationSet.  For example dataPath


thank you


Found em on Codepen.io


Oh, ur using it in ur SW App? Okay, then we won't use it here! Uniqueness = good


@gabrielsaldana We should verify a little more info about the AssociationSet.  For example number of tuples in graph


Yes . For example if DynamoUI is open I will not allow certain operations in the external component .
Also , the model state tells me how should I Execute a DynamoRevit command and what parameters are needed. For example if dynamoModel is started UIless already , I will use the existing model if I want just to run a graph . If it's not started I'll have to start it first before anything else .


You removed the negative here! Should be:

    && !paths[path_idx + 1].config->isTravelPath() // next move is extrusion


Can we at least use C++ delete?


why not use the local `pos` variable as vector for `render.ComputeLightning` too?


`sort` is unlikely to be helpful here.  For example:

```javascript
var b = [ {a: 2}, {a: 1}]
b.sort()
```

doesn't change the order of `b`.  My `setEquals` function is O(n^2), but will work for this case.


Add the ability to `include` and `extend` jade modules


I'm not sure if this has some consequences: @Sharad what do you think - is it safe to not start the ViewModel for Dynamo Revit or was this necessary?


Make an enumarating class with these strings as attributes?
Then pylint can tell us when it's typoed.


Not offhand; looks legit to me. I presume that there are different escaping/quoting rules in windows cmd.exe to bash? Try swapping single and double quotes perhaps?  I can try and have a look at work, but my windows computer is in disarray at the moment.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.93.33326a7831998f9079836a6d3f5807b360351f76](http://build.palaso.org/viewLog.html?buildId=73388&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1252, ignored: 17 Build time: 00:04:52


I see , so actually a combination between a valid dynPath and Automatic set true will result in the execution of the graph . Yes , I have graphs that will always be on manual mode but I need to execute them on demand from the external component .


So? @keithkfng?


Hehe, your comment message scared me!


@casey-collab


@sharadkjaiswal I think we should be sure to discuss these changes in a PR in the context of our API that we discussed the other day. Any thoughts on the addition of public properties and constants?


We've agreed to revert the change to 299.


Тут буде помилка \V такого спеціального символа не існує. З слеша починаються спеціальні символи: \n, \t тощо. Навіщо там той слеш?


This method posts to a queue - and we have two of those. I renamed it to "AppendMessage".


Done.


That is only 1 game per every 2000 games. Also, statistical error is much lower after 174K games. This is [-3,1] test like any other.


I've already added the CSD portion of the changes which are what are preventing us from having voice working. I haven't been able to test tho I can't be 100% sure. It is linked somewhere in the comments for the original commit.


I would prefer to have queues of the same types - and carrying String16 allows me to do less copies (as there is no need to convert char array to std::string when passing a message from IO to main threads).


From what I see here: http://stackoverflow.com/questions/2422798/python-os-path-join-on-windows , it is preferable to just use the `os.path.join` function and not hardcode either slashes or backslashes.


This is great idea. Good luck with this.


Not sure I understand this comment...


hi hi


Based on your input above I guess I should check if we are in automatic mode , and if we are and the dynPath is valid I should let the model "run by itself" and not force it with modelToRun.Run() ?


I know... I'm working on a private implementation right now - so this will go away soon.


done.


@arhuertah @afeld If this request is still actual, could you please create a usual pull request according this repository rules? If in doubt or something does not work, please drop us an email to support@jetbrains.com saying that you can not add your university domain to SWOT database on GitHub. Thank you!


Sure, but it would add more memory copies.


I'm wondering if authenticate should create the code at all. Authenticate shouldn't be creating objects as a side effect.

Seems to me that all this method should to is to validate any passed code and return user if authenticated. This seems to be also handing username/password validation which should be handled by `django.contrib.auth.backends.ModelBackend`


There is more confirmation here that it may be better just to use forward slashes (or preferably `os.path.join`), despite the Windows environment. Probably easier to not make mistakes around escaping, etc.


ah i see.
you are right. pacli-simple runs in bash, because of the first line of code.

i have just spend around 20min reading the source code of pacaur... interesting.


I've also moved initialization of dynamoViewModel here since I believe is not needed if you don't show the UI . Am I correct ?


which comments ? cant get u


I replied to comments. The new CL is here:
https://github.com/eugeneo/node/commit/b6fe1cd6776cf51aadf23d92f30ddbabf5931473

On Tue, Jun 7, 2016 at 12:25 PM Eugene Ostroukhov <eostroukhov@google.com>
wrote:

>
> https://github.com/eugeneo/node/commit/9a009a8e5443295aec40149260af308e6c0b1baa
>
> On Mon, Jun 6, 2016 at 12:37 PM Eugene Ostroukhov <eostroukhov@google.com>
> wrote:
>
>>
>> https://github.com/eugeneo/node/commit/165564317ffaa975469ef90da76d548374c89d6e
>>
>> On Mon, Jun 6, 2016 at 12:26 PM Pavel Feldman <notifications@github.com>
>> wrote:
>>
>>> I don't think so. We should not accept connections while connected.
>>>
>>> —
>>> You are receiving this because you authored the thread.
>>> Reply to this email directly, view it on GitHub
>>> <https://github.com/eugeneo/node/commit/dd65067f07d2dd269a3f84f22bc98e722618e13c#commitcomment-17758504>,
>>> or mute the thread
>>> <https://github.com/notifications/unsubscribe/AARkrUKHu0-cquMtKM66YjxCKk2PtuHhks5qJHRUgaJpZM4IvOLr>
>>> .
>>>
>>


I'm assuming you want to revert this commit eventually?


@valendesigns yeah, I think it's not relevant anymore because of #34.


I am not sure, see note above. I will talk with @sharadjawiswal about this.


The results file (for the expanded search) and the one created for the dendrogram had the exact same data, except one was "tab separated" and the other was "comma separated".
I unified the process so it will work with just one file, there was no need to keep them both.


Thanks.  Unfortunately it failed
http://tests.stockfishchess.org/tests/view/5755c0000ebc59029919b278


This ended up working in preprod?


I think this syntax will fail on our build machines using an older version of visual studio.


The `minimum_value` for avoid distance in FDMPrinter is 0mm. This effectively makes it 0.01mm. Wouldn't it be better to initialise it to 0?


This solution has a small drawback: In case the MAC seen on the wire is 00:00:00:00:00:00, it won't be propagated. To my knowledge this should not happen but does happen from time to time (e.g. in case of misconfigured devices). In contrast to the solution with an extra pointer it also introduces some `memcmp` and `memcpy` calls, which might be a tiny overhead, aren't they?


https://github.com/eugeneo/node/commit/6daa4539ed558f5820a6e7f261480f338acf3daa

On Tue, Jun 7, 2016 at 1:58 PM Pavel Feldman <notifications@github.com>
wrote:

> Can we at least use C++ delete?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/eugeneo/node/commit/1694f4b85a81cb78601f9d59f6bfa89c091a7ac6#commitcomment-17778198>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AARkrbIt_q-eRVAOr7Q8vIuIg-NiHrhSks5qJduFgaJpZM4IwLMe>
> .
>


Cool!


GroundDomainUpdate (mitchute) - x86_64-MacOS-10.9-clang: Build Failed

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


there's assertedly 100% coverage, it'll raise under test


Are the version updates _required_? I think we'd like to keep the libraries themselves (not necessarily the examples) bound to as low a version as we can.


Nah the change that I copied over from the one cyanogen did for the legacy hal was here

http://review.cyanogenmod.org/#/c/145218/

So that would most likely fix calling. Also the lastest hal stuff is on the gerrit the gist might be a bit outdated. http://review.cyanogenmod.org/#/c/144754/


The whitespace seems off here.


https://github.com/eugeneo/node/commit/73ffd51108b6ac04b8f71d547f959ef2abdc38b4

On Tue, Jun 7, 2016 at 2:08 PM Eugene Ostroukhov <eostroukhov@google.com>
wrote:

> I replied to comments. The new CL is here:
> https://github.com/eugeneo/node/commit/b6fe1cd6776cf51aadf23d92f30ddbabf5931473
>
> On Tue, Jun 7, 2016 at 12:25 PM Eugene Ostroukhov <eostroukhov@google.com>
> wrote:
>
>>
>> https://github.com/eugeneo/node/commit/9a009a8e5443295aec40149260af308e6c0b1baa
>>
>> On Mon, Jun 6, 2016 at 12:37 PM Eugene Ostroukhov <eostroukhov@google.com>
>> wrote:
>>
>>>
>>> https://github.com/eugeneo/node/commit/165564317ffaa975469ef90da76d548374c89d6e
>>>
>>> On Mon, Jun 6, 2016 at 12:26 PM Pavel Feldman <notifications@github.com>
>>> wrote:
>>>
>>>> I don't think so. We should not accept connections while connected.
>>>>
>>>> —
>>>> You are receiving this because you authored the thread.
>>>> Reply to this email directly, view it on GitHub
>>>> <https://github.com/eugeneo/node/commit/dd65067f07d2dd269a3f84f22bc98e722618e13c#commitcomment-17758504>,
>>>> or mute the thread
>>>> <https://github.com/notifications/unsubscribe/AARkrUKHu0-cquMtKM66YjxCKk2PtuHhks5qJHRUgaJpZM4IvOLr>
>>>> .
>>>>
>>>


Removed the `* 2`?


wrong indent


Can I leave that to you? Empirically my current test on Beam Rider with the latest code seems to be working, and Tom Schaul said that my plot of Frostbite scores "looks more convincing" (compared to the old results which were only reaching tuned DDQN levels).


trying now and reporting
all rest has be done


Похожих борд все равно нет. Давно надо было.


I missed the response to this one.


Hmm, the build announces itself as a 'trusty' dist, but the apt sources are from 'precise'


Mauvaise idée le . à la fin de l'URL car si on le copie colle avec l'URL ça ne fonctionnera pas


Суть коммита: там изменился класс тредов.


Hi,

I'd then use the xml format. Is the issue that you need to loop through the xml multiple times?

-brady


They initially pushed it to PM


I think here what should be printed is

```
print(r.t+dt, r.integrate(r.t+dt))
```

`r.integrate(r.t+dt)` is the value at `t+dt`. I don't see the point of printing `r.t` alongside with `r.integrate(r.t+dt)`.


Et plutôt que "The app" peut-être dire "ezmaster" pour éviter toute ambiguïté avec les "app" au sens ezmaster


can I ask... what is the advantage of using the bourbon mixin +order over the regular CSS? I'm new to Jekyll and sass, what's the reason for doing this?

Thanks!


Mettre l'exemple en commentaire je pense (avec un #)


>= non ?


">= 1.7"


Is it necessary to precise `browser` in `import 'angular2-meteor-polyfills/browser'` the meteor modules system is supposed to look at the browser field in packages.json ?


@Ruben0406 graag translations toevoegen ipv hardcoded


lets translate these:
``````
$tpl->assign('ActiveTitleYes', xl('Yes') );
$tpl->assign('ActiveTitleNo', xl('No') );


Added using to graphics to ensure resources freed


@lake4790k I've had a suspicion that it might be the model architecture that's the reason (because all of these adjustments top out at ~0.9), but haven't investigated further...


Remember to maintain consistent spacing between your arguments and equations. Space them evenly to make it easier to read


Pour éviter ces merges qui arrivent si on a comité en locale alors que des modifications distantes sont présentes et qu'on a part exemple oublié de faire un git pull avant de commiter.
Ok peut faire un git rebase avant de git pusher ses commits locaux.


:+1: nice find @gourlaysama!


looks like a type with 2 $$ here


"+     Your ImagicalMine Server needs Visual Studio C++ Redistributable 2015. "
 - Only if the server OS is Windows. Useless on Linux :)


I just had time to look again at this.
The uuid_js.go file only contains the necessary code in order to be used with gopherjs build and work with the goa client generated files.
goagen requires the NewV4(). So this method isn't exported in the uuid_js.go if we want to add it by copying we actually have to pull more code, and depend on the crypto/rand package, and from what I understant it would increase the generated javascript code.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.94.44c7b566beeb0421a9c85305516b3c6a8b5dec23](http://build.palaso.org/viewLog.html?buildId=73390&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1252, ignored: 17 Build time: 00:05:16


I've tested this script and didn't get that error so not sure that error is specific to this script


Who doesn't love a little ascii?


Use single quotes when declaring a string. JS convention. Also, spacing between your statements in `else if` statements


Because it's a hassle to pass the retracted status on from the previous layer. I don't think it really mattes given that the time estimates are naive anyway.

Shouldn't your comment have been on line 430?


TODO (review marker for bradymiller)


You'll need this for external API calls like from the CampyDB web app.


We'll need the authorization/authentication methods for later so add them back.


We'll be using basic HTTP auth later when we establish user management.


Indentation on closing bracket, circle variable and return statement.


I think it is. I'm going to look into it tomorrow.

The least we could do is use the global_only property to distinguish between global/per-meshgroup vs per-extruder/per-mesh.


`k` should never finish with `_id`, should it? Even on the cases it is added, the `_id` bit gets removed.


Append "API" to API resource class names. "CgfNewickTree" -> "CgfNewickTreeAPI"


This covers only the case where the finally blocks returns. It will still leak if it throws an exception instead:

```php
function test() {
    try {
        return str_repeat("a", 2);
    } finally {
        throw new Exception;
    }
}

try {
    test();
} catch (Exception $e) {
    echo $e, "\n";
}
```


No print statements anywhere. Let's use the logging module e.g. https://bitbucket.org/peterk87/sistr_backend/src/e037a7ff8d095eddc273617817081535b7947d03/app/__init__.py?at=master&fileviewer=file-view-default#__init__.py-25


> This solution has a small drawback: In case the MAC seen on the wire
> is 00:00:00:00:00:00, it won't be propagated.

Yeah, I figured that's ok given that 00:00:00:00:00:00 doesn't have
much meaning anyways. But you're right that it does loose the
information that something has been extrated from the packet at all.

> In contrast to the
> solution with an extra pointer it also introduces some `memcmp` and
> `memcpy` calls, which might be a tiny overhead, aren't they?

I'm not concerned about those, that should be fast. The advantage is
avoiding lots of tiny memory operations. But if we did want to make
sure we show the occasional 00:00:.. in the logs, we could use two
boolean flags instead simply saying if the addresses are set and also
get rid of the memcps that way as well.


please add comments here, is this the case that the external component has started Dynamo but now Dynamo wishes to start from the Revit UI and so the other model is shut down?...

if so, I am not sure we should enforce this behavior here - consider the case of some other external component which actually intends to connect to a running instance of DynamoRevit. Is it possible to enforce this constraint in the external component or is this not possible?


I think it's time to develop a 3D freezer.


If you're just including the code you don't need to wrap it in the block. In fact, I'm surprised that this works...


Underscores instead of camelCase for function/method names
`createGeoDict` -> 'create_geo_dict`

Review https://google.github.io/styleguide/pyguide.html


This should probably be `content.contributions` right?


`GeoDict` -> `GeoAPI`


I'm showing pexExceptions, afwCG, afwDetection, and afwTable as unused.


The following doc(s) have been deployed to the [1626reduce](https://1626reduce-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/b6183b622502bd9b396b39dc78492c667cdb8668/app/config/sculpin_site.yml)
- [docs/reducing-large-repos](https://1626reduce-static-docs.pantheonsite.io/docs/reducing-large-repos)


I've changed them this afternoon, in a separate branch which introduces four properties to replace global_only.


Can you pre-compile this output rather than re-reading/re-compiling every time the API is called?


Good job on your css, but formatting could use some work.

This could be written better as such:

```
body {
  backgroud-color: #ffe;
  padding: 50px;
}
```

and likewise with all the css selectors below...


This could be a module too. And `imports = import ./foo.nix` becomes `imports =[ ./foo.nix]`


Furthermore this assumes that the RETURN is directly after the FAST_CALL, which is not necessarily the case. There might be other FAST_CALLs or FE_FREEs in the between. For example:

```php
function test($x) {
    foreach ($x as $v) {
        try {
            return str_repeat("a", 2);
        } finally {
            return 42;
        }
    }
}

test([1]);
```


Nice job using a CSS class to create this concept of a black border that you then use on your image and on your body! 👍


Please review PEP8 specifically style guide around blank lines https://www.python.org/dev/peps/pep-0008/#id21


2240_CmToIn (macumber) - Win64-Windows-7-VisualStudio-12: OK (2401 of 2423 tests passed, 0 test warnings)

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


Overall great job. Well done. 👍


CatchNanInHeatBalance (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


If the issue is that you simply need to do a separate sql insert for each row of data, then recommend placing the following code around your inserts if you don't want it to take hours(innodb can be inefficient when doing large number of inserts in a row):
https://github.com/openemr/openemr/pull/130/files#diff-cffbacdb1f5a81dc622aeb505e08b0fcR91
(and use sqlStatementNoLog() to wrap the inserts)


I'd like you to describe your commits better. "more changes" doesn't tell me anything about what you did. Likewise with the previous commit.


This one didn't import gcodeExport.h...


We're going to need a `create_app(config='default')` function.

See https://bitbucket.org/peterk87/sistr_backend/src/e037a7ff8d095eddc273617817081535b7947d03/app/__init__.py?at=master&fileviewer=file-view-default#__init__.py-51


Ok , I'll change it.


@dan131riley Thanks.  This was also added to the master.


XP, right! Gonna add that now!


The reason why this import statement and other API resource import statements are after the above code that's been deleted in this commit is to prevent circular dependencies which may be a problem later on as the complexity of the project increases with the addition of a DB/task queue/etc.


Overall, code looks good. Need to test it though(hopefully will get to do this today).
2 remaining items are:
1. Importing xml valueset file
2. version/upgrade mechanism when upgrading the valueset file (can mimick what is done by rxnorm)


Maybe; could you elaborate?


os_2_0_develop (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Why is `max_speed_layer` a double again?


Thanks! Fixed.


> But if we did want to make sure we show the occasional 00:00:.. in the logs, we could use two boolean flags instead simply saying if the addresses are set and also get rid of the memcps that way as well.
Seems hard to predict whether it will be worth the effort. So I think in the end its more or less a style decision :)


Thanks for catching that - I refactored the code.


Best to keep this as json.dumps instead of jsonify. The `output_json` method was added to handle JSONP (https://en.wikipedia.org/wiki/JSONP)


https://github.com/eugeneo/node/commit/929ba156f1d57c99ac8704a2ff956f877aa739b6

On Tue, Jun 7, 2016 at 2:16 PM Eugene Ostroukhov <eostroukhov@google.com>
wrote:

>
> https://github.com/eugeneo/node/commit/73ffd51108b6ac04b8f71d547f959ef2abdc38b4
>
> On Tue, Jun 7, 2016 at 2:08 PM Eugene Ostroukhov <eostroukhov@google.com>
> wrote:
>
>> I replied to comments. The new CL is here:
>> https://github.com/eugeneo/node/commit/b6fe1cd6776cf51aadf23d92f30ddbabf5931473
>>
>> On Tue, Jun 7, 2016 at 12:25 PM Eugene Ostroukhov <eostroukhov@google.com>
>> wrote:
>>
>>>
>>> https://github.com/eugeneo/node/commit/9a009a8e5443295aec40149260af308e6c0b1baa
>>>
>>> On Mon, Jun 6, 2016 at 12:37 PM Eugene Ostroukhov <
>>> eostroukhov@google.com> wrote:
>>>
>>>>
>>>> https://github.com/eugeneo/node/commit/165564317ffaa975469ef90da76d548374c89d6e
>>>>
>>>> On Mon, Jun 6, 2016 at 12:26 PM Pavel Feldman <notifications@github.com>
>>>> wrote:
>>>>
>>>>> I don't think so. We should not accept connections while connected.
>>>>>
>>>>> —
>>>>> You are receiving this because you authored the thread.
>>>>> Reply to this email directly, view it on GitHub
>>>>> <https://github.com/eugeneo/node/commit/dd65067f07d2dd269a3f84f22bc98e722618e13c#commitcomment-17758504>,
>>>>> or mute the thread
>>>>> <https://github.com/notifications/unsubscribe/AARkrUKHu0-cquMtKM66YjxCKk2PtuHhks5qJHRUgaJpZM4IvOLr>
>>>>> .
>>>>>
>>>>


@angus, any reason why we didn't use the OGC WPS defined mimetype below to determine the required output format (i.e. the output is available in either application/x-netcdf or text/csv rather than application/octet-stream)?


Hey @chulsupark!
Please could you let me know your name and an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed this Dockerfile :-)   Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530


Hey @heavyplayer2!
Please could you let me know your name and an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed this edit :-)   Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530


Здравствуйте, Филип. У меня возникли с добавлением своего домена на гитхаб
JB, вернее я добавил, но его не приняло, не знаю почему, возможн я допустил
ошибку в своих действиях.Но меня этот вопрос больше не беспокоит т.к. я
выслал документы из своего учебного заведения и мне уже пришел ключ. Еще
раз спасибо Jet Brains за ваши замечательные продукты и все остальное в
целом.


Hey @MelissaCole!

Please could you let me know an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort:

https://github.com/ethereum/webthree-umbrella/issues/530


Might as well follow the code style here. I'd fix it for you, but I'm not doing this review on a computer where I can easily compile it...


Hey @pompomJuice!

Please could you let me know your name and an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed this change :-)   Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530


Thanks!  ui_testbench and ProDBG uses different main code so you need to update windows.rs as well (used be ProDBG)


os_2_0_develop (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Hey @psalami!

Please could you let me know an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed 3 small changes.   Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530


Hey @SharpCoiner!

Please could you let me know an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed 6 commits. Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530


Disculpa!!!


Hey @CodeShark!

Please could you let me know an e-mail address for you?

It is in relation to this cpp-ethereum re-licensing effort!

You contributed 14 commits.   Thanks!

https://github.com/ethereum/webthree-umbrella/issues/530

PS.   Hey, @programmerTim!   Is this e-mail address for you correct too?   Thanks!

https://github.com/ethereum/webthree-umbrella/wiki/Contributors


Hey @autolycus

See https://github.com/ethereum/webthree-umbrella/wiki/Contributors

Please could I get an e-mail address for you?   This is in relation to the planned re-licensing of cpp-ethereum.   See https://github.com/ethereum/webthree-umbrella/issues/530.

You contributed 10 commits.   Thanks!


kok ti je pa  to ratal?


Should this be apt.pkg.read?


* Auto-show


@MCSotgiu Maria Chiara aggiorna il progetto...


@josevalim Correct -- I'm not sure where this commit is, it should be using "k_id" (as in {k, k_id, m, _}) which will have the trailing "_id" bit. This is the correct commit: https://github.com/GriffinMB/phoenix/commit/fb9c601459083ecad2a90c8dc04fe78094f0bff3 and is the one I made the PR with!


CatchNanInHeatBalance (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1655 of 1655 tests passed, 0 test warnings)

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


@MalQu Aggiornamenti del progetto?


We currently didn't include these incremental scripts in our CI tests.
And I am sure that this build fail was caused by the problem on the ROS server side.
So don't worry.


Yes. This file has a few undefined variables, which is a serious bug, in addition to many linter warnings. It is so bad that I decided to make only minimal changes, rather than try to clean it up. I filed a ticket in March: https://jira.lsstcorp.org/browse/DM-5684


@r4ff4ell4 aggiorna il progetto...


@giovip giovanna aggiorna il progetto


Good point


@JensLincke this breaks the karma tests, because in the browser (outside of Lively), there will be no node_modules. Instead, chai is available globally there becase it is included as another <script>. Please have a look at the test-node script in package.json how chai is supplied to the mocha tests. But I see that we need to look for a solution for the Lively test runner as well, now that the module can be imported there.


Argh, indent!
https://github.com/eugeneo/node/commit/21be0c431a921276dbe79d16bf5f1b570145bd30

On Tue, Jun 7, 2016 at 2:47 PM Eugene Ostroukhov <eostroukhov@google.com>
wrote:

>
> https://github.com/eugeneo/node/commit/929ba156f1d57c99ac8704a2ff956f877aa739b6
>
> On Tue, Jun 7, 2016 at 2:16 PM Eugene Ostroukhov <eostroukhov@google.com>
> wrote:
>
>>
>> https://github.com/eugeneo/node/commit/73ffd51108b6ac04b8f71d547f959ef2abdc38b4
>>
>> On Tue, Jun 7, 2016 at 2:08 PM Eugene Ostroukhov <eostroukhov@google.com>
>> wrote:
>>
>>> I replied to comments. The new CL is here:
>>> https://github.com/eugeneo/node/commit/b6fe1cd6776cf51aadf23d92f30ddbabf5931473
>>>
>>> On Tue, Jun 7, 2016 at 12:25 PM Eugene Ostroukhov <
>>> eostroukhov@google.com> wrote:
>>>
>>>>
>>>> https://github.com/eugeneo/node/commit/9a009a8e5443295aec40149260af308e6c0b1baa
>>>>
>>>> On Mon, Jun 6, 2016 at 12:37 PM Eugene Ostroukhov <
>>>> eostroukhov@google.com> wrote:
>>>>
>>>>>
>>>>> https://github.com/eugeneo/node/commit/165564317ffaa975469ef90da76d548374c89d6e
>>>>>
>>>>> On Mon, Jun 6, 2016 at 12:26 PM Pavel Feldman <
>>>>> notifications@github.com> wrote:
>>>>>
>>>>>> I don't think so. We should not accept connections while connected.
>>>>>>
>>>>>> —
>>>>>> You are receiving this because you authored the thread.
>>>>>> Reply to this email directly, view it on GitHub
>>>>>> <https://github.com/eugeneo/node/commit/dd65067f07d2dd269a3f84f22bc98e722618e13c#commitcomment-17758504>,
>>>>>> or mute the thread
>>>>>> <https://github.com/notifications/unsubscribe/AARkrUKHu0-cquMtKM66YjxCKk2PtuHhks5qJHRUgaJpZM4IvOLr>
>>>>>> .
>>>>>>
>>>>>


very nice, thanks for debugging and fixing this.

In future, please consider adding a unit test which demonstrates both the bug and the fix when doing this kind of work -- both as a way of communicating the issue, and as a nice regression test going forward :)


regression ?


@jmarrec FYI


Hello @coding-horror ,
IMHO I suggest NOT to use _/var/discourse_ as the prefered or default path, in docs or practice.
According to [FHS](https://en.wikipedia.org/wiki/Filesystem_Hierarchy_Standard) , /var/ is not the place for applications or software packages. We sysadmins might use some special device or filesystem with fast Write performance instead of Read at this mountpoint.

A better suggested path is in _/opt/discourse_ .
Or, even _/usr/local/discourse_.


Yes , that's the case ; I'll add comments. Initially I just initialized the UI on top of existing UIless revitDynamoModel but than I taught that it might be wiser to shutdown the existing one in order to preserve the same startup context as before when you press Dynamo in Revit .
I'm not sure how a constraint should apply depending on external apps ; I guess DynamoRevit will have to know who started the model and what to do when Dynamo is pressed in Revit .
I would prefer just put the UI on top of existing model if that's ok .


@ekohl @ares @domcleal sorry, pushed into the wrong remote... however, if there are no objections, this should get cherry-picked into 1.12


"Dilophosurus"
One of these?
http://i.imgur.com/LDajAls.jpg


Looks like you forgot to update something:
```
    [javac] /home/lucas/git/ceylon/ceylon/compiler-js/src/main/java/com/redhat/ceylon/compiler/js/CeylonTestJsTool.java:88: error: method loadDependencies in class CeylonRunJsTool cannot be applied to given types;
    [javac]                     loadDependencies(localRepos, repoman, artifact);
    [javac]                     ^
    [javac]   required: List<File>,RepositoryManager,File,Set<String>
    [javac]   found: List<File>,RepositoryManager,File
    [javac]   reason: actual and formal argument lists differ in length
```


description


Hi @bennylava10100,

I guess it's the rebel in me that's at fault. I'm not particularly a fan of `EntityFramework` and its design. I think the particular DAO design I wrote here was intentional. However, I sense the underpinnings of a religious war brewing. :smile_cat: Just kidding. Haha.:+1:

##### Option 1: `DescriptionAttribute`
Okay. So, a couple issues I see with `DescriptionAttribute` or custom model layer attributes:

```
[Description("PersonTable")]
public class Person : Document<Guid>
{
    public string FirstName { get; set; }
    public string LastName { get; set; }
}
```
Now, what happens if I want to map `Person` into different/multiple tables? For whatever reason (for better or for worse), it doesn't seem possible with a single `[Description("PersonTable")]`. Additionally, regarding *data locality* just seems awkward to have database physical layer **configuration information** (such as `"PersonTable"` table name) tacked onto a **Domain Object** at a higher abstraction level than the **DAL**.

But with the current DAO design, you could:
```
public class PersonDao : RethinkDao<Person,Guid>, IPersonDao
{
    public PersonDao(IConnection conn)
        : base(conn, dbName: "MyDb", tableName: "PersonTable")
    {
    }
}
public class WhateverDao : RethinkDao<Person, Guid>, IWhateverDao
{
    public WhateverDao(IConnection conn)
        : base(conn, dbName: "OnADifferentDB", tableName: "AnotherPersonTable")
    {
    }
}
```
Okay, granted, it's a little more verbose, but **all** the database **configuration information**, physical layer stuff, is all here, in the same file, in the DAL. And no need to jump around multiple files to spatially piece together how everything should work at the physical DAL layer.

And what happens if we want to `FindPersonByLastName` using an index? A super generic `RethinkDao` I don't think would do very well. IIRC, you'd have to create your own derived DAO (anyway). Or, maybe you'd have to rely on some kind of `ExtensionMethod` to compose an underlying ReQL query. You'd also lose some protected variables of `RethinkDao`. I dunno but personally, just feels like I've lost control of the way I want to design my DAL for my app.

##### Option 2: The all mighty **EF** `TypeConfiguration`.

I'm not well versed in EF, so, I might be totally wrong, but having something like `TypeConfiguration` would, I think, require us to carry around some kind of `DbContext` god object. Now, contrast this to the current implementation. **The DAO _is_ the context itself**. No need to fiddle or pass around a context object. **It's simple. It works. I like.**

Maybe just me, the rebel, the **EF** hater, maybe personal preference too, but I think `DbContext` is way too automagical. `DbContext` now becomes a bloated (over-engineered?) :whale2: monster of thing. One `DbContext` object to rule them all vs a collection of DAOs that make up the DAL. ¯\ _ (ツ)_/¯

 Taking this current approach with an `IoC container` like `Ninject` or `SimpleInjector` basically amounts to:
```
kernel.Register<IPersonDao, PersonDao>(Lifestyle.Singleton);
kernel.Register<IWhateverDao , WhateverDao>(Lifestyle.Singleton).
```

-----

Now, I know not everyone will agree with me, but the current `IDao` code is what I'm currently willing to maintain at the moment. However, if you feel very strongly the driver should have something **"EF like"** as you've described, I'd be more than open to accepting a PR for the driver. Since it is *kinda* out of scope of the driver, perhaps, we could release a `RethinkDb.Driver.Data` or `RethinkDb.Driver.Extras` assembly that contains **"EF like"** `DbContext` stuff and all those bells :bell: and whistles.

All I ask is ...
* ... please, please, please, keep it simple.
* The **"EF like"** features should be pretty close to being feature complete.
* Some unit tests that run though the basics would be cool.
* Some accompanying documentation would be cool too.
* Also, create a GitHub issue with some example API usages so we can all discuss the API before committing to it. :sunglasses:

Forgive me if the PR bar for the driver is kinda high, but hopefully everyone realizes I really care a lot about maintaining a high-quality, cool, and kick-ass, C# driver for us to all use.

Peace homie,
Brian

:chocolate_bar: :cookie: :lollipop: [***Ronald Jenkees - Stay Crunchy***](https://www.youtube.com/watch?v=GeIAXlwVlZc)


Who knows? Mercury has a glass transition at -39 degrees C. We could print something with it!

Better open up the windows first though.


Nice refactoring. I was having some errors with the passing of sourceMaps through all the getAllLoadRecords methods making our build to fail.

Waiting for this to be published to update to latest version.

Good job.


@waywaaard This is the commit that broke the working behavior for unittest where I could say

```
from unittest.gui import TestCaseGui
```

git bisect shows that this is the release that broke this behavior.  Could you either fix it or give me an idea of why it might be broken?

Brad


Oh nice job copying F/A with the ModState file :clap: .


develop (kbenne) - x86_64-MacOS-10.10-clang: OK (2379 of 2419 tests passed, 0 test warnings)

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


Les fichiers HTML, personne n'en a besoin de le VCS. Mettez le conf.py qui permet de les regénérer plutôt.


the idea was to have everything here and return only object... This could be reused in other tools...


It could be useful to change saber stance while dead so that you quickly spawn with the desired stance afterward. Just a thought.


Remember to split up the passing tests into their own commits.


The `number` argument is close to the JS `Number` keyword. Better to use `n` or `num` as argument name just to avoid confusion.


test **test** comment.
`var abc = 4;`


Use single quotes when declaring strings. JS convention.


Indentation on `if` is one tab too many.


G


H


Sure, will do.  Thanks for integrating our fixes!

*Keith Mason*
Vice President | Application Platform and Quality Engineering

Conning | One Financial Plaza, Hartford, CT 06103, USA
<x-apple-data-detectors://0/0>
*D* +1 860 299 2671   *F* +1 512 692 9134   *E* keith.mason@conning.com

<http://www2.conning.com/e/22072/2014-07-08/5tycz/122916975>
<http://www2.conning.com/e/22072/user-ConningCentennial/5tyd4/122916975>
<http://www2.conning.com/e/22072/company-conning/5tyd6/122916975>
<http://www2.conning.com/e/22072/ConningNews/5tyd8/122916975>

On Jun 7, 2016, at 5:58 PM, dls <notifications@github.com> wrote:

very nice, thanks for debugging and fixing this.

In future, please consider adding a unit test which demonstrates both the
bug and the fix when doing this kind of work -- both as a way of
communicating the issue, and as a nice regression test going forward :)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/dls/DynamoDB.jl/commit/4f2da00ddf28f1dd5fb129df49f5a1cd5bd71568#commitcomment-17779089>,
or mute the thread
<https://github.com/notifications/unsubscribe/AIuCDCe20PP7hmh_0tWzl_Z8_CD4s34wks5qJel9gaJpZM4IwZzt>
.

--

Please click here
<http://www.e-disclaimer.com/conning/EE06B3FA5D199E20A3F7D0AE080A38FF.htm> for
important information regarding this e-mail communication.


Adicionando bibliotecas


@mantelt Can this branch (fix/windowsDeployment) be deleted? If yes, go ahead...


@trueadm should these also be props/className?


+There is some stuff to care about:


You might want to update the [Reprap page](http://reprap.org/wiki/G-code#M204:_Set_default_acceleration) too then. It says that Marlin uses P and T, but not so for Ultimaker now.


Good solution. Only suggestion is to either use a `case` statement or `if () else if() else` block. Also, you could extract `score/total` from every conditional, to a variable and use that instead. i.e. ` var myGrade = score/total`


Nice spot :) although in this case it won't affect performance, as they're completely static and thus never get evaluated or created again (they're attached to a DOM node that is cloned upon creation).


How about just "Acceleration"?


@tcyrus You should use more meaningful short summaries.


Well, `wof-pylibs` doesn't actually update anything. This is just meant to alert you that you should attend to the libraries when you're in the process of doing server deploys. Currently the Python updating process is a little complex due to `mapzen.whosonfirst.pip.utils` still being on a separate branch.


Project implementing ML techniques such as classification and clustering


Remember to use single quotes instead of double. JS convention.


Ah, I see now. The issue is linking to this incorrect commit, while the PR is linking to the correct one. Sorry for the confusion. Again, you'll find the correct PR here: https://github.com/phoenixframework/phoenix/pull/1733/commits/fb9c601459083ecad2a90c8dc04fe78094f0bff3


what typos where


@Connor-Vickers `.swp` files are just for Vim to manage your undo history, so you don't need to put them in source control.


yes interesting
you have also with bash
- [yaourt](https://github.com/archlinuxfr/yaourt) , all functions are in /lib/
- [apacman](https://github.com/oshazard/apacman/blob/master/apacman)
...


Great! Just remember to keep your spacing consistent between your arguments to make things easier to read.


You might want to activate this only when skirt is enabled.


Typo: `ar` -> `or`


@jasonrudolph @spicycode @gdagley @glv @karnowski @abedra @glv @muness @stuarthalloway @rsanheim @jgehtland @eyowell @donmullen

Memories...

The [history on this file](https://github.com/relevance/etc/commits/master/scripts/pair) is actually pretty interesting. Kind of a live updating roster of Relevance/Cognitect employees and contractors, up through late last year...

Miss you guys! :relaxed:


Closes #4


You can ditch the `!` here *IFF* you update to 0.1.8 of XTest.


typo?


I see, makes sense. I'm good with the PR then if you are ready to submit it.


This is the hard dependency on US denominations that I mentioned. Can you think of any strategies to make this class work with the Euro, for instance?


Typo: `effect` -> `affect`


Hummm...
Maybe changing `[^]` with `[\s\S]`.

@butchmarshall I can't test on IE8 right now... do you mind testing it for me please?


Typo: Missing period after the first sentence.


I think mpz_t's support up to 2^31-1 limbs or something like that. That's
about 2^37 bits. It seems like a 32 bit long might be a bit underpowered
for that. Admittedly there aren't likely to be many applications using the
maximum, but I'm wondering if we will consider ourselves shortsighted in 10
years or so if we make it a long.

Bill.

On 7 June 2016 at 22:43, Brian Gladman <notifications@github.com> wrote:

> On 07/06/2016 19:10, jzmaddock wrote:
> > This change introduces a source-level incompatibility between MPIR and
> > GMP, given that the GMP documented signature for this function takes a
> > long/, while on Win x64 yours now takes an __int64/ and never the two
> > shall mix. Further the change appears to be unnecessary since internally
> > the exponent type on win x64 is still only type long (ie 32-bits).
> >
> > Found while investigating this bug report:
> > https://svn.boost.org/trac/boost/ticket/12234
>
> Fortunately, unless I have missed some cases, there are only two
> functions in the MPIR public interface where this is a possible issue:
>
> double mpz_get_d_2exp __GMP_PROTO ((mpir_si *, mpz_srcptr));
> double mpf_get_d_2exp __GMP_PROTO ((mpir_si *, mpf_srcptr));
>
> A simple solution might be to provide:
>
> double mpz_get_d_2exp __GMP_PROTO ((signed long *, mpz_srcptr));
> double mpf_get_d_2exp __GMP_PROTO ((signed long *, mpf_srcptr));
>
> in addition to the mpir_si variants when MPIR is built on Windows x64.
>
> I can look into doing this if we decide that it is desirable to have a
> fix for this.
>
> Brian
>
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/wbhart/mpir/commit/fb616eed78d0a32dc9e53b355b0c2d42c1209907#commitcomment-17777935>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAOzpq1VZCfgyaMXfJJvk8kAHI8Pftbhks5qJdfcgaJpZM4IwNBy>
> .
>


:exclamation: This should not change.


Should this have been a PR? :-)


:exclamation: This should be the following (`t` is not required to be a `RuleContext` for this method)

```
int altNumber = ((RuleNode) t).getRuleContext().getAltNumber();
```


You also might want to enable this only if skirt is enabled. Does this also affect brim, by the way?


Note: current parse query is counting all possible cheers and not the spectator-verified cheers - we'll probably want to change this in the future


Heyyyy you have to compile the JS source again to this works... I'm getting your data....
Never clone my own blog, use the template instead [here](https://github.com/willianjusten/cards-jekyll-template)...

Please, run `gulp -p` and deploy a real version of your analytics...


La *B* en minúscula.


Now all the arrow point to this...


Nothing in your tests requires you to define dimes and quarters. Think back to Rule 3 of TDD. Even if the algorithm is straight-forward to you, having tests that ensure correct behavior at every step is invaluable when refactoring.


I'm not opposed to it but we're using 'Product' up above without using the singular labels for the label of this item.
`<strong><?php echo apply_filters( 'edd_product_type_options_heading', __( 'Product Type Options:', 'easy-digital-downloads' ) ); ?></strong>`


You should probably say "the raft surface layers" instead of "the surface raft layers".


El nombre del archivo `AppBase.js` -> `Appbase.js`.


Why not do a cast to the entity and see if it's nullptr?
Then you can rid of the while EntityType thing and you can also make this writing better pluggable (future)

For example:
`
const auto ellipse = std::dynamic_pointer_cast<const lc::entity::Ellipse>(entity);
    if (ellipse != nullptr) {
       // writeEllipse(ellipse);
    }
`


Having high values doesn't really make any sense. As long as the entropy pool is not completely drained, the values don't matter too much and it really won't make much of a difference anyway.


@mindctrl we shoudl update our docs then, I pulled it directly from there. :)


Again, "raft interface layer"? What do you think?


Again, "raft base layer"?


Again, "raft surface layers"?


This looks pretty cool.  A shame you didn't make this branch from the 1.8.9 branch.  It could have saved you some time.


Again, "raft interface layer"?


Hey, doesn't crash at least @tivie!  I don't know if the regexp is still performing the same operation though - does this function have tests?

![image](https://cloud.githubusercontent.com/assets/339763/15876809/abbe7ec0-2cdd-11e6-9764-116e910a71b5.png)


This needs to change, but I don't know how to not hardcode it. @mkwm any idea?


From: http://stackoverflow.com/questions/22687940/python-ryu-handling-packets-using-a-switch-after-flow-was-added-to-switch


@lileeyao 你的意思是会用generic class实现这个LRU是么？


Cool, I never knew about `process.exitCode`


@dookiedude  hey dude...
is it BOARD_HAVE_AUDIENCE_ES325_2MIC := true
or
BOARD_HAVE_AUDIENCE_ES325_MIC := true

?


I'd have allowed sending the kick packet in handshake phase instead as the handshake phase is client-side only a very short time as there is no server -> client handshake packet


There is no "visible" property any more.


@snicolet
I know you had lots of success with this kind of stuff.  If you think my constants are way off or have any other comments here or here
http://tests.stockfishchess.org/tests/view/5755c0000ebc59029919b278
http://tests.stockfishchess.org/tests/view/575608950ebc59029919b286
I'm definitely interested.


Boolean types are now labelled "bool" rather than "boolean".


Can you explain how this method works?
Perhaps it's my lill knowledge about math, but I currently don't see how the mirror works.
Are axis1 and axis2 start end end points for a line to mirror at? If so, can we comment this?


is it really BOARD_HAVE_AUDIENCE_ES325_2MIC := true ?
I thought it was BOARD_HAVE_AUDIENCE_ES325_MIC := true

without '2'


Does this change means that Django 1.7 is no longer supported for Mezzanine 1.4.1?
I'm currently running into an issue because of that change.


feelsbadman;


This looks like a very good addition to the base function.


`"boolean"` -> `"bool"`


Noted


*in last commit

513620f0f9ed7d4419395b102ac638cec611d35f


Hi, just wanted to let you know that I won't be making it on today. I
caught some sort of cold and my asthma isn't helping.
Will the notes be on slack?

-Chantal Montanez


I think you mean Mezzanine 4.1. Then yes, only the two latest Django
versions are supported by any particular release.


aaaaah @arco  ....in a previous post u wrote MIC and i got frustrated :P


At the moment it's completely unuseful, we miss that part of code in our audio repo... That flag does nothing now.. Bul like arne suggested, i think that we need it or audio during calls won' t work...!


`Right-click on the selected text and choose "Quote selected text" in the context menu` sounds good to me :)


just a test


i confirm calls doesnt work.now building with "2MIC" and report here


Won't work, that flag is missing in the new hal, it's just in legacy code


Need to delete BetaOnly.


Can't we not re-use DxfLinePattern here?


It's be nice if it mimiced the lodash package and has `lodash-bound/mapValues`
instead of `lodash-bound/lib/_mapValues`.


I added my comments on this one in a other commit... I think we can use a cast and check for nullptr here.


@lileeyao I got it. 我在一个网站上看到了generic实现。http://my.oschina.net/apdplat/blog/479050


why?


even after those changes?
https://github.com/AOSP-JF-MM/android_hardware_qcom_audio/commit/eeb2c047d9e5eca19c7906184d38562c7aefcdd1 ?


It looks a bit different on the app no worry


We should add a toUpper in StringHelper and use that.


@jdalton Right! I wanted to ask you how did you generate the files in the root only for npm in `lodash`. Are you building it into another separate dir which gets published, or is there another trick?


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2362 of 2422 tests passed, 0 test warnings)

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


regexxx


Honestly, I don't remember. There's a chance that that initial assertion library did not offer a deep equals operation, but outside of that, I think that the xml representation might have been slightly less typing.

Which ever case, it was most likely out of laziness vs. creating better tests.


@glennon can you undo this commit?


Is there a check here for the homepage links?


A beer for the effort @changetip


Okay, I'll take it back into consideration!


@glennon: can you undo this commit?


Hi @dongcarl, @MaxFangX sent you a tip worth A beer ($3.50), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/948013)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


updated rules


@MaxFangX 🍻


Squashed en rebased <3


Yep. I output the files to a `./foo` folder, then I switch to the `npm` branch and move the files from `./foo` to the root.


So we should add it in Handshake protocol


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

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


120796739_ObjListAutoPopulate (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2211 of 2211 tests passed, 0 test warnings)

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


Added 13 streamable anime by R-Freak:

• Akuma no Riddle
• Ano Natsu de Matteru
• Cuticle Tantei Inaba
• Denpa Onna to Seishun Otoko
• Donten ni Warau
• Hai to Gensou no Grimgar
• Haikyuu!! Second Season
• Hamatora The Animation
• Hentai Ouji to Warawanai Neko.
• Inou-Battle wa Nichijou-kei no Naka de
• Kamisama Hajimemashita
• Kamisama Hajimemashita Second Season
• Kekkai Sensen


Neither did I! :)


Wow.....
On Jun 7, 2016 6:46 PM, "remote_vase" <notifications@github.com> wrote:

> Okay, I'll take it back into consideration!
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/Inactive-to-Reactive/Inactive-to-Reactive.github.io/commit/a80aefce26441dbbb7fd0586883db5a1b7a2eb07#commitcomment-17779675>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AOKfN2U-DufCt63NloobnpkL7rl8EXQkks5qJfSzgaJpZM4IwWDw>
> .
>


small change


So:

1. Build to `lib`
2. Create branch `release`
3. Copy all files from `lib` to root
4. `npm publish`
5. Revert changes and switch back to `master`
6. Delete branch

?


That commit adds only the support for the CSD client that use a non-stardard name, not the dual mic feature ..


I've so done this before too. Pretty sure ESLint catches this though.


Hi Guzzardo,

A proofset with this commit has been created for nose-parallel. Just for you!

[View Proofset](https://www.bluecore.com/admin/email_proofs/nose-parallel/proof_set/5649391675244544)


so there is no way to make calls work with new hal ?


os_2_0_develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Ah, I understand. Yeah it might've been on 430.


@ausdkunst

Hi Ausdkunst

I know you are really good into FG Automobiles.
I wanted to make you aware of this beautiful FG Jewel.

I wonder, now that Jack clarified the License, if you wish to give it some love :question:

@IAHM-COL


GroundDomainUpdate (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


[![Deer Mandala of the Kasuga Shrine](http://api.artsmia.org/images/6612/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=6612)


[![Monju Bosatsu](http://api.artsmia.org/images/6613/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=6613)


[![The Doryphoros](http://api.artsmia.org/images/3520/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3520)


[![Roman Matron](http://api.artsmia.org/images/488/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=488)


[![Winged Genius](http://api.artsmia.org/images/611/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=611)


[![Hikaru Genji](http://api.artsmia.org/images/123836/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123836)


[![Ganymede and the Eagle](http://api.artsmia.org/images/1629/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1629)


[![Grand Salon from the HÃ´tel de la BouÃ«xiÃ¨re](http://api.artsmia.org/images/3161/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3161)


[![Dressing table](http://api.artsmia.org/images/425/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=425)


[![Half armor](http://api.artsmia.org/images/49709/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=49709)


[![Wine cistern](http://api.artsmia.org/images/1451/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1451)


[![Warrior with Shield](http://api.artsmia.org/images/1244/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1244)


[![Large Seated Nude](http://api.artsmia.org/images/1888/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1888)


I keep my [npm branch](https://github.com/lodash/lodash/tree/npm) around so I can link to it in the readme.
Folks like to browse package source without dloading the package.

Build release. Switch branch. Move files from release to root. Commit and tag release. npm publish.


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

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


Just realized the extra lint commit message eats the more important commit message; I'll be sure to duplicate the commit message if there are formatting issues moving forward.

> component skeletons, styles for new `BraveryTab` items


TeamCity Mobile :: Android :: Android Develop (Nightly UAT) [Build 13](http://teamcity.hq.judo/viewLog.html?buildId=40163&buildTypeId=Mobile_AndroidDevelopNightlyUat) is now running


I made an attempt to write a test for this new feature, but could not get it to work, for some reason. Here is what I tried:

```javascript
  it('should set search text to initSearchText if set', function() {
    var el = createUiSelect({
      initSearchValue: 'Hello World',
    });

    $(el).click();

    // el.find(".ui-select-container").click();

    var searchInput = el.find('input[type="search"]');

    expect($(searchInput).val()).toEqual('Hello World');
  });
```


We could remove the second `.ini` for that. I believe I caused the `troposphere_variables` typo bit.


Since this is really _just_ an intermediate file, we really never notice that double file extension in the deployment.


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

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