Could we change the name to a more straightforward one: copyFilesFromHdfs? Import can imply some conversion taking place.


The same here, maybe readAvroDatastoresFromHdfs?


s/send/sent


agreed


s/settings/setting


@ArmandH develop is usually merged into master once a week.


s/names/named


hi, pls send me yr and the classe's contact info so we mach keep it touch keepIn.js
:D
thanks if u do!
theda 845 596 2455 poetbridge@gmail.com
theda takeapictureofamonsterr on facebook


이건 아닌거 같습니다.


Fine by me too. I don't mind either if we keep "awakeness", it's understable enough :)


「研究」の30％程度を翻訳。
訳文を複数変更。
訳文の誤字や誤訳を複数修正。


Do you mean something like `CONFIG_OBD_CALC_SIGNATURE` (TRUE/FALSE)?


changed


done


@dexX7 How can we make `OmniValue`s when this object is created if we don't know the `PropertyType`?


Grammar nit.


@dexX7 Any thoughts?


OK, we actually saved some lines here.


We doing this wrong...


It takes a fairly long time <15 seconds or so.


@dviramontes, @wdoug: I'm not sure what to do with `redirectTo` at this point. It appears as though `this.props.login(email, password);` might be using code in `Application.js` to redirect, but I don't really understand how to pass this information through to there... Or if I even have to.


@joergoster
If you look just a couple of lines below you will see that update_pv() gets called by any thread.  If we let only the main thread clear easy move we run a chance of making easy move blunders.


@dexX7 We have a choice here between using a strong type and matching the naming convention of the JSON...


@sebque Una consulta, sera necesario saber el ultimoAgregado?? no se puede sacar la cantidad con el metodo del arraylist?


Wait, /quit tells it to nuke everything it has done?  That seems deeply dangerous.  Why?


>> do you have your own github fork of the repo?

Hu ?

>> do you have a local clone of your fork repo?

Stop talking greek, please !!!!

>> the general method that github uses is the two above and then you edit your
local repo's .git/config to add an "upstream" entity and point it to this master
repo...

By hand ???

>> Then you... git pull upstream

Oh my GOSH ! I swear to you that tomorrow i will not do any blend thing. I will learn git and what that strange commands mean.

>> and... git push origin

I am lost, dude.

By the way, i shared all i done on gdrive.

>> I just pushed your additions @thevirtualfer.

Hehehehe ! Sorry for sharing this way, man. But, at least you got the mods.

>> I also pushed the first attempt to rotate the skis. Please test and I am open to any improvements.

Look. To me it is ok. But i never saw a real plane with skis. Here is pure amazon forest and very hot the whole year.

>> I'm not real happy with it.

I am very happy. But if you see possible improvements, who am i to say the contrary ?

Cheers, bro.


looks good


The plugin was recording only global lockTime and totalTime which were not being used in any of the charts. Instead, I should capture all the global lock metrics for version 2.x too to make it report the same metrics as 3.x The metrics would be more consistent in all the hosts, irrespective of the mongo version deployed.


Doc LG.  There's a LOT here, so I want to go over it at some point, but don't block on me.


looks like, damn intellij.


It doesn't hurt to do it twice! :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.639](http://teamcity.monogame.net/viewLog.html?buildId=10443&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Thank you!!


`host = NULL`! That's what I was missing and was a bit surprised you asked for `_cleanup_`  here. I'll fix that.


should the value be populated always by the caller/lib? the implementation should not need to know what is the default.


I think it's safe(I actually think this way is more similar to the original behavior) but the test is showing it is performing worse.  I reduced priority to -1.


OK


You know it already worked right? But this is maybe more obvious..


eline koluna sağlık, builddep teki 5.4.1 leri de düzeltmişsin.


This 'static' here, I'm not sure if it is the best idea... looks strange in the client of this class. Maybe it should be moved to another class, let's talk about it a bit at our sprint meeting.


Evitar usar nombres como «cosa» en el código.


My git(hub) skills are rather poor.  I know you can run tests using the SHA ID as a branch name but I don't know if it has to come from your own repo.


You don't need to edit your `.git/config` file.

What you can do is execute `git remote -v`. It will usually show you one remote: origin. It should point to https://github.com/Juanvvc/c172p-detailed.git. A remote is a location from which git will pull/push code.

If you work like @dany93 for example, then you have two remotes: origin and upstream. Upstream points to https://github.com/Juanvvc/c172p-detailed.git and origin points to https://github.com/dany93/c172p-detailed.git. In that case he pushes to his own fork on github. He gets new code from us via `git pull upstream master` (if he wants to get the latest code from the master branch).

If you have one remote, you can just do `git pull` which should basically do `git pull origin <current branch>`

If I look at https://github.com/thevirtualfer?tab=repositories, I see you have a fork, so you can work like @dany93 and add a new remote and push to your fork instead. That way you can't screw up (or do weird things) our repository if you're not an expert in git :smile:

So the question is: do you want to work directly on our repository (Juanvvc/c172p-detailed) or via yours (thevirtualfer/c172p-detailed)? (And paste the output of `git remote v` in your answer)


@renchak There seems to be a mergeconflict here


How could it have worked??


response


Should merge commits at gerrit to build ;)


done


Yes


I'm very happy to help!  And yes, choose a good editor.  I figured out the other day that I've used the same editor (Emacs) nearly every day for the last 23 years.  Having good tools makes a big difference to productivity.  If you haven't really got a good day-to-day editor that you use, I'd suggest trying a few different ones for a day or two to work out what you like -- some people like vi, some people like Emacs, some people like things like Sublime Text, and so on.  There are a lot of possible choices.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.639](http://teamcity.monogame.net/viewLog.html?buildId=10443&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:20:44


This should be false?


I think we should not use fc.follow_symlinks. This should be "true" instead. This is the initialization of the find emulator and this is done only once.


You might want to move "MAX_SIZE_OF_VM_NAME" to vm.h


yay!


I just pushed a commit adding the ski rotation to the flight recorder (no aliases for mp). I wanted to get it in there for ease in testing the rotation code. After looking at it in the recorder from different angles, it's not as bad as I first thought. In fact, with some adjustments to the skis back edge contact point, it might actually work well enough. That is until onox shows me a better way forward, ha!


@thevirtualfer, all you work, including your latest, is also pushed to ski branch.


these two shouldn't need to be there?


@jcfr Thanks.  I started down the path of changing to std::string, but then determined that two separate patches (one that removed the compiler warning, and one that changes to std::string) would be the better option.

It should be a quick refactoring.


First comment was right after all. I have a separate problem with some decisions not taken immediately, but that may be either an other problem in simpleflow, or most likely SWF not being consistent.

FYI I dropped the majority of this commit as it's a breaking change, and will reintroduce something similar later. Things happen in https://github.com/botify-labs/simpleflow/tree/bugfix/fix-optional-activities for now.


I think that length is more suitable than size.
Anyway in this case, just define LENGTH. It is enough for us.
Take a look at the "char name[MAX_SIZE_OF_VM_NAME];" in vm.h


If the intent is to have a chart for Avg wait time for a r/R/w/W lock, the formula for which would be timeAcquiring/acquireCount, shouldn't the count and time both be gauge?


yup, should be gauge.


DO NOT USE VALUE IN ENUM.
You can control state with STRING.


Ditto.


Don't chagne the API please, its already released


this is beautiful


Do we need to distinguish type of VM?
Please let me know your intention.


I am fixing it.


which exception class do you mean here ... could you add id as a comment pls, thanks


Awesome! That's what I use. :+1:


I see you're doing polling in Nasal again :sob: Didn't you try some mathematical expressions in JSBSim or `<autopilot>`?


========== Build: 12 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


s/INITIALIZED->RUNNING->SLEEP->WAIT/INITIALIZING->RUNNING->SLEEPING->WAITINIG/


Oh, good catch!


:+1: danke! wirdi im verlauf vo de wuche no ibaue


JC,

I used the clang compiler on Mac.  I have been compiling with std=c++11 in order to prepare for the future of being able to use C++11.

NOTE:  Slicer now builds completely with C++11 on Mac.

 Hans

FROM ITK COMMIT LOG:

commit 671d04333f5c43c82bc9aabab92f2d66140e653e

Author: Hans Johnson <hans-johnson@uiowa.edu>

Date:   Sat Dec 13 21:29:09 2014 -0600

    COMP: Add c++11 override designation

    C++11 provides the override keyword to assit with identifing

    functions that are meant to be override the base class

    implementation.  This refactoring adds the override keyword

    in order to enhance compile time identification of

    errors that may be introduced during refactoring.

    The following substitutions were heavily used during

    the code instrumentation of the override keyword:

    :s/)$/) ITK_OVERRIDE/ge

    :s/);$/) ITK_OVERRIDE;/ge

    :s/) *const *;$/) const ITK_OVERRIDE;/ge

    :s/const *{/const ITK_OVERRIDE {/ge

    :s/) *const$/) const ITK_OVERRIDE/ge

    :s/() *{}$/() ITK_OVERRIDE {}/ge

    :s/) = 0;$/) ITK_OVERRIDE = 0;/ge

    :s/) *const *= *0;$/) const ITK_OVERRIDE = 0;/ge

    :s#();  *//#() ITK_OVERRIDE;  //#ge

    Change-Id: I199c677b6420831ada5b009fc3d222b00687321a

From:  Jean-Christophe Fillion-Robin
Reply-To:  hjmjohnson/Slicer
Date:  Wednesday, October 7, 2015 at 3:34 AM
To:  hjmjohnson/Slicer
Cc:  Hans Johnson
Subject:  Re: [Slicer] COMP: Add ITK_OVERRIDE designations for C++11 (8bdf606)

Thanks Hans.

Reviewing these changes also identified some dead code

Are you talking about the commented code in ResampleDTIVolume/itkWarpTransform3D.txx ?

In the commit message, what did you mean by:
[...] override a parent class an provide
exceptions or no action, but infact did not override any
parent virtual function.
Finally, did you use some vim or python code to automatically identify the ITK virtual method and update their usage in Slicer ?

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


@onox said :

>> A remote is a location from which git will pull/push code.

Ok. Putting it on my glossary.

I understand that there will be a place from where i need pull, or clone. It is called "remote". Now i know.

>> What you can do is execute git remote -v. It will usually show you one remote: origin.

A bit confuse, but i will try to live with it. There is a "remote" also called "origin". What is the difference ? Never mind. I will live with these two names.

>> If you work like @dany93 for example, then you have two remotes: origin and upstream.

Ahhhh !
Two remotes ? Ok. I am getting it. But why two remotes ? Who is upstream ?

>>  Upstream points to https://github.com/Juanvvc/c172p-detailed.git and origin points to https://github.com/dany93/c172p-detailed.git

Ahhhhh, again !

The remote "origin" will be my own remote copy of the original remote "upstream". Is it ? How i can get that copy, like https://github.com/thevirtualfer/c172p-detailed.git ?

>> In that case he pushes to his own fork on github.

Ahhh, by the third time !

My copy is a remote, called origin, and is a fork. I saw that word somewhere. the "fork" word.

But @onox. After i got my own remote (called 'origin') the original remote (called 'upstream') will be modified. How i import that mods ? Creating a new fork ?

>> He gets new code from us via git pull upstream master

Ok. It goes to my local, no ? And now i need to push that mods to my remote, no ? It is a simple git push, or there are some others tricks ?

