[R/composite.R:2:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L2) *style:* **Use two spaces to indent, never tabs.**
```r
	sum_mat <- mat_from_ret_obj(retina_list[[1]], rotation=0, ...)
^
```
[R/composite.R:2:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L2) *style:* **Commented code should be removed.**
```r
##' @param retina_list A list containing retina objects, with names(retina_list) all defined.
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:2:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L2) *style:* **Use two spaces to indent, never tabs.**
```r
  									spin_spatial_res=128,
  ^~~~~~~~~
```
[R/composite.R:3:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L3) *style:* **Use two spaces to indent, never tabs.**
```r
	n <- length(retina_list)
^
```
[R/composite.R:3:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L3) *style:* **Commented code should be removed.**
```r
##' @param spin_spatial_res Resolution passed to the spin-optimization.
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:3:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L3) *style:* **Use two spaces to indent, never tabs.**
```r
  									plot_spatial_res=512,
  ^~~~~~~~~
```
[R/composite.R:4:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L4) *style:* **Use two spaces to indent, never tabs.**
```r
	for (i in 2:n) {
^
```
[R/composite.R:4:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L4) *style:* **Commented code should be removed.**
```r
##' @param theta_interval number of degrees to traverse in spin-optimization
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:4:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L4) *style:* **Use two spaces to indent, never tabs.**
```r
  									theta_interval=10,
  ^~~~~~~~~
```
[R/composite.R:5:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L5) *style:* **Use two spaces to indent, never tabs.**
```r
		sum_mat <- sum_mat + mat_from_ret_obj(retina_list[[i]],
^~
```
[R/composite.R:5:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L5) *style:* **Use two spaces to indent, never tabs.**
```r
  									plot=FALSE,
  ^~~~~~~~~
```
[R/composite.R:6:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L6) *style:* **Use two spaces to indent, never tabs.**
```r
											  rotation_indices[i],
^~~~~~~~~~~
```
[R/composite.R:7:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L7) *style:* **Use two spaces to indent, never tabs.**
```r
											  ...)
^~~~~~~~~~~
```
[R/composite.R:8:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L8) *style:* **Use two spaces to indent, never tabs.**
```r
	}
^
```
[R/composite.R:9:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L9) *style:* **Use two spaces to indent, never tabs.**
```r
	rotation_degree_list <- compute_rotation_matrix( retina_list,
^
```
[R/composite.R:9:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L9) *style:* **Use two spaces to indent, never tabs.**
```r
	return(sum_mat)
^
```
[R/composite.R:9:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L9) *style:* **Commented code should be removed.**
```r
##' @author Brian Cohn \email{brian_cohn14@@pitzer.edu} and Lars Schmitz
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:10:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L10) *style:* **Use two spaces to indent, never tabs.**
```r
												spatial_res=spin_spatial_res,
^~~~~~~~~~~~
```
[R/composite.R:11:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L11) *style:* **Use two spaces to indent, never tabs.**
```r
												theta_interval=theta_interval, ...)[1,]
^~~~~~~~~~~~
```
[R/composite.R:14:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L14) *style:* **Use two spaces to indent, never tabs.**
```r
	n <- length(retina_list)
^
```
[R/composite.R:15:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L15) *style:* **Use two spaces to indent, never tabs.**
```r
	sum <- map_vec_sum(retina_list, rotation_degree_list, spatial_res=plot_spatial_res)
^
```
[R/composite.R:16:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L16) *style:* **Use two spaces to indent, never tabs.**
```r
	composite_mat <- sum/n #get the average
^
```
[R/composite.R:17:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L17) *style:* **Use two spaces to indent, never tabs.**
```r
	if (plot){
^
```
[R/composite.R:17:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L17) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/composite.R:18:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L18) *style:* **Use two spaces to indent, never tabs.**
```r
		message('Plotting')
^~
```
[R/composite.R:18:32:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L18) *style:* **Put spaces around all infix operators.**
```r
if (class(rotation_degree_list)!="matrix") {
                              ~^~~
```
[R/composite.R:19:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L19) *style:* **Use two spaces to indent, never tabs.**
```r
		dev.new()
^~
```
[R/composite.R:19:30:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L19) *style:* **Put spaces around all infix operators.**
```r
	rotation_degree_list <- compute_rotation_matrix( retina_list,
                            ~^~~
```
[R/composite.R:20:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L20) *style:* **Use two spaces to indent, never tabs.**
```r
		plot_from_MAT(  MATRIX=composite_mat,
^~
```
[R/composite.R:21:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L21) *style:* **Use two spaces to indent, never tabs.**
```r
						extrapolate=TRUE,
^~~~~~
```
[R/composite.R:22:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L22) *style:* **Use two spaces to indent, never tabs.**
```r
						spatial_res = plot_spatial_res,
^~~~~~
```
[R/composite.R:22:9:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L22) *style:* **Only use double-quotes.**
```r
message('No coordinates submitted, spin optimization chosen')
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:23:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L23) *style:* **Use two spaces to indent, never tabs.**
```r
						col_levels=50,
^~~~~~
```
[R/composite.R:24:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L24) *style:* **Use two spaces to indent, never tabs.**
```r
						contour_levels=20,
^~~~~~
```
[R/composite.R:24:11:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L24) *style:* **Put spaces around all infix operators.**
```r
	n <- length(retina_list)
         ~^~~
```
[R/composite.R:25:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L25) *style:* **Use two spaces to indent, never tabs.**
```r
						contour_breaks_source =  c(min(composite_mat),max(composite_mat)),
^~~~~~
```
[R/composite.R:25:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L25) *style:* **lines should not be more than 80 characters.**
```r
	sum <- map_vec_sum(retina_list, rotation_degree_list, spatial_res=plot_spatial_res)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:25:9:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L25) *warning:* **no visible global function definition for ‘map_vec_sum’**
```r
	sum <- map_vec_sum(retina_list, rotation_degree_list, spatial_res=plot_spatial_res)
        ^~~~~~~~~~~
```
[R/composite.R:25:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L25) *style:* **Put spaces around all infix operators.**
```r
	sum <- map_vec_sum(retina_list, rotation_degree_list, spatial_res=plot_spatial_res)
           ~^~~
```
[R/composite.R:25:74:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L25) *style:* **Put spaces around all infix operators.**
```r
	sum <- map_vec_sum(retina_list, rotation_degree_list, spatial_res=plot_spatial_res)
                                                                        ~^~
```
[R/composite.R:26:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L26) *style:* **Use two spaces to indent, never tabs.**
```r
						col_breaks_source     =  c(min(composite_mat),max(composite_mat)),
^~~~~~
```
[R/composite.R:26:23:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L26) *style:* **Put spaces around all infix operators.**
```r
	composite_mat <- sum/n #get the average
                     ~^~~
```
[R/composite.R:26:29:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L26) *style:* **Put spaces around all infix operators.**
```r
	composite_mat <- sum/n #get the average
                           ~^~
```
[R/composite.R:27:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L27) *style:* **Use two spaces to indent, never tabs.**
```r
						z1 = Pmol_753$azimuthal_data.datapoints[[1]]$z,
^~~~~~
```
[R/composite.R:27:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L27) *style:* **Place a space before left parenthesis, except in a function call.**
```r
	if (plot){
           ^
```
[R/composite.R:28:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L28) *style:* **Use two spaces to indent, never tabs.**
```r
						z2 = Pmol_752$azimuthal_data.datapoints[[1]]$z)
^~~~~~
```
[R/composite.R:28:25:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L28) *style:* **Only use double-quotes.**
```r
		message('Plotting')
                        ^~~~~~~~~~
```
[R/composite.R:29:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L29) *style:* **Use two spaces to indent, never tabs.**
```r
	} #end plotif
^
```
[R/composite.R:30:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L30) *style:* **Use two spaces to indent, never tabs.**
```r
	return (composite_mat)
^
```
[R/composite.R:30:39:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L30) *style:* **Put spaces around all infix operators.**
```r
		plot_from_MAT(  MATRIX=composite_mat,
                                     ~^~
```
[R/composite.R:31:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
						extrapolate=TRUE,
                       ^
```
[R/composite.R:35:71:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L35) *style:* **Put spaces around all infix operators.**
```r
						contour_breaks_source =  c(min(composite_mat),max(composite_mat)),
                                                                     ~^~
```
[R/composite.R:36:71:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L36) *style:* **Put spaces around all infix operators.**
```r
						col_breaks_source     =  c(min(composite_mat),max(composite_mat)),
                                                                     ~^~
```
[R/composite.R:37:52:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L37) *style:* **Put spaces around all infix operators.**
```r
						z1 = Pmol_753$azimuthal_data.datapoints[[1]]$z,
                                                  ~^~
```
[R/composite.R:38:52:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L38) *style:* **Put spaces around all infix operators.**
```r
						z2 = Pmol_752$azimuthal_data.datapoints[[1]]$z)
                                                  ~^~
