@disusered Nobody is arguing about COCs. It is up to every project to decide how they want to interact with each other and which standard they want to uphold for their community.

But that is exactly the point, it should be up to the community of the project, and not to the provider of the hardware. Imagine github.com got removed from DNS because it contains the word git.

Nobody is arguing about the removal of illegal content either.

Is this change of a word a minor thing? Sure, but this makes it even worse. If github starts policing repos based on "minor" things and we give them a free pass, we've created a precedence, where they can arbitrarily influence projects, and github has become a too monopolised centre of open source work to just allow that.

Lets also not forget that we are paying them to provide just the hardware and a nice interface to host git repos. Would you be happy if your ISP started filtering out bad words for you?

Github itself has become the google of code. If your open source project isn't on here, it's nonexistent.
Besides the fact that people basically have their CV in their github profiles, and get hired through them.


@onodera-punpun I mean, your github repo is nothing but scripts and themes, but why am I not surprised that the SJW is a hypocrite?


The mio dependency on bytes master is a problem.  Can you please instead track specific revisions of the bytes repo, to avoid breakage? Thanks!


Hey @ehazlett, @tonistiigi,

Can you go build with this? (line 258)
```go
for range decodeChan {
}
```
In my case it works only if I add `_ =`:
```go
for _ = range decodeChan {
}
```


It's possibly a little late for that since it uses a generator, but I can take a pass at trimming out the extraneous stuff.


An extra newline gets injected here by `myFSep` in the pretty printer. I don't know why and I don't know how to fix it, do you have any suggestions.


The mio dependency on bytes master is a problem. Can you please instead track specific revisions of the bytes repo, to avoid breakage? Thanks!


Warning: Illegal string offset 'no' in ...vendor/m1ke/easy-site-utils/display.php on line 311
Call Stack
#	Time	Memory	Function	Location
1	0.0016	302672	{main}( )	.../admin_landlord.php:0
2	0.0895	5567840	html_table_quick( )	.../admin_landlord.php:55
3	0.0895	5569968	make_table_head( )	.../display.php:372


Is returning a pointer (of any flavor) the nicest way to do this here, or would it be better to return the value? Independent of how this function is used, is there a general principal for things created with `new`?


Shouldn't this go into the [Data Structures](https://github.com/avelino/awesome-go#data-structures) section?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.411.08eaafd33556a1c776da74352cbc8880f3e0c7e6](http://build.palaso.org/viewLog.html?buildId=60189&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 130 Build time: 00:06:06


5051_fix_local_win_test (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1777 of 1777 tests passed)

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


Thanks! :+1:


SF :)
Just asking a loosely related question: I'm under the impression that we have some code redundancy between rooms, traps and spells. Is it not possible for us to move part of this code to a parent object? Or would it add unnecessary complexity?


Yep, pretty cool.


Blizz must stop with patches - and LET US HAVE ALL OPCODES!:)))


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (744 of 744 tests passed)

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


@litten 背景太黑和主题不搭，边角是圆角和主题不搭，还是改回原来的吧


@bumblebritches57
Yeah but I don't go around saying that other people opinions are less valuable because the "probably don't even contribute to open source" (which is false btw, plenty of people here who are on the anti-censorship side of the argument contribute to interesting projects.).


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/5351deb04310ff4b4f36ccac44386a41ea4b9e77


merging graydon/stellar-core/perf-caches = 5351deb0 into auto


graydon/stellar-core/perf-caches = 5351deb0 merged ok, testing candidate = 7458449c


Can do.  Does that need releasing?


Zapomniałeś zamknąć body


Co to jest? Co to tu robi?


Yes, the comment seems worthless as it is. I'll see if there's anything beyond it worth mentioning, and if not, I'll just remove it.


Do we want to change the assignment even if `currentAssigned` already contains `tp`?


@aduermael  I think it's a new feature of golang 1.4


@onodera-punpun  You wanna find a source for that? otherwise it's just hearsay, and your side doesn't have a great track record of not talking out of their ass.


`Splitter.on(DELIMITER)` could be made a private static constant `Splitter` so compare doesn't have to construct 2 instances every time it is run.


@Diego-F-Aguirre Comment out this line and it works.


@bumblebritches57 you realize my profile is a click away and I have a handful of repos. The guy who said I only have dotfiles only has dotfiles and a visual theme.... I guess I can't expect much from people like you I suppose.

@j-pb fair point, it was unexpected to see Github step up. Though note that LKML and other huge projects are not hosted on GitHub and not subject to any tone.

@onodera-punpun when people who don't contribute to open source talk about how code's of conduct hurt open source, yes I think their opinions are less valid.


@bumblebritches57
Source of what, this:

>all I see is a group of activists that don't contribute to open source trying to force a private business to bend to their will.

?


@erictune Hmm, if `newAuthenticatorFromOpenIDEndpoint()` fails, then `NewAuthenticator()` will fails, so `newServiceAccountAuthenticator()` is not touched. But that's the same case for `newAuthenticatorFromTokenFile()` or `newAuthenticatorFromClientCAFile()`. Do you mean we should check all the authenticators before return an error?


I wonder if this breaks something. Anyway it's just a temporary fix as the whole concept should be rewritten and test output directory should be assigned by `job`.


the function `before(DELIMITER)` creates could also be made as a static constant function so it doesn't have to be created every time isPasswordHistorical is called


@rodrigok  - IMHO ...  with the current features list, as a minimum:

* server-side activities related to file upload - for 'copyrighted material, censored material' concerns and audit-ability
* RESTfull-API calls  - especially after we add more 'system state changing' APIs beyond the current post message API

Enable/disable/filter on several levels
* level 1 : mandatory --> like the two items above should be always on and cannot be turned off
* level 2:  admin controlled ->  ones that can be turned on and off via admin settings;  log msgs that are of interest to administrators / configurators/ integrators but not necessarily to code devs
* level 3:  code devs only -> what we're familiar with, INFO/WARNING/DEBUG/FATAL and perhaps enabled only via a private console command  or special role setting in DB (no possible way to accidentally trigger using Rocket.Chat itself)

Server side is tricky because whereas with 1,000 concurrent users - we have 1,000 browsers doing the client-side logging ...  we only have one thread handling all 1,000 log generators on the server-side.  BTW... I've used https://papertrailapp.com/ for aggregation and they have been best in their class for ages (free account is more than enough for our demo deployment; they can work off rsyslog to get logging thread unblocked asap)

Thanks :smiley:


Dlaczego dałeś ENTER w środku napisu pomiędzy " " ?
Dlaczego na końcu linii przed średnikiem jest nawias klamrowy zamiast okrągłego?


Memory leak here due to naked news.  `org1` is a copy of the returned `simple_config_origin` and the one allocated by `new_simple` is lost forever.


@gregorysyrkin
Думаю стоит оставить или заменить на Dependences
Ориентировался на [этот репозиторий](https://github.com/robbyrussell/oh-my-zsh)


Should this be final?


Personally I don't quite see the utility of this as it seems to just new a `simple_config_origin` with what presumably could be default parameters.  Also, returning by value would make more sense unless this were some sort of factory function that returned a pointer-to-base, but here (and for the other member functions like this) it is returning a pointer to the same type.


Это термин, так и должно быть


We should rely more on IsStarted to determine if it is running... Always nice to make sure that the back-end functionality is not dependent on the UI. :)


they won't stop, even with expansions: "Next WoW expansion will be unveiled at gamescom 2015!"


кстати, тут вроде множественное число
processes должно быть?


Nice catch :)


you are auto-boxing a `boolean` into a `Boolean`, you shouldn't need the `Boolean` object here since you never return a third null state


There are a couple of files you need to change if you want the admin backend to work.

articles/config.php
Set Database Name, Username and Password.
Set admin Password and Username.

things_to_do/connect.php
Set Database Name, Username and Password.
Set admin Password and Username.

upcoming_articles/config.php
Set Database Name, Username and Password.
Set admin Password and Username.

contact_process.php
Set YOUR email address in the $to section

go into the 'admin' table that you made and set the admin password and username

go into the '_comments' table that you made and set isadmin for one of the users you made to '1'.
( To add a user, just comment in the comment section in the things_to_do page. )

This is not meant to be used commercially because it has security flaws.


Nie rób tak, dla Ciebie to może być teraz czytelne, ale dla kogoś kto to czyta już nie, ty też zapomnisz za jakiś czas co dokładnie to oznacza i będziesz musiał sobie to wykminić, zamiast po prostu przeczytać i wiedzieć.

Każdą zmienną inicjuj w osobnej linii. Wokół operatorów zawsze dawaj spacje.


Thanks for explaining - LGTM.


Or something similar... I believe I made a variable for this in our PictureBoxEx. Not sure how many changed I made are different in the remote-desktop branch...


warning C4127: conditional expression is constant


Reverse after bad conception


Agreed!


Git is a british insult.

This offends me. In the world of SJWs, being offended gives me special rights. Therefore, please take down this offensive repo, ***Hub?


@onodera-punpun  Nope, a source for your claim that SJWs contribute to open source in a meaningful way.


Why does it help to remove this "do" (and others below)?  Does it make the code more readable?  I think not.  Having a "do" here let's me easily introduce extra statements like debug printing without having to think whether the "do" is already present.  As a rule, I always put a "do" when starting a new block.  This is more robust than the other extreme, only putting "do"s when there are >=2 statements.


@disusered Except that GitHub isn't a subset of open source. I can have my own private repositories on this website (or, repos that disallowed redistribution of modified code), and still give opinions as valid as anybody's else. Ad-hominem is never the solution.


@Dwar3xwar DWMR. Compact code. Good. Press on.


This was a bug in the old code. Should be incrementing `i` instead of `s`.


can be replaced by `secondDateTime.compareTo(firstDateTime)`


@blesh because the timing on the CI server was causing this passing test to fail. Should probably refactor to virtual time scheduling when we get that.


