The difference is not easily apparent here. Note there is no ending semicolon (;) before my deeply indented line.


Где объявления через интерфейс или абстрактный класс?
Я знаю что говорил, что допускается объявление через имя класса, но в данном случае тебе нужно сделать объявление через интерфейс и привести тип, там где нужно, на одном методе.


I guess that there is a typo here.


indentation can be just 2 whitespace instead of 4


missing extra empty line at the end


nice one


@kornholi wouldn't this be the same as:

```python
        if len(array.shape) > 2 && order == 'image':
                return array.transpose((1, 2, 0)), metadata

        return array, metadata
```


nice :)


this was an important feature of the weapon: to occupy target's AM systems for the carrier's heavier ordinance (especially with the very low shield-damage attribute)...=(


wrong commit message (there are no modifications)


Общее замечание:
Учись правильно называть классы. Где телевизор и пульт Горизонт?


extra line, not needed


I would like to suggest using `signed-in`.
And if you would like to change `guest` to `guests`,  please also change it in line.


I think there's a mistake here, ReactJS SP has been put on the wrong event.


Ok you suggest to use relative path instead of absolute one.
Something like:
```cmake
include_directories(include/LIEF)
```


extra lines, not needed


This actually belongs to the first part of the diff.


Very neat!! :)


Please place a more descriptive name!


extra line


Lol


tro for se loik


Isn't one already defined by Android?


according to java styleguide, you can use the ' else ' right after the closing ' } ' of the if statement, in the same line.


friend of one class member of another class


I mean parent blocks of the new ones that have been added, like `header-cell` etc.


:+1: great 😄


sorry about the whitespaces


afaik just `out _` will do the job


