If I'm right, there are totally no any free code for those strdup allocated
strings, it's a seriously problem.

2016-02-23 20:19 GMT+08:00 Ulion <ulion2002@gmail.com>:

> when override existed key/value, previous key and value should be
> released, who allocated memory, who should release them, external caller
> have no idea about these details, the library should do it correctly.
>
> 2016-02-23 19:04 GMT+08:00 Benoît Blanchon <notifications@github.com>:
>
>> This should be clearly stated in the documentation.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/bblanchon/ArduinoJson/commit/8733f95e51a372a85fd9d5c00f79bacc46700e65#commitcomment-16264814>
>> .
>>
>
>
>
> --
> Ulion
>

--
Ulion


made it tempoarary global so i can write all my code i tests/common.js


gell geil @Trolleybus


@bestander 😔


True, but I think that sites dedicated to hosting infringing content are the rare exception.
Otherwise, the question ends up being how much is 'some'..

But as an open-source project that is run by volunteer developers, and not volunteer lawyers, I'd suggest that you stay out of the business of judging websites and putting them in categories.

In dubio pro sitō


I'm not sure but maybe you need
MaxIteration = int(d);


tried to hack loadQML a bit, doesnt seem to work


I think it's wise to keep this seperate. We officially support Raspbian, but it is known to work on other distros (and we will unofficially support them!).

Maybe make it `Designed for Ras...... (also works on most debian based distros!)


found the issue which was due to a typo which I failed to spot as timed action lists where duplicated on server using both correct and incorrect ids.


I would prefer that the .meta files in baserock were read in as yaml, as the above code is just reading the file as a text file and looking for lines matching the target filename, rather than looking for a 'files' attribute in the yaml metadata and just searching that.


ook :) not that it is so important but...
```perl
if ($self->auto_migrate && !$self->{migrated}) {
    $self->migrations->migrate;
    $self->{migrated}++;
}
```
Would be understandable to anybody. And we do not mix operators either.


Once this PR is merged and we're satisfied that it is working as intended, will we be in a position to re-enable URI equivalence for the stream?


+1 for this pull my only comment is that the link function need $attrs too so the line 13 should be like this
```link: function ($scope, $element, $attrs) {```


je tam chyba, pricom tam ma byt function->getType() != Repository::Git::GitType::MODIFIED
bude opravene v najblizsom commit-e


saw approval from gordonguthrie
at https://github.com/basho/riak_kv/commit/e2811ca5bd1e261ab3acc7bd59b81c44a480d5d7


merging basho/riak_kv/at-negate-aggregations = e2811ca5 into borshop-integration-1354-at-negate-aggregations


basho/riak_kv/at-negate-aggregations = e2811ca5 merged ok, testing candidate = 913393eb


make description == summary (it'll still be a short sentence :) )


not available with the Src folder


d should be always integer, is it not ?


Array<Array<Object>>


Array\<Array\<Object\>\>


This has been used on other steps and according to the docs (http://help.eclipse.org/luna/index.jsp?topic=%2Forg.eclipse.platform.doc.isv%2Freference%2Fapi%2Forg%2Feclipse%2Fswt%2Fwidgets%2Fpackage-summary.html - see Widget#sleep) it does something in regards to saving cpu cycles by putting the ui thread to sleep when it's not used.

Haven't studied the exact usage and implications but this can be safely removed if needed, maybe you guys have more insight into this


LGTM, thanks.


woobie doobie


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


What you say is true on a normal program running on a computer.
However, on an embedded system the caller have to care about memory allocation.

ArduinoJson uses a very efficient memory allocation strategy using a preallocated memory pool.
The advantage is that it allows ArduinoJson to work without dynamic allocation and have a very small code.
The drawback is that the memory is released when the memory pool is discarded.
This feature is what makes ArduinoJson so good in embedded environment and will not be changed.

Now, if you think there is a problem with the library, please open a new issue, because here is not the right place.
I'll delete further messages.


Sorry, I realized the strdup using the special alloc function which use
static buffer, then it not need a free call. but, for the simple key/value
override operation, specially the
   JsonObject::setNodeAt method, which should be enhanced to not set key if
the node was already existed.

2016-02-23 20:21 GMT+08:00 Ulion <ulion2002@gmail.com>:

> If I'm right, there are totally no any free code for those strdup
> allocated strings, it's a seriously problem.
>
> 2016-02-23 20:19 GMT+08:00 Ulion <ulion2002@gmail.com>:
>
>> when override existed key/value, previous key and value should be
>> released, who allocated memory, who should release them, external caller
>> have no idea about these details, the library should do it correctly.
>>
>> 2016-02-23 19:04 GMT+08:00 Benoît Blanchon <notifications@github.com>:
>>
>>> This should be clearly stated in the documentation.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/bblanchon/ArduinoJson/commit/8733f95e51a372a85fd9d5c00f79bacc46700e65#commitcomment-16264814>
>>> .
>>>
>>
>>
>>
>> --
>> Ulion
>>
>
>
>
> --
> Ulion
>

--
Ulion


Is your local instance of the cucumber boilerplate up-to-date? Because in the current package.json of this repository, selenium-standalone gets installed as a npm postinstall command after you've run npm install.


Call this function `postcssExtractToFile` rather than `myplugin`, the stack trace will be a little more descriptive.


Tsq file is an ambiguous term. It can hapilly be an ODBC script or tranaction-SQL script.


using System;
using System.Collections.Generic;

 class Program
{
	public static void Main()
	{
	   int Total = 0;
	   int Product = 0;
	   Calculate(10, 20, out Total, out Product);

	   Console.WriteLine("Sum = {0} && Product = {1}", Total, Product);
	}

	public static void Calculate(int FN, int SN, out int Sum, out int Product)
	{
	   Sum = FN + SN;
	   Product = FN * SN;
	}
}


You assume that there is only one output operator for each plan node. I am not sure if that is true. See `LocalExecutionPlanner.addLookupOuterDrivers()`. It seems that there might be 2 output operators (in different pipelines) for outer join node. Additionally, operators above original join node are duplicated with same plan node id. I think stats should be added here too.


{@link cdf.dashboard.Utils.addArgs|addArgs}


I ensure a "\n" at the end of the extra code because, if extra code does not end with "\n", markdown rendering is ugly. i.e:

This markdown....

```markdown
```hs
foo = 1```
```

makes this rendering

```hs
foo = 1```


That is correct, please open an issue.


Comment magic constatns.


Comment magic constant.


Right, that usually happens on exit and is probably an odering issue. In master I will rewrite the code to not use an extra list just for the number of synced chats. ChatView has a IsSynced property anyhow.


jo to musím smazat


good idea!


I will make a pull request, thx.

2016-02-23 20:32 GMT+08:00 Benoît Blanchon <notifications@github.com>:

> That is correct, please open an issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bblanchon/ArduinoJson/commit/8733f95e51a372a85fd9d5c00f79bacc46700e65#commitcomment-16266497>
> .
>

--
Ulion


Pretty bad form on my part but I'm more interested in getting the discussion going than learning Groovy particulars :)

A method on_every_node that takes a block would be the ideal here.
That for loop would then yield to the block. Clean efficient and a nice addition to the DSL :)


Because your plugin is doing asynchronous things with the filesystem, you should return a Promise so that PostCSS knows that the operation is async.

```
// ...
return function (css) {
  return new Promise(function (resolve, reject) {
    // rest of the code
  });
}


Make these methods promises.


Yes, this would be better; I will change this


not really. I would change it to yourproposal


Looks good. Spoken already about the possible options for the sort functionality and until we fix the cypher this works. <thumbs up>


Removed in https://github.com/CSC-IT-Center-for-Science/ansible-role-slurm/commit/e8493a25c7f79c0eace1b3621e947e33f07d009b


what is this?


In XBMC/Kodi there are more then one player possible which can play content simultaneously. So you have to determine the 'false' player (e.g. audio) and stop it. At this moment its only possible with json.


Ok, perfect.

For now, I will use your multichanel+DataIointerface (it works so good).
Maybe later, following your ideas, i will do mine.

PS: Your code is excellent, thousand times better than ladspa code.
And it does not need a external library (like ladpsa does).

Maybe it could be done for bs2b too (... ok, I exaggerate here ;-) ).

Many thanks Andrew.

Fre;D


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3252


Thanks.
By the way, I would like to ask is it reinventing the wheels that ss reimplement the resolver?
I apologize if this is a dumb or repeated question.


########################    R - ROUTINE FOR KEY INDICATORS    ######################

## 01. Download of data

GDP01_LCU_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/001_GDP by Type of Expenditure at current prices - National currency.csv", header=T, sep=",")
GDP02_USD_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/002_GDP by Type of Expenditure at current prices - US dollars.csv", header=T, sep=",")
GDP03_LCU_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/003_GDP by Type of Expenditure at constant (2005) prices - National currency.csv", header=T, sep=",")
GDP04_USD_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/004_GDP by Type of Expenditure at constant (2005) prices - US dollars.csv", header=T, sep=",")

GDPpercap05 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/005_Per capita GDP at current prices - US dollars.csv", header=T, sep=",")

GVA06_LCU_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/006_Gross Value Added by Kind of Economic Activity at current prices - National currency.csv", header=T, sep=",")
GVA07_USD_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/007_Gross Value Added by Kind of Economic Activity at current prices - US dollars.csv" , header=T, sep=",")
GVA08_LCU_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/008_Gross Value Added by Kind of Economic Activity at constant (2005) prices - National currency.csv" , header=T, sep=",")
GVA09_USD_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/009_Gross Value Added by Kind of Economic Activity at constant (2005) prices - US dollars.csv", header=T, sep=",")

GFCF010_LCU_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/010_Gross Fixed Capital Formation by Type of Expenditure at current prices - National currency.csv", header=T, sep=",")
GFCF011_USD_current <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/011_Gross Fixed Capital Formation by Type of Expenditure at current prices - US dollars.csv", header=T, sep=",")
GFCF012_LCU_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/012_Gross Fixed Capital Formation by Type of Expenditure at constant (2005) prices - National currency.csv", header=T, sep=",")
GFCF013_USD_2005 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/013_Gross Fixed Capital Formation by Type of Expenditure at constant (2005) prices - US dollars.csv", header=T, sep=",")

## 02. Divide the Value of the files by 1 million (except for GDP per capita) - This way, Value is expressed in Millions

install.packages("dplyr")
library(dplyr)

