Those tests are pending because we will change the interface to structures


A lot better. Thx ;)


yeah, although it's conditional on vmware_fusion and thus probably only applies to Gregory, it really ought to be in its own commit


@ropilz hey man. could you fix the issue or not? a team of app developers are looking forward for your changes to be happended! :+1:


are you gaining something by having this logic in a function? it's only used once, and its return value is kind of a hodgepodge.


Sure. I always have to delete the stuff later. Just had not done it this time, yet. Is there a clever way to avoid collecting all the files there when building the packages?


picky: `getBubbleDisplayValue`


5110_stdbaropress_sealevel (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


If we're going to standardise one way or the other, I'd prefer it to be this one. We've previously (https://github.com/torchbox/wagtail/commit/d344dc39a16cb92a085989fabfde80cea5a0df04) found that passing URL names around makes it hard to deal with views that take extra parameters - for example, if/when we get round to converting snippets to this template, we'll need to pass the app_label and model name in the URL.

(Admittedly I was also trying to sidestep the need for an `object` variable, because I didn't have the Django docs to hand to look up what it should be called. I'll do that properly now - and also make `context_object_name` optional, because I suspect the site views aren't using it any more...)


we should squash this with the previous commit


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.912.63673d17ac65e7433585724dfe9fa2293a6b7fed](http://build.palaso.org/viewLog.html?buildId=61442&buildTypeId=bt431) outcome was **FAILURE**
Summary: Tests passed: 410, ignored: 4; exit code 1 (new) Build time: 00:02:46


:+1: ,  looks good.


Is there a way u could do it? I'm migrating over to a new computer and I won't have any computer access for most of the day.

Ur the best :)

Jack

> On Aug 18, 2015, at 11:41 AM, Evelyn Fifi Yeung <notifications@github.com> wrote:
>
> we should squash this with the previous commit
>
> —
> Reply to this email directly or view it on GitHub.
>


I'm not sure I follow. This is to keep from crashing when calling "jsrun.timeout_run(Popen(cmdline, stdout=PIPE), None, 'Compiling')".

As I understand it, initially all response files (except for DEFERRED_REPONSE_FILES) are expanded on the Settings object. Before emscripten.py is called, it looks like library and required exports are then appended to this list. So I let EXPORTED_FUNCTIONS become expanded on the Settings object for appends and then change it back to a response file (now only if necessary) before opening the emscripten.py process.


@ZyX-I The reason for those results is that we avoid all forms of "neovim" in technical/source code competely, preferring `nvim` and `NVIM` where possible. Even in user docs, we prefer `Nvim` where possible. Ideally for your generator name, I suggest `nvim`, but *if* you prefer "neovim"`, then it's better to avoid the `NeoVim` capitalization which is explicitly not used in any of our materials (in particular, http://neovim.io).

For better or worse, "Neovim" was the agreed capitalization in https://github.com/neovim/neovim/issues/149


Small typo there with *utilities* :)


Actually it happened with **git add <folder>**. Normally if I use the git to build in it also (yes I do that), then I do always a **git add <folder>/PKGBUILD** first and check with **git status** if any folder(s)/file(s) is/are tracked before I commit. The best solution is to have a clean commit repo and another one you build in. For some repositories I've some files tagged in **.gitignore**, but for git-header-folders there is no proper solution.


print("\nSürüm  kontrolü yapıldı, 3.X sürümü kullanılıyor\n")

karşılama = open("karşılama.txt","r")
print("\n",karşılama.read())
karşılama.close()

def kimlik():
    #KİMLİK FONKSİYONU
    print("1.) Merhabalar Yeni Kullanıcı Detaylı Kimlik Kartını Oluşturmak\n\
    ister misin?  Evet için 'E', Hayır için 'H' Olarak Cevap veriniz.")

    cevap = input("Cevap :")
    # BURADA LOWER() VE UPPER() İLE GİRİLEN VERİYİ BÜYÜK HARF KÜÇÜK HARF KONTROLÜNÜ SAĞLADIK
    cevap = cevap.lower().upper()

    if cevap == "EVET":
    # kimlik fonk başladı
        try:
            # Hata kontrolü başladı
            ad = input("\nAD :")
            ad = ad.lower().upper()
            #-------------------------------------------------------------
            soyad = input("SOYAD :")
            soyad = soyad.lower().upper()
            #-------------------------------------------------------------
            tc = input("TC :")
            doğum = int(input("DOĞUM YIL :"))
            #-------------------------------------------------------------
            ana_ad = input("ANA ADI :")
            ana_ad = ana_ad.lower().upper()
            #-------------------------------------------------------------
            baba_ad = input("BABA ADI :")
            baba_ad = baba_ad.lower().upper()
            #-------------------------------------------------------------
            memleket = input("MEMLEKET :")
            memleket = memleket.lower().upper()
            #-------------------------------------------------------------
            doğum_yer = input("DOĞUM YERİ :")
            doğum_yer = doğum_yer.lower().upper()

            print(
"""
 |-------------------------------------------------|
 |              | KİMLİK BİLGİSİ |                 |
 |-------------------------------------------------|
 |                                                 |
 |   TC       = {}
 |                                                 |
 |                                                 |
 |                                                 |
 |   AD                    = {}
 |   SOYAD                 = {}
 |   DOĞUM TARİH           = {}
 |   ANA AD                = {}
 |   BABA                  = {}
 |   MEMLEKET              = {}
 |   DOĞUM YER             = {}
 --------------------------------------------------
 \n""".format(tc,ad,soyad,doğum,ana_ad,baba_ad,memleket,doğum_yer))

        except (ValueError,ZeroDivisionError):
            hata1 = "Girdiyi kontrol ediniz. Rakam veya özel kaakter kullanımına dikkat ediniz"
            çiz = (len(hata1)*"-")
            print(çiz,"\n",hata1,"\n",çiz,sep="")

        except:
            hata = "Nedeni bilinmeyen bir hata oluştu..."
            çiz = (len(hata)*"-")
            print(çiz,"\n",hata,"\n",çiz,sep="")

    # kimlik fonk bitti
    else:
        print("Kimlik aşaması atlandı...\n",("-"*50),sep="")
        exit
#-------------------------------------------------------------------------------------
def kayıtlı_kullanıcı():
#-------------------------------------------------------------------------------------
    try:
        kaydet = open("kaydet.txt","w")
        ad = input("Kullanıcı AD :")
        ad = ad.lower().upper()
        kaydet.write(ad)
        parola = input("Parola :")
        kaydet.write("  ")
        kaydet.write(parola)
        havuz = "ALI","VELI","AHMET","CENGİZ","ŞEKER"

        if ad in havuz and len(parola) in range(3,8):
            tanım = open("tanım.txt","r")
            print("\nGiriş başarılı bir şekilde sağlandı...{}\n".format(ad),tanım.read())
            tanım.close()
        elif ad not in havuz or len(parola) not in range(3,8):
            yeni_kullanıcı()
        else:
            exit

    except(ValueError,ZeroDivisionError):
        hata1="Hata oluştu, ya sayı girmediniz yada 2. rakamı '0' girdiniz."
        print(("-"*len(hata1)),"\n",hata1,"\n",("-"*len(hata1)),sep="")
    except:
        hata2="Beklenmeyen bir hata oluştu."
        print(("-"*len(hata2)),"\n",hata2,"\n",("-"*len(hata2)),sep="")
#-------------------------------------------------------------------------------------
def yeni_kullanıcı():
    try:

        kaydet = open("kaydet.txt","w")
        print("Yeni Kulanıcı Kayıt Ekranına Hoş Geldin...\n")

        ad = input("Kullanıcı AD :")
        ad = ad.lower().ad.upper()
        kaydet.write(ad)
        e_posta = input("E-posta :")
        e_posta = e_posta.lower().upper()
        kaydet.write(e_posta)
        kaydet.write("  ")
        parola = input("Parola :")
        kaydet.write("  ")
        kaydet.write(parola)
        havuz = ad
        kaydet.close()

        if ad in havuz and len(parola) in range(3,8):
            tanım = open("tanım.txt","r")
            print("Giriş başarılı bir şekilde sağlandı...{}".format(ad),tanım.read())
            tanım.close()
        else:
            print("Hatallı Giriş")
            exit

    except(ValueError,ZeroDivisionError):
        hata1="Hata oluştu, ya sayı girmediniz yada 2. rakamı '0' girdiniz."
        print(("-"*len(hata1)),"\n",hata1,"\n",("-"*len(hata1)),sep="")
    except:
        hata2="Beklenmeyen bir hata oluştu."
        print(("-"*len(hata2)),"\n",hata2,"\n",("-"*len(hata2)),sep="")
#-------------------------------------------------------------------------------------
def dos_uzantı_sıralama():
    try:
        print("2.) Merhabalar, dosya uzantısına göre sıralanacak ögeleriniz var\n\
        bu sıralamayı yapmak için Evet için 'E', Hayır için 'h' basınız...\n")

        girdi = input("Cevanınız nedir ? = ")
        girdi = girdi.lower().upper()

        if girdi == "E":
            dosyalar =open("dosyalar.txt","r")
            liste = dosyalar.read()
            liste = liste.split()

            print("\n\n'dosyalar.txt' içerisinde en çok tekrar eden uzantılı\n\
            karakter dizilerini istedik...")
            for i in liste:
                if i.endswith(".com"):
                    print(("-"*15),i,sep="\n")
            print("\n")
            dosyalar.close()
        else:
            print("Dosya uzantısı sıralama uygulamasındna çıkıldı...\n",("-"*30),sep="")
            exit
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
#-------------------------------------------------------------------------------------
def dos_baslangıc_sıralama():
    try:
        print("3.) Merhabalar, dosya uzantısına göre sıralanacak ögeleriniz var\n\
        bu sıralamayı yapmak için Evet için 'E', Hayır için 'h' basınız...\n")
        girdi = input("Cevanınız nedir ? = ")
        girdi = girdi.lower().upper()
        if girdi =="E":
            dosyalar = open("dosyalar.txt","r")
            liste = dosyalar.read()
            liste = liste.split()
            print("\nDosyaları ilk harfine göre sıralama yapıyoruz\n")
            for i in liste:
                if i.startswith("d"):
                    print(("-"*15),i,sep="\n")
            print("\n")
            dosyalar.close()
        else:
            print("Dosyanın ilk harfine göre sıralama uygulamasıdnan çıkıldı...\n",("-"*30),sep="")
            exit
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
#-------------------------------------------------------------------------------------
def toplama():
    try:
        bilgi1 = open("toplama.txt","r")
        print("\nAÇIKLAMA METNİ :\n",bilgi1.read(),"\n",sep="")
        sayı1 = int(input("1. sayıyı gir :"))
        sayı2 = int(input("2. sayıyı gir :"))
        print("\n","\t",sayı1,"\n","\t",sayı2,"\n","+","\n",("-"*20),"\n\
        ","\t",(sayı1+sayı2),"\n",sep="",end="\n\n----FİNİŞ----\n\n")
        bilgi1.close
    except(ZeroDivisionError,ValueError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
#-------------------------------------------------------------------------------------
def çıkartma():
    try:
        bilgi2 = open("çıkartma.txt","r")
        print("\nÇıkartma işlemine hoşgeldiniz...\n",bilgi2.read(),"\n",sep="")
        sayı1 = int(input("1. sayıyı gir :"))
        sayı2 = int(input("2. sayıyı gir :"))
        print(("-"*50),"\n","\t",sayı1,"\n","\t",sayı2,"\n","-","\n",("-"*20),"\n\
        ","\t",(sayı1-sayı2),"\n",sep="",end="\n\n----FİNİŞ----\n\n")
        bilgi2.close()
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
#-------------------------------------------------------------------------------------
def çarpma():
    try:
        çarpma = open("çarpma.txt","r")
        print("\nÇarpma işlemine hoşgeldiniz...\n",çarpma.read(),sep="")
        sayı1 = int(input("1. sayıyı gir :"))
        sayı2 = int(input("2. sayıyı gir :"))
        print(("-"*50),"\n","\t",sayı1,"\n","\t",sayı2,"\n","*","\n",("-"*20),"\n\
        ","\t",(sayı1*sayı2),"\n",sep="",end="\n\n----FİNİŞ----\n\n")
        çarpma.close()
    except(ZeroDivisionError,ValueError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
def bölme():
    try:
        bölme = open("bölme.txt","r")
        print("\nBölme işlemine hoşgeldiniz...\n",bölme.read(),sep="")
        sayı1 = int(input("1. sayıyı gir :"))
        sayı2 = int(input("2. sayıyı gir :"))
        print(("-"*50),"\n","\t",sayı1,"\n","\t",sayı2,"\n","/","\n",("-"*20),"\n\
        ","\t",(sayı1/sayı2),"\n",sep="",end="\n\n----FİNİŞ----\n\n")
    except(ZeroDivisionError,ValueError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
def ü_alan():
    try:
        sayı1=int(input("'a' köşesini gir :"))
        sayı2=int(input("'b' köşesini gir :"))
        print("Sonuç :",(sayı1*sayı2)/2)
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
#-------------------------------------------------------------------------------------
def hacim():
    #HACİM FONKSİYONU
    try:
        print ("""Hacim formül = π.r.r.h \n(π=3,14 alırız, r taban yarıçapı, h yükseklik)""")
        sayı_r =int(input("r taban yarıçapı :"))
        sayı_h =int(input("h yükseklik :"))
        sayı_π = 3.14
        sonuç = (sayı_π*sayı_h*pow(sayı_r,2))
        print("'r'taban yarıçap :",str(sayı_r),str(sayı_r),"'h' yükseklik :",sayı_h,sonuç,sep="")
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    #HACİM FONKSİYONU
def üs_al():
    #KAREE AL FONKSİYONU
    try:
        print("Üs alma işlemine hoş geldiniz...")
        sayı1 = int(input("Sayı gir :"))
        print(sayı1,"  girilen sayısının üssü ",(sayı1*sayı1))
    except(ValueError,ZeroDivisionError):
        hata ="Hata oluştu, harf veya 2. sayıyı '0' olarak girdiniz , lütfen kontrol ediniz"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    except:
        hata ="Bir hata oluştu üzgünüm"
        print((len(hata)*"-"),"\n",hata,"\n",(len(hata)*"-"),sep="")
    #KAREE AL FONKSİYONU
def yardım():
    #YARDIM FONKSİYONU
    print("Yardım menüsü çalıştı...")
    yardım = input("Yardım alam istediğin işlemi gir :")
    print("\n",help(yardım))
    #YARDIM FONKSİYONU
#-------------------------------------------------------------------------------------

while True:
    kimlik()
    kayıtlı_kullanıcı()
    girdi = input ("Yapmak istediğiniz işlemi giriniz  :")

    # İf bloğu ile girdi değerlendirmesi yapılacak
    if girdi == "+":
        toplama()
    elif girdi == "-":
        çıkartma()
    elif girdi == "*":
        çarpma()
    elif girdi == "/":
        bölme()
    elif girdi == "a":
        ü_alan()
    elif girdi == "h":
        hacim()
    elif girdi == "k":
        üs_al()
    elif girdi == "y":
        yardım()
    else:
        print("\nProgramdan Tamamen Çıkıldı...")
        exit


Strongly agree! For my app, I've forked this repo and basically reverted this commit.

Similar reasons:
- I'd like to disable CSRF for most tests, but also want to enable it for specific tests.
- This also allows users to specify `app.config` settings either before or after `CsrfProtect(app)` and have it still work.


It is too good a kluge to let die.


ok. That's basically what I'm doing. :) will just have to pay better attention...


Simply do this and all your problems will be gone
https://github.com/SlimLG2/device_g2/commit/41196c7e6f5cb200621201f40aa95d2a16383e12


Эээх:)))

вторник, 18 августа 2015 г. пользователь brus-fabrika написал:

> хорошо, но очень сложно )))
>
> if(secondname1.equals(secondname2)) {
>    return name1.compareTo(name2);
> } else {
>    retutn secondname1.compareTo(secondname2);
>
> или даже еще проще
>
> int compareTo(...) {
>    return (surname1+name1).compareTo(surname2+name2);
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlgaGrygoryan/homework/commit/0fc0bfa6edeef27a4284a27f339218ec4846c1ea#commitcomment-12767765>
> .
>


It happened to me also several times, no biggy. We all learn ;)


This is not an accurate docstring.


I have replaced this with `Neovim` in all my docs, also generator is now named after the executable.


I'm going to be adding it to the Hybrid Base theme, starting today.


Doesn't your graph class have an API that covers this?  Why not use it?


TeamCity ATK :: QA :: Integration Per Branch [Build 1883](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545918&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests failed: 1 (1 new), passed: 439, ignored: 15, muted: 8 Build time: 02:33:54


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.913.c6b4d3877293188ef45049adaeac13a401aad106](http://build.palaso.org/viewLog.html?buildId=61444&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:03:12


\* after the standard executable name, not `argv[0]`. I did not think this is a good idea because this is easy to fake.


Done


Done. I should’ve followed convention instead of linking from Google in the first place :-1:

On Tue, Aug 18, 2015 at 11:38 AM, robes <notifications@github.com> wrote:

> so far we have chosen to import all dependencies into our project's "scripts/vendor/" subdirectory. Could you do the same for this (jquery-ui) script?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/informatics-isi-edu/chaise/commit/b4edfb75e025e0be04cce4ff2bdb016172f6d75c#commitcomment-12767856


Done - https://github.com/forcedotcom/SalesforceMobileSDK-iOS/issues/1147

Thanks


For this case, this should be `depends_on :cask => 'virtualbox500101573'`.


TeamCity ATK :: QA :: Integration Per Branch [Build 1885](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546011&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests passed: 422, ignored: 15, muted: 9 Build time: 01:48:51


Don't you want to put this before container.Start()?


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 391](http://192.168.30.225:8111/viewLog.html?buildId=2131&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Ah, I see - probably should join with `/` (or maybe `' '`) or something then to avoid e.g. `['a', 'b']` colliding with `['ab']` or something... annoyingly then you have to filter out `undefined` sub-paths.


Take a look at [this](http://csswizardry.com/2013/01/mindbemding-getting-your-head-round-bem-syntax/) site for a quick intro to BEM naming conventions.

The gist is:
 - `.block` represents the higher level of an abstraction or component.
 - `.block__element` represents a descendent of `.block` that helps form `.block` as a whole.
 - `.block--modifier` represents a different state or version of `.block`.

In this particular example the class describes a single component. If you separate the component's name with double underscores you are you are indicating a hierarchy that doesn't exist. To separate the different words, just use single dashes.


Was the deletion of the two log messages intentional, or a mistake resolving merge conflicts?


Know your whitespace!


Annoyingly we can't fully standardise on `get_add_url` right now, because wagtailadmin/shared/header.html still expects to receive a URL name when rendering the 'add' button. (Snippets currently sidestep this by not using the header include - which is further proof that passing URL names in template variables is harmful...)


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1804 of 1805 tests passed)

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


PopupViewImpl in it's own right is perhaps wrong; it represents a popup but extends Composite and wraps a Bs3Modal... surely extending Bs3Modal is more correct?


comment


I also created Issue #145 with a lot more information (some examples/discussion of workarounds) about this... I'm a bit new to Github, so I apologize for bringing this up in multiple places.


This has the ugly side-effect of us trying (and failing) to deactivate the device before proceeding to deactivating the parent devices recursively. I think it needs some expansion to prevent this.


hi Dave is doze working on d2 right now?
thanx


comment. this loop is...replacing all xpath expressions in the given val with their HTML bubble equivalents?


git remote add origin https://github.com/JasonHus66/ss-YWVzLTI1Ni1jZmI6MTMzODYwMzc2ODVAMjA4LjExMC44My4yNDI6ODM4OA-.git
git push -u origin master


definitely `\\s*` and not `\\s+`?


This is the `div` which wraps around the two columns of form fields. It's not really a separator (i.e. a thing which separates two other things) but more of a section wrapper. Hence I renamed it.


Will this always overwrite the username with the email address? if so I think it will cause issues for mobile workers or for web users that have a different email


same comment as above


O! This fixes pocket issue too? :smile:


@anthonydb Are you planning on doing any more tweaks to this page? I actually moved it out of `documentcloud` to the BPCS repo, over on the `bpcs` branch. This'll cause a merge error (I can resolve it, no worries), but can you check out the BPCS repo and make this same change there? It'll be in a view called `apply.html.erb` (some renaming occurred).


oh, i see there are some high-level comments with the tests.


Done!


Seems good. Just, example code seems estrange now that `Solo` became `solo`.

To avoid errors just, change `solo` (function parameter on test.js line 11) change `solo` to `worker` or some like that.


Is this bitwise OR on purpose? Or should it be a logical or || ?


any objects to making richtext two words? (throughout this PR)


Also, I'd _love_ to see this committed: I think I can stop using my fork once this is merged. So :+1:!


Won't this overwrite the username if the email and username are different?


(same comment as above)


This highly depends on the Distribution used. See: https://wiki.apache.org/httpd/DistrosDefaultLayout


If you take a look at the actual class definition of the element in the html you will see that it's

```html
<div class="button button--light donation-case-form__add-donation-case-button">
```

This is a very powerful feature of BEM: Components can contain other components.
In this particular case, the element is both: an individual component (`.button`) and as a descendent of `.donation-case-form` (`.donation-case-form__add-donation-case-button`).

This allows us to separate concerns and make styles more reusable.


Since errors on this are common and somehow expected, there si no need to log them.

Please remove `console.log` and we're done.


oops.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 391](http://192.168.30.225:8111/viewLog.html?buildId=2131&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:11:51


is NS_MAIN not a namespace or did you forget + at the beginning of the wiki name?


TeamCity ATK :: QA :: Integration Per Branch [Build 1884](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545933&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests failed: 10 (10 new), passed: 431, ignored: 15, muted: 7 Build time: 02:47:53


Unless I'm much mistaken, this is not actually Dijkstra's algorithm.  What you appear to be doing here is finding the path from `start` to `end` where *at each step* you choose the smallest weight.  But this does not guarantee that you've found the shortest path through the entire graph, merely that you took the easiest step at each junction.  Consider a graph with these two paths:

A -> (B, 3) -> (C, 1) -> (D, 1)
A -> (E, 1) -> (F, 10) -> (D, 1)

If I'm reading your algorithm above correctly, then when you leave A you will choose the path that goes via E because it's 'easier' than the path through B.  But the total weight of the paths shows that the 'shortest' path should be the one that goes via E.


Another thing you might notice is that the element has a `.button` as well as a `.button--light` class. This is the correct way of adding a modifier to an element, since css doesn't know about hierarchies.


This is just a test repo.


no. The delay is meant to be how long to wait for things to be started in the container, so it does need to be after the start.

As for the Atoi, it's going to fail when the key isn't set, leading to a 0s delay which is the expected default.


Albo: `order(count: :desc)`


@gembarrett This will aide in the UX for the form- different UI elements for success/fail messages


`and` is a part of the QL and should be uppercased to `AND`


bhai lala wo front page pura chahiye kese bhi sunday tak,,

kam se kam layout complete rakhna,
banglureuniversity jesa......

content apn bad me daal dnge...


Done.


There is no documentation D:


good catch, fixed.


Is this available in Melpa?


BTW `&__section` reads out as `.donation-case-form__section`. You can do some pretty fancy stuff with Sass's ampersand selector: http://seesparkbox.com/foundry/how_to_use_ampersands_to_simplifiy_your_sass

For more information on how to use BEM modifiers correctly, refer to this comment (or just read along):
https://github.com/Tafelstellwerk/stellwerk/commit/304c7cb73d38640f75a51a573f8fa19b7b152aa4#commitcomment-12768426


Maybe you need to do
```
git pull upstream master
```
check what your remotes are by doing
```
git remote -v
```


It's a separate component!


Hi Cyril,
 What's the error you are getting? I have topoplot working ok when I call it from limo_results.
 Best,
 Ramon


These tests do not prove that you can find the shortest path, because the shortest path also happens to be the path that starts with the smallest hop.  Your tests need to test that the goal of the algorithm can be achieved even with "bad" starting conditions.


Hitting "c" from the problems buffer seems to be intended (hence the key binding), but this was not actually working becaue it needed to switch buffer first.  Note the eclim-java-correct line which only takes a line number into the *current* buffer.


Excellent change - we definitely don't want to have to update this code *every* time the Pro version changes (yikes)

Note: for things like this I was checking the python version - but that assumed that Arcmap would never upgrade to 3 - so I like your approach.


Arthur, I am trying to run the MvcMusicStore.Spa application with a local SQL Server database, and I'm hitting the issue below on the call to await db.SaveChangesAsync();

System.Data.SqlClient.SqlException
Cannot insert explicit value for identity column in table 'Genres' when IDENTITY_INSERT is set to OFF.

Looks like your change was intended to fix this.


s/red/read


lol


nit: In `your schema design will reflect that difference` change `will` to `should`.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 352](https://teamcity.labkey.org/viewLog.html?buildId=267120&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:00
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


Should be `'time'`, but not a big deal.


@Drup Right. Do you actually have any real world usage of `with_strf` or you just put it here along side `with_file` ? I have the impression that this combinator will lead to ugly code rather than forcing the user to have a good decomposition of its formatting process into pretty-printers.


Or maybe not, given the fact that it probably makes sense to name the class `TimeUtility` instead of just `ITime`. So let's just keep this ass is.


check this https://github.com/cooler-SAI/WoWSCore548 gonna be usefull :)


:+1:   Nice work!


Only the template does.


Эта штука тоже специфична только для клиента. Верни ее обратно.


Why not use a ConcurrentStack and remove the need for locks?


Hum...


Messages zipping by (saving file ... saved file ... refreshing) are quite distracting. For example, in this case one wonders who is refreshing what. Whereas the * is the modeline is discreet and directly adjacent to the counts about to be updated.


Hehe sorry this might be a bit out of the blue... The backstory is that a colleague of mine recently upgraded his app to use the latest MvvmCross UserDialogs plugin, and wondered why he got errors that `IUserDialogService` could no longer be found. I remembered that `IUserDialogService` is the old name for `IUserDialog`, and decided on a whim to see if there were any references to the old `IUserDialogService` in the repo.

This is the only reference to `IUserDialogService` in the repo, hence my confusion; it seems this file might no longer be in use though?


Not like this. You need to keep the length of msgstr to be nplurals, otherwise PO file will not be valid. When deleting strings, you basically need to set them to empty string: "".


nice. it's good for building new structure too.


curious about the outer paragraph tags here...is that something ckeditor does? can users enter newlines, and will `contenteditable=true` cause more markup to be added if they do? (sorry this is vague, but i've had issues with contenteditable divs misbehaving, especially regarding newlines, especially cross-browser, and am curious how much if ckeditor protects us from this kind of thing)


output-value is...true if the xpath points to a question in the form and false otherwise?


Isn't this unused?


This patch is inaccurate, the addition of flags to $(LD) should be done as part of the definition of the LD_FLAGS macro.

you should have an additional

LDFLAGS += -fprofile-arcs -ftest-coverage -lgcov -coverage

on the line 230 ..


We have an error count in the modeline, but no way to actually *get* to those errors easily. One could go to the problems buffer, find the current file (NB: I was fixing / ignoring lots of warnings in a project when I was doing this), try to remember where they were,  only to be booted back to the top at the first refresh.

In Eclipse, the visual aid is the red markers in the scrollbar. We needed something to fill the same need in emacs.


actually, no, output values don't have to be questions. what's output-value for?


This merely allows clicking around in javadoc. Can't think of any negative side effects.


No idea. They need a contributor agreement signed, but I'm perfectly fine with them just taking this and committing it themselves. It's a nasty bug.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 353](https://teamcity.labkey.org/viewLog.html?buildId=267124&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:24
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


The problem this solves is that variable completion gets mixed in with types because of different casing. Since there are A LOT of types, completion becomes useless. In any event, the option ignore-case=nil seems to have this intention, but it doesn't go all the way.


is `bundle.obj` an instance of CouchUser? If so then we should remove `.username` here


(same comment as above)


:+1: Thanks!


It looks like this change: https://github.com/devknoll/relay-nested-routes/pull/3/files#diff-1fdf421c05c1140f6d71444ea2b27638R11 got dropped in the merge. Without it, the data gets requested repeatedly due to updates to the root from changes in router transition state.


This patch is a mistake - in the most recent one there is a new edition to Make.common.in


@Myoldmopar I changed the example file name in the CMake file to be consistent with the idf but I see the same error.  Let me double check.


peut etre mais ce n'est pas ici que ça plante malheureusement, cela dit je vais essayer de lock le bordel pour voire (le code qui fait panté est ce que j'ai ajouté dans le EntityManager en commentaire


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.914.c6b4d3877293188ef45049adaeac13a401aad106](http://build.palaso.org/viewLog.html?buildId=61448&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:03:14


Implement was pretty broken in a lot of cases, beginning with the requirement that yas be available and on; eclim should not require yas, whis is the case everywhere but here.

For the details, see commit message.


97204612_MultiSpeedFan (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1805 of 1805 tests passed)

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


Fixed


minor: `else` should be indented below `then`


we lost several files with this commit and everything is breaking. at the very least, restore .travis.yml and .gitignore please!


Done.

hmm what do you mean? Something like eddystone.chrome?


I've noticed a performance slowdown and this may be the culprit


Done.


Set up for Selenium Webdriver demo session.


The '|| true' here has no purpose, it needs to be after the last statement.
The problem is not introduced by this commit but we might as well fix it...


Yes sorry I forgot to update version number before tagging.
Ideally I would avoid having to sync async on marmalade each time I make a change.
Isn't marmalade capable of syncing from github ?


Do you consider we lost more precision if the result is even?


find ich nicht gut..


hm


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (2222 of 2241 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.15%25-yellow.svg)


I think we should commit to not breaking the API in a minor release. We can make additions (and it might be worth mentioning what can be considered an addition). We probably have done this before, but I think we've reached a point in v3 adoption that we can't break the API anymore.


change to themes/themes.profile


themes/package.json


There appears to be a typo: "canols oil" should be "canola oil"


Change to whatever the compiled version of the theme will be (once built with Stylus)


thanks for the pointer, i keep forgetting to monitor TC commits :/

there is an issue though.
the new relationship between go_template and go_template_locale is one => many instead of one => one. with your change you are still only matching one per row.

you can do it like this
```sql
       SELECT
            go.name,
            gtl2.`name` AS name_loc2,
            gtl3.`name` AS name_loc3,
            gtl6.`name` AS name_loc6,
            gtl8.`name` AS name_loc8,
[...]
        FROM
            gameobject_template go
        LEFT JOIN
            gameobject_template_locale gtl2 ON go.entry = gtl2.entry AND gtl2.`locale` = 'frFR'
        LEFT JOIN
            gameobject_template_locale gtl3 ON go.entry = gtl3.entry AND gtl3.`locale` = 'deDE'
        LEFT JOIN
            gameobject_template_locale gtl6 ON go.entry = gtl6.entry AND gtl6.`locale` = 'esES'
        LEFT JOIN
            gameobject_template_locale gtl8 ON go.entry = gtl8.entry AND gtl8.`locale` = 'ruRU'
```


Oui mais tu joue avec le feu a faire des find et des insert dans une map ! hahaha
C'est priorite number one.
Ton crash pourrait venir du fait que tu appelle un reload de material alors que le material n'est pas encore loadé. Je n'ai pas regardé ton code en detail, mais j'avais mis un load assynchrnone, donc quand tu ajoute un component meshRenderer il peut se passer plusieurs frame avec le material invalide parce qu'il est en train de loader


Yes, for code where you will have various branching and non branching part and you want to compose a string like that. Sure, you could define a pp and then use it, but that's forcing a "datatype" vision on a problem that does not necessarily need it.

I don't see how it's ugly.


need to add .style to this list of file names to ignore.


should remove robot and robotx...


)
sudo kextload /System/Library/Extensions/AppleHDA.kext
reboot
start terminal - /GA-Z77X.sh --inject-hda    and ...watch screenshot

./GA-Z77X.sh --install-clover watch screenshot

2015-08-18 11:34 GMT-07:00 Alex James <notifications@github.com>:

> Hmm, what happens when you force AppleHDA to load before running it?
> sudo kextload /System/Library/Extensions/AppleHDA.kext
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theracermaster/Gigabyte-GA-Z77X-DSDT-Patch/commit/788781a32f7840ae0a9330ab58e724a2cd2daf3f#commitcomment-12767757>
> .
>


The fact that the IDs are always odd suggests to me that it might actually be creating the even-numbered IDs somewhere and they're just not getting rendered to the XML file.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.915.eef6f12ea7c403607e9dfd9ceac3f11f82779bad](http://build.palaso.org/viewLog.html?buildId=61449&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:02:43


I read somewhere that rendered partials inherit access to the instance variables of their parents, so I figured why use two local variables? It makes the code simpler to drop them, and saves on resources.

In response to your questioning my actions, I found this: http://rails-bestpractices.com/posts/2010/07/24/replace-instance-variable-with-local-variable/

And:  http://thepugautomatic.com/2013/05/locals/

Now I understand a bit better, and will change it back.


What if we did it this way instead:

Place the closing `</informalexample>` or `</example>` inside an `if` block that says "if the next block is a `<calloutlist>`, then don't render this tag."

Then at the end of `block_colist.docbook45` we add an `if` block that checks whether it was an informalexample or an example and adds the corresponding closing tag.

This way we wouldn't need to override where the callouts render or mark them to be skipped or anything.

Seems like this should work; I just don't know how to write that second `if` block.


Oh, I've just forgot to add to pull request DISTINCT right after SELECT


Just realized that I got preempted and forgot to comment here; Indentation should be 4, not 2


:+1: :birthday:


why looking specifically for `\r\n` and `\n\r` instead collapsing all whitespace with `\s+`?


commented here


comment left


@racodond both of these rules will traverse the entire folder structure, could we not simplify it by searching it once and when we find a manifests directory checking the siblings for 'tests' and 'metadata.json'?


if you can address the indentation, and this, then I'm happy to pull this. I'm about to make a new release sometimes this week, so the sooner the better.


Je viens de trouver pourquoi ça crasher c'etait pas du à ça, juste un probleme de variable temporaire. Normalement tout les appel son lock comme j'utilise les meme methode pour reload au niveau de l'asset editor. Et que je lock avant l'update dans le world editor


thank you


Minimum of 1.0.0?


@dtulig, 8d29ed7 fixes that. Thank you for the bug report.


Does this add an upgrade code? Sure it doesn't replace it?


done.


Nit on commit message:

```
This enables the MSI to upgrade old Node.js versions and
early versions of io.js before the upgrade code was changed.
```


No, it was too minor.  It was only made for some small repoman changes to be used with the new git based ebuild tree.

I threw in a couple man page updates as well that will be in the next release.


Dma channels are only used/assigned when the spi driver is loaded.
The corresponding ids 6 and 7 are only describing the DREQs that will trigger further transfers from/to the spi-hw block.


Comentario


Caused by https://github.com/ZeroK-RTS/Zero-K/commit/013492f213e93a30b745fe13f19074b98470269a

Why was yardmap removed in the first place? It's the first way engine determines things to be structures.


why this?don't you like the "home" ?


disp might have paragraphs but definitely won't have line breaks? is calling `.text()` an option? it's less sketchy than regexing out individual HTML elements.


I've added another commit to fix this.
It probably makes sense to squash that commit, but for now I'm leaving it separate for ease of review.


NODE_0X_DETECTED and EARLY_IO_DETECTED aren't used anywhere. Do we need the separate checks or can it be done with one single check?


You can change `doneMessages` instance to `linkedSetOf` and use it instead of `messagesForHashCode`


Renamed `findCreateEntry` to `emplace`, and changed it to return `std::pair<Entry&, bool>`. (Please double-check that I used `std::ref` correctly to initialize the `pair`.)

Also cleaned up the callers, including changing the return type to `void` on one that wasn't checked, and removing another that wasn't used.


:facepalm:

The easiest solution I can imagine is a slightly customized string format that uses RegExp-like behavior. Say, a question-mark character following a zero character makes it optional.

In the `Map` population loop, each iteration will add two keys: One with question mark characters stripped, and one with "0?" sequences stripped. It's not a future-proof solution, but it would fix the weird driver discrepancy without changing the `Map` behavior.


We could but I prefer that each rule remains independent. For my whole code base, it adds three seconds to the analysis (30 seconds for 100,000 lines of code). So, performance is not an issue for now from my point of view.


Need to update with URLs for GitHub and Bitbucket also


Should this generate a getter for `click`? I would think so, but I don't think there's code to do it currently.

Would we use `fullName` from `EventBinding` in `processor.dart#_processProtoViewDtoEventBindings`?


^ saying that in the PR is helpful. putting it as a comment in the code would be more helpful.


Add URLs here also


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 354](https://teamcity.labkey.org/viewLog.html?buildId=267134&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:52
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


yup :)


