ResultPresenter does not necessarily need an explicitly defined model.. since both MainPresenter and ResultPresenter only references the single data model that holds the operands, the operators, and the operations they need to calculate..


close to beer time ;)


LGTM.


Hi,

A temporary table only exists inside the current MySQL connection, so no collision is possible between different users. What kind of issue are your experiencing with those temp tables ?


weird to me that we've to hide the FB import button when there's no exportCallback (under `this.exportCallback_ === goog.nullFunction`). intended? or should the keyring import button be also here?


This would read slightly better as `if mm.out() is not None`.


Since MV are created many batchlogs internally, which have to be played by C* to sync MV with the base table, replaybatchlog blocks untill all batchlog have been replayed.


 That's how java_test does it. The getBuildRules method describes how to `buck build`, the runTests method describes how to test (`buck test`), so the user can build without testing.
Also, AndroidInstrumentationApk doesn't compile anything by itself, it makes the apk and apk_under_test compile by depending on them.


I having issues with the Block object still, any time I reach the bottom and a new block is created and the original block variable is replaced and I'm unsure how to fix it so that I can store the information about all the other blocks. For the willFit function I was planning on checking the coordinates of all the other blocks currently on the board to see if I can move.


RIP :egg:


Ah indeed :-) I'll remove it from the other instances in that file then (these were the odd ones out).


`0xababab` isn't ASCII and so isn't valid for that key. Were you wanting it to be a binary key?


I wonder if "ignore" really needs an int or if could be part of "flag".


Typo in this commit message.


Cross-reference:
a0dac3cf2fedab24a9e5ffff4f1ca4d79813e234


It seems like the call to `.pipe($.size({title: 'vulcanize'}));` is the only difference between my code and the one from `polymer-starter-kit`. I am not really sure what difference that should make, but I'll try adding it anyway.


Because GTK requires gchar **
I didn't write this stuff


cross reference:
eb8869822c7bf691ce5c1b76aceac624ea08cbe4


should be `constants.ElementId.FB_IMPORT_DIV`, right?


Using FC branch? This added to composer there.


> Am 22.07.2015 um 17:47 schrieb ArcEye <notifications@github.com>:
>
> Because GTK requires gchar **
> I didn't write this stuff

sorry, looked up examples - saw it

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


yeah, it can stay @ 100% for 10 hours if the Phone was charged in airplane mode and it'll stay all the time in deep sleep ;-)


Gotcha. Maybe the `debug` statement should be something more like "Block until MV batchlogs are finished playing", then.


Wrong commit message; this actually fixes it for times when the pwd isn't writable.


Ah, right...my bad.


To include with #277


`unset`ing here seems like a bug since `$data` is not passed by reference. The `GetSetMethodNormalizer` will continue to work with the unmodified copy.

In the case of `GetSetMethodNormalizer` the construtor is called with the correct parameter and then a setter is also called for the same parameter, if defined.


Good catch! Want to send us a PR on this one?


No, using Zikula 1.4.0.


Safer in which sense? That replace is only used in the write routine to revert the encoding done in the previous line with `encodeURIComponent`, see #35.

I did a fast research on the RFC http://tools.ietf.org/html/rfc3986#section-2.1:

> ... For consistency, URI producers and normalizers should use uppercase hexadecimal digits for all percent-encodings.

Although lowercase and uppercase digits are equivalent per spec, is there any browser where those characters are treated differently after being encoded with `encodeURIComponent`?


AdbHelper


Trは、TableRowで、行を意味しているので、可読性は損なっていないと思います！
内容がイッパイ書いてあるとちょっと見づらいかもしれませんが、少ない内容なら充分だと思います。


The commit message is quite an understatement.


@jorge9200 this messages are public, everybody can read it... :unamused:


I didn't know you can do this. Neat.

Any reason why you're not re-using `@add_experiment` here, like you did with `add_goal` / `add_goals` ? Or alternatively, refactor `add_experiment` so it uses `add_experiments`?


I really don't follow what this loop is supposed to be doing :/ It might be more readable to create a list of `Process` objects and operate over them, rather than reasoning about indices.


I literally fail to see what was fixed.


Since we working with an external [API](https://ember-101-backend.herokuapp.com) that was written in Rails, we need to extend ember's default adapter `DS.RESTAdapter` to write our own adapter that will match our backend's payload.

This commit simply logs ember's attemps to find a model or application adapter. If none are found, then **ember data** falls back to the default adapter, `DS.RESTAdapter`.


this PR also made that fast.

On Wed, Jul 22, 2015 at 1:43 AM, Matthias Langer <notifications@github.com>
wrote:

> I have been following your changes over the past days. To be honest, the
> best thing that could happen to me right now would be you adding a
> equivalently efficient implementation for "CanTransformValues" and issue
> release 11.3 within the next couple of weeks. How far are we from 11.3?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scalanlp/breeze/commit/a81e0a055c0429d9bbaaee3ca5606673632f71c5#commitcomment-12293375>
> .
>


@caijun this test stops rendering with an error in your code for copying the common/varied chunks. Can you please find out why and fix this bug?


I am not 100% sure whether it is worth to add this commit to staging. If we follow our plan to employ remote-revoke in the future, it becomes obsolete. On the other hand, it would be quite interesting to see if there is a performance benefit to it (noux_tool_chain comes in mind). What do you think?


Perhaps default values for the new parameters would be appropriate here to maintain backwards compatibility. While I realize the action shouldn't be externally called this types of changes can often be problematic if people have tried to extend it themselves.


Ok, surprise: Google released today Chrome 44 as stable :-)


aici mi-a scapat un experiment


おお！よかったです！！！


Great addition!


Sorry, it does not have fixed the issue.

![screen shot 2015-07-22 at 17 53 10](https://cloud.githubusercontent.com/assets/13376752/8830083/99d480f0-309a-11e5-9441-ff188f089a3b.png)


I'll admit I don't understand what ```get=get_sync``` does, but it seems it's a necessary kwarg to get to_csv() to work.  Should it be made a default argument, so that it doesn't need to be specified explicitly?


Java plugins are installed as jar files - the ddl won't be on the filesystem.


better to apply `trim()` at line 392 (before the `!username` check)


this commit is problematic in multiple ways


I'm gonna have to find out whether or nor this "new2" enables regexp, which would require some more edits.


Indentation and {} needed here?


Done


This doesn't work anymore with the new Ruby plugins packaging format (starting with Kill Bill version 0.14.x).


试了一下，效果不错。

不过我发现，当客户端关闭的时候，服务器端的TCP状态仍然是EST状态。 这时候客户端的TCP链接实际已经全部关闭了

算bug吗？ 建议增加握手机制，定时收不到对端响应，关闭TCP链接。


Should this be `self.service` to match the tests below?


Yes, all these run  `holder.processDirtyFiles` but they used different conditions with few(1-3) parameters. For example, `erlangSourceRootDescriptor.isTests() && file.getName().endsWith(".erl")`  in org.intellij.erlang.jps.builder.ErlangBuilder#receiveDirtyErlangSourceModules. Maybe create new interface like that
`interface ProcessDirtyFileCondition{ public boolean value(ErlangTarget erlangTarget,
                           File file,
                           ErlangSourceRootDescriptor erlangSourceRootDescriptor);`


OK


oh, I see.. in that case you kind of have an AndroidInstrumentationApk without an APK under test, except that buck doesn't allow for that, so you just use an AndroidBinary. ok, seems reasonable


I fix it.


CodeBook.md
Ladan Rabbani

July 21, 2015

Getting and cleaning data Course project
The purpose of this project is to demonstrate your ability to collect, work with, and clean a data set. The goal is to prepare tidy data that can be used for later analysis. You will be graded by your peers on a series of yes/no questions related to the project. You will be required to submit: 1) a tidy data set as described below, 2) a link to a Github repository with your script for performing the analysis, and 3) a code book that describes the variables, the data, and any transformations or work that you performed to clean up the data called CodeBook.md. You should also include a README.md in the repo with your scripts. This repo explains how all of the scripts work and how they are connected.

One of the most exciting areas in all of data science right now is wearable computing - see for example this article . Companies like Fitbit, Nike, and Jawbone Up are racing to develop the most advanced algorithms to attract new users. The data linked to from the course website represent data collected from the accelerometers from the Samsung Galaxy S smartphone. A full description is available at the site where the data was obtained:

http://archive.ics.uci.edu/ml/datasets/Human+Activity+Recognition+Using+Smartphones

Here are the data for the project:

https://d396qusza40orc.cloudfront.net/getdata%2Fprojectfiles%2FUCI%20HAR%20Dataset.zip

### You should create one R script called run_analysis.R that does the following. ####1.Merges the training and the test sets to create one data set. ####2.Extracts only the measurements on the mean and standard deviation for each measurement. #### 3.Uses descriptive activity names to name the activities in the data set #### 4.Appropriately labels the data set with descriptive variable names. #### 5.From the data set in step 4, creates a second, independent tidy data set with the average of each variable for each activity and each subject.

Description of the Data
The features selected for this database come from the accelerometer and gyroscope 3-axial raw signals tAcc-XYZ and tGyro-XYZ. These time domain signals (prefix ‘t’ to denote time) were captured at a constant rate of 50 Hz. and the acceleration signal was then separated into body and gravity acceleration signals (tBodyAcc-XYZ and tGravityAcc-XYZ) – both using a low pass Butterworth filter.

The body linear acceleration and angular velocity were derived in time to obtain Jerk signals (tBodyAccJerk- XYZ and tBodyGyroJerk-XYZ). Also the magnitude of these three-dimensional signals were calculated using the Euclidean norm (tBodyAccMag, tGravityAccMag, tBodyAccJerkMag, tBodyGyroMag, tBodyGyroJerkMag).

A Fast Fourier Transform (FFT) was applied to some of these signals producing fBodyAcc-XYZ, fBodyAccJerk-XYZ, fBodyGyro-XYZ, fBodyAccJerkMag, fBodyGyroMag, fBodyGyroJerkMag. (Note the ‘f’ to indicate frequency domain signals).

Description of abbreviations of measurements
1. leading t or f is based on time or frequency measurements.

2. Body = related to body movement.

3. Gravity = acceleration of gravity

4. Acc = accelerometer measurement

5. Gyro = gyroscopic measurements

6. Jerk = sudden movement acceleration

7. Mag = magnitude of movement

8. mean and SD are calculated for each subject for each activity for each mean and SD measurements.

