I think this version should be better.
If it's a new change we won't loop into the existing states. And if it's a modification we will loop and fallback for the non-existing case.

```
    protected function updateState($type, $change) {
        // Change can be a change or an add
        if($type == "change") {
            $this->syncstate[] = $change;
        } else {
            $change_id = $change['id'];
            foreach ($this->syncstate as $i => &$state) {
                if ($state['id'] == $change_id) {
                    switch ($type) {
                        case 'change':
                            $state = $change;
                            break;
                        case 'flags':
                            $state['flags'] = $change['flags'];
                            break;
                        case 'star':
                            $state['star'] = $change['star'];
                            break;
                        case 'delete':
                            array_splice($this->syncstate, $i, 1);
                            break;
                        default:
                            throw new Exception("updateState: type '$type' is not supported");
                    }
                    return;
                }
            }

            $flags = empty($change['flags']) ? "<no flags>" : $change['flags'];
            $star = empty($change['star']) ? "<no star>" : $change['star'];
            $mod = empty($change['mod']) ? "<no mod>" : $change['mod'];
            ZLog::Write(LOGLEVEL_WARN, sprintf("updateState: no state modification found!!! %s|%s|%s|%s|%s", $type, $change_id, $flags, $star, $mod));
        }
    }
```


thanks for padding bro!


@ariya

Do you have a plan to apply this change to harmony branch?


which one should i delete?
![capture](https://cloud.githubusercontent.com/assets/11464822/6652867/86ccd712-caa8-11e4-93b7-818c9e72aee4.PNG)


Oops sorry, thanks for fixing it.


Why this lib?


Completely remove the spaceminer repository, then re-fork it :)

Rob

On Mar 14, 2015, at 6:17 PM, "Anup Barua" <notifications@github.com> wrote:

> which one should i delete?
>
>
> —
> Reply to this email directly or view it on GitHub.
>


parts of speech tagging. note to self: automate check for TreeTagger scripts and parameter files or initiate download later..


@joolswills
We were both referring to "systems", not "programs"

If you feel this strongly regarding root usage by end users, i'am just curious why it has taken you 2+ years of work on this project before enforcing that choice.


Because VM is a time traveler and pushes from tomorrow.


hmm it feels like we shouldn't allow the user to make that kind of request.
But to your point the error would give them wouldn't be the correct one.
Perhaps we need to be checking for this kind of invalid request elsewhere in the code.
But cool, thanks for explaining your thought process to me.


“类列表类型”可以考虑改为“类似于列表的类型”或者“与列表相仿的类型”。


Андрей, надеюсь, что Вы не пугаетесь моего большого количества вопросов. Мне нравится, как Вы работаете, поэтому хочется, чтобы в коде все было ну если не идеально, то почти))
--
С уважением, Кравчук А.И.

> On 14 Mar 2015, at 21:03, andrew-minich1 <notifications@github.com> wrote:
>
> О, спасибо что посмотрели, ошибки исправлю, вроде бы ничего серьёзного, всё
> понял. Я вот кстати до этих курсов вообще не понимал для чего интерфейсы
> нужны, а теперь понимаю что без них вообще никак)
>
> 14 марта 2015 г., 20:00 пользователь AnzhelikaKravchuk <
> notifications@github.com> написал:
>
> > Название интерфейса IGetDiscount лучше заменить на IDiscount, вообще,
> > интерфейсы, если вспомнить FCL именуются чаще как причастия -
> > форматируемый, сериализуемый и т.д. или как существительные ICompare
> > (компаратор) , а вот методы наооборот, как глаголы или словосочетания.
> > Здесь все наоборот.
> > Опять реализация интерфеса оказалась не в том проекте
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/andrew-minich1/ASP.NET.1501.Minich.Day4/commit/93734367e7d64d46a9c1fdb0e97490f7d122a212#commitcomment-10199352>
> > .
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/andrew-minich1/ASP.NET.1501.Minich.Day4/commit/93734367e7d64d46a9c1fdb0e97490f7d122a212#commitcomment-10199634>.
>


``\ *递归*\的`` ，这里的“的”字前面少了个空格。


Uh oh! But how do we know is a line?


It's unlikely that @response_set is not set by the time you get here.  It should be getting set in the controller.  If you are generating a report, should bounce out here https://github.com/kjayma/surveyor_gui/blob/master/app/controllers/surveyor_gui/reports_controller.rb#L35.  If you are displaying a single survey response, should bail here:https://github.com/kjayma/surveyor_gui/blob/master/app/controllers/surveyor_gui/responses_controller.rb#L33


no, I think open sans is ugly so I want it to be the last resort.


cc @jeffmo re @Constellation 's question.


If we make this change it will mess up the styling.


“当我们使用 `Cons` 构造器来创建一个值时”可以考虑去掉其中的“来”，句子的意思不变。


The href at least references a rails path, so I don't


Ah, good to see `:init` being fixed! :+1:


This line here throws an NPE for this model: https://www.mediafire.com/?z87689iz8uk8hbb
What could be the problem here?


