My primary motivations behind wanting to keep the plugin id's camelCased is because configuration is prefixed with the plugin id, and we want to keep things BWC. While none of the plugins that are changing have documented configuration the tests_bundle plugin specifically is used elsewhere, see https://github.com/spalger/kibana/commit/36cdae8281b9065bb050a3d823efaef3cf23e2b8#diff-4d343b812281fca44b2afeab2d3f9e76R89


please add min version to oslo libs, pbr, clients, six etc ...


RE:  01-install.md
hello @zhannar
cc:  @reshama @aramnath @Vinoad @JoelWesley

Good job in completing the 01-install (python) section.  :thumbsup: emoji

To print the version of python on your computer, the below two commands will work.
```
$ python --version
$ python -V
```


I might rename this to `isDiploidWithLikelihoods` since I think people might reasonably skip the javadoc and assume it's just checking the ploidy.


Also, please follow the consistent coding style.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1596 of 1596 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)


nitpick: not aligned


Quick note, we're back to 100% unit test pass again.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1022 of 1022 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-30.56%25-green.svg)


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


@chermidap Could you revert the commit where I change this?


Can you rephrase that in the form of a unit test assertion?


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


develop (mjwitte) - 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)


The following doc(s) have been deployed to the [simpleurls](http://simpleurls-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/a43aa5607f8dc8f995a9048568a6bf996b7f43b3/app/config/sculpin_site.yml)


__API Description file syntax error on line 1083__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Well, we have been talking about making the config prefix for plugins configurable. Maybe it's time to do that and keep the id snake_cased


#include<stdio.h>

int main (){

    int m;

    while(~scanf("%d",&m)&&m!=0) printf("%d\n",m/2);

    return 0;

}
//数序分析，整除2，想想也是，三个就能换回来一个，然后整除又能去除余数。


I'd suggest to just use the pattern as the ID, not also the correction. I think that's what we usually do so it would be nice to have it consistent.


/*
递归问题
3个瓶子换1瓶水+1个空瓶子，两个瓶子换1瓶水+0个空瓶子，1个瓶子换0瓶水。
f(1) = 0
f(2) = 1
f(3) = 1
f(4) = f(2)+1    //4个瓶子，其中3个可以换1瓶水+1个空瓶，所以是f(2)+1
f(5) = f(3)+1    //3个瓶子换1瓶水+1个空瓶，所以是f(3)+1
...
f(n) = f(n-2)+1 */
#include <iostream>

using namespace std;

int f(int n)
{
    if(n==1) return 0;
    if(n==2) return 1;
    return f(n-2)+1;
}

int main()
{
    int n;
    while(cin >> n){
        if(n==0)
            break;
        cout<<f(n)<<endl;
    }
    return 0;
}


oslo.sphinx?


RE:  02-editors.md
hello @rachaelrho
cc:  @reshama @aramnath @Vinoad @JoelWesley

This section has not been completed.  Perhaps it was an oversight?  Can you please complete and add mentions in your commit:  @reshama @aramnath

Thanks!


@spalger that's exactly the sort of thing I had in mind, isolate snake_casing to filenames and imports and use camelCase for the id in the code. But why not make it a convention that plugin ids get normalized to camelCase instead of requiring additional configuration? Similar to how Angular translates directive names between html and JS.


Ah okay! Cet ID c'est en effet celui des tickets orateurs. Ils sont déjà listés dans la page programme mais oiuay, pourquoi pas les lister aussi dans les participants.


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


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 283](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=834&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Compilation error: Xbim.Common\Xbim.Common.csproj Build time: 00:01:34


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


So some more thoughts:  I think we could simplify this though by using the core "comparator" function
so that we can just operate in terms of numbers in Clara.  It looks like for that one the Clojure [1] and
ClojureScript [2] are the same.  Good suggestion there!

1. https://github.com/clojure/clojure/blob/clojure-1.8.0/src/clj/clojure/core.clj#L2985
2. https://github.com/clojure/clojurescript/blob/master/src/main/cljs/cljs/core.cljs#L10191


Any reason not to run unit tests?


Yes, I was going to mention that.

I was assuming that the statement with the "IMPORTANT" in all caps was there on account of that giant, gaping security hole that we used to have - that allowed anyone to fake coming from localhost and thus gaining access to the admin api. Now that it's been closed, I felt it was ok to soften it. Because I feel having the admin APIs restricted to the world, but open to localhost is a fairly sensible out-of-the-box setup.

Feel free to change it back to "IMPORTANT", etc. But then I feel it would leave one wondering - if it's so important, why are you making it an optional step? Just make that script part of setup-all.sh, and leave a note explaining how to relax the security, if needed.

Maybe it should say that it's all-caps IMPORTANT to consider the security implications of having the APIs open to localhost, and if that's not good for your installation, this is how you close it.

As always, I may be missing something.

________________________________
From: Philip Durbin <notifications@github.com>
Sent: Friday, March 11, 2016 10:05 AM
To: IQSS/dataverse
Cc: Andreev, Leonid
Subject: Re: [dataverse] Documentation changes reflecting the 4.3 installer fixes and improvements. (a83066f)

@landreev<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_landreev&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=Wyoaab-62YLl40kAZBx0_PvdnyX0x8nOcegnuRFUlgM&m=Rwn4bZcIDExDdv5VIPSMTVcfgdIN1E6uu0XemIAEoDA&s=XGdmzq5IbUFCfZLifIRfDsk-nnWBKxCaSCQES-500I8&e=> I see you softened my warning about the importance of running the post-install-api-block.sh script. I'd go the other way and put VERY IMPORTANT in all caps. :)

On a related note, the installer ends with a fairly bland statement that we should probably improve to emphasize the security implications of only considering the running of the script and not actually running it: "Setup done. Consider running post-install-api-block.sh for blocking the sensitive API."

See also:

  *   Any Last Minute Security Details Before Going LIVE - Google Groups<https://urldefense.proofpoint.com/v2/url?u=https-3A__groups.google.com_forum_-23-21topic_dataverse-2Dcommunity_BhOFjm7BOAc&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=Wyoaab-62YLl40kAZBx0_PvdnyX0x8nOcegnuRFUlgM&m=Rwn4bZcIDExDdv5VIPSMTVcfgdIN1E6uu0XemIAEoDA&s=xppVC_pXS3d8pNrUNwKRkU0OnaAremMLM3jZCeY3YCc&e=>
  *   Enhancements to API blocker #1886<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IQSS_dataverse_issues_1886&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=Wyoaab-62YLl40kAZBx0_PvdnyX0x8nOcegnuRFUlgM&m=Rwn4bZcIDExDdv5VIPSMTVcfgdIN1E6uu0XemIAEoDA&s=JPz-P031MNtA4EwsjgYj9WcVlEiJlf0dsX3AqPunbmA&e=>

-
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IQSS_dataverse_commit_a83066f1e67a57a34b75398256898e8d3d6fbab8-23commitcomment-2D16646231&d=CwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=Wyoaab-62YLl40kAZBx0_PvdnyX0x8nOcegnuRFUlgM&m=Rwn4bZcIDExDdv5VIPSMTVcfgdIN1E6uu0XemIAEoDA&s=yTXmiMGRsM7Vq3ML1BW84yH0aWOjPqLUCdWOMu_eQiw&e=>.


I'm cool with automatic transformation of the plugin id for the require part. But it smells like the issue @rashidkpc and @epixa had that lead us to https://github.com/spalger/kibana/commit/0574f59d2468f55847eeeb9227d1b6290405448f


that should bump the major version to 2 :)


I think this is actually an unrelated bug where it accidentally skips evaluating the GlobalRef. It should have visited this before doing the call in the first place. When I stepped through, I saw it skipping the GlobalRef, so this may be a bug in my tree walking code.


hello @deenhe91 cc:  @aramnath

Can you please complete Questions 3 and 4?


I think it would be useful to say in the commit note that the reason we're happy to update the checksums is because all the regression tests pass with no changes to the artefacts i.e. all the changes in the PR are refactoring not changes in behaviour.


Actually, it is VideoToolbox filters that should be commented. VideoToolbox and FFMpeg filters must not be in the list of filters since they are automatically added at run time when Mediastreamer detects that the Apple platform is recent enough or, in the case of FFMpeg, the version of the installed FFMpeg library support H264.

Furthermore, you should know that this header file is not used to build Linphone's SDK. It is use by an XCode project we internally use to develop Liblinphone on Apple platforms. The list of filters included while building the SDK is automatically generated at build-time and is available in **linphone-iphone/WORK/ios-arm64/Build/ms2/src** directory.


'hebrew' : 'Hebrew',


Yeah, `SimulationError` sounds good.

I guess I'm not super convinced that every exception we ever raise has to be a subclass of `NengoException`. First, there are still exceptions that will come from nengo code that aren't, for example when we call things in `filter_design.py` or things from Numpy/Scipy in general. For the most part our validation will prevent this, but it's still possible for users to elicit exceptions from these libraries. So using a `try: except NengoException` block around nengo code won't get every exception that comes from our code. Second, if a user is running nengo code and just catching general exceptions, they'll still be able to see where they came from and know they're from our code. I don't think the changes in this PR are bad, and I wouldn't undo anything, I guess I just don't see a whole lot of advantage to having a blanket rule that "every exception has to be a NengoException".


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


The changes are failing all tests in PatientServiceAllergyTest.
The code https://gist.github.com/jdegraft/17ac3ef4719a186636da does not work. It seems adminService is not properly initialized, though I have loaded the database.


it is not clear what do you want to do here. Could you explain?


Multiple times "defaultValue(null)" is used. This doesn't actually do anything since the code for default value is:

        public Builder defaultValue(final String value) {
            if (null != value) {
                this.defaultValue = value;
            }
            return this;
        }


else if ( language == 'hebrew' ) {
// Hebrew is Windows-1255
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1255'), targetEncodingCharset );
}


hello @hayattokun
cc:  @aramnath

Can you please complete this section?  Also include mentions:  @reshama @aramnath in your commit.  Thanks!


queue name is not assigned to class scope here


latex-documentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