The units given are g’s for the accelerometer and rad/sec for the gyro and g/sec and rad/sec/sec for the corresponding jerks.
These signals were used to estimate variables of the feature vector for each pattern:
‘-XYZ’ is used to denote 3-axial signals in the X, Y and Z directions. They total 33 measurements including the 3 dimensions - the X,Y, and Z axes.
tBodyAcc-XYZ tGravityAcc-XYZ tBodyAccJerk-XYZ * * tBodyGyro-XYZ
tBodyGyroJerk-XYZ
tBodyAccMag
tGravityAccMag
tBodyAccJerkMag
tBodyGyroMag
tBodyGyroJerkMag
fBodyAcc-XYZ
fBodyAccJerk-XYZ
fBodyGyro-XYZ
fBodyAccMag
fBodyAccJerkMag
fBodyGyroMag
fBodyGyroJerkMag The set of variables that were estimated from thease signals are :
mean() : Mean value
std(): Standard deviation
Data Set Information
The experiments have been carried out with a group of 30 volunteers within an age bracket of 19-48 years. Each person performed six activities (WALKING, WALKING_UPSTAIRS, WALKING_DOWNSTAIRS, SITTING, STANDING, LAYING) wearing a smartphone (Samsung Galaxy S II) on the waist. Using its embedded accelerometer and gyroscope, we captured 3-axial linear acceleration and 3-axial angular velocity at a constant rate of 50Hz. The experiments have been video-recorded to label the data manually. The obtained dataset has been randomly partitioned into two sets, where 70% of the volunteers was selected for generating the training data and 30% the test data.

The sensor signals (accelerometer and gyroscope) were pre-processed by applying noise filters and then sampled in fixed-width sliding windows. From each window, a vector of features was obtained by calculating variables from the time and frequency domain.
Download the Data and unziping
filesPath <- "/Users/LadanMac/Desktop/coursera/geting and cleaning data /data/UCI HAR Dataset"
setwd(filespath)
if(!file.exists("./data")){dir.create("./data")}
fileUrl <- "https://d396qusza40orc.cloudfront.net/getdata%2Fprojectfiles%2FUCI%20HAR%20Dataset.zip"
download.file(fileUrl,destfile="./data/Dataset.zip",method="curl")
unzip(zipfile = "./data/Dataset.zip",exdir="./data")
load the packages

library(dplyr)
library(data.table)
library(tidyr)
Files in the Dataset Folder that will be used
1.subject files

   * test/subject_test.txt
   * train/subject_train.txt

2. ** activity files **

    * test/X_test.txt
    * train/X_train.txt

3. data files

   * test/Y_test.txt
   * train/Y_train.txt

4. features.txt - Names of column variables in the dataTable

5. activity_labels.txt - Links the class labels with their activity name.

Read the above files and create data tables.
dataSubjectTrain <- tbl_df(read.table(file.path(filesPath, "train", "subject_train.txt")))
dataSubjectTest <- tbl_df(read.table(file.path(filesPath, "test" , "subject_test.txt" )))

dataActivityTrain <- tbl_df(read.table(file.path(filesPath, "train", "Y_train.txt")))
dataActivityTest  <- tbl_df(read.table(file.path(filesPath, "test" , "Y_test.txt" )))

dataTrain <- tbl_df(read.table(file.path(filesPath, "train", "X_train.txt" )))
dataTest  <- tbl_df(read.table(file.path(filesPath, "test" , "X_test.txt" )))
1. Merges the training and the test sets to create one data set
#merge the training and the test sets by row for Activity and Subject
alldataSubject <- rbind(dataSubjectTrain, dataSubjectTest)

#name the variables
setnames(alldataSubject, "V1", "subject")
alldataActivity<- rbind(dataActivityTrain, dataActivityTest)
setnames(alldataActivity, "V1", "activityNumber")

#combing the Data training and Test files
dataTable <- rbind(dataTrain, dataTest)

dataFeatures <- tbl_df(read.table(file.path(filesPath, "features.txt")))
setnames(dataFeatures, names(dataFeatures),      c("featureNumber", "featureName"))
colnames(dataTable) <- dataFeatures$featureName

activityLabels<- tbl_df(read.table(file.path(filesPath, "activity_labels.txt")))
setnames(activityLabels, names(activityLabels), c("activityNumber","activityName"))

#merging columns for Subject and Activity
alldataSubjAct<- cbind(alldataSubject, alldataActivity)
dataTable <- cbind(alldataSubjAct, dataTable)
2. Extracts only the measurements on the mean and standard deviation for each measurement
dataFeaturesMeanStd <- grep("mean\\(\\)|std\\(\\)",dataFeatures$featureName,value=TRUE)

dataFeaturesMeanStd <- union(c("subject","activityNumber"), dataFeaturesMeanStd)
dataTable<- subset(dataTable,select=dataFeaturesMeanStd)
3. Uses descriptive activity names to name the activities in the data set
dataTable <- merge(activityLabels, dataTable , by="activityNumber", all.x=TRUE)
dataTable$activityName <- as.character(dataTable$activityName)

dataTable$activityName <- as.character(dataTable$activityName)
dataAggr<- aggregate(. ~ subject - activityName, data = dataTable, mean)
dataTable<- tbl_df(arrange(dataAggr,subject,activityName))
4. Appropriately labels the data set with descriptive variable names
1. leading t or f is based on time or frequency measurements.

2. Body = related to body movement.

3. Gravity = acceleration of gravity

4. Acc = accelerometer measurement

5. Gyro = gyroscopic measurements

6. Jerk = sudden movement acceleration

7. Mag = magnitude of movement

8. mean and SD are calculated for each subject for each activity for each mean and SD measurements.

The units given are g’s for the accelerometer and rad/sec for the gyro and g/sec and rad/sec/sec for the corresponding jerks.

#first looked at head of the data to see the name before the change
head(str(dataTable),2)
#descriptive variable names
names(dataTable)<-gsub("std()", "SD", names(dataTable))
names(dataTable)<-gsub("mean()", "MEAN", names(dataTable))
names(dataTable)<-gsub("^t", "time", names(dataTable))
names(dataTable)<-gsub("^f", "frequency", names(dataTable))
names(dataTable)<-gsub("Acc", "Accelerometer", names(dataTable))
names(dataTable)<-gsub("Gyro", "Gyroscope", names(dataTable))
names(dataTable)<-gsub("Mag", "Magnitude", names(dataTable))
names(dataTable)<-gsub("BodyBody", "Body", names(dataTable))
# then to check again with the new discriptive names
head(str(dataTable),6)
5. From the data set in step 4, creates a second, independent tidy data set with the average of each variable for each activity and each subject
# creates the .txt file on the direcotory
write.table(dataTable, "TidyData.txt", row.name=FALSE)


The commit comment is messed up for some reason, it was meant to be : The bank now always starts with $50k, rather than with a random amount of cash.


Fixed by Colin Ian King.


I tried just changing it over to a UINavigationController and there are other problems that happen like the hard coded status bar heights and frame that's used for the animation. Probably would need more additional work to get it going with that.


It's rather a cometic thing, so we must not add it. It's rather the answer to questions I get sometimes why we get  "overmaps" attempt messages.


Do we need this file at all? Seems all that's left is the helper method which is no longer used.


Thanks!


How does this line of code not crash? I don't see `subcomponents` defined. Should this be `self.subcomponents`?


Hrm, if that's the case then it's a definite error.  `get_sync` will use a single-threaded scheduler for the entire graph.  I'll check this out.


I am not exactly following. In the old code the callback was always established, but there were different means for calling it at the end:

* Through a synchronisation in case of JTA (which was registered in `join()`)
* Through `rollback()` (non-JTA)

The new code sets up the collector upon first usage and invokes the error handler through a TX observer (which works for local and JTA). It should all be fine unless I am missing something.


Na razie wyświetliłam wszystko w index, żeby sprawdzić, czy pójdzie :rabbit:


@SeanNguyen  bug when retrieving asset price fixed. The bug happens because id is a special keyword in bookshelf and it must be a number


Thanks, deployed !


Dňa 22. júla 2015 17:49:28 CEST používateľ Fagner Brack <notifications@github.com> napísal:
> Safer in which sense? That replace is only used in the write routine
> to revert the encoding done in the previous line with
> `encodeURIComponent`, see #35.

Ah. In that case, it's probably OK. If there indeed is some problematic browser out there, users of it will fill an issue.

> I did a fast research on the RFC
> http://tools.ietf.org/html/rfc3986#section-2.1:
>
> > ... For consistency, URI producers and normalizers should use
> uppercase hexadecimal digits for all percent-encodings.
>
> Although lowercase and uppercase digits are equivalent per spec, is
> there any browser where those characters are treated differently after
> being encoded with `encodeURIComponent`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/js-cookie/js-cookie/commit/6b66ed4e29dbfa1a223804cbdb8169e3ea636b52#commitcomment-12301270


I see - it is definitely good to remove these false-positives. So let's add it. But we somehow must keep a note to remove it once we have the remote-revoke implementation in place. Maybe by adding a note in a corresponding issue?


hi


No need, just edit it yourself.

Dňa 22. júla 2015 17:49:24 CEST používateľ Fagner Brack <notifications@github.com> napísal:
> Good catch! Want to send us a PR on this one?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/js-cookie/js-cookie/commit/a38840b7b97ff86ebb9d4edab812a336cedc6119#commitcomment-12301268


Fixed in https://github.com/ContinuumIO/dask/commit/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5


Here too I wonder if "abbrev", "verbose" and "detached" could just use a bit like "with_commit_tag_algo" and "match_as_path"?


That the helper method isn't used is itself a problem. It means we're going to end up with `99.99` instead of `99.99%`. Will think about this.


Seems like a bad rebase here as I did that change in a recent commit (renamed imageFromContainerPostReq to imagePostReq when I added support for direct image copies between servers).


哇塞 ，竟然有好多发翻译不准确的地方，还得继续学习


Specifying nrows in read_csv causes an error "read_csv() got multiple values for keyword argument 'nrows'"

```
def test_read_csv():
    with filetext(text) as fn:
        f = read_csv(fn, chunkbytes=30, nrows=3)
        assert list(f.columns) == ['name', 'amount']
        #assert f.npartitions > 1
        result = f.compute(get=dask.get).sort('name')
        assert (result.values == pd.read_csv(fn).sort('name').values).all()
```


Should `content` be `service_questions` so there's an association between the loader and this parameter?


感谢组长的认真批阅


Actually I think I may have thought of something that might work. I'll upload the file later when I wrote the code for it.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1247 of 1247 tests passed)

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


yeah, commented on this, but rebase made the comment go away. :dancers:


document lower limit (preservation), upper limit


By the way maybe this commit could be split in two. The first one could make some changes to "ref-filter.h" and maybe "ref-filter.c", and the second one could just change "builtin/branch.c".


Tagged it. Would be nice to make these real packages. Tagging with seemingly random commits just seems dirty to me.


@Gilbert09 when no enquiries, pls add class **no-enquiries**:

```<div class="layout-offset-top no-enquiries">```


Apparently I can just use my android tablet to do coding. With git support and everything. DroidEdit's one HELL of an app.


sm used for letting the caller bock until kernel memory is freed up (RCU)


Hrm, yes that would be an issue.  In the case of a user-specified `nrows` I would be tempted to punt with an informative error suggesting that the user use Pandas.  In the case where nrows is small this is ideal.  In the case where nrows is large it's actually fairly hard to do without diving through the file sequentially (potentially very slow).