Ogólnie staraj się dawać angielskie nazwy zmiennych. Bo się źle czyta kod który w połowie jest po angielsku a w połowie po polsku. Ma to zwłaszcza duże znaczenie jak będziesz pracował z innymi ludźmi. W dużych zespołach takie
'lokalne' nazewnictwo zmiennych jest niedopuszczalne.

Do tego rób wcięcia, tak żeby się łatwiej czytało twój kod.


However, even after unselecting everything, the previous current research progress is filled again, as it's still the current research until "Apply" is clicked, so the effect is maybe not as good as you intended.


@bumblebritches57
Eh I never said that, I said the people who are on the ANTI censorship side of the argument contribute to plenty of interesting projects.


That was quick!


Use *nullptr* for null pointers.


cool. land this guy.


@disusered  I don't know who you are, or why your profile matters... In fact this is the first time I've seen you in this conversation.


@onodera-punpun so we are on the same side, good.


I did not spot it, I had to find out the hard way (by debugging). Btw using `# FIXME:` or `# XXX:` usually helps (although I fail on that on my own and usually send my patches with `prints` incorporated...)


The method is called `import_and_return_unsaved_record`, you're missing the `unsaved`.


I don't really agree with this. There is a difference between tab width, which only relates to the `'\t'` character, and indent amount, which is how much you indent code blocks. They don't have to be equal. The standard tab width is in fact 8. Changing the tab width to change indent amount is incorrect. I set my editor to tab width of 8 and indent amount of 4.


I'd be more for not putting "room" in the readable name, but that's open to discussion. I would put "room" directly in the formatBuildRoom function; the rationale is that we might want to reuse the readable name in other places, e.g. in tutorials, or in the tooltips of the research tree, and then "room" might be redundant.


TEste de tesdte


@ybur-yug Making fun of homeless people doesn't magically become ok, just because you got it from a book.


The defensive copy was not actually needed?


Would a single trailing underscore be also sufficient in the changed identifiers?

Do you care about [the C++ standard specification](http://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier#answer-228797 "What are the rules about using an underscore in a C++ identifier?")?


This is a type, it's further down the file that the default is set.


Even if you agree with me, no need to change it right now IMO, I intend to have a look at dehardcoding the CEGUI tooltips of the research tree now that we have these readable names, so I can do the change then.


a


Unused variable?


@blesh if we had a fast/easy way to do that, I'd say ok. But since this is only hit by internal code paths, it's safe to assume anything that lands here is a `Subscriber`.


Now when I see the last commit merged I noticed, that I did not change the commit message... The caching is actually not enabled... :disappointed: Well, I hope that's not that bad. I can actually enable caching, but at least my trial runs did not bring any improvements, that's why I removed it.


@ndeloof
Please follow the @reviewbybees policy


> At the moment the system assumes that all `CycleCourseGroup` has `ProgramConclusion`.

Well, from what I've been given to understand that is not entirely true. The issue here I believe is the integrated master degree. Whereas some schools issue a different graduation according to each of the cycles comprehended within these degrees (which is the case you are modeling), some other schools issue a graduation for the 1st cycle alone and then a final graduation that refers  to the whole curricular plan. In this case the `CycleCourseGroup` relative to the 2nd cycle will not have have a `ProgramConclusion` associated with. And this code will not work.
Please consider adding the filter I suggested since it's harmless for your use case. To be fully compliant with both scenarios, the algorithm needs to look into the `RootCourseGroup` besides seeking all `CycleCourseGroups`
Thanks


What about people like me that are actually on the spectrum?

I tell ya, I feel a bit more upset that people think I need to be treated like a baby and that I somehow won't be able to withstand some mean words used as a joke. That I need to be protected from jokes because my feelings might get hurt.

I'm an adult, stop treating me like I'm not.


This is also true for output_var_process, can you do the same fix for it?


This change breaks the binary compatibility FYI


/me UI newbie, is truly grateful for every piece of advice :) but here, I am not quite sure if your comment implies that I should change something here around line 51? Maybe you mean that I can remove line 51?


Even stranger is, that it used to fail all the time, then I added it, it started to work and when I removed it afterwards, it kept working .... I should probably test it on a fresh copy.


I'll add a second tab here to remain consistent with the rest of the file.


That's a good point. For now I'll just fix this, but I wonder if it would be easier for this mock to provide a hook to change assignment directly?


If that is all your `LauncherDecorator` is doing, much better to use `EnvironmentExpander`. In fact this whole commit could have been done without any Workflow dependency by just implementing `SimpleBuildWrapper`.


I'll add a tab here too.


@j-pb

You don't understand, @ybur-yug just a *toy repository*, and just a *joke* of course, that makes it okay. *OBVIOUSLY* WebM for [censored] is not a joke.


allConsumed() now returns a copy instead of the underlying collection.


`who_has` is a set mapping keys to addresses of workers that have that key.  We want to iterate over all keys in who_has where there is at least one worker that holds that key.


No, `performOperation()` does not return anything. I just was not sure about the keyword in typescript. Thus +1, I'll change it to `void`.


A good question. Sounds sensible, but I am still getting familiar with this code, so not totally sure. I am learning by reviewing and asking questions at this point. :)


I don't think he's talking about construction of the authenticator, but about the authentication of a particular token


Oh, I had missed that. Thanks.


Although of course Arch went crazy and made python point to python3. :-p


Please check for "!trace"


The message box on this page looks odd due to the addition of the `<p></p>` wrapper in this commit.
![image](https://cloud.githubusercontent.com/assets/3905798/8964604/8633bd22-35f4-11e5-9d49-239afaebcfb4.png)


return -EINVAL; ?


Вполне возможно


Я не до конца понял код, но мне кажется, что тут некорректно используется localization->Get().
```
if (coloringPropertyName == localization.Get("Cancel")) return false;
```
Потому что язык может в любой момент измениться на другой и эта функция вертнет другое значение. Поэтому эту функцию не стоит использовать для идентификации чего-либо. Лучше сранивать свойство name. Также лучше вообще ограничивать себя от использования localization->Get. У любого элемента, который отображает текст, есть дочерний элемент Text, для которого нужно задать свойство autoLocalize = true. localization->Get() лучше использовать для строк, которые формируются на лету и сразу же выводятся, то есть часто обновляются. Может здесь как раз такой случай, я не до конца понял, но сравнивать с этим значением точно не стоит.


Thanks so much. This ability really needs to be a permanent fixture in Cordova


* No data migration
* Test connection button has been removed
* Binary compatibility issue

I'm strongly :-1: :-1: :-1: on such breaking changes without a review

:bug:


No default value?


Looks like you misspelled "canola"! Let me know when you've updated the change.


There's a lot of overlap between checking for single-topic path and single-topic host. Please factor this code out into a separate function to use for both.


Binary compatibility


Valid point. I will fix. However, I would recommend tracking v0.4.x for a bit :)


Ну если plural то точно processes


There should probably be some exponential backoff between retries.


Got it


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/73229478


Oh dear, I think merging #11774 managed to break these again, sorry. Is there a bug somewhere or are these just fragile due to #265?


Поправь, я солью изменения


In legacy ECLWatch, the job name can be modified for a completed WU.


Oh right! It's because I'm using an old dev environment, I didn't realize. Thanks @vieux!


should choose 307 (or another prime number) or delete the subsequent comment


should we add this to our `.gitignore`?


I guess since it is a 0.x plugin there is no compatibility guarantee. Still, should at least edit the mojo in the POM to specify the compatible-since version for settings.


Please put multi-line comments on their own lines rather than next to code.


@Snkz
What. :question:


It's probably just overly fragile.  I like the exposition, but this sort of incremental definition is flirting with bugs… especially as folks will likely try things on their own and then wonder why it broke.  I'll change it to define everything all at once.


Escopo padrão para apenas eventos cuja a data ainda não tenha expirado.


Thanks


@erictune @liggitt I am thinking about adding user info bundles(username, uid, groupid) into the claims. Users can change their email, and still be mapped to the same `sub`, thus the same (username, uid, groupid). The drawback is these bundles are not standard...


Ideally everything would be stored in an env var.


Escopo nomeado (admins) que refere-se a lista de usuários administradores do sistema.


This is terrible code


(facepalm)

That's whay whe have the --python2_path command line argument. We should
have some sort of detection, but for the time being, this is the best we
can offer.

On Wed, Jul 29, 2015 at 7:22 PM, Steven Dee <notifications@github.com>
wrote:

> Although of course Arch went crazy and made python point to python3. :-p
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/bazel/commit/a18964096887e592a14d9553f776e70ce9495131#commitcomment-12426295>
> .
>

--
Lukács T. Berki | Software Engineer | lberki@google.com |

Google Germany GmbH | Dienerstrasse 12 | 80331 München | Germany |
Geschäftsführer:
Graham Law, Christine Elizabeth Flores | Registergericht und -nummer:
Hamburg, HRB 86891


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1769 of 1771 tests passed)

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


This is test comment


Alternately, write a

```java
public FormValidation doCheckServerUrl(@QueryParameter String value, @QueryParameter String credentialsId, @QueryParameter String serverCertificate) {…}
```

which will get automatically invoked any time any of those three fields change.


Ajustando para a ordem decrescente, pois por padrão a ordem seria crescente.


This looks like it was duplicated from your other patches. It may cause a collision on the merge. We'll see.


Feel free to revert it in that case. I'm not even sure why we have these functions in base at this point.


OK like this?


Opa, com certeza estou entrando em contato :-)
Muito obrigado pela força. Te adicionei no Facebook para podermos conversar. O meu é o https://www.facebook.com/gabrielcostatocadotux


fast-forwarding master to auto = 7458449c


Can do


Alteração do escopo padrão para um escopo nomeado em virtude problemas ocorridos com sua utilização como padrão. Criado então o escopo 'Ativos' que refere-se apenas aos eventos ainda não expirados.