doesn`t need 2 empty lines at end of the code


r code for correlation heatmap on tmdb


2 empty lines instead of 1


Make sure these are invalidated


Jajajaja q hacés por acá? Encima no puedo hacerlo andar al muy taimado. Es infinito lo que me falta aprender.


@Sami32 Yes, I understand that we use both, but none are defined. That's why I said that we should add both.


do we need a constructor, if we don't have any specification in it/parameters?


extra line and the one below too


Thank you.
But I'm not copy to SYSTEM permissions.
Cannot finish the test.
I am using KODI 17.1 arm64 v8a.
Because 60Hz is not compatible with the TV Memc function.
Could you tell me how to set the 24Hz for KODI or RKMC?
Although RK3399TVBOX is set to 24Hz,The KODI is always 60hz...


should be declared before the methods if used in more than one method.


I think one commit should only have one file in it


Thanks for the pointer, fixed.

On Tue, Apr 4, 2017 at 6:53 PM, Shashidhara T D <notifications@github.com>
wrote:

> Probably, we need to fix this todo before this pr gets merged, since
> federation namespace is not a fixed one. You can make use of this function
> FederationSystemNamespace(https://github.com/kubernetes/
> kubernetes/blob/master/test/e2e/framework/federation_util.go#L27) to get
> the federation namespace
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/kubernetes/kubernetes/commit/9d8f93538c66d7c405d11707d9a5aef584db2e22#commitcomment-21630160>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAbjleat-mGNHYRiwWx1gjeSJGK6quf6ks5rsvQzgaJpZM4MzpqO>
> .
>


you don't need this extra line here


You made me realize that I had many mistakes today . Thanks! :)


Maybe a more detailed description of what is fixed...


So... Yeah... Something definitely happened here. I don't think it messed anything up though. Looks like it just removed and then added everything back... So I don't think anything was actually changed. We shall see. Sound engine is still there.


By @vuntz, not untz. Sorry.


you should leave an empty line at the end


The package name should be all lowercase and should not contain "_".

I'm just gonna copy this part from the styleguide, I also learned this today:

Package names are all lowercase, with consecutive words simply concatenated together (no underscores). For example, com.example.deepspace, not com.example.deepSpace or com.example.deep_space.


there should be an extra line at the end of the file


Well I got Software OpenGL working on Ivy Bridge using the same method you did with clip control :)


thats not going to work, in line 424 and below some stiff in the same function reffers to this variable


do leave an empty line after the file


extra line


- conversion to/from double is missing in decfloat library, inside engine it's done using intermediate ascii string. I.e. I suppose that one can get a string (or sign/BCD/exponent trio) and after it use any method to build double.


This looks so nice, clean and readable


add link to commit used


cool 👍


we don't need empty test


fix formatting ("...cortex ")


After this change I am getting this now:


Do a pr ;)


what script? point to where you set your path


- scatter\gather - because it's not just BCD, it's group of 3 objects -  sign, BCD and exponent, but I can rename if you wish


say which figures (e.g. "Figures 1-4")


always put a new line at the end of each class (I don't know why, Peti told me - ask him :P )


say which figures (5-7?)


say which figures


do you need this @BeforeEach section at all? :)


I retrieved that from a 1 year old issue report:
`SEVERE - 20:22:07:250 - AWT-EventQueue-0 : ContentDirectoryService Browser : Error: Current state of service prevents invoking that action. Can't parse DIDL XML response: org.teleal.common.xml.ParserException: org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 2579; Le préfixe "pv" de l'attribut "pv:subtitleFileType" associé à un type d'élément "res" n'est pas lié..`

Taken from an old Kodi thread, but Rigel and MiniDLNA say the same:

> Basically, in the ContentDirectory SOAP request, 8player asks for:
> ...<Filter>...,sec:CaptionInfoEx,pv:subtitleFileType,res@pv:subtitleFileUri,pv:subtitlefile</Filter>...
>
> And minidlna returns:
> ... pv:subtitleFileUri="http://192.168.1.7:8200/Captions/6069.srt" ...
> sec:CaptionInfoEx sec:type="srt">http://192.168.1.7:8200/Captions/6069.srt</sec:CaptionInfoEx>
>
> A simple http get to subtitleFileUri returns the subtitle data.

I remember having asked about that here some months ago, and get how to disable it, it's as i discover that my external subtitles will not be available but the internal was still working.

It's seem that something is incorrect, i'll do some more test later.


and fix typo "independandly"


didn't thought about reusing the count letters function here, but looks very interesting.


@papr @mkassner


use bulletpoints or some other list layout, right now this is all displayed together making it hard to read


shouldn't it still raise so that we know something is wrong? depends the conditions for which paypal sends a pdt error tbh... not sure


Ты убрал отсюда объявления переменных `$falseBtn` и `$rightBtn`, но вместе с ними ты убрал и `let`.
Без него код будет работать, но так делать очень не рекомендуется.
Верни `let` для переменной `counter`, а ещё лучше — объяви её как `const`.


we should use horizontal whitespaces between the 'if' and the '( )' just like you do after 'for'. Apart from this, the code looks very neat and easy to read. I like how you name you variables and your functions as well.


Don't leave outcommented text in a code :)


it's an interesting solution :)


You could define fibonacciSequence[] outside test so you shouldn't have to repeat it all here.


Could be tested with negative input. Otherwise super neat!


Nice solution:)


I assume self.pricelist_ids only contains pricelists that are selectable? if yes, lgtm (but you're the expert here)


First fetch


This is the nice and clean way to do this, I think many people would just put it in 1 line and it would be unreadable. Good job! :)


according to google java styleguide you need a space before {
otherwise nice code :)


Why are you importing commands from discord.ext when the code you are using doesn't require it as far as I can see


Something's not right, Perry. I have to brush up on my sync methods.

If I grab a lock then sem_wait, I remember something that would give up the lock if I go to sleep. Can't remember. Last time you used this stuff was only last term. You're as bad as your students!


I hope it's a temporary measure, so I'm willing to uncomment it back when the multiplatform limitation is lifted.


You can compare Strings with '.equals()' because they are not primitive types.  '==' works only with primitive type variables.


try to commit the exercises separately :)


Nice code


pour tout ce qui est style, mieux vaut que tu passes par un fichier css externe et que tu crée les classes dont tu as besoin


nice fix thank you !


Yes, I think so. That script shouldn't be getting installed. I don't think it is if you install it with `pip install .` rather than `python setup.py install`, though, and the former is always recommended now.


line 28 of this file:
```
output: {
          path: `./public/assets/`,
```
seems like another duplication.


You do not need these imports


Yes I think so, thanks oleohi !


Test it for a nexative number, what will happen? Hmmmmm :)


Great job!


Leave one empty line at the end of the file... I was told as well to do it so :)


Love this commit message :D :D


you are comparing null to "list" which is given the value null just a line before -- it doesn't do anything with your Sum methods


mt bom!


@Sami32 See if #1250 fixes this.


clean job! ;-)


Sauf le drag'n'drop d'images, qui est top!


recursion, nice! :)


put spaces. Same as before


@sarahpearson , you can do the pull request on this.


Bluemix [toolchain](https://dev-console.stage1.ng.bluemix.net/devops/toolchains/00de3605-dfd5-439e-a720-757ec84db3af "simple-toolchain-20170405145532368"): [Delivery Pipeline](https://dev-console.stage1.ng.bluemix.net/devops/pipelines/7daacc6b-0269-47fc-8120-8746a8f2f45b "simple-toolchain-20170405145532368") deployed [simple-toolchain-20170405145532368](https://github.com/jerome-lanneluc/simple-toolchain-20170405145532368/releases/tag/deploy-ibm-ys1-us-south-jerome_lanneluc-fr.ibm.com-dev-20170405-150641 "deploy-ibm-ys1-us-south-jerome_lanneluc-fr.ibm.com-dev-20170405-150641") to [dev](https://dev-console.stage1.ng.bluemix.net/apps/1c0fcc3d-8da7-4751-8768-eb1a53c84dff "ibm:ys1:us-south:jerome_lanneluc@fr.ibm.com:dev"), including this commit


It might be better to do separate commits for at least each exercise.
Smaller commits and more specific commit messages can be useful later, for you as well as when collaborating with others.
(I'm mostly just parroting what Tojás said :wink:)


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


Move the implementation of `avx_mul` to a new file such as `matrix_avx.c`.


@ecarmichael, I added some comments on the readme file. Also, I'm wondering about the two folders Eric_naris and Naris_paper, those are confusing names: are both needed? Can we come up with more informative names?


put spaces also here: }else{ --> } else {


woops, meant to say "linear_extrude" in that commit comment


test


Initial occurence will always be 0, should be 1.


# CANBAN

## СБОРКА
Проект разрабатывался с использованием сборки Yeoman Gulp-Angular Generator. Детальное описание сборки, а также ее установки и настройки доступны по ссылке: [https://github.com/Swiip/generator-gulp-angular](url)

## ВНУТРЕННЯЯ СТРУКТУРА ПАПОК ПРОЕКТА (корневая папка src)

## ОПИСАНИЕ ПРОЕКТА
Проект выполнен с использованием Angular 1.5, в компонентном стиле и содержит 6 компонентов:

- main - основной, родительский
- add - отвечает за создание новой задачи
- doit, inProgress, done, aborted - соответствующие статусы выполнения задачи.

Вся логика содержится в main.service.

Сортировка задач по приоритету и времени также реализована в сервисе, выполняется на этапе создания или редактирования массивов задач. Использование фильтров в данном проекте показалось мне неуместным, т.к. сортировка "на лету" при редактировании приоритета "выдергивает" задачу из поля видимости пользователя и переставляет в место с новым приоритетом, что явно не есть usability.

Место хранения задач - localStorage.


Oh yeah.


Very smart way of using enhanced loop,  well done!


I'm not sure if you really need these empty lines inside your '@Test' section


Refactoring to this scale will most certainly allow us to uncover bugs that we never knew about. I followed other native functions — nearly all of them use C types rather than Ruby VALUE.

For example, `shoes/native/gtk/gtkcheck.c`:

```c
void shoes_native_check_set(SHOES_CONTROL_REF ref, int on);
```

Let's just say that if you have found a bug, we may have to fix a whole lot of things.


Typo: should be "actual".


relatedly, there seem to be multiple instances of the BuzCSD folder, pls remove all but one


Would have been nicer to upload the original code then the changes. We can't see what yo changed. Otherwise, good.


I fixed some of the comments.  I am not sure about this path setting script.  The paths used are set at the top of the "...MASTER" files.

The Eric_Naris is the 'sandbox' code while the Naris_paper is the final code.


@cpicanco I can't follow. Could you provide some context? From where are you calling
> `super().__init__(gpool)`


Thanks a lot @xavijam! Less code to worry about!


leave spaces before the equation sign as well, like this: ' = '


We don't need this, we can easily fix this in another way with your existing implementation. I'll raise a PR, kindly have a look.


When SobelX and SobelY are being used in collision detection, there will need to be a default/adjustable threshold that will allow certain collisions to be ignored, that is ignore edges that are not significant enough to allow objects to pass through them. Make the threshold adjustable to the user to allow for a better adaptability from one image to a completely different image.


it looks like you are using the circstat toolboxes and also something called altmany? make sure those are acknowledged somewhere


Why is this removed from here?
Now, `_last_gps_origin_time_us` is updated only [after all gps checks pass](https://github.com/wingtra/ecl/blob/90150d6fcab1ce0c6298e5fa3f6f4009c7b234cd/EKF/gps_checks.cpp#L80), so this will never get updated if some test fails from the very beginning. Since this timestamp is used to calculate `dt` (line 153), `dt` will just grow and eventually saturate at `filt_time_const`.


if there isn't a path setting script then the readme doesn't need to mention one


I have no idea what's going on in this excersize but i like the package name! :D


you don't need this empty line


You forgot to check for negative numbers... Just like me :)


I think it's safe to say GBA support has moved beyond experimental. Largely stable. NDS support, on the other hand...


nor this one :-P


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/ff0a3bab-1e0d-4687-a2f5-da830e98bbbd "dev0-1"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/b947e447-9d2d-486f-98c3-ff166c73b66c "dev0-1") deployed this commit to [fisrtDev](https://console.ng.bluemix.net/apps/101cded5-90bb-46f7-a3c5-a4ad48f701dd "ibm:yp:us-south:projectBen11:fisrtDev")


Will do after I do some tests , wanna test if OpenGL works on my old nvidia 9300 gs xD


TeamCity Glyssen :: Glyssen pull requests [Build *.*.528.7a6f058f018c3ce61fa5bca6b2c317a0e54fdc68](http://build.palaso.org/viewLog.html?buildId=87514&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1666, ignored: 18 Build time: 00:07:13


root@f02ps:/usr/src/sys/dev/ixl# pciconf -lvBv
hostb0@pci0:0:0:0:      class=0x060000 card=0x00000000 chip=0x71928086
rev=0x03 hdr=0x00
    vendor     = 'Intel Corporation'
    device     = '440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled)'
    class      = bridge
    subclass   = HOST-PCI
isab0@pci0:0:7:0:       class=0x060100 card=0x00001414 chip=0x71108086
rev=0x01 hdr=0x00
    vendor     = 'Intel Corporation'
    device     = '82371AB/EB/MB PIIX4 ISA'
    class      = bridge
    subclass   = PCI-ISA
atapci0@pci0:0:7:1:     class=0x010180 card=0x00000000 chip=0x71118086
rev=0x01 hdr=0x00
    vendor     = 'Intel Corporation'
    device     = '82371AB/EB/MB PIIX4 IDE'
    class      = mass storage
    subclass   = ATA
none0@pci0:0:7:3:       class=0x068000 card=0x00000000 chip=0x71138086
rev=0x02 hdr=0x00
    vendor     = 'Intel Corporation'
    device     = '82371AB/EB/MB PIIX4 ACPI'
    class      = bridge
vgapci0@pci0:0:8:0:     class=0x030000 card=0x00000000 chip=0x53531414
rev=0x00 hdr=0x00
    vendor     = 'Microsoft Corporation'
    device     = 'Hyper-V virtual VGA'
    class      = display
    subclass   = VGA
none1@pci43312:0:2:0:   class=0x020000 card=0x00000000 chip=0x152f8086
rev=0x01 hdr=0x00
    vendor     = 'Intel Corporation'
    class      = network
    subclass   = ethernet

2017-04-05 17:12 GMT+03:00 Dexuan Cui <notifications@github.com>:

> dev/ixgbe/if_ix.c:2644: "Unable to map MSIX table \n");
> dev/ixgbe/if_ixv.c:1382: "Unable to map MSIX table \n");
> dev/ixl/if_ixlv.c:1195: "Unable to map MSIX table\n");
> dev/ixl/ixl_pf_main.c:1398: "Unable to map MSIX table\n")
>
> Can you please find out where "Unable to map MSIX table" message comes
> from and check the rid before the line bus_alloc_resource_any()? the
> rid/BAR number must match with the BAR number shown in "pciconf -lvBv".
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/dcui/freebsd/commit/0d205f947a262bd0cde2bfa4d783ad66fe50fb92#commitcomment-21638169>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACJn3vD1UxFUZhfSy_I4bU6ZtGyJ1TP4ks5rs6FegaJpZM4Mxp9->
> .
>


This can be done with only one loop
define occurence outside loop to 1 but you could skip occurence with dictionary.get(wordToCount.charAt(i))


fine indentation


you could have used bit more informative function names


``ResponsAbility``? :-)


@rianby64 разобрался, работает правильно


if I'm going to test the whole workflow I'd like to run a version as close to final as possible, and want to avoid something like that BuzCSD no longer works because we deleted a version that is referred to some place -- what do you think about removing everything we don't need first before I do a run through?


Hi, This is a problem if we want to run a config server (with command: "--configsvr" ) because it runs on port 27019, so the test is never a success.
This commit fails all my stack since I updated it :/


leave one empty line on the bottom of your files


See issue #62.


we upgrade the osticket to latest version and we try to implement this code there its not working. Pls assist.


Finally something to comment on... You forgot to check for negative numbers... Just like me... But Patrick commented on this my code :)


Just a question about the volume filter deprecation.
When using --volume (speaking of sotfvol), will it operates at the end of the chain?
I'm asking because when using audio compressors effects, having the volume filter after them is required to not feed the audio compressor itself with a modified volume of course.
thanks.


Also "} else {" should be in the same line, without line breaks.


could be simplified, there is no need to type in elementvalue on each test and then type it in again at the assertEquas' expected value


Your code looks quite good altogether. It's interesting that you made some mistakes in some of the exercises but the same stuff was correct for others. Altogether it's easy to read and clear. Not much to criticised.


note that the bin folder of the original project is not necessarily available anymore after deployment, so if this fixes the problem we have to think about a more general solution where cached configs are not written anymore, for example if `org_pcfg != current_pcfg`, or if `!exists(org_pcfg.bin)`, or something?


no need for empty line here :)


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


Niether me :) but the code looks neat and readable, hope it works perfectly.


put spaces before and after operator ('something + 1' instead of 'something+1')


root@f02ps:/# pciconf -lbv none1@pci43312:0:2:0:
none0@pci43312:0:2:0:   class=0x020000 card=0x00000000 chip=0x152f8086
rev=0x01 hdr=0x00
    vendor     = 'Intel Corporation'
    class      = network
    subclass   = ethernet
    bar   [10] = type Memory, range 32, base 0xfc000000, size 16384, enabled
    bar   [1c] = type Memory, range 32, base 0, size 16384, enabled

2017-04-05 18:11 GMT+03:00 Michael Pro <michael.adm@gmail.com>:

> root@f02ps:/usr/src/sys/dev/ixl# pciconf -lvBv
> hostb0@pci0:0:0:0:      class=0x060000 card=0x00000000 chip=0x71928086
> rev=0x03 hdr=0x00
>     vendor     = 'Intel Corporation'
>     device     = '440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled)'
>     class      = bridge
>     subclass   = HOST-PCI
> isab0@pci0:0:7:0:       class=0x060100 card=0x00001414 chip=0x71108086
> rev=0x01 hdr=0x00
>     vendor     = 'Intel Corporation'
>     device     = '82371AB/EB/MB PIIX4 ISA'
>     class      = bridge
>     subclass   = PCI-ISA
> atapci0@pci0:0:7:1:     class=0x010180 card=0x00000000 chip=0x71118086
> rev=0x01 hdr=0x00
>     vendor     = 'Intel Corporation'
>     device     = '82371AB/EB/MB PIIX4 IDE'
>     class      = mass storage
>     subclass   = ATA
> none0@pci0:0:7:3:       class=0x068000 card=0x00000000 chip=0x71138086
> rev=0x02 hdr=0x00
>     vendor     = 'Intel Corporation'
>     device     = '82371AB/EB/MB PIIX4 ACPI'
>     class      = bridge
> vgapci0@pci0:0:8:0:     class=0x030000 card=0x00000000 chip=0x53531414
> rev=0x00 hdr=0x00
>     vendor     = 'Microsoft Corporation'
>     device     = 'Hyper-V virtual VGA'
>     class      = display
>     subclass   = VGA
> none1@pci43312:0:2:0:   class=0x020000 card=0x00000000 chip=0x152f8086
> rev=0x01 hdr=0x00
>     vendor     = 'Intel Corporation'
>     class      = network
>     subclass   = ethernet
>
> 2017-04-05 17:12 GMT+03:00 Dexuan Cui <notifications@github.com>:
>
>> dev/ixgbe/if_ix.c:2644: "Unable to map MSIX table \n");
>> dev/ixgbe/if_ixv.c:1382: "Unable to map MSIX table \n");
>> dev/ixl/if_ixlv.c:1195: "Unable to map MSIX table\n");
>> dev/ixl/ixl_pf_main.c:1398: "Unable to map MSIX table\n")
>>
>> Can you please find out where "Unable to map MSIX table" message comes
>> from and check the rid before the line bus_alloc_resource_any()? the
>> rid/BAR number must match with the BAR number shown in "pciconf -lvBv".
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/dcui/freebsd/commit/0d205f947a262bd0cde2bfa4d783ad66fe50fb92#commitcomment-21638169>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/ACJn3vD1UxFUZhfSy_I4bU6ZtGyJ1TP4ks5rs6FegaJpZM4Mxp9->
>> .
>>
>
>


I have a class that inherits from the offline surface tracker and I must initialize in `__init__.py`:

``` python
class Offline_Screen_Tracker(Offline_Surface_Tracker,Screen_Tracker):
    """
    Special version of screen tracker for use with videofile source.
    It uses a seperate process to search all frames in the world.avi file for markers.
     - self.cache is a list containing marker positions for each frame.
     - self.surfaces[i].cache is a list containing surface positions for each frame
    Both caches are build up over time. The marker cache is also session persistent.
    See marker_tracker.py for more info on this marker tracker.
    """
    def __init__(self,g_pool,mode="Show Markers and Surfaces", min_marker_perimeter = 100,robust_detection=True, matrix=None,test=None):
        #self.g_pool = g_pool
        Trim_Marks_Extended_Exist = False
        for p in g_pool.plugins:
            if p.class_name == 'Trim_Marks_Extended':
                Trim_Marks_Extended_Exist = True
                break

        if not Trim_Marks_Extended_Exist:
            from trim_marks_patch import Trim_Marks_Extended
            g_pool.plugins.add(Trim_Marks_Extended)
            del Trim_Marks_Extended

        # heatmap
        self.matrix = matrix
        self.heatmap_blur = True
        self.heatmap_blur_gradation = 0.12
        self.heatmap_colormap = "viridis"
        self.gaze_correction_block_size = '1000'
        self.gaze_correction_min_confidence = 0.98
        self.gaze_correction_k = 2
        self.heatmap_use_kdata = False
        super().__init__(g_pool,mode,min_marker_perimeter,False,robust_detection,)
```


I think you should handle the case, when any two are equal. Now if a=b or b=c or a=c,  it will also return c, not just in the case when c is thevbiggest.


I just learned this today myself, It is fortunate to leave an empty line at the end of your files, after the last closing bracket.

Quoting the reason for this:
"...lines not ending in a newline character aren't considered actual lines. That's why some programs have problems processing the last line of a file if it isn't newline terminated."


Tohle bych nevyžadoval. Dává mi smysl, abys volitelně mohl povolit read-write přístup přes git i neregistrovaným uživatelům, přičemž daná editace by byla asociovaná s použitým SSH klíčem a standardní git identitou (jméno a emailová adresa autora a commitera). Může to fungovat až tak, že se takovému uživateli vygeneruje nějaký token, který dostane ve zprávě od git serveru při pushnutí, pro přihlášení přes webové rozhraní.


OK, from chat I now understand: I just forgot to delete the comment at line 96. Will fix that now.


neat formatting, good job


Created multidev environment [ci-drops-8#c8-51](https://dashboard.pantheon.io/sites/689219ca-6583-4af8-ab05-2cebf6ef79a0#c8-51).

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


NO!


This key has now been changed and the file will be ignored.


smart solution! I have not thought about negative input parameters!


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/ff0a3bab-1e0d-4687-a2f5-da830e98bbbd "dev0-1"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/b947e447-9d2d-486f-98c3-ff166c73b66c "dev0-1") deployed this commit to [fisrtDev](https://console.ng.bluemix.net/apps/101cded5-90bb-46f7-a3c5-a4ad48f701dd "ibm:yp:us-south:projectBen11:fisrtDev")


actor_id убрать


No the on exit handler gets executed every time we exit the server doesnt it? So in two events:
1. We're out of work
2. Somewhere process.Exit(...) Was called

The pid file does stay if the server gets started a second time since neither of these cases occured then.


This is function.h for mbed flash compiler


thirst and hunger are increased when play, not decreased


'absent' is also a string


`__init__` is called on both super classes. `Screen_Tracker` probably does only take 5 arguments (incl `self`) and this is the place that throws the exception. Python Mixins are confusing. Also: I would super.__init__ in the beginning, not in the end.


indent this line by two spaces


there are some comments left in both files :)


Можно убрать эту таблицу


Useful auto-formatting in IntelliJ using a pre-created Google Style Guide template:

Download this file:
[intellij-java-google-style.xml](https://raw.githubusercontent.com/google/styleguide/gh-pages/intellij-java-google-style.xml)

How to put it in IntelliJ:
[Installing the google styleguide](https://github.com/HPI-Information-Systems/Metanome/wiki/Installing-the-google-styleguide-settings-in-intellij-and-eclipse#macintosh)
Use the Macintosh guide for Windows & Linux also.


getting there! ;-)


Awesome, thank you, @HelenCampbell ! Fix the one thing I marked and go ahead and squash the commits, and I'll merge it after that. Yay!


Эту тоже


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


You do not need this to run the function just call the print


No we dont! :)


> Každý předmět má jeden repozitář (vlastní wiki)…

Tu volbu, zda wiki, resp. jmenný prostor, bude samostatný repositář či třeba větev v jednom repositáři, bych nechal zatím otevřenou. Použití větví pro jmenné prostory mi začíná připadat jako lepší řešení. Zprvu jsem měl obavy o transparentnost pro uživatele při přístupu přes git, ale ono to vlastně vyjde úplně nastejno.


Call the function during the test


Good, clean formatting! :+1:


Much appreciated!


```
my old nvidia 9300 gs
```
Not with the Nvidia driver (fine on Linux with Mesa). It misses the mandatory `arb_buffer_storage` extension.


I'm not quite sure whether you really need this part in your Sum class


It doesn't seem to be working; it's just sitting in the queue, but still hasn't started.


just curious why?  I'm assuming it has something to do with Vagrant or Windows or something since I haven't needed to do that.  Thanks!


you don't need this empty line


An easy solution to your problem would be to add `*args, **kwargs` to your `Screen_Tracker.__init__` function. This way you do not have to update you class every time that we change the constructor signature...


Following our [styleguide](https://google.github.io/styleguide/javaguide.html#s4.1.1-braces-always-used) you sould use `{ }` even when it works without it.


Good commit message.


Ano, je to tak. https://about.gitlab.com/2015/03/03/gitlab-acquires-gitorious/


Good idea!


hy!! its  me suman stha,here  from  nepal.


To support #194


Can we get a unstable release with this fix included?


Indeed.


Nemá žádnou odlehčenou verzi, ani není modulární. Je to stále více a více kynoucí monolitický moloch. :/


you should pay more attention to spaces. I can see that you know how to use them correcrtly, because in line 19 you left a space between ')" and '{' but sometimes you forget about it, or it's just a typo.


Fix for issue #49


Leave one empty line on the bottom of the files


Currently it works that way, and it will keep working that way. Whether it will insert a volume filter is a different question.


Tested and submitted https://git.eclipse.org/r/#/c/94474/
Thank you for finding this.


The doctests actually are still being run on the CI, via the tox command.
You can do:
pytest --pyargs fontTools
To run them all, or
pytest --pyargs fontTools.ttLib
To only run that module's doctests


What did you mean by 'all the tests'? Once I opened it I saw that it was for the Sum exercise, but you should be more specific in your commit messages. I'm also not good at this, but I think we should get used to it :)


no need for empty lines here :)


|`line 395`|
|---|
I think this function, along with moveFilesOrDirectories(), should be extracted to a separate file. There's a lot of stuff in moveUtils.kt, and these two functions are on a different level from the rest of the logic here.


用户模块完成，4-5日


you forgot to check is for negative numbers (-->Patrick's comment on Norbi's-->Norbi's comment on mine as well)


If you still use that run-tests.sh script I can make it do so if you like.


Suggestion: say what class / package / exercise etc. is the commit about. (e.g. "add Sum test case").

Don't know if is actually recommended by the mentors or not though. :blush:


If I understand correctly you start counting from n=1 and assume the value is 0, while mathematically it is more fortunate to calculate from n=0 (which = 0). The reason why I spotted this is, that in your test-case the 6th element has a value of 5, but mathematically the 6th element = 8, and the 5th element = 5. I hope it's clear what I'm trying to say. Your function will work well and error-free nonetheless, this is just a suggestion.


no


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:583`|
|---|
I think this is far too compressed; would be more readable without `apply()`.


I think the docstring in `function` is not accurate enough: `Defines a function, making all words found in body local. ` It should be `all set-words`.


Very nice, clear commit. :+1:


this looks cool!


We should just use `bool` instead. Cocoa doesn't understand `gboolean`.

http://wiki.inkscape.org/wiki/index.php/Booleans


Maybe you should put spaces between the parameters to make them easier to read, like you did some lines above. `Arrays.asList(10, 20, 40, 30)`


leave a space after the closing parenthesis: '(int) waterAmount'


Fix


Hmm what about sw? Is it needed on sw render ?


clear structure!


Nice clear code.


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:524`|
|---|
Does this work correctly when we're moving an inner class of an inner class? Looks like it handles only one level of nesting.


> scatter\gather

even library used decXXXToBCD\decXXXFromBCD :)
I think it is more user-friendly names.
Also, please, consider to expand toStr\fromStr to toString\fromString :)


<!DOCTYPE html>
<html>
<head>


which function


Great job!


#17 Is this more clear, or should I create a second example?


What are the issues we encounter with the python-networkmanager patch ?


Exemplary :sunglasses:


``CRISwitch`` would be a better name I think.


this. otherwise it's perfect


`return if` ... `.exists?` would be more readable inmho


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


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:554`|
|---|
Move isAncestor() to DescriptorUtils.kt and make it an extension function on DeclarationDescriptor?


Hi Shawn, I noticed these changes made for non-linear ads and was hoping they might fix some issues I have been seeing, but unfortunately the issues remain.
I have raised an issue and I am happy to look further into them myself, though it would be good to have a chat with you first, so that I don't waste any time.
The issue is that the controls are not visible when a non-linear overlay ad is shown and you can see this on the demo page I have setup here: http://coullwebsdk.herokuapp.com/nonlinearvpaid/examples/simple/

Thanks!


Easy to see through.


@braamgenis What's this comment for?


Whoops, I think I forgot that too! :blush:


**Whoops!**

There were *1 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-issues-and-reporting-bugs.md#L258`](https://github.com/Google/WebFundamentals/blame/d70e836106b295b868956df3723ba19b5fdb865d/src/content/en/fundamentals/engage-and-retain/push-notifications/common-issues-and-reporting-bugs.md#L258) - Line must be at most 100 characters


LGTM, except node_modules dir being committed as well. Could you please add a node_modules in .gitignore?


ok chked!


Geroge, very neat and clean code, but are you sure it gives correct result? I checked with simple example of mine, it didn't.


it's clearer, but more examples would be great!


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:604`|
|---|
"Member extension function call" should be more clear for users than "Call with two implicit receivers"


It's fine, that's why I have ReSharper 👍

Thanks for `out _` suggestions as well, I didn't even know about that myself. It looks much better than previous `out ignored`, finally I'm not that angry that there is no way to skip out parameter declaration if I don't need it.


Not sure how to restructure the whole thing to call `super.__init__` in the end. I am overriding the `self.recalculate` method and the new method requires those vars in above.

The traceback shows that the place is the line 60:

```
Traceback (most recent call last):
  File "main.py", line 637, in <module>
  File "main.py", line 521, in session
  File "pyglui/ui.pyx", line 169, in pyglui.ui.UI.update (pyglui/ui.cpp:9233)
  File "pyglui/ui.pyx", line 137, in pyglui.ui.UI.handle_input (pyglui/ui.cpp:8730)
  File "pyglui/menus.pxi", line 525, in pyglui.ui.Scrolling_Menu.handle_input (pyglui/ui.cpp:43091)
  File "pyglui/menus.pxi", line 542, in pyglui.ui.Scrolling_Menu.handle_input (pyglui/ui.cpp:42872)
  File "pyglui/ui_elements.pxi", line 409, in pyglui.ui.Selector.handle_input (pyglui/ui.cpp:20253)
  File "pyglui/ui_elements.pxi", line 420, in pyglui.ui.Selector.handle_input (pyglui/ui.cpp:20095)
  File "pyglui/ui_elements.pxi", line 439, in pyglui.ui.Selector.finish_selection (pyglui/ui.cpp:20549)
  File "pyglui/ui.pyx", line 332, in pyglui.ui.Synced_Value.value.__set__ (pyglui/ui.cpp:45324)
  File "main.py", line 300, in open_plugin
  File "plugin.py", line 260, in add
  File "/home/pupil/pupil_player_settings/plugins/screen_tracker_offline.py", line 77, in __init__
    super().__init__(g_pool,mode,min_marker_perimeter,False,robust_detection,)
  File "offline_surface_tracker.py", line 60, in __init__
TypeError: __init__() takes from 2 to 5 positional arguments but 6 were given
Failed to execute script main
```


Good work, thanks for the PR


you can use the `pdf_attachment.attachable.document_id` here, would keep you mind in the pdf_attachment mindset


you need a space before the opening and after the closing parenthesis, like this: 'if (elts ... null) {'
(the same applies to line 11)


TeamCity Glyssen :: Glyssen pull requests [Build *.*.529.c84d1fa630eb694e74ea91e31c7ad7cff8bf50f2](http://build.palaso.org/viewLog.html?buildId=87515&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1666, ignored: 18 Build time: 00:07:17


nice! I've never seen that iso8601 function before.


It enables code completion in IDE. :smile:


`theme_xhred_global_help=Hjälpa` should be `theme_xhred_global_help=Hjälp`


But let me take a look at the screen tracker.


I see that it does


Suggestion: Easier to see changes if you first commit the original, unmodified exercise files and then commit the changes.

(Current commit message is very good for the changes. But without the original file versions, the committed content is shown as the whole contents, not just your fixes.)


Thank you,am acting on it now


frgfgbfgr


jeste to dokoncim


In this case you have a valid reason to set the variables first and call `super().__init__` after.


[function reference](https://msdn.microsoft.com/en-us/library/windows/hardware/hh439511(v=vs.85).aspx)


Here it is:
``` python
class Screen_Tracker(Surface_Tracker):
    def __init__(self,g_pool,mode="Show Markers and Surfaces",min_marker_perimeter = 100,invert_image=False,robust_detection=True):
        super().__init__(g_pool=g_pool,mode=mode,min_marker_perimeter=min_marker_perimeter,invert_image=invert_image,robust_detection=robust_detection)
        for p in g_pool.plugins:
            if p.class_name == 'Marker_Detector':
                p.alive = False
                break
```


This approach looks good to me.
I would now move all of this into a Service object

```
class Worker
  def perform(attachment_id)
     ...
     SuperService.new(attachment).call # something around those lines
  end
end
```


You could just add the old on use text as a description so it's clear what it does.
_Use: Deals 100 Shadow damage and restores 100 mana to you. (cooldown 30 sec)_

http://web.archive.org/web/20051114083928/http://thottbot.com/?i=25000


Nice, cleanly organised code. :+1:


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


![](https://github.trello.services/images/mini-trello-icon.png) [Pont helyzetének vizsgálata a háromszöghöz képest](https://trello.com/c/neyN7Dym/57-pont-helyzetenek-vizsgalata-a-haromszoghoz-kepest)


Nice code, I implemented this part in a completely different way, it was good to see your solution :)


I am int he process of cleaning everything up.  I will push it in about 10
mins

On 5 April 2017 at 11:11, mvdm <notifications@github.com> wrote:

> if I'm going to test the whole workflow I'd like to run a version as close
> to final as possible, and want to avoid something like that BuzCSD no
> longer works because we deleted a version that is referred to some place --
> what do you think about removing everything we don't need first before I do
> a run through?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/vandermeerlab/EC_Naris/commit/bd96c57668a1092f21c179e27c856a156b0c568e#commitcomment-21639414>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AFL2AXrkTHMFgwN54uysIg6lVNulz5wjks5rs68ygaJpZM4M0Xdi>
> .
>


don't leave commented out statements in your code


![image](https://cloud.githubusercontent.com/assets/26910619/24713676/ba13716c-19fc-11e7-8653-5042e9045e88.png)
o outro alfabeto e a chave q ele deixou.


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:104`|
|---|
I don't see why this function is called "lazilyProcess", given that it always resolves all references in the given set of elements and passes them to the given callback.


é o alfabeto maçonico.


Sounds good - didn't know the syntax for this!


Will do.


Is this an official logo, or is it improvised?

I can't see it anywhere in [cmu-sei/BigGrep](https://github.com/cmu-sei/BigGrep).


@Davidslv we'll be moving it into a service in another commit at some point.


Why assert?
Assert statement will not be executed in release mode.


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:106`|
|---|
Introduce a type alias for the type of `usageFactory`?


This is out of scope, but if we could render the HTML from whitehall that would make this process simpler without having to republish the document.
It's unfortunate we can't use the functionality here.


Let's hope EB doesn't move the app ;)


is there a chance that this caused https://github.com/realm/realm-java/issues/4437 ?


Nice code. :+1:

Commit message: "add" has happened before. This commit is about changes, new implementations, etc.

But otherwise everything is good :smiley:


What if there was a different type of ammo for the gatling guns that did next to no damage but was specifically designed to spoof anti-missiles?


use the braces even if it's only one statement inside


Isn't this useful in the case where we're generating a META file in the top level package? E.g. we have a findlib structure like:

```
cohttp
cohttp.lwt
cohttp.async
```

which mimics the opam package structure (cohttp, cohttp-lwt, cohttp-async). But only cohttp will install the META file for the entire 3 projects. This seems to work only with `exists_if`


Yeah - once that fix is pushed (which should hopefully be soon), then we can ignore all this looping madness.


For me is OK too :-)


Could you try to change it to the following:
```diff
- def __init__(self,g_pool,mode="Show Markers and Surfaces",min_marker_perimeter = 100,invert_image=False,robust_detection=True):
+ def __init__(self,*args, **kwargs):
-        super().__init__(g_pool=g_pool,mode=mode,min_marker_perimeter=min_marker_perimeter,invert_image=invert_image,robust_detection=robust_detection)
+        super().__init__(*args, **kwargs)
```


This is a fairly slow operation and should be performed after the fast checks that currently follow it.


How you showing the diffs?????


Yes it is used to send the vertices and the texture.


Hey, it is actually not possible as this is not a part of the material design spec per: https://material.io/guidelines/components/snackbars-toasts.html

Under **Behavior** : "Snackbars animate upwards from the bottom edge of the screen."


You can also do
```
AttachmentData.new(
file: uf,
attachment_data: pdf_attachment_data,
...
)
```

That's less typing


Awesome! :)


absolutely !


Wouldn't it go above `$the_widge` line, and be on the same level indent? Also, it's technically a `WP_Widget`.


The following doc(s) have been deployed to the [905deployb](https://905deployb-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/deploybot](https://905deployb-static-docs.pantheonsite.io/docs/deploybot)
- [source/docs/articles/sites/deploybot.md](https://github.com/pantheon-systems/documentation/commit/57d6a3da9b5e3cf27007562ecd305df2f176882d/source/docs/articles/sites/deploybot.md)


Use `diff` as code highlightling language and add +/- add the start of the line.


Understood.  Thing is,  there's several that could get past short-term impasses because of this.  We either need this in the short-term to work around not having BT Classic going or we need the Bluedroid Classic support starting to gel.  How do you validate for FCC and EC purposes, both LE and Classic on the BT side before it rolls out to the factory?  Even a bit of whatever firmware as a blob that we could load could take short-term pressures off us on this.


Thanks, was fixed before I read your comment ;)


Do you mean didn't I need to delete tabs in the template? Should I only have to add `gofmt` to the tests and fix expected results?


Does this mean when you have no favicon.ico in your template, the survey will start all over again (loop)?


Same here. I've been scratching my head a while after having updated to 2.0.1
I tried quickly to use support functions from sklearn with no success; I also tried to refer symbolic functions from TF with the same result...
For the time being I rolled back to 1.2.2


The following doc(s) have been deployed to the [earlyaccess](https://earlyaccess-static-docs.pantheonsite.io/docs) Multidev environment:


Thank you! Will try your tips out.


I ❤ React


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


@paflopes it's been a while since I wrote scripts for rooting Zenfne. Is something has changed with command that you commit? Maybe is it useful for pull request?


passing a function as an argument. Excellent! :)


Gah, indentation will be the death of me. It's technically only really needed before we want to use one of it's methods, but yeah, might be neater to put it before `$the_widget` is first declared/assigned.
Ah, yeah should be `WP_Widget`, I keep forgetting we didn't invent widgets. :stuck_out_tongue_winking_eye:


hehe 👍


good


|`line 146`|
|---|
If I understand correctly (which may not be the case), we will currently report a conflict if we move a nested class to a subclass of its containing class, where the members of the companion object of the outer class will still be visible. (Take nonInnerToTopLevelCompanionConflict as the base and move B into a subclass of A). This shouldn't be a conflict.


Just perfected them :P


#### Live Demo: [http://demo.znote.eu](http://demo.znote.eu)


I have problem after updating db schema to 184 version.

Component | Version
--------- | -------
LibreNMS  | 0fd0b7cd6cdeec0dd996e57e72648640f176931e
DB Schema | 184
PHP       | 7.0.17
MySQL     | 10.1.22-MariaDB
RRDTool   | 1.4.8
SNMP      | NET-SNMP 5.7.2
==========================================================

[OK]    Database connection successful
[FAIL]  MySQL tables collation is wrong:  [FIX] http://bit.ly/2lAG9H8
         cmpMemPool latin1 latin1_swedish_ci
         current latin1 latin1_swedish_ci
         fanspeed latin1 latin1_swedish_ci
         frequency latin1 latin1_swedish_ci
         voltage latin1 latin1_swedish_ci
         port_in_measurements utf8 utf8_general_ci
         port_out_measurements utf8 utf8_general_ci
[FAIL]  MySQL column collation is wrong:  [FIX] https://t.libren.ms/-zdwk
         cmpMemPool Index latin1 latin1_swedish_ci
         cmpMemPool cmpName latin1 latin1_swedish_ci
         cmpMemPool cmpValid latin1 latin1_swedish_ci
         current current_oid latin1 latin1_swedish_ci
         current current_index latin1 latin1_swedish_ci
         current current_type latin1 latin1_swedish_ci
         current current_descr latin1 latin1_swedish_ci
         fanspeed fan_oid latin1 latin1_swedish_ci
         fanspeed fan_descr latin1 latin1_swedish_ci
         frequency freq_oid latin1 latin1_swedish_ci
          and 5 more...
[FAIL]  Database: missing column (users/remember_token)
[FAIL]  Database: incorrect column (users/updated_at)
[FAIL]  Database: extra table (cmpMemPool)
[FAIL]  Database: extra table (current)
[FAIL]  Database: extra table (fanspeed)
[FAIL]  Database: extra table (frequency)
[FAIL]  Database: extra table (port_in_measurements)
[FAIL]  Database: extra table (port_out_measurements)
[FAIL]  Database: extra table (voltage)
[FAIL]  We have detected that your database schema may be wrong, please report the following to us on IRC or the community site (https://t.libren.ms/5gscd):
         ALTER TABLE `users` ADD `remember_token` varchar(100) NULL AFTER `realname`;
         ALTER TABLE `users` CHANGE `updated_at` `updated_at` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP;
         DROP TABLE `cmpMemPool`;
         DROP TABLE `current`;
         DROP TABLE `fanspeed`;
         DROP TABLE `frequency`;
         DROP TABLE `port_in_measurements`;
         DROP TABLE `port_out_measurements`;
         DROP TABLE `voltage`;


![](https://github.trello.services/images/mini-trello-icon.png) [Criar método de checagem de credenciais autenticadas anteriormente](https://trello.com/c/eiwS9c2H/50-criar-metodo-de-checagem-de-credenciais-autenticadas-anteriormente)


Feel free to get inspired by solution **after you have given it a try**. Just avoid copying it.
I know the exercices was about jQuery so you might have felt stuck in JQ mindset


Just to be clear, this fix reverted the code back to the previous 'working' state. If there are any loot bugs associated with this then the issue is elsewhere, as this *should* be purely visual.


I'm not sure. It might have been to work around a julia issue; we should try restoring them.


Why not keep using DescriptorUtils.isCompanionObject()?


TeamCity Platform Services :: Environment Manager 2 :: Package [Build 3.39.8-pre-2-ga2e051d](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=57742&buildTypeId=PlatformServices_EnvironmentManager2_Package) outcome was **SUCCESS**
Summary: Running Build time: 00:00:56


NTP on everything!

...what were you talking about?

2017年4月5日(水) 1:16 Jason Perrin <notifications@github.com>:

> As I said in ocf/dns@96508e4
> <https://github.com/ocf/dns/commit/96508e41cf454ada0218fed09e770b689189db45>,
> I don't know if we want NTP on this server too, I mean it isn't a big deal
> to have, just seems unnecessary...
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ocf/puppet/commit/6fedfb0b11752f6049678c1ccd39a434ccf9be4f#commitcomment-21632948>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAs6FJlQ-hQCKm_qfeWbIuBif6CZkLEmks5rs03DgaJpZM4Mz5fs>
> .
>


you are getting there, though why would you remove handler? You need to combine JS logic with javascript logic.
Those exercies are mostly for JS logic and has only a small part of jQuery. I'd suggest you try to implement requested functionality without jQuery first and then add jQuery on top.
For example in this exercise you can just `console.log('red');` instead of displaying color on a screen as starting point


<details>

<summary>Submitting the task to TaskCluster failed. Details</summary>

```js

Schema Validation Failed!
Rejecting Schema: http://schemas.taskcluster.net/queue/v1/create-task-request.json#
Errors:
  * data.metadata.owner should match format "email"
----
method:     createTask
errorCode:  InputValidationError
statusCode: 400
time:       2017-04-05T15:58:08.347Z
```

</details>


Обрати внимание,, как разросся JS-код.
Чтобы не путаться, вынеси его в отдельный файл `quiz.js` и подключи в HTML через тег `<script src="quiz.js"></script>`.


Nice! We need to do more of this throughout the codebase 👍


@lucasbento Oops! ... Really. I'll fix it, thank you.


Generally bad form to commit notes that aren't direct literal comments, I typically create a `NOTES.txt` to keep track of what needs to be done, and include it in the `.gitignore` for the project so it doesn't get checked in to source control.


This really must be fixed : a lot of user have bad ressources url for logo / js / css etc ....


<details>

<summary>Submitting the task to TaskCluster failed. Details</summary>

```js

Schema Validation Failed!
Rejecting Schema: http://schemas.taskcluster.net/queue/v1/create-task-request.json#
Errors:
  * data.metadata.owner should match format "email"
----
method:     createTask
errorCode:  InputValidationError
statusCode: 400
time:       2017-04-05T16:00:35.767Z
```

</details>


Ok, nice! I added a tips block. An editor can review the text later :)


story.txt


1.what is a network?
2.Discuss various types of network?
3.what are the goals of TCP/IP model?
4.what is OSI model?
5.what are connection oriented and connectionless service?
6.what  is the difference between IP address and port number?
7.different topology types?
8.some important characteristics of bus topology?
9.compare ring and bus?
10.compare star and ring?
11.name the transmission media used for bus LANS


Добавь счётчику атрибут `readonly`, это не позволит пользователю его так просто изменить.
```html
<input class="quiz_counter"
       type="number"
       value="0"
       readonly>
```

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


@rockbmb


remove this debug statement before doing the PR


Focus on logic first. After you figure out how to print "red", "green", "yellow" you just need to replace `console.log` with different things. Easies thing is to add a special class - active to one of three lights.
```
.yellow.active {
  background: yellow
}
```


d


|`idea/src/org/jetbrains/kotlin/idea/refactoring/move/moveUtils.kt:594`|
|---|
Why not make it a proper instance method so that you wouldn't need the `this !is` check?


Dziękuje.


Typo: should be site-lisp, not site-list.  Better yet, just inherit emacs and do emacs data/meson.el.


It says missing seperator.


x + y=sum
x-y =(absolute) different
sum must greater then different
and sum+diff= always positive


Typo, should be site-lisp.  But better to inherit emacs and doemacs instead.


inherit vim, then viminto syntax; dovim misc/ninja.vim.


@MZO9400 I'm working on that one as we speak


It was wrong :(((


@frenzymadness, pushnul jsem tohle – domácí úkoly závisely na Šibenici.


Good use of the object-oriented principles.
(Like storing inputs inside the object as variables.)
Was useful for me to see your code. :+1:


This push seems to have broken the build, although I don't think this caused it.

The issue looks like its related to ros not installing from apt-get I think. Someone should investigate.


As far as I can see this isn't used any more and should be removed.


Who wrote this code.  Terrible.


Are you sure this is the right place for this?  Other packages install zsh completions somewhere under /usr/share/zsh.


Proszę o sprawdzenie.


@diptodip pushed to master. Was it his fault?

On Wed, Apr 5, 2017, 12:07 Josh Morton <notifications@github.com> wrote:

> This push seems to have broken the build, although I don't think this
> caused it.
>
> The issue looks like its related to ros not installing from apt-get I
> think. Someone should investigate.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/gtagency/buzzmobile/commit/34d006afe8483439912d9c1c890cb39a16daf99a#commitcomment-21640512>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHC0h8YzTljFrXb-lHv5DG6HEz0mGpVaks5rs7xRgaJpZM4M0eFQ>
> .
>


confused


Testing done:

1. Ran docker system prune --dry-run
2. Ensured expected networks & containers were reported as "Will Delete {conatiner-name or network-name}:"
3. Ensured that none of them were actually deleted.

```
root@2d05e13c394f:/go/src/github.com/docker/docker# docker system prune -n
Will Delete Containers:
1b264184425a1132938361cc6e517047310d717c1afe045f8372d2c7923c8a52
a579e22e49f10d2d59f34b4d70d319675dee683b2319ec1fcfbc95e99f4c0f56
b66fe1248fa5876e26e2761b0049fa22a0b8bf7f77459a6837ccb8c6764a5b31

Will Delete Volumes:
helo
helo1
helo2
helo3

network prune opts: {true true {{map[dryRun:map[true:true]]}}}Will Delete Networks:
my-bridge-network1
my-bridge-network3
my-bridge-network2

Total reclaimed space: 161B
root@2d05e13c394f:/go/src/github.com/docker/docker# docker container ls
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
root@2d05e13c394f:/go/src/github.com/docker/docker# docker container ls --all
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS                         PORTS               NAMES
1b264184425a        3e314f95dcac        "/bin/bash"         About an hour ago   Exited (0) About an hour ago                       distracted_jackson
a579e22e49f1        3e314f95dcac        "/bin/csh"          About an hour ago   Created                                            vibrant_snyder
b66fe1248fa5        3e314f95dcac        "/bin/zsh"          About an hour ago   Created                                            nervous_panini
root@2d05e13c394f:/go/src/github.com/docker/docker# docker network ls
NETWORK ID          NAME                 DRIVER              SCOPE
ac699222e1a0        bridge               bridge              local
85d97445be49        host                 host                local
0ce6fa036b38        my-bridge-network1   bridge              local
af2364df0f9a        my-bridge-network2   bridge              local
24b6e5b77b6b        my-bridge-network3   bridge              local
20ccd5270f43        none                 null                local
root@2d05e13c394f:/go/src/github.com/docker/docker#

```


@MZO9400 its fixed here: @8deacbf


I was confused


Like I said, I don't think so. Check the circle logs, its a weird seemingly unrelated thing.


I think it makes sense to add `val project: Project` here as well, so that you won't need to pass it around in some of the calls.


把 propTypes 的定义放在最前面会好一些，方便看


TeamCity libpalaso :: palaso-trusty64-lfmerge Continuous [Build 2.6.242.1fd0761fc7cf1c6085847880db966d589bf276b0](http://build.palaso.org/viewLog.html?buildId=87529&buildTypeId=PalasoTrusty64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3389, ignored: 124 Build time: 00:04:32


I don't think the isValid check makes sense here; it's perfectly fine to update the userdata of an invalid element.


Section transition can sometimes cause perfomance drop in levels with moving layers.


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1257.c45b10d8b9eabdfe73debaf5a642ac657861e9c0](http://build.palaso.org/viewLog.html?buildId=87528&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3393, ignored: 125 Build time: 00:05:55


Wew


I think that various methods in MoveDeclarationsDelegate shouldn't be abstract, and should have empty implementations in the base class instead.


Pro accounts still only have one simultaneous build (which can be parallel though). One of my other projects has a 1-hr long test suite and is currently running.


![](https://github.trello.services/images/mini-trello-icon.png) [Create Charts in Excel](https://trello.com/c/bM9Sw9yc/4-create-charts-in-excel)


![](https://github.trello.services/images/mini-trello-icon.png) [Create server api which will return pre-selected tickets information](https://trello.com/c/0cFzc3T4/72-create-server-api-which-will-return-pre-selected-tickets-information)


[452]
[452]
[452] --> FOAM FATAL IO ERROR:
[452] error in IOstream "/mnt/../GitTEST/Cable-conference/Re_p\
ower_of_five_pimpleFoam/processor452/0.0012/k" for operation Ostream& operator<<(Ostream&, const Scalar&)
[452]
[452] file: /mnt/../../../GitTEST/Cable-conference/Re_power_of_five_\
pimpleFoam/processor452/0.0012/k at line 63031.
[452]
[452]     From function virtual bool Foam::IOstream::check(const char*) const
[452]     in file db/IOstreams/IOstreams/IOstream.C at line 96.
[452]
FOAM parallel run exiting
[452]


I think this method will be easier to read if you extract the TObjectHashingStrategy implementation into a top-level object.


Charts


我看他的[官方文档](https://facebook.github.io/react/docs/typechecking-with-proptypes.html) 是这么写的啊 。 这个应该怎么放到前面？


Current implementation of moving layers sucks because causes one huge global lag. To resolve it, is implementing the new system which will allow every layer have unlimited elements and never cause lag (except moment that all of them are visible on screen)


## This is a markdown file


а теперь ты видишь комментарий?


There were the following issues with your Pull Request

  * Your commit message does not contain a body
  * Invalid type. Valid types are **add, refactor, remove, move, fix, update, version, cleanup**

Guidelines are available at https://github.com/pagarme/pagarme-js

---
This message was auto-generated by https://gitcop.com


Hey Mary, stick to one version of JS when you code.If you want to write ES6, then you need to lose the `var`.


Windows executable


oh, nice catch, I've seen dat


Could potentially add browserify, where we can simply call the js lib we want in the js files like
var $ = require('jquery'); and then have a file that bundled all our js.
Did not add for now as I did not want to force those who are not using npm to compile js.


Ah, okay, that explains it.


Mildly annoyed at how "Hideous Destructor" is not strictly and explicitly defined, but no idea how to do that without ultimately useless bloat.


Apart from being unnecessary and misleading, I don't think that deleting the QLocalServer in Server's destructor would create a problem, like a runtime error, for the program.  The QLocalServer is never accessed outside of the Server.

"You can also delete child objects yourself, and they will remove themselves from their parents."
"When any QObject in the tree is deleted, if the object has a parent, the destructor automatically removes the object from its parent."
-- http://doc.qt.io/qt-5/objecttrees.html

As is, the QLocalServer will be deleted before the Server. Upon removing the QLocalServer's explicit deletion from Server's destructor, a Server will be deleted first, and the QLocalServer will be deleted once its parent's lifetime ends.


@SumanthKP @parvatihiremath Given we are using role code. This needs to be updated too.


De vuelta después de un largo descanso!


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.1.211.88cfdfc2c4c0ad0b4f06f0f5fd469157f6236671](http://build.palaso.org/viewLog.html?buildId=87530&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3554, ignored: 111 Build time: 00:07:07


@jcredding FYI.


@gopusatya These lines are added by the task runner Grunt


a small test running on orca using latest 0/ system/ constant/ but a smaller mesh without any error.

It indicates that the mesh may have some problems.

So I re-decompose the 30M mesh to see any trouble with I/O.


It took me a long time to find React-Navigation project. It will be reffered on the Navigation guide later?


Hmm, it's not intended, it's some kind of mistake.
Is 1.5.6 needed?
If none, I will check our release script. We need to clear garbages before packaging.


已改进 https://github.com/rsuite/rsuite-clipboard/commit/903942f979f8ec474147e3a734ab7079a1dc0dd2#diff-c0fc770b342598b30cc2417ffa74ef90R10


That won't work cause you already wrote the Headers and already send something in the body


@parvatihiremath This dependency can be removed.


It's building now, but it still doesn't seem to be working in parallel, e.g. [the current build](https://ci.appveyor.com/project/wjakob/pybind11/build/1.0.1596) is running just the fifth job (with 6-8 just queued), and the total build time is just the sum of the individual job times (plus several few seconds of the overhead between jobs).


Просто немного мусора.Метод в обычном классе вроде как ничего никому не должен=)


Delightful. Also, axed another typo. :)


Обрати внимание, что эта часть функции `init` — со 119 по 129 строки — совершенно отдельная и никак не пересекается с остальным кодом: здесь объявляется и наполняется содержанием переменная `answers`.
Её можно и нужно вынести в отдельную функцию, чтобы упростить код.
Что это будет за функция?
Это функция, которую дают на вход список ответов, а она выдаёт готовый HTML для их кнопок.
Поэтому назвать её целесообразно `generateAnswerListHtml`
У неё будет один входящий параметр — массив ответов на вопрос. Кроме этого, она будет возвращать результат с помощью конструкции `return`.
Её код должен выглядеть примерно так
```javascript
function generateAnswerListHtml (answerList) {
	let answers = ''

	answerList.forEach(
		answer => {
			if (answer.isCorrect) {
				answers += `<button class="quiz_answer quiz_answer-right">${answer.text}</button>`
			} else {
				answers += `<button class="quiz_answer quiz_answer-false">${answer.text}</button>`
			}
		}
	)

	return answers
}
```

При это из кода функции `init` убираются 119–129 строки, а строка 131 заменяется на вызов нашей функции.
```javascript
$quiz_answerList.innerHTML = generateAnswerListHtml(question.answerList)
```


@parvatihiremath The domain classes should not have any jackson usage.


At leas, as in current build, levels with moving layers usually work great.


I fixed it locally. Lol


Use `mapToNewOrThis`?


![](https://github.trello.services/images/mini-trello-icon.png) [Registration path from app to firebase](https://trello.com/c/IWM1U99g/1-registration-path-from-app-to-firebase)


Need also add PR for DB table now is empty as i know  and need rewrite some scripts and add bd loading text for them becouse some of npc texts hardcoded by pussywizard and xinef to scripts(I write it for understending why some texts still in english, you can find it by yorself by look in saurfang icc script he has hardcoded text and not only him )


line 42 refers to 'this' but you're in an anonymous function, so 'this' is no longer referring to MainCanvas. You'll need to bind this to the function eg:
`this.Canvas.mousedown(function(){down(this.Canvas[0])}.bind(this));`


This is logically incorrect. If there were multiple references at the original element, most likely they resolved to different targets, and you're rebinding them to the same target element.


1.5.6 is not needed, but I want to ask you to avoid such stuff in the future.


@parvatihiremath There is a slash missing. i.e it should be /v1/actions/_search


## This is a markdown file


모달 sass 작업했습니다.
급하게 만드느라 코드가 좀 지저분한 감이 있습니다.
우선 작업하고 리펙토링 할때 수정하겠습니다.


Do we need the `fill_background` parameter? It doesn't seem to be used anymore, as `draw_char` now draws character pixels only if their brightness isn't 0.0.


Sweet!  May want to have a few more tests here though. For methods like this I enjoy using the `[TestCase] ` attribute. I see three cares.

Trailing zeros,  1.200
Trailing  zero whole number,  1200 (I think the old suggested impl would have broken here)
No trailing zeros,  12345.6789


`canvas` that you've passed through to the 'down' function is already `canvas[0]` so you only need `canvas` on these two lines, not `canvas[0]` surely.


@MZO9400  only thing that was actual missing was a hard TAB before the line ;p


can you add javadocs like the other similar classes to explain why it is ok to throw an UOE for those methods?


Looks like you are right. Strange..


The following doc(s) have been deployed to the [905deployb](https://905deployb-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/deploybot](https://905deployb-static-docs.pantheonsite.io/docs/deploybot)
- [source/docs/articles/sites/deploybot.md](https://github.com/pantheon-systems/documentation/commit/57d6a3da9b5e3cf27007562ecd305df2f176882d/source/docs/articles/sites/deploybot.md)


Yep! Spaces cause problems with compiler lol.


@parvatihiremath Change this to
`return new ActionResponse(responseInfo,actionContracts);`
since the default behavior of Spring is to return 200 ok.


/cc @jcredding


that should not be necessary anymore: if advanceExact returns true, then there is an ord


Hello pedramphp.
This error did exist briefly on January 17, but it is fixed in both the experimental branch and the testnet master branch.
See:
 https://github.com/aeternity/testnet/blob/master/src/consensus/txs/channel_team_close_tx.erl#L25
https://github.com/aeternity/testnet/blob/experimental/src/consensus/txs/channel_team_close_tx.erl


The name doesn't seem to match the behavior: the name says "and class files" and the code has "!includeClassFiles"


@parvatihiremath The bindingResult should be removed since we don't do error handling in the controller.


Hello, this updates are not visible on CocoaPods, yet.


Unused import


@MZO9400 please contact me on Hangouts if you are interested in joining us...

mhrvisser21@gmail.com


Is "Max jobs" set on the project Settings?  (Or is it unset and perhaps it needs to be set?)


@parvatihiremath Typo in the package name i.e contact instead of contract.


If you replace this with a loop that waits for the `/healthz` endpoint to return `ok` then it should be more reliable.


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/588f383c-31f4-41ff-a07a-b28b3f931c80 "WestFieldServlet"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/c12fb70d-7bd5-49b2-a05b-1c8339e6d87d "WestFieldServlet") deployed this commit to [WestField](https://console.ng.bluemix.net/apps/e24bfad3-2ee1-4f43-8f66-6892de442d14 "ibm:yp:us-south:POT_Org:WestField")


@parvatihiremath Remove the setter and noArgs annotation. Since we are setting the fields of the object in the constructor.


There's an issue with k8s 1.6 and the provisioner - I'll push a new image that fixes this before lunch.


No, its currently not working. When I get it to a working point ill move it to a different branch


@parvatihiremath This should be camel case.


I think it's simpler not to register these as top-level services, and instead to make them properties of PackageIndexUtilModificationListener.


Аналогичная ситуация в 133–142 строках — они отдельные и не пересекаются с остальной частью функции `init`. Единственное, что нужно принять во внимание — они должны выполняться строго после строки 131, потмоу что они навешивают обработчики на созданные в 131 строке кнопки.

Что именно делают строки 133–142? Они _устанавливают обработчики на кнопки_.

Поэтому подходящим названием для функции, в которую мы их вынесем, будет `setBtnListeners`.

Код её будет идентичен строкам 133–142.

```javascript
function setBtnListeners () {
    const $falseBtn = document.getElementsByClassName('quiz_answer-false'),
          $rightBtn = document.getElementsByClassName('quiz_answer-right')

    for (let i = 0; i < $falseBtn.length; i++) {
        $falseBtn[i].addEventListener('click', counterReduce);
    }

    for (let i = 0; i < $rightBtn.length; i++) {
        $rightBtn[i].addEventListener('click', counterAdd);
    }
}
```

Никаких отличий.

А вместо строк 133–142 в функции `init` теперь можно просто выполнить эту новую функцию, примерно так

```javascript
setBtnListeners()
```

Обрати внимание, как после этих манипуляций сократился код функции `init`, потмоу что мы вынесли из него побочные операции в отдельные функции.

```javascript
function init () {
    const question         = questionList[0],
          $questionText    = document.getElementsByClassName('quiz_question')[0],
          $quiz_answerList = document.getElementsByClassName('quiz_answerList')[0]

    $questionText.innerHTML    = question.text
    $quiz_answerList.innerHTML = generateAnswerListHtml(question.answerList)

    setBtnListeners()
}
```

Чем меньше избыточного кода в каждом месте программы, тем проще его читать и понимать, а, значит, и проще работать.


it's unset -- I set it to "10" just now, and I also set "MSBuild options: Enable parallel builds". AFAIK all of this is overridden by .appveyor.yml so technically shouldn't be necessary.


git revert?


The following doc(s) have been deployed to the [audittaxon](https://audittaxon-static-docs.pantheonsite.io/docs) Multidev environment:
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/916a25d3a9dd794bdd9391286312e56d2e869f7f/source/docs/assets/css/main.css)


I think "Util" should be removed from the names of this and the following class.


this method returns the total number of values of the field. Instead it should consume the first ord, and make sure the next ord after than is SortedSetDocValues.NO_MORE_ORDS


blablabla


@parvatihiremath This class should be removed since we have it in the common jar.


"createEvents" is not a good name for a list which also contains delete events.


Yeah, this test case used only DMA1. Ok to add DMA2 also.


-<?php require "login/loginheader.php"; ?


გხფგფჰჰჯჰჯგფფგცვ


@parvatihiremath Remove setter and only use allArgsConstructor


TeamCity Glyssen :: Glyssen pull requests [Build *.*.530.acdab773cd4cf5ddbdb7f6597833401a201a2e68](http://build.palaso.org/viewLog.html?buildId=87525&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1666, ignored: 18 Build time: 00:12:34


it is guaranteed to be >= 0 if advanceExact returns true


guaranteed to be >= 0


@parvatihiremath This should be set to false to avoid seeing a lot of noise in the logs.


@elharo @chanseokoh good for release?


yup! I wasn't sure if there were more `order`s incoming


LGTM


Indentation errors


@parvatihiremath TenantId should be 256 characters.


No, this is sync realm only. What was I even thinking.


Hi,
Didn't see a way to comment on your blog so I'm leaving this here: there's a corner case where the pre-commit hook doesn't have anything to stash, but the stash list is not empty (e.g. from a manual `git stash` command). The end of the script will silently pop the unrelated stash, which is a bit unexpected.
You can name your stash to check if the initial command saved anything, and only pop at the end if it did. Here's how I did it:
```
STASH_NAME="pre-commit-$(date +%s)"
git stash save -q --keep-index $STASH_NAME

# Test prospective commit
 ...

STASHES=$(git stash list)
if [[ $STASHES == *$STASH_NAME* ]]; then
  git stash pop -q
fi
```


`one()`, `first()`, and `get()` have their own usefulness in sqlalchemy.  If you have the id of an object, then using `get()` is the better call.  If you actually only expect one object to exist and it's a programming error to have more than one, you want to use `one()`.  If potentially many objects exist, and you only want the first one, you want to use `first()`.

In this case, you have the `item_id`, (since that's what you're passing back through the form), so `get()` makes sense here. :)


Escapes html tags in JSON custom fields, see #192


@parvatihiremath Is boolean a PostGres specific data type?


@jcredding FYI.  I randomly noticed this b/c I think redis-namespace released a new version that one of our apps unlocked and installed.  I thought I'd check this against hella-redis and this is the result.


resolves issue #18


Caching all scopes in this map is a bad idea; there are both "stable" scopes such as project and library scope, which are OK to cache, and "temporary" scopes such as GlobalSearchScope.fileScope(), which can be created in an unbounded number and never reused again.


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/7c1a1049-628c-4188-bf58-bb5e034a8047 "1stChatBot"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/16a045d3-1f7d-4b1d-9b61-af818aab264e "1stChatBot") deployed this commit to [dev](https://console.ng.bluemix.net/apps/67b631f4-7589-40ec-9aa1-4e49ce30df2a "ibm:yp:us-south:jerryobrien30@gmail.com:dev")


what happen??


@parvatihiremath Remove `throws Exception` if the test does not actually throw an exception.


Thanks Jake,

I will have a look at that.

Have a good evening

Stu Last
Phone: +44(0)1326 722701
Mob:  +44(0)7805 456 944

Email:  stuart@last-family.co.uk
Linked in:  http://uk.linkedin.com/in/stulast
Twitter:  http://twitter.com/Stu_L

On 5 April 2017 at 16:32, Jake Gordon <notifications@github.com> wrote:

> canvas that you've passed through to the 'down' function is already
> canvas[0] so you only need canvas on these two lines, not canvas[0]
> surely.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/Stuartlast/sample.canvas/commit/5f643fb8ce055dff1eb13ec25d028cd7e55a2024#commitcomment-21640948>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AA-Er4UPPfZzCiit4EuhcPFHQg0-yXH-ks5rs8IWgaJpZM4M0PWn>
> .
>


Simple randomizer


That seems as good as deleted for this purpose...


I agree, we probably don't want to put things like ntp masters, mesos masters, etc. on this server since it's often going to be using high % of CPU and such.


A weak map should actually handle this correctly.


????????????????????????????????????ß


Doh! :)

Can you update https://docs.google.com/presentation/d/1cw7A4HbvM_Abv320rVgPVGiUP2msVs7tfGbkgdrTy0I/edit#slide=id.gbb3c3233b_0_162

I'll try to take out this block manually.


@parvatihiremath The assertions need to be more exhaustive. How do we know the action fields are rightly populated? And the count is as expected?


Move these functions to an implementation file.


Should be `const std::string&`


Please stick to the naming conventions. All variables and function names are lower case.


How did you get this key?


This should be a separate function to generate mesh.


@parvatihiremath The domain package should not be excluded.


This should be const.


Hahaha


Hey Tobin, I'm seeing 4 new unexpected errors in the layer validation tests with this patch.


not an issue for now, but later we can remove the `+1` since it was only useful because we wanted to avoid a branch that we now have to do anyway (because we need to check the result of advanceExact)


@parvatihiremath Also exclude unknown fields in objectmapper globally.


here to you need to consume ords until you get SortedSetDocValues.NO_MORE_ORDS, there is no way to know the value count up front. getValueCount() will return the number of values for the entire segment


![](https://github.trello.services/images/mini-trello-icon.png) [Да се позволят заявки без access_token на /api/users](https://trello.com/c/g5SyR0j4/86-access-token-api-users)


Interesting idea..
Maybe you should compare the original value because the first move may decrease the difference but may not create a cutoff
so you can code like this:

before the move loop have:
Value tValue = value;

Then you can compare the alpha difference with tValue since it will not get updated.


It went through bud


same here


![](https://github.trello.services/images/mini-trello-icon.png) [Да се сложи template на facebook_authorizer и facebook_token_handler, за да се вижда нещо докато потребителя чака](https://trello.com/c/vQ8Rm42b/88-template-facebook-authorizer-facebook-token-handler)


Acredito que devamos trocar o atual simbolo do ' i ' pelo simbolo do ' u '. O que voces acham?


ssm架构


You should use `unpkg.com` rather than `npmcdn.com`, it was renamed a while back.


耶！


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.767.e65e8b866425e23ec8b3fc47718069fa7ac9d614](http://build.palaso.org/viewLog.html?buildId=87535&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 938, ignored: 57 Build time: 00:20:00


Nope, this is accessible from IL only, and even there it's weird and buggy - e.g. LDTOKEN of a function pointer type (corresponds to C# `typeof()`) will result in a `RuntimeTypeHandle` of `System.IntPtr` (see more examples in #1929). Even if you get a `System.Type` instance representing a function pointer there's not much you can do with it, besides comparing for equality (not even `ToString` works right). It's better to pretend these don't exist, really.


yes


Thanks!


TeamCity Code :: 1. Development :: DDK :: Feature [Build 236](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=19352&buildTypeId=Code_Development_Ddk_Feature) is now running


My text editor was being a saboteur. He's been disciplined.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.1.253.88cfdfc2c4c0ad0b4f06f0f5fd469157f6236671](http://build.palaso.org/viewLog.html?buildId=87544&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3576, ignored: 128 Build time: 00:04:54


@AdSchellevis me again :D do you want any of these merged into stable/17.1 yet or shall we skip for 17.1.6 or you have more work here anyway so it could wait?


@VoyagerOne thanks. Not sure I fully understand your comment.


First: the same problem affect master and lts branch. Just remove` {TEMPLATEURL} `in front of favicon.ico in startpage.pstpl to see it. I noticed it in twig branch because my TEMPLATEURL was wrong. It took me a long time to understand why I needed to click twice on next....

Second: it doesn't loop. Here why:

With a wrong url for favicon, when you start your survey, your url is like: `index.php/survey/index/sid/{sid}/newtest/Y/lang/en`

So, when the url of the favicon is wrong, the browser call:
`index.php/survey/index/sid/{sid}/newtest/Y/lang/favicon.ico`
You can try it: that url show the survey again, it changes in the session the value of LEMpostKey. So, when you click on next, surveyRuntime compare the posted value of LEMpostKey with the one in session, they are different, so it display again the same page.

But:  When you click on next reload the first page, it also change the url to index.php/survey/index.php/{sid}.  (it removes the newtest/Y/, lang, etc). So with a wrong url for favico:  the browser will load index.php/fav.ico. In this url, there is no sid, so the session datas about the current survey remain untouched ($SESSION['survey_{sid}']). So, when you'll click on next, LEMpostKey in POST and in session will be the same, and you'll can see the next page.


Should there be a semicolon after:

result = this._userService.addUser(this.user)


i love code


I have a change for this, it also fires more requests in case we haven't hit the retranslateAll threshold.  Ill push it.


Oh well at least Ivy Bridge works now which is enough I think since it's still recently newer , dunno If Sandy will work but I think not since I think it doesn't support gl 4 at all.


TeamCity Code :: 1. Development :: DDK :: Feature [Build 236](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=19352&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Tests passed: 42 Build time: 00:04:31


https://514-53988135-gh.circle-artifacts.com/0/tmp/circle-artifacts.rcwN7uP/build.txt

It seems like the ros ubuntu repos are down? Not much you guys can do about it, unfortunately...


@sooti


 * Version update
 * modules list in config file
 * nginx-module-graphite
 * Modules upstream update


Changed Ordered list for professional Summary


Yeah that's what it seemed like, bleh.

On Wed, Apr 5, 2017 at 10:11 AM Jay Kamat <notifications@github.com> wrote:

>
> https://514-53988135-gh.circle-artifacts.com/0/tmp/circle-artifacts.rcwN7uP/build.txt
>
> It seems like the ros ubuntu repos are down? Not much you guys can do
> about it, unfortunately...
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/gtagency/buzzmobile/commit/34d006afe8483439912d9c1c890cb39a16daf99a#commitcomment-21641646>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEz4AJ7he1-mS1govFq1G1MDjVRdIK2cks5rs8tRgaJpZM4M0eFQ>
> .
>


Jbuilder doesn't support this scenario. It will generate one META per package. What you can do currently is manually add cohttp.lwt and cohttp.async as aliases for cohttp-lwt and cohttp-async


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.405.e65e8b866425e23ec8b3fc47718069fa7ac9d614](http://build.palaso.org/viewLog.html?buildId=87557&buildTypeId=bt2) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:01:35


Could you clarify "not written anymore"? Do you mean situations where the cached config can be used but will never be modified? I think one challenge we are getting into here is that we are starting to view these as components if we do this, versus having a "whole program" view like we have now, but then we need to start thinking about versioning and dependencies (e.g., if a module we use from another Rascal project uses a Rascal function that either no longer exists, has been modified, or is in a newer version than what we are using).


Don't do this. Node creation is called from the `create_node` route, which is decorated with `@db.serialized` so that the entire route runs in a database transaction with an increased isolation level that will be retried if it conflicts with any concurrent calls to that route from other clients. That's important to make sure that the network gets filled consistently when multiple clients are calling the route simultaneously. If you make your own commits rather than relying on the commit made by the decorator, then in the case where the decorator's commit fails, it will only roll back some of the changes before trying again.

What was the motivation for this change?


这里是不是缺少了：`CONFIG_TCP_CONG_ADVANCED=y` ？ 不然 bbr 的不会生效？


#17 Alright, I've added a second example and added some clarification to show how each XP point is added.


about fixing that bug: I agree. First we must test if only the favicon is affected (what about wrong url for css files, js files, etc). Then, calling index.php/survey/index/sid/{sid}/newtest/Y/lang/ with a wrong value for lang should be forbidden (and maybe other urls could be concerned).


it's not a good idea to pass index to key


Thanks but some people may want to use it... i'll consider it in any case


you can skip constructor


you can use => <Task .....    instead of return


Changing what `__dict__` is seems like a bad idea. What problem are you trying to solve? If it's just being able to do `config['setting_name']`, that's what `__getitem__` is for.


Add some class names

Preview: documentation | landing | table
Coverage: core | datetime


Yes, I know this, but I didn't understand, what change did you have on mind? Give some concrete example please :)


``$DeleteFromDB->bindParam(':id', $_GET['id'] );``


props should be passed to constructor and super


@a-frobot docs-v0.5.0


Hello Kevin,

Just a few quick ones

 - are all tests passing on `master` right now? On my setup I have a few failing tests on `master`, that is why I'm asking
 - if they all pass, which version of Python you are running your tests against?
 - are you directly running `py.test` or `tox`?
 - it seems like the kquick/Thespian fork is the new home of the project
    - if so, will you submit a PR from kquick/Thespian into godaddy/Thespian since there are the fork master branches now contain similar commits under different refs, by different authors which makes tracking both reps harder


``header("Location: index.php");``


better to not use this method (it creates new function every time)


bad idea to pass index as key


That's correct.

Did you look at the order of deletion question? If you removed the delete call from the destructor (or deleted the destructor as you did), and just relied on Qt's object lifetime management? Whose memory would be vacated first: QLocalServer's or Server's? (Speculate first; then see if you're right. Then try to figure out why.)

Hint: to see which object got deleted, connect to each object's ```destroyed``` signal. You can do it with a lambda syntax:

```C++
QObject::connect(&obj1, &QObject::destroyed, [] () { qDebug() << "obj1 destroyed"; });
```


Created multidev environment [drupalcon-github-magic#ci-4](https://dashboard.pantheon.io/sites/710b4a22-7cfc-445b-bd19-e576911ed625#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-drupalcon-github-magic.pantheonsite.io/)


@mdejuan : Can you push OptinCustomerDTO as well to master, seems your changes are breaking build due to OptinCustomerDTO


not from here - dev/ixgbe/if_ix.c:2644: "Unable to map MSIX table \n");
not from here - dev/ixgbe/if_ixv.c:1382: "Unable to map MSIX table \n");
not from here - dev/ixl/if_ixlv.c:1195: "Unable to map MSIX table\n");
not from here - dev/ixl/ixl_pf_main.c:1398: "Unable to map MSIX table\n")
from here ---> /usr/src/sys/net/
  4878:iflib.c

/*
 * bar == -1 => "trust me I know what I'm doing"
 * https://www.youtube.com/watch?v=nnwWKkNau4I
 * Some drivers are for hardware that is so shoddily
 * documented that no one knows which bars are which
 * so the developer has to map all bars. This hack
 * allows shoddy garbage to use msix in this framework.
 */
if (bar != -1) {
            ctx->ifc_msix_mem = bus_alloc_resource_any(dev,
            SYS_RES_MEMORY, &bar, RF_ACTIVE);
            if (ctx->ifc_msix_mem == NULL) {
                        /* May not be enabled */
                        device_printf(dev, "mark-E Unable to map MSIX
table\n");
                        goto msi;
            }
}

2017-04-05 18:15 GMT+03:00 Michael Pro <michael.adm@gmail.com>:

> root@f02ps:/# pciconf -lbv none1@pci43312:0:2:0:
> none0@pci43312:0:2:0:   class=0x020000 card=0x00000000 chip=0x152f8086
> rev=0x01 hdr=0x00
>     vendor     = 'Intel Corporation'
>     class      = network
>     subclass   = ethernet
>     bar   [10] = type Memory, range 32, base 0xfc000000, size 16384,
> enabled
>     bar   [1c] = type Memory, range 32, base 0, size 16384, enabled
>
> 2017-04-05 18:11 GMT+03:00 Michael Pro <michael.adm@gmail.com>:
>
>> root@f02ps:/usr/src/sys/dev/ixl# pciconf -lvBv
>> hostb0@pci0:0:0:0:      class=0x060000 card=0x00000000 chip=0x71928086
>> rev=0x03 hdr=0x00
>>     vendor     = 'Intel Corporation'
>>     device     = '440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled)'
>>     class      = bridge
>>     subclass   = HOST-PCI
>> isab0@pci0:0:7:0:       class=0x060100 card=0x00001414 chip=0x71108086
>> rev=0x01 hdr=0x00
>>     vendor     = 'Intel Corporation'
>>     device     = '82371AB/EB/MB PIIX4 ISA'
>>     class      = bridge
>>     subclass   = PCI-ISA
>> atapci0@pci0:0:7:1:     class=0x010180 card=0x00000000 chip=0x71118086
>> rev=0x01 hdr=0x00
>>     vendor     = 'Intel Corporation'
>>     device     = '82371AB/EB/MB PIIX4 IDE'
>>     class      = mass storage
>>     subclass   = ATA
>> none0@pci0:0:7:3:       class=0x068000 card=0x00000000 chip=0x71138086
>> rev=0x02 hdr=0x00
>>     vendor     = 'Intel Corporation'
>>     device     = '82371AB/EB/MB PIIX4 ACPI'
>>     class      = bridge
>> vgapci0@pci0:0:8:0:     class=0x030000 card=0x00000000 chip=0x53531414
>> rev=0x00 hdr=0x00
>>     vendor     = 'Microsoft Corporation'
>>     device     = 'Hyper-V virtual VGA'
>>     class      = display
>>     subclass   = VGA
>> none1@pci43312:0:2:0:   class=0x020000 card=0x00000000 chip=0x152f8086
>> rev=0x01 hdr=0x00
>>     vendor     = 'Intel Corporation'
>>     class      = network
>>     subclass   = ethernet
>>
>> 2017-04-05 17:12 GMT+03:00 Dexuan Cui <notifications@github.com>:
>>
>>> dev/ixgbe/if_ix.c:2644: "Unable to map MSIX table \n");
>>> dev/ixgbe/if_ixv.c:1382: "Unable to map MSIX table \n");
>>> dev/ixl/if_ixlv.c:1195: "Unable to map MSIX table\n");
>>> dev/ixl/ixl_pf_main.c:1398: "Unable to map MSIX table\n")
>>>
>>> Can you please find out where "Unable to map MSIX table" message comes
>>> from and check the rid before the line bus_alloc_resource_any()? the
>>> rid/BAR number must match with the BAR number shown in "pciconf -lvBv".
>>>
>>> —
>>> You are receiving this because you commented.
>>> Reply to this email directly, view it on GitHub
>>> <https://github.com/dcui/freebsd/commit/0d205f947a262bd0cde2bfa4d783ad66fe50fb92#commitcomment-21638169>,
>>> or mute the thread
>>> <https://github.com/notifications/unsubscribe-auth/ACJn3vD1UxFUZhfSy_I4bU6ZtGyJ1TP4ks5rs6FegaJpZM4Mxp9->
>>> .
>>>
>>
>>
>


Ouuuuuuuuuuulà attention c'est une copie de pointeur, cela ne va pas faire une copie des éléments de la grille et on pourra potentiellement y accéder via plusieurs boards, ce qui n'est pas le top du top. Je corrige ça :)


dlaczego ? ;)


``$Result = $readFromDB->fetch();``


:+1: However, could it be that Laynon does not properly minify the code?


Thank you @daelvn for the alternative GetOpt link 👍


rm ``foreach ($Result as $item)``


Am confused about the prime_number problem, I feel I don't understand what
it should do. I understood the problem as it is to generate all prime
numbers from 0 to n,where n is the number given to the function.
Please,explain to me what is required.

On Wed, Apr 5, 2017 at 6:20 PM, Loice Andia Kivisi <notifications@github.com
> wrote:

> Call the function during the test
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/meshack-mbuvi/slc-17/commit/844f35746369d5a919662ef17b25634748d4a7bf#commitcomment-21639587>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AI1bIpeqhTohUZP7d1EMozIienFTZ1Krks5rs7FSgaJpZM4M0Z5f>
> .
>


Hahaha u deleting stuufff wonder yalo vo hahaha stupid


btw, jak byś to poprawił ?;)


Yep, see the bottom paragraph of my previous comment.


Play haha


you can also add this to your `<head>`
`<style>
 div#sheet > h5, div#sheet > h4, div#sheet > p { text-align: center; }
</style>`


@jeremy-lunarg I can certainly believe that as various error messages were slightly modified and a couple of IDs were merged into a single ID. I suspect that they're existing unexpected errors that fall into these cases and so aren't being correctly masked now. Send me the details and I can fix it up.
Are unexpected errors going to cause test fails soon?


Lmao. We'll just wait and rebuild later today


Haga to late


@adefelic : Um... the method I suggested may be misleading, and I didn't mean to mislead. I just looked up Qt's source code, and ~QObject fires the ```destroyed``` signal near the top of its code, before it takes care of child objects.

Let me think...

Suppose in addition to logging QLocalServer's and Server's destroyed signals you also re-write the tail-end of your main() function like this:

```C++
 int main(...) {
    ...
    int result = 0;
    { // extra scoping to force Server destruction
        Server server(...);
        ...
        result = app.exec();
        qDebug() << "About to exit server variable's scope";
    }
    qDebug() << "About to exit main";
    return result;
}
```

Does the order of *those* printouts surprise you? (It surprised me - and I was expecting to be surprised.)


``$item = $Result[0];``


Hey, just wondering what addition this is since I could merge it into my fork?


Any reason for removing this row? This seems to be use when you don't specify a fixed scale and step, which then allows you to zoom to week level.


~~the~~


Autonomous soccer playing robots


Gracias me abristes los ojos con mas info


I'm suggesting to move the `(MASTER, NATIVE)` after the first sentence. Sentence 1. One of MASTER, NATIVE. Sentence 2.


rm form from indexAction plz


nice change


No default value  in PUT.


так погано робити. Почитай про формат дати і як з ним працювати.
https://developer.android.com/reference/java/text/SimpleDateFormat.html
https://developer.android.com/reference/java/text/DateFormat.html
і ще оце можеш глянути
https://developer.android.com/reference/java/util/Calendar.html


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 10](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1535&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


> Yep, see the bottom paragraph of my previous comment.

I see. You managed to answer the question without discovering the mechanism by which the child gets deleted (or at least you didn't mention it). That's why I'm probing further. :-)


Do not change this line, it makes no sense. Also, it may affect migrations.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 11](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1536&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Haha


test


Leave me alone u stupid fuck u


this must be logical or ('||') not bitwise or ('|')


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 11](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1536&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:03:04


TeamCity Glyssen :: Glyssen pull requests [Build *.*.531.3a095a9a8d57a19d5221dac01b5ce3bc501d2cd1](http://build.palaso.org/viewLog.html?buildId=87572&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1666, ignored: 18 Build time: 00:07:18


Nice work dealing with the "collapsing parent"!


Looks like you may have left out a `0` here


@papr , using `*args, **kwargs` and the same error occured:

```
Traceback (most recent call last):
  File "main.py", line 637, in <module>
  File "main.py", line 521, in session
  File "pyglui/ui.pyx", line 169, in pyglui.ui.UI.update (pyglui/ui.cpp:9233)
  File "pyglui/ui.pyx", line 137, in pyglui.ui.UI.handle_input (pyglui/ui.cpp:8730)
  File "pyglui/menus.pxi", line 525, in pyglui.ui.Scrolling_Menu.handle_input (pyglui/ui.cpp:43091)
  File "pyglui/menus.pxi", line 542, in pyglui.ui.Scrolling_Menu.handle_input (pyglui/ui.cpp:42872)
  File "pyglui/ui_elements.pxi", line 409, in pyglui.ui.Selector.handle_input (pyglui/ui.cpp:20253)
  File "pyglui/ui_elements.pxi", line 420, in pyglui.ui.Selector.handle_input (pyglui/ui.cpp:20095)
  File "pyglui/ui_elements.pxi", line 439, in pyglui.ui.Selector.finish_selection (pyglui/ui.cpp:20549)
  File "pyglui/ui.pyx", line 332, in pyglui.ui.Synced_Value.value.__set__ (pyglui/ui.cpp:45324)
  File "main.py", line 300, in open_plugin
  File "plugin.py", line 260, in add
  File "/home/pupil/pupil_player_settings/plugins/screen_tracker_offline.py", line 73, in __init__
    super().__init__(*args, **kwargs)
  File "offline_surface_tracker.py", line 60, in __init__
TypeError: __init__() takes from 2 to 5 positional arguments but 6 were given
```