Why are we returning an array now?


Here, I think that you could take the v3.0.8 files and these two lines to another commit.


Any particular reason for this removal? I didn't see an associated issue.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Done


This tag doesn't exist so bower is failing in the post-npm install step.


So this really, really looks like something that should be a bound parameter prepared statement, right? A glance at this here interpolation / literal-expansion-in-SQL-stringery should rightly make you scream.

...except `$subscription` is a table alias and not some arbitrary value. I'm pretty sure this is, if not The Right Thing, fairly safe.


I wonder if there are not already functions in ref-filter.{h,c} doing similar things, and if this function should be in ref-filter.{h,c}.


I appreciate that this is ported from the `devise_security_extension` gem, but for the benefit of developers working on this code in the future, it'd be helpful if the message for this commit explained why this code is needed and maybe where it came from originally.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1244 of 1244 tests passed)

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


enum (why?)


Thx man, my fault probably should have looked there.

> On Jul 22, 2015, at 7:09 AM, Jeremy Marc <notifications@github.com> wrote:
>
> check the jQuery cycle options:
> http://jquery.malsup.com/cycle/options.html <http://jquery.malsup.com/cycle/options.html>
> The following command strings are supported:
> 'pause' // pauses the slideshow, slideshow can be resumed with 'resume' command
> 'resume' // resumes a paused slideshow
> 'toggle' // toggles the pause/resume state of the slideshow
> 'next' // advances slideshow to next slide
> 'prev' // advances slideshow to previous slide
> 'stop' // stops the slideshow
> 'destroy' // stops the slideshow and unbinds all events
>
> For example:
> $('#slideshow').cycle('next');
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jeremymarc/jquery.cycle.transition.mask/commit/d769f690ac408c1bff6b922161d9e9b009e9cddd#commitcomment-12299039>.
>


Hey Rafael - this is probably not the way to do this. Instead of merging
from master, try:

    git checkout master
    git pull upstream master
    git push origin master  # this is just to make sure that your origin is
in sync
    git checkout my-branch
    git rebase master
    # Resolve conflicts and along the way add files where confilcts are
resolved:
    git add file.py
    git rebase --continue
    # When the rebase is done:
    git push -f origin my-branch  # Careful with that -f flag!

On Wed, Jul 22, 2015 at 9:05 AM, Rafael Neto Henriques <
notifications@github.com> wrote:

> Merge branch 'master' of git://github.com/nipy/dipy into
> DKI_PR3_kurtosis_statistics
>
> # By RafaelNH (14) and others
> # Via Eleftherios Garyfallidis (3) and others
> * 'master' of git://github.com/nipy/dipy:
>   Fixed memory leak in clusteringspeed.pyx. Added a tool to test refcounts.
>   DOC: Whoops, correcting typos in half
>   NF: Fetch and Read the MNI T1 and/or T2 template.
>   DOC: Another small change in documentation as suggsted by @arokem
>   DOC: Use the rst link syntax as suggsted by @arokem
>   BF + NF: Provide dipy version info when running dipy.get_info()
>   BF: Bug was caused by code in test. Basically when checking if
> directions are sampled by multiples of 2*pi / num, I was including the
> comparison between the first perpendicular vector with itself. In a world
> with perfect percision, this would not be a problem since np.dot(d, d)=1
> and np.arccos(1)=0. However, the percision of np.dot can result in values
> slightly larger than 1, like 1.000000000000002. In this cases, np.arccos
> will give NaN. To solve thisissue I remove the comparison between the
> selected ref perpendicular direction with itself.
>   Insure that vector v is always a float
>   BF, BW: Insure compatibility with numpy 1.6. Use dtype optionin np.array
> to insure that vector is saved as a float
>   Revert "TEST: Attempt to solve nan problem in python 2.7"
>   Revert "TEST: Attempt to solve nan problem in python 2.7"
>   TEST: Attempt to solve nan problem in python 2.7
>   TEST: Attempt to solve nan problem in python 2.7
>   DOC, TEST: Improve documentation and tests according to the suggestions
> done by @arokem. In particular, 3 random defined vectors are tested as well
> as vector [1, 0, 0] in a for loop
>   PEP8: Revising full geometry.py and test_geometry.py files
>   TESTS, BF: perpendicular_directions is tested and bugs fixed
>   NF: Implemented a function that computes n equaly spaced perpendicular
> directions given a reference vector v
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/RafaelNH/dipy/commit/1b06bfcbd5b213ea0b1ce7d85a936a232526fcf6
>


Similar to https://github.com/alphagov/signonotron2/commit/7a9e4f5e32be6c5984af7463c1d95b3383510247#commitcomment-12301722, it'd be helpful if the commit message explain the what and why a bit more.


This is not wrong, but I find it a bit ugly...


everyone can POST, PUT, DELETE when on a different domain over javascript? Very dangerous... certainly because your session is stored using a cookie.

Implication: someone could direct you to a page which does a DELETE request to the leiedal code, deleting entries. This will work as you're still logged in.

@coreation @NtynMichiel


Ok.  what's the best way to go?  Should we make the console and eventBus in the AdbHelper optional as I had it first, or just let adbhelper post the log output to them even for the tests?


I left  PRODUCTION because renaming causes changes in org.intellij.erlang.jps.rebar.RebarBuilder. I wanted to call this target "ALL" or "PRODUCTION_TEST"


Ok thanks, will do.


Patchs with your local user home directory might be suboptimal...


I should also add that this can be added to most current kernels without messing anything up.
It can be enable and disabled in the .config file by changing between CONFIG_FB_S5P_S6E8AA0_0 (the original) and CONFIG_FB_S5P_S6E8AA0_3 (the black crush fix).


Hi, thanks for the suggestion. I think one of the goals is to make the examples as simple as possible so that it only shows what we are trying to demonstrate. Doubling the size plus doing some if statements complicates it a bit more. Can you simplify what you are trying to achieve.


The method under test `getEntityPersisterClass(EntityBinding)` went away in 5.


make accessors const


It be better if the client did this, and the argument was a reflection::Object


@caijun this data viz is not rendering correctly (some geoms are not being downloaded) and now this test is correctly failing. Can you please investigate why this is happening and try to fix it?


OK


Hmm. ok.  When doing testing, it's often convenient to use a small data set (often by grabbing the first few rows from a large dataset).  This is a reason to keep a user-specified nrows around.  We use this in dev mode for pandas-related things a lot, for this reason.  For us, this approach has become standard operating procedure.  I can see that setting nrows=1000000000 could be problematic, but I'd say it's buyer beware.


Would it be better to return e.String()??


Well, ErlangTargetType is now a singleton, so you can name it INSTANCE.


document changed semantics of the create_ec portal argument (the syscall uses two subsequent selectors)


s/Open Source/Free Software/


Hi,

This code doesn't have any timestamp checks.  It is only trying to speed up the existing file copy transform.  I'll probably take a look at *avoiding-copy-if-possible* later.

The `useFsCopy` option is present as a sort of test flag, so that the approach can be tested only by explicitly enabling it.  I'm only testing on Windows at the moment, so I'd like some help on testing other platforms, and using an option seemed the safest way.  If the proposed approach turns out the quickest in all (most?) cases, then the option can be dropped.

I've used `undefined` as the encoding in an attempt to avoid character encoding issues.  I believe this is how it works - https://nodejs.org/api/fs.html#fs_fs_readfile_filename_options_callback - *If no encoding is specified, then the raw buffer is returned.*

And yes, I think that a new `copyFile()` abstraction could be the best approach so that it could be tuned independently for Node and Rhino as required.


As a user, I'm hoping to not have to make a decision up-front to use either dask or pandas based on data size.  I'd like to just use dask, and know that it'll work on a full data set, but will also work on a small test sample as well.


Done.


Added d3.min.js


POLICY -> Policy


you can say copyleft here instead of LGPL (and link to http://www.gnu.org/copyleft/copyleft.en.html)


I was looking at the race from a different perspective: I can't assert that the remove returned `true`. However, you are right, the current code could call `notifyTerminated()` multiple times. Fixing, approximately as you suggested (using a local variable for the return code of `remove()`).


Лишняя строчка, видимо.


good idea for readability to use static import for STATUS_CLOSED, would be nice to have it for all STATUS_*


Лучше убрать трубования к версиям (или выставить какой-то минимум, если есть необходимый).


Essentially it's the same as before.


unrevert


OK


OK.  In this case `nrows` could create a dask `DataFrame` with a single partition.  So you would lose any sort of out-of-core or parallel advantage in this case but would remain within the same dask API.


So reading this, I realize that I never understood what this specific constructor does - and there isn't a comment for it in the .h file... could I trouble you to add one?

In particular, I don't understand why this constructor only doesn't have the state map populated...  if your comment in the .h file were convincing enough, we could probably lose the comment in the .cpp entirely!  :-)


Use cases for this function. Motivation behind the arguments.


Absolutely. Not sure what happened there.


Fixed in `7a751ed`. Was getting ahead of myself there.


This brings in the latest dependency versions, and more importantly the latest enforcer rules.


Yep.  That sounds ok.


Only needed within core. Outside of core, OOM always results in a OOM portal traversal and is handled by core.


This purges the bogus enforcer configuration, in favor of the upstream one, which would have caught the problematic 1.0.1 release before it was deployed.


Anyways, a space is in need.


Dumb question. Why not fill all the frambuffer with your finger print instead of all of the data like it was originally being done with the fill color?


A Response object must be returned. See the base controller:

https://github.com/campusdomar/PuMuKIT2/blob/2.1.x/src/Pumukit/WebTVBundle/Controller/MultimediaObjectController.php#L39


потому что тут путь к файлу, а там к таргету.


Hi, do you can add option to disable craft guide fo players, so that only administrator will have it?


Fact. “staying” should be “saying”; “show it” should be “shown”.


нет, тут файл


Okay I see now, it's a convenience for the += operator in get_project_repos.  I don't like the idea of returning an array where it doesn't make sense, wouldn't it be better to have something like this in get_project_repos:

```python
for repo_url in proj_urls:
    # filter(None, <list>) removes all instances of None from the list
    secondary_proj_urls.extend(filter(None, get_secondary_proj_url(repo_url, workspace_dir)))
```


I am not particularly keen in having C cast (int) to then convert the result, which can also cause problems and be forgotten. I would be interesting to see the difference between 0.32 and 1 / 3 in the result. Further more, fractions are more precise than decimal if this is really an issue. But we can discuss hours about this and there is no point doing it.

What we needs to discuss is the use of the defines. They should be there for storing temporarily values that we eventually want to be able to change through a styling option. We should not use them now to change values because this will generate endless changes in Git. More precisely, since the DrawMensurCircle method is already used for drawing mensur signs it would be better not to change the result output and I would prefer to leave it as it is with a CIRCLE_RADIUS_FACTOR 1.0.