GDP01_LCU_current_mutate <- mutate(GDP01_LCU_current, Value_millions=Value/1000000)
GDP02_USD_current_mutate <- mutate(GDP02_USD_current, Value_millions=Value/1000000)
GDP03_LCU_2005_mutate <- mutate(GDP03_LCU_2005, Value_millions=Value/1000000)
GDP04_USD_2005_mutate <- mutate(GDP04_USD_2005, Value_millions=Value/1000000)

GVA06_LCU_current_mutate <- mutate(GVA06_LCU_current, Value_millions=Value/1000000)
GVA07_USD_current_mutate <- mutate(GVA07_USD_current, Value_millions=Value/1000000)
GVA08_LCU_2005_mutate <- mutate(GVA08_LCU_2005, Value_millions=Value/1000000)
GVA09_USD_2005_mutate <- mutate(GVA09_USD_2005, Value_millions=Value/1000000)

GFCF010_LCU_current_mutate <- mutate(GFCF010_LCU_current, Value_millions=Value/1000000)
GFCF011_USD_current_mutate <- mutate(GFCF011_USD_current, Value_millions=Value/1000000)
GFCF012_LCU_2005_mutate <- mutate(GFCF012_LCU_2005, Value_millions=Value/1000000)
GFCF013_USD_2005_mutate <- mutate(GFCF013_USD_2005, Value_millions=Value/1000000)

## 03. Deleting variable 'Value' that has become useless.

GDP01_LCU_current_mutate$Value <- NULL
GDP02_USD_current_mutate$Value <- NULL
GDP03_LCU_2005_mutate$Value <- NULL
GDP04_USD_2005_mutate$Value <- NULL

GVA06_LCU_current_mutate$Value <- NULL
GVA07_USD_current_mutate$Value <- NULL
GVA08_LCU_2005_mutate$Value <- NULL
GVA09_USD_2005_mutate$Value <- NULL

GFCF010_LCU_current_mutate$Value <- NULL
GFCF011_USD_current_mutate$Value <- NULL
GFCF012_LCU_2005_mutate$Value <- NULL
GFCF013_USD_2005_mutate$Value <- NULL

## 04. Selecting the variables we are interested in.

GDP_LCU_current.1 <- subset(GDP01_LCU_current_mutate, Item=="Gross Domestic Product (GDP)")
GDP_USD_current.1 <- subset(GDP02_USD_current_mutate, Item=="Gross Domestic Product (GDP)")
GDP_LCU_2005.1 <- subset(GDP03_LCU_2005_mutate, Item=="Gross Domestic Product (GDP)")
GDP_USD_2005.1 <- subset(GDP04_USD_2005_mutate, Item=="Gross Domestic Product (GDP)")

GDPpercap05.1 <- subset(GDPpercap05, Item=="Gross Domestic Product (GDP)")

GVA_LCU_current.1 <- subset(GVA06_LCU_current_mutate, Item=="Agriculture, hunting, forestry, fishing (ISIC A-B)")
GVA_USD_current.1 <- subset(GVA07_USD_current_mutate, Item=="Agriculture, hunting, forestry, fishing (ISIC A-B)")
GVA_LCU_2005.1 <- subset(GVA08_LCU_2005_mutate, Item=="Agriculture, hunting, forestry, fishing (ISIC A-B)")
GVA_USD_2005.1 <- subset(GVA09_USD_2005_mutate, Item=="Agriculture, hunting, forestry, fishing (ISIC A-B)")

GFCF_LCU_current.1 <- subset(GFCF010_LCU_current_mutate, Item=="Gross fixed capital formation (including Acquisitions less disposals of valuables)")
GFCF_USD_current.1 <- subset(GFCF011_USD_current_mutate, Item=="Gross fixed capital formation (including Acquisitions less disposals of valuables)")
GFCF_LCU_2005.1 <- subset(GFCF012_LCU_2005_mutate, Item=="Gross fixed capital formation (including Acquisitions less disposals of valuables)")
GFCF_USD_2005.1 <- subset(GFCF013_USD_2005_mutate, Item=="Gross fixed capital formation (including Acquisitions less disposals of valuables)")

## 05 Reshaping the object in 'wide' format.

install.packages("reshape2")
library(reshape2)