>> If I look at https://github.com/thevirtualfer?tab=repositories, I see you have a fork, so you can work like @dany93 and add a new remote and push to your fork instead.

Do i have a fork ? Guy !!!!

Ahhhh, one more time.

I remember now where i saw that word. Was on the main page. I clicked there and, to me, it was in vain. Ok. I have a remote (called origin) that is a 'fork' from the original remote (called upstream).

>> So the question is: do you want to work directly on our repository (Juanvvc/c172p-detailed) or via yours (thevirtualfer/c172p-detailed)?

I think i will mess my remote. But look. If i mess it so much, a 'git pull upstream master' will clean my local, i guess. And a 'git push' to my remote will clean it, will not ?

>> Paste the output of 'git remote -v'

Ok. It is :

origin	git://github.com/Juanvvc/c172p-detailed.git (fetch)
origin	git://github.com/Juanvvc/c172p-detailed.git (push)

Thanks for the help again, @onox


Root paths are generally used by pages such as `/login` and `/about`. Resources are usually nested such as `/videos/:id`.


I think this is wrong for loops:

$ mkdir -p d/d
$ (cd d/d && ln -s .. l)
$ find -L d
d
d/d
find: File system loop detected; `d/d/l' is part of the same file system loop as `d'.
$ ./find_test find -L d


LGTM


Brady
We didnt notice that this could be executed as a parameter list. So tried declaring the variables in the script before running and executed it.
Just now tested using parameter list with $1, $2,$3,$4 variable names and it works good. Has committed the parameter list execution script with the intended changes, here is the link: https://github.com/visolve-openemr/openemr420/commit/676d95ad522bf6b1e5609a1f6e223ea3bf6fa6f4


This happens. https://travis-ci.org/google/kati/builds/84017040#L577


SessionFactory rework commited to Rework_SessionFactory branch.
I am not quite understand how should I handle exception which can be thrown in SessionFactory constructor.


Covention for variable definition, do not allow plural.


hi,千里码的用户,欢迎来到github。 这里是世界上最大的开源代码托管社区,算是一个程序员的宝藏了。php,ruby,python,apache等众多明星项目的代码都托管在这里。你可以完全免费的下载下来,阅读,编译,调试,更改,学习。当你通过不断的学习理解，你也可以以提交代码的方式回馈社区。 在这里,我希望你可以掌握git&github的基本用法,了解基本的操作命令。

每个项目都有个基本的 README.md 说明文件,现在你看到的就是本项目的 README.md 文件。

请fork该项目 https://github.com/qlcoder-oreo/welcome2git.git ,并将你的name添加在README.md 最后的通过列表,我们会在第一时间merge你的提交,并设置你通过此题。

以下是通过列表,恭喜以下coder熟练掌握的git&github的基本使用方法。请自行修改本文件,将你的名字添加至改列表最后:

shinian
oreo
jiaye
mdep
orinew
zhaoxj
shujia2012
stryandk
zzliux
墨池
情趣复制你我
zayne
名码农柯南
端着木鱼敲代码
白青黑
zhengtaishuai
乾坤数据
Tobe
ltm1994
thinkfree1992
mindthink
hong
wentao_tang
qlctangsir
Luck Lamb
cy


Mixed tabs and spaces! If you use VS I'd suggest using [Productivity Power Tools](https://visualstudiogallery.msdn.microsoft.com/34ebc6a2-2777-421d-8914-e29c1dfa7f5d) extension to avoid this. To satisfy my OCD I also always enable "View White Space"!


thinking about this. I think it is okay to log error. But i dont think returning a warning for a filter is not appropriate. Also this is consistent with other filter operations that we have today for containers and such.


Ditto.


Code seems to be good, but with some C++ issues.
Firstly you can not create a vector of referenzes, you must use pointers intead of this.
You should really declar every const member function as const, other wise you would not be able to read Data from a const object.
And you mix up the ElementWrapper with the Element Object.

I will try to fix these issues today, but i cannot garante anythinig


@bossmanpl Wrzuciłem pierwszą bardzo koncepcyjną wersję, na końcu obrazek trzeba 2x kliknąć by się wypełniło. Zerknij czy jeśli chodzi o rysowanie kształtu jest ok.


Arrow functions have a lexical `this`, meaning the `this` on this line now refers to the global `this`, so this won’t work.


Same here.


What is the difference between _xxx and xxx?


Good, I'm glad you figured that out already. You can safely ignore my last comment obviously :smile:


Looks like a candidate for a template literal?


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/27b23eb9a0dc9124289e5f9fa8c9d7d3004e7b5d


merging basho/riak_cs/bugfix/dvv-for-rt = 27b23eb9 into borshop-integration-1259-bugfix/dvv-for-rt


basho/riak_cs/bugfix/dvv-for-rt = 27b23eb9 merged ok, testing candidate = 3c1d06b6


Why did you change this from if fromStyle == toStyle ? You end up with two conditions instead of one.


Do you want to declare vCPU like a double pointer?


You can save your created categories above to variables then use those variables instead of calling `Category.find_by`.


Please remove trailing spaces.


You don't need a blank line in the structure.


What do you mean '18'?


you could make count a gauge as well as a counter

I don't think an average over the lifetime of the process is particularly useful because the later samples have less weight than the earlier ones.

you could also do an average over the past hour (say) by summing acquireCount over an hour and subtracting time-shifted timeAcquiringMicros from itself (not great for an hour after restart)


Where are u printing the ouputGeometry result ?


He should maybe consider something kinda like this:

`Object.assign(Array.prototype, { pick() {...}, pickRemove() {...} })`


Command tested in game and working! SCRIPT_COMMAND_FOLLOW also working.


@onox

I figured out that 'origin' and 'upstream' are pointer names. I can set it to where i want. Is it true ? Or i can set only the 'origin' pointer ?


sorry, I meant timeAcquiringMicros should be a gauge


You are right, my fault. Didn't notice this, because originally I only fixed one condition. Then I realized the other one crashes as well, and returned nil for both.

So now we can merge the conditions again. Sorry about that. I'll send an update soon, unless you are faster :-)


regs32_t is not right because size of register is changed by architrecture.
s/reg32_t/uint32_t/.


Seems correct.


Wrong file name.
s/VCPU_REGS/ARMV7_REGS/.


Could you provide a comment why are you doing this?


Thanks, fixed in 949bda.


@DefinitlyEvil I have school and shit. Maybe try pushing a change to one of the startup console messages to the glowstone repo to test? And fyi if you did the command above to push do

git push http://github.com/DragonetMC/Dragonet.git
And press enter.


An I think the checkNeutral() and the getE() Methods were good, because the bool getE(), can not actually return which element is neutreal.


Simply reverted in 949bda7071c07f698e43a3d75d77167daeafb05b


@onox

Why 'git pull upstream master' ? Who is 'master' ?


Reverted this in 949bda7071c07f698e43a3d75d77167daeafb05b


I'd prefer the more formal:
`Since this overload is a generalized case of the standard one it will also work fine in plain sequential mode.`


Do not need to use core and gpr in the same line.
Choose an one word such as core_regs or gprs.
I prefer core_regs.


@webbedspace Cool, I'll take a look at `Object.assign`, I'm brand new to most of this...


It did work because it treated the string as a single-item array with the string as the item but it may be hard to understand the code so that could justify your change.


When transitioning back to the original screen, my view was offset by 64px and partially hidden unter the navigation bar (and it also had the height increased by 64px).


There is a condition on line 33, the FAQ page is opened only after upgrade which is once per like two months!


Yeah, definitely. I'll make this change immediately. There are so many new tools, I forget what's at my disposal...


+1.Flappy Bird : No matter the current velocity, the bird will gain the same amount of height when the screen is tapped (T/F)T
+2.The bird accelerates due to gravity; i.e. vertical velocity is always increasing downwards.(T/F)T
+3.The rotation of the bird is correlated to its vertical velocity. Animation (flapping) only occurs when moving upwards. (T/F)T
 4.counter strike : kecepatan lari saat membawa knife adalah 250 units/seconds (T/F)T
 5.kecepatan lari saat membawa pistol adalah 245 units/seconds (T/F)F
 6.kecepatan lari saat membawa awp adalah 210 units/seconds (T/F)T
-
+link 1-3 http://www.kilobolt.com/day-1-flappy-bird---an-in-depth-analysis.html
http://gaming.stackexchange.com/questions/114991/how-to-speed-up-your-movement-in-counter-strike


You should use quote_plus() instead, quote() does not encode spaces and '/' (then you don't have to specify a blank safe argument).


When you attach a comment the end of code in single line, please add a space before and after comment directive.
e.g. ; // Only guest_switchto
Anyway, we don't need a comment in this line.


Period is a time, not sche32_t.


While I agree, I still think we should *always* do due diligence and discuss if it isn't a clear cut decision.

Turns out that typetools 0.4.3 is binary compatible with 0.4.1, so it's a no brain decision to backport it.


id_from_vmcb is the same as VMID, isn't it?


Use braces when you write conditional statement.


#include <openssl/evp.h> is required


@alex-ab thanks!


Классы achievments и photos используются в качестве контейнеров. Думаю их тоже стоит сделать частью формы, т.к. это не отдельные блоки. Что нибудь вроде form__container


s/core_gprs/core_regs/ or s/core_gprs/c_regs/.,
s/gp_regs/regs/


> Why 'git pull upstream master' ? Who is 'master' ?

master is the branch name.

@thevirtualfer Forks are OK. It just means you make a copy of a repository. The point is that eventually changes in your fork are copied to the original repository via Pull Requests (PR). That's how open source collaboration works on github (and other services).

OK, so you have this:

```
origin git://github.com/Juanvvc/c172p-detailed.git (fetch)
origin git://github.com/Juanvvc/c172p-detailed.git (push)
```

So you might want to do:

`git remote rename origin upstream`
`git remote add origin git://github.com/thevirtualfer/c172p-detailed.git`

Then push to origin. Check in which branch you are with `git branch`. You'll see a `*` in front of the active branch. Can you show me your `git branch` output?


So after banging away on this for couple hours, and being force fed the StandardCharsets as a hint in javadocs, I ended up with something very close to your spoiler. Which flipped my memory switch and i ended up using your spoiler code anyway.


You didn't change it in the README.


Again, great idea. Addressed this in 1c1e5833ca33f2b839a443bfe550f06cb0f937ad


При верстке по БЭМ лучше всего каждый блок БЭМ описывать в отдельном файле. Таким образом получается четкое разделение.


Hello I guess this is related with the asset pipeline issue. The current version do not work in production mode. Could not find the ckeditor/init.js  in production mode.


Most of the time `local_address_excess` is **4**.
Using the following results in 5.5% less scan time when searching for 0 in zeroed memory.
````
            if (local_address_excess != 4)
                memset(local_address_beyond_last_element(swath), 0, local_address_excess);
            else
                *(int32_t *)local_address_beyond_last_element(swath) = 0;
````


Это также нужно стилизовать через классы. БЭМ старается избегать стилизования по тегу.


Btw, if you work in your own fork on github, @wlbragg can still directly pull in your code with `git pull https://github.com/thevirtualfer/c172p-detailed.git <branch name>`.


Will you use array or pointer?


This is all very ugly. I know that. I know(or think I know) that i need to use HashMap, but I have not figured out just how yet.


HELLO


sure i had in mind that we (on cleanup) creating a thrid importer (like copy this file to castxml.rb)


Hi,

The wpa_supplicant in mesh mode has HT-40 enabled by default (default eap_workaroud).
If I want to disable HT-40 then how should I go about?

My wpa_mesh.conf file is something like this:

#disable_ht40:1       this gives an error
network={
        ssid="CDOT-MESH"
        mode=5
        frequency=2412    # frequency=2437 for channel 1
                          # frequency=2437 for channel 6
        key_mgmt=SAE
        psk="cdot1234"
}
OR

The config_ssid.h has a field to diable it : #define DEFAULT_DISABLE_HT40 1
should I set this and recompile ?

On Thu, 01 Oct 2015 17:54:26 -0700, Chun-Yeow wrote
> The proper way to bring down your mesh interface is terminating your wpa_supp.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/cozybit/wpa_supplicant/commit/02bf5235c07168053e357d356f2c363bb7f4e2d
f#commitcomment-13554187

Disclaimer:
----------
This email and any files transmitted with it are confidential and intended solely for
the use of the individual or entity to whom they are addressed.  If you are not the
intended recipient you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly prohibited. The
sender does not accept liability for any errors or omissions in the contents of this
message, which arise as a result of e-mail transmission.

--
Open WebMail Project (http://openwebmail.org)


std::vector does not allocate memory if you assign with menge[i]=
you have to do the push_back() method, or resize the vector first.


You are right...


time is enough, vm_runtime64_t is too long.


I will replace this.


@thevirtualfer Read this https://www.atlassian.com/git/


Makes sense!


I'd reformulate this a bit:
`Standard RapidCheck stateful testing can be used to find bugs occur as a result of single threaded interaction with the system under test. However, another very common class of bugs are race conditions which by definition only appear when more than one thread is involved. RapidCheck can provoke race conditions in stateful tests by executing a subset of the commands in a command sequence in parallel. RapidCheck then uses a verification function that is returned by the run function (see below) in each command to check if there is any valid interleaving of the commands in the sequence that can explain the result of the execution.`


Do not use plural in array.


It seems that castxml imports more types. Specially in some setups signature members that are only defined but not used.
Simliar to the gccxml workaround you introduced. Some meber are simply not as symbol availible because they were never used a member. Therefore the later code have told they have no members (the members are not created by castxml). The error message in this case was wrong.


I think that you just have to set key_2 (second argument of mptcp_hmac_sha1) to 8 bytes set to zero. If I remember correctly a HMAC-SHA1 takes a max of 512 bits of keying material. As we only have 8 bytes, that's all we can provide.


project internationalization
и я бы the убрала перед project


Ah - I guess I can't avoid loop detection. I'll fix it up :)


Mistake in initializing test arrays which lead to confusion.


Sorry, debgging because the output from autoproj overlapped should be indeed removed on the final


The output is :

  master
* skis


node.js —> Node.js


>I see you're doing polling in Nasal again :sob: Didn't you try some mathematical expressions in JSBSim or <autopilot>?

OK, now that I have a framework, I will give it a shot. Your taxing my puny brain again.


Хотел как то немного сгруппировать текст :)


