heat-emission (XuanLuoLBL) - x86_64-Linux-Ubuntu-18.04-gcc-7.3-UnitTestsCoverage-Debug: OK (1809 of 1809 tests passed, 0 test warnings)

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


I have not very concrete worries about this parameter, mostly about bindings not being generated consistently depending on when this is called with `true` or `false` with otherwise identical arguments.


This depends on porting `rqt_logger_level` which doesn't look like it will happen soon. This branch can be deleted


there are some errors on multiple lines, e.g. line 18, 22 and 51


I don't know how to use this thing!(I'm trying!!!)


No longer needed as a result of https://github.com/ros2/rclcpp/pull/587


typo, alignments, grammar enhancements


My Gentoo box is a bit slower. Same result here with cython-0.29.4. I get the `MemoryError` on the first doctest of `tableau.py` and subsequent doctests pass.


we have convention about naming classes that is used by js - `js_` prefix, e.g. `js_new-task`


## Awesome! :) And Hints! 🚀

Loving those excursions into **functional programming**!! And your code works just fine - you're simply looking somewhere where your result is not.

Some operations **work directly on the object** -  in this case `unsorted_list` - and `.sort()` is one of them.

Check out [this REPL](https://repl.it/@martin_martin/functional) for a working version that prints what we're looking for.


Thanks.

On Sun, 10 Feb 2019 at 02:07, Kumar Nityan Suman <notifications@github.com>
wrote:

> Good Work. Nice Touch.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/colorpal/dark-jupyter/commit/af5093b8d4382d918aa9859c03d40bb986a1e624#commitcomment-32256857>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AZfI_-dc5c9fMFmjMMvVObNMlRHdlpOCks5vLzF5gaJpZM4ayumo>
> .
>


넹


OK, fair enough. Thanks for the reply.


Successfully aliased the URL <https://leerob-qh1wk4kga.now.sh> to the following aliases.

* https://leerob.io
* https://www.leerob.io


Successfully aliased the URL <https://search-discordservers-com-mbkhak0i9.now.sh> to the following alias.

* https://search.discordservers.com


I don't think this makes sense for other execution modes, where `process.argv[1]` is likely `undefined` or not a path e.g. `node inspect`.


@un-tethered Please add the error payload as part of the response object for easy debugging.


@un-tethered Why do we need line `6` ?


FEAT: In the function named "check_survivor_bias" (later renamed by "check_target_distribution"), not use .plot(kind="scatter") for memory limit and not use .plot.scatter() for not a dataframe


BlueJ didn't recognize previous package restructuring, and lead me to believe that I was pushing changes to the package structure

ignore second half of commit message, no changes were made in that regard.


@enriquezmark36 yes booting but many stuff broken, camera, ril, video dec/encoder


Successfully aliased the URL <https://leerob-7s78nmf7f.now.sh> to the following aliases.

* https://leerob.io
* https://www.leerob.io


Successfully aliased the URL <https://jrpdebug-7yugpvg30.now.sh> to the following alias.

* https://jrpdebug.now.sh


These step is where if successfully logged in, should navigate to another page i.e welcome page .by calling constructor


Wikipedia like page


Created multidev environment [delongd8#ci-17](https://dashboard.pantheon.io/sites/545434d9-28d2-4ff3-8440-7a607ccc190e#ci-17).

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


Good workaround and worth it for practicing thinking about it differently! Feels clunky, though, when considering the functional approach that you tried first, right?


学习GitHub


啦啦啦


## Awesome!

You could also assign it like so:

```python
my_dict[i] = i**2
```


Upvotes for politeness!! ;)


new updates (method, var, detect password)


Hi **titipata**, I have sent the PubMed file to your mail (my.titipat@gmail.com). If it is not your mail, then, kindly tell me your email address and I will re-send it to you. Thanks.


## Awesome! 👏

Well applied cycling through different types to make certain operations easy! Cool that you switched to `set`s and applied `.union()`.


sd dsdfwserwewrwerewrfdsfsdfsfds


## Well done!

Also very nice how you're applying the `.get()` method together with its **default value**. In this case, that works perfectly, since adding `0` to a number doesn't make a difference, and yet the dictionary lookup won't fail if the key does not exist in one of the dicts. 👍


@garrigue note this is not necessary anymore after #44


这是针对期权、股票、数字货币之类市场的，只有国内期货不给5档


Please forgive my ignorance, but are the Dwarf mounts different at night from what they are during the day?
"night spawns" - sounds like they have got some kind of vampiric life at night.


Nice


## Suggestion ♻

A quicker way to do this would be to create the tuple right away: `result = (items, numbers)`


6.9.0 is bad


更新到8号的历史版本


更新8号版本


Good job for solving the task! :)

## Challenge 🚀

Can you solve this task using **functional programming** (as you did in a few other exercises) - using e.g. `sorted()`?

Doing so you could reduce the code needed to solve this task to **one line** of logic :)


huhuhuioopoppooo


## Awesome 👍

Very cool that you introduced some **error handling** into your code here! Congrats for thinking of this and implementing it :)


Created multidev environment [delongd8#ci-20](https://dashboard.pantheon.io/sites/545434d9-28d2-4ff3-8440-7a607ccc190e#ci-20).

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


I noticed `node inspect` doesn't work with `node -r`. Do you know if there is a technical reason or if it's likely an oversight?


## Suggestion ♻

You mentioned you were interested in [Big-O](https://en.wikipedia.org/wiki/Big_O_notation) (code performance of different data structures). Nested for-loops are baaaad - try to avoid them if you're looking to increase performance.

Often there are great methods in python that do just what you want them to do without needing to re-implement the whole logic. And usually these are much faster than what we mortals come up with :)

Can you think of a **method** that you could apply directly on a string (no need for the list) that would be a better fit and allow you to reduce your code's complexity? You've already used in other places!


Lol :)
I see I'm going into older commits so maybe these comments are obsolete. But still fun! hehe...


Upload your code to the github , I'll look into it

Daksh Semwal

On Wed, Feb 6, 2019, 8:20 PM ilyasma5656 <notifications@github.com wrote:

> i already use your db it's show or view just category bro but for food it's
> does't show could you suggest to me
>
> ในวันที่ พ. 6 ก.พ. 2019 เวลา 11:33 DAKSH SEMWAL <notifications@github.com>
> เขียนว่า:
>
> > @ilyasma5656 <https://github.com/ilyasma5656> use this Eat It Video
> > Lecture <https://www.youtube.com/watch?v=Ad41Bh704ms> for further
> queries
> >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub
> > <
> https://github.com/DAKSHSEMWAL/Eatit/commit/f34205b7a76d979216185dc7bb2c66fd5106c312#commitcomment-32208087
> >,
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe-auth/AsfTsOWatW8sGfKCDBpLCUbdEHehpu_zks5vKlshgaJpZM4akgjU
> >
> > .
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/DAKSHSEMWAL/Eatit/commit/f34205b7a76d979216185dc7bb2c66fd5106c312#commitcomment-32214675>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ANbCVqKjlkyCs2jFvvlvsxDrmDUPMdtLks5vKuvHgaJpZM4akgjU>
> .
>


Executing the daily benchmark build, I will reply here when finished:

@nanosoldier `runbenchmarks(ALL, isdaily = true)`


Successfully aliased the URL <https://sociallinker-8k9c62o6r.now.sh> to the following alias.

* https://sociallinker.now.sh


When will you be releasing as a new version?


@ih24n69 , yep that many. I tried to fix many stuffs there: the single/dual sim ril, taking photos in either of the camera, and the video decoder. But still those are pretty much ugly hacks. In the end before my hard disk fails, the only outstanding thing I was not able to work out is the [camera video recording](https://github.com/diepquynh1501/android_hardware_sprd/issues/15) (also maybe along with the video encoder). Regarding the video decoder, vp8(.webm) codec works though.

P.S. The camera source in your cm 12.1, can be used as a drop in replacement but only the rear camera works. The front camera won't work at all with that source.


![](https://github.trello.services/images/mini-trello-icon.png) [Basic Login with Email](https://trello.com/c/dp78DVGF/19-basic-login-with-email)


@lawlz Gotcha. Yeah, I hadn't noticed the tiny different between the logos, so thanks for pointing that out.


Deployment failed with the following error:

~~~
The property `static` is not allowed in now.json when using Now 2.0 – please remove it.
~~~


Deployment failed with the following error:

~~~
You tried to create a Now 1.0 deployment. Please use Now 2.0 instead: https://zeit.co/upgrade
~~~


仍然有问题，如`jdbc:mariadb://db.xxx.com/postgresql`这种就会误判


smmsm


@l181211UsamaZahid should return Int type value.


Successfully aliased the URL <https://sociallinker-ap9x9vqia.now.sh> to the following alias.

* https://sociallinker.now.sh


@l181211UsamaZahid should return area


@l181211UsamaZahid should also check for null ptr.


@l181211UsamaZahid here you should set null ptr after removing memory.


@l181211UsamaZahid for pointers you should transfer values not address stored in pointers.


don't see anything


Practice Lesson 9


@l181211UsamaZahid not correct way for deleting pointer variable.


## Suggestion ❗

Here's a cool construct you could apply fittingly to how you constructed this code logic: `for - else`.

It's not a common way to write python code, but it's totally valid. So if you're interested, you can [read up on it here](http://python-notes.curiousefficiency.org/en/latest/python_concepts/break_else.html) and give it a try to implement it here.

But this is certainly not a topic you need to learn as a beginner!! :)


will change this one back to what it was, no worries.


:question: why? As one of the more hard-line opposers of IVTs, this strikes me as strange since there is little to no value in removing this one.


@l181211UsamaZahid value for education background not copied


going back and looking at the code it should be working like this video

https://www.youtube.com/watch?v=5uZfKI9YPHc

which was how it was working before I stopped using the bargraph. I think what is wrong is that the code for the bargraph sequence one is not in the powered on but safety off section at line 414. It should look like this I think.

    } else {
      // and do the standard bargraph sequence
      barGraphSequenceOne(currentMillis);

      // if the safety is switched off play the click track
      if (safety == true) {
        setWandLightState(1, 4, 0);    // set back light off
        setWandLightState(2, 4, 0);    // set body off
        safety = false;
        playAudio(clickTrack, playing);
      }
    }


@182182basitAliAhmad should be array list.


## Awesome 👍

Well done, works fine and is concise! Good job! :)

## Challenge 🚀

How can you adapt the code to change both uppercase as well as lowercase characters from a user input? E.g. that: `Hello here I am!` would change to e.g. `%ello %ere I am!`?


This could also be `str[0]`, which might be easier to read and understand.


♻ Please avoid using `str` as a variable name, since it is the name of a built-in function `str()`.


test dictionary


@180997zuhaibsohail function prototype is changed.


Definitely we need to return `copy`. The fact if copy was made or not cannot depend on data. There is non-exported `funique` function that uses similar logic: https://github.com/Rdatatable/data.table/blob/master/R/setops.R#L50


