[R/data.R:28:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/data.R#L28) *style:* **Lines should not be more than 80 characters.**
```r
#' @source Taken from \url{http://www.ats.ucla.edu/stat/r/dae/logit.htm}, direct download
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helper_funs.R:78:14:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/helper_funs.R#L78) *warning:* **no visible global function definition for ‘no_plot’, Did you mean 'boxplot'?**
```r
  plot_df <- no_plot(model)
             ^~~~~~~
```
[R/insert_or.R:90:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/insert_or.R#L90) *style:* **Lines should not be more than 80 characters.**
```r
insert_or <- function(plot_object = NULL, or_object = NULL, line_col = "red", # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/insert_or.R:146:24:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/insert_or.R#L146) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      arrow_xloc_l <- -(max(plot_object$data$y) - min(plot_object$data$y)) *
                       ^
```
[R/or_gam.R:80:54:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L80) *style:* **Variable or function name should be snake_case.**
```r
                   percentage = NULL, slice = FALSE, CI = NULL) {
                                                     ^~
```
[R/or_gam.R:130:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L130) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam1_CI_low <- pred_gam1[1] - (2 * pred_gam1[2])
      ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:131:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L131) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam1_CI_high <- pred_gam1[1] + (2 * pred_gam1[2])
      ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:139:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L139) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam2_CI_low <- pred_gam2[1] - (2 * pred_gam2[2])
      ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:140:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L140) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam2_CI_high <- pred_gam2[1] + (2 * pred_gam2[2])
      ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:142:24:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L142) *style:* **Use <-, not =, for assignment.**
```r
      result$predictor = pred
                       ^
```
[R/or_gam.R:151:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L151) *style:* **Put spaces around all infix operators.**
```r
      result$perc1[x] <- percentage*x - percentage
                                  ~^~
```
[R/or_gam.R:152:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L152) *style:* **Put spaces around all infix operators.**
```r
      result$perc2[x] <- percentage*x
                                  ~^~
```
[R/or_gam.R:178:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L178) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam1_CI_low <- pred_gam1[1] - (2 * pred_gam1[2])
  ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:179:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L179) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam1_CI_high <- pred_gam1[1] + (2 * pred_gam1[2])
  ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:186:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L186) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam2_CI_low <- pred_gam2[1] - (2 * pred_gam2[2])
  ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:187:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L187) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam2_CI_high <- pred_gam2[1] + (2 * pred_gam2[2])
  ^~~~~~~~~~~~~~~~~
```
[R/or_glm.R:62:39:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L62) *style:* **Variable or function name should be snake_case.**
```r
or_glm <- function(data, model, incr, CI = 0.95) {
                                      ^~
```
[R/or_glm.R:80:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L80) *style:* **Variable or function name should be snake_case.**
```r
  CI_low <- list()
  ^~~~~~
```
[R/or_glm.R:81:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L81) *style:* **Variable or function name should be snake_case.**
```r
  CI_high <- list()
  ^~~~~~~
```
[R/or_glm.R:86:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L86) *style:* **Variable or function name should be snake_case.**
```r
      CI_list <- as.data.frame(suppressMessages(confint(model,
      ^~~~~~~
```
[R/or_glm.R:92:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L92) *style:* **Lines should not be more than 80 characters.**
```r
      odds_ratios[[i]] <- round(exp(as.numeric(coef[[i]]) * as.numeric(incr[[i]])), 3)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:94:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L94) *style:* **Lines should not be more than 80 characters.**
```r
        CI_low[[i]] <- round(exp(CI_list[i, 1] * as.numeric(incr[[i]])), 3) # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:95:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L95) *style:* **Lines should not be more than 80 characters.**
```r
        CI_high[[i]] <- round(exp(CI_list[i, 2] * as.numeric(incr[[i]])), 3) # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:98:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L98) *warning:* **local variable ‘or’ assigned but may not be used**
```r
      or <- odds_ratios[[i]]
      ^~
```
[R/or_glm.R:116:5:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L116) *style:* **Variable or function name should be snake_case.**
```r
    CI_low <- c(rep(NA, length(preds)))
    ^~~~~~
```
[R/or_glm.R:117:5:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L117) *style:* **Variable or function name should be snake_case.**
```r
    CI_high <- c(rep(NA, length(preds)))
    ^~~~~~~
```
[R/plot_gam.R:4:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L4) *style:* **Lines should not be more than 80 characters.**
```r
#' @description This function plots the smoothing function of selected GAM(M) models
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plot_gam.R:27:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L27) *style:* **Lines should not be more than 80 characters.**
```r
#' @param ci_alpha Numeric (range: 0-1). Opacity value of confidence interval shading.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plot_gam.R:66:9:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L66) *warning:* **no visible global function definition for ‘gam_to_df’**
```r
  df <- gam_to_df(model, pred)
        ^~~~~~~~~
```
[R/plot_gam.R:103:28:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L103) *warning:* **no visible global function definition for ‘ggtitle’, Did you mean 'title'?**
```r
    plot_gam <- plot_gam + ggtitle(title)
                           ^~~~~~~
```
[tests/testthat/test-insert-or.R:5:69:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L5) *style:* **Trailing whitespace is superfluous.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1 ^ 2)) + s(x2) + offset(x3) + x4,
                                                                    ^
```
[tests/testthat/test-insert-or.R:7:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L7) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:8:21:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L8) *style:* **Trailing whitespace is superfluous.**
```r
  library(oddsratio)
                    ^~~~~~~~~~~~
```
[tests/testthat/test-insert-or.R:10:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L10) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:14:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L14) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:16:8:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L16) *style:* **Use <-, not =, for assignment.**
```r
  ymin = ggplot_build(plot_object)$layout$panel_ranges[[1]]$y.range[1]
       ^
```
[tests/testthat/test-insert-or.R:17:8:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L17) *style:* **Use <-, not =, for assignment.**
```r
  ymax = ggplot_build(plot_object)$layout$panel_ranges[[1]]$y.range[2]
       ^
```
[tests/testthat/test-insert-or.R:18:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:23:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L23) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:24:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L24) *style:* **Trailing whitespace is superfluous.**
```r
})
  ^
```
[tests/testthat/test-or-gam.R:9:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L9) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-gam.R:23:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L23) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-gam.R:43:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L43) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-glm.R:18:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-glm.R#L18) *style:* **Variable or function name should be snake_case.**
```r
  fit_glmmPQL <-glmmPQL(y ~ trt + week, random = ~1 | ID,
  ^~~~~~~~~~~
```
[tests/testthat/test-or-glm.R:18:15:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-glm.R#L18) *style:* **Put spaces around all infix operators.**
```r
  fit_glmmPQL <-glmmPQL(y ~ trt + week, random = ~1 | ID,
              ^~~
```


uverbs_release_pd is called when pd->ref reach zero. in normal conditions this happen w/o race with other ib_pd users and so the pd->usecnt == 0. but if the code is not written well and someone manage to share the ib_pd and inc the useccnt or do extra dec on the usecnt we will see it here. not sure why you say this can't happen. please explain.


Ho avuto problemi di accesso, avevo caricato probabilmente solo sulla cartella desktop. Può andare come immagine? Senza le scritte.
Grazie


also, even the n900 with 256MB ram is reported to be "usable": https://github.com/postmarketOS/pmbootstrap/issues/438
I think the droid4 is well fit for running a slim linux.


The restructuring seems to take more time than expected. :/


第二次作业


Cool!


Still needs further updates : currently, it should work on every 64bits system, but maybe not on 32bits ones


spaces instead tabs please


Note that the whole subject started as a fix for the times losses reported by Ronald, and it is a little bit ironic that you now try to create more time losses :-)

Do you think you could make a version that keeps the Elo gains (if any) and never loses on time?


line duplicated?!


Bonjour Gaëtan,

J’espère ne pas trop te déranger.
J’ai une interrogation sur du code php. Ayant discuté avec Salwa elle m’a dit que tu pourrais peut être m’aiguiller.
J’ai créé un dépôt sur GitHub avec mon code et t’ai ajouté en collaborateur.

Mon problème est le suivant :

Je dois faire un code permettant :
1 - de vérifier qu’il y ait bien la variable email dans l’url, sinon ça affiche un message.
2 - S’il y a la variable dans l’url, vérifier si elle n’est pas déjà dans la base de donnée. Si oui, un message.
3 - S’il y a bien la variable et qu’elle n’est pas déjà enregistrée dans la base de donnée :
	- Afficher le formulaire
	- Enregistrer la variable dans la base de données au submit.

Mon problème étant que je n’ai pas accès à la page où va l’action du formulaire.
C’est un formulaire réalisé par un client de mon lieu de stage.
Sa fonction étant de récolter les informations du formulaire ailleurs que sur wordpress.
Du coup je ne peux pas ajouter l’action d’enregistrer l’email dans ma table.

J’ai pensé à faire du JS mais dans le doute et comme c’est pour un de leurs clients. Je ne veux pas faire n’importe quoi et donc je viens quémander des conseils.

Si tu ne peux pas, je comprendrais.

Merci malgré tout.

Sylvie
06 76 45 31 32 (si tu préfères répondre par tel)


Que rápido has ido en borrarlo, jajaja, pon tú nombre hombre, jajaja.
Y como siempre ni puñetera idea de traducir, adelante sigue así, eres el mejor.


It's a server variable, you can find it either from the `phpinfo()` or from debugbar.


franchement le support de cette plateforme pue la merde


For work joinable, need modify LoginPacket.


Hey, I checked it, looks nivce!