IMO this patch can posted to mailing list seperately - now as well !


hrm


To add translations, use `decl` method:


Posted a few inlined comments, but I really hope we don't need to support symlinks in the find emulator. There are some tricky cases such as file system loops or symlinks which point paths the find emulator doesn't know. At least all tests in find_command.mk should pass and we may want more tests.

Also, kati's --regen flag heavily depends on the find emulator. I think we should do something for it. Maybe just adding referred symlinks into |fc.read_dirs| would be enough.

I thought the current find emulator implementation was OK. Its RunFind returns false when it finds a symlink and -L is specified, and then it falls back to not using the find emulator. What kind of problem have you met?


ok understood - this is fine - some of what u explained can go into chagelog to convince upstream guys as well !


Algorithm for relocating variables was incorrect.

Segmentation fault as well for accessing indexes out of bounds


I think you should also rename `dialog-overlay` and `dialog-container` in CSS and JS.

https://github.com/aurelia/dialog/blob/master/src/dialog-renderer.js#L47


А как это назвать? Коррекция? Редактирование?


Ping every second? That seems like a lot of pointless traffic for slack's API.

The ping line should probably be run only every n loops, where n is the most seconds you can handle waiting to know the connection's down, like at least 30.

For example
```
    try:
        sincePing = 0
        while True:
            start = time.time()

            if sincePing > 30:
                # This will cause a broken pipe to reveal itself
                server.slack.server.ping()
                sincePing = 0
            else:
                sincePing += 1
```


With respect to flooding, it is very difficult to see whether the peer is attacking us, or whether he is just announcing the addresses of his 42 interfaces :)
I think that unless this becomes an issue, we should be ok with accepting those ADD_ADDR_2 unconditionally.

One thing to consider would be to discard ADD_ADDR_2 messages immediately if the path-manager is not able to actually handle them (e.g., as is the case for ndiffports).


Already in.

Sent by MailWise<http://www.mail-wise.com/installation/4> – Your emails, with style.:)

-------- Original Message --------
From: Vineet Gupta <notifications@github.com>
Sent: Wednesday, October 7, 2015 09:33 AM
To: EZchip/linux <linux@noreply.github.com>
Subject: Re: [linux] tracing: Fix for non-continuous cpu ids (c58ee43)
CC: Noam Camus <noamc@ezchip.com>

IMO this patch can posted to mailing list seperately - now as well !

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/c58ee43eb1dd871146deca401d3fdbf46292d57d#commitcomment-13631502>.


I would not use #if #endif, because this would be a compile-time option then. ADD_ADDR_2-support should be dynamic based on the version that has been negotiated.

In the future we might drop support for version 0, but that's in the very far future only.


>> Btw, if you work in your own fork on github, @wlbragg can still directly pull in your code with

But he doesn't need, right ?

I can pull from upstream and push to origin, ever i want.

Will read the link you sent me.

Very thanks, bro. Will execute the commands you said, and see if i can push to myself !!!   ;-)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1658


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/82b319f1a9f7981c27f1e26739ce014733877d27


merging basho/riak_cs/refactor/hide-bags-absolutely = 82b319f1 into borshop-integration-1260-refactor/hide-bags-absolutely


basho/riak_cs/refactor/hide-bags-absolutely = 82b319f1 merged ok, testing candidate = 8be64424


Also add a BUILD_BUG_ON in arc_mm_init() just like we did for VMALLOC_SIZE just as a safety net.
Looks fine ! ACK


Ah, yes, I meant to say that the gradients can have numerical error.

I'll fix that. Thank you

CC #209


What is the problem ?
you can always edit the cache with ccmake or set the variables with `-D...` in the command-line


> But he doesn't need, right ?

I can pull from upstream and push to origin, ever i want.

Correct.

> Very thanks, bro. Will execute the commands you said, and see if i can push to myself !!! ;-)

You should always be able to push to your own repository on Github.


That's part of virtual-dom. It's an unfortunate (but necessary for performance) piece of virtual-dom's API, which in turn is a small piece of Cycle DOM's API, which in turn is a piece of Cycle itself.


It seems that .htaccess creates bugs with Wamp, so I deleted it. Sorry...


Add "c++" after the backticks to get C++ syntax highlighting.


>Didn't you try some mathematical expressions in JSBSim or <autopilot>?

As a matter of fact that is how I started out. But I was to far from a solution to work it out. I have plenty of structured language experience so that is where I can pseudo a solution (so to speak). Now that I have a  working framework, I think I can transfer it either method.


@voff12

推荐目录下的主页命名成 `README.md`，这样的好处是

用户直接浏览目录时，也会展示出这个文章内容，如

https://github.com/voff12/awesome-product-thought/tree/master/money-ball
![image](https://cloud.githubusercontent.com/assets/1063891/10330598/db16442a-6d00-11e5-96a4-d952f0f3a33a.png)

否则浏览目录时，无文章内容，如

https://github.com/voff12/awesome-product-thought/tree/674c45a29d19d451fc1183c56176da0c4ee2c4d6/apus-founder-share
![image](https://cloud.githubusercontent.com/assets/1063891/10330625/25ebaaa8-6d01-11e5-8d69-1c8feef3866f.png)


I'm not sure we should use terms like "previously" or talk about past tense in general since users don't necessarily read docs in order.


So where is the kernel code which does this mapping ?


в документах мы уходим от please. Может, заменить на you can find...


It looks a bit confusing since there is no indentation here but indentation is hard and I don't really know what else to do.


requried -> required


Minor improvement? Reword/rephrase description?


В репозитории мавена нет 1.3.0. И и при таком коде:

```
<parent>
        <groupId>org.springframework.boot</groupId>
        <artifactId>spring-boot-starter-parent</artifactId>
        <version>1.3.0.BUILD-SNAPSHOT</version>
    </parent>
```

у меня ничего не происходит - не скачивается этот билд. Может я что то делаю не так.
Однако ты учил меня не кидаться на сырые релизы :)


所有数据读取放到OpenCV里面


I don't think we should speak in terms of "does not have to be executed in parallel". I'd rather say that in the failing case, these were not executed in parallel. The "reason" might be that they don't have be executed in parallel but I think we should be clear about what this represents, i.e. "this is what was done to repro this bug".


Plz, mae the transform_volume_name() be just a function in volume.c. From my perspective there's no need in having it per-backend virtual method.


You do have the Output section below but I think it would be nice with some more explaining how the execution actually happens (i.e. sequential + two branches) and maybe some short explanation of why it works this way.


I think the code would be more readable ("don't mix abstraction levels"), but don't have a strong opinion either, so you can do as you prefer.


See comment below about "previously".


We should try to look for the new version of this one, it should have versioncode 107954630, with the last number increasing per specific DPI


Не знаю, в официальном гайде указана версия https://spring.io/guides/gs/spring-boot/


nice job anthony


fast-forwarding 2.1 to borshop-integration-1259-bugfix/dvv-for-rt = 3c1d06b6


Merge sha 8be64424 is stale.


merging basho/riak_cs/refactor/hide-bags-absolutely = 82b319f1 into borshop-integration-1260-refactor/hide-bags-absolutely


basho/riak_cs/refactor/hide-bags-absolutely = 82b319f1 merged ok, testing candidate = 5c6625b6


hi i am very new to android and i'm making a school project implementing pdf on the app i hope you would create a tutorial or a step by step readme.md ^^ thanks for your post i will do my best to make this work for me. hope you'd  do my request. sorry for bothering


@mstembera Thanks for explaining. Makes sense now. :-)


I never knew this event existed, neat


@mudassarn , instead of a test client could you write a test case?


just wondering if we could clean this part and don't use `external/someone.someone` in permissions repo


"methods is a default package, coming with any installation of R" - so the comment of "Known issue: package 'methods' is not available for R version 3.2.1. In which case, please downgrade R to version 3.1.1." is incorrect?


@diegobz Some links are pretty huge, I believe they won't look nice in the command line and it wouldn't be convenient for the users.


@neebz Is this open source? or your forgot to make it private?


spring-boot-starter  -  The core Spring Boot starter, including autoconfiguration support, logging and YAML.


Would you consider using http://github.com/shurcooL/sanitized_anchor_name instead of `makeAnchorName` (and contributing your rich test suite there)?


meh, only added that for debugging and forgot to remove it again. Will send a fix


@due1 according to best practices, no IDE-specific files should be versioned by git. Is there a specific reason for this?


@mudassarn  could you expose a dummy function listFiles in fileTransfer/localServer.js, which throws a "Not Implemented Exception". This way we can keep the public interface of both amazonS3 and localServer in sync.