Thanks Scott!


Great :smile::+1:


This is actually the part that could help out performance-wise; would be interesting to run the QP benchmarks again


This will register all level-1 instances.

If I go to the search page, and then open 4 pages from there, you will register 4 events, no?


using the mixins for `.box-shadow` and `.transition` would make this more readable


The `GraphExecutor` calls this on the component immediately after modifying its state to `SUSP_SEND` and `SUSP_RECV`. If I modify the executor impls to directly call `self.suspend_thread()` instead, I don't see any problems enabling this assertion.


OK, we can review this if we add more of this type of check in future.


Can be called after `generateForMessage` in `generate`, so `messagesWithDiff`can be local val


> Sure, you could define a pp and then use it, but that's forcing a "datatype" vision on a problem that does not necessarily need it.

This doesn't entail a datatype vision. What it does is make you define `pp` functions in the first place. Which is good if you later want to expose them or compose them. I don't feel that introducing a name readers will have to learn is worth it for such little functionality, since you are really not far from `strf`:
```ocaml
let pp ppf () = ... in
strf "%a" pp ()
```

So I'll drop it.


I agree that it smells. The intention of the `detail` strings are to store the `reason` determined in `checkValidity`.

On second thought, though, since the reasons are only used for logging, there really is no need to store them for any length of time. If the cached flags indicate a failure, then the reason can indicate that.

