[inst/extdata/asalmon_mcw/main.r:2:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L2) *style:* **Variable or function name should be snake_case.**
```r
LAMBDA_var <- 0.1 #Thin plate spline smoothing. Lambda=0 would be interpolation.
^~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:3:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L3) *style:* **Variable or function name should be snake_case.**
```r
RESOLUTION_var <- 500 #Plot width in pixels
^~~~~~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:5:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L5) *style:* **Lines should not be more than 80 characters.**
```r
#TODO change path. make sure this main.r script is just outside the diagram_retina folder.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:8:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L8) *style:* **Lines should not be more than 80 characters.**
```r
#todo verify these. see the tutorial here: https://github.com/bc/retina/blob/master/tutorial.md search for the part that starts with "#[V point of the tear,"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:10:10:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L10) *style:* **Commas should always have a space after.**
```r
    c(59,52,67),
         ^
```
[inst/extdata/asalmon_mcw/main.r:10:13:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L10) *style:* **Commas should always have a space after.**
```r
    c(59,52,67),
            ^
```
[inst/extdata/asalmon_mcw/main.r:11:10:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L11) *style:* **Commas should always have a space after.**
```r
    c(79,73,85),
         ^
```
[inst/extdata/asalmon_mcw/main.r:11:13:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L11) *style:* **Commas should always have a space after.**
```r
    c(79,73,85),
            ^
```
[inst/extdata/asalmon_mcw/main.r:12:10:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L12) *style:* **Commas should always have a space after.**
```r
    c(34,31,37))
         ^
```
[inst/extdata/asalmon_mcw/main.r:12:13:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L12) *style:* **Commas should always have a space after.**
```r
    c(34,31,37))
            ^
```
[inst/extdata/asalmon_mcw/main.r:15:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L15) *style:* **Lines should not be more than 80 characters.**
```r
assemble_markup_file('left', path_to_retina_data_folder, nasal_outline_index=1, dorsal_outline_index=NA)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:15:22:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L15) *style:* **Only use double-quotes.**
```r
assemble_markup_file('left', path_to_retina_data_folder, nasal_outline_index=1, dorsal_outline_index=NA)
                     ^~~~~~
```
[inst/extdata/asalmon_mcw/main.r:15:77:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L15) *style:* **Put spaces around all infix operators.**
```r
assemble_markup_file('left', path_to_retina_data_folder, nasal_outline_index=1, dorsal_outline_index=NA)
                                                                           ~^~
```
[inst/extdata/asalmon_mcw/main.r:15:101:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L15) *style:* **Put spaces around all infix operators.**
```r
assemble_markup_file('left', path_to_retina_data_folder, nasal_outline_index=1, dorsal_outline_index=NA)
                                                                                ~^~
```
[inst/extdata/asalmon_mcw/main.r:29:21:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L29) *style:* **Commas should never have a space before.**
```r
        height = 45 ,# height of the counting frame in microns
                   ~^
```
[inst/extdata/asalmon_mcw/main.r:29:22:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L29) *style:* **Commas should always have a space after.**
```r
        height = 45 ,# height of the counting frame in microns
                     ^
```
[inst/extdata/asalmon_mcw/main.r:34:28:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L34) *style:* **Commas should never have a space before.**
```r
        extrapolate = TRUE ,          #Predicts densities to the equator.
                          ~^
```
[inst/extdata/asalmon_mcw/main.r:36:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L36) *style:* **Lines should not be more than 80 characters.**
```r
        rotation_ccw = -90, # when set to -90 degrees, the rotation is unaltered from the measured orientation.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/extdata/asalmon_mcw/main.r:37:22:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L37) *style:* **Put spaces around all infix operators.**