Nil 之后应该加空格，并且应该用 \` 包围。


You forgot to update the doc:

> Use the :init keyword to do some stuff to initialize foo. If loading was deferred, the code is run immediately; otherwise the package is required before running the code. See below for options that defer loading of the package.


@bors: retry


I'll add Helvetica Neue for newer versions of OSX


saw approval from jdm
at https://github.com/yodalee/servo/commit/314bf795e8744199a7754d4f8bca6c98c7a146de


merging yodalee/servo/issue4768-textencoder = 314bf795 into auto


yodalee/servo/issue4768-textencoder = 314bf795 merged ok, testing candidate = 0b2d9839


Merci !! ^^

Par contre je remet les strongs, les balises <b> ne respect pas la norme W3C ;)


“……有两个叶节点（叶节点这个词习惯上是指没有子节点的节点）的树”这里可以考虑把括号的内容移到句子的末尾。


Tree 、 null 没有使用 \` 包裹。


“仿造 Java 示例里那样”这里应该为“仿照 Java 示例”。


Why this change?
casting to bool or boolean should be equivalent even using HHVM
http://docs.hhvm.com/manual/en/language.types.boolean.php

> To explicitly convert a value to boolean, use the (bool) or (boolean) casts


Don't write 'Git' in all caps.


I was asking myself the same question... Hence I was wondering if converting a line from `BufferGeometry` to `Geometry` made any sense...


Why? Storage is much more generic than database.


I haven't been working on the project for 2 years.

It was fixed as it cropped up that people were running es as root accidentally after install, then rebooting and having a non working set up as it was then launched under the pi user. I added this as an additionally safety to avoid this.

I haven't enforced anything. I have made the retropie shortcut, created by out system do an additional check to protect users from making a mistake. you are still free to change it to suit your needs.


If the initialization fails, shouldn't this just abort?


move oc constant keys to OcMessageConstants class


chane wos2 -> wso2


Wow much comment. Such info!


You should use multicatch.


Use a guard clause instead of wrapping the code inside a conditional expression.


And proper logging.


TeamCity Akka.NET :: Akka.NET PR Build [Build 153](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=257&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Hola Senior


Lets hope no one hits this. Converting `BufferGeometry` to `Geometry` is, hopefully, a temporal thing while we figure out how to make `BufferGeometry` more noob friendly.


check serverId is needed or not


this is rather confusing I think (you don't say that this is about the script field, people could think it concerns the other fields too)


Is this equivalent to "#{t}"?


new merge


That's not transparent at all xD


[Info]: clif_parse: Disconnecting session #8 with unknown packet version (p:0x068a,l:19).

Client: 2013-08-07Ragexe


Добрый вечер.
Мы пишем метод сортировки, который сортирует строки(!) матрицы согласно некоторому ключу, значение которого получается на основе обработки строки матрицы (она же одномерный массив). Поэтому в качестве параметра компаратора (название интерфейса ISortModes не очень удачное, лучше а-ля IComparer) следует передавать интерфейс, у которого метод имеет прототип вида int IComparer (int[] x, int[] y). При этом, класс, который реализует этот интерфейс и передается в метод сортировки уже в себе содержит логику сравнения, поэтому никаких дополнительных булевских параметров метод сортировки принимать не должен. Постарайтесь изменить с учетом указанных замечений.


Need to use .items() instead of .iteritems() for the tests to pass in Python 3


Fixes #1


#The following will disallow sudo but allow root. That resolves the issue of letting the user choose, and, prevents sudo being used.

username=$(who am i | awk '{print $1}')
userid=$(id -u)

if [ "$userid" = 0 ] && [ "$username" != "root" ]; then
        echo "Please run without sudo."
        exit 1
fi


haberi yok henuz :smile:


thanks - but I am going to keep it as is for now, as retropie is still designed by default to be run under a specific user.


I think it should be `../0000-msys2ize.patch`, or `${srcdir}/0000-msys2ize.patch`.


I'm not quite good in expressing myself in written based except oral..can you help me with this? hehehehe


@martinsvalin :+1: https://github.com/himynameisjonas/spotbot/commit/ba6e4f31443de49d1174508008715151ae0293c7


Fixed!


I have not tried again lately, but initial development was done with gss-ntlmssp testing and it should work.
Do you have GssapiConnectionBound turned On ? That's the only limitation GSS-NTLMSSP has for now, it cannot work unless you turn that one on, as it does multiple roundtrips and we need to stach the incomplete context between requests.


don't hard code this in a string - just state: "unknown RSpec type"


\\@megawac

I don't think the bug is reproable in the iOS simulator which is what Sauce uses.


It need to be
````patch -p1 -i ${srcdir}/0000-msys2ize.patch````


@joolswills
Considering this change was only made today, i disagree with your comment. You claim " retropie is still designed by default to be run under a specific user", yet it never was, until today.


Add "r" before number


Does this break build with VS2013 < Update 3?

I'd also mention:

>The /Zo switch is available in Visual Studio 2013 Update 3, and it replaces the previously undocumented /d2Zi+ switch. We recommend you use /Zo instead.


How's this better?


And in Cata? And see you in TBC already added.


yeah I actually went further and renamed newBuf => buildBuffer, newCBuf => newBuffer, but wasn't sure how far we wanted to go with it all. Thoughts on those names?


how would this line look like if we'd support 20 different kind of RSpecs? Could you think about a better way of checking if there is a delegate that supports the according RSpec? Maybe using reflections?
@dnehls : ideas?


it was never designed to be run under root. Nor should it be. That is why a lot of work is done to get the file ownerships correct around the code, and in various modules.

I don't see why you are making such a fuss about this, you already know how to run it as root, so carry on.


okay that's fine, although a part of me feels like these are the base cases for reading frames and its useful to know when those most basic of cases has failed. I might argue to leave them in for now, and remove them along with a larger debug cleanup prior to a larger release


I may be missing something but this makes the Simple Oauth 2.0 (linkedIn) example fail with `NullPointerException` because `getRequestInitializer() == null`. This is expected since there is no statement setting the request initializer when setting up the AuthorizationFlow in `SimpleOauth2ExplicitActivity onActivityCreated`.


some tests failed:
failure: http://build.servo.org/builders/mac2/builds/1228


Fixes #2


Just had a thought - would it be better to have `$order_number = $order->get_order_number();` as a global variable within the plugin?

There are 3 other instances using `. $order->get_order_number() .` which could be replaced.

I don't really know which is better/faster/neater but thought I would mention it (I have no idea what the "best practice" is here for PHP).

Also it is probably worth noting that once this fix is pushed out in a subsequent plugin update, the following Knowledgebase article will become obsolete - https://support.bitpay.com/hc/en-us/articles/203032559-Mismatched-order-invoice-numbers-on-WooCommerce-when-using-Sequential-Order-Number-plugin


Worked!  I was focused on the particulars of the syntax I hadn't thought to go refer to the docs again. Now I know. I added these changes in the [next commit](https://github.com/woodall/learn-requireJS/commit/996f0ef34a9d706ac1471e4b9bac7031de8fde0d). Thank you again. HIGH :hand:


@eomuraliev,
Worked!  I was focused on the particulars of the syntax I hadn't thought to go refer to the docs again. Now I know. I added these changes in the [next commit](https://github.com/woodall/learn-requireJS/commit/996f0ef34a9d706ac1471e4b9bac7031de8fde0d). Thank you again. HIGH :hand:


@jdonaldson DONE [3369 tests]


FYI, assimp internal importers oftentimes use <> or $ for generated nodes. It's not very consistent, but something we do and thus all code should be able to handle it.


I was somewhat expecting this ...
Our (semi)automatic tools that run on the server silently overwrite the additions madre recently, so I would suspect that the same warnings will show up again in older versions of the gettext tools.

Sorry, I dont think there is anything we  can do here :(


Usually you would declare the variable here and initialize it in a constructor. So, just Random random; here and random = new Random() inside the constructor.


Make sure your client doesn't have the diff to remove packet obfuscation. Also make sure packet_keys_use matches your packet_db_ver. Since you're using 2013-08-07 you need to define packet_keys_use as 45.


While we're at it - is there any other special characters we should filter out here? What about '{' and '}'?


What I most often see people sacrificing readability for is what they believe to be optimization. It seems hard to pick a best practices guide here, and what would be appropriate depends on the experience level of the contributor.


It will come to Cata soon.
I will finish ToC25 in a few days then I'll push everything to cata


You can keep the `const` here. Just change the `aiString&` you pass to `toXFileString()` to be a `const aiString&`.


There is no need to create a new variable here. You can directly use randomNumberToPrint. Since the function is just calculating the mean of an array of numbers, the variable can be named something like numbers or samples


Again, you don't need a new variable; you can directly use randomNumber.length when you need it.


It's best to declare one variable per line.


ок


I'm not sure why you wouldn't just use `if (value)` here


yes, I implemented the filters previously, but no theme could not actually use them because they where fired before the `functions.php`-file was loaded :/ kinda meaningless filters then.. so i'm trying to load them a bit later and see if that works


Да мне наоборот очень нравятся курсы! я ведь говорил что у нас на вечернем
вообще программированию не учили, и не у кого что то спросить, или кто бы
сказал как правильно, а тут вот как раз ответы на все вопросы. Так что
спасибо Вам)

14 марта 2015 г., 21:22 пользователь AnzhelikaKravchuk <
notifications@github.com> написал:

> Андрей, надеюсь, что Вы не пугаетесь моего большого количества вопросов.
> Мне нравится, как Вы работаете, поэтому хочется, чтобы в коде все было ну
> если не идеально, то почти))
> --
> С уважением, Кравчук А.И.
>
> > On 14 Mar 2015, at 21:03, andrew-minich1 <notifications@github.com>
> wrote:
> >
> > О, спасибо что посмотрели, ошибки исправлю, вроде бы ничего серьёзного,
> всё
> > понял. Я вот кстати до этих курсов вообще не понимал для чего интерфейсы
> > нужны, а теперь понимаю что без них вообще никак)
> >
> > 14 марта 2015 г., 20:00 пользователь AnzhelikaKravchuk <
> > notifications@github.com> написал:
> >
> > > Название интерфейса IGetDiscount лучше заменить на IDiscount, вообще,
> > > интерфейсы, если вспомнить FCL именуются чаще как причастия -
> > > форматируемый, сериализуемый и т.д. или как существительные ICompare
> > > (компаратор) , а вот методы наооборот, как глаголы или словосочетания.
> > > Здесь все наоборот.
> > > Опять реализация интерфеса оказалась не в том проекте
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/andrew-minich1/ASP.NET.1501.Minich.Day4/commit/93734367e7d64d46a9c1fdb0e97490f7d122a212#commitcomment-10199352>
>
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub <
> https://github.com/andrew-minich1/ASP.NET.1501.Minich.Day4/commit/93734367e7d64d46a9c1fdb0e97490f7d122a212#commitcomment-10199634>.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andrew-minich1/ASP.NET.1501.Minich.Day4/commit/93734367e7d64d46a9c1fdb0e97490f7d122a212#commitcomment-10199722>
> .
>


think u should commit this first, that way itll ignore the .idea or any file ending in .iml


If you look at what this function should do without considering the rest of the problem then it's input is a set of numbers and the output is the variance. So you shouldn't have to pass it the mean which you would have calculated somewhere else. It's best to calculate the mean inside the function (which you can do by calling the calculateMean function) so that you're sure the mean is calculated on the same set of numbers on which you're calculating the variance.


Same comments as earlier. No need for extra variables and one variable declaration per line.


The input to the calculateStandardDeviation function should be a set of numbers. You can call the calculateVariance function from inside to get the variance. This is where the block diagrams come in handy :)


Hmm, I'm not sure. Looks fine to me at a glance. I'll look some more and let you know.


make APIRequests a var in ur RetrofitRequest class, created a getter for it.
make it a singleton. check out how we do getNumiRequest()


:rainbow:


:+1: , i guess instead of void, return the Listing obj here


`new(this) ExportProperties();` is not necessary, in fact it causes a memory leak. Before the constructor is entered, all fields are default-constructed. Here, you re-run the default constructor on the already constructed object, which leaks any storage allocated by the maps.

A vanilla copy constructor that we can use here is this:

```
ExportProperties::ExportProperties(const ExportProperties &other)
 : mIntProperties(other.mIntProperties),
   mFloatProperties(other.mFloatProperties),
   mStringProperties(other.mStringProperties),
   mMatrixProperties(other.mMatrixProperties)
{}
```

which is almost the same (but better) than

```
ExportProperties::ExportProperties(const ExportProperties &other)
{
   mIntProperties = other.mIntProperties;
   // ...
}
```

However my entire point was that the first version is what the compiler automatically provides if you do not provide a copy constructor at all (and no default constructor either).

So if you just drop both constructors, it should still work. We can also keep them to clarify the intent that the object is copyable, but let's still use the initialization list syntax and drop the placement new.


30414/Dolineanu A. Vlad Ionut/Assignment 2/functions.h

neat little trick, but I don't think it's a good practice - a function for this however could be.


Might have been better to keep these as doubles as well, just so that you're not mixing data types when calculating percentage and absolute difference. But it ok here.


I see you've taken my advice about commit titles. Nice catch too. I would have been panicking not knowing why the lab check didn't work


Oh, as long as this is the way this is going, great! :)


try something more suggestive
```numberOfElements```
```length```
```size```


You can add documentation for each function by typing /** + Enter above the functions. A short description of what the function does and also an explanation of the parameters to the function and the return variable (if any) should be enough.


why 7.0? is that font/DPI dependent?


Works well!


No se porque lo detecta asi, igualmente lo unico que hice fue copiar el mapeo, pasarlo al map editor editar las rejas y volverlo a meter, igual hay unas cuantas lineas comentadas si no me equivoco


New data coming?


30414/Dolineanu A. Vlad Ionut/Assignment 2/functions.c

what if they are both NULL?
won't ```ourList->number``` reach -1 for example?


But why would you want that? The result would be that the manually edited lines are intended using spaces and the lines added by git use tabs. I generally prefer spaces for indentation, but still I would prefer spaces over a random mixture of the two indentation styles.


Suggest to bump up the test priority. .as I think this will pass


This would be better as a class variable


Okay, i wait, thanks!


It would be safer to use gaussianSamples.length than N. You also don't need to pass N to the function then.


When you look at the function independent of the rest of the class, it should just take a set of numbers and compute the variance. Passing the mean as a parameter means it is prone to errors (mean could have been calculated on a different set of numbers, could be wrong etc.). Calling the computeMean function from inside this function would be a better way to do it.


Just a suggestion: if you're already collaborating with the original owner of the repo, you're probably better off deleting this forked version repo.


You should pass sampleMean instead of trueMean.


piola entonces!
no, sabes pq preguntaba, porq a alan le habian pasado un mapeo, pero le habian pasado cualquier cosa, era el mapeo de la carcel, pero con un monton de removes de cualquier otras cosas, y encima removes de exteriores, pero sin el mapeo del exterior, habia espacios azules, etc, y por eso queria saber


test


nice work @Vincent-Michael ...but this "bug" - righ now pushed- seems to be only on @Vincent-Michael  commits


Documentation for the functions! :)


> Does this break build with VS2013 < Update 3?

Will throw `cl : Command line warning D9002: ignoring unknown option` but will compile. Of course enhanced debugging information won't be included in build compiled with < U3.

I can drop the path if you think we should maitain compatibility with older compilers. But my impresion was that we always encourage to use latest release.


Looks like you pushed to master without a PR.

In any event, you've got some debugging code here. The same don't-put-this-is-the-core-repo advice applies as with the parser.

Also, there's already a mechanism for passing list markers (see `list_type`) below. Why not just extend that to account for no markers?


Не нашла логику расчета возможной скидки на продукты в корзине.


Yes, will publish later tonight


Ignore the comment above. I looked it up and people recommend you initialize the variables the way you did it (i.e. not inside a constructor). They both do the same thing, but your way is easier to read since it's all in one line.


Thanks for your quick reply,

I will do that,  starting by checking out the latest versions of the GLXXX
packages (GLAbstraction  GLFW  GLPlot  GLText  GLWindow).

Best,
Alain

ps: I was not able to reproduce the issue concerning println+Reactive in a
minimal environment (only Reactive and Base). There everything worked fine:
I got a print and a backtrace. (I did not try a case simply with GLPlot),
so I was wondering whether there was a signal tied to the STDOUT or STDERR
(I suppose that STDIN is used to recover
keys pressed).

-----------------

#==
   This test is intended to show that try/catch works with signals. The
   idea is to use a function inside fold that will give run time errors.
==#
using Reactive
const  z = [0,1,2]
const  y = [10,11,12]

in=Input(z)
push!(in,y)
in

#this function is will throw, since * does not work
#when both args are column arrays
ftr1 = s -> foldl( (x,y) ->   x*y, z, s)

function ftr2(x,y)
     try
        x * y
     catch e
        rethrow()
     end
 end

function ftr3(x,y)
     try
        x * y
     catch e
        println("error:$e")
        catch_backtrace()
        backtrace()
        rethrow()
     end
end

f1 = ftr1(in)
f1
push!(in,z)

in2=Input(z)
push!(in2,y)
f2= foldl( ftr2, z, in2)
push!(in2,z)

in3=Input(z)
push!(in3,y)
f3= foldl( ftr3, z, in3)
push!(in3,z)

---------------------------
In the output, the printed diagnostics are quite useful (both with julia
0.3 and 0.4)

Warning: using Base.Graphics in module Main conflicts with an existing
identifier.

 in anonymous at /home/alain/test/BigData/JULIA-TRY/reactiveTest3bug.jl:16
 in anonymous at /home/alain/.julia/v0.3/Reactive/src/Reactive.jl:323
 in update at /home/alain/.julia/v0.3/Reactive/src/Reactive.jl:85
 in push! at /home/alain/.julia/v0.3/Reactive/src/Reactive.jl:245
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354

ERROR: `*` has no method matching *(::Array{Int64,1}, ::Array{Int64,1})
 in push! at /home/alain/.julia/v0.3/Reactive/src/Reactive.jl:271
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
while loading /home/alain/test/BigData/JULIA-TRY/reactiveTest3bug.jl, in
expression starting on line 39

alain@alain-linux:~/test/BigData/JULIA-TRY$ Julia4  reactiveTest3bug.jl
WARNING: uint(x) is deprecated, use UInt(x) instead.
 in depwarn at ./deprecated.jl:40
 in uint at deprecated.jl:29
 in anonymous at no file:33
 in include at ./boot.jl:250
 in include_from_node1 at ./loading.jl:129
 in reload_path at ./loading.jl:153
 in _require at ./loading.jl:68
 in require at ./loading.jl:51
 in include at ./boot.jl:250
 in include_from_node1 at loading.jl:129
 in process_options at ./client.jl:318
 in _start at ./client.jl:402

 in anonymous at /home/alain/test/BigData/JULIA-TRY/reactiveTest3bug.jl:16
 in anonymous at /home/alain/.julia/v0.4/Reactive/src/Reactive.jl:323
 in update at /home/alain/.julia/v0.4/Reactive/src/Reactive.jl:85
 in push! at /home/alain/.julia/v0.4/Reactive/src/Reactive.jl:245
 in include at ./boot.jl:250
 in include_from_node1 at loading.jl:129
 in process_options at ./client.jl:318
 in _start at ./client.jl:402

ERROR: LoadError: MethodError: `*` has no method matching
*(::Array{Int64,1}, ::Array{Int64,1})
Closest candidates are:
  *(::Any, ::Any, !Matched::Any)
  *(::Any, ::Any, !Matched::Any, !Matched::Any...)

*{T<:Union(Complex{Float64},Complex{Float32},Float64,Float32),S}(!Matched::Union(SubArray{T<:Union(Complex{Float64},Complex{Float32},Float64,Float32),2,A<:DenseArray{T,N},I<:(Union(Int64,Colon,Range{Int64})...,),LD},DenseArray{T<:Union(Complex{Float64},Complex{Float32},Float64,Float32),2}),
::Union(DenseArray{S,1},SubArray{S,1,A<:DenseArray{T,N},I<:(Union(Int64,Colon,Range{Int64})...,),LD}))
  ...

 in push! at /home/alain/.julia/v0.4/Reactive/src/Reactive.jl:271
 in include at ./boot.jl:250
 in include_from_node1 at loading.jl:129
 in process_options at ./client.jl:318
 in _start at ./client.jl:402
while loading /home/alain/test/BigData/JULIA-TRY/reactiveTest3bug.jl, in
expression starting on line 39

2015-03-14 13:00 GMT+01:00 Sdanisch <notifications@github.com>:

> Okay please checkout Romeo master and GLWindow master.
>
> This example
> <https://github.com/SimonDanisch/Romeo.jl/blob/master/test/simple_display_grid.jl>
> should explain how to make screens in a grid.
> It should also explain the nature of lift. Lift takes a signal (a value
> which can change over time) and registers a transformation/action, which
> will get applied to that signal.
> So as all window regions are signals, which ultimately stem from the
> window size, your screen grid changes when you resize the window.
> If you don't want them to change dynamically, just create a signal without
> varying dependencies,
> e.g. Input(Rectangle(0,0,50,50)).
>
> how to ensure the mouse actions for moving a plot come from same area as
> the graph.(right now I get mouse action to the right of the screen, drawing
> on the left)
>
> This shouldn't happen... Probably you defined windows which overlap?
> Otherwise I need a clear reproducible example.
>
> where is the use of signals / mouse actions to orient the drawing in
> plot_screen represented in the code
>
> It's in GLAbstraction, together with GLWindow.
> Some relevant code:
> reactglfw.jl
> <https://github.com/JuliaGL/GLWindow.jl/blob/master/src/reactglfw.jl> :
>
> #Screen constructorfunction Screen(
>         parent::Screen;
>         area                             = parent.area,
>         children::Vector{Screen}         = Screen[],
>         inputs::Dict{Symbol, Any}        = parent.inputs,
>         renderlist::Vector{RenderObject} = RenderObject[],
>
>         hidden::Signal{Bool}             = parent.hidden,
>         hasfocus::Signal{Bool}           = parent.hasfocus,
>
>         nativewindow::Window             = parent.nativewindow)
>     #checks if mouse is inside screen
>     insidescreen = lift(inputs[:mouseposition]) do mpos
>         isinside(area.value, mpos...) && !any(children) do screen
>             isinside(screen.area.value, mpos...)
>         end
>     end
>     # creates signals for the camera, which are only active if mouse is inside screen
>     camera_input = merge(inputs, @compat(Dict(
>         :mouseposition  => keepwhen(insidescreen, Vector2(0.0), inputs[:mouseposition]),
>         :scroll_x       => keepwhen(insidescreen, 0, inputs[:scroll_x]),
>         :scroll_y       => keepwhen(insidescreen, 0, inputs[:scroll_y]),
>         :window_size    => lift(x->Vector4(x.x, x.y, x.w, x.h), area)
>     )))
>     # creates cameras for the sceen with the new inputs
>     ocamera      = OrthographicPixelCamera(camera_input)
>     pcamera      = PerspectiveCamera(camera_input, Vec3(2), Vec3(0))
>     screen = Screen(area, parent, children, inputs, renderlist, hidden, hasfocus, pcamera, ocamera, nativewindow)
>     push!(parent.children, screen)
>     screenend
>
> Camera code is in GLCamera.jl
> <https://github.com/JuliaGL/GLAbstraction.jl/blob/master/src/GLCamera.jl>
>
> I actually only develop with 0.4, so this should work with 0.4... If not,
> try to checkout master in all GLxxxxx packages.
> Only runtest.jl and simple_display_grid.jl are tested by me. The others
> are just leftovers and I'd be surprised if they work.
>
> 1) I had to circumvent run time compile error diagnostics with too low
> level looping code in/src /visualize/color.jl ''ERROR: error compiling
> color_chooser_boundingbox: too many parameters for type Vector3''
>
> I'm not really sure what you mean by this... I've pushed a change which
> might fix this.
>
> To the last point: reactive + println is terrible... It's nearly never
> working correctly to print inside a signal.
>
> Best,
> Simon
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SimonDanisch/Romeo.jl/commit/2e04ab8617fb79dd7217570ab85a9919bb204269#commitcomment-10198203>
> .
>


+# Sources
+* .cpp


Csak így tovább!


merging basho/yokozuna/zl-yz-index-hashtree-eqc = f8a1724f into borshop-integration-445-zl-yz-index-hashtree-eqc


saw approval from seancribbs
at https://github.com/basho/yokozuna/commit/f8a1724f1b9f0297d7801d79a6dfd37d8b74a1f3


basho/yokozuna/zl-yz-index-hashtree-eqc = f8a1724f merged ok, testing candidate = 6f0281c4


Check.

Have a look to copy constructor of class Importer. There is the same memleak, i did not fixed.


why you prefer discover module ? and not nosetest or pytest

https://nose.readthedocs.org/en/latest/
http://pytest.org/latest/


Has no influence to code quality.


@bsmiles32 *conversation continued from commit 8265c54c17ea75e9e18a4e979d48245b556bddc8*

In this, I'd think the performance regression comes from the decoupling, which kills opportunities to:

* query the validity of a page before invalidating its physical alternates;
* revalidate a page which contents have not changed but which gets remapped later.

Run the code after this commit with this statement inside [`invalidate_r4300_cached_code`](https://github.com/Nebuleon/mupen64plus-core/blob/b07829dc32782df6e7e7c17a0db2032d23cffb7e/src/r4300/r4300_core.c#L81-L96) and see how many invalidations happen:
```C
DebugMessage(M64MSG_INFO, "Invalidating code at %08" PRIX32 " +%zi", address, size);
```

Commits like this one also justify the proposal I made for TLB accessor functions in R4300 emulators [1], because then the R4300 emulators can do specific actions to handle invalidating their code when something gets unmapped, and then handle *revalidating* their code when something later gets *remapped* elsewhere (or even at the same virtual address).

That, or TLBWrite should become something like `CHECK_MEMORY`, defined before including the opcodes but containing core-specific instructions to handle overwriting data seen as code.

[1] https://groups.google.com/d/msg/mupen64plus/4rOO7cX0toY/8f9yFmfpQf0J


Check.


you have already solved the problem .. unfortunately we must necessarily use this and not what else is on alliance because it is perfectly adapted to our skin
thaks andy


Thanks for giving me the heads up. Looking through these files now.


@stefano-k Please review this for inclusion in MATE 1.10


Lowercase `kb/s` more common than `Kb/s`. Another is `Kbps`, but `Kb/s` is rare imo.

Rounding is for the case when file have almost constant bitrate. There is no sense to change the displayed value if it dips below certain value by a fraction. Should give more stable output.


IntelliJ says that using that shouldn't ignore annotations.xml  because its dumb :-1:
I'll try adding it though


Should be myradio


The idea was to calculate the mean and standard deviation of the same set of samples. So you should generate only one sample set (for a given mean and std dev) and then calculate the mean and std dev on that (instead of generating a new sample set inside of the mean and std dev calculation functions).

So, for example, the calculateMeanOfSamples function should be passed an ArrayList of samples and the mean should be calculated on that. The same for the other functions.


`"T" "U"` doesn't compile. Thanks szatz from irc for pointing this out!


@dstogov @weltling The x86 VC11 build is broken now:
http://windows.php.net/downloads/snaps/master/re78971c/logs/make-nts-windows-vc11-x86-re78971c.txt
I am suspecting this patch, but the both of you are probably better in finding out what went wrong.


Thanks, I just added it.


My intuition tells me this code is going to be extremely buggy.  I think I saw with the holodeck code that setting .x, .y, and .z with vars[] leads to just unbelievably buggy byond behavior--objects thought they were in two places at once.

Also I am not sure why Remie thinks that numerous string comparisons for every var in the varlist is going to be significantly faster than creating and destroying simple objects.  I mean yes, that's slow, but that's a lot of string operations on top of the rest of the library's overhead.

The whole exercise strikes me as a bad idea.  That has nothing really to do with you, though.


Not sure how to interpret this name


gorgeous!


temp file


Seeing the message now. Thank You :-)


*because :^)


just leaving a note in case this line didn't get deleted. many more commits to look through


This is not the same pattern anymore...
```
>>> ur'\x00' == u'\x00'
False
```


disregard the previous comment. line was deleted


This commit is just like the one I did before - 2 commits instead of one? What is happening on my computer :confounded:


hmm, maybe this was why I moved to explicitly destroying events and streams rather than relying on the GC...


Is it reasonable to assume that we should not be calling this method and passing in the geometry of a line, which has no faces?


Found it. I did some OP_RETURN tests earlier, but I just saw those were not created with zero amounts.

This is the full blown check:
https://github.com/zathras-crypto/mastercore/blob/0.0.9.2-Z-ClassC/src/main.cpp#L570-L589

So how using that in combination or as replacement of:
https://github.com/zathras-crypto/mastercore/blob/0.0.9.2-Z-ClassC/src/wallet.cpp#L1265-L1269

...?

Might even make sense to remove all those checks from wallet.cpp and simply call `IsStandard(wtxNew, strFailReason)` at the end.

Nevertheless, you still have a valid point, unrelated to the wallet code, but I'd change the `if-else` to:

```c++
bool IsDust(int64_t nMinRelayTxFee) const
{
    if (!scriptPubKey.IsUnspendable()) {
        return false;
    }

    return ((nValue*1000)/(3*((int)GetSerializeSize(SER_DISK,0)+148)) < nMinRelayTxFee);
}
```


require"d" packages.


can you please try:
```python
ur'[\u0000-\u0008\u000B\u000C\u000E-\u001F\uD800-\uDFFF\uFFFE\uFFFF]',
```


You could align all these based on `=`


Good job using xpath :)


@timaschew @jonathanong have fixed all the issues I've been running into to get to standalone app bundles and have pushed my changes to the [`standalone/example`](/componentjs/bundler-example/tree/standalone/example) branch. It does depend on the custom implementation of require referred to in the earlier comment.

@timaschew thanks for handholding me through the process.


Good job!


I would be more explicit about this here and not use macro. The reason for this is - you want to show at least 1 feature spec what it takes to sign in. Also test `sign out` too.


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1148 of 1148 tests passed)

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


Use `unless` for `if !`


Travis just reported a failure... https://travis-ci.org/xmlrunner/unittest-xml-reporting/builds/54306673


A lot of file restructuring was needed for it to work... More will be needed to allow the game to be easily modified.


"both"?


que es lo que se modifico del mapeo?


Just so you're aware I tested this function fairly extensively both before and after it's merge.


Found BROKEN by full.
Here's the log file for step 'trunktesterbot':
```
python internal/test_py.py python -skipfiles="alltests.py helpers.py complexity.py speed.py" -passoptions="--ignore_memory_light --run_slow"
Running test in python
01. tools.py
02. qcqpsolver.py
03. linearsolver.py
04. control.py
05. misc.py
06. homotopynlp.py
07. qpsolver.py
08. typemaps.py
09. sparsity.py
10. symbolic.py
11. sdp.py
12. implicitfunction.py
13. integration.py
14. simulator.py
15. typemaps_futurediv.py
16. matrix.py
17. ocp.py
18. mx.py
19. nlp.py
20. vectortools.py
21. function.py
22. lpsolver.py
23. ad.py
24. socpsolver.py
25. sx.py
26. mx_jacobian_sparsity.py
Ran 26 tests, 0 fails.
python internal/test_py.py python -skipfiles="alltests.py helpers.py complexity.py speed.py" -memcheck -passoptions="--ignore_memory_heavy --run_slow"
Running test in python
01. tools.py
02. qcqpsolver.py
03. linearsolver.py
04. control.py
05. misc.py
06. homotopynlp.py
07. qpsolver.py
08. typemaps.py
09. sparsity.py
10. symbolic.py
11. sdp.py
12. implicitfunction.py
13. integration.py
14. simulator.py
15. typemaps_futurediv.py
16. matrix.py
17. ocp.py
18. mx.py
19. nlp.py
20. vectortools.py
21. function.py
22. lpsolver.py
23. ad.py
24. socpsolver.py
25. sx.py
26. mx_jacobian_sparsity.py
Ran 26 tests, 0 fails.
python internal/test_py.py '../docs/api/examples'
Running test in ../docs/api/examples
01. ssym.py
02. jacSparsity.py
03. generateCode.py
04. SXFunction_constr_py.py
05. symbolicsubstitution.py
06. countNodes.py
07. tolerance.py
08. simulator.py
09. controlsimulator.py
10. monodromy.py
11. idas.py
12. jacobian.py
13. dulmageMendelsohn.py
14. coloring.py
15. stronglyConnectedComponents.py
16. createParent.py
17. matrix_expand.py
18. veccat.py
19. expand.py
20. customfunction.py
21. parallelizer.py
22. limitPrinting.py
23. assertion.py
24. printme.py
25. dotdraw.py
26. callback.py
27. KinsolSolver.py
28. NewtonImplicitSolver.py
29. NLPImplicitSolver.py
30. exacthessian.py
31. ipopt.py
32. LinearSolver.py
Ran 32 tests, 0 fails.
python internal/test_cpp.py '../docs/api/examples' -memcheck
Running test in ../docs/api/examples
01. SXaddition.run
02. customfunction_c.run
03. limitPrinting_c.run
04. range.run
05. all.run
Ran 5 tests, 0 fails.
python internal/test_py.py '../docs/tutorials/python/src' -memcheck
Running test in ../docs/tutorials/python/src
01. matrix.py
02. mx.py
03. sxfunction.py
04. sx_advanced.py
05. sx.py
06. sundials.py
07. modelica_ms.py
08. structure.py
09. ipopt.py
Ran 9 tests, 0 fails.
python internal/test_py.py '../docs/examples/python' -skipfiles="c_code_generation.py parametric_sensitivities.py reduced_hessian.py"
Running test in ../docs/examples/python
01. accessing_sx_algorithm.py
02. vdp_multiple_shooting.py
03. vdp_collocation.py
04. dae_collocation.py
05. mhe_spring_damper.py
06. simulation.py
07. vdp_collocation2.py
08. sensitivity_analysis.py
09. lqr_control.py
10. simple_nlp.py
11. ipopt_nl.py
12. robust_ocp.py
13. rosenbrock.py
14. implicit_runge-kutta.py
15. biegler_10_1.py
16. vdp_indirect_single_shooting.py
17. vdp_dynamic_programming.py
18. vdp_indirect_multiple_shooting.py
19. vdp_single_shooting.py
20. rocket.py
21. dae_single_shooting_in_30_lines.py
22. thermodynamics_example.py
In ../docs/examples/python/modelica/fritzson_application_examples, thermodynamics_example.py failed:
==============================
returncode:  1
No jmodelica installation, falling back to precompiled XML-files
Warning: Skipped TiXmlNode::TINYXML_DECLARATION