```
[R/composite.R:44:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L44) *style:* **Commented code should be removed.**
```r
##' @param retina_list A list containing retina objects, with names(retina_list) all defined.
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:48:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L48) *style:* **Commented code should be removed.**
```r
##' @author Brian Cohn \email{brian_cohn14@@pitzer.edu} and Lars Schmitz
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:50:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L50) *warning:* **no visible global function definition for ‘mat_from_ret_obj’**
```r
	sum_mat <- mat_from_ret_obj(retina_list[[1]], rotation=0, ...)
            ^~~~~~~~~~~~~~~~
```
[R/composite.R:50:17:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L50) *style:* **Put spaces around all infix operators.**
```r
	sum_mat <- mat_from_ret_obj(retina_list[[1]], rotation=0, ...)
               ~^~~
```
[R/composite.R:50:63:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L50) *style:* **Put spaces around all infix operators.**
```r
	sum_mat <- mat_from_ret_obj(retina_list[[1]], rotation=0, ...)
                                                             ~^~
```
[R/composite.R:51:11:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L51) *style:* **Put spaces around all infix operators.**
```r
	n <- length(retina_list)
         ~^~~
```
[R/composite.R:52:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L52) *style:* **Place a space before left parenthesis, except in a function call.**
```r
	for (i in 2:n) {
            ^
```
[R/composite.R:53:25:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L53) *style:* **Put spaces around all infix operators.**
```r
		sum_mat <- sum_mat + mat_from_ret_obj(retina_list[[i]],
                       ~^~~
```
[R/composite.R:53:36:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/composite.R#L53) *style:* **Put spaces around all infix operators.**
```r
		sum_mat <- sum_mat + mat_from_ret_obj(retina_list[[i]],
                                  ~^~
```
[R/matrix_retobj.R:2:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L2) *style:* **Use two spaces to indent, never tabs.**
```r
	x2 <- retina_object$azimuthal_data.datapoints[[1]]$x
^
```
[R/matrix_retobj.R:3:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L3) *style:* **Use two spaces to indent, never tabs.**
```r
	y2 <- retina_object$azimuthal_data.datapoints[[1]]$y
^
```
[R/matrix_retobj.R:4:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L4) *style:* **Use two spaces to indent, never tabs.**
```r
	z2 <- retina_object$azimuthal_data.datapoints[[1]]$z
^
```
[R/matrix_retobj.R:6:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L6) *style:* **Use two spaces to indent, never tabs.**
```r
	# Rotation
^
```
[R/matrix_retobj.R:7:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L7) *style:* **Use two spaces to indent, never tabs.**
```r
	ori <- c(-90, 90, rotation)
^
```
[R/matrix_retobj.R:8:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L8) *style:* **Use two spaces to indent, never tabs.**
```r
	az <- mapproject(x=retina_object$trimmed_data[,2], retina_object$trimmed_data[,1],
^
```
[R/matrix_retobj.R:9:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L9) *style:* **Use two spaces to indent, never tabs.**
```r
					   projection="azequidistant", orientation=ori)
^~~~~
```
[R/matrix_retobj.R:10:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L10) *style:* **Use two spaces to indent, never tabs.**
```r
	x2 <- az$x #overwirte old value
^
```
[R/matrix_retobj.R:10:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L10) *style:* **Commented code should be removed.**
```r
##' @author Brian Cohn \email{brian_cohn14@@pitzer.edu} and Lars Schmitz
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:11:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L11) *style:* **Use two spaces to indent, never tabs.**
```r
	ifelse(reflect,x2<-(-1.0*x2), message(''))
^
```
[R/matrix_retobj.R:12:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L12) *style:* **Use two spaces to indent, never tabs.**
```r
	y2 <- az$y #overwrite old value
^
```
[R/matrix_retobj.R:12:17:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L12) *style:* **Put spaces around all infix operators.**
```r
mat_from_ret_obj<- function(retina_object, rotation=0, n=1, reflect=TRUE, ...){
               ~^
```
[R/matrix_retobj.R:13:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L13) *style:* **Put spaces around all infix operators.**
```r
	x2 <- retina_object$azimuthal_data.datapoints[[1]]$x
          ~^~~
```
[R/matrix_retobj.R:14:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L14) *style:* **Use two spaces to indent, never tabs.**
```r
	# Fitting models
^
```
[R/matrix_retobj.R:14:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L14) *style:* **Put spaces around all infix operators.**
```r
	y2 <- retina_object$azimuthal_data.datapoints[[1]]$y
          ~^~~
```
[R/matrix_retobj.R:15:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L15) *style:* **Use two spaces to indent, never tabs.**
```r
	map2fit <- fit_plot_azimuthal(x2,y2,z2,
^
```
[R/matrix_retobj.R:15:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L15) *style:* **Put spaces around all infix operators.**
```r
	z2 <- retina_object$azimuthal_data.datapoints[[1]]$z
          ~^~~
```
[R/matrix_retobj.R:16:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L16) *style:* **Use two spaces to indent, never tabs.**
```r
									plot_suppress=TRUE, extrapolate=TRUE,
^~~~~~~~~
```
[R/matrix_retobj.R:17:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L17) *style:* **Use two spaces to indent, never tabs.**
```r
									outer.radius=pi/2.0,
^~~~~~~~~
```
[R/matrix_retobj.R:18:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L18) *style:* **Use two spaces to indent, never tabs.**
```r
									falciform_coords = retina_object$azimuthal_data.falciform[[1]],...)
^~~~~~~~~
```
[R/matrix_retobj.R:18:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L18) *style:* **Put spaces around all infix operators.**
```r
	ori <- c(-90, 90, rotation)
           ~^
```
[R/matrix_retobj.R:19:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **Use two spaces to indent, never tabs.**
```r
	MAT <- map2fit[[2]]$z
^
```
[R/matrix_retobj.R:19:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **lines should not be more than 80 characters.**
```r
	az <- mapproject(x=retina_object$trimmed_data[,2], retina_object$trimmed_data[,1],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:19:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **Put spaces around all infix operators.**
```r
	az <- mapproject(x=retina_object$trimmed_data[,2], retina_object$trimmed_data[,1],
          ~^~~
```
[R/matrix_retobj.R:19:23:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **Use two spaces to indent, never tabs.**
```r
	MAT <- map2fit[[2]]$z
                      ^
```
[R/matrix_retobj.R:19:27:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **Put spaces around all infix operators.**
```r
	az <- mapproject(x=retina_object$trimmed_data[,2], retina_object$trimmed_data[,1],
                         ~^~
```
[R/matrix_retobj.R:19:84:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L19) *style:* **Trailing whitespace is superfluous.**
```r
	az <- mapproject(x=retina_object$trimmed_data[,2], retina_object$trimmed_data[,1],
                                                                                ^
```
[R/matrix_retobj.R:20:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L20) *style:* **Use two spaces to indent, never tabs.**
```r
	#reflect the eye
^
```
[R/matrix_retobj.R:21:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L21) *style:* **Use two spaces to indent, never tabs.**
```r
	return(MAT/n)
^
```
[R/matrix_retobj.R:21:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L21) *style:* **Put spaces around all infix operators.**
```r
	x2 <- az$x #overwirte old value
          ~^~~
```
[R/matrix_retobj.R:22:26:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L22) *style:* **Put spaces around all infix operators.**
```r
	ifelse(reflect,x2<-(-1.0*x2), message(''))
                        ~^~~
```
[R/matrix_retobj.R:22:28:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L22) *style:* **Place a space before left parenthesis, except in a function call.**
```r
	ifelse(reflect,x2<-(-1.0*x2), message(''))
                           ^
```
[R/matrix_retobj.R:22:33:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L22) *style:* **Put spaces around all infix operators.**
```r
	ifelse(reflect,x2<-(-1.0*x2), message(''))
                               ~^~
```
[R/matrix_retobj.R:22:47:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L22) *style:* **Only use double-quotes.**
```r
	ifelse(reflect,x2<-(-1.0*x2), message(''))
                                              ^~
```
[R/matrix_retobj.R:23:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L23) *style:* **Put spaces around all infix operators.**
```r
	y2 <- az$y #overwrite old value
          ~^~~
```
[R/matrix_retobj.R:26:17:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L26) *style:* **Put spaces around all infix operators.**
```r
	map2fit <- fit_plot_azimuthal(x2,y2,z2,
               ~^~~
```
[R/matrix_retobj.R:30:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L30) *style:* **Put spaces around all infix operators.**
```r
	MAT <- map2fit[[2]]$z
           ~^~~
```
[R/matrix_retobj.R:30:23:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/matrix_retobj.R#L30) *style:* **Trailing whitespace is superfluous.**
```r
	MAT <- map2fit[[2]]$z
                      ^
```
[R/polar_plot_from_mat.R:4:9:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L4) *style:* **Variable and function names should be all lowercase.**
```r
require(RColorBrewer)
        ^~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:22:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L22) *style:* **Commented code should be removed.**
```r
##' @param single_point_overlay Overlay "key" data point with square (0 = No, Other = number of pt)
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:32:14:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L32) *style:* **Put spaces around all infix operators.**
```r
plot_from_MAT<- function(
            ~^
```
[R/polar_plot_from_mat.R:33:4:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L33) *style:* **Commented code should be removed.**
```r
  ### Plotting data (in cartesian) - will be converted to polar space.
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:34:18:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L34) *style:* **Trailing whitespace is superfluous.**
```r
  z1, z2, MATRIX,
                 ^
```
[R/polar_plot_from_mat.R:36:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L36) *style:* **Use two spaces to indent, never tabs.**
```r
	minitics <- seq(-outer.radius, outer.radius, length.out = spatial_res)
^
```
[R/polar_plot_from_mat.R:37:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L37) *style:* **Use two spaces to indent, never tabs.**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
^
```
[R/polar_plot_from_mat.R:38:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L38) *style:* **Use two spaces to indent, never tabs.**
```r
	Mat = MATRIX
^
```
[R/polar_plot_from_mat.R:42:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L42) *style:* **lines should not be more than 80 characters.**
```r
  col_breaks_source = 2, # Where to calculate the color breaks from (1=data,2=surface)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:44:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L44) *style:* **lines should not be more than 80 characters.**
```r
  col_levels = 10,    # Number of color levels to use - must match length(col) if
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:44:82:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L44) *style:* **Trailing whitespace is superfluous.**
```r
  col_levels = 10,    # Number of color levels to use - must match length(col) if
                                                                                ^
```
[R/polar_plot_from_mat.R:46:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L46) *style:* **lines should not be more than 80 characters.**
```r
  col = rev(colorRampPalette(brewer.pal(11,"PuOr"))(col_levels)),  # Colors to plot
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:46:44:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L46) *style:* **Commas should always have a space after.**
```r
  col = rev(colorRampPalette(brewer.pal(11,"PuOr"))(col_levels)),  # Colors to plot
                                           ^
```
[R/polar_plot_from_mat.R:49:30:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L49) *style:* **Put spaces around all infix operators.**
```r
  contour_levels = col_levels+1,
                            ~^~
```
[R/polar_plot_from_mat.R:51:20:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L51) *style:* **Put spaces around all infix operators.**
```r
  outer.radius = pi/2,
                  ~^~
```
[R/polar_plot_from_mat.R:51:23:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L51) *style:* **Trailing whitespace is superfluous.**
```r
  outer.radius = pi/2,
                      ^~
```
[R/polar_plot_from_mat.R:52:28:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L52) *style:* **Commas should always have a space after.**
```r
  circle.rads = pretty(c(0,outer.radius)), #Radius lines
                           ^
```
[R/polar_plot_from_mat.R:54:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L54) *style:* **Use two spaces to indent, never tabs.**
```r
	contour_breaks = seq(min(z,na.rm=TRUE),max(z,na.rm=TRUE),
^
```
[R/polar_plot_from_mat.R:54:64:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L54) *style:* **Trailing whitespace is superfluous.**
```r
  single_point_overlay=0, #Overlay "key" data point with square
                                                               ^
```
[R/polar_plot_from_mat.R:55:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L55) *style:* **Use two spaces to indent, never tabs.**
```r
						 by=(max(z,na.rm=TRUE)-min(z,na.rm=TRUE))/(contour_levels-1))
^~~~~~
```
[R/polar_plot_from_mat.R:57:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L57) *style:* **lines should not be more than 80 characters.**
```r
  interp.type = 1, #1 = Thin plate spline , null if you are inputting your own predicted surface
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:58:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L58) *style:* **Use two spaces to indent, never tabs.**
```r
	contour_breaks = seq(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE),
^
```
[R/polar_plot_from_mat.R:58:28:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L58) *style:* **Commas should always have a space after.**
```r
  lambda=0.001, xyrelief=1,...) {
                           ^
```
[R/polar_plot_from_mat.R:58:33:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L58) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  lambda=0.001, xyrelief=1,...) {
                                ^
```
[R/polar_plot_from_mat.R:58:34:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L58) *style:* **Trailing whitespace is superfluous.**
```r
  lambda=0.001, xyrelief=1,...) {
                                 ^
```
[R/polar_plot_from_mat.R:59:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L59) *style:* **Use two spaces to indent, never tabs.**
```r
						 by=(max(Mat,na.rm=TRUE)-min(Mat,na.rm=TRUE))/(contour_levels-1))
^~~~~~
```
[R/polar_plot_from_mat.R:59:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L59) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:61:33:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L61) *style:* **Commas should always have a space after.**
```r
  spatial_res <- length(MATRIX[,1])
                                ^
```
[R/polar_plot_from_mat.R:62:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L62) *style:* **Use two spaces to indent, never tabs.**
```r
	print(paste0("Manual contour range set from ", contour_breaks_source[1], " to ", contour_breaks_source[2]))
^
```
[R/polar_plot_from_mat.R:63:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L63) *style:* **Use two spaces to indent, never tabs.**
```r
	contour_breaks = pretty(contour_breaks_source,n=contour_levels)
^
```
[R/polar_plot_from_mat.R:63:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L63) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:64:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L64) *style:* **Use two spaces to indent, never tabs.**
```r
	contour_breaks = seq(contour_breaks_source[1],contour_breaks_source[2],
^
```
[R/polar_plot_from_mat.R:65:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L65) *style:* **Use two spaces to indent, never tabs.**
```r
						 by=(contour_breaks_source[2]-contour_breaks_source[1])/(contour_levels-1))
^~~~~~
```
[R/polar_plot_from_mat.R:66:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L66) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:67:18:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L67) *style:* **Put spaces around all infix operators.**
```r
	minitics <- seq(-outer.radius, outer.radius, length.out = spatial_res)
                ~^~~
```
[R/polar_plot_from_mat.R:67:65:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L67) *style:* **Put spaces around all infix operators.**
```r
	minitics <- seq(-outer.radius, outer.radius, length.out = spatial_res)
                                                               ~^~
```
[R/polar_plot_from_mat.R:68:2:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L68) *warning:* **local variable ‘grid.list’ assigned but may not be used**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
 ^~~~~~~~~
```
[R/polar_plot_from_mat.R:68:19:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L68) *style:* **Use <-, not =, for assignment.**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
                  ^
```
[R/polar_plot_from_mat.R:68:19:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L68) *style:* **Put spaces around all infix operators.**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
                 ~^~
```
[R/polar_plot_from_mat.R:68:27:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L68) *style:* **Put spaces around all infix operators.**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
                         ~^~
```
[R/polar_plot_from_mat.R:68:38:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L68) *style:* **Put spaces around all infix operators.**
```r
	grid.list = list(x=minitics,y=minitics) #choose locations to predict at
                                    ~^~
```
[R/polar_plot_from_mat.R:69:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L69) *style:* **Use <-, not =, for assignment.**
```r
	Mat = MATRIX
            ^
```
[R/polar_plot_from_mat.R:69:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L69) *style:* **Put spaces around all infix operators.**
```r
	Mat = MATRIX
           ~^~
```
[R/polar_plot_from_mat.R:71:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L71) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  else {stop("interp.type value not valid")}
       ^
```
[R/polar_plot_from_mat.R:71:44:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L71) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  else {stop("interp.type value not valid")}
                                           ^
```
[R/polar_plot_from_mat.R:78:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L78) *style:* **lines should not be more than 80 characters.**
```r
#   ###;; ________________________________________________________________________
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:80:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L80) *style:* **Variable and function names should be all lowercase.**
```r
  markNA <- matrix(minitics, ncol = spatial_res, nrow = spatial_res)
  ^~~~~~
```
[R/polar_plot_from_mat.R:80:69:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L80) *style:* **Trailing whitespace is superfluous.**
```r
  markNA <- matrix(minitics, ncol = spatial_res, nrow = spatial_res)
                                                                    ^
```
[R/polar_plot_from_mat.R:81:13:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L81) *style:* **Variable and function names should be all lowercase.**
```r
  Mat[!sqrt(markNA ^ 2 + t(markNA) ^ 2) < outer.radius] <- NA
            ^~~~~~
```
[R/polar_plot_from_mat.R:81:28:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L81) *style:* **Variable and function names should be all lowercase.**
```r
  Mat[!sqrt(markNA ^ 2 + t(markNA) ^ 2) < outer.radius] <- NA
                           ^~~~~~
```
[R/polar_plot_from_mat.R:81:62:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L81) *style:* **Trailing whitespace is superfluous.**
```r
  Mat[!sqrt(markNA ^ 2 + t(markNA) ^ 2) < outer.radius] <- NA
                                                             ^
```
[R/polar_plot_from_mat.R:82:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L82) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:84:7:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L84) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if ((length(contour_breaks_source == 1)) & (contour_breaks_source[1] == 1)){
      ^
```
[R/polar_plot_from_mat.R:85:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L85) *style:* **Use <-, not =, for assignment.**
```r
	contour_breaks = seq(min(z,na.rm=TRUE),max(z,na.rm=TRUE),
                       ^
```
[R/polar_plot_from_mat.R:85:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L85) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(z,na.rm=TRUE),max(z,na.rm=TRUE),
                      ~^~
```
[R/polar_plot_from_mat.R:85:41:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L85) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(z,na.rm=TRUE),max(z,na.rm=TRUE),
                                       ~^~
```
[R/polar_plot_from_mat.R:85:59:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L85) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(z,na.rm=TRUE),max(z,na.rm=TRUE),
                                                         ~^~
```
[R/polar_plot_from_mat.R:86:52:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L86) *style:* **Put spaces around all infix operators.**
```r
						 by=(max(z,na.rm=TRUE)-min(z,na.rm=TRUE))/(contour_levels-1))
                                                  ~^~
```
[R/polar_plot_from_mat.R:86:53:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L86) *style:* **Place a space before left parenthesis, except in a function call.**
```r
						 by=(max(z,na.rm=TRUE)-min(z,na.rm=TRUE))/(contour_levels-1))
                                                    ^
```
[R/polar_plot_from_mat.R:86:65:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L86) *style:* **Put spaces around all infix operators.**
```r
						 by=(max(z,na.rm=TRUE)-min(z,na.rm=TRUE))/(contour_levels-1))
                                                               ~^~
