@kbrock
We don't need this change either, we just need to ensure that the user that last parameter is always passed in


yay :tada: I <3 it


@kbrock
The user = nil will not work anymore since user is mandatory now. If the user is nil we should raise an exception


it was still giving issues sometimes was about to ask you the same if  vasishath  missed something


There are a few places it is used by itself.  The alternative would be to pass a flag and that would certainly be doable.
On Oct 6, 2015 12:34 PM, Michel Pollet <notifications@github.com> wrote:Do we still use avr_timer_cancel_all_cycle_timers by itself? If not, what about renaming the function to something shorter? I know I'm to blame originally, but it's getting a bit ridiculous now :-)

—Reply to this email directly or view it on GitHub.


This also could be win10 specific - looks like old windows behavior was not to change desknames when ss activates/deactivates.


Really? Forked for one change? Can you not just run
```
syn keyword remindExpiry	UNTIL FROM SCANFROM SCAN WARN SCHED THROUGH
```

in your `vimrc`


@skorlir Sure, just trying to give you the feedback you asked for on gitter. I think you'll end up implementing this one way or another if the library takes off.


I think you mean `UNCOMPRESSED`? Ethereum uses uncompressed keys.


Should be a command line parameter that is polices from the browser process.


@maltoe actually what i was showing in this spec is that the same `lock_manager` won't automatically extend a lock


@bloo that's cool. btw we can factor out reusable api client and publish it as a separate npm package, so that our front-ends and cli clients could depend on it


It is advisable to _code against the interface_ which means that we can have `HashMap<Integer, List<Pastry>>` instead of `HashMap<Integer, ArrayList<Pastry>>`. This is because with the interface, the one who uses this map has the flexibility to to put as a value any implementation of `List` not just the concrete `ArrayList`. In other words we are not coupling this to a particular implementation.

What you did is not wrong, this is more of a thing to keep in mind.


Enough with 1 cast?
```
--- a/print-icmp.c     2015-10-08 20:34:46
+++ b/print-icmp.c        2015-10-08 20:43:35
@@ -598,7 +598,7 @@
              * however not all implementations set the length field proper.
              */
             if (!ext_dp->icmp_length) {
-                vec[0].ptr = (const uint8_t *)(const void *)&ext_dp->icmp_ext_version_res;
+                vec[0].ptr = (const uint8_t *)&ext_dp->icmp_ext_version_res;
                 vec[0].len = plen - ICMP_EXTD_MINLEN;
                 if (in_cksum(vec, 1)) {
                     return;
@@ -618,7 +618,7 @@
             }

             hlen = plen - ICMP_EXTD_MINLEN;
-            vec[0].ptr = (const uint8_t *)(const void *)&ext_dp->icmp_ext_version_res;
+            vec[0].ptr = (const uint8_t *)&ext_dp->icmp_ext_version_res;
             vec[0].len = hlen;
             ND_PRINT((ndo, ", checksum 0x%04x (%scorrect), length %u",
                    EXTRACT_16BITS(ext_dp->icmp_ext_checksum),
```


is this temporary until we fix the bug with the project name being doubled?


What's this?


# -*- coding: utf-8 -*-
"""
Created on Wed Sep  9 16:49:35 2015

@author: pasca

Call reconall on set of subjects, create bem meshes and source space::
    python run_FS_segmentation.py

"""

# Import modules
import os
import os.path as op
import nipype.interfaces.io as nio
import nipype.pipeline.engine as pe

from nipype.interfaces.freesurfer import MRIConvert, ReconAll
from nipype.interfaces.utility import IdentityInterface, Function

from nipype.interfaces.mne import WatershedBEM

from FS_params import home, is_nii
from FS_params import sbj_dir, MRI_path, FS_WF_name, subjects_list, BEM_WF_name

from FS_utils import get_first_file, get_MRI_sbj_dir
from FS_utils import copy_surfaces, create_bem_sol, create_source_space

def create_main_workflow_FS_segmentation():
    ### check envoiroment variables
    if not os.environ.get('FREESURFER_HOME'):
        raise RuntimeError('FREESURFER_HOME environment variable not set')

    if not os.environ.get('MNE_ROOT'):
        raise RuntimeError('MNE_ROOT environment variable not set')

    ### (1) iterate over subjects to define paths with templates -> Infosource and DataGrabber
    #       Node: SubjectData - we use IdentityInterface to create our own node, to specify
    #      the list of subjects the pipeline should be executed on
    infosource = pe.Node(interface=IdentityInterface(fields=['subject_id']), name="infosource")
    infosource.iterables = ('subject_id', subjects_list)

    ### grab data
    if is_nii:
        datasource = pe.Node(interface=nio.DataGrabber(infields=['subject_id'], outfields=['struct']),
                             name='datasource')
        datasource.inputs.base_directory = MRI_path
        datasource.inputs.template = '%s/%s.nii.gz'
        datasource.inputs.template_args = dict(struct=[['subject_id', 'struct']])
        datasource.inputs.sort_filelist = True
    else:
        datasource = pe.Node(interface=nio.DataGrabber(infields=['subject_id'],outfields=['dcm_file']),
                             name = 'datasource')
        datasource.inputs.base_directory = MRI_path # dir in cui cercare i file
        # the template can be filled by other inputs
        # Here we define an input field for datagrabber called subject_id.
        # This is then used to set the template (see %s in the template).
        if home:
            datasource.inputs.template = '%s*/*.dcm'
        else:
            datasource.inputs.template = '%s*/*/*T1*/*.dcm'
        datasource.inputs.template_args = dict(dcm_file = [['subject_id']] )
        datasource.inputs.sort_filelist = True

    get_firstfile = pe.Node(interface=Function(input_names=['dcm_files'], output_names=['dcm_file'],
                             function = get_first_file), name = 'get_firstfile')

    get_MRI_sbjdir = pe.Node(interface=Function(input_names=['dcm_file'], output_names=['struct_filename'],
                             function = get_MRI_sbj_dir), name = 'get_MRI_sbjdir')

    ### MRI_convert Node
    mri_convert = pe.Node(interface=MRIConvert(), infields=['in_file'] , outfields=['out_file'],
                          name='mri_convert')

    ### ReconAll Node to generate surfaces and parcellations of structural
    # data from anatomical images of a subject.
    # the output of mriconvert is the input of recon-all

    recon_all = pe.Node(interface=ReconAll(), infields=['T1_files'], name='recon_all')
    #recon_all.inputs.subject_id = subject_list
    if not op.exists(sbj_dir):
        os.mkdir(sbj_dir)

    recon_all.inputs.subjects_dir = sbj_dir
    recon_all.inputs.directive    = 'all'

    # reconall_workflow will be a node of the main workflow
    reconall_workflow = pe.Workflow(name = FS_WF_name)
    reconall_workflow.base_dir = MRI_path

    reconall_workflow.connect(infosource, 'subject_id', datasource,  'subject_id')

    if not is_nii:
        reconall_workflow.connect(datasource, 'dcm_file',   get_firstfile,  'dcm_files')
        reconall_workflow.connect(get_firstfile,  'dcm_file', get_MRI_sbjdir,  'dcm_file')

        reconall_workflow.connect(get_MRI_sbjdir,  'struct_filename', mri_convert, 'out_file')
        reconall_workflow.connect(get_firstfile,   'dcm_file',        mri_convert, 'in_file')

        reconall_workflow.connect(mri_convert, 'out_file',  recon_all, 'T1_files')
    else:
        reconall_workflow.connect(datasource, 'struct', recon_all, 'T1_files')

    reconall_workflow.connect(infosource, 'subject_id', recon_all, 'subject_id')

    ### BEM generation
    main_workflow = pe.Workflow(name = BEM_WF_name)
    main_workflow.base_dir = sbj_dir

    #
    bem = pe.Node(interface=WatershedBEM(), infields=['subject_id', 'subjects_dir', 'atlas_mode'],
                  outfields=['brain_surface', 'inner_skull_surface', 'outer_skin_surface', 'outer_skull_surface'],
                  name='bem')
    bem.inputs.subjects_dir = sbj_dir
    bem.inputs.atlas_mode = True

    main_workflow.connect(reconall_workflow, 'recon_all.subject_id', bem, 'subject_id')

    copy_bem_surf = pe.Node(interface=Function(input_names=['sbj_dir', 'sbj_id','brain_surface',
                                                            'inner_skull_surface', 'outer_skin_surface',
                                                            'outer_skull_surface'], output_names=['sbj_id'],
                            function = copy_surfaces), name = 'copy_bem_surf')
    copy_bem_surf.inputs.sbj_dir = sbj_dir

    main_workflow.connect(infosource, 'subject_id',   copy_bem_surf, 'sbj_id')
    main_workflow.connect(bem, 'brain_surface',       copy_bem_surf, 'brain_surface')
    main_workflow.connect(bem, 'inner_skull_surface', copy_bem_surf, 'inner_skull_surface')
    main_workflow.connect(bem, 'outer_skin_surface',  copy_bem_surf, 'outer_skin_surface')
    main_workflow.connect(bem, 'outer_skull_surface', copy_bem_surf, 'outer_skull_surface')

    bem_sol = pe.Node(interface=Function(input_names=['sbj_id'], output_names=['sbj_id'],
                             function = create_bem_sol), name = 'bem_sol')

    main_workflow.connect(copy_bem_surf, 'sbj_id', bem_sol, 'sbj_id')

    source_space = pe.Node(interface=Function(input_names=['sbj_dir','sbj_id'], output_names=[''],
                             function = create_source_space), name = 'source_space')
    source_space.inputs.sbj_dir = sbj_dir

    main_workflow.connect(bem_sol, 'sbj_id', source_space, 'sbj_id')

    return main_workflow

if __name__ == '__main__':

    ### run pipeline:
    main_workflow = create_main_workflow_FS_segmentation()

    # main_workflow.write_graph()
    main_workflow.config['execution'] = {'remove_unnecessary_outputs':'false'}
    main_workflow.run(plugin='MultiProc', plugin_args={'n_procs' : 8})


rev6 starts up now and works, but no gyro sync :)
Op 8 okt. 2015 19:33 schreef "Larry Davis" <notifications@github.com>:

> Is #1376 <https://github.com/cleanflight/cleanflight/issues/1376>
> addressed in these binaries?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/borisbstyle/cleanflight/commit/f5b00031ea161952fccec9a3f80333981aad80d2#commitcomment-13668988>
> .
>


Oh, so I guess that the GMAIL_PASSWORD_PLACEHOLDER is valid for this new mail service.


I think it would make more sense to use delete the CLIENT_* lines and then delete newlines:
```bash
kvargs=$(echo "${@:4}" | sed -e '/^CLIENT_/d' | tr -d '\n')
```
(Sidenote: I prefer `'\n'` over `"\n"` because the latter makes me think about or double-check Bash's weird rules for interpreting backslashes.)


I wrote some cleanup.

Can you provide me tests vectors ? pcap files ?

Best regards

--
  Benoît Canet
  http://www.nodalink.com

Le Jeu 08 oct 2015, à 13:57, Eric Leblond a écrit :
> It is for C code not for Python :P but refactoring patch is accepted ;)