GDP_LCU_current.2 <- dcast(GDP_LCU_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GDP_USD_current.2 <- dcast(GDP_USD_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GDP_LCU_2005.2 <- dcast(GDP_LCU_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GDP_USD_2005.2 <- dcast(GDP_USD_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")

GDPpercap05_USD <- dcast(GDPpercap05, Country.or.Area + Year ~ Item, value.var="Value")

GVA_LCU_current.2 <- dcast(GVA_LCU_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GVA_USD_current.2 <- dcast(GVA_USD_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GVA_LCU_2005.2 <- dcast(GVA_LCU_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GVA_USD_2005.2 <- dcast(GVA_USD_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")

GFCF_LCU_current.2 <- dcast(GFCF_LCU_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GFCF_USD_current.2 <- dcast(GFCF_USD_current.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GFCF_LCU_2005.2 <- dcast(GFCF_LCU_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")
GFCF_USD_2005.2 <- dcast(GFCF_USD_2005.1, Country.or.Area + Year ~ Item, value.var="Value_millions")

## 06. Changing the new variable's name.

install.packages("data.table")
library(data.table)

setnames(GDP_LCU_current.2, "Gross Domestic Product (GDP)", "GDP")
setnames(GDP_USD_current.2, "Gross Domestic Product (GDP)", "GDP_USD")
setnames(GDP_LCU_2005.2, "Gross Domestic Product (GDP)", "GDP_2005")
setnames(GDP_USD_2005.2, "Gross Domestic Product (GDP)", "GDP_USD2005")

setnames(GDPpercap05_USD, "Gross Domestic Product (GDP)", "GDP_percap_USD")

setnames(GVA_LCU_current.2, "Agriculture, hunting, forestry, fishing (ISIC A-B)", "AFF_VA")
setnames(GVA_USD_current.2, "Agriculture, hunting, forestry, fishing (ISIC A-B)", "AFF_VA_USD")
setnames(GVA_LCU_2005.2, "Agriculture, hunting, forestry, fishing (ISIC A-B)", "AFF_VA2005")
setnames(GVA_USD_2005.2, "Agriculture, hunting, forestry, fishing (ISIC A-B)", "AFF_VA_USD2005")

setnames(GFCF_LCU_current.2, "Gross fixed capital formation (including Acquisitions less disposals of valuables)", "GFCF")
setnames(GFCF_USD_current.2, "Gross fixed capital formation (including Acquisitions less disposals of valuables)", "GFCF_USD")
setnames(GFCF_LCU_2005.2, "Gross fixed capital formation (including Acquisitions less disposals of valuables)", "GFCF_2005")
setnames(GFCF_USD_2005.2, "Gross fixed capital formation (including Acquisitions less disposals of valuables)", "GFCF_USD2005")

## 07. Merging all the objects into one.
GDPmerge1 <- merge(GDP_LCU_current.2, GDP_USD_current.2, by=c("Country.or.Area", "Year"),all=TRUE)
GDPmerge2 <- merge(GDP_LCU_2005.2, GDP_USD_2005.2, by=c("Country.or.Area", "Year"),all=TRUE)
GDPmerge <- merge(GDPmerge1, GDPmerge2, by=c("Country.or.Area", "Year"),all=TRUE)

GDP <- merge(GDPmerge, GDPpercap05_USD, by=c("Country.or.Area", "Year"),all=TRUE)

GVAmerge1 <- merge(GVA_LCU_current.2, GVA_USD_current.2, by=c("Country.or.Area", "Year"),all=TRUE)
GVAmerge2 <- merge(GVA_LCU_2005.2, GVA_USD_2005.2, by=c("Country.or.Area", "Year"),all=TRUE)
GVA <- merge(GVAmerge1, GVAmerge2, by=c("Country.or.Area", "Year"), all=TRUE)

GFCFmerge1 <- merge(GFCF_LCU_current.2, GFCF_USD_current.2, by=c("Country.or.Area", "Year"),all=TRUE)
GFCFmerge2 <- merge(GFCF_LCU_2005.2, GFCF_USD_2005.2, by=c("Country.or.Area", "Year"),all=TRUE)
GFCF <- merge(GFCFmerge1, GFCFmerge2, by=c("Country.or.Area", "Year"),all=TRUE)

GVA_GFCF <- merge(GVA, GFCF, by=c("Country.or.Area", "Year"),all=TRUE)

KeyIndicatorsVariables <- merge(GDP, GVA_GFCF, by=c("Country.or.Area", "Year"),all=TRUE)

## 08. Adding new variables including calculations based on the original variables.

KeyIndicators <- mutate(KeyIndicatorsVariables,
                        Def_GDP =GDP/GDP_2005,
                        Def_GDP_USD =GDP_USD/GDP_USD2005,
                        Def_GFCF = GFCF/GFCF_2005,
                        Def_GFCF_USD =GFCF_USD/GFCF_USD2005,
                        Def_AFF_VA = AFF_VA/AFF_VA2005,
                        Def_AFF_VA_USD = AFF_VA_USD/AFF_VA_USD2005,
                        EX_Rate_USD = GDP/GDP_USD,
                        INV_Ratio_USD = GFCF_USD/GDP_USD,
                        INV_Ratio_USD2005 =GFCF_USD2005/GDP_USD2005,
                        AFF_VA_GDPShare_USD =AFF_VA_USD/GDP_USD,
                        AFF_VA_GDPShare_USD2005 =AFF_VA_USD2005/GDP_USD2005,
                        Population_thousands = GDP_USD*1000/GDP_percap_USD,
                        GDP_percap_USD2005=GDP_USD2005*1000/Population_thousands)

## 09. Setting a new name for the 'Country.or.Area'  variable (= 'Country').

setnames(KeyIndicators, "Country.or.Area", "Country")

## 10. Creation of a copy of the KeyIndicators file and changing of the Country names into FAOSTAT Contry names:

install.packages("DataCombine")
library(DataCombine)
install.packages("car")
library(car)
install.packages("plyr")
library(plyr)

KeyIndicators.names <- KeyIndicators

KeyIndicators.names$Country <- mapvalues(KeyIndicators.names$Country, from=c("Bolivia",
                                                                             "Former Ethiopia",
                                                                             "Former Sudan",
                                                                             "Former USSR",
                                                                             "Former Yugoslavia",
                                                                             "Iran, Islamic Republic of",
                                                                             "Netherlands Antilles (former)",
                                                                             "United Kingdom of Great Britain and Northern Ireland",
                                                                             "United Republic of Tanzania: Mainland",
                                                                             "United States",
                                                                             "China: Hong Kong SAR",
                                                                             "China: Macao SAR"),
                                         to=c("Bolivia (Plurinational State of)",
                                              "Ethiopia PDR",
                                              "Sudan",
                                              "USSR",
                                              "Yugoslav SFR",
                                              "Iran (Islamic Republic of)",
                                              "Netherlands Antilles",
                                              "United Kingdom",
                                              "United Republic of Tanzania",
                                              "United States of America",
                                              "China, Hong Kong SAR",
                                              "China, Macao SAR"))

## 11. Deleting the Country names not included in FAOSTAT and the former Countries in order to avoid double-countings.

KeyIndicators.names.1 <- KeyIndicators.names[!KeyIndicators.names$Country=="United Republic of Tanzania: Zanzibar", ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!KeyIndicators.names.1$Country=="Yemen: Former Democratic Yemen", ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!KeyIndicators.names.1$Country=="Yemen: Former Yemen Arab Republic", ]

KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="USSR" & KeyIndicators.names.1$Year==1990), ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Czechoslovakia" & KeyIndicators.names.1$Year==1990), ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Yugoslav SFR" & KeyIndicators.names.1$Year==1990), ]

KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Ethiopia PDR" & KeyIndicators.names.1$Year==1990), ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Ethiopia PDR" & KeyIndicators.names.1$Year==1991), ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Ethiopia PDR" & KeyIndicators.names.1$Year==1992), ]
KeyIndicators.names.1 <- KeyIndicators.names.1[!(KeyIndicators.names.1$Country=="Ethiopia PDR" & KeyIndicators.names.1$Year==1993), ]

## 12. Importing FAOSTAT codes file and merging it with Key Indicator's Country names

IND_Flatfile2015 <- read.csv("/home/ileeo/Scrivania/R_stuff/KeyIndicators_RacheleBrivio/IND_Flatfile2015.csv", header=T)
IND_Flatfile2015_select<- subset(IND_Flatfile2015, select=c("Country", "Year", "FAOCode", "region", "regioncode", "region2", "subregion", "subregioncode", "developing", "LLDC", "LDC", "LIFDC", "NFIDC", "SIDS", "OECD", "EU", "MAFAP", "BRICS", "FAO_member"))
KeyIndicators_FAOSTAT <- merge(KeyIndicators.names.1, IND_Flatfile2015_select, by=c("Country", "Year"),all.x=TRUE)
colnames(IND_Flatfile2015_select)[colnames(IND_Flatfile2015_select)=="GDP_percap_USD"] <- "GDP_percap_USD_FAO"
KeyIndicators_FAOSTAT <- merge(KeyIndicators.names.1, IND_Flatfile2015_select, by=c("Country", "Year"))

## 13. Changing the variables' order

KeyIndicators_FAOSTAT.1 <- KeyIndicators_FAOSTAT  # backup Copy of the 'KeyIndicators_FAOSTAT' object

install.packages("DataCombine")
library(DataCombine)

KeyIndicators_FAOSTAT.1 <- KeyIndicators_FAOSTAT.1[c(28,1,2,3,4,6,5,27,7,12,14,13,15,8,10,9,11,23,24,25,26,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,16,17,18,19,20,21,22)]    # Changing variables' order

## 14. Adding growth rates for GDP_USD and GDP_USD2005 and ordering those new variables according to our needs

# Please, export KeyIndicators_FAOSTAT.1 and do a quick check in Excel that row
# ordering in terms of Years is ok. In particular, no years missing/gap within series
# write.csv(KeyIndicators_FAOSTAT.1,file="KeyIndicators_FAOSTAT.1.csv",row.names=FALSE)

KeyIndicators_FAOSTAT.1_Growth <-transform(KeyIndicators_FAOSTAT.1, GDP_growth_USD=ave(GDP_USD, Country,
                                                                                FUN=function(x) c(NA,exp(diff(log(x)))-1)))
KeyIndicators_FAOSTAT.1_Growth.a <-transform(KeyIndicators_FAOSTAT.1_Growth, GDP_growth_USD2005=ave(GDP_USD2005, Country,
                                                                                FUN=function(x) c(NA,exp(diff(log(x)))-1)))

KeyIndicators_FAOSTAT.1_OrderedVariables <- KeyIndicators_FAOSTAT.1_Growth.a [c(
                                                                                "Country",
                                                                                "Year",
                                                                                "GDP",
                                                                                "GDP_USD",
                                                                                "GDP_USD2005",
                                                                                "GDP_2005",
                                                                                "GDP_growth_USD",
                                                                                "GDP_growth_USD2005",
                                                                                "Population_thousands",
                                                                                "GDP_percap_USD",
                                                                                "GDP_percap_USD2005",
                                                                                "GFCF",
                                                                                "GFCF_2005",
                                                                                "GFCF_USD",
                                                                                "GFCF_USD2005",
                                                                                "AFF_VA",
                                                                                "AFF_VA2005",
                                                                                "AFF_VA_USD",
                                                                                "AFF_VA_USD2005",
                                                                                "INV_Ratio_USD",
                                                                                "INV_Ratio_USD2005",
                                                                                "AFF_VA_GDPShare_USD",
                                                                                "AFF_VA_GDPShare_USD2005",
                                                                                "FAOCode",
                                                                                "region",
                                                                                "regioncode",
                                                                                "region2",
                                                                                "subregion",
                                                                                "subregioncode",
                                                                                "developing",
                                                                                "LLDC",
                                                                                "LDC",
                                                                                "LIFDC",
                                                                                "NFIDC",
                                                                                "SIDS",
                                                                                "OECD",
                                                                                "EU",
                                                                                "MAFAP",                                                                                "Def_GDP",
                                                                                "Def_GDP_USD",
                                                                                "Def_GFCF",
                                                                                "Def_GFCF_USD",
                                                                                "Def_AFF_VA",
                                                                                "Def_AFF_VA_USD",
                                                                                "EX_Rate_USD")]

View(KeyIndicators_FAOSTAT.1_OrderedVariables)

## 15. Ordering the dataset per both Country and Year

KeyIndicators_FAOSTAT_order <- with(KeyIndicators_FAOSTAT.1_OrderedVariables, order(Country, -Year))

KI <- KeyIndicators_FAOSTAT.1_OrderedVariables[KeyIndicators_FAOSTAT_order, ]

View(KI)

################################## KEY INDICATORS MAPPING ROUTINE ##################################################

## 00. Packages needed #######################################
library(rgdal)
library(rgeos)
library(maptools)
library(dplyr)
library(tidyr)
library(tmap)
library(sp)
library(plyr)
library(dplyr)
install.packages("data.table")
install.packages("ggplot2")
library(data.table)
library(gisfao)
library(sp)
install.packages("devtools")
devtools::install_github("UNFAOstatistics/gisfao")
data(fao_world)
plot(fao_world)
data(fao_world_centroids)
plot(fao_world_centroids)
data(graticule)
plot(graticule)
library(ggplot2)

## 01. Selection of variables we are interested in from the original KI (Key Indicators) file.
KIgraph <- KI[, c("Country", "Year", "GDP_percap_USD")]

## 02. Sum of GDP values (the variable could be changed) according to Country and Year.
KIgraph <- aggregate(data=KIgraph, GDP_percap_USD ~ Country + Year, FUN=sum)

## 03. Changing the name of variable "Country" into "ADM0_NAME" in order to rendere the two dataset (values and geo) mergeable.
setnames(KIgraph, "Country", "ADM0_NAME" )

## 04. Chenging the Country names in order to render them alike in both datasets.
KIgraph$ADM0_NAME <- mapvalues(KIgraph$ADM0_NAME, from=c("China, People's Republic of",
                                                         "Bolivia (Plurinational State of)",
                                                         "Democratic People's Republic of Korea",
                                                         "Republic of Moldova",
                                                         "United Kingdom",
                                                         "Venezuela (Bolivarian Republic of)",
                                                         "Iran (Islamic Republic of)",
                                                         "State of Palestine"),
                               to= c("China",
                                     "Bolivia",
                                     "Dem People's Rep of Korea",
                                     "Moldova, Republic of",
                                     "U.K. of Great Britain and Northern Ireland",
                                     "Venezuela",
                                     "Iran  (Islamic Republic of)",
                                     "West Bank"))

KIgraph$ADM0_NAME <- mapvalues(KIgraph$ADM0_NAME, from="China, Hong Kong SAR",
                               to="China")

## 05. Merging the two datasets through the common "ADM0_NAME" variable.
KIgraphMap <- merge(KIgraph, fao_world, by="ADM0_NAME", all=T)

## 06. Merging data and FAOSTATgis package.
fao_world_Ilio  <- fao_world
fao_world_Ilio <- spTransform(fao_world_Ilio, CRS("+proj=robin"))                               ## Assigning coordinates reference system (CRS)
fao_world_Ilio$id <- rownames(fao_world_Ilio@data)                                              ## Assigning row names.
map.points <- fortify(fao_world_Ilio, region="id")
fao_world_Ilio.df <- merge(map.points, fao_world_Ilio, by = "id")
KI_map <- KIgraphMap                                                                            ## Making a copy of KIgraphMap.
names(KI_map)[names(KI_map)=="Reporter_fao_world"]<- c("ADM0_NAME")                             ## turning KI_map "Reporter_fao_world"  variable into "ADM0_NAME" to render it mergeable.

KI_map_select <- subset(KI_map, select=c(ADM0_NAME, Year, GDP_percap_USD))                      ## Subseting variables we are interested in.

library(tidyr)
KI_map_select$Year <- paste0("Year_",KI_map_select$Year)
KI_map_wide <- spread(KI_map_select, "Year", "GDP_percap_USD")

fao_world_Ilio.df <- merge(fao_world_Ilio.df, KI_map_wide, by="ADM0_NAME", all.x=T)

map.df <- fao_world_Ilio.df

## 07. Preparation of the map.
p <- ggplot(data=map.df, aes(long,lat,group=group))

p <- p + geom_polygon(aes(fill = Year_2013),colour="white",size=.1)

p <- p + xlab("Longitude")                                                                               # Assigning names to x and y axes (optional)
p <- p + ylab("Latitude")

space <- function(x, ...) {                                                                              # Formatting number with '000  separator.
  format(x, ..., big.mark = ",", scientific = FALSE, trim = TRUE)
}

p <- p + scale_fill_gradient(low = "white" , high='#FF0000',                                             # Assigning map colour and interval  breaks=seq(minimum value, maximum value, breaks)
                             labels = space, limits=c(0, 100000), breaks=seq(0, 100000, 10000))

## 08. Adding a grid and refining both map colors and legenda.

library(scales)
library(grid)
                                                                                # Graticule
grat_robin <- spTransform(graticule, CRS("+proj=robin"))                                                  # Reprojecting graticule
grat_df_robin <- fortify(grat_robin)

p <- p + geom_path(data = grat_df_robin, aes(long, lat, group = group, fill = NULL),                     # Outlook of the grid.
                   linetype = "solid", color = "Dim Grey", size = .5)
p <- p + geom_polygon(data = map.df, aes(long,lat),fill=NA,colour="white",size = .7)
p <- p + geom_polygon(aes(fill = Year_2013),colour="white",size=.2)
p <- p + scale_fill_gradient(values=c("Dim Grey","#edf8fb","#b2e2e2","#66c2a4","#2ca25f","#006d2c"))     # Legenda settings.
p <- p + theme(legend.position = c(0.05,0.05),
               legend.justification=c(0,0),
               legend.key.size=unit(6,'mm'),
               legend.direction = "vertical",
               legend.background=element_rect(colour=NA, fill=alpha("white", 2/3)),
               legend.text=element_text(size=12),
               legend.title=element_text(size=12),
               title=element_text(size=16),
               panel.background = element_blank(),
               plot.background = element_blank(),
               panel.grid.minor = element_blank(),
               panel.grid.major = element_blank(),
               axis.ticks = element_blank())

p <- p + guides(fill = guide_legend(title = "GDP per capita USD - Year: 2013",
                                    title.position = "top",
                                    title.hjust=0))

p


Convert to a promise:

```js
function writeExtractedToFile (extracted) {
  return new Promise(function (resolve, reject) {
    fs.writeFile(options.extracted, extracted, function (err) {
      if (err) reject(err);
      // ...
      resolve();
    });
  });
}
```


or just an alias to s3 on route53 ?


Haha, gaining experience after levveling up


it would be great to collect also input stats, so that filtering stats could be derived. Operator is input operator if there is other upstream operator in pipeline with different plan node id. Otherwise it is input operator only if the pipeline is input too.


Your plugin needs tests. :smile:


It was not very explaining so it went away during one of subsequent review/address cycles.
The clue here is that coercion codebase does not play well for parameters with calculated types. So we do not consider it in such case.

There is finished work by @arhimondr which refactors method parameter matching and simplifies this code very much. But it is based on Decimal PR so we would like to land that in order.


Looks like I forgot to remove django-piston-oauth2 when I removed the piston-related code. It should not be used anymore, AFAIK


I think its right about the errors, here is the sample about page and another contrast checker.

I've put out a patch to AccessSniff to Error if there are errors, tested it on your site just then. It fails correctly now.

Also If you turn on verbose it'll show you all the files its testing, lots of them!


{?object} The normalised value or `null`


when could this be the case?


Can you lowercase this? :)


Cool - thank you!


I think this comment can fit easily in two lines!


@FeeJai slim has a syntax for it. Just do

```slim
 p
   strong Title:
   '
   = @article.title
```


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


That may be true, but this is something that is being ensured by the editor, not bibliotheca. So  that means that if you edit extra fro github, you could still have problems


I'm not sure. In enum Depth an underlying type is maybe short or byte!?
I think an explicit cast is safe, see here:
http://www.dailycoding.com/Posts/enum_coversion_operations_int_to_enum_enum_to_int_string_to_enum_enum_to_string.aspx


Is a double question mark even a valid statement? Maybe it's just a typo for a single question mark?


'?' without wall time seems to be confusing. Consider printing time too.


Thanks! I'll have a look soon. BTW, those are upstream Bootstrap errors. We fixed many of them some time ago, but those are trickier to fix due to `table-striped`.

I plan to add grunt-accessibility eventually in Bootstrap docs itself eventually.


I'd keep it in foreman::cli as a require, just not here.


Maybe it would be better to use here named constants instead of plain numbers?


BTW, is `Line:undefined Col:undefined` normal?


## This is a markdown file


You'll see later in the PR that I remove this entirely.


I would extract this method outside the plugin function so that it is only defined once. And, less verbose variable names are good. So, your consuming code looks like this:

```js
if (checkOverrides(options.extractors, rule.selector)) {
  extracted.push(rule);
} else {
  remaining.push(rule);
}
```

And the function:

```js
function checkOverrides (list, possibleOverrides) {
  for (var i = 0, len = list.length; i < len; i++) {
    let item = list[i];

    if (possibleOverrides.indexOf(item) > -1) {
      return true;
    }
  }
  return false;
}
```


added


30 seconds or 10 seconds is an infinite value for a computer to calculate ;) . No, this service is not syncronized to real time so that 30 seconds is a fuzzy value. Switching could vary in range of 30 seconds before or just immediate before broadcast begins. This could confuse users as they ask 'while switch immediate today and 30 secs before yesterday.'


k


ok


Anyway, I guess you are right.


Suggest to use eslint or similar so that unused variables are caught by the machine rather than a manual review. :smile:


What is  angular-todo app
On Feb 23, 2016 3:18 PM, "Muhammad Kamran Qadri" <notifications@github.com>
wrote:

> are you asking for what code is this? or what is in this page?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smkamranqadri/learn-typed-angular2/commit/3a1e4b7573fd4e48202efbc2259fea6f4d5b7636#commitcomment-16263901>
> .
>


This is similar - we could possibly use Chris's PR. But then we would have to declare each scalar in separate class (which I personally believe would be more readable).

I do not feel strongly about this patch. If Chris's work lands we can rework function definitions to use his framework.


Same here: let's define 3 constants for 3 modes and use their anywhere


Yeh comes up now and then, I think when its all one line / part of the HTML tag it has issues finding where it sits in the source file. Will make an issue around it :)


@lv-zheng Yes. Python SS is based on epoll/kqueue, an async IO model. Non-blocking IO could be easily implemented, while the dns lookup that provided by the system is blocked IO. Thus implement asyncdns is essential in pyss. I've wrote a [http proxy](https://github.com/htc550605125/http-proxy) based on epoll, in order to do dns lookup async I use signal fd, you can see [here](https://github.com/htc550605125/http-proxy/blob/master/core/net_utils.c). The framework of my http proxy is just like the golang's goroutine, in other words, golang solves this problem itself.


shouldn't operators be consistent across tasks (e.g. all have output statistics or none)?


Do you need all this?

```
node_modules
npm-debug.log
```


@maethu why did you switch all settings to "stored:true"? I think it makes sense to not include everything (at least by default). Has this anything to do with the atomic update pr?


Note that `deps` should be an array. Also this is the jspm 0.17 config. For jspm 0.16 compatibility you would want to use:

```
  "shim": {
    "dist/foundation": {
      "deps": ["jquery"]
    }
  }
```


Because of that, I think that **atheneum** and **editor** should ensure this


View.jade is missing the addition of "ctrl.formData._reportWhen" to the form declaration


@charris like this?


The new icon is a little too big and too different from the official logo.
Could you:
* use a smaller icon
* use only the mountain shapes of the logo without "c2c" (as in v5)
?
Thanks!


@oliver---- Having method in this.off makes it impossible to remove methods that are lambdas or anonymous functions.


Should I also make a task to remove the entries from /etc/pam.d/sshd ? If they remain there the user will be told twice that they don't have a job on a node..


u


thanks


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


changes.changes.changes? Wow that's deep :D
Couldn't we manage to simply retrieve a single changes array?


@flavour Please review


done


LOL - no, I thought my .gitignore would have caught that but clearly not!


Excellent. Will do.


I don't want to change it, because then I have to change the style elsewhere as well.

The code was a direct copy/paste from Mojo::Pg: https://github.com/kraih/mojo-pg/blob/master/lib/Mojo/Pg.pm#L85


Oh also, yeh if you wanna add it to the Bootstrap docs let me know if you need me to do anything to help out :+1:


Yes, will set up. :)


fixed


well, it's better to have less differences to mainstream if we can avoid, right? Sooner or later we'll have to send everything that differs, so things that will not be upstreamed and bring no benefits could be removed. Why have it?! I'll do it next then


Much improved.


I've needed a good excuse to try Babel so will probably use that.


If no `duplicateStageID` option was specified then the "Duplicate event" error should probably still occur.


It is 3 dollars for a single ride ticket. I haven't lived in NYC too long,
but it has gone up since I arrived. - Josh

On Mon, Feb 22, 2016 at 11:26 PM, jue <notifications@github.com> wrote:

> Did MTA raise the price??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JoshFutt/MetroCardTrue/commit/720992ecbe44bffa49113285ed3fb0382e67f4e4#commitcomment-16259103>
> .
>


privacy should be 'Privacy' :+1:


I haven't got the time around it yet, but my plan is to add it in the v4-dev branch and not the current master.

So many things, and so little time :/


As soon as I figure out how to write them I'll add some (honest) ;)

Been using them a little with a rule I'm contributing a little too on Stylelint so will see what I can borrow from that. I barely writing basic sane JS so to be honest I'll probably come to that last.


I love to eat good home cooked food


all drivers have bug - oscam EMM not work.. czech ICE card  0D97 and 0D96. Please send my log manufacturers
[log(1).txt](https://github.com/oe-alliance/oe-alliance-core/files/142580/log.1.txt)
thanks martin


It would probably be cleaner to just do this in-line in the payload object with a ternary operator.


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


It's just a key, so it can be named anything. Its not what ends up on the screen :smiley:


way to go guys, hadn't seen the server code in a while, good to see you're using a more efficient protocol over the wire. #win


ok


Yes, I have not dedicated some time to learn Groovy stuff. So my Groovy code is very close to Java.

> A method on_every_node that takes a block would be the ideal here.

I agree.


using System;
using System.Collections.Generic;

 class Program
{
	public static void Main()
	{
	   int[] Numbers = new int [3];

		Numbers[0] = 101;
		Numbers[1] = 102;
		Numbers[2] = 103;

	//	ParamsMethod();
		//ParamsMethod(Numbers);
	ParamsMethod(1, 2, 3, 4, 5);
	}

	public static void ParamsMethod(params int [] Numbers)
	{
	  Console.WriteLine("There are {0} elements",Numbers.Length);
	  foreach (int i in Numbers)
	  {
	     Console.WriteLine(i);
	  }
	}
}


You can always add downloaded (from somewhere else) extractors by dropping them into the directory and adding the import line to the `__init__.py` ... easy as pie.


Can you keep that test ? Since this is fairly fragile I prefer keeping a test that will tell us when jingo breaks :)


@Creeperface01 ?


Why repeat this query inside the loop?
In fact, the query is already done once outside the loop too:
ltemplate = db(ltable.area_id == area_id).select(*lfieldnames)


Yeh tell me about it lol, always things to fix


À nouveau un souci d'indentation


@borshop merge


re: 1) dd/mm/yyyy dates are ambiguous in some cases -- 3/12/2016 would be March 12th in the US and December 3rd in most other places -- and so will import inconsistentlyy, I believe depending on the users locale, which isn't good (i.e. in your case, US users or those using a en-US installs would get wrong dates). yyyy-mm-dd avoids that.

I believe you're right re: 2), but I'll let @aurimasv  comment (and fix if necessary) since I just had a quick glance.


Thanks JImmy, this is a good catch before launch!


Ici tu as mis `angular.module` alors qu'au reste des endroits tu avais mis un `\n\t` entre `angular` et `.module`


This commit is fine. This is as much testing as can be done for this particular issue.


Ici d'un coup l'indentation passe à 8 espaces ; normal ?


I know, but that is not the point..


row.wkt should be extracted from row outside the loop


I cast it just in case.


Même remarque que pour `TransHomeCtl`


EXTI registers in L4 are now split into two. EXTI1 - EXTI31 in one register and EXTI32 - EXTI39 in another. Consequently the EXTI functions (which expect a bit mask) will fail to work because they can't distinguish between EXTIs for one register or another. I replaced the functions to expect an integer between 1 and 39. This is then mapped to the first or second register. However, this has the downside that only one EXTI can be assinged at one time.


upGIF


Extracting/comparing WKTs is fairly expensive, so we should do this only when absolutely required.
If this usecase is only for imports, but the function is called in other cases too, then can do the check only for imports by checking response.s3.bulk


Is there one reason why calibrate function must be inside reference camera and not inside wizard?
If it could be inside wizard it can be automatized , head could move 1mm and take snapshot if it detects pattern and make a set automatically. User have to rotate between sets, if used on down cam.


Holy smoke! Wow, what an amazing job Laz


yea, noticed that too


Using random in the test suite is a no-go.


Having these local variables is slightly overkill, and if they must be used should only be inside the "if (name_.EndsWith("IOS", false))"


Fixed #185


Toujours les mêmes commentaires que pour les 2 précédentes PR :p


It should no longer be needed. I do not recall details now but in some flow it was possible that type signature with missing parameters were constructed.

On the other we recently added separate commit which allows users to construct. DECIMAL with 0, 1 or 2 arguments. But 0-arg and 1-arg constructors are just wrappers over 2-arg one. They just providing default values for precision and scale. 38 and 0 respectively.


Again, indent de 8 espaces ? C'est bizarre alors qu'on semble standardisés à 4 espaces ailleurs


yeah, I did have this, the problem is still that the parameters from foreman::params are not evaluated in foreman::cli


Try to run bench more times to see if it is the same each time.

On Tue, Feb 23, 2016 at 2:01 PM, Lyudmil Antonov <notifications@github.com>
wrote:

> I cast it just in case.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lantonov/Stockfish/commit/706504ffd8fcffcc262f4e833093096877d789fd#commitcomment-16267074>
> .
>

--
IQ World <http://www.ultimaiq.net/>


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!!


guteMberg? ;)


Fixed.


Same here, `angular.module` au lieu de `angular\n\t.module` fait étrange


This one can also be replaced by a map


Yes, this entire part will hopefully be removed today.


Seems odd though: fn is called 'add_area_from_template'
That process would never happen during imports would it?


Actually @ben-eb ,for someone who has never written any tests before. Do you know any good resources? E.g. Tutorials or examples to help me get started?


@oceanzus  What sort of exception handling is _fs_verify_password_ doing here?  It seems like this because we now have two different hashing systems -- because they didn't drop their user tables I'm guessing.


Perfect


You do switch to dictionary config below, do you still need that TODO ? Do we still need `REDIS_BACKEND` string ?


Pour éviter d'avoir des diff de 3 lignes là où tu ne fais qu'ajouter une ligne, il peut être utile de mettre le `;` final sur la ligne du dessous ; en plus ça montre la fermeture d'un "bloc" angular :)


it's mean todo application build with AngularJs 2.


Un niveau d'indent de trop ici


LGTM


mongoResmiQueryBuilder already constructed query, this change has no effect. An alternative can be:

`Query query = mongoResmiQueryBuilder.relationSubjectId(uri).query(resourceQueries.orElse(null)).textSearch(textSearch.orElse(null)).pagination....`


Une ligne blanche en trop. Oui, je suis lourd, je sais :p


Yes, `ghc --numeric-version | cut -d. -f1-2` is better. Writing the commit...


Bench is the same 10 times in a row. No change from master.


Just a small note: there is no end of line at the end of the file here. Otherwise, thanks for all the other fixes! :)


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