Везде `chatAPI` :))


I am not sure what do you mean, @lubieniebieski


Will refactor this in the future, currently just focus on functions implementations first.


This looks like shit but it's what the IDE wants


@kumajaya I left it over night yet it still did not complete...


Is this really a hard dependency or just a "lets use the latest"?

I am confused this is not part of a commit that starts using new APIs or removes deprecated ones.


Скорее всего так, если только String Boot сам автоматом не тусует бины по нужным контекстам.


@kumajaya Then why you name it like that?


Remove prebuilt kernels too... dontcha?


@Godweed what version are you using?
I use spree editor on multiple 3.0 stores without problems


@ndalliard This won't remove the files from the repository, it only ignores new files which matches the specified pattern. But these files are already committed and therefore they are still stored at the repo. I propose to wait with the deletion...


Hello


Checkmarx scan results 07 Oct 2015 10:00 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
39
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

19a974971b960a93fd9db110981376e4a81c8bfb	@dudug


Да, здесь я попался как раз. Не глянул на версию референса, который читал (а он был как раз по версии 1.3.0) и потом долго рвал волосы - почему у меня консоль не работает после `spring.h2.console.enabled=true`


السلام عليكم هل يمكن مساعدتي لفتح apple id لايفون مغلق ولكم جزيل الشكر


Не может. Thymeleaf сыпет ошибки на всех Security тегах.


two small comments
a) we don't ever use the github system for pull requests or comments on commits - please just send things like this to the mailing list
b) you are of course correct. This was supposed to be a calloc(), not a malloc() but I fat fingered it (and then apparently copied it to the macos.c file for good measure. Thanks for catching that.


@imscaradh Yes my bad. I am not an expert in using git. @Tschanzt reverted my commit. Especially because the eclipse files doesnt really bother us.


oops


No problem!


That's ParamSklearn. I hope this will be fixed during the next few days.


copyright (c) 2015 RealUnknown


The man page documents that `emacs` or `vi` are the expected values, and (I had a quick look at the sources of libedit-20150325-3.1 and) there seems to be support for both styles of key-bindings. It is not completely clear to me what level of feature support is provided by the present patch (for example editline supports `M-d` or `M-D` to delete the next word, and `M-p` and `M-n` for incremental search forward/backward in history, are those enabled by this patch or does it require more editline-plumbing code?), but if the more advance Emacs keybindings work, I think it could make sense to expose a runtime function to change the editor mode (which we could decide expose in Toploop or through the toplevel configuration).

I think it would be useful to expose a `


Yep, it is public by design :)

> On Oct 7, 2015, at 11:51 AM, Afeef Janjua <notifications@github.com> wrote:
>
> @neebz <https://github.com/neebz> Is this open source? or your forgot to make it private?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/caremerge/hfp-utils/commit/41b94cf321839aef180047d85708288d31fbaff3#commitcomment-13631710>.
>


А, нашел:
```
.csrf()
				// Allow unsecured requests to H2 consoles.
				.requireCsrfProtectionMatcher(new AllExceptUrlsStartedWith("/console"))
```
классно, я не знал что так можно! :) Надо будет вернуть `csrf`.


why was this updated ? unless there's an explicit commit referring to JTS version port, a new JTS port was not performed


fine for me


When incrementing CURRENT you also have to increment AGE unless backward compatibility was broken


Je corrige ça, merci :+1:


@javadev yes, but I think `HashMap` is good for default implementation. Also, you can set another Supplier for `groupingBy` and `toMap` methods.


Работало, но в `MainController`'e подсвечивало поле. Мне это не понравилось :)
![image](https://cloud.githubusercontent.com/assets/13660819/10331165/c75d1e88-6cdb-11e5-91b5-4e3f345a55a8.png)


This plugin is already activated in the parent pom (https://github.com/exoplatform/maven-parent-pom/blob/master/pom.xml#L1329), not sure it is useful here.


modified the trajectory class: add acc in one DWA simulate step. modified the eo energy cost function: if DWA single simulate step, how to find the vel_mean acc_mean traj_length rot vel_end

another more robust approach to be done in the future:
1. due the trajectory class is certain: DWAxa_(0.0), DWAya_(0.0), DWAthetaa_(0.0) may not work exactly as expected.
2. link odometry_msgs to the eo_energy_cost_function.cpp. Find the instant velocity of the robot and calculate the vel_mean acc_mean traj_length rot vel_end for DWA single simulate step.


This particular change shouldn't effect whether or not you get deadlocks - can you submit an issue with more details and an example layout for us to try?

Thanks!


... as you can see here (jumped from 1.9 to 1.11 ?)


sdds


i tried importing the fle to my android studio but it wont finish building. any advice?


and same thing here, CURRENT raises with AGE unless backward compatibility was broke


im struggling  to down load wazapp to my nokia n9


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1662


Why do we need to exclude this pattern ? Can't we rename test classes instead, to avoid to have specific rules by project ?


you have to use a while loop. A single if-statement assume that points are always in neighboring images. But you can't assume that.


The version should be put in a property and dependency management.


Yep, seems nice to me. I won't be able to add it to i-score until next week I think.


it's a android app.


נשלח מה-iPhone שלי

ב-7 באוק׳ 2015, בשעה 01:34, Or Shachar <notifications@github.com> כתב/ה:
Right, I haven't notice this method

Thanks!
> Seems alright!
> Check out the static method String.join, probably familiar to you from Python. It was added on Java8.
> For instance:
>
> String.join("\n",words);
> would have given you same results without having to use substring and replaces.
>
> Cheers!
>
> —
> Reply to this email directly or view it on GitHub.
>


I am using 3.0 stable version and getting
Request URL:http://www.domain.com/javascripts/ckeditor/init.js
Request Method:GET
Status Code:404 Not Found


Actually, what would happen by not raising AGE would be that CAPI version would jump to 2.8.0.
Anyway, I'm fixing this.


There's no need for this to be a global variable, it should be returned by the `wait_for_requested...` function.


i lie you


finish read me


The check here is in a loop which checks if the exited process is one of the user requested processes:
1. It will also capture the exit status of reparented zombie processes.
2. The exit status for the same process will be checked multiple times needlessly (except in the case where it relates to the first user-requested process).


Can you turn this into an object lookup or a Set? All of the moduleMocker code is super-hot and run *a lot*. Metadata is generated once per worker per file, mocks are created whenever the registry is reset, which is at least once per test.


@nirvn I see anyway the scrollbar not need to resize the window (15'' screen).

> you'll see the code does not use the QTextDocument width to set the item's width. Some dark Qt voodoo at play here.

hmm, I guess ```option.rect.width()``` is related to ```doc.setTextWidth( 2800 )```, but I am not sure.

Anyway, I expect the horizontal scrollbar must be shown as needed and if I resize the window I  also would expect the text must be wrapped and not cut.


You should not specify this version at all because it is supposed to be already defined in the pluginManagement section of the eXo Parent Pom. (If it is not the case, ping @mgreau)


this must be defined in the eXo Dependency POM


I don't think this comment about 'offloads some of the optimization work to the compiler' is accurate.

My understand is that the rationale for this is to match the x64 addressing mode when the index is an uint64 and the offset is an int32 and is implicitly signed extended and added with 64 bit precision (and not wrapped to 32 bits in wasm32 code). Also that this is a scheme optimized for using a memory protected guard zone.

It's not even clear if the index is interpreted as a uint64 in a load on wasm64, and a uint32 on wasm32?

If so then using inline bounds checks these will need to firstly check that the addition of the offset will not wrap and then check that the effective address is within bounds - slow.


you can also just bail out early in line #130. The eval piece of this is just to preserve he function names. Also, would you mind adding a test for this?


I think the check for the exit status should instead be inside this `if`, then you will know it relates to a requested process rather than a reparented process. This also brings up the issue of smell-baron being able to run multiple processes (separated by the `---` argument). In this case I think it would be desirable to return the exit status of the "left most failing requested process" rather than arbitrarily returning the exit status of the last requested process to fail. It might even be desirable only to consider the exit status of the first requested process, e.g in the case of `lighthttpd --- runit` you probably don't care if `runit` fails, only `lighttpd`.


@WHPODOO, self.ref(xml_id) works only in test cases because it is defined in openerp/tests/common.py, so it can not be used in normal model. in normal model we need to use self.env.ref(xml_id)....


It's duplicated in `else` so I guess it can be called just after `if () { ... } else { ... }` so it will be single call.


The same as https://github.com/ezsystems/PlatformUIBundle/commit/1f844bfda0348267c73c9db92de22c0ad00d9cda#commitcomment-13632076


@OM2
You should include `bower.json` file in this PR also.

Also you are using window.innerHeight to get height and calculate initial amount of elements, that should be seen, but I think it would be nice to calculate actual needed amount by taking height of the list itself (it can be done using $element in link). `Math.floor` should be `Math.ceil` since you want to show a little bit more than fits into viewport to show scrollbar.


@sbimikesmullin Thanks for the nice comment! :smiley:

I still have it on my list to create PRs for our patches, and there is a good chance that we can work on that in the next weeks.


The "notification" feature seems to support a fixed number of keys. This implementation treats it as a free key value object. I think a seperate object like gcm.Message would be a better fit with the appropriate json tags.


It seems to be easy…..but !


We should move the string in here into strings.xml


Added in commit 7dcea341671c274f1fea3ad14c64763847f68886


Hmm, sure, what would be the condition for that?


Too much Ruby


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


I'm afraid `Documentation/security/securelevel.txt` is missing from the tree. Could this be added?


Hello @mgp25

I am using this api for sending message but no message sent to my number.

code i am using:
$sender = 	"918010202721"; // Mobile number with country code (but without + or 00)
$password =   "***********"; // Password you received from WhatsApp
$nickname="fhruy";
$dst = "919717474431";
require '../src/whatsprot.class.php';
$w = new WhatsProt($sender, $nickname, true);
$w->connect();
$w->loginWithPassword($password);
$newContact = array($dst);
$w->sendSyn($newContact);
$w->sendPresenceSubscription($dst);
$w->sendMessageComposing($dst); // Send composing
$w->sendMessagePaused($dst); // Send paused
$message = 'Hi! :) this is a test message';
$w->sendMessage($dst , $message); // Send message
$w->pollMessage();

and output i am getting is:

http://hybrid3.dintex.in/wa/examples/NakulSend.php

no message sent


In the process window, in the 'Execution' tab, in the 'Result' section.


Fixed in commit 8ec853687cf7bea1c91baa02973190849b06d11b


@romancieslik done at https://github.com/betterplace/fundraising_handbuch/commit/a2cbba9a19d93c2c06ea7643290d2089393e564f


Do these even work? I've never had success with such form of subscription before.


TB means nothing to me, but in general, you can allow me to do the compression


Наверное, нужно в бинарном режиме открывать `'rb'`. Для windows критично.


TeamCity pull requests :: netty [Build 378](https://garage.netty.io/teamcity/viewLog.html?buildId=2102&buildTypeId=pulls_netty&guest=1) is now running


出力のFormatですが、このように区切りますか？


should the reported repro be part of the unit tests?


Addresses issue #27.


Estás comparando dos hostings "clásicos" entre sí. Los dos te van a costar lo mismo los uses un 1 o un 10%. Se trata de comparar alguno de ellos con servicios en nube en los que sólo pagas por tiempo consumido.


Makes me wonder; why isn't NIX_REMOTE part of environment.sessionVariables? Wouldn't that have made sense?


Well, this one would work even without game. see the function argument.


Hm `asrent` and `asbuy` would be more AreaShop specific rather than "abuy" and "arent" ;)


Same as for line 122


コメント書いてよ


ya puestos también podías haber puesto el resultado de ejecutar el programa empaquetado, pero gracias por la corrección


This could even be required lazily, but it seemed reasonable to avoid introducing the "laziness magic" given that `jasmine-test-runner` is pretty lightweight. (it takes only `1.5ms` to require it the first time)


I'm not sure I get you. Do you mean for eslint rules? If so, do you mean adding a blank line after/before block?
Also, does it interferes with the PR itself? I can do the PR without the eslint fixes. Thanks.


You missed out starting and closing brackets - "(" and ")"


This one is indeed an error. I've forgot that there's no tick variable defined here in this function.


> How might we solve that problem and pass the test proposed in my previous comment?

    getExpensiveResource = function () {
            if( persona._closed == false )
				return _expensive_resource;
			else
        		return {value:null};
		}

> Care to add some instructions to your README file?

Done! :)


Polygon isn't displaying as the GeoJSON is malformed...it seems that the server is missing this patch:
https://github.com/flavour/eden/commit/90c92d4f51c56b451eefec26c5450f60b89e9576


You missed out starting and closing brackets - "(" and ")".


So my general rule of thumb is to declare explicit types for method signatures (arguments and return types) and properties. I use `def` for local variables because the type can almost always be inferred and the code ends up being much less noisy. That said, it _is_ just a rule of thumb and you can go with whatever you feel comfortable with. You should just aim to come to some agreement amongst the team so that you have a consistency of style across code bases.


The link here points to the same article.


This link also points to the same article


Ok np I will change that


:smile:


I made the change because there are already some other API changes introduced for 7.0.0.

@bamedro  Should I revert this change or is it ok to keep it for 7.0.0?


Sure, but pointers does exactly the same. At the time it seemed like the best and simplest solution.. Are we agreeing it leaks now?


fixes #44


Does the ButtonGroup work in resizing the text?
Continue working on this branch, we can merge later in the week.


great! some remarks:

1. `assert` is no function so to avoid confusion, consider removing the parentheses
2. we could wrap the `__main__` block into a `def test_version_utils()` function, then e.g. `py.test` would find it.


@thevirtualfer From the output I can see you're working in the `skis` branch. Maybe (if you're working together with @wlbragg), it's easier to just directly use our repo. Just make sure that you use `git branch` or `git status` to verify that you're in the `skis` branch and not in the `master` branch.

If you want to get back to one remote, you can do:

```
git remote remove origin
git remote rename upstream origin
```

(It's really a matter of how you want your workflow to be).

It's a good practice to always do `git status` before you plan to commit or push anything. Check the output to verify you're on the correct branch (`skis`) and which files you're about to commit (the files/changes that have been staged).

Also be aware that (especially with binary files like `.blend` files) you start with a recent version of the file. With `.blend` files you will basically override the previous version because git cannot see how the file was changed (it can see it has changed, but not how because the file is binary). So if you start with a very old file, you might undo fixes that have been made in the mean time.


Map `TryModule(namespace)` with folder `src` :notebook:


Is it all tridonic specific stuff?


База данных для учета студентов

        Настройки базы: /studentdb/db.py
        Загрузка демо-данных: python manage.py loaddata --app=student demo_data.json


Do you think that returning -1 is better than fowarding the exception to the caller?


And that's not a correct git commit message.


Because it still doesn't work with xblock.


Also < should come before >= dep.


hmm true it supports `body, title, icon` all as strings, looks like even the readme is wrong ^^ will update


Really? Wasn't WebKit2 just removed from next?


@huynh-tam-mulodo please set starting number is 1 for auto_increment.
Thanks


NVM*L* Libraries?


> Hey sorry was at work.

No worries. :wink:

I'm unable to compile/set a working WotLK core+client for a while, so it will be hard for me to debug this.
First, we need to isolate the part of the code causing the crash. Could you try the following patch, please?
```diff
 src/game/playerbot/PlayerbotAI.cpp | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/src/game/playerbot/PlayerbotAI.cpp b/src/game/playerbot/PlayerbotAI.cpp
index 67c2580..1bcfe36 100644
--- a/src/game/playerbot/PlayerbotAI.cpp
+++ b/src/game/playerbot/PlayerbotAI.cpp
@@ -7111,13 +7111,6 @@ void PlayerbotAI::findNearbyCorpse()
         if (!corpse->IsCorpse() || corpse->IsDespawned() || m_bot->IsFriendlyTo(corpse))
             continue;

-        Loot* loot = sLootMgr.GetLoot(m_bot, corpse->GetObjectGuid());
-        LootItemList lootList;
-		loot->GetLootItemsListFor(m_bot, lootList);
-
-        if (lootList.size() > 0 || loot->CanLoot(m_bot))
-                continue;
-
         uint32 skillId = 0;
         if (corpse->HasFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_SKINNABLE))
             skillId = corpse->GetCreatureInfo()->GetRequiredLootSkill();