@soumagne, do you know why there are 3 NA_Progress() calls here?


Yup, that is correct, but safety first. I'd suggest to use D9 to go back. Can you also tell Plailect to include that in the guide?


use PRIu8 here


@ reshama yes that's what I'm working on today. I had completed the tasks but didn't realise there were things to fill in, so I am further ahead


Refers to AMQP


AMQP


These arguments are backwards I do believe.


Check return type?


@spalger ah ok, I'm cool with that. I thought it was rewriting a regular module's name, which I think would be going too far with the snake_casing.

This error message should probably be updated to say `ng_mock`. https://github.com/elastic/kibana/blob/8358ca43567698e8fab770b09fe7705e15c59448/src/plugins/dev_mode/public/ng_mock.js#L8


What does the Donald Trump Pho™ even taste like? Does anyone want to know?


@davidyan74 please implement proper wait condition


You can remove this conditional clause as this test method is only present on a class with `USE_TZ=True`.


__API Description file syntax error on line 1083__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


That's what I thought of when I ran into the issue, but is there a way
to get into HBL to run D9 when you've got 2.1 on sysnand?

On 3/11/16, d0k3 <notifications@github.com> wrote:
> Yup, that is correct, but safety first. I'd suggest to use D9 to go back.
> Can you also tell Plailect to include that in the guide?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/d0k3/OTPHelper/commit/6ff6f776b1d570206c1d2207aca330a4d56e3bc7#commitcomment-16649279


could you `render partial _form` here instead of duplicating this logic?


Нельзя вынести в отдельный файл?


There's an extra `v`, which prevents building the debian package.


A second str_replace for just `Trump` would be effective.


Все фрагменты выглядят абсолютно одинаковыми. Кажется, что большую часть можно вынести в базовый класс.


Final Code with all original code and borrowed base code


ok, on the first batch where kazz was added it can be but 2.4.3 was nerfed a lot for this reason i was realy sure about the 60sec.

but i can be wrong as well,

if OC wants to be blizzlike 2.4.3 or some kind of 2.4.3 without nerfs is a other topic, i don't even want to take a position on this


Можно вынести и переименовать в что-то типа ActionsLoader?


Not really I guess this is to try to complete operations. NA_Progress returns when an operation completes, so if there are 3 operations to be completed, it may be called 3 times.


@martint I recall we did just that, looked at how different DBs do it, and made a choice there. Do you think we can keep this as currently implemented at this point?


Длина строки.


else if ( language == 'hebrew' ) {
// Hebrew *is* Windows-1255
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1255'), targetEncodingCharset );
}
else if ( language == 'russian' || language == 'ukrainian' || language == 'bulgarian' ) {
// Cyrillic *is* Windows-1251
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1251'), targetEncodingCharset );
}


I think we need to basically follow

https://cran.r-project.org/doc/manuals/r-devel/R-exts.html#Linking-to-other-packages

On Fri, Mar 11, 2016 at 9:25 AM, Daniel Lee <notifications@github.com>
wrote:

> @bgoodri, I was trying to figure out library linking yesterday and failed
> horribly. Want to talk about it?
>
>
>
> On Mon, Mar 7, 2016 at 4:24 PM, Bob Carpenter <notifications@github.com>
> wrote:
>
> > +1 for sans-#; I don't even like it in Jupyter/knitr.
> >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16546966
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16645204>
> .
>


What about just `NetworkError`? or is that too ambiguous? `NetworkStructureError` is okay, just a bit long, but there are other long exception names.


s/loca/local/


s/requst/request/


Условия лучше  поменять местами.