> —
> Reply to this email directly or view it on GitHub[1].

>

Links:

  1. https://github.com/regit/suriwire/commit/01865aa9e48c81a9c613d8bb7752b620f15fb345#commitcomment-13660912


I should have checked the code before opening my mouth :-). Feel free to go ahead and do the header base allocation I might get to it tomorrow.


# 7. The variable ArtisqtBio lists the birth place of each painter. Use this
#information to create a world map where each country is colored
#according to the stock of paintings in MOMA's collection.

df_map=df

# if not born
df_map$birth1 <- sub(",","",sub(" .*","",str_extract(df_map$ArtistBio, "[A-Z].*[a-z]")))

#If born is followed by numbers
df_map$birth2 <- str_extract(str_extract(str_extract(df_map$ArtistBio,
                                                  ".* born [1-9]"), "[A-Z].*,"), "[A-Z].*[a-z]")

#If born is followed by country
df_map$birth3 <- str_trim(str_extract(str_extract(df_map$ArtistBio, "born .*"),
                                   " [A-Z].*[a-z] "))

#Using ifelse function to combine the three scenarios
df_map$b2b3<-ifelse(is.na(df_map$birth3),df_map$birth2,df_map$birth3)
df_map$Birth_place<-ifelse(is.na(df_map$b2b3),df_map$birth1,df_map$b2b3)

df_map$birth <- sub(",","",sub(" .*","",str_extract(df_map$Birth_place, "[A-Z].*[a-z]")))

# Scrape page for nationalities and country.

nat1 <- html("https://www.englishclub.com/vocabulary/world-countries-nationality.htm") %>%
  html_nodes("td:nth-child(2)") %>%
  html_text() %>%
  as.character()
nat2 <- html("https://www.englishclub.com/vocabulary/world-countries-nationality.htm") %>%
  html_nodes("td:nth-child(1)") %>%
  html_text() %>%
  as.character()

nat<-data.frame(nat2,nat1)
names(nat)<-c("Country","birth")

merge_birth<-full_join(df_map, nat, by=c("birth"))
merge_birth$Country<-as.character(merge_birth$Country)
merge_birth$region<-ifelse(is.na(merge_birth$Country),merge_birth$birth,merge_birth$Country)

map = map_data("world")
map$iso2c = countrycode(map$region, origin = "country.name", destination = "iso2c")
df.map.2 = left_join(merge_birth, map)


@ttyS0 - my preference is to have all *packages* exclusively install to ```/usr``` (and not ```/usr/local```), while still letting individual local *builds* go wherever (ex via ```PREFIX```)

Do all the repository 'watchers' get all of the notifications automagically, or should I ```@``` them?


@tmat Actually, I'd prefer to explicitly set the search paths here instead of creating a reference resolver for the project. This is intended to provide nothing more than #r completion for script in VS, therefore the change should be limited to this completion provider, until we come up with a proper fix in the future (a script workspace maybe).


[USPA Aircraft Manual] (http://www.uspa.org/Portals/0/Downloads/GM_USPAAircraftManual.pdf)


woot?


[USPA Jump Pilot Training Syllabus] (+http://www.uspa.org/Portals/0/Downloads/GM_JumpPilotTrainingSyllabus.doc)


It is the standard way in orion code to navigate through the file system;) Probably when the project name is correct, then using "explorer.scope" method would work properly but I cannot guarantee that.


A widzsz tego nie wiedziałem:/ a ja durny że starych  picon nazwy
kopiowałem:d tak poradził em sobie właśnie z tym i wrzuciłem duże do picon,
tylko gdy chciałbym ustawić coś z klasycznymi piconami są one wtedy na
sztywno przeźroczyste.

Wygląda to mniej więcej jak w załączniku.
08.10.2015 8:22 PM "herpoi" <notifications@github.com> napisał(a):

> Jak to ręcznie przypisywać kanały do ikon? Przecież OpenPli już od niemal
> roku rozpoznaje pikony o nazwach kanałów.
> InfinityHD już od dość dawna widzi tylko pikony w
> /usr/share/enigma2/picon/. Wrzuć tam po prostu duże pikony zamiast małych i
> będzie po sprawie.
>
> W dniu czwartek, 8 października 2015 mihail1 <notifications@github.com>
> napisał(a):
>
> > Robisz super picony;) szkoda, że się nie znam na tym i muszę ręcznie
> > przypisywać kanały do ikon.
> > W Twojej wersji infinityHD mój nBox nie widzi picon w katalogu
> > /usr/share/enigma2/BigPicon/picon/ widzi tylko z tego
> > /usr/share/enigma2/picon/. W związku z tym jak chce gdzieś duże picony to
> > dekoder skaluje te małe standardowe i wszystko wygląda jak jeden wielki
> > piksel
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13668443
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670139>
> .
>


I'm not sure I understand why you have this method. @friedmud said that there is a single virtual in the libMesh base class that you need to implement but this isn't the way to do that. Your interface has to satisfy libMesh's interface not create it's own right?


Does this filter out the entire CLIENT_* line, or just the CLIENT_* word at the start of the line?


This above the others, under the constructor (see other tests for formatting)


[USPA Aircraft Operations Pilot Training] (http://www.uspa.org/GroupMembers/GroupMemberProgram/AircraftOperationsPilotTraining/tabid/544/Default.aspx)


@dmn47 Here is how your changes translated to the content and themes folder.


Why do we even have these single-quotes?


hello world


Why don't you remove the `_ = *` ?


We should push the indent only when the `format_variant_list` succeeded.


> Do all the repository 'watchers' get all of the notifications automagically, or should I @ them?

I dunno.

Marco!


Hi Sir,
I need some information about your code, can you please send me your emai address to have some conversation.
Thanks alot


LGTM


iniciando projeto


Oké, akkor ötletelgessünk :D csináljuk meg az office 2020-t :DDD
Komolyra fordítva valami olyat csináljunk akkor, amit esetleg még
használnánk is. Nekem mondjuk elkélne egy fasza naptár cloud
szinkronizációval. Vagy mondjuk egy esemény/taskkezelő, vagy
csináljunk egy mobilos ticket hibajegykezelőt rft-hez passzol is :D
Kezdetnek akkor ennyi hirtelen :D

uszalab <notifications@github.com> írta, 2015. 10. 08.:
> Túl komolynak nem kell lennie, inkább valami irodai hasznos dolog legyen.
> Nem szoktam játszani:D
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13662131


I'm not sure what this code is supposed to do, but in any case, I don't think it accomplishes it:

```console
$ foo() {
>   IFSBAK="$IFS"
>   IFS=$'\n' snippedargs=($(printf "%s\n" "${@:4}"))
>   IFS="$IFSBAK"
>   declare -p snippedargs
> }
$ foo 1 2 3 foo $'bar\nbaz\n' quux
declare -a snippedargs='([0]="foo" [1]="bar" [2]="baz" [3]="quux")'
$
```


The Overrun checks can come out later, will just be interesting to see if any other games do it.


Witam
No i przy przełączaniu kanału gdy ustawie środek z bigpicon jest tylko
cyfra kanału nazwa żółtymi literami i tak gdzie ma być ikona jest puste
pole:(
Sorki za moją nie wiedze  ale dopiero raczkuje w temacie.

Pozdrawiam
Michał
08.10.2015 8:55 PM "Michal Kawaler" <michal.kawaler@gmail.com> napisał(a):

> A widzsz tego nie wiedziałem:/ a ja durny że starych  picon nazwy
> kopiowałem:d tak poradził em sobie właśnie z tym i wrzuciłem duże do picon,
> tylko gdy chciałbym ustawić coś z klasycznymi piconami są one wtedy na
> sztywno przeźroczyste.
>
> Wygląda to mniej więcej jak w załączniku.
> 08.10.2015 8:22 PM "herpoi" <notifications@github.com> napisał(a):
>
>> Jak to ręcznie przypisywać kanały do ikon? Przecież OpenPli już od niemal
>> roku rozpoznaje pikony o nazwach kanałów.
>> InfinityHD już od dość dawna widzi tylko pikony w
>> /usr/share/enigma2/picon/. Wrzuć tam po prostu duże pikony zamiast małych
>> i
>> będzie po sprawie.
>>
>> W dniu czwartek, 8 października 2015 mihail1 <notifications@github.com>
>> napisał(a):
>>
>> > Robisz super picony;) szkoda, że się nie znam na tym i muszę ręcznie
>> > przypisywać kanały do ikon.
>> > W Twojej wersji infinityHD mój nBox nie widzi picon w katalogu
>> > /usr/share/enigma2/BigPicon/picon/ widzi tylko z tego
>> > /usr/share/enigma2/picon/. W związku z tym jak chce gdzieś duże picony
>> to
>> > dekoder skaluje te małe standardowe i wszystko wygląda jak jeden wielki
>> > piksel
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13668443
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670139>
>> .
>>
>


What is the use of restartLevel()? It is empty. Otherwise add javadoc.


I don't think it's useless.

https://developers.facebook.com/tools/debug/og/object/


Это лишнее, у тебя в super() вызывался конструктор Object, где сделалось то же самое


Added Travis CI Badge


This is definitely better, but wasn't exactly what I had in mind. I was thinking that we'd tell the SimpleJSONRPCServer to listen on 0 and provide a way to get the port number out via the SimpleJSONRPCServer's interface. Pedantically speaking we're still not truly letting the OS select a guaranteed free port to give the server. Also if it wasn't for the `allow_reuse_address = True` in SimpleJSONRPCServer then this would have caused issues.


This may crash unless checks in the constructor are slightly modified as I suggested above.


I actually don't understand what this is. Github shows it as basically redoing all the changes that have been made in the past week or so but `git diff 53d532c 52a45d1` has no output.


Laat maar hier is restartLevel() weer weg :p


Given that we're casting to a "pointer to byte", it might suffice in this case.  I think that you can get alignment warnings in the more general case - I added the cast to `void *` to squelch *some* warning, as I remember.


+    public void registerObserver(Observer observer) {
+        if(observer == null || observersList.contains(observer))
+            return;
+        observersList.add(observer);

Voor hier accolades toe voor de if voor findbugs


у тебя x*scale это this.x, используй его


Fixed in 832571f8f0fbe0ec3ba4f961c6d4fb682ac742b6 and edddd32396b1f21a7374e52ea099b1e5ded2106a


here https://android.googlesource.com/kernel/common/+log/android-3.4/
almost done :+1:


Later nog een keer


New pull request for same function created.


I remember that I did a diff between the myriad doc and the myria doc and there was no difference. But thanks for the other changes!


This is complicated.  How about the following?

```bash
arg="${arg##*([[:space:]])}"
arg="${arg%%*([[:space:]])}"
```


borra este archivo


No bueno


yuck.


Awesome. Is it possible to change the size of the game without a restart, too?


borra este archivo, esta en la raiz

NO BORRES el que esta en public/


Check out parents for this commit, in particular https://github.com/letsencrypt/letsencrypt/commit/0ecfec56e26f40333879d17ec8755ae75b7798f2. Then https://github.com/letsencrypt/letsencrypt/commit/321ae99035f14b2252651ebf59ad837d526011a2. Then...


This is certainly a nice improvement. However, I'm not quite clear on why this needs to be a class, rather than a stateless function. (I haven't fully grokked the docstring below either, though all this might be due to mental exhaustion.)


Er is een javadoc annotation @author


Using Leaflet 1.0 beta, with `map.scrollWheelZoom: false` specified, the scroll wheel still zooms the map. If I set that line to check specifically for `map.scrollWheelZoom === true` I can disable and reenable `scrollWheelZoom` again, but then continuous zoom seems to be disabled, even if it is set to `true`.


"or is it a problem of the rule engine? what if the same term instances are used again and again and then modified on substitution?"

^ I just wanted to post that, and yes "derivedTerm = derivedTerm.cloneDeep();" did it.
Sorry this I should have found earlier, especially since nal.meta is currently primarily for me to maintain.
Nice. Only 4 failed testcases anymore!!


Er staat hier nog een syso


@jchodera
Cell 17 in HSA fluorescence analysis - ligand2.ipynb is where the error is. Do you have any idea what might be causing this?


If we're going to repeat this, we probably should just define a `trim()` function in the SDK.


74564912Issue4347BBRadConvHW (wfbuhl) - x86_64-MacOS-10.9-clang: Tests Failed

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


Whoa, nice!


s/Computation/Computational


The new implementation is more succinct and this is good. But why did you inline it?


curly quotes


Commit LGTM


Interesting, why were we browserifying in the first place?


These instructions were so clear that I succeeded on the first try. super rare for me. :P thanks thy!


merging latest code


ah, okay - thank u for answering


I assume by "stateless function" you mean something that returns a function that closes over the variables that are currently being set in __init__ below.

No particular reason other than this way probably being a bit more what Python people would probably expect and possibly more extensible/future proof. Mostly just because it is what came to mind first.


Why were the level, threshold, and standbypct variables changed to const? We'll need to be able to modify these later once the settings menu is built.


@dmmiller This could be interesting to share between iOS and Android.


Epic! :)