@180997zuhaibsohail not correct way to transfer data from pointer memory.


i really need to remember to `push` before adding new stuff from the web UI


@180997zuhaibsohail deletion of memory is missing.


this should test non-equality


## Challenge 🚀

Currently, you are **looping twice** - one time over the string, and one time over the list of vowels.

Try to replace the second loop with a simple **check**: "Is the letter `in` the list of vowels? If so, increment!"

Can you translate these instructions to code and re-write your logic? :)


: ) You're catching the fact that language changes between singular and plural. Very user-experience centric thinking! Well done 👍


@181092muhammadehssan prtotype of function is changed.


## Awesome! :)

Nice research on the English language, and cool that you included this **edge case**!


@181092muhammadehssan code similar as @180997zuhaibsohail


## Awesome 👏

Great code comment that gives additional information here, alerting the reader that there might be an issue with whitespace being counted as a character as well - and that this choice was made in favor of counting it.

Full disclosure is always good :)

You could add a short note on _why_ you chose to count whitespace as a character - even though in this case it might just be convenience (but that's a good reason!) :)


That's a great practice on **nested conditionals**!! :D

## Suggestion ♻

However, there are simpler ways to write this logic through connecting conditional checks with **logical operators** (`and`, `or`, `not`).

## Challenge 🚀

Rewrite this part using logical operators, to slim it down!


## Awesome! 👍

Well implemented to make the user input easier through splitting by comma! :)


Такая практика - плохая.
Обычно принято освобождать ресурсы (закрывать файлы в данном случае) в том пространстве имён, где они были созданы. `translate` должна возвращать int, которая сообщает об ошибке или её отсутствие, а вызывающий контролировать значение после вызова и решать об освобождении ресурсов.


test c commit review====================


OK thanks for your review


@180965muhammadahmad good clean code 👍 . Memory deletion is not proper in main.


how to get this file in pdf or any other format so, I can see reports


actual joy


kgjkgb


@l181107abdullahkhan Q1 not implemented


Here is Dynamic update


@181027naumanAhmed Q1 is missing.


Update


Successfully aliased the URL <https://nteract-site-ocfekfqlty.now.sh> to the following alias.

* https://nteract.io


@182171Naeemraza other memory also needs to be deleted.


# every people here our universe need respect in every aspect of life


to prevent a conflict most in religious issues


You're totally right, they probably should. But it was making me have to handle more than is necessary in the GraphQL layer and this was an easy fix. Errors are handled correctly, though- if you pull open the query UI or GFD UI and turn off the backend & make a query, you'll see the error reflected in the places you'd expect.

I'm sure there's some type info being lost in the middle there for all this to work as it does, but it's just for the compiler anyway and as you say, it works, no need to mess with it IMO.


pylint имеет свойство не обращать внимание на некоторые ошибки
https://lists.logilab.org/pipermail/python-projects/2011-January/002832.html


Это же не оптимально. `printf `- это print format - удобно при выводе строки с заданным форматом. Примитивный вывод букв нужно делать функцией `int fputc(int c, FILE *stream)`.


Нужно читать документацию
https://lists.logilab.org/pipermail/python-projects/2011-January/002832.html


good


Эта строчка до сих пор здесь
Нужно закоптить и запушить


я не вижу


@182171Naeemraza there is no need for allocating array of memory here.


Looks fine!


> All rights reserved.