TeamCity Civic Assembly :: Build all [Build 94](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1870&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 70, passed: 1348, ignored: 3 Build time: 00:17:53
### Failed tests
```
Venue tests: Venue type: Should Not Add Venue if Cancel button pressed: <no details avaliable>

Venue tests: Venue type: Should Add Venue: <no details avaliable>

Venue tests: Venue type: Should Add an Venue and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Not Delete a Venue: <no details avaliable>

Venue tests: Venue type: Should Delete an Venue: <no details avaliable>

Venue tests: Conference place type: Should Not Add Conference place if Cancel button pressed: <no details avaliable>

Venue tests: Conference place type: Should Add Conference place: <no details avaliable>

Venue tests: Conference place type: Should Add an Conference place and CONTINUE: <no details avaliable>

Venue tests: Conference place type: Should Not Edit a Conference place: <no details avaliable>

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


I think even these circle does by default for rails app, right?


I believe we should add some wheels to this aka: (in training*wheels)


Thank you. I couldn't find that in their doc yesterday. Are you going to
give it a try or shall I?

btw, the StanHeaders build creates a StanHeaders.so, not libStanHeaders.so,
which is the typical naming convention. know how to change it?

On Fri, Mar 11, 2016 at 12:03 PM, bgoodri <notifications@github.com> wrote:

> I think we need to basically follow
>
>
> https://cran.r-project.org/doc/manuals/r-devel/R-exts.html#Linking-to-other-packages
>
>
> On Fri, Mar 11, 2016 at 9:25 AM, Daniel Lee <notifications@github.com>
> wrote:
>
> > @bgoodri, I was trying to figure out library linking yesterday and failed
> > horribly. Want to talk about it?
> >
> >
> >
> > On Mon, Mar 7, 2016 at 4:24 PM, Bob Carpenter <notifications@github.com>
> > wrote:
> >
> > > +1 for sans-#; I don't even like it in Jupyter/knitr.
> > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16546966
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16645204
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16649467>
> .
>


Yea :) - this key belongs to the mitro team and we've already removed a fair amount of their private stuff and alerted them that they should revoke, reset etc.


I mean that i have to revert your change. true?


Is there a reason for so many methods to have empty comment blocks before them? Specifically like this:

    /**
     *
     */
    public JMSResponse consume() {
        Message message = this.jmsTemplate.receive();
    ...


I felt the bio was lacking


Это выглядит очень сомнительно.


Looks good!


Can we use an existing class for this and we avoid all the `setSection`s?


Issue #8 should be fixed by this together with removing the ResetEncoders.


I have a script which will  do Exec::run and save the output into a separate log file here separate  log is creating  with out error but the Exec::run command output not entering into the log

please give some ideas

@load base/frameworks/notice

module Test;

export {
	redef enum Notice::Type += {
		Test::Found
	};
redef enum Log::ID += { LOG };

    type Info: record {
        ts: time        &log;
    tester: vector of string &optional;

    };
}

redef record connection += {
test: Info &optional;

};

event bro_init()
	{
Log::create_stream(Test::LOG, [$columns=Info, $path="testing"]);
	local r1 = " ls \ ";

local cmd = Exec::Command($cmd=r1);
        when (local res = Exec::run(cmd))
        {
local rec: Test::Info = [$ts=network_time(),$tester=res$stdout];

    Log::write(Test::LOG, rec);
# print fmt ("%s",res$stdout);

        }

   }


Уберите, пожалуйста, все математические расчеты из этого класса.


The following doc(s) have been deployed to the [simpleurls](http://simpleurls-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/0fad1f84ffcc7abbcc9f700aa5213ce278c24911/app/config/sculpin_site.yml)
- [scripts/merge_conflicts.sh](https://github.com/pantheon-systems/documentation/commit/6d1e305bf42050609e9a77217f9603fb1e73fbcc/scripts/merge_conflicts.sh)


Hi, am working on a project that involves saving sports data from a 3rd party API.
i just figured out how to save the data to my Postgresql database but some of the data is Images.
How do i save and query images in Postgres and SqlAlchemy?
Thanks


Yes, I love  to see a separation. Much easier to read. Spring habit, but won't fight over it.


training + wheels


Why not:

```
for host, block_cidr in self._get_pool_affine_blocks(pool)
```

I'm also not a big fan of the name `_get_pool_affine_blocks` - it doesn't really say what the method is doing.  Maybe `_get_host_block_pairs(pool)` or similar?


Thanks! Noted regarding printing Python version

On Fri, Mar 11, 2016 at 11:46 AM Reshama <notifications@github.com> wrote:

> RE: 01-install.md
> hello @peter-gray-rasmussen <https://github.com/peter-gray-rasmussen>
> cc: @reshama <https://github.com/reshama> @aramnath
> <https://github.com/aramnath> @Vinoad <https://github.com/Vinoad>
> @JoelWesley <https://github.com/JoelWesley>
>
> Good job in completing the 01-install (python) section. [image:
> :thumbsup:] emoji
>
> To print the version of python on your computer, the below two commands
> will work.
>
> $ python --version
> $ python -V
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/peter-gray-rasmussen/dsp/commit/f56462f574b576c88b15c24512b4da8120286657#commitcomment-16648994>
> .
>
--
Sent via mobile


@mattsm I'm fine moving the defaults into there. I'll do that and resubmit.


What are the other objectives?


The instructions say we need to change to build process to put
libStanHeaders.a under lib/ rather than putting StanHeaders.so under libs/.
I have makeup exams going on ATM, but I can try to figure it out later.

On Fri, Mar 11, 2016 at 12:05 PM, Daniel Lee <notifications@github.com>
wrote:

> Thank you. I couldn't find that in their doc yesterday. Are you going to
> give it a try or shall I?
>
> btw, the StanHeaders build creates a StanHeaders.so, not libStanHeaders.so,
> which is the typical naming convention. know how to change it?
>
> On Fri, Mar 11, 2016 at 12:03 PM, bgoodri <notifications@github.com>
> wrote:
>
> > I think we need to basically follow
> >
> >
> >
> https://cran.r-project.org/doc/manuals/r-devel/R-exts.html#Linking-to-other-packages
> >
> >
> > On Fri, Mar 11, 2016 at 9:25 AM, Daniel Lee <notifications@github.com>
> > wrote:
> >
> > > @bgoodri, I was trying to figure out library linking yesterday and
> failed
> > > horribly. Want to talk about it?
> > >
> > >
> > >
> > > On Mon, Mar 7, 2016 at 4:24 PM, Bob Carpenter <
> notifications@github.com>
> > > wrote:
> > >
> > > > +1 for sans-#; I don't even like it in Jupyter/knitr.
> > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16546966
> > > >
> > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16645204
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16649467
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16649511>
> .
>


Instead of manually deleting the file it would be more reliable to open the file with automatic deletion on dispose. This would alsoe guarantee that the file gets deleted when the process dies or is manually killed.


Можно переписать следующий код без использования такого количесва констант?


受教了


fixed the changelog and tagged 10.0.5 (no other changes).


into bft I mean :)


there is no difference of what was being done. The function would return something different if type was null (that is not the case)


typo: no -> not


TeamCity ReOpenLDAP :: master [Build 46](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1876&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


Looks good!


Вообще это больше похоже на enum. Или на отдельный класс, с возможностью изменения формата и локализации.


Can we move this to a separate commit `Add contributor logos`?


上次的


Same


use Copy-Item and throw an exception if it fails, saying that you need to have the windows-curtin-hooks submodule


All of them, actually :)


Nice


:x:


else if ( language == 'hebrew' ) {
// Hebrew *is* Windows-1255
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1255'), targetEncodingCharset );
}
else if ( language == 'russian' || language == 'ukrainian' || language == 'bulgarian' ) {
// Cyrillic *is* Windows-1251
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1251'), targetEncodingCharset );
}


@poqudrof @filib needs a bit more love, but coming on


Same here


Yay!


hello @reshama @aramnath

isn't that what i put above? `python --version` in the terminal

sorry, i'll do a better job labeling answers!


Кстати, лучше ставить пробелы рядом с присваиванием. Да и вообще любыми операторами. И после знаков препинания.


Yup, exactly. Sorry about that. Keyur and I got crossed up there.


Yeah, `base.py` would work in that there are no circular imports. As long as we think it's "base" enough to deserve a place in there. Since we're now inheriting it for a lot of things, I think I would be okay with that. And it's possible that even more things (i.e. NeuronTypes) will inherit from it in the future.


Two things:

1. If we express the variance in matrix form, we should also express the expectation the same way for consistency. Would you mind amending the commit accordingly?

2. We could express the matrix as in Wasserman (per the shot below), which be a bit clearer at the cost space. Have you tried the more verbose version?

![multinomial](https://cloud.githubusercontent.com/assets/53797/13709505/43694b98-e769-11e5-8d3a-7f26a6b3f048.png)


Check out the patch I pointed you to. Those changes look required to me.


we're all on the edge... of our seats for 2.0 :P


Don't you miss BtnFailure here?


The following doc(s) have been deployed to the [simpleurls](http://simpleurls-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/0fad1f84ffcc7abbcc9f700aa5213ce278c24911/app/config/sculpin_site.yml)
- [scripts/merge_conflicts.sh](https://github.com/pantheon-systems/documentation/commit/6d1e305bf42050609e9a77217f9603fb1e73fbcc/scripts/merge_conflicts.sh)
- [source/_partials/navbar.html](https://github.com/pantheon-systems/documentation/commit/5ad9f2c0ad0f59b6b43a3ac951584a72a1710ce9/source/_partials/navbar.html)


'hebrew' : 'Hebrew',


@landreev I think the only thing you're missing is that @michbarsinai is the one who worked on all the API blocking code, not me :)

So it's not me making it an optional step. Maybe we should change the blocking script to run automatically... the only problem is that I rely on the API endpoints being open in my REST Assured tests at https://build.hmdc.harvard.edu:8443/job/phoenix.dataverse.org-apitest-develop/ ... maybe setup-all could accept a flag for --insecure and I'd use it on that http://phoenix.dataverse.org server... this is all #1886 territory.

Oh, speaking of docs, since you're hacking on the Installation Guide anyway, you **could** try to knock out a few of the dozen or so open issues at #2944.


int will never be less than 8 bits so there is no problem here. Am I missing something ?


@amn34
Yes, it just gives two possible ways of doing it.  (your answer is correct too :smile: )


打印一个函数


Or maybe it's your attitude?


You're totally right. It's good as-is.


Is this needed?


Shouldn't it be
@timestamp ||= Time.now.utc


"xmpcore" dependency should go under dependencies tag.


Hi @flovilmart, here is the log:

http://pastebin.com/GB9TThv1

I printed all the request so you can see what was sent.
When he started he didn't have account, he login anonymous, after facebook, then logout and tries to login again and then fails.

Thank you for your help.


To minimize penalty points.


@klq, ready for HW#1 review.


You are actively wasting my time. This has NOTHING to do with semantic versions. I doesn't matter is version are semantic or not, they have to be compared.

I built Package Control to help users. Making version numbers in the trillions is actively *user hostile*.

Either way, I will remove any packages or sources that break the Package Control site/channel/crawler until someone submits a patch to fix a given issue (in this case the comparison of version number components > 2 billion), or I get around to it.


Hey @webMattson, I'd appreciate you stop mentioning me in commit messages. ;-)

The way github works I get an email every time you write '@maciej'.

Thanks!


input lines for the salted format are:

$restore_tester$valid$SALT

SALT can be any 4 character string, I like using SLT1..SLT9 but others can certainly be used.


Sure:
```javascript
var parentObj = { constructor: 'foo' };
var childObj = Object.create( parentObj );
assert.ok(!jQuery.isPlainObject(childObj), "isPlainObject(Object.create({constructor:'foo'}))");
```
Was okay in 1.x because of the try ... catch, now throws TypeError instead.


This looks a bit insane. Are we sure it doesn't break things like pending actions on the old controller, etc?


This is not in the PR description - and thats important change (fix)


@oritwas yeah.. should have been in a separate commit, but otherwise the test/rgw can't initialize


//好的


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1580 of 1580 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)


Haha, I just wrote the same code, but then I changed it again, because we are not sure our users have such modern Node versions. Everything else here is OK for 0.10.x, so maybe this should be too:

```javascript
var util = require('util');
...
...

  if(opts.headers) {
      Object.keys(opts.headers).forEach(function(key) {
          cmd += util.format(' -H  \'%s: %s\'', key,  opts.header[key]);
      })
  }
```


Yes, because trace element:
 * has no derivatives in cell space.
 * is undefined on the cell interior.

And it now throws exception for attempting to do these instead of returns NaN.


This line is also problematic, because we are getting string from calling `dumps()` and it breaks. The way to reproduce it is for example set `default={}` on the Model field.


@qbikez Unfortunately, it hasn't been publicly released yet. I think it will be available around the same time as ASP.NET Core RC2.


I honestly don't have strong opinions about this.  If I were implementing it, I'd probably just say that plugin ids must be snake cased because it seems like the easiest solution.

Regarding the angular direct names thing: they are moving away from transforming direct names in angular 2.  If you have a camel cased directive name, you have to invoke it as camel cased in your template.


Actuator **is** optional


Correct parameter is s:<no>


What about adding bool argument `clearHistory` to `::clear()` defaulting to true instead?


As far as I can see the logs, I know what's going on and this will fixed by #952.


Specifying min and max versions no longer necessary?


nice catch!


imports - pep8


You can shorten this to `foldLeft(0)(_ + _.length)` or `map(_.length).sum`.


aqui esta la base de datos


test


But why

> Am 11.03.2016 um 15:52 schrieb Tim (robske_110) <notifications@github.com>:
>
> Complete update for deu.ini
>
> @thebigsmileXD need help
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/robske110/ClearSky/commit/bc50dc906430f5451b72163242b1a74093656b26


Why don't you define all colours in one file?


@mlomnicki have you tested it with rails_event_store Active Record repository?


yeah I will extract it to a separate PR


unit? seconds?


It's a bit confusing the way this is broken up. I'd do it like this:

```
var result = ans && this.bindings[this.bindings.length - 1];
var traceEntry = this.getTraceEntry(origPos, expr, result);
```


oh got it! i saw your slack message and thought it seemed like i had skipped the question. thanks! :smiley:


Without this patch, you can already return fields that are stored in raw json using the [json] response writer.

The other difference between the GeoJSON response writer and the [geo] transformer is that
* [geo] just deals with the field value
* wt=GeoJSON modifies the SolrDocumentList >> FeatureCollection and SolrDocument >> Feature (w/ geometry)

If the Shapes are only accessible in ValueSource, you would need to use both to get a the expected output (GeoJSON with geometry)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.705.8629a6f08289052f2c6c7a3e8fba3d8561e0e1e3](http://build.palaso.org/viewLog.html?buildId=70261&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3438, ignored: 126 Build time: 00:04:54


Alas, the problem is not completely solved by this patch. The top block in a repeat block flow remains highlighted.


number of google pages?


I'm with court, just make plugin ids snake cased


Nit: Variable names shouldn't start with an uppercase letter; I'd probably name this `cstNode`.


@mpraglowski nope, but I can do it. I just wanted to open this PR to make sure you are okay with that proposal. Are you?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.714.8629a6f08289052f2c6c7a3e8fba3d8561e0e1e3](http://build.palaso.org/viewLog.html?buildId=70262&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3418, ignored: 107 Build time: 00:05:50


Ditto, I'd go with `cstChildrenNames`.


I think you can shorten it to: ``Seq(Return, CursorUp(1), ClearLine) ++ text.diff(Seq(NewLine)) ++ Seq(newLine)``


absolute/relative path?


Hi;
Perhaps you could drop the 4th clause from the new file(s)?
We prefer this template for new files:
https://www.freebsd.org/doc/en_US.ISO8859-1/articles/committers-guide/pref-license.html


nice catch


Actually I would remove this method completely. It's used only once, in the constructor.
How about?
```ruby
metadata[:timestamp] ||= Time.now
```


Putting something in lib was done in rstan 1.0 up to when Stan was totally just header files. See
https://github.com/stan-dev/rstan/blob/v1.3.0/rstan/rstan/src/install.libs.R
and https://github.com/stan-dev/rstan/blob/v1.3.0/rstan/rstan/src/Makefile.win

@bgoodri do you refer to that?


This should be `t.equal(manyCSTNode, altCSTNode)`.


good point


That is great!!! I will start working with your branch.
Thank you.


Lol


Same as above.


or u could just add `startdicehelp: ['/startdice <bet> - Starts a game of dice. Requires +'];` below the commands themselves


a


I had forgotten that, but it is similar to what we need for StanHeaders.
I'm try to figure out what needs to go into StanHeaders/src/Makevars to do
it at installation time.

On Fri, Mar 11, 2016 at 12:28 PM, maverickg <notifications@github.com>
wrote:

> Putting something in lib was done in rstan 1.0 up to when Stan was totally
> just header files. See
>
> https://github.com/stan-dev/rstan/blob/v1.3.0/rstan/rstan/src/install.libs.R
> and
> https://github.com/stan-dev/rstan/blob/v1.3.0/rstan/rstan/src/Makefile.win
>
> @bgoodri <https://github.com/bgoodri> do you refer to that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16650120>
> .
>


I would've just used Microsoft.VisualStudio.Shell.ThreadHelper.Generic.Invoke, it is easier to use than NuGetUIThreadHelper (since it doesn't need async delegates and await).

But anyways... it seems like Initialize() can be called each time GetProvider() is called. Initialize() shouldn't spend time moving to the main thread if it doesn't need to do the import.


Use `++` operator here which is defined on `Seq`s: `Seq(this) ++ text ++ Seq(ctrl.Reset)`


Nice...I knew there had to be a better solution than that. :)


lgtm, with the suggestions that I mentioned.

@alexwarth, to answer your question -- maybe name it `result`?


Thank you for this great work!
We are already using LLVM-3.8 and currently using -fstack-protector-strong.
It would be really nice if you could update it, or perhaps even post it in phabricator (reviews.freebsd.org).


Not sure why the difference, but this way works in both Hive versions I have available.


Thank you very much @flovilmart .


Or you could leave the method but return timestamp from metadata


I could, I'm just really not a fan of the `colors.scala` representation.  It makes it harder to find on the command-line or browsing a repo so I just create separate files to keep it clean.


the fix here is similar to that staticToInstance helper... we need to convert an anonymous function into an "instance" function that is called with a "self" argument.


sounds good!


This need to be tested - it might break a few things ;)


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1064.8629a6f08289052f2c6c7a3e8fba3d8561e0e1e3](http://build.palaso.org/viewLog.html?buildId=70263&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3418, ignored: 107 Build time: 00:05:40


s/and/an


Fixed and pushed changes.


Sure feel free to remove it! :)


>
> @alexwarth <https://github.com/alexwarth>, to answer your question --
> maybe name it result?
>

Hmm, I don't know. That's just as undescriptive as `ans`. But I don't want
to make a big deal about it, so I'm happy to go with `result` if you guys
are OK with it.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cdglabs/ohm/commit/16cc9879b2d0df849ae61fc9acf17a4cf9769a18#commitcomment-16650183>
> .
>


@luan did you have food on your mind?


Reference to object `$e` is already defined - no need to create new object `$(e)` here.


hey you there?


OK

On Fri, Mar 11, 2016 at 9:34 AM, Juan Julián Merelo Guervós <
notifications@github.com> wrote:

> Rewrite abstract
>
> I have done several changes, mainly on the client, as well as extensive
> experiments. Main change now is that overlap is almost entirely eliminated,
> that is, it is quite improbable that the clients started in a previous
> experiment contribute to a new one. However, this also makes changes in
> behavior of the users: they see changes if they leave it long enough. So
> let's see what goes on.
>
> So far only thing changed is title and abstract. Please @mariosky
> @unintendedbear @fergunet take a look
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JJ/modeling-volunteer-computing/commit/b4f6154c86a0c1a621ff51c4748ae897d8bea286
>

--
Dr. Mario García Valdez
Instituto Tecnológico de Tijuana
Móvil: +52 (664) 123-7806


Don't we have volumes without persistency?


@paveljanik It would work, but is it necessary to do that?


This technique carries some caveats... assigning to/from instance methods will create wrappers around the assigned function.  As you can guess, this adds overhead every time the wrapped method is called.  So, it's not a great idea in general.


Hello,
it seems that running rpi2-gen-image.sh from its own directory is now required, is that really wanted ?
I usually run it from an empty dir, using its absolute path...


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.706.91da297dbf2475daa14e83433b72f89a9b6a0b13](http://build.palaso.org/viewLog.html?buildId=70265&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3438, ignored: 126 Build time: 00:06:02


I do not like the look with `.fold-line`. Showing `delete_attachment` only if there is an attachment makes sense.


Why use a lambda here?


Please describe in the Changelog file what your code does instead of what the feature request was. E.g. start with "Add action to open..."


This is the incomplete fix for the issue [#20](https://github.com/audiocogs/flac.js/issues/20), because in case the `stream` does not have enough data while the last header block is being parsed, loop condition will never be `True` again.
I used the next approach for resolve that problem:
```javascript
if (!stream.available(this.size)) {  // Check if there are enough data in the `stream` for parse current header block
    if (!this.readBlockHeaders) {    // Check if it's block headers parsing stage
        stream.rewind(4);            // Rewind stream 4 bytes back (one UInt8 and one UInt24 values have been read from the `stream`)
        if (this.last)               // Check if it's last header block
            this.last = false;       // Reset value to default (it will be parsed again in the next loop repeat
        }
}
```


I'd say just remove those comments completely. The `clojure-mode` package comes with `clojurescript-mode`, `clojurec-mode`, and `clojurex-mode`. And the package already sets those in `auto-mode-alist`, so there's nothing to gain with this line above.


And here?


looks good


I'm open for pull requests here. I think there is some room for
improvement.
On Mar 11, 2016 18:08, "William Blum" <notifications@github.com> wrote:

> Instead of manually deleting the file it would be more reliable to open
> the file with automatic deletion on dispose. This would alsoe guarantee
> that the file gets deleted when the process dies or is manually killed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fsprojects/Paket/commit/66ca1d50e9f84291a32948e4f3f3dff02aba399f#commitcomment-16649601>
> .
>


Interesting. Thanks for digging into this! In my particular case, I am using [a Node package that spawns a child process](https://github.com/steelbrain/consistent-env/blob/992aec54198c4ddc05e43a6c6e591857cd5ba986/src/helpers.js#L30) to collect some information. Do you know if there is a way to do this in a way that will run `.zshenv` (and similar files for other shells) only?


Actually I don't know what `e` is referencing here. That variable does not exist.


TeamCity libpalaso :: palaso-precise64-lfmerge Continuous [Build 2.6.22.8629a6f08289052f2c6c7a3e8fba3d8561e0e1e3](http://build.palaso.org/viewLog.html?buildId=70264&buildTypeId=PalasoPrecise64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3418, ignored: 107 Build time: 00:06:06


What is the reason to have the first letter in upper case?


+## Functionality include 2 items to cache, calculate and retrieve/set the inverse of given matrix:
 +## 1 - makeCacheMatrix()
 +## 2 - cacheSolve()


Return a  ConcurrentLinkedQueueBuilder here instead of ```null```


Done.
During the fixing documentation i found that we are raising wrong exception type `INVALID_FUNCTION_ARGUMENT` for the overflow situation.
I changed the exception type for overflow to `NUMERIC_VALUE_OUT_OF_RANGE` and changed the message to `format("DECIMAL value '%s' is out of range", value.toString())`.


Ok.


yup agree.  wasn't too happy with the name myself.


then here will be just pool = builder.build


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1580 of 1580 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.85%25-green.svg)


I like it, but could it be renamed to just `reset()`? That way if there was anything else we need to reset it'd all happen in one command.


Please be sure to not squash this commit but rather remove so that there isn't somehow binary data kept


I *think* if you don't have the SCC admission controller (Kube) or not turned on (for some reason) then we shouldn't show this field at all.  So would transform this into an if annotation set, print, otherwise do nothing.


small comment on the queue builder


test


This should be documented in doc string since it's not obvious from signature. Nevertheless I think it should not be ignored for a `Request`.


Why `boost::find_if` instead of `std::find_if`? Also, is this guaranteed to always return an iterator that can be dereferenced?


This looks like a typo?


Why `boost::find` instead of `std::find`?


`boost::find` takes a Range concept while `std::find` requires two iterators.


If I'm reading it correctly, this block of if/if else statements will only ever add one of the headers. I believe what is wanted is just a series of if statements so that all JMS headers could be added.

If that is what is desired I believe this section should be reworked so that there is a single for each loop and then checks against the key for each type. Perhaps, start with the the attributes that don't start with the JMS header since that is likely where most attributes will hit.


This broke run tests


@girving I would like to disagree on this since the tests are relatively contained in their own class, but I'll change it if you like.


`insert` is idiomatic


IMHO they do not share that much. E.g. we do not store a name of the original file of an invoice. Generation of filenames is different. Invoices allow only JPEG and PDF, while documents support everything. ...
The common stuff is already in the `FileMagic` and `MIME::Type` modules so I do not see much improvement to share code. An those which might can be shares is about 2-3 lines, where moving it to a helper method would make the code less readable.


Yes , It should be parameterizable on exo-property {exo.addons.lecko.dump.name}


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


I hate to say it but I like it better with the lambda instead of trotting out a clumsy wrapper around `std::vector`.


the clock already has it. `this.clock` is just the plain sinon clock instance.


I think it is best if you extract the code to open the consoles to the class JabRefDesktop.java as this seems to be the class taking care of any desktop related actions. See open folder action as a hint


I'd love to profile the sandbox vs just using timers. I have a hunch setting it up and tearing it down is relatively expensive. But I may be crazy.


Ok; fair enough. Fixed.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.715.91da297dbf2475daa14e83433b72f89a9b6a0b13](http://build.palaso.org/viewLog.html?buildId=70266&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3418, ignored: 107 Build time: 00:07:03


What is the impact of this change to composite users?


Now, that's a good catch


this is the right thing to do by default imo.  you need the .so symlink in order to link against the library.


Can you add these dmsetup commands to dmsetupClient so we can unit test this function?


The test for this needs to change


A precondition of the function is that at least one path element has a currency. Debug builds should throw an exception if this precondition is violated.


latex-documentation-myoldmopar (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


FYI, I have successfully debugged from Android Studio when opening `epub-samples/mantano-test-lcp.epub` with the following branch tips (each repository / submodule):
https://github.com/clebeaupin/SDKLauncher-Android/tree/mantano-lcp
https://github.com/clebeaupin/mantano-lcp-client/commits/android
https://github.com/clebeaupin/readium-sdk/tree/mantano-lcp
https://github.com/readium/readium-shared-js/tree/develop

Side note: I noticed that the Gradle build script does not contain the epubReadingSystem task, so I guess at this stage it is not strictly a fork / descendant of the readium gradle-experimental branch, right?


I put image with attribute primary in the begin of array in order to it be the first image in carousel


Can you add this command to dmsetupClient so we can unit test this function? We may need to rename it to something more general than dmsetupClient, but that could come later.


Multiple times "transfered" is used (validateJmsHeadersAndPropertiesAreTransferedFromFFAttributes). The correct spelling is with two 'r's, "transferred".


This is not right. It should be бла-го-4слов-лѧ́-хꙋ.


Ditto here.


This is not right. дом-ні́-ка was correct.


I'd rather import these


Hi, Emmanuel


Don't know if necessary, but it would be elegant :+1: Simple default + one `if` in `clear` and no hara-kiri with temporary storage for history etc.


See https://github.com/CounterpartyXCP/counterwallet/issues/774

I suggest s/spec=//


Seems this introduced a bug in counterwallet.io


It's showing up as a circle but the label and the class name say "mt_intratio--square'


For some reason, some of the entries got messed up. I think I must have made a mistake.


This is not correct.


Nothing. This simply changes the default size, not the minimum, so if the resolution is smaller than 800x600 the window size will adapt. But if it's equal or bigger then there's no need to be in 720x480 by default, which would mean the user having to scroll to do things such as accepting the license :)


o.o


forgot to select the postfix package for installation


I'm guessing we won't want to call this on the fly for each container individually, as that would result in multiple calls to create/remove the metadata snapshot and to thin_ls in parallel. At the very least, you'd need a mutex to guard creating the metadata snapshot and removing it.

I think a better approach might be to have a single goroutine that's responsible for periodically getting the thin_ls data (snapshot, thin_ls, remove snapshot). Then the individual container updates could just key off of that data, which may be slightly out of date (on the order of a few seconds, presumably, and not minutes or worse). This way we're not overloading the system with O(# of containers) calls to thin_ls.


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


Use len(string) checks instead of equality/inequality when looking for empty/nonempty strings.


I like this a lot better :+1:


Looks good.


Try changing semantics. For example:
`parser::State::Pointer state(parser::State::Data::create());`
It probably could make:
1. class hierarchy simpler;
2. builder unnecessary;


Use query attribute (project.repo_url?)


Looks good!


There was a typo. The code should be:
`name="'+(optName!=undefined?optName:"multiselect_"+id)+'"`

I will send a new pull request with it revised.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18902](https://teamcity.mesosphere.io/viewLog.html?buildId=161258&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:28


gör inget mer :P


That would be better yes. It is a long time that I wrote it and I dont remember well everything.
First thing that I see is that spare is of type int (4 bytes) and uc_usp_ptr is of type void* (4 or 8 bytes). That's probably why I choosed to use a union.


const ?


Где проверка на асинхронность, таймаут?


Now that we're keeping the scripts in a single branch, we could remove this block and the separate `.scripts` directory entirely, and use the script file from the local checkout instead. This would improve stability in use cases like mapbox-gl-native. The current behavior can produce broken builds of past revisions, for example during git bisects, when the tip version of some `script.sh` is incompatible with the version of `mason` / `mason.sh` that mapbox-gl-native has pinned. Using `mason`, `mason.sh`, and a `script.sh` all from the same checkout would make that much less likely.


jag e klar men jag kan inte ladda upp koden


Yeah, you're right, sorry -- it's not better.

Now that I think about it more, I think it should be `cstNode`, and we should either pass a node or null.


This may work on Endless, but we can't really depend on `dpkg` being generally available, can we? Can you find a way to do this which does not depend on `dpkg`?


Alguna razón para no querer .ruby-version en el proyecto?


Likely a mistake.


i think we should have stable id / link to object for these layers.


circle runs `rake db:schema:load` and we need to run `rake db:migrate`


Thanks, Jason!


I opened the issue mentioning that PC is now requiring something that I dont need, and that I consider the current approach OK , as it is working right now. https://github.com/wbond/package_control_channel/issues/5288
You telling I wasting your time giving you feedback about something I consider an issue. I'll remember that.

 I, then adapted the version of the package based on the feedback you gave https://github.com/wbond/package_control_channel/pull/5289 and it broke something, that I  didn't expect to broke, and that, is not even my fault if PC/site/software does not validate the tags/versions correctly.

> I built Package Control to help users. Making version numbers in the trillions is actively user hostile.

I disagree, even that I didn't want to give a trillion, just a date with the format of your requirement, looks like it didn't work, Im not interested to discuss it.


Yeah, that works for me!

On Fri, Mar 11, 2016 at 9:57 AM, Patrick Dubroy <notifications@github.com>
wrote:

> Yeah, you're right, sorry -- it's not better.
>
> Now that I think about it more, I think it should be cstNode, and we
> should either pass a node or null.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cdglabs/ohm/commit/16cc9879b2d0df849ae61fc9acf17a4cf9769a18#commitcomment-16650754>
> .
>


A lot of duplicated code. I do not like that


lägger in det du skrev nu


I was trying to save the trouble of an anonymous class, do you still want it?


What about naming convention?


As it is only one time use.


Good spot! I notice it happens when you click mrtgview of a graph and then open the real-time window of that graph as well.

One of the latest commits seems to have broken real-time functionality altogether as well, i'm trying to find out why.


I don't remember well, I think I had hesitations and I made it compatible with the regular stack.


I'd like to keep the getters and setters consistent,  changing the name of the new method to `setCreated` and the description to:

>Set when this entry was created.  If not explicitly set, the channel update time is used. Useful when entries are created and updated at the same time (offline mode, queuing to avoid rate-limiting, etc.)

...or something similar that isn't specific to rate-limiting, as this method will have other uses.    If you want to add a commit to PR #5 with these changes, the rest is ready for merging.


It's better to store images in a storage service like [Amazon S3](https://aws.amazon.com/s3/), [Amazon Cloudfront](https://aws.amazon.com/cloudfront/), or [Google Cloud Storage](https://cloud.google.com/storage/), and then store a URL to the image in a Postgres database.

This way, you use each technology for what it was made for - object storage for storing media; database for organizing and querying data.


В задаче JS-9 предполагалась работа модуля с асинхронными функциями. Следовательно, тестировать нужно на асинхронных функциях. Это касается всех тестов.


Don't we still want to take clock drift into account?


Men Jonas chilla nu det är fredag kväll! :) vi jobbar vidare tillsammans på söndag!

Skickat från min Samsung-enhet

-------- Originalmeddelande --------
Från: jonazeiselt <notifications@github.com>
Datum: 11-03-2016  18:59  (GMT+01:00)
Till: axelmagnussen/DA353A_GU <DA353A_GU@noreply.github.com>
Rubrik: Re: [DA353A_GU] Förslag på hur man lånar/lämnar tillbaka (6aafb60)

lägger in det du skrev nu

---
Reply to this email directly or view it on GitHub:
https://github.com/axelmagnussen/DA353A_GU/commit/6aafb6012a91962735c9f037b40252256847197d#commitcomment-16650798


It is easy: `$scope.community.images = [new swellRT.FileObject(file)];` to set it and `$scope.community.images && $scope.community.images[0]` to get it.

Is angular-swellrt supporting FileObject arrays, @atfornes?


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.2.20.cf3ce239e033f3a65e275d114fa51e3ed9675b2e](http://build.palaso.org/viewLog.html?buildId=70279&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1045, ignored: 12 Build time: 00:08:30


2043-heatpump-translation (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (2145 of 2397 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-89.49%25-red.svg)


haha, kunde inte bärga mig :P


Based on my comment [here](https://github.com/cdglabs/ohm/commit/16cc9879b2d0df849ae61fc9acf17a4cf9769a18#commitcomment-16650754), I'd revise my suggestion to:

```
var cstNode = ans ? this.bindings[this.bindings.length - 1] : null;
var traceEntry = this.getTraceEntry(origPos, expr, cstNode);
```


Please just `== production` We shouldn't allow any derivative thereof of the word production because that just complicates everything in the long run.  However, that said, I think this is fallible anyways, if the user disables safe in production it should count.


i already opened the PR for discussion #8819, and as @yan12125 suggested, i ignored the params as they can be passed to sanitizedRequest(headers and data) and also https://github.com/rg3/youtube-dl/pull/8819#discussion_r55806836.


Fixed


I'd expect function named like this one will return True or False


It works for me as-is. No need to change


iaf, jag lade tillbaka din unBorrow-metod, tror jag börjar förstå det här med merge-conflicts o det, haha :P


Sounds good.  Thanks.


`now` is never used after this


lol


PKGOPTION_dropbear_OPTION should be exported, otherwise ignored.


2043-heatpump-translation (kbenne) - Win64-Windows-7-VisualStudio-12: 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)


Okej säg till när man får kika haha

Skickat från min Samsung-enhet

-------- Originalmeddelande --------
Från: jonazeiselt <notifications@github.com>
Datum: 11-03-2016  19:04  (GMT+01:00)
Till: axelmagnussen/DA353A_GU <DA353A_GU@noreply.github.com>
Kopia: lelanova <jelena_novacic@hotmail.com>
Rubrik: Re: [DA353A_GU] Förslag på hur man lånar/lämnar tillbaka (6aafb60)

iaf, jag lade tillbaka din unBorrow-metod, tror jag börjar förstå det här med merge-conflicts o det, haha :P

---
Reply to this email directly or view it on GitHub:
https://github.com/axelmagnussen/DA353A_GU/commit/6aafb6012a91962735c9f037b40252256847197d#commitcomment-16650891


Got what I needed from this.  Thanks!!


good to know that we can give tuple  inside tuple to isinstance.


Make it a PR ?


Bonus: resilience against domains that contain "Address"


В ближайшем будущем залью эквивалентный билд для 32-битной винды.


Нет теста на проверку именно параллельности работы. По тестам нет гарантии, что функции выполняются параллельно.

Это можно реализовать с помощью setTimeout с разными значениями и проверкой общего времени выполнения.


I'd combine both try/except blocks, lines 101-113


This will be removed shortly along with the corresponding line that adds it to the attributes. Just a workaround until apipie-bindings gets updated.


aa, ja e klar nu :)
om return saken funkar de vete katten :P


I've split this in two, following your suggestion - removing the partial in the first commit, and then moving the rendering of both partials up into the outcome so we don't need to pass through `applicataion_address` to `making_your_application`.


This is what the community website editor suggests. You might want to change its behavior as well.


Strangely, this semi-required some Javascript changes. Which, frankly, is ridiculous? Parts of code are badly written and begging to be rewritten. Then again, parts of this website almost ask to have a CMS so a webmaster could do changes instead of requiring HTML/CSS/etc knowledge.


I haven't yet time to look into the PR but I see no problems patching request on fly as I did in [`http_request`](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/utils.py#L750-L755).


@Slair1 Thanks for the pull request. I'd suggest we probably want to keep this as generic as possible, as we don't want to change this between releases. How about changing the this to /usr/share/cloudstack-mysql-ha/lib/*.jar to be in line with the other pull request.

We'll pull this in a test usage.


I don't think that deleting tests for all possible options is good idea. You say that checks are excessive as the code identical, however this is implementation details. If implementation changes (for example, because of optimization), you can easily pass an error. Won't resist to reduce a code coverage.


`map` тоже про параллельность. Такое поведение стоит проверить.


Cthulhu <notifications@github.com>于2016-3-12 1:09 周六写道：

> 受教了
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/racaljk/hosts/commit/6ae2cd1d687c344d6d497a175e0222b40e6b477a#commitcomment-16649610>
> .
>
@racaljk 没想到你那么晚睡，我都睡一轮了。
其实，上面的操作在维护程序代码的时候，最好别那么做，尤其是代码已经 push 到远程仓库后。对于程序代码，撤销改动应该毋庸置疑地用 `git
revert [commit-id]`
具体的以后有时间，你可以去看 git 官网的文档，是中文的而且很详细，我这里说的都不全面。

至于为什么要删掉那个 commit，是因为避免不明真相的群众 get 到不可用的 IP。

早点休息啊，睡眠可是很重要的，现在就应该对应高考的时间，去调节生物钟了。


In this commit, the activityInterface files were changed by mistake. Corrected in https://github.com/robotology/poeticon/commit/1d8d0e29848dbac9939fc8af1fcd35f3f9e8a542


2129_radiance_server_perf (rpg777) - i386-Windows-7-VisualStudio-12: 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)


I added this comment because upstream's code fix problems like double-promotion, for example.


@WillNetsky
Thanks for being the first person to submit! :+1: :star2:


hehe the "ize" transition


The first element always has a currency, so we are guaranteed to find one. The `assert` was overly paranoid. There is no reason I can't use `std::find_if` here, I just thought we preferred `boost::range`.


No need anymore! :-) The new version doesn't have those -U flags.


@Eric, this code is wrong, you fix it.


You positive?


15253064 - Süleyman ÜSTEK


Here we are optimising for reads, but in general, do we optimise for reads or writes? We should have a group discussion some time.

The reason I'm mentioning this is because having many indexes will make writes less performant.


I'm not familiar with the syntax, but I assume the TARGET_FILE hidl/esdl-xml has been defined elsewhere, and this construct declares hild and esdl-xml as targets which need to be built.


@typiconman I made a mistake by applying changes to the old words.txt, thus prior changes from "fixed.csv" got lost. Now it should be better. Please review cumulative changes after my second commit.


Note that this is not a new file, but the old AuthorizationFilter.java which now shares a shared abstract base class. There are just a few changes to enable passing the AuthSubject returned from login into the AuthorizedRequestWrapper.


Well, at this point, the `calculator` isn't used within the current question, so I didn't think to use a `next_node_calculation` until in a later commit when it is. I can update it, but it's expanding the scope of a value further than it's currently needed, so not sure if we have some 'guidance' on that.


2129_radiance_server_perf (rpg777) - Win64-Windows-7-VisualStudio-12: 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)


lol, trodde jag pratade med viktor :P
jag var tvungen o ändra layouten litta, knapparna blev helt konstiga när jag addade den Media-tablen :/


Нет проверки, что все функции запустятся, даже если в одной из них будет ошибка


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#RkwqLenZQBe-0qQEDV2l4A


Looks good to me.  During a 2-day period not so long ago I completely rewrite SMC to use Redux (and immutable.js) instead of Flummox and mutable state, and this massive rewrite broke surprisingly little.  This is one thing it did break. Your fix looks exactly right to me.


Yes, I left the code unchanged (other than moving to env vars instead of hard coding names, but conceptually the result is identical to what it was before). From what I could gather it is trying to always use the 32 bit msbuild even when targeting a 64 bit architecture. Not sure why, wasn't relevant to my fix so left as is.


Нет проверки, что в результирующем массиве порядок получается правильный, когда запускаются функции с разным временем выполнения.


Hash Rocket.


This can be very useful to new users and seasoned users, alike.  :+1:


Weren't you also looking at using a sprite sheet to alleviate all the separate image loads?


@blumu, using IDisposable would make sense, although I'm unsure it would provide any guarantee in case of process dying or being killed.


Indeed it should, thanks for pointing it out


The changes in this file seems unrelated to the bootloader.


Okej :)

Skickat från min Samsung-enhet

-------- Originalmeddelande --------
Från: jonazeiselt <notifications@github.com>
Datum: 11-03-2016  19:11  (GMT+01:00)
Till: axelmagnussen/DA353A_GU <DA353A_GU@noreply.github.com>
Kopia: lelanova <jelena_novacic@hotmail.com>
Rubrik: Re: [DA353A_GU] Förslag på hur man lånar/lämnar tillbaka (6aafb60)

lol, trodde jag pratade med viktor :P
jag var tvungen o ändra layouten litta, knapparna blev helt konstiga när jag addade den Media-tablen :/

---
Reply to this email directly or view it on GitHub:
https://github.com/axelmagnussen/DA353A_GU/commit/6aafb6012a91962735c9f037b40252256847197d#commitcomment-16651057


`platform.architecture()` should work


Maybe change this VERIFY to VERIFY0 to be consistent with the new code below.


For some reason that's beyond me, the way netmdcli/libnetmd works, all `printf()`s are ignored until the very end of the operation. `puts()` is the only way to get an output while the operation is running.


This have been fixed in numpy 1.8. I'll make an issue to clean that up when we drop support of numpy 1.7.1.

https://github.com/Theano/Theano/issues/4215


undefined or global?


Thanks, I should've put that in the description :P


LOL


Are all three event types required? I would have expected just change to work.


Many times chat application developers will use a debouncer or a propagation mechanism. Would consider refactoring this @kesha-antonov. This could also be a perfect use-case for a conditional operator here as well :+1: .


Updated https://github.com/google/closure-compiler/wiki/@suppress-annotations to reflect this change as well.


I made a lambda called `insert` and removed the `StrandSet` struct.


It seems a bit strange that your bootloader project deletes a bunch of USB code, especially when it leaves things in a state that seems like they're broken following this change. For instance, target_usb_setup() still exists and says it's appending interfaces, but it doesn't appear to do so anymore and no longer registers a callback. Could this be moved to a change where you make usb modifications in a working set?


Fixed


Redundant commits can be backed out. I did push as much as I could since I wasn't up-to-date with core and don't remember each of the fixes. Sorry. Next time I won't push to main branch next time.


angular-swellrt should work with arrays of images.

Anyway, I think that a single project.image or project.main-image makes sense, and if some day we need an array of images of the project/comunity, we can add the array you are proposing.


In task1, the common words also includes numbers.


So better use `@site.safe` as the only condition?


Передавать нужно const Complex<T>& obj, в следующем конструкторе - const Complex<T>* obj


Sir, I was not considering it as a bug . It remains highlighted till it's execution of it's child block is not completed. For example if forward block is inside repeat so repeat block remain highlighted till execution of forward is not complete.  Is it still a bug ?


nice work around. We had other not so nice work around elsewhere.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.191.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=70273&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:38:26


But the forward block stays highlighted even after the repeat block completes.


OK, How do i make sure that the data i am saving to my database isn't duplicated everytime i update my data from the 3rd party API.


IMO yes, because afaiw Github already enforces safe, as we do on some of our production builds here where I am.  /cc @benbalter @parkr for confirmation.


I changed it. https://github.com/achow101/bitcoin/commit/21e45a097e154be6be4a503f2181896c5f761385


Writing additional elements under if (value.hasAltitude()) means we should throw when point WITH additional element but WITHOUT altitude is created.


This entire export declaration is a noop now.


What's avg runtime of check_proxy()?


ok


I find in task 1, your result also count numbers as the most common words.


Pending actions are flushed and will be used by get_upgrade. This is the only way I know of to upgrade the  current vistrail without switching the version in the GUI, (which I have also tried to do.) The diffing code is tied to the GUI and and the upgrade code is tied to the current controller and it would require major changes to fix this. I think this is worth having because being able to diff the upgraded pipelines is very useful. The only side-effect I can think of is the version tree, but it hides upgrades anyway so should not be affected.


We need to mention the schema migrations here too.


IANAL, but I have been warned about touching other people licenses ;).


Initial version (well I forgot to upload the text based one), kinda works, tho card flip animation can take too long on slow devices and end up starting the next animation before finishing the previous, like the dealer has 3 hands or something. Should rewrite the animation to calculate itself based on time since it began. So we effectively frame skip when the device can't draw fast enough.

Works in chrome & firefox, no guarantees anywhere else.

Didn't know what to do with the images, but they're here because it won't work without them.


I don't think you need this variable


I don't think it's intended for this if-statement to go ahead if `focussedThread.stoppedDetails.reason` is `null` or `undefined`.
And since it's different than 'step', there should be a check for its falsyness.

I saw you changed this recently. Previous to https://github.com/Microsoft/vscode/commit/ff7d4095bca5de06b9a8733020471befeedfba7f, the if-statement was checking for the falsyness of the `stoppedReason`.


So instead of this, everything else need to be indented, and the line with focusToolbarRequest has one space too many. Anything I'm missing?


I'd use the same string formatting method through the code


What do you think about making these system dependencies more explicit by pulling them out into their own `Prerequisites` subsection as a bulleted list?


why have this in a before suit block?  Can't these just be statically initialized?


The 20 most commmon words or the whole output file?


thanks for updating so fast <3


We should probably have cli support that does not involve "django migrate" to stamp of migrations.


Thanks, still waiting on that zed :dogecry:


YESSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS SweetAIO kappa


I think generally it's good to optimise for reads if a bigger part of your visitors consume data.
Given that most users go to ShipmentTracker for consuming data and a big part of the insertions happen in the background, we can argue that we probably want to optimise for reads. Just my opinion.


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1580 of 1580 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)


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


> why? It's not in the make-based build system

Since PX4 build uses `-Werror`, I needed to add `-Wno-error=undef` since there were one or more piece of code using undefined macros. That seems to be fixed on master now. I'm sending the new version without this.


I'm OK with it


By repeat block completes I meant that blocks inside the repeat block is complete over specified number of times.


@timchai thanks for updating.

It's missing code blocks around code, for readability. Also, it'd be nice to add a note about linking the PR to an issue in the pull request.

Excited to see these changes as a PR!


why need FUNC_VALUE and not use iput directly?


Fixed again.


@BurgerZ Андрей, файл удалил, украинский можно возвращать


Here I'd call out the warnings specified by those `CS0*` codes


@JEG2 did you write those yasnippets or you copied from some place?


> And some others are:
> (...)

About those ones, I remember while writing the PX4 build, I went on adding only the necessary flags of type `-Wno-error=*` and `-Wno-*`. That was easy, since `-Werror` was enabled. I think the only one missing is `-Wlogical-op`.


The 20 most common words.

Sent from Outlook Mobile<https://aka.ms/qtex0l>

On Fri, Mar 11, 2016 at 10:25 AM -0800, "Alicia Reigel" <notifications@github.com<mailto:notifications@github.com>> wrote:

The 20 most commmon words or the whole output file?

-
Reply to this email directly or view it on GitHub<https://github.com/biolprogramming/assignment-11/commit/8885f44787aff7898729fa020cd126fd3c8793d3#commitcomment-16651334>.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2334 of 2397 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.37%25-yellow.svg)


there is still a minor change to be done here, but I'll merge this first. :)


@annakovale Excellent, thanks. Very ambitious testing as well!


yes delete


This wont work


Nah, I scripted those things which weren't working on OC, since September 2015 release of OC - there wasn't that much commits from others anyway, so 95% of my scripts shouldn't clash with others, even those SAI. Unless you made a lot of script to SAI changes.

Also Prince Maalchazar should be working for you now - I mean infernals from the sky xD if you added the trigger, if not some people may get crashes. Would be good if someone checked it.


I made them myself.


With this change, the script should run with the current Tax Calculator.


this and `exactTest` can probably be static


It's 2016, #pragma once is cleaner.


Apparently this artifact hasn't yet been deployed to nexus which is causing this [build failure](https://ci.openmrs.org/browse/REFAPP-OMODDISTRO/latest)


:collision: Ooops! ICL 16 fails here deducing `version` to `initializer_list` type. And the same issue exists for 'dtsample' project.

Will revert it back on the next commit :smiling_imp:


it isn't imported


SCEnter and no SCReturn ? ;)


I'd reword the last sentence to something like:

"This is used when targeting .NET Framework and Portable Class Library(PCL) targets.  You can only build a project with this specified on Windows."


And funny enough that likely won't be true in the future when we put targeting packs out, but for the meantime it is true.


Better not to hardwire the magic constant "UAZ" in the code: use the constant defined in ReachKBConstants which creates this namespace (note: which is lowercase):

```
object ReachKBConstants {
  /** The default namespace string for KBs. */
  val DefaultNamespace: String = "uaz"
```


Your wasting my time because you know that using a version number in a specific format is a requirement, yet you don't like it so you are protesting by actively trying to subvert the system, and thus breaking things for all users. Any complex system is going to have requirements that a given user doesn't need. Thankfully the requirement is pretty trivial, and gives benefits to the users.

Just start with version 1.0.0. Increase it to 1.x.0 every time you add a feature. Increase to 1.0.x every time you fix a bug. Ideally change it to 2.0.0 when you break something that worked before. Then as new features of Package Control are added, your package won't throw users for a loop. Also, it doesn't require much thought since you just increment a group of numbers based on the change you made.

Or start at version 100.0.0. I don't really care. But starting at version 20160310153451 is unfriendly to users because if they try to compare they have to look at 14 digits. Instead of say, 6 digits in three groups. Also, using a huge version breaks the current software, and I care more about the existing thousands of packages being available to millions of users than I care that you don't like the semantic version format.


It seems to be quite common for FTP/torrent apps to download files directly into the destination folder. Moving files possibly from one disk to another is annoying especially when there are bundles and the client should figure out when they are finished and ready for sharing.

The whole process of finishing bundles is still too complicated and removing this feature also helps when rewriting the code later on.

> Additionally, sometimes the client will report "TTH inconsistency" and the only way this can be resolved is to delete the temp file and allow the client to restart the download.

This isn't directly related to the temp files though. Either the other client sends invalid data, the data gets corrupted somewhere along the way to you or AirDC++ can't validate it correctly. If the data gets downloaded successfully from the same user if you delete the temp files, the issue is probably somehow related to that specific chunk that was being downloaded earlier. If you see a pattern with the errors, please submit a bug report.


Uhh ahh well.... This is stupid and ugly as hell! You should not copy a users private key to any server!
What about creating an own key and referencing it in the config?


ннннн


commenting


# TODO


>> why? It's not in the make-based build system

> Since PX4 build uses -Werror, I needed to add -Wno-error=undef since there were one or more piece of code using undefined macros. That seems to be fixed on master now. I'm sending the new version without this.

Hm, the absence of `-Wno-error=undef` actually breaks build for `px4-v4`:
```
In file included from ../../libraries/AP_HAL_PX4/GPIO.cpp:20:0:
/home/gjsousa/ardupilot/modules/PX4Firmware/src/drivers/boards/px4fmu-v4/board_config.h:137:25: error: "SPIDEV_FLASH" is not defined [-Werror=undef]
 #if (PX4_SPIDEV_BARO == SPIDEV_FLASH)
                         ^
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
```


develop (axelstudios) - 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)


please change _scrollwheel to scrollwheel (and no need to declare i as private)


_scrollwheel to scrollwheel


111247952_Remove_QUUID (lefticus) - i386-Windows-7-VisualStudio-12: Build Failed

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


The need of the addition of `-Wno-error=undef` was probably because of the recent addition of `-Wundef` on master.


please reconsider timeout


I know. This wss just a quick patch until yours is ready. We were getting some stupid stuff in production.


Hello Brady,

[Here](https://github.com/epsdky/openemr/commits/Appt-display) is the next item. It is the Onsite Patient Portal appointments display fix.

Regards,

Ian.


You could add some code before saving new data that checks if the data already exists in the database.


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


I wonder if it is possible to expand the AttributeValue array into multiple claims instead, given the usage of it will be mainly to see if an identity has a specific claim type with a specific value usually.


LK style is to use #pragma once


:tada:


Безусловно. Это я забыл обновить конфиг в insales, он должен был быть такой же в ekam: https://github.com/insales/ekam/blob/master/.rubocop.yml


:aries: :taurus: :gemini: :cancer: :leo: :virgo: :libra: :scorpio: :sagittarius: :capricorn: :aquarius: :pisces:


Can this function be replaced with upstream's CheckConnectivtyToHost?


Can this function be replaced by GetMultipleNodes?


ok hvala necu nikome dati taj url  kad mi posaljes a ti stavi neku sifru na svoj post da nemoze niko doci do te nove liste kad je napravis


please remove this setter and handle changes in the `ngOnChanges` method


Ooops, wrong commit message.  Fixed tags element to use the backend tag model to save, instead of inserting directly, so nested set stuff happens.


sucks!


TeamCity ReOpenLDAP :: master [Build 46](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1876&buildTypeId=ReOpenLDAP_Master&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:31:25


TeamCity ReOpenLDAP :: ps-stable [Build 116](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1877&buildTypeId=ReOpenLDAP_PsStable&guest=1) is now running


the map has no scrollwheel attribute. It's part of the MapsOptions


@etagwerker @mauro-oto
This changes the logo width. I pushed this directly to gh-pages.
Let me know if it's ok!


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


In theory yes, but when using with the map after it causes compilation error.


Same answer here ;-)


Mr. Nelson,

I made the code changes above to the BB-BONE-AUDI-02-00A0.dtx file located at /opt/source/lib/bb.org-overlays/src/arm/.  The top section of code for file bb-boneblack-overlay.dts is no where on my BBB, so I was not able to make that modification.

I still have no audio working.  I am using 4.1.15-ti-rt-r43 currently.  I am using (CircuitCo) BB Black Audio Cape Rev B with my BBB Rev C.  I am also using 4D Systems (4D 7.0" LCD Cape II ) and still have no audio.  Is there a workable patch that allows the two to work together?  I have been at this a while with no success.


please remove this method and use the `setMapOptions` method of this class in `SebmGoogleMap` component


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


No, because no such function exists...

(The next patch gets rid of getMultipleNodes() anyway though.)


I'd recommend hiding action links altogether instead of having dysfunctional ones. Also, there are similar translation links in the extended stats, which I think are not covered in this PR.


is this only way?


Не уверен, но походу ты где-то с кодировкой слегка накосячил.


Seth  WHAT is the int ctoi??  We talked about atoi, and i cannot find anything on the web about the c to i. And your code is awesome,


Oh, we lost the history :'(


not here, inside of terminal_validate


@dneuman64 Like this?


Palette files for creating KMZ files.


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


no (as noted in the commit message) because the upstream method doesn't take a timeout, so only works for the expected-to-succeed case, not the expected-to-be-isolated case


Во все перегруженные операторы нужно передавать const Complex <T>&a вместо неконстантной версии, так как данные переданного объекта в этих функциях не меняются. Также эти методы не должны возвращать ссылки (кроме +=, <<, =). Если есть составной оператор +=, то имеет смысл определить + через него. По той же логике != желательно определить через == (или наоборот), а их можно сделать возвращающими bool. После этого сделать все операторы, кроме присваивания, friend-методами, добавить туда ещё один аргумент.


@surrim @envygeeks I think doing either `safe` OR `production` makes the most sense because not all production builds use the safe flag (e.g. if they want to use `_plugins`) but may still want other production-like outputs (e.g. their CSS/JS minified). In a production environment, we should consider the destination and source independent of each other, whereas only relying on `safe` would cause on non-safe production build to fail if you changed/updated/removed the source.


Basic polling survey web app developed using Python Django framework.


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


Some funny variable naming going on in this file, EHresult1 is a bit weird when there is no EHresult2, and it's starting with a capital which is a bit odd.  Similar things in other tests.


Great contribution, thank you.
Can you clarify how I can access the site from my browser?
I did install Docker tools on my Mac, I downloaded the code, did a make start with a path specified.
Now what?


:bow: :+1: :cool:


Why "moot"? It's not very descriptive to someone poking around the source.


This would be a good way to start enforcing that notion though.  Before safe was a meh situation for all parties, now we enforce them to know what it is and how to use it, otherwise why have it there if it's only an internal feature?


add comment 'comes here if nothing expected'


You where the one that showed me this :)


yeah, also need to add the continuous queries.


remove TODO


Palette files for the creation of KMZ files


__API Description file syntax error on line 67__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


We should be trashing this right? This is just discarding.


@pprint possible?


use .at()?


Same bug here. 0 will not trash the card.


Изменил на стандартный метод.
return new DateFormatSymbols().getMonths()[x].substring(0, 3);


Было бы куда удобнее писать this->_im, разве нет? Тем более с указателем а таки происходит


This shows up in a few places, would it be worth having a container type with named ref, het, and home variables instead of using the triple?


commenting every conditions, please!


Correction to typo BB-BONE-AUDI-02-00A0.dtx should be BB-BONE-AUDI-02-00A0.dts.


add comment


I'd make the RRRhody and LIFEcycle logo's a separate commit as well


rename it


I didn't catch this before but 80 character wrapping is not our convention.  Can you go through and clean up these short lines in one pass?


do i need to make a new --timeout flag for this?


120 or 140 is more common.


Да, передаваемые даты всегда корректны. Теперь игнорирую исключение.


that's a good point, thanks


Yes it is.  It was added in the previous big commit.


Исправил


Клонирование подразумевает создание еще одного объекта, идентичного существующему. Сейчас же у тебя в полученном словаре ссылки на те же фигуры. Это раз.
Дальше. Раз мы, как следует из предыдущего пункта, создаем новый набор фигур, то нужно дать им какие-то имена, так как если мы попытаемся добавить в список фигур сцены еще один набор фигур, идентичный уже имеющемуся, то произойдет ошибка, ведь такие имена уже используются.

Подсказка. Используй метод Clone каждой фигуры, и какой-нибудь префикс к именам фигур.


nice find! thanks. Btw I think it should be safe to do:

if (tmp_len != 20) return (PARSER_HASH_LENGTH);


I'm okay with everything here other than the bugs I pointed out above.


Mismatch in godoc


if ((this->_re != a._re) && (this->_im != a._im)) return 0;
Логическое И не обрабатывает правые аргументы, если встретился ложный, поэтому нет смысла разделять это на две проверки


hi teacher. :santa:


Почему это maxy, если мы берем снова Х, как и в первых двух случаях?


Are there any other types of output that would return a substring "xfs" that would not actually be xfs?


Dead code?


Can you make it so attributes are in alphabetical order? Highlighting the list of attributes and pressing f5 should automatically arrange them in alphabetical order


Don't repeat this check twice: $wp_url != $wp_site_url
Check it once for both screens


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.599.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=70285&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:15:55


I don't think this will discard an entire hand. Discarding handPos repeatedly will not get rid of all the cards. Should be from 0, right?


Эт че за хрень еще?


Please review my Codes


Changed to StatementResult as per your suggestion in the discussions yesterday


Same comment as above. Should discard from 0.


Why would this be v3?  When would I get this?  Is this something I have to fix as an admin?


Sorry we need change "in your website settings" to "with your website settings"


OK so I confirm that everything is escaped, so there is no issue. However, CDATA is required because description contains HTML tags (auto-linking or markdown) which breaks the XML syntax. If you put `]]>` in your link description, the `>` will be escaped, so there is no problem.


Assumes that xfs_quota is in the path.  Is that a safe assumption?  Should you check for xfs_quota being in the path before you finish initializing the plugin?


Should this be a fuzzy equality?  Any chance of -0.0 showing up here?


use > instead of >


Not v3


Thanks.  Didn't think to look for something like this!


If I don't need to know this as an admin, it should be v4 or v5.  It should be v5 if it is extremely unlikely I would ever need this when debugging something.


app_descriptor is unused.


I've had the same doubt but checked with @carlocaione and concluded that dpkg was the best option since what platform.architecture() does is simply check the result of `file /usr/bin/python3`, which also spawns a process and is apparently worse than checking the output of dpkg.

Also, `dpkg` is available in our image even if things like `dpkg -l` won't work, but `dpkg --print-architecture` does (our chrome plugins updater relies on it, fwiw), so I think it's not a bad option. After all, we will be downloading .deb package from OpenPrinting, so in my mind it makes sense to trust what `dpkg` has to say.

All this said, I can certainly combine the output of platform.machine() and platform.architecture() to determine the right debian architecture, so let me know if you still prefer that way and I'll do it, no problem.


Up til this day, this is still an issue with active_admin_editor gem


Nit: add comma before "use"


Separate selectors to be on two separate lines


No, it doesn't solve the issue. As far as I know, the only way to overcome it is to use find but it returns a const_iterator instead of the actual value. Compiler should optimize it but I don't know if it's more readable.


comment


removed in 9ea23ac25de4009b2f3e40f3291773f7329222be


Why isn't the code above that does the same check not using this method?


Этот же метод используется для удаления композиции. Так что нужно обработать и этот случай тоже.


is this effectively duplicating (some portion of) the `Callee` logic earlier in this commit?


I'm good with these changes as soon as other comments are addressed.


spacing.


Sorry we need one mote text edit:

Please, check WP Admin > Settings > Reading page. Values in  Wordpress Address and Site Address fields should be the same. Otherwise, you will not be able to edit your website with Live Composer.


Why are you printing the error with logging vs returning it in the error?


Why? The emphasis was done on purpose.


Not v3


seems ok, if maybe not as DRY as it could be, but I'm not 100% sure. I'm kind of trusting you on this one that you shuffled all the bits into their appropriate places :)


Мы можем посчитать площадь не только композиции, но и отдельной фигуры


Аналогично


Why is this not a method on xfsQuotaApplicator?  I don't like that you have to use a &realQuotaCommandRunner here but you have an abstraction up above to handle it.


> 141 additions and 522 deletions.

Anyway, the stats look good :)


Who calls this?


nit: order these alphabetically.


lgtm


И тут тоже


The Pair<> return type is a bit strange.  It makes sense that you'd want to return how many samples it actually operated on, but the internal filtering to computeDiploidGenotypes isn't guaranteed to keep in step with this filter expression.  I'm not sure it's worth changing, but it's potentially a bit brittle.   (although the tests should catch if one but not the other was changed)


Does that mean that meteor could install NPM packages on launch as it does for the one on atmosphere? The two lines gist

```
$ git clone repoURL
$ meteor
```

is very convenient (and reproducible!).


Why is this a log and not returned with the error?


When I run this migration, I get an error

```NoMethodError: undefined method `using_mysql?' for ActsAsTaggableOn::Utils:Module```