```
(Basically remove lines 7114 to 7120 in PlayerbotAI.cpp). This may have some side effects on bots behaviours, but I can't be sure. This check was initially added to prevent bots to repeatedly go back to a corpse previously looted but when I switched to the new loot I added some similar checks elsewhere. Also loot items availability is handled differently in core, now.


ditto


-m should say **readme** not ready (damn autocorrect!)


@posita

> How might we solve that problem and pass the test proposed in my previous comment?
>> I'm looking to it

https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/fa8ea9233501441f06278659c9bc94ae60169b55


For more information, that's http://search.maven.org/#artifactdetails|org.robotframework|swinglibrary|1.9.5|jar


Please remove commented code.


(descriptor, result) < 0) {


ここまでのブロックの書き方がまずい。
top.php からボタンを押されると、必ず$_POST["id"], 必ず$_POST["pass"] にはから文字列が入ってくる。
だから、else 以下が実行される。
ここではセットされてるのは確実なのだから、処理はいらない。
あえていえば、
$id = htmlspecialchars($_POST["id"]);
のあとで、$id や $pass が空文字列のときに、top.php に戻す処理をいれる


Well, the HUD rectangle does not use a projection matrix, so technically there is no near plane. But there is still the normal depth range of [0...1] used for z-testing and clipping. So yes, the 1E-5 was chosen to be as close as possible to 0.0 while still being bigger (because there may exist WebGL implementations that would clip the rectangle away if it lies exactly at depth 0.0). The value 1E-5 is just an educated guess. It is definitely bigger than 0.0 when 32 bit floats are used (as is the case on desktop hardware), and is hopefully still bigger than 0.0 on mobile devices where 16 bit half-floats may be used instead.


@posita ... your test is passing, look: https://travis-ci.org/web2solutions/Veritaseum-proof-of-concept

I just dont trust on this code ::):


Seems reasonable. I'm not too worried about micro-optimizations in the context of test runners. But it something to keep in mind though :thought_balloon:


Maybe could you check if we can factor ````bufferDuration```` (and the if-statement) out?
So instead of

````
// so speed = 1 sets the sample to play for 1 cycle, and 2 for half a cycle
		if (unit == \c) { speed = speed * bufferDuration * cps };

		endSpeed = speed * (1.0 + (accelerate.abs.linexp(0.01, 4, 0.001, 20, nil) * accelerate.sign));
		if(endSpeed.sign != speed.sign) { endSpeed = 0.0 }; // never turn back
		avgSpeed = speed.abs + endSpeed.abs * 0.5;

		// sustain is the duration of the sample
		switch(unit,
			\r, {
				sustain = bufferDuration * length / avgSpeed;
			},
			\c, {
				sustain = bufferDuration * length / avgSpeed;
			},
			\s, {
				sustain = length;
			}
		);
````

It could be:

````
	endSpeed = speed * (1.0 + (accelerate.abs.linexp(0.01, 4, 0.001, 20, nil) * accelerate.sign));
		if(endSpeed.sign != speed.sign) { endSpeed = 0.0 }; // never turn back
		avgSpeed = speed.abs + endSpeed.abs * 0.5;

		// sustain is the duration of the sample
		switch(unit,
			\r, {
				sustain = bufferDuration * length / avgSpeed;
			},
			\c, {
				sustain =  length / (avgSpeed * cps);
			},
			\s, {
				sustain = length;
			}
		);
````

I don't have a test, so this is speculative algebra.


How this works? This method doesn't return anything.


ここで、セッション内のキーid に NULL を設定している。
作り方にもよるけれど、エラーなのにセッション内にキーid をセットしているから、
セッションが必要な他のページにも遷移出来てしまう。
エラーの時は何もセットしない方が好ましい。


        videoitem.title = " ".join([item.title, videoitem.title.replace('Enlace encontrado en ','[COLOR green][B][')+'][/B][/COLOR]'])
;)


You can set them via the command line, sure, but this is not the default behavior, it can be confusing. Is there a specific reason to do so ?


@nguyen-tien-mulodo : done. thank you


ここでセッションからid を取得しているだけで、何もしていないから、
他のページに遷移出来てしまう。
例えば、idが取れない場合に、top.php にリダイレクトするとかの処理が必要。


在 ee0b2b8db9584a962614cd9cfe0dc6cfde1c6c45 修改了～ :smile_cat:


Why do I have to use `request.args` for the GET argument? And when should I use `request.values`?


Why do we have sleep here?


Nice update :-) It's important to do this stuff to show the library is alive, hence to attract new users. By _split-screen text panels_, do you refer to the horizontal/vertical split panels I talked about ? Because I'm not sure I'll push them, I'm not done yet ^^


Ok, @onox

But, can i push to the upstream without the renaming steps ?

I mean, something like :

```
git checkout skis
git pull
```

and after making my changes...

```
git push  upstream skis
```

or else

```
git pull upstream
```

Can i do this ?


Да, действительно, упустил. В документации для совместимости советуют добавлять 'b'.


I think you referenced the wrong issue. This fixes #13


```python
    @cached_property
    def args(self):
        """The parsed URL parameters.  By default an
        :class:`~werkzeug.datastructures.ImmutableMultiDict`
        is returned from this function.  This can be changed by setting
        :attr:`parameter_storage_class` to a different type.  This might
        be necessary if the order of the form data is important.
        """
        return url_decode(wsgi_get_bytes(self.environ.get('QUERY_STRING', '')),
                          self.url_charset, errors=self.encoding_errors,
                          cls=self.parameter_storage_class)

    @cached_property
    def values(self):
        """Combined multi dict for :attr:`args` and :attr:`form`."""
        args = []
        for d in self.args, self.form:
            if not isinstance(d, MultiDict):
                d = MultiDict(d)
            args.append(d)
        return CombinedMultiDict(args)