```r
        plot_suppress=TRUE)
                    ~^~
```
[inst/extdata/asalmon_mcw/main.r:39:29:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L39) *style:* **Put spaces around all infix operators.**
```r
retinaplot(my_retina, lambda=0.001, polynomial_m=   2, rotation=0)
                           ~^~
```
[inst/extdata/asalmon_mcw/main.r:39:49:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L39) *style:* **Put spaces around all infix operators.**
```r
retinaplot(my_retina, lambda=0.001, polynomial_m=   2, rotation=0)
                                               ~^
```
[inst/extdata/asalmon_mcw/main.r:39:64:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L39) *style:* **Put spaces around all infix operators.**
```r
retinaplot(my_retina, lambda=0.001, polynomial_m=   2, rotation=0)
                                                              ~^~
```
[inst/extdata/asalmon_mcw/main.r:40:10:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L40) *style:* **Put spaces around all infix operators.**
```r
par(mfrow=c(2,2))
        ~^~
```
[inst/extdata/asalmon_mcw/main.r:40:15:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/inst/extdata/asalmon_mcw/main.r#L40) *style:* **Commas should always have a space after.**
```r
par(mfrow=c(2,2))
              ^
```
[R/composite.R:2:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L2) *style:* **Lines should not be more than 80 characters.**
```r
##' @description This is a subfunction, that is useful when creating average maps.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:3:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
##' @param retina_list A list containing retina objects, with names(retina_list) all defined.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:4:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L4) *style:* **Lines should not be more than 80 characters.**
```r
##' @param rotation_indices Each value in this vector will correspond to the amount of rotation applied to each retina in the list.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:5:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L5) *style:* **Lines should not be more than 80 characters.**
```r
##' @param ... Arguments passed from other functions. You can access the thin plate spline interpolator here.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:6:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L6) *style:* **Lines should not be more than 80 characters.**
```r
##' @return sum_mat The sum of all retina objects in the list, with respect to the rotation indices.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:9:16:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L9) *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:12:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L12) *style:* **Lines should not be more than 80 characters.**
```r
        sum_mat <- sum_mat + mat_from_ret_obj(retina_list[[i]], rotation_indices[i],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/composite.R:12:85:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/composite.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
        sum_mat <- sum_mat + mat_from_ret_obj(retina_list[[i]], rotation_indices[i],
                                                                                ^
```
[R/matrix_retobj.R:3:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
##' You can decide how much to rotate the retina before returning the matrix. You can also pass arguments to the polar plotter.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:6:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L6) *style:* **Lines should not be more than 80 characters.**
```r
##' @param n the number by which to divide the matrix by. Useful when generating partial averages.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:7:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
##' @param reflect logical, by default it is TRUE, in order to view the retina from outside the eye (rather than from within.)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:12:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L12) *style:* **Lines should not be more than 80 characters.**
```r
mat_from_ret_obj <- function(retina_object, rotation = 0, n = 1, reflect = TRUE,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:12:81:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
mat_from_ret_obj <- function(retina_object, rotation = 0, n = 1, reflect = TRUE,
                                                                                ^
```
[R/matrix_retobj.R:17:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L17) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/matrix_retobj.R:20:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L20) *style:* **Lines should not be more than 80 characters.**
```r
    az <- mapproject(x = retina_object$trimmed_data[, 2], retina_object$trimmed_data[,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:20:87:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L20) *style:* **Trailing whitespace is superfluous.**
```r
    az <- mapproject(x = retina_object$trimmed_data[, 2], retina_object$trimmed_data[,
                                                                                ^
```
[R/matrix_retobj.R:25:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L25) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/matrix_retobj.R:27:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L27) *style:* **Lines should not be more than 80 characters.**
```r
    map2fit <- fit_plot_azimuthal(x2, y2, z2, plot_suppress = TRUE, extrapolate = TRUE,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:27:88:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L27) *style:* **Trailing whitespace is superfluous.**
```r
    map2fit <- fit_plot_azimuthal(x2, y2, z2, plot_suppress = TRUE, extrapolate = TRUE,
                                                                                ^
```
[R/matrix_retobj.R:28:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L28) *style:* **Lines should not be more than 80 characters.**
```r
        outer_radius = pi/2, falciform_coords = retina_object$azimuthal_data.falciform[[1]],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/matrix_retobj.R:28:26:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L28) *style:* **Put spaces around all infix operators.**
```r
        outer_radius = pi/2, falciform_coords = retina_object$azimuthal_data.falciform[[1]],
                        ~^~
```
[R/matrix_retobj.R:28:93:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L28) *style:* **Trailing whitespace is superfluous.**
```r
        outer_radius = pi/2, falciform_coords = retina_object$azimuthal_data.falciform[[1]],
                                                                                ^
```
[R/matrix_retobj.R:30:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L30) *style:* **Variable or function name should be snake_case.**
```r
    MAT <- map2fit[[2]]$z
    ^~~
```
[R/matrix_retobj.R:32:15:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/matrix_retobj.R#L32) *style:* **Put spaces around all infix operators.**
```r
    return(MAT/n)
             ~^~
```
[R/plotting_functions.R:3:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
##' Extracts xy and falciform coordinates from datapoints collected from retistruct. Falc and Datapoints are stacked upon one another
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:5:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L5) *style:* **Lines should not be more than 80 characters.**
```r
##' @param datapoints_from_retistruct data structure that comes from the datapoints.csv, which is automatically generated by Retistruct.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:7:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
##' @return L list of pixel xy positions, and falciform coordinates in [1] and [2], respectively
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:9:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L9) *style:* **Lines should not be more than 80 characters.**
```r
extract_xy_and_falciform <- function(datapoints_from_retistruct, number_of_sampling_sites) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:10:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L10) *style:* **Lines should not be more than 80 characters.**
```r
    pixel_xy_positions <- datapoints_from_retistruct[1:number_of_sampling_sites,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:10:81:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L10) *style:* **Trailing whitespace is superfluous.**
```r
    pixel_xy_positions <- datapoints_from_retistruct[1:number_of_sampling_sites,
                                                                                ^
```
[R/plotting_functions.R:12:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L12) *style:* **Lines should not be more than 80 characters.**
```r
    falciform_len <- length(datapoints_from_retistruct[, 1]) - length(pixel_xy_positions[,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:12:91:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
    falciform_len <- length(datapoints_from_retistruct[, 1]) - length(pixel_xy_positions[,
                                                                                ^
```
[R/plotting_functions.R:16:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L16) *style:* **Lines should not be more than 80 characters.**
```r
    falciform_coordinates <- datapoints_from_retistruct[(xy_len + 1):total_len, ]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:22:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L22) *style:* **Lines should not be more than 80 characters.**
```r
##' ImageJ values don't always show up the right way, because Y values get larger and positive while moving down the image.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:24:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L24) *style:* **Lines should not be more than 80 characters.**
```r
##' @param falciform_xy_coordinates 2 columns of coordinate (X,Y) data in ImageJ pixel format
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:28:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L28) *style:* **Lines should not be more than 80 characters.**
```r
reflect_IJ_y_values_for_falciform <- function(falciform_xy_coordinates, xy_img_dimensions) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:28:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L28) *style:* **Variable and function names should not be longer than 30 characters.**
```r
reflect_IJ_y_values_for_falciform <- function(falciform_xy_coordinates, xy_img_dimensions) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:28:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L28) *style:* **Variable or function name should be snake_case.**
```r
reflect_IJ_y_values_for_falciform <- function(falciform_xy_coordinates, xy_img_dimensions) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:29:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L29) *style:* **Lines should not be more than 80 characters.**
```r
    falciform_xy_coordinates_transformed <- falciform_xy_coordinates  #make a copy
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:29:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L29) *style:* **Variable and function names should not be longer than 30 characters.**
```r
    falciform_xy_coordinates_transformed <- falciform_xy_coordinates  #make a copy
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:31:52:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L31) *style:* **Put spaces around all infix operators.**
```r
    half_of_vertical_pixels <- xy_img_dimensions[2]/2
                                                  ~^~
```
[R/plotting_functions.R:32:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L32) *style:* **Lines should not be more than 80 characters.**
```r
    falciform_xy_coordinates_transformed$cyan <- (falc_y - half_of_vertical_pixels) *
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:32:86:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L32) *style:* **Trailing whitespace is superfluous.**
```r
    falciform_xy_coordinates_transformed$cyan <- (falc_y - half_of_vertical_pixels) *
                                                                                ^
```
[R/plotting_functions.R:38:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L38) *style:* **Lines should not be more than 80 characters.**
```r
##' Shows a visualization of the outline, as well as the location of each datapoint, with fit plot.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:40:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L40) *style:* **Lines should not be more than 80 characters.**
```r
##' @param pixel_xy_positions 2 columns of coordinate (X,Y) data in ImageJ pixel format
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:41:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L41) *style:* **Lines should not be more than 80 characters.**
```r
##' @param z_vector numerical vector of density values for each of the coordinates
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:42:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L42) *style:* **Lines should not be more than 80 characters.**
```r
##' @param xy_outline_points 2 columns of coordinate(X,Y) data in ImageJ pixel format for the outline coordinates.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:45:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L45) *style:* **Lines should not be more than 80 characters.**
```r
traditional_flatmount_plot <- function(pixel_xy_positions, z_vector, xy_outline_points) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:46:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L46) *style:* **Lines should not be more than 80 characters.**
```r
    fit <- Tps(pixel_xy_positions, z_vector)  # fits a surface to ozone measurements.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:46:12:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L46) *warning:* **no visible global function definition for ‘Tps’, Did you mean 'Ops'?**
```r
    fit <- Tps(pixel_xy_positions, z_vector)  # fits a surface to ozone measurements.
           ^~~
```
[R/plotting_functions.R:47:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L47) *warning:* **no visible global function definition for ‘set.panel’**
```r
    set.panel(2, 2)
    ^~~~~~~~~
```
[R/plotting_functions.R:49:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L49) *warning:* **no visible global function definition for ‘set.panel’**
```r
    set.panel()
    ^~~~~~~~~
```
[R/plotting_functions.R:52:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L52) *warning:* **no visible global function definition for ‘surface’**
```r
    surface(fit)  # Quick image/contour plot of GCV surface.
    ^~~~~~~
```
[R/plotting_functions.R:61:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L61) *style:* **Lines should not be more than 80 characters.**
```r
##' Shows a visualization of the outline, as well as the location of each datapoint so the user can identify where the tears are.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:64:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L64) *style:* **Lines should not be more than 80 characters.**
```r
##' @param path_to_retina_data_folder The path to the folder that contains the outline.roi file.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:65:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L65) *style:* **Lines should not be more than 80 characters.**
```r
##' @return roi_coordinates XY vals of pixel coordinates of the outline points, in order.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:69:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L69) *style:* **Lines should not be more than 80 characters.**
```r
    roi_object <- read.ijroi(file.path(path_to_retina_data_folder, "outline.roi"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:69:19:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L69) *warning:* **no visible global function definition for ‘read.ijroi’**
```r
    roi_object <- read.ijroi(file.path(path_to_retina_data_folder, "outline.roi"))
                  ^~~~~~~~~~
```
[R/plotting_functions.R:70:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L70) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/plotting_functions.R:72:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L72) *style:* **Lines should not be more than 80 characters.**
```r
    # format plots has the coordinate (0, 0) in the top left.  We have the coordinate
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:73:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L73) *style:* **Lines should not be more than 80 characters.**
```r
    # (0, 0) in the bottom left. We need to transform P so that the outline appears
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:76:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L76) *style:* **Variable or function name should be snake_case.**
```r
    roi_XY_coords <- roi_object$coords
    ^~~~~~~~~~~~~
```
[R/plotting_functions.R:80:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L80) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/plotting_functions.R:81:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L81) *style:* **Lines should not be more than 80 characters.**
```r
    plot(roi_XY_coords, type = "l", col = "#d3d3d3", asp = 1, xlab = "X Pixels",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:81:81:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L81) *style:* **Trailing whitespace is superfluous.**
```r
    plot(roi_XY_coords, type = "l", col = "#d3d3d3", asp = 1, xlab = "X Pixels",
                                                                                ^
```
[R/plotting_functions.R:83:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L83) *style:* **Lines should not be more than 80 characters.**
```r
    text(roi_XY_coords[, 1], roi_XY_coords[, 2], labels = 1:length(roi_XY_coords[,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:83:83:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L83) *style:* **Trailing whitespace is superfluous.**
```r
    text(roi_XY_coords[, 1], roi_XY_coords[, 2], labels = 1:length(roi_XY_coords[,
                                                                                ^
```
[R/plotting_functions.R:85:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L85) *style:* **Lines should not be more than 80 characters.**
```r
    message("If you are getting this message, you are using a special feature. Contact me at brian.cohn@usc.edu and I will happily teach you how to use it.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:93:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L93) *style:* **Lines should not be more than 80 characters.**
```r
##' @description Creates a markup.csv file. One of the dorsal_outline_index or nasal_outline_index must be filled in. Not both.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:96:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L96) *style:* **Lines should not be more than 80 characters.**
```r
##' @param dorsal_outline_index The index of the point that points toward dorsal, or NA.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:97:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L97) *style:* **Lines should not be more than 80 characters.**
```r
##' @param nasal_outline_index The index of the opint that points toward nasal, or NA.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:98:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L98) *style:* **Lines should not be more than 80 characters.**
```r
##' @param path_to_retina_data_folder The path where the markup.csv file will be saved.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:100:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L100) *style:* **Lines should not be more than 80 characters.**
```r
assemble_markup_file <- function(eye_left_or_right, path_to_retina_data_folder, dorsal_outline_index = NA,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:100:107:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L100) *style:* **Trailing whitespace is superfluous.**
```r
assemble_markup_file <- function(eye_left_or_right, path_to_retina_data_folder, dorsal_outline_index = NA,
                                                                                ^
```
[R/plotting_functions.R:103:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L103) *style:* **Lines should not be more than 80 characters.**
```r
        ## https://stackoverflow.com/questions/18509527/first-letter-to-upper-case
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:109:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L109) *style:* **Lines should not be more than 80 characters.**
```r
        line2 <- paste0("NA,", nasal_outline_index, ",",phi0,",NA,TRUE,\"", eye_side_string,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:109:57:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L109) *style:* **Commas should always have a space after.**
```r
        line2 <- paste0("NA,", nasal_outline_index, ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                        ^
```
[R/plotting_functions.R:109:62:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L109) *style:* **Commas should always have a space after.**
```r
        line2 <- paste0("NA,", nasal_outline_index, ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                             ^
```
[R/plotting_functions.R:109:93:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L109) *style:* **Trailing whitespace is superfluous.**
```r
        line2 <- paste0("NA,", nasal_outline_index, ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                                                ^
```
[R/plotting_functions.R:112:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L112) *style:* **Lines should not be more than 80 characters.**
```r
        line2 <- paste0(dorsal_outline_index, ",NA", ",",phi0,",NA,TRUE,\"", eye_side_string,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:112:58:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L112) *style:* **Commas should always have a space after.**
```r
        line2 <- paste0(dorsal_outline_index, ",NA", ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                         ^
```
[R/plotting_functions.R:112:63:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L112) *style:* **Commas should always have a space after.**
```r
        line2 <- paste0(dorsal_outline_index, ",NA", ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                              ^
```
[R/plotting_functions.R:112:94:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L112) *style:* **Trailing whitespace is superfluous.**
```r
        line2 <- paste0(dorsal_outline_index, ",NA", ",",phi0,",NA,TRUE,\"", eye_side_string,
                                                                                ^
```
[R/plotting_functions.R:120:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L120) *style:* **Lines should not be more than 80 characters.**
```r
    message(paste("Wrote markup file to ", output_path, "\n Ensure that it exists beside your outline.roi file"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:130:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L130) *style:* **Lines should not be more than 80 characters.**
```r
##' @param tear_coordinates_dataframe the dataframe of 3 columns, with c('V0','VB','VF'), and n columns, where n= number of tears
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:131:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L131) *style:* **Lines should not be more than 80 characters.**
```r
##' @param path_to_retina_data_folder The path where the T.csv file will be saved.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:135:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L135) *style:* **Lines should not be more than 80 characters.**
```r
assemble_tear_file <- function(tear_coordinates_dataframe, path_to_retina_data_folder) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:138:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L138) *style:* **Lines should not be more than 80 characters.**
```r
    write.table(tear_coordinates_dataframe, output_path, sep = ",", row.names = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:139:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L139) *style:* **Lines should not be more than 80 characters.**
```r
    message(paste("Wrote tear file to ", output_path, "\n Check that it exists beside your outline.roi file"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:147:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L147) *style:* **Lines should not be more than 80 characters.**
```r
##' @param path_to_retina_data_folder The path where the P.csv file will be saved.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:151:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L151) *style:* **Lines should not be more than 80 characters.**
```r
assemble_point_coordinates_file <- function(outline_coordinates, path_to_retina_data_folder) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:151:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L151) *style:* **Variable and function names should not be longer than 30 characters.**
```r
assemble_point_coordinates_file <- function(outline_coordinates, path_to_retina_data_folder) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:155:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L155) *style:* **Lines should not be more than 80 characters.**
```r
    message(paste("Wrote coordinates file to ", output_path, "\n Check that it exists beside your outline.roi file"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:177:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L177) *style:* **Variable and function names should not be longer than 30 characters.**
```r
plot_degree_label_for_latitudes <- function(outer_radius, circle.rads) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:177:59:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L177) *style:* **Variable or function name should be snake_case.**
```r
plot_degree_label_for_latitudes <- function(outer_radius, circle.rads) {
                                                          ^~~~~~~~~~~
```
[R/plotting_functions.R:178:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L178) *style:* **Lines should not be more than 80 characters.**
```r
    axis(2, pos = -1.25 * outer_radius, at = sort(union(circle.rads, -circle.rads)),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:178:85:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L178) *style:* **Trailing whitespace is superfluous.**
```r
    axis(2, pos = -1.25 * outer_radius, at = sort(union(circle.rads, -circle.rads)),
                                                                                ^
```
[R/plotting_functions.R:180:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L180) *style:* **Lines should not be more than 80 characters.**
```r
    text(-1.26 * outer_radius, sort(union(circle.rads, -circle.rads)), sort(union(circle.rads,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:180:95:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L180) *style:* **Trailing whitespace is superfluous.**
```r
    text(-1.26 * outer_radius, sort(union(circle.rads, -circle.rads)), sort(union(circle.rads,
                                                                                ^
```
[R/plotting_functions.R:195:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L195) *style:* **Lines should not be more than 80 characters.**
```r
    fields::image.plot(legend.only = TRUE, col = col, zlim = zlim, family = "Palatino")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:202:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L202) *style:* **Lines should not be more than 80 characters.**
```r
##' @param endpoints a 4 element numeric vector describing the xy and x'y' for the line segment.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:206:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L206) *style:* **Lines should not be more than 80 characters.**
```r
    segments(endpoints[1], endpoints[2], endpoints[3], endpoints[4], col = color_hex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:216:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L216) *style:* **Lines should not be more than 80 characters.**
```r
##' @param endpoints vector of 4 numerics, x,y and x',y' defining the line segment
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:217:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L217) *style:* **Lines should not be more than 80 characters.**
```r
write_labels_at_endpoint_locations <- function(r_label, l_label, degree, endpoints) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:217:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L217) *style:* **Variable and function names should not be longer than 30 characters.**
```r
write_labels_at_endpoint_locations <- function(r_label, l_label, degree, endpoints) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:226:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L226) *style:* **Lines should not be more than 80 characters.**
```r
##' @description We do not need points plotted in the corners of the plotted circle.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:233:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L233) *style:* **Lines should not be more than 80 characters.**
```r
nullify_vals_outside_the_circle <- function(minitics, spatial_res, heatmap_matrix,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:233:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L233) *style:* **Variable and function names should not be longer than 30 characters.**
```r
nullify_vals_outside_the_circle <- function(minitics, spatial_res, heatmap_matrix,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:233:83:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L233) *style:* **Trailing whitespace is superfluous.**
```r
nullify_vals_outside_the_circle <- function(minitics, spatial_res, heatmap_matrix,
                                                                                ^
```
[R/plotting_functions.R:235:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L235) *style:* **Variable and function names should not be longer than 30 characters.**
```r
    matrix_position_is_within_the_circle <- function() {
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:238:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L238) *style:* **Variable or function name should be snake_case.**
```r
    markNA <- matrix(minitics, ncol = spatial_res, nrow = spatial_res)
    ^~~~~~
```
[R/plotting_functions.R:246:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L246) *style:* **Lines should not be more than 80 characters.**
```r
##' @description Find the location to put the label around the circle at each of the lines
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:251:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L251) *style:* **Variable and function names should not be longer than 30 characters.**
```r
compute_longitude_label_location <- function(axis.rads, outer_radius) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:251:46:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L251) *style:* **Variable or function name should be snake_case.**
```r
compute_longitude_label_location <- function(axis.rads, outer_radius) {
                                             ^~~~~~~~~
```
[R/plotting_functions.R:252:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L252) *style:* **Lines should not be more than 80 characters.**
```r
    return(c(RMat(axis.rads) %*% matrix(c(1.1, 0, -1.1, 0) * outer_radius, ncol = 2)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:252:14:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L252) *warning:* **no visible global function definition for ‘RMat’, Did you mean 'hat'?**
```r
    return(c(RMat(axis.rads) %*% matrix(c(1.1, 0, -1.1, 0) * outer_radius, ncol = 2)))
             ^~~~
```
[R/plotting_functions.R:261:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L261) *style:* **Lines should not be more than 80 characters.**
```r
    endpoints <- zapsmall(c(RMat(axis_radian) %*% matrix(c(1, 0, -1, 0) * outer_radius,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:261:88:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L261) *style:* **Trailing whitespace is superfluous.**
```r
    endpoints <- zapsmall(c(RMat(axis_radian) %*% matrix(c(1, 0, -1, 0) * outer_radius,
                                                                                ^
```
[R/plotting_functions.R:263:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L263) *warning:* **no visible global function definition for ‘draw_line_segments’**
```r
    draw_line_segments(endpoints)
    ^~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:267:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L267) *style:* **Lines should not be more than 80 characters.**
```r
##' @description put a degree label at the ends of the endpoints for each longitude
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:274:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L274) *style:* **Lines should not be more than 80 characters.**
```r
plot_longitudinal_labels <- function(axis.rad, outer_radius, r_label, l_label, degree) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:274:38:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L274) *style:* **Variable or function name should be snake_case.**
```r
plot_longitudinal_labels <- function(axis.rad, outer_radius, r_label, l_label, degree) {
                                     ^~~~~~~~
```
[R/plotting_functions.R:275:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L275) *style:* **Lines should not be more than 80 characters.**
```r
    write_labels_at_endpoint_locations(r_label, l_label, degree, compute_longitude_label_location(axis.rad,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:275:108:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L275) *style:* **Trailing whitespace is superfluous.**
```r
    write_labels_at_endpoint_locations(r_label, l_label, degree, compute_longitude_label_location(axis.rad,
                                                                                ^
```
[R/plotting_functions.R:285:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L285) *style:* **Variable or function name should be snake_case.**
```r
RMat <- function(radians) {
^~~~
```
[R/plotting_functions.R:286:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L286) *style:* **Lines should not be more than 80 characters.**
```r
    return(matrix(c(cos(radians), sin(radians), -sin(radians), cos(radians)), ncol = 2))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:290:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L290) *style:* **Lines should not be more than 80 characters.**
```r
##' @description plots radial lines, degree label for latitudes, and plots radial spokes with labels
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:295:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L295) *warning:* **no visible global function definition for ‘plot_circle_radial_lines’**
```r
    plot_circle_radial_lines(radius_vals)
    ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:296:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L296) *warning:* **no visible global function definition for ‘plot_degree_label_for_latitudes’**
```r
    plot_degree_label_for_latitudes(outer_radius, radius_vals)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:297:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L297) *warning:* **no visible global function definition for ‘plot_radial_spokes_and_labels’**
```r
    plot_radial_spokes_and_labels(outer_radius)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:324:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L324) *style:* **Variable and function names should not be longer than 30 characters.**
```r
define_color_breaks_based_on_source <- function(col_breaks_source, z, Mat) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:324:71:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L324) *style:* **Variable or function name should be snake_case.**
```r
define_color_breaks_based_on_source <- function(col_breaks_source, z, Mat) {
                                                                      ^~~
```
[R/plotting_functions.R:329:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L329) *style:* **Lines should not be more than 80 characters.**
```r
    } else if ((length(col_breaks_source) == 2) & (is.numeric(col_breaks_source))) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:346:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L346) *style:* **Lines should not be more than 80 characters.**
```r
##' @param extrapolate whether to predict out to the equator of the eye, even if no data exists nearby.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:347:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L347) *style:* **Lines should not be more than 80 characters.**
```r
interpolate_input_data <- function(minitics, x, y, z, lambda, polynomial_m, extrapolate) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:348:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L348) *style:* **Lines should not be more than 80 characters.**
```r
    grid.list = list(x = minitics, y = minitics)  #choose locations to predict at
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:348:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L348) *style:* **Variable or function name should be snake_case.**
```r
    grid.list = list(x = minitics, y = minitics)  #choose locations to predict at
    ^~~~~~~~~
```
[R/plotting_functions.R:348:15:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L348) *style:* **Use <-, not =, for assignment.**
```r
    grid.list = list(x = minitics, y = minitics)  #choose locations to predict at
              ^
```
[R/plotting_functions.R:349:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L349) *style:* **Lines should not be more than 80 characters.**
```r
    t <- Tps(cbind(x, y), z, lambda = lambda, m = polynomial_m)  #computationally intensive
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:349:10:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L349) *warning:* **no visible global function definition for ‘Tps’, Did you mean 'Ops'?**
```r
    t <- Tps(cbind(x, y), z, lambda = lambda, m = polynomial_m)  #computationally intensive
         ^~~
```
[R/plotting_functions.R:350:12:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L350) *warning:* **no visible global function definition for ‘predictSurface’**
```r
    tmp <- predictSurface(t, grid.list, extrap = extrapolate)
           ^~~~~~~~~~~~~~
```
[R/plotting_functions.R:351:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L351) *style:* **Variable or function name should be snake_case.**
```r
    Mat <- tmp$z
    ^~~
```
[R/plotting_functions.R:363:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Variable or function name should be snake_case.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
    ^~~~~~~~~
```
[R/plotting_functions.R:363:25:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Put spaces around all infix operators.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
                       ~^~
```
[R/plotting_functions.R:363:31:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Put spaces around all infix operators.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
                             ~^~
```
[R/plotting_functions.R:363:37:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Put spaces around all infix operators.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
                                   ~^~
```
[R/plotting_functions.R:363:47:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Put spaces around all infix operators.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
                                             ~^~
```
[R/plotting_functions.R:363:57:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L363) *style:* **Put spaces around all infix operators.**
```r
    axis.rads <- c(0, pi/6, pi/3, pi/2, 2 * pi/3, 5 * pi/6)
                                                       ~^~
```
[R/plotting_functions.R:364:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L364) *style:* **Variable or function name should be snake_case.**
```r
    r.labs <- c(90, 60, 30, 0, 330, 300)
    ^~~~~~
```
[R/plotting_functions.R:365:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L365) *style:* **Variable or function name should be snake_case.**
```r
    l.labs <- c(270, 240, 210, 180, 150, 120)
    ^~~~~~
```
[R/plotting_functions.R:367:9:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L367) *warning:* **no visible global function definition for ‘plot_longitudinal_lines’**
```r
        plot_longitudinal_lines(axis.rads[i], outer_radius)
        ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:368:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L368) *style:* **Lines should not be more than 80 characters.**
```r
        plot_longitudinal_labels(axis.rads[i], outer_radius, r.labs[i], l.labs[i],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:368:83:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L368) *style:* **Trailing whitespace is superfluous.**
```r
        plot_longitudinal_labels(axis.rads[i], outer_radius, r.labs[i], l.labs[i],
                                                                                ^
```
[R/plotting_functions.R:389:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L389) *style:* **Lines should not be more than 80 characters.**
```r
##' @description Useful for making the falciform process look more smooth and refined. This is purely aesthetic.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:390:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L390) *style:* **Lines should not be more than 80 characters.**
```r
##' @details enhance the resolution of a polygon vertices dataframe by creating a spline along each vertex.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:391:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L391) *style:* **Lines should not be more than 80 characters.**
```r
##' @param xy vertices in dataframe with x and y columns, in order (not all are used).
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:398:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L398) *style:* **Lines should not be more than 80 characters.**
```r
##' @references http://gis.stackexchange.com/questions/24827/how-to-smooth-the-polygons-in-a-contour-map
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting_functions.R:407:5:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L407) *style:* **Variable or function name should be snake_case.**
```r
    data.spline <- spline(1:(n + 2 * k), data[, 1], n = vertices, ...)
    ^~~~~~~~~~~
```
[R/plotting_functions.R:411:1:](https://github.com/bc/retina/blob/07fa212b346b641cec5f57344a6c32c888bfcb9d/R/plotting_functions.R#L411) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```


hello


使命


```
LinearLayout linearLayout = listItemView.findViewById(R.id.words_view);
linearLayout.setBackgroundColor(parent.getResources().getColor(mColorResourceId));
```

This code worked for me, but I'll go with the solution provided. I'm curious though... would it be wrong to use the above code, considering that we were not taught anything about the ContextCompat or the ability to cast a LinearLayout View as simply a View?


Django does not set ON DELETE CASCADE for sqlite3.

And it _is_ a problem for developers. A Postgres multi-DELETE query will not run on sqlite3, meaning one wouldn't be able to get a dev environment up and running.


This was a breaking change and your version update should have reflected that. Our entire application broke and it took quite a bit of debugging to realize it was because of this change, which was only marked as a minor update.


A separate one (maybe preceding)


## Jenkins Build Summary

Built from [this commit](https://github.com/scottschurr/rippled/commit/75cf4d0b7056460afa570c4f562c70076b4460c3)

Built at __20180621 - 01:17:24__

### Test Results

Build Type | Log | Result | Status
---------- | --- | ------ | ------
msvc.debug | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/msvc.debug.txt) | 1029 cases, 0 failed, t: 566s | PASS :white_check_mark:
gcc.release.unity
-Dassert=true,
MANUAL_TESTS=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/gcc.release.unity_Dassert_true_MANUAL_TESTS_true.txt) | 1093 cases, 1 failed, t: 776s | FAIL :red_circle:
docs | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/docs.txt) | 1 cases, 0 failed, t: 2s | PASS :white_check_mark:
msvc.debug,
PROJECT_NAME=rippled_classic | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/msvc.debug_PROJECT_NAME_rippled_classic.txt) | 1029 cases, 0 failed, t: 1241s | PASS :white_check_mark:
coverage | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/coverage.txt) | 1032 cases, 0 failed, t: 686s | PASS :white_check_mark:
clang.debug.unity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/clang.debug.unity.txt) | 1032 cases, 0 failed, t: 251s | PASS :white_check_mark:
clang.debug.nounity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/clang.debug.nounity.txt) | 1030 cases, 0 failed, t: 148s | PASS :white_check_mark:
msvc.release | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/msvc.release.txt) | 1028 cases, 0 failed, t: 516s | PASS :white_check_mark:
clang.debug.unity,
PARALLEL_TESTS=false | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/clang.debug.unity_PARALLEL_TESTS_false.txt) | 1032 cases, 0 failed, t: 491s | PASS :white_check_mark:
gcc.debug.unity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/gcc.debug.unity.txt) | 1032 cases, 0 failed, t: 247s | PASS :white_check_mark:
gcc.debug.nounity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/gcc.debug.nounity.txt) | 1030 cases, 0 failed, t: 141s | PASS :white_check_mark:
clang.release.unity
-Dassert=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/clang.release.unity_Dassert_true.txt) | 1031 cases, 0 failed, t: 336s | PASS :white_check_mark:
gcc.release.unity
-Dassert=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/gcc.release.unity_Dassert_true.txt) | 1031 cases, 0 failed, t: 352s | PASS :white_check_mark:
gcc.debug.unity
-Dstatic=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/joeloser_deprecate_dirdelete/5//artifact/gcc.debug.unity_Dstatic_true.txt) | 1032 cases, 0 failed, t: 248s | PASS :white_check_mark:


spring.rabbitmq.host=localhost
spring.rabbitmq.port=5672
server.port=8882
hystrix.command.default.execution.timeout.enabled=false
management.security.enabled=false


Revised project proposal based on review feedback.


The knobs and sliders work well but as if there is no other banks, just one bank.


#6712-Zone-Coil-Autosizing-does-not-include-impact-of-AirTerminalSingleDuctMixer (rraustad) - i386-Windows-7-VisualStudio-14: OK (2929 of 2929 tests passed, 0 test warnings)

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


@OrenSchwartz Hi, these changes didn't fix bugs, just improved performance. I will overwrite this package with native javascript, doesn't rely on any frameworks and libraries.


Maybe `raise HandshakeFailure() from e`?


and here


This appears to break loading of Airborne Ranger ("airborne" on the CPC floppy softlist).
upd765 Read Deleted Data command is returning different status byte compared to 0.197.
ST1, in 0.197, returned 0x80 (End of Cylinder), and ST2 returned 0.
In 0.198, ST1 returns 1 (Missing Address Mark), and ST2 returns 1 also (Missing Data Mark), and the loader just keeps retrying.


as long as none runs `editorconfig: fix file` on anything that has the last line removed its OK :) That includes css files as I found :)


tsdfsdf


Hahaha sounds fun