Traceback (most recent call last):
  File "thermodynamics_example.py", line 77, in <module>
    p = vertcat((ocp.pi,ocp.p))
  File "/home/casadibot/slaves/linuxbot/full/source/python_install/casadi/casadi.py", line 39140, in <lambda>
    __getattr__ = lambda self, name: _swig_getattr(self, SymbolicOCP, name)
  File "/home/casadibot/slaves/linuxbot/full/source/python_install/casadi/casadi.py", line 57, in _swig_getattr
    raise AttributeError(name)
AttributeError: pi

==============================
Ran 22 tests, 1 fails.
make: *** [examples_code_py] Error 1

```


Both versions use ONE_PLY=2 in types.h

ONE_PLY=1 has been introduced three days later. (commit 222f59b9c1537f22db65c6104072f5e0f68c1fbc, 2014-09-29)


@joergoster what algorithm do you use to tune these values?


Hi Remy, good to see you back coding.

Not sure this calc is correct, e.g. if my target size = 140MB, LOW_BUFFER = 10%, HIGH_BUFFER = 20%, then the low_size_limit should be 126MB and the high_size_limit 168MB but in the above the low_size_limit would be 14 and the high_size_limit 28


I'm confused about this commit.

The rootfs will be rsynced so that on the target host it will have the same idmap as it did on the source.  You send an idmap from the source, and use that to shift the rootfs.

I would think you'd want to claculate the delta between the source idmap and the idmap at the target - or just verify that the source idmap falls within the target idmap range.

I'm mostly trying to check my assumptions - I don't think getting that correct now is necessary for this patchset to be merged.


@calebcole203 "Links
need to be updated as index.html has been renamed to index.php"


asd


That could be it. But also, the technique you were using to attach a finalizer using `mkWeak#` doesn't work, at least not anymore. I'm not sure it's possible to attach a finalizer to something of type `Addr#`. I think we have the same problem with contexts.