If Giuliano wants them smaller, then I see two options. We implement a styling system so this can be then specified in the MEI file. But I cannot really see this happening until we have a good discussion with the MEI community. The other option is to use a font for these symbols and to design a font with the shapes Giuliano would like to have. I feel that this will be needed at some point. Since we have SMuFL it would not be complicated.

But again, in the meantime, we can have a 1.0 factor which can be changed locally for testing but no changed value should be committed. We can later see how to have this in the Style class. How does it sound?


Эти два `places.get()` хорошо бы тоже на новые строчки, если автоформатер не против.


Лишняя пустая строчка.


Sorry to come to this late, but I think this needs `groups = {"R"}` adding to the test declaration.


lyapunov_symn is a utility function provided by Dynare that's likely to be useful in a whole number of places when solving and estimating models. If I recall correctly, I had something in my own code where I needed to solve a Lyapunov equation, so I understandably called the dynare function. Since this wasn't in likelihood initialization, I didn't ever notice the lyapunov=doubling option, and the fact that it led to a different function being called.

Indeed, even if you just compare where lyapunov_symn and disclyap_fast are called within Dynare, you'll see that many bits of other code only call the former, e.g. global identification, so I was not the only person led into the trap of calling the inefficient function.

So this issue looks like it was caused by some bad design. It does not make sense for disclyap_fast to be a separate file that must be called as an alternative to lyapunov_symn, particularly since lyapunov_symn already contains code for multiple methods. Either lyapunov_symn should be a shell function with a big "switch" that dispatches to other methods (each with their own function, not just doubling), or it should directly contain those other method's code. The current set-up is terrible for code reusability.

The issue with sparse matrices is another reusability one. The period doubling algorithm is designed for large matrices, and in most reasonable applications, large matrices are sparse. The cost of a few "full" statements is essentially zero when full matrices are passed in to start with. (Incidentally, in large models, significant performance improvements arise from replacing the matrices in dr with sparse ones, so this is perhaps something dynare ought to do anyway. My dynareOBC does this by default.)

In any case, is there really any situation where the algorithm with doubling is beaten by the one without it? The algorithm without doubling seems to be dominated in all regards in my experiments, so it ought to be the default.


Why static and out of function?

What is missing here is a const on data_bits_table.


d


We do neither. AdbHelper should use a mock Console in tests (you can use a real EventBus if you want to - just "new EventBus()"). Production code should use a real AdbHelper that's hooked up to a real Console and a real EventBus.

This is why creating AdbHelper yourself is a bad idea, because you create it the same way when you run in production and test. If you have it passed in via the ExecutionContext it's easier to provide one that uses a fake Console in tests and a real Console in production. Does this make sense?


This change mistakenly reverts https://github.com/sblessing/homebrew/commit/96b28e8f95130876fbe3fcbb09065633033bab2d. See also https://github.com/Homebrew/homebrew/pull/39209.


That'd mean I'd have to call `serverTransportListener.streamCreated()` from within the constructor. I'd rather not do that. I could move the `serverStream.setListener` into the constructor, but it seems more clear to have the client and server mirror each other.


TeamCity CommitStatusPublisher :: Github.Original [Build 11](http://192.168.2.85:8111/bs/viewLog.html?buildId=5523&buildTypeId=CommitStatusPublisher_GithubOriginal) is now running


TeamCity CommitStatusPublisher :: Github.Original [Build 11](http://192.168.2.85:8111/bs/viewLog.html?buildId=5523&buildTypeId=CommitStatusPublisher_GithubOriginal) outcome was **SUCCESS**
Summary: Success Build time: 00:00:00


> On a side note, do you know if vs2013 u5 has the same version number for the compiler?

I don't and I'm honestly too lazy to check this out since this update 5 does not seem to have much gain.


@imsimond

This makes sense to me. I'll remove the alert/tag notifications for the first iteration and move it to another branch so it's not lost, then perhaps it can be revisited. There's potential of the tags being clickable for the feed, which then get added to the users tag notification list, or a similar mechanism to make it simple for the user to add tags

For wording, what do you think of these two options:

1. Enable browser notifications
2. Enable browser notifications for key events

Number 1 keeps it short and simple, and number 2 gives a little more detail on what notifications they are.

Once we have this updated, we'll look into updating the docs for the plugin now we have a final feature set.


Move Pager_object::handle_oom to new interface

```
struct Oom_handler
{
  virtual handle_oom() = 0;
};

class Pager_object : public Oom_handler
...
```

Pass argument as reference.


I'm gonna install it in my VM anyway and check.


We need to pass 'null' here, Zephir will convert null to false to match the specified data type.


They might not do it *now*. I get your point, but better safe than sorry imho.
It's a tiny feature but it could save hundreds of numbers from being blocked in the future.
All it takes is one clever person being hired.


@teppeis https://github.com/closure-gun/closure-gun/commit/27a9b1fa534d55dfdf20fcae1c25b959022c03b1


Oh, this is because the .editorconfig file has `trim_trailing_whitespace = true` and that's why you see those three changes on those blank lines. My editor cleared out the extraneous whitespace. Should those changes stay then?


LGTM


there's no reason to protect only the includes an implementation below. Move the "#if" to the beginning and also check by LINUX, so the build doesn't fail for apm2 as it's currently failing (apm doesn't have termios.h): https://travis-ci.org/diydrones/ardupilot/jobs/72138633


Hmmm.. maybe we should protect this by a `finally` block.


변수 이름을 갑자기 바꿔버리면 어떡합니까..


Where does the function "wp_cache_confirm_delete" sourced? I don't see it in Wordpress or Supercache, and it's causing fatal errors on my sites..


Hmm ok, so thinking of what the api would look like for that,

```
android_instrumentation_apk(
  name = 'xy',
  apk = ':app',
  manifest = 'AndroidManifest.xml', # This would be optional for buck test, but not for buck build
  deps = [...],
)
```

That could work, though I don't particularly like the part about the android manifest.  Also should we still keep the name `android_instrumentation_test`, so we are consistent with other test rules that are called with `buck test`?


Just had a first quick look. Does not really work. Using static member vars means to initialize the dates only once in a Kodi process lifetime. For example, my production Kodi installation "never" shuts down, it only suspends to RAM. You need to initialize the boundary dates in SetTimer. Put them in normal member vars, then.

More to come.


I'm pretty sure that was Android Studio. I'll change it back.


OK if you say so. Otherwise I'm sure @jfoug or @frank-dittrich will make us aware soon :)


#include <iostream>

using namespace std;

	int main()
{
	cout<< "Hello world!"<<endl;

	cin.get();
	cin.get();

	return 0;

}


ok


Just my two cents, I'd have done sth like below
`cb(new Blob([this.responseText], {type: 'text/plain'}));`


OK


When I retrieve depth points from the depthDisp Map, I find that they are either very small or very large compared to the seemingly reasonable points from the undistorted map in Protonect. Here is an example dump for corresponding depth of detected right eyes from my console:
```
right eye depth: 4.59177e-41
right eye depth: 1.17549e-38
right eye depth: -1.79366e-43
right eye depth: -1.59694e+29
right eye depth: -6.23804e+26
right eye depth: -1.79366e-43
right eye depth: -4.08817e+31
right eye depth: -1.79366e-43
right eye depth: -1.59694e+29
right eye depth: 1.17549e-38
right eye depth: -1.59694e+29
right eye depth: -1.79366e-43
right eye depth: -1.59694e+29
right eye depth: -nan
right eye depth: -1.79366e-43
right eye depth: -1.79366e-43
right eye depth: 2.35056e-38
right eye depth: 4.59177e-41
right eye depth: -nan
right eye depth: -4.08817e+31
right eye depth: -1.59694e+29
right eye depth: 1.18468e-38
right eye depth: 1.17549e-38
right eye depth: -6.23804e+26
right eye depth: -1.79366e-43
right eye depth: -6.23804e+26
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: -nan
right eye depth: -nan
right eye depth: -nan
right eye depth: -nan
right eye depth: -1.79366e-43
right eye depth: 4.59177e-41
right eye depth: -1.59694e+29
right eye depth: -4.08817e+31
right eye depth: 1.17549e-38
right eye depth: -1.59694e+29
right eye depth: -6.23804e+26
right eye depth: -1.04657e+34
right eye depth: -nan
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: 4.59177e-41
right eye depth: -4.08817e+31
right eye depth: 4.59177e-41
right eye depth: -1.04657e+34
right eye depth: 1.17549e-38
right eye depth: 4.59177e-41
right eye depth: -1.79366e-43
right eye depth: 1.17549e-38
right eye depth: -nan
right eye depth: 1.17549e-38
right eye depth: 2.3507e-38
right eye depth: 2.3507e-38
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: -1.79366e-43
right eye depth: -1.79366e-43
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: -1.79366e-43
right eye depth: 4.59177e-41
right eye depth: -4.08817e+31
right eye depth: 1.17549e-38
right eye depth: 4.62751e-41
right eye depth: -1.59694e+29
right eye depth: -nan
right eye depth: -nan
right eye depth: 1.18468e-38
right eye depth: 1.17549e-38
right eye depth: 1.17549e-38
right eye depth: -1.79366e-43
right eye depth: -1.58456e+29
right eye depth: 4.59177e-41
right eye depth: -1.79366e-43
right eye depth: -nan
right eye depth: 1.17549e-38
right eye depth: 4.59177e-41
right eye depth: -1.04657e+34
right eye depth: -nan
right eye depth: 4.59177e-41
right eye depth: 4.59177e-41
right eye depth: -1.59694e+29
right eye depth: -nan
right eye depth: 1.17549e-38
right eye depth: -1.04657e+34
right eye depth: 4.59177e-41
right eye depth: 1.17549e-38
right eye depth: -1.04657e+34
right eye depth: 1.17549e-38
right eye depth: 4.59177e-41
right eye depth: -1.59694e+29
right eye depth: -nan
right eye depth: -nan
right eye depth: -nan
right eye depth: -1.59694e+29
right eye depth: 4.59177e-41
right eye depth: -nan
right eye depth: -4.05648e+31
right eye depth: -6.23805e+26
```


Who blocks when? important: not core


you could use `project_name` here too


Small typo: this should be `day_point` instead of `time_point`.


This emits `!notnull` metadata on the pointer. Are you sure it's only used when the pointer is dereferencable?


0 DyTech Informations/Migration Script.lua

Isnt used yet. When used, credits will be given :D


Also, I think it is `Project.find_by(name: project_name)`, the notation you use is deprecated afaik


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1247 of 1247 tests passed)

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


Sure.


Will do.


When backed by JTA (e.g. in the case of Ehcache), the `commit()` attempt (which is causing the error we are asserting on here) will have called roll-back already. Then trying to rollback it here another time will fail.


ACK


emoji test!!!! :+1:


remove


...sorry my built-in python compiler did not give any error :) seriously I checked the patch and it seemed ok but I cannot make sure that it works and we do not have automatic checks for this code...


const


I don't get how it is more appropriate… method name says `update_user` but all you do is verify permissions