Add animation to destination headings


let me show u sth marvellous


#6712-Zone-Coil-Autosizing-does-not-include-impact-of-AirTerminalSingleDuctMixer (rraustad) - x86_64-MacOS-10.9-clang: OK (2922 of 2929 tests passed, 9 test warnings)

Messages:

 * 8 tests had: AUD diffs.
 * 8 tests had: EIO diffs.
 * 5 tests had: ESO big diffs.
 * 6 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 4 tests had: MTR big diffs.

Failures:

regression Test Summary
 * Passed: 616
 * Failed: 7

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


Created multidev environment [wilder#ci-600](https://dashboard.pantheon.io/sites/14808007-3d80-488e-b078-193b788aa4a3#ci-600).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-600-wilder.pantheonsite.io/)


hello, when i run your code, there is an error: value error: string_input_producer requires a non-null input tensor. please could you tell me how to solve?


assign wrong variable:  should be tx_max_cpu_usage


请问一下这个demo怎么在本地运行啊


NB @rsignell-usgs if you have GDAL < 2.3, the `AWS_NO_SIGN_REQUEST` configuration will have no effect and GDAL may continue to dig up some configuration from your environment or AWS config file.


You need verify for us that the device sends those precise sysex messages when you press the bank button 4 times.


This could be useful as the name of the function composition function...

```
(define (. f g) (lambda (x) (g (f x))))
(define cadr (. cdr car))
```

or to express relative library paths : ```(. sublib)``` and ```(.. parent)```

It is an extension to Scheme, so non portable.


Thanks. I have this change in my local repo but had not pushed it.


Ah, seems you did that already on IRC.


There are no bank-dependent controls, so banking does nothing here


> wouldn't that get messy fast? (If i want to try out different saturation values, for example), why not overwrite the file? if the user wants to go back to the original value, just substract the value that was added or viceversa, the operation should be rather instantaneous.

Not really. The scheme files are tiny. It doesn't matter if there's a large number of them since they're not meant to be modified or managed by the user. I made this change since the color-schemes with modified saturation become different color-schemes entirely (*old values are lost*).

> this also breaks compatibility with the previous filename patterns.

This shouldn't matter, the color-schemes will just be regenerated. If the user has modified the files in the `.cache` directory then they aren't using `pywal` properly. The `.cache` directory is a cache of `pywal`'s output and not a permanent storage space for color-schemes.


Ah right, the whole map is 100% global commands.


I'm unsure why you commented on a commit instead of opening an issue. This is hard to test and all of my tests worked, as well as all of the RogueTech testing. I don't know what to tell you -- it might not work in all situations.


hector - you need some bindings that are bank-sensitive.


完善牛人信息


compare to https://github.com/Ardour/ardour/blob/master/midi_maps/Korg_nanoKONTROL.map#L26-L63  "B1" .. "B9"


I like what you did here. I might port it over to the main fork, just to keep things consistent. Again, sorry I completely missed your PRs.


Build version: __`0.10.0-master.171`__ :crossed_fingers: [`master`](https://github.com/streamich/ci-scripts/tree/master) on [*CircleCI*](https://circleci.com/gh/streamich/ci-scripts/171) :tada:


Build version: __`0.10.0-master.258`__ :crossed_fingers: [`master`](https://github.com/streamich/ci-scripts/tree/master) on [*Travis*](https://travis-ci.org/streamich/ci-scripts/builds/394826786) :tada:


flex


And I got "Breath of the Wild" for my birthday.


## Jenkins Build Summary

Built from [this commit](https://github.com/scottschurr/rippled/commit/11ab98cced69e1135a8a1e7ce55610273b46a4c0)

Built at __20180621 - 01:54:09__

### Test Results

Build Type | Log | Result | Status
---------- | --- | ------ | ------
rpm | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/rpm.txt) | 1032 cases, 0 failed, t: n/a | PASS :white_check_mark:
msvc.debug | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/msvc.debug.txt) | 1030 cases, 0 failed, t: 564s | PASS :white_check_mark:
gcc.release.unity
-Dassert=true,
MANUAL_TESTS=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/gcc.release.unity_Dassert_true_MANUAL_TESTS_true.txt) | 1101 cases, 0 failed, t: 796s | PASS :white_check_mark:
docs | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/docs.txt) | 1 cases, 0 failed, t: 2s | PASS :white_check_mark:
coverage | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/coverage.txt) | 1033 cases, 0 failed, t: 687s | PASS :white_check_mark:
clang.debug.unity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/clang.debug.unity.txt) | 1033 cases, 0 failed, t: 248s | PASS :white_check_mark:
msvc.debug,
PROJECT_NAME=rippled_classic | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/msvc.debug_PROJECT_NAME_rippled_classic.txt) | 1030 cases, 0 failed, t: 1237s | PASS :white_check_mark:
clang.debug.unity,
PARALLEL_TESTS=false | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/clang.debug.unity_PARALLEL_TESTS_false.txt) | 1033 cases, 0 failed, t: 494s | PASS :white_check_mark:
clang.debug.nounity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/clang.debug.nounity.txt) | 1031 cases, 0 failed, t: 556s | PASS :white_check_mark:
gcc.debug.unity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/gcc.debug.unity.txt) | 1033 cases, 0 failed, t: 248s | PASS :white_check_mark:
msvc.release | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/msvc.release.txt) | 1029 cases, 0 failed, t: 515s | PASS :white_check_mark:
clang.release.unity
-Dassert=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/clang.release.unity_Dassert_true.txt) | 1032 cases, 0 failed, t: 336s | PASS :white_check_mark:
gcc.debug.nounity | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/gcc.debug.nounity.txt) | 1031 cases, 0 failed, t: 514s | PASS :white_check_mark:
gcc.release.unity
-Dassert=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/gcc.release.unity_Dassert_true.txt) | 1032 cases, 0 failed, t: 350s | PASS :white_check_mark:
gcc.debug.unity
-Dstatic=true | [logfile](http://ci.ops.ripple.com/job/scott-schurr-rippled/job/develop/14//artifact/gcc.debug.unity_Dstatic_true.txt) | 1033 cases, 0 failed, t: 247s | PASS :white_check_mark:


🔥 [Jenkins job](http://xamarin-jenkins.guest.corp.microsoft.com:8080/job/macios/job/xcode10/26/display/redirect) (on internal Jenkins) failed in stage 'Test run' 🔥 : hudson.AbortException: script returned exit code 2

✅ [Build succeeded](http://xamarin-jenkins.guest.corp.microsoft.com:8080/job/macios/job/xcode10/26//console)
✅ [API Diff (from stable)](http://xamarin-storage/jenkins/xamarin-macios/xcode10/99a71fcfe494e38671ef6ca7fc94de811fe4e400/26/jenkins-results/api-diff/index.html)
✅ [API Diff (from PR only)](http://xamarin-storage/jenkins/xamarin-macios/xcode10/99a71fcfe494e38671ef6ca7fc94de811fe4e400/26/jenkins-results/apicomparison/index.html) (🔥 breaking changes 🔥)
✅ [Generator Diff](http://xamarin-storage/jenkins/xamarin-macios/xcode10/99a71fcfe494e38671ef6ca7fc94de811fe4e400/26/jenkins-results/generator-diff/index.html) (please review changes)
🔥 [Test run failed](http://xamarin-storage/jenkins/xamarin-macios/xcode10/99a71fcfe494e38671ef6ca7fc94de811fe4e400/26/jenkins-results/tests/index.html) 🔥

# Test results

1 tests failed, 160 tests passed.

## Failed tests

 * introspection/watchOS - simulator/Debug: Failed


Oops?


It looks like `pwrite` should be present, though was buggy (https://github.com/kripken/emscripten/issues/4837).  Or is there some other problem with it?


interfaceの定義の方が間違ってますね、すみません...
masterブランチの方で直しておくので後程取り込んでおいてください―


I think it's better to do a helm list every 2 seconds here and see whether it returns a non-zero exit code


Please helm in /usr/local/bin/too?


The deletion of this file is causing builds to fail: https://circleci.com/gh/stdlib-js/stdlib/3220#tests/containers/2.


Hi, First of all, thanks a lot for helping us out. This firmware is working fine, there is only a small problem, the speed at which the filament is pushed in or out is a bit slow, because of which when the filament is pushed to the nozzle, it does not reaches the nozzle completely, and similar is the case when the filament is removed. Can you please see what might be the problem..


Что за command? Какие-то непонятные цифры. Обзови их как-нибудь по-человечески, хотя бы )


👍 corrected


Actually I just looked at current page structure and IMHO it would be better to place API usage guide under advanced section


I tried but I couldn't set the placements of the blocks like in this. So, I made it by hand.
It is being used to generate the interaction diagram in the internals page though.


Какой смысл в этой операции? Мап просто ничего не делает и возвращает те же элементы что и были. Так как это объекты, то возвращаются ссылки.


This sync technique adds a number of input clock periods to all SYSREF outputs using the HMC7043 slip feature, while ensuring s/h is met at the DAC and the FPGA using fixed values for the analog delay and the digital coarse delay.

Since the input clock (1.2GHz) is 2x the DAC clock (600MHz), after RTIO/SYSREF alignment there are two phase possibilities for the SYSREF edge phase within the DAC clock cycle. Doing the selection (which depends on clock delays throughout the board, and the coax cable between the clock input and output on the DRTIO target) before the DAC SYSREF scan ensures that the SYSREF margins measured at the DAC are good with the phase that is selected.


A typo. It should be "Insert exist document as subdocument"


With the clang 6.0 release `langOpts.CPlusPlus1z` was changed to `langOpts.CPlusPlus17`.
But you can start cling with the arguments `-std=c++1z` and `-std=c++17`. Both enables c++ 17. I tested it with Structured binding.


Remove the underscores in the class fields. Use them only when initializing in the constructor.


Remove the underscores in the class fields.


should probably use command module instead of shell module to prevent shell exploitation bugs, since we are passing in user provided data through cm_bootstrap_data


If we configured the HMC830 to output 600MHz, we would not need this alignment here. But then we have a problem because the HMC7043 analog delay range is less than one half-period of 600MHz, and the combination of analog+coarse digital delay gets a "hole".


We don't need this error handler.


Wow... this file now contains a whole bunch of absolutely useless documentation, like "Size of the COCO dataset" for `size()` or "Loads an image at the image index" for the method `load_image(image_index)`. I honestly recommend that you take a look at https://sourcemaking.com/refactoring/smells/comments and avoid such documentation in the future.


This repo is being closed and the past, present, and future project content is being moved to its new home in the DLT Nexus repo as all captioning/ transcription/proofreading and sundry ventures will be sourced from within the new DLT Nexus organization going forward.


Smfh, no


smfh. no


Have fixed this in cloudman dockerfile as well.


A generated website with your changes can be accessed here: https://doc-review.qlikcore.com/910


but, isUpvoter is a promise


Are you referring to the fact that those two tests are very much alike ? In that case I could just extract the logic in a method and feed that method the account to work with. beforeEach wouldn't work as it would do it for all test cases, not just these two.


This also works because DRTIO re-syncs implemented in e29536351dfd41334f1081dc76bbbb831ffe7113 (NB: not tested on HW yet) only give an even number of slips, since in the worst case we are shifting by one RTIO coarse clock cycle (150MHz).


Можно просто:
`!requestString`


@rouault That's embarrassing :(


What was wrong with `data_dbc` ?


Why put an `IGNORE` here ? There are no UNIQUE/PRIMARY constraints on this table, so it won't change anything


This needs to be fixed on the e93 branch, not just master


This system don't support **Multi** Bossbar.


Pass one context parameter to the constructor. Define the dbset in the universal repository. Delete the underscore in the class fields.


Странное название метода с точки зрения как логики, так и инглиша :)


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


Have you checked that the `createOtherJobs` factory fans out the same jobs when re-run ? If not it might end up dumping the same blocks several times (in different jobs)


Thanks. My oversight.


changed the readme


🔥 [Jenkins job](http://xamarin-jenkins.guest.corp.microsoft.com:8080/job/macios/job/xcode10/28/display/redirect) (on internal Jenkins) failed in stage 'Test run' 🔥 : hudson.AbortException: script returned exit code 2

✅ [Build succeeded](http://xamarin-jenkins.guest.corp.microsoft.com:8080/job/macios/job/xcode10/28//console)
✅ [API Diff (from stable)](http://xamarin-storage/jenkins/xamarin-macios/xcode10/05cc126dc3a1ab18a4d5061257517906433e52d6/28/jenkins-results/api-diff/index.html)
✅ [API Diff (from PR only)](http://xamarin-storage/jenkins/xamarin-macios/xcode10/05cc126dc3a1ab18a4d5061257517906433e52d6/28/jenkins-results/apicomparison/index.html) (please review changes)
✅ [Generator Diff](http://xamarin-storage/jenkins/xamarin-macios/xcode10/05cc126dc3a1ab18a4d5061257517906433e52d6/28/jenkins-results/generator-diff/index.html) (please review changes)
🔥 [Test run failed](http://xamarin-storage/jenkins/xamarin-macios/xcode10/05cc126dc3a1ab18a4d5061257517906433e52d6/28/jenkins-results/tests/index.html) 🔥

# Test results

1 tests failed, 160 tests passed.

## Failed tests

 * introspection/watchOS - simulator/Debug: Failed


what does this do?


Update after Takuvik request


What do you think about moving `valJson` to module `Validators.Json` and renaming it to `json` ?


Я специально не делал такое дефолтное значение для ориентации так как LinearLayoutManager и GridLayoutManager(extends LinearLayoutManager) ссылаются на одну константу LinearLayoutManager.VERTICAL/HORIZONTAL, а StaggeredGridLayoutManager на другое RecyclerView.VERTICAl/HORIZONTAL и теоретически если что то поменяется в их константах могут вылезти проблемы хоть и числовые значения совпадают, правда не уверен стоит ди брать это во внимание


showForAdd и showForEdit - должно быть одним и тем же методом showFor


Apparently unrelated to parent bug but said bug unfixable without this being dealt with


The external mail program is a bit misleading. When in the EspoCRM UI I want to use the built-in mail support but when on my mobile phone, I want to use the phone's email program. I found several issues when I use my phone's mail program, that the folder in EspoCRM are not synched properly (Sent, Drafts, my own Archive folder, Trash).


@leon I think this PR should be separated in at least two parts:
- Changes needed in the store itself (perhaps one PR for each single change in it)
- EntityBase (plugin maybe)

I believe merge little pieces of code is easier and with the changes in store there will be another possibilities...

Nowadays I'm using EntityBase in my new project (copied/patched in my project itself), but changes in store are another thing. In this case I would have to follow apart from upstream or merge it often... for instance!


Component status also has etcd status in it and the other masters know about all the etcds. These statuses are reported separately.
Checking for `running` is not reliable because before going into `starting` state, the VM goes into `running` state, so the test passes before actually verifying anything.
Checking for `starting` state is also not reliable because sometimes it is very fast and the tests become flaky.


Слишком много всяких setБлаблаблаValue, почитай доку к форме, там есть способ как насетить значения во всем поля формы одной командой. Аналогично с getБлаблаблаValue - нет необходимости вручную выбирать каждое значение в отедльности, все можно забрать одной командой.


#2 here


Dear @YassenTrick:
I regret to inform you that the release you submitted, named "**SizePlayer**" (v1.0.2), for the project [SizePlayer](https://poggit.pmmp.io/ci/YassenTrick/SizePlayerPMMP/SizePlayer) on 2018-06-20T19:41:16.000Z, has been rejected.

Not all return points in onCommand returns a bool

Please resolve the issues listed above and submit the updated plugin again.

> Note: This comment is created here because this is the last commit when the released build was created.

> via Poggit (@poggit-bot)


Awsome


NVM that message


Для простоты все поля которые пользователь не может заполнять (список suppliers например, а так же id товара если он есть) можно сделать hidden полями (также как ты бы делал если бы делал это на нативной html форме), тогда не надо будет вводить локальные переменные для хранения этих значений (ведь эти переменные надо потом при закрытии очищать и вобще куча доп действий)


So that it reverts any accidental payments made to the contract


It was one or the other. no need for both.
The `INSERT IGNORE` I was talking about is in `createOtherJobs`, and _requires_ a unique / primary to work


Возможно стоит оставить интерфейс паблик? что бы его можно было заекстендить например если подключать через джитпак


Спорно, может надо не закрывать форму пока товар не сохранится (и показывать на ней спиннер пока идет сохранение)? Или еще что. Не всегда закрытие должно происходить сразу, я бы убрал отсюда этот метод.


Dear @MineBuilderFR:
Regarding the release you submitted, named "**DynoPM**" (v1.0), for the project [DynoPM](https://poggit.pmmp.io/ci/MineBuilderFR/DynoPM/DynoPM) on 2018-06-20T15:25:04.000Z, there are some problems with the values in the submit form.

In the description, advertisement of non-Poggit release channels as "release" is not acceptable.

Your release has been reset to draft. Please [edit the release](https://poggit.pmmp.io/edit/MineBuilderFR/DynoPM/DynoPM/7) to resolve these problems, then click "Submit" on the edit page to have the plugin reviewed again.

> Note: This comment is created here because this is the last commit when the released build was created.

> via Poggit (@poggit-bot)


I'm wondering if this line is necessary. We are currently running into issues because the 'home' directory in our Hadoop cluster containers is read-only (or non-existent in some cases, depending on the user). It's sad that this stops us from running dask on top of knit/yarn, just because it tries to write a file with config variables that it already uses to update the defaults right afterwards.


Yes, that's OK -- it is only the namespace -- used also in the Crossref documentation, s. for example https://data.crossref.org/reports/help/schema_doc/4.3.6/4.3.6.html... -- probably originating from some time ago...


## este es mi villano favorito


Corrected it in new commit


Логика понятна - кликнула Add - триггернули эвент на всем приложении - споймали его где надо и что-то сделалии. Система сломается как только этот компонент будет использован в приложении больше одного раза, и когда надо будет делать разные дейтсвия по нажатию на add (на однойстранице надо открыть окно edtiGoods, на другой editSuppliers на трейтей еще что-нибудь). Вывод: глобальные события приложения только для действительно единственных в своем роде событий (например application:loaded, page:change и т.п.)


GANGANdazeeeeee


I'm concerned that your clang-format is doing something different to my clang-format here!
Sure this was with the `Google` style?


pragma solidity ^0.4.16;

interface tokenRecipient { function receiveApproval(address _from, uint256 _value, address _token, bytes _extraData) external; }

contract TokenERC20 {
    // Public variables of the token
    string public name;
    string public symbol;
    uint8 public decimals = 18;
    // 18 decimals is the strongly suggested default, avoid changing it
    uint256 public totalSupply;

    // This creates an array with all balances
    mapping (address => uint256) public balanceOf;
    mapping (address => mapping (address => uint256)) public allowance;

    // This generates a public event on the blockchain that will notify clients
    event Transfer(address indexed from, address indexed to, uint256 value);

    // This notifies clients about the amount burnt
    event Burn(address indexed from, uint256 value);

    /**
     * Constructor function
     *
     * Initializes contract with initial supply tokens to the creator of the contract
     */
    function TokenERC20(
        uint256 initialSupply,
        string tokenName,
        string tokenSymbol
    ) public {
        totalSupply = initialSupply * 10 ** uint256(decimals);  // Update total supply with the decimal amount
        balanceOf[msg.sender] = totalSupply;                // Give the creator all initial tokens
        name = tokenName;                                   // Set the name for display purposes
        symbol = tokenSymbol;                               // Set the symbol for display purposes
    }

    /**
     * Internal transfer, only can be called by this contract
     */
    function _transfer(address _from, address _to, uint _value) internal {
        // Prevent transfer to 0x0 address. Use burn() instead
        require(_to != 0x0);
        // Check if the sender has enough
        require(balanceOf[_from] >= _value);
        // Check for overflows
        require(balanceOf[_to] + _value >= balanceOf[_to]);
        // Save this for an assertion in the future
        uint previousBalances = balanceOf[_from] + balanceOf[_to];
        // Subtract from the sender
        balanceOf[_from] -= _value;
        // Add the same to the recipient
        balanceOf[_to] += _value;
        emit Transfer(_from, _to, _value);
        // Asserts are used to use static analysis to find bugs in your code. They should never fail
        assert(balanceOf[_from] + balanceOf[_to] == previousBalances);
    }

    /**
     * Transfer tokens
     *
     * Send `_value` tokens to `_to` from your account
     *
     * @param _to The address of the recipient
     * @param _value the amount to send
     */
    function transfer(address _to, uint256 _value) public {
        _transfer(msg.sender, _to, _value);
    }

    /**
     * Transfer tokens from other address
     *
     * Send `_value` tokens to `_to` on behalf of `_from`
     *
     * @param _from The address of the sender
     * @param _to The address of the recipient
     * @param _value the amount to send
     */
    function transferFrom(address _from, address _to, uint256 _value) public returns (bool success) {
        require(_value <= allowance[_from][msg.sender]);     // Check allowance
        allowance[_from][msg.sender] -= _value;
        _transfer(_from, _to, _value);
        return true;
    }

    /**
     * Set allowance for other address
     *
     * Allows `_spender` to spend no more than `_value` tokens on your behalf
     *
     * @param _spender The address authorized to spend
     * @param _value the max amount they can spend
     */
    function approve(address _spender, uint256 _value) public
        returns (bool success) {
        allowance[msg.sender][_spender] = _value;
        return true;
    }

    /**
     * Set allowance for other address and notify
     *
     * Allows `_spender` to spend no more than `_value` tokens on your behalf, and then ping the contract about it
     *
     * @param _spender The address authorized to spend
     * @param _value the max amount they can spend
     * @param _extraData some extra information to send to the approved contract
     */
    function approveAndCall(address _spender, uint256 _value, bytes _extraData)
        public
        returns (bool success) {
        tokenRecipient spender = tokenRecipient(_spender);
        if (approve(_spender, _value)) {
            spender.receiveApproval(msg.sender, _value, this, _extraData);
            return true;
        }
    }

    /**
     * Destroy tokens
     *
     * Remove `_value` tokens from the system irreversibly
     *
     * @param _value the amount of money to burn
     */
    function burn(uint256 _value) public returns (bool success) {
        require(balanceOf[msg.sender] >= _value);   // Check if the sender has enough
        balanceOf[msg.sender] -= _value;            // Subtract from the sender
        totalSupply -= _value;                      // Updates totalSupply
        emit Burn(msg.sender, _value);
        return true;
    }

    /**
     * Destroy tokens from other account
     *
     * Remove `_value` tokens from the system irreversibly on behalf of `_from`.
     *
     * @param _from the address of the sender
     * @param _value the amount of money to burn
     */
    function burnFrom(address _from, uint256 _value) public returns (bool success) {
        require(balanceOf[_from] >= _value);                // Check if the targeted balance is enough
        require(_value <= allowance[_from][msg.sender]);    // Check allowance
        balanceOf[_from] -= _value;                         // Subtract from the targeted balance
        allowance[_from][msg.sender] -= _value;             // Subtract from the sender's allowance
        totalSupply -= _value;                              // Update totalSupply
        emit Burn(_from, _value);
        return true;
    }
}


There are at least a couple of other (container) formats that one would want, like mp4, mkv and even gif (probably more). I guess it would be OK to automatically use the given filename extension of the target file as `-f` parameter, instead of requiring the user to do that.

> And why does the FFmpeg format auto-detection not work here anymore?

Because the 1. pass doesn't have the actual target filename. Instead it writes to `/dev/null` or `NUL`. You can take a look at the [FFmpeg VP9 Two-Pass guide](https://trac.ffmpeg.org/wiki/Encode/VP9#twopass) if you want.

> Is there any specific reason for using two-pass encoding in this case?

Better encoding results, smaller filesize and so on. It's a bit overkill for this particular use-case, but there are a few animations that span more than 10 seconds and contain a few 100 frames. I would hope it has at least some effect on those .


Hmm... This is good question... Missing context could only happen when executing Crossref export using CLI... I will test that case... :+1:


По названию я подумал что он возращает заголовок окна (текст). Поправь на что-то типа getWindowHeader


글자가 깨져있다.


I haven't changed that. I must have had an older version that actually overwrote the new _data_dbc_. I'l revert that.


How do you feel about defaulting to `ENV.fetch('BUNDLE_GEMFILE', 'Gemfile.lock')` to be consistent with the rest of bundler? The CLI invocation still doesn't seem configurable and this was the first approach I tried.


Ok


Правильно будет переименовать во что-то типа showErrorMessage. Названия методам всегда даются в виде глаголов (doSomething, openSomething, showSomething и т.п.). В противном случае непонятно из навзания то ли метод отображает ошибку, то ли возвращает ее текст, то ли запрос на сервер с ошибкой отравляет, то ли еще что.


Ah, and maybe you thought if this can be executed form the site level or so... This should not be the case...


JetApp как раз по умолчанию имеет события. Надо расширять JetView


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


I don't think etcd running means all processes are running though. What is our expected return state from a master going down?


Just tested the difference with https://www.pixiv.net/member_illust.php?mode=medium&illust_id=69322926

1-pass encoding gives a 93.8KB file, 2-pass reduces it to 35.2KB.