I'm getting a harmless lockdep warning with gr3d, but I think you should get it too with VIC

    178.153807] tegra-host1x 50000000.host1x: cdma_timeout_handler: timeout: 4 (04-54180000.gr3d), HW thresh 1, done 2
    178.156650] INFO: trying to register non-static key.
    178.156762] the code is fine but needs lockdep annotation.
    178.156867] turning off the locking correctness validator.
    178.156979] CPU: 0 PID: 20 Comm: kworker/0:1 Tainted: G         C       4.14.0-rc4-next-20171012-00078-g1be490603561-dirty #283
    178.157184] Hardware name: NVIDIA Tegra SoC (Flattened Device Tree)
    178.157424] Workqueue: events syncpt_thresh_work [host1x]
    178.157557] [<c0113d00>] (unwind_backtrace) from [<c010da88>] (show_stack+0x20/0x24)
    178.157714] [<c010da88>] (show_stack) from [<c0b76694>] (dump_stack+0xb0/0xdc)
    178.157856] [<c0b76694>] (dump_stack) from [<c018c8ac>] (register_lock_class+0x404/0x608)
    178.158011] [<c018c8ac>] (register_lock_class) from [<c0191510>] (__lock_acquire+0x88/0x1534)
    178.158172] [<c0191510>] (__lock_acquire) from [<c0193494>] (lock_acquire+0xe4/0x2c8)
    178.158324] [<c0193494>] (lock_acquire) from [<c0b90a08>] (__mutex_lock+0x8c/0xb50)
    178.158469] [<c0b90a08>] (__mutex_lock) from [<c0b914f8>] (mutex_lock_nested+0x2c/0x34)
    178.158761] [<c0b914f8>] (mutex_lock_nested) from [<bf045578>] (tegra_drm_context_put_channel+0x28/0x50 [tegra_drm])
    178.159095] [<bf045578>] (tegra_drm_context_put_channel [tegra_drm]) from [<bf0d5944>] (gr3d_submit_done+0x50/0x5c [tegra_drm])
    178.159449] [<bf0d5944>] (gr3d_submit_done [tegra_drm]) from [<bf0188d4>] (host1x_cdma_update+0x1c0/0x254 [host1x])
    178.159753] [<bf0188d4>] (host1x_cdma_update [host1x]) from [<bf016efc>] (action_submit_complete+0x2c/0x218 [host1x])
    178.160051] [<bf016efc>] (action_submit_complete [host1x]) from [<bf016d10>] (process_wait_list+0x2b8/0x384 [host1x])
    178.160352] [<bf016d10>] (process_wait_list [host1x]) from [<bf016e18>] (syncpt_thresh_work+0x3c/0x40 [host1x])
    178.160595] [<bf016e18>] (syncpt_thresh_work [host1x]) from [<c0148954>] (process_one_work+0x270/0x868)
    178.160774] [<c0148954>] (process_one_work) from [<c0149c34>] (worker_thread+0x60/0x550)
    178.160930] [<c0149c34>] (worker_thread) from [<c0150258>] (kthread+0x180/0x1bc)
    178.161077] [<c0150258>] (kthread) from [<c01086e8>] (ret_from_fork+0x14/0x2c)


Wow, INTeresting chalenge.


Is this sufficient to mitigate the attack? Or an update on the client side is required as well?


kref = amount of times the ib_pd object pointer is referenced
usecnt = amount of times the ib_pd object was shared
ib_pd object can be shared say x times but referenced y times.
the kref is used to start the ib_pd object destruction if and only if no one hold ref to that object
at that time it is assumed that ib_pd usecnt is already zero.
if not we have a bug.
so i do not see how i can remove the kref here.


I'd rather say "this overload"


Jajaja, le sobra "la", que bueno eres


Are there some inspections that could enforce this formatting style?


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/288510263?utm_source=github_status&utm_medium=notification


Otra vez le sobra "la", muy bueno por tu parte


FYI I had done this in another branch which has now been merged into master. That being said, in the TS branch I've updated much of the dependencies.

After the next couple of small PRs and fixes go in, we can release the next major version.


Ole , otra vez "la", como cambia el significado de la frase, que pasada de traductor, jajaja


It's the default (in 2017.3, at least)


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/c43ca27d-b26e-4156-921e-5337d72e05e4 "BlrApp"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/1627f65f-1a8e-4818-8fc0-1cc7835cf973 "BlrApp") deployed [BlrRepo](https://github.com/reni-varghese/BlrRepo/releases/tag/deploy-ibm-yp-us-south-Information-Technology-Training-dev-20171016-114722 "deploy-ibm-yp-us-south-Information-Technology-Training-dev-20171016-114722") to [dev](https://console.bluemix.net/apps/145c3472-d3a2-4fa3-b858-db3d06556d8c?env_id=ibm:yp:us-south "ibm:yp:us-south:Information Technology Training:dev"), including this commit


I think you can just include the code, I wouldn't want another dependency. There are tiny decompressors available. It is just the effort of coding it, but the decompression code is freely available, sometimes under 20 lines of C (so I read somewhere). I think it would significantly shrink the binary!


Use `TEST_PASS()` to indicate no failures.


can't... the kref decide when ib_dealloc_pd can be safely called...


nice!


Why is this not in 1.8.2? It's still a bug with the resizing. As described here: https://community.xibo.org.uk/t/png-file-and-resolution-downscale/5818


fix #31


what is this madness!


ah then there is no HTTPS but there is REQUEST_SCHEME with https as a value


I did not manage to find any description of `scale` in `compose-file-v2`.


No se donde has sacado (by dpzjsy), por que la traducción mía pone (dpzjsy), sin el by, para que quepa en el programa, pero eso a tí da igual, hay que modificar, para que salga tú nombre.


This breaks `master`, please fix!


fast-forwarding master to auto = 59149264


ADDED getThemesInUse (using [@vrann fix](https://github.com/magento/magento2/pull/8142/commits/35e3225f0478a7b952119dca2a800a76b9ae5f3e#diff-dbf04232abcedbe2e67996b043041b48) + added check for used themes in products, categories and cms pages)

REF https://github.com/magento/magento2/issues/10363

UnitTests are failing due to mocking problems:
```
1) Magento\Catalog\Test\Unit\Model\Product\Image\CacheTest::testGenerate
Error: Call to a member function getId() on null
```


We fixed it in 1.8.2 with respect to the Library upload, which was the original bug. We didn't realise at the time that it also effected other areas of the software. See https://github.com/xibosignage/xibo/issues/1287


Lo mismo que lo anterior, estaba sin el by, ahora aparece con el by, hay tanta gente modificando, que al final salle lo que sale


:tada::tada::tada:


Otra con lo mismo, la mía sin el by, ahora aparece el by,


innerWidth med litet i


I'd say:
> Then no undefined behaviour happens.


Otra vez sobra "la", una pasada tío, que traducción.


TeamCity Selenium POM :: CI [Build 100](https://teamcity.sponte.uk/viewLog.html?buildId=1377&buildTypeId=SeleniumPom_Ci) is now running


Más cambios, aparece el by de nuevo, y sobre todo que sobre "la"


KKBox - competition on kaggle


Más cambios, la y by


Seems 'sleeping' bug here as well

    233.078086] BUG: sleeping function called from invalid context at ./include/linux/percpu-rwsem.h:33
    233.082576] in_atomic(): 1, irqs_disabled(): 128, pid: 20, name: kworker/0:1
    233.087056] INFO: lockdep is turned off.
    233.091488] irq event stamp: 805195
    233.095891] hardirqs last  enabled at (805195): [<c02a3554>] kfree+0xd8/0x270
    233.100345] hardirqs last disabled at (805194): [<c02a34fc>] kfree+0x80/0x270
    233.104742] softirqs last  enabled at (800384): [<c0101820>] __do_softirq+0x2a8/0x69c
    233.109134] softirqs last disabled at (800377): [<c01302a8>] irq_exit+0x10c/0x170
    233.113499] CPU: 0 PID: 20 Comm: kworker/0:1 Tainted: G      D  C       4.14.0-rc4-next-20171012-00078-g1be490603561-dirty #283
    233.118004] Hardware name: NVIDIA Tegra SoC (Flattened Device Tree)
    233.122564] Workqueue: events syncpt_thresh_work [host1x]
    233.127070] [<c0113d00>] (unwind_backtrace) from [<c010da88>] (show_stack+0x20/0x24)
    233.131611] [<c010da88>] (show_stack) from [<c0b76694>] (dump_stack+0xb0/0xdc)
    233.136153] [<c0b76694>] (dump_stack) from [<c0159814>] (___might_sleep+0x23c/0x270)
    233.140676] [<c0159814>] (___might_sleep) from [<c01598b8>] (__might_sleep+0x70/0xa8)
    233.145203] [<c01598b8>] (__might_sleep) from [<c013cc7c>] (exit_signals+0x34/0x330)
    233.149718] [<c013cc7c>] (exit_signals) from [<c012cd68>] (do_exit+0xb0/0xd10)
    233.154239] [<c012cd68>] (do_exit) from [<c010def4>] (die+0x468/0x4cc)
    233.158749] [<c010def4>] (die) from [<c010df80>] (arm_notify_die+0x28/0x60)
    233.163266] [<c010df80>] (arm_notify_die) from [<c01012fc>] (do_DataAbort+0xc4/0xc8)
    233.167800] [<c01012fc>] (do_DataAbort) from [<c010e6fc>] (__dabt_svc+0x5c/0xa0)
    233.172339] Exception stack(0xef221bf0 to 0xef221c38)
    233.176844] 1be0:                                     6b6b6b6b 00000000 6b6b6c2f 00000000
    233.181421] 1c00: c19ead6c c12600fc c12d31c4 00000001 ed9cbac4 00000000 ef1e0140 ef221cb4
    233.185997] 1c20: ef220000 ef221c40 c0193494 c0191568 20000093 ffffffff
    233.190554] [<c010e6fc>] (__dabt_svc) from [<c0191568>] (__lock_acquire+0xe0/0x1534)
    233.195166] [<c0191568>] (__lock_acquire) from [<c0193494>] (lock_acquire+0xe4/0x2c8)
    233.199782] [<c0193494>] (lock_acquire) from [<c0b90a08>] (__mutex_lock+0x8c/0xb50)
    233.204411] [<c0b90a08>] (__mutex_lock) from [<c0b914f8>] (mutex_lock_nested+0x2c/0x34)
    233.209127] [<c0b914f8>] (mutex_lock_nested) from [<bf045578>] (tegra_drm_context_put_channel+0x28/0x50 [tegra_drm])
    233.213995] [<bf045578>] (tegra_drm_context_put_channel [tegra_drm]) from [<bf0d5944>] (gr3d_submit_done+0x50/0x5c [tegra_drm])
    233.218952] [<bf0d5944>] (gr3d_submit_done [tegra_drm]) from [<bf0188d4>] (host1x_cdma_update+0x1c0/0x254 [host1x])
    233.223958] [<bf0188d4>] (host1x_cdma_update [host1x]) from [<bf016efc>] (action_submit_complete+0x2c/0x218 [host1x])
    233.229052] [<bf016efc>] (action_submit_complete [host1x]) from [<bf016d10>] (process_wait_list+0x2b8/0x384 [host1x])
    233.234233] [<bf016d10>] (process_wait_list [host1x]) from [<bf016e18>] (syncpt_thresh_work+0x3c/0x40 [host1x])
    233.239380] [<bf016e18>] (syncpt_thresh_work [host1x]) from [<c0148954>] (process_one_work+0x270/0x868)
    233.244476] [<c0148954>] (process_one_work) from [<c0149c34>] (worker_thread+0x60/0x550)
    233.249580] [<c0149c34>] (worker_thread) from [<c0150258>] (kthread+0x180/0x1bc)
    233.254644] [<c0150258>] (kthread) from [<c01086e8>] (ret_from_fork+0x14/0x2c)


