Yes, they are working
On Mar 31, 2015 3:40 PM, "nshaheed" <notifications@github.com> wrote:

> Are the functions working now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nshaheed/eecs448_lab06/commit/91cd6292db7c37703da16e2d5d66064703e9c441#commitcomment-10505075>
> .
>


asking for an inexist env var & not presenting a default value is an **error**, not an acceptable result.  `env()` should not return `null` in this situation - this is a bad implicit.

current `env()` implementation also cause trouble when user choose `null` as default value (unable to distinguish if value is explicitly presented).

talk is cheap, this is my preferred `env()` implementation:

```
function env($key, $default = null)
{
    $value = getenv($key);

    if ($value === false)
    {
        if (func_num_args() == 1) {
            throw new \RuntimeException("Environment variable $key not provided.");
        } else {
            return value($default);
        }
    }

    switch (strtolower($value))
    {
        case 'true':
        case '(true)':
            return true;

        case 'false':
        case '(false)':
            return false;

        case 'null':
        case '(null)':
            return null;

        case 'empty':
        case '(empty)':
            return '';
    }

    if (Str::startsWith($value, '"') && Str::endsWith($value, '"'))
    {
        return substr($value, 1, -1);
    }

    return $value;
}
```

this allows following usage:

```
var_dump(env('REQUIRE_AND_NULLABLE')); // Require, null must be specified
var_dump(env('NULL_BY_DEFAULT', null)); // User must specify null as default value in need
```


Returning a map of Enums is going to cause churn in the generator since map iteration is unordered. This should just be a slice of Enum types or at least a slice of 2 element tuple (key, value) pairs.


Makes sense, so I've changed it. Nothing really depends on the absolute time of the waveform.


Side note,  I also saw a  Page1- block 3 in the ATA5577c.pdf.   Something to implement. :)


Surely `elementCounts` should also have 6 items now?


@tWido this thing is a bit too long and should be in a constant somewhere I believe. Also, using `.format` might save you time, energy, and might make this part of code much cleaner.


i think we should think about ways we can test the app without relying on globals. it's enough that there's global state in K9, we don't need to have it in the tests as well


@Glenpl Gicior, jak proponuję jakąś nazwę to z reguły coś co pierwsze mi przychodzi na myśl, ale nie traktuj tego jak biblię – sam często zmieniam nazwy zmiennych czy funkcji po 10 razy zanim zdecyduję się na coś sensownego ;)


@tWido if you can do `trs.__len__()-2`, you should be also able to do `len(trs)-2` which is once again much cleaner.


This was devilishly difficult to find and debug.  I know the iterator state is *supposed* to be private, but the Array iterator state was previously so simple (and useful) that there may be other places like this.


I would have done this differently.  I would have made targets into a set, and then tested for membership of "JS", "c", "ast", "lex", etc in the set where we now tat for equality.  This would then allow multiple outputs from one run.  "Both" seems like a particular special case, which is all that we need for now ... Which is better?   I usually go for simple, but there is some value to generality, too.


hm is there no way in greenmail to let it pick an arbitrary port? and why are there multiple instances running? shouldn't they get cleaned up after a test? it's been a while i worked on this so don't remember the specifics


> So we can both avoid rebuilding a list, have the fold, keep using descriptive atoms rather than booleans, and avoid having to make update_erlcinfo require some form of ad-hoc accumulator it has no business knowing about.

This seems to be a good compromise to me, although I will still feel little bit strange when seeing things like `Modified =:= modified`. Generally, when I see a variable named Modified, Updated, Expanded etc., I always asume it is boolean (what else could that be?). On the other hand I heartily agree that Modified (whatever it is) shouldn't polute `update_erlcinfo` logic. So what about this?
```
 Modified = lists:foldl(
    fun(Erl, Modified) ->
        case update_erlcinfo(G, Dirs, Erl) of
            modified -> true;
            _ -> Modified
        end,
    end, false, Erls),
 Modified andalso store_erlcinfo(G, InclDirs),
```
I have to say that the last line reads much more flowly for me than the version with case. I admit though that I am pretty used to it so for me it means exactly the same as `if Modified then store_erlcinfo(G, InclDirs)` (if that could be written in Erlang) and it even looks almost the same. And also it's not true that result of this expression is always a boolean - it's either false or the result of the right hand side. See:
```
1> true andalso ok.
ok
```


Looks good!


I absolutely agree with you, but the architecturally 'correct' way of doing this requires (IMHO) appropriate layering within the main application to allow fast test setup/teardown.

Or, each test can delete its own account afterwards. Slower, but cleaner.


@MacDada tutaj musiałem zmienić, bo razem z wcięciami to się zrobiło po prostu zbyt długie :) przy okazji udało się usunąć 2 ify


To chociaz uzywaj nowego('93) castowania, a nie tego z C.
static_cast<int>(...)


There are multiple instances running because the tests need to have multiple accounts (e.g. when deleting an account from a list). I think it's cleaner if each account owns its own server instance.


@Glenpl refaktoruj ile się da – zobacz, że czym mniejsze i wyspecjalizowane funkcje, tym kod mniej wymaga komentarzy – sam się opisuje – nazwa funkcji doskonale mówi co robi, więc tylko czytasz i nawet się nie zastanawiasz.


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/59


Yes, I regenerated it with Roxygen. See https://github.com/vsbuffalo/rivr/commit/8a0f2132d34808203c301408944d156b222aa291


Actually, because the piece of code in question is changed substantially in b86a07e, I will not send a fixup right now, but rather after first round one review during rebase, so that I can also modify subsequent commits accordingly.


I tak się produkuje wtedy tych funkcji pierdylion – klasy w OOP pomagają je grupować, żeby łatwiej było to ogarniać.

Zauważ, że jak będziesz chciał wydzielić klasy, to będziesz miał ich kilka z kilkoma publicznymi metodami, a reszta metod będzie prywatna, bo to będą takie właśnie „funkcje pomocnicze ułatwiające zrozumienie”.


r=pcwalton


@tcnghia Yep.

To understand this, please try to describe what should be deleted when deleting a tag. There is more detail in the original PR for this, but the main issue is that a tag does not uniquely identify a resource.

Do you delete every manifest referenced by that tag?
Do you delete every tag object that carried that name?

It makes much more sense to make the delete specific.


your ALLOW_NOGIL does not seem to do anything, is it necessary?


problème ici : il faut plutôt faire une boucle pour vérifier l'ensemble des clés


Aha, I see.  I'm still caffeinating :sleeping:


i always ask myself, why we need to double text in locale file and in jade? can't we just put data-i18n?


Nit: the `null` check below for `entitlement.getAccountId()` could be removed.


I think that this is the **only** place where you need ```countOneAssertion```, since all of the other assert methods are implemented in terms of this one.


+1 for the pre-commit hook.  While not perfect, it is what is used on the Metasploit side of the house and I think it works well.


@Zimnx Kolejny wykopowicz? :D


@Zimnx W najnowszym commicie już w ogóle nie ma tego rzutowania bo warunek jest w osobnej funkcji inline, tutaj chciałem zrobić jak najkrótszy zapis tylko po to, aby kompilator się odczepił :)
(int) jest znacznie krótsze od static_cast<int>(), chociaż wiem, że "byle krócej" to nie jest dobre podejście


Nit: the `else` on line 165 below could be removed, as well as the `null` check on line 170.


Cool, I'll finish my end and send everything in before friday.


"Page1- block 3"  some day :)
after 9 it would roll to 0 for 2010

looks like the ata5577 was released in 2007


I think that I would rather put in a test for ```numberOfNones ≠ 0``` and print the number only if it is.   Tests without assertions ought to be fixed pretty quickly, and then the briefer message is more pleasant.


PGF: external `PDF`, `JPG` and `PNG`; embedded `ImageRGB8`


@vampiricwulf
Also, concerning the language settings, I thought it would be nice to enable people like @underr to easily add other languages w/o scanning and rewriting code. As for bloat, most of it will be moved server-side at some point and will be hardcoded on template generation. So very little performance impact. I just want to get the client core done before polishing auxiliary systems.


If you adopt the previous suggestion, then you don't need to change these at all.


gurl is the only thing that can decide whether to follow the link, as no other level ever gets the redirect destination. Or am I missing something?


Isn't this the same as alreadyCancelled = _cancelled ?
Also, this is repeated, so making a thread safe getCancelled function might be a good idea.


Thanks. I implemented it into https://github.com/sinkillerj/ProjectE/pull/592
Your code was of giant help.


hello


maybe they only plan a max of 10 year roll out then replaced with a new chip.


yoinks, i guess this had to get deleted?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1229
success: http://build.servo.org/builders/linux2/builds/1227
success: http://build.servo.org/builders/android/builds/1263
success: http://build.servo.org/builders/mac1/builds/1442
success: http://build.servo.org/builders/mac2/builds/1445
success: http://build.servo.org/builders/gonk/builds/409


Why is this a double?


You can file a feature request for a config knob which "removes" line 128:
```js
if (api.engine === "flash") timelineApi.disableAnimation(!!conf.terseFlashTimeline);
```


Thanks for catching this. +1


develop (lefticus) - i386-Windows-7-VisualStudio-12: OK (1167 of 1168 tests passed)

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


Thanks, I'll add those.


self.hashtags.to_a.each { |t| out.push(t.getText) } finde ich persönlich viel eleganter als die dreizeilige Variante. Bei längeren Loops sollte aber die ausführlichere Variante genommen werden.

Was die Return Statements angeht: Können wir uns das mal gemeinsam anschauen? Weshalb ist es best practice, diese wegzulassen? Erschwert das nicht die Lesbarkeit?


@MacDada Zagrałem sobie teraz raz i zauważyłem, że znacznie spadła szybkość działania programu i chyba część funkcji będę musiał zrobić jako inline. Przy każdym wykonanym ruchu dochodzi do około 100 wywołań funkcji, nic dziwnego że o jakiś mały ułamek sekundy wzrósł czas wykonania programu - ale każde 0.01 jest niestety widoczne


svg also supports native `PNG` and `JPG`


@crosbymichael why do we need a server on windows?


Oh, right, how does the "native" stuff work again?


Nice catch.  Now you're in the ancient code.  But yeah... doesn't look like there is a need for this section anymore.


It's so things like systemd and unix file handling are not imported in the windows build where only tcp is supported.


DesignDaysGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


@ivankravchenko  Thanks a lot :)


fast-forwarding master to auto = b79d06c8


viewName is never used after this either so might as well kill it too.


saw approval from cmeiklejohn
at https://github.com/basho/riak_dt/commit/1562c735eaf047308dc0039466fa838d7728468a


merging basho/riak_dt/bugfix/jrd/edoc = 1562c735 into borshop-integration-116-bugfix/jrd/edoc


basho/riak_dt/bugfix/jrd/edoc = 1562c735 merged ok, testing candidate = a2986bcc