This statement is both [contradictory and obsolete](https://web.archive.org/web/20080101091757/http://www.iusmentis.com/copyright/allrightsreserved/) and should be removed from all files.

Ref: https://github.com/rubygems/rubygems/pull/1249


ping @kemitchell


@182171Naeemraza not checked if the values are nullptr or not.


new comment added


@182171Naeemraza not correct. you should assign value to pointer directly not to deference.


@182171Naeemraza not transferred education background. You need to assign memory for folders as well .


I wish. Mix of a bit of a burnout due to 50 hour weeks in my day job as a
software developer and currently a bad cold that I thought I could distract
me from. And yesterday also a hangover 😉

On Sun, 10 Feb 2019, 00:35 Joe Groocock <notifications@github.com wrote:

> Drunk releases?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/theotherp/nzbhydra2/commit/32f76dce8372668acf4fb8ccbcd0db53dd72abfd#commitcomment-32257494>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ANtAeYHFmBR88UHBuk6OjUZQvUklnqO9ks5vL1tQgaJpZM4ayzH3>
> .
>


우선 이번 문제(16915)를 풀기위하여 2-SAT문제인 https://www.acmicpc.net/problem/11280 먼저 풀어보기로함.
이 문제를 풀기위한 사전지식으로 SCC(Strongly Connected Component)에 대한 사전지식이 필요하다.


@mqudsi the extra `S` looks like a typo?


Много повторений усложняет чтение и может быть причиной ошибок. Перепиши как-нибудь так после while :
```
const int len = strlen(s);
fprintf(out,"%c",alpha[(buf[0] >> 2) & 0x3F]);
fprintf(out,"%c",alpha[((buf[0] & 0x3) << 4) | ((buf[1] & 0xF0) >> 4)]);
if (len >= 2)
  fprintf(out,"%c",alpha[((buf[1] & 0xF) << 2) | ((buf[2] & 0xC0) >> 6)]);
else
  fprintf(out,"%c",'=');
if (len >= 3)
  fputc(alpha[(buf[2] & 0x3F)], out);
else
  fputc('=', out);
```


Для связи с приемо-передатчиком будем использовать стандарт "СТЫК С2". Тут можно посмотреть описание http://vsegost.com/Catalog/31/31414.shtml


Add the Checkboxes to the collapse and get rid of the blue hover glow on them.


> . or it will even be possible to just modify the size variables without creating a new lv_disp, I'll see when this is done.

That's the goal! It will be a `lv_disp_set_resolution` function to change the resolution in runtime.

> If you need anyone to test these changes, I'll implement them in AviTab as soon as you're ready for testing.

Awesome! Thank you!


要放在“成员”部分的内容，按照word文档的形式先放上去就可以啦，图片的原图都在压缩包里面


@l180921usmanbutt submission should be made on each lab submission link. Q1 is missing.


The change was needed. It had some grammar problem. I just make them solved. That's all!


Probably an oversight, though technically the help text of `node inspect` does not claim to support Node.js flags.


@l180921usmanbutt should have separate cpp and header file


TOP 10 THAILAND LOTTERY TIPS & THAI LOTTERY RESULT 16/02/2019 THAI LOTTERY KING [ THAI LOTTERY 123 ]
http://thailotterytip.org


Successfully aliased the URL <https://personal-site-silkfhlegk.now.sh> to the following alias.

* https://site.sergiodxa.com


Created multidev environment [d8mag1#ci-2](https://dashboard.pantheon.io/sites/04f7837a-a16f-46de-9af4-182ad7341617#ci-2).

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


Created multidev environment [d8mag1#ci-4](https://dashboard.pantheon.io/sites/04f7837a-a16f-46de-9af4-182ad7341617#ci-4).

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


![](https://github.trello.services/images/mini-trello-icon.png) [Install project Dependecies](https://trello.com/c/ifuDOdzR/2-install-project-dependecies)


You missed this: 1 -> 3


commit


Вместо этой жути можно было написать:
if ('A' <= alpha[j]) <= 'Z')
  buf[i] = alpha[j] - 'A';
else if ('a' <= alpha[j]) <= 'z')
  buf[i] = 26 + alpha[j] - 'a';
else if ('0' <= alpha[j]) <= '9')
  buf[i] = 52 + alpha[j] - '0';
...


heat-emission (XuanLuoLBL) - x86_64-Linux-Ubuntu-18.04-gcc-7.3-IntegrationCoverage-Debug: OK (2446 of 2448 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 637
 * Timeout: 2

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


Created multidev environment [d8mag1#ci-6](https://dashboard.pantheon.io/sites/04f7837a-a16f-46de-9af4-182ad7341617#ci-6).

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


heat-emission (XuanLuoLBL) - x86_64-Linux-Ubuntu-18.04-custom_check: OK (2 of 2 tests passed, 0 test warnings)

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


Yessir. Must have been in insert mode and didn't realize!


Successfully aliased the URL <https://jrpdebug-g9ld1b1uc.now.sh> to the following alias.

* https://jrpdebug.now.sh


Tried to connect MongoDB. Doesn't work at all. Problems with external libraries and nothing is working. I'm confused as hell


![](https://github.trello.services/images/mini-trello-icon.png) [Initialize a Node project](https://trello.com/c/40S7HBC7/1-initialize-a-node-project)


https://github.com/fish-shell/fish-shell/commit/2445a76d2ef5fd6f7725679c36797c5f4e81cfe3

I hope you enjoy the new completions.


The MongoDB driver used was version 3.10.0
The driver package [https://repo1.maven.org/maven2/org/mongodb/mongodb-driver-legacy/3.10.0/](url)
Why legacy? Legacy had this MongoClientURI class when mongodb-driver-sync had not.
Is it working? No. What to do? I have no clue.


Cheers, thanks for your work on them 🙂


This is cool, I didn't know this was a feature (using VS Code). How'd you discover that?


Copy of this build can be found at
https://www.mewbuilds.com/build/e48a1546cafabe09c391dc0dec6c2c00955ffc05/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmcvNf95c4raUADHBLrDY2LVx8Sm6yywZFKwnHLbEg2F4G


Never seen or used smee before, pretty cool!


Copy of this build can be found at
https://www.mewbuilds.com/build/077e32677e3df536bc751b356de9f898aa88e1d1/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmUFg2GL9WuKHphcMSUXyH2jdxRL3dmCGSXEbtAQSiyY6X


Very good efforts!


Just curious, did you manually replace `*` with `-` or was it auto-formatted? It would be nice if it was auto-formatted (ala prettier) and the rule was set in version control.


I'm guessing this file was created and/or used by probot? Mind if we add a comment near the top explaining that?


Updated


It's happening!

![BCG emulation](https://i.imgur.com/35XwSjT.png)

It's kinda unstable in MMBN4.5, but it works some times. Works rather flawlessly for unlocking minigames in Megaman Zero 3.


Successfully aliased the URL <https://jrpdebug-5sktvsqfs.now.sh> to the following alias.

* https://jrpdebug.now.sh


The current app doesn't require write access to checks. We should probably disable this.


We don't need read access to contents either.


We only need read access to pull requests.


Need to setup the following access:
![image](https://user-images.githubusercontent.com/32533397/52530375-96145e80-2cb9-11e9-8977-7c8a6c32fd40.png)


Ummm... I don't know why it @ "ReBuild"


не надо коммитать эти конфигурационный файлы


не надо аббревиатуры писать капсом, все в camelCase:
BookDto


вешай @AUtowired аннотация над конструктором все равно - хороший тон


We should be listening for all events (opened, edited, deleted, transferred, pinned, unpinned, closed, reopened, assigned, unassigned, labeled, unlabeled, milestoned, or demilestoned.) since that meta data exists on the pull request and users of PRLint might be creating/using rules on any one of those pieces of meta data.


This file is basically just a template for when someone goes down the self-hosted route?


1. HttpServletResponse не используй явно, у спринга есть готовые решения для этого
2. Лучше используй GetMapping в данном случае
3. Для универсальности советую использоть octet-stream в качестве респонса, а не text/csv, ну и для этого есть константы в спринге в классе MediaType


StandardCharsets c константами кодировки, не нужно хардкодить


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


честно говоря, нет смысла в выносе отдельно в интерфейс, когда у тебя будет ровно одна реализация с постфиксом *Impl, это раньше было модно


FORM* ADDED*


ну так и передавай свою дто, а не кучу параметров отдельно!
Идеальная функция - это функция, которая принимает ровно один параметр и один возвращает (из функционального программирования)


вот здесь ты повесил Autowired)


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


[modify] the output of get_train_test() will not contain CUST_ID


а как же паттерн билдер?)


aod


а где транзакции на уровне сервисов??


вот любишь ты статику... завязывай))


Created multidev environment [d8mag1#ci-7](https://dashboard.pantheon.io/sites/04f7837a-a16f-46de-9af4-182ad7341617#ci-7).

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


если так боишься пустых строк и налов в списке, то просто отвалидируй это на уровне контроллеров своих


точка нужна на этой строчке, а не там, где stream)


как-то сложно вышло... :(


it appears my secondary editor was configure to use tabs

i shall change this tab nonsense when i get to an actual computer and not a crappy android tablet


try with resources так не пишут с большими отступами, сразу try (CSVPrinter csvPrinter = new ...)


Looks about right, seems to just be a copy/paste from the old `src/app.js` 👍


csvPrinter и так должен влашиться, когда закрывается поток, посмотри


а почему здесь нет Java 8 stream?))


Не хватает
```
#include <cstdio>
#include <cstdlib>
```


пробрасывать исключение надо выше, иначе пользователь ничего не поймет! для него как будто все хорошо, а файла не получит


подключай логирование, а не sout в консоль


CollectionUtils.isEmpty и все, не усложняй логические условия


обязательно оба года разве? А если я хочу только начало указать? Так часто и делают... например, хочу посмотреть фильмы с 2000 года)


Now could not deploy this pull request since the `now.json` file does not contain valid JSON.

	Click here to see the contents of `now.json`

{
requirement.txt
discord4.py
}

For more information on now.json visit https://zeit.co/docs/features/configuration#now.json.


这真是好码


Copy of this build can be found at
https://www.mewbuilds.com/build/90551700505bfad77d830718841cc7d1063e504a/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmcRHKfekwZwihPMq4VXbQSGx5nFr8dsRATfRaDzgfkoKr


Hm, seeing the two files `src/core/lint.js` and this one `src/linter.js` is a little confusing to me, I'm not sure what the difference would be (without inspecting the files). It's pretty minor, but I might sleep on it and try to rename in the future.


HI @mathewmeconry
Thanks alot
Yest this is exactly what is missing.
Suggestion, make the template check as optional argument since most users already have there checks in` conf.d/ `

```
parser.add_argument('-tc', '--template-check', dest='templateCheck', type=open,
                        required=False, help='Which Template should be used for the checks')
```

Thanks again


SCC를 이해하기 위하여 https://www.acmicpc.net/problem/2150을 풀어보자.


Yes, using a generic solution that does the job without callbacks was my first choice too.

However, in the current implementation, the objects (peripherals, registers, fields, etc) are automatically created from a JSON (read in when QEMU starts). This JSON is also automatically created from the CMSIS SVD, plus some patches.

The CMSIS SVD files do not have the persistent/non-persistent information. This means it normally must be added in the patch. It can be done, but it requires some effort, and I don't have time for it now.


Thinking again, there is a workaround, we can set these bits manually in the peripheral initialisation after the automatic creation is performed with the all-persistent default.

I'll add the new field shortly.


Oh I see what `src/core/defaultConfig.json` is for now. I think this might cause too much confusion and it would probably be safer to send:

```js
{
  state: 'success',
  description: 'No rules have been setup - Click details to the right',
  target_url: 'https://github.com/ewolfe/prlint#example-rules'
}
```


а смысл какой менять знак у координаты? лишняя ненужная логика, упрощай


looks good


не за чем добавлять префиксы rectangle*, ты ведь и так пишешь эти методы в класса Rectangle, тавтология получается


rectangleLength и rectangleWidth  рассчитываются на основании других полей, не нужно их делать полями, иначе при изменении через сеттор или еще как-нибудь твоей точки любой - придется вспоминать, что нужно перерасчитывать другие поля


This is good stuff!


> `* add --private/-P`

Er, that one was already there.


2019-02-09전 자료


ну, здесь то же самое у тебя, не надо дублировать в методе название класса figure*


название пакетов с маленькой буквы


Hi Sebastien,
	The Slave structure covers common logic for all vhost-user backends and helps to reduce duplicated code, and VhostUserSlave trait implements device specific logic. On master side, there’s only implementation of the VhostUserMaster trait, but on slave side there’s one  implementation of the VhostUserSlave trait for each type of vhost-user device type.

> On Feb 8, 2019, at 2:41 AM, Sebastien Boeuf <notifications@github.com> wrote:
>
> I have some hard time understanding the need for generic definition here. It feels to me like we could simplify this code by doing the same thing you did for the master implementation:
>
> pub struct Slave {
> with later in the code the actual implementation of VhostUserSlave for Slave:
>
> impl VhostUserSlave for Slave {
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/jiangliu/vhostuser_rs/commit/4e38e87797bcd7e5f73ef6dbff4f41903f41b60f#commitcomment-32234206>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB14_Nxhyr0aAU_JkJXcWipCrJA8vH_Cks5vLHNPgaJpZM4aoN6x>.
>


та же ошибка, используешь поля, которые можно рассчитать


это же поле базового класса, почему ты решил именно его отдельно устанавливать?


отступ между классом и полями


нельзя наследоваться от утилитного класса... утилитный класс должен быть final


геттеры и сеттеры пихай в самый низ, чтобы не захламляли нужную логику


Uh, can we have a config setting for allowing this?


ааа. так это у тебя не утилитный класс, а интерфейс... неправильно назвал интерфейс значит!


в конструкторе обычно делай для всех полей this, чтобы была установка полей на одном уровне со всеми остальными, принято так


все поля private


опять же, не называй класс утилитным, если он таковым не является


а зачем этому классу поле figure??


Борщ, прежде чем отправлять изменение, ПРОТЕСТИРУЙ, ХОТЯ БЫ ДО МЕНЮ ДОЙДИ.


не должно быть магических чисел, выноси все в константы


@gforcada Hey Gil, I am sorry if my comment sounded harsh to you. I know you are working your ass off for the community and I really appreciate your work here. Applying changes to all our packages is a boring and unthankful job, I can tell that from own experience. I understand that you don't have time to create pull requests in that situation and I am sorry that I did not take this into account when asking for a PR.

I got frustrated lately with plone.restapi development a bit because other devs with good intentions do things that lead to additional work for me as a maintainer.

plone.restapi uses semantic versioning and my goal is to do very frequent releases after every single meaningful pull request. This is only possible if I can merge PRs at any time and then do a release right away. This wasn't possible because of your commit to master. With a PR I could have easily postponed the merge of this important and valuable contribution to after the next release.

The simple reason why I did not complain about Maurits PR is that this one did not require me to manually go through all open PRs and having to amend them.


то, что rectangle.getCounterHightPoints() == 4 не означает для компилятора 100% вероятность того, что это класс Rectangle! просто на instanceOf проверь


Thanks for the info @nonumeros. I don't typically pay mind to low-severity vulnerabilities for development dependencies. In fact, to use HoM you don't even need to run an NPM install as I've copied the necessary scripts into the codebase along with their `/*! licensing info */`, if any. The particular dev dep mentioned is used by `standard-changelog` and will be updated when I roll out the next release, if warranted.


здесь то же самое


[ hacker / cool-anonim-hackers.jpg ](http://www.pageresource.com/wallpapers/wallpaper/cool-anonymous-hackers.jpg)


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


везде устанавливай через this


почему у фигуры 3 точки??  4 не может? это же абстракция, круг тоже фигура, зачем ему в полях 3 точки? неправильно используешь наследование


Thanks for the info @nonumeros. I don't typically pay mind to low-severity vulnerabilities for development dependencies. In fact, to use HoM you don't even need to run an NPM install as I've copied the necessary scripts into the codebase along with their `/*! licensing info */`, if any. The particular dev dep mentioned is used by `standard-changelog` and will be updated when I roll out the next release, if needed


Thanks for the info @nonumeros. I don't typically pay mind to low-severity vulnerabilities for development dependencies. In fact, to use HoM you don't even need to run an NPM install as I've copied the necessary scripts into the codebase along with their `/*! licensing info */`, if any. The particular dev dep mentioned is used by `standard-changelog` and will be updated when I roll out the next release, if needed.


Wouldn't background services run on UI thread if the Activity is still there? If the app is opened, then the service will essentially be running on the UI thread


Plural rules implementation is part of translation (generated file). User should  not know anything about it. It just call `_p(n, str)` and don't care about internals.


название класса не по код конвешену


Background services are always running in the background, the UI thread doesn't exist in a service because there is no UI


выбрал не верную реализацию списка, здесь лучше бы подошла LinkedList (много вставок в середину)


Thanks for the info @nonumeros. I don't typically pay mind to low-severity vulnerabilities for development dependencies. In fact, to use HoM you don't even need to run an NPM install as I've copied the necessary scripts into the codebase along with their [`/*! licensing info */`](https://git.habd.as/comfusion/hall-of-mirrors/src/branch/master/static/modules/hall-of-mirrors/photoswipe.min.js#L1-L4). The particular dev dep mentioned is used by `standard-changelog` and will be updated when I roll out the next release, if needed.

Thank you for choosing [After Dark](https://after-dark.habd.as).


т.к. только прошли итераторы - то ок, но если можно без итератора, а просто обычным foreach пройтись, то так и нужно делать, упрощай


оптяь магические числа, выноси в константы


IMHO this exposes to public internal things without any serious benefits. Your just need to "load generated translations". 3 params is a bit strange for such action.


если класс параметризован - то ты обязан параметризовать его!


Theres a comma missing here :(


Yes, it is so much easier. 👍


Yes, but normal services runs on the main thread, and main thread will be same as UI thread if the UI is there


indexOf надо провеять на -1, а не на >= 0, добавляя или отнимая какие-то числа, не надо запутывать других


"****" тоже в константы


I don't believe that is true


A service is always a background service


Copy of this build can be found at
https://www.mewbuilds.com/build/eb5453479490d2a3bcf04c96ab0b8b73663aff6e/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmdgBbSnVg1bwwpeNATSSPnZUf7v2fqkbBgA41zP3oa8vM


From https://github.com/webpack/webpack/issues/8768.


You forgot to remove a mention of "static" from https://github.com/classgraph/classgraph/blob/e1e0126db90fa5675d2d2f11daf87fa3b32aee8e/pom.xml#L241


я же показывал, как сортировать инты - Integer.compare


sir aap ko mein ne ptaya tha keh mein ne sawal samjha tha jis ki waja se
hmari logic same hai
aap plegiarism ka case tou na lgain

On Sun, 10 Feb 2019, 10:19 a.m. yahyaali19 <notifications@github.com wrote:

> @181092muhammadehssan <https://github.com/181092muhammadehssan> code
> similar as @180997zuhaibsohail <https://github.com/180997zuhaibsohail>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/OOP2019lab/lab3-181092muhammadehssan/commit/22b3171be902dc965b81268360115020f2f0dfeb#commitcomment-32258434>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/Assi6uG1RA__hSeUvz8E1CcA3khCGUvpks5vL6vggaJpZM4ay5Ty>
> .
>


Nope, only Intent service are the true background services.
From the docs -
Note that services, like other application objects, run in the main thread of their hosting process. This means that, if your service is going to do any CPU intensive (such as MP3 playback) or blocking (such as networking) operations, it should spawn its own thread in which to do that work.


So what was the segfault cause? I see two potentials but want to know what you found.


отступ между классом и полями


класс не по код конвеншену


I could be wrong but I've never had issues with UI threads and services 🤷♂

And, the only thing that really needs to be in a non UI thread is networking - maybe large content resolver queries but that's unlikely with music.


зачем писать new ArrayList<>(Arrays.asList(
если можно просто Arrays.asList(


toString и так вызовется, если запихнуть объект в sout


Odd.

Well, you can use `GlobalScope.launch {}` to run coroutines from anywhere


а если студентов не будет в списке? - ошибка


код не отформатирован


Successfully aliased the URL <https://jrpdebug-f83tkylru.now.sh> to the following alias.

* https://jrpdebug.now.sh


Btw, I've been testing the app on my phone with no performance issues or freezing


支付宝手机号枚举工具


Thanks! I will keep that in mind. I feel the more I type the easier it is to make mistakes. I should probably change to a better naming practice.


зачем брать в скобочки (convertToArabic(romanValue))? не усложняй


There will be a slight delay in playback actions (not significant on newer devices) especially if the queue is very large


Yeah, I was using clang 6 libc++ on Linux, with a command like:

```
LDFLAGS=-stdlib=libc++ make
DEBUG=1 \
TOOLS=1 \
ARCHOPTS=-msse4.2 \
ARCHOPTS_CXX=-stdlib=libc++ \
OVERRIDE_CC=clang \
OVERRIDE_CXX=clang++ \
SEPARATE_BIN=1 \
TOOLS=1
```

What are the actual elements of `m_save`?  Are they safely movable?


I will test this on some low end devices to see how it is performing


здесь не вижу смысла создавать переменную i1, да еще и с таким названием)


по заданию нужен был метод distance, а у тебя иначе он называется, почему?)


Copy of this build can be found at
https://www.mewbuilds.com/build/fb004290718441dbe460a42bfc7f2ac40955db40/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmaVvCa1pvJ5ezfXgVeUHrXsLWjjUmvn43t23mtZw75kaF


Successfully aliased the URL <https://lucleray-e7ol8jkgo.now.sh> to the following alias.

* https://lucleray.me


нет такого слова sout)


See https://github.com/naman14/TimberX/commit/5dc7ace0d54cfbe830bb6bc4b45a2fea158e00e8


> No regression tests?

I wrote a test but somehow missed committing it to git. Good catch! It's fixed now.


слишком сложно, 4 уровня вложенности, причем 3 из них - циклы, т.е. сложность алгоритма твоего O(n^3), что похуже сортировок)


Я нашел ошибку!Очень глупая ,и мне стыдно про нее говорить. Но теперь все
работает. Я изменил код , как вы посоветовали. Спасибо!

вс, 10 февр. 2019 г. в 13:01, Sergey <notifications@github.com>:

> Вместо этой жути можно было написать:
> if ('A' <= alpha[j]) <= 'Z')
> buf[i] = alpha[j] - 'A';
> else if ('a' <= alpha[j]) <= 'z')
> buf[i] = 26 + alpha[j] - 'a';
> else if ('0' <= alpha[j]) <= '9')
> buf[i] = 52 + alpha[j] - '0';
> ...
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ArtemGomer/LabsNsu/commit/2aa91e78a165d5a70598f59cc68f024780ef254b#commitcomment-32258534>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ApbaTfT6uNLvUmCZnSM5FafyScsORuzeks5vL7W7gaJpZM4ay5ae>
> .
>


By this I mean `setQueue`


This also should not run if we don't have the storage permission because the content resolver queries will throw


опять же, для индексации по одному и тому же массиву используешь сразу 2 переменные i, n, что усложняет алгоритм, нельзя так


расчет размера массива лучше тоже вынести в отдельный метод


[CodeFactor](https://www.codefactor.io/repository/github/tman540/t.a.r.b.s.-engine) found an issue: Redefining built-in 'type'

It's currently on:
[TARBSengine.py:440
](https://www.codefactor.io/repository/github/tman540/t.a.r.b.s.-engine/source/master/TARBSengine.py#L440)Commit 5c62704efff30146dd63e8c47d086651c3354a62


тернарным бы тут зашло красиво
sout(isPalindom ? "Введенная строка является полиндромом!" : "Введенная строка не является полиндромом!")


не изменять параметры методов, представь, что там стоит final


I honestly feel that if you are using the development branch, you know things can go wrong. Only master needs to be stable.

Will leave this message up for a few weeks first.


не за чем усложнять, тебе лишь надо дать дефолтный результат true, а если символы будут не равны, то устанавливать false и выходить из цикла... упрощай


придерживай одного return из функции


I did add this check in 1.5 release adf131ed709e9092deb060b1214b8cbbf29e6f37
Think it got lost between commits


название переменной не по код конвеншену


в квадрат возводить можно Math.pow


This and loading media items is the cause of https://github.com/naman14/TimberX/issues/44


по заданию метод должен был называться Diagonal


But got it, `context.isPermissionGranted(READ_EXTERNAL_STORAGE)` could be used there now 😄


тут вообще хорошо все)


прочти про то, как использовать файлик .gitignore, а то ты комитаешь всякую чепеху в виде идеевских файликов и бинарников


мне здесь код писать?) или для чего этот класс закомичен?))