I'm tempted to agree. The presence of default parameters or constructor overloads is really inconsistent in the original project, for whatever reason... Which means for some classes (like this one) there's a lot of passing null around, which doesn't work the same way in C++ as in Java.

So then the question becomes, how concerned are we with staying parallel to the original? If not so much, I would gladly replace this with an overloaded constructor/default parameters. So far as I have seen, it's only used for testing ...


cherry-picked to 3.0


Use <=. if var_name_size == name_len, strcpy will copy name_len + 1 bytes.


Yeah, that's what I figured. But it's not documented, so I doubt anyone's actually using it. And it's not tested, so I doubt that it actually works, and doubt it more every time this code changes. In fact, I've suspected that flipping that setting would just make the validation request spin forever for the past few weeks.

If we do decide to keep it, I'd rather just have `run_validator` return a mock success result. But that would need tests. And since I don't think this is currently being used (on account of it probably not working), I'd rather not do that until it's necessary.


As usual, getting rid of them would be even better.


I'd just get rid of this.


this broke compatibility with Windows XP. `CreateSemaphoreEx` is only after Vista.


Ao contrário do escopo 'ativos', o 'expirados' refere-se aos eventos já ocorridos.


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2179 of 2215 tests passed)

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


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 222](https://teamcity.labkey.org/viewLog.html?buildId=263368&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


what is this comment? should we remove it?


And this one


Can we not always use ModuleID.fromString()? It internally takes care of whether there is a version separator or not.


Same comment as on the certificates patch. This may cause a collision once any of the other integration test patches get merged. We'll see.


No complaints; I'm not using Arch for anything serious atm anyway.


should probably call the callback whether there was an error or not


Regarding declaring `before(DELIMITER)` statically, by the same logic would I you not propose I declare all the methods herein statically? What distinguishes this function from the methods?


Are you having trouble with comments? These statements don't look like they should be commented.


i need to test this again.


Yes, this is very different, thanks!


shouldn't this default be `false` to prevent merging arrays for configurations?


I meant the google guava `Function` object it generates.


Thanks for catching this and helping fix.

Just a minor comment above about a redundant check for the version separator. Can you fix that? Then we can merge this PR.


TeamCity xBIM :: XbimScripting :: Build [Build 28](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2161&buildTypeId=XBIM_XbimScripting_Build) is now running


what does the "addr" parameter mean?


Sorry, bad habit.


the underline is too short


@louizo: the point is, to store any data as an array of unsigned chars (i.e. uint8_t) as a fallback. But in practice, we've never used this as fallback.


TeamCity xBIM :: XbimScripting :: Build [Build 28](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2161&buildTypeId=XBIM_XbimScripting_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:55


@vcunat OK thanks. Sometimes it happens but sometimes it's fine. Guess I'll have to debug later.


If you want to interoperate with arbitrary openid providers (which I think the goal of this should be), we either have to use standard openid claims or allow choosing which claims to use, not hardcode non-standard claims.


this is not entirely accurate. This makes it sound like the string is the entire command, but the string is just for representing the type of message that is sent.


true = merge
false = concat

or is there something I'm missing?


@aalsum is there a reason that you pinned this gem to this particular version?


@pollygee FYI this is an example of why we need to make these variables easy-to-configure for nonprogrammers


Without a persisted identity->user mapping like openshift, I'm not sure how to both identify the user unambiguously (from a remembered immutable claim like `sub`) while allowing the username to be the human-readable claim (like `preferred_username`/`email`)


Same as certificates and passphrases.


Yes, agree. But we don't have any exponential or jitter backoff retry in our codebase.

Another [issue](https://github.com/elastic/elasticsearch/issues/11934) is tracking adding jitter and exponential backoff policy from our side. I could cancel this PR and merge the jitter one together, but I think this one should be done first.


Why not to push this method to some factory class?


@floydpraveen Can you tell me the logic here?
`Math.abs(suggestion.length - autosuggest.length) < 3` (what is the logic for less than 3 here)
and
`suggestion.indexOf(autosuggest) != -1 || autosuggest.indexOf(suggestion) != -1` (this logic as well)
/cc @santosh1994


so, something like:
```
private static final Function<String, String> FUNCTION_RAW_HASH = new Function<String, String>() {
    public String apply(String from) {
        return Iterables.getFirst(Splitter.on(DELIMITER).split(from), from);
    }
};
```

and then `return isHashed(password, Lists.transform(Lists.newArrayList(history), FUNCTION_RAW_HASH));`


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1258 of 1258 tests passed)

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


The repl code can probably get away with something much simpler than these functions for the purposes of bracketed paste.


RadianceRefactor (rpg777) - 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)


I think this should go inside the `if(creation)` check, we don't want the slider to change to the querystring-slide every time we do a breakpoint?


Discussion at https://groups.google.com/d/msg/cryptopp-users/B4ePKBCPMs4/nm5UivX3AQAJ.


shall we change this log import as well?


Fixed


Yes, I can do that. I lose the generic quality of the function, but that's not essential.


My fault. I got mixed up because Magento 2 needs p tags and these don't due to the different stylesheet. Thanks, fixed now.


Please review the diff and advise amendments that are needed.


So with the added method page.clearMemoryCache() is it correct to conclude that the signature for the callback would be the following?

```javascript
page.clearMemoryCache(funcition(err){
  //can assert against !err here
});
```


Should I use `$query->getBindings()` here as well?


Update to Sinch SDK 3.6.2


@ulion version 0.8.1 in NPM should have this fixed. Thanks again for letting me know about this!


Maybe this could do with a better name, something which reflects we are getting a query parameter from the URL ? -- currently it is very similar to `_.goTo()` :smile:


`if (err)` like above will be enougth.


I think css should be inlined like scss/sass, but maybe @jhnns knows this for sure.


Just a note: from #1176,  `CUSTOM` is also added, so you may need to fix this when merging with #1176.


You may want to make the console sound a bit more excited.


Added a test for `send` without args. Done the same for `public_send` and `__send__`


Thanks for fixing this! We should move toward using pod UID as the identifier. Mirror pods make things like this more complicated (my bad =( )


I think it was set to `False` by default, so unless someone explicitly activates it... for example people using the docker install without adding any settings won't validate the addons they submit.

@wagnerand is the latest to have used the docker install, maybe you can confirm?


```
page.clearMemoryCache(funcition(err){
  if (err) {
    done(err);
    return;
  }

  browser.exit(done)
});
```


You no like?


This will overwrite the min_axis image


Looks like this commit should have been a PR before it got committed, then it would have been discovered that the `throw PacketArrayMaxCapacityException(newSize, _limit)` lines would cause the build to fail.
Just like jackpoz commented above, VS2013 and GCC needs `_storage.size()+1` instead of `newSize`.


97902934_ZoneMixing (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)


pkg/donut/bucket.go
+ please format Go code with 'gofmt -s', "hash" will be sorted.


Remove brackets, its old fashion. Also feels like the TeamSpeak log now.


There is nothing?


I think you asked me to do this but I didn't get it done. Sorry about that.


Looks good


Dammit, MSVC, "while(1)" should be explicitly special-cased by your warning. Grumble


How do we want to plan or communicate the next release?


Could you move the other formatting functions here too? There's one for the entire buffer and one for the selection.


Here \o/ I see its mentioned in the tutorial. But they should be self containing in my opinion.


I'm so happy about these tests!


I think if the tests are passing and the code review smell test passes then we merging to master w/o regard to release cycles is what we want right?


Apologies, @jckilmer I should have clarified. So the javadoc still gets created in the html format. However, if you run javadoc:jar the javadoc jar is not being created. These jars are getting pulled into the distribution during a release. Is there another command or option to get the jar created?


In terms of a roadmap or something like that? can you elaborate?


@waldoj you changed it to SHA-56 yesterday but it should be SHA-256


BTW I'm not entirely sure I like the idea of all the text fields being `varchar 255`, I would generally prefer to tighten these up further, but for now I wanted to add validations that would prevent db errors for too-long text.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1261 of 1261 tests passed)

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


What about using
typeof obj == 'function'
to detect if it is function or not

Also missing a semi-colon at the end


@waldoj whats the idea behind using `-1.` repeatedly instead of `1)` `2)` etc?


I'd love some context on the name switch from `Future` to `Task`.


This formats the currently line only? We could emphasize that by having the braces already formatted.


So in this case, config_double::new_copy overrides the virtual abstract_config_value::new_copy. So far as I have been able to tell, the only way to allow a different return type on the derived method is to return a pointer to a subclass. How is this best handled? I get override errors when I try to either return by value or return a unique_ptr.


As discussed with @shenoybr, this was mistakenly committed directly to master. Reviewing here.

:shipit:


How about... ?

```javascript
page.clearMemoryCache(function(err){
                assert.ifError(err);
                browser.exit(done);
});
```


@ngoldman Yeah I guess im just not sure what constitutes "ready to go out with the next release". Maybe that just means if the review in step 2 passes...


Is it because Futures are associated with being a different name for Promises?


I was trying to add a hyphen, but somehow replaced the `2` while doing so. :) It's correct as published.


yeah... ; formats the line, } formats the block

I'll change it.


The server is okay with not having a closing brace here? That's pretty nice.


LGTM.


@chelm I think that only works if the PR is patch or minor. Major changes shouldn't go straight to master.


Although I've not tested it...


new functions should have code and docs


yeah... just in the middle of that :)


:+1: :+1:


That allows Markdown to do the counting, so that if you reorder the list or add new items, the numbers will still be correct. Anyway, I left it in your format as published.


It's not redundant.  If there's no pthreads, the "if (spawn)" statement has nothing inside of the "if" block.  It's better to remove the entire "if" statement when there's no pthreads.


are you on drugs? @tkrokli


i think a little elaboration on submitting [DNM] pull requests if more changes are pending might be enough clarification for @chelm here