We most likely would not implement it. It degrades the flash engine behavior and makes it unsuitable for any serious use and makes the player look bad. So the ugly timeline is a price we have to pay for decent playback.

If you are deploying a flash-only source you can consider some player that implements the control bar with flash (Flowplayer Flash for instance).


It uses the raw png or jpeg data, instead of going though juicy pixels. PDF supports raw jpeg but I haven't implemented it yet.


@vangheem because moving everything to p.a.z3cform and make Archetypes depend on it would be too effort?


Why was the "Parent" column removed from this view?


the newly created csrf shouldn't be added to the response ?


@crosbymichael but why would the Windows client need Server ? Or is this "future proofing" ?


Closes #18.


It took me a while to figure out that you are pushing an object onto the callStack, and that ```GraceCallStackToString``` is a method.  Very nice!   But why do ```var errorString = this```?  ```this``` is a callFrame object, not an errorString.   And then please don't reuse the same variable for a different purpose!  Can't you just use ```this``` on lines 2309, 2311 and 2313?


it was already there and probably for future windows daemon


saw approval from jdm
at https://github.com/servo/servo/commit/66902d2c902243004ad5a158b16fccecf9b8a18f


merging servo/servo/heap-unclassified = 66902d2c into auto


servo/servo/heap-unclassified = 66902d2c merged ok, testing candidate = 8976bbfe


This file should not be committed as these Configurations reflect your specific project config details.


Should be the same as alreadyCancelled = _cancelled

It might be worth making a thread-safe function for void getCancelled() and bool setCancelled(bool val) where set returns the value of _cancelled before setting it to val.


Is this now only applicable to the campaigns type since you are no longer getting the current campaign name when looping over option sets?


the best way would be to bring the whole app into a known state between tests (reinstall), but that's even slower


 Добрый день! Все сделано замечательно!
Мерджить не буду, чтобы у других товарищей из группы была возможность "с нуля" сделать.


Yeah - it's just my mis-understanding GitHub's highlighting


It turns out the order matters.


The `managedObjectContext` returns all the reports of the database. As a result, the sent report contains the totals of all the months, and not only the ones of the current month page. This needs to be fixed.


update tel number


wait, what's happening now..? Is `addr3` being set to `00:00:00:00:00:00` or `uninitialized memory` or .. ?


They can hit 190 or more


my guess is this is a typo and it's supposed to be `addr3`, but it clearly has not affected much :)


1. Make it work
2. Make it right
3. Make it fast

Miałeś działający kod, zrefaktorowałeś go, żeby był czytelny i uporządkowany, jak wydajność do nieakceptowalnego powodu, to natualnie na koniec optymalizacja :)

Tylko znów, zanim zaczniesz inline'ować – zastanów się czy problem tkwi w liczbie funkcji czy dizajnie aplikacji? Bo dużo tych funkcji tak na prawdę nie ma…

Może to algorytm jest do kitu (jak np wielokrotne pętle po wszystkich polach)?


Ahhhhhhhhhhhhh...


Could you please explain the reason for remove the above link from this view.


Whoa, thanks!


Maybe we should change it to `isImage: true`?


This change is not permitted because it fucks up my MacPorts-based OS X dev environment. You can override it locally, of course, just do not commit it. I do want you to contribute, and I do not care much about Q_UNUSED. I was just about to merge this, but then I noticed this. Sorry :-(


It is a docstring.


Yeah, data-i18n is enough. The text in the jade file is only for people not needing to go back and forward between en.coffee and the jade file.


@jasnell that makes sense to me, I'd like to get input from @mkdolan before I rewrite this though. Are we assuming that the main TSC charter can be used for these other projects or will each one have it's own similar to the charter we're working on now? If we're going to share we'd need to do a fair amount of work.


But why not just do an `#include` of that header `#if defined(__linux)` ?


ich hab eben nochmal rein gesehen. Der KeyStore in der KeyStoreHelper Klasse ist nicht Passwort geschützt. Der private Schlüssel schon, aber der KeyStore selbst nicht. Und es wurde ja zur Anforderung gemacht, dass der KeyStore selbst ebenfalls verschlüsselt sein soll. Dabei soll halt noch jedes RZ selbst entscheiden, ob für KeyStore und private key das gleiche, oder verschiedene Passwörter gelten. In meiner Lösung hab ich jetzt jeweils das Gleiche genommen.


hm,  my tag didn't have a P1-B2...  Do you have some more identifications based on the CID?  we only implement two  ATA5577M1 &  ATA5577M2 at the moment...   Should have ATA5567 & ATA5557?


Hi Maciej, Two things:

 At the end of your post, you're thanks Bill & William Lynch. They are both me. So either William Lynch or Bill Lynch (or neither for that matter) is fine.

Your code is better, but still problematic. `errn()` is expected to be treated as a function. So users would expect to call it as you did in your examples with a semicolon after it. The fix that you installed will still cause some issues because of that:

    if (condition[1])
        if (condition[2])
           errn("whatever");
    else
        x = x + 3;

This code will fail to compile. It fails because it will expand to:

    if (condition[1])
        if (condition[2]) {
             fprintf(...);
        }
    ;
    else  // No connecting if
        x = x + 3;

Because of this, I would recommend that you use the `do {} while (0)` idiom as recommended in the original email. It will fix this problem.

Enjoy!
Bill Lynch


r+


@MacDada ilość funkcji i ifów od czasu kiedy kod był ostatni raz szybki spadła, za to ilość wywołań funkcji wzrosła o kwadrat długości boku planszy + kilka. Dla planszy 9*9 funkcja update_field() wowoływana jest 81 razy przy każdym ruchu, i myślę, że to właśnie to jest sprawcą tego spowolnienia :)


@Glenpl Masz np 3 reprezentacje pól: `board `, `board_int`, `shown` – i tym samym robisz po trzy pętle, dla każdej z tych reprezentacji.

Może jakaś struktura? Może da się to połączyć w jedną pętle? Tego typu zmian szukałbym najpierw (poprawy algorytmów), a dopiero jak już model nie może być zoptymalizowany, to wtedy haxiory typu inline'owanie.


I came up with this value after some light testing. I'm sure more thorough testing might give a slightly different result.


Glad that we clarified this in Slack group chat!


all tests pass:
success: http://buildbot.bos1/builders/test-riak_dt/builds/143


http://forum.xda-developers.com/htc-one-m8/development/rom-resurrection-remix-v5-4-0-t3069129
https://www.androidfilehost.com/?a=show&w=files&flid=15720


I would think each Project would have it's own charter modeled after the
initial one. They wouldn't be shared but the Node.js TSC charter would act
as the template for any new ones.
On Mar 31, 2015 2:10 PM, "Mikeal Rogers" <notifications@github.com> wrote:

> @jasnell <https://github.com/jasnell> that makes sense to me, I'd like to
> get input from @mkdolan <https://github.com/mkdolan> before I rewrite
> this though. Are we assuming that the main TSC charter can be used for
> these other projects or will each one have it's own similar to the charter
> we're working on now? If we're going to share we'd need to do a fair amount
> of work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikeal/nodejs-advisory-board/commit/552c1ecae5f86936b6cde6096924478f33935339#commitcomment-10505644>
> .
>


John, can you explain why adding these two fields fixes the issue? I'm not following your logic here.


I finally get you intention. Will send a fixup.


Actually, `range` can be anything iterable with exactly two elements. (But you probably know that and the comment is still okay. I just wanted to explain the motivation behind my suggestion.) There are other places in Lasagne where we could relax the tuple/list requirement at some point.


Fürs each könnt ihr natürlich beide Varianten verwenden. Für mich ist ein Statement pro Zeile lesbarer, aber ist letztlich eure Entscheidung. Die Return Statements sind in der Ruby-Community anscheinend ein prominentes Thema, vorallem Einsteiger tun sich schwer damit. Ruby gibt das zuletzt evaluierte Statement immer automatisch zurück, was das explizite Schreiben von return unnötig macht. Die allgemeine Convention scheint, dies so zu tun, also "return" wegzulassen. Die Gründe dazu sind eher obskur, von behaupteter besserer Lesbarkeit bei einzeiligen Methoden bis zu Performanceeinbussen bei explizitem return (finde ich ein schwaches Argument, Ruby est eh lahm und der Unterschied ist vorallem bei älteren Ruby-Versionen spürbar). Allerdings scheint dieser Stil von vielen Ruby-Programmierern angewendet zu werden. Wenn Ihr wollt, könnt ihr explizite Returns verwenden, ich werde euch sicher deswegen nicht Abzüge machen.

Ich hoffe, das klärt deine Fragen, ansonsten können wir gerne am Freitag weiter darüber diskutieren.


Wouldn't our [make file](https://github.com/acquia/acquia_lift/blob/CH-2304/acquia_lift.make.example#L70) predict the JavaScript file's location?


1st commit


To not break module functional consistency I wouldn't return just the first match, just yield everything.
Also using `position_escaped` is somehow inperformant since the `RegexObject` inside checks the string from beginning. For larger strings this would really cost.
And really use thetest strings in the module, you really can miss critical cases...


IMO, this is only partly done. The default values for the environment values only come into play in the presence of the config json file you are trying to remove. Default env vars should be present in the Dockerfile as an indicator for users who wish to override them.

For example
```
ENV REDIS_HOST 127.0.0.1
ENV REDIS_PORT 6379
```


before only the last id in the list was actually getting posted


Haha same here!


You are not 100% sure to have an 'f' key at this point so this is buggy and needs to be refactored.


thanks


ok


It would make more sense to have the countries list in the models rather than the helpers, I think.
Also by convention the name of the class should be CountriesList. I don't think we use underscores for the names of the classes.

This reminds me of #78 which is about having the languages list in the models.


Cool!


And a big penalty (at least for me): you can't use regexes. The StringProcessing module relies on them and implementing simple processing can easily done with regexes. So maybe we don't allow such functions for this module.
In this case the issued function `unescaped_search_for()` would do the same job.


@jasnell makes sense to me.


repeating this comment in issue #8


@MacDada nie do końca rozumiem, gdzie robię trzy pętle? Każda plansza odpowiada za coś innego i przy każdym ruchu wykonywane jest 0 akcji dla board_int, 1 akcja dla shown i (ilość pól, dla 9*9 jest to 81) akcji dla board
Nawet przy inicjalizowaniu gry i czyszczeniu plansz, wykonywana jest tylko 1 pętla wspólna dla wszystkich plansz


Hi Jens, thanks for stopping by! There is no fixed timeline but with some luck the client side will be complete by the end of April, no idea about the server side yet, have not really looked into it. Good idea about the mailing list, will do it after the travel next week.


@dfberry some of the payment providers do not create and save a payment before returning a failure on a payment attempt.  In the cases where the payment is null in the payment result, the audit was failing as it expected an instantiated payment.  This adds a null check.