я домашки не перепроверяю, только по ООП мы договаривались


@79Vlad
Присваивание статуса убрал на бэклог.
Это необходимо делать через отдельную настройку.


import re

# take string
sentence = input("Input a string please: ")

# take symbol
sym = input("Input a symbol please: ")
# first letter of the string
fl = sentence[0]

fl_re = re.compile(fl, re.IGNORECASE) # turn off case-sensitive
sentence_re = fl_re.sub(sym, sentence)
print(sentence_re)


геттеры и сеттеры в конце класса пиши, они не несут какой-либо логики


код не отформатирован


@79Vlad переделал несколько условий на отдельную функцию


ну вот, сча с 5 все ок, но т.к. это второе решение, то макс балл не получится


This should be fine i think cause onLoadChildren will only be called when a mediabrowser subscribes. The subscription happens in MediaItemFragmentViewModel which is tied to the media item fragments. So onLoadChildren shouldn't be called until the fragments are added to the viewpager


고쳐주세요


зачем брать в скобочки? (linearize(arrayNumTwodimen))
усложняет понимание кода


лучше вынести расчет длины нового массива в отдельную функцию


Ahhhh, got it, nice


url, root_url


If the initial crash is fixed, though, and the user denies the permission, we get stuck with a white activity since `setupUI` isn't called


i,j,k - это так индексы называют в массивах, а если это значения - то называть надо как-то покрасивше)


어떤 예외인지 명시


сразу int finishNum = arrayNum[arrayNum.length - 1];
стараться сразу же инициализировать и объявлять переменную в одной строчке


Yeah it's good to go. It's been tested for a month or so. Give it a spin and publish (if you have premmy).

Will work on the non-premmy side, when I find the time ;)


Thanks, Martin. This works too.
# take string
str = input("Input a string please:")

# take letter
l = input("Input a letter please:")

i = 0 # counter
for x in str:
    i += 1
    if x == l:
        print(i)
        break
else:
    print("The letter is not in the string.")


над названием еще надо поработать, но уже гораздо лучше!


API d'http:~~~~/?%s'
url = API % (params)


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


довольно сложное условие)


Maybe it's possible to just return the lists? At least in diffPackages() which seem to be the only place where issue_messages is used, the first thing that is done is splitting them on newlines.


4 уровня вложенности... как-то слишком много


This is by far the hardest question. I just don't have time now to finger out the syllables and how to define if the string is even English.


I mindlessly massaged it into working after looking at the postinstall script. If I remember correctly, there were some calls with rc05 in the filename but the files created didn't have rc05 in the name. I will grab the updated deb and try upgrading again tomorrow.


We can add MainFragment to the UI that has the viewpager and tabs even if the permission is not granted and let's add a permission check to not add the actual media item frgaments inside MainFragment


неплохо, ты даже сделал больше, чем я просил, красава! только попроще бы было - вообще шикарно!


This check doesn't seem to work for the jvm backend. One example for the failure mode (taken from a spectest) is
```
$ ./perl6-j -e 'say Date.new("2000-02-28").day-of-week'
Type check failed for return value; expected Int:D but got Int (Int)
  in block <unit> at -e line 1
```

Testing for nqp::isconcrete would help:
```
diff --git a/src/core/Dateish.pm6 b/src/core/Dateish.pm6
index b3ced5fad..0a40dc378 100644
--- a/src/core/Dateish.pm6
+++ b/src/core/Dateish.pm6
@@ -50,7 +50,8 @@ my role Dateish {
     multi method gist(Dateish:D: --> Str:D) { self.Str }

     method daycount(--> Int:D) {
-        nqp::ifnull(
+        nqp::if(
+          nqp::isconcrete($!daycount),
           $!daycount,
           $!daycount := self!calculate-daycount
         )
```