me too :)


link to gh-release


nope.

If it's unused, delete it.


The second point here makes no sense and has been rewritten in https://github.com/JetBrains/kotlin/commit/22791a171cc4b5ff85a20efa577ac8cdf94b7257


@waldoj oh ok cool. just saw the other thread


Otherwise I agree, any patch or minor update should be fine to merge to master and should help keep the project healthy.


Great choice of columns. This is enough information to act on, isn't it?


Daca folosesti side_effect, return_value e ignorat


yeah. Roslyn is far more tolerant than NRefactory was.


LGTM


hmm.. that test looks wrong though


:ok:


Yababadabababa doo :)


## more comment


@liggitt Ahh, gotcha, thank you! Then I think union [authenticator](https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/auth/authenticator/request/union/union.go#L37) will takes care of this as you said.


Agree on doc (we should probably specify http://usejsdoc.org/), what do you mean by code?


@dmfenton ^


meant tests


Linia asta e echivalenta cu side_effect=Exception. Ar trebui sa faci assert-uri si pe call-urile efectuate asupra mock_execute si de cate ori s-a intamplat asta.


@MariagraziaAlastra Can we have this wrapped in a `<span class="tag">` so that it's easy to target this text?


**I have the LAST word**


No, not unless you count tobacco and coca-cola as drugs. Should I delete my comments?


I am not seeing where cov.unscaled is used. We have an element called
cov.scaled as part of the list and there is also covmat from stanreg(). For
posterior_predict, I guess you need to copy that part of stanreg() or else
we need to return stanmat also.

On Wed, Jul 29, 2015 at 11:44 AM, Jonah Gabry <notifications@github.com>
wrote:

> @bgoodri <https://github.com/bgoodri> If we take stanfit = stanfit$stanfit
> for non-mcmc models then we will lose things like cov.unscaled, right? If
> we're going to have posterior_predict work for non-mcmc then won't we
> need to recreate L?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12423867>
> .
>


Change it to deleteMessage


Tot timpul cand in cod apare time.sleep, trebuie mock-uit. In cazul de aici, poti sa faci assert ca s-a apelat cu "retry_interval", si sa verifici si de cate ori in functie de caz.


Hey Mark,

I'm just now seeing these emails. I was experiencing the second crash you
mentioned before the app started crashing on launch. I may need some help
with the search bar crash as well.

On Tue, Jul 28, 2015 at 9:59 PM dothework <notifications@github.com> wrote:

> @alexblair1 <https://github.com/alexblair1> resetting ios simulator and
> fixing the self.context issue allows the app to run. but it crashes when
> attempting to search on the main tableView. Do you need help with that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexblair1/BlocSpot/commit/fef6f80fb9bbce33aed747471dc3eebd2c419803#commitcomment-12411798>
> .
>


@jagtalon sure ofc!


?? Ignore bundle.js, it may as well be assembly.


Actually, never mind. :)


Ar fi de preferat sa pui assertRaises in _test_execute ca sa poti sa faci si assert-uri dupa.


This restriction will go away when we introduce a way of calling extension functions uniformly with ordinary ones


Looks good. I might have coded it as:

    def subscribe(action: String) = { i =>
         if (sub.subscribesTo(topic.id)) {
              ....

    Since 'sub' (the subscriber) is in scope, and doesn't need to be looked up each time in the foreach loop

:+1:


This file appears to have been checked-in by accident.


:100: so goooood!


RadianceRefactor (rpg777) - i386-Windows-7-VisualStudio-12: OK (2195 of 2215 tests passed)

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


@ches's workaround in comment 49c4af43ec5819d8f5c1a91f9b84296c927ce6e7 is incomplete in some situations.

When the assets:environment task is run, initializers have already run, so simply requiring the gems in the assets group won't run the initializers defined in them.  I found that I had to pull together a list of all Rails Engines & Railties, filter out whichever ones had already run (i.e. from gems not in the assets group), and then run the rest of them.  This is what finally worked.

    namespace :assets do
      task :environment do
        Bundler.require(:assets)
        already_initialized = Rails.application.railties.collect(&:class)
        railties = Rails::Railtie.subclasses
        engines = Rails::Engine.subclasses
        (railties + engines - already_initialized).each do |engine|
          engine.instance.run_initializers(:default, Rails.application)
        end
      end
    end

Also, you need to put this task in your Rakefile **ABOVE** `Rails.application..load_tasks`


Ah I really like this but not sure if we want to mention Jenkins since open source contributors won't have access to it. So maybe edit it as "There are also acceptance and accessibility tests that can run against a sandbox. For more information..."


@ldoktor I suppose it's a good point. Do you want the virttest callers to use re.escape instead of `astring.shell_escape` and drop this patch?


u r welcome:)

2015-07-30 1:41 GMT+08:00 Camilo Aguilar <notifications@github.com>:

> @ulion <https://github.com/ulion> version 0.8.1 in NPM should have this
> fixed. Thanks again for letting me know about this!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buglabs/node-xml2json/commit/1af71b6c20f6e705b837c39a7d126462f9c56e6c#commitcomment-12426731>
> .
>

--
Ulion


It was `True` by default in `lib/settings_base.py`, and not touched anywhere else. I'm about 99% certain that flipping it to false would just break add-on submission.


That is great! Thank you!


fast-forwarding 2.1 to borshop-integration-1153-bugfix/fd/RIAK-1850 = 635d45b2


The wiki was referring to integer division.


@flavour Please review


This command fails in Mac OS X.


Surprised I didn't catch that myself. This is a no-brainer to merge.


This will return an error in the latest systemd version (208-20.el7) because systemd unit files are case sensitive. Please use Type=notify instead. Note that it may have been ignored silently before, which would have caused systemd to revert to Type=simple.


nope me no likey


I'm ok with dropping particular functions if they seem unnecessary; I'd complement that with a comment about why the function doesn't exist. So for this, leave a comment with the constructor about why this function is redundant with default constructors.


Redundant submodule definition, looks like you can remove one.


        commit 091a29fb26f9e4aa3c247d14980a1a9a533fc208
        Author: Lucas Meneghel Rodrigues <lmr@redhat.com>
        Date:   Thu Jul 2 14:00:46 2015 -0300

            avocado.utils.download: Add url_download_interactive

            Add a utility function useful for interactive download
            programs (it shows a progress bar on the terminal, great
            for users to get a better understanding of where the download
            process stands).

            Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>

        commit 987b78dbdc627a05013106f6e5ac3b0859074b7d
        Author: Lucas Meneghel Rodrigues <lmr@redhat.com>
        Date:   Thu Jul 2 14:02:08 2015 -0300

            avocado.utils: Add output module

            Add avocado.utils.output, a library with utility functions
            for user friendly display of information.

            Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>


plus was breaking build so gonna disable it til i figure why it was


of course :) thanks for checking!


So I hope the comment above puts your concerns to rest, @clebergnu and @ldoktor


Created issue [ACDM-897 | https://jira.fenixedu.org/browse/ACDM-897]

`RootCourseGroup` is achieved by `ProgramConclusion.conclusionsFor(this)`


This also shouldn't be checked in. Git doesn't make use of .gitignore until it's part of the repo, so to get ignore you would have first had to stage .gitignore, then everything else.


I'm using a work factor of 12 with bcrypt hashing (where 2**work factor == number of hashing rounds). http://wildlyinaccurate.com/bcrypt-choosing-a-work-factor/ shows some benchmarks, with work factor 12 taking between .2 and .5 seconds to compute (depending on the server). This is quite long in terms of cpu cycles, but something I wouldn't think the would feel much at all. Let me know if you have preference/rationale for a lesser/greater work factor.


oh, true.


Should change the license to say cpp-hocon.


README will need an update.


Can clear out the cpp-project-template lines in this file.


Thanks, I needed `sp_compare = add` in my uncrustify config.

(although it's technically allowed here by the coding guidelines, I dislike the lack of spaces)


Why are PARAM_RAW suddenly used for these options where as before those variables had PARAM_BASE64, PARAM_TEXT or PARAM_BOOL as type indicated?


not really, the `avocado.utils: Add output module` is created on `14:02:08` and is used by a patch `avocado.utils.download: Add url_download_interactive` which is created earlier on `14:00:46`. So GH displays the patches correctly and it's the incorrect order.


![screen shot 2015-07-29 at 2 06 49 pm](https://cloud.githubusercontent.com/assets/1682369/8965707/2175d8b4-35fb-11e5-9c94-ba8ece62aa51.png)


testing....


The data model doc also specifies a `:voting-option/note` string attr.


good point... updating that now.


I hope that it is better


Yeah, the way variables are in scope for nested methods in Scala is still not native to my thinking. I'll refactor as it'll only take a minute and it'll be easier to maintain. Thanks.


If it's a date issue with GitHub, then `git rebase --ignore-date master` should fix it.


That's not the same at all.  The "lock" field says whether or not we should use the lock, or avoid it.


Well it would be nice to not reimplement the wheel. For now we can simply redirect it, though.


97902934_ZoneMixing (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Yes, in javascript we use empty variables as potential placeholders for values.


I think it's supposed to be this based off of Win32 code:
```C++
wheelEvent.trigger(getWindowId(), 0, evt.wheel.x, evt.wheel.y);
```


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


OK!

2015-07-29 10:48 GMT-07:00 Juan Julián Merelo Guervós <
notifications@github.com>:

> OK, perdón @fergunet
>
> Ha sido @mariosky, que no sabe lo de la bibliografía, quien lo ha
> incluido. En todo caso, Mario, si quieres incluir algún item de biblio que
> sea de nuestro grupo usamos directamente un fichero que tenemos también en
> GitHub.
>
> Perdón a ambos. Ya está corregido.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JJ/modeling-volunteer-computing/commit/3bbcdd0c4489b6dcdce238c0bfaaab3d94ecdef8


@gschlossnagle Did you mean
`from urlparse import urljoin`
and not
`from urlparsr import urljoin`
?


But, the value will not be set if the following two 'if statement's do not match.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1261 of 1261 tests passed)

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


The 'lock' field is used/checked in the macro.

```
28  #ifdef HAVE_PTHREAD_H
29 #  define PTHREAD_MUTEX_LOCK(_x) if (_x->lock) pthread_mutex_lock(&((_x)->mutex))
30 #  define PTHREAD_MUTEX_UNLOCK(_x) if (_x->lock) pthread_mutex_unlock(&((_x)->mutex))
31 #else
32 #  define PTHREAD_MUTEX_LOCK(_x)
33 #  define PTHREAD_MUTEX_UNLOCK(_x)
34 #endif
```


I think a workfactor of 12 is fine (for now), the only time it will be triggered is on password changes. The worst case scenario is when a secure account changes their password and have a full history. Then it will have to hash up to `MAX_HISTORY` + 1 times


No need to call `html_safe` here or specify a get request.

http://apidock.com/rails/String/html_safe


the only thing that may affect it is once we see how it actually performs on our VMs


You're missing a `REFERENCES` clause here, e.g., `... doctor_id INTEGER REFERENCES doctors(id), ...`. This acts as a constraint, preventing inconsistent states such as an appointment being updated to an invalid `doctor_id`.


0.5 seconds * 9 hashes = 4.5 seconds to process setting a password


Yes, we'll need to see how it performs there. Assuming worst case bench mark from above, and worse case scenario (full history), it will be 4.5 seconds, correct.


97902934_ZoneMixing (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)


These backslash commands are not SQL statements. The semicolon is optional here.


True, eliminating the invalid addresses is the first step. Later we can do passes for tightening.


Something's wrong here. If this ledger entry must have a signerListID, then it has to be possible to have more than one of them per account. But if the ledger key is a hash of only the account, that won't work. We need to decide now whether multiple signer lists will live in the same ledger entry or in different ledger entries. If the former, the structure of this has to change. If the latter, the way the key is computed has to change.


after we integrate the code I might look into including tracking a custom metric in new relic for password hashing time.


Should these be sets? Or is the vector just how you do this w/ schemas?


What was the reason for this, buginess of the objects?


11% of that userbase is far from the sunset.


The 'lock' field is already used/checked in the existing macro.

```
28 #ifdef HAVE_PTHREAD_H
29 #  define PTHREAD_MUTEX_LOCK(_x) if (_x->lock) pthread_mutex_lock(&((_x)->mutex))
30 #  define PTHREAD_MUTEX_UNLOCK(_x) if (_x->lock) pthread_mutex_unlock(&((_x)->mutex))
31 #else
32 #  define PTHREAD_MUTEX_LOCK(_x)
33 #  define PTHREAD_MUTEX_UNLOCK(_x)
34 #endif
```

In the existent code will be transformed in double if() statements

```
#ifdef HAVE_PTHREAD_H
	if (cbuff->lock) PTHREAD_MUTEX_UNLOCK(cbuff);
#endif
```

will be

```
	if (cbuff->lock) if (cbuff->lock) pthread_mutex_unlock(cbuff->mutex);
```

agree @alandekok ?


Toilets were disabled,


It's an error code for invalid arg


it causes an issue when sending option_id on api... when you send an existing option_id as value, it used to work, but after merging this pull request, it duplicated every option with a new one, within option_id as label.
Evey pull request is an excelent contribution, that should be practiced by every developer, but this one gave me work for a week, restoring the association of these misupdated attributes. :(


**o**r


Let me verify if jobname is editable when job is completed.


ReadVarsSmallDiffTest (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1775 of 1777 tests passed)

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


https://github.com/mozilla/olympia/blob/master/settings.py#L69

I personally haven't used it in a while, but I did use it in the past.


That would be great. I'd like to see how that's done as well.


LGTM.


ACE3 default)**.** (one dot too much)


gubbatron her is an axp user http://gph.is/17VVEO0


new relic documentation on it: https://docs.newrelic.com/docs/agents/java-agent/custom-instrumentation/java-custom-instrumentation


@JBYoshi Getters are fine, you can easily call `getInstance()` on the returned `PluginContainer`. This is for method parameters.


I'm seeing the following performance on my mac:
milliseconds 418
milliseconds 401
milliseconds 375
milliseconds 377
milliseconds 383
milliseconds 370
milliseconds 384
milliseconds 367
milliseconds 374
milliseconds 365
milliseconds 398
milliseconds 376
milliseconds 384
milliseconds 382
milliseconds 382


Is it really worth optimizing for non-visible tooltips? What's so bad about just always setting the text? If `:set_text` on a textbox is really all that bad, then you could safe the text internally. Or did I miss something?


hidden**Se**lection
**whiteboard, TV, PC Screen** being


Just wondering, why was FINISHED added as a deletion reason, as opposed to leaving it null?


Should probably be noted somewhere that if there are no defined remotes (controllers), objects themselves become remotes (please word that nicely though -.-).


Thanks guys , all working for me now.


Ow, my comment was related to line return false;. You need to replace it to return -EINVAL. I was not sure what error code we were using for this error.


Since `\\copy` is not a SQL command, the semicolon here is actually optional.


Might be nice to change OC to RL.


Use `delay_timeout.started` instead.


믿고 하는 pull :) 고맙습니다!


@mapkyca what IDE do you use?
*just curious to know*


It's not necessary but I kept it here on purpose because this is an internal implementation thing and we shave off an extra fcall each time we need to use `immediately()` for a `yield` by storing the reactor instance and accessing it directly.


Not a lot of repeating here, but a little; I would have still left this part setting `@result` in the `before()` function.


Yup, using fully qualified namespace declarations here for 5.5-compat


You still need to specify the full path `https://x.x.x.x:443/auth/login/` and/or disable `follow redirects` and change the alert criteria to !300 code class...


This comment got outdented; also I'd have put it at the top of the function.


No. See https://github.com/mono/monodevelop/blob/master/main/src/core/MonoDevelop.Core/MonoDevelop.Projects.Formats.MSBuild/MSBuildProjectService.cs#L509


D3DBSP_Lib will be released at a later time

Sent from my iPhone

> On Jul 29, 2015, at 5:41 AM, Maurice Heumann <notifications@github.com> wrote:
>
> It's missing some headers and sources, don't you think?
>
> —
> Reply to this email directly or view it on GitHub.
>


Wait, why was this necessary?  The `suites.sh` script should take care of making sure this is correct, and that it will update correctly when supported suites come and go.


Right, thanks for advice


Ok then


Related to issue #12.


This function is for finding the intersection of 2 sorted arrays of numbers. You're using it on unsorted arrays of strings, so I'm not sure how this would do what you want at all.

It also returns the intersection which you don't use either. You just need to know whether the two arrays intersect.

Instead, you can simply iterate over arr1 and check for `arr2.indexOf(<arr1 item>) != -1` then return true (intersects) false otherwise.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf


I'm not sure how to change that actual commit header and details now.  I can append here, apologies as i'm still getting used to git.

The def insync? stuff is all commented out there in that commit because I was just holding onto the code while working on it for use in other parts.  It has been removed from the final commit so far.

The change from newparam to newproperty was to facilitate basically if I wanted to cause the zone to get updated, I could make a change in the description or shortname and it would.  If it's a param then it wont detect your change, so it wont cause it to run a should != want against it.  It was annoying during development and testing.  I don't see a huge problem with leaving them as properties now that it's done though.  You may in your life change a description but shortname should most likely match the zone name.

The change from param to property really helped when changing rich rules or some other properties didn't cause it to detect a change made and cause puppet to push the update to the zone.  So I'd end up having to change the description by a letter or something to cause it to fire off.  All of that should be fixed though!


I'm seeing a WildFly test failure now, not sure if it was this commit but might of been.  We are using a query cache that is configured to be non-transactional.  Test output is at http://pastebin.com/kg3YhxxL

I'll post a link to the test source in a minute.


@kristineds Can you please add a space between the name and Pro suffix?

```php
`$this->name.' <sup`
```

_For the benefit of screen readers that may otherwise read this as `Comment MailPro`._


:clap: :clap: :clap:


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 35](http://teamcity.otenv.com/viewLog.html?buildId=1428805&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 36](http://teamcity.otenv.com/viewLog.html?buildId=1428806&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


A**ACE Slideshow** - probably best to decide for uniformed style for all modules, or just write "This module", for example even the first one is weird "ACE rally points".


What do you mean 'ack for this' like acknowledged?


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 35](http://teamcity.otenv.com/viewLog.html?buildId=1428805&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


Thanks for the new upload_sources.sh script @mfonville and @bgiesing!


**O**bjects


I guess I meant should I keep preventInContainer for the before/after bits (in case people tried to build an image while they were inside a docker container - for whatever reason)


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 36](http://teamcity.otenv.com/viewLog.html?buildId=1428806&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


90946736_SHRgt1_4567 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1770 of 1773 tests passed)

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


**I**mages
from the mission **folder/file** or another mod (or something like it)


This isn't bad, but the built-in [url node module](https://nodejs.org/api/url.html) would be even better. (Think we want `hostname` from that.)


That's because it's a static function so we don't have more information at this point. However, it is always good to add a small log so we know that something went wrong here (even if we don't know why)


We should return as early as possible in `classify()`. This check should be moved near the top of the function.


disabled**,** number (missing comma)


Yes, Atronix builds are compiled with builtins. This is actually one of the reasons I integrated these builtins, they are just simpler, more robust and efficient.


Following up with https://bugzilla.xamarin.com/show_bug.cgi?id=32494.


hidden**S**election


o/ o/ o/


In this case, we would have to check nullptr so there would be 1 check as well


good solution! although I didn't see/encounter this issue myself


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 44](http://teamcity.otenv.com/viewLog.html?buildId=1428819&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


Faça o logout e login novamente, reindexe e limpe o cache.


Should note that `ace_interaction` is not needed in that case (somewhere, not necessarily here), hence why it's not in dependencies (I have to add failsafes at some point though, refer to #1945).


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 44](http://teamcity.otenv.com/viewLog.html?buildId=1428819&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 45](http://teamcity.otenv.com/viewLog.html?buildId=1428820&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 45](http://teamcity.otenv.com/viewLog.html?buildId=1428820&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


sure, actually for output vars there are two different checks that could return before unlocking.
thanks for spotting that.


No big deal, but odd that this rename shows up as a file-deletion and a file-addition, though `company_lookup` is ok. May as well rename the `_spec` files to match those underscores, too.


**N**ote that all names and interaction names **are examples.**


missing space before the controller name.


Yep, which is why I made the API branch ;)

You'll want to be implementing some stuff like XPHandler soon, so if you want to merge in the API branch let me know


I agree many things are common but not that much can be shared (functions required for traps, rooms and spells are different so the function pointers also are and the data). They are clearly copy/pasted but not identical.


**O**bjects


**I**mages


Oops, sorry I meant cov.scaled not cov.unscaled. What I was trying to say
was that if stanreg doesn't return cov.scaled then we can't recreate L
inside posterior_predict since the input to posterior predict is the output
from stanreg.

On Wed, Jul 29, 2015 at 1:56 PM, bgoodri <notifications@github.com> wrote:

> I am not seeing where cov.unscaled is used. We have an element called
> cov.scaled as part of the list and there is also covmat from stanreg(). For
> posterior_predict, I guess you need to copy that part of stanreg() or else
> we need to return stanmat also.
>
> On Wed, Jul 29, 2015 at 11:44 AM, Jonah Gabry <notifications@github.com>
> wrote:
>
> > @bgoodri <https://github.com/bgoodri> If we take stanfit =
> stanfit$stanfit
> > for non-mcmc models then we will lose things like cov.unscaled, right? If
> > we're going to have posterior_predict work for non-mcmc then won't we
> > need to recreate L?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12423867
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12427070>
> .
>


ack


Hmm, the test might make assumptions that it shouldn't.  Test is at https://github.com/wildfly/wildfly/blob/master/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/jpa/secondlevelcache/JPA2LCTestCase.java#L229

Test bean createEmployee() that is invoked from above test class is at https://github.com/wildfly/wildfly/blob/master/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/jpa/secondlevelcache/SFSB2LC.java#L358 and queryCacheCheck() is at https://github.com/wildfly/wildfly/blob/master/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/jpa/secondlevelcache/SFSB2LC.java#L232

In queryCacheCheck(), the second call to query.getResultList() is expected to result in a cache hit but from the test failure message (Expected 1 hit in cache(hitCount=0, missCount=2, putCount=2). expected:<1> but was:<0>) that is not happening.


Thanks Willy. Here's the [updated pull request](#https://github.com/mapsense/mapsense.js/pull/27)


Yaaay, empty lines! :grinning:


@jonsterling That seems like good reasoning, this goal sounds possible but I'm having trouble seeing through the term-soup to see how to prove it :/

Thought though, perhaps we could add a primitive operator for subtyping and then we can add `rec` and `corec` to the standard library and derive the normal mathematical (co)induction principles *once* by proving them parametrically over the function we apply `rec` to under the assumption it's monotone/(co)continuous. Isn't that what NuPRL does?


no**,**really (makes it easier to understand)


@jozefg Yes, I think that's what Nuprl does...


Nice work here! But a few errors were made


This commit appears to have broken the junit tests for the CommandLineParser class as well as others.


RadianceRefactor (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)


@jonsterling Worth doing here do you think? It'd be really cool in my mind if we had a simple way to do induction/coinduction *in the standard library*.


More tests please: `%Math_pow%(_base_, _exponent_)` in particular.


`ace_slideshow` (code tags? no idea what the standard is)
information (remove **s**)
mission**-**tools (or whatever)?


**S*ettings


No need, that will be prevented automatically.


Yes, I would very much like to have that rather than do these gnarly
proofs one-by-one! :)

On Wed, Jul 29, 2015, at 11:42 AM, Danny Gratzer wrote:
> @jonsterling Worth doing here do you think? It'd be really cool in my
> mind if we had a simple way to do induction/coinduction *in the standard
> library*.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jonsterling/JonPRL/commit/0ec9c341e1f9c796f71f4a99f23eb3cfeb8e0a3d#commitcomment-12428154


@jonsterling I'm gonna have a go at this and see how far I can get :) Will report back!


Excellent! Super excited to hear how it goes :)

On Wed, Jul 29, 2015, at 11:43 AM, Danny Gratzer wrote:
> @jonsterling I'm gonna have a go at this and see how far I can get :)
> Will report back!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jonsterling/JonPRL/commit/0ec9c341e1f9c796f71f4a99f23eb3cfeb8e0a3d#commitcomment-12428177


whatever you clic while defining the research tree, the current research is the current research as long as you validate with the apply button. For the same reason, if you open the research tree, unselect everything and let the tree open, if a grimoire is brought to the dungeon temple giving enough points, the current research will be validated. That's because when you open and play with the tree, the server doesn't know what you are doing and, thus, will continue what was planned. Of course, we could warn the server when such events occur but we would have to clearly define what we want (stop when the tree is open ? when the currently research is unselected ?). That's a source of problem for something that might not be an issue IMHO.


(y)


Note that I've done it that way to do the same as for traps. And for traps, the wooden door do not show "trap". that's why I did it this way. However, that can be changed later as you say


maybe you should cache your hits? if your trying to rely on cache-control.. its not going to work.. as xem sends headers like: `< Cache-Control: max-age=0, private, no-store, no-cache, must-revalidate`


@manucorporat - quick question.  You mentioned:

>We encourage developers to use a middleware to handle the error responses, so they can split the error logic, from the normal flow logic.

>We recommend this:

```go
func handler(c *gin.Context) {
    var json JSON
    if c.Bind(&json) == nil { // error is managed by a middleware (NOT HERE)
        // do stuff, success flow!
        c.JSON(200, gin.H{"status": "ok"})
    }
}
```

I'd like to do this, but I don't see how to make it work (lacking generics in golang) without creating a custom error handling middleware for each type that can be posted.

For example:

```go
type Foo struct {
   Name string `json:"name" binding:"required"`
}
type Bar struct {
   OtherName string `json:"othername" binding:"required"`
}

func FooValidationCheck() gin.HandlerFunc {
  return func(c *gin.Context) {
    var foo Foo
    if err := c.Bind(&foo); err != nil {
      c.JSON(400, err)
    } // else continue
}

// need a separate one for Bar
```


that was like this in the old code too ( getcwg ), I just commented it.


Isn't the input to posterior_predict the output from stan_glm? In that
case, cov.scaled is an element of that but not of the stanfit element.

On Wed, Jul 29, 2015 at 2:38 PM, Jonah Gabry <notifications@github.com>
wrote:

> Oops, sorry I meant cov.scaled not cov.unscaled. What I was trying to say
> was that if stanreg doesn't return cov.scaled then we can't recreate L
> inside posterior_predict since the input to posterior predict is the output
> from stanreg.
>
> On Wed, Jul 29, 2015 at 1:56 PM, bgoodri <notifications@github.com> wrote:
>
> > I am not seeing where cov.unscaled is used. We have an element called
> > cov.scaled as part of the list and there is also covmat from stanreg().
> For
> > posterior_predict, I guess you need to copy that part of stanreg() or
> else
> > we need to return stanmat also.
> >
> > On Wed, Jul 29, 2015 at 11:44 AM, Jonah Gabry <notifications@github.com>
> > wrote:
> >
> > > @bgoodri <https://github.com/bgoodri> If we take stanfit =
> > stanfit$stanfit
> > > for non-mcmc models then we will lose things like cov.unscaled, right?
> If
> > > we're going to have posterior_predict work for non-mcmc then won't we
> > > need to recreate L?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12423867
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12427070
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428067>
> .
>


@brandonly101 Should clean up the commented out code. Other than that, it looks good.


why?


I know, a perfectly fine response would have been "no, stop following
master."  I was following a particular commit, but that commit contained a
Cargo.toml which followed bytes/master, so I screwed myself :)  I forked
and updated Cargo.toml and am tracking my own fork; I wanted the 0.4.x
changes over 0.3.x.  Now that you have the 0.4.x branch I'll follow that
instead, that's awesome.  Thanks :)

On Wed, Jul 29, 2015 at 1:39 PM, Carl Lerche <notifications@github.com>
wrote:

> Fixed
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/carllerche/mio/commit/272fb3d06e8f7134c9611e1877b3ff71642ced67#commitcomment-12426685>
> .
>


These need to be virtual to actually override the parent.


it was not working... im looking into it seperately but i needed something
so i could at least build experimental without it messing up the
distributions file

On Wed, Jul 29, 2015 at 11:28 AM, Tianon Gravi <notifications@github.com>
wrote:

> Wait, why was this necessary? The suites.sh script should take care of
> making sure this is correct, and that it will update correctly when
> supported suites come and go.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/calavera/docker/commit/2eee19236650c5146274172e08751cfce52c2fbe#commitcomment-12427839>
> .
>

--

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>


still trying to understand how we're getting this case.  The port_id in portbindings table has a FK reference to ports, so this should not occur because of constraint checking.


Yes. And I use "cooldownTurns - 1" turns to compute the cooldowntime. Which is what I originally used as it was what I expected. But after trying, It appeared to work better with "cooldownTurns" so I thought I made some mistake and changed. Now, I understand why ^^


Is it really necessary to read the file? Any reason not just to compare with the values in ENV?


it was removing utopic, then when reprepro-ing it would fail with utopic
has been removed

On Wed, Jul 29, 2015 at 11:47 AM, Jessica Frazelle <me@jessfraz.com> wrote:

> it was not working... im looking into it seperately but i needed something
> so i could at least build experimental without it messing up the
> distributions file
>
> On Wed, Jul 29, 2015 at 11:28 AM, Tianon Gravi <notifications@github.com>
> wrote:
>
>> Wait, why was this necessary? The suites.sh script should take care of
>> making sure this is correct, and that it will update correctly when
>> supported suites come and go.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/calavera/docker/commit/2eee19236650c5146274172e08751cfce52c2fbe#commitcomment-12427839>
>> .
>>
>
>
>
> --
>
>
> Jessie Frazelle
> 4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
> pgp.mit.edu
> <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>
>

--

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>


@terrellt : So why did we do this? We've ingested a heck of a lot records using the old predicates...


lolololol


@ldoktor, the order of the patches is correct in the branch graph, github likes to display in date order, but this shouldn't affect the actual result.


@miigotu so our changes to xem_refresh https://github.com/SiCKRAGETV/SickRage/pull/2179 won't work either. We still access xem for every show even if it's no mapped


no, nothing you're missing.

i just didn't follow the code path correctly.

you're good.


Why?


@clebergnu I'm going to start using this `--ignore-date` thing, so that github behaves.


![maduro gozon](https://cloud.githubusercontent.com/assets/163977/8966778/3eaf54e0-3601-11e5-952a-1021508a6a99.jpg)


I'll pm you.


Is that always the case, or only in certain circumstances? Because I've found several examples that don't have it on the derived function, only the base function.


Se a solução do @rafaelstz não funcionar, tente salvar o perfil de admin.


Thanks for fixing the translations!


Because these things actually happened...
<img width="638" alt="screen shot 2015-07-29 at 2 44 00 pm" src="https://cloud.githubusercontent.com/assets/163977/8966796/4e8a7188-3601-11e5-992b-5ded936abedc.png">


@alexmurray There you go; I amended the commit and force pushed it upstream (b3b9a566927a8242f0b1afcf5db4f8381f077818)


iojs vrs nodejs perhaps


maybe we have to do this way: https://github.com/SiCKRAGETV/SickRage/pull/2175/files


@thet you need to be careful update resources.

See https://github.com/plone/Products.CMFPlone/commit/2d3865805efc6b72dce236eb68e502d8c57717b6
and
https://github.com/plone/Products.CMFPlone/commit/bd1f9ba99d1ad40bb7fe1c00eaa32b8884aae5e2


nice! lgtm


Vadim, DON'T COMMIT PROPERTIES FILE!!!!


make sure program handles empty channel


This function is also defined in application.js. If they are not different, then you can DRY up your code. Here's one way of doing it.:

Use a namespace for your program.

```js
$(function() {
  window.contactApp = {   // <--- Create an object contactApp that will be global. It will hold all your functions.
    getContacts: function() {
       // ...
    },
    otherFunction: function() {
       //...
    }
  };

  contactApp.getContacts(); // <--- call your functions here for when the dom loads.
});


Merge


Did you add this because the video is funny or because you genuinely like the music?

If it is the former, then I vote to remove it, if it is the latter then I will have to live with it.


now I see. The [Bitnami VM](https://bitnami.com/stack/gitlab/changelog.txt) uses ruby 2.0.0-p645, which [didn't have the method yet](https://github.com/ruby/ruby/commit/394d5dfa9ba625c99a1e6a411f81b628bfebd60a), thus the search is broken. I guess, it's a bug for Bitnami


Nah, pinning to a commit is a good idea. I forgot to make a bunch of breaking changes (removing bytes re-exports and making `mut_bytes()` unsafe) before cutting 0.4, so I wanted to make those changes on master before I forgot again.


Delete debug information in the email content


Also, once you read the body in the middleware, you cannot read it again from the request.  What's the workaround there, to stuff the deserialized object into the Context?


I'm not sure, I'll follow up.


The red record button in the toolbar is a little distracting in my opinion.
And the location could also lead to misclicks.

I think "Record Stream" in the "Output" menu is enough for a feature that is not used regularly.
But it would be nice if this menu item has an icon.

What do you think?


This was the easiest way I could think to ensure the mark was a whole number.


This is previous branch I made and forgot to delete (post-icon) - you probably wanted to merge (post-icons) new one: https://github.com/gravitystorm/openstreetmap-carto/pull/1702


@Snkz
Does this mean the photo question works?


What do you mean, just "Today's" without " activities" at the end?


Is this  something which will work for other users like this?  I guess this looks for test_ve in ~/.conda -- but that would not match for other users?


im still working out a few kinks but soon yea


all other fields are staring with lowercase, I believe it should be id.


Wow I'm totally attending this.


OK. The comment led me to think that we were delaying closing the file_reader. I dislike how this is organized but we deal with this in another change.


Yep.


No it wont, so this doesnt resemble the final solution.


1718_setpoint_managers (mbadams5) - 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)


5051_fix_local_win_test (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1777 of 1777 tests passed)

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


[Deployment available online](https://s.brightspace.com/lib/ifrau/0.7.2/ifrau.js)


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2173 of 2215 tests passed)

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


The interval could be moved to a variable


http://www.cplusplus.com/doc/tutorial/polymorphism/ and http://stackoverflow.com/questions/4895294/c-virtual-keyword-for-functions-in-derived-classes-is-it-necessary are two things I was looking at.


Huge thanks for that!

You should add yourself to CONTRIBUTORS :-D


More direct link: http://ampl.com/products/solvers/all-solvers-for-ampl/


Thanks, though I had been hoping for information as comprehensive as in your previous post where you listed the partitions and explained which partition numbers are holding the user data (sdb5, sdb6 etc.). I couldn't find that information reading your link. Is there an equally easy scheme for GPT?


Please do not change 2 space indentation. Unless we change all files 4 space indented, I want to keep it map.js with two space indented.


warum zwei methoden für die gleiche sache?


@rgerkin thanks for this one -- one thing; could we do these changes in the dev branch first, rather than master directly?  I think folks are sticking to staging all new changes in the dev branch first these days on this project rather than having to merge master back into dev to take advantage of new fixes like this.


If the coordinator knows about other nodes that fabric doesn't know about, would that be a useful thing to include? (not necessarily as part of this patch).


The code that uses the "path" variant doesn't need this behavior if I recall correctly. Verify this.

There are a lot of changes already, so I would prefer not changing the behavior in the middle of these other changes.


GIT WIT Commits


`aspen/website.py` isn't where the algorithm module you want lives. It's in `aspen/algorithms/website.py`.


@nixxquality
If you keep the Github repository up, I would strongly recommend asking them to change your project from a fork to a standalone repository, and just referencing the original project in the readme file.  Github seems to interpret its "forked from X" as "fork created to make a pull request to X".  Not only would you be more likely to be contacted about shit like this, but the code would be searchable, and people's contributions would show up in their stats.


ReadVarsSmallDiffTest (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1775 of 1777 tests passed)

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


5051_fix_local_win_test (mjwitte) - 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)


warum umbedingt auf den heap? füg doch bei AIConstruction auch eine SetAI hinzu?


interesting question. maybe with a "not administratable by presto-admin" thing? might be helpful.


The current plan is to compute the ledger index using the signerListID (so there can be multiple SignerLists per account).  For the moment I'm jam-setting the signerListID as zero (since we have no user-visible signerListID yet).  Here's the code: https://github.com/scottschurr/rippled/commit/638bbddb7b75d3ea902d2385368cfaafb2d7440f#diff-29168e48c3327cd30f7ca91dd98bd74aR191.  If we support multiple SignerLists per account in the future, then that code will need to change so the signerListIndex is passed in to the computation.


Oops


Isn't the last / in the source the problem? cp -r should take care of copying build_sysroot/prefix into destdir/prefix just fine.


Isn't the case of mkdir with prefix?


Steve,
Did a crash course on Coffeescript on Monday (mainly running code in a browser based IDE for coffeescript).  Just got some basic stuff to work on Tuesday, like the redis and puma servers and then tried to figure out how to update the game waiting screen with ActionCable.  Thomas Ingram is running a hack afternoon/night at Common Grounds Coffee in Apex from roughly 5:30PM to 7:30PM or 8PM.  He briefly looked over the examples and gave me some hints.  To begin with I am not sure I have a complete understanding of how DHH defines connections and channels.  Quoting from DHH, "A single Action Cable server can handle multiple connection instances. It has one connection instance per websocket connection. A single user may well have multiple websockets open to your application if they use multiple browser tabs or devices. The client of a websocket connection is called the consumer."  To begin with it seems as if an ActionCable server serves up WebSocket connections to the requesting clients.  Based on what DHH says, it seems as if a connection = a WebSocket, as he states "It has one connection instance per websocket connection."  A single physical user can have multiple connections (WebSockets) if they have multiple browsers, devices, or browser tabs opened to one's Rails application.  In my case each user would typically have one connection to the application but there would be multiple WebSockets open on this one application as each player has one browser window open to the GoodCleanGame application.  Next is where I might get a bit confused.  DHH says the following, "Each consumer can in turn subscribe to multiple cable channels. Each channel encapsulates a logical unit of work, similar to what a controller does in a regular MVC setup. So you may have a ChatChannel and a AppearancesChannel. The consumer can be subscribed to either or to both. At the very least, a consumer should be subscribed to one channel."  So a consumer may be on a certain connection but that connection can have multiple channels, moreover DHH describes a channel as a unit of work, which initially confused me, but when he make a channel analogous to what a controller does in the Rails MVC architecture it started making more sense to me.  So I was thinking once a connection is established (and I do not know how that happens, maybe in the coffeescript which sends messages to the server?) then I can set up different channels which correspond to different game states.  My first thought was to set up a "WaitingPlayersChannel" where players go and wait for others to join.  However, an update is broadcast not by entering a comment on a comment form but by clicking the name of the player who has game slots available.  Now once all players are joined then maybe there needs to be another channel called GamePlayChannel in which the opening screen is setup and by which players take turns within the card game.  I am further confused on how the GameController might interact with these different channels.  It seem logical that when a new game is created then a connection is created (I would not know how to do this) and the first player subscribes to the WaitingPlayerChannel.  Likewise with the join action another connection should be established and then that player subscribed to the WaitingPlayersChannel.  When all players are joined then everyone needs to subscribe to the GamePlayChannel.  However, I do not know if such code should go in the GamesController or if the code should be kept separate under the channels directory.  Another long post but I am a bit lost right now.


no, no...


Consider the null coalescing approach to simplify the logic.
```javascript
var whatIWant = null || new ShinyObject(); // is a new shiny object
var whatIWant = undefined || "well defined"; // is "well defined"
var whatIWant = 0 || 42; // is 42
var whatIWant = "" || "a million bucks"; // is "a million bucks"
var whatIWant = "false" || "no way"; // is "false"
```
 http://stackoverflow.com/questions/476436/is-there-a-null-coalescing-operator-in-javascript


:+1:

There are enough params that as the app grows, this might be a candidate for a separate conf file, but seems good now


This commit caused the CommandLineParserTest to fail


The `virtual` keyword is not necessary on derived types, but I've used it historically to let viewers of the derived types know that the member is virtual.  However, with C++11's `override` keyword, that's a bit unnecessary to do these days, provided it is use (which is should always be to ensure the base member doesn't suddenly change its signature).


Ohhh right because it was never in a release.  Hmmm.


Alternatively, you can convert arr2 into a Set for quicker lookup than `indexOf()`


I like gold.


RadianceRefactor (rpg777) - 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 but the output from stan_glm is the same as the output from stanreg
except maybe with x,y, and/or model removed. I think cov.scaled is included
in the list returned by stan_glm.fit, but that gets fed into stanreg before
being returned by stan_glm.

On Wednesday, July 29, 2015, bgoodri <notifications@github.com> wrote:

> Isn't the input to posterior_predict the output from stan_glm? In that
> case, cov.scaled is an element of that but not of the stanfit element.
>
> On Wed, Jul 29, 2015 at 2:38 PM, Jonah Gabry <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > Oops, sorry I meant cov.scaled not cov.unscaled. What I was trying to say
> > was that if stanreg doesn't return cov.scaled then we can't recreate L
> > inside posterior_predict since the input to posterior predict is the
> output
> > from stanreg.
> >
> > On Wed, Jul 29, 2015 at 1:56 PM, bgoodri <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> > > I am not seeing where cov.unscaled is used. We have an element called
> > > cov.scaled as part of the list and there is also covmat from stanreg().
> > For
> > > posterior_predict, I guess you need to copy that part of stanreg() or
> > else
> > > we need to return stanmat also.
> > >
> > > On Wed, Jul 29, 2015 at 11:44 AM, Jonah Gabry <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > > wrote:
> > >
> > > > @bgoodri <https://github.com/bgoodri> If we take stanfit =
> > > stanfit$stanfit
> > > > for non-mcmc models then we will lose things like cov.unscaled,
> right?
> > If
> > > > we're going to have posterior_predict work for non-mcmc then won't we
> > > > need to recreate L?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12423867
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12427070
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428067
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428248>
> .
>


Yeah, I did a find usages of PluginContainer and these were all I could find in the API as method parameters.


develop (kbenne) - 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)


It worked! :rocket:
http://hacks.io/


Wow... I might need some help decideing if that is the right licence Im looking for, I tried to understand it all but just seems like it repeats over and over. Never have been a strong reader though... HELP!!!!!


Why does it matter?


The correct Twitter author is @jjaybrown98
Nice one @raphaelpor :+1:


I'm sorry that was so confusing, if you are ever looking at the configuration of something you know works once deployed you could consider [looking at the config](https://github.com/opentable/service-discovery/blob/master/server/config/ci-uswest2/config.properties#L1)

I don't think renaming this makes sense, the interpretation is left up to the actual implementation you use.  For example for most clients which use the "OT" strategy you are connecting to our API, not etcd.


Nothing is completel done :)