Podés simplemente quitar el tag si querés.

```javascript
.on('click', '.calendar [data-turn-id]', function (evt) {
```


Why are these constants?


Was macht das hier eigentlich?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#O6OOyNXaRmy215ZyeWlcYw


Space in between colon


> Your wasting my time because you know that using a version number in a specific format is a requirement

A requirement that was just introduced, does not sound that crazy to me to request to you to keep the current system that all of my packages are using right now on your own software on this very moment. I don't think is a waste to ask for to keep current behavior, you see otherwise, fine, I'm not asking anymore.

> yet you don't like it so you are protesting by actively trying to subvert the system, and thus breaking things for all users

I didn't do that on purpose, I want a date you want a version so I just appended ".0.0" to a date. My first attempt was to make it more readable but semantic version does not want leading zeros. It didn't work, Im sorry if broke something.

About everything else bellow of what I quoted, wbond, please understand this: I disagree and Im not interested. Thanks


Need to use that new line at end of file config for all files


TODO


added line so tile is displayed


what is the point of this match here? isn't it just `if f.abi == Abi::RustCall { .. } else { .. }`?

(After all, `fn_type.sty` is now known to be either `FnDef` or `FnPtr`, right?)


В этом семестре пишем на С++, поэтому в классах должны быть конструкторы (не функции init), деструкторы. Память выделяется new и new[], удаляется соответственно delete и delete []. Также желательно всё, что можно, делать const


Add space between selector


Seems to work so far


La indentación estaba bien. Igualmente, para evitar el ruido es mejor que los cambios de indentación sean commits aparte.


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


> similar to how `variables` and `scopes` expansion is only remembered in the `tree`
By this, do you mean the static member `ExpressionContainer.allValues` (whose purpose seems to be determining whether the values changed)

or by implementing `getChildren`, which will cause the `tree` itself to automatically refresh the items that are expanded?