I will remove the `get/setDetail` functions and members.


I'm not an expert on TPL - but does it make sense to use `ConfigureAwait(false)´ here?

Rebus has worker threads whose `SynchronizationContext` is set to [this bad boy](https://github.com/rebus-org/Rebus/blob/master/Rebus/Workers/ThreadBased/ThreadWorkerSynchronizationContext.cs), causing continuations to be added to the queue.

Worker threads then do their work by either pulling a continuation from the queue, or try to receive the next message.

This `ConfigureAwait(false)` thing - doesn't it somehow cause the continuation to escape the Rebus worker threads and be run on the thread pool?


Sure


rax retry tests


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (753 of 753 tests passed)

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


Agreed. I'll fix it.


They are constants.  Two questions.
1) I felt that leaving them in the method that uses them helped to keep the math clear.  Someone looking through the code can see exactly what formula I'm using to calculate ABV (or whatever else) without having to check multiple places.  Is that a good idea?
2) Most of these constants are used only once by one method, so isn't it a little redundant to say ```WEIGHT_OF_ETHANOL = 1.05``` at the top and then ```WEIGHT_OF_ENTHANOL``` again in the method?  42 characters instead of 4. Or is it more of a style issue?


What is the difference?


oh, nevermind, you're copying sections from hqstyle-core.css, that's why this isn't really LESS. can you explain more about why this commit is necessary?