Why are you importing Tag?


@tomekwi I had to make this really dirty hack. Because sometimes the `_` is an object with a `_`. Do you know what's wrong???? :confused:  (only in the es2016 tests)


That's a huge improvement, really nice.


You're right! Originally I was iterating over elements in an array. When I changed the type to a string and started using find, I missed cleaning up the extra loop.


This commit is just about moving stuffs in others files, and passing the base address in MFuncs_Initialize().


Scrutinizer mag behaupten was es will - aber mit diesem commit hast Du Magic Numbers in deinen Code eingebaut, was meiner Meinung nach schlimmer als das bisschen Wiederholung ist...


@mfsny I think we should keep more than one partition in order to test also the cases where there is actual distributed stuff going on.


I can file a ticket


wall has to do the same thing (I think?) but
1) triggers off window resize event
2) debounces before any dom reading
https://github.com/Livefyre/streamhub-wall/blob/rc/src/wall-view.js#L55


Comment 1


@mfsny should be `cannotBeBooleanCount`


Okay.


I’ll come back to it tomorrow. I’m already lagging behind with some of my stuff.


Numeric


Instead of doing that try... finally, can you use this new mecanism with a decorator:

```
@theano.configparser.change_flags(**{'vm.lazy': True})
def test...()
```


where was it hosted before?