W C++11 masz juz w standardzie te dwie metody:
std::to_string(T) i odpowiednio std::stoi(string) - ( ͡° ͜ʖ ͡°)
Propsy za swoje, ale to duplikacja ;)


Using again `position_escaped()` results in bad performance. Nearly all comments from the previous commit ef59f9c apply here too.


Move the block above to line 1377 (`trap _lp_reset runtime DEBUG`) that is currently bash only.


Bumping the version number is not appropriate, only Grant should be doing this


`raise StopIteration` is not needed.


temporary (hopefully) senility.


Those changes in lp_reset_runtime are not necessary because lp_reset_runtime is bash-only.


Especially it won't get called btw...


Aw, thanks! :smile:


I think this is the wrong location for this feature.
The file caching should happen in the Aoe_LazyCatalogImages/Catalog_image helper.
Same with the token 'validation' step.
Keeping the details of how everything past the '/catalog/product/media/' portion is handled out of lci.php is probably best.


Can we capitalize API?


wait now this is never fired
```
if (_isCompletionActive || cursorKeys.contains(e.keyCode)) {
        _handleHelp();
}
```


at some point we should take a look at Jackson, it's supposedly a way faster parser.


we already had the CFP link last week ;)


This function could be replaced by yielding versions of the previous commits, depending if they should yield.


Squash this change with the previous commit to correct the line ending issue.


@dfberry  There are going to be a ton of these.  Due to the way I have the Bazaar site setup, every time I open it in the browser to check something, there is a SqlCE db change.  I probably should exclude the database from the commits (ignore) but have found it easier to simply commit them so that I have my sample data backed up and since we now have a couple of different version of the database, switch branches is important.

Also, the Bazaar site has the package definition and I have not found a good way to back / export it ...


paramters is misspelt here.


do we need to do this setting of servo_out to zero?


@Zimnx według http://www.cplusplus.com/reference/string/stoi/ funkcja stoi jest zdefiniowana w nagłówku string, ale kiedy próbowałem jej użyć dostawałem błąd, że funkcja taka nie istnieje :(


Err, actually, this fixes #26; typoed the issue number.


