Created multidev environment [pantheon-d8-composer-test#pr-jeff-slo](https://dashboard.pantheon.io/sites/957d97ea-0393-4d17-a25e-bcedd59daae8#pr-jeff-slo).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-jeff-slo-pantheon-d8-composer-test.pantheonsite.io/)


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.583.8c49832ecca41eab29984b6b66df3df835511cf9](http://build.palaso.org/viewLog.html?buildId=100637&buildTypeId=bt223) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.586.3ee1c8dc1442267b6359a0870e4333b1aeae857b](http://build.palaso.org/viewLog.html?buildId=100640&buildTypeId=bt223) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:08


I made decoding utf8 strings a little more efficient, by not uselessly reallocing as much.


Missing an empty line about "GC:".


In general if someone makes a change I assume they do that change
consciously. Tests help documenting and verifying assumptions, checking
edge cases etc, and help reviewers to make determinations about the change.

It is not all that possible that initial assumptions were incorrect and, as
a result, the implementation is incorrect, but with tests it may be easier
to see that than in absence of tests.

If I need to make a change in an area I do not have good knowledge about,
presence of tests can provide me with a degree of confidence that my
changes do not introduce regressions.

On 23/10/2017 8:33 AM, "Gerhard Olsson" <notifications@github.com> wrote:

> No existing tests, no tests added. I believe there are some a-b diff swaps
> currently, but not sure where....
>
> What test would really add something here?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/gerhardol/gitextensions/commit/7234d28588cad4b22f3dbc6ad58d8542449904c7#commitcomment-25122683>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEMyXqAxA16etCh14SBWKdAZf961Q_fzks5su7SugaJpZM4QCG0S>
> .
>


Here is a build to this point

magnet:?xt=urn:btih:f6c79c670a717087c92ca983237bbf399ce608de&dn=gargoyle-ispy%202017-October-23%2002_24.7z&tr=udp%3a%2f%2ftracker.openbittorrent.com%3a80&tr=udp%3a%2f%2fopen.demonii.com%3a1337&tr=udp%3a%2f%2ftracker.coppersurfer.tk%3a6969&tr=udp%3a%2f%2ftracker.leechers-paradise.org%3a6969

I would describe this build as touchy, sometimes stuff works, sometimes it doesn't.

I try to make a procedure for known problem then it works perfect.

Need somebody else to confirm my observations though

Wait for the next build


Oh, and an empty line here.


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.584.b1b22c713dc97c835958e1d6f006e51283862c9c](http://build.palaso.org/viewLog.html?buildId=100638&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3359, ignored: 129 Build time: 00:02:15


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.585.49f789901e7c6cbd4d90f1c462ef07ab7acf0aa2](http://build.palaso.org/viewLog.html?buildId=100639&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3359, ignored: 129 Build time: 00:02:19


Should be "libs", right?


Hi @jannes-m  - what do you think about introduction to the scientific notation?


I used the equivalent when calculating duration for a timer or histogram. Typically I would have used the `record(Runnable)` signature, but in the cases I had I had to wrestle with some checked exceptions (which Runnable doesn't handle) and wanted to minimize the code restructuring.

Also I typo'd the math (had it backwards) so having a util kept me screwing it up again.


@rayane
Oui j'ai testé ton code et j'arrive à bien mettre le mm mail plusieurs
fois.

Le 21 oct. 2017 20:36, "rayaneayoubi" <notifications@github.com> a écrit :

> bonsoir Moussa j`ai recu cet email mais j`ai pas compris quoi faire? ?? :$
>
> bonsoiree.
>
> ________________________________
> From: Moussa Camara <notifications@github.com>
> Sent: Saturday, October 21, 2017 6:36:58 PM
> To: mouskito/poec-agefos
> Cc: rayaneayoubi; Author
> Subject: Re: [mouskito/poec-agefos] tp (29ce142)
>
>
> Le mail doit etre unique:
>
> mouski | sqdd | test@test.fr<mailto:test@test.fr> |
> | mouski | sqdd | test@test.fr<mailto:test@test.fr> |
> | mouski | sqdd | test@tedsdsst.dd<mailto:test@tedsdsst.dd> |
> | mouski | sqdd | test@tedsdsst.dd<mailto:test@tedsdsst.dd>
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub<https://github.com/
> mouskito/poec-agefos/commit/29ce14256db131a14a93b74726017b
> 48910200dc#commitcomment-25109121>, or mute the thread<https://github.com/
> notifications/unsubscribe-auth/Ae-iibFP040dNq4LpwOuyCvYvThGKcpNk
> s5sug-agaJpZM4QBmq9>.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/mouskito/poec-agefos/commit/29ce14256db131a14a93b74726017b48910200dc#commitcomment-25110167>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AI548wJvSKdoImZMPrRAzpihnI1q_1OMks5sujmQgaJpZM4QBmq9>
> .
>


All the code in Autonomous after init HW and before wait for start could be moved to the HW init method in the HW class.
Last code update was not saved, missing
	New timer setting for sensing
	new color ranges
	comment out of lift block
	restructure of while loop for hitting jewel

References in TeleOp to hardware can be replaced with robot. references


## makeCacheMatrix creates a special matrix object, and then what the cacheSolve does is it


Oye no, el `x = NULL` y `y = NULL` eso no había que borrarlo :octocat:


如果知道值只能是true或者false的话，没必要在if利用equal判断
如果要用的话最好用triple equal。


Wouldn't this remove all zones and markers if at least 1 obj is defined?


Please use `R.utils::withTimeout()`; the `evalWithTimeout()` is there for historical reasons but should be formally deprecated (https://github.com/HenrikBengtsson/R.utils/issues/76)


Verified by Louise


You and your cargo cult programming... :|


I starting to wonder if my router is faulty?


wifi_on의 값이 0과 1만 존재할 예정이면 boolean 형이 좋습니다.


This is wrong if motor number is bigger than 8.


Nice!


Same.


Any plans to support `PERMISSIONS.ACCESS_COARSE_LOCATION` ? :)


trocar para ricardobresolin,pythonanywhere.com


![](https://github.trello.services/images/mini-trello-icon.png) [Reply to comment: ‚cancel‘ on same height as ‚submit‘ button](https://trello.com/c/dgQ5RWtd/22-reply-to-comment-cancel-on-same-height-as-submit-button)


상수값을 오른쪽에 쓰는 것이 일반적입니다.
setor == FAT_...


Ah, yes, that is correct, v1.1.4 does have a `dist` directory.  Yet, after cleaning the cache and installing afresh (and confirming the `dist` directory is there with the built files) I get this error.
```
ERROR in (webpack)-dev-server/client/socket.js
Module not found: Error: Can't resolve 'sockjs-client/dist/sockjs'
```


OBS: deixei a instancia como unicornio só para teste


while 문 내에서 count가 감소하기만 하고 증가하지는 않는데 맞나요?
그렇다면,  count <= 32의 조건은 while문 앞에서 이런 경우에는 리턴하게 해주는 것이 좋습니다.

 while문에 걸린 조건은 while 내에서 변하는 값들입니다.


Too late homie


You can just use the standard Array.filter here


import Requests


@slate71 what's the dir structure of `node_modules/sockjs-client` look like?


Actually this was important. Not putting it as part of the big `if-else` chain is what lets the egg validator in aforementioned `if-else` chain actually go into effect.


El controlador se creó usando el comando:
`php artisan make:controller PagesController`


@cjolivier01 @piiswrong the hard-coded search path and extension for tcmalloc makes it hard for pre-built binary to include this with static-link. Other dependencies in mxnet have "USE_X_PATH" config entries for this exact reason. Consider conforming to the same convention.


/cc @bckim92


We need to make sure -1 is removed for Complemultipart to succeed ..


if 문의 조건이 true (긍정) 인 경우, 더 먼저 제시하는 것이 맞긴 합니다만,
예외적으로
false에 해당 하는 내용이 더 주요 로직인 경우에는 이 로직을 먼저 놓일 수 있도록 합니다.

즉, if(!dir)이 되게 하는 것이 읽기에 좋습니다.

또한, 이런 경우 사실 if(file)이라고 했다면 긍정이 먼저 오는 것도 만족되겠죠


Se cambió el nombre del namespace con el comando:
`php artisan app:name SistemaVentas`


Se instaló laravel con el comando:
`composer create-project --prefer-dist laravel/laravel SistemaVentas`


Normally the error criteria is `norm(delta_u) > eps`. The above is not time scale invariant (e.g., do you measure time in seconds or hours) and would force rejecting a step even when the accuracy is much better than expected.


Also, normally you would use the `continue` statement to retry the step from the top of your `while` loop instead of copying the step re-evaluation below.


no falta donde importas la libreria?


> Should probably add tests where the input data ends right before the end of a page, followed by an unmapped page, to catch over-reads. This will also catch out of bounds writes,

Show examples.

> although we could catch those easier with a canary otherwise.

What is canary, except bird, I don't know.


还可以使用aop，纯xml的配置
***
🙈 🙈 🙈


the value used with ui-sref has to be one of the labels in the state object in app.js. ie. 'landing', 'album', or 'collection' not index.html


Hi there, it might seem like a silly question.
But I really want to know the reason this implementation of writing every field byte by byte is chosen over just calling java.io.ObjectOutputStream#writeObject to let JDK take care of writing the CacheHeader, is it because of the performances?
Thank you for any clarifications!


I think it would be easier if those `compareTo` methods only compare terms of the same type and the ordering of terms of different types is relayed to some other method (e.g. in `Term` or some new helper class). Then there is only one location where ordering of different term types is located.


 get/set 동사는 메모리에 있는 변수 값을 가져오거나 메모리에 저장하는 정도의 행위만을 할 때 사용합니다.

그래서 get 함수를 통해서 값을 가져오는 경우는 별도의 변수에 저장하지 않고 반복 호출해서 사용하는 경우가 많습니다.
함수 이름을 바꿔주세요.


if (set_wifi(ssid, pw) < 0) 도 좋습니다.

ret가 저장해서 필요한 값이 아닌 경우라면요


RESULT_FAIL과 RESULT_ERROR의 차이는?


@ryanbr `ouo.press` is a mirror of `ouo.io`.
update https://forums.lanik.us/viewtopic.php?p=126452#p126452


amazing code change.


A couple reasons. One is performance as you guessed: see the block comment lower in the file about the implementation of ObjectOutputStream using reflection and being slow (and allocating a ton of memory / producing garbage). The other is for versioning: I wanted to be explicit about when the file format changed. If you just use writeObject then things can move around if you rename a field. I also think it's less compact since IIRC it stores field names in the output. But I wrote this over 4 years ago, so I'm surprised I remember as much as I do :)


이렇게 대다수의 경우를 비워두는 거라면 if 문으로 처리하는 것이 낫습니다.


我们有这个域名？


This is a repository of my completed Udacity projects.

The file names start with a 4-letter code to indicate which nanodegree they belong to:
**dlnd** - Deep Learning NanoDegree
**dand** - Data Analyst NanoDegree


meta description and license, please!


why rk3399 and rk3328 uses armhf instead of arm64 package?


Remove


2017 updates:
1. cast sentinels to pass new gcc -Wall muster.
2. add a log file for troubleshooting, netpong.log
3. improve error messaging when handling sockets, to get better diagnostics.
4. Note in README about MacOS Sierra, to create listening sockets


commiting??


delete LICENSE


Yes


We only maintain arm32 debian.

ARM64 libmali could be get form here.
https://github.com/rockchip-linux/libmali/tree/rockchip/lib/aarch64-linux-gnu


These are both taken care of in #65, since they fail the pre-commit hooks there.


建議變數 ja 用有意義的名稱


建議變數 obj 用有意義的名稱


I don't see your code... It seems as if you just pushed an empty folder


bundle install


This is c99 comments, and, as I remember, we are permitting c99 in this project. They are added for 2D commenting - short comments on the right side to mark sub-sub-sections. Yes, "not too long" at the bottom (and "too long" below) is a typo, should be `/* */`-comment.


会报什么错误？


1. 建議變數 ja, obj, sche, j, o用有意義的名稱
1. 建議參數 s 用有意義的名稱


we did not use hashers.py.


I advise against using minus here. Once had an ugly but where the order contained cycles due to doing minus. It caused `TreeMaps` to not contain elements that were added. I suggest something like `thisPrio > otherPrio ? 1 : -1;`


rip, I’ve attached it here… ill see whats up with github

On Oct 22, 2017, at 9:38 PM, Maryam Archie <notifications@github.com<mailto:notifications@github.com>> wrote:

I don't see your code... It seems as if you just pushed an empty folder

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/maryam-a/mini-places/commit/56038ab41fba790b559fe51503f275da52a761ae#commitcomment-25124573>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AOVJ4ebbcKUwlOxYgYWkuOVRRKxpUQ89ks5su-3_gaJpZM4QCMUK>.


Trying to modify script for transformation according to JSON file


You can’t use this with static link because the distributed libraries
aren’t compiled with -fPIC.

This convention was taken from another lib search algorithm in the
Makefile, so it does conform.

I am not sure why we don’t stop using the archaic Makefile and use Cmake,
which can search for things more intelligently.

On Sun, Oct 22, 2017 at 5:56 PM Sheng Zha <notifications@github.com> wrote:

> @cjolivier01 <https://github.com/cjolivier01> @piiswrong
> <https://github.com/piiswrong> the hard-coded search path and extension
> for tcmalloc makes it hard for pre-built binary to include this with
> static-link. Other dependencies in mxnet have "USE_X_PATH" config entries
> for this exact reason. Consider conforming to the same convention.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/incubator-mxnet/commit/01ae629c6593e0352fd30979bccd0196854ef882#commitcomment-25124231>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AKts_Uc-cRev58CCay9f5D65ooDhEUJeks5su-Q4gaJpZM4QCLMR>
> .
>


You will need to disable for pip static build.  Note that pip static build
is problematic as long as other symbols from mxnet dependencies are not
hidden. It is known to cause problems, especially regarding OpenCV.

On Sun, Oct 22, 2017 at 6:44 PM Chris Olivier <cjolivier01@gmail.com> wrote:

> You can’t use this with static link because the distributed libraries
> aren’t compiled with -fPIC.
>
> This convention was taken from another lib search algorithm in the
> Makefile, so it does conform.
>
> I am not sure why we don’t stop using the archaic Makefile and use Cmake,
> which can search for things more intelligently.
>
> On Sun, Oct 22, 2017 at 5:56 PM Sheng Zha <notifications@github.com>
> wrote:
>
>> @cjolivier01 <https://github.com/cjolivier01> @piiswrong
>> <https://github.com/piiswrong> the hard-coded search path and extension
>> for tcmalloc makes it hard for pre-built binary to include this with
>> static-link. Other dependencies in mxnet have "USE_X_PATH" config entries
>> for this exact reason. Consider conforming to the same convention.
>>
>> —
>> You are receiving this because you were mentioned.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/apache/incubator-mxnet/commit/01ae629c6593e0352fd30979bccd0196854ef882#commitcomment-25124231>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/AKts_Uc-cRev58CCay9f5D65ooDhEUJeks5su-Q4gaJpZM4QCLMR>
>> .
>>
>


저녁 온라인미팅때 말하려하긴했지만 기록겸 먼저 남깁니다.
스터디는 같이진행하고 추후 프로젝트를 진행할때 2-way로 진행하는건 어떠하신지 해서 의견을 구합니다.

개인적으로는 아직은 일정이 정해진 프로젝트를 진행하는데 부담이있어,
팀원분들이 프로젝트를 진행할 시기가 오면
따로 혼자 1인 프로젝트(라기보단 놀이터)를 진행 하는것도 좋지않을까 하는데....
두개의 프로젝트가 각각 진행되며 서로의 제3자의 눈을통한 의견공유하는것도 제생각엔 나쁘지않을것 같아 생각해봤습니다.


程式應實際讀取檔案，而不是用常數取代


这个我没验证过， 会不会是 get_security_bars 调用的时候的报错？ @JaysonAlbert @yutiansut


Caller assumes it calls `open()`, which then replaced with `idx_open()`, but assumption remains. Caller does not know if syscall is ever called in new 'open', thus it can not handle `errno` properly. Except zeroing it before call to `idx_open()`. Ok, I will do that style of fix. Caller also had other type of problem, in my opinion, it setting [`error = errno`](https://github.com/aabc/blists/tree/3a0c24742e9092db5c8a8c5276e54e910add0d5a#L281) variable unnecessarily. So, instead of wrestling with that mess, I decided to quickly and with less side effects zero `errno` in `idx_open()`.


from azoe import ... está repartido por todos lados en lo que es rundata. El package en sí no se "importa" entero en ningun lado, sino que se usan los widgets individualmente segun convenga.


Typo: "conjugating" -> "conjoining". The method [`__or__`](https://docs.python.org/3/reference/datamodel.html#object.__or__) is a magic method (see https://github.com/RafeKettler/magicmethods for a good overview).


remember you can use the `resources` method here.


A quaternion is not a vector, nor a complex number. I'll fix the mathematical jargon later on...


You can achieve the same thing as this with a uniqueness validation on Application


Are there guidelines for adding documentation somewhere?


:D

2017-10-22 19:47 GMT-03:00 JC Ivancevich <notifications@github.com>:

> 👏👏👏
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/alnocetti/QuickDemo/commit/90e907672bc44f500e37b87690f0517a4dc088f8#commitcomment-25123230>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACVxewQce2DahsoY_6mssCK3HqsBpW9Sks5su8YXgaJpZM4QCItV>
> .
>


I dislike not outputting proper or ever informative error message. To debug that is really hard. There is ~20 calls to `html_error(NULL)`, also, a lot of cumulative `if (condition1 || condition2 || condition3...)`s with single call to `html_error()`. Not user friendly.


![](https://github.trello.services/images/mini-trello-icon.png) [Reply to comment: ‚cancel‘ on same height as ‚submit‘ button](https://trello.com/c/dgQ5RWtd/22-reply-to-comment-cancel-on-same-height-as-submit-button)


I like this code but it's worth mentioning that the shortcode `%invoice_url%` generates the URL however it fails for the user when they click the email link when they are no longer logged into the website as WordPress dies `You do not have permission to download this invoice`.


UC DSC Files


Contribution guide: http://docs.godotengine.org/en/latest/community/contributing/updating_the_class_reference.html

Writing guidelines:
http://docs.godotengine.org/en/latest/community/contributing/docs_writing_guidelines.html


Ah yes, this commit was part of a larger patch-set bitcoin#9011 - will revert this one.


Oh, great solution, tests fail, so just disable them all… 🙄 Have you verified why it fails? If the test is broken, then you should disable just that test, not all tests.


And you should **ALWAYS** write a comment after `options="!check"` why the heck it’s disabled!


Initially the test had double indentation and test was not failing(should have failed), but after we did the fix, the test failed as the indentation was reduced to one level, hence we fixed the test.


Made some changes


done


I think the indentation was ok in the initial output file as it was relative to the function call and incremented with one because of the open parenthesis.


Yes, it can be `lateinit`.


Yes, looks like it can be an `object` (`DataFlowValue.nullValue(...)` uniqueness is not important here)


Хм, интересное ограничение у eset.
Тестировал на разных. И свой логин:пароль использовал купленный на год к EAV\ESS 10.  Сроком до сентября 2018.


/* http://meyerweb.com/eric/tools/css/reset/
   v2.0 | 20110126
   License: none (public domain)
*/

html, body, div, span, applet, object, iframe,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, dfn, em, img, ins, kbd, q, s, samp,
small, strike, strong, sub, sup, tt, var,
b, u, i, center,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
table, caption, tbody, tfoot, thead, tr, th, td,
article, aside, canvas, details, embed,
figure, figcaption, footer, header, hgroup,
menu, nav, output, ruby, section, summary,
time, mark, audio, video {
	margin: 0;
	padding: 0;
	border: 0;
	font-size: 100%;
	font: inherit;
	vertical-align: baseline;
}
/* HTML5 display-role reset for older browsers */
article, aside, details, figcaption, figure,
footer, header, hgroup, menu, nav, section {
	display: block;
}
body {
	line-height: 1;
}
ol, ul {
	list-style: none;
}
blockquote, q {
	quotes: none;
}
blockquote:before, blockquote:after,
q:before, q:after {
	content: '';
	content: none;
}
table {
	border-collapse: collapse;
	border-spacing: 0;
}


This header is MSMQ specific so its still added if running with MSMQ

https://github.com/Particular/NServiceBus.Transport.Msmq/blob/develop/src/NServiceBus.Transport.Msmq/MsmqUtilities.cs#L153


"browsing context is overflowed"보다는 "page is scrollable" 이라는 표현이 Web 기술이 익숙하지 않은 일반 사람들도 더 이해하기 쉬울 것 같습니다.


we dont want to indent for assign chuck which are not followed by a newline, the indent of the assign chunk not followed by a new line will be previous stack entry + indent size (done before if condition.) but these chunk which are not followed by new line should not be considered when indenting the next line. only assign chunk followed by newline should be considered for indentation.


Great !


OK


아래에서도 (In several references below, you can get the help to understand the Snav operations well.) "operation"이라는 표현을 사용하고 있어, behavior 보다는 더 어울리고 통일을 위해 operation으로 사용하는 것은 어떨까요?


Created multidev environment [sushd8#ci-5](https://dashboard.pantheon.io/sites/fff84385-21b1-4e45-831c-18644431324c#ci-5).

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


just like this


입력 방식으로 사용되는 "사용자 시선 방향"의 표현은 보통 "eyes direction"이라는 표현보다는 "gaze direction"이라는 표현을 많이 사용합니다. (Computer Vision 도메인에서)


@vozgorkov Плохая практика передавать так аргументы. Исправь на передачу переменной! Измени название функции через camelCase.


There was double indention in the output files initially, which contradicts https://github.com/uncrustify/uncrustify/issues/1260 (which says they only want one indentation for the next line), as both are talking about the indentation of paren open after assign chunks


![](https://github.trello.services/images/mini-trello-icon.png) [Something](https://trello.com/c/P5TxZpdA/8-something)


@peterhoeg, any explanation?…


Thank you, I missed that one! Good you always install new version on your server as I update it :)


The " (1)" was automatically added by android studio and i removed it in another commit


"i.e. the first byte SHALL NOT be zero unless the Partial IV is the byte string 0x00."

This basically says that is should not start with zero unless it starts with zero. And it does not follow from the rule that comes before it.


This bothers me greatly w/ NixOS, that people are committing straight to master, w/o any discussion. @edolstra


yes, The problem is identified and it's an issue in cython's Pythran.py. I'll make a PR to solve that.


Did you also add the note that Christian wanted about therequirements on nonce length?


Just rearranged the code in Android studio thats why all the lines are highlighted as changed


works fine if remove line
state = {};


原来是你改挂了！


"Note that in this specification only algorithms that use nonces equal or greater than 6 bytes are supported."

- This needs to be 7 when the kid length is included in the nonce.


avoid to use the name `resourcesCenterProps` here. At least, use ES6 destructuring to get all props so we will have to change the code only in one place :

`const { headerTitle, headerImage, resources } = resourcesCenterProps`


额，我每次都 `run lint` 过了才提交的 ＝。-


Yeah , just for testing :)


Multihunter user have a problem on installation when you try to set a password. Support user works smoothly


Hello,
I have one issue on my woo commerce website
http://nhafurniture.com/shop/
when i'm open shop page or categories page, and sub categories page bool(true) error show in top of the website. i did not find file where i look this type of code
please help to find this error code


tried that one hr ago, error.

Works fine and no warning is removed altogether.


It is preferable to pass "signifiant" props to `ResourceBlock` and not a generic object.

```<ResourceBlock title={resource.title} ...```
you can also use object destructuring here in the `.map`

Also, avoid to use an index for key (even if here it wont cause any problems for now)


Nice catch!


"and no other fields are present"

What other fields? Wasn't it better without this addition? or....


I'll check it out, just installing it myself...


I am new to Python and I am practicing what I learned in class with this game.


just remove the line state = {}; , that worked, tested on local build


good


it may break if `resource` has no media. Please add a check to be able to have resources with no media (even if I'm not sure that it will happen)


![](https://github.trello.services/images/mini-trello-icon.png) [Something](https://trello.com/c/P5TxZpdA/8-something)


まとめてかえられるようにできるだけクラス化
@keisukeFukumitsu
ってこんなかんじでソース変更にコメント打てます


Hi! We can merge this, if you make a PR, that would be awesome! Cheers ;)


for this kind of components (same for `ResourcesCenter`), I would use functionnal components. Please also add flow typing for those 2 components, those are very simple use cases to start with it (you'll find plenty of examples in the code)


Oh ok, I think I see it. I think it's a bit unfortunate that the variable name is short mixing it up with the normal indentation. Can you also rename `x` it to be longer ?


16 is a bit low for my uses... maybe make it 256 or something. This hits bigger projects way more.

How come you're not using the sbt util? 0.13 support?


100 is a bit low for my uses... maybe make it 256 or something. This hits bigger projects way more.

How come you're not using the sbt util? 0.13 support?


Please stop suggesting to remove line. Removing lines will only break the code from functioning.
Might as well I remove all the lines and it will have no errors.


@talh78 just fyi, we have babel section in `package.json`


"encoding"


I was just wondering what happens with the previous code when the next chunk is not a newline as now it will take the `else` path whereas before it was doing an indentation here. We need to see if we regress in some other way.


The change of the end condition (line 637) introduced a bug:
If a word doesn't fit in the entire line width, an infinite loop (or a crash) happens due to the `break` before `i++` is done.

The possible crash happens due to a buffer overflow in line 639 below, I don't know why it may continue in the infinite loop without actually crashing .

Previously the misfitting word just caused a line fold (which is not good if we want to maintain a fixed width but it is better than a loop//crash...).

I have a simple fix that splits very long words across lines, so the fixed width is maintained. It works nicely and solves this problem but the postscript code doesn't support it (unless changed too). However, in the current way postscript is generated (hard-wired width=8000). But it turns out this big width is also not supported by the postscript code, it just has badly broken output for sentences that re longer than a page width. It seems it was designed to use the `row_starts` array generated by the text graph generator when it fold line.

### Repeat by:
- Open a terminal with width of say 100 or less.
- Start `link-parser`.
- Use the following test sentence from `n/corpus-fixes.batch:`:
92 Chapter 10.............................................................................................................................................]

A shorter repeat-by:
`$ echo x | lg -width=10 | cat`

An infinite loop (or a crash - depending on how the library has been compiled) will result.

---
### Possible fixes:
1) Revert the change to use the original loop condition (with its known screen line overflow).
2) Apply my fix to split long words that doesn't fit in an entire line (**I can send a PR**).

In any case, the current postscript stuff **should be fixed too**, as it is totally unusable just  now for sentence that doesn't entirely fit in the page width! I can open an issue on that and suggest how to fix that.


[file1.txt](https://github.com/MaalRobotics/repo2/files/1406672/file1.txt)
[file2.txt](https://github.com/MaalRobotics/repo2/files/1406674/file2.txt)


Hi! We can fix this on our own, but we don't want to bring a version conflict, so... we can merge it, if you would make a PR, that would be awesome ;)


works on removing the line state = {}; on local build test.


This is not working for Fedora 26


The main web page


@michalrus This was discussed in https://github.com/NixOS/nixpkgs/commit/e01bb0c0d05e29b764fa090d85ec5babd03c84b8#commitcomment-24975428


It's better, but still duplicated code. I think the best way would be to create a `isDimension` and `isMetric` function, which checks for `isNumber` resp. `!isNumber` and then create a another function that takes a filter function as parameter, which could be either `isMetric` or `isDimension`


will test now.


@tudor2k3: would a a patch be of use for you then?


Looks good. Well done! 😃


I realized that this should probably `sessionStorage`


$(function(){

console.log("Hello world");

});


@joergoster would actually be interesting to have the actual data on how often a draw evaluation is actually a draw eval and how often it is a threefold in sight. I didn't do statistics on it.

I used the function because, at the rootPos I want to know if the PV contains a threefold, rather than a draw by eval.

From my tests, I had the feeling it is important to only try and search deeper when we're actually ahead in time.


arent these WDB?


We might want to use the same optimization flags as for blists itself, in order to more likely expose the same bugs.


nearly don't stop the compiling though.

![655645](https://user-images.githubusercontent.com/32848227/31885170-793ae7a4-b7e7-11e7-8aee-2d2d8ef007cc.jpg)


initial commit


just learn


add new files


Maww


@mpenkov Did you create a pull request of this commit?


Cool! Should’ve been linked in the commit message, @peterhoeg.


This rule is overmatching and impacts victims like `http://www.bilibili.com/video/av15649737?from=search&seid=15098962397753758570` . Please make it hostname-specified.


I wrote this, and I used the term geolibrary because it is defined in Longley et al. (see [link](https://books.google.de/books?id=zyFR8uxHM9oC&pg=PA283&lpg=PA283&dq=geolibrary+geographic+information+systems+and+science&source=bl&ots=bRCBRwHASw&sig=ROYptHohoj67VKpekMeNHdRaYTQ&hl=de&sa=X&ved=0ahUKEwj6rr7jx4bXAhXCJVAKHUuNBCQQ6AEINzAB#v=onepage&q=geolibrary%20geographic%20information%20systems%20and%20science&f=false)). Hence, GEOS and GDAL is wrong here, instead we can use as an example a typical geolibrary (if you can think of one).
Geoportals, in turn, combine several geolibraries to fascilitate data access.


That is why `master` access should be restricted.


This still says "non-const".


I'm aware removing line could have consequences, ok so it solved the warning, but yes i agree might brake some think to.


If you do **READ** the commit message, you'll understand.


Ok, thanks!


> ***
* 辅助类


Does this change the very line or is it just a comment to the line we clicked...?


Perhaps use "size_t" here, but more importantly I feel this const'ification is not worth complicating the code. When I asked for const's, I certainly didn't intend us to introduce extra artificial constraints (the length limit here) and extra data copying. So I suggest that instead we revert the const's that we couldn't introduce without this data copying, and document that this function alters the passed "charset" and why.


Make put it up to the devs, see if they have any ideas about this issue ?..


looks like just a comment to the line above...


Why two space chars here?


@tvs Was this a necessary change? The KUBO_DEPLOYMENT_LOCATION value was used in the ODB pipelines to override the default location.


I think, I would put GIS in parentheses. The advantage of Python really is, that it has become the least common denominator bridging the divide between programmers. And the other good thing is, that you can do web and frontend development, you can't do that in R. And shiny is one escape (though I wouldn't say the best solution). So I would keep this paragraph a bit more generic. I can do so, if you want me to.


And yeah right null terminator in front of the string lol.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/73511828-153e-4116-982d-cd7af20ed76d "HULUpload"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/790ac3cd-a474-4b62-92f0-a8d134917190 "HULUploads") deployed [HULUpload](https://github.com/tinusm/HULUpload/releases/tag/deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-105433 "deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-105433") to [cloudnative-dev](https://console.bluemix.net/apps/fe4ed76f-15d6-45ae-b390-3bc19e83ee67?env_id=ibm:yp:us-south "ibm:yp:us-south:TEAM17040311301800949:cloudnative-dev"), including this commit


Indeed it should - I just had one of those "oh shit" moments and got cracking on reverting as quickly as I possibly could...


Great! Thanks!


Hi ,  is it possible to publish a new release with this fix ?  Our production depends on this fix .  Thank you !


No matter what bullshit explanation you'll paste into readme this code is bad and you should feel bad.
At least parametrize it with some external file or #define. C offers you preprocessors for something.
"I will make stupid change and explain it in readme so it seems less stupid." logic is not a serious programming methodology.


Sorry for the late reply (I've been on holiday 🌞).

I presume `BYTE_ORDER` gets set somewhere reliably (I don't see it in here).
I'm not sure how `__extension__` behaves on MSVC (it's a GCC extension). It might break things there.

Other than that this looks good to me.


Сделано.


Дошло. Сделал.


weird

https://github.com/TrinityCore/TrinityCore/blob/00329fe9a505c437af0b7591d8321bf3b77ad7fb/src/server/game/AI/SmartScripts/SmartAI.cpp#L176
https://github.com/TrinityCore/TrinityCore/blob/00329fe9a505c437af0b7591d8321bf3b77ad7fb/src/server/game/AI/SmartScripts/SmartAI.cpp#L200

those checks where there before the rework...
did those scripts even work in the past?

also removing those checks can lead to some unkwown behaviour... example scenario: script calling to stop path when there is no current path. Has this been tested?


Мне кажется, что "separated" подходит больше чем "divided". ^__^


Minor: formatting (here and in overloads below).


Should this remain as a show of courtesy since we are not only using the library but redistributing it as well?


My friend, that's what you often do in C.
char array may be 50 bytes but you put null terminator wherever you want to end your string.
You put that null in the front so the default signature may be empty. I don't think you understand how strings work.
That's nuff said. Have fun with shitty code.


Yup, I moved renaming of config from Multihunter to Support... moving it back. Also, making some massive optimizations in the installation SQL routines ;-)

Expect a commit soon.


maybe best to leave it as is, the best is what you have done and good work.


I fully agree that this work-flow isn't great. The fact that people will have to write their own wrappers to deal with getting stuff merged means many people (re)implementing things badly. I just don't see the alternative. I commit maybe one or twice a day but for some of the really prolific guys that would mean a LOT of PRs that would have to be processed via the web UI which would be insanely painful.


I don't like it


То, что оставил - это нужно (как варианты реализации)


ithhx真好!


Created multidev environment [sushd8#pr-custom-t](https://dashboard.pantheon.io/sites/fff84385-21b1-4e45-831c-18644431324c#pr-custom-t).

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


It fixes inventory transactions?


@DominicWatson Not sure this is actually a good idea. It breaks the tests, as the `filterParams` passed in (by reference) are modified; and so if they are used in a second query the keys have already been expanded to include the table reference; and so will not match the queryparam placeholder the second time around.

If we don't want to duplicate _all_ the arguments, we could maybe move the duplication of critical arguments - like `filterParams` - further down the line. I'm not sure how much performance gain we get from this change anyway?


I'm realizing I actually erred here originally, but the size field should be set to 3, not 1 (the current code to send NMEA 2000 messages doesn't actually look at the size when sending, only when receiving, so it's not a problem yet).


Maybe:

    enum { CONTENT_TYPE, CONTENT_DISPOSITION } type;


Is there any reason why you placed it here instead of implementing `FunctionIntrinsicFactory`?


Ues


Why not just embedd this files to repository inside directory named `fixtures`?


For now, fine, but raw_ballast_ is a PWM value, not a position, so it may be misleading. Just add a TODO comment or the such.


I agree. Fixed.


Minor: code below deserves some simplification. I understand that you've just moved it here from KotlinUastApiTest.kt. Yet, 1) it's not the best idea to cram everything into a single expression, and 2) `dropWhile { !it.text.contains(refText) }.mapNotNull { it.toUElementOfType<T>() }.first()` definitely contains some redundancy.


I'll bet if you were to run this manually you would get an error message and `scamp_main` would crash. You need to add a line in `util/msg.proto` in the `LogEntry` protobuf.


IMO, it's a bit more reliable to get descriptor for `kotlin.coroutines.experimental.Continuation.context` and then get corresponding `JsName` via `TranslationContext` (which is unavailable here).


Good girl!!!


PROVA


Yes please do update the text in the `rev1` branch - I agree with everything you say here and have tried to make it sound like a bridge although hadn't appreciated the web element. Any changes to this prose building on this will be very much appreciated.


use option 'osd_scrub_cost' instead of 'osd_snap_trim_cost'


Yes, but argv[1] isn't a proper error message anyway, and it might not be safe (per our own code's checks, it is invalid at this point, so who knows what's in there). If we want to improve error reporting, we could handle it as a separate task (as you say, there are many such places).


![](https://github.trello.services/images/mini-trello-icon.png) [Como Jugador quiero que los palos que hacen de portero se muevan solos, y reboten por toda la pantalla](https://trello.com/c/d6R2V4Yw/30-como-jugador-quiero-que-los-palos-que-hacen-de-portero-se-muevan-solos-y-reboten-por-toda-la-pantalla)


Great thanks. I thought I was risking making it *too* concise and simple. Want to avoid dumbing down complex issues so if at any point the prose does this let us know!


![](https://github.trello.services/images/mini-trello-icon.png) [Como Jugador quiero que cada 10 seg aparezcan un nuevo palo de portería](https://trello.com/c/LpsLeYXx/33-como-jugador-quiero-que-cada-10-seg-aparezcan-un-nuevo-palo-de-porter%C3%ADa)


![](https://github.trello.services/images/mini-trello-icon.png) [Como desarrollador quiero cronometrar 10s para la creación de un nuevo palo de portería.](https://trello.com/c/B68CStjR/37-como-desarrollador-quiero-cronometrar-10s-para-la-creaci%C3%B3n-de-un-nuevo-palo-de-porter%C3%ADa)


Kek


Sure. After RL conversation, I like the auto-GitHub-sync option.
We can even make it Apache-based (and not requiring a VPS)


Why no PR ?


![](https://github.trello.services/images/mini-trello-icon.png) [Como desarrollador quiero implementar un método de creación de nuevos palos de portería.](https://trello.com/c/sHlFF7ze/39-como-desarrollador-quiero-implementar-un-m%C3%A9todo-de-creaci%C3%B3n-de-nuevos-palos-de-porter%C3%ADa)


This test will be shown as `TestFilterAssigneeNullAfterWIUpdate/create_work_item/update_work_item/search_for_work_item`.

Please ask yourself these questions:
1. Do you test the creation of work items or why do you explicitly make this a subtest?
1. What is updated?
1. What is searched for?

If you want to test that a assignees are null after removing the assignees with an *update* operation, then try to come up with a better name. Since this describes the success case, I'd make the sub tests more user-friendly:

```go
func TestUpdateWorkItem(t *testing.T) {
    t.Run("assignees", func(t *testing.T) {
        t.Run("remove", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that assignees is null
            })
        })
    })
}
```

I've written this [12 days ago](https://github.com/fabric8-services/fabric8-wit/pull/1699#discussion_r143936317). You should not just put in sub-tests to organize your one test but instead structure it in order to be extensible. Let's say I want to test the same update process for labels. I would do it like this.

```go
func TestUpdateWorkItem(t *testing.T) {
    t.Run("assignees", func(t *testing.T) {
        t.Run("remove", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that assignees is null
            })
        })
    })
    t.Run("labels", func(t *testing.T) {
        t.Run("remove", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that labels is null
            })
        })
    })
}
```

And in order to test the *add* operation of labels and assignees I would do this:

```go
func TestUpdateWorkItem(t *testing.T) {
    t.Run("assignees", func(t *testing.T) {
        t.Run("remove", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that assignees is null
            })
            t.Run("not found", func(t *testing.T) {
            })
        })
        t.Run("add", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that assignee was added
            })
            t.Run("not found", func(t *testing.T) {
            })
        })
    })
    t.Run("labels", func(t *testing.T) {
        t.Run("remove", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that labels is null
            })
            t.Run("not found", func(t *testing.T) {
            })
        })
        t.Run("add", func(t *testing.T) {
            t.Run("ok", func(t *testing.T) {
                // double check that label was added
            })
            t.Run("not found", func(t *testing.T) {
            })
        })
    })
}
```

Notice that I can have *success* and *error* cases for each operation by structuring the test like this.


Выпавшее число не сможет оказаться тройкой, т.к. random() возвращает число из промежутка [0, 1), т.е. единица не включается. Тогда при домножении на длину массива, равную 3, ты получишь число из промежутка [0, 3), после чего достаточно просто отбросить от него дробную часть и получить целое число из множества {0, 1, 2}. Т.е. проблем не возникнет.

У тебя сейчас получается число из такого же множества:
round(  [0, 1) * 3 - 1  ) = round(  [0, 2)  ) = {0, 1, 2}.

Но вообще говоря, при твоем подходе 0 будет выпадать чаще, чем 1, а 1 -- чаще чем 2. Вот почему:
- чтобы в итоге получился 0, надо, чтобы random() вернул число из [0, 0.5).
- чтобы в итоге получилась 1, надо, чтобы random() вернул [0.5, 5/6).
- чтобы в итоге получилась 2, надо, чтобы random() вернул [5/6, 1).

Как можешь видеть, длины промежутков, т.е. вероятности получения чисел, не одинаковые => какие-то цитаты будет появляться чаще остальных.

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


![](https://github.trello.services/images/mini-trello-icon.png) [Como desarrolador quiero modularizar el código para hacerlo más mantenible.](https://trello.com/c/owPHIOgL/38-como-desarrolador-quiero-modularizar-el-c%C3%B3digo-para-hacerlo-m%C3%A1s-mantenible)


OK, although we don't have `nofollow` e.g. on download links to tarballs on the Openwall website. I don't know what effect the `download` attribute has.


I cannot think of any. Agree an example would be good. New words not in everyday usage should be put in 'quotes', defined and ideally referenced on first use I think.


No, use `workitem.SystemAssignees` and `workitem.SystemLabels` here.


juste. thx!


Sure, I didn't mean we'd have any issues compiling this - it's only about which style we choose to use. Personally, I tend to use C++ style comments only for temporary hacks, not in any C code I commit, but I understand there's also the approach you describe and I don't mind using it.


Like seriously? So much shitstorming in a WIP commit?


ok, so:
1. using "Upload file" do not highlight the different, it will mark all lines as deletion and insertion.
2. the the way is to "Edit file" and copy paste the content. But how to combine multiple change in one commit in this way?


I think you *can* also do this if you want a cleaner more meaningful way:

```sql
SET LOCAL fields.assignees = '{{index .0}}';
SET LOCAL fields.labels = '{{index .1}}';
update work_items set fields=fields - current_setting('field.assignees') where Fields->>current_setting('field.assignees') is null;
update work_items set fields=fields - current_setting('field.assignees') where Fields->>current_setting('field.assignees')='[]';
-- Remove 'system.labels' from the fields if the value is 'null' or '[]'
update work_items set fields=fields - current_setting('fields.labels') where Fields->>current_setting('fields.labels') is null;
update work_items set fields=fields - current_setting('fields.labels') where Fields->>current_setting('fields.labels')='[]';
```


![](https://github.trello.services/images/mini-trello-icon.png) [Como Jugador quiero mover la pelota pero que solo se mueva de arriba a abajo (no izquierda - derecha), si me tocan las porterías pierdo](https://trello.com/c/jKCMJoK4/31-como-jugador-quiero-mover-la-pelota-pero-que-solo-se-mueva-de-arriba-a-abajo-no-izquierda-derecha-si-me-tocan-las-porter%C3%ADas-pier)


No need to create a work item manually first. Instead do this:

```go
fxt := tf.NewTestFixture(t, s.DB, tf.WorkItems(1, func(fxt tf.TestFixture, idx int) error {
    fxt.WorkItems[idx].Fields = map[string]interface{
        workitem.SystemTitle: "some title",
        workitem.SystemState: workitem.SystemStateNew,
    }
    return nil
})
```


啥。。


Copying is so small and beauty of _const_ is so big, that, I think, it's better to leave it as is.

Currently, MAX_CHARSET_LEN does not constraint anything, since _encoded-word_ is limited to 75 bytes, minus 8 (for `=??1?1?=`), it could be even 68. Also, we white-list charsets anyway, and our white-listed charsets probably would not exceed 16 bytes. So I suggest to decrease MAX_CHARSET_LEN to fit into cache line.


create about


what about white theme ;)


[R/AddinLoginTab.R:22:24:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinLoginTab.R#L22) *warning:* **no visible global function definition for ‘.getTitleAndMessage’**
```r
    titleAndMessage <- .getTitleAndMessage(response = response)
                       ^~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:27:30:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinSubmitTab.R#L27) *warning:* **no visible global function definition for ‘.testsPassedPercentage’, Did you mean 'testsPassedPercentage'?**
```r
    testsPassedPercentage <- .testsPassedPercentage(testResults)
                             ^~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:38:27:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinSubmitTab.R#L38) *warning:* **no visible global function definition for ‘createSingleResultDisplay’**
```r
      testResultOutput <- createSingleResultDisplay(testResults = testResults)
                          ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:66:32:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinSubmitTab.R#L66) *warning:* **no visible global function definition for ‘.createDetailedMessageWithButton’**
```r
    elements <- list(elements, .createDetailedMessageWithButton(index, message))
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:73:10:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinSubmitTab.R#L73) *warning:* **no visible binding for global variable ‘tags’, Did you mean 'rtags'?**
```r
  btn <- tags$button(id = paste("button_", index, sep = ""), "Toggle details")
         ^~~~
```
[R/AddinSubmitTab.R:94:10:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/AddinSubmitTab.R#L94) *warning:* **no visible global function definition for ‘.createTestResultElement’**
```r
  return(.createTestResultElement(name = "All tests", status = "pass"))
         ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Authentication.R:40:5:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Authentication.R#L40) *style:* **Commented code should be removed.**
```r
  # url <- "https://tmc.mooc.fi/api/v8/core/org/hy/courses"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:2:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L2) *style:* **Variable or function name should be lowerCamelCase.**
```r
download_exercises <- function(token, exercise_id, target, exercise_directory) {
^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:2:39:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L2) *style:* **Variable or function name should be lowerCamelCase.**
```r
download_exercises <- function(token, exercise_id, target, exercise_directory) {
                                      ^~~~~~~~~~~
```
[R/HTTPQueries.R:2:60:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L2) *style:* **Variable or function name should be lowerCamelCase.**
```r
download_exercises <- function(token, exercise_id, target, exercise_directory) {
                                                           ^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:5:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L5) *style:* **Variable or function name should be lowerCamelCase.**
```r
  base_url <- "https://tmc.mooc.fi/"
  ^~~~~~~~
```
[R/HTTPQueries.R:6:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L6) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_url <- paste(sep = "", base_url, "api/v8/core/exercises/",
  ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:9:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L9) *style:* **Variable or function name should be lowerCamelCase.**
```r
  url_config <- httr::add_headers(Authorization = token)
  ^~~~~~~~~~
```
[R/HTTPQueries.R:11:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_response <- httr::GET(exercises_url,
  ^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:23:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L23) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
^~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:23:37:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L23) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
                                    ^~~~~~~~~~~
```
[R/HTTPQueries.R:23:50:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L23) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
                                                 ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:24:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L24) *style:* **Variable or function name should be lowerCamelCase.**
```r
  base_url <- "https://tmc.mooc.fi/"
  ^~~~~~~~
```
[R/HTTPQueries.R:25:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L25) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_url <- paste(sep = "", base_url, "api/v8/core/exercises/",
  ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:28:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L28) *style:* **Variable or function name should be lowerCamelCase.**
```r
  url_config <- httr::add_headers(Authorization = token)
  ^~~~~~~~~~
```
[R/HTTPQueries.R:30:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L30) *style:* **Variable or function name should be lowerCamelCase.**
```r
  submission_file <- httr::upload_file(file_location)
  ^~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:32:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/HTTPQueries.R#L32) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_response <- httr::POST(exercises_url,
  ^~~~~~~~~~~~~~~~~~
```
[R/TMC_plugin.R:22:23:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/TMC_plugin.R#L22) *warning:* **no visible binding for global variable ‘.loginTab’**
```r
    shiny::callModule(.loginTab, "login")
                      ^~~~~~~~~
```
[R/TMC_plugin.R:23:23:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/TMC_plugin.R#L23) *warning:* **no visible binding for global variable ‘.courseTab’**
```r
    shiny::callModule(.courseTab, "courses")
                      ^~~~~~~~~~
```
[R/TMC_plugin.R:24:23:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/TMC_plugin.R#L24) *warning:* **no visible binding for global variable ‘.submitTab’**
```r
    shiny::callModule(.submitTab, "testAndSubmit")
                      ^~~~~~~~~~
```
[R/Zipper.R:2:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L2) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_zip <- function(folder, target) {
^~~~~~~~
```
[R/Zipper.R:3:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
  files_to_zip <- dir(folder, full.names = TRUE)
  ^~~~~~~~~~~~
```
[R/Zipper.R:7:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
^~~~~~~~~~
```
[R/Zipper.R:7:24:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
                       ^~~~~~~~~~~~
```
[R/Zipper.R:7:38:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
                                     ^~~~~~~~~~~~~
```
[R/Zipper.R:11:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_tar <- function(folder, target) {
^~~~~~~~
```
[R/Zipper.R:12:3:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L12) *style:* **Variable or function name should be lowerCamelCase.**
```r
  files_to_tar <- dir(folder, full.names = TRUE)
  ^~~~~~~~~~~~
```
[R/Zipper.R:16:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
^~~~~~~~~~
```
[R/Zipper.R:16:24:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
                       ^~~~~~~~~~~~
```
[R/Zipper.R:16:38:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
                                     ^~~~~~~~~~~~~
```
[tests/testthat/testLintr.R:3:5:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/tests/testthat/testLintr.R#L3) *style:* **Commented code should be removed.**
```r
#   context("lints")
    ^~~~~~~~~~~~~~~~
```
[tests/testthat/testLintr.R:5:7:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/tests/testthat/testLintr.R#L5) *style:* **Commented code should be removed.**
```r
#     lintr::expect_lint_free()
      ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:1:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/tests/testthat/testZipper.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
test_resources_dir <- paste(sep = "", getwd(), "/resources")
^~~~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:3:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/tests/testthat/testZipper.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
pack_folder_path <- paste(sep = "", test_resources_dir, "/pack")
^~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:5:1:](https://github.com/RTMC/tmc-rstudio/blob/a8ed6fa01d06a749ab0cf08fc85ff68a6d317f8d/tests/testthat/testZipper.R#L5) *style:* **Variable or function name should be lowerCamelCase.**
```r
.clean_dirs <- function() {
^~~~~~~~~~~
```


Editing accident.


I tried that with Same error..


Not sure about simplifying `dropWhile { !it.text.contains(refText) }.mapNotNull { it.toUElementOfType<T>() }.first()`. Is there a `firstNotNull` method in Kotlin? and `dropWhile` it could be moved into "map" in theory... but It will be a little bit different behaviour and not sure about better code. Or maybe I am missing something?


Thought of this too.


Nope


These should be compiled to save time and not compile each execution


Awesome! :metal:


I removed `argv[1]` from there. But, `argv[1]` was admin passed, not web client passed parameter, so it should be quite safe.


I think, Android Studio puts higher preference to single quote than double quote :)


[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,
              ^~~
```


You mean put `UastKotlinPsiVariable` on line below, or what?


@igr-santos é realmente necessário utilizar uma biblioteca externa? Ela me pareceu pequena, com pouca manutenção.

Será que não vale tentar usar api direto? https://developer.mozilla.org/en-US/docs/Web/API/FileReader/readAsDataURL


Trainning file


@andir thanks again for your contribution. Could you please sign the Google Corporate CLA at
cla.developers.google.com to ease relicensing and moving the repository into the the Google organization? Thanks!


Download links probably have `.tar` in file name, while attachment links have `1` as file name. In the first case search engine would guess and not attempt to download tarball, in the second case it could not distinguish text document from tarball by the url.


/* 评分css */

.rating {
	width: 80px;
	height: 16px;
	margin: 0 0 20px 0;
	padding: 0;
	list-style: none;
	clear: both;
	position: relative;
	background: url(../images/star-matrix.gif) no-repeat 0 0;
}

.nostar {
	background-position: 0 0
}

.onestar {
	background-position: 0 -16px
}

.twostar {
	background-position: 0 -32px
}

.threestar {
	background-position: 0 -48px
}

.fourstar {
	background-position: 0 -64px
}

.fivestar {
	background-position: 0 -80px
}

ul.rating li {
	cursor: pointer;
	float: left;
	text-indent: -999em;
}

ul.rating li a {
	position: absolute;
	left: 0;
	top: 0;
	width: 16px;
	height: 16px;
	text-decoration: none;
	z-index: 200;
}

ul.rating li.one a {
	left: 0
}

ul.rating li.two a {
	left: 16px;
}

ul.rating li.three a {
	left: 32px;
}

ul.rating li.four a {
	left: 48px;
}

ul.rating li.five a {
	left: 64px;
}

ul.rating li a:hover {
	z-index: 2;
	width: 80px;
	height: 16px;
	overflow: hidden;
	left: 0;
	background: url(../images/star-matrix.gif) no-repeat 0 0
}

ul.rating li.one a:hover {
	background-position: 0 -96px;
}

ul.rating li.two a:hover {
	background-position: 0 -112px;
}

ul.rating li.three a:hover {
	background-position: 0 -128px
}

ul.rating li.four a:hover {
	background-position: 0 -144px
}

ul.rating li.five a:hover {
	background-position: 0 -160px
}


@ged-odoo
Thank you for the pointer. I have checked on some newId fields in the wild and none of them is having any reference set. Maybe I am getting it wrong @rco-odoo

Should I have now with every newId in any x2m field a corresponding reference set pointing to the original record?

I tried to get the ref via ```record.one2many_field_ids``` resulting in ```some.model(4, 5, 6, 11, <odoo.models.NewId object at 0x07762610>)``` and trying to fetch on the newId record via ```record_new_id.id.ref``` which does work in terms of non-attribute errors but having no value set.


@evsh even though I don't plan to make another 3.3.x release, IMO there's no harm to push commits into the v3_3_x anyway.


Temp


if more than 8 gpus hsr core dump


@pixelb why not just modify readlink? It seems what you have done is this:

http://xkcd.com/927


# kindly follow these steps
- open your git repo
- `git checkout master`
- `git pull origin master`
-  'git branch add <your branch Name>'
- `git checkout <your branch Name>`
- now you're ready to insert your code
- once you finished, commit your changes and then
- `git push origin <your branch Name>`


> Show examples.

`mmap()` a page, put your string to parse right at the end of the page. In another call, right at the beginning. Ensure adjacent pages are either not mapped or are `PROT_NONE`.

There should be examples of this sort somewhere in:

http://repo.or.cz/libc-test.git/shortlog

> What is canary, beside bird, I don't know.

https://en.wikipedia.org/wiki/Buffer_overflow_protection#Canaries


Not DamageModifier.


Temp 3


See #19 pls


1 +<!DOCTYPE html>
2 +
3 +
4
My First Heading for BT2201

5
My First Heading for BT2201

6
7


練習問題なので卍不要です


Okie, further updates in #19


praveen


There are already papers out there showing that OSM is comparable to or even better than professional data providers (here TomTom, previously TeleAtlas):

> With a relative completeness comparison between the OSM database and TomTom’s commercial dataset, we proved that OSM provides 27% more data within Germany with regard to the total street network and route information for pedestrians. On the contrary, OSM is still missing about 9% of data related to car navigation. According to our projection for the future, this discrepancy should disappear by the middle or end of 2012, and the OSM dataset for Germany should then feature a comparative route network for cars as provided by TomTom.

[article](http://www.mdpi.com/1999-5903/4/1/1/htm)

Of course, here we talk about Germany (one of the countries with the most OSM activity), and we are only looking at roads. However, this result is still astonishing, would be interesting to know if there is a follow-up study proving the claim that the OSM dataset is now complete for Germany...


`rapid://zk:stable`

I didn't touch launcher behaviour; don't know what happens if stable version changes while Chobby is running.

Didn't increment config version because I wasn't sure whether we wanted to deploy right now.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/73511828-153e-4116-982d-cd7af20ed76d "HULUpload"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/790ac3cd-a474-4b62-92f0-a8d134917190 "HULUploads") deployed [HULUpload](https://github.com/tinusm/HULUpload/releases/tag/deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-113147 "deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-113147") to [cloudnative-dev](https://console.bluemix.net/apps/fe4ed76f-15d6-45ae-b390-3bc19e83ee67?env_id=ibm:yp:us-south "ibm:yp:us-south:TEAM17040311301800949:cloudnative-dev"), including this commit


お気に入りのページ（スチーム！）


@AlinMoldovean, @hansgschossmann just reported there is still aproblem:

`public ExpandedNodeId(NodeId nodeId, string namespaceUri)
         {
             Initialize();
              if (nodeId != null)
             {
                 m_nodeId = nodeId;
            }`

Should also set new NodeId(?


I was unable to get it fixed with the `selected` attribute. With that approach, I had the original problem. I leave as is; one can open a follow-up.


Changed title


The interface should be such that if the call returns a negative value, `errno` is set to indicate the error, otherwise `errno` might not be set. The callers should use the interface accordingly. I've just checked the four calls from `html.c` (as committed in CVS): one of them properly saves `errno` in the `error` variable to maintain it across its call to `free(list_file);`, but the remaining three don't need such saving anymore (I guess they needed it before your introduction of `idx_open()`) and they should now be changed to use `errno` directly instead.


> So the test succeeded on Windows before the change, and now fails?

I'm not sure when/how it failed.

>  the problem is just that the MSVC gmtime() returns NULL for negative time_t values

True. MSVC's `gmtime()` handles only this era (or > Jan 1-1970). But testing `strftime()` with `tm->tm_year` < 1900 works fine.


jhasfjklasdkjasdöf


I wonder is possible to use hardware watchpoints to control access.


Suggest rephrasing it as "Que tal, que cual, Ramón y Cajal" for the sake of perfect rhyme (rima consonante)


The following does not match the byte they are over:
8(1+n)       8(2+n)

(Are they needed?)


this is the first comment made from Github web.
your code is shit!
well done anyway lol
![trollface](https://user-images.githubusercontent.com/31865045/31887076-2700e1f2-b821-11e7-94e6-9265c4aa7f2f.jpg)


Gotta love them hot fixes!


这东西向下兼容么？


So please do use `enum` here.


Idea by @mstembera, see discussion there:
https://github.com/mstembera/Stockfish/compare/287e2e2...24c3d64


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/73511828-153e-4116-982d-cd7af20ed76d "HULUpload"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/790ac3cd-a474-4b62-92f0-a8d134917190 "HULUploads") deployed [HULUpload](https://github.com/tinusm/HULUpload/releases/tag/deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-113900 "deploy-ibm-yp-us-south-TEAM17040311301800949-cloudnative-dev-20171023-113900") to [cloudnative-dev](https://console.bluemix.net/apps/fe4ed76f-15d6-45ae-b390-3bc19e83ee67?env_id=ibm:yp:us-south "ibm:yp:us-south:TEAM17040311301800949:cloudnative-dev"), including this commit


# -- Let's see if we can figure out why Daniel is doing this second data type

I couldn't remember at the time but now I've bumped into the issue again:
- if you apply a bit mask on a numpy signed integer array the result will be an unsigned array.
- to get the correct values you have to apply the initial dtype again


[R/AddinLoginTab.R:22:24:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinLoginTab.R#L22) *warning:* **no visible global function definition for ‘.getTitleAndMessage’**
```r
    titleAndMessage <- .getTitleAndMessage(response = response)
                       ^~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:27:30:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinSubmitTab.R#L27) *warning:* **no visible global function definition for ‘.testsPassedPercentage’, Did you mean 'testsPassedPercentage'?**
```r
    testsPassedPercentage <- .testsPassedPercentage(testResults)
                             ^~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:38:27:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinSubmitTab.R#L38) *warning:* **no visible global function definition for ‘createSingleResultDisplay’**
```r
      testResultOutput <- createSingleResultDisplay(testResults = testResults)
                          ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:66:32:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinSubmitTab.R#L66) *warning:* **no visible global function definition for ‘.createDetailedMessageWithButton’**
```r
    elements <- list(elements, .createDetailedMessageWithButton(index, message))
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/AddinSubmitTab.R:73:10:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinSubmitTab.R#L73) *warning:* **no visible binding for global variable ‘tags’, Did you mean 'rtags'?**
```r
  btn <- tags$button(id = paste("button_", index, sep = ""), "Toggle details")
         ^~~~
```
[R/AddinSubmitTab.R:94:10:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/AddinSubmitTab.R#L94) *warning:* **no visible global function definition for ‘.createTestResultElement’**
```r
  return(.createTestResultElement(name = "All tests", status = "pass"))
         ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Authentication.R:40:5:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Authentication.R#L40) *style:* **Commented code should be removed.**
```r
  # url <- "https://tmc.mooc.fi/api/v8/core/org/hy/courses"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:3:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
download_exercises <- function(token, exercise_id,
^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:3:39:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
download_exercises <- function(token, exercise_id,
                                      ^~~~~~~~~~~
```
[R/HTTPQueries.R:4:25:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L4) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        zip_target = getwd(),
                        ^~~~~~~~~~
```
[R/HTTPQueries.R:5:25:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L5) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        zip_name = "temp.zip",
                        ^~~~~~~~
```
[R/HTTPQueries.R:6:25:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L6) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        exercise_directory, remove_zip = TRUE) {
                        ^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:6:45:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L6) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        exercise_directory, remove_zip = TRUE) {
                                            ^~~~~~~~~~
```
[R/HTTPQueries.R:7:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
  base_url <- "https://tmc.mooc.fi/"
  ^~~~~~~~
```
[R/HTTPQueries.R:9:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L9) *style:* **Variable or function name should be lowerCamelCase.**
```r
  zip_path <- paste(sep = "", zip_target, "/", zip_name)
  ^~~~~~~~
```
[R/HTTPQueries.R:11:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_url <- paste(sep = "", base_url, "api/v8/core/exercises/",
  ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:14:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L14) *style:* **Variable or function name should be lowerCamelCase.**
```r
  url_config <- httr::add_headers(Authorization = token)
  ^~~~~~~~~~
```
[R/HTTPQueries.R:16:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_response <- httr::GET(exercises_url,
  ^~~~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:30:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L30) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
^~~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:30:37:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L30) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
                                    ^~~~~~~~~~~
```
[R/HTTPQueries.R:30:50:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L30) *style:* **Variable or function name should be lowerCamelCase.**
```r
upload_exercises <- function(token, exercise_id, file_location) {
                                                 ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:31:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L31) *style:* **Variable or function name should be lowerCamelCase.**
```r
  base_url <- "https://tmc.mooc.fi/"
  ^~~~~~~~
```
[R/HTTPQueries.R:32:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L32) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_url <- paste(sep = "", base_url, "api/v8/core/exercises/",
  ^~~~~~~~~~~~~
```
[R/HTTPQueries.R:35:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L35) *style:* **Variable or function name should be lowerCamelCase.**
```r
  url_config <- httr::add_headers(Authorization = token)
  ^~~~~~~~~~
```
[R/HTTPQueries.R:37:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L37) *style:* **Variable or function name should be lowerCamelCase.**
```r
  submission_file <- httr::upload_file(file_location)
  ^~~~~~~~~~~~~~~
```
[R/HTTPQueries.R:39:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/HTTPQueries.R#L39) *style:* **Variable or function name should be lowerCamelCase.**
```r
  exercises_response <- httr::POST(exercises_url,
  ^~~~~~~~~~~~~~~~~~
```
[R/TMC_plugin.R:22:23:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/TMC_plugin.R#L22) *warning:* **no visible binding for global variable ‘.loginTab’**
```r
    shiny::callModule(.loginTab, "login")
                      ^~~~~~~~~
```
[R/TMC_plugin.R:23:23:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/TMC_plugin.R#L23) *warning:* **no visible binding for global variable ‘.courseTab’**
```r
    shiny::callModule(.courseTab, "courses")
                      ^~~~~~~~~~
```
[R/TMC_plugin.R:24:23:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/TMC_plugin.R#L24) *warning:* **no visible binding for global variable ‘.submitTab’**
```r
    shiny::callModule(.submitTab, "testAndSubmit")
                      ^~~~~~~~~~
```
[R/Zipper.R:2:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L2) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_zip <- function(folder, target) {
^~~~~~~~
```
[R/Zipper.R:3:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
  files_to_zip <- dir(folder, full.names = TRUE)
  ^~~~~~~~~~~~
```
[R/Zipper.R:7:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
^~~~~~~~~~
```
[R/Zipper.R:7:24:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
                       ^~~~~~~~~~~~
```
[R/Zipper.R:7:38:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L7) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_unzip <- function(zipfile_name, target_folder) {
                                     ^~~~~~~~~~~~~
```
[R/Zipper.R:11:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_tar <- function(folder, target) {
^~~~~~~~
```
[R/Zipper.R:12:3:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L12) *style:* **Variable or function name should be lowerCamelCase.**
```r
  files_to_tar <- dir(folder, full.names = TRUE)
  ^~~~~~~~~~~~
```
[R/Zipper.R:16:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
^~~~~~~~~~
```
[R/Zipper.R:16:24:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
                       ^~~~~~~~~~~~
```
[R/Zipper.R:16:38:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/R/Zipper.R#L16) *style:* **Variable or function name should be lowerCamelCase.**
```r
.tmc_untar <- function(tarfile_name, target_folder) {
                                     ^~~~~~~~~~~~~
```
[tests/testthat/testLintr.R:3:5:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/tests/testthat/testLintr.R#L3) *style:* **Commented code should be removed.**
```r
#   context("lints")
    ^~~~~~~~~~~~~~~~
```
[tests/testthat/testLintr.R:5:7:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/tests/testthat/testLintr.R#L5) *style:* **Commented code should be removed.**
```r
#     lintr::expect_lint_free()
      ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:1:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/tests/testthat/testZipper.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
test_resources_dir <- paste(sep = "", getwd(), "/resources")
^~~~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:3:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/tests/testthat/testZipper.R#L3) *style:* **Variable or function name should be lowerCamelCase.**
```r
pack_folder_path <- paste(sep = "", test_resources_dir, "/pack")
^~~~~~~~~~~~~~~~
```
[tests/testthat/testZipper.R:5:1:](https://github.com/RTMC/tmc-rstudio/blob/d01cd421d8642f4130dedeb0c8316874a0d9ec28/tests/testthat/testZipper.R#L5) *style:* **Variable or function name should be lowerCamelCase.**
```r
.clean_dirs <- function() {
^~~~~~~~~~~
```


Oh. I wouldn't worry about the cache here. Rather, I worry that we need to somehow test that the condition `i < sizeof(charset_buf) - 1` is triggered right when it should be and that our action on it is reasonable. I would prefer not to have to worry about that, which is why I suggested reverting.


hello i am vinay kumar


`$event::CAUSE_CONTACT` might be better


以谷歌的尿性，难说

不过无所谓的，不兼容再改咯


Shopping edit


wer hat hier das wort titel durch musik ersetzt? für mich klingt das nach sehr schlechtem deutsch, ev. mit migrationshintergrund. Statt einem Titel runterladen, lädt man jetzt ein Musik runter???

Isch geh jetzt aldi... ;)


Nice, Thanks


So in the current release 5.3.0 it would appear that `data-ckeditor-basepath` does not work. It will not work until you do a 5.3.1  Correct ?

At the moment it is required to use   `window.CKEDITOR_BASEPATH = '/static/ckeditor/ckeditor/';`  ?


By new words, you mean, words new to the text because geolibrary is not a new word but it appears it is not that much in use. Nevertheless, I would refrain from introducing new words (such as spatial libraries) when there are words already in existence (such as geolibrary).
Have a look at: https://catalog.data.gov/dataset
This is probably the most popular geoportal and it just allows access to various geolibraries.


> I wonder is possible to use hardware watchpoints to control access.

Yes, but `mmap()` is more portable, simpler, and catches more errors (not only sequential over/under-flows, but also small jumps over the addresses you would have watched). Anyway, implementing this is not a high priority.


OK, this is convincing.


Yes, that's correct.


I don't understand where worries is there. It's usual (even typical) and simple use of index limiting. There is so much more realy complicated code in blists to worry about.


fix for https://github.com/zendframework/zend-psr7bridge/issues/25


OK, there were a couple of things there, all of them fixed now. Multihunter & Support passwords work, Oasis step now correctly redirects to the final step and the world and oasis population routines are lightning fast now as well - so no more timeouts :)


SZAFDAEEEFCDR


Okay, similar to my initial pull request, what seems to fix it (whyever), is putting an additional `Locale` into `LocaleUtils`, so the diff would look something like this:

```diff
 public class LocaleUtils {
+    private static final Locale DEFAULT = Locale.getDefault();
     private static final Locale SPANISH = new Locale("es");
     private static final Locale PORTUGUESE = new Locale("pt");
     private static final Locale RUSSIAN = new Locale("ru");
     public static final List<Locale> SUPPORTED_LOCALES = Arrays.asList(
-        Locale.GERMAN, Locale.ENGLISH, SPANISH, Locale.FRENCH, Locale.ITALIAN, PORTUGUESE, RUSSIAN,
-        Locale.SIMPLIFIED_CHINESE, Locale.TRADITIONAL_CHINESE);
+        DEFAULT, Locale.GERMAN, Locale.ENGLISH, SPANISH, Locale.FRENCH, Locale.ITALIAN, PORTUGUESE,
+        RUSSIAN, Locale.SIMPLIFIED_CHINESE, Locale.TRADITIONAL_CHINESE);
     private static final String SEPARATOR = "_";
```

Do you want me to send you a patch or would you prefer to commit that?


Found some information about thread on changelog 2008-09-13 18:49

And here is a answer from Viktor:

```
The correct way to declare thread local variables in Harbour-context on C level is to use static HB_TSD_NEW( ... ) (grep for examples in the source). I doubt the above could work correctly with Harbour threads and HVM and it for sure isn't a portable solution.
```
full answer:
https://github.com/vszakats/harbour-core/issues/326#issuecomment-338627912


I'm currently trying out this option on a local test server but i believe this won't be an issue.


but rotten flesh hasn't been fully implemeted, it currently fake-changes food bar whenever you consume it.


oh... thank you!


yes, I am aware.. this will work correctly when it is added.


isn't that for block damage ?


Project 2fk9


Thats why the I did not make any mention of the rework in this commit - not treating as related


No


Remarkable project


redundant code


They already perfectly works)


Wow


This shows some syntax error. So, I am retaining the current form.


Fixing the spacing issue here to pass `lint`


Tried to express myself better in 8e15f6d :)


mainly did it for consistency - because single quote doesn't work with variable names


typo?


I mean words that are either not in the dictionary or which most people will not have heard of, like geolibrary.

I don't think spatial library is a new word - it's a new concept. Would still need defining though.

I suggest adding a brief definition and link to academic reference or websites would resolve the issue, whether we use geolibrary or spatial library or geoportal. Looking at the content of that web page and the [wikipedia page](https://en.wikipedia.org/wiki/Geoportal) on geoportal I think geoportal is a more appropriate word and cannot see how geolibrary is any different. So my suggestion for that line and the next one would be:

```
Various 'geoportals' (web services providing geographic data such as the geospatial datasets in [Data.gov](https://catalog.data.gov/dataset?metadata_type=geospatial)) are a good place to start, providing a wide range of geographic data.
Geoportals are a very useful data source but often only contain data for a specific locations.
```