@elia - Several of these are already covered in lib/opal/rspec/rspec.rb as stubs. I had kind of meant for this file to be auto generated as well. The next time someone runs the task, all of these will go way and then the person will wonder why all of this broke, right?


ack


When a series have flags SER_BARE or SER_EXT, it's not really a block. We should change the macro IS_BLOCK_SERIES to exclude these series, and remove the checks here


I see. We may have to return cov.scaled from stanreg or take the upper left
block of covmat.

On Wed, Jul 29, 2015 at 3:10 PM, Jonah Gabry <notifications@github.com>
wrote:

> Yes but the output from stan_glm is the same as the output from stanreg
> except maybe with x,y, and/or model removed. I think cov.scaled is included
> in the list returned by stan_glm.fit, but that gets fed into stanreg before
> being returned by stan_glm.
>
> On Wednesday, July 29, 2015, bgoodri <notifications@github.com> wrote:
>
> > Isn't the input to posterior_predict the output from stan_glm? In that
> > case, cov.scaled is an element of that but not of the stanfit element.
> >
> > On Wed, Jul 29, 2015 at 2:38 PM, Jonah Gabry <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> > > Oops, sorry I meant cov.scaled not cov.unscaled. What I was trying to
> say
> > > was that if stanreg doesn't return cov.scaled then we can't recreate L
> > > inside posterior_predict since the input to posterior predict is the
> > output
> > > from stanreg.
> > >
> > > On Wed, Jul 29, 2015 at 1:56 PM, bgoodri <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> > >
> > > > I am not seeing where cov.unscaled is used. We have an element called
> > > > cov.scaled as part of the list and there is also covmat from
> stanreg().
> > > For
> > > > posterior_predict, I guess you need to copy that part of stanreg() or
> > > else
> > > > we need to return stanmat also.
> > > >
> > > > On Wed, Jul 29, 2015 at 11:44 AM, Jonah Gabry <
> > notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > > > wrote:
> > > >
> > > > > @bgoodri <https://github.com/bgoodri> If we take stanfit =
> > > > stanfit$stanfit
> > > > > for non-mcmc models then we will lose things like cov.unscaled,
> > right?
> > > If
> > > > > we're going to have posterior_predict work for non-mcmc then won't
> we
> > > > > need to recreate L?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12423867
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12427070
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428067
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428248
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428781>
> .
>