Have no idea what this means?


OK, maybe I can add a sign out test at the last :-)


So `git-link-exec` uses return value style error handling while `process-lines` uses exception style (calls `error`).
If buffer has no file or its file is not part of a git repo calling `proces-lines`' will terminate the `git-link` call and the user friendly error message [is not sent to the mini buffer](https://github.com/sshaw/git-link/blob/v0.1.0/git-link.el#L182-183).


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1030


Many machines have ```use_power = 0``` when they are turned off. That's why I suggested what I did.


the problem is that when loading group from client m_sortedMembers is empty


The interpolation will call `String.Chars.to_string/1`. However this code is faster because we skip the protocol altogether. So instead of calling String.Chars.to_string that then calls the implementation, we already know the implementation, so we just dispatch to it instead.


Well, when i open a gitconfig file in emacs i want things to behave as i
am used to.  And i am used to that emacs inserts spaces for
indentation.  Just because the tool that generated the file inserts tabs
don't mean that i want emacs to do that.

And there are gitconfig files that git config never writes to.  For
example in my ~/.gitconfig there is an include statement that includes a
file that i share among different machines.  That included file was
created by emacs and will only be updated by emacs.

But maybe this is just me.  And it is not a big deal to me either.

Jonas Bernoulli <notifications@github.com> writes:

> But why would you want that? The result would be that the manually
> edited lines are intended using spaces and the lines added by git use
> tabs. I generally prefer spaces for indentation, but still I would
> prefer spaces over a random mixture of the two indentation styles.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/johanclaesson/git-modes/commit/6df99b1dcca0e1489113e574f168e93ef0013d4d#commitcomment-10199940


Добрый вечер.
1. Название интерфейса на трасянке недопустимо, как и использование транслитерации. Метод интерфейса должен заявлять о намерении сравнения по ключу порядка строк, а не сравнении их равенства, а, значит, ответа равно не равно не достаточно, поэтому, обычно, возвращается целочисленное значение (-1, 0, 1)
2. Реализация шаблона декоратор здесь мне кажется надуманной.


On remarque aussi les prépare statement


variable name should start with lower case


Very well structured code. Apart from generating samples multiple times, everything else looks fine.


The problem is that some users don't have a choice i.e. enterprise users, but until ownCloud itself and its official apps support IE, I don't see a reason to spend too much time on this.


не совсем понимаю, зачем в тесте проверка условия


ah I see I'm an idiot :p


Ah, I didn't think about that! :+1:


Clean!


Should it be 'spec' instead of 'test'?


Add an actual flag, rather than magic constant?


Maybe reword the “reviews code not authors” part. It currently speaks too much of bad experience.

»Code review helps improving the code quality, ensures that multiple people know the codebase and makes it infeasable to pressure contributors into adding malicious code.«


Why isn't the generator just using the boilerplate repo as a submodule? Seems like much easier than maintaining 2 different versions of the boilerplate.


oops. Seems to be an addition from a local change?