TeamCity Selenium POM :: CI [Build 100](https://teamcity.sponte.uk/viewLog.html?buildId=1377&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:21
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

```


Also removed !invite, as the bot is becoming private.


Ska vara

```javascript
    context.strokeStyle = 'hsl(' + hue + ', 50%, 50%';


Awesome, thanks for your swift reply Dan.


Du har alltså glömt variabeln hue


!spookyme is temporary, and will be commented out in November.


Also fixed something with .then statements.


Léelo es para partirse de risa, estaba bien y lo has puesto mal.
"Esta entrada es la de desinstalar "del" controlador NVIDIA.....


I'd split this stuff out into a separate method or just move it into the `receiveAnnotations` method below. That way this method only deals with transmitting.


TeamCity Selenium POM :: CI [Build 101](https://teamcity.sponte.uk/viewLog.html?buildId=1378&buildTypeId=SeleniumPom_Ci) is now running


Woohoo!
Logo sucks but still yayy!


Why do you pass a `IList<string>` type here? This should be simply `string`.


You could use `var` here.


The issue occurs primarily on the client side and cannot be mitigated by the AP.


TeamCity Selenium POM :: CI [Build 101](https://teamcity.sponte.uk/viewLog.html?buildId=1378&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:03
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

```


You forgot to open an read the file here based on the input url.


Maybe you should get a text editor that detects when the file has changed on disk?


Otherwise your solution is good, it's a shame that you missed the things I commented above.


@SntsDev the change you highlighted was introduced in #12276, I just fixed the formatting here.

Could you open a new issue about this, referencing that PR?

Also, can you share the model?


typeo on the master `html5mode` should be `html5Mode`.

Fixed in next commit on the master branch


Looking at the implementation, I think it makes no difference.
See https://golang.org/src/log/log.go?s=9379:9407#L299
But since it is the right way to do, I will change this.


It's an ugly solution in my book, but it works; he's looping through an array that looks like this "sharpness:5:speed:10:poison:3...." using %2 and casting with (int) and (string) to alternate setting $ench to the Enchantment using $name_level as a string on the first loop, then setting the level using $name_level as an int on the next loop


And you put this method inside your Main method. Why? This should an individual method outside your Main.


@paulmassen Can you create bug out of this. This was broken in older versions, too, but with the opposite way.


Where is this going to be used?


I don't see this being needed anywhere?!


![](https://github.trello.services/images/mini-trello-icon.png) [Create Models](https://trello.com/c/nDXL5EbD/2-create-models)


Thanks for clarifying.


![](https://github.trello.services/images/mini-trello-icon.png) [Add sidebar to web server layout](https://trello.com/c/5Z1WQyT1/1-add-sidebar-to-web-server-layout)


![](https://github.trello.services/images/mini-trello-icon.png) [Fix Url](https://trello.com/c/Ry2yMkib/3-fix-url)


![](https://github.trello.services/images/mini-trello-icon.png) [Setup db](https://trello.com/c/sEmdyqkA/4-setup-db)


https://msdn.microsoft.com/en-us/library/mt260482.aspx

Looks about the same as what I'm using.  With strtod(), strtol() and strtoul() you can "split" a string by passing a pointer to the pointer to receive the starting address of the first character after the C standard library finishes parsing a number out of the string.

Although, your approach fails if the user specifies just "12" or "3" with no `:` character.  The results should be time is 12 o'clock PM and 3 o'clock AM for both; yours will just complain about an invalid time.


Although sleep is caused by DataAbort, probably this bug is unrelated to this code.


You can generate those, btw:

```
s <- (dungeon.Y << 16) + dungeon.X + currentProvince
s <- -rol32(s, 5)
mifName <- s.toString[0:8] + ".MIF"
```


What exactly is the meaning of this field? Please add documentation :)


You put this method inside your Main method again. Why?


Чтобы не считать сумму последних элементов в условии и в теле цикла, можно в условии проверять значение последнего элемента массива (метод last)


Лучше использовать цикл с условием


This is a working solution, however it's not the easiest way to solve this :) Anyway, this way I saw that it's definitely your own code, and you used the needed building blocks in a correct way.


object ArrayOps {
  def main(args: Array[String]): Unit = {

    val array = new Array[Int](5) //此方法创建数组需要写明数组类型以及数组的固定长度  -->  “ new Array[Int](5) ”
   array(5) = 10 //对数组访问的时候，数组下标从0开始，最长长度不能超过“ length-1 ”，该数组赋值下标越界 会报错 “ java.lang.ArrayIndexOutOfBoundsException ”
    for(item <- array) println("item -->  " + item)

    val array1 = Array(1,2,3,4,5) //此方法创建数组已经给出参数，因为Scala有自动推导类型的方法，所以可以不用书写该数组的类型
    array1(1) = 20
    for(item1 <- array1) println("item1 -->  " + item1)
  }
}


Why did you trim the returning value here? It's not necessary.


This is to fix below parodus build failure .

nopoll_conn.c: In function 'nopoll_init_recursive_mut_once':
nopoll_conn.c:590:2: error: passing argument 1 of 'pthread_mutex_init' from incompatible pointer type [-Werror]
/usr/include/pthread.h:733:12: note: expected 'union pthread_mutex_t *' but argument is of type 'pthread_once_t *'
cc1: all warnings being treated as errors
make[5]: *** [nopoll_conn.lo] Error 1


maybe a more intuitive name would be `children`?


![](https://github.trello.services/images/mini-trello-icon.png) [Add page info to sidebar](https://trello.com/c/90W1spCX/7-add-page-info-to-sidebar)


I used a wrong issue id. The correct one is https://github.com/OpenConceptLab/DATIM-Metadata-Project/issues/119


OK will do that in ib_uverbs_alloc_pd & __ib_alloc_pd . Thanks.


Gotcha, so scrap this one until we get the TS version going?


Ali je za skoke klasičnega množenja že naložena najnovejša slika? Meni se zdi, da ni nisem pa siguren.


This comment no longer holds true.


Only the starting position for the initial level matters, when moving between levels you are placed in the middle of the same cell you were on the previous level, turned by 180°.
Random dungeons have 25 fixed entry points, 1 per block. (The game never generates dungeons more than 2x2, though).


At the first time we run this application, opendata_postal_codes will be empty because the database isn't fill yet, is it?
So the assert on line 150 will fail.


why find is with upper case?


Ne, za skoke je se stara.

On Oct 16, 2017 14:11, "MitjaR" <notifications@github.com> wrote:

> Ali je za skoke klasičnega množenja že naložena najnovejša slika? Meni se
> zdi, da ni nisem pa siguren.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/mihic/clanek-matrike/commit/e175907c8b5e117de4f1c57e17609741f7711bf5#commitcomment-25005240>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAeJpiC-jey058kY3F-hNK3XU-MVsKBlks5ss0gLgaJpZM4P6hh8>
> .
>


why as any?


it is already handled 2 lines above


Is this method mapped to the repository? Can't see how this gonna work.


why do we need both database name and database record?


Copy paste I will fix it


I'm thinking about having the `dte` command language eventually compile to Lua bytecode, which should make it considerably smaller when embedded.

Obviously static linking `liblua.a` will more than cancel out the space saving though...


I got compile error when I didn't use it


probably we should catch exception here? even if it fails I think entire operation was successful


Hi,
Have you thought of extending this library to features of [AudioMeterInformation](https://msdn.microsoft.com/en-us/library/windows/desktop/dd368227(v=vs.85).aspx)?


c fo


cc:// @ealonso


Looking further, I can see how the current seqlock routines would work in the single writer case but I think they'd need work to cope with multiple writers. I also don't know if they would need more elaborate fences to protect the work done between the two points.


I tried that, but it is indeed much more difficult to test.
Reasons:

1. The string returned is multi-line. When using a logger outside the function, the log output looks very scrambled which makes it harder to review for humans.
2. Using a new logger inside the function: output is not correctly prefixed. Colors apply, though.

I can think about handing the logger into the function. That should solve the problem. Will try that.


Nope. Milan hard coded the style. I just replaced it by the appropriate class.


удалил данную строчку, ибо она была создана только для теста


TeamCity Selenium POM :: CI [Build 102](https://teamcity.sponte.uk/viewLog.html?buildId=1379&buildTypeId=SeleniumPom_Ci) is now running


TeamCity Selenium POM :: CI [Build 102](https://teamcity.sponte.uk/viewLog.html?buildId=1379&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:38


Yes good catch.


I don't get this where is try here ? no sub calls ca throw ?


![yummy recipe uml](https://user-images.githubusercontent.com/31155081/31611679-c0ab581a-b285-11e7-8eab-1bb1e9985e9f.jpg)


why not EPERM here ?


For me, it's not:
<img width="308" alt="screen shot 2017-10-16 at 15 22 11" src="https://user-images.githubusercontent.com/6651625/31611724-e742795e-b285-11e7-8788-4b0133b43e2f.png">


Please correct this


Oh I see, that's all new type definitions


I don't understand :( Can you explain?


Thanks !


Yes. On 1 column, the subject is mandatory. On multi columns, no.


The commit's message is `add flow annotation`, but you're doing a lot more editing that just adding flow. I think that's what he meant.


Why not use ESP8266WiFiMulti instead? the library does this for you.


I don't know lol. @ayyazdaniaryan?


This `import` statement should go up to the first paragraph because that is a standard library module and not third party. Also there should be two blank lines below since top-level blocks shall be separated by two blank lines according to PEP-8 which I stick to.


To accurately emulate a 4K drive, shouldn't the zone descriptors also operate on logical block boundaries on line 1371 and 1373?


so I bet there is some other issue related to this. What compilation error do you get?


Maybe `{id:var}list`? `list` is a generic word and not suitable to use as translation template.

Is it intended not to apply such change to another `list` in the `controls_for_of` block?


looking to the code with @regiskuckaertz, this feature has been [added 4 years ago](https://github.com/guardian/mobile-apps-article-templates/pull/125).
If found a[n issue for Safari 11](https://github.com/cferdinandi/smooth-scroll/issues/392).


Yes good catch again.


55px looks like a magic number, where does it come from?


@StevenBlack some false positives removals in latest commit. Still a work in progress but I am getting them cleared out.


maybe we can remove the DatabaseName here


weiss nicht


@afgane I'm not sure this will work. What if we want to set the description to an empty string? If we always filter out empty strings, it won't work.


We also need to think of a good test for this, to prevent regressions.


Do you know why, after I have the Plaso app installed in Splunk and import a timeline csv, why the Plaso app doesnt see any of the data? It tells me its waiting for data, but other apps such as Search and Report can see it already.


yes I know but @ayyazdaniaryan told me to use `for/against`. I'm more confident with aryan's english than fred's english ;)


@SLambrakis can you please check your settings to see if "Reduced Motion" is enabled?


Tried it with ak2i_ntrcardhax_flasher_dsi, it showed me `Flashcard setup failed`
I'm using [this cart](http://www.r4isdhc.com/r4-product#a2)


Do you have `allow_url_fopen=1` in php.ini?


@sergiokopplin ❤❤

Thanks!


Salut Sylvie, je regarde ça ce soir en rentrant :)

Le 16 oct. 2017 13:36, "sveyres" <notifications@github.com> a écrit :

> Bonjour Gaëtan,
>
> J’espère ne pas trop te déranger.
> J’ai une interrogation sur du code php. Ayant discuté avec Salwa elle m’a
> dit que tu pourrais peut être m’aiguiller.
> J’ai créé un dépôt sur GitHub avec mon code et t’ai ajouté en
> collaborateur.
>
> Mon problème est le suivant :
>
> Je dois faire un code permettant :
> 1 - de vérifier qu’il y ait bien la variable email dans l’url, sinon ça
> affiche un message.
> 2 - S’il y a la variable dans l’url, vérifier si elle n’est pas déjà dans
> la base de donnée. Si oui, un message.
> 3 - S’il y a bien la variable et qu’elle n’est pas déjà enregistrée dans
> la base de donnée :
> - Afficher le formulaire
> - Enregistrer la variable dans la base de données au submit.
>
> Mon problème étant que je n’ai pas accès à la page où va l’action du
> formulaire.
> C’est un formulaire réalisé par un client de mon lieu de stage.
> Sa fonction étant de récolter les informations du formulaire ailleurs que
> sur wordpress.
> Du coup je ne peux pas ajouter l’action d’enregistrer l’email dans ma
> table.
>
> J’ai pensé à faire du JS mais dans le doute et comme c’est pour un de
> leurs clients. Je ne veux pas faire n’importe quoi et donc je viens
> quémander des conseils.
>
> Si tu ne peux pas, je comprendrais.
>
> Merci malgré tout.
>
> Sylvie
> 06 76 45 31 32 (si tu préfères répondre par tel)
>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/sveyres/wordpress_ch7/commit/1fdde8c90176d686c64a4974ef27501eced0551c#commitcomment-25004658>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAO7xqZQjhoWZTob8KpGRh2SA494qczKks5ssz_LgaJpZM4PG99h>
> .
>


C'est un test pour utiliser Git


Salut Sylvie, je regarde ça ce soir en rentrant :)

Le 16 oct. 2017 13:36, "sveyres" <notifications@github.com> a écrit :

Bonjour Gaëtan,

J’espère ne pas trop te déranger.
J’ai une interrogation sur du code php. Ayant discuté avec Salwa elle m’a
dit que tu pourrais peut être m’aiguiller.
J’ai créé un dépôt sur GitHub avec mon code et t’ai ajouté en collaborateur.

Mon problème est le suivant :

Je dois faire un code permettant :
1 - de vérifier qu’il y ait bien la variable email dans l’url, sinon ça
affiche un message.
2 - S’il y a la variable dans l’url, vérifier si elle n’est pas déjà dans
la base de donnée. Si oui, un message.
3 - S’il y a bien la variable et qu’elle n’est pas déjà enregistrée dans la
base de donnée :
- Afficher le formulaire
- Enregistrer la variable dans la base de données au submit.

Mon problème étant que je n’ai pas accès à la page où va l’action du
formulaire.
C’est un formulaire réalisé par un client de mon lieu de stage.
Sa fonction étant de récolter les informations du formulaire ailleurs que
sur wordpress.
Du coup je ne peux pas ajouter l’action d’enregistrer l’email dans ma table.

J’ai pensé à faire du JS mais dans le doute et comme c’est pour un de leurs
clients. Je ne veux pas faire n’importe quoi et donc je viens quémander des
conseils.

Si tu ne peux pas, je comprendrais.

Merci malgré tout.

Sylvie
06 76 45 31 32 (si tu préfères répondre par tel)

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sveyres/wordpress_ch7/commit/1fdde8c90176d686c64a4974ef27501eced0551c#commitcomment-25004658>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAO7xqZQjhoWZTob8KpGRh2SA494qczKks5ssz_LgaJpZM4PG99h>
.


*Mode (TrackingSpanMode)


Well, it does check at startup but requires the use to redownload it themselves. Maybe in v1.0 we an have self updating Rack.


No. You are directly accessing the column here, not a python property


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/c3c40904-ca0e-4de6-ae72-b5e8ea496054 "simple-toolchain-20171016110647432"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/ea00952d-0688-4284-82e6-03ad43652874 "simple-toolchain-20171016110647432") deployed [simple-toolchain-20171016110647432](https://github.com/OttoCara/simple-toolchain-20171016110647432/releases/tag/deploy-ibm-yp-us-south-otello_caramaschi-it.ibm.com-dev-20171016-123514 "deploy-ibm-yp-us-south-otello_caramaschi-it.ibm.com-dev-20171016-123514") to [dev](https://console.bluemix.net/apps/33003793-6459-4840-ae63-4a907ebd75f7?env_id=ibm:yp:us-south "ibm:yp:us-south:otello_caramaschi@it.ibm.com:dev"), including this commit


can you plz document what exactly this method does?


no capitalized variable names


Super ! Merci beaucoup !

> Le 16 oct. 2017 à 14:33, Gaëtan PRIOUR <notifications@github.com> a écrit :
>
> Salut Sylvie, je regarde ça ce soir en rentrant :)
>
> Le 16 oct. 2017 13:36, "sveyres" <notifications@github.com> a écrit :
>
> Bonjour Gaëtan,
>
> J’espère ne pas trop te déranger.
> J’ai une interrogation sur du code php. Ayant discuté avec Salwa elle m’a
> dit que tu pourrais peut être m’aiguiller.
> J’ai créé un dépôt sur GitHub avec mon code et t’ai ajouté en collaborateur.
>
> Mon problème est le suivant :
>
> Je dois faire un code permettant :
> 1 - de vérifier qu’il y ait bien la variable email dans l’url, sinon ça
> affiche un message.
> 2 - S’il y a la variable dans l’url, vérifier si elle n’est pas déjà dans
> la base de donnée. Si oui, un message.
> 3 - S’il y a bien la variable et qu’elle n’est pas déjà enregistrée dans la
> base de donnée :
> - Afficher le formulaire
> - Enregistrer la variable dans la base de données au submit.
>
> Mon problème étant que je n’ai pas accès à la page où va l’action du
> formulaire.
> C’est un formulaire réalisé par un client de mon lieu de stage.
> Sa fonction étant de récolter les informations du formulaire ailleurs que
> sur wordpress.
> Du coup je ne peux pas ajouter l’action d’enregistrer l’email dans ma table.
>
> J’ai pensé à faire du JS mais dans le doute et comme c’est pour un de leurs
> clients. Je ne veux pas faire n’importe quoi et donc je viens quémander des
> conseils.
>
> Si tu ne peux pas, je comprendrais.
>
> Merci malgré tout.
>
> Sylvie
> 06 76 45 31 32 (si tu préfères répondre par tel)
>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/sveyres/wordpress_ch7/commit/1fdde8c90176d686c64a4974ef27501eced0551c#commitcomment-25004658>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAO7xqZQjhoWZTob8KpGRh2SA494qczKks5ssz_LgaJpZM4PG99h>
> .
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/sveyres/wordpress_ch7/commit/1fdde8c90176d686c64a4974ef27501eced0551c#commitcomment-25005613>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AXx5MKc11SpsaDGzGN_A0Rhg8FX1E4HDks5ss00OgaJpZM4PG99h>.
>


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2620](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45671&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Instead of substringing, you could split the text first and iterate through its words.


Przeczytaj do końca zanim zaczniesz.

1. Rejestrujemy użytkownika gość
2. Tworzymy nową role w metodzie Register w AccountController.
3. Tworzymy użytkownika admin ( z nową rolą ).
4. Usuwamy lub komentujemy kod o nowej roli w AccountController.
5. Tworzymy pustą migrację do bazy danych.
6. Do migracji dodajemy Userów, Role i UserRole.
7. Usuwamy Userów, Role i UserRole z bazy danych.
8. Uruchamiamy migracje.


threadapi-feature.jam: No such file or directory


You don't need a bunch of these libraries, you should delete these.


@eran-stratoscale see https://github.com/Stratoscale/skipper/commit/9af56b7d4c2c8edc9555abcd2da100cd2440946e


How is this different than just adding EPERM to the doGetStat error handling ?
Seems like we want to handle EPERM for any file not just for a directory ?


ooops :D
fixed 4200a56


`for/against` is slightly more casual way of asking. Also, it avoids the British/US english issue with the word "Favor/Favour". If there is ambiguity, we can use the "In Favour/Not in Favour" format, it is not wrong at all, and is more professional.


It get's even more complicated then and the code gets unreasonably ugly, just for two lines of Printf(). I don't even think tests running in parallel use the same instance of the program, and therefore also do not use the same logger instance. `go test -v -race .` did not throw any errors.
Unclear if we are really having a problem here.


It does not return `undefined` but a file stat without any children.


will do .... probably later ... thanks for the reply


Merged with 8d99d5cf7f94393833ba50d2befb2864f08432a6 on staging.


Haa right right thx.

We should still add it the doGetStat in case it's a regular file however ?


this is a code


Fixes #616


Fixed in Object Viewer before the merge, but got missed from the main program / Route Viewer :/


is installed without errors, but the safetynet does not pass
[log.zip](https://github.com/kantjer/MagiskFiles/files/1387431/log.zip)


Done.


week 1 *


Also it's not clear in this code that it's the directly itself that is throwing the EPERM it could be one of the children.. and in this case should return an empty list.


To tell the truth, all these from-to-addr-t casts are ugly in both versions ;-)


Looks like it's problem with a2ki flasher, other people have it too https://github.com/d3m3vilurr/ak2i_ntrcardhax_flasher/issues/23


It should be around readdirSync only and it should the the same errors flags as doGetStat assuming we add EPERM there too...


@karimbahgat I think this change makes problems on current master if `value=None` and field type is `'C'` (in my case: `(fieldName, fieldType, size, deci, value)` is `('OriUncDesc', 'C', 40, 0, None)` inside `def __dbfRecord(self, record)`). On current stable release these values end up as `'None'` in the dbf (which is also not ideal probably) but now I'm getting en exception:
```
../../../../../pyshp/shapefile.py:1147: in __dbfRecord
    value = b(value, self.encoding, self.encodingErrors)[:size].ljust(size)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

v = None, encoding = 'utf-8', encodingErrors = 'strict'

    def b(v, encoding='utf-8', encodingErrors='strict'):
        if isinstance(v, unicode):
            # For python 2 encode unicode to bytes.
            return v.encode(encoding, encodingErrors)
        elif isinstance(v, bytes):
            # Already bytes.
            return v
        else:
            # Error.
>           raise Exception('Unknown input type')
E           Exception: Unknown input type
```
because `v=None`
```


I'm fine with both options - do as you like.


I think I'd misunderstood what the previous behavior was on local tiles, I'll try to figure out how to restore this. The previous set of options was problematic because it can prevent you from ever being able to exit the game, if you have `restart_after_save = true` (that's what led to me to accidentally change this behavior).

Also, when you have `restart_after_game = false` or `restart_after_game = maybe` and a name set, it should exit after C-q, death, or saving, not return to the main menu.


You don't need a bunch of these libraries, you should delete these.


You should return the `Pokemon` object, not just the `maxStrength`.
Otherwise, your solution is nice!


duplicate https://github.com/LiquidRemix/android_device_oneplus_cheeseburger/blob/6d276ad472e346f2c3b90a67dc69d6f63d550c56/system.prop#L131


Why did you set the `int.MinValue` (which is -2,147,483,648) as a default value? :) Use `0` next time.


Created multidev environment [natalie-sandbox#pr-drupal-8](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-drupal-8).

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


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2620](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45671&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 299, ignored: 3 Build time: 00:15:30


fixed


test


Done.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2621](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45679&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


neo xmeter


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2622](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45677&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


А зачем так делать инициализацию? Почему нельзя сразу при объявлении переменных filteredPaths и allowedRolesForConfiguration их бы не проинициализировать?


Why not. Changed.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2623](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45680&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


To be decided


This method shoud be return the Dictionary...
We use void if the method print to console, paint to a canvas, but now we want to get the result.


This field is never used.


Thank you! :)


@zcbenz Cool! Thanks.


We will address this with upstream version. This is the UEK5 version which is 1 phase before upstream one that is based on fd as you wrote in the comments.


According to the specification, your method should take the file path as an input (not the character to search for) and should return with a Dictionary (not an object).


Probably not but who cares?


@rafibomb
$size: 100% doesn't work anymore as the off-canvas component's position mixing has a new if function since Foundation 6.4.2:

```
$sizes: if($orientation == horizontal, $offcanvas-sizes, $offcanvas-vertical-sizes)
) {
  @if $position == left {
    top: 0;
    left: 0;
    height: 100%;
    overflow-y: auto;;

    @each $name, $size in $sizes {
      @include breakpoint($name) {
        width: $size;
        transform: translateX(-$size);
      }
     @}```
So it should be sizes: variables. I have tried using sizes: small 100%, but unfortunately it didn't work.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/b67360b9-c62d-4c3a-8b62-6913af21cd08 "devops-insights-tut201710"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/2acdc53e-e790-4da4-b29c-4679ba1d013b "devops-insights-tut201710") deployed [devops-insights-tut201710](https://github.com/jarkko-koski/devops-insights-tut201710/releases/tag/deploy-ibm-yp-us-south-Jarkko.Koski-fi.ibm.com-staging-20171016-130133 "deploy-ibm-yp-us-south-Jarkko.Koski-fi.ibm.com-staging-20171016-130133") to [staging](https://console.bluemix.net/apps/cabe65e4-37ba-4ad3-9274-9e1d6ad11640?env_id=ibm:yp:us-south "ibm:yp:us-south:Jarkko.Koski@fi.ibm.com:staging"), including this commit


Added code to calculate users input to get an answer to the equation


@mdoering I'd recommend double checking that the 400 error response SanitizeHtmlFilter sends (when XSS is found in the request), is handled as expected.

There is a problem with XssException exception handling - missing in this method signature, called in the XssRequestWrapper constructor: https://github.com/gbif/ipt/blob/01e76cc11e7418685dce47af2bb48560d6d4e5cb/src/main/java/org/gbif/ipt/struts2/SanitizeHtmlFilter.java#L108

Previously I believe XSS errors were handled using a standard error response as per https://github.com/gbif/ipt/blob/master/src/main/resources/struts.xml#L109 I would need to dig deeper but handling the 400 explicitly here could be nice.


I have no idea why is there any condition for printing.


Why do you need this for?


maybe add a comment that this is a tree?


![](https://github.trello.services/images/mini-trello-icon.png) [add view of page](https://trello.com/c/3W43XlnB/5-add-view-of-page)


Hi archana please message me its a very important


Uhm. True. Removed.


test


please understand me


It is a Trie, not a Tree, but I can add a comment.


Yes, it is easy at this point when I am looking a the commit. But 50 commits afterwards it will be hard for anyone else to know about it. It is not so self discoverable and in order to understand what they do we have first to track the location in the code where they are defined and then the location where they are used to have some meaning. I will for sure, forget about them in a few days and will have to do the whole process I described again and again.

Miguel Carvajal
Licenciado en Física
Instituto Balseiro
8400, Rio Negro, AR

On 16 Oct 2017, 06:24 -0300, Pietro Delugas <notifications@github.com>, wrote:
> Dear Miguel
> In this particular case it's very easy to read what you are asking. There are just 3 files changed. For the documentation about the variables just scroll up this page and read file exx.f90 lines 61,62 and 67 to 72.
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.


Please see reply above for previous comment.


it can't get the value here(environment.getProperty(prefix + "[" + i + "]")), so , the first while is not need


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/1af20cf2-bc4e-4aeb-a6fe-ce9abe9238c4 "simple-toolchain-20171016"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/83f03d78-e346-4c67-8962-3119e31193fe "simple-toolchain-20171016") deployed [simple-toolchain-20171016](https://github.com/CristianRuocco/simple-toolchain-20171016/releases/tag/deploy-ibm-yp-us-south-Cristian-Ruocco-Test-20171016-130532 "deploy-ibm-yp-us-south-Cristian-Ruocco-Test-20171016-130532") to [Test](https://console.bluemix.net/apps/97241d58-731f-480c-ac03-16c8459655fe?env_id=ibm:yp:us-south "ibm:yp:us-south:Cristian Ruocco:Test"), including this commit


plz document what this does


It's unnecessary to convert your `string` to a `char[]`, because a `string` **is a** `char[]`.


Tricky solution :)


Why using `8.8.8.8` address? This is one of the Google's DNS server. And yes, it should contain the name of the network interface.

The idea of having an optional configuration sounds good. So feel free to adjust this later.


what does `ls` stand for?


This module is in Boost.Build. You need to update it.


__pokemon_type__ parameter unnecessary, you don't use it in the function


Нет, в потомках ты тестируешь только измененния логики. Все тестировать не нужно


Нет, что тут тестировать?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2622](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45677&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 299, ignored: 3 Build time: 00:13:16


Да, public_send лучше


8.8.8.8 because then you are sure it uses the NIC with the default gateway set. 10.x used now could use an eventual existing LAN only NIC without gateway used in a local 10/8 network.


drop to private method


В принципе вариант, правда возникает вопрос в необходимости этого подстчета. Ну то есть, может быть, проще вообще его было не делать? ;-)


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2621](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45679&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 299, ignored: 3 Build time: 00:15:12


__init__ shoud take type and hp as parameter, so you could use it when you inherit in different classes eg:
bulbasaur:  super().__init__("bulbasaur", 12).


Using this code causes the layout I use (oUF_Neav) to not show a player portrait until something changes. Switching back to "UNIT_PORTRAIT_UPDATE" seemed to fix the problem.


hi bro this file is not exicuting shell


这是我的葫芦娃第三次作业第一次提交，请老师看一下


``` return (is_object($staff) && $staff->getDeptId() == $this->getId()); ```


Maybe rename to `getNode` or `findNode`? Took me a while to understand what's going on here.


Ok I get it. I need to improve how I split my commits and their associated messages.


U don't have to print an emty line in this case


Consider picking a more intuitive name (see below).


speak quote could be a field, so you can set it in __init__ , with inheritence you have to define __speak__ method once


create user and use before


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2623](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45680&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 299, ignored: 3 Build time: 00:15:41


when admin, when standard user - contexts


What's this?


You should return with the strongest `Pokemon` object, not a `string`.


Пусть будет, интересно же.


I have 'Reduced Motion' disabled (the default).


class name is misleading, you construct only one pokemon with it's __init__


1. KNN metric classifier/DatasetProcessing.py

вот тут не обязательно перемешивать, мы делаем перемешивание только когда разбиваем на `train` и `tеst` точки


You should store the strongest `Pokemon` object itself here as a default value.
`var strongest = pokemon;`


For some reason DAO appeared in the main Maven project, removed it, as well as a neatbeans configuration file from a module, not needed.


Should pass tests when travis IMP is updated


I don't fully understand why you need this struct. Couldn't you just use `node` directly instead?


You are returning with the name of the integer type here :)


@Grimsbain Is it 3D or 2D?


code duplication and magic chars, this could be separate functions, and should able to handle every character, not just 3


This `Speak` method shouldn't have any arguments.


@p3lim 2D


also you dont use file_name parameter


Checked iCloud Keychain on Mac for its existence - it does exist.


@mdoering I believe this new method is the culprit. Debugging, I trigger the following exception

```
Caused by: javax.ws.rs.core.UriBuilderException: java.net.URISyntaxException: Illegal character in query at index 35: http://localhost:8080/ipt/about.do?""" style="background:url(javascript:alert(6268))" "=
	at com.sun.jersey.api.uri.UriBuilderImpl.createURI(UriBuilderImpl.java:723)
	at com.sun.jersey.api.uri.UriBuilderImpl._build(UriBuilderImpl.java:651)
	at com.sun.jersey.api.uri.UriBuilderImpl.build(UriBuilderImpl.java:641)
	at org.gbif.ipt.action.BaseAction.getRequestURL(BaseAction.java:119)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at ognl.OgnlRuntime.invokeMethod(OgnlRuntime.java:897)
	at ognl.OgnlRuntime.getMethodValue(OgnlRuntime.java:1464)
	... 144 more
```


Elisa hihihihih


Whoops! 😁


This should be `int strength = random.Next(1, 11);` to allow `10` as an option as well.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/c3c40904-ca0e-4de6-ae72-b5e8ea496054 "simple-toolchain-20171016110647432"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/ea00952d-0688-4284-82e6-03ad43652874 "simple-toolchain-20171016110647432") deployed [simple-toolchain-20171016110647432](https://github.com/OttoCara/simple-toolchain-20171016110647432/releases/tag/deploy-ibm-yp-us-south-otello_caramaschi-it.ibm.com-DevOpsPracticesPrototypes-20171016-131653 "deploy-ibm-yp-us-south-otello_caramaschi-it.ibm.com-DevOpsPracticesPrototypes-20171016-131653") to [DevOpsPracticesPrototypes](https://console.bluemix.net/apps/61986195-2c86-4182-bf16-8089c7b10d18?env_id=ibm:yp:us-south "ibm:yp:us-south:otello_caramaschi@it.ibm.com:DevOpsPracticesPrototypes"), including this commit


@lmagder I've successfully intercepted method call on ubuntu (message sent using dbus-send tool). I've managed to check how does the dbus configuration on CMU looks like and it allow eavesdropping the method calls, so it should work in the car also.
There is one thing to fix: a segmentation fault error when starting headunit having phone connected. It breaks on releasing dbusFilterFunc pointer. I did not check if the same applies to car executable.


indeed. The BaseAction.getRequestURL method fails with the following which should be caught:

Caused by: javax.ws.rs.core.UriBuilderException: java.net.URISyntaxException: Illegal character in query at index 31: http://localhost:8080/about.do?""" style="background:url(javascript:alert(6268))" "=
	at com.sun.jersey.api.uri.UriBuilderImpl.createURI(UriBuilderImpl.java:723)
	at com.sun.jersey.api.uri.UriBuilderImpl._build(UriBuilderImpl.java:651)
	at com.sun.jersey.api.uri.UriBuilderImpl.build(UriBuilderImpl.java:641)
	at org.gbif.ipt.action.BaseAction.getRequestURL(BaseAction.java:119)
	at sun.reflect.GeneratedMethodAccessor77.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at ognl.OgnlRuntime.invokeMethod(OgnlRuntime.java:897)
	at ognl.OgnlRuntime.getMethodValue(OgnlRuntime.java:1464)
	... 141 more
Caused by: java.net.URISyntaxException: Illegal character in query at index 31: http://localhost:8080/about.do?""" style="background:url(javascript:alert(6268))" "=


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131838 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131838") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


first version


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131844 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131844") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20171016-131850 "deployed-2-env-Prod-20171016-131850") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131859 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20171016-131859") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