```

values = GET+POST. but since you only expect this as a GET argument it makes more sense to use args


Hello..i need you please


I'm not so sure that HP/Power values should not be sent to client for mobs.
Blizz client in classic always displayed %, I agree. But I pretty sure that addons were displaying current values, meaning that they were sent to client.

I agree with the current patch for duellers and pets.


Just now with going in the world.


@paulirish Works also with OS X's own sed if you replace `-r` with `-E` in https://github.com/paulirish/dotfiles/blob/master/bin/diff-so-fancy#L59 and https://github.com/paulirish/dotfiles/blob/master/bin/diff-so-fancy#L64


@relshire, I think it would be nice to expand on subgroup topics e.g. QC group - with a bullet point and an example image to prompt discussion for each area


Слав, я, конечно, извиняюсь за доставленные неудобства. Я сделал так не потому что специально хотел доставить хлопот или мне лень было. Наоборот, я не торопился и проделал это пункт (перевод на Spring Boot) дважды. Сначала, в тестовой ветке заставил все работать и пофиксил все что не работало или вызывало у меня вопросы. Потом сделал патч с изменениями, и согласно ему (просто читая изменения) я переводил уже рабочую ветку на Spring Boot, пытаясь разбить все операции на коммиты. Собственно, два прохода для этого и задумывались. "Ну и где твои разбиения на коммиты?!" - скажешь ты. Я и сам себе это говорю, потому что маловато коммитов получилось. Но я следовал принципу - коммит должен быть не только атомарным, но и не деструктивным. Т.е. в нем мы фиксим баг/изменяем/удаляем/добавляем какой-либо фунционал, но проект при этом работает. В моем случае, только все, представленные в этом коммите операции, позволили запустить проект, не нарушив функционал - как тогда дробить на более мелкие коммиты, если проект просто не будет работать?
Может стоило тогда отойти от этого принципа в угоду читабельности, я не знаю.


I think for some reason condition on line 33 is always true (on Safari), perhaps because plugin is not be installed from apple's 'extensions.apple.com' due error 'This developer's server appears to be busy. Please try again later.'
And on each browser start or enable/disable extension from Safari setting -- opened new tab and generating URL
on Safari 5.1.10 - http://add0n.com/google-translator.html?version=undefined&p=undefined&type=upgrade
on Safari 6.2.8 - http://add0n.com/google-translator.html?version=0.3.9&p=0.3.6&type=upgrade


https://github.com/joyent/sdc-dockerlogger link 404s


essai correct
JCA
7 octobre 2015
layout(matrix(1:2, nrow=1))
x<-rt(200,5)
hist(x, density=5, angle=30, ylab="Effectifs", xlab="", main="Histogramme")
hist(x, prob=TRUE, ylim=c(0,0.5), ylab="Proportions", xlab="",main="", col="red")
curve(dt(x,5), add=TRUE, lty=3)
lines(density(x),col=7,lwd=3)
￼
QI=rnorm(10,mean=100, sd=15)
QI
##  [1]  97.62805  81.32668  95.07349 108.25726  83.22509  85.11071  82.34146
##  [8]  96.41002 104.47708  81.16412
QI=round(QI,0)
QI
##  [1]  98  81  95 108  83  85  82  96 104  81
mean(QI)
## [1] 91.3
table=numeric(1000)
for(i in 1:1000)+{table[i]=mean(sample (QI,10,replace=TRUE))}
hist(table)
hist(table,breaks = seq(from =80, to =130, by = 1))
lines(frequency(table),col=7, lwd=2)
￼
hist(table,freq=FALSE,breaks = seq(from =80, to =130, by = 1))
lines(density(table),col=7, lwd=2)

hist(table,freq=FALSE,breaks = seq(from =80, to =130, by = .5))
lines(density(table),col=7, lwd=2)
￼
table.sorted=sort(table)
c(table.sorted[25],table.sorted[975])
## [1] 85.7 97.8


このコメントgif画像必要？ｗｗｗｗｗ


Thx for the comment. But I disabled the key, thats enough for now.


Indeed :) default phase installs (from devmanual):
README* ChangeLog AUTHORS NEWS TODO CHANGES THANKS BUGS \
				FAQ CREDITS CHANGELOG

Ebuild is now 6 lines shorter

--
Bernard Cafarelli (Voyageur)
Gentoo developer (NX, GNUstep, net-misc, llvm/clang, ...)

Le 07/10/2015 1:55, Julian Ospald a écrit :

> looks like src_install() can be omitted, since the defaults pretty much
> do the same afais
>
> --
> Reply to this email directly or view it on GitHub.


Noooooo......


Don't create a new main.c. Instead, please utilize the consistent `main.c`.


:heart:


eğleniyor muyuz gençler ?


もとに戻す


We have to compare the performance between the original and the optimized routines. Thus, keep the consistent `main.c` is critical.


Here I want to introduce you a music software called Kugou. It's an application designed to search for and listen to music and radio online, watch music video , record music online and sing karaoke online. Meanwhile, there are other functions including recommending music and lists of music on any stage of level, phone rings making,transforming file format and so on. More than these, it establishes good relationships with many famous corporations and organizations at home and abroad, such as Yahoo, Sina, eBay, BENQ, Rolling Stone, R2G etc. Thus it services a huge number of resources of music for us, which is one of important reasons for using it.I have to say that it is an excellent application, but I still have a few personal suggestions for better-performing of Kugou. First of it, it is necessary to unify type of title of music. For instance, some titles are formed by singer-song, while the others are formed by song-singer. Most songs are listed by their initial. When you want to find a song, but you don’t know its initial, and you don’t know it is singer-song or song-singer. That could make you much confusing. And if you are just in a harry, you might be in trouble.Second, Kugou has an unique advantage that it recommends music according songs that you already stored in your phone. But here's a problem. Some songs you've forgotten for centuries on your list, but they're still viewed as norms of style that you like now. In my opinion, it should remember every frequency of song an user is listening to at that time. So just on the basis of songs that are played frequently during that period.Third, although thousands of lists are recommended by net friends or ranking lists, there are at least fifty songs in their lists generally, such a large quantity; many users hope that some popular radio anchors and well-known musicians make a list. This list doesn't have to cover lots of songs, but it should be in high quality.And at last, it is better to add a brief introduction of songs and of singers. We should not only focus on songs, but pieces of short information about this production.For years I've been listening to music on Kugou, though, there are a few flaws in it. There are defects I think could be made up followed. The first point is, in its interface, you couldn’t find a break key, but a pause button. Some people explain that they couldn’t log on Kugou by their third ID. So they have many songs stored at other apps don’t permitted to play on Kugou. And the last thing is you can’t press either fast-forward or rewind when you lock your screen. Sometimes you just have to listen to a song from the beginning when you only want a short section of it.As for the requirement unsatisfying me, maybe I don't have all the lyrics of songs I have stored, especially English songs. It does be a big enough private library of songs, but not all lyrics included. That’s could be a pity.Anyway, it wins high public praise. Its nearly perfect humanized design still catches my loyalty definitely.


Declare it in `ILoggingManager`, add a default implementation in `AbstractLoggingManager` (returning 0). Override it for `GCLoggingManager`. Let other connectors as is.


comment text to 'Test commit title'


You don't have to mention `Modify Makefile`.


I only noticed it recently: this change also makes it possible for me to release zc.recipe.egg with zest.releaser. zc.recipe.egg is in a subdirectory if the zc.buildout github repo :-)


- A SpecFunc::Ei entry in SpecFunc_doc.i.in should be added
- Because of the whitespace changes in t_LogUniform_std.cxx, we do not see what changed, you could revert it and modify just the relevant lines


Since you already have implemented the routines, you can remove the comment "FILL YOUR OWN..." safely.


Oh right, I also improved performance for $quit.


Please use `astyle` for consistent coding style!


thanks, ain't tested it, but should solve storage problem in my case


You should clean the generated images as well.


Did you release all allocated entries already?


Favourite points count would be better as an `private int` in `LogCacheActivity` and `GCLoggingManager`.

In `GCLoggingManager` add a public accessor to retrieve the value (as done for `possibleLogTypes`)


The function `hash` is implementation specific, and you should not expose here.


The hint should be there in my opinion, @davve94


Is it correct that these comparisons are strictly-greater and not greater-or-equal? So a 1920x1080 device with 192 dpi / 200% scale does **not** qualify as "high resolution" here? (Just wondering, I don't own a high resolution device so I can't check what typical values are.)


Is this something that we shouldn't have on github :D?


TeamCity pull requests :: netty [Build 378](https://garage.netty.io/teamcity/viewLog.html?buildId=2102&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5623, ignored: 21 Build time: 00:38:59


BTW, I guess these 2 elements are kept for backward compatibility so that a XAR created with a new version of XWiki can be imported in an old XWiki version?

Maybe we could generate a comment too in the XAR export to explain that they are deprecated and kept for backward compat reasons?


i think it would be better to use
```
make V=1 -j$MAKE_THREADS PREFIX="$butch_prefix"
make PREFIX="$butch_prefix" DESTDIR="$butch_install_dir" install
```
instead of adding a patch (the variable can be overridden like this)


Bij het selecteren van 1 connectie worden alle connecties geselecteerd. Waarschijnlijk mist er hier een check of klopt deze niet helemaal.


@PennyQ - I'm getting:

```
Traceback (most recent call last):
  File "volume_test_axis.py", line 35, in <module>
    from vispy.visuals.transforms import (STTransform, MatrixTransform,
ImportError: cannot import name MatrixTransform
```

Is this a new feature in the development version of VisPy?


You can if @Juanvvc added you as a collaborator. But what's the point of having two remotes then? :smile:


Couldn't this just be `m := append(self.modifiers, metrics.Tenant(k))` ?


directive


please add
```
tarball=smu-1.4.tar.gz
```
so the tarball gets saved with a proper name


typo here, should be >


Two things:
1) The value of R is never used, apart from returning it. The variable is then not necessary as the case itself will return that value.
2) Unless there is a reason to pass the DefaultSorts, this function shouldn't deal with it but let the defaults to the function itself. The previous function rabbit_mgmt_util:reply_list(Facts, ReqData, Context, Page) seem more appropiate and doesn't require to check the page in this function.


Yes. I measured it at about extra 0.1ms / 90 kb memory allocations per razor function that contains at least one XHTML entity (f.e. ©). Shouldn't be a significant change for the most of the websites.


Just so we're clear here, the `/file/` endpoint needs multipart-mime uploads not to upload a ton of files in the same go, but for being able to upload a file as well as accompanying metadata (JSON formatted) in the same request.

It should not allow uploading *n* documents at the same time and them return an invalid `Location` header.


message


Tror det är lugnt men key hashen ska vi inte ha på github i alla fall men
kanske råkade pusha den med :O

onsdag 7 oktober 2015 skrev Maxim Goretskyy <notifications@github.com>:

> Is this something that we shouldn't have on github :D?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/signal-unknown/stormy-street/commit/404e1bcf804e004248b01107ff97c24ea9623a0e#commitcomment-13632683>
> .
>


You seem to have inverted the condition here. This being a create operation, you would want to create a new user if `findUserByName` returns a `NullUser`.


You should mention the motivation and exact reason why you would like to make such modifications.


The correction in `findPathLarge` is correct, but it could consume less memory; because with the actual version, at each recursive call (i.e. a call to `findPathLarge` within `findPathLarge` itself), the input `impassable` (if non-`null`) set is copied; whereas the copy is only required at the very first call to this method. Hence I'd refactor the method as follows (introducing an indirection to distinguish the initial call from the recursive calls, the key point being that `findPathLarge0` calls `findPathLarge0`, not `findPathLarge`!):

~~~~~~~~~~~~~~~~~~~~~~~~~~~{.java}
    public ArrayList<Coord> findPathLarge(int size, int length, Set<Coord> impassable,
                                     Set<Coord> onlyPassable, Coord start, Coord... targets) {
    	final Set<Coord> s = impassable == null ? new LinkedHashSet<Coord>() :
    		new LinkedHashSet<Coord>(impassable);
    	return findPathLarge0(size, length, s, onlyPassable, start, targets);
    }

    private ArrayList<Coord> findPathLarge0(int size, int length, Set<Coord> impassable,
                                     Set<Coord> onlyPassable, Coord start, Coord... targets) {
    	if (impassable == null)
    		throw new NullPointerException("Public caller (findPathLarge) is wrong");

        if(!initialized) return null;
        /* Initialize the list with the (a priori) appropriate size */
        path = new ArrayList<Coord>(length);

        if(onlyPassable == null)
            onlyPassable = new LinkedHashSet<Coord>();

        resetMap();
        for (Coord goal : targets) {
            setGoal(goal.x, goal.y);
        }
        if(goals.isEmpty())
            return path;

        scan(impassable, size);
        Coord currentPos = start;
        double paidLength = 0.0;
        while (true) {
            if (frustration > 500) {
                path = new ArrayList<Coord>();
                break;
            }
            double best = gradientMap[currentPos.x][currentPos.y];
            Direction[] dirs0 = rng.shuffle((measurement == Measurement.MANHATTAN)
                    ? Direction.CARDINALS : Direction.OUTWARDS);
            Direction[] dirs = Arrays.copyOf(dirs0, dirs0.length + 1);
            dirs[dirs0.length] = Direction.NONE;
            int choice = rng.nextInt(dirs.length);

            for (int d = 0; d < dirs.length; d++) {
                Coord pt = Coord.get(currentPos.x + dirs[d].deltaX, currentPos.y + dirs[d].deltaY);
                if (gradientMap[pt.x][pt.y] < best) {
                    if (dirs[choice] == Direction.NONE || !path.contains(pt)) {
                        best = gradientMap[pt.x][pt.y];
                        choice = d;
                    }
                }
            }

            if (best >= gradientMap[currentPos.x][currentPos.y] || physicalMap[currentPos.x + dirs[choice].deltaX][currentPos.y + dirs[choice].deltaY] > FLOOR) {
                path = new ArrayList<Coord>();
                break;
            }
            currentPos = currentPos.translate(dirs[choice].deltaX, dirs[choice].deltaY);

            path.add(currentPos);
            paidLength += costMap[currentPos.x][currentPos.y];
            frustration++;
            if (paidLength > length - 1.0) {
                if (onlyPassable.contains(currentPos)) {

                    closed.put(currentPos, WALL);
                    impassable.add(currentPos);
                    return findPathLarge0(size, length, impassable, onlyPassable, start, targets);
                }
                break;
            }
            if(gradientMap[currentPos.x][currentPos.y] == 0)
                break;
        }
        frustration = 0;
        goals.clear();
        return path;
    }
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

ps: I also initialized `path` with an appropriate size, to minimize resizing of the array backing the array list.
pps: I did not check other methods changed in this commit.
ppps: I did not test the code above.


`update_user` and user_service.update names seem misleading since now they take care of both create and update.


1.0 的時候 Local 會跟 Skirmish 和 Ranked 出現在一起，是第三個選項，但是沒法點，呈現灰色的


I was more inclined to make these all capitals like AUTH instead of auth.

Thoughts?


This should probably be


You should allow the linkage from `main.c` to several implementations as comparisons.


You are welcome :)


I love Tux even on normal boot. Reminds me when I first boot Redhat 6 :)


@hasufell nah @klausman is just preventing tmux from linking to a specific version of libevent. See the bug in question and this thread https://groups.google.com/forum/#!msg/tmux-users/fTJCwcSEwvw/Rbmwyt1dBAAJ.
@mgorny does it really matter?


The point is, i can work mainly on my particular remote.

But sometimes, like changing a single blend file, or making a single texture (as i done with @wlbragg), i can prefer change the upstream skis branch directly.


Top! Ik zal het checken zodra ik terug ben

On Wednesday, 7 October 2015, fadyyecob <notifications@github.com> wrote:

> Bij het selecteren van 1 connectie worden alle connecties geselecteerd.
> Waarschijnlijk mist er hier een check of klopt deze niet helemaal.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VincentBeltman/IlwisCore/commit/35202d25260291b05c314e8d563fcbd17e64c7b0#commitcomment-13632699>
> .
>


@RideShare-OZ-orange should probably end files with a new line to be defensive. Broad issue across all files. You can probably modify your IDE to do this. http://stackoverflow.com/questions/4700312/empty-new-line-at-the-end-of-the-java-source-files


@RideShare-OZ-orange should avoid commenting out code. Version control will keep track of old code for you.


Fore some reason anchor generated for that section is invalid (contains a space):
<img width="310" alt="screen shot 2015-10-07 at 10 16 57" src="https://cloud.githubusercontent.com/assets/985504/10332298/a42cfe64-6cdc-11e5-91e3-a033b5ad6b7f.png">


Could you rename the variable as well (and replace the other occurrences)?


Du kan pröva med en annan bakgrundsfärg


DAFUQ IZ DIZ????????????????!!!!!!!!!!!!!!!!!!!11111


That's intentional because it's just a forward reference to  the spec text with the normative SHOULD (which we do not want to duplicate).


========== Build: 12 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


line


@RideShare-OZ-orange consider extracting this code out to a method to keep methods short and readable. http://refactoring.com/catalog/extractMethod.html


mek it mor pretti


Ну это скорее относится к концепции создания отдельных моделей для каждой роли, дублирование полей, как в примере аватара, как в админа, так и в продавца. Основной совет, который уже звучал на всех обсуждениях, использовать  Form objects


I mean, only on exceptional situations, like this, instead of renaming my remotes, to point to the original repository (not the fork one), i can address it by a well described push. Then, go back to my normal work on my remote.


1) Alt-Svc is optional, so we can't put a SHOULD in here. 2) That being said, this is just an intro with the detail being in section 2.4, so I believe we should say "can" here.


**no-undef**: "describe" is not defined.


Fixed #1


Change this to "name". I want "name" keyword to be used in the email template instead of "userName".


@jsquyres thanks for the review, i revamped that part the fix in #990
now configury only test for PMPI if the user wants to try to build f08 bindings,
only master aborts if usempif08 is requested and PMPI is not built (the test is done a bit later in configure)
i never backported this logic (that was quite a bit change iirc) to the v1.10 series
(e.g. configury does not abort in v1.10 is the requested fortran binding cannot be built),
shall i do that now or can it wait for v2.0.0 ?


Ta en hamburgare istället!!!


You should use the maven property ${elasticsearch.version} so that it works when the version will be updated:

`<workingDir>elasticsearch-${elasticsearch.version}/</workingDir>`


@RideShare-OZ-orange is final needed here? Was this added because there are concurrency concerns or to ensure that the lift is not modified? Seems potentially overkill to do it for a local variable


LOL, thats why now tux is my profile picture


Byt färg på paddlarna!!


useful comment thanks :)


Thank you @hazendaz for your work!
I made some messy commits, but found out it was the server id in the settings.xml and the deployment is working now.
https://github.com/mybatis/hazelcast-cache/commit/d9cdbb8db9a08298c4a76d1b7e3482f55fae3e0c
We may have to edit the server id again when switching to parent-26 or later.


@RideShare-OZ-orange I find this comment confusing. Can you clarify?


You can push changes like this directly to master if you like.


@RideShare-OZ-orange probably shouldn't have sysout debugs in the production app


use objects


Hi Josh,

Thanks, I still have to learn my ways around the platform, probably this is why this didn't occur to me...

There are probably many such trivialities still lurking around.

Thanks for fixing,
Peter


@RideShare-OZ-orange have you got test coverage to encode this business rule?


wow


Yes: it has been identified and fixed


@RideShare-OZ-orange is this comment accurate? You said earlier that it's not always the suburb name


if this is just css for the calendar, why is it in application.scss?


this will give you com/pratilipi/email/EmailUtil/template/ - which is not a correct path.


Warum sollte das nicht funktionieren? Die Rolle "firewall" installiert ein Init-Script/systemd-Unit/whatever, das z.B. `/etc/firewall.d/*` oder so der Reihe nach abarbeitet (irgendwie nach IPv4 und IPv6 getrennt, aber idealerweise mit der Möglichkeit dass Dateien auch für beide Protokolle gelten) und die anderen Rollen legen dann Dateien in `/etc/firewall.d/` an. Da muss Ansible gar nix tun.


Great thank you very much for this fix.


Git sum


empty classes party!


Varför?


@RideShare-OZ-orange would recommend avoiding abbreviations to avoid confusion. latitude and longitude have less scope for error and it is trivial to auto-complete long variable names using modern IDEs.


@RideShare-OZ-orange is this a to-do?


Finish  README


We could make kappa_multinomial_stats an internal function that is called within kappa_multinomial.


Fix #1


The text is wrong :)
And this should be a localised text.


example added


why not using some method like "apointment_services_params" to clean this up?


@mekdrop : do you have a time frame for a first tutorial? It doesn't have
to be finished, but something with the basics would help us along :-)

Op zo 27 sep. 2015 om 10:25 schreef Raimondas Rimkevičius <
notifications@github.com>:

> Oh, that was unplanned for me for me... thanks @skenow
> <https://github.com/skenow> ! :)
>
> I plan to write a tutorial how to use this feature. Maybe the next week...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ImpressCMS/impresscms/commit/478cbd2028a19ebcdb910e4d173bcabf2b77dcd1#commitcomment-13455209>
> .
>


Finish README


see src/Graviton/FileBundle/FileManager.php::120 » only the first uploaded file will be recognized .. so the Location header is vaild.


it could be extended to handle raster objects....


colors


@RideShare-OZ-orange is this a magic number?


You also want to hide the wallets, if there are now accounts?

E.g if in mist i remove permissions, would you want to hide my wallets? This could be very confusing


det sämsta jag sett på länge


thanks


what's goin on there? is this line drunk?


Since, these variables are being used inside try block only. Move both declaration to inside the try block.


nicks


make this an internal function?


@RideShare-OZ-orange this should probably go in an issue tracking system. Are you using JIRA?


The `implode()` here looks very dangerous nevertheless.


@RideShare-OZ-orange I find this variable name a bit confusing. What is the m prefix?


This check has still to be there, as this is tresting wether you are on the main or test net


Better use `EthAccounts.find().count() > 0 `


Do you know the output of this generator? Maybe the license header is included somewhere in the generator output.


why "&" ?


Apart from that comments, it's pretty nice


Very good point. I was thinking about a singleton which locks when acquired and unlocks through RAII. Since SQLite3 already has threadsafety, that would probably not be necessary or can be done in user code.

Thanks.


@RideShare-OZ-orange confused by this comment too


Let me explain it better.

Suppose i am doing a lot of untested changes on my remote, and someone contacts me to do a simple remap on a almost ready blend file he is working.

First of all, i push all i have done to my remote.
Now i pull from upstream, to get the current version of the blend file
And then, i can :

(1) Push it to my remote and tell this to my friend
(2) Rename the remotes, making upstream my default origin, and push
(3) push directly to upstream, without renaming.

Never mind.

It is just me learning a new stuff and doing cheaper philosophy about it. :-)


Git sum


@RideShare-OZ-orange consider using the command pattern to avoid the big conditional statement


Looks good :D


Declare "String line = null" before the while statement.


Sure, in that case you can have two remotes :smile:


I used rdirichlet for this instead.


re-throw an exception here.


'Spinner': 'スピナー（回転）',


Please note that this has been done in: https://github.com/gregjacobs/Autolinker.js/pull/123


'Form Control': 'フォーム・コントロール',


im gonna kill you


You need to make a habit of doing a self-review before as well as after submitting your code changes.


related: https://github.com/galetahub/ckeditor#deployment


It's better to define maven properties here, so that we can override these values it if needed, for the CI for example:

Example for *minPortNumber* element:
* Define a property in the **social-rdbms-addons** pom: `<es.port.range.min> 9215</es.port.range.min> `
* Use it here `<minPortNumber>${es.port.range.min}</minPortNumber>`
* Then it is possible to override it like that:  `mvn verify -Des.port.range.min=9415`


@RideShare-OZ-orange perhaps replace all sys out.printlns with `Log.d`


@Abhinav1997  @the-dagger  Thanks for the support.


@RideShare-OZ-orange //todo? Is this in the issue tracking system?


kkkkkkkkkkkkkayyyyyetgsdfvzfvsd


Comment


ok, point taken


+1.Flappy Bird : No matter the current velocity, the bird will gain the same amount of height when the screen is tapped (T/F)T
+2.The bird accelerates due to gravity; i.e. vertical velocity is always increasing downwards.(T/F)T
+3.The rotation of the bird is correlated to its vertical velocity. Animation (flapping) only occurs when moving upwards. (T/F)T
 4.counter strike : kecepatan lari saat membawa knife adalah 250 units/seconds (T/F)T
 5.kecepatan lari saat membawa pistol adalah 245 units/seconds (T/F)F
 6.kecepatan lari saat membawa awp adalah 210 units/seconds (T/F)T
-
+link 1-3 http://www.kilobolt.com/day-1-flappy-bird---an-in-depth-analysis.html
http://gaming.stackexchange.com/questions/114991/how-to-speed-up-your-movement-in-counter-strike


@RideShare-OZ-orange suggest removing this code if no longer required. Git will keep track of it for you if you need it.


fitta...


agreed .. will change this.


fast-forwarding 2.1 to borshop-integration-1260-refactor/hide-bags-absolutely = 5c6625b6


Det ser ut som en 5åring har gjort denna skit


yup


Test comments. Can you see me?


I would say that some forgot to do that.


Avoid white space in pom.xml, use hyphens instead.


@RideShare-OZ-orange suggest removing author attribution (git will show individual contributions) and replacing verbose comment with descriptive method name `public void handleOfferRideButtonClick`

Suggest you avoid using comments that explain what the code does and instead refactor code so it's clear on its own.
http://programmers.stackexchange.com/questions/1/comments-are-a-code-smell


good


Thanks for reporting. I have never seen this problem as both my project and testing framework require 'securerandom' somewhere else.

This PR fixes the problem: https://github.com/snowplow/snowplow-ruby-tracker/pull/87


This commit was not tested and it doesn't solve the problem:
```
CMake Error at CMakeLists.txt:12 (cmake_policy):
    Policy "CMP0020" is not known to this version of CMake.
```

Why policy `CMP0020` is enabled?

I would prefer to create a pull request to get review before merging.


TeamCity pull requests :: netty [Build 379](https://garage.netty.io/teamcity/viewLog.html?buildId=2107&buildTypeId=pulls_netty&guest=1) is now running


I did the same but
```
if (layout.columns !== this.$$scope.layout.columns) {
```
https://github.com/sunshineo/angular-deckgrid/commit/66582102d9f8a58e551717e37ae7a616d7b52eb2


Думаю, сделать базового юзера и конкретных юзеров наследников (админ, продавец, гость) с STI и Form object'ами.


@RideShare-OZ-orange Again consider using the command pattern (http://www.javacodegeeks.com/2012/11/by-your-command-command-design-pattern.html) or look at whether this class is potentially becoming a god object and should be split up so that each view can handle its own clicks


Delete commented out code


<fd>


che dal'we delat'?


TeamCity pull requests :: netty [Build 380](https://garage.netty.io/teamcity/viewLog.html?buildId=2108&buildTypeId=pulls_netty&guest=1) is now running


This revision is clean.


@RideShare-OZ-orange consider moving this to another file. Seems like this file is becoming a god class violating the single responsibility principle.


Utiliza mejor la plantilla jsonlp, que aporta más información sobre lugar, fecha y todo eso.


This is one of those things that just seems so obvious, now that you say it... :bow:.


после добавления этой строчки всё ломается (


Shouldn't this be `ez_content:viewContentAction` or `ezpublish.controller.content.view:viewContentAction`?

There's no service called `ezpublish.controller.content`.


We should probably do this in constructor since it won't change. And maybe **isArray** is a better name here?


To break out of the while loop after reading 3 lines. When all the variables are set, reading next couple of lines is not needed.


Feel free to squash this into `1de6c8722de3b7b4d84c4ee512b0cf6ba6d7089f`


@RideShare-OZ-orange would suggest refactoring to the below to avoid comment. Comments are a maintainability risk
```
bool successfullyRetrievedIDToken = result != null;
if (successfullyRetrievedIDToken)
{
...
}
else
{

}


It works perfectly.


@WHPODOO : who are you ?


From [webpack documentation](http://webpack.github.io/docs/plugins.html#emit-async):

> emit(c: Compilation) async
> The Compiler begins with emitting the generated assets. Here plugins have the last chance to add assets to the assets array
>
> after-emit(c: Compilation) async
> The Compiler has emitted all assets.

So we use "after-emit", because we might possibly miss assets using "emit".


I'm lacking for any idea how to write this tutorial :|

Also, I think about make some update to the code before the tutorial.


Port 50000 is the jenkins port that is used to connect slave agents.

Why are we removing it from the jenkins service and we add it as container port to the jenkins swarm client?


This is one way to do it, yes :)


@RideShare-OZ-orange confused? Why are we using openweathermap?


typo in title : /

should be **isolate http basic auth key**


@akbertram here is change


@RideShare-OZ-orange hardcoding URL feels like a bad idea. Can you change to `endsWith('ride/create?driverid=")`?


Can they be both places, maybe we should add a disclaimer on the project page, "Select a suggestion to vote on it"?


@RideShare-OZ-orange confused by OpenWeatherMap. Make sure comments are relevant. Would suggest extracting this block to a method with a descriptive name and kill the comment as per Uncle Bob's clean coding principles.


TeamCity pull requests :: netty [Build 380](https://garage.netty.io/teamcity/viewLog.html?buildId=2108&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 3412, ignored: 12; compilation error: Compiler (new) Build time: 00:06:29


TeamCity pull requests :: netty [Build 381](https://garage.netty.io/teamcity/viewLog.html?buildId=2109&buildTypeId=pulls_netty&guest=1) is now running


這CSS也太亂，用到的根本只有兩個阿 囧


> "UNKNOWN" is to just shut up false compiler warning.

This is unclear behaviour.
What if you add new Type entry and forget to update ```type_str```?


you shouldn't need a new instance of Text - just use PROTOCOL_STATUS_CODE_KEY directly.


It should also have `contentId: \d+` requirement, to allow fallback to `/content/view/full/42` URLs.


@RideShare-OZ-orange will this impact performance? Consider putting this on a debug flag.


JAR diff at ~keghani/GitHub_diffs/20151007_02/java-build/1.0-SNAPSHOT-jar-with-dependencies_old_to_1.0-SNAPSHOT-jar-with-dependencies/changes_report.html


weather data again?


Promoted 7-10-2015


@RideShare-OZ-orange variable name is a bit unclear. Can you give it a descriptive name?


@RideShare-OZ-orange name is unclear. Do what in background?


Done


Thanks for spotting this @bamos !


@RideShare-OZ-orange can you delete commented code and remove unnecessary whitespace? >1 line = unnecessary imo


-.link:hover {
-	color: #888888;
-	position: relative;
-	top: 1px;
-	left: 1px;
-}

這個好像要留
不然不會位移


那個時候就太急拉~~~QQ


@RideShare-OZ-orange as mentioned - very long file. Suggest you look at how you can split it up.


You should be able to remove the `<executions> </executions>` block as it is already defined here https://github.com/exoplatform/maven-parent-pom/blob/master/pom.xml#L1335L1350 in the same maven profile.

To be sure, you can check it with this command `mvn help:effective-pom`


@RideShare-OZ-orange am confused as to the purpose of this file?


```enum class``` is no about safety. Well in general it is. But in this case using ```enum class``` allows anyone to pay less attention when they add new entry to ```Type``` or ```Status```.


Why was this needed?


any asserts after this ?


I don't like too much this import inside the function...


@RideShare-OZ-orange why MyRequest? Would RequestQueue be more appropriate?


@omerkirk updated, can you have a look?


should we give a default value by get_tag? like before this commit, the value will be '' if the given tag not found.

or add something like
```
if name is None:
    continue
```
otherwise if given tag not exist, there will be exception


@flavour Isn't the check done here: https://github.com/flavour/eden/blob/master/modules/s3db/gis.py#L2838

This would give me error as this is already loaded as list?


@RideShare-OZ-orange why the m prefix? Will make it harder to auto complete variable names


因為位移沒有什麼意義，對UX也沒有任何幫助XD


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


What is the benefit of this preference?


@RideShare-OZ-orange author names should come from git rather than from comments imo.


好吧~~QQ


@RideShare-OZ-orange commented code


me too


@RideShare-OZ-orange NB: most files end with a line break so could consider making this a convention


I find the naming of around/around_index.html and around/_index.html a bit confusing - maybe there's a better name for the latter? (It seems like it's really the form?)


@RideShare-OZ-orange necessary to override default constructor?


why the revert? I think unbearable is correct.


bardzo podobne do mojego kodu :)


Using out parameters rather then return value makes your code less clear for a reader.


@RideShare-OZ-orange comment seems unncessary


if return None, then all the callers must make sure they are handling the case that returned type is NoneType, so maybe add a default value will be better


ja mam 2 spacje  odstepu


This commit is being reviewed as a part of review issue #4.


@troyliu0105 fix 了两个小 bug
1. 发送日志前，需要判断是否存在 log.（『现在一切正常哦』这句话我先随意写的，你换成你想提示的话）
2. 数据获取方式默认值没改成『http+』

我再测下其他的， Update log 记得增加新功能的截图


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.39](https://pixelated.design/build/viewLog.html?buildId=146&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:09


@RideShare-OZ-orange suggest extracting this block out to a method with a descriptive name e.g. `checkRideType`


Should be of type integer.


@appleguy Thanks for you interest in our library! We haven't released our Pod yet, have to improve our public API a bit, and ofc we're super excited to try ASDK 2.0 new features. Made some changes in https://github.com/hyperoslo/Wall/pull/95 https://github.com/hyperoslo/Wall/pull/96 to be up-to-date with ASDK master in our Tests and Example projects, and we're pretty sure that our 1.0 release will be pointed on ASDK 2.0.


This one too.


This is actually adressing: https://jira.ez.no/browse/EZP-24893


I have done it already, but because of git problem con't upload it


*[-4, 0] typo from description...


@RideShare-OZ-orange recommend extracting this to its own method as per Robert Martin's clean coding principles.


@RideShare-OZ-orange is this in an issue tracking system?


The `memset()` init is called frequently in other functions as well. So the better solution would be introducing static inline helper functions to `value.h`. There it is also easier to maintain if they do the right thing. Names could be `zero_match_flags()` and `zero_value()`.