FYI, you might be able to use the ReStructuredText plugin for PyCharm to preview markup changes: https://plugins.jetbrains.com/plugin/7124


*Rubocop :x


Nice touch :-)

(The purist in me asks why C++ still does not have a mechanism to attach `requires` clauses to template arguments to simply remove the whole function from function lookup if the constness does not match :-( )


I noticed at one point it looked like there might have been a newline after the . in the minified code.

Might have been my editor though.... not sure...


Do you really need to call toString() in the line above?


```jl
julia> m = Model();

julia> @time m == m
  0.000006 seconds (153 allocations: 10.480 KB)
true

julia> @time m === m
  0.000003 seconds (4 allocations: 160 bytes)
true
```


Code LGTM. Well done doing some much-needed curation and clean up.


Suggested refactoring, that solves both problems: https://github.com/typeinference/kotlin/commit/a89b3b0a3d9b405ef7893fcd33fa3aff3b51d1f8


Да, переименуется.


Consider initializing timeout in code that calculates _indexOfLastPageAdded.

Consider testing whether the thumb cover actually exists yet and wait again if not.

Or if possible (or todo?) do something with async so this happens as soon as possible


(Lol bad example, that's just compilation time)


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2217 of 2241 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.93%25-yellow.svg)


```
julia> @time m === m
  0.000001 seconds (4 allocations: 160 bytes)
true

julia> @time m == m
  0.000004 seconds (4 allocations: 160 bytes)
true
```


#issue #89


* Support XBMC/KODI 13.x and later (https://github.com/Diblo/KODI-Popcorn-Time/issues/3)
* Support Linux ARM (https://github.com/Diblo/KODI-Popcorn-Time/issues/18)
* Removed Android x86 support (https://github.com/Diblo/KODI-Popcorn-Time/issues/15)


It's the standard access path for libraries on MacOS X.


@buggtb Could you help tell me where I can find >4.3.0.1-SPARK? Thanks.
<dependency>
                 <groupId>pentaho</groupId>
                 <artifactId>mondrian</artifactId>
-                <version>4.3.0.1-130</version>
+                <version>4.3.0.1-SPARK</version>
             </dependency>


see? you do it on mondays.


They do it on mondays :)


TeamCity ATK :: Source Code Compile and Test [Build 8608](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545487&buildTypeId=Gao_SourceCodeCompileTest) is now running


This is mostly due to a security fix in android: https://github.com/swig/swig/commit/a1771cb8a0cbba65ffd07bee96a2cb41a9f112fd


谢谢哈。这个库不想继续维护了，只是为了用起来方便而已。如果添加没有国家化的中文文字的话，其他语种用起来会感觉很诡异，你自己添加上方便就好。


s


I appreciate the effort, but your solution slightly blurs the game icon and I think the raised selection doesn't really fit the rest of the design, so I made my own version, which simply restores the original selection background color.


No more ******! :)


Thanks! :+1:


-1 It is not duplicate it's provided to make some IDE's happy


Issue: https://github.com/Diblo/KODI-Popcorn-Time/issues/15
Issue: https://github.com/Diblo/KODI-Popcorn-Time/issues/14
Issue: https://github.com/Diblo/KODI-Popcorn-Time/issues/19


I didn't mean to revert the per_page thing -- you may want to fix this


Issue: https://github.com/Diblo/KODI-Popcorn-Time/issues/15


Quoting the standard
>§5.3.1 Unary operators, Section 3
>The result of the unary & operator is a pointer to its operand. The operand shall be an lvalue or a qualified-id.

The 1st one applied the address-of operator to a non-lvalue while the fixed one applied it to a lvalue. Some compiler give warnings others give an error since it's ill-formed.


On Mon, Aug 17, 2015 at 5:32 PM, Tim Moody <notifications@github.com> wrote:

> install-init has a couple of simple jobs:
> create /etc/xsce
> copy local_vars there
> copy config_vars there
>
Once those are in place they become authoritative, what is in vars/ is
ignored and I'm ok with that, just need to have cli users aware that their
config is now at /etc/xsce/local_vars.yml

> I purposely did not call 0-once in install-init, because it does all kinds
> of platform-specific work.
>
That should only done once, it was left in runansible's xsce.yml for
backwards compatibility at the time. Now it's time to address that, I've
altered runansible to use xsce-from-console.yml and to call install-console
only when needed on a fresh install.

> If you want to remove the creation of /etc/xsce from 0-once I don't
> object, but I don't like calling 0-once to create the directory and getting
> a whole bunch of other stuff.
>
The only time anything really copied is on the first run, on the rest of
the runs it's noise, lets cut the noise down by running it only once from
install-console.

> 0-runonce is currently a misnomer as it runs everytime, though it probably
> only does a given task once, as do all ansible roles.
>
Yea with runansible I know, I dislike the noise, that is why I switched the
layout a bit.

> The reason install-init has to be completely separate is that the
> variables loaded in a task, even at the top level are not global, so you
> may not get the same results from running 0-once there as you would from
> running it in the role.
>
It's still standalone, just run conditionally from install-console first
when really needed.

The logic for runansible is if ./xsce-prepped is absent run install-console
first.
The logic for install-console is if /etc/xsce/config_vars.yml is absent
call install-init first where we can really do 0-once only once.


test


I'm sorry, but why would you add a docker installation to the system root?


안녕


@CurlyMoo
Why did you do this? The banner is already in the file /etc/motd - now I get 2 banners if I login via ssh


@levicole here are my additional blocmetrics snippets


@levicole and here


t


@codewithpassion do you recall why the reset was added?


Yes, you're absolutely right.
I've made the fix: https://github.com/ripple/ripple-lib/pull/511


for #1 , fail, I think I should move the part for the view in next commit.


Thanks!


:yellow_heart: `jshint` is not required for the application. However, if you choose to keep it, it would be more appropriate to place it in the `devDependencies` category.


:green_heart: The `User` model is more appropriate for the tests.


LGTM


:yellow_heart: This would no longer be necessary since it is not used. It is recommended that redundant code be deleted.


I think this bug is from umbraco, because if I use earlier umbraco version, it's has no issue with the authorization, but I think this need to implemented till Umbraco has been fixed for this issue


I don't think I agree ... that would be testing implementation, not behaviour. The behaviour I actually want is reflected in the test: if I make every method this class adds protected, can I still create a new instance of my object?

It just so happens that the current implementation uses `immutable!`, but if we refactor to be another name (e.g. `make_immutable!`) I shouldn't have to change the test.


:green_heart: Good work clearing the database after the tests.


Oops - I left "commit directly to stable8.1 branch" selected and it committed it.
It all looks good to me :)


98812296-SkyTemperatureShiftsSchedules-4866 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


Commnet was wrong. Fix it please.


EvansHVACObjects (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


TeamCity ATK :: Source Code Compile and Test [Build 8608](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545487&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1196, ignored: 6 Build time: 00:24:22


If we really don't necessary this line, then you have to remove the line before you send pull request.
Could you explain why did you remain this comment at L304.


Interesting about the thread issue. Can you provide more details?

The executor is customizable, so you can use a same-thread executor if desired. I tried to be intelligent about fork-join threads, but more real-world experience would help me tune that heuristic. The intent was to defer the maintenance work to a background thread instead of adding latency to a user-facing thread.


Haha, I see you've found the experiment! If you create a database called `todos` and then `./run-app.sh`, you'll see for yourself.


5110_stdbaropress_sealevel (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1778 of 1797 tests passed)

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


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


need to add entry in tests.full.xml and tests.android.xml


Yay!

Dumbass... :+1:


need also add a usecase use async intialization method


This method is in there twice. REALLY be careful with what you're doing. If you have 2 methods with the same name, you will have the same problem as you had before. Since, this method is already declared as private "as it should be so you don't let anyone else use this method", you should REALLY remove this one.


Hi m，

How about your tests?


TeamCity ATK :: source code Package [Build 8231](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545490&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:23:06


TeamCity ATK :: QA :: Integration Per Branch [Build 1877](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545491&buildTypeId=Atk_IntegrationPerBranch) is now running


This should work, as long as you can use `id: user.id` in the 3rd argument.


Nope, it will still only select a single forme for each of the six dex numbers it generated. It's just no longer guaranteed that randomly selecting say, 6, and choosing Mega Charizard X will give it a Charizardite X before reverting it to Charizard.

I'm not sure why that condition was part of the code anyway, since it never worked properly before my fix the other day (most formes don't have their own learnsets and were being rejected before it even got to this part).


This one is perfect, keep this here.


close quote
echo the whole thing without _e


Maybe, your intention was that VMPIDR is determined by CPU ID that VM is owned.


Please change from nr to number or ID.


這行不應該放進 PR 裡面


@adsouza We got that part done today, we weren't sure of how exactly we should limit it.


Pls, initialize the connection queue in init method and remove the check for nil and then alloc.

If block is nil, this code would crash. Pls check for nil.


@adsouza we changed this because we are now having users log in at the end of the checkout process as you suggested.


:yellow_heart: The condition in which the user details are checked for undefined values should come before the condition in which the error code is sought. For example:

```javascript
if (user.username === undefined || user.email === undefined || user.password === undefined) {
  // code blocks
}

if (err) {
  // code statements
}
```


Thank you! @adsouza


Why does bootloader configure SMP? Linux kernel can do the same action, am I worng?


Ditto.


This isn't really necessary, but it makes greping for `this.whatever` easier.


We don't allow any comment inside code.


AdequateDispatch? ;)


About damn time👍


Remove unnecessary space after if ( and the end of ).
if ((result == HVMM_STATUS_SUCCESS) && (virq < 16)) {


I feel like giving you a hard time :)


:yellow_heart: The spec should read `it ('should not create new user if user already exists')`.


Ah ah ah... no newline! This is fun!


:green_heart: Good work covering all paths in the tests. :+1:


You can remove unused variables.


Done: https://github.com/meltwater/puppet-monit/commit/5fbb6b710ceac705a26b3049271611fe30ddc7a5


OK


Cos for me, it didn't work on Python 2.6 before this change, and it did afterward :-) Are you seeing otherwise?


:yellow_heart: The `username` and `password` is not sufficient for a login. The user's client would also be required to supply a token in the header. I'll be willing to pair with you to work this out.


Each condition have to enfold with brace () because sometime we can be confused when we read this line.


Nope.


EvansHVACObjects (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Nope :P


:green_heart: Good test!


If the number of chracters in a line is not over a hundred, then you don't need to split a line to two lines.


Ditto.


What is the meaning of preg_s?


TeamCity ATK :: source code Package [Build 8232](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545488&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:22:14


Ditto. Please make name of variable easily.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1280 of 1280 tests passed)

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


Do not split a line.


I couldn't pass the npm test because test cases use CRLF as the end of line.


rax retry tests


Thanks @therealklanni, I'll get these in soon. Just haven't had a second yet


noclip = fly?


The boolean View field named "highlight" could be renamed to "highlighted".


Could you please reindentation and arrangement?


Modifier inconsistent :P


Remove empty line.


:umbrella:


ライブラリのソースコードをいじってはいけません。
この修正だと.form-groupをもちいる全画面に影響がでてしまうのでは無いですか？
この修正がベストな理由はなんですか？

いい加減な仕事をしていないですか？


Will #6015 still be needed with that diff?


Did you mean to say "Problem" in the line above?


![Image Alt](http://www.picgifs.com/disney-gifs/disney-gifs/pinocchio/disney-graphics-pinocchio-329861.gif)


I think that

    modbus_close(ctx);
    modbus_free(ctx);
    EXIT_ERROR++;
    return -1;

won't work: it closes ctx global session...


Change name to _indexOfPageToSelect


It's not suppressing anything, and there's nothing that needs suppressing, and.... don't know what else to say :-)


```jl
julia> function f1(n)
       for i in 1:n
       m == m
       end
       nothing
       end;

julia> function f2(n)
       for i in 1:n
       m === m
       end
       nothing
       end;

julia> f1(1);

julia> f2(1);

julia> @time f1(1_000_000)
  0.055878 seconds (4 allocations: 160 bytes)

julia> @time f2(1_000_000)
  0.003201 seconds (4 allocations: 160 bytes)
```


*cough*


Do we want to consider updating [or even excluding] the version numbers of other browsers?

Newest versions:
* Chrome - ```44.0.2403.130```
* Firefox - ```40.0.2```
* Safari ```8.0.7```


didn't realize fd-input and fd-textarea were new in this PR. i see what this is doing now. it's an unfortunate amount of duplicated code, but the style sharing between HQ and vellum is itself kind of unfortunate...hmm. what if instead of changing vellum, you modified the HQ styles to add `div[contenteditable=true]` to the form styles (or a more specific class, but it might be reasonable to expect all contenteditable things to look like form inputs)?


Oh, right, I see. Sorry, I didn't notice the `list` changed to `file`, and from there I guessed wrong about the rest.

Ok, this looks good. But, please make this conditional as well, as in the previous comment, better to only use a file when the list is actually very long. Also as above we can use `configuration.get_temp_files().get(suffix='.response').name` to get a temp file to write into.


This commit causes issues - I am encountering at least one instance where a simple blind sql injection using  parameters similar to --data='x=1&y=2&z=3' no longer works


I think it does not matter here, but I would rather keep both of these constructors around. The reason is that you remove the copy constructor, whereas the constructor you leave is a conversion-constructor. It will never be called for `_Constness==Constness`, in which case the compiler will actually call an automatically generated copy-constructor that simply copies each member. This automatically constructed copy constructor does the right thing here, but it is certainly a confusing corner case most people will miss and it will yield surprising results if we ever make the constructors do something that goes beyond just copying members.

If you want to try it out, use this program:
```
template <typename T>
class X
{
public:
  X();

  template <typename U> X (const X<U> &x)
    {
      x->foo;   // purposeful error
    }
};

X<int> x;      // default constructed object
X<int> y(x);   // no error -> uses automatically generated copy constructor
X<double> z(x);// error -> uses conversion constructor above
```


Expanded tabs to spaces


Oooo! This looks interesting. Got any more details for us, Chris?


lol


So I would leave the copy constructor and simply write the other one as
```
Iterator (const Iterator<BlockVectorType,!Constness> &c);
```


shouldn't this be:
```
if @input_data && @expected_output
```


perhaps WM_ACTIVATE could occur while everything is ok ?
Perhaps you should set to devicenoreset only if it was in devicelost state ?


Looks like this is related to #809 as well.


I think `update_every_n_cycles` is very descriptive, but not as technical as `update_freq_divider`.
I would prefer `update_clock_divider` or `*_prescaler` since these terms are IMO more common in embedded programming.


Added comments, changed some names


cooperative, not coorperative.


Huh, neat. I guess `===` is checking the pointer and nothing else?


@Myoldmopar Corrected the file name in CMake file and pushed the branch again.


I don't see a reason to separate these weights in this particular feature. You still assign all of them the same value every time you modify one. I agree it might be a good idea at some point if we use these differently, but it makes no sense in this commit.


1.2 will be the next release, right ?


last minute


DONE


Would it not be better to check to see if appEndpoint contains an ? Then old apis/custom urls would work


I'll include this commit, thanks.


Thanks for catching this. It will actually 1.1. However, this still needs fixed :) Thanks again


Wasn't this... que must just be long today


Godd point, fixed.


The `Z` is interpreted as the end of the uppercase character range, the hyphen after it is a literal hyphen in this regex pattern. This is okay here.


agreed, I'll remove or split it out.


@ropdor thx for the triage. On it.


Should we change the spelling of Ace here as well? Id est, "Ace" instead of "ACE".


what do you think about actually use the "y" key to determine which path to take here?


This fixes #243, but now this issue has risen again with players approaching 2500


because technically a valid "r" response could have an "e" entry too..


This is to allow you pass, only an `expected_output` for tcp service that doesn't require an input to test hearbeat


why should these errors not be reported?


At the heart of this is a discussion that will never be solved. Small and Obvious vs Magical and Hidden.

The small and obvious camp wants tiny classes that only do one thing, but do it really well. In that case, yes, you lose 38 characters, but you gain obviousness. And since the class is only going to be < 30 lines long anyways (since we're in the tiny class camp), it doesn't matter if it's a couple lines above, it makes this math line read more easily.

The Magical and Hidden camp isn't malicious though, they honestly believe that difficult math should be hidden away so you can get to the act of writing the web app. They use gems and package code up in the neatest packages, so that the people that need to look at the math, already know what 1.05 is doing there; it's obvious because _if you're in this gem, you already know the math._

Unfortunately for you, you have no idea which camp you're going to get hired in to. You can ask some very pointy questions, and we expect senior candidates to do exactly that, but it would be difficult for you to understand the ramifications of which side a company is (if it can even choose internally, which is rare).

So, long answer longer, my current job is small and obvious, and I'm prepping you to do our code challenge in the off chance that you decide that you hate chicago and want to move out to Portland and hang with Cam and I. For our code challenge, we would likely prefer to see it written out and assume that we don't know any of the math, and we don't mind a little duplication because your classes are so small as it is, that it doesn't matter.


Should be `keylet::signers(account_)`. We might consider removing the `uint256` overload in `keylet::signers`.


Damn!  This was a git commit for a different git project. Why did it propagate up?


`this.sfx` is `undefined`but `if (opts.sfx)` does work


do you know where this "r" entry comes from? is it because we report errors and base it on whatever half-built response message we have at the time of the error? (in which case that seems wrong, when an error is reported, I would expect to build the error response message from scratch, and not add an "r" field)


Just b/c there was no longer a dependency on `m`, but I guess you could view it as a precondition check and so it belongs at the beginning?  Either way.


In your setValue() method, i see you check for both datatypeClassname and datatypeConfig. But i do not see you set them both here.


Yeah, pretty much


but you always puts the input


This will cause "conditional expression is constant" warnings at some point


I didn't know that. Is there any document about request/response message format like http://www.libtorrent.org/dht_store.html?


This is called an inclusion guard.


You should have a look at `describeOwnerDir` in View.h I think it makes this easier


Shouldn't this be `return None`?


I think this is it: http://www.bittorrent.org/beps/bep_0005.html#errors


Funnily enough, if you ``const m = Model()``, the difference goes away, since julia is able to replace ``==`` with ``===``:
```jl
julia> @code_typed f1(1)
1-element Array{Any,1}:
 :($(Expr(:lambda, Any[:n], Any[Any[Any[:n,Int64,0],Any[symbol("#s1"),Int64,2],Any[:i,Int64,18]],Any[],Any[UnitRange{Int64},Tuple{Int64,Int64},Int64,Int64],Any[]], :(begin  # none, line 2:
        GenSym(0) = $(Expr(:new, UnitRange{Int64}, 1, :(((top(getfield))(Base.Intrinsics,:select_value)::I)((Base.sle_int)(1,n::Int64)::Bool,n::Int64,(Base.box)(Int64,(Base.sub_int)(1,1))::Int64)::Int64)))
        #s1 = (top(getfield))(GenSym(0),:start)::Int64
        unless (Base.box)(Base.Bool,(Base.not_int)(#s1::Int64 === (Base.box)(Base.Int,(Base.add_int)((top(getfield))(GenSym(0),:stop)::Int64,1))::Int64::Bool))::Bool goto 1
        2:
        GenSym(2) = #s1::Int64
        GenSym(3) = (Base.box)(Base.Int,(Base.add_int)(#s1::Int64,1))::Int64
        i = GenSym(2)
        #s1 = GenSym(3) # line 3:
        Main.m === Main.m::Bool
        3:
        unless (Base.box)(Base.Bool,(Base.not_int)((Base.box)(Base.Bool,(Base.not_int)(#s1::Int64 === (Base.box)(Base.Int,(Base.add_int)((top(getfield))(GenSym(0),:stop)::Int64,1))::Int64::Bool))::Bool))::Bool goto 2
        1:
        0:  # line 5:
        return Main.nothing
    end::Void))))
```


That `finishJoining` function feels like a workaround, are you sure you can't place that logic in the [async.series](https://www.npmjs.com/package/async#series-tasks-callback) callback?


The function argument should be `Keylet const& k`. And the first line can be


But also, lol:
```jl
julia> @code_native f1(1)
	.section	__TEXT,__text,regular,pure_instructions
Filename: none
Source line: 5
	pushq	%rbp
	movq	%rsp, %rbp
Source line: 5
	popq	%rbp
	ret
```


Cool. I did not know about that. I was using an online previewer but there were still differences with github when it came to formatting and white space inside tables.


(Alright, back to work now)


I think these look good. We should keep track of the recommendation from https://wiki.duraspace.org/display/hydra/Rights+Metadata+Subgroup


DONE


Can you also add a test for delete?


https://docs.djangoproject.com/en/1.8/ref/models/querysets/#when-querysets-are-evaluated

> Testing a QuerySet in a boolean context, such as using `bool()`, or, and or an `if` statement, will cause the query to be executed. If there is at least one result, the QuerySet is True, otherwise False. … Note: If you only want to determine if at least one result exists (and don’t need the actual objects), it’s more efficient to use `exists()`.


Are these the only properties you expect for full representation?


That is true, good note. It was a leftover from the separate yaw while banking feature that you might know of.

In any case it is irrelevant for this feature, but let's see if the need for separate term tuning arises in testing. I will remove them later if not, and add it to a separate branch where I will put the yaw while banking feature.


Indeed it should


environment variable!


добавил первый commit


good iea


what does this line?
Maybe this row is responsible for ugly terminal emulation?

http://superuser.com/questions/715125/openssh-freebsd-screen-overwrite-when-closing-application

TERM var must be transferred from terminal emulator @ login.


I had to reverse this change in bbed7340eae5dcdc1a271f6bf7fda7273b83b0c6 since this altered what was displayed as an excerpt when you manually specify an excerpt.


What is in /share/nano? FHS[1] doesn't say.

[1]: http://www.pathname.com/fhs/pub/fhs-2.3.html


```
        // this isn't necessarily because the other end is doing
        // something wrong. This can also happen when we restart
        // the node, and we prematurely abort all outstanding
        // requests. Also, this opens up a potential magnification
        // attack.
//      entry e;
//      incoming_error(e, "invalid transaction id");
//      m_sock->send_packet(e, m.addr, 0);
```

Any different? Can we handle a error for a response? Do we cache a request and waiting a error response of our response?
In my mind, DHT is more like a simple RPC, request->response, if response is incorrect, re-request or ignore it.


should add an error check like

```js
if (err) {
  done(err)
}
```


I'm not sure right now which one is proper. If shipment can be shipped, we should apply the transition. No matter what state it is in,


@azturner Thanks!


We need to make sure that we don't compute the keylet more often than necessary since it involves a somewhat costly hash operation.


Move the declaration of `unquoted_meta` meta here?


done. thanks niphlod!


Is the idea that the actual rich text widget will be contenteditable, so `contenteditable=false` grabs all of the bubbles? but won't that also grab all of the icons and line break tags?

A comment like `// Each bubble has a tooltip displaying its path and a deletion button` (or whatever the truth is) would make these first three functions easier to quickly grasp.


And /share/org? Is it emacs specific?


`auto const signers = `...


I thought we rebased and removed these?


if you're ever going to need to select these, giving them a `bubble` class would probably be useful


hash router logging is like `std::vector` logging


I do not think we should just swallow this exception and simply make the above call. Let us instead notify the caller about the failure.


I had a look at the original paper; there is some argument to changing the goodness-of-fit to be discrete since that one has some defined meaning. However, the rest of the plot colours are rather arbitrary, and I don't see any reason why they must be the same by _default_.


@jgarboza813 DWMR (done with my review). Good stuff here. Press on!


If it's unneeded, simply remove.


We should simply turn off this warning instead of individually ignoring every instance of it in the codebase.


Instead of calling setPropertyValue in the above else, we should tell the caller about the failure to get the custom data type matching their parameters.


I do. I have pointed it out in my second comment. In node.cpp
In node.cpp,

```// build response
void node_impl::incoming_request(msg const& m, entry& e)

    entry& reply = e["r"];
    m_rpc.add_our_id(reply);

    // mirror back the other node's external port
    reply["p"] = m.addr.port();
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
```


мне нужна эта штука, возможно её стоит вынести в утил, но реально она мне нужна.


This slipped past me when the code was introduced, but is it possible to have a name like `something/another/mynode`? If so, we have a strrstr() somewhere ;-)


I will change that.


This expression is valid. Since this is a string literal instead of a raw string literal the double escaped closing bracket (`\\]`) is actually treated as `\]` in the regular expression character group.


Yeah, i was just thinking wrong about this.

I think it would be marginally clearer if there weren't two verbs in teuthology_user_created|changed.  How about calling the variable "teuthology_user_existence" and we'll call this good?  (I have to request *some* change after all the fuss I've made over nothing.  ;))

no seriously I think that would be clearer.


97204612_MultiSpeedFan (Nigusse) - x86_64-MacOS-10.9-clang: OK (1799 of 1799 tests passed)

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


Is this supposed to be part of the keystone-ldap feature?


Shouldn't this return globalProperty.getPropertyValue() unless datatypeClassname and datatypeConfig are set?


DONE


Created a redmine service to authenticate against this redmine plugin:
https://github.com/suer/redmine_oauth_provider


That's from the client.
Twitter haven't implemented Oath2 protocol yet, so there's a different logic needed.

In the client, this is how the controller function looks like for twitter:

twitter = ->
    $auth.authenticate('twitter').then (response) ->
      Resource.POST "auth/twitter_step_2", { twitter_token: response.data.token }

Let me know if you need more info :)


`var`? I was under the impression that now we should use either `const` or `let`.


Cool, busy procs are gone!


rax retry tests


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Awesome!! Thanks a lot :)


Why is this not a boolean?


I don't think we will again any time soon, either, but to date we have been using MINOR version to do so. this whole thing kind of written after-the-fact for now (for one particular reader). should we just strongly word that 3.2 for mq and 1.2 for auth are just unique cases?


> What is in /share/nano? FHS[1] doesn't say.

Highlighting configs for `nano(1)` editor, added for symmetry with vim
and emacs.

> And /share/org? Is it emacs specific?

Yep. Note that it was there before this patch.


the best way to go about making your own agents is that which is presented in the pool request. However, if you want those agents to stay private to you, you don't want to submit a merge request


why does this hide the popovers instead of removing them altogether?


instead of `check_output`


Set up for Selenium Webdriver demo session.


Yeah, sorry, forgot about that ;)


thanks!  was just about to ask for a new release


I will add that


Maybe ```$MountPoint ($Partition)```?


comment on what this is for


Wow. Such voodoo. Also, `be_truthy` is evil and tells me that you aren't really sure what might come out of this test. :)

(Also, I recognize that you're removing this code, but I'm commenting on it anyways and there's nothing you can do about it!)


@Notsosof1 The problem is swift specific. In order to use strings with the switch case structure you must force the variable to unwrap (as it's called). ie. you must add the bang ! like this:

`switch segue.identifier! {`


git pull origin worked better.

Thanks!

On Tue, Aug 18, 2015 at 3:06 PM, Rich Signell <notifications@github.com>
wrote:

> Maybe you need to do
>
> git pull upstream master
>
> check what your remotes are by doing
>
> git remote -v
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/USGS-CMG/usgs-cmg-portal/commit/0b2a4c969f70703b9414b293e22068141f836329#commitcomment-12768495>
> .
>

--
Ellyn Montgomery, Oceanographer and Data Manager
U.S. Geological Survey
Woods Hole Coastal and Marine Science Center
384 Woods Hole Rd., Woods Hole, MA, 02543-1598
(508) 457-2356


Are those `global` statements necessary?


This can cause compilation error so hold on...


Yeah, that's fine.


На сервере должен быть апачевский http клиент.
https://hc.apache.org/httpcomponents-client-ga/httpclient/apidocs/


Using :loose will allow scraping for multiple categories, and fall back to SimpleRSS format only if necessary
https://github.com/aasmith/feed-normalizer/blob/master/lib/feed-normalizer.rb#L105


Wouldn't using `input.on` instead of `input.find` above allow getting rid of this handler? Not a major improvement, but a little bit less code.


You can throw a `pending` in here also.

Using `it` without the block is so that you can sort of draft the tests that you want to write, but fill them in later.


Hi @mgruner,

If fact was my first option but it didn't look so nice along with other checks e.g.
OTRS Disk Partition (/dev/disk1)

But I will change it anyway :-)

((enjoy))
cr


comment


So, 6 independent requests? Is there no better way to do this?


Create RoadMap for the project (wiki) #21


Really no way to make it work?


$ cd C: X


Thank you for this; this is exactly what I needed to get logstash working with bosun. The bosun.org documentation needs to be regenerated: http://bosun.org/configuration.html


Having a `find_by_name` in here feels weird. Also, what is the type of this hash? `{ Object => Float }` is a little weird. Or is it implicitly turning a Malt into a String by calling `to_s` on it to make the key name?


Uh? What is that for?


No, I'm thinking that this library could be "eddystone-advertising" or something, and could (eventually) check whether chrome.bluetoothLowEnergy, mozBluetooth, or navigator.bluetooth.advertise exists, and implement its API appropriately for each.


Yeah - I had a feeling a lot of my ```after```s are unnecessary.  Related question - if I write some value to the test DB during a test, e.g. ```@recipe.save!``` do I need to explicitly ```destroy``` that in my test file, or is that something the database cleaner will take care of generally?


Missing a space after the comma


`// A single-line input supporting stylized "bubbles" in place of xpath expressions`


We can't, because `unquoted_meta` would go out of scope. See the last line of this block.


`// A multi-line version of richInput above`


On Tue, Aug 18, 2015 at 12:03:46PM -0700, Stéphane Graber wrote:
> no. The delay is meant to be how long to wait for things to be started in the container, so it does need to be after the start.

Sorry, I don't understand this. What's the purpose of having a delay
after Start()?

> As for the Atoi, it's going to fail when the key isn't set, leading to a 0s delay which is the expected default.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stgraber/lxd/commit/066ed37ac3fc83cad4454b370b21083472dccccc#commitcomment-12768439


@coreytegeler it won't go through basic checks when I submit it to addons.mozilla.org. I think there's something up with migrating the install.rdf file.


Awesome work :clap::clap:


I can see your point but I believe saying "decimal uses 0-9, hex uses 0-9A-F" is as harmful as being taught "there are only 10 digits" in school without a better explanation of other numeral systems. I could remove "As long as you only have X digits, your system can be called Y, but " and leave it only as "It is conventional to use from 0 to 9/0 to 9 and A to F", etc.


I will add datatypeConfig as null here.


You have created the directory here, which looks correct. So, theoretically, when you install nova-common, which will pull rjil-cicd package, you will have the /etc/nova/nova.conf file available.

So, since I don't see the entries for nova-control, my gut feel is that you need to ad a section like python-nova, also for nova-control.


We should remove `[nodoc]` from [chrome.bluetoothLowEnergy.registerAdvertisement and friends](https://code.google.com/p/chromium/codesearch/#chromium/src/extensions/common/api/bluetooth_low_energy.idl&l=308-330) so that there is documentation.


asdasd


it should be.


adsadasd


Yup, didn't mean to leave this one in.  Thanks for fixing!


Any suggested comment to fold into  #15310?


Is this flag used?


Doh, thanks for the catch + fix!


I think "To copy over source code from a project and start independent work on it" suffices for GitHub's use or forking. I'm not experienced in using GitHub, however, so please feel free to correct me.


Same thing as we support in LXC, it's so that you can have a container which you know will take about 30s to start, you can then set the delay to say 40s and all containers starting after it should be able to use the resources exposed by the container.


Want to submit a PR about this?


at least fix that readme :)
Cataclysm 5.4.8 Emulator - V1
and some of those rules looks pretty silly on public repo


The relevant tests moved here:
wamibrew/spec/models/recipe_unit/name_spec.rb
Haha - yeah.  I'm not sure what comes out each time, and that's part of the point - I want it to be one of a small number of outcomes, and I wasn't sure how to say ```expect(method_output).to eq('string_1' || 'string_2' || 'string_3')```


Just like a `case` statement is generally a smell that the code could be restructured to be more readable/extensible, having a bunch of `elsif` statements can indicate that there might be a better way of doing things.

When I see that code, the main thing that stands out is that your only trying to do two things:

* Render based on the property
* Render based on the occupant

So you could probably create two new methods (something like `render_property` and `render_occupant`) that handle the complexity of what it means to render the property of a cell and the occupant.

But _don't_ try to get it done before the Meetup :-) It'll be better to have what you've got working, than to have better organized code that may not work when you try to demo it in front of everyone :-)

Also, I'll be leaving in a few minutes, so I'll see you soon.


Minor nitpick, but please add a period after the "Hex grid file" here.


Hey, just noticed this. Looks like you're missing a comma on line 8. A malformed .bithoundrc file will cause our analysis to ignore it and user our defaults again.


The quick brown fox... (just testing Brendan)


It's so people can set some minimal kind of container dependencies by just ordering them through the use of priorities and then setting delays when a container is known to take a while to be ready for the next container to contact it.


I think `context` is supposed to be something like:

A Recipe # Noun (Object under test, `subject`/`described_class`)
With a bittering hop # With a Noun (Context describing how we set up above mentioned Object)
Does a thing # Verbs (Method under test)


Thanks. Now I see. I was thinking that you somehow cusomized satellizer for this 2-step flow.
Maybe it's possible to handle both steps in a single Rails action, to avoid having a specialized JS controller.


https://github.com/LoomSDK/LoomSDK/blob/ea62b6a4da8da3501b035601224978424d306b8f/loom/graphics/gfxQuadRenderer.cpp#L127 Here


Are these cases switched?


PR coming your way


`sol-flow-resolver-conffile.c` use the same approach and unless things changed, we also have code to validate this in the JSON schema (for the generated types).


I stand corrected sir!

On Tue, Aug 18, 2015 at 1:44 PM, Jure Gregorin <notifications@github.com>
wrote:

>
> https://github.com/LoomSDK/LoomSDK/blob/ea62b6a4da8da3501b035601224978424d306b8f/loom/graphics/gfxQuadRenderer.cpp#L127
> Here
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LoomSDK/LoomSDK/commit/ea62b6a4da8da3501b035601224978424d306b8f#commitcomment-12770351>
> .
>


空行入れましょう！


yes.

```
>>> FOO="bar"
>>> print FOO
bar
>>> def foo():
...   FOO=4
...   print FOO
...
>>> foo()
4
>>> print FOO
bar
>>> def bar():
...   global FOO
...   FOO="baz"
...
>>> bar()
>>> print FOO
baz
>>>```


Hi barbudreadmon,

My change description should have been:

"Added two-player support.  Fixed bug in ppu.cpp which caused the both the first player controller and the second player controller to control the first player."

This change DOES NOT add support for a third or fourth player via a multitap or USB hub.

My apologies.


ah, I see. I'm inclined to fix this simply by correctly parsing the message type from the "y" field. There are potential benefits to include this here. It confirms what our node ID is and it echoes back the other end's port (which really should be the IP and port these days).

Would that be sufficient you think?


Not that i could understand.  Warren's comment about parsing /proc/version is better than this i think.


Should this version line be wrapped inside an else clause?   Seems if you successfully call prepareForSegue:sender:userInfo, that prepareForSegue:sender: shouldn't be called.


Capital argument "-N" names the job, lowercase "-n" decides what queue to put the job in, in a torque RM system.
AUGUSTUS_CONFIG_PATH needed to be exported to be used by "autoAugTrain.pl"
"autoAugTrain.pl" would die if the configuration file for the species already existed, removed the new_species script to stop this.


I do not see a way to batch requests: http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-metadata.html#instancedata-data-retrieval


Why can't we use Contact here?


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (753 of 753 tests passed)

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


Nice simple solution. I was pondering how I was going to get rid of the null terminated strings when moving back and forth between screen saver and login window.


We can use Contact here also.


@ropdor it should be ok now with the latest commit (383316fcb35328f5867565a2dd9024f8b7cde3bf). Used better approach for dealing with original and your issue


Hello Beloved in Christ.

I apologize for this intrusion , my name is Metthey Françoise
originally from France . I can contact you in this way because I want
to do something very important . This will seem a little suspect true
that you do not know me and I also do not know you . In fact, i am
suffering from cancer of leukemia which the doctors has confirmed to
me that i will not last long due to the last diagnoses result which i
received right here at the Hospital in  Ivory Coast where i am been
treated right now., I have this disease for more than 4 years. I am a
widow and I have not had children . I am hospitalized. I plan to
donate all my possessions . I almost sold my business with an oil
company in Benin , part of this money will be paid to various
associations, centers for orphans and homeless. I do not know what
area of work you carry but I would help you help others. I currently
in a briefcase sum of $ 1,750,000.00 ( One Million Seven Hundred Fifty
Thousand Dollars ) in a bank in Benin to babysitting session. This is
money that I kept for a construction project. I will be rigged to give
you money that can help you in your business and your projects ,
please accept this as it is a gift I made you and without asking for
anything in return. I suffer a lot and I am very afraid of dying
without being a destination of my property , I was almost unable to
sleep at night as during the day because I do not want to die without
having donated all the money otherwise I think it would be a mess. For
more information write me on my personal address :
franoisemetthey@yahoo.fr

May Peace and mercy of God be with you.

Metthey Françoise
( franoisemetthey@yahoo.fr )


Good Job


Proof of life for FUSE-4593, also guarantees creation of host dimensions these specially formatted events are going to decorate if no other plugins exist.


@DEZERALDOlivier I :heart: this part!! well done!


This fixes #4


I'm a bit confused by this sentence. I prefer the first paragraph of the class comment.


Is this semicolon here for a reason?


Nice :) Less confusion now, could finally pull up latitudes and longitudes for cities.


Ну хоть что-то стало лучше. API надо выносить (не прямо сейчас) в отдельные интерфейсы, сейчас это не API, а винегрет из кучи методов класса. После этого можно подумать над справкой. Но я скорее за документацию для разработчика в виде http://ru.cppreference.com/w/cpp/container/vector/erase. И формальное описание есть, и пример. Он даже запускаемый, вообще класс. JavaDoc для приведенных двух методов конфигуратора выглядит как табличка `"Дерево"` на дереве. Возникает вопрос - зачем подписали, и так видно, что это дерево. Лучше бы инструкцию приложили, как из него дом построить. Но если её написать не отдельным документом, а в виде JavaDoc, то от исходников одни комменты останутся. Выглядит так, что JavaDoc не совсем подходящий формат для документирования, потому
- или таблички Дерево, но хотя бы более-менее читаемый java-код программы
- или отличная документация, с примерами, но нифига не видно кода

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

Предлагаю остановиться на первом пункте.


Cool. Sorry for the noise.


Ok.


I wouldn't put back-quotes around splat, as it's not a word that appears in code.


From  travis-ci/travis-ci#4125


Recommend `yyyymmdd` format so that the images are in chronological order.


@cpennington  Fixed one bug - and flipped the default `include_empty` since it raises a ValueError for unsuspecting users (like me). Review?


Ты же говорил, что проверки нужны


:clap:


np! :)