Markdown is spelled Markdown, no camel case


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/22037/testReport/) for this commit**


I know this is orthogonal but at some point IOException should die


Code Thief. Its our code for Hackathon...


I have just implemented a proof-of-concept libsndfile based sound source. Only 150 lines of code. With your permission, George, I will include libsndfile in Marsyas sources and integrate the new sound file source into the existing system (automatic selection in SoundFileSource, etc...)


I thought you would add yet another troubleshooting button for shooting troubles with troubleshooting button.


for a ByteBuf impl we can use putInt if the Endian-ness matches, same for long


Thanks, it sound good


Protip: You can probably use [#map_to](http://www.rubydoc.info/github/jirutka/corefines/master/Corefines/Enumerable/MapTo) from Corefines here. ;)

```rb
semester_periods = semester.semester_periods_dataset.where(type: 0).order(:starts_at)
  .map_to(PlannedSemesterPeriod)
```


Now everyone are able to see my comments. Please make this repository private.


Reason: some old trarslations still shows TRDS title


You are the man - just make sure that it can be turned on/off with CMake just to avoid full dependence on it.


volli absicht :P nai weiss nöd wasi da komischs gmacht han, danke für de hint :)


I suspect benchmarking will indicate that you can get rid of this


Ja, de Game class kan je nog ff negeren ;)
Is onderdeel van MVC, maar nog niet af.


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 8](https://teamcity.labkey.org/viewLog.html?buildId=276636&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:24


Shouldn't this be?
```rust
Philosopher::new("Michel Foucault", 4, 0),
```


@jessdtate We forgot to add this line, which actually uses the port string. My simple network worked with this change. The rest below is all reformatting and adding the needToExecute() call around the rest of the code.


I think this fix is correct.


Do you think we should stop this test to free up some resources?
As it seems neutral...


Não rola colocar o Slack da Cloudwalk já que só o time tem acesso.

Acho que o correto é deixar o Linkein ou Github dele: https://github.com/scalone


`RoutesHash | any` union is no different than only `any`, but had to have it this way because due a typescript bug, if we only used `RoutesHash`, then object assignments to inherited types would not be contextually typed. See related [issue](https://github.com/Microsoft/TypeScript/issues/5181).


hello


Ok. I won't come in tomorrow. You are welcome to do a pull request on the selfie repo. I will then improve it further and merge it into all repos eventually. -ck


EoL comments are wrong


Typo detected: “…from from…”.


Yea, Markdown again


This is NOT US Desert with M4 Rifles


Again, comment is wrong


I made a nice typo creating the branch, but here's the functional version for comparison: https://github.com/cdent/gabbi/compare/cleanup-from-yaml...functional-cleaup-from-yaml?expand=1


:trophy: moar moar


"@hellermf In either case, /usr/local isn't the right place to put RPMs' files ...."

You misunderstand my purpose in making this change it is actually to force the packages to install to their proper location, /usr. I package everything up with PREFIX=/usr and this makefile was previously hardcoded to /usr/local and breaking my build. Also for other packages I did go ahead and use GNUInstallDirs and so far I like it. It gives even more flexibility and automatically-doing-the-right-thing-ness (like detecting the multi-arch conventions of different distros: the miserable lack of standardization of /usr/lib vs /usr/lib64 vs /usr/lib/x86_64-linux-gnu/ ) So I may considering reworking CMake to use that module in the future.


Potential optimization to use pre-computed table of int -> varint bytes. Possibly useful as most protobufs have a small no. of fields with low ordinals.


UCP, not desert. M4 rifles is right though


Just "Loading..." will do (consistent with elsewhere in the app)


like i said, let me handle it lol


Same note about int's here


SequenceNumber++; es algo más comodo


Avoid unused local variables such as '$id'.


Method GroupController::edit does not have a Docblock comment.


Method GroupController::delete does not have a Docblock comment.


s/started/start/


can't you return `self.objectForKey(key) as? NSDate` directly?


static methods should move into the WireFormat class


Use rewriteSequenceNumber() here?


Please don't use general exceptions.


I don't think these guys are Ukrainian (it's also spelled wrong)


Looks good! Will download and verify functionality as well.


Should be `NetworkStreamTest` (singular) to match the filename.


Wait, no -- I just meant getting rid of the `<i>`, not getting rid of "Unrated". I think "Unrated" should still show, just not italicized.


IMHO these don't need to have `key` at the end


s/this.intervals/#intervals/ (or is it just my IDE that doesn't interpret "this.intervals" properly?)


Ukrainian misspelled.


Isn't the key part of the object itself?  Can't we just put these in a list?


I guess this is part of the "which end are we at?" detection.  I suspect you want to take an average rather than a difference, thoughts?


Avoid object construction here if possible


Added head


el macro UPDT_utilsNtohl debe incluir esta inteligencia, es más el macro deberia ir no aca sino en modules/libs.


adding the offset here is to roll it into the new calendar before zeroing out the day ?


also wondering if this is a good use case for swift enums


And FWIW, this filename/class should be singular as well to be consisteent, i.e. `DisposedSocketTest.cs`.  We follow the "Art of Unit Testing" guidelins for naming.


YAGNI?


Shouldn't this raise an exception?


You might wanna make sure that idx stays in bounds here (same applies to the REDTransformEngine, I expect)


Incurs object creation overhead :(


Nope; this is probably because I remain ignorant, but the only potential state I can spot is ` self.loader` - presumably that's a collection of tests? (I'm not counting `prior_test` as that's passed in from the outside, not retained/determined internally.)

So, in my current naïveté, I would have expected something like this (pseudo-code):

```python
context = { ... }
tests = []
for test_descriptor in test_descriptors:
    predecessor = tests[-1]
    test = make_test(test_descriptor, predecessor, **context)

suite = TestSuite(tests)
```

This might be way off though, so if that's the case, we should clarify my understanding another time.


Thanks :+1:


Remember to add view.


@vncdias como assim post em ajax? é o JS que envia?


typo "would would"


Please use math here, in stead of making new instances of network.


Could you please explain this change?


Removed unused detail setting from arg parsing, struct, and config logging


Thanks for your patch, I see some problems though. Can you have a look?


Also on my to-do list is that (last time I checked) prefer shared lib vs prefer static lib was hardcoded differently package by package and I would like to clean that up. Thoughts? The option to set an env var, for example PREFER_LIB=shared, to override any hardcoded setting is the best I am coming up with.


Sparsam umga mit maven dependencies, jungs ;)


Method GroupController::edit does not have a Docblock comment.


Method GroupController::delete does not have a Docblock comment.


This should probably be i<=pktCount, as the latter is initialized to 0 (and will be zero in the case of a RED packet with 1 primary block and no other blocks).


Check bounds for off+3?


@Anaethelion This one is for Heroku ? No possibility to move it to requirements folder ?


@bgerstle I don't think… if its not a date we want to fail if we can't actually cast to an NSDate


Since you are touching these lines already. `<ul></ul>` and `<li>`


I think this should use 0xff00 and 0x00ff.


Oh never mind… I see you mean get rid of the else statement…


I think we're agreeing aggressively - the packages should go to /usr and they weren't.

Running with this patch locally popped RPMs into the wrong place, which is how I ended up on this commit. If we're both on the same page about where to put it, I'll keep tidying up (I have a branch in toolbox) and double check it puts things in the right location.

Once I get done cleaning things out and factorizing them, I'll loop back around and add the GNUInstallDirs bits and see how they match up with the standards.


Thanks :+1:


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1374 of 1374 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-70.36%25-green.svg)


Does this work if seqnumOrig is the last (max) packet of an interval?


Right, thanks :+1:


To be clear, I don't think the `if` or `else` statements are necessary


Adding ControllerHelper module to assist with testing of user sign in with devise


In 10-community-cookbooks


Ouch, probably not. Thanks :+1:


Honestly? No idea.


You need `<p>` before `<ul>` and after `</ul>`. See http://blog.joda.org/2012/11/javadoc-coding-standards.html


Also problem #3 ^^