`if(textSearchQuery!=null)` {
        final TextCriteria textCriteria = TextCriteria.forDefaultLanguage().matching(textSearchQuery);
        query.addCriteria(textCriteria);
}
return this;`


This is not used anywhere ?


Deux petites questions ici :
1. Il y aurait pas un espace de trop juste entre <a> et <event-short> ?
2. <event-short></event-short> ne pourrait pas se raccourcir en <event-short /> ?


It's thread safe because key mapping doesn't changes during program run. `mapKey(key)` always returns the same value (mapped key) for the same key.


I remember to have read somewhere that enum variables have an underlying int type.


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


This line should be deleted.
We should explain in the javadoc that the type returned depends of the path parameter.


:bear:


That bear was supposed to be a beer, but I pressed enter too soon.


Here is the error output:
```
env/bin/python manage.py runserver
Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7f3be78b67b8>
Traceback (most recent call last):
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/core/management/commands/runserver.py", line 109, in inner_run
    autoreload.raise_last_exception()
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/utils/autoreload.py", line 249, in raise_last_exception
    six.reraise(*_exception)
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/utils/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/utils/autoreload.py", line 226, in wrapper
    fn(*args, **kwargs)
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/__init__.py", line 18, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django/django/apps/registry.py", line 115, in populate
    app_config.ready()
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/apps/dpnk/apps.py", line 36, in ready
    post_save_changed.connect(change_invoice_payments_status, sender=Invoice, fields=['paid_date'])
  File "/home/petr/soubory/programovani/Auto-mat/DPNK/dpnk/env/src/django-fieldsignals/fieldsignals/signals.py", line 28, in connect
    "django-fieldsignals signals must be connected after the app cache is ready. "