You can use `Range#join` to create the combined range.


I removed some  resource metadata settings that are already defined in adhocracy_core.resources.proposal.proposal_meta.

What exactly do you mean?


Thank


А почему было недостаточно поменять `+` на `*` ?


why is the   necessary, isn't this the usual empty-element icon that applies the image with ::before styles? (space between the icon and the id would be nice, though)


I've tried to do it similar to https://github.com/openmrs/openmrs-core/blob/master/web/src/main/java/org/openmrs/web/controller/maintenance/SettingsController.java#L80


Hey Massi, this context object is a good idea. Can we have it be a simple object in the pluginmanager.js file, instead of creating a new class?


Actually, it would more accurately be in {INITIALIZED, ACTIVE}


comment the key codes, or make them into named constants


:+1: phew ;-)


Thanks, I've corrected this.


Hello Beloved in Christ.

I apologize for this intrusion , my name is Metthey Françoise
originally from France . I can contact you in this way because I want
to do something very important . This will seem a little suspect true
that you do not know me and I also do not know you . In fact, i am
suffering from cancer of leukemia which the doctors has confirmed to
me that i will not last long due to the last diagnoses result which i
received right here at the Hospital in  Ivory Coast where i am been
treated right now., I have this disease for more than 4 years. I am a
widow and I have not had children . I am hospitalized. I plan to
donate all my possessions . I almost sold my business with an oil
company in Benin , part of this money will be paid to various
associations, centers for orphans and homeless. I do not know what
area of work you carry but I would help you help others. I currently
in a briefcase sum of $ 1,750,000.00 ( One Million Seven Hundred Fifty
Thousand Dollars ) in a bank in Benin to babysitting session. This is
money that I kept for a construction project. I will be rigged to give
you money that can help you in your business and your projects ,
please accept this as it is a gift I made you and without asking for
anything in return. I suffer a lot and I am very afraid of dying
without being a destination of my property , I was almost unable to
sleep at night as during the day because I do not want to die without
having donated all the money otherwise I think it would be a mess. For
more information write me on my personal address :
franoisemetthey@yahoo.fr

May Peace and mercy of God be with you.

Metthey Françoise
( franoisemetthey@yahoo.fr )


This commit is on the deleted code, so we have no problem now.


Hmm, for beginners, is it really necessary to introduce the idea that a decimal number system might use anything other than "0-9"?  I can't think of another decimal numbering system, and the glossary is aimed at very early learners.  The less noise the better.  Some simplification will be needed.