Make sense. [Deleted](https://github.com/acquia/acquia_lift/commit/13ef377d224d576c0b99c933cbe5f619299899c1).


@jgawor can you give me a feedback with this changes? Thanks!


Redis saves strings into its storage, so true will be converted to value "1", and false to "0"


@dimroc Welp, took me a while to actually run this project, but this commit broke tests for me. It keeps thinking that context isn't a valid method now, even though I've got all the dependencies installed. Can you do "bundle exec rake test" in this project without any errors? I'm currently digging into it.

`dhl_tracking_number_test.rb:4:in `<class:DHLTrackingNumberTest>': undefined method `context' for DHLTrackingNumberTest:Class (NoMethodError)`


Why not instead

```python
for msg in step.messages.filter(visibility=VISIBLE, direction=msg_direction_filter)
```


This method isn't in the latest diff


I still don't understand what is so bad about short, concise (but not cryptic!) code, especially in language with so small syntax :-) That said the version with `lists:filter` is clearly the right thing to do. My version is a vestige of eventually abandoned logic, where I also wanted to utilize tail recursion. But when I changed the logic, I realized that tail recursion is no longer neccessary, but didn't take that second step of cleaning up the code properly.


No i zapomniałem dopisać, że wykorzystuję wersję biblioteki Marka sprzed dodania komentarzy (biblioteka z dnia 27.03)


good point, thanks. I updated commit as advised.


@Zimnx  używam gcc 4.7.1 więc standard z 2011 obsługuje, tym bardziej że w której funkcji używam nawet rzeczy dodanych do języka w tym standardzie - nie wiem dlaczego z tą funkcją jest problem


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 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)


Modifications to Boardconfig.mk


Is this related to https://github.com/meteor/meteor/pull/4053 ?


Section 4.2 (in its current version) was quite useful, thanks for making this public.


You are deciding whether or not to allow redirects based on the scheme of the NEW url. If AllowHTTPRedirects is 'https", this means http->https redirects are OK, https->https redirects are OK, but https->http is not. If AllowHTTPRedirects is 'all' then even https->http redirects are OK.

There's an argument to be made that if AllowHTTPRedirects is 'https', we assume the original server is under admin control and we should trust any and all redirects. IOW, all we care about is the scheme of the original URL.

In any case, this brings up another thing that should be discussed on munki-dev, namely, what is the expected/desired behavior if the scheme changes during a redirect?

(I told you this is more complex than it might initially seem...)


It could go either way... the token name and value are generated to be used/rendered for the controller that answers the current request. You could easily argue the other way. I'm fine keeping them in the request.


not too much effort. Archetypes depending on plone.app.z3cform seems weird but it's not like anyone other than plone uses Archetypes so it probably is fine.


Looks good. I'll merge this in!


https://github.com/Fedjmike/Thoughts/commit/ecba1eb3558362436f2c4dfcc8fe9612e8a9a917

Get reckt


Looks good. (The stuff was hard to edit because the online edit is a pos)


@holycrap872, can I ask why this check is here?  Shouldn't arraysInFactor.size() always be != 0 ?


ok


I think `cmd = 'sudo -u "%s" -- %s' % (sudo_user, cmd)` would be even better (even though unix login cannot contain spaces)


typo "cancelledn"


@chibill Yeah, cause poses are terrible! (whatever a 'pos' is...)


Is this a real comment?


Hi, this modification is due to professor's requirement. During a discussion with professor Gehringer, he asked us to show buttons for 'view dependencies graph' and 'due/start time' instead of showing 'manage bookmark rating rubrics'.


Glad to help and thanks for your feedback. Regarding this error, could you please try these steps until something works:

1) Deinstall py-fann2 completely and reinstall it using the 1.0.7 tag;
2) If that doesn't work, could you use pip or easy install to install the package?
3) If that doesn't work, could you provide a listing of the contents of /usr/local/lib/python2.7/dist-packages/fann2 ?


Hi Eran,

For the count_substring method, the condition should be  "while pos < len(s)" (rather than while pos >= 0)
It works correctly even now because the loop runs till no more occurrence of the substring is found.

Alternately, it might be better to use str.count (https://docs.python.org/2/library/stdtypes.html#str.count)

Also, in context.py (line 76) why do you use shallow copy rather than deepcopy as elsewhere?


@Glenpl Proponuję `BOMBS_COUNT`


Fixes issue #3


I also removed an unneccessary mraa_init


what OS are you running (in ref to your hex editor..)?

I use Notepad 2 (I'll do a pick)
![pj64 rdb_viewed in notepad 2](https://cloud.githubusercontent.com/assets/1712842/6928790/7b672bba-d7c4-11e4-9569-5cde31470947.png)

http://www.flos-freeware.ch/notepad2.html

I noticed those three files had a LF line ending, thankfully Notepad 2 can read all three line endings type just fine ^_^  so I just double lick the line-ending status bar space to change the line ending and save done as easy as pie ^_

so I tried something different, using TortoiseGit I tried committing to a new branch but I could not see the new branch ^_^


Shouldn't there be a space? `if (true) {`

"Separating any reserved word, such as if, for or catch, from an open parenthesis (`(`) that follows it on that line."


@Glenpl Proponuję `hidden_fields_count`


I think this function is somehow not needed. The user can collect the yields manually via list (and the common python programmer should know that).


Was this ever added to SASS? I still can't get imports to work with variables.


@MacDada chodzi o zmianę nazwy stałej? to chyba lepiej BOMBS_AMOUNT


I tried to do this in a tricky way... without success :open_mouth:

![fail](http://gifs.joelglovier.com/fail/cat-mirror.gif)

Thanks for help! :+1:


Not portable; may also be www-data:www-data


Looks like we are missing the restart on installation.


@Glenpl Yep, tak czy siak coś co powie czym jest `bombs`, bo analizując teraz kod musiałem prześledzić użycia, żeby skumać co to reprezentuje ;)


Stuff happens. NP :)


python stdlib usually has such mini-wrappers but I think I agree, I prefer having only iterators and converting to list manually as needed.


@Glenpl 4.8.1 wspiera już na tyle C++11, że nie powinieneś mieć żadnych problemów. W Twojej wersji kompilatora, jeszcze paru rzeczy brakuje, możliwe że std::stoi ;)


Create a iconList with the fonts


Also need lib64 as in RHEL version of PYTHONPATH


Reword this. Maybe:
Provide comma separated list of words that will cause a file to be ignored if they appear in the file's path or name.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1166 of 1168 tests passed)

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


Programa del PIC en CCS


DesignDaysGridView (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2196 tests passed)

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


Comments also to that we were no worse off.


r=jdm


Nice catch!


Fairly happy with my progress in this commit. I have a functional click move, but need to add checkWin, endGame, and squareReset functions.

After that, I need to link my splash page to the "Mode Selection" page, and link input to the name of ctrl.players[0], so that people can be personalized. Then, I need to build the "Mode Selection" page, that allows people to select whether they wish to play another person using FIREBASE app, or a CPU using Math.Random to generate moves.


Does this mean secure paste is working again in R3?

Hope so :)  If so time to update again...


So this should verify that popover is no longer used and can be removed?


@Glenpl Srry, nie wnikałem jakoś głęboko, ogólnie rzucam sugestie, gdzie możesz poszukać problemów (bo nie wierzę, że wynikają z liczby funkcji – w normalnym kodzie masz ich kilkaset i to nie robi ;)).

Jutro odpalę u siebie i zobaczę gdzie realnie jest problem – bo „na sucho” bez profilowania to nie ma sensu.


Oh, I'm sorry. I am not very sure that I can remember the reason very clearly but I'll try to explain it by my best. For this project I need to create my own users but there are some bugs for creating users so I try to create users manually with MySQL commands. I don't want to input all attributes for every user so I just input name, password, role_id. Because the lack of inputting parent_id, this line will give a mistake. And I deleted it to make the server work because this line is not related with my topic. I'm sorry I should get this line back when I committed.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I see. Do I need to remove it now or replace it with database.yml.example?


Зачем убран код загрузки обработки?
Он был нужен для автообновления кода тестов при выполнении. Увеличение юзабилити, чтобы не заставлять пользователя лишний раз не нажимать `Перезагрузить`
Прошу вернуть или предоставить веские причины удаления.
```
-			Попытка
-				лИмяОбработки = ПодключитьНаСервереВнешнююОбработкуОтКлиента(полныйПуть, ТестовыйСлучайОбъектМетаданныхИмя, Объект.РежимОтладки);
-			Исключение
-				СостояниеВыполнения = ВывестиОшибкуВыполненияТеста(ПолноеИмяТестовогоМетода, СтрокаДереваТестов, ОписаниеОшибки(), ИнформацияОбОшибке());
-				Возврат СостояниеВыполнения;
-			КонецПопытки;
```


Ahhhh, that's the function! Thanks!

I may want to lump this with the typo issue as well.


thanks! submitting a fix


@MacDada stwierdzilem, że trochę przesadziłem z tą paniką ( ͡° ͜ʖ ͡°) najwolniejsza rzecz w tym kodzie to chyba wypisywanie na ekran i system("CLS") i jak to często bywa w grach konsolowych przy każdym ruchy widać taki lekki flash


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1230
success: http://build.servo.org/builders/linux2/builds/1228
success: http://build.servo.org/builders/android/builds/1264
success: http://build.servo.org/builders/mac1/builds/1443
success: http://build.servo.org/builders/mac2/builds/1446
success: http://build.servo.org/builders/gonk/builds/410


I’m okay with this change but I’m not convinced by your run-once benchmark. The 100 milliseconds difference could easily be your system system busy with some other task or just random noise.

It’s true, `pry` was not the default console, but it was occasionally useful to run `bundle exec pry` to debug something. That said, I can always add it back to the `Gemfile` when I occasionally want to do that.


Times (seconds):
Input =  0
Solve =  487.54
Output = 0.02
CPLEX 12.6.0.0: optimal integer solution within mipgap or absmipgap; objective 99.6
4282918 MIP simplex iterations
973486 branch-and-bound nodes
absmipgap = 0.0099547, relmipgap = 9.99468e-05


indent/align?


This solves a great many problems in simple way.
:+1:


maybe rename to mergeJSONOrDie and panic from the function?


Here's a couple of wrong ones.


This is wrong.


This might be better defined inside of `init`


Do we need this here?


This commit reduced the build time in the Docker Hub from 25 to 11 minutes!


fast-forwarding master to auto = 8976bbfe


We should probably implement output_warning_spin as a method in the AP_Motors library instead of MotorsMatrix because otherwise Tricopter is missed.


is this needed?


-To add plugin support for the Local login
Run the following command,
postgres@debian$ psql poem < workspace/oryx-editor/poem-jvm/data/database/local_login.sql


I think we should stick with 7 for a while.

Most people use 7 to run there server.
Spigot
http://i.mcstats.org/Spigot/Java+Version.borderless.png

All servers
http://i.mcstats.org/All+Servers/Java+Version.borderless.png


Ok. I've reverted my change, so plone.app.widgets is back in Plone 5.


I'd move these unimportant types to the end of the file, having ListTodos and the other handlers at the beginning.


> The 100 milliseconds difference could easily be your system system busy with some other task or just random noise.

Thats right. The benchmark is really brittle.

I dont know.. TBH I dont care much, if we wanna add it back, it is fine.


I didn't even know Java 1.9 was out....


create a static final for expected size


s/enc/k/


Related to the change from "add output_warning_spin" this might cause the pos-controller to output zero which would lead to tricopters stopping stabilization.


I think the TODO belongs with the implementation


saw approval from jdm
at https://github.com/snf/servo/commit/1cc5bd4287d34afcf65395c59343c726e9c93cd5


merging snf/servo/send_storage_event = 1cc5bd42 into auto


snf/servo/send_storage_event = 1cc5bd42 merged ok, testing candidate = c0c54875


Because it's unique and often shorter.


remove this line


On Tue, Mar 31, 2015 at 02:39:43PM -0700, nrgaway wrote:
> Does this mean secure paste is working again in R3?

Yes. Now we have also automated test for that, to notice such problem
earlier in the future :)

--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?


I think that might just be people who are customizing there java...


This is not needed until you call `writeJSON`, right? move it there


I understand the decision tree, and the code in this PR implements exactly the thing you describe (see also the tests?). My comment was that gurl is the only thing that can even see what the NEW url is.

Gurl could get some object to callback into that can make the decision whether to allow the redirect; open to alternative suggestions, but from where I stand, even the callback object will look kinda messy.


If pageBuilder is not empty add the last page to pages.


delete?


I have systemd-timesyncd on Raspberry Pi with Arch for Network Time Sync,
so starting ntpd fails, as workaround changed on my system line 93 and 94 to:
```
    requirements_arch_service is-enabled enable systemd-timesyncd &&
    requirements_arch_service is-active  start  systemd-timesyncd ||
```
Worked fine for me.
Filed an issue for that: #3363


Your comments sound reasonable. I'm OK if you remove it, if it's not used.


_potential = potential;


I agree that your PR implements this. I'm not sure this is what people are expecting. Thus the need to discuss on munki-dev.


Use ExecStatus::Ptr typedef here instead?


Help me understand this statement. So `c.Bool("guid")` will look for `guid` in context and return `true` if present. Correct?


The naming is surprising here: `RenderSidebarJSON` prints `learnJSON`


Simply yes.


You are right. I have tried, and found the non-campaign else piece no longer reachable. [Deleted](https://github.com/acquia/acquia_lift/commit/1dc70cd4dc66707036ce11428ba345dfc24d7019).


extra line, remove


Just tried it and I can indeed run `bundle exec rake test` with no errors. I took the liberty of checking out a clean copy and running it from scratch and all seemed fine. It seems that the Gemfile.lock was .gitignored, which I'm not a fan of. I've pasted it below so you can track down any discrepancies you might have.

What ruby version are you using? I'm on 2.2.0.

```
PATH
  remote: .
  specs:
    tracking_number (0.8.0)

GEM
  remote: http://rubygems.org/
  specs:
    activemodel (4.2.0)
      activesupport (= 4.2.0)
      builder (~> 3.1)
    activesupport (4.2.0)
      i18n (~> 0.7)
      json (~> 1.7, >= 1.7.7)
      minitest (~> 5.1)
      thread_safe (~> 0.3, >= 0.3.4)
      tzinfo (~> 1.1)
    builder (3.2.2)
    docile (1.1.5)
    i18n (0.7.0)
    json (1.8.2)
    minitest (5.5.1)
    multi_json (1.10.1)
    rake (10.4.2)
    shoulda (3.5.0)
      shoulda-context (~> 1.0, >= 1.0.1)
      shoulda-matchers (>= 1.4.1, < 3.0)
    shoulda-context (1.2.1)
    shoulda-matchers (2.8.0)
      activesupport (>= 3.0.0)
    simplecov (0.9.2)
      docile (~> 1.1.0)
      multi_json (~> 1.0)
      simplecov-html (~> 0.9.0)
    simplecov-html (0.9.0)
    thread_safe (0.3.4)
    tzinfo (1.2.2)
      thread_safe (~> 0.1)

PLATFORMS
  ruby

DEPENDENCIES
  activemodel
  minitest
  rake
  shoulda
  simplecov
  tracking_number!
```


License header?


Inutile, tu peux directement mettre ```return (this.currentSize == 0);```


Makes sense. I'll open a thread.


same as above, remove extra line


Should this be in global/intTypes.h?


Wow this seems really weird ! :-)

Why not `Throwables.propagate(exceptions)` in catch clauses ?

Since the `ex.getMessage()` seems a very weird/tricky thing, I would use an intermediate variable (with explicit naming) to describe the fact that the filename is carried by the exception message (people don't always read javadoc, it will enforce this thing in code :-))


Not public?

I would have expected this would have been the preferred form of creation.


Help me understand this statement. So `c.Bool("guid")` will look for `--guid` in context and return `true` if present. Correct?


So awesome!!!

It's been such a PITA with no copy/paste.


Or is a helper function coming that would parse the JSON file?


b


Oh .. didn't noticed the code was already here before...

Thus, my remarks might concern @xhanin as well :)


Isn't it better to have feedback/suggestions in one place, so devs and most of the community will see it?


Let’s leave it out for now and see how painful that is.


saw approval from javajolt
at https://github.com/basho/clique/commit/454b5eca825d746936b40ddd4ecfc60634a9d776


merging basho/clique/feature/sdc/test-group-leader = 454b5eca into borshop-integration-56-feature/sdc/test-group-leader


basho/clique/feature/sdc/test-group-leader = 454b5eca merged ok, testing candidate = f6e42997


Помоєму потрібно внести даний рядок під умову перевірки індексу, якщо цього не робити, то нема ніяких гарантій стосовно поведінки програми, якщо прийшов не валідний індекс. По любому ж воно буде намагатись звернутись до неіснуючого елемента QList, на що відразу можна отрмати болт.


How does this fix it?

append() is used in order to make the cursor go to the end of the text. That shouldn't be the cause of a null pointer.


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2109 of 2196 tests passed)

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


Unused.


resolves #333


resolves #334


i guess the set_stabilize(true) can be moved outside the if statement.


Pas d'idée pour sa :/


maybe clarify in the comments that this disables attitude control in the motors.


Credit to Hendi


`<UIActionSheetDelegate>` no longer needed


develop (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1167 of 1168 tests passed)

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


What movies should we bring to annoy the mentors??

Sent from my iPhone

> On Mar 30, 2015, at 8:55 PM, Cody Burrows <notifications@github.com> wrote:
>
> I can't edit the code right now which makes me sad.
> Pete:
> Sudo check your email so you can edit it for me!!!
>
> —
> Reply to this email directly or view it on GitHub.
>


Why do you copy everything from jclouds?!


Код не убран, а перемещен в функцию `ВыполнитьТестВЗависимостиОтВидаТеста_МодульИлиФорма`, чтобы подхватывались изменения и для тестов, выполняемых на сервере.


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/60


OK...followed your directions and no joy.   Listing of my efforts attached.  Cheers John

From: Future Link Inc.
Sent: Tuesday, March 31, 2015 2:31 PM
To: FutureLinkCorporation/fann2
Cc: jfperez
Subject: Re: [fann2] fixing issue #6 -- bumping version (beb24b3)

Glad to help and thanks for your feedback. Regarding this error, could you please try these steps until something works:

1) Deinstall py-fann2 completely and reinstall it using the 1.0.7 tag;
2) If that doesn't work, could you use pip or easy install to install the package?
3) If that doesn't work, could you provide a listing of the contents of /usr/local/lib/python2.7/dist-packages/fann2 ?

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

*** DE-INSTALLED OLD VERSION - NOW INSTALLING AFTER CLONING fann2-1.0.7 FROM GIT ***

pi@raspberrypi ~/Git/fann2 $ sudo python setup.py install
running swig
running install
Checking .pth file support in /usr/local/lib/python2.7/dist-packages/
/usr/bin/python -E -c pass
TEST PASSED: /usr/local/lib/python2.7/dist-packages/ appears to support .pth files
running bdist_egg
running egg_info
creating fann2.egg-info
writing fann2.egg-info/PKG-INFO
writing top-level names to fann2.egg-info/top_level.txt
writing dependency_links to fann2.egg-info/dependency_links.txt
writing manifest file 'fann2.egg-info/SOURCES.txt'
reading manifest file 'fann2.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'fann2.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-armv7l/egg
running install_lib
running build_py
creating build
creating build/lib.linux-armv7l-2.7
creating build/lib.linux-armv7l-2.7/fann2
copying fann2/__init__.py -> build/lib.linux-armv7l-2.7/fann2
copying fann2/libfann.py -> build/lib.linux-armv7l-2.7/fann2
copying fann2/fann2.i -> build/lib.linux-armv7l-2.7/fann2
copying fann2/fann2_wrap.cxx -> build/lib.linux-armv7l-2.7/fann2
copying fann2/fann_cpp_subclass.h -> build/lib.linux-armv7l-2.7/fann2
running build_ext
building 'fann2._libfann' extension
creating build/temp.linux-armv7l-2.7
creating build/temp.linux-armv7l-2.7/fann2
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DSWIG_COMPILE -I./include -I../include -Iinclude -I/usr/include/python2.7 -c fann2/fann2_wrap.cxx -o build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for Ada/C/ObjC but not for C++ [enabled by default]
fann2/fann2_wrap.cxx: In function ‘void init_libfann()’:
fann2/fann2_wrap.cxx:11294:21: warning: variable ‘md’ set but not used [-Wunused-but-set-variable]
g++ -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-z,relro build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o -ldoublefann -o build/lib.linux-armv7l-2.7/fann2/_libfann.so
creating build/bdist.linux-armv7l
creating build/bdist.linux-armv7l/egg
creating build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/__init__.py -> build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/fann2_wrap.cxx -> build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/libfann.py -> build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/fann_cpp_subclass.h -> build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/fann2.i -> build/bdist.linux-armv7l/egg/fann2
copying build/lib.linux-armv7l-2.7/fann2/_libfann.so -> build/bdist.linux-armv7l/egg/fann2
byte-compiling build/bdist.linux-armv7l/egg/fann2/__init__.py to __init__.pyc
byte-compiling build/bdist.linux-armv7l/egg/fann2/libfann.py to libfann.pyc
creating stub loader for fann2/_libfann.so
byte-compiling build/bdist.linux-armv7l/egg/fann2/_libfann.py to _libfann.pyc
creating build/bdist.linux-armv7l/egg/EGG-INFO
copying fann2.egg-info/PKG-INFO -> build/bdist.linux-armv7l/egg/EGG-INFO
copying fann2.egg-info/SOURCES.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying fann2.egg-info/dependency_links.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying fann2.egg-info/not-zip-safe -> build/bdist.linux-armv7l/egg/EGG-INFO
copying fann2.egg-info/top_level.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
writing build/bdist.linux-armv7l/egg/EGG-INFO/native_libs.txt
creating dist
creating 'dist/fann2-1.0.7-py2.7-linux-armv7l.egg' and adding 'build/bdist.linux-armv7l/egg' to it
removing 'build/bdist.linux-armv7l/egg' (and everything under it)
Processing fann2-1.0.7-py2.7-linux-armv7l.egg
creating /usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg
Extracting fann2-1.0.7-py2.7-linux-armv7l.egg to /usr/local/lib/python2.7/dist-packages
Adding fann2 1.0.7 to easy-install.pth file

Installed /usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg
Processing dependencies for fann2==1.0.7
Finished processing dependencies for fann2==1.0.7
pi@raspberrypi ~/Git/fann2 $ python
Python 2.7.3 (default, Mar 18 2014, 05:13:23)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from fann2 import libfann
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "fann2/__init__.py", line 4, in <module>
    from fann2 import libfann
  File "fann2/libfann.py", line 26, in <module>
    _libfann = swig_import_helper()
  File "fann2/libfann.py", line 18, in swig_import_helper
    import _libfann
ImportError: No module named _libfann

*** CHANGING DIRECTORY OUT OF THE GIT/FANN2 DIRECTORY ***

pi@raspberrypi ~/Git/fann2 $ cd /
pi@raspberrypi / $ ls
bin   dev  home  lost+found  mnt  proc  run   selinux  sys  usr
boot  etc  lib   media       opt  root  sbin  srv      tmp  var
pi@raspberrypi / $ python
Python 2.7.3 (default, Mar 18 2014, 05:13:23)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from fann2 import libfann
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg/fann2/__init__.py", line 4, in <module>
    from fann2 import libfann
  File "/usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg/fann2/libfann.py", line 26, in <module>
    _libfann = swig_import_helper()
  File "/usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg/fann2/libfann.py", line 22, in swig_import_helper
    _mod = imp.load_module('_libfann', fp, pathname, description)
ImportError: /usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg/fann2/_libfann.so: undefined symbol: fann_copy
>>>

*** DE-INSTALL GIT COPY - NOW INSTALLING FROM PIP ***

>>>
pi@raspberrypi / $ sudo pip uninstall fann2
Uninstalling fann2:
  /usr/local/lib/python2.7/dist-packages/fann2-1.0.7-py2.7-linux-armv7l.egg
Proceed (y/n)? y
  Successfully uninstalled fann2
pi@raspberrypi / $ sudo pip install fann2
Downloading/unpacking fann2
  Downloading fann2-1.0.7.tar.gz (63Kb): 63Kb downloaded
  Running setup.py egg_info for package fann2

Installing collected packages: fann2
  Running setup.py install for fann2

    building 'fann2._libfann' extension
    gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DSWIG_COMPILE -I./include -I../include -Iinclude -I/usr/include/python2.7 -c fann2/fann2_wrap.cxx -o build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o
    cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for Ada/C/ObjC but not for C++ [enabled by default]
    fann2/fann2_wrap.cxx: In function ‘void init_libfann()’:
    fann2/fann2_wrap.cxx:11294:21: warning: variable ‘md’ set but not used [-Wunused-but-set-variable]
    g++ -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-z,relro build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o -ldoublefann -o build/lib.linux-armv7l-2.7/fann2/_libfann.so
Successfully installed fann2
Cleaning up...
pi@raspberrypi / $ python
Python 2.7.3 (default, Mar 18 2014, 05:13:23)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from fann2 import libfann
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/fann2/__init__.py", line 4, in <module>
    from fann2 import libfann
  File "/usr/local/lib/python2.7/dist-packages/fann2/libfann.py", line 26, in <module>
    _libfann = swig_import_helper()
  File "/usr/local/lib/python2.7/dist-packages/fann2/libfann.py", line 22, in swig_import_helper
    _mod = imp.load_module('_libfann', fp, pathname, description)
ImportError: /usr/local/lib/python2.7/dist-packages/fann2/_libfann.so: undefined symbol: fann_copy
>>>

*** CONTENTS OF /usr/local/lib/python2.7/dist-packages/fann2 DIRECTORY ***

pi@raspberrypi /usr/local/lib/python2.7/dist-packages/fann2 $ ls
fann2.i         fann_cpp_subclass.h  __init__.pyc  libfann.pyc
fann2_wrap.cxx  __init__.py          libfann.py    _libfann.so


I think this changes behaviour because now won't it try and stabilize the vehicle when the throttle is above zero?  It's likely that that is ok with the new thr-low-comp changes from Leonard but if the vehicle is leaned over more than 15degrees on take-off it could act quite differently than before in that it would jump up to level once the throttle is lifted just a bit.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1168 tests passed)

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


Can you also document the content-type please?


The first `(` here is unneeded, so remove it instead


Same here. The outer `()` can be removed.


亲，这个是nodejs吧，最基础的http module，得装了nodejs的才work


Lmao didn't see the manifest file haha. Sorry mate


Do I have permission? It says I don't


No space after function


Here's the current workaround from MSHADE-183, I'll try to see if we can work around this on the mvn install


Please use this translation:
```
 "Obtenir clef {{.ServiceKeyName}} pour instance de service {{.ServiceInstanceName}} étant {{.CurrentUser}}..."
```


Space after if


@rbf : Can you review this commit ?


This outfit name is Spirit Caller


- Typo: judul mater, Tutotial
- Buat branch baru sekaligus checkout itu bukannya `git checkout -b <nama-branch>` ya?
- Jadinya mau tetap pakai nama `toki-origin` ya? Setelah gw baca-baca, konvensinya di GitHub itu `upstream`·


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This should be reverted (or the commit dropped if you rebase/squash).


This outfit name is Puppeteer


No space after if


Use:
```
"Pas trouver clef de service {{.ServiceKeyName}} pour instance de service {{.ServiceInstanceName}}"
```


double space after ,


This still didn't fix #7746 it doesn't err-out, however it still doesn't take care of a private repo failing to install.


Is it suppose to hardcoded or it suppose to come from the form selection?


Double space after,


Iterators provide a concrete example of this, which I think is worth mentioning. Something like:

> For example, iterators in Rust are often defined by chaining methods like `map` and `filter` that take closures.  Thanks to static dispatch and inlining, this will often compile down to a flat loop.


Double space after ,


Volgens mij is dit `// Object {a: "example"}`


```
"Récupérer et montrer le guid pour la clef service-key.  Tous autres sortie pour le service ont été suprimer."
```


```
"Montrer information pour clef de service"
```


Space after if. Space after =


@naterkane Is it really not installed? https://github.com/npm/npm/issues/7807#issuecomment-88210390


No space after (


Ik zou hier van maken "Aan een prototype kun je dingen toevoegen" :wink:


No space before :


For ATA5567 and T5557 the CID stood for an optional Customer ID, the
ATA5577 had separate chip options so this was converted to chip type.


Use single quote


Actually this is not quite correct. This should read something like:

"The same statement and proof of Lemma~\ref{lem:asymptotic} works with $\curvature{T,S}$ and $\ric{T,S}$ replaced by the MH random walk curvatures $\curvature{\MH; T,S}$ and $\ric{\MH; T,S}$, respectively."


haha, awesome titles!


You could otherwise take the one I did, it has the correct messages etc compared to an npc that does nothing.


@tWido this is much better, obviously but two things here:

1. wouldn't it be even better to have a constant, say `HLADNYSTUDENT_URL` defined in such a way that you could just directly do

```python
HLADNYSTUDENT_URL.format(day, month, year)
```

and get the string you are expecting in lin right now?

2. you seem to use the same literal URL twice here, could we replace that with a single constant as  described above?


Hmm, Go fmt should complain about this `{` not being on the same line as the rest of the `func` declaration...


just click on the document and then click the pencil. it should let you edit.

-----Original Message-----
From: "Sh1dowdude" <notifications@github.com>
Sent: 3/31/2015 6:15 PM
To: "NovusGames/Deceptus" <Deceptus@noreply.github.com>
Cc: "Colby Newman" <colby.newman2000@gmail.com>
Subject: Re: [Deceptus] Create PLANNED_FEATURES.md (f025d7e)

Do I have permission? It says I don't
—
Reply to this email directly or view it on GitHub.


Any particular reason for that?


:+1: for porting good stuff to R


in this context we have a single known instance of DocumentStore defined above. why would we resolve it from the container?


ahh, it's very simple like you said =)


Yeah, that seems normal considering templates reference timestamp and according to the commit log, it's never initialized. Only I didn't commit that, so I don't really know where these changes are coming from.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


https://github.com/manjaro/packages-community/commit/f64acc359d4e0ed1758eaf704dda68ba23495983
The package is deprecrated, no longer used by MSM. Removed glibc=${pkgver} dependency so eventually we can remove it from the repos without causing problems.


Should we leave this assert here?


always use strict comparison everywhere.
also L85-L90 should stay, b/c what about if show more that 1 result?


EditText.append()'s parameter annotatetd as `@NotNull`.

Try the following code snippet:
```
EditText editText = new EditText(getContext());
editText.append(null);
```
I use Android 4.1.2


Grown ups 2.

Reason:
Deer-urine quality humor, and Adam Sandler


This is unused -- left here by mistake?


That was a huge help! Minitest 4.2 kept getting installed, and adding a version dependency in the gemspec file for 5.5 fixed the problem.

I also took out the Gemfile.lock from the .gitignore file. Thanks!


The corresponding header file is not included, and TravisCI compilation fails.


Oh, and should I get Gary's mod?

Sent from my iPhone

> On Mar 31, 2015, at 6:27 PM, Cody Burrows <notifications@github.com> wrote:
>
> Grown ups 2.
>
> Reason:
> Deer-urine quality humor, and Adam Sandler
>
> —
> Reply to this email directly or view it on GitHub.
>


It's not clear what you mean by "moving mass according to $W_1$ where possible".


Seems you're quite correct. I guess we're rarely trying to look up static symbols, so we've never noticed this. I'm typically working with ARM binaries and, while none that I have on hand have multiple `$a`, `$t` or `$d`, I believe you're correct that these could appear multiple times.

What would your preferred solution to this be? We could store a list in `_symbol_name_map` and return this instead of a single entry? I'm not sure if @eliben has an opinion on this.


Shouldn't this `cmd.ui.Ok()` be after the next statement (the `cmd.ui.Say(...)`)? I ask because it should follow all other CLI interactions where it seems that the `OK` is always last. Please check with Shannon.


You copy the libraries **.so, ...** result of compile http://sourceforge.net/projects/fann/files/fann/2.2.0/FANN-2.2.0-Source.zip/download inside your system shared library folder?. So **swig** and **pip** can locate it before install the bindings.


lowercase error strings


How dare you commit such atrocities!


Same as above. If we decide to move `OK` last then do the same here.


Any Adam Sandler movie or mean girls
On Mar 31, 2015 6:28 PM, "johndunaske" <notifications@github.com> wrote:

> Oh, and should I get Gary's mod?
>
> Sent from my iPhone
>
> > On Mar 31, 2015, at 6:27 PM, Cody Burrows <notifications@github.com>
> wrote:
> >
> > Grown ups 2.
> >
> > Reason:
> > Deer-urine quality humor, and Adam Sandler
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10506943>
> .
>


LatestTaggedImage already does this.


type fatalError error


Этот yjdsq кусок как-то коряво получается если близко к тексту стараться перевести:

> Rust поддерживает мощную систему вывода локальных типов в телах функций. В
сигнатурах же элементов это запрещено, поэтому делать выводы о типах в
сигнатурах разрешено только лишь на основывании этих самых сигнатур элементов. И
тем не менее, из эргономических соображений, для сигнатур функций применяется
очень ограниченый вторичный алгоритм вывода, который называется "элизия срока
жизни". Он делает вывод только на основе компонентов сигнатуры, а не на основе
тела функции, делает вывод только о сроках жизни параметров, и делает это исходя
из трех легко запоминающихся и однозначных правил. Это делает неявные срок жизни
лишь сокращением для написания сигратуры элемента, хотя и не прячет конкретные
типы как если бы к нему был применим полностью локальный вывод.

Есть у кого какие мысли?


Maybe the throttle filter should also be implemented in AP_Motors instead of the motors matrix?  This would make it available to tricopters (and other copters that don't use the matrix).
Should the default be zero to ensure no change from the current throttle behaviour?


I've been fighting bad mouse wheel events since 2007 -- had no idea there's finally a sane standard. Thanks!


Type info is carried when you are converted to an interface - so a check later for fatalError will be true.


Better yet, get "8 Crazy Nights". Adam Sandler,  Deers eating "droppings"
and with the MOST ANNOYING VOICE a character can ever have
On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com> wrote:

> I can't edit the code right now which makes me sad.
> Pete:
> Sudo check your email so you can edit it for me!!!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823>
> .
>


Is this just moving the constraint from somewhere else in the code?  I'm wondering why it's so wide..


> Hmm, it should validate the certificate using home/https-certs.pem instead of ignoring it

Do we expect someone doing MiM attack on 127.0.0.1? I can check if there is way how to give GLib certificate to use...


I'm not sure how to distinguish between the 3 chips other than ata5577 is
the only one with a page 1 block 3
On Mar 31, 2015 5:12 PM, "Iceman" <notifications@github.com> wrote:

> hm, my tag didn't have a P1-B2... Do you have some more identifications
> based on the CID? we only implement two ATA5577M1 & ATA5577M2 at the
> moment... Should have ATA5567 & ATA5557?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marshmellow42/proxmark3/commit/a4669d6ef3a423ff4382326571247ca0615117d3#commitcomment-10505670>
> .
>


Misschien iets meer uitleggen over de chain. Als je een property van een object gebruikt, wat komt eerst, wat daarna, etc.


Move `OK` after if we change based on comments above.


@stevensotelo does this newly added Test (Utf8CharactersTest) cover your contributions?


buy mounts and outfits are only for premiums?


/CC @mdo @cvrebert: maybe we should reconsider the box sizing approach and merge this and https://github.com/twbs/bootstrap/tree/bump-ff-esr


The merge commit went bad. Should be fixed now, please test.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1231
success: http://build.servo.org/builders/linux2/builds/1229
success: http://build.servo.org/builders/android/builds/1265
success: http://build.servo.org/builders/mac1/builds/1444
success: http://build.servo.org/builders/mac2/builds/1447
success: http://build.servo.org/builders/gonk/builds/411


for free players too but I guess @marksamman don't care since in tfs we don't really follow cipsoft way of doing things :P


If you change this from 'slides' to 'Slides', it will fix the capitalization inconsistency in the sidebar and on the slides page.


And about Gary's mod?

Sent from my iPhone

> On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com> wrote:
>
> Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating "droppings"
> and with the MOST ANNOYING VOICE a character can ever have
> On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com> wrote:
>
> > I can't edit the code right now which makes me sad.
> > Pete:
> > Sudo check your email so you can edit it for me!!!
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


@EPuncker is right. If it bothers you, feel free to create a pull request and I'll merge it.


Sure I guess idk
On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com> wrote:

> And about Gary's mod?
>
> Sent from my iPhone
>
> > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com> wrote:
> >
> > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating "droppings"
> > and with the MOST ANNOYING VOICE a character can ever have
> > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> wrote:
> >
> > > I can't edit the code right now which makes me sad.
> > > Pete:
> > > Sudo check your email so you can edit it for me!!!
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> .
>


Here a start of what i want to do: http://uppix.com/f-Sans_titre551b1fb80018a5db.jpg
- C++ template wrapper around linphone api
- Bidirectionnal binding using Swig (one object on python side = one object on c++ size)(can be done due to a custom smartptr handling inside swig and the c++ template wrapper)
- Python deamon using REST webservice(+ web gui) + persistence of services in database
- One audio & video conference service (audio part is already existing), the video part uses FreeJ :)
- All that services are quickly and easily modifiable (not the piping part which is tricky)


Please open an issue, or I will forget this.


+1
that would be a big branch :-)


fast-forwarding master to auto = c0c54875


Элизия - "стирание"?


Restore authorship to @montamer


s/that that/that accept/


saw approval from jdm
at https://github.com/mbrubeck/servo/commit/fb87258fb7cbbf90fb5b4ceaceef42478e6e1eb5


merging mbrubeck/servo/yodalee-issue4768-textencoder = fb87258f into auto


mbrubeck/servo/yodalee-issue4768-textencoder = fb87258f merged ok, testing candidate = dd3c63b6


This change is not neccessary. Undone.


Yes please, we all need replicate, we all need efficient code (even if we don't know) :pray:


I didn't understand the significance of 0x1FF. Is that all the other defines or'd in?


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


My main issue now is the hard link between some feature(TextLayer for example) and sdl (the Freej 2.0 window). And the lack "rotozoom" in linear blits. I'm not sure that my work can be used back in the original project. It will require a huge retro compatibility, adding back the JS interpreter using swig javascript binding and a clean up. Doable but long :)


Maybe call these beforeP and afterP to emphasize their type (and that will help in the debugger I think as well).


This should read if the rover receiver knows the "surveyed position of the base station".  We don't want people to key in an inaccurate base station position in general.

I'm also thinking that this might be too much detail for the message specification.  Since the message could be used by any device using SBP, maybe we'd be better off just explaining what each flag is intended to convey without implementation details.


dido above, should say "Surveyed" base station position


Я думаю, "сокрытие" хорошо подходит.


I have an addon pack at this link: http://steamcommunity.com/sharedfiles/filedetails/?id=395450732

Click subscribe to all to get them and we can play multiplayer with more than vanilla gmod

Sent from Windows Mail

From: Ayyyyyyyy lmao
Sent: Tuesday, March 31, 2015 6:36 PM
To: Team1559/RecycleRush2015

Sure I guess idk
On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com> wrote:

> And about Gary's mod?
>
> Sent from my iPhone
>
> > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com> wrote:
> >
> > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating "droppings"
> > and with the MOST ANNOYING VOICE a character can ever have
> > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> wrote:
> >
> > > I can't edit the code right now which makes me sad.
> > > Pete:
> > > Sudo check your email so you can edit it for me!!!
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> .
>

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


Looks like another space/tab formatting issue here.


I meant, updated bult


Also, spread the link to anyone who wants to play gmod with us

Sent from Windows Mail

From: Ayyyyyyyy lmao
Sent: Tuesday, March 31, 2015 6:36 PM
To: Team1559/RecycleRush2015

Sure I guess idk
On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com> wrote:

> And about Gary's mod?
>
> Sent from my iPhone
>
> > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com> wrote:
> >
> > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating "droppings"
> > and with the MOST ANNOYING VOICE a character can ever have
> > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> wrote:
> >
> > > I can't edit the code right now which makes me sad.
> > > Pete:
> > > Sudo check your email so you can edit it for me!!!
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> .
>

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


Nice! Great to reduce friction like this.


It looks like the CloudFront generation is being generated with a different API version. There might be another generator bug here, but it should be using the latest version.


Same issue with CloudFront here-- this is a different API version.


I guess this is a debug print statement, isn't it?


What version of fann are you using as well?


Do you want it to apply to lab pages AND 'blog' posts or solely lab pages? Do you want the sidebar, related pages, etc. sections to be affected, too?


Minor issue - the comments in lines 314 & 317 need to be updated to use version rather than image_info.


Yes


The challenge instructions just say to return an empty list ( https://docs.google.com/a/teamtreehouse.com/document/d/16nYu-LDgU5YLRPZGRFVMz4ivPjnm_1-kcqfxd0Ehzh0/edit#bookmark=id.yrr5n1mv4n4r ), so I'll leave lines 5-8 off. Let me know if we need to add additional instructions to include this part.


Rather than add a level, let's add a `deprecate` method that delegates to the `warn` method, and print all deprecations at the `warn` level.


Попробовал:

В Rust есть мощный локальный вывод типов. Однако, сигнатуры объявлений верхнего уровня не выводятся, чтобы можно было рассуждать о типах на основании одних лишь сигнатур. По соображениям удобства, очень ограниченный механизм вывода типов сигнатур функций, называемый "сокрытие сроков жизни". Он выводит типы на основании только элементов сигнатуры - тело функции при этом не учитывается. При этом его назначение - это вывести лишь параметры срока жизни аргументов. Для этого он реализует три простых правила. Таким образом, сокрытие срока жизни упрощает написание сигнатур, одновременно не скрывая реальные типы аргументов.


Technically, I think you need to call this callback two different ways, in case the callback fails:
```
client.beforeP().then(function (ignoredResult) { cb(); }, function (err) { cb(err); })
```
You can test this with a callback that raises an exception.  Would be good to test that for both callback and promise versions.


With the eigen part we can now have a full GLscreen using 3D ... you can easily image a video conferencing using this feature


I think this should be done ONCE ever, not every time _get_locale_encoding is called. Keep the value in a state var or my closure or package var. If the value is undef, start require'ing and calling the functions. IDK where you got "GetOutputCP" in Win32.pm from. It doesn't exist in Win32.pm or Win32.xs.


Sorry, didn't notice the attached state had been added.


I'm certainly not a fan of reddit.  Just a personal opinion.
I got all my #! info from the forums ... good enough for me.


You could try resolving with something other than null to ensure that it is ignored.


So the extended start script also uses `-s init stop`. Is it that you are only using the basic bin script that you only care about it there?


+1, looks good to me


@masters3d can you please sort it?


This didn't work


This worked


Work now, thanks


@andreasohlund,
I thought I just mention a few things for your consideration.

IMO this sample can confuse users, it says that uses a UoW but is implemented as a behavior, as a user I would start raising questions about why is not using the publicly exposed `IManageUnitsOfWork` instead.
Also it mentions "Child Containers" why mention it, I don't think it adds any benefit to it, if we use  `IManageUnitsOfWork` all the life-cycle of the component is handled by that abstraction.


Yes, I just never got around to finish putting it up here. Meant to clean it up a bit. I don't have it right here though, but remind me if I haven't put it up in a week.


I would leave the space blank to avoid further convolution. I think a blank space conveys avoid the same meaning as a question mark symbol.


Comentar


What do you mean with VB? These refactorings apply to formulas/worksheets, not VB.


add comment explaining why you are checking for `urlParams['metrics']`


gjk


huh?


Yay, thanks for extracting this!


@alexgreen You'll want to grab this update too. Happy to have some input on the visual styling for these types of graphs.


no need to check-in this.


No, it doesn't change behavior, because the relax_bf_rate_controller(); and set_yaw_target_to_current_heading(); cause the attitude controller to output zeroes to the motor matrix torque inputs.


I did wonder about this. I'll remove it, thanks for the feedback.


hah! thank you for catching, this was debugging #702 :/


Yup, and not there anymore.


no need to check in this file.


iteration (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2172 of 2196 tests passed)

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


90179004-ConstrNumBackNotInitializedComplexGlazing (mjwitte) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

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


By not having a laptop, I am not vulnerable to a physical computer attack.


errors addressed in push


Actually, I am not sure why this file is getting checked in. I just do the
usual git add, commit -m "", push.

WHich file is this?

I had never seen the commit message coming up as this huge thing before.

On Tue, Mar 31, 2015 at 3:55 PM, Xiao Meng <notifications@github.com> wrote:

> no need to check in this file.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simbatech/hive/commit/5abc41c33d15a49d651c342a4a96c2f6194e3247#commitcomment-10507274>
> .
>


we should add this one + hive.resultset.compression.enable instead of checkin the whole configuration here.


Oh, that makes sense.

It seems like a better solution to still use the old code with `append()` if getText() is not null. Otherwise use the new solution. My users have generally liked when the cursor is put at the end of the input.


Question: does an app have to set both tx_attr::msg_order and rx_attr::msg_order? If both are set, which takes precedence? or is that undefined.

Basically, I was looking at the ordering flags and saw that they are duplicated for both TX and RX. If the text is the same, can we just refer back to one section? Otherwise, one must scan it to see if there are any differences when ordering applied to TX or RX?


<3 x ∞

Im at a workshop right now and cant really test - but thanks so much for the PR!


This reminds me that we should use hive logging mechanism instead of use stdout.


how about now? changed hive-site.xml to  only have two config properties.

On Tue, Mar 31, 2015 at 4:00 PM, Xiao Meng <notifications@github.com> wrote:

> This reminds me that we should use hive logging mechanism instead of use
> stdout.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simbatech/hive/commit/5abc41c33d15a49d651c342a4a96c2f6194e3247#commitcomment-10507349>
> .
>


I bought it

Sent from my iPhone

> On Mar 31, 2015, at 6:42 PM, MergeyMergeMerge <notifications@github.com> wrote:
>
> Also, spread the link to anyone who wants to play gmod with us
>
>
>
>
>
>
> Sent from Windows Mail
>
>
>
>
>
> From: Ayyyyyyyy lmao
> Sent: Tuesday, March 31, 2015 6:36 PM
> To: Team1559/RecycleRush2015
>
>
>
>
> Sure I guess idk
> On Mar 31, 2015 6:34 PM, "johndunaske" <notifications@github.com> wrote:
>
> > And about Gary's mod?
> >
> > Sent from my iPhone
> >
> > > On Mar 31, 2015, at 6:31 PM, MaxSpiegs <notifications@github.com> wrote:
> > >
> > > Better yet, get "8 Crazy Nights". Adam Sandler, Deers eating "droppings"
> > > and with the MOST ANNOYING VOICE a character can ever have
> > > On Mar 30, 2015 8:55 PM, "Cody Burrows" <notifications@github.com>
> > wrote:
> > >
> > > > I can't edit the code right now which makes me sad.
> > > > Pete:
> > > > Sudo check your email so you can edit it for me!!!
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10479823
> > >
> > > > .
> > > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Team1559/RecycleRush2015/commit/e988916aa0486744c6b367f2f78142639b2b0c67#commitcomment-10507032>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub.
> —
> Reply to this email directly or view it on GitHub.
>


I Think solely lab pages...that is where it starts to be a drag because the
labs are long...

Sidebar seems fine

related pages I don't care

Julin Maloof
Professor
Department of Plant Biology
1 Shields Ave
University of California, Davis
Davis, CA 95616
(530) 752 8077

On Tue, Mar 31, 2015 at 3:44 PM, Mike Covington <notifications@github.com>
wrote:

> Do you want it to apply to lab pages AND 'blog' posts or solely lab pages?
> Do you want the sidebar, related pages, etc. sections to be affected, too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jnmaloof/BIS180L_web/commit/5f7d08eada8cfc1ace40195df9e812d766681940#commitcomment-10507135>
> .
>


Aligning the values makes it easier to read:

```javascript
_.extend(ipc.config, {
   appspace       : "alloy.",
   socketRoot     : "/tmp/",
   id             : SERVICE_ID,
   maxConnections : 100,
   retry          : 500,
   silent         : true
});
```


Why is it `&T[0,0]` instead of just `T[0,0]` here?


`radius` misspelling.


develop (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Removing this line makes m_resultLightBB[i].Min.Z always uninitialized. I have an alternantive fix here that stops things from growing indefinitely and as far as I can tell does not have a problem of frontal asteroids falling into shadow when they shouldn't: https://github.com/jeske/SimpleScene/commit/ea96599af1e15dd370e1793e0cfe0dda51c24c46


technically, we only need an entry for ::update, as the other two plugins only have their tests affected and not runtime code, but no harm will come by advising the user to upgrade everything :)


No, I didn't notice it in the extended script. Ideally, IMO, both would change to using halt()


if you send a PR to do that for both I can ask others to weigh in quickly and we can give it a try.


Doesn't `VideoDescriptorTestBase` inherit from `TestCase` already?


only decode when needed


I think this will cause a reference error in PyPy.  This can be avoided by moving `m_METADATA` to the bottom.

It still needs to be moved to core though, it shouldn't be an extension as it's a core IRCv3.2 component.


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/1230


You can't just change the OS in the VM settings? There shouldn't be any way to have a GuestId that doesn't match any of the options...

That said, it's a minor change, though it does add more settings to configure on first run, which IMO is getting a bit unwieldy as it is.


Nitpick: If more than 255 binfmt entries fail, the error counter will be outside the allowable range of Linux exit codes.

I would replace the error counter with something simpler.


Hi Jay,

Enjoyed you KGI demo https://www.youtube.com/watch?v=RfNqYyfiKaI

I was exploring psl and its application and found Knowledge Graph Identification in github. I ran iswc13 branch last week successfully. However we're experiencing target server failure even for PSL and KGI/iswc13 since yesterday with the following build error :
Caused by: org.apache.maven.artifact.resolver.ArtifactResolutionException: Could not transfer artifact edu.umd.cs:psl-archetype-example:jar:1.1 from/to psl-archetype-example-repo (https://scm.umiacs.umd.ed
u/maven/lccd/content/repositories/psl-releases/): The target server failed to respond.

We also like to git clone branch slg13 and akbc13 but didn't find it in Github. Are these branches are available for experiments?

--Harihar


Nit: Can you add a quick comment in the code of what the invalid data is, so the next developer doesn't have to look carefully for it?


gdrouet,

sorry but the servlet spec does not allow passing of mocked request/response objects.   See the javadoc of include which says:

     * <p>The request and response parameters must be either the same
     * objects as were passed to the calling servlet's service method or be
     * subclasses of the {@link ServletRequestWrapper} or
     * {@link ServletResponseWrapper} classes that wrap them.

Sorry about that, but the threadlocals needed to track the internal jetty request without relying on this wapping were really having a significant impact of garbage collection.    So it is a valuable optimisation.


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


Forgive my ignorance.   Are you asking me to do something here or are you asking me if I have already done that?  Which system shared library folder are you referring to?

I have the latest version of Fann installed...Fann-2.2.0

pi@raspberrypi ~ $ cd /usr/local/lib
pi@raspberrypi /usr/local/lib $ ls
libdoublefann.so        libfann.so.2       libfixedfann.so.2.2.0  pkgconfig
libdoublefann.so.2      libfann.so.2.2.0   libfloatfann.so        python2.6
libdoublefann.so.2.2.0  libfixedfann.so    libfloatfann.so.2      python2.7
libfann.so              libfixedfann.so.2  libfloatfann.so.2.2.0  python3.2
pi@raspberrypi /usr/local/lib $
You copy the libraries .so, ... result of compile http://sourceforge.net/projects/fann/files/fann/2.2.0/FANN-2.2.0-Source.zip/download inside your system shared library folder?. So swig and pip can locate it before install the bindings.

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

From: Future Link Inc.
Sent: Tuesday, March 31, 2015 3:44 PM
To: FutureLinkCorporation/fann2
Cc: jfperez
Subject: Re: [fann2] fixing issue #6 -- bumping version (beb24b3)

What version of fann are you using as well?

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


Comment?


saw approval from Ms2ger
at https://github.com/kylc/servo/commit/f55c85f42f4e624928d250313c5402beb1d23398


merging kylc/servo/fix_mach_help_msg = f55c85f4 into auto


kylc/servo/fix_mach_help_msg = f55c85f4 merged ok, testing candidate = b20cc69a


typo here.


It inherits from unittest.TestCase, not django.test.TestCase. This was necessary for database cleanup.


Ok, will do. Thanks!


I could make this clearer by using the qualified name of the class if you think that would be useful.


Please [file a new bug](https://github.com/pypa/pip/issues/new) against pypa/pip, I'm probably the only one getting notifications if you comment on my fork. You might want to reference the original issue (pypa/pip#1137). I'll see what I can do then, if you provide steps to reproduce :)

@catox0 you shouldn't need to "apply" anything since this patch is in releases 6.0+. But of course they probably broke it again.


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


Travis runs `git submodule init` automatically after fetching the repo, so this shouldn't be doing anything


I would prefer if we just fully remove this :)


is it possible to disable that limit completely?


I have read the comment in the docs at least 3 times, but hadn't understood it's meaning till I saw this diff.
Thanks!


今天要上nodejs，所以看看怎么弄的hoho。


fix spacing?


submit a PR? happy to accept this.


the idea of the sdks to be just plugins is very smart :D


Will do


the SDKs are just plugins, remember recursion? ;)


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


ack


I accidentally worked on "master" branch instead of "testing_astropy_scripts".


fixed in master. thanks.


Okay. I think I addressed all of your comments. :-)


I've corrected it.

---
Kristen Beck
PhD Candidate UC Davis Genome Center
Biopharma Partnership Coordinator, Corporate Relations
NIH T32 Biotechnology Fellow
West Coast Technical Liaison, Girls Who Code
775-287-1543
www.twitter.com/theladybeck

On Tue, Mar 31, 2015 at 3:34 PM, Mike Covington <notifications@github.com>
wrote:

> If you change this from 'slides' to 'Slides', it will fix the
> capitalization inconsistency in the sidebar and on the slides page.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jnmaloof/BIS180L_web/commit/d689d8e7f24a160ae6ada3dd5914cd8675760980#commitcomment-10507031>
> .
>


In ubuntu 14.04 you can use **/usr/local/lib/** and also can create links in **/usr/lib/**
This what I have in my OS before install with pip.
lrwxrwxrwx 1 root root     18 ago  7  2014 /usr/local/lib/libdoublefann.so -> libdoublefann.so.2
lrwxrwxrwx 1 root root     22 ago  7  2014 /usr/local/lib/libdoublefann.so.2 -> libdoublefann.so.2.2.0
-rw-r--r-- 1 root root 126928 ago  7  2014 /usr/local/lib/libdoublefann.so.2.2.0
lrwxrwxrwx 1 root root     12 ago  7  2014 /usr/local/lib/libfann.so -> libfann.so.2
lrwxrwxrwx 1 root root     16 ago  7  2014 /usr/local/lib/libfann.so.2 -> libfann.so.2.2.0
-rw-r--r-- 1 root root 126927 ago  7  2014 /usr/local/lib/libfann.so.2.2.0
lrwxrwxrwx 1 root root     17 ago  7  2014 /usr/local/lib/libfixedfann.so -> libfixedfann.so.2
lrwxrwxrwx 1 root root     21 ago  7  2014 /usr/local/lib/libfixedfann.so.2 -> libfixedfann.so.2.2.0
-rw-r--r-- 1 root root  83295 ago  7  2014 /usr/local/lib/libfixedfann.so.2.2.0
lrwxrwxrwx 1 root root     17 ago  7  2014 /usr/local/lib/libfloatfann.so -> libfloatfann.so.2
lrwxrwxrwx 1 root root     21 ago  7  2014 /usr/local/lib/libfloatfann.so.2 -> libfloatfann.so.2.2.0
-rw-r--r-- 1 root root 126927 ago  7  2014 /usr/local/lib/libfloatfann.so.2.2.0


I am not fully abreast of what's going on with the Fedora Project, but is anything here worth incorporating back into the main repository? Thanks!


Do you know why this timeout happens on RFM22b especially ?


"A fast, keyless, powerful cipher"... Really, Frank? You're really gonna add and document this? :p


hahaha well thank you, it just made it much easier ;)


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2102 of 2196 tests passed)

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


Peak latency is printed out in oldisim's script  (/workloads/search/search_qps.sh).


Fixed.


And have one inside of MySqlConnection? It doesn't seem worth it. MySqlConnection is the wrapper for MYSQL*, so I'm confused as to what we'd gain by wrapping it again within the wrapper. Did I misunderstand?


os.matchdirs now results in an empty table in pretty much all cases, since the default is to exclude everything.


You should research if in the  **raspberrypi OS** it is the same folder **/usr/local/lib/**.


Added.


Is it `2014-04-01` already? ;)


tyurtyu


I'm concerned about the legibility... Do you think it would be better?


"Intverval"! thanks


Man... You removed the most important of all my comments... ¬¬


Also you do not need to create all the links, if you compile http://sourceforge.net/projects/fann/files/fann/2.2.0/FANN-2.2.0-Source.zip/download with one parameter it has to select where place the compiled libs.


Maybe not? You can open a (tiny) ticket to experiment and make the appropriate change. I'd prefer it not to delay this one though.


```
php composer.phar update
rm -rf app/cache/*
php app/console cache:clear -e prod
php app/console assetic:dump -e prod
php app/console assets:install -e prod
```


sure


Looks good.


Well the reason I didn't PR is because I realised there's a functional difference: merge works recursively (and handles arrays somehow), so you can pass on granular nested objects and keep the vast structure of defaults. I thought that might be the case for ngAnnotate where there's a large map of presets the end user might only want to tweak some of.


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


Lol. Didn't even think about it. @jedisct1 you prankster! :p


hmm.. seems like there is still a missing @Nullable on 'initialState,  line 1213.


Actually it's me being a python noob. Your current script _will_ exit with non-zero if the import fails, sorry! :)


Nah, I misread this while skimming through -- thought it was a helper class, not just a helper func.


You asked for a rationale on this branch earlier. Hopefully this cleans it up a bit? (though I am still unsure as to whether to create an issue on ircv3, all the other commands say something along the following lines:
```
Servers MAY omit certain metadata, which is considered not visible for
the requesting user, or replace it with `ERR_KEYNOPERMISSION`.
```
But CLEAR just says ``The server MUST respond with one `RPL_KEYVALUE` event per cleared key and one
`RPL_METADATAEND` event.`` so I just take that as we should omit keys for which the user doesn't have permission to set.

This should only affect certain custom roles that can see a key but can't set it, so I'm not too concerned.


Thanks man!


Hi Harihar,
We're currently migrating the PSL repos at

https://scm.umiacs.umd.edu/maven/lccd/content/repositories/psl-releases/

and expect them to be unavailable for the next couple of days. In the
meantime, it's always possible to install PSL from source. Just clone
version 1.1 and install:
https://github.com/linqs/psl/wiki/Building-psl

On Tue, Mar 31, 2015 at 7:14 PM, hnahak <notifications@github.com> wrote:

> Hi Jay,
>
> Enjoyed you KGI demo https://www.youtube.com/watch?v=RfNqYyfiKaI
>
> I was exploring psl and its application and found Knowledge Graph
> Identification in github. I ran iswc13 branch last week successfully.
> However we're experiencing target server failure even for PSL and
> KGI/iswc13 since yesterday with the following build error :
> Caused by: org.apache.maven.artifact.resolver.ArtifactResolutionException:
> Could not transfer artifact edu.umd.cs:psl-archetype-example:jar:1.1
> from/to psl-archetype-example-repo (https://scm.umiacs.umd.ed
> u/maven/lccd/content/repositories/psl-releases/): The target server failed
> to respond.
>
> We also like to git clone branch slg13 and akbc13 but didn't find it in
> Github. Are these branches are available for experiments?
>
> --Harihar
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/linqs/KnowledgeGraphIdentification/commit/c20096dfd073ce052e94c4fbbe1a273fa4f949e4#commitcomment-10507509>
> .
>


I just fixed this.


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


If this method will not perform its expected behavior under certain conditions (the room isn't adjacent to the current one), I'd say it would be a good practice to return a `bool` indicating if the operation was completed, so the caller can react if needed.

P.S. You could take a reference to a `Room` instead of the ID to save yourself the work of searching for the proper instance.


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes, this makes a lot more sense now.


I don't know if a mutex is needed. I feel like if the caller races queryUnbuffered and cancel there are bigger problems. I'm not sure how the mysqlclient client implementation address this. If queryUnbuffered is called by multiple threads, there is a much larger problem--I think MYSQL* is tied to the thread that created it.
Anyway, I protected the setting of _isExecuting and _interrupted (it seemed harmless to do, and possibly helpful).


`map` is the name of a built-in function: https://docs.python.org/2/library/functions.html#map


added, not pushed though, copy-pasta


Unclosed tags here. The browser is probably correcting this for you, but be aware of it. It might cause strange issues.


Can we get some unit tests here?


Maybe you should fix it inside https://github.com/aars/loopback-connector-postgresql/blob/7824baa53f52f535810745a3247add1f99d7a6fc/lib/postgresql.js#L232


On 1 April 2015 at 10:28, Scott Arciszewski <notifications@github.com>
wrote:

> Is it 2014-04-01 already? ;)
>
2015-04-01 actually :)


Greetings,

I am inquiring about the possibility of you releasing centro under a BSD like license (same as django itself).
I was hoping to explore building upon it.  Feel free to contact me gmorhart@gmail.com anytime.

Cheers