django.core.exceptions.AppRegistryNotReady: django-fieldsignals signals must be connected after the app cache is ready. Connect the signal in your AppConfig.ready() handler.
```

So maybe it is something wrong in my project, but the `apps.ready` is `False` in `AppConfig.ready()`.


I could swear I tried to install jsonnet a few days ago and it wasn't in nixpkgs. Did you just add it recently?


fixed


wouldn't you need an "Ms::" prefix to preferences here?


Can you use ``list-checks``?


ditto for mscoreGlobalShare


@Jameskmonger Why did you remove this?


:heart:


Hmm....


Why do we reference `django` here? Can't we just use a `foobar` module or something below `connexion`?


the first code


Bra jobbat!


This post is pretty great; https://medium.com/javascript-scene/what-every-unit-test-needs-f6cd34d9836d#.2nfq33iff

It uses tape but for new projects now I'm recommending https://github.com/sindresorhus/ava as it's much faster and easier to reason about.


`.npmignore` automatically inherits `.gitignore`, so I thought it would be better to put the line in `.gitignore` because we don't want it uploaded to the repository either. Having it in `.gitignore` prevents it from being uploaded to npm.


Yes good point, moved to a separate file in https://github.com/everypolitician/jekyll-pmo-docs/commit/24bc36c5d8f3a7fc8ecdd0a02217904ed02ff53c.


Bad news, though. It seems like this breaks squirrel and won't go out via auto-update :( I'm not sure how to fix it yet.


Comit initial


Wow cool I didn't know that, I learn something new everyday!

Thanks for dealing with these issues so quickly! :smile:


The test we employ to judge whether a site is included is simple:

 - If the example URL obviously infringes copyright, reject.
 - If https://yourextractor.com/ shows more than one obviously infringing video, reject.
 - Otherwise, use our good judgement; usually include.

We most definitely put websites into categories. It would be far more damaging - both from a moral as well as a legal standpoint - to include a website that obviously infringes copyright than not to support one in the grey area. Fortunately, with all the extractors (~800 included, ~20 rejected) I have encountered so far, there were none where I had to make a difficult decision.


> An other approach might be to try to also lex the file backwards from the end, to maybe more accurately determine the start/end of the error, and helping with recovery, but that also sounds tedious, as I guess a whole new lexer would need to be written, looking at what to expect BEFORE some construct.

Wouldn't this create another instance of the same problem? An underminated character literal is unterminated no matter which direction we are lexing. E.g. which tokens and errors would the lexer produce for the following input:

```c
// multichar character literals are analogous to lexing backwards or forwards.
int c = 'cc';
```

```c
// unterminated character literals may be missing their first or second apostrophe.
int c = c';
```

```c
// unterminated character literals may be missing their first or second apostrophe.
int c = 'c;
```

It is possible that lexing it backwards would facilitate better error messages. From a quick glance, I fail to see how the problem is made easier.

> As stated, I think approach 3 is good enough, but better with some guidance to the user.

Ok, lets go with approach 3 for now, and try to help the user by *outlining* the first error, and including any additional errors below.


path.isResource() ?


ompa hiano.


jeejee


wetwet


As I don't believe in the good of people, I would keep CONTRIBUTING.md /  SECURE_CODING.md in the root directory.


missing spaces after commas


This is __extremely__ dangerous this nukes all session values setup during bootstrap. I'm of the opinion this should NOT be done here. I've got a fix waiting in a PR which will hopefully be ready by the end of today.


ok


Ajouter Makefile


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


@schatten I think disconnecting lambdas is troublesome in JS. At least I am not aware of a way how to do it and also don't see other libraries doing this (jquery, backbone). Do you know more?

What is possible in our case however, but probably not you would be looking for:
```
emitter.on('foo', function() { ... }, this);
...
emitter.off(this);
```

Let me know if you see any solution to that problem.


Yes, that is a good idea.  Thanks.


What is catherine.dll?


Added links to lessons


Niestety nie udało mi się również zaimplementować ścieżki do pliku jako parametr. W rozwiązaniu ścieżka jest podana statycznie w kodzie programu.

Paweł Czernek


Cette méthod fait-elle vraiment un `parse`?

Votre programme ouvre un fichier XML, le parse (via `doc.Load`) puis le réaffiche en fabricant du XML à la main? Curieux, non?

Un moyen plus de simple de refaire une sérialisation correcte. Même si le passage par un `XmlDocument` est relativement inutile tant qu'aucune manipulation n'est faite.

```cs
var doc = new XmlDocument ();
doc.Load (".../foo.xml");