I find this, and the `weston_remoting_api::create_outputs(compositor, number)` function, perhaps a little awkward, mainly because of the `sscanf()` call. Instead, I would suggest to have `weston_remoting_api::create_output(compositor, name)`. Then `load_remoting()` could loop over the config sections and for each "remote-output" section, call `create_output` with a name from the config. That way `drm_backend_remoted_output_configure()` could use `weston_config_get_section()` to find the right section by the name. This approach would be much closer to the existing `load_x11_backend()` and `load_wayland_backend()`, except that you don't have to count outputs and you don't have to match name prefixes because the section name "remote-output" already does that for you.

I remember I was ambivalent about this before, but now that I see the code, a specific solution is starting to feel more attractive to me. What do you think?


To me "ip" would imply a numerical address, but I assume you intend to support also host names. How about "host" instead?


@p3lim I just check and the current code works with 3D turned on (I normally don't use it.) but "UNIT_PORTRAIT_UPDATE" worked with both.


Would it not be better to require host? What value would you use if host is not given?
I suppose it is easier to come up with fallback values for port and bitrate.


This should check if there actually is `remoting_api` before using it. It will be NULL if `weston.ini` has no remote output sections.


I left some comments about the `create_outputs()` API above.


The very first thing in every `.c` file should be `#include "config.h"`. It sets up some defines that the system and other headers use.

This file is also missing the copyright and license blurb. Please copy the license blurb from `libweston/compositor.c` and replace the copyrights. This applies to all new code files you add.


I would like these to be called `saved_destroy`, `saved_enable` and `saved_disable` or similar, to convey that they are saved values from elsewhere.


This can never happen.


Need to `wl_list_remove(&remoting->destroy_listener.link)` so that the caller of this function does not hit use-after-free.


This is exactly what I referred to, very good.


It would be better to store the `api` pointer in `remoted_output` or `weston_remoting`. There is no need to look up the API every frame and you avoid handling an error case.


Lookup looks ok.


Sure, but I'm not sure about what is the issue: is that the fact that SERVER['HTTPS'] is not set?

It looks like behind load balancer or proxy, Server https is not set most of the time. I'm testing this on cloud9 as well as locally.
Could we use this instead? : $_SERVER['HTTP_X_FORWARDED_PROTO']


This failure path should probably call `remoted_output->saved_disable()` and return an error. The caller needs to know it failed.


The pointer is already in `remoting`.

Yes, this is the wrapping I suggested. Ok.


This should ensure that `output` really is a remoted output, and fail otherwise. A libweston user could make the mistake of calling this for a wrong output.

A libweston user might also call this function multiple times for the same output. That is not really handled below, the modes will just accumulate in the list and they all are flagged as "current".


The unit is milli-Hertz, not micro-Hertz.


`free(remoted_output->ip)` first in case it was already set.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/b67360b9-c62d-4c3a-8b62-6913af21cd08 "devops-insights-tut201710"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/2acdc53e-e790-4da4-b29c-4679ba1d013b "devops-insights-tut201710") deployed [devops-insights-tut201710](https://github.com/jarkko-koski/devops-insights-tut201710/releases/tag/deploy-ibm-yp-us-south-Jarkko.Koski-fi.ibm.com-staging-20171016-132206 "deploy-ibm-yp-us-south-Jarkko.Koski-fi.ibm.com-staging-20171016-132206") to [staging](https://console.bluemix.net/apps/cabe65e4-37ba-4ad3-9274-9e1d6ad11640?env_id=ibm:yp:us-south "ibm:yp:us-south:Jarkko.Koski@fi.ibm.com:staging"), including this commit


As the very first thing, this function should probably check the virtual output API is available. If it's not, there is no way this plugin can work.


This doesn't have any affect to the array, the ToString() return a string that represents the object.


Are you sure about owner-group separator syntax? I would replace your point by a colon:

`chown -R ${NAGIOS_USER}:${NAGIOS_GROUP} "${NAGIOS_HOME}/etc/htpasswd.users"`

Reference: http://manpages.ubuntu.com/manpages/trusty/man1/chown.1.html


This declaration should not be necessary, there is already one in `compositor.h` common for all generic libweston plugins.
Other than that, this header file looks good, sans license blurb.


should be fixed with https://github.com/gbif/ipt/commit/849f1d70bca847762343cbcc11fe7390ddc6cd3d


This seems to be only active in our farm mode, where the 2nd serial line is active for communication and the block you mentioned just echoes the data received from the 2nd serial line into the 1st serial line.


Funções - Huanna


In the README:MD there was:
"What is inheritance? When would you use it? [2p]

Your answer:

[add your answer here]"

Why create 2 new files for answer?


Hi Esaki-san,

the two patches compile nicely without warnings, and the coding style is very much the Weston style, excellent!

When adding non-trivial portions of code, you can add your copyright line to the copyrights at the top of the file if you wish. Official patches should also have Signed-off-by line in the commit message, the meaning is the same as in the Linux kernel.

Again almost only minor comments, the code structure is looking good.

It could be nice to add text file explaining how this plugin is used. Maybe the plugin should be in a new source code directory as well, there will be much more code in the future once we get to Waltham and input.


@mdoering - please revert this particular change


tag 用例怎么挂了。。


controller layout, controller namespace


TeamCity Selenium POM :: CI [Build 103](https://teamcity.sponte.uk/viewLog.html?buildId=1380&buildTypeId=SeleniumPom_Ci) is now running


Don't follow, can you elaborate more on this?


TeamCity Selenium POM :: CI [Build 103](https://teamcity.sponte.uk/viewLog.html?buildId=1380&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:38


第15行多了个空格：
```
'&categoryId=10000000&sortId=5&sin= ' + offset + '&ein=' + (49 + offset);
```
应改成：
```
'&categoryId=10000000&sortId=5&sin=' + offset + '&ein=' + (49 + offset);
```


This program asks the user for his name and his age. Then it displays how old will the user be next year.


TeamCity Selenium POM :: CI [Build 104](https://teamcity.sponte.uk/viewLog.html?buildId=1381&buildTypeId=SeleniumPom_Ci) is now running


王启凡的个人博客


Is pip really required?


Здорово


Зачем тебе эти текстовые переменные? Ты вполне можешь использовать локальные переменные в методах.


Тут стоило своим классами наследникам дать какие-то свои поля, иначе получается в них нет смысла. Но идея правильная. Ставлю 9 за всё ДЗ


TeamCity Selenium POM :: CI [Build 104](https://teamcity.sponte.uk/viewLog.html?buildId=1381&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:07
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

```


I'm gonna kill someone for convincing me to use UMC instead of UPU :grin::gun:


[Test](http://overfront.netne.net/Audio/) Online


Нет, тут должен быть выбор из уже созданных станций, и из них мы создаем маршрут


put it in the support folder


Тут достаточно локальной переменной


why?


Зачем формировать массив и возвращать его? У тебя есть массив @routes, тебе нужно на экран его вывести  с номерами, затем запросить номер у пользователя и взять из него же нужный маршрут по номеру.


These mutex locks have already been introduced in https://github.com/orocos-toolchain/rtt/commit/58cb4bb733fc14de0ca520620cfcb689e2155797 as part of #91, but removed again in b9f2f348959216f32819967d1b49910deeb4bed6 after a discussion in https://github.com/psoetens/rtt/pull/3#issuecomment-280170267.

We observed priority inversions with the original patch because it introduced locks between components calling or sending operations or writing to event ports and the "receivers" of these events. The way these mutexes are used in the ExecutionEngine and Activity implementation *should* make sure that the processor either already processed the message when the condition is broadcast or is unblocked at least once afterwards. But I cannot exclude that there is a potential race condition at the moment and would have to wrap my head around it again. Intuitively, locking the mutex only to broadcast the condition variable does not introduce any additional guarantees. If there is no simple way to avoid locks completely, then the sender has to already hold the mutex before enqueuing a message in the queue in [ExecutionEngine::process()](https://github.com/orocos-toolchain/rtt/commit/3345b989164f1f9446cac0d0ab7192498fc7f0ad#diff-9dbf4334fae7dbc810530fc0126e9309L254).

Does Xenomai explicitly forbid to broadcast a condition variable without holding the associated mutex?


What is dis?


То же, что и с get_route_list, не нужно делать новый массив


Пробел перед открывающей фигурной скобкой не нужен


This is the first program from week 4


test with sign_me_in method


Не нужно удалять станцию. Если удалили ее из маршрута, то она может позже быть добавлена в другой маршрут.


Зачем эта проверка? Можно же просто containsKey использовать, его достаточно


fixed


Зачем перебирать все станции и искать по имени, если у тебя метод delete_mid_station и так удалит станцию? А если ее нет, просто ничего не произойдет.


テストなので…


TeamCity Selenium POM :: CI [Build 105](https://teamcity.sponte.uk/viewLog.html?buildId=1383&buildTypeId=SeleniumPom_Ci) is now running


@bgruening I am working on this already in PR #191


改一下


Эти две переменные и объявлять можно внутри цикла, так как они вне него не используются


The problem was that mocha depends on it.


Отлично. 9,5


TeamCity Selenium POM :: CI [Build 105](https://teamcity.sponte.uk/viewLog.html?buildId=1383&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:04
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

```


wow, such message


oh my god ,who can teach me how to play


Лучше переменную создавать типа интерфейса:
```
Map<Character, Integer> letterRepeat = new LinkedHashMap<>();
```


There's DEFAULT_SCRIPT_NAME constant, can it be used?


There are a few things which need to be fixed:
1. The CMD2 module should not be translated in the `sermon-manager-for-wordpress` text domain but in the module’s text domain, if at all.
2. Spanish uses guillemets `« U+00AB` (`«`) and `» U+00BB` (`»`) as primary quotation marks.
3. Translations of plurals are missing.
4. The `You are running <strong>PHP %s</strong>, but Sermon Manager requires at least <strong>PHP %s</strong>.` message id is deprecated/does not exist.
5. Preferably, all non-ASCII characters should be encoded as HTML character entities.
6. Although I do not speak Spanish this translation reads to me like an unreflected word by word translation which is often okay to do between English and Spanish — because both languages are part of the same family of languages — but sometimes you cannot or should not do that.
7. The `Language:` header suggests that it is a translation for the `es_CL` locale but it has been committed as `es_ES` locale. Which one is it? If there is nothing region specific in the `es_CL` locale then it should probably or could be taken as the base `es` locale, for which effectively the `es_ES` locale is the fall back locale for Spanish (unfortunately WordPress handles fall back or region agnostic locales incorrectly).

Besides, this is an odd way to review stuff. You have deviated from the preferred work flow. Review should be done in pull requests **before** they are merged. This is what they are for: To make modofications before they are merged into the default or `master` branch. Otherwise, if you apply fixes to commits on the default or `master` branch you are making it a work or feature branch. Hence, I am unable to suggest any changes to this now.


Пробел после =


The method name says _empty_ so it's inline with that... we cloud special case an empty string though:
`... if v == '' or v`?


Гуд. 9,5


Servono tutti questi prefix? Sia `-moz` che `-webkit` sono inutili sui browser dopo il 2012. Il `filter` può servire se si vuole supportare perfettamente IE8-9.

[link a caniuse](https://caniuse.com/#feat=css-gradients)


TeamCity Selenium POM :: CI [Build 106](https://teamcity.sponte.uk/viewLog.html?buildId=1384&buildTypeId=SeleniumPom_Ci) is now running


Bluemix [toolchain](https://console.stage1.bluemix.net/devops/toolchains/ee7af0c7-eb4b-4808-84ae-65a1ebf892f4 "UI-ys1int-KEEP-TC"): [Delivery Pipeline](https://console.stage1.bluemix.net/devops/pipelines/91fde061-3564-4ded-a29e-3c930cc52871 "UI-ys1int-KEEP-TC") deployed [UI-ys1int-KEEP-TC](https://github.com/idsb3t1/UI-ys1int-KEEP-TC/releases/tag/deploy-ibm-ys1-us-south-idsb3t2-us.ibm.com-dev-20171016-134051 "deploy-ibm-ys1-us-south-idsb3t2-us.ibm.com-dev-20171016-134051") to [dev](https://console.stage1.bluemix.net/apps/ba06642c-6d24-4099-84a8-d95b6a78f1a2?env_id=ibm:ys1:us-south "ibm:ys1:us-south:idsb3t2@us.ibm.com:dev"), including this commit


local variable


Personalmente mi pare troppo alto (soprattutto sul portatile). In ogni caso bisogna aggiustare il javascript per la sticky nav, perché adesso "salta".


You don't need a bunch of these libraries, you should delete these.


TeamCity Selenium POM :: CI [Build 106](https://teamcity.sponte.uk/viewLog.html?buildId=1384&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:03
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

```


Отлично. 10


fixed


Не используй импорты со звездой. Непонятно, что конкретно импортится. Лучше настройки в идее поменять


That's almost a perfect solution :) The only small difference between the specification and your solution is that this method should take the file path as a parameter, so the file reading part should be called from this method, not from the `Main`.


good, I have only my mobile around :(


True, the method name is not great - but Boto only really cares about None params as far as I've gathered. Since this is a top level method, it seems risky to filter out all empty values here - e.g. what if v is a single empty space. I think the root bug lies in cloudlaunch, which is sending a dict instead of a string as the user_data is supposed to be.


Гуд. Всё верно и аккуратно. 10


Может просто:

```
return garage.get(car);
```
?


lmao


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


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

Offending files: js/sql.js


You don't need to convert your `string` into a `char[]`, because a `string` **is a** `char[]`.


Это неправильно. Метод должен возвращать сформированную строку, а не печатать в консоль


@BitDen Use

```
const {translators, initialState}  = require('../../_shared.js');
```

instead! And then

```
let expectedState = initialState.clone();
```


You could use `+=` here to simplify this line.


Promise does not work with Internet explorer.

I see you added `promise-polyfill`. Is it for that ? How to use it, plz ?


Стоило в классы-наследники добавить поля. Иначе эти классы бесполезны


TeamCity Selenium POM :: CI [Build 107](https://teamcity.sponte.uk/viewLog.html?buildId=1385&buildTypeId=SeleniumPom_Ci) is now running


Would be better if u count with the dictionary's value, BTW it works too


За всю ДЗ 8


<title>¡Largo de mi rama, plebeyo!


TeamCity Selenium POM :: CI [Build 107](https://teamcity.sponte.uk/viewLog.html?buildId=1385&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:02
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

```


close() must be call, for free server side resources, so I could remove only `metaData.close()` and related.


@saradickinson Is this the correct spot for _getdns_upstream_reset()?
I think it might need to move one line up to only reset when the fd is not writable?  Otherwise there is not really a point in testing for that, right?


FYI, dte doesn't need a C11 compiler. If `$CC` doesn't support `-std=gnu11` it will fall back to `-std=gnu99` (and disable C11 `static_assert`).


Why it's not specified for a particular file path? `endsWith()` looks suspicious.


TeamCity Selenium POM :: CI [Build 108](https://teamcity.sponte.uk/viewLog.html?buildId=1386&buildTypeId=SeleniumPom_Ci) is now running


Одного пробела перед char хватит


Отлично. 10


TeamCity Selenium POM :: CI [Build 108](https://teamcity.sponte.uk/viewLog.html?buildId=1386&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 10 Build time: 00:01:06
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.FrameTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

```


not bad idea, but now it is unnecessary, it sorts the lines of the file not the characters :/


This method should be in your `Pokemon` class and it shouldn't have any arguments.


Awesome, thank you! 🎉


This method should return the strongest `Pokemon` object itself, not its type.


Why are you feeding in `ttl` here? Please document.


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


unnecessary empty lines, why? styleguide...


update file with a little comment


Ah ok...so pretty off mark I was. Alas. However, thank you generously for
that clarification. Cheers

On Oct 16, 2017 12:10 AM, "Bradley Sepos" <notifications@github.com> wrote:

> We generally use that term to mean point-to-point in HandBrake—any video
> encode where the start or stop point is in the middle of the source. So,
> for example encoding from 30 seconds into a source, to 90 seconds, would
> yield a 60 second encode from the middle of the source. We call that p-to-p.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/HandBrake/HandBrake/commit/5cc544302e6596239ff01ef3b35502a013d67de8#commitcomment-24997829>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AULfDZXTaeZqTNTpxHqKk6O1CPlP3UPyks5sstcqgaJpZM4P6ATe>
> .
>


@evilaliv3 please review. These are the strings included in the browser-crypto branch to handle frontend key generation, decryption failures and password changes.

I think we can cut the last two strings without a problem because they are too specific.


Are there any standard imports?


However it's a beautiful solution, you should aware lamba functions till you don't understand them completely :)


this function call returns the following error:

ERROR: fixed.optimize is not a function
    at eval (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:9216:35)
    at AST_SymbolRef.eval [as optimize] (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:5313:23)
    at Compressor.before (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:5296:24)
    at AST_SymbolRef.eval [as transform] (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:2955:35)
    at eval (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:3108:33)
    at Compressor.before (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:5291:9)
    at AST_ObjectKeyVal.eval [as transform] (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:2955:35)
    at eval (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:2974:25)
    at doit (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:134:23)
    at MAP (eval at <anonymous> (/usr/local/lib/node_modules/uglify-js/tools/node.js:21:1), <anonymous>:160:52)


printlns?


TeamCity Selenium POM :: CI [Build 109](https://teamcity.sponte.uk/viewLog.html?buildId=1387&buildTypeId=SeleniumPom_Ci) is now running


Is commented code needed for reference?


This way the possible strength will be between `0` and `9`. You should use `random.Next(1, 11);` instead.


❤   :heart: :heart: :heart:


This is by Intention: https://golang.org/doc/effective_go.html#Getters


nice 🥇


Btw, please stop abusing the `master` branch as your development branch. Do your development on the `dev` branch or on per feature branches. As you can see now, you have introduced *ugly* commits again into the revision history. If you do not care about the revision history then you do not need a version control system at all. One important reason for having a version control system in place is to have a comprehensible history of modifications which is easy to read. If you have to read about fixes of fixes in the commit chain then it is more confusing than clarifying. Just follow the guidelines of the GitHub guys. They know what they are doing and saying. Commits should be *crafted* not dumped.


TeamCity Selenium POM :: CI [Build 109](https://teamcity.sponte.uk/viewLog.html?buildId=1387&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 45 (43 new), passed: 13 Build time: 00:02:29
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.ui.AnimationsTest.waitsForElementToStopMoving: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.refreshingModelReducesNumberOfItems: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.refreshingModelReducesNumberOfItemsForPageElements: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.canRefreshModel: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.returnsNewElementsAfterInvalidateWhileAccessingOriginalItemsFirst: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.returnsNewElementsAfterInvalidate: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.returnsNewListWithLessElementsThanOriginalAfterRemovingItems: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingWithoutImplictInvalidateTest.returnsMultipleNewElementsAfterInvalidateWhileAccessingOriginalItemsFirst: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest.refreshingModelReducesNumberOfItems: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest.refreshingModelReducesNumberOfItemsForPageElements: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.CachingTest.canRefreshModel: <no details avaliable>

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


Done with https://github.com/gbif/ipt/commit/fcbbb6edec0c34b408338f495c159992b7d8c802


One single, simple pokemon, not many of them


Не надо закрывать, закроется автоматически блоком try-with-resources


If I understand the code correctly you could eliminate `idx` and just pass on `ls[1:]` to the recursive incovation


Аналогично


Will push it later to the main repo.


It just occurred to me that you didn't change the `rdfs:label` but the `rdfs:comment` for this property. Thus, the `rdfs:label` remains the same despite the fact that I updated 'demo'.


Hello!


Можно было оставить только IOException блок, так как FileNotFoundException от него наследуется, а обрабатываешь ты их одинаково


[R/methods-aggregateReplicates.R:46:9:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-aggregateReplicates.R#L46) *style:* **TODO comments should be removed.**
```r
        # TODO Slot `sampleNameAggregate` into `sampleMetadata()`
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/methods-knownMarkersDetected.R:44:19:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-knownMarkersDetected.R#L44) *warning:* **Use `[[` instead of `$`  to extract an element.**
```r
    if (!any(known$ensgene %in% all$ensgene)) {
                  ^
```
[R/methods-knownMarkersDetected.R:44:36:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-knownMarkersDetected.R#L44) *warning:* **Use `[[` instead of `$`  to extract an element.**
```r
    if (!any(known$ensgene %in% all$ensgene)) {
                                   ^
```
[R/methods-knownMarkersDetected.R:56:9:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-knownMarkersDetected.R#L56) *style:* **TODO comments should be removed.**
```r
        # FIXME Need to update this step
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/methods-sampleMetadata.R:28:1:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-sampleMetadata.R#L28) *style:* **Lines should not be more than 80 characters.**
```r
                # `description` is missing in some older bcbio objects because we
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/methods-sampleMetadata.R:29:1:](https://github.com/hbc/bcbioSingleCell/blob/36045a36ad86d01e8a1dd1aaae243cd101f20b99/R/methods-sampleMetadata.R#L29) *style:* **Lines should not be more than 80 characters.**
```r
                # used `fileName` and `sampleName` initially to define the minimal
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Vivek

COPY python_mysql.py /root/bot/python_mysql.py

would be better.

It's in the same repo, we don't need http download. Also different branch/tag may build different docker images.


Отступы + исключения (выше писал)


Можно не закрывать


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.445.6cb7f65c168a11c4cca8d19ea350781bf842d6fe](http://build.palaso.org/viewLog.html?buildId=100184&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Failed to start build Build time: 00:00:04


OK, I understand.

I was able to recreate the problem locally, but I could easily fix it without the try-finally-clean procedure by marking the thread a daemon thread. Wouldn't that be a simpler solution?

https://github.com/mmarquee/ui-automation/pull/57

Pascal


TeamCity libpalaso :: palaso-win-master Continuous [Build 4.0.131.6cb7f65c168a11c4cca8d19ea350781bf842d6fe](http://build.palaso.org/viewLog.html?buildId=100187&buildTypeId=Libpalaso_PalasoWinmasterContinuous) outcome was **FAILURE**
Summary: Failed to start build Build time: 00:00:03


TeamCity Selenium POM :: CI [Build 110](https://teamcity.sponte.uk/viewLog.html?buildId=1388&buildTypeId=SeleniumPom_Ci) is now running


Можно же вот так:

```
return Integer.compare(firstNumber.length(), secondNumber.length());
```


Fixed in 88e9c2d


@nelsonvarela So this information is not lost, could you please create a new Issue for this bug with a self-contained standalone JS test case and uglify options?


this give number between 0-9
each time I want to get/check the pokemon's strenght it will become weaker or stronger


A sketch of the user interface design of the yummy recipe.
contains the registration form, where the user will enter the required details for registration of their accounts.
User interface design has been built with html/css/js using boostrap framework


Вместо:

```
hashMap.keySet().contains(
```

можно использовать:

```
hashMap.containsKey(
```


:+1: We can merge / deploy / break tonight.


TeamCity Selenium POM :: CI [Build 110](https://teamcity.sponte.uk/viewLog.html?buildId=1388&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 53 Build time: 00:02:30
### Failed tests
```
uk.sponte.automation.seleniumpom.tests.DependencyInjectionTest.canUseMyOwnDependencyInjection: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.BritishAirwaysTest.useOfCalendars: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest.timesOutWhileWaitingForElementToBecomeVisible: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest.timesOutWaitingForAnElementToBeGone: <no details avaliable>

uk.sponte.automation.seleniumpom.tests.ui.PageTest.timesOutWhileWaitingforElementToBeHidden: <no details avaliable>

```


Стоило во все классы-наследники добавить доп поля, иначе нету смысла в них никакого


This looks really weird. It might be just me, but I'd recommend avoiding such unusual constructions.
I guess this should invert the condition but quick googling lead me to nowhere.


Bluemix [toolchain](https://console.stage1.bluemix.net/devops/toolchains/5453a732-962a-4b83-856e-5e2c6a95b124 "UI-ys1int-SimpleTC1016T100114"): [Delivery Pipeline](https://console.stage1.bluemix.net/devops/pipelines/67705def-95a1-4c03-89eb-21ee6ab0d415 "UI-ys1int-SimpleTC1016T100114") deployed [UI-ys1int-SimpleTC1016T100114](https://github.com/idsb3t1/UI-ys1int-SimpleTC1016T100114/releases/tag/deploy-ibm-ys1-us-south-idsb3t3-us.ibm.com-dev-20171016-140722 "deploy-ibm-ys1-us-south-idsb3t3-us.ibm.com-dev-20171016-140722") to [dev](https://console.stage1.bluemix.net/apps/8539b7ba-ab11-4d99-80bd-a1c2f8126eda?env_id=ibm:ys1:us-south "ibm:ys1:us-south:idsb3t3@us.ibm.com:dev"), including this commit