@lizmat Do you think that change would be ok-ish? (It's spectest clean for moar.)


не сокращай, пиши String


Nit working


1. switch довольно объемный, вынести бы его в отдельную функцию
2. должен присутствовать default в switch


стараться не использовать break and continue. В твоем случае можно было обойтись тернарным оператором:
result += currentNum < pastNum ? -currentNum : currentNum;


делай отступ между классом и полями


геттеры-сеттеры в самом низу, чтобы не захламляли


не надо дублировать в методе название твоего класса, и так понятно, если вызвать Rectangle.square, что это будет площать прямоугольника


### 2019.02.10일까지의 새로운 업데이트 사안.
#### 새로운 8문제를 풀었습니다. 1문제는 못푼문제입니다. 푸는대로 다시 올려드리겠습니다.
#### main.java는 아주 쉬운 문제들만 건들었기때문에 자주 바뀝니다. 양해부탁드립니다.
-----
##### 백준 문제 번호

1. 1316

2. 1357

3. 1457

4. 16435

5. 1011 -> 아직 못푼 문제입니다.

6. 1919

7. 1920

8. 2921

----
##### 이번에는 대부분 문자열 처리위주로 풀었습니다.


вот здесь нужно было сделать лишь pointUpperLeft.distance(pointUpperRight)


Redundant README file.


for i in str.upper():
    if i in v:
        n += 1


@stas-demydiuk , can you take a look at https://github.com/domoticz/domoticz/issues/3033? People are complaining that sub-menus do not appear. Strange thing is that i am on latest Beta and do not see any changes at all after this commit


[R/create_alignment_params.R:44:3:](https://github.com/richelbilderbeek/pirouette/blob/e92325615af71ab0c5a1f1b1123366cf837f706a/R/create_alignment_params.R#L44) *warning:* **no visible global function definition for ‘create_alignment_params’**
```r
  create_alignment_params(root_sequence = "acgt", mutation_rate = 0.1)
  ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/pir_run.R:147:5:](https://github.com/richelbilderbeek/pirouette/blob/e92325615af71ab0c5a1f1b1123366cf837f706a/R/pir_run.R#L147) *warning:* **no visible global function definition for ‘check_experiment’**
```r
    check_experiment(experiment)
    ^~~~~~~~~~~~~~~~
```


somehow the indentation vanished...


I wasn't able to retrieve PDD puzzles from the code base and submit them to GitHub. If you think that it's a bug on our side, please submit it to [yegor256/0pdd](https://github.com/yegor256/0pdd/issues):

> set -x && set -e && set -o pipefail && cd /tmp/0pdd20190209-4-1fobfwa/miroque/fast-cook && pdd -v -f /tmp/20190210-12372-9z3e5i [1]: bash: warning: setlocale: LC_ALL: cannot change locale (UTF-8) + set -e + set -o pipefail + cd /tmp/0pdd20190209-4-1fobfwa/miroque/fast-cook + pdd -v -f...

Please, copy and paste this stack trace to GitHub:

```
UserError
set -x && set -e && set -o pipefail && cd /tmp/0pdd20190209-4-1fobfwa/miroque/fast-cook && pdd -v -f /tmp/20190210-12372-9z3e5i [1]:
bash: warning: setlocale: LC_ALL: cannot change locale (UTF-8)
+ set -e
+ set -o pipefail
+ cd /tmp/0pdd20190209-4-1fobfwa/miroque/fast-cook
+ pdd -v -f /tmp/20190210-12372-9z3e5i

My version is 0.20.4
Ruby version is 2.6.0 at x86_64-linux
Reading /tmp/0pdd20190209-4-1fobfwa/miroque/fast-cook
11 file(s) found, 93 excluded
Reading .gitignore...
Reading src/main/java/ru/miroque/fastcook/services/ServiceMenu.java...
ERROR: src/main/java/ru/miroque/fastcook/services/ServiceMenu.java; puzzle at line #6; @todo found, but puzzle can't be parsed, most probably because @todo is not followed by a puzzle marker, as this page explains: https://github.com/yegor256/pdd#how-to-format
If you can't understand the cause of this issue or you don't know how to fix it, please submit a GitHub issue, we will try to help you: https://github.com/yegor256/pdd/issues. This tool is still in its beta version and we will appreciate your feedback. Here is where you can find more documentation: https://github.com/yegor256/pdd/blob/master/README.md.
Exit code is 1

/app/objects/git_repo.rb:66:in `rescue in block in xml'
/app/objects/git_repo.rb:63:in `block in xml'
/app/vendor/ruby-2.6.0/lib/ruby/2.6.0/tempfile.rb:295:in `open'
/app/objects/git_repo.rb:62:in `xml'
/app/objects/puzzles.rb:36:in `deploy'
/app/objects/job.rb:38:in `proceed'
/app/objects/job_starred.rb:33:in `proceed'
/app/objects/job_recorded.rb:32:in `proceed'
/app/objects/job_emailed.rb:35:in `proceed'
/app/objects/job_commiterrors.rb:36:in `proceed'
/app/objects/job_detached.rb:48:in `exclusive'
/app/objects/job_detached.rb:36:in `block in proceed'
/app/objects/job_detached.rb:36:in `fork'
/app/objects/job_detached.rb:36:in `proceed'
/app/0pdd.rb:354:in `block in <top (required)>'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1635:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1635:in `block in compile!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:992:in `block (3 levels) in route!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1011:in `route_eval'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:992:in `block (2 levels) in route!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1040:in `block in process_route'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1038:in `catch'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1038:in `process_route'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:990:in `block in route!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:989:in `each'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:989:in `route!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1097:in `block in dispatch!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `block in invoke'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `catch'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `invoke'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1094:in `dispatch!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:924:in `block in call!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `block in invoke'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `catch'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1076:in `invoke'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:924:in `call!'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:913:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/xss_header.rb:18:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/path_traversal.rb:16:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/json_csrf.rb:26:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/base.rb:50:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/base.rb:50:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.4/lib/rack/protection/frame_options.rb:31:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/logger.rb:15:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/common_logger.rb:33:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:231:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:224:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/head.rb:12:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/method_override.rb:22:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:194:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1957:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1502:in `block in call'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1729:in `synchronize'
/app/vendor/bundle/ruby/2.6.0/gems/sinatra-2.0.4/lib/sinatra/base.rb:1502:in `call'
/app/vendor/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/handler/webrick.rb:86:in `service'
/app/vendor/ruby-2.6.0/lib/ruby/2.6.0/webrick/httpserver.rb:140:in `service'
/app/vendor/ruby-2.6.0/lib/ruby/2.6.0/webrick/httpserver.rb:96:in `run'
/app/vendor/ruby-2.6.0/lib/ruby/2.6.0/webrick/server.rb:307:in `block in start_thread'
```


Created multidev environment [slotelaw-d8#ci-7](https://dashboard.pantheon.io/sites/29255401-7951-4224-89e5-95951feeb8d0#ci-7).

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


![image](https://user-images.githubusercontent.com/37661524/52531265-df19f080-2d55-11e9-87a5-21856e12ee5c.png)

![image](https://user-images.githubusercontent.com/37661524/52531270-e93bef00-2d55-11e9-9871-5e99dcedb747.png)

_20190210131031_model_cnn_01_31_dqn_var1_part11_20190210131031_ model_cnn_01_31_dqn_var1_part11_20190210131031_ lr=0.01, GAMMA=0.999


[Your benchmark job](https://github.com/JuliaLang/julia/commit/0680f19907051ba95046397367b3de2bb1afe6e0#commitcomment-32258374) has completed - possible performance regressions were detected. A full report can be found [here](https://github.com/JuliaCI/BaseBenchmarkReports/blob/92cfa477e9bc4818733200ac077415b62a4f05b9/daily_2019_02_10/report.md). cc @ararslan


quick question, why was this removed?


public! protected  в наследовании используется, у тебя тут наследования нет


1. Поля private
2. В данном случае не корректно дефолтная инициализация


то же самое, public


а где конструктор? Как ты будешь инициализировать свой прямоугольник


OK


Puzzle `1-d5473f3b` discovered in   [`src/main/java/ru/miroque/fastcook/services/ServiceMenu.java`](  https://github.com/miroque/fast-cook/blob/master/src/main/java/ru/miroque/fastcook/services/ServiceMenu.java)   and submitted as #1. Please, remember that the puzzle was not   necessarily added in this particular commit. Maybe it was added earlier, but   we discovered it only now.


# compare the number of characters
if c1 == c2 == c3:
    print("They have the same number of character.")
elif c1 > c2 >= c3 or c1 > c3 >= c2:
    print(i1)
elif c2 > c1 >= c3 or c2 > c3 >= c1:
    print(i2)
elif c3 > c1 >= c2 or c3 > c2 >= c1:
    print(i3)
elif c1 == c2 > c3:
    print(i1, i2)
elif c1 == c3 > c2:
    print(i1, i3)
elif c2 == c3 > c1:
    print(i2, i3)
else:
    print("Something is wrong...")


why not


не за чем разбивать инициализацию и объявление переменных, все одной строчкой


public! прочти еще раз про модификаторы доступа, а то совсем не та песня чет


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


поля private должны быть, изменения состояния объектов - только через set*


@Jermolene
Is it possible to also add the titlecase filter.
Example:
`[jeremy ruston]titlecase[]`

produces
**Jeremy Ruston**


Successfully aliased the URL <https://shinjuku-lt-backend-kasotzvtim.now.sh> to the following alias.

* https://shinjuku-lt-backend.now.sh


это toString, нет необходимости в этом методе


почему опять protected?? старайся не использовать его вовсе


сначала идут константы, потом поля, потом конструторы, потом остальные методы


can you help me how to program
 BT handshake (R)
- [ ] Obtaining an action list (R)
- [ ] Implementing an action list (W)
- [ ] Play stop (W)
- [ ] Sound switch (W)
- [ ] Play control (W)
- [ ] Heartbeat packet (W)
- [ ] Reading robot state (R/A)
- [ ] Volume adjustment (W)
- [ ] Powering off all servos (W)
- [ ] Controlling all servo indicators (W)
- [ ] Clock calibration (W)
- [ ] Reading clock parameters (R)
- [ ] Setting clock parameters  (W)
- [ ] Reading the software version (R)
- [ ] Low voltage alarm (A)
- [ ] Reading the hardware version number of the robot (R)
- [ ] Controlling the motion of a single servo (W)
- [ ] Controlling the motion of multiple servos (W)
- [ ] Reading back angle of a single servo (powered off) (R)
- [ ] Reading back angle of multiple servos (powered off) (R)
- [ ] Setting offset value of a single servo (W)
- [ ] Setting offset value of multiple servos (W)
- [ ] Reading offset value of a single servo (R)
- [ ] Reading offset value of multiple servos (R)
- [ ] Reading version of a single servo (R)
- [ ] Reading version of multiple servos (R)
- [ ] Play completion (A)
- [ ] Allowing charge during play (W/A)
- [ ] Reading the SN of the robot (R)
- [ ] Reading the UDID of the main chip (R)
- [ ] Sending the action list (A)
- [ ] Completing action list sending (A)


то же самое, поля private, не придерживаешься принципа инкапсуляции в ООП


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


Ok. This implements the grace period concept discussed in issue #11.


а тут почему final? Нет единообразия в коде


I will look at the issues.


Now could not deploy this pull request as it was unable to determine the deployment type.
The repo contains both a Dockerfile and a package.json.

To fix this, set the type field in now.json.

For more information on now.json visit https://zeit.co/docs/features/configuration#now.json.


I had to also add following:
LICENSE_FLAGS_WHITELIST = "commercial_gst-plugins-ugly commercial_lame commercial_libmad commercial_mpeg2dec commercial_ffmpeg commercial_libav commercial_x264 commercial_gstreamer1.0-libav commercial_gstreamer1.0-plugins-ugly commercial_mpg123 commercial_libomxil commercial_faad2 commercial_faac commercial_lib32-faac commercial_lib32-faad2"


You use `data.frame()` with a string variable (rownames). Thus, you should probably use `stringsAsFactors=FALSE`. I suggest using a little helper function (see https://github.com/strengejacke/sjstats/blob/master/R/helpfunctions.R), so we won't forget this issue.


Also oe-alliance-core/meta-oe/recipes-distros/openvix/image/openvix-image.bb
add following to get the package lists for the ipk's
do_package_index[nostamp] = "1"
do_package_index[depends] += "${PACKAGEINDEXDEPS}"

python do_package_index() {
    from oe.rootfs import generate_index_files
    generate_index_files(d)
}
addtask do_package_index after do_rootfs before do_image_complete


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

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

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

### Deployments updated

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

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


like it


This is likely libc++ vs GNU libstdc++ behaviour.  On the past few releases of macOS, libc++ is the default and the only viable option, while on Linux GNU libstdc++ is the default.


이미지 빌드 시도를 하면

Exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/pip/_internal/cli/base_command.py", line 176, in main
status = self.run(options, args)
File "/usr/local/lib/python3.6/site-packages/pip/_internal/commands/install.py", line 346, in run
session=session, autobuilding=True
File "/usr/local/lib/python3.6/site-packages/pip/_internal/wheel.py", line 886, in build
assert have_directory_for_build
AssertionError

라는 에러를 내뱉으면서 생성이 되지않네요.. 왜그런걸까요


NumOfSells


Move ; up (next to text)


You just commit this row
<DistFiles Include="$(SpaRoot)node_modules\**" Condition="'$(BuildServerSideRenderer)' == 'true'" />

and node_modules do not exist in publish folder.
Therefore node_modules begin work from your local root folder!


Could you suggest the change? I am not sure how


NumOfOrders


названия пакетов в одной слово маленькими буквами без нижних подчеркиваний


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


не нужно использовать статический импорт для вывода на консоль


зачем добавлять постфиксы M? Они ничего не говорят


достаточно одного объекта сканера для считывания с консоли, не надо их плодить


не должно быть закоментированного кода


Форматировать код ctrl + alt + L


Created multidev environment [slotelaw-d8#ci-10](https://dashboard.pantheon.io/sites/29255401-7951-4224-89e5-95951feeb8d0#ci-10).

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


называть переменные так, чтобы было понятно, что там. И не использовать сокращения


разве я так показывал switch? Должно быть:
case 1:
    dataNumber1=31;
    break;


читаем код конвеншен, у оператора switch должна быть инстркция default


дубликата кода много, можно ведь объединить месяцы, где один и тот же dataNumber1 твой


а если високостный? Мы же специально функцию писали на занятии


а переход на новый год?


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


else должен быть на той же строчке, что и закрывающая скобочка после if, т.е.
} else {


мы же проходили Math, там есть Math.max and Math.min.


Successfully aliased the URL <https://shinjuku-lt-backend-vdqaxobxzi.now.sh> to the following alias.


To dobry początek.


этого недостаточно, чтобы оба дома поместились на участке


зачем столько отступов строк? не нужно этого


это x and y, зачем давай названия не общеизвестные?


сразу return b * b - 4 * a * c;


комментарии над классом не по код конвеншену


вот я же говорил, что можно было проще value != 0


все выносить в отдельные функции, в main - только вызов их и тестирование


Why is this change necessary? If this change is not necessary, please revert.


зачем это ;?


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


все классы должны быть в пакетах!


Форматировать код ctrl + alt + L


зачем везде статик импорт делать? есть же быстрые клавиши sout + tab и все, не надо самому ничего писать


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


choose = chose probably.. 😄


и смысл от отрицального ввода? оно не имеет решения


не увидел решение с рекурсией


делать отступ в одну пустую строку между названием класса и методом, а также между методами


ооой, это я что-то нащелкала и идея сконвертила так, я не заметила, у меня в одну строчку было


one tiny comment.. I like the additions / changes.


Можно геттер сделать, который задает начальное значение:

```
module ClassMethods
  attr_writer :instances

  def instances
    @instances ||= 0
  end
end
```

Или задавать в `register_instance`

```
def register_instance
  self.class.instances ||= 0
  self.class.instances += 1
end
```


lesson2 - что это такое?? не импорта, ни чего, какая-то мистическая переменная


среднее - это double


why?


kang


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


зачем создавать этот массив, если тебе вернется новый массив из твоего метода sumMassive??


@scorpeon я думаю можно отправить пул реквест в оригинальный реп с этим

Только форматирование поправить нужно


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


もう動くようになったみたい。 https://github.com/epoberezkin/ajv/issues/941#issuecomment-462114537


Лучше просто `attr_accessor`


你这么怀念啊，要不我给你换上如何。


по заданию было же написано, создать метод linearize, зачем придумывать другое название, тем более совсем не описывающее того, что делаем этот метод


потому что это метод, а distance это существительное)


давай, пожалуйста, нормальные названия коммитам, вообще ничего не понятно, что происходит в твоих классах


直ってた。 https://github.com/terser-js/terser/issues/251#issuecomment-460221829


Оптимальнее использовать хеш, где ключом является номер, а значением - объект. Тогда поиск по массиву будет просто доступом к значению по ключу за константное время.


смысл от переменной workMassive ??


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


давай нормальный названия переменным


ну вообще и глагол есть, но тогда он будет значить совсем другое, а я - вводить людей в заблуждение) я бы с бизнес аналитиком поговорила по этому поводу)


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


а это зачем? дубликат кода, только тип другой


не сокращай название переменных


why remove TFS?


First of all: I recommend future discussions to take place either on [the issue tracker](https://github.com/OHDSI/TheBookOfOhdsi/issues) or on [the forums](http://forums.ohdsi.org/). A comment on a commit is unlikely to be found by anyone but the repo maintainers.

To respond to your question: We currently have code mapping in [the ETL chapter](
https://ohdsi.github.io/TheBookOfOhdsi/ExtractTransformLoad.html), but if it's too much we can give it its own chapter.


работа метода не должна зависеть от предварительной подстановки правильных параметров, тебе может без проблем прийти сначала больший, затем меньший массив и наоборот


не вижу смысла в этих переменных, тем более с такими непонятными названиями они еще больше усложняют понимание кода


Dit is geen "bad request", de resquest syntax is correct. Je kunt beter een 409 (conflict) gebruiken, of een 200 (technisch gezien is de request volledig correct afgehandeld) met de melding dat de credential al bestaat


sure, why wouldn't I ?


зачем этот отступ в конце метода?


все также продолжаешь писать классы не в пакетах


That's good.  I think I spotted a copy-n-paste bug.  I think `writable_bits` should be `persistent_bits`.

```c++
    cm_object_property_add_uint64_callback(obj, "persistent-bits",
            &state->persistent_bits);
    state->writable_bits = 0xFFFFFFFFFFFFFFFF;
```


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


Maybe, but I usually wait until I have at least 3 variants until I try to generalize. And I'm not sure it is worth the added complexity.
I had it generalized in a previous attempt, so I get where you come from :)


`TimeUnit.DAY.toMillis(10)` geeft je 10 dagen in milliseconden. Dit is iets leesbaarder, en minder foutgevoelig.


метод должен возвращать true or false, и не писать в консоль и возвращать непонятную строку. А потом уже ты решаешь, что делать с результатом true-false. Метод должен делать что-то одно! а у тебя он и на экран выводит и полиндром рассчитывает, а зачем несвязанный с методом результат возвращает


Форматировать код ctrl + alt + L


1 cache-line is not always 64 bytes.


название методов должны отражать действие, т.е. быть отглагольным словом convertToArabNumber


asmkefvr kv
v dJV NK;
VBJ;VRWKJ;


читаем код конвеншен, у оператора switch должна быть инстркция default


RENJ;LESJ;
VNAJ;EJK;


вынести лучше switch в отдельный метод, громоздкий он


I have no idea - I just didn't change them, and since that code isn't called any longer for the File/Reg providers, it is less important.
But maybe that should be a separate PR?


названия пакетов в одной слово маленькими буквами без нижних подчеркиваний


And the same with the rest of the PSProperties, right?


Deze constants zou ik in een property file zetten die je ook nog eens niet op git zet. Nu kan iedereen zien wat bijvoorbeeld je secret key is.

Het voordeel van een property file is dat je deze voor verschillende omgevingen kun opzetten, en je hebt als je configuraties in 1 plek, Hierdoor is aanpassen ook simpeler.


просто return Math.sqrt(Math.pow(this.x, 2) + Math.pow(this.y, 2) );
нет смысла в создании переменной, если ты ее сразу возвращаешь и никак больше не используешь


distanse до чего? это точка! там нет никакой дистанции!!!


а вот здесь есть, дистанция до другой точки


Если посмотреть на код подклассов, то они почти одинаковые. Разница только в том, что у пассажирских вагонов занимается 1 место, а у грузовых - произвольное кол-во. Ну и ошибки разные. Подумай, как можно сделать код более универсальным и не дублировать его в подклассах, а вынести в базовый как можно больше.


сгенерируй геттеры и сеттеры идеей через Alt + insert и посмотри, как они должны выглядеть


не надо дублировать название класса в методе, а так понятно, если ты вызовешь rectangle.square


The vague worries are the best! You are a lot stronger than me on the DLR, so this is actually the part I most wanted to get input on. Any idea on how I can find this out?


Successfully aliased the URL <https://deepdashio-52vqgs6sq.now.sh> to the following aliases.

* https://deepdash.io
* https://www.deepdash.io


нужно было просто сделать return pointUp.distance(pointDown);


одной строки пропуска достаточно между методами


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


все классы должны быть в пакетах
названия пакетов в одной слово маленькими буквами без нижних подчеркиваний


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


Please remove this commented line.


hmm. so the idea is to ask users to do the conversion? if so, a bit unfriendly design, but may be fine.


елать отступ в одну пустую строку между названием класса и методом, а также между методами


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


не экономь на названиях переменных или параметров функции, пиши полными словами


все выносить в отдельные функции, в main - только вызов их и тестирование


```
def each_train
    @trains.values.each { |train| yield(train) }
end
```


Arrays.toString


Dear @SavionLegends,
> I regret to inform you that your plugin "**Grenade**" (v1.2 submitted on 2019-02-08T13:06:53.000Z) has been rejected.

B1b — No redundant API versions:
> Plugins must not list **redundant** API versions in the plugin.yml. Only the earliest supported API in each major version needs to be listed, e.g. `3.2.0` is not necessary if `3.1.0` is already listed.

Please resolve these issues and submit the plugin again.

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


не надо делать отступов в первой строчке метода, тем более в первых двух


что за странное название flag? Вообще ничего не говорит название такое


Reasoning expressed by @weshaggard: https://github.com/dotnet/corefx/issues/1604#issuecomment-98319086

Additional motivation since that post was written (and updated):
- ILLinker can't trim assemblies with internals access
- ILC (.NET Native) doesn't work well with internals access. We currently have 750 failing tests on uqpaot which will be fixed by this PR.
- The .NET Standard test suite doesn't work with internals access given to just some implementations.


Please remove this empty line.


Please remove the commented block.


ты используешь одну и ту же переенную для совершенно разного хранения данных, нельзя так!


ты же так занулишь вообще все значения! а нужно только дубликаты убрать


I would suggest to get rid of this :) It's commonly used as module operator in a lot of other languages :)


why, only need to remove the copy line, we have the rest


вообще не понятно, где заканчивается инициализация данных, а где начинается основной алгоритм - все выносить в функции, в main только тестирование


will need fixing now


так себе название a1, a2... давать более осмысленные, чтобы читались хоть как слова


Changed Java project to Maven Java Project
Happened to be some problem with installing MongoDB Driver if you install it manually
This code allows to connect to external mongodb


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


нужно использовать StringBuilder при конкатенации строк в цикле! я же показывал разницу


Try it


не эффективное решение, куча памяти сжирает, а нужно было всего лишь проверить симметричные относительно центра строки символы


никакого вывода на консоль внутри метода, это только для тестов выводим, а тестируем мы в методе main


Practica 1


лучше вынести swtich в другой метод, громоздкий он


пока ок с этим, но как пройдем исключения - там должно пробрасываться исключение


при каждой итерации цикла ты создаешь новую, но одну и ту же строку, в аппер кейсе, не эффективно


не сокращай, lastValue, currentValue


Please remove empty line in javadoc.


отступ между классом и полями


Если версия руби новая, можно так записать:
`sum = @cards.sum(&:point)`


все также продолжаешь писать классы не в пакете!


public конструктор


Please remove empty javadoc.


Van wat ik begrijk maken jullie een applicatie die alleen rest-calls gaat verwerken. Als je zeker weet dat alles met een bepaalde url prefix een bepaald recht moet hebben, kan ant dat makkelijk. Mochten jullie straks een lijst van 20 rechten hebben, dan wordt het wat onoverzichtelijk. Dan kun je meerdere security filters gaan aanmaken, of je kunt eens kijken naar beveiliging op methode en class niveau met bijvoorbeeld `@Secured({ "ROLE_A", "ROLE_B" })` of `@PreAuthorize("hasRole('ROLE_ADMIN')")`.


Successfully aliased the URL <https://svgr-bhornhnweg.now.sh> to the following alias.

* https://svgr.now.sh


Please remove this javadoc. It is enough it is reflected in the copyright header that you are the initial contributor.


очень неудобно оперировать массивом, более того, фиг пойми, в каком индексе у тебя x, а в каком y, нужно залазить и смотреть. А потом ты еще и поменяешь местами. Вообще не вижу смысла в этом методе, лучше отдельные геттеры для коодинат


то же самое для сеттеров


이슈 #4 연결


Config files met wachtwoorden of paths naar je db nooit op git gooien,


сразу инициализируй, не разбивай на два этапа объявление и инициализация


10 можно в константу
`Card::ACE_CORRECTION`


fralx
many thanks!


сразу return, нет смысла в создании отдельной переменной answer


то же самое, сразу return getWidth() * getHeight();


не сокращай название в методах также, как и в переменных


hi bro i have problem in your app


тут не за чем использовать this, и так понятно/


не сокращай, point1,point2


public


my problem is
when i check true in Ramadhan Notifaction and come time of alarm
he not run sound duaa_suhoor he run default sound if phone


Unrelated to this commit, but the module currently doesn't compile w/ kernel 4.10 or higher.

```
/home/heechul/Projects/RT-Gang/throttling/kernel_module/src/overflow.c:114:35: error: request for member ‘tv64’ in something not a structure or union
   delta_time = (u64) (ktime_get ().tv64 - cinfo->core_throttle_start_mark.tv64);

```
check the following memguard commit to support them.
https://github.com/heechul/memguard/commit/56d7deaa8af98f577d3fd7c00b6af91489bcd483


Oho! I will watch this with excitement!

So glad that someone is picking it up :)
Best of luck!

/Tilly


Als je iets als trello/jira/etc gebruikt, kun je beter daar een issue in aanmaken dan TODO's in je code plaatsen.

De reden hiervoor is dat een issue aanmaken een beter overzicht geeft van wat allemaal nog moet gebeuren, en de mogelijkheid geeft om prioritering aan te brengen.


Если дальнейшее выполнение метода не имеет смысла, лучше сразу возврат сделать (early return).
Код будет более линейным

```
return if @user.hand.full?
deal_cards(@user, 1)
@interface.user_cards(@user)
if @user.score > Hand::MAX_SCORE
  @interface.lose_info
  @status = :judge
end
```


السلام عليكم ورحمة الله وبركاته
كيف حالك اخي علي ان شاء الله تكون بخير، اخوك من العراق النجف
اكو مشكلة بالسورس كود الخاص بيك

عندما اختار تفعيل تنبيهات شهر رمضان واختار قبل وقت السحور بـ 1 ساعة مثلا
عند قدوم موعد التنبيه لا يقوم بتشغيل صوت دعاء السحر الذي قمت انت بوضعه داخل مجلد
raw
داخل التطبيق

يقوم بتشغيل صوت التنبيه الافتراضي للجهاز
كيف احل هذه المُشكلة


`return if @status == :judge`


You are right. Changed it. Thanks


The way suggestions/feature requests work on GitHub is that you submit a Pull Request


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


Successfully aliased the URL <https://overreactedio-2ear9q5br.now.sh> to the following alias.

* https://overreacted.io


필요없어졌으므로 삭제하기


初回


How can I use this number to generate all the permutations in an array


Je AccountRole object heeft 2 velden. Het eerste veld is id. Het tweede veld is de daadwerkelijke role. Ik zou zeggen dat je de role moet ophalen aan de hand van de meer permanentere waarde, namelijk het id. Dit heeft ook het voordeel dat dit je primary key is (of in ieder geval hoort te zijn) en dat het zoeken hierop sneller gaat.

Mocht je zeggen "Ja, maar we veranderen de waarde van het `role` veld toch nooit", waarom haal je deze dan uit de database en zet je die niet hard hierin?

Voor de mogelijke rechten zou ik sowieso een statische lijst aanhouden ipv database entries. De waarden uit die lijst kun je dan overal gebruikern en dat voorkomt typefouten. Bovendien lijkt het me niet dat je dynamisch de roles die bij methodes mogen gaat aanpassen in runtime. Dat verslaat mijnsinziens het doel van toegang beperken tot bepaalde rollen een beetje (verbeter mij vooral als jullie een heel goede reden hebben om het over de database te laten lopen).


Thank god it's not Contributor Covenant


```
game_controller.rb:149:22: W: Lint/UnusedBlockArgument: Unused block argument - time. You can omit the argument if you don't care about it.
    number.times do |time|
game_controller.rb:122:29: C: Style/SymbolProc: Pass &:score as an argument to max_by instead of a block.
    [@user, @dealer].max_by { |player| player.score }
game_controller.rb:108:5: C: Style/ConditionalAssignment: Use the return of the conditional for variable assignment and comparison.
    input == 2 ? @status = :finish : @status = :in_progress
game_controller.rb:34:33: C: Style/WhileUntilDo: Do not use do with multi-line while.
      while (@status != :judge) do
game_controller.rb:34:13: C: Style/ParenthesesAroundCondition: Don't use parentheses around the condition of a while.
      while (@status != :judge) do
game_controller.rb:34:7: C: Style/WhileUntilModifier: Favor modifier while usage when having a single-line body.
      while (@status != :judge) do
game_controller.rb:25:32: C: Style/WhileUntilDo: Do not use do with multi-line while.
    while (@status != :finish) do
game_controller.rb:25:11: C: Style/ParenthesesAroundCondition: Don't use parentheses around the condition of a while.
    while (@status != :finish) do
game_interface.rb:99:26: C: Style/UnneededInterpolation: Prefer to_s over string interpolation.
    puts DEALER_SCORES + "#{dealer.hand.score}"
game_interface.rb:30:5: C: Layout/IndentArray: Indent the right bracket the same as the start of the line where the left bracket is.
    ].freeze
game_interface.rb:24:5: C: Layout/IndentArray: Indent the right bracket the same as the start of the line where the left bracket is.
    ].freeze
```


Successfully aliased the URL <https://jameswlanecom-4tbvvtfd0.now.sh> to the following aliases.

* https://jameswlane.com
* https://www.jameswlane.com


Ik zou dit niet in een productie omgeving gebruiken. Het is ideaal voor snelle prototyping, maar je geeft wel heel veel opties om je data te manipuleren die je niet per se wilt hebben. Je kunt bijvoorbeeld een tabel hebben waar nooit zo maar zaken uit verwijderd mogen worden. Jij weet dit, maar weten mensen over 5 jaar dit ook nog? Wil je dat er programatisch zaken aan je tabel met autorisatie roles toegevoegd kunnen worden tijdens runtime? Of dat iemand die per ongeluk leeggooid omdat ze op de verkeerde repository de delete aanroepen en je de ROLE_ADMIN kwijt bent?


[I suggest](https://github.com/idea4good/GuiLite/issues/22 "reserved identifier violation") to [rename this identifier](https://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier#answer-228797 "What are the rules about using an underscore in a C++ identifier?").


Ook dit wordt afgeraden in productieomgevingen. Een kleine wijziging in je entity kan een column, of zelfs een tabel, droppen en dan ben je alle data kwijt. Beter implementeer je iets als LiquiBase om je database wijzigingen bij te houden.


Please remove this empty line.


Acts as a Birthday reminder


Да, туда. Да руки не дойдут


Richard Smith sent me this, so I'm not sure my commit message accurately displays what's going wrong here: https://godbolt.org/z/4LkXCA


Would you please make a PR for this issue?

Sent from Mail<https://go.microsoft.com/fwlink/?LinkId=550986> for Windows 10

________________________________
From: Markus Elfring <notifications@github.com>
Sent: Sunday, February 10, 2019 5:46:57 PM
To: idea4good/GuiLite
Cc: idea4good; Author
Subject: Re: [idea4good/GuiLite] 修改部分英文表达以及修正部分标识符风格 (#23) (356fbd1)

I suggest<https://github.com/idea4good/GuiLite/issues/22> to rename this identifier<https://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier#answer-228797>.

―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/idea4good/GuiLite/commit/356fbd146b0b2964116fad3a085d8b7a9b03cd85#commitcomment-32259269>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AXEfPUR7hDlEnQiJ4dsXMzAX72rM5WOoks5vL-qRgaJpZM4ay-bu>.


Created multidev environment [mgrt#ci-1](https://dashboard.pantheon.io/sites/e6b961cf-1d74-44c6-bb9f-fb1ca37a302f#ci-1).

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


https://youtu.be/fy1AsFrj4qU?t=34

these scripted call for helps seem to ignore LOS, ours dont due to good reason, but i've found that e.g send ai event should also ignore los (Ghaz'an 18105 - Kill Fish in Pond on Death)


> Dear God if your poor phone takes 2.5 seconds to turn on the screen you might have some underlying issues here.

Lmao, this boost it to improve UX right after you unlock your phone.


`$id = $_GET['id'];` и передавай id в url


Убрат `if @user.score > Hand::MAX_SCORE`
Дилер должен ход сделать.


Mijn persoonlijke voorkeur is om velden met annotaties een witregel te geven boven de annotaties en onder het veld. Dat maakt het makkelijker om te lezen welke annotaties waarbij horen, en je hebt een beter overzicht in 1 oogopslag waar zaken beginnen en eindigen.


после удаления, отредиректь пользователя на список
`header('Location: http://127.0.0.1:8000');`


удаляй


## **Question**: Why GSource+GMainLoop were not calling the `ResetEvent` and the `WaitForMultipleObjectsEx ` will not be retriggered if no new message is coming?

normal `g_main_context_iterate`:
1. `g_main_context_prepare` -> `socket_source_prepare_win32` -> `update_condition`
2. `g_main_context_poll` -> `g_poll` -> `poll_rest`
3. `g_main_context_check` -> `socket_source_check_win32` -> `update_condition`
4. `g_main_context_dispatch`
  -> (if new message coming) `socket_source_dispatch`
  -> `update_condition`
  -> `udp_received` (GSource callback by `g_source_set_callback(source, (GSourceFunc)udp_received, NULL, NULL)`)
5. `g_main_context_release` -> x


Are you serious? You stole my N# idea!


Исправь `define_winner`

```
def define_winner
  return if @user.score > Hand::MAX_SCORE && @dealer.score > Hand::MAX_SCORE
  return if @user.score == @dealer.score
  return @dealer if @user.score > Hand::MAX_SCORE
  return @user if @dealer.score > Hand::MAX_SCORE

  [@user, @dealer].max_by(&:score)
end
```


oops!

fixed, thank you.

if you have no further suggestions, I'll run the build scripts.


Geef in het geval van collections liever een lege terug dan null. Op die manier hoef je nooit te kijken of de lijst bestaat als je met het resultaat van de methode wilt gaan werken. Dit scheelt een hele lading null-checks

```getAuthorities().forEach(item -> ???);```

is makkelijker leesbaar minder foutgevoeling dan:
```
if(getAuthorities() != null) {
    getAuthorities().forEach(item -> ???);
}
```


echo 'delete	';


NP


$id = $_GET['id'];


Github pages?


Neutral and Earth not shown for all radial circuits but are present! 2.5mm Twin & Earth cable used for all mains circuits. Lighting circuit 1.5mm T&E cable. Miniature Circuit Breakers (MCB) rated at 16A for radial circuits and 6A for lighting. Kill Switch is a heavy duty 20A (might even be 30A) device with Neon. Easy to acquire in UK and equivalents certainly available in other parts of the world. Do NOT underspecify any cable, connections or switch ratings.


`echo '<td><a href="update.php?id='.$val['id'].'">update</a></td>';`


@SimpleBinary


Successfully aliased the URL <https://feedly-proxy-nvm9ial5v.now.sh> to the following alias.

* https://zo-feedly-proxy.now.sh


@BonfireScratch Read the README


*ToOne is altijd default eager. Je haalt nu dus iedere keer dat je het persoon ophaalt ook het account met alle data op. Het is een beetje jammer omdat dit gewoon onnodige overhead is, en ook gevoelige info zoals het wachtwoord ophaalt.

Dit kun je oplossen door de koppeling tussen account en persoon alleen op account te zetten, of in ieder geval door de fetch op lazy te zetten.

Mocht je het op willen lossen door de koppeling alleen op account te zetten, zet dan ook `@OneToOne(fetch = FetchType.LAZY)` omdat je het persoon natuurlijk alleen maar nodig hebt als het inloggen slaagt.


inverse-model (XuanLuoLBL) - Win64-Windows-7-VisualStudio-14: Build Failed

Failures:

integration Test Summary
 * Passed: 1
 * Failed: 639

regression Test Summary
 * Passed: 636
 * Failed: 2

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


Can you please follow the coding style of the function you're modifying?  Right above this is an `if (...)` with space between keyword and control expression but you've added two `if(...)` with no space.


Super nice. If you want the one liner, `functools.reduce(lambda x, y: x * y // math.gcd(x,y), range(1,21))`?


Nope. It exposes the entire instance but the default chip method does exactly this.


Could you please put a link to this file in the README.md ?


albo przykładowo pobrać tylko właściwą implementację repozytorium z metodą edytująca votes count ;)


Same to you!


As I read, the articles about this identifier mention a lot that people should not use `localhost`.

I have in my case postfix installed on the web server. And the `localhost` as the SMTP setting. This line of code will therefore produce `localhost` as the `EHLO` identifier.

Maybe a better default `host` (if the user does not provide it in `options.heloidentifier`) should be computed from the sender's email address (`emailsender`): `gabriel@mydomain.com` -> `mydomain.com`


Thank you so much :) What an exceptional application joplin is. Deep respect... Donation coming...


Well, it was this:

> Yeah, just a barebones one to get someone jumpstarted. Also makes it easier if someone is just test-driving the app.

With a default config with just PR title rules, it lets people install the app and quickly try out things without having to read or figure out rule details. One may also argue that this may cause folks to _NOT_ explore further.

I'm wondering if we should combine the two appraoches - have the default lint just the title but then the message could be something like:

```
{
  state: 'success',
  description: 'PRLint is setup with the default rules. To provide your own, click details to the right',
  target_url: 'https://github.com/ewolfe/prlint#example-rules'
}
```
_(feel free to mess with the messaging above)_


[./acronym/test_acronym.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./acronym/test_acronym.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/acronym/acronym.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./bin/run_lints.R:12:1:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./bin/run_lints.R#L12) *style:* **lines should not be more than 80 characters.**
```r
files <- list.files(path = ".", pattern = ".+\\.R$", recursive = TRUE, full.names = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./bob/bob.R:6:1:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./bob/bob.R#L6) *style:* **lines should not be more than 80 characters.**
```r
    if ((input == toupper(input)) && (unlist(gregexpr("[A-Z]", input))[1] > 0)) "Calm down, I know what I'm doing!"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./bob/bob.R:9:1:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./bob/bob.R#L9) *style:* **lines should not be more than 80 characters.**
```r
  else if ((input == toupper(input)) && (unlist(gregexpr("[A-Z]", input))[1] > 0)) "Whoa, chill out!"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./bob/test_bob.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./bob/test_bob.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/bob/bob.R")
        ^~~~~~~~~~~~~~~~~~~~~~
```
[./difference-of-squares/test_difference-of-squares.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./difference-of-squares/test_difference-of-squares.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/difference-of-squares/difference-of-squares.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./grains/test_grains.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./grains/test_grains.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/grains/grains.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./perfect-numbers/perfect-numbers.R:5:30:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./perfect-numbers/perfect-numbers.R#L5) *style:* **Put spaces around all infix operators.**
```r
  small_divisors <- numbers[n%%numbers == 0]
                            ~^~~
```
[./perfect-numbers/test_perfect-numbers.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./perfect-numbers/test_perfect-numbers.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/perfect-numbers/perfect-numbers.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./raindrops/test_raindrops.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./raindrops/test_raindrops.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/raindrops/raindrops.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./rna-transcription/rna-transcription.R:1:28:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./rna-transcription/rna-transcription.R#L1) *style:* **Commas should always have a space after.**
```r
replace <- function(strand,rna_map){
                           ^
```
[./rna-transcription/rna-transcription.R:9:59:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./rna-transcription/rna-transcription.R#L9) *style:* **Put spaces around all infix operators.**
```r
  paste(sapply(dnal, replace, rna_map = rna_map), collapse='')
                                                         ~^~
```
[./rna-transcription/test_rna-transcription.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./rna-transcription/test_rna-transcription.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/rna-transcription/rna-transcription.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./secret-handshake/test_secret-handshake.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./secret-handshake/test_secret-handshake.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/secret-handshake/secret-handshake.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./space-age/space-age.R:2:1:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/space-age.R#L2) *style:* **lines should not be more than 80 characters.**
```r
  earth_age = list("earth" = 1.0, "mercury" = 0.2408467, "venus" = 0.61519726, "mars" = 1.8808158,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./space-age/space-age.R:2:13:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/space-age.R#L2) *style:* **Use <-, not =, for assignment.**
```r
  earth_age = list("earth" = 1.0, "mercury" = 0.2408467, "venus" = 0.61519726, "mars" = 1.8808158,
            ^
```
[./space-age/space-age.R:3:1:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/space-age.R#L3) *style:* **lines should not be more than 80 characters.**
```r
                  "jupiter" = 11.862615, "saturn" = 29.447498, "uranus" = 84.016846, "neptune" = 164.79132)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[./space-age/space-age.R:4:16:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/space-age.R#L4) *style:* **Put spaces around all infix operators.**
```r
  round(seconds/(31557600* earth_age[[planet]]), 2)
              ~^~
```
[./space-age/space-age.R:4:26:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/space-age.R#L4) *style:* **Put spaces around all infix operators.**
```r
  round(seconds/(31557600* earth_age[[planet]]), 2)
                        ~^
```
[./space-age/test_space-age.R:1:9:](https://github.com/tqa236/r_exercism/blob/ab1bd4ba16ff12deb22d35d30e4b0de2da464a96/./space-age/test_space-age.R#L1) *warning:* **Do not use absolute paths.**
```r
source("~/exercism/r/space-age/space-age.R")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Hi! This is the friendly automated conda-forge-webservice.

I updated the Github team because of this commit.

- @nathan-toubiana was added to conda-forge. Welcome to conda-forge!
  Go to https://github.com/orgs/conda-forge/invitation see your invitation.

You should get push access to this feedstock and CI services.

Feel free to join the community [chat room](https://gitter.im/conda-forge/conda-forge.github.io).

NOTE: Please make sure to not push to the repository directly.
      Use branches in your fork for any changes and send a PR.
      More details [here](https://conda-forge.org/docs/conda-forge_gotchas.html#using-a-fork-vs-a-branch-when-updating-a-recipe)


hehe love it! tag yours as q5-m.py and push for record :D


didn't know python had reduce


### Visual regression report:

[![Visual report](https://circleci.com/gh/stevector/nerdologues-d8/2433/artifacts/0/artifact/backstop_data/bitmaps_test/20190211-000749/0600294051_Clips_by_person_0_document_0_desktop.png)](https://circleci.com/gh/stevector/nerdologues-d8/2433/artifacts/0/artifact/backstop_data/html_report/index.html)


Me faltó un espacio, plz help


Successfully aliased the URL <https://allostasis-oy6567d7e.now.sh> to the following alias.

* https://allostasis.sandboxneu.com


Most of this is either ancient code I never used (gradient stuff) or unfinished, beta code (ScreenSelectMusicCasualManageGroups) and can be removed from the theme.

I think all you really want is BGAnimations/ScreenEvaluation common/PerPlayer/Pane5/default.lua


`std::hex` and `std::setfill` are "sticky" - you can lift them out of the loop