using (var stream = new MemoryStream ()) {
	var writer = new XmlTextWriter (stream, Encoding.UTF8) {
		Formatting = Formatting.Indented,
		Indentation = 4
	 };

	doc.WriteTo (writer);
	Console.WriteLine (Encoding.UTF8.GetString (stream.ToArray()));
}
```

De manière générale, il faut éviter de sérialiser du XML à la main. Car XML peut devenir compliqué.


Instead of simply commenting out the call to randombytes() I suggest to do something like this:
```
#ifndef DETERMINISTIC_TESTS
randombytes(sk, 32);
#endif
```
and compile with -D for those tests.


test


It can be anything. I will change it.


prefer `||` over `or`


Done (221c621e54e8a3f59c4c0b9df80a088530aa7e50).


I'm sorry -- I did not mean to check in either src/js/config.js or src/js/routes/More/SignIn.jsx.


Yeah, you can remove it... it must have been copied over by me by mistake.


Trying to implement your changes throws the following exception.
`js: uncaught JavaScript runtime exception: TypeError: Cannot call method "pop" of undefined`


Instead of having 2 different methods, I would have just an `extra` method. In order to access the primitive field, just use self[:extra]


this link does not exist anymore. Please - anyone - upload the pdf
It can be found here: https://www.rdw.nl/opendata/Paginas/handleidingen.aspx?Path=Portal/Zakelijk/Open%20data


Shouldn't this be `a < 0 && b < 0`? It differs from `sol-types.c`...


FYI: More readable again would be to use
`Byte[] b = "TEST".getBytes();`


That method is probably already there, since we are merging exercise and guide extra somewhere


something more like this?
```go
func RegisterPlugins(r cmdplugin.Registration) {
  r.Register(&acsPlugin{})
}
```


#failBuild :p


Very true, thanks!


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


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!!


Não deveria ser "pilha.h"? "Pilha.h" não existe.


Wait, that is not the same commit as we need to change mergeCoaddMeasurements.  The commit you show is of mergeCoaddDetections.  It looks like this was never corrected on the HSC side.


I want the key.


It's implemented in [PHP 7](https://wiki.php.net/rfc/isset_ternary) and also on the [HHVM](https://github.com/facebook/hhvm/commit/fcf8d4dea785cdbcba02baa7a0d94020d14a9a10). Apparently Travis uses [`HipHop VM 3.6.6 (rel)`](https://github.com/facebook/hhvm/releases/tag/HHVM-3.6.6) which shouldn't have support yet but it's working on the other builds.


Hmmy wouldn't `{}` suffice here? Why `{{}}`?

Maybe we should define ETHER_ADDR_NULL or so as `(const struct ether_addr) {}`?


Well I want the key, I didn't know I could get it by doing that.


@NehrDani @gdi2290
this does not play well with google chrome's inspector. After hot module replacement all other files from the source folders are gone. I've reverted this in my project


a config-driven plugin system would probably work OK for our use case. if plugins need configuration (e.g. DCOS_TOKEN) does that mean that we're writing versioned config types for each plugin impl (or maybe plugins can just consume a config map)?


I guess this
* https://github.com/jboss-openshift/ce-arq/blob/master/fabric8/src/main/java/org/jboss/arquillian/ce/fabric8/F8OpenShiftAdapter.java#L330
doesn't clean all it created?


Please do rename "filter.cpp" to "reduction_chain_filter.cpp" in a separate commit. It is really cumbersome to see changes to the file on github when it is also renamed.


Ditto.


А это что?


I finished the import, export and update process.


While I think about it, it would be nice if mumbledj itself would place its cache directory within `XDG_CACHE_HOME`; that way I can drop the `~mumbledj/.mumbledj/songs` symlink. See [basedir-spec](https://specifications.freedesktop.org/basedir-spec/basedir-spec-0.8.html) for more information.


I think that issue is separate from whether we need the dependencies in foreman::install.


Indeed, sorry @yorick-ne


Лажа


This implementation is incorrect on some inner layer rotation.
It might because of the preservation of some vector/array data inside the iteration.


So chrome folder is only for menda ? If fixing the name profile folder ff for all profile is possible create a pkgbuild to install chrome folder into this ... or not ? Is also possible write optional dependence for maia theme for ff or not ?


Of course it would be my change set.


Nope. It cleans all within the template. DC is in it. RC not. RC is created as a result of processing a DC.


If you get back a `true` value of `mAnswerShown` from the `savedInstanceState`, that also implies that the answer was displayed prior to rotating the device. A user wouldn't ordinarily expect a rotation to make something disappear from the display, so it would be appropriate to redisplay it here.


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!!


commit


dgfdg


Hi,
I wanted to edit these quickly on the github interface and it looks like I messed up something in the process.
Sorry about that, you can ignore this commit, I'll resubmit it properly after a pull and edition locally...
Awesome tutorial BTW !!
Cheers,
Brice


Thanks for the collaboration.


paths copy?


@bryk can we make sauce labs to run tests sequentially instead of in parallel? The problem is that all integration tests run on same cluster and if they run in parallel then there are conflicts. In order to run tests in parallel we would need every browsers to run tests on different cluster.


Copy in batch the paths


s/regisration/registration/


can or cannot reduntante.


completely reworked 01 code


mesma coisa do outro


Is the backslash at the end of the comment line necessary?


:joy:


we don't needed to add this null parameter ?


Yay. :)


You could eliminate this assignment to `mIsCheater` as well as the corresponding one in `previousQuestion` and a bunch of other code if you entirely eliminated `mIsCheated`, just using `mCheatedBank[mCurrentIndex]` everywhere instead. Essentially the `mIsCheated` variable has outlived its useful life once you've got the more general `mCheatedBank`, but instead of allowing it to die a natural death, you've kept it on life support, painstakingly keeping it up to date with the appropriate element of the array.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 115](https://teamcity.labkey.org/viewLog.html?buildId=305125&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 217 Build time: 00:13:34
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.api.data.ScopeQueryLoggingProfilerListener$TestCase: <no details avaliable>

```


using System;
using System.Collections.Generic;
using  ProjectA.TeamA;
using System.Configuration;

class Program
{
	public static void Main()
	{
	   ClassA.Print();
	   ClassA.Print();
	}
}

namespace ProjectA
{
	namespace TeamA
	{
		class ClassA
		{

     }
	}
}

namespace ProjectA
{
	namespace TeamB
	{
		class ClassA
		{

		public static void Print()
		{
		     Console.WriteLine("Team B Print Method");
		}

     }
	}
}


if **none** was detected


Sim SIm kkkk

Date: Tue, 23 Feb 2016 05:31:16 -0800
From: notifications@github.com
To: torre_hanoi_C@noreply.github.com
CC: gisellyazevedo@hotmail.com
Subject: Re: [torre_hanoi_C] First Function - validarJogada (cc8fce3)

Não deveria ser "pilha.h"? "Pilha.h" não existe.

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


@anosach-luxoft, usual meaning of Trim is to remove "white spaces" from right and left sides of the string. "Tab" is the kind of "whitespace". Actually my goal was to "show" in the header that we could trim more that just one space.


not needed too


10 s should be enought


After downgrading PHP I tested pdf with encouter/SOAP and patient file (patient photograph) it is working but if I upload patient id card (pdf document) then I see error "undefined method HTML2PDF_myPdf::setSourceFile()"

I am looking into this issue


arbitrary local error


This issue can be closed then.


local timeout


wu


`{}` would work too.

I'll add `ETHER_ADDR_NULL`.


Does not work  to find players, alliances, planets etc.
in the search after entering the nickname or the first letter,
show  a table with the name, the alliance, planet position and ranking.
However, it does not show the player list etc.

Any help?


test


Do we not want to reset all session values before running each test? Out of interest, which situations is this causing problems in?


Shreya - modified solution to solutions


Will this work with older OpenSSH and SunSSH?


@anosach-luxoft, I don't agree with you. During debug I was interested in the info which is actually fed to the `VerifyImage`. And this is debug so it shouldn't go to the production.


@alexdowad , option `-v` on `bison`, old version `racc` will create <filename>.output log file,  now `Display extra diagnostic information` break the compatibility.
The `-S` `Output summary of parser states as HTML` is amazing,
But can we still keep output parser states as TEXT on `-v` option ?

if not, some comment line about option `-v` on `lib/racc/parser.rb` file need change.


@MartinPaulEve should replace this with a genericFK, so we can link log entries to ANY model not just article...


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


Always try to avoid "this" keyword


Oups it's not the full README


test ok ;0)


An issue is here!


You should be able to clean these conditions up using lodash's _.get() function. Something like the following might work (untested):