Automatic wasn't entirely wrong, but it refers to storage. I think the locality of the variable is what this comment glorify.


I'm really trying to follow the logic for why this alias is necessary, but can't.  What is happening here?  Given the cbuf allocation, and the map_at, I don't see why we need anything other than a memcpy to populate the memory (which is elsewhere)?

What's with the alias?


what is the error that had?


Perhaps use an npm module for this like https://github.com/jonschlinkert/index-of or https://github.com/jonschlinkert/indexof ?


Could we have these grouped like:

```yml
shallow_clone: true
version: '{build}'
skip_tags: true
```


nice!  I look forward to trying it out!


Was something not working with ava?


you don't necessarily need an extra variable here. writing
```
    free(List->head);
    List->head=NULL;
    List->tail=NULL;
```
should suffice


same as above


:ok: All? or these 3 items?


This however, is correct.


@jdonaldson Lua has some veeeeeery strange bechaviours... This makes passing tests complicated...


Nice use of the extra ```length``` field


@chimerast
この辺りでtsvをこんな感じに受け取る（or 変換する）方法ってありますか？
```json
[
  {"Year": "1997", "Make": "Ford", "Model": "E350", "Length": "2.34"},
  {"Year": "2000", "Make": "Mercury", "Model": "Cougar", "Length": "2.38"}
]
```
https://github.com/mbostock/d3/wiki/CSV#parse


why exactly do you need this line?


Lovely work! Very well organized.


looks good


У меня есть интерфейсы IStrokeComparer, ICalculator и IPurchaseable. Не
совсем понимаю, какой из них на трасянке или написан транслитерацией. Учту
на счет компаратора! Показалось, что ответа true в случае, если строки
нужно менять местами достаточно для метода сортировки. Кстати, мне не видна
ваша вставка на счет проверки условия в тесте

14 марта 2015 г., 23:04 пользователь AnzhelikaKravchuk <
notifications@github.com> написал:

> Добрый вечер.
> 1. Название интерфейса на трасянке недопустимо, как и использование
> транслитерации. Метод интерфейса должен заявлять о намерении сравнения по
> ключу порядка строк, а не сравнении их равенства, а, значит, ответа равно
> не равно не достаточно, поэтому, обычно, возвращается целочисленное
> значение (-1, 0, 1)
> 2. Реализация шаблона декоратор здесь мне кажется надуманной.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VladislavMedved/ASP.NET/commit/18adf06725f47801db7901d06b982c82591df915#commitcomment-10200060>
> .
>


It's better to calculate the average inside this function (by calling calcMeanDeviation instead of passing it as a parameter. This way, there's no possibility of a wrong average value being used.


Passing Client instead of C to this function likely forces a repeated lookup of the same client record in the backend.


Nope, just those since they are one-liners anyway.


aun marca 3 shift/reduce conflicts


All you need to do is increment the ASSET_VERSION constant at the top of this file.  This is exactly why it's there - to force a re-unpacking of the assets onto the user device.


Why do we no longer lint line endings in OS X and Linux?


Better to make this a class variable so that a new object is not created each time you call this function.


The min height and width won't work when twisting the hand. If you could calculate the distance between the highest and lowest point of the blob (length of finger) this might be a workaround.


IStrokeComparer - stroke - строки?
14 марта 2015 г. 23:22 пользователь "VladislavMedved" <
notifications@github.com> написал:

> У меня есть интерфейсы IStrokeComparer, ICalculator и IPurchaseable. Не
> совсем понимаю, какой из них на трасянке или написан транслитерацией. Учту
> на счет компаратора! Показалось, что ответа true в случае, если строки
> нужно менять местами достаточно для метода сортировки. Кстати, мне не
> видна
> ваша вставка на счет проверки условия в тесте
>
> 14 марта 2015 г., 23:04 пользователь AnzhelikaKravchuk <
> notifications@github.com> написал:
>
> > Добрый вечер.
> > 1. Название интерфейса на трасянке недопустимо, как и использование
> > транслитерации. Метод интерфейса должен заявлять о намерении сравнения
> по
> > ключу порядка строк, а не сравнении их равенства, а, значит, ответа
> равно
> > не равно не достаточно, поэтому, обычно, возвращается целочисленное
> > значение (-1, 0, 1)
> > 2. Реализация шаблона декоратор здесь мне кажется надуманной.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/VladislavMedved/ASP.NET/commit/18adf06725f47801db7901d06b982c82591df915#commitcomment-10200060>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VladislavMedved/ASP.NET/commit/18adf06725f47801db7901d06b982c82591df915#commitcomment-10200113>
> .
>


Nice find!  No more using 'example_user', huh?


derps.


I think it's just called mean. Mean deviation is something else ([see here](http://mathworld.wolfram.com/MeanDeviation.html))


Works well, and well written code!


Все, понятно. Я имел в виду строка, поскольку думал, что она так и переводится.


// hello world


This looks like a typo (`mesage` instead of `message`)

I have no idea if this can lead to a bug...


// hellow owr


Если expected массив и actual не совпадают по длине, то можно сразу же выдать Assert.Fail не углубляясь в проверку почленно


Well, these are overriden by the local install anyway, right. If anything they should default to localhost:4443


These not existing was causing ALL tests to fail, as 'coffee-rails' and 'sass-rails' couldn't be loaded.

If you want to defer the to the versions specified in the app the engine is within, I guess you'd be best off making the version requirements for each gem as fuzzy as possible, but they are definitely dependencies of the engine.


Why remove TakePicture? It was really helpful, when I wanted to determine if a player took a picture for my news reporter mission I checked this for reference.


Bikeshed name time. \@megawac & @jridgewell. Since this is allowing a single rest param should the method be called `restParam` singular.


The problem came with non-MMX builds (the 'any' builds). Not sure why it did not show up until then, but that is when it showed up  (cygwin build).  Removing count does not seem to impact either cygwin or linux builds, so I will leave it like this.


never a read / write hdd going to be faster than the speed of RAM.
Having a terminal with 1 gb of RAM is unneeded zram only generates lag for its low bandwidth, the zram is long gone with 1GB of RAM terminals


The conditional is a hack. This will not work where NULL is not all zeros at runtime. C standard mandates NULL being zero only at *compile* time.


fast-forwarding develop to borshop-integration-445-zl-yz-index-hashtree-eqc = 6f0281c4


What if you annotate the CoreMethod with `@RubiniusOnly`? I think it would be clear while keeping things simple.


Please give the docs a once over and clean up.


This is simply a back port for the performance commits that @rkorytkowski made for TRUNK-4618


Perviously I could find some problems by using tape instead of ava. https://github.com/imagemin/imagemin-jpegtran/commit/a9d8a2cf99d318f94f96a859ee91867e9061645e So I expected to find the problem related to imagemin/imagemin#89 with tape again. But, [with Node 0.12, ava also produce valid logs](https://ci.appveyor.com/project/ShinnosukeWatanabe/imagemin-optipng/build/23#L49). ([With Node 0.10, ava doesn't seems to produce valid logs.](https://ci.appveyor.com/project/ShinnosukeWatanabe/imagemin-optipng/build/22#L45))

Now this branch became unnecessary.


That's not what I told you :)
Is it on purpose? The idea was having the same value in both left-margin's but with opposite signs


This is superfluous.  By definition Elasticsearch output _must_ be in JSON format, so this configuration line is ignored.


Is this compatible with BSD make?


I certainly wouldn't be against pulling in a module. But the readme points
out that this is currently dependency free, and I didn't want to change
that. If the authors want to pull something in, that's fine (though my vote
would be for the lodash implementation, as it gets more usage/review)
On Mar 14, 2015 4:14 PM, "Nathan Rajlich" <notifications@github.com> wrote:

> Perhaps use an npm module for this like
> https://github.com/jonschlinkert/index-of or
> https://github.com/jonschlinkert/indexof ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jasonkarns/enum/commit/bc2476bf507cf854e4d5471b12a252ca0af42a3c#commitcomment-10200089>
> .
>


This will result in degraded indexing performance.  I recommend commenting this and using the default now that you have a working setup.


I'm not sure if line 182 is the "right" behavior or not.  Right now, the prompt always gets formatted, regardless of whether it came from a function call or not.  Maybe this isn't necessary, and `format_prompt` should only be called if `p` wasn't callable to begin with...


It makes no sense to change all transitive callers to pass it non-const if a simple `const` added to `toXFileString` fixes it. Also, if something is non-const, readers will assume that it changes the data. In an exporter I would be seriously surprised by this, even though we do make a copy of the scene and allow modifications.


Great!


Or May 2015..


test


Hi, this API is iOS8+ only :)


It's a TypeError to have a named param after a rest param (so it's always just a single rest param), and ES6 still calls them "rest parameters".


@Sharlikran Instead of having all the after metadata, I just made the priority higher. The weather mods should load very late instead of having to keep adding all those mods that LOOT would sort behind it.


I hear you, I've had to rethink a lot of approaches lately.  I think the haxe foundation has done a good job covering all kinds of cases with these tests.


This crashes if cornerRadius1 is non-nil but cornerRadius2 is nil. My local copy now reads:

```    return (!cornerRadius1 && !cornerRadius2) || ((!!cornerRadius1 && !!cornerRadius2) && [cornerRadius1 isEqualToNumber:cornerRadius2]);```


Indeed it does, now I see where you got it from. Importer tries to avoid replicating the init code here. It is not a resource leak because the only data member of Importer - `pimpl` - is a POD, where in this case `std::map` is not.

It's a confusing hack, I'll fix it by pulling the init logic out into a separate method.


Congrats, you and everyone else who has been working on the SMP has really done a great job these last weeks.


Interesting. With Node 0.10, Windows build always [failed](https://ci.appveyor.com/project/ShinnosukeWatanabe/imagemin/build/91). But, with 0.12 it [passed](ShinnosukeWatanabe) the test at long last.

I don't know the reason.


Why is the `_closure` not necessary?


:clap:


`if not np.all(np.iscose(...`


Same here


I approve of this ;)


dfsdf


Because the algorithm is guaranteed to give a closed solution.

Think of if this way.

You have an array like [0.25, 0, 0, 0, 0.75] and you set a delta=0.1
Then the nonzero components will all be scaled by (1-3*0.1), so the sum of the nonzero components is 0.7, just enough room for three zero components to fill up the remaining proportion.


Well, that's not so strange that the tangent is missing since is far less useful when programmatically planning the aliens' wave pattern in a shoot 'em up and the period movements of a metaballs field. :wink:

Just kidding, I guess this is due the fact that Wren is not born as a full featured general purpose language (will it become such?), but a game-scripting oriented language with the set of features more commonly used in that context.

:D


If is it is still not clear, I would be happy chat about it sometime.


well, p is a pointer to the node that is being deleted. And i had it pointing to the next node of the list as i want to continue deleting elements in case more of them have x as a data. Have i not approached it correctly? or am I missing smth?


I think it might be due to npm version.


1 okno.png:
	- Usuń błąd w nagłówku - "pizzerią".
	- "Utworzenie własnej pizzy" brzmi dziwnie. Zastanów się, czy nie lepiej zamienić to na tryb rozkazujący. Np. "Zamów pizzę z menu" i "Zamów pizzę według własnego przepisu".