I've readded the gaps.


Hey so I'm a developer at WPILib working on the FRCSim project. I was looking for a linux driverstation that implements the NI Communication protoco,l and this was the first thing I found. Is this in a working state? It seems you haven't been updating it much lately.

If it is, there's a chance we will encourage other teams to use it for simulation.


I implemented @elliottt 's fix here: https://github.com/GaloisInc/cryptol/pull/262.


why is this needed?


well, you need to build it before you can run it, so I don't think the manifest is optional (right now it's mandatory anyway).
I don't feel particularly strongly about the name. We could add an alias so you could use either name.


document assertion


indentation


Good catch! I lazily removed it as I got compiler warnings about implicit move and copy constructors.

Fixed in e2ef733


besides the wrong style for switches... why are you bothering with AK8963_14BIT_ADC?  we don't use it and as far as I can see there's no intention to use it.


Forgot to remove. Disregard.


check pds, not src/dst


Вроде сейчас Boost исправлен. Посмотрите. У меня осталась проблема, что libxml2 (только на windows!) не находит iconv.h


s/knowning/knowing/


enum


Ding this extra line.


대댓글 버튼이 사라졌네요...


wssww


That's a good point.


great


OK, it seems to be different after all

```
Microsoft (R) C/C++ Optimizing Compiler Version 18.00.40629 for x86
```


+1

Josh Holmes
Director of Architecture
+1.424.242.8816 | josh.holmes@microsoft.com | blog http://www.joshholmes.com

-----Original Message-----
From: Felix Rieseberg [mailto:notifications@github.com]
Sent: July 22, 2015 9:30 AM
To: felixrieseberg/case-studies <case-studies@noreply.github.com>
Cc: Josh Holmes (DX) <Josh.Holmes@microsoft.com>
Subject: [case-studies] Fix all case studies (2cd6ffb)

Fix all case studies

For the first time, we finally have a non-broken build.

@SaraSp @joshholmes: I’d recommend that going forward, we only accept case studies that do not break the build and pass html-proofer.
Html-proofer checks if all images and links are reachable.

---
Reply to this email directly or view it on GitHub:
https://github.com/felixrieseberg/case-studies/commit/2cd6ffbc20c52951f5363606a6331a111e12ecd4


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (729 of 729 tests passed)

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


os


@matthewelse @0xc0170

This is a simpler pgen with some things torn out. Supports single project.


ugh, thanks!


uff, this was more than 3 years ago, for some reason we chose to ignore messages from the lowest priority programatically?


I suggest using linux-g++* {} scope, because I use OS X which is also unix, but not Linux


FIxed.


:+1:  :+1:


I was worried that the definition might change though (and we'd somehow miss that when updating), and then the below wouldn't apply. This way we'd get an assert.


Just ran into this.  Any idea if it's a bug in Android?


Yeah seems so. Could we make this configurable perhaps? I'd like to have some debug messages in scripts.


what's that for?


Спасибо! :+1:


Для джарников стоит сделать отдельную папку `thirdparty`.


I suspect this change on line 1 was unintended. `PostsController` is no longer inheriting from `ApplicationController`.


Not a bug in Android, the option that launches the OverlayConfigActivity is defined in `preferences.xml`, where it must specify the correct android package name in order to be permitted to launch the activity.

For this application, the debug and release builds have slightly different package names (see the two different `strings.xml` contents), allowing side-by-side installation. This is not done by default in Android applications, so unless you made it happen yourself, this is not a problem you face.


Lately I learned a better way
`validates :title, :campaign, :user, presence: true`


Hrm I have a feeling we are talking about two different things.  All this is not needed for tests in the `test/` directory.  It is so when `buck test` is used in production, in a project that has an `android_instrumentation_test`, there is no unnecessary output.

Below are two examples, the first with the real console and buckeventbus, and the second with them absent.  Note that in the first example the superconsole also gets stuck at `install apk`.

```
Not using buckd because watchman isn't installed.
[-] PROCESSING BUCK FILES...FINISHED 0.1s
[-] BUILDING...FINISHED 0.2s (1/21 JOBS, 0 UPDATED)
[+] TESTING...4.7s
 |=> //apps/myapp_test:test...  4.2s (running install apk[4.2s])
[+] INSTALLING...4.2s
Log:
No Android platform target specified. Using default: Google Inc.:Google APIs:21
Installing apk on emulator-5554 (4.0_without_google_api).
Successfully ran install apk on 1 device(s)
Successfully ran install exopackage on 1 device(s)
RESULTS FOR ALL TESTS
PASS    <100ms  1 Passed   0 Skipped   0 Failed   com.example.tests (emulator-5554)
TESTS PASSED
```

```
[-] PROCESSING BUCK FILES...FINISHED 0.1s
[-] BUILDING...FINISHED 0.8s (21/21 JOBS, 21 UPDATED, 0.0% CACHE HITS)
[-] TESTING...FINISHED 7.3s (1 PASS/0 FAIL)
[-] INSTALLING...FINISHED 3.0s
Log:
No Android platform target specified. Using default: Google Inc.:Google APIs:21
built APK for //apps/myapp:app-exo at buck-out/gen/apps/myapp/app-exo.apk
built APK for //apps/myapp_test:app_test at buck-out/gen/apps/myapp_test/app_test.apk
RESULTS FOR ALL TESTS
PASS    <100ms  1 Passed   0 Skipped   0 Failed   com.example.tests (emulator-5554)
TESTS PASSED
```


Looking at this code, compareAndSet followed by getAndIncrement seems like a recipe for disaster, lets relook at the whole multithreaded round robin strategy and see if we can come up with a cleaner overall solution (since we are cleaning some of it up)


Hi, this is not related to this commit.
Please use the mailing list for questions like this, or if the url is outdated, open an issue...


@adaikalaraj This method is deprecated. Need to find a different solution. What does this code do?


This change actually causes a segfault for me when using editline:

```
$ gdb ./step1_read_print
GNU gdb (Ubuntu/Linaro 7.4-2012.04-0ubuntu2) 7.4-2012.04
Copyright (C) 2012 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
For bug reporting instructions, please see:
<http://bugs.launchpad.net/gdb-linaro/>...
Reading symbols from /home/joelm/personal/programming/mal-kariya-mitsuru/c/step1_read_print...done.
(gdb) run
Starting program: /home/joelm/personal/programming/mal-kariya-mitsuru/c/step1_read_print
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
user> nil

Program received signal SIGSEGV, Segmentation fault.
0x000000000040490a in append_to_history () at readline.c:48
48	        fprintf(fp, "%s\n", he->line);
(gdb) bt
#0  0x000000000040490a in append_to_history () at readline.c:48
#1  0x000000000040498a in _readline (prompt=0x7fffffffdd10 "user> ") at readline.c:66
#2  0x0000000000401ab1 in READ (prompt=0x7fffffffdd10 "user> ", str=0x0) at step1_read_print.c:17
#3  0x0000000000401bc6 in RE (env=0x0, prompt=0x7fffffffdd10 "user> ", str=0x0)
    at step1_read_print.c:50
#4  0x0000000000401c62 in main () at step1_read_print.c:70
(gdb)
```

Without the change it works fine. gnu readline build works fine either way. Perhaps a different version/behavior of editline? Mine is libedit2-2.11-20080614-3ubuntu2

Let me know if you are able to address it soon. Otherwise I'll revert the change.


From the doc:
> Note: this method is only intended for debugging and presenting task management user interfaces. This should never be used for core logic in an application, such as deciding between different behaviors based on the information found here. Such uses are not supported, and will likely break in the future. For example, if multiple applications can be actively running at the same time, assumptions made about the meaning of the data here for purposes of control flow will be incorrect.


Interesting.  I assumed it was a bug because I thought you were likely changing `applicationId` but not `packageName` between debug and release.

Looking at [`AndroidManifest.xml`](https://github.com/sigmabeta/dolphin/blob/master/Source/Android/app/src/main/AndroidManifest.xml#L2), the `packageName` is set here, but it's the `applicationId` which is changing in [`build.gradle`](https://github.com/sigmabeta/dolphin/blob/master/Source/Android/app/build.gradle#L48).

I still think it may be a bug in Android, as `targetPackage` seems to be referring to the `applicationId` rather than `package` as might be expected.  I'm quite new to the topic though - just read https://sites.google.com/a/android.com/tools/tech-docs/new-build-system/applicationid-vs-packagename recently.


The likely cause was that TastyPie returned a time that matched the EST timezone, which JavaScript incorrectly assumed to be a GMT time, and thus rolled back an additional 4 more hours.

Solution: http://stackoverflow.com/questions/15517024/how-to-assume-local-time-zone-when-parsing-iso-8601-date-string


Port should be a variable, it is set in multiple places:
https://github.com/redhat-openstack/delorean-instance/blob/master/delorean-user-data.txt#L59
https://github.com/redhat-openstack/delorean-instance/blob/master/delorean-user-data.txt#L223
etc.

Some environments have issues connecting to ports < 1024 (openshift gear for prod-rdojenkins.rhcloud.com) so I wanted to change it to e.g. 3300

Also ssh port change is currently commented out https://github.com/redhat-openstack/delorean-instance/blob/master/delorean-user-data.txt#L283-L285
@javierpena was there an issue and those ^ lines didn't work?


Hello World.md

## This is a Markdown file


Nice!


return here?


Ah thanks, much cleaner (as well as the use of reject). Thanks for the link too!


cli-bug-fix (nealkruis) - x86_64-MacOS-10.9-clang: OK (1748 of 1755 tests passed)

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


continue please


what about the case when the Apk is instrumenting itself?  Should the user just use the same AndroidManifest as for the `android_binary` then?


Hi @arokem!
Yup, my bad! I was following this procedure however I did a typos in git checkout master and I only noticed at the end of the rebase! I already rerun the rebase! I think now everything is fine.


This isn't really an appropriate forum for discussing this, but just to be clear: line 2 of `AndroidManifest.xml` defines *a* package, but this definition is only used to shorten references to classes within the `AndroidManifest.xml` file.

For example, line 42 contains `android:name=".activities.AddDirectoryActivity"`, instead of the fully-qualified name `org.dolphinemu.dolphinemu.activities.OverlayConfigActivity`. Removing line 2 would make it so you'd have to use the fully-qualified name at line 42, but would otherwise have no impact.

The package name that matters at runtime is defined in `build.gradle`.


Конвертировать `order` в `places` придется так или иначе. Либо при чтении из файла джейсона, либо при записи в него. Мне кажется, чем делать это при записи (в текущем варианте это пока не делается, но придется) и хранить в джейсоне два массива, лучше хранить в джейсона только `order` и делать конвертацию при парсинге.

Т.е. перед записью в `printStream` будет что-то такое:
```java
Map<Integer, Integer> places = getPlaces(JSONArray order);
```
(ну или `String` вместо `Integer`)


Is it a fix bug to the issue #257?


Thanks, and sorry for the noise in your repo.


Still new to C# Async. Fixed


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


he he


Hi @robbell

Good work on the script for BackConnection host names - but doesn't this also require to disable strict name checking (DisableStrictNameChecking). [Please see the description for the first step here](https://support.microsoft.com/en-us/kb/896861)

It would be nice to also include DisableStrictNameChecking in this script. Maybe by an optional parameter?

Cheers.
![capture](https://cloud.githubusercontent.com/assets/2877439/8831599/adea90f8-30a3-11e5-86ea-c55b0424572b.PNG)


please include a related_name field here.


This needs to require CPAN::Meta too


No worries at all - I just don't believe I'm enough of an expert to be of much assistance to any followup questions.


Main function can be void?


В принципе, хранить строки и с маленькой буквы, а перед записью в модель делать
```java
object.get("computeRight").substring(0, 1).toUpperCase() + object.get("computeRight").substring(1);
```
Но выглядит так себе.


Why null=True? My understanding is that 'blank' allows a field to be optional, null simply allows a field to be of type null. Why do we want age and date of birth to be nullable? (or even optional)


Please include the migrations file for any model changes. (generation from makemigrations)


Merge failed in Gerrit, would like to abandon this as the code is already in the other subsequent commits that have already merged.


You might want to pull from upstream again. It's been moving ahead in the
meanwhile :-)

On Wed, Jul 22, 2015 at 9:57 AM, Rafael Neto Henriques <
notifications@github.com> wrote:

> Hi @arokem <https://github.com/arokem>!
> Yup, my bad! I was following this procedure however I did a typos in git
> checkout master and I only noticed at the end of the rebase! I already
> rerun the rebase! I think now everything is fine.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RafaelNH/dipy/commit/1b06bfcbd5b213ea0b1ce7d85a936a232526fcf6#commitcomment-12302740>
> .
>


Nie wiem dokładnie, jak to jest, bo póki co nam i tak zaokrągla niestety :( Wydaje mi się, że w samym kodzie byłoby tak poprawniej.


Changes look good! We have some different stylistic habits, e.g. `kwarg = ...` vs `kwarg=...` -- to make things simple I vote that where we differ, we go with django's style. In this case, it is the `kwarg=...` style. What do you think?


I solved this another way but don't remember how 😊 maybe look through pages module FC branch?


hello from github


There are a number of issues with this, I think all of them would be solved by actually loading CPAN::Meta when testing for its presence.


If set, the filter string is "XYZ):".
The threadtime logcat format only has "XYZ", so it would not match.

Not entirely moot; I am seeing past instances of my process in the logs; the purpose of this filter would be for me to limit the log to only the current process.


Не находит в стандартных библиотеках или где? В сандартных под Windows его нет, надо ставить отдельно.


`transistion`?


Good call making it `"""` -- I've been super inconsistent about that already. One other stylistic bit is whether we should have a space after the quotes. I'm ambivalent.


Sure, I presume the problem was that we couldn't filter by package name in the logback file? try to comment it out and see what happens...
Just let me know  and I'll try to come with a friendlier solution


I don't even know...


Does a relationship make sense with a `NULL` start time? I suppose if we don't know when it started.... Would that mean that a relationship without a `start` or `end` is currently active?


Ok, that makes sense. I don't see anything wrong with the second output, although I haven't applied the patch, so maybe it looks weird when it's running. I don't think we should be sending events from AdbHelper to /dev/null because that makes it impossible to figure out why your tests aren't running (AdbHelper uses the console to signal errors).
If there are specific messages you'd like to inhibit (like the "successfully ran... message") maybe add a verbosity setting to AdbHelper?


I will always hate these method names :) Completely OT, but there is somewhere in the code where we call one manually which seems wrong. We should fix it...


@imsimond In response to notifications:

Mobile notifications are very limited. On iOS, no browsers look to support notifications. On Android, Chrome and Firefox state they do with a HTML5 browser test (https://html5test.com/).

We check if the notification API is supported in the browser using the method from MDN, so any browser that doesn't support them, the checkbox will not show.

It appears Chromes implementation on Android has doesn't work fully with just the Notification API, despite it passing feature checks, and Firefox's implementation auto closes after a number of seconds (https://bugzilla.mozilla.org/show_bug.cgi?id=875114). Unfortunately these issues are down to the vendor, so a solution could be to disable notifications on mobile.

The MDN has a great table at the bottom to show which browsers (desktop and mobile) support notifications — https://developer.mozilla.org/en-US/docs/Web/API/notification.


new


Should we check for intersections elsewhere, e.g. the `validate_time` method?


Yep. Ran into issues streaming out of normalize and into another instance of unique-kmers, though.


should rethink this logic so wmware tools wait does not into negative.


Atualização README.md e remoção do dd()


love this


I guess programming after midnight isn't very productive :)


Fixes https://github.com/Anaconda-Server/anaconda-server/issues/385


Can you open a bug to track that?


Maybe we could warn about how to modify this for embedding people who probably will? :///


@mpc-hc/developers I have this commit ready for next push. It resolves few more issues.


![Blub](http://i.imgur.com/Acd4qfM.jpg) - Ingameresult
https://gist.github.com/Rushor/e6069c800b5b070b965c - See you on 6.x :)


OK.  This is off-topic and weird.  ```.strip()``` is not doing what you expect whenever you use it on an intented, triple-quoted string:

See here:
https://github.com/ContinuumIO/dask/blob/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5/dask/dataframe/tests/test_io.py#L269-L281

I'm certain you're expecting text2 to be
```
name,amount
1,400
2,-500
Frank,600
```
at this point.

It's actually still
```
name,amount
    1,400
    2,-500
    Frank,600
```

This appears to be the case throughout your test cases if the string is defined within a function, and thus indented.  It's a little surprising that .strip() isn't chopping the leading spaces.

I realized this by opening various temp output csvs prior to their being deleted as part of the test cases.

There's probably a more elegant way to do this, but I guess you could replace your instances of ```.strip()``` with ```.replace('    ','')```

I think this might be important because auto-detection of dype ***might*** be fouled up if the data being received looks like ```<space><space><space><space>1``` instead of ```1```. I'm not sure.


@ilanschnell please submit contributions via pull requests.


This seems wrong to me. i.e. I would think we'd shift the lock off if its hidden and let these constraints on the urlbar never change (ideally we'd animate?). Maybe I'm crazy. Probably better to avoid churn anyway and stick with what we had.


One could make this return the components of the tensor in res, so it does not need to be called for each component indiviually?


See https://github.com/ContinuumIO/dask/pull/465


Thankyou based maxpowa


https://gist.github.com/Rushor/e6069c800b5b070b965c - See you on 6.x :)
![Bild](http://i.imgur.com/slc4oOg.jpg) - Ingameresult


Similarly, I would return a vector in `res` here.


Good practice (of putting comments to signify the end of larger sections. :thumbsup:


We only need `libcrypto`, not the full `openssl`, no?


hh


Being that I have no idea what this was supposedly doing, I cannot really comment on this.


Ah, indeed.  This isn't removing space within the textblock

On Wed, Jul 22, 2015 at 10:13 AM, Michael Rea <notifications@github.com>
wrote:

> OK. This is off-topic and weird. .strip() is not doing what you expect
> whenever you use it on an intented, triple-quoted string:
>
> See here:
>
> https://github.com/ContinuumIO/dask/blob/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5/dask/dataframe/tests/test_io.py#L269-L281
>
> I'm certain you're expecting text2 to be
>
> name,amount
> 1,400
> 2,-500
> Frank,600
>
> at this point.
>
> It's actually still
>
> name,amount
>     1,400
>     2,-500
>     Frank,600
>
> This appears to be the case throughout your test cases if the string is
> defined within a function, and thus indented. It's a little surprising that
> .strip() isn't chopping the leading spaces.
>
> I realized this by opening various temp output csvs prior to their being
> deleted as part of the test cases.
>
> There's probably a more elegant way to do this, but I guess you could
> replace your instances of .strip() with .replace(' ','')
>
> I think this might be important because auto-detection of dype *might* be
> fouled up if the data being received looks like
> <space><space><space><space>1 instead of 1. I'm not sure.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ContinuumIO/dask/commit/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5#commitcomment-12303042>
> .
>


Work work zug zug


I used to.be.at.least,
Would it be better to say within(numericDelay-5, numericDelay+5)


@crisla and @Falter89
I have refactored the code for computing the distribution of firm size based on @crisla previous work.  For reference I was able to reduce it to 3 lines of code...
```python
sorted_data = viz.solution.sort('theta', ascending=True, inplace=False)
theta_pdf = workers.evaluate_pdf(sorted_data.index.values) / sorted_data.theta
theta_cdf = theta_pdf.cumsum() / theta_pdf.sum()
```
...see the notebook for details.  I will incorporate this functionality into the `Vizualizer` class in the July `pyAM` release.  I will also extend to compute the distributions of wages, profits, output (per worker). Anything else?


I'm so sorry for breaking that on accident. Thank you for fixing it!


https://gist.github.com/Rushor/e6069c800b5b070b965c - See you on 6.x :)
![Bild](http://i.imgur.com/6R5CN1o.jpg) - Ingameresult


See https://github.com/ContinuumIO/dask/pull/466/files

On Wed, Jul 22, 2015 at 10:15 AM, Matthew Rocklin <mrocklin@continuum.io>
wrote:

> Ah, indeed.  This isn't removing space within the textblock
>
> On Wed, Jul 22, 2015 at 10:13 AM, Michael Rea <notifications@github.com>
> wrote:
>
>> OK. This is off-topic and weird. .strip() is not doing what you expect
>> whenever you use it on an intented, triple-quoted string:
>>
>> See here:
>>
>> https://github.com/ContinuumIO/dask/blob/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5/dask/dataframe/tests/test_io.py#L269-L281
>>
>> I'm certain you're expecting text2 to be
>>
>> name,amount
>> 1,400
>> 2,-500
>> Frank,600
>>
>> at this point.
>>
>> It's actually still
>>
>> name,amount
>>     1,400
>>     2,-500
>>     Frank,600
>>
>> This appears to be the case throughout your test cases if the string is
>> defined within a function, and thus indented. It's a little surprising that
>> .strip() isn't chopping the leading spaces.
>>
>> I realized this by opening various temp output csvs prior to their being
>> deleted as part of the test cases.
>>
>> There's probably a more elegant way to do this, but I guess you could
>> replace your instances of .strip() with .replace(' ','')
>>
>> I think this might be important because auto-detection of dype *might*
>> be fouled up if the data being received looks like
>> <space><space><space><space>1 instead of 1. I'm not sure.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ContinuumIO/dask/commit/1d338ffe644b01f1d2b79a9fa291c1bf4cb572f5#commitcomment-12303042>
>> .
>>
>
>


bc break


removeevent


Before:
![before](https://cloud.githubusercontent.com/assets/2971112/8832009/34af3728-30e1-11e5-87bf-1b13e952b0b4.png)


Nice

Le mer. 22 juil. 2015 17:40, Navid <notifications@github.com> a écrit :

> i will rename the tables
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RomainGoussault/Deepov/commit/c8926f2b211460098fdbf930fa1c1ceb28604204#commitcomment-12301067>
> .
>


Hi,
Why don't you linearize the model by changing the reference frame ?
In robot's reference frame it's model is linear.


For that we would need to ask person who added this thing. Basically when fullscreen D3D device is created FocusWindow need to be provided. Focus window is the one which will be on top of everything obviously. And that's handled internally. I have no idea why we were creating dummy window for that. Seems like terrible hack for something. But can't really tell. I can tell that it works fine when focus window is set for our fullscreen window.


After:
![after](https://cloud.githubusercontent.com/assets/2971112/8832025/4ab864a4-30e1-11e5-8950-db8b98ff9597.png)


In Chrome Inspector, select a modern phone and view the website. It still comes up full width. With modern phones, you typically have a pixel density > 1. While that might mean 640px wide is actually 1280px wide, that doesn't mean it'll be convenient to the viewer. They'll have to strain their eyes to see everything on the smaller screen.
Try expanding the max-width attribute to 640px. That will utilize the greater pixel density displays on modern phones and make viewing the site on those phone a more pleasant experience.


in component


Thanks! But did you notice that new builds of ST3 have swiched to a [new default Javascript syntax](https://github.com/sublimehq/Packages/blob/82b58838389969208d7391fe3ede52e6a78fb851/JavaScript/JavaScript.sublime-syntax#L3) based on the JavaScript Next syntax? They also seem to have moved the default language packages to [sublimehq/Packages](https://github.com/sublimehq/Packages/). Do you still see a need for this change?


Ilan, why would `platform` and `arch`  not exist in a conda package? Shouldn't these default to:

```py
import patform
index['arch'] = patform.machine()
index['arch'] = patform.system()
```


@Vezzra, suffix `InSystem` does not tell me that this field is special. Field in system, system in universe, so field in universe and field created with coordinates in universe too. It is confusing.  If you add you explanation about difference to docstrings it would be good.  But now all documentation about field classes (mobile and static) is in function name.  I want to see information about field classes(mobile or stationary) in function names.

> but also inserted into the system, bound to it,

Yor previous explanation is good. But this means nothing to me. I think It will confuse other.


I commented it out and it seems does what I want. Now I'll see what happens if I run some other scripts, that I don't get too many messages.

But why would we want to filter messages and why would we do it in logback? I imagine we would just have some global config setting in either the framework or cli/webui that sets the level of job messages.


ah sorry for that @aheckmann :( my fault


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1247 of 1247 tests passed)

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


VERY good job!!! it's awesome!:)


Krasava


EnergyPl.us (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


Yes. Crap. Good catch.


Also, will this really fix the issue?  The `KeyError` in the old code would not be coming from the  `subdir` lookup. because it uses `index.get('subdir')`


bc break


Woops good catch


I think this `if` is superflous


throw a LogicException ?


I prefer hi!


to drop


What does this mean though, to the democracy if the mesh network?


+        $this->objectManager->persist($version);
+        $this->objectManager->flush($version);

use a version saver


Not sure if this is OK for use in the wild but I found that this helped with chrome firing transitionend events more consistently. This bug for chromium helped a lot for reference of the problem itself: https://code.google.com/p/chromium/issues/detail?id=388082 (particularly comment #9 by dstockwell@chromium.org)


inject the entity class


What do you need these blanks for?


I'm running this for 1st time!


saver


El fork hace una copia exacta en crudo, permite a los desarrolladores contribuir a un proyecto de forma segura.
La rama es para añadir nueva funcionalidad al proyecto creo.


This should not be removed


added the first construction of the db


pim_versioning.object_manager.versionto be injected in the saver


Kill the printf spam (10000+ syslog entries in 10 min).


These imports are not valid json. Different imports need to be in separate objects.


missing specs + comments to fix


The `KeyError` in the old code came from the `arch_map` lookup.  The `arch_map` lookup should only be used in case the `subdir` key does not exist, which is the case for older conda packages.


Thanks Rob


Yes, the commit description only mentions only about the change in chefspec execution result output messages. It's not mentioning about provider or anything about the change in how this cookbook/provider works. The only reason I created the PR was to clean jenkins output while executing chefspec in it.


I think ultimately we'd want to follow pep, which django is occasionally a little lax on (ie docstrings in the tutorials), but no strong opinion either way. I'd agree with ``kwarg=...`` style. Shown in PEP 0008: https://www.python.org/dev/peps/pep-0008/#other-recommendations


:heart_eyes:  :+1:  Going after :8ball:


At least this makes sense to me. Maybe @alexmarsev will know better.


@bytekast this is inline with what I was thinking


The key should argument should be a string like "token" or something. Plus, you're saving the local userModel not the response from the server.
Check the documentation, this wont work.


This is a compile-time pessimization.


I like spaces after quotes for the marginally increased readability.


@lundman: No bird crashes, but sample syslog errors:

```
7/22/15 6:28:35.109 PM bird[975]: can't get permanent storage of path:'/Users/xxx/Library/Mobile Documents/com~apple~TextInput/Dictionaries/mobile~E5009972-3420-50F5-A5AD-2B5DF0DAFE27/UserDictionary/SAlQVUhF7208e6_gvZx_zdKx1U1AzKGem3HO2pLKjgY=/270028BC-3A96-4A06-B20D-DF5BCE875548.1.cdt' parentFileID:556698 fileID:557568 size:735 mtime:1407370390 mode:-rw-r--r-- documentID:2615243109 generationID:1 fd:20 refs:1: Error Domain=GSLibraryErrorDomain Code=2 "The operation couldn’t be completed. (GSLibraryErrorDomain error 2 - invalid path)" UserInfo=0x7fcfb0ea9840 {NSDescription=invalid path}
```
```
7/22/15 6:27:13.618 PM bird[975]: Sqlite request <PQLStatement:0x7fcfb0d40610 {  SELECT next_retry_stamp, throttle_id, zone_rowid, item_id, apply_kind     FROM client_unapplied_table    WHERE throttle_state = 1 ORDER BY next_retry_stamp, throttle_id    LIMIT ?}> failed on <PQLConnection:0x7fcfb0e245a0 clientTruth {batch:1:0 txn:0 auto-commit:0}> with error [Error Domain=SqliteErrorDomain Code=4 "The operation couldn’t be completed. (SqliteErrorDomain error 4 - callback requested query abort)" UserInfo=0x7fcfb0c629e0 {NSDescription=callback requested query abort, SqliteExtendedCode=4, SqliteSQL=  SELECT next_retry_stamp, throttle_id, zone_rowid, item_id, apply_kind     FROM client_unapplied_table    WHERE throttle_state = 1 ORDER BY next_retry_stamp, throttle_id    LIMIT ?}]
```
```
7/22/15 6:27:13.615 PM bird[975]: Sqlite request <PQLStatement:0x7fcfb0f4a6f0 {INSERT INTO client_items ( rowid, zone_rowid, item_id, item_creator_id, item_sharing_options, item_localsyncupstate, item_in_flight_diffs, item_local_diffs, item_notifs_rank, item_stat_ckinfo, item_state, item_type, item_mode, item_birthtime, item_parent_id, item_filename, item_hidden_ext, item_finder_tags, item_xattr_signature, item_doc_id, item_file_id, item_generation, item_localname, item_processing_stamp, item_staged_file_id, item_staged_generation, item_bouncedname, version_name, version_ckinfo, version_mtime, version_size, version_thumb_size, version_thumb_signature, version_content_signature, version_xattr_signature, version_device, version_conflict_loser_etags, version_quarantine_info, version_uploaded_assets, version_upload_error, desired_version, item_live_conflict_loser_etags) VALUES ( ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)}> failed on <PQLConnection:0x7fcfb0e245a0 clientTruth {batch:1:2 txn:1 auto-commit:0}> with error [Error Domain=SqliteErrorDomain Code=19 "The operation couldn’t be completed. (SqliteErrorDomain error 19 - UNIQUE constraint failed: client_items.zone_rowid, client_items.item_doc_id)" UserInfo=0x7fcfb0d12830 {NSDescription=UNIQUE constraint failed: client_items.zone_rowid, client_items.item_doc_id, SqliteExtendedCode=2067, SqliteSQL=INSERT INTO client_items ( rowid, zone_rowid, item_id, item_creator_id, item_sharing_options, item_localsyncupstate, item_in_flight_diffs, item_local_diffs, item_notifs_rank, item_stat_ckinfo, item_state, item_type, item_mode, item_birthtime, item_parent_id, item_filename, item_hidden_ext, item_finder_tags, item_xattr_signature, item_doc_id, item_file_id, item_generation, item_localname, item_processing_stamp, item_staged_file_id, item_staged_generation, item_bouncedname, version_name, version_ckinfo, version_mtime, version_size, version_thumb_size, version_thumb_signature, version_content_signature, version_xattr_signature, version_device, version_conflict_loser_etags, version_quarantine_info, version_uploaded_assets, version_upload_error, desired_version, item_live_conflict_loser_etags) VALUES ( ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)}]
```
```
7/22/15 6:31:55.966 PM bird[975]: UNREACHABLE: saving items should really always work
```


Weird formatting here


the function's signature is inconsistent with trackPageView


good catch


test comment


another


Should this be hardcoded?


static because there is no need to allocate this table in the stack every time the function is called. Outside of the function because improves readability of the function while making no practical difference if they can be access by other functions since it will be "static const" anyway.


s/to//


@ameyagholkar @olegsych @scsouthw @AlexBulankou do you guys think *"properties?: {[string]:string}, measurements?: {[string]:string}"* is better than pure JS syntax?


This is a silly nitpick, but in JonPRL, I try to use the following conventions:

1. two space indentation
2. no vertical alignment ("ascii art") of things like equals signs or colors or arrows


Would you mind fix this in Classic Xabber or that's too much to ask?


Yeah, you're right. I'm going to get rid of it. Thanks!


The comment was a little confusing. the script also set $host_id to something but that never got used, it was always a hostname. So your fix just changes the variable name from id to name which is more descriptive, eliminates the host_id being set even though its never used, and uses cut instead of jq.  I like that you are using the specified host but if the user enters nothing it chooses the first host in the list as I did before. This is a nice update, thanks for simplifying it a bit.


Here, I'm using `SinglePass` when deciding whether to use a sentinel or a cursor to terminate the range. Certainly if the base range is Bounded, the `intersperse` view can be too. When the base range is single-pass, the state stored in the end cursor is never referenced. In those cases, is a space optimization to use an empty sentinel instead.

What is your rationale for changing this?


It might be nice to put some of these protocols in extensions... or maybe not. I'm still a bit up in the air on my preference there as well.


OK, so I'll move those .class files back to Java 6 for now-- I've already
done so in a branch.

On Tue, Jul 21, 2015 at 5:14 PM, Aaron Tomb <notifications@github.com>
wrote:

> The main difficulty with moving to more recent Java versions is that the
> bytecode parser we use doesn't support the more recent instructions, and
> therefore can't parse the JDK .jar files included with those versions. I
> definitely think we should update the parser, but it's a non-trivial amount
> of work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GaloisInc/jvm-verifier/commit/1d48a560b40e0d41aa444e2ca891ae847539f270#commitcomment-12287913>
> .
>


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1246 of 1247 tests passed)

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


> Does a relationship make sense with a ``NULL`` start time? I suppose if we don't know when it started

Yeah, it was for the case where we don't know when the relationship started. In that case, we at least know it's <= created date.

>  Would that mean that a relationship without a start or end is currently active?

I'd say so, since you can't say definitively it was dropped. The problem would be the case where you have report 1 and 2 for the same source, where 2 was released after 1, and inclusion is the only information we have. From a pull with the simple ``start or end`` criteria, that would mean that those from 1 would still be considered active. Should think on this a little more.


We recently enabled automatic flush on window closing and page navigation away. We need mention it here (and people don't need to call it "manually" on window closing anymore)


add EOL


How a field behaves / moves (or not) is generally determined by the field's own movement-related effects (or lack thereof). Being created in a system or at some other location in space is irrelevant to those effects.


You mean make sure we're not overriding the method? or prevent overriding in a subclass?


no problem


The problem is that when the focus window is set to our fullscreen window we will lose fullscreen when you click on a window on your second monitor. This worked around that by creating a dummy window that is always in focus.


0744 means other uses can read files inside the directory, but can't list the directory contents. Is that the desired behavior? If so, why?


a comment


fixes_for_VS2015 (mbadams5) - x86_64-MacOS-10.9-clang: OK (1757 of 1757 tests passed)

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


Yo tengo un fork, que viene a ser como una especie de copia que hago yo de tu proyecto, paa trabajar de forma segura con él. Creo que tu podrías hacer fork de mis proyectos por ejemplo, yo lo he hecho del tuyo.


Thanks for the patch. I think a slightly shorter version, and one which would support non-numpy values (it possible that arrays or normal scalars could be passed to the function) would be to do

    if N.any(vals < xcoords[0]) or N.any(vals > xcoords[-1]):


This won't do anything as there's no "properties" property in default config properties :)


Totally not a bot.


Done.


WTF! You got termites in your unit test!


It actually was a keyword list already, but this is some syntactic sugar for keyword lists that only use atoms as indexes :)


This doesn't compile on some platforms. Can it be deleted?


Why this change?


I'm suspicious about this definition of factor, the one I know and love:

`factor :: Prob -> Measure ()`

It gets called that since if we view our programs as graphical models, this function would
add a factor to the factor graph.


Maybe you should take a look at the [Code Style Guidelines](https://source.android.com/source/code-style.html) for Android before doing this.


(That is, avoid the " = default" syntax?)


We previously had the continue in place because gotuf was trying to sign things with the wrong cryptoservice, and returning an error was causing problems (but returning no signatures was ok). I wouldn't change this unless you're confident that gotuf is no longer doing this.

Also, if you remove the continue, remove the comment explaining why we continue.


There's no measurements on TelemetryContext


Don't do a look up here, you already have the instance, just do:

```
instance.slug = slugify(instance.name)
instance.save()
```


There's no properties on TelemetryContext


ok


Sorry, I mean that should a relationship fail validation if there are other intersecting ranges. This line used to check for intersecting ranges on clean, with the changes I don't think the check is still happening. That being said, the check does involve a query against the db -- perhaps expensive for an implicit call.


Unused import


3 days in a row without commits. Any updates? I am afraid, that development of PGE is slowed to screeching crawl for indefinite amount of time. Sorry for asking such stupid question...


See a lot of C# 6.0 usage which isn't entirely supported in VS2013. Are we moving to VS2015 already?


:+1: I'll stick with that from here on out.


test


Sounds good


getAndIncrement uses CAS. So I'll just make the strictly ordered cyclic counter.


Fixed. And in zip_with. Any short explanation as to why? They seem fairly equivalent to me (instantiating `operator value_type` vs. instantiating `type`).


Fixed


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.799.4ff52fbbf33fbefe68810170efb82bdc7a484bf8](http://build.palaso.org/viewLog.html?buildId=59925&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 390, ignored: 3 Build time: 00:03:08


missing description for properties and measurements


I like this syntax. I would also prefer the string "Map of key (string) to value (string)" added to line 47 below just to explicit.


> suffix `InSystem` does not tell me that this field is special

Well, because it isn't. The difference between the functions has (in theory) nothing to do with the *type* of field you want to create (you can pass all field types to both functions), only with the *location* at which the field is created. `CreateField` just places the new field at the specified coordinates. `CreateFieldInSystem` does exactly what the name says - it creates a field *in* a system, which means, at the location of the system and kind of *attaches* it to the system. Think of it as a universe object that is part of / contained within the system like planets.

In practice that of course makes only sense with stationary fields.

> Field in system, system in universe, so field in universe and field created with coordinates in universe too. It is confusing.

Probably, but that's an issue of the underlying engine, I just exposed the functionality, I didn't implement it. All universe objects have their own coordinates, even when they are contained within other universe objects (like planets within systems). If an universe object is contained within another universe object, its coordinates must be the same as those of the container. That of course also applies to fields that are contained by a system.

> If you add you explanation about difference to docstrings it would be good.

Well, yeah, of course, but this docstring for interface functions thing is part of the revision of the whole server side Python interface, which is still on the todo list. Anyway, this doesn't really affect the function names.

>  I want to see information about field classes(mobile or stationary) in function names.

See above. Doesn't make sense, because the functions don't determine the field type, that's done by the field type name parameter. Adding an indicator to this type name if the field is stationary or moving might make sense, though.

> > but also inserted into the system, bound to it,

> Yor previous explanation is good. But this means nothing to me. I think It will confuse other.

Well, made part of the system like a planet - better...?


Dumb question: Presumably I should just revert view_mensural.cpp to the
previous version, commit, then push, yes? But I don't see how to go back
to a previous version in SourceTree, nor do I see a commit (in Git view)
for this change or a commit with the hex number in the subject line
above. ??

--Don

On 7/21/15 8:15 AM, Laurent Pugin wrote:
>
> This needs to be changed back, sorry for this. The problem is that
> there is no default value for all the MEI attributes, including for
> boolean. As a result, data_BOOLEAN can be "true", "false" but also
> "NONE", which means that no value is given in the MEI. Then it is up
> to us to decided what to do for each individual case. So HasColor()
> can be used to check if there is value (true or false) in the MEI and
> GetColor() to get the actual value. I know this is confusing but I
> think it is the best we can do.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rism-ch/verovio/commit/75a7ea6d8ee66de821c3918058a9f2198a19247b#commitcomment-12273569>.
>

--
Donald Byrd
Woodrow Wilson Indiana Teaching Fellow
Adjunct Associate Professor of Informatics
Visiting Scientist, Research Technologies
Indiana University Bloomington


Talking with Tim Robertson, he says that the URL should be: gbif.org/occurrence/<id>


Maybe the commit message (relies on ordering of the sequence) should be copied into a comment here to clarify that this method is stricter than the json specification requires.


@nmerket Debug builds went yellow.  Looks to be an assertion error on array bounds similar to [this](https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-beef593f2f08d31ec2ffb6ce13f99d652cc26764-x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug.html).  Chat at the next break?


This change didn't work correctly in production, as the CSS file is not loaded in this URL:
https://play.freeciv.org/civsocket/7002/status


I only do this so things look consistent when I work in the browser. I don't think this is needed once the app is on the device.


LGTM


When I print update_player, I am getting both values... the former and the new information. I think that is due to the fact that I am calling self.create_player. But replacing information...doesn't that require creating a new name or is it simply replacing? Do I write a for loop? to replace the old with the new or do I simply override by assigning the variable = old value with a new value? I feel like I am on the cusp of getting this.


Bootstrap v3 support for horizontal collapsing via width class seems like maybe they decided to remove it?
so for this to work, we need to add CSS:
```
.collapsing {
  height: auto;
  -webkit-transition-property: visibility;
  transition-property: visibility; }

.collapsing.width {
  width: 0;
  -webkit-transition-property: width, visibility;
  transition-property: width, visibility; }

.collapsing:not(.width) {
  height: 0;
  -webkit-transition-property: height, visibility;
  transition-property: height, visibility; }
```


Looks good, but can we do "{$key}-{$value}" on line 46? It'll work either way because of the double quotes, but http://stackoverflow.com/questions/10512452/php-using-a-variable-inside-a-double-quotes makes the case that it's better to always wrap vars in brackets to make it explicit which part is the var and which part is the string.

I'm also okay with $key . '-' . $value, but I like the look of "{$key}-{$value}" more.


@jonsterling Fixed in the latest commit :) I also have everything implemented (in a rough form) except the parsing currently. This file contains all the ugly slow implementation details so I think it'll be possible to iteratively improve this.


OK. I believe this is TS syntax.


Tab vs space?


Awesome, glad that's gone.


This works now?


Yes.  It can safely be deleted.


Yay!


I've pushed a new commit onto this branch / pull request to remove it.


lol you just made it slower this won't fix shit :p


Thank you. That drove me crazy.


Sure thing.

________________________________
From: Edwin Lee <notifications@github.com>
Sent: Wednesday, July 22, 2015 11:52 AM
To: NREL/EnergyPlus
Cc: Merket, Noel
Subject: Re: [EnergyPlus] Merge pull request #4698 from NREL/hpwh_residential (e07692e)

@nmerket<https://github.com/nmerket> Debug builds went yellow. Looks to be an assertion error on array bounds similar to this<https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-beef593f2f08d31ec2ffb6ce13f99d652cc26764-x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug.html>. Chat at the next break?

-
Reply to this email directly or view it on GitHub<https://github.com/NREL/EnergyPlus/commit/e07692e592865fadb3ed77c2d8a1bedaa838ecf7#commitcomment-12303904>.


Does TS 1.5.3 now support JSPM bundling?


`SnippetExtractor` seems to be the the only dependency on private APIs. Should we copy that one over, too?


STDOUT


arguments


Strange, I had comments all over this commit, but now they're gone. What happened?


This seems like it should be at the end of this method, not the first thing in it.


I had already thought (but not thought through) that manuscripts should not closed *at all* when doing something with the sessions. Maybe they should rather stay around? But I'm not sure what's worse performance-wise: keeping copies of open manuscript objects or loading them again after switching sessions.

Anyway, good catch, may be the solution to that issue.


In fact, should it happen before or after the animation runs below?


We don't call GL_RenderReadPixels anymore, so the alteration is not required anymore


I don't know where my comments went. But basically, I disagree with every edit in this commit. Please revert it.


Looks good. I will wait for the followup changes and merge.


@FWAJL
This was committed under the wrong Issue Number.  Correct Issue number is #1056.