```js
if (_.get(options, "noEmptyLinesBeforeNonFirstDecls" && _.get(firstPropData, "orderData.lineSeparatedGroup") === _.get(secondPropData, "orderData.lineSeparatedGroup")) {..}


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


Oh, this'll end up below!!


Oh, this'll too end up below!!


When the device is rotated, you only save and restore the cheated status for a single question (the one currently on display), setting the entire rest of the `mCheatedBank` back to `false`, so that the user can pretend to be knowledgable about all the other questions despite having actually peaked at their answers prior to the rotation. It would be more appropriate to save and restore the entire `mCheatedBank`.


Done! Come and see!


Crap. Sorry about that.


s/you/your/


Sure, just re-added it in 0cd7a44


try setting the environment variables inside `.travis.yml` - https://docs.travis-ci.com/user/environment-variables/#Encrypted-Variables


looks good to me.


Is this faster? Just curious


I think the `if` statement in it's current form doesn't work (I still get 308 passing and 1 failing which is this one). I added this amount to stop 150 or so breaking.

I will purge a few of these off i imagine once i understand it a bit more.

I think the `firstPropData.orderData.lineSeparatedGroup === secondPropData.orderData.lineSeparatedGroup` is the relevant bit i need to work around for this/


ok, 661f4dd on top of this change would move this to init.pp


Read button does nothing if click from a notification..

Steps to reproduce the issue:

1. Go to inbox
2. Select any message and mark as read and then notification will show
3. Click the read button and as expected it does nothing..


This is ok.


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

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


1."AddTrash" - are you serious? That is not by industry standards.
2. I believe we can avoid adding .suo files to git. Add them to git.ignore


remove **New** ?


All good I think;)


리스트 목록과 썸네일을 나타내는 부분에서 IE10에서의 오작동 부분을 수정하였다.


it `sounds` better to me **Globally registers**, disregard if it doesn't to you.


The coefficientsCache_ and recurrenceCoefficientsCache_ attributes have a meaning only in the context of orthogonal polynomials, but not for other orthogonal functions families. The polynomialsCache_ should be renamed functionsCache_ here, and you should check that only the evaluation of the polynomials stored in this cache are needed (and not more polynomial specific information).
It looks like a class hierarchy is needed here: on top, for general functions, as a child, for polynomial functions.


Interesting. That looks like a legit failure to me. We can investigate more by adding `grunt test --verbose --stack`.

In fact, another thing I think we should normalize is the `package.json`:

- Check for `"test": "grunt test --verbose --stack"`
- Check for `"files": []`
- Check that `"main"` points to the task
- Check for `"engines": { "node": ">=0.10" }`
- Normalize license, repo and author fields (author will be going to jQuery Foundation and contributors in the future so we should probably wait on that)
- Any other fields?


what's wrong with comparing to the bewit that's actually being generated?  If we're patching the time function, we should be getting deterministic bewits, right?


good catch, removed in fa9ffd9


I forgot a closing parenthesis in there. Amended.

The point of the _.get() method here is that if you have a nested property `e` whose path is `a.b.c.d.e.`, but `b`, `c`, and `d` *might* be undefined (in which case you can't just access `e` with `a.b.c.d.e` reliably), you can use the get() method to check the path in one statement and get undefined if any of the parts of the path are undefined, instead of getting a syntax error. So `_.get(a, "b.c.d.e")` is like `(a && a.b && a.b.c && a.b.c.d) ? a.b.c.d.e : undefined`

Something to experiment with.


Modifying git.ignore file is not related to adding interface entities. In future always try to separate it by commits or branches.


I can help doing that, as long as we have a final decision 'cause I reverted the jQuery foundation license header in grunt-contrib-internal; I thought it was a mistake since I didn't see any other info :/


not sure if this is fixed in a later commit or not, but if not is the plan to fix it?


can we ditch away the Main prefix in constructors, in particular since we have qualified imports?

```
P.href_ P.SpaceAll
```


@giox069 Oops, commented in the wrong window and with a typo too :/ should of course be a cast to ```INT16```, but the pull is already merged. Nice find btw.


Ah sweet, thanks. Will make the the learning a bit easier i imagine.


bool property will be always false by default. No need to initialize it by this value.


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


Generally, yes. `reduce` does `in` and `length` checks we do not need because `Object.keys` does not return array-likes with negative length or sparsed arrays.


Isn't this `cl-eval-when` redundant? I don't think code at the top level evaluates during compilation (unless there is an explicit `eval-when-compile`).


@anosach-luxoft, is it ok good for you: "Trying to resume application by hashID:"


ALWAYS write accessibility level of each member! (public, private, internal, etc.)


This breaks the switch later in the file, making the detail info has no label.


LGTM


This didn't work for shamu, no bootanimation after flashing a fresh build.


again, you forgot about accessibility levels


these 0's are offset constants and the 32 is max size of sha256 digest. Should I add comments or change them into local (global) constants?


Thanks!


Remove "get"


"Remove this order"


ALWAYS use space symbol

Desciption = task.Desciption


Supersu didn't flash on shamu after a fresh install. I had to reboot and flash supersu.


This if statement is too long - add a new line
also use empty() for checking for an empty string?


Yes.  Necessary for tcl to skip over this "exec" line once wish has been
invoked.  Otherwise it will choke.  See beamnrc_gui.tcl and
dosxyznrc_gui.tcl.  This is how these GUI's have invoked wish all along.

Blake

On Tue, Feb 23, 2016 at 8:41 AM, Frédéric Tessier <notifications@github.com>
wrote:

> Is the backslash at the end of the comment line necessary?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nrc-cnrc/EGSnrc/commit/9eaa2b241b22586829ae0443a7d3b5e5d7f32ceb#commitcomment-16267923>
> .
>


avoid using this keyword


These tests are just verifying URL well-formedness.  The tests below validate that the bewit is valid.  Just separation of concerns.  That said, with this change I can drop the time patch.


ریدی


it's fixed in a later commit


If you do things this way, the polynomial representation of the gradient is done in the canonical basis, which is not the best basis for accurate computation eg for orthogonal polynomials. It would be better to implement Clenshaw's algorithm in a specialized version of the derivative() method for the orthogonal polynomials and to compute the gradient of a polynomial by calling this method.


매년 업데이트 되야하는 유머가 아닐까요?
촤하하하


use 'var' keyword always where it's obvious


Thank you! This needed to be done.


I can understand it, but nobody other does. Just add comments.


That should not be necessary and sounds like a weird installation problem, where they forgot to do
```
ln libvulkan.so.1 libvulkan.so
```


Note that if you retrieve a `true` value from the `savedInstanceState`, that indicates that `mCheated` was `true` prior to the rotation, but you don't set that value back into `mCheated`. As a result, if the user does a second rotation, you'll lose track of the fact that they cheated. Also, it would be good to keep the display intact after rotation; if the answer was showing before, it ought to still be showing afterwards.


isCosine_ is more readable


Makes sense. I usually use Object.keys but I use forEach to iterate. I'm guessing the for loop is a little faster too. I was hoping es6 would have optimized these sort of things.


What if not? Should not happen? Should you add a check here?


This needs a changelog entry.


I like this.


3 spaces for indentation seems a bit much


Issue is resolved please try again, and make sure to always backup your wallets.


Why did you merge your own Pull Request?


That's the object you pass that's constructed like that.


You might not want to enable this on a development build, though. Preoptimizing makes it a bit harder to test patches because you can't simply exchange the jar files. You'll have to upload the odex as well. At least mmmp doesn't do that correctly.


The session value is responsible for storing amongst other things username and DiracEnv which are both NOT set to sane defaults within the config system.

Yes there is the bug that the Config should always have a sane default value but in many cases values are modified within bootstrap which ```GangaUnitTest``` is designed to rely on at the moment.

This is causing major headaches with the Dirac Parallel test (at least) and is likely a cause of other problems so I'm treating it as a bug.

Besides it's not entirely unreasonable to require a test reset the config if it chooses to modify it, in fact I think this is a very good requirement to check that things can be set and unset as expected without funny things happening.

And yes, all of this becomes mute if we're able to run all tests in a strictly isolated way unlike at the moment.


@malirod :+1:


Should we check rkt version here, if we still use deprecated `stage1Image`  field?
IMHO it'll fail, if we use rkt build below 1.0, because of lack `--stage1-path` flag (is there still need support for rkt < 1.0).


See https://github.com/Kinto/kinto.js/blob/master/src/collection.js#L671-L674


these are logged on lines 231, 245, 270 but I understand this approach is not the best.. should I change it into one log line, add comment or made the getTSResponse() method to three steps?


Hi, Thanks.  Was trying to confirm the fix, but  I can't even get to the screen to send a payment.  Instead, I get this error  message when I try to import my wallet ( http://bits.owocki.com/16032M3X0J2n/Screen%20Shot%202016-02-23%20at%207.01.07%20AM.png )  Maybe I have a pending payment that's in an invalid state or something?


Perform small fixes - why should i open this commit and look which particular fixes you've added? The name of the commit must say what you've done in it.


Слишком сложная структура с While case и if else внтури. Такой код не очень гибкий. склонный к ошибкам


I hope this code is not required. The cleanup's shouldn't depend on the journal abbreviations.


Gah, true.


AFAICT, it's not "me" but "us". Also, what I was suggesting is to find a way to prevent it from being passed that way, or possibly being prefiltered before being passed to the hook function; isn't this possible?


Ecma 262


you get from the application's context.


Нельзя кетчить NullPointer Exception


How about moving the 3 lines above after the 'if content = ignore' to save unnecessary processing?


I submitted a PR to correct this without reintroducing the Calligra import. It's up to you to decide which fix you prefer.


слыком сложный Switch case с if else внутри. Нужно упростить этот код


if i recall correctly just calling remove() didnt update the saved options, i'll have a look at this later today.


It works fine with openssh-5.3p1-111.el6.  I am not sure about SunSSH -- are you implying that RSA is not support for host authentication by SunSSH?


@TimCootes Thank you.  Sorry for missing this.


The change in mainworker.cpp breaks the Philips Hue slider.


just upgrade your programming on html5 and css


This is definitely wrong pattern. Avoid returning null pointers, throw an exception instead.


Devia sim :-)


What does the `bool` mean? I'd name the output arguments.


Better pass the repository to the class in the constructor instead of the method class. Makes dependencies more explicit: if you want to use the class you need to give me the following information: a reader and the journal abbreviations...


really like this :)


remove () - you don't need it when doing such initialization


cool, if so, then please do drop it.


This should be "RIGHT"


Maybe explain the logic of resource set? I know one can always read the spec, but it'd be cool if we had the idea here.... I got that you must have more than 3 slashes...


Yeah, figured it was the only logical solution :)


I replaced Session with an instance variable in 03856be6656cec62bf05aea318fc09c4a9115dbc


Use mock(JournalAbbreviationLoader.class) instead of the Globals object.


is this a boolean? if so, it can be shortened to if(response.passwordSuccess) no?


then*


This is from remote edition


move else up a line


I need to a background that is mactch to our business. thanks


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2076 of 2081 tests passed, 543 test warnings)

 * 547 tests had: AUD diffs.
 * 477 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.
 * 2 tests had: Table small diffs.

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


In a pretty-printing context, it is more logical to print 1/sqrt(Pi) instead of a 16-digits (or even 5-digits) number. In the comment, it is 1/sqrt(2*pi) instead of 1/(2*pi). The first value is 0.5 * SpecFunc::ISQRT2PI (maybe you should create SpecFunc::ISQRTPI) and the second value is SpecFunc::ISQRT2PISpecFunc::ISQRT2PI=0.398942... instead of 0.15915494309189535


good morning


same as above


done


here too


Wrong indenting?


the "p" had gone wandering, he returned


This presents a potential (while remote) risk that you will get a null reference.  I'd rather this either called `First` which will fail if no match is found, or that we performed a null check and throw a KOS exception (one of those "you should never see this" kinds) so that we can give the user a friendly message.
```
var event = MMengine.Events.FirstOrDefault(kspEvent => kspEvent.name == "ModeEvent");
if (event == null) {
    // throw exception
}
event.Invoke();
```


Oh, of course, since we need to perform the reassignment for the property persist to happen. Hmm.


And no PR? why?


Same note as above


@svd260 I prefer the fully spelled out version of the name. Was there a practical reason to abandon this or was this simply for aesthetics? If not practical, let's switch back to the fully spelled out version as it's unambiguous and doesn't require any special knowledge to understand the abbreviation.


move up line


Why such a lot of free space?


Not only because of time jumps but also because a TimePoint contains additional information.


Separate logical parts of code by empty lines


Try to avoid doing such changes in commit


Maybe you wanted to write 'when the LWM2M server requests...'


I like this


You should just call it `delta_monotonic`. It is unusual in Ruby to give the `get` prefix to method names.


2 --- WTF?

Why not 100500 ?


Looks fine


@simon-hofer Thanks for reminding, I'll fix it.


Well, it should't be null anyway, so the whole if statement disappears.


added app


Currently all configurations have added the ``HAVE_RX_SUPPORT`` macro, so this feature is enabled for all built drivers. I just thought maybe one day someone would like to have a way to remove this feature somehow.


> See https://github.com/Kinto/kinto.js/blob/master/src/collection.js#L671-L674

So it seems it's because the object carries a lastModified property along the list of changes. Maybe that's just a naming issue, but I'm clueless for a better name.

Well, your call, if you feel this is okay I'm not against landing this as is.


Bit of a false condition, as static abilities are removed when source is removed, but matches other options


Check the norming constants


a few general trivial comments:
- I think it would be nicer to review if indendation is 2 spaces
- else should be on the same line as the closing if bracket
- checking boolean variables can be checked in if statements more shortly

otherwise looks good!


Should this be a variable with a default value?

Then we could set production to ERROR and pre-prod to DEBUG?


I mean "LEFT"!


The twitter account is up and running. I just made a test post on it. Stephen


quite good , continue like that


i guess it's better just pass `-w` and inhibit all warning once


Hi @dma61, this was a test project and is not in production nor maintained anymore. I wouldn't use that in production if I were you ;-)


鼠标拼命划过“启动Minecraft”按钮的测试：
原：20%的CPU占用，因为原写法近乎是while(1)等待1ms。
现：2%的CPU占用，因为在等待下一毫秒的时候会sleep 1ms。


there is some redundancy with these two lines, move to a function or store globally?


Please don't do that.


Thanks for that information!


The bug with `DiracEnv` only happens because all the code in `GangaLHCb/__init__.py` is being run on *import* rather than inside the bootstrap. I think this is likewise for the `user`. This is something that we've discussed before as definitely not wanting to do wherever possible.

The reason I am resetting back to the default value is that in a test I am expecting that if a test requires a certain setup then it is passing that as `extra_opts` in its `setUp()` method so those values will be set on every applicable test function.


there is some redundancy with these two lines...perhaps it can be moved into a function?


Please no.


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


This is the comment from the remote repository


wala! wala kang inadd!


pull!


It says that above it already.


Yes, I changed it into try-catch statement and added comment.. will be shown in the next commit with all comments addresed.


ok...


is it bad practice to open this file for reading and writing and store it globally?


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: templates/getDivForSliderEffect.phtml


DEleted border off the header.


@brunogamacatao Hexadecimal values should always be in lower case. See: [https://github.com/OSBI/saiku-styleguide/tree/master/css#3.1](https://github.com/OSBI/saiku-styleguide/tree/master/css#3.1)


if (!$tokenFile) ?


good idea


Here you return a NumericalMathFunction, which is probably overkill in the context of PCE or low rank tensor expansion. There is no need for gradient or hessian, and the cost of manipulating a NumericalPoint of dimension 1 instead of a NumericalScalar is quite large. It is why the polynomials are manipulated like they are. If the burden of creating a hierarchy for evaluations from R into R is not too high it should be the correct solution.


I don't know exactly how the test suite works but the problem is reproducible with hopper.spider from repo. Be aware that this will exhaust your memory very quickly :)
I'm testing by setting ulimit -Sv 2000000 and getting a MemoryError.


Unusual perhaps, but pretty confusing considering there's also the instance variable with the same name.


Rawr Eclipse.


Instantiate the driver class here such that the returned value will be a list of objects rather than strings.


For what this `href` is supposed to be used ?


can this check for valid filesize be a function?


@brunogamacatao Look at the file *_variables.styl* [https://github.com/OSBI/meteorite-core-ui/blob/development/src/main/resources/src/styl/_variables.styl#L50](https://github.com/OSBI/meteorite-core-ui/blob/development/src/main/resources/src/styl/_variables.styl#L50)


Why is this extra step needed? is that to solve a manifest-merger error ?


magic constants


Thanks very much!


스톡기반에서 다시 튜닝해봐야겟네요 ㅎㅎ 감사드립니다 ㅁ.ㅁ


pas pire pas pire


pan toute


Bindings in the constructor.


qui qui


href is the path to the resource. In previous versions, it was written to packet payload. It is not anymore


Two alternatives:

    self.autoJoinChannels = [channel for channel in self.autoJoinChannels if channel != channel]

    self.autoJoinChannels = filter(lambda c: c != channel, self.autoJoinChannels)


@brunogamacatao Remove this token "<". at the end of </Content>


This is a minor change but very useful. I was just counter-checking the dialog and came across it


@namuzra the database creation depends on your Django version and setup. Please check the Django documentation or the [tutorial chapter 2](https://docs.djangoproject.com/en/1.9/intro/tutorial02/).


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (984 of 985 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-99.9%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-28.57%25-green.svg)


Shouldn't this be in PodToSystemd() instead of appToSystemd()? In a pod with several apps, the shutdown service should still be written only one time.


cannot parse {% it needs <% instead.


this is probably better as a `Set`


I don't understand what the value of `<optional>` is.  The [Maven documentation](http://maven.apache.org/pom.html#Dependencies) states that marking a dependency as "optional" means that the dependency is not required when a project uses ez-vcard as a dependency.  But this is not true if the project decides it wants to parse jCards using jackson-databind.


草头行= [];
草头行.push('草');
草头行.push('头');
草头行.push('行');
console.log(草头行);
JSON.stringify(草头行);


My page implemented, may be  :+1:


This assignment to `mIsCheater`, the corresponding one in `prevQuestion`, and a lot of other code could go away if you simply eliminated `mIsCheater` and used `mCheaterBank[mCurrentIndex]` everywhere. Essentially `mIsCheater` has now outlived its useful life because you've introduced the more general `mCheaterBank`, but rather than allowing it to die its natural death, you are keeping it on life support, painstakingly keeping it up to date with the relevant element of the array.


Why throw a `JsonParseException` here?  `JCardParseException` extends `IOException`, so it can be thrown from this method

`JsonParseException` means that there is something wrong with the JSON syntax.  `JCardParseException` means that there is something wrong with the jCard syntax.


I know it's not really part of the PR but, why is this synchronized? might be worth a comment


init_agent expects a fourth parameter which is a Queue object


Sorry about that. Grunt is in progress of transitioning to the jQuery foundation: https://github.com/gruntjs/grunt/issues/1404

There are some outstanding issues to address before we can make the transition at this time. But we'll get there eventually.

Thanks for the help on this!


++ but we don't want to mege this until 0.7.16 is released


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


Nit: capitals and full stops on comments would be good haha. also space after // and when the comment begins
e.g. // This


Yes, but this line is still causing problems which have not __yet__ been fixed elsewhere so at least for now I'm removing it and fixing problems elsewhere in a separate PR


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


Yay, no diffs here as expected.  OK, this branch can die now over here.  The only change needed is to merge in https://github.com/NREL/EnergyPlusDevSupport/pull/46 and the audit file diffs will ignore those found/notfound lines with paths in them.


Yes, I also need to make sure the update functionality works with the new S3 storage locations and the vault-updater.


Please move this to test/data


Adding this, makes the package dependent on `eslint-plugin-react` even though `"extends": "airbnb/base"` shouldn't need it. Travis seems to fail with `npm ERR! peerinvalid Peer eslint-config-airbnb@x.x.x wants eslint-plugin-react@^y.y.y` due to this.


This belongs to previous commit.


Quem diria. Quando eu sugeri mudar para o HAML, fui detonado. No final...


DBS is an English thing, isn't it?


It's not ideal atm (the parameters could be 2D arrays maybe?), but I could try to fix it by using some index arrays, etc.


You should not implement methods that just return. If (which it doesn't) unittest.TestCase had a setUp/tearDown that does something, you've just overridden it.

You could use super() but in this case, you should just drop these.


Bindings in the constructor please.


I don't see it being used.


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2114 of 2119 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SHD diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.
 * 2 tests had: Table small diffs.

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


Here you should add a Log2(const UnsignedInteger n) method to SpecFunc, near to the BitCount() and NextPowerOfTwo() methods, using one of the good ideas here:
http://stackoverflow.com/questions/11376288/fast-computing-of-log2-for-64-bit-integers
Except for the fun of hacking, you should care about correct rounding.


`edition1` and 2 would be nicer changed/renamed.


"int i" is scoped locally to the if statement and is basically a NOP, what is intended here?


good


Yes, you are right. Thanks.


I believe `len(cfg.ServiceAccount) == 0` is now obsolete. Let the validation do its job.


LGTM


Can do but what should I use instead?

Dne 23. 2. 2016 v 13:16 Jan Chvojka napsal(a):
>
> Avoid using System.in. This enhardens automatic testing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CESNET/TSAConnector/commit/255b9d27dc85b26df171486b7d6819c89e81a95f#commitcomment-16266178>.
>


Same remark as for the FourierSeriesFactory clas. You create quite large objects (constant gradient&hessian) and it will probably hurt performance when only the evaluation is needed.


Description is wrong. Also, remove any vm info from line 37


My pages code :+1:


Foi quando você falou de HAML que eu fiz esse PR, @dpassarini!

Mas no fim, acho que tiraram a gem e estamos usando HTML mesmo


It would be better not to include private methods during header traversal as this adds overhead, will attempt to parse functions that will never be used, etc... and these will be filtered later anyways.  It would be better to catch the private constructor and mark the class as having one