@dtenenba I'm not extremely experienced in Python, so apologies if this is incorrect.  Presumably it's possible to catch all classes that derive from [`psutil.Error`](http://pythonhosted.org/psutil/#psutil.Error) here. Would it be useful to add a secondary catch (_except_) block which handles all `psutil.Error` instances?


Leave it to me, I'm writing a common module that handles option()s properly. Once I get done swizzling everything to using that single module, we can figure out what the best default is (for now, I have it set to build/link static unless the "package" flag is set)


I usually do this… because it is easy to copy past the variable name to the string. I append Key so its obvious it is a key which is a string… I would assume `WMFAppLaunchDate` to be an NSDate.


Sorry, I completely forgot to include a summary in my commit message. After Thomas introduces fixes for rate calculation, I inquired about the timestamp used for the rate data point. It seemed counter intuitive to me that we would take the average of the two timestamps. He agreed and I opened https://issues.jboss.org/browse/HWKMETRICS-284 so that I would not forget about it. Well, I still forgot about it :) With this change we are basically saying this is the rate known as of this time.


Ah, so the issue I'm trying to avoid is having a very long argument list to the function where:

* it is very long
* where some of them are the same between each call for all the tests in one file
* where the solution would be to use **kwargs or *args, which I pretty much hate and think is evil

So this solution, and the one in the related branch is to:

* prior to the loop establish the unchanging stuff for that file
* make a TestMaker to hold that unchanging stuff
* call a method or function from that TestMaker with the changing stuff

This would probably make more sense if we were both less tired.


> IMO using the %foo% form of environment variables is appropriate here, as the docs are referring specifically to Windows, which uses that format natively.

Since the rest of the document refers to environment variables in the `$NAME` form, it would introduce more confusion than helpful information, though. Let's not do that.

> The location of the ProgramData folder is assigned to an environment variable in Windows because it's location can vary. It's usually (although not required to be), located at %SYSTEMDRIVE%\ProgramData, with %SYSTEMDRIVE% usually being C: (once again, not required).

Indeed, on XP the location is actually not `$SYSTEMDRIVE/ProgramData` at all. But there is also no `$PROGRAMDATA` to begin with. We use the appropriate replacement: `$ALLUSERSPROFILE/Application Data`.

See how useful all of this information is? If we really want to improve the documentation, we really have to include this. The best way to convey this would be to keep referring to `C:\ProgramData\Git\config`, detailing after that sentence that the path is not hard-coded at all, but instead respects the environment variable `$PROGRAMDATA` which defaults to `C:\ProgramData`. And after that, we should note that on XP, the `Application Data` directory is used instead that lives inside the location to which the environment variable`$ALLUSERSPROFILE` refers, because the `$PROGRAMDATA` concept was only introduced with Vista.

I think that will truly clarify things, better than what we have now, and better than replacing the hard-coded path by a correct-but-incomplete mentioning of `%PROGRAMDATA%`.


Hi Travis,
I'm unclear what items were comited and what needs to be moved to a
different repo.
Not sure where to start with fixing this...
Maybe I can track you down for some help?
-ek

On Thu, Oct 8, 2015 at 11:30 AM, tgaff <notifications@github.com> wrote:

> this often indicates that you copied another github repository under your
> 23-modules repository. Unfortunately that won't work. If this is a lab that
> you were asked to fork, please remove it from here and put it in your dev/
> directory. Then push to the fork and make a pull-request on *that_repo*
>
> If this is just a few files you can instead remove the hidden .git
> directory (*warning* don't remove 23-modules/.git)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ekingan/23-homework/commit/d35482ad5c559c57d58507d103548e56310f851b#commitcomment-13670304>
> .
>


I didn't know if they would be customizable. We can switch that back without a problem.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.992.fcdc8bd4c33ea29dd5ba055f0eae7887c9ebb18c](http://build.palaso.org/viewLog.html?buildId=64108&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 542, ignored: 6 Build time: 00:03:56


you could probably just return the result of this expression w/o all the if checks because `[nil timeIntervalSinceNow] == 0` will always be less than the threshold.


It's currently a JSONArray of JSONObjects, the keys aren't being stored separately.


this is also unnecessary (see above)


Thanks for letting us know Sabir. I changed the language slightly in <DESC> and will update all relevant docs.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 27](https://teamcity.labkey.org/viewLog.html?buildId=276634&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3 (1 new), passed: 22 Build time: 00:36:31
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

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

```


oh, my bad


Thanks Blake


not clear why a nested if statement is necessary here. can they be combined into one `&&` expression?


This code could be a lot less obfuscated—but maybe cleaning it up should be done in a separate commit.

What I have in mind (although I could be misunderstanding the intention):
```bash
# Take a list $kvargs of the form "k1=v1l:v1r k2=v2l:v2r ..." and produce a list
# $list of the form "v1l[v1r],v2l[v2r],...".
for arg in $kvargs; do
  value="${arg#*=}"
  left="${value%%:*}"
  right="${value#*:}"
  list="${list:+${list},}${left}[${right}]"
done
```


Yep, already discussed it with Krzysztof, and the same solution should work for Android. Not sure what the schedule is yet though.


actually, now that i've grokked it, you could just return the result of this (consolidated) expression w/o the if statements etc.


Yeah I think both are good things to have.


I'm not completely sure about this. I copied the previous format but the anchor doesn't work on GitHub.


Nit: 'improvements', not 'improvments'


so i guess this might be worthy of a "best practice" proposal to prefer returning results of expression when possible instead of manually returning `YES`/`NO`. do you prefer writing out the if statements because it allows you to do stuff before returning if needed later on?


does the link work in the existing docs?


Peti, a taskkezelőn meg az utolsó 5leted alatt mit értesz pontosabban?
2015.10.08. 21:02 ezt írta ("alper061" <notifications@github.com>):

> Oké, akkor ötletelgessünk :D csináljuk meg az office 2020-t :DDD
> Komolyra fordítva valami olyat csináljunk akkor, amit esetleg még
> használnánk is. Nekem mondjuk elkélne egy fasza naptár cloud
> szinkronizációval. Vagy mondjuk egy esemény/taskkezelő, vagy
> csináljunk egy mobilos ticket hibajegykezelőt rft-hez passzol is :D
> Kezdetnek akkor ennyi hirtelen :D
>
> uszalab <notifications@github.com> írta, 2015. 10. 08.:
> > Túl komolynak nem kell lennie, inkább valami irodai hasznos dolog legyen.
> > Nem szoktam játszani:D
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13662131
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13670932>
> .
>


:joy:


Actually the full set of graph tags are included above it.


Token is duplicated


@b-long Thanks for the comment. I'm not sure I want to do that yet. I'd like to try this and see how it works. If other exceptions are being thrown, we can look into why, and perhaps catch them. But I am wary of trying to catch too many exceptions and thereby hide behavior that we should maybe be paying attention to.


It does: https://nodejs.org/api/console.html#console_console_time_label


Ok, cool. Any other major changes pending for this?

On Thu, Oct 8, 2015 at 12:34 PM, Jeremy Koritzinsky <
notifications@github.com> wrote:

> I didn't know if they would be customizable. We can switch that back
> without a problem.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BROKE-KSP/BROKE_Defaults/commit/54a2a878873c21261c6e5a7d533ee274f19239a4#commitcomment-13671635>
> .
>


I prefer the bail early to show the intent here rather than relying on the nil side effect - it is more explicit in regards to the intent.


Should be unnecessary to use `U.nullify` here -- counts should never go negative.


Ah, right, good call.  Yeah, I should have expanded on my comment to include re-throwing the error (not just masking it).


with a default expiry of 7 days.


This actually isn't overly robust - I ran into this problem on osx and was just about to submit a pull request with this change.


Oh, and won't this implementation create a new invoice for EACH employee?

On Thu, Oct 8, 2015 at 12:41 PM, Ryan Pershing <ryan.pershing@gmail.com>
wrote:

> Ok, cool. Any other major changes pending for this?
>
> On Thu, Oct 8, 2015 at 12:34 PM, Jeremy Koritzinsky <
> notifications@github.com> wrote:
>
>> I didn't know if they would be customizable. We can switch that back
>> without a problem.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/BROKE-KSP/BROKE_Defaults/commit/54a2a878873c21261c6e5a7d533ee274f19239a4#commitcomment-13671635>
>> .
>>
>
>


Sure. Be sure to add a comment with associated 1.2 work-item


then I wouldn't worry about it. unless you know of a different syntax that will also work in github. but I wouldn't put this as a blocker.


IMO having the _variable_ name have `Key` in it is important. however, for [defaults keys & env vars](http://nshipster.com/launch-arguments-and-environment-variables/), the convention for the value is to not have a "Key" suffix .


I don't like to join that many tests together when the operators are different got a `<=` a `!` and an `==` all in there. Makes a really long if statement or a return statement harder to understand to me.


Is this supposed to be "<KEY>" ?


update_unit_testing (mbadams5) - x86_64-MacOS-10.9-clang: OK (1908 of 1908 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)


does this mean I am now an opensim developer :8ball:  ?


Like in this image
![image](https://cloud.githubusercontent.com/assets/12142968/10377870/8ba32056-6dba-11e5-8eda-867add20f36f.png)


This is outdated diff. The line you are commenting no longer exists. Please check here: https://github.com/pgjdbc/pgjdbc/pull/387/files


That's the goal. On the simple view they will be combined, but on the detailed view you can see who is getting paid what. Additionally, this way we can see who is not paid each quarter and act appropriately.


are you working on use with non-cartographic projection?


Yes, we have enough information to conclude that 23-threads test is at least neutral. I'm stopping it now.


LGTM


what about separating them into multiple lines, indenting "sub expressions" as needed? for example:

```objc
if (foo == 1
   && bar == 2
   && (buz == 3
         || baz == 4)) {
  // then foo is 1, bar is 2, and either buz or baz are 3 or 4 respectively
}
```


what if escaped_len + 2 > SIZE_MAX?


That's fair. If [multiglob](npmjs.com/package/multiglob) was synchronous, or if there was a version of a node-glob wrapper for arrays that had a nice implementation, I could get behind using that instead of just node-glob. But honestly I think this is a small use case. I think except for this exact situation, or something very similar (transitioning tasks from one format to another), there's no use for more complicated globbing.


wow this the best addition i've ever seen


The idea is to use new_size here, right?


Looks good to me. I'm not ecstatic about adding an "MLContext" flag to the parser's top-level entry point, but can't think of a better alternative offhand.


I would rather see `|| 0%{?suse_version} == 1315` tacked on here.


Top, channi das bimier wider drus neh :wink:


no problem, but them we may have to remove my additions in `config.py`, otherwise if you try to build the logs and don't have the package it will raise an exception


How does that make it any different from the vanilla system, then?


Updated the pr - but for me this is just how I think - I write out the conditions with separate returns because that is literally what the logic is (if this, do this, else that) - so I tend not to condense these. I am not opposed to returning like you suggest, especially in this circumstance - just not my usual style.


Like in this image
![image](https://cloud.githubusercontent.com/assets/12142968/10377966/19bdd49e-6dbb-11e5-8c51-1c1882c43c0a.png)


Hi @smeyer-ilias is there any concern against this PR? maybe this would help any load-balancer-based installations


5.0. not 4.0


Зачем этот лишний пробел перед скобкой?


After this line, we would have two if-endif clauses, one `%if 0%{?fedora} || 0%{?rhel}` for the Red Hat packages and one `%if 0%{?suse_version}` for the SUSE ones.

This would make more sense in my mind, because it would include all the distros/versions that support LTTng in the `_with_lttng`.


Зачем вдруг понадобилась эта настройка?


I don't remember either, but for what it's worth: I have screenshots taken with patch 2.4.3 where I see absolute values for both hp and mana on my own warlock and my imp, while I see "100%" in the health bar of another player of my own faction. Remarkably, it says nothing in his energy bar (rogue) but perhaps that was an option somehow. Also, most of my screenshots show no values at all when I have NPCs selected. And as far as I can see (and remember), I was still using the default UI back then...


Fixed [here](https://github.com/chapmanb/bcbio-nextgen/commit/902f7cd5fdff22cf77531b0e721564abab8a232b)


Do you mind if I remove the default argument for the delay in change_delay?
I think calling task.change_delay() without any arguments is not really useful and the effect is not obvious for the reader.


I have the following comments/concerns for this commit:
1. git ignore should not be gutted as it removes unneeded stuff from the project. Mainly the game boxes used to test which should be maintained in the gamebox project.
2. I have some changes of my own which I would like to merge in before I do yours.
3. This changes a lot of code. I will use it as is if you want, but I would be more comfortable with a staged implementation. Perhaps the main and tray menus first followed by the rest. Or even splitting the main and tray menus. Am I being too cautious?


Is this comment accurate? I would expect there to be an import statement here.


Is this a fix to the previous material?


Hi @w8r, yes we are. We wanted to leverage some of the functionality (e.g. drag events) from your library, but needed to reproject the actual points in multiple ways depending on the mode in which we will use it.


Awesome stuff :+1:

Played around 10 minutes or so. Game seems to be working great.


except there's no "do" anything here. in these cases where you're abstracting the details behind a condition, returning the expression directly allows devs to focus directly on what is actually being evaluated.  at the end of the day not a big deal, but IMO this is all goes back to procedural vs. functional, where having explicit `if X return Y else Z` is procedural and `return foo == 0` is functional (although, in some languages you wouldn't even need the return statement).


Почему тут `Object`? Разве оно не `Person` возвращает?


Seems like bike shedding at this point… 1 if statement with 3 lines vs 2 if statements with 2 lines…


Some background reading....

http://psy-lob-saw.blogspot.com/2012/12/encode-utf-8-string-to-bytebuffer-faster.html

also I believe the built in String.getBytes() encoding has gotten a lot faster in Java 8. Take a look at java.lang.StringCoding


Возможно, что проще было бы удалять на HQL -- оно вроде должно уметь. Меньше запросов к БД, кстати, должно быть)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 746](https://teamcity.labkey.org/viewLog.html?buildId=276638&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 13, passed: 230, muted: 23 Build time: 00:34:31
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

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

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

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

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

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

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


poke me if you need me to customize something


Это по идее должно идти отдельным коммитом.


Awesome solution, this request should be merged, although it has one bug. When you click on a tab instead of scrolling, shouldClicked is set to YES causing the tabs to move non asynchronously once again. What i did to fix it was, reset the variable (i.e. self.shouldClicked = NO;) in the scrollViewDidEndDraggin method.


I'm looking to this with suspicious. I'm getting TWI_MTX_DATA_ACK while expecting for TWI_MTX_ADR_ACK... when doing a write operation. What is TWI_COND_WRITE supposed to mean?...


This is because the Tray is mostly obscured (IMO doesn't look good) and
it isn't needed because the stock market uses a single tray.

It is was more cleanup that I did when I went in to fix the 'In-Map' menu

On 10/08/2015 03:54 PM, Richard Price wrote:
>
> Is this a fix to the previous material?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672056>.
>


Not valid if the padding character is part of the map, no?


@Hollyn This is a merge error and needs to be fixed


This `wrksrc=` seems to be redundant; the default is `wrksrc=${pkgname}-${version}` already.


У коммита неправильная ссылка на таску


Do not translate :)


LGTM :)


On second thought, no I agree with you. Done. The constructor now takes a digest constructor from hashlib (e.g. `hashlib.sha1`).


LGTM, and please feel free to apply this setting to rest of specification. I like this style. :100:


Good point!  But ##eqv? is going to be slow... given that we know both are flonums, I think I'll add a ##flonum-eqv? primitive that compares the bits are the same, and use that.  Does that seem right?


Prefer early return here.

Presumably you want to find the highest tech level of a unit, so you should flip the order of the checks and simply return the value immediately.


:D You're welcome

On Tue, Oct 6, 2015, 22:47 Ben West <notifications@github.com> wrote:

> rename to ifttt-notify for consistency
>
> Look what you started @audiefile. ;)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/openaps/openaps-js/commit/474343f088f6fe0426e6aca18f650598a0706ba5
>


Odd place to initialize values. These should be initialized in the constructor.


Eso que es?


Setter functions considered harmful :-1:


build failing, leo...


@TomasVotruba you simply have to make sure that the root of the inheritance is also in the inheritance map.


Sorry John. We delete your DFA keyword, since we decide not to mention how we deal with keyword explicitly.


It's best to manually modify the "react.d.ts" so that you do not have to write `import * as React from 'react'` but `import React from 'react'`

The change is from `export = react;` to `export default react`, yet you are working with a bootstrapo version that does not have this.


"Remove blocks..." is a good idea! It bothers me that most of the items in the File menu begin with verbs but not all. It makes sense for menu actions to be verbs.

Yeah, my concern is that someone will see "Unused blocks..." and not realize they'll be removing them...


The way this is used in setOriginalSequenceNumber (and also for consistency), this should include "this.offset".


This is not a detonation event...


Also not every block metadata is 15


什么呀！


Docs missing :)


@klauszhang this is for saving data. Why do we need this here?


AI level 3 moves are *very* random: http://lichess.org/J91xFB6X for an example.
Perhaps a more effective tact (than increasing randomness as is done here) would be to limit the number of nodes searched (the "go" command accepts a "nodes" argument for this purpose).


And we probably don't even want to log fire spread but only the destruction of blocks by fire


Sweet, thanks for the help @cpbuckingham !


You wouldn't be able to cache it.

I don't think it's too important for dexsearch.


I'm going to move this conversation over to google group but it's being rather slow tonight.  I guess bumping this to parent 26 snapshot would have done the trick as well.  Anyway, figured we are doing this too many times as well.  We are running this on all 4 java builds so 4 times each go.  So i've corrected that.  I've also added version eyes and trying to get coveralls working now.  Should look rather nice when all done.  Probably will add in the site generation as well.  Anyway...once I can get google groups responding a little better I'll move this convo over there.  Thanks for helping out!


solved on Discord; The change is good! :)


Thanks :+1:


Yes, current stream time. I think in video editing case, we need to know each frame with timestamped. So the playback time just like the time stamp.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.649](http://teamcity.monogame.net/viewLog.html?buildId=10503&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


I haven't tested this at all, and I have no idea why I committed it...
This is just how one would be implemented.


You should really get on this!


to generate the dropdown list, without this _for helper, we can't access other models other than EvidenceItem. The select helper in form helper of EvidenceItem require access to SoftwareEngineeringMethodology and SoftwareEngineeringMethod to build the list.


LGTM :100:


Again, this is for *saving* data, not *reading*.

> Nested attributes allow you to save attributes on associated records through the parent.

[API doc](http://api.rubyonrails.org/classes/ActiveRecord/NestedAttributes/ClassMethods.html)

All good, I'l remove it in the next commit. :)


Woo! Action. =D


Thanks for doing this! Finally someone's looking into making the random hacks section in the LilyPad config window less confusing.

You could also disable "Send escape on close" since PCSX2 suspends emulation (Esc.) when you close the GS window anyway. It's irrelevant.

The "Exit emulator on window close" option doesn't seem to be working for me. Could someone verify please?


LGTM. Welcome Anssi. Thanks for your great help. Kaku and I learned a lot from you in writing spec side. :)


Yeah...well...I forgot what I used this for... anyway I'll make a delete commit.


I apologise for the branching. I accidentally created branches which I do not know how to break, but I can merge "develop"(branched-out paths) back into "master"(main path).

Command Line Prompt previously had most of its notification messages leaving 2 lines each.
This has been changed to mostly one line each.
Those which are not leaving one line each are currently mostly intentional.

Previously, Flex Display did not leave a line for notification messages (when it is not displaying a list of tasks). It is now leaving a line for notification messages, even if there is only one notification message.

Thank you.


I copied original one from master and pushed in the branch in my next commit to ensure it is not changed.


Another thing you can do to see if you might have non-numeric values, is to use the .info() function on your data frame to find anything that looks unbalanced, then investigate.  What you have is great though.  Just an FYI.

```

<class 'pandas.core.frame.DataFrame'>
Int64Index: 467 entries, 0 to 466
Data columns (total 15 columns):
Unnamed: 0              467 non-null int64
name                    467 non-null object
age                     467 non-null int64
gender                  467 non-null object
raceethnicity           467 non-null object
month                   467 non-null object
day                     467 non-null int64
year                    467 non-null int64
streetaddress           463 non-null object  <-------------- AH HA!
city                    467 non-null object
state                   467 non-null object
lawenforcementagency    467 non-null object
cause                   467 non-null object
armed                   467 non-null object
county_income           467 non-null int64
dtypes: int64(5), object(10)
memory usage: 58.4+ KB
```


You should pick this as well:
https://github.com/CandyDevices/kernel_motorola_msm8916/commit/b24b385d36c53b7b8595ad505b39e456210127d4
GCC 5 Fixups
it shouldn't break gcc 4 build but I guess anything is possible


Implemented this fix per http://christopher-schuster.me/technology/2015/08/22/travis-ci-fun/


Would be useful for this to be some sort of default template that I can pass in the `[]Group` results to as well as other fields


LGTM. I don't have any idea better than null now. :)


Merged


Yeah it should, thanks for catching it!


Routes file?

Only a few so putting them here for now while I learn more.


This should have been handled by the templates - not merged


Yes, good point, using `base < size - offset` it is only necessary that the `size - offset` is not negative, but this applies to both 32 bit and 64 bit runtimes so becomes marginal for 64 bit runtimes running wasm32 code which should scale to a 4GB memory size in which case no offset could be accommodated using inline bounds checks (a memory protected guard zone could be used).

One significant question people will probably want to understand is why the offset is even included in the instruction, and this is due to the interpretation of the base as an unsigned value rather than the `base + offset` as an unsigned value. This will be important for producers to know too because they need to take care not to fold offsets applied to an index (that might be signed) before it is cast to an unsigned value into the instructions offset.

In any case these are all implicit implementation details and quite the opposite of 'offloads some of the optimization work to the compiler'.


I'm going to hold off merging these - this really should be handled with `m.command( 'stop' )` for example


Need to figure out how to increment ID's on Groups. Some options:

1. Start fresh when server is run with an initialized constant named `var groupSeq = 1`
2. Make a `func init()` that connects to Redis, grabs the largest `groups:id` and sets that to some constant like `var groupSeq = value`.
3. Find out if Redis does this with `Id` fields on its own.


Nếu đã dùng preg_match thì khỏi cần kiểm tra cái ! empty( $module_name )


snprintf puts null as ```(null)``` with ```%ls```.


Same with these - I understand the motivation, but I'd rather that the user adds these to their own library. You'll end up with "helpful" suggestions like "I think `run-timed()` should also have a default `duty_cycle_sp` but it should be `50` not `75`. Let the users handle this themselves :-)


Resolved, transferred to separate file locally "HashFunction.java"


sweet


Resolved, same issue as that in following comment.


Đoạn này cần đưa xuống sau dòng 71, bởi nếu người dùng sửa url, bỏ cái op = rỗng sẽ phải chuyển trang là không cần thiết.


Now its getting larger depths earlier (function is bigger). Maybe try 5*log(1+x/4) which stretches graph


space missing between words.


Is a web worker better than a worker?


Looks like this is still a problem. Have you tested it?


Shouldn't have been removed, I changed that line back. @mgsmith1000 was working off an earlier copy and missed that. Thanks Phil!


LGTM :100:


Can you remove the .idea files (optionally with a .gitignore)?


LGTM


Hmm… I should have defined LI with platform check, but this can not be done with a clean solution. Going for gcc and clang first.


Merged credit, the getters/setters/init are handled by the autogen template updates. I'm leaving the doc string alone for now - we may need to add some features to autogen to make the strings look better.


Please consider moving this (and the other helpful motor command helpers) to a separate module. The base bindings between attributes and files are almost all autogenerated - I'd like to keep the base binding as simple as possible


This doesn't look right. Have you tried this? The 3rd field is supposed to be the fstype (fuse), but in this change fstype is the 4th field.


Same here - it's a helpful function, but I'd prefer it in a separate module.


nice!


Should I send a PR then?


There are so many things I would like to do but are difficult to do on my own. Until I get this bat cave operational (and probably for the next few months while it's being used), I may be pushing updates specific to the needs of this system (focusing on humidity), such as sensor verification (two sensors in one area must both read within X% of each other). The other sensor types will eventually be updated with the same functionality.


再次评论


Thanks!  I didn't even notice that :)


Please!

--

*José Valimwww.plataformatec.com.br
<http://www.plataformatec.com.br/>Founder and Director of R&D*


wat?


Can you provide a reasoning behind the change? Do you have specifics in mind?

To my mind, this is the most usable. Any additional options with template engines could be provided through additional methods in the library.


reviewed


I agree, something like that completely went over my head. Good thinking :+1: .


@dalanlan Er ... why this place use `KUBE_APISERVER_STOP` instead of `FLANNEL_STOP`? Something wrong merged in?


Interesting. I didn't realize this would be popular. Of course, if you want to tweak the sorting it's very easy.


软工第一次会议记录
10月7日晚8:30 online dictation system项目小组在 D26楼下进行了第一次站会。
本次会议的议程主要由3部分组成：
1.组员相互介绍
2.po详细阐述需求
3.任务分块

根据讨论结果，第一轮scrum开发的目标是完成批改和反馈模块。

任务块：
1.对这一模块进行进一步的细化
2.对每一细化模块进行算法设计
3.具体代码编写
4.测试和维护
请组员自己对有意向的任务报名，明天中午前（周五）会完成调剂。


gurl pls what is this crap


LinkedIn, y u no return 200-level response code?
```
$ curl -IL https://linkedin.com/in/joelsmiller
HTTP/1.1 999 Request denied
Date: Thu, 08 Oct 2015 02:57:21 GMT
X-Li-Pop: prod-lva1-SPDY
X-LI-UUID: Dvpa8q0WCxQAyE3gOisAAA==
Content-Length: 956
Content-Type: text/html
```

Also, some useful documentation pertaining to the html-proofer gem:
> https://github.com/gjtorikian/html-proofer/issues/223
https://github.com/gjtorikian/html-proofer


I am thinking should we add more explanations about the different between video monitor event and video processor event. For example, the monitor is for the case you only want to get the video frame from MediaStreamTrack. You don't want to pass the processed video frame back to a new MediaStreamTrack. Otherwise you need video processor to create a new MediaStreamTrack contained processed video frame. That might help web developer to figure out which one is best for them. @anssiko how do you think?


У коммита нет ссылки на таску


@Hollyn This is a merge error and needs to be fixed


This is outdated


These comments are outdated.


I didn't understand the rationale for the suse_version check being only in one location, so that's why I left the logic the same -- as if it assumed it already had the necessary packages pre-installed.


@marinofull exatamente

Att.
Vinícius Dias
Gerente de Treinamento
Vice-Coordenador Geral da SEMCOMP UFBA 2015

Email enviado via Gmail para Android
Em 08/10/2015 16:29, "Marinofull" <notifications@github.com> escreveu:

> @vncdias <https://github.com/vncdias> como assim post em ajax? é o JS que
> envia?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/InfoJrUFBA/caixinfo/commit/3a8627a2f4c8d0d626a0dc1675a2029aae7c657a#commitcomment-13671499>
> .
>


null checking*
but I somehow didn't actually do it, done here: https://github.com/Tanjoodo/autoproxy/commit/fc30b9fc97de9f96d21a2005e1ebe1d077c27c13


er... this was before I changed the code to take the padding as the last byte. will change locally.


Мне кажется, что надо игнорить все скрытые файлы
```xml
<exclude>.*</exclude>
```


Производить изменения с помощью GET -- грешновато =)

На самом деле еще и опасно. Есть различные истории когда люди вешали операции по изменению на GET запросы, а потом по этим ссылкам приходил браузер (например, при восстановлении вкладок) или поисковик как-то их находил (например, из твоей почты) и в итоге люди теряли данные. Поэтому не делай так :)


I don't think many people are building these locally and it's not like we had sphinx installed before, which is where all the dependencies come from. Could we just add documentation in case someone wants to do this?


OK. Thinking was that it would be nice to not expose the internals of
NodeInfo across two packages if we didn't need to. Understand the desire to
keep executor/ purely mesos.

On Wed, Oct 7, 2015 at 7:56 PM, Dr. Stefan Schimanski <
notifications@github.com> wrote:

> See below. The cAdvisor code is part of the kubelet internals. This
> shouldn't bleed into the executor. Moreover, the Mesos code shouldn't bleed
> into the service. Hence, this struct is the compromise: a simple interface.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mesosphere/kubernetes/commit/498a62af4e9e3917ff1ecacd60957a2a4df05cd7#commitcomment-13652227>
> .
>


It's an issue in the syntax file which needs fixing. The Vim contribution guidelines say to contact the file maintainer before submitting your own pull request, but I only saw that after making the change. Now I'm waiting on a reply. I'm being a part of the open source community :)


Isn't the global.ArtifactCollectors a table?


Thanks!


This seems like a content issue to me. The OA will automatically split `<h1>` tags into separate sections with their own completion checkbox.

If we want more sections with checkboxes, they should be H1s. If we want fewer H1s for SEO, then there should be fewer sections, or the content itself adjusted. In particular, this file is super long and our students would probably appreciate if it was broken into multiple pages.


See notes below.

On 10/08/2015 03:52 PM, Richard Price wrote:
>
> I have the following comments/concerns for this commit:
> 1. git ignore should not be gutted as it removes unneeded stuff from
> the project. Mainly the game boxes used to test which should be
> maintained in the gamebox project.
>
I didn't gut it, I put it in my .gitignore so it wouldn't be updated to
the repo.
>
> 2. I have some changes of my own which I would like to merge in before
> I do yours.
>
You are free to merge in any order you would like.
>
> 3. This changes a lot of code. I will use it as is if you want, but I
> would be more comfortable with a staged implementation. Perhaps the
> main and tray menus first followed by the rest. Or even splitting the
> main and tray menus. Am I being too cautious?
>
It is needed as a whole to be usable by iOS. If you split it up, iOS
users won't be able to use it until the final piece is implemented.
If you want to be cautious, upload your changes to GitHub, then merge in
my Pull Request. Fetch it onto your test box and use/test it for a while
till you are comfortable with it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajprog/board18/commit/47766c2e75614d2a16cbeb93f077b7c0ca0c5837#commitcomment-13672028>.
>


Facebook's debugger was unable to find it. It's fine now.


Так локализовывать тоже можно, особенно если оно работает, но я сделал не так:

Во-первых, старался универсальные сообщения резюзать между разными полями. Т.е. вместо 4х переводов (Не указано имя/Не указана фамилия/Не указано отчество) для NotNull я использовал одно универсальное -- "Поле обязательно для заполнения"

Во-вторых, я не использовал при локализации название самого объекта (если буду переименовывать и забуду переименовать здесь, то это потенциальная ошибка в будущем).


we lost the non GV_ADD for main, we have a patch for it lol


You already were!


HipChat API change requiring this:

https://www.apichangelog.com/changes/3f30140d-066b-4345-9050-c124f32369a3#first-change


Use comma after the last argument.


Если можно, то переименую `replace` -> `set_rao_perspective`


>Если можно, то переименую `replace` -> `set_rao_perspective`

И тут. В общем везде, где можно


Как параметр уже можно задавать ?


import numpy as np
import matplotlib.pyplot as plt

ort1, ort2, ss1, ss2, n = 3, 4, 1, 1, 10000

array1 = np.random.normal(ort1, ss1, n)
array2 = np.random.normal(ort2, ss2, n)
histo1 = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]
histo2 = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]

print(array1)
print(array2)

z, bins, count = plt.hist(array1, 40, normed=True)
z, bins, count = plt.hist(array2, 40, normed=True)
plt.plot(bins, 1/(ss1 * np.sqrt(2 * np.pi)) *
 np.exp( - (bins - ort1)**2 / (2 * ss1**2) ), linewidth=2, color='r')
plt.plot(bins, 1/(ss2 * np.sqrt(2 * np.pi)) *
 np.exp( - (bins - ort2)**2 / (2 * ss1**2) ), linewidth=2, color='orange')
plt.show()


I could, if I have knew about them.


Пробовал удалять из манифестов ? Как ведет сборка через Эклипс ?


add newline


Are we sure that there are no games that have more than one Stock Market
menu?
I can't think of any off the top of my head. But I am gun shy after my
map orientation fiasco.

On 10/08/2015 02:58 PM, ajprog wrote:
> This is because the Tray is mostly obscured (IMO doesn't look good) and
> it isn't needed because the stock market uses a single tray.
>
> It is was more cleanup that I did when I went in to fix the 'In-Map' menu
>
> On 10/08/2015 03:54 PM, Richard Price wrote:
> >
> > Is this a fix to the previous material?
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672056>.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672129>.
>


To do mag weg.


Do you know why does Gregory horror show uses this strange address??


I could probably make the logic in here simpler, if people agree that adding this feature is a good idea. This was just the quick first cut, and I stopped once the tests were passing.


Think about how you could format these so they show up as dollars (dollar sign, two numbers after the dot).  This line and line 127 are too long - since you're in parentheses, you can move some of this onto the next line after any comma.


> ultimatt, can you tell me why true from the plugin is false in the test?

I'm sure sure what you are asking, but it appears that the functions you are referencing are not working exactly as you expect.


>>> class Foo(object):
...     def __init__(self):
...             self.a = 'a'
...
>>> hasattr(f, 'a')
True
>>> hasattr(f, 'b')
False

This can be written as

if hasattr(t, 'perfherder_artifact) and t.perfherder_artifact:
...

without the use of the try except block.


Initialising


These values could surest be better. Discussion with Cayorion.


Done


Uncomment  this


Lines 35 and 36 are still running after this - so the user has to exit twice.  You can fix this by getting rid of those lines and putting your main_menu() calls somewhere else - either inside your if logic or at the end of your other functions.


Wait what


Also, a few files had some weird names and git removed them because of file/destination requirements


for the quotes


Oops my bad. got it back


```mvn clean package [-Declipse-path=PathToEclipse]```
Если путь содержит пробелы, то кавычки ставятся вокруг параметра целиком, т.е.
```mvn clean package "-Declipse-path=C:\Path with spaces\Eclipse"```


This is preventing the image preview from showing up at all for me. #68


and yes, it was my mistake to use escaped len


Can you make this only print the most recent donation, not the total donation amount?


ahh ok.. I never even knew that.. I've hardly ever used tr() so apologies for muppet questions..


Why are you using a static variable instead of a stack variable?


A task kezelő olyan lenne, hogy beírod hogy mit kell megcsinálnod
mikor, meg mikor értesítsen, és felszinkronizálja felhőbe, hogy más
telefonokon is működjön/jelezzen, és lényegében ennyi. Az rft-es meg
létrehozhatsz projekteket és ahhoz megoldandó feladatokat,
hibabejelentéseket, meg stb, amit a többi csapattag lát, és értesíti
őket a készülék ha jön új cucc

uszalab <notifications@github.com> írta, 2015. 10. 08.:
> Peti, a taskkezelőn meg az utolsó 5leted alatt mit értesz pontosabban?
> 2015.10.08. 21:02 ezt írta ("alper061" <notifications@github.com>):
>
>> Oké, akkor ötletelgessünk :D csináljuk meg az office 2020-t :DDD
>> Komolyra fordítva valami olyat csináljunk akkor, amit esetleg még
>> használnánk is. Nekem mondjuk elkélne egy fasza naptár cloud
>> szinkronizációval. Vagy mondjuk egy esemény/taskkezelő, vagy
>> csináljunk egy mobilos ticket hibajegykezelőt rft-hez passzol is :D
>> Kezdetnek akkor ennyi hirtelen :D
>>
>> uszalab <notifications@github.com> írta, 2015. 10. 08.:
>> > Túl komolynak nem kell lennie, inkább valami irodai hasznos dolog
>> > legyen.
>> > Nem szoktam játszani:D
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13662131
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13670932>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GoldHiroz/ANDROIDRFTb/commit/f0b185824210921447f5a211873c1a8ae4c9e662#commitcomment-13671738


I don't know of any. But I am new to 18xx.

I don't know the "map orientation fiasco". But my opinion is not to have
something there on the possibility that it will be used in the future.
If there aren't any current game boxes that use it, then remove it. It
is easy to put back in (copy the functions over from the Map.js files)
if a new game box is introduced with multiple trays for the Stock Market.

On 10/08/2015 04:11 PM, Richard Price wrote:
> Are we sure that there are no games that have more than one Stock Market
> menu?
> I can't think of any off the top of my head. But I am gun shy after my
> map orientation fiasco.
>
> On 10/08/2015 02:58 PM, ajprog wrote:
> > This is because the Tray is mostly obscured (IMO doesn't look good) and
> > it isn't needed because the stock market uses a single tray.
> >
> > It is was more cleanup that I did when I went in to fix the 'In-Map'
> menu
> >
> > On 10/08/2015 03:54 PM, Richard Price wrote:
> > >
> > > Is this a fix to the previous material?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> >
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672056>.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672129>.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajprog/board18/commit/48fdf489897eaa3bdcdca8a297600ddd2787a612#commitcomment-13672386>.
>


Woot!


Why not implement this? It seems like it would be very easy.


That space


I'm not sure if I should trust you... don't you know you are on the develop branch 'ya fool?


74564912Issue4347BBRadConvHW (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Why is this 'next_id'?
Looks more like 'last_id'?
This seems like a risky strategy to me to assume this would be the correct ID
Since the info is *always* created as a component of the alert (correct?) then we should be able to get this as current.request.args[0]
Can always verify this is an ID via int() as a x-check


Yah, at least you are sure now.


<goal>replace</goal>
```goal``` это часть life-cycle плагина ```replacer```


What is the point of this enumeration?


This code only fix it on iOS, right?

I'm looking for the Portrait view on Android devices.


it's there again :D I see you are closer and really fast in it.. thank you for your hard work


Has top priority


include intel copyright here?


any chance we could change the naming ?
Example,
archer_aprun -> archer
archer -> archer_orte

Would avoid the user from having to specify the resource as "archer_aprun" + avoid changes in enmd.


Magic number in expression (1e6). Please change to a const or macro with a meaningful name.


Potentially dangerous.


До сборки там только ```.gitignore```, сделал, чтобы после ```clean```, оставался только он


@Tobschinski That only explains why most of my screenshots don't show values, and not why I have one that shows "100%" on the health bar and nothing on the energy bar. I'll have to check my TBC client's options again (custom UI now).


To fix the ims libs, all you need to do is remove .so;)


The two os* consts should probably be moved to https://github.com/bosun-monitor/bosun/blob/master/cmd/scollector/collectors/collectors.go#L30 with the others. The first is for metrics, and the second is for descriptions (both sorted alphabetically)


74564912Issue4347BBRadConvHW (wfbuhl) - 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)


I don't care -- this is just suppose to be a generic way to create tasks without having to provide task name info. I could define the bsp-dependent struct passed in to include a proper RTEMS object name.


Nit: which version of the spec are you using for section references? I see Section 5.4.1 referenced in the [current version](https://httpwg.github.io/specs/rfc7540.html#rfc.section.6.6) of the spec.


I'm getting two flake8 errors - this should be indented to match the last parenthesis, and there is a space in your print statement on line 82.


Jaki soft wgrany?

czw., 8 paź 2015, 21:03 mihail1 użytkownik <notifications@github.com>
napisał:

> Witam
> No i przy przełączaniu kanału gdy ustawie środek z bigpicon jest tylko
> cyfra kanału nazwa żółtymi literami i tak gdzie ma być ikona jest puste
> pole:(
> Sorki za moją nie wiedze ale dopiero raczkuje w temacie.
>
> Pozdrawiam
> Michał
> 08.10.2015 8:55 PM "Michal Kawaler" <michal.kawaler@gmail.com> napisał(a):
>
> > A widzsz tego nie wiedziałem:/ a ja durny że starych picon nazwy
> > kopiowałem:d tak poradził em sobie właśnie z tym i wrzuciłem duże do
> picon,
> > tylko gdy chciałbym ustawić coś z klasycznymi piconami są one wtedy na
> > sztywno przeźroczyste.
> >
> > Wygląda to mniej więcej jak w załączniku.
> > 08.10.2015 8:22 PM "herpoi" <notifications@github.com> napisał(a):
> >
> >> Jak to ręcznie przypisywać kanały do ikon? Przecież OpenPli już od
> niemal
> >> roku rozpoznaje pikony o nazwach kanałów.
> >> InfinityHD już od dość dawna widzi tylko pikony w
> >> /usr/share/enigma2/picon/. Wrzuć tam po prostu duże pikony zamiast
> małych
> >> i
> >> będzie po sprawie.
> >>
> >> W dniu czwartek, 8 października 2015 mihail1 <notifications@github.com>
> >> napisał(a):
> >>
> >> > Robisz super picony;) szkoda, że się nie znam na tym i muszę ręcznie
> >> > przypisywać kanały do ikon.
> >> > W Twojej wersji infinityHD mój nBox nie widzi picon w katalogu
> >> > /usr/share/enigma2/BigPicon/picon/ widzi tylko z tego
> >> > /usr/share/enigma2/picon/. W związku z tym jak chce gdzieś duże picony
> >> to
> >> > dekoder skaluje te małe standardowe i wszystko wygląda jak jeden
> wielki
> >> > piksel
> >> >
> >> > —
> >> > Reply to this email directly or view it on GitHub
> >> > <
> >>
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13668443
> >> >
> >> > .
> >> >
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670139
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670945>
> .
>


Unnecessary extend


i get the error... its bad that its not possible to get kind() on ipaddr.js parseCIRD object...


Revert to old behavior


No -- everything is identified via osThreadId which is actually an rtems_id


I'll change it.


I don't think basic lexical sorting will work?


Good point -- I'll change it for all the non-implemented functions that return a status code.


This patch allow the definition of multiple class in schema.yml.

[...]
objects:
    zacacia_platform:
        type: Platform
        class:
            - 'top'
            - 'organizationalRole'
            - 'zacaciaPlatform'
[...]


Why not `return "", "", err` ?


Causes issue #62?


I will investigate.


Explain why


oh, never mind


add_examples (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.5: Build Failed

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


Well on the system image they have them linked to there from lib so I figured I would just put them there anyway.


"also masking out top part which is a "Mode" cache stuff, we don't care :)"


i.e. 1.2 > 1.10 when 1.10 is the newer version


Nevermind, I'm looking at the HTTP/2 spec. Carry on.


add_examples (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.5-LibC++: 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)


> This would probably be confusing to co-developers sharing this repository. It also leads to problems like the ones we discussed before about failing to use the tools properly to afford proper inspection of change sets over time.

>  The consequences of these bad habits may be acceptable if one is working alone, but team members come to expect certain conventions to be followed. When they are not, the team's efforts can be disrupted (sometimes significantly).

Fully agree

> Do you have much experience using revision control in a team environment? For example, have you used topic branches to propose or submit changes?

Sure

 Do you do so regularly?

The last 3 years of my life, I've been working on a company (CAIRS) which I couldn't even write tests, because the CEO was always under pressure to finish up all features he always promises to client .. from him, tests are expensive

They don't uses github, any methodologies, etc ... all things of this kind were my effort there ...  all with no success, there is no way throw a war with 30 indians when you are alone and when your CEO and CIO are permissive and negligent ....

Not blaming no one, not point fingers ... just giving you an idea

But in a short answer, no, I'm not doing it regulary


yes... we can change it in the fugutre to just MAIL instead of GMAIL


:)

2015-10-08 22:23 GMT+02:00 Daniel Sikorski <notifications@github.com>:

> it's there again :D I see you are closer and really fast in it.. thank you
> for your hard work
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/manups4e/android_device_lge_d855/commit/aaa5c74bbfb7f3831bc5f7b2160041956b97dc7b#commitcomment-13672609>
> .
>


Lets discuss this in the meeting.


Maybe add a comment saying it expects "resource" and "group/resource" will result in an error.


@blesh couldn't we change this in the [BidirectionalSubject](https://github.com/ReactiveX/RxJS/blob/master/src/Subject.ts#L179) instead? Seems like overkill to add an extra conditional to every `_subscribe` along the chain.


lgtm.


It does not improve metrics.  It costs ~0.6% in rate for intra only (subset1) and 1.2% in rate for video (ntt-short-1).  It would improve metrics if the numbers were negative.


and added destructor


OK I will do it this way:
Load my changes.
Do the merge of your pull request.
Fetch it onto my test box.
Test that it works with non iOS web browsers. I leave the iOS testing to
you.
I will discuss any concerns that I have with you and after they are resolved
I will create a new release with all of these changes and close the
relevant
bugzilla issues. I will then upload the new release to demo1.
How does that sound?

I am going to assume [trembling as I do so] that one SM menu is enough.

On 10/08/2015 03:06 PM, ajprog wrote:
> > 3. This changes a lot of code. I will use it as is if you want, but I
> > would be more comfortable with a staged implementation. Perhaps the
> > main and tray menus first followed by the rest. Or even splitting the
> > main and tray menus. Am I being too cautious?
> >
> It is needed as a whole to be usable by iOS. If you split it up, iOS
> users won't be able to use it until the final piece is implemented.
> If you want to be cautious, upload your changes to GitHub, then merge in
> my Pull Request. Fetch it onto your test box and use/test it for a while
> till you are comfortable with it.


You should source the blippd.opts file that gets installed in /etc/sysconfig/blippd instead of hardcoding the config file here.  Other scripts (like ibp_configure.py) update that options file to include the appropriate blipp config path.  You can source it like the functions file above to pull in $OPTIONS

The other flags should be -D and -l <logfile> to properly daemonize the process.


Thanks :+1:


It has few differences from cpu/6502/net/ethernet-drv.c.


Also, I'm not considering several aspects here in this repository, I'm strictly following what you ask. The initial requirement is far way from the actually scenario :)

if you provide me your own list of the best practices that you wait from my side, I will really appreciate it and follow.


only checking `name`? what about `organization`


Understood, and thanks for the explanation.


@palvarez89 fixed


This is still not quite right; we are leaking during error cases in `test/runTests` despite passing #19


are these really signed?


same here?


:+1: let's merge it!


and especially here?


1f313f1eb07310dc4ab4c9b8506d6117f83af33a looks good now - thanks!


Possible division by zero.


Why do we not hash the full address here?


Is this something that should be removed? It get's updated to `include_once __DIR__.'/bootstrap.php.cache';` in app.php

Was having issues so I added that line in here under $loader, not sure if this is my own fault or a real issue.


why do we hash the peer here? what's going on? /me is confused...


Need to change this line to map[y][x] from map[y][y] as soon as I get home but I can't from work and I'm losing my mind because of it. Didn't catch it before I left


oh... there's a fall through....

still wondering what the peer thing is about...


That sounds good to me.

What site is demo1? I have heard of Prod1, dev1, and demo2. Which site
is use mostly for active/real games?

On 10/08/2015 04:37 PM, Richard Price wrote:
> OK I will do it this way:
> Load my changes.
> Do the merge of your pull request.
> Fetch it onto my test box.
> Test that it works with non iOS web browsers. I leave the iOS testing to
> you.
> I will discuss any concerns that I have with you and after they are
> resolved
> I will create a new release with all of these changes and close the
> relevant
> bugzilla issues. I will then upload the new release to demo1.
> How does that sound?
>
> I am going to assume [trembling as I do so] that one SM menu is enough.
>
>
> On 10/08/2015 03:06 PM, ajprog wrote:
> > > 3. This changes a lot of code. I will use it as is if you want, but I
> > > would be more comfortable with a staged implementation. Perhaps the
> > > main and tray menus first followed by the rest. Or even splitting the
> > > main and tray menus. Am I being too cautious?
> > >
> > It is needed as a whole to be usable by iOS. If you split it up, iOS
> > users won't be able to use it until the final piece is implemented.
> > If you want to be cautious, upload your changes to GitHub, then merge in
> > my Pull Request. Fetch it onto your test box and use/test it for a while
> > till you are comfortable with it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajprog/board18/commit/47766c2e75614d2a16cbeb93f077b7c0ca0c5837#commitcomment-13672929>.
>


we don't have any sockets other than AF_INET and AF_INET6 i presume... maybe hit an assert here? sounds wrong to hash basically nothing for them...


How about:
- create class `AvroHdfsTimePartitionedWithRecordCountsWriter.FilenameFormat implements FileNameWithRecordCountFormat`. `AvroHdfsTimePartitionedWithRecordCountsWriter` will use this class for file names.
- create another class `AvroKeyCompactorOutputCommitter.FilenameFormat implements FileNameWithRecordCountFormat`, which will be used for compaction output file names.
- Since files in "late" folder adds an additional component from the original file name, use an implementation of `FileNameWithRecordCountFormat` that wraps around another `FileNameWithRecordCountFormat`.


This order is not stable regarding endianess,, is that intended?


especially given that the ipv4 order does honour endianess before ordering...


@bobzoller FYI.  Didn't install dropbox via brew yet, tho seems like that'd be nice.


add_examples (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.6-AddressSanitizer: OK (376 of 376 tests passed)

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


This is going to screw with cycle mode hack in add_surface_entity_tool::cycle_mode()


It's weird to have an interface in `gobblin-utility`. You can put these classes in `gobblin-api/gobblin.util`.


Were you addressing an issue with this?
Why did you +x to the solution as well?


Store input from console to myData.json as String


This change does not resolve #68


testing


I suspect I must have a bug in my implementation for this to have failed so catastrophically.


Commit message should be "adds comments"


I agree, when changing this I kind of blindly changed it to those because I already used them on my dev server a couple of times, but `[asrent]` and `[asbuy]` are indeed better options. Changed it now: https://github.com/NLthijs48/AreaShop/commit/eb21b9567e0b6894b50576228d0399aec8c22836


Lol. No I can't remove you. :p don't want to.
And stop with the "last commit commit of your life". You're a coder, like it or not.


Mo-no-id, Mo-no-id :D


No problem:)


https://github.com/bwidawsk/engineers-nightmare/blob/flashlight/src/component/component_system_manager.cc#L236

That ^^ is probably screwing with your light. I'd move those three lines (234-236) below the conditional they're above.

Also, you may need to investigate skipping the conditional we have on previewing tools. Currently we don't call preview if the initial tool raycast didn't hit. I'd try just always calling preview and see how that goes.
This whole setup is bogus, but we can at least tweak it to work for now.


Shouldn't this be `plan.snapshot_plan or plan`? If we are allowing snapshot then we need the snapshot plan (provided one exists)


Shouldn't use `COUNT_INDEX`, because, e.g., `AvroHdfsTimePartitionedWithRecordCountsWriter` just adds row count to the files written by `AvroHdfsTimePartitionedWriter`, like `originalFileName.rowCount.avro`. No assumption should be made on how many dots `originalFileName` contains.


@apg i would assume that the env variable will override the necessary resolution, rather than be a fail-safe?


yes ... there is a counter, which can be used on NAMESPACE level ... the `resource._closed` may be used as check on `NAMESPACE.resource` level .... not sure if it is necessary, how I told before, my intention was to maintain the API compatibility

in fact, if we destroy the resource using the close(), your test will not pass, because getExpensiveResource() and all other mehods and properties does not longer exists


ref #120


Tror att detta borde hanteras av varje widget istället. Flytta till sopmaker som lyssnar på eventen och stänger sig själv.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/22038/testReport/) for this commit**


My bad. I meant to say dev1.
demo1 and demo2 are both defunct.

On 10/08/2015 03:45 PM, ajprog wrote:
>
> What site is demo1? I have heard of Prod1, dev1, and demo2. Which site
> is use mostly for active/real games?


in no moment there were some requirement to "clear" the code by itself ... then if there is not specifying to remove things like that, and if you ask me to pass your test, I presume I should to maintain the object structure, even clearing the original references ...


There is no reason to sleep here, if connection is NIL its either because we are out of the IO.select loop because of something weird OR usually its because we are currently shutting down which will be taken care of if its the case in the next iteration.


shhhhhhh


We need to implement the same notion of equality that the kernel does. And the kernel does indeed consider addresses different if their peer prefixes are different:

# ip address add 1.1.1.1 peer 1.1.1.2/24 dev br0
# ip address add 1.1.1.1 peer 2.1.1.2/24 dev br0
# ip address add 1.1.1.1 peer 2.1.1.3/24 dev br0
RTNETLINK answers: File exists
# ip address show dev br0
5: br0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default
    link/ether 4e:3b:2d:10:59:2d brd ff:ff:ff:ff:ff:ff
    inet 1.1.1.1 peer 1.1.1.2/24 scope global br0
       valid_lft forever preferred_lft forever
    inet 1.1.1.1 peer 2.1.1.2/24 scope global br0
       valid_lft forever preferred_lft forever


Remove member initializers from function declarations in the header file.
Also add semicolons at the end of declaration.


You need to update your version of `assaytools` by installing from the github repo:
```bash
python setup.py install
```
I had fixed this error in my last PR, but the `assaytools` conda package hasn't been updated yet.


@grepory @snorecone what should be done about testing? We fail hard when etcd doesn't exist now


Your second param has an invalid origin.


This, and the others, should be
```c++
BinarySearchTree<Comparable>::BinarySearchTree(): root{ nullptr }
```


add_examples (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.6-ThreadSanitizer: OK (376 of 376 tests passed)

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


>all hail our release manager

Hear hear!


Good


it's double-optional


here it should be `lowerStorebi`, right?


I would use cast in 2 of the callsites, and `if (... instaceof ...)` in the other.


Remove?


is this protocol-agnostic for a reason?


I'd suggest to move that entire block right before the "service_start" block to maintain some logic in the ordering.


Just saying: this doesn't influence anything, but it reads better in my opinion.


@deads2k I remember talking with you about this - are we certain we want to fail the job at this point?


`callm` vasm implementation can be completely removed. We lower it to load+callr, right?


add_examples (lefticus) - x86_64-Linux-Ubuntu-15.04-gcc-4.8: OK (376 of 376 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)