2 okno.png:
	- Usuń walutę z tabeli.
	- Pierwszy wiersz w tabeli zawiera nazwy kolumn. Trzeba to uwidocznić.
	- Zmień wygląd pierwszej kolumny w tabeli - jest mało czytelna.
	- Zmień nagłówek wyszukiwania na "Wyszukaj pizzę", bo nie wyszukujemy dodatków.
	- Usuń "Płatność kartą".
	- Dodaj podgląd zamówienia - zwykła ramka z nagłówkiem powinna wystarczyć.
	- Dodaj aktualny koszt zamówienia.

3 okno.png:
	- Jak wcześniej - "Utworzenie własnej pizzy" nie brzmi dobrze.
	- Pierwszy wiersz w tabeli ze składnikami zawiera nazwy kolumn. Trzeba to uwidocznić.
	- Zmień nagłowek "Wykreowana przez Ciebie pizza" na coś innego.
	- Zamień kolejność "modułów". Najpierw lista wybranych składników, później wybór liczby pizz, sosu i koszt.
	- Zmień "Suma" na "Koszt".


nice fix!


Looks good.


THAT IMAGE IS SWEET


Зависимости!


Nice catch!  I stand corrected.


Merged commit from branch.


@y-higuchi I'm not sure if this is applicable anymore with the new v1.0 OFSwitchManager. If you'd like to check and run into the issue there, please reopen an issue/pull request with the current master branch as the head. This cannot be merged now though with the net controller structure.


This cat test is setting all of the attributes manually, this is the same as using the .create as opposed to factory girl. Looks good though.


Noch entfernen


Wichtig Negative Zahlen einlesen und einrechenn


How do I upload a mod


I've argued a bit with @megawac on this. I don't think it's useful to ever provide a `start`.

```javascript
var fn = restParams(function(names, alwaysUndefinedParam) {
    assert(alwaysUndefinedParam === void 0);
}, 0);
```

What's the point of ever having having a `start` anything after it is always going to be undefined?

Additionally, if we're emulating ES6, the rest param is **always** the [last parameter](https://github.com/lodash/lodash/commit/7c730eed7720f21ea39dc12d32b86da6be7d9983#commitcomment-10200194). And, you're going to have to put the param in the function's params list (`function fn(...args)`), so why support a param-less function?


Will fix.


`var otherArgs = Array(start + 1);`?


## Ellipsis and Horizontal Axis size in chart scripts

SVG has nothing like `text-overflow: ellipsis`, that's why many charts had overlapping labels.

In this commit I've created the `ellipsis` function in `ChartUtils` that solves the problem.

Additionally many charts have now a Horizontal Axis property to change the Width of the labels for the Y-Axis.

Remember to *import* the new charts scripts using **Load -> ImportExportChartScripts**.

Enjoy!