WinXP is past the point where it is reasonable to expect software to continue to support it. If you insist on running an old, unsupported OS then stick with an old version of libtorrent.


What is the purpose of this change?  As far as I know, `@stdout` is not a valid Doxygen command, but `@return` is


fixes #2


You need to make your commit messages more accurate. I don't see any big query tests added here, just old tests deleted. `git diff` your files to see what changes you are actually staging.


@tchx84, we change the env variables at sugar restart, then, if the user do more than one change in the UI, the file and the env variables will have different values. The file is saved on a timeout
https://github.com/godiard/sugar/blob/master/extensions/cpsection/language/view.py#L366 then race conditions can happen if the control is done outside of the _write_i18n() method.
Finally, read the file is not dangerous, write the file is, then if we can avoid write, better.


Shouldn't "inputs" be capitalized?


fixes #3


Is there any chance that we could run into an `https` url? If that's the case then this regex wouldn't match.


Danke für den Hinweis :) Scrutinizer bringt mich ehrlich gesagt manchmal auch echt auf die Palme, weil ich oft nicht verstehe, was das Tool jetzt schon wieder hat. Aber ja, ich versuche meistens eher Scrutinizer zu "befriedigen", weswegen ich dann sowas produziere,... Der Dienst ist wohl noch nicht so intelligent wie ein Mensch und übersieht daher einiges, und ich sollte mehr meinen Verstand anknipsen, als einfach zu versuchen, die Beschwerden von Scrutinizer loszuwerden. :)


@VideahGams But someone has to hold you hand in the new future internet, where no one can hurt your feelings.


purpose: add a `revision_override` parameter to the job configuration. format:

```json
{
  ...
  "revision_override": {
    "cassandra-2.2": ["host1", "host3"]
  }
  ...
}
```

Hosts not named in the keys will use the version specified as `revision`, as before; this dict specifies overrides to that.


It'd be nice to fold our recommendations here into the style/best-practices page. I know I forget about `override`. And in past lives, I've seen inconsistent use of `virtual` in derived classes (though I've always favored `virtual` in the derived class, for the reason Peter gave, as a bit of hint/context to readers).


It was on my Digital Ocean droplet