```
[R/polar_plot_from_mat.R:87:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L87) *style:* **Use two spaces to indent, never tabs.**
```r
	CL <- contourLines(x = minitics, y = minitics, Mat, levels = contour_breaks)
^
```
[R/polar_plot_from_mat.R:88:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L88) *style:* **Use two spaces to indent, never tabs.**
```r
	A <- lapply(CL, function(xy){
^
```
[R/polar_plot_from_mat.R:88:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L88) *style:* **lines should not be more than 80 characters.**
```r
  else if ((length(contour_breaks_source == 1)) & (contour_breaks_source[1] == 2)){
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:88:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L88) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  else if ((length(contour_breaks_source == 1)) & (contour_breaks_source[1] == 2)){
           ^
```
[R/polar_plot_from_mat.R:89:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L89) *style:* **Use two spaces to indent, never tabs.**
```r
	  lines(xy$x, xy$y, col = gray(.2), lwd = .5)
^
```
[R/polar_plot_from_mat.R:89:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L89) *style:* **Use <-, not =, for assignment.**
```r
	contour_breaks = seq(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE),
                       ^
```
[R/polar_plot_from_mat.R:89:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L89) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE),
                      ~^~
```
[R/polar_plot_from_mat.R:89:43:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L89) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE),
                                         ~^~
```
[R/polar_plot_from_mat.R:89:63:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L89) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE),
                                                             ~^~
```
[R/polar_plot_from_mat.R:90:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L90) *style:* **Use two spaces to indent, never tabs.**
```r
	})
^
```
[R/polar_plot_from_mat.R:90:52:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L90) *style:* **Put spaces around all infix operators.**
```r
						 by=(max(Mat,na.rm=TRUE)-min(Mat,na.rm=TRUE))/(contour_levels-1))
                                                  ~^~
```
[R/polar_plot_from_mat.R:90:53:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L90) *style:* **Place a space before left parenthesis, except in a function call.**
```r
						 by=(max(Mat,na.rm=TRUE)-min(Mat,na.rm=TRUE))/(contour_levels-1))
                                                    ^
```
[R/polar_plot_from_mat.R:90:67:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L90) *style:* **Put spaces around all infix operators.**
```r
						 by=(max(Mat,na.rm=TRUE)-min(Mat,na.rm=TRUE))/(contour_levels-1))
                                                                 ~^~
```
[R/polar_plot_from_mat.R:91:4:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L91) *style:* **Trailing whitespace is superfluous.**
```r
  }
   ^
```
[R/polar_plot_from_mat.R:92:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L92) *style:* **lines should not be more than 80 characters.**
```r
  else if ((length(contour_breaks_source) == 2) & (is.numeric(contour_breaks_source))){
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:92:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L92) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  else if ((length(contour_breaks_source) == 2) & (is.numeric(contour_breaks_source))){
           ^
```
[R/polar_plot_from_mat.R:93:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L93) *style:* **lines should not be more than 80 characters.**
```r
	print(paste0("Manual contour range set from ", contour_breaks_source[1], " to ", contour_breaks_source[2]))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:94:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L94) *style:* **Use <-, not =, for assignment.**
```r
	contour_breaks = pretty(contour_breaks_source,n=contour_levels)
                       ^
```
[R/polar_plot_from_mat.R:94:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L94) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = pretty(contour_breaks_source,n=contour_levels)
                      ~^~
```
[R/polar_plot_from_mat.R:94:56:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L94) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = pretty(contour_breaks_source,n=contour_levels)
                                                      ~^~
```
[R/polar_plot_from_mat.R:95:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L95) *style:* **Use two spaces to indent, never tabs.**
```r
	# internals for axis markup
^
```
[R/polar_plot_from_mat.R:95:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L95) *style:* **Use <-, not =, for assignment.**
```r
	contour_breaks = seq(contour_breaks_source[1],contour_breaks_source[2],
                       ^
```
[R/polar_plot_from_mat.R:95:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L95) *style:* **Put spaces around all infix operators.**
```r
	contour_breaks = seq(contour_breaks_source[1],contour_breaks_source[2],
                      ~^~
```
[R/polar_plot_from_mat.R:96:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L96) *style:* **Use two spaces to indent, never tabs.**
```r
	RMat <- function(radians){
^
```
[R/polar_plot_from_mat.R:96:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L96) *style:* **lines should not be more than 80 characters.**
```r
						 by=(contour_breaks_source[2]-contour_breaks_source[1])/(contour_levels-1))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/polar_plot_from_mat.R:96:52:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L96) *style:* **Put spaces around all infix operators.**
```r
						 by=(contour_breaks_source[2]-contour_breaks_source[1])/(contour_levels-1))
                                                  ~^~
```
[R/polar_plot_from_mat.R:96:53:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L96) *style:* **Place a space before left parenthesis, except in a function call.**
```r
						 by=(contour_breaks_source[2]-contour_breaks_source[1])/(contour_levels-1))
                                                    ^
```
[R/polar_plot_from_mat.R:96:78:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L96) *style:* **Put spaces around all infix operators.**
```r
						 by=(contour_breaks_source[2]-contour_breaks_source[1])/(contour_levels-1))
                                                                            ~^~
```
[R/polar_plot_from_mat.R:97:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L97) *style:* **Use two spaces to indent, never tabs.**
```r
	  matrix(c(cos(radians), sin(radians), -sin(radians), cos(radians)), ncol = 2)
^
```
[R/polar_plot_from_mat.R:98:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L98) *style:* **Use two spaces to indent, never tabs.**
```r
	}
^
```
[R/polar_plot_from_mat.R:98:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L98) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  else {stop("Invalid selection for \"contour_breaks_source\"")}
       ^
```
[R/polar_plot_from_mat.R:98:64:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L98) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  else {stop("Invalid selection for \"contour_breaks_source\"")}
                                                               ^
```
[R/polar_plot_from_mat.R:99:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L99) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[R/polar_plot_from_mat.R:99:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L99) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:100:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L100) *style:* **Use two spaces to indent, never tabs.**
```r
	circle <- function(x, y, rad = 1, nvert = 500){
^
```
[R/polar_plot_from_mat.R:101:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L101) *style:* **Use two spaces to indent, never tabs.**
```r
	  rads <- seq(0,2*pi,length.out = nvert)
^
```
[R/polar_plot_from_mat.R:101:7:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L101) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if ((length(col_breaks_source) == 1) & (col_breaks_source[1] == 1))
      ^
```
[R/polar_plot_from_mat.R:102:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Use two spaces to indent, never tabs.**
```r
	  xcoords <- cos(rads) * rad + x
^
```
[R/polar_plot_from_mat.R:102:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
  ^
```
[R/polar_plot_from_mat.R:102:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Use <-, not =, for assignment.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
       ^
```
[R/polar_plot_from_mat.R:102:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
      ~^~
```
[R/polar_plot_from_mat.R:102:17:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                ^
```
[R/polar_plot_from_mat.R:102:22:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                    ~^~
```
[R/polar_plot_from_mat.R:102:29:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                            ^
```
[R/polar_plot_from_mat.R:102:35:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                                  ^
```
[R/polar_plot_from_mat.R:102:40:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                                      ~^~
```
[R/polar_plot_from_mat.R:102:47:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L102) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  {zlim=c(min(z,na.rm=TRUE),max(z,na.rm=TRUE))}
                                              ^
```
[R/polar_plot_from_mat.R:103:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L103) *style:* **Use two spaces to indent, never tabs.**
```r
	  ycoords <- sin(rads) * rad + y
^
```
[R/polar_plot_from_mat.R:103:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L103) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  else if ((length(col_breaks_source) == 1) & (col_breaks_source[1] == 2))
           ^
```
[R/polar_plot_from_mat.R:104:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Use two spaces to indent, never tabs.**
```r
	  cbind(xcoords, ycoords)
^
```
[R/polar_plot_from_mat.R:104:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
  ^
```
[R/polar_plot_from_mat.R:104:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Use <-, not =, for assignment.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
       ^
```
[R/polar_plot_from_mat.R:104:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
      ~^~
```
[R/polar_plot_from_mat.R:104:19:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                  ^
```
[R/polar_plot_from_mat.R:104:24:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                      ~^~
```
[R/polar_plot_from_mat.R:104:31:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                              ^
```
[R/polar_plot_from_mat.R:104:39:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Commas should always have a space after.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                                      ^
```
[R/polar_plot_from_mat.R:104:44:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Put spaces around all infix operators.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                                          ~^~
```
[R/polar_plot_from_mat.R:104:51:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L104) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  {zlim=c(min(Mat,na.rm=TRUE),max(Mat,na.rm=TRUE))}
                                                  ^
```
[R/polar_plot_from_mat.R:105:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L105) *style:* **Use two spaces to indent, never tabs.**
```r
	}
^
```
[R/polar_plot_from_mat.R:105:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L105) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  else if ((length(col_breaks_source) == 2) & (is.numeric(col_breaks_source)))
           ^
```
[R/polar_plot_from_mat.R:106:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L106) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[R/polar_plot_from_mat.R:106:3:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L106) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {zlim=col_breaks_source}
  ^
```
[R/polar_plot_from_mat.R:106:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L106) *style:* **Use <-, not =, for assignment.**
```r
  {zlim=col_breaks_source}
       ^
```
[R/polar_plot_from_mat.R:106:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L106) *style:* **Put spaces around all infix operators.**
```r
  {zlim=col_breaks_source}
      ~^~
```
[R/polar_plot_from_mat.R:106:26:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L106) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  {zlim=col_breaks_source}
                         ^
```
[R/polar_plot_from_mat.R:107:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L107) *style:* **Use two spaces to indent, never tabs.**
```r
	# draw circles
^
```
[R/polar_plot_from_mat.R:107:8:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L107) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  else {stop("Invalid selection for \"col_breaks_source\"")}
       ^
```
[R/polar_plot_from_mat.R:107:60:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L107) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  else {stop("Invalid selection for \"col_breaks_source\"")}
                                                           ^
```
[R/polar_plot_from_mat.R:108:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L108) *style:* **Use two spaces to indent, never tabs.**
```r
	if (missing(circle.rads)){
^
```
[R/polar_plot_from_mat.R:108:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L108) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/polar_plot_from_mat.R:109:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L109) *style:* **Use two spaces to indent, never tabs.**
```r
	  circle.rads <- pretty(c(0,outer.radius-.4))
^
```
[R/polar_plot_from_mat.R:110:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L110) *style:* **Use two spaces to indent, never tabs.**
```r
	}
^
```
[R/polar_plot_from_mat.R:111:1:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L111) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[R/polar_plot_from_mat.R:111:12:](https://github.com/bcohn12/retina/blob/eeb6326c634d31d16c94a2945b75169b6cc1ff6a/R/polar_plot_from_mat.R#L111) *style:* **Use <-, not =, for assignment.**
```r
  Mat_plot = Mat
           ^
```


Nice idea to enter several values in one run!


It's good that you handle the case when both values are equally far from the 3rd one.


Let's talk about how you can squash this nested if/else statement with boolean short circuiting!


Did clean git checkouts of both the USD dev branch and the usd-build-club...

The patch works perfectly now!

There are some new bugs and quirks it seems...

1. The build-macos-maya.sh script does not get past the prerequisites because of "exit 0" at line 140 in the build_prerequisites-macos-maya.sh script it calls

2. After commenting out the exit in build_prerequisites-macos-maya.sh, the build-macos-maya.sh script does continue onto to run "configure.sh Maya" but even without errors it does not continue on to the cmake --build . --target install --config Release   Is there another exit or abort somewhere?


Worst commit I've ever seen


Wow, much needed improvements to the XML handling and looks like you did it pretty cleanly. good job!


Thank you for cleaning up the code! Always nice to have more readable and maintainable code.


hgignore is for mercurial. Likely this can be removed


Wow, this is a really helpful commit! Good job!


Like Jared mentions, the property was not used anywhere. I'm not sure I understand how an unused property can break anything on your end @aaronpowell


Thank you for fixing the find-bugs errors! Master has some find bug errors that have leaked to each branch, and hopefully when merged back into master, this will fix most of the issues.


I checked the demo app and there was a regression with the Search component which has a parent element. The search now on demo is not showing the results and If I remove this new block , everything works fine.


ok change1


creature_template_equipment_raw is no longer used and can be removed


The #9 in the title is meant to refer to IskyN/pyta#9.


Error: ADD `reg_mail` VARCHAR(255) NOT NULL DEFAULT '' AFTER `mail`,
Fix: ADD `reg_mail` VARCHAR(255) NOT NULL DEFAULT '' AFTER `email`,


I also added a displayInventory() method. It shows a dialog window (a very ugly one at the moment, I still need to style it).


Uh oh!  The longitude and latitude are flipped!


Thanks)


Do we want to use an unordered_map or a list for [pingers](https://github.com/MichaelWallace30/CommProtocol/blob/pinger/CommProto/include/CommProto/ping/pingmanager.h#L100)?  The unordered_map would give us O(1) access to Pingers given their destination ID.  This would help us when receiving packets and calling [Pinger::ResetReceiveTime()](https://github.com/MichaelWallace30/CommProtocol/blob/pinger/CommProto/include/CommProto/ping/pinger.h#L57).  However, the list can be sorted by the time before a ping needs to be sent.  This means we won't have to iterate through all of the Pingers.  Once [Pinger::GetNextPingTime()](https://github.com/MichaelWallace30/CommProtocol/blob/pinger/CommProto/include/CommProto/ping/pinger.h#L63) is positive and we know that [Pinger::ResetReceiveTime()](https://github.com/MichaelWallace30/CommProtocol/blob/pinger/CommProto/include/CommProto/ping/pinger.h#L57) wasn't called after our last check, we can stop iterating through the list.  Maybe we can use both both data structures?


Definitely true... I don't know how I'd fix this blunder in a backward compatible way though.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.695.e59a7ac6b1d44660d3a810c3f949f94e68ef9e78](http://build.palaso.org/viewLog.html?buildId=79970&buildTypeId=bt323) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 935, ignored: 56 Build time: 00:14:35
### Failed tests
```
LibChorus.Tests.dll: LibChorus.Tests.VcsDrivers.Mercurial.HgResumeTransportTests.CalculateEstimatedTimeRemaining_ExpectedTimeRemainingString: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.VcsDrivers.Mercurial.HgResumeTransportTests.Push_UserCancelsMidwayAndBeginsAgainWithAdditionalPush_Resumes: <no details avaliable>

```


> (2)：このカードが手札からの特殊召喚に成功した場合に発動できる。
http://yugioh-wiki.net/index.php?%A1%D4%A3%CC%A3%CC%A1%DD%A5%BF%A1%BC%A5%B3%A5%A4%A5%BA%A1%A6%A5%EF%A1%BC%A5%D6%A5%E9%A1%BC%A1%D5

the text do say if spsummon from hand


## Write a short comment describing this function
## Caching the Inverse of a Matrix:
## Matrix inversion is usually a costly computation and there may be some
## benefit to caching the inverse of a matrix rather than compute it repeatedly.
## Below are a pair of functions that are used to create a special object that
## stores a matrix and caches its inverse.

## This function creates a special "matrix" object that can cache its inverse.


comment


To identify each notifications uniquely,


No worries - thanks for the prompt reply!  ; )


Тогда есть риск получить NPE. strategy.selectElevator может вернуть null, который передастся в dispatchElevator. А там e.elevatorId() от null.


Corrected


![November 21. - Hétfő](http://ujlipot.vakvarju.com/data/napimenu/20.jpg?v=1479507529)
 Zellerkrémleves
Rozmaringos csirkecomb gombás pároltrizzsel
Túrós-barackos rétes


Скобки не обязательны в данном случае, выражение все равно полностью считается.


ok


Пиши внутри else, не на той же строке


checkdeps shows no problems in the xorg area.....
/var/log/Xorg.0.log is not created....

I've uploaded my test build iso, the zipped savefolder, checkdeps output and dmesg output to:
[http://www.smokey01.com/peebee/downloads/test/](http://www.smokey01.com/peebee/downloads/test/)


Лучше разделить на 3 метода для текущей/следующей/предыдущей станций


TeamCity Units.NET :: Build and Test [Build 268](http://teamcity.chump.work/viewLog.html?buildId=912&buildTypeId=UnitsNet_BuildTest) is now running


TeamCity Units.NET :: Build and Test [Build 268](http://teamcity.chump.work/viewLog.html?buildId=912&buildTypeId=UnitsNet_BuildTest) outcome was **SUCCESS**
Summary: Tests passed: 840 Build time: 0:0:42


TeamCity Units.NET :: Build and Test [Build 269](http://teamcity.chump.work/viewLog.html?buildId=913&buildTypeId=UnitsNet_BuildTest) is now running


TeamCity Units.NET :: Build and Test [Build 269](http://teamcity.chump.work/viewLog.html?buildId=913&buildTypeId=UnitsNet_BuildTest) outcome was **SUCCESS**
Summary: Tests passed: 840 Build time: 0:0:39


Все эти методы можно вынести в private, т.к. из вне класса они не вызываются


add mobile redirect


![](https://github.trello.services/images/mini-trello-icon.png) [Adicionado abstracao](https://trello.com/c/FO9jREL1/2-adicionado-abstracao)


J'ai vite fait regardé le jeu d'instructions MIPS, toutes les fonctions proposées me paraissent nécessaires (presque). Je sais pas trop ce qu'on est censé rajouter, bref voilà.


## These are a pair of functions that cache the inverse of a matrix.


Why is the product findings output limited to 10 results and not a long line of all product findings?


Проверка должна быть внутри класса Train


@smithua


Indeed - otherwise it would get messy for other API users too I guess :)


It would be more efficient to probably do something like this:

```cpp
template<typename... TChains>
auto wait_until_complete(TChains&&...chains)
{
    latch l{1};
    auto final_chain = wait_all(FWD(chains)...).then([&]{ l.count_down(); });
    final_chain.start();
    l.wait();
}
```

I think that to do `wait_all(...)` you probably need to use the context here to build a new chain.


Не надо сравнивать с nil и true/false. Это бессмысленно, т.к. это и так логические выражения (nil приравнивается к false). В данном случае подойдет unless res, либо, если важно, подчеркнуть, что сравниваем именно с nil, можно использовать метод `nil?`


LOL


Нужно инийиализировать переменную класса, иначе будет ошибка при добавлении.


Then you can generalize/abstract away the latch stuff with something like `start_synchronized()` *(since the latch will always be initialized to `1`)*:

```cpp
template<typename... TChains>
auto wait_until_complete(TChains&&...chains)
{
    auto final_chain = wait_all(FWD(chains)...).then([&]{ l.count_down(); });
    final_chain.start_synchronized(); // blocks
}
```

```cpp
void continuable::start_synchronized()
{
    latch l{1};
    auto final_chain = this->then([&]{ l.count_down(); });
    final_chain.start();
    l.wait();
}
```


А где @@trains инициализируется, как руби должен догадаться, какой там тип? Там ведь nil по-умолчанию, а у nil нет операции <<


Класс-переменные не могут быть приватными или публичными и их принято инициализировать в начале класса.


Take a look at [my `latch`](https://github.com/SuperV1234/ecst/blob/generalized_instances/include/ecst/utils/cv_operations/latch.hpp), which does not have any `generation` stuff. I don't think we need it since the latch will be used once then destroyed.


Методы класса обычно пишут в начале, после конструктора и до инстанс-методов


@jamesmartin it makes sense =)


This duplicate the first line of resulting dataset.


Missing `'`


This is the new and improved version of the program


Got green light from lars and tested against his use case, seems we're good.


would name this record_state to prevent confusion


Гораздо лучше  будет, если хранить данные в хеше, тогда поиск  - это поиск в хеше по ключу, а это основная операция для хеша и вполняется она очень быстро, в разы быстрее, чем перебор массива и сравнение.


It looks smashing. This is a cleaner solution than my fix, as it isolates sanitation for both offline and online exporting to a separate function.


We don't care too much about the version IMO. What we do care about is that "xwiki" feature is provided only by proper packages exposing their own id.


You can lock this mutex in single place. In the place where all the proc* functions are called.


This ruins eveything


would extract a (possibly inner) function _find_record(records) that implements the inner each in the case it is grouped, and the whole thing if it isn't grouped, to prevent code duplication


Alright !


ok


In Travis, you can run test builds using a configuration matrix. I think, such a feature could help to run these tests more thoroughly. Running with configuration A on one test server and B on another introduces new ways of bugs. Running with configuration A AND B on one server and the same configurations on another server sounds much better


would write the start just below init, by convention, but this is very pedantic


Нет закрывающего end для if,  а вообще, лучше объединить условия через && и использовать if.


fixed in PR https://github.com/wso2/wso2-synapse/pull/675


To make it more readable you should use if cases within { }.


Вообще, в задани ничего не было про то, что надо при создании поезда создавать его с вагонами, ну да ладно.


This comment had been removed from https://github.com/odoo/odoo/blob/10.0/addons/web_kanban/static/src/js/kanban_widgets.js#L106

It's for https://github.com/odoo-dev/odoo/commit/42f155c683555f7a8a748841ab255651f00c1e67#diff-d9fe4c084619a0a6aa0b72cbf30cd9fcR678 I think


同学您好，你的问题我已在问题 #13 中作出了回复，请移步查看！
:haircut_man:


第一次进来，感觉懵懵懂懂。


ok !


Кстати, есть гораздо более красивый способо что-то сделать n раз. Метод times. Например, пишешь `5.times {  |i| ...} ` и блок выполняется 5 раз.


![](https://github.trello.services/images/mini-trello-icon.png) [summary_stats](https://trello.com/c/grDpoM7e/10-summary-stats)


You might want to have a look at my repo, I already implemented full WPF support (including crashes [with crash reporter key], custom events, session telemetry) for the current HockeySDK. If you have any feedback please let me know.
https://github.com/edoust/HockeySDK-Windows


@budius do you have a quick better idea, how to do the threading? I feel a little bad, documenting this new Thread() code ^^
@stanleyguevara maybe?


The gauge widget is not ok. It uses a reference to its parent whereas it shouldn't. Widgets aren't supposed to do RPC by themselves anymore. Instead, they should use trigger_up().


line 198 :    'index_poll_name' => "Site Poll" . or  'index_poll_name' => "Site Poll", ???


Ok you can remove it


@tatani kein problem, alles gepostet!


Need to discuss how this file is used. It looks quite strange to read it every time you process some command. You can store current state in memory. Moreover this state can be stored in for of convenient to use OOP style data structure.


Все можно сделать в разы проще, и, главное, эффективнее, если хранить данные в хеше. Так как хеш заточен на операцию поиска по ключу


Oh nooo! I won't be able to use my dark magic anymore. :'(


nice spelling


Change Table Color to Red


This description is the same as for the function above. I assume this one is correct and the one above isn't.


Тут будет синтаксическая ошибка, т.к. case не закрыт. Ну и стоит все же дописать меню.


This variable mMaxClientNumber still looks strange for me. Do we have some implicit upper bound on the number of clients?


나도 오류뜨는듯


the file does not exists, never used, forgot to delete


Just an idea, do not know if possible, define a const which can be changed at new expansion?


Arc-dark LXQt Panel Theme


What is the reason for removing lombok?
Don't get me wrong..just trying to gather arguments against it :)


Thanks Victor - that's much better. I'm thinking we might need to rewrite `asus_report_contact_down` as I currently have too much happening on each line. Especially for the assignment of `touch_major` which goes over four lines!

I'll revisit tomorrow morning. But feel free to make your own changes in the mean time.


We could do that but that variable should global so we need to add it to Carbonite.lua.


Hey Bro i really like what you doing i can see that you are an Expert, thanks a lots it's encourage me a lot's


It looks more like current number of clients. If yes then don't use it in constructor for vectors capacity initialization. And call it not max but call it current number.


Need to discuss this.


Uploaded files directly and a shitload of changes are showing. Probably bypassing gitignore or something.


@delete, sim. O ES6/ES2015 já comtempla melhor orientação a objetos(definição de classes, etc) e simplifica a estrutura geral do projeto.


Just wondering if MS-MPPE-Encryption-Policy should still be removed by this list? (as it is related to -Encryption-Types)


const AuthStruct&


Better to have single hashmap that maps authStruct.login to all the information about client. Don't need many hashmaps that map user login to scalar values.


1. ты просто строку возвращаешь? не выводишь на экран?
2. Пиши внутри else, а не на той же строке


А если вагонов и так 0? Будет отрицательное число.


Где проверка на тип вагона? По-прежнему можно любой вагон добавить к любому типу поезда


Harsh,i updated the path and fix up lunch error and renamed the repository.so plz pay attention to it :) @dev-harsh1998


@aaronpowell ah, that makes sense. But it sounds like you have a solution. And that the reasons for removing it is clear to you. (cleaner interface)


I'm not against the chaining, (not sure if @mattbrailsford has any opinions on this?) ... of course I'd prefer to cut down on the number of requests.  But I'm cool to have it chained for this feature branch/work.  Just add a TODO note on it to remind ourselves 👍


remove on_change --> field_changed on max_value


Score of pgosf vs pgobase: 194 - 192 - 614  [0.501] 1000 ELO difference: 0.69 +/- 13.37 tc=8+0.08


I felt like `eslintConfig` within `package.json` was making a resurgence because it must have been new but has [been around for a while, apparently](https://github.com/eslint/eslint/commit/5cebba45df31c76cb6a9e64abb54f1a1fbf4edc4).  The order of precedence in loading configs is outlined here:

http://eslint.org/docs/user-guide/configuring#configuration-file-formats

I also used soley `.eslintrc` up until earlier this year, but have moved to `eslintConfig`.  Also, it seems that `.eslintrc` is listed as "deprecated" on that page so I guess that option should not be recommended anywhere. :smile:

For hierarchy succinctness, I would probably root for `eslintConfig` versus the other alternatives (like `.eslintrc.js`), especially since the actual `eslint` configuration is quite simple (thanks to this package).


remove parent --> use record_data


+ no RPC


is this for #90?


Does this change need to be reflected in distributions/pom.xml as you did in [your PR](https://github.com/openhab/openhab-distro/pull/326/files)?


This is on cmake level. This doesn't work on make level:
```
make WARN_MODE=early all
```


> _Code MUST use 4 spaces for indenting, not tabs._
>
> http://www.php-fig.org/psr/psr-2/


Wää


Do you have made any progress about device recognition by the router? Thanks.


cookie


Hi @bernardosrulzon - that has been fixed: https://github.com/snowplow/snowplow/commit/059bdbe12d5a433709ab46207d527480d5e58d34


railsに直依存はちょっと重そうと思いました！


And it works! I need to tweak it a little and will commit soon.


@clintonb can you confirm that running acceptance tests locally does not result in ``WebDriverException`` error or other selenium related issues?


Hey @alexeiva,

Thanks for trying this out!

The script will get better over the course of this week. it is quite unreliable for the time being.


readme很接地气嘛


TestHest


It looks like the base ADAL fork doesn't have this. I'm surprised if they were asking us to add support for it yet their own library doesn't yet. Why is that?


It was more meant like that. Now you have to login to keycdn to see what was done and even to aws console to check the DNS.

I don't know where this kind of informations belong. Speaking of infos it also makes me thing of `keycdn__key` that overwrites: `keycdn: key` in the config file... I don't where that could be written. Maybe it just does not need to be written.

BTW: I removed the specific stuff from the keycdn documentation as asked.


well done


Il faut annuler cette MERDE!!


I think it was ment to be "as of 1.13" ?


select_related voorkomt een tweede hit bij de database


Remove not used methods, code.


Ook hier


> Tested pipeline "gst-launch-1.0 videotestsrc ! video/x-raw,width=800,height=600 ! vaapisink" in https://bugzilla.gnome.org/show_bug.cgi?id=771035 is not fixed with this patch.
> => I looked into it, found out gst_base_src_decide_allocation_default in basesrc/videotestsrc calls set_config with no allocator. ( Can be fixed by moving gst_buffer_pool_config_set_params call to
> somewhere outside of "if (allocator) bracket" or we can set allocator to query in propose_allocation by gst_query_add_allocation_param)

Purrfect... can you cook a patch removing the size modification in the plugin base and add the allocator in the propose_allocation() query????

> If it's through
> https://github.com/ceyusa/gstreamer-vaapi/blob/master/gst/vaapi/gstvaapipluginbase.c#L582,
> same issue happens. It seems to change allocator's videoinfo(Quark) but not change allocator->image_info, which leads to mismatch too. => Maybe this is different issue..

Yeah, that's an awful hack for decoders only. We need to rethink that.


Test voor 1 van de char filters?


This is displayed at the bottom. Needs to be removed?
![screen shot 2016-11-21 at 10 56 30](https://cloud.githubusercontent.com/assets/16060989/20480135/34acfba2-afd9-11e6-909e-4e207ef0b066.png)


string imported maar niet gebruikt?


We can remove Signup/Signin/Sign out and the welcome User as these are handled elsewhere. But we need to check if a user is logged in when typing the url messages/erehwon.


Waarom niet helemaal weg halen?


content =  Erehwon team


`generateConcentric3DInfill` would call it in the same way (only the `first_wall` parameter needs to be last since it has a default argument). Since `first_wall` is not `nullptr` then, it executes the code inside lines 59 through 67.

I realise that we delayed this such that you probably don't _want_ to fix this up, but this pattern does make the code more readable: Just fill in `first_wall` if it's missing, instead of having two different functions for it.


APC support => Emulated
combined with
apc.enabled => Off (local value)
seems quite odd. Could that be the cause?


Thanks, yeah, I'm just trying to do something useful :) I feel myself half-blind, because Linux code and documentation don't quite agree with each other or at times too silent to figure out the proper way of doing things. But with the help of kernel maintainers we have a chance to do it the right way in the end!


It was a missing dep (not found by checkdeps) - libXfont2 has been added to Slackware Current


Didn't you see the notice in VMGS/design.md? We won't make support for macOS unless *everyone* in the development team has access to macOS.
— Just a joke. Ha ha. It would always be a good thing to implement support for yet another platform... maybe.

The Unix building way rarely cooperates with Windows. If I didn't make a make.rb, Windows users will never get a chance to play this uninteresting game. For building consistency between Windoge and Dogix, we have to use make.rb. Probably cmake is a good solution, but it's too hard for such a weird project and me.

I was surprised when I found that your Makefiles link VMDE and VMGS dynamically. I supposed that they were statically linked everywhere. Nonetheless, it should be quite easy to make make.rb link them dynamically since there's already a super simple project configuration system.

SOIL is linked in through the archive SOIL.a on my system and there's no problem.

HOWEVER, editing make.rb to adopt macOS is not so easy. There's currently a variable called ‘@@windows’, so we need to change that into ‘@@platform’ that holds a symbol: any of `:windows`, `:linux` or `:macos`. There will be a lot of work.

I'd be very happy to switch our building system to clang++, because gcc on Windoge is as fast as doge.

And at last, I wonder if your IME on macOS works. I've hardly seen you write such a long reply completely in English.

*comment not revised*


I'm close to having a working patch. It works for start/stop/status but fails with ping. Problem is, I'm not sure what the syntax should be. Here a trace with the error message at the end, if one of you could help me:
```
+ exec_cmd /usr/bin/erl -sname ping-console-ejabberd@localhost -hidden +K true -smp auto +P 250000 -eval io:format("~p~n",[net_adm:ping("localhost")]) -s erlang halt -output text -noinput
+ [ as_install_user = as_install_user ]
+ su -c "$0" "$@" ejabberd -- /usr/bin/erl -sname ping-console-ejabberd@localhost -hidden +K true -smp auto +P 250000 -eval io:format("~p~n",[net_adm:ping("localhost")]) -s erlang halt -output text -noinput
{"init terminating in do_boot",{function_clause,[{net_adm,ping,["localhost"],[{file,"net_adm.erl"},{line,61}]},{erl_eval,do_apply,6,[{file,"erl_eval.erl"},{line,670}]},{erl_eval,expr,5,[{file,"erl_eval.erl"},{line,228}]},{erl_eval,expr_list,6,[{file,"erl_eval.erl"},{line,878}]},{erl_eval,expr,5,[{file,"erl_eval.erl"},{line,404}]},{init,start_it,1,[]},{init,start_em,1,[]}]}}

Crash dump is being written to: /var/log/ejabberd/erl_crash_19700103-205023.dump...done
init terminating in do_boot ()
```


@akosyakov, please have a look at the changes to MonacoModel.


logout probably not needed


interface das aulas


Is that correct on a frame? Is it because when front-end comes it will be displayed in a small drop down?


That's bad idea

On Sat, Nov 19, 2016 at 9:14 PM, Jent Zheng <notifications@github.com>
wrote:

> you can just simply add WORKDIR /app to dockerfile rather than adding -w
> flag every time in docker run commands :-)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/TeamWiizmi/docker-node-image/commit/4052d8d625bcda1bbbeeff017869cb49db77c704#commitcomment-19892094>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABTO-jfXkpFLj16u5pW8mVoShPowV3trks5q_1ijgaJpZM4K3WqE>
> .
>

--
Sincerely,
Buck


no need for `finally`?


Resource bars already use "powerplants" in the tooltip of +energy label.


Die "Idee" dafür kam aus dem Feedback. "Warum fehlt hier der Selektor?"
Lassen wir ihn lieber drin, dann ist keiner verwirrt


use static const for the private variables so that they only exist once in the whole class and not in every instance of it.


what's wrong with you?


good point, hmm, maybe it should rather not be try-finally in the other ones, if the stage throws that might make tests pass unexpectedly when the stage failed


You may want to remove this constructor altogether. Because if you rename your contract and forget to rename the constructor, you just end up with a useless orphan function.


@MalteWunsch Does that (for example) work for version 3.2?

major >= 2 => yes
minor >= 7 => no


`this.balance` also works.


Seems fine to me, this is just how promises work, short of creating a web service to handle both at the same time, but no point in that.


I noticed that when I have a new message and I click the Notifications link it doesn't display the message:
![screen shot 2016-11-21 at 10 59 37](https://cloud.githubusercontent.com/assets/16060989/20480546/e017b2b0-afda-11e6-9087-f2c6d0b1f882.png)
and click Notifications link above Inbox. It goes to messages/notifications and displays: hello Received Messages Received


This isn't quite right, Crispy Doom has left the fields of "limit raising" for ages. It is now limit-removing (i.e. no static arraas but raising the limits dynamically) for all but a handful of limits, so please don't state here that you raise the limits to the level of Crispy Doom and then not doing so. Maybe you mixed this up with Doom-Plus?


Now that you know there may be 1 wei that remains in the contract, how do you retrieve it? Remember, avoid trapping, aka burning, Ethers.


To save gas, you may want to calculate `msg.value/2` only once.


after updating to this release, and latest gstreamer-imx, cpu usage has increased by a lot especially on crowded scenes. I am able to observe changes in CPU usage when i cover the lens or open it up(up to %130-140 on single core). I used to convert colors before this patch in order to get the encoder working (without green or pink artifacts on grayscale image) but even with the converter CPU usage was not this high(about %40 50 on single core).

I put a log together with 2 seconds of stream on rtsp
[gstreamer-imx-log.txt](https://github.com/Freescale/libimxvpuapi/files/603357/gstreamer-imx-log.txt)


I'm doing the same thing you do! We work in parallel. I however moved over to CompressedImage, but am struggling with decompression of the one channel 16 bit png images. Have you managed to do this yet?


Thanks!


Земля не однородна. Её безразмерный момент инерции с хорошей точностью равен 1/3.


Let's better prefix the member with a `_`, like a private member. It is not suppose to be accessed directly as value anyway, or is it?


Yeah - I've found the documentation can be missing a lot of detail too. So I've taken to reading the source. But there is a lot to read! And there's nothing like documentation to detail best practices.


@ebaydayz do you think we could add traders to this system? Would that then help with the cursor target issue in trader zones?


What is the reason behind this change? I do not understand the comment.


Like @jmesserli already mentioned elsewhere you can use `listOf()` instead of `Arrays.asList()`


@modus75 can you review branch feature/optimize-history which includes #370 and #509 before I merge on master please ? :-)


What is retriever-regexp? Is it retreve the span body?


This check is good. Maybe it would make sense to also print a warning - because this condition is the result of a programming error.


@khanfrd ofcourse I will make one. 4.9 is essential for aarch64 kernel development. I started with 6.x since I have been using Uber and linaro 6 tc's for quite sometime and they were stable. But 4.9.x is needed if I ever run into problem with 6.x. So in a day or two I will upload 4.9 tc.
Also thanks for showing interest in my toolchain. 😄


Apart from the few inline comments, the patch looks very good to me!


Unnecessary, because the validity of the items category is checked in the beginning of the method.


Hi Mathias!

thanks for the comment, I just discovered your repositories. I think it would be great to integrate improvements from both of us into a single repository. I'm new to unity and to C#, I come from the pure ROS and C++ world.

Regarding the decompression, I found the BinaryReader, which I haven't tested yet. It has the method ReadUInt16() that should suit you.


sdsdas


Excellent thankyou 👍

When you save the PNG files please can you make sure you **don't** use interlacing because it means I have to open them and re-save. Also, try and remember to save the images without colour management, otherwise `libpng` raises sRGB warnings


What was the hero position issue? I thought it's the size that was the issue?


OK, that is a good idea.


Hi

another line which can use the same fix is at line 629:

  echo("Compiling ", bin, " (", pkgInfo.name, ") using ", backend,
       " backend...")
  doCmd("\"" & getNimBin() & " $# --noBabelPath $# \"$#\"" %
        [backend, args, bin])


  This bug will abort 00_rootfs_build.sh. Now I remove this part and check the reason why,
and I only add audio service by hand on running system. It's not good way to add audio service.
I'll get a good way to solve this issue.

**Good Luck**


No, the Scala support has been available since 1.12.


Hi.
Please tell me how to start "Mac Multi-Tool", where to write"enter"
I would like to install a new drive Sierra10.12.1
I tried already with Pandora's box, but it is not.Hi everyone.
Recently I've got some time to put aside some time to configure my hackintosh. I'm trying to install Sierra 10.12.1 on my system.
I've got the below configuration (hardware):

 Intel Core i7-4770K  3.50GHz,  LGA1150,  3.5 GHz, 4cores (2 logical cores per physical)
ASRock Z87 Extreme9/ac, Realtek ALC1150 Audio Codec), Supports DTS Connect
GPU: Intel HD 4600 (I'm connecting 27"Monitor to PC via HDMI cable)
Mem: 4x8GB  Corsair Vengance CL9
SSD disc Samsung 250GB
Any help would be much appreciated!
Thankyou


The slower clock speed limits the possible sampling rates, I use 11025 which works just fine


取消删除


faltou vincular a entrada de manutenção com o pneu que estava com problema.
deve colocar o registro do pneu na tabela part_entry, vinculada a entrada recém criada


@samaaron I was just browsing the code and was confused by this, should it be `res.merge(h)`?


If the issue is small vs wide screens, we can try to differentiate the behavior. If I'm the only one bothered by closed current form twisties, we can revert the commit. Both ways are fine for me.

> Should we just assume the user knows what they're doing?

No :-) It's not always possible to respect users choice *à la lettre*. Say, the last open pane on "Preferences" tab was "Publish". Now you click some plugins options, that have their own pane in "Preferences". Should we still land on "Publish" pane or switch to the plugin's one? Same here: clicking an `adi_form_links` can be seen as "bring me there and open the pane". Or not, there is a choice to do.


```js
render(){
    const { message: { id,type,text }, deleteFlashMessage } = this.props;

      return(
        <div className={classnames('alert', {
        'alert-success': type === 'succes',
        'alert-danger': type === 'error'
})}>
      <button onClick={() => deleteFlashMessage(id)} className="close"><span>×</span></button>
         {text}
        </div>
      );
    }
```
and then you can remove the onClick() method above


I made that comment and also check out the version command's description too!


Good catch. I was under the impression it's a capital S, as the web page so prominently features that S part of the logo. Fix in 34002d8.


não vejo necessidade de tu buscar os tipos de alertas existentes, tu pode definir os tipos no fleetany-alerts e colocar aqui diretamente no código, quando tu enviar um alerta, já manda o tipo baseado no problema que o fleetany-api identificar


See above.


also fix indentation


This should follow the same pattern.

`get /trip/:id`, PageController, :view_trip`


Just use `:id` instead of `:tripid`. It's the convention. You can always pattern match to a different variable name in your controller.


considering it doesnt even connect its just decoration as of now as i have no use for it with the Creative Energy Cell and your Cake Converter available, wasnt even planning on doing conduits this early into the mod, and dont have to anymore until i finish up the main part


hmm.. strange I don't understand why you don't need an ASSERT_THROW here. As far as I know the syntaxError throws an InternalError.


I like this logic. However I recommend that we put it in Tokenizer::findGarbageCode() instead so it's detected earlier.


You may want to explore using `bytes32` instead of `string`. Smaller, but hey, this is expensive blockchain.
`web3.toUtf8()` will be your friend.


If `symbol` shall be unique, it might as well be a `bytes32` and act as the id.


Did all the changes and checked it 3 times, but it't didn't work.
I got no usernames in the scanned locations.


Agree that is better and more pythonic


Okay.


Aqui tá rolando problema no escopo { fechar e abrir aspas


I expect you to have a very, _very_ good reason to use `tx.origin` instead of `msg.sender`. Can you tell me?


O resto tá de acordo
Select e insert


And user is logged in


hello


It's better to put `.gitignore` the root of the repository.


"Regexp that is used to retrive the modes symbol from the head of the chunkmode chunk."

This regexp is renamed into `mode-matcher` in the future version of polymode.


> Any advice how to test for Errata 843419 changes? Kernel boots at least :)

Works fine enough for me too, but looks like gt9xxf_ts driver crashes on shutdown, so it should not be enabled by default.


Жалко что в Android нет стримов : (


Create **suggestions-scraper.reportobserver** directory for **in** and **out** directories


thanks, I will fix that later


That looks like it is only useful for debugging or testing. `return false` should be enough, have a go at refactoring this piece.


i*1000

и как, работает с разной задержкой? было бы круто, думаю, если б заменить i на index


Исправил (добавил правило в Access list для FOSUserBundle и аннотацию для контроллера)


This magic `10000` should be a parameter.