![image](https://cloud.githubusercontent.com/assets/720953/6653546/47a8b59e-ca97-11e4-925a-ad404ce14466.png)


Added functionality to delete filename (#) rather than filename


repoarch/cmst - cmst is in manjaro repos

and please remove geniso.log it's seems that you accidentally committed it


My counter argument to this is wrapping functions which may not be user defined. For example

```js
var restArgs = _.restParams(_.rest, 0);
restArgs(1,2,3,4,5,6,7);
```


make it non-random..!


rfm69_config.c can be deleted since the header now contains the setup array


Good job getting these tests written and passing. Try to commit after each test though so it's more broken up and easier to take in.


I see, thank you! (I'm sorry I got a bit confused from the commit message). The tests were not right because the input was not a composition?


The cmst of stable don't do what I will. I took this of unstable.

Sorry for geniso.log, I remove it.


Doesn't work for Mac OSX clang, you need the patch from #290 for that.


Fixed in version 0.82.

Thanks for the report and input.

John.


>In this, I'd think the performance regression comes from the decoupling, which kills opportunities to:
>
>    query the validity of a page before invalidating its physical alternates;
>    revalidate a page whose contents have not changed but which gets remapped later.

Indeed.

> Commits like this one also justify the proposal I made for TLB accessor functions in R4300 emulators [1],
If you have some code to show, I'll gladly look at it :)

The macro approach you also suggested, could also work, but I guess it is cleaner to use a well defined interface ?

On another note, I would also suggest that :
- you fix the "recomp.c: Remove duplication in code for alternate physical addresses" commit as discussed earlier
- split this PR such that all commits except this one and the adler field removal can be merged.

This would allow to merge all the good stuff you've done, and give you more time to work on the tlb decoupling interface. What do you think ?
(If it's too much work, or if @richard42 prefers to merge that after the next release, forget what I said).


Technically yes.  Compositions can't have zeros.


Avoid toString ... this is usually used for debugging.


wrong spelling datasciencecoursera


Ya me llego deja lo reviso


cmst in all branches should be the same since yesterday's stable update


Muy bien Gerardo


Your  train.txt is  the same as rank.txt, so  when I train your network ,accuracy won't converge till accuracy  up to 1.  What's more,I find your network won't converge even if I changed the train.txt. Could you tell me what's the problem??


I gave it a bit of thought and it actually makes more sense to remove them. If it didn't create a screenshot, it shouldn't show up in the report. Plus, suites that don't have any valid specs should definitely not be reported.
d3b2b2a081f27fbd7a9452064df66e171250b6ae


I don't want to jinx it but the 16-core test is looking very good :)


Thank you @mortonjt!


This is the default equals, remove it or add some semantics for equality.


**cmst** is the same. Please use our default build-server **netzspielplatz.de** to have a synced server.


Awesome to see this got fixed before I got started :)


ghjkl;


I knew that conversation was somewhere.


This file is okay to commit.


These two blank lines will make this code not `gofmt`ed.


I love that emial reg ex lol


nice


Workshopped


What is the motivation for this change?

Is it because users should be less concerned about the generate code and just take that it works?


Then simple rename plugin and bb.


i want to check the behaviour of the old observables with this version (the formulas given in the references). should be easy to change the calculation if this does not give reliable results. i still have to find an efficient way to calculate the stddev with mpi


Hello, Dongdong!

My name is Jonathan, and I don't know if Mike told you, but I signed up to work on the website for viewing the CS100 grades as well.

 I've already begun work on the website, which you can see [here] (https://github.com/jduga002/gitlearn). I've got it set up so that there is a basic website where you can enter your account, and it will return your grades for you on a webpage. Currently, I'm working on adding nice formatting to the site so it looks good.

I was wondering since you're working on this project, too, if you wanted to meet up to collaborate on it, or if you wanted to divy up the different parts for the website.

Go ahead and shoot me an email at jduga002@ucr.edu so we can talk further about this.


:fireworks: :balloon:


Java 8 stream can do it in a better(elegant) way. This should be rewritten to something similar to:
AlarmController.getController().getViewMap().entrySet()
.stream() /* maybe parallelStream() ? */
.filter(x -> x.equals(this)
.findAny() /* returns optional */
.get() /* get the value from optional */

Another alternative: BiDiMap (from commons-collections)


#24 Step 1/?


#24 Step 2/?


#24 Step 3/?


Avoid magic numbers


Stream this


Zonder de package kreeg ik een foutmelding "org.hibernate.hql.internal.ast.QuerySyntaxException: PersonsEntity is not mapped [from PersonsEntity]


De pagina wordt getoond door naar localhost:8080/persons te gaan, maar de lijst wordt niet gevuld vanuit de DAO.


Check Optional class from Java 8. This is actually what you want:

public Optional<Connection> getConnection() {
try {
...
return Optional.of(dbConnection);
} catch (Exception ex) {
return Optional.empty();
}

}


godoc ain't markdown :)


Never ever catch for NPE .... do NOT use error handling for logic handling. If a variable is null, check for it(exceptions are super expensive)


`*[]ConfigurationSet` :( :( :(


Remove this space


Did you do this? verification note in homepage footer


is this signature complying to some typedef? why do we return `error` if it's always nil?


Following the comments for connectToDB:
getConnection().ifPresent(conn -> /* use connection here */).orElse(/* handle cannot establish connection here */)


empyfile?


Never catch NPE ...


getConnection().ifPresent(..).orElse(...)


A proper TLB interface basically implies that each R4300 emulator must first become a plugin, and *then* each one can be properly told what to do with the TLB. It's way too much work, mostly due to the mess that is the New Dynarec. But it will indeed be cleaner, much like your `r4300_???` pointer-gathering functions from #83.

Do you want to see a macro version, perhaps? Because, technically, TLBWrite belongs to the R4300 opcodes; it's just defined with `r4300emu != CORE_PURE_INTERPRETER` checks.


True, so i don't need this of unstable. I will change that. netzspiellplatz is often very, very slow here.


SHA1 checksum still needs to be provided somewhere


Weird? I was expecting the order to be `newX, newY, oldX, oldY`. The purpose of this explicit ordering is so I can lazily define a callback that only cares about the new values:

```javascript
function callback(x, y) {
    // ...
}
```


Nice class.


How can container have two different kinds simultaneously?


I were tring to 'center' without the text-align property. I will make it right.


Missing callback args.


Also missing callback args.


Can you change it to use the conditional operator here? I don't like the idea of container being false.


Resource leak ...


add setLogLevel(RestAdapter.LogLevel.FULL)


extra data


Need to change the javadoc on this one


IMO we don't need a synonym for %d. If we use a mixture of %d and %i in the code, then it's likely that format strings that could have been shared will end up being distinct and duplicates would be stored in flash (e.g. printf("%d") and printf("%i") will require both "%d" and "%i" strings to be stored, whereas if they were both "%d" they would share their string).


And now the callback doesn't even get the old X value anyway! Same with Y below...


With this removed, the safety net in the `set` method is no longer used. This regresses the bug that allows `NaN` into the scene transformations.


Ouch, I forgot about BSD make. No, it doesn't work on BSD make, and in fact the objcheck change doesn't work either. I've rebased the branch removing this commit (don't have time to learn how to do this on BSD make) and fixing the other one. Will send a new pull request.


Need More Info.


no me gusto


I can't see this changes in october-min.js of latest 225 build. Who should to rebuild october-min.js?


So for block scoped variables defined in loop initializers (for, for in, for of), this is always implicitly false, right? Same with catch parameter.


This is a test


Thats why we have http://repo.manjaro.org/ as one of our services. Please use no outdated mirrors for building install medias.


This is a test ,too


I'm not sure if there was error, but now he will send in lines... No more ACC&PASS


Do not close streams in try blocks


Fixed OK. Thanks.


When do you and when do you not statically import `Expressions`' creation methods?


This to encourage consistency, I am just wondering.


Good point, it should be more consistent. What do you think, when should it be imported and when not?
I thought about using the static import in the *Expression classes.


Why this?
You forgot the comma.


LGTM.


very nice!


looking good!


Have you thought of just using .sbt files, now that in 0.13.x you can define multiple projects?   Everything in one place.


How this comment skipped the Travis build?!


TeamCity Helios :: Helios Unstable Latest [Build 4](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=262&buildTypeId=Helios_HeliosUnstableLatest) is now running


TeamCity Helios :: Helios Unstable Latest [Build 4](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=262&buildTypeId=Helios_HeliosUnstableLatest) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:19


MSVC warned here about converting bool to QAbstractSocket::SocketState.

I believe the intention was `stdoutSocket.state() != QAbstractSocket::ConnectedState` and I'm updating the code to do that.

Ditto below.


c'est verifie -> ok


Ah whoops. I meant to do that with value, but that's the new one lol. It was an order problem though. updateBounds depends on the new value, not the old one. And refactoring it to use parameters didn't seem right.


My intention was to fix standards compliance (got burned some time ago). As for saving space - I am not big fan of such micro-optimizations.
Bus using format specifiers consistently is good idea and omitting %i is way to enforce it.


Looks good.


Ich glaube nicht, eigentlich ist "a impliziert nicht c" doch äquivalent zu  "nicht a oder nicht c", oder liege ich da falsch?


Merge


Не компилируется потому, что требуется сперва принять PR #39 в ylab


is this correct? what does it mean to `get_winsize` on ios?


Still immediately and obviously wrong. Please *test* stuff before submitting it.


Congrats @xeno-by. \O/ for all!


Passiert in der Initialisierungsschleife nicht folgendes: Ist adjMtrx[i][j] true bleibt es so, ist es false auch und wenn es noch nicht initialisiert wurde, gibt es einen Fehler...


Sure, I will test. Using this project: https://github.com/Pfiver/groovy-RootLoader-issue Feel free to do so as well. While this minimal project that triggers my issue does execute a lot of pre-compiled byte-code, it is, for me, still just a script. And thus, should, hopefully, be covered by the RootLoader intended use-cases. But, I do not *really* know, what those use-cases exactly are. I just noticed that IntelliJ is using it. And I usually trust, that those guys are doing the right thing. I could again be wrong. I.e. JetBrains might be wrong.

But anyway. Please have a look at the repo linked above. I think it is simple enough and has a clear and simple, straight-forward intent. And I (again, it might of course be that I am alone here) think that this *should* just work out of the box (and it actually did until recently, when I upgraded from selenium-firefox-driver 2.44 to 2.45, where they obviously started to include xerces).

And: 1. If you could give me a short overview (or a pointer to it) of the main use cases that RootLoader/GroovyStarter is supposed to cover, that would be great. 2. If you could *show* me a one (just one) case where this kludge actually does any good, that would be even greater.

True, I do not really understand the intent of RootLoader/GroovyStarter. Specifically, what I completely fail to understand is how the org.w3c.dom.Node special-casing kludge should resolve any issues that people are having with RL/GS. If it is supposed to load classes from a *different* source than the sytem classloader, then why would anybody *not* want to load org.w3c.dom.Node from that different source? Because that class is in "rt.jar"? But then why *only* "org.w3c.dom.Node"? There are probably many other classes (outside of "java.*" and "javax.*") in "rt.jar". Therefore, should not the kludge be adjusted to load *any* class that is present in "rt.jar" (or the bootstrap classpath?) through the system classloader? Why is "org.w3c.dom.Node" arbitrarily special-cased just because some "userland" jars that are *unrelated* to groovy are broken.


@sdt You are correct that you shouldn't have to do it that way. Most other implementations (apart from the mal impl itself) do not use metadata for the macro flag. Even the existing perl implementation does not. What sort of problem were you seeing?


I'd move the board definitions above the 32u2 family since they are nearly the same but the at90usbxx are older. Then came the u2 and then u4.


utiliser pretty_binet plutôt.


sdfsdf


* clap clap clap clap *

Congratulations!


> Do you want to see a macro version, perhaps? Because, technically, TLBWrite belongs to the R4300 opcodes; it's just defined with r4300emu != CORE_PURE_INTERPRETER checks

No need to if it doesn't bring that much value for the decoupling.

> It's way too much work, mostly due to the mess that is the New Dynarec.
Agreed. We will need to talk about that after the next release richard has planned.


Heather


il faut encapsuler ça dans une condition has_editing_rights($binet, $term) en effet.


ONe question arter, as i have been researching and not be able to change. Is it possible to use GCC androideabi for ROM and chooose another toolchain to compile kernel instead of this one? has been a headache i cant find the answer to this.Thanks!


Project is broken. Next time I'll keep tests off the index until they pass.


Missed this before: now we need to check here if `pProperties` is null. I'd suggest we leave it as is for now. I'll commit a fix that will make sure that we never pass a null ExportProperty to exporter implementations (if the user doesn't specify one, we use an empty one). It makes it easier for exporter writers if they do not need to handle this extra case.


Sieht schon gut aus, meine Idee wäre gewesen, wenn der Mittelstein schwarz ist, dass man gar nicht schaut wie viele weiße Nachbarn etc es gibt. Es sollte ja ausreichen nur Left mit Right und Up mit Down zu vergleichen und dementsprechend die Fälle beide sind weiß, einer ist weiß, keiner ist weiß zu betrachten...


Make sure this is a text file


¿Entonces la conexión a la base de datos solo se efectuaría cuando hago la instancia de mi modelo?


here is src img should be /img instead.
I'd update the master with this section in a minute


Oh, that makes sense. Thanks.


`__getitem__` accepts a Slice if you want to retrieve multiple values, or a `Py_Int` if you only want to retrieve one.

There's also no need to convert this to a string and back. `UNPACK_SEQUENCE` merely pushes the individual elements in the tuple onto the stack.

You should do:

```
f.push(val.__getitem__(new Py_Int(x)));
```


Perhaps a little radical this deletion.

May be something in this style
https://github.com/zhao-gang/wifidog-gateway/commit/47d2b07f63dd845837b2677cb72fb6c6828364c1


It looks like `i` isn't reused. You can remove `i` and do:

```
for (var x = f.readArg(); x > 0; x--) {
```

....or remove `x` and do:

```
for (; i > 0; i--) {
```


(Style) We typically put curly braces on the same line as `for` / `if`/ etc.


Not related, but these should all be at latest version 22703.


This is just bad. It needs 3 different Factions jars merged into one another to build... there has to be a better way to do something like this.


I'd rather have this be the same no matter the OS, so just have `gchar **argv=NULL` on Windows, and set `cmd = NULL` otherwise (after filling `argv`), what do you think?


Thanks. Can you open a separate issue for that?  Not sure how many other projects need to be updated as well.


do we want to manually check the argument passed to the shell?  I agree it might give better error handling, but OTOH the check has to never find false negatives or people would get annoyed…


ICSSolarCollector_#4723 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1655 of 1656 tests passed)

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


Any chance to disable the sorting? Messes up a hell alot of things. Especially numbering if it goes past 9th number.


Just do

```
(and (symbol? register)
     (member ...)
     #t)
```

instead of adding explicit `if`, `#t`, and `#f`.


What is `car` of an instr? Why not define `instruction-mnemonic` or so somewhere else?


merging upstream changes into jdsteve2's fork


Moi je trouve ça suffisamment clair, mais le switch d'avant ne me choquait pas non plus plus que ça. Ton avis @Newbienathou ?
Sinon, tu peux essayer d'utiliser fa-toggle-on, fa-toggle-off ?


ICSSolarCollector_#4723 (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1649 of 1650 tests passed)

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


ICSSolarCollector_#4723 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


"spread" is weird here.


Could you please help me update mdss that's all I need help with? android.c357@gmail.com if not at least say you can't help me.


Shouldn't this be `instrs`? Or are we really going across each component of a single instruction?

Basically we just want to see if any vector resides in the instruction. Can you think of a better way to encode the English sentence "Is there any FOO? in the list L?"?


I find it more common in Golang that instead of defaults being explicitly set for you, it's more common to make available a Default configuration that people can use.

I've also see people use NewConfig functions that return the default configuration


As you wish.

https://github.com/nickgammon/arduino_sketches/commit/cca2778336ff232a0df247b10c9b9d7763b8c01b


Override PANEL:OpenMenu in your panel and use whatever sorting you want.


How is this different from the previous function? Is this one just auxiliary? How about calling it `compute-instruction-length-aux` or so?


Is this really meaning `symbol?` or do you want something more akin to `register?`?


Oyeah, I also did this, so I can eventually fix that error.


What does `autoincrement?` mean anywhere? Is this underground CPU voodoo that cultists know about?


@jangrewe Per your request, I have added the ability to use aliases. There are three high-level alias related commands, but all existing commands now accept the exact job name or the alias. You create new aliases by using `jenkins setAlias myalias, my-long-job-name that-is-annoying-to-type`. Mind giving it a go and letting me know how it works for you? Please make sure to read my notes in the README regarding persistence in case you aren't already persisting Hubot's brain.


Made the output box autoscroll to the bottom
(Just putting this stuff here because I forgot it in the commit message)


Use `(newline)` instead.


Thx!


convert to what?


Fixed inconsistent spacing in which the last line used a tab character rather than spaces


LOVE IT!


You can break the first line after `(` and then indent the rest the normal 4 spaces.


what is RSRD


what is RS


I've slightly modified the 2.1 assignment, according to your suggestions. I also put the first version of the 2.2 assignment.


Basic error handling, break fast, break hard.


what is STD-JMP


there's a typo in canola.


This line caused the first Origin to hit the server to be the only valid origin until the server was restarted, I've changed this to using the forceOriginMatch for values of "*".


`catch_and_clear_warnings`, or maybe just `this` ;)


These two tests have the same name :)


Looks mostly awesome, 45 additions, 234 deletions, less bugs, less duplication, and better behavior on Windows… what more could we ask for? :)


You were too quick - I already fixed that :)


It also won't have any CORS headers on the response, so it will fail with a proper error msg when hit with a client that respects it.


Not crazy, that definitely improves readability. :)


awesome work, hombre!


we talked this one over...all good.


@amarbirgill Looks good. Did you write this from scratch or was this code given out during a breakout?


Fine job!


This mman_alias has to do with the booter's version of the spd's memory. dsrc is part of a contiguous block of memory in b_spd that the shared "booter.c" will populate with the spd's sections. Since we can't alloc the cbufs to those contiguous blocks of memory, we instead need a way to make sure the contiguous blocks are the same as the cbufs. We could (maybe) memcpy from dsrc to caddr later on after the booter finishes populating but before it activates the spd.


This doesn't parse answers that have more than one line in them correctly (e.g. a matrix). `strsplit(q[3],NL)` just looks for a new line in an answer and assumes that when it hits a new line the next answer begins. It should check for a new line **and** a character that represents a bullet point in markdown, either `*`, `+`, or `-`. Turns out everyone is using asterisks, so I am suggesting an easy fix for this in Pull Request #2.


very nice!


Looks like a recursive call. Btw, those two generic methods are YAGNI in NET2/4 projects.


Clever! Really like this solution of performing the operation first and then adjusting later if the final value is invalid.


Fine fluff!


More fine fluff!


This looks really sketchy, but this was my commit from our AWS EC2 instance. I even pushed using my username and password...


please be careful with the number of characters


this space shouldn't be here


lots of good stuff!


Окееей, допишем

14 марта 2015 г., 22:36 пользователь AnzhelikaKravchuk <
notifications@github.com> написал:

> Не нашла логику расчета возможной скидки на продукты в корзине.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MatthewKoreysha/EPAM.Homework/commit/f582899e0bcb4aacf805ef5e8d9845f240ba243d#commitcomment-10199980>
> .
>


This could be rejigged a bit.

```ruby
if equipped_weapon
  equipped_weapon.hit(enemy)
else
  enemy.wound(5)
end
```

You can leave out the `self` when calling `equipped_weapon`.
Dealing with the positive case in the if statement seems to read a little better.


Issue #24, Step 4/?


be careful with 100 characters (java conventions)


The generated code became more complex due to the blocking stuff. I don't want to scare new users with it. ;-)


This should be

```ruby
robot.wound(damage)
```


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


It will ignore all files, except dist files. See latest commit  f9e465ae6974cbec50a880608de9c4172af9fde0


Hi @kanaka. This was my final test failure in self-hosting mode. All the step 9 swap! tests were failing much like this.

```
TEST: (swap! a inc3) -> ['','6'] -> FAIL (line 337):
    Expected : '(swap! a inc3)\r\n6'
    Got      : '(swap! a inc3)\r\n(reset! (atom 3) (#function (deref (atom 3))))'
```

This one was a bit of a head-scratcher until I remembered that I'd implemented my own swap! as a macro.

```clojure
(defmacro! swap! (fn* (atom f & args) `(reset! ~atom (~f (deref ~atom) ~@args))))
```

Finally it dawned on me that the mal implementation was hoisting the core functions directly from the host implementation, so it was installing my swap! macro locally as if it were a regular function.

Once I'd realised that the mal implementation was using metadata for the ismacro flag I switched to using the same approach, which meant my macros got detected as macros, and my tests all passed.

I wonder if mal should include a `macro?` core function which would get used when installing the host language core functions?

I had plans to try this out and possibly make a PR, but it was already way past my bedtime :)


franchement, devrez y avoir un hook là :/


It would be nice if Browser.prompt and Browser.confirm also could be implemented at this place.


I love the minimalist nature of this change.  Reading between the lines, I can see tumbleweeds blowing through here...


:+1: Yay good catch!


there are npe in normal use, it's not ideal, but I've not found side effects so far


ICSSolarCollector_#4723 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (639 of 640 tests passed)

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


Right now, if your `#heal!` raises a SyntaxError the test will still pass.
Please modify the test to make sure that the right kind of error is being raised. In this case `RobotAlreadyDeadError`


Nice addition


This looks suspicious. You associate `modelsJvmSettings` with `jsSettings`, and vice versa.


creates -> Creates


@m6w6
Hi Mike,
In config.w32 there should also be this:

	diff --git a/config.w32 b/config.w32
	index 13c24ab..ad074d7 100644
	--- a/config.w32
	+++ b/config.w32
	@@ -115,6 +115,7 @@ if (PHP_HTTP != "no") {
			AC_DEFINE("PHP_HAVE_CURL_FORMGET", 1, "");
			AC_DEFINE("PHP_HAVE_CURL_MULTI_SETOPT", 1, "");
			AC_DEFINE("PHP_HAVE_CURL_MULTI_TIMEOUT", 1, "");
	+		AC_DEFINE("PHP_HTTP_HAVE_CLIENT", 1, "Have HTTP client support");

			if (CHECK_HEADER_ADD_INCLUDE("event2/event.h", "CFLAGS_HTTP") &&
				CHECK_LIB("libevent.lib", "http", PHP_HTTP) &&

You have added PHP_HTTP_HAVE_CLIENT to config9.m4, but did not yet add it here.


Makes sense.

I agree with the decision, I just wanted to get a better context. Thanks. :)


This line is redundant: the JS part of a `CrossProject` is already configured as a JS project.


you don't have it with the big letters at the beginning of a sentence, do you?


maybe should remove this.


the rightmost opening delimiter which is before this closing delimiter, that's an important bit of info for me.


A couple of things to change:

1. Make this function not be global by including it in a clojure. Here is an explanation for why globals are bad: http://stackoverflow.com/questions/2613310/ive-heard-global-variables-are-bad-what-alternative-solution-should-i-use

2. This code should be in application.js. Easier to read, easier to write.


Um, there is no Subsection (a) and (b).  Perhaps this should be 7.7.1 and 7.7.2?


Was still getting this error in Windows.

roamn's fix worked for me.


wow, why do we need two for loops out of a sudden here?


Should read release 3.1.5


jo ist mir auch aufgefallen, als ich nochmal deinen code angesehen hab. werde ich morgen oder so mal ändern


This is a new line of code that I am adding a comment to. It doesn't do much at all.


We should remove this and just have the SetVerbosity method on this interface.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2232](http://teamcity.monogame.net/viewLog.html?buildId=5671&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 419, ignored: 6 Build time: 00:14:17


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


Could we rename this to something like LogAction?


Could we rename this to something like LogActionEntry?


could you add `(a)` and `(a)(b)(c)`?


Parameter should be called `logAction` or `action` since it does not produce a value.


weiß nicht genau, ich dachte eigentlich, dass das else auch den uninitialisierten teil abarbeitet. hab das aber in meinem branch schon besser/fehlerunanfälliger implementiert


"just like how" would be better as "in the same way that" or "similar to how"


I would prefer to group all methods together: all errors and warnings together etc. etc.


Missing period.


Looks good


"nicht a oder nicht c" ist äquivalent zu "(a impliziert nicht c) und (c impliziert nicht a)"


Try to use `and` instead of `&`.


ICSSolarCollector_#4723 (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1655 of 1656 tests passed)

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


@gorhill ah, shoot — I should've noticed that. Sorry. I'll rename the Safari ones to have the `safari-` prefix.


Fixes:
>Customizer option to display/hide header cart
>Fix issue with default buttons display when there's still no options saved after theme install
>header cart positioning fixed -> pushed to right from header right content


whoaa crazy. Looks good


Remove deprecated parameters and add clean-css.
https://github.com/roamn/sublime-less2css/commit/9a06ff399b92137f84d9207948197e44f5cf571a


ICSSolarCollector_#4723 (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1655 of 1656 tests passed)

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


nice


PS: Added IL Code optimization. Tmp variables in il is not more needed


well done!


"XO" should not be hardcoded! During the lesson I just do not have time to extract all values to the static final variables but here we need to maintain "ideal" variant


new line after


Sir, I believe you've discovered that I'm an idiot! I should have looked in the file before deleting it, d'oh!


DITTO


99% ready to be merged, however small changes still need to be made


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


by any chance you know whats this error about?

http://pastebin.com/3najfjgn

Thanks


hahaha


This is a fix for https://github.com/pimoroni/unicorn-hat/commit/cabf748a9d36fa28a46c14af2945a3e39941a44f which I broke here: https://github.com/pimoroni/unicorn-hat/commit/656556fd817d6ea6f1d1e27f5371b23646658953


Was this change for performance or to fix a bug? (Just interested about the effects of Streams)


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


add gitignore


this.addEffect(cantBeBlocked) is missing!


There is a trivial typo here "Make sure every flag is in" => "Make sure every flag in"


I am a nice comment


Solid test. I see some conflicts with something I put in though, I'll see if they're in the master merging.


This is great but this is exactly the kind of commit that is very hard to work with.
 - Cheats enhancements should be worked on a new branch
 - Bug fixes for what?  Which commit introduced the bug?
 - Fixed what controller issues?

Please, for my sanity, if you can't put any of this on a separate branch, at least split it into atomic commits that are easy to cherry-pick and rearrange.


Yeah here it was.


Any chance you can hack up some ASCII art to describe this.  It still isn't
obvious to me.
On Mar 14, 2015 7:01 PM, "Gedare Bloom" <notifications@github.com> wrote:

> This mman_alias has to do with the booter's version of the spd's memory.
> dsrc is part of a contiguous block of memory in b_spd that the shared
> "booter.c" will populate with the spd's sections. Since we can't alloc the
> cbufs to those contiguous blocks of memory, we instead need a way to make
> sure the contiguous blocks are the same as the cbufs. We could (maybe)
> memcpy from dsrc to caddr later on after the booter finishes populating but
> before it activates the spd.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gedare/Composite/commit/a28632d75b59013158133bd0ed06ed86ce7c8b66#commitcomment-10200583>
> .
>


We'll have to robustify these tests later so they don't only expect the path to be correct but also the information on the page to be correct.


It's using the observer explicitly, there's no reason not to use the parameters! It's a better option than getting either the private `_x` property or getter `x` function.


@bigdaz, shouldn't this be `instantiator.newInstance(DefaultResolutionStrategy.class)` instead, like it is above in `doCreate()`?


ah. nice one. I didn't thought of reverse loop... always went forward. Nice one Rimelek.


嗨,

hjwdzh@gmail.com 想關注您

****** hjwdzh@gmail.com 是您的朋友嗎？ ******
好/是: http://invites.infoaxe.net/signup_e.html?fullname=Wchargin/car2014&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=cc3430e6-7070-4583-8069-380a98d3f3b9&from=hjwdzh@gmail.com&template=invite_reminder_2_b&src=txt_yes

不/否: http://invites.infoaxe.net/signup_e.html?fullname=Wchargin/car2014&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=cc3430e6-7070-4583-8069-380a98d3f3b9&from=hjwdzh@gmail.com&template=invite_reminder_2_b&src=txt_no

按一下這裡，將您自己從所有這類電子郵件移除
http://invites.infoaxe.net/uns_inviter.jsp?email=reply@reply.github.com&iid=cc3430e6-7070-4583-8069-380a98d3f3b9&from=hjwdzh@gmail.com


good stuff.


looking good!


nice admin work!


style points!


Thanks! Haven't used it in a while. Fixed in 80b4206f4375991a2b00a80b941d9b7d0b34a349


:crystal_ball: :gem:


Note: Branch "issue39" has been deleted, and issue #39 has been closed.


Ok, fixing many errors... :P


You did not regenerate `descriptor.rs`, which could be done with command `./full-rebuild.sh`. I did that for you, and pushed. b24ec457e398ebb359515170b996b94a231c1113. Thanks!


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


I been thinking about this for two weeks now but am usure if this is all it takes looking at https://github.com/xbmc/xbmc/search?utf8=%E2%9C%93&q=configure.in results.


?????. Do not touch?
Also, why delete all that code?
:negative_squared_cross_mark:


This class certainly does not belong in the `model.entity` package. More like `instancemanager.HeroManager`.


ggggggggggggg


Okay great, I believe I finally got it right here https://github.com/miguelgraz/FocusFileOnSidebar/commit/45cca28f94b449a206ef383f027e0f23af5ca7ce :smile:


This makes not much sense (I know it's been there for some time) for me, what's the point of converting (probably data) public keys to addresses, just to check nothing? Or worse: stop and fail here?


This should complete unit tests for TranslatorTest.cs


Be sure to ask @thpang67 about the looping error still showing up.


真特么帅


Thanks @elliottt

Weird, I didn't even realize you could put a type on to those annotations!


``` ruby
category    = Fabricate(:category)
monk        = Fabricate(:video, title: 'monk', category: category)
futurama    = Fabricate(:video, title: 'futurama', category: category)
south_park  = Fabricate(:video, title: 'south park', category: category)
```


@LevelX2 Thanks for the feedback. I would also prefer to use internal frameworks. I'm curious about the use of this. `CantBeBlockedByCreaturesAllEffect` seems to be geared towards abilities on creatures, where as I _think_ the `applies` method on an effect for attached permanents (auras or equipment) needs to accommodate this. Is there another class I should be using that is specific for attached effects?

Thanks!