I know what it's been too long since i commited this. However this codes still use in multiwii and baseflight. Can I get a little help? please :)


using the `:as foo` with require means you have to specify the namespace for these methods, i.e. `set/difference` and `set/union`


Note dist/ files are just the grunt generated equivalent of the REAL files in src/. Only the latter is annotated.


>I concur with some people here: you should probably go over to gitlab or something and replace this repository with a message "We moved to GitLab, because github is shit (or an explanation)"
>
>Don't encourage them by complying with their request when it's so easy to just leave them. You staying here brings your users here, making them more popular.

Moving properly would require more than 24 hours because of limitations in my code; the auto-updater depends on using GitHub releases and so any user that wouldn't launch in the 24 (- however many hours it would take for me to move to another website and whip up a new update system) hour window would be locked out of future updates, barring word of mouth from users who did get the news.

This is the only move I can do in such a short period of time.


Added these defaults back in, which were mistakenly deleted.


ui.grid.expandable is in alpha? Any way to see milestones and progress on that?


Please note that the above changes contains the removal of the Card Type dropdown.  Because the card type is provided in the call back from the hosted fields, it's pointless to have a drop down.  We've added an anonymous function (line 1215 of master.js above) that sets the creditCardType on the cart when the credit card number is successfully entered.

We've seen a lot of carts that already do this and provide a nice gui of card icons, highlighting which card was used.  To implement this, you would only need to add the gui code within that anonymous function as well.


@krainboltgreene https://sr.ht/1191.png


Shoo.


@rodrigopavezi actually common-app-components repo is set to use `"` for quotes. Inconsistent I know...

Will create constants names. Thanks!


creando las vista de ejemplo de primer proyecto


I understand canProxyAs expects a user name? but input.getName will return the name of a file. Am I understanding something wrong?


You maintain all of the content in `top_level_ranking` and spend the time sorting it, but you only ever reference the top 2 items in it...

Instead, you can maintain a record of the `top_level` item and its `sum_of_child_values` as you iterate over matches.


Nice touch, thank you!


develop (kbenne) - 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-87.86%25-red.svg)


This entire if-statement will not be required if you're maintaining the top level category as I mentioned in the comment above.


Verstehe. Dann wäre ein PropertyConcern das richtige. @Mandatory und @Immutable sind eigentlich nur vorgefertigte build-in concerns. Du kannst selbst einen schreiben und damit kompliziertere Bedingungen prüfen. Ich hab das schonmal mit url und urls beim TileWMSSource gemacht.


@nixxquality If Github is causing you hassles, and you aren't getting paid for this, why not just quit? Let Github disable the account and walk away.

While a lot of people appreciate what you do, you don't owe anyone a damn thing. Let everyone execute their rights and tragedy of the commons be damned:

* People have the right to complain about your language
* GitHub has the right to disable your account
* **You** have the right to walk away and stop supporting a project.


You won't need this - refer to comments around where you use it.


Hmmm does the missing trailing `();` actually work for IIFE's??? ... never tried that.


There is a usage caveat: the plugin only adds a link to the archived link on archive.org _(yup, I know it's the exact intended usage of the plugin ^^)_

Consequently, the following use case is broken:
- a link to `http://websi.te` is archived
- no one really cares about that particuliar site, thus it is never archived on archive.org
- the site goes down
- some time later, the Shaarli user attempts to retrieve the archive... which doesn't exist

Is it possible for a plugin to add `postAdd`, `postEdit`, `postDelete`... hooks that would be called when a link is modified?

In this case, this would mean:
- request the page from the archive.org cache
    - it exists: that's a good start, maybe we can ask for it to be refreshed?
    - it does not: ask for it to be added to the archive

This could be done through the [JSON API](https://archive.org/help/json.php) or the [Advanced Search](https://archive.org/advancedsearch.php#raw) form


@krainboltgreene Shoo.
I would like a reason why people should 'Shoo'. Otherwise we can both say it and both uses are equally valid.


@LegoStormtroopr Because that would be selfish of me and my users come first.


Why?

`ourKeyID = 0`, means we have never generated a DH keypair which means we cant generate session keys.


This is cool!  What are x and y axis?  Some mix of scores from Splatoon?


90946736_SHRgt1_4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1779 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'm inclined to treat `undef` as false since it is falsey in Perl, but I'm open to counter arguments.


1718_setpoint_managers (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2266 of 2288 tests passed)

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


@nixxquality who put the "retard" text in the code base? Was it a big deal to remove it?  Was it actually a different fork responsible?

—
Sent from Mailbox

On Wed, Jul 29, 2015 at 2:15 PM, nixxquality <notifications@github.com>
wrote:

> @LegoStormtroopr Because that would be selfish of me and my users come first.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12431287


Please just don't copy this stuff from Golang otr. The fragmentation file already takes care of most of this stuff.


That's true. :)


Mm, will the rgb defaults trimm to 255 (max) too? If so, cool, we're done, +1.


um.. wouldn't easier just ```write!(f, "WindowsError({})", self.errno()))``` ?


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


90946736_SHRgt1_4567 (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)


WARNING: all is not as rosy as that commit message suggests. In some un-pushed work, I am continuing to figure out best way to handle the vignette / token issue.


And thus, after almost a month of `extendedType` failing (and it probably would have failed longer if the tests hadn’t failed even sooner due to various compiler bugs), we finally have green tests again…

…on the JVM. Runtime error on JS. ¯\\\_(ツ)\_/¯


Put this into a translation variable.


As posted on the Stripe/Ruby repo:

The trick to verify a bank account:

1
:headers => { 'Content-Type' => 'application/x-www-form-urlencoded', 'Accept' => 'application/json'}

Because the parameters Stripe wants are invalid as JSON.

And the Body like this

 :body => "amounts[]=14&amounts[]=26",


@thezoggy we just found out that. we tried to fix with those PR


I think the firehose of information just blew up your comment :-)

There is a lot of stuff in your post. Can you try to pare it down to the main things you're having trouble with, preferably in list format? I think I'll be able to give you better answers if I can address one thing at a time.


Isn't # already part of the translation string?


Can't you make these names match the ones at MENUITEM above, or the inverse?


My two cents.

This sure would be a very different conversation if it was 'WebM for Niggers' or 'WebM for Kykes'. Those words are more accepted as being derogatory in society. I think a lot of people use 'retard' as a colloquial, playful word, but really, I think it isn't acceptable; a community, however minority, is [voicing it's opinion](https://specialolympicsblog.wordpress.com/2012/10/23/an-open-letter-to-ann-coulter/) more openly about this. I think they are right.

We must, I believe, be wary and think critically about any form of censorship. In this case, I believe the censorship is legitimate because the change does not affect any functionality of the repository or suppress opinions (blowing up the forks was quite silly though). Each case of censorship should be exposed to this amount exposure and community thought. I think we are doing well in that regard.

Github are enforcing their TOS and I believe this change is for the positive. People implying that Github are restricting their freedom, I think, will need to try and empathise more with the community of people affected by this word.


Yep, it's guaranteed by sol_rgb_set_max(&color, 255)


This fixes the issue for now. Are you working on a better fix, or should I work on it?


Should the previously loaded providers (`mProviders`) also be cleaned here?


- Remove from criteria
- Don't need to have them to enter stage 2. But do need to have sign off to enter stage 3.


KSM is working in caf ?


From experience, the actual need here will probably arise only when pushing toward i18n support.
Most of our sentences are badly formatted in regards to that area, so this can be redone in a more global effort later, IMHO.


Emphasize that the committee chooses reviewers, not the champion.


@nixxquality Thats a noble attitude of you, but no one would blame you if you pulled an Xkit-guy. He also got pilloried by Tumblr and stopped building his open-source project... which was a tool to make Tumblr better.

@jgrigonis The original repo was "WebM for Retards", @nixxquality just forked it and never changed it - because there was no reason to.


Missing if statement body. Please add this in and commit and mention me on github with `@monica`


Isn't `item_tree[top_level]` already sorted on line 374?


Very good!


r=jedmccaleb


@lwn

> People implying that Github are restricting their freedom, I think, will need to try and empathise more with the community of people affected by this word.

But where it ends? We will need to rewrite every single fucking project to make sure, that it doesnt bring butthurt to some retard. Nope. That isn't the right way to go.


This code works, but just a reminder that you could also use `is_a?` here.


No need to ceil, unless `attack_power` is a decimal.


It would be desirable if the `user` parameter would give the user-configurable name (upper left corner in Simquadrat dashboard on the tab which as default shows "Bob's mobile") instead of the user's full name ("Firstname Lastname").


Oh! didn't know you could do that. Yes, actually it did work, now i see
what you're talking about. Thanks Mark!

On Wed, Jul 29, 2015 at 4:48 PM, dothework <notifications@github.com> wrote:

> Yes. Exactly.
> I thought I made my comment on the exact line of code we are referring to.
> GitHub allows for that. It didn't work?
>
> Sent from Outlook
>
>
>
>
> On Wed, Jul 29, 2015 at 1:38 PM -0700, "Diego-F-Aguirre" <
> notifications@github.com> wrote:
>
>
>
>
>
>
>
>
>
>
> Hey Mark,
>
>
>
> Looks like maybe you forgot to paste the line that I should comment out.
>
> Did you perhaps mean the following line:
>
>
>
> [tableView deleteRowsAtIndexPaths:@[indexPath] withRowAnimation:
>
> UITableViewRowAnimationAutomatic];}
>
>
>
> Since [realm deleteObject:self.array[indexPath.row]]; is already telling
>
> the cell to delete itself?
>
>
>
> On Wed, Jul 29, 2015 at 1:06 PM, dothework <notifications@github.com>
> wrote:
>
>
>
> > @Diego-F-Aguirre <https://github.com/Diego-F-Aguirre> Comment out this
>
> > line and it works.
>
> >
>
> > —
>
> > Reply to this email directly or view it on GitHub
>
> > <
> https://github.com/Diego-F-Aguirre/BlocNotes/commit/74275a2cffdbdc2dea984d505dfc5a8ba305f482#commitcomment-12425885
> >
>
> > .
>
> >
>
>
>
>
>
>
>
> --
>
> :::::::::::::::::::::::::
>
> Diego F. Aguirre
>
> 973-885-0724
>
>
>
> http://www.linkedin.com/in/diegoaguirre3d
>
> http://diegoa3d.com/
>
>
>
>
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Diego-F-Aguirre/BlocNotes/commit/74275a2cffdbdc2dea984d505dfc5a8ba305f482#commitcomment-12430724>
> .
>

--
:::::::::::::::::::::::::
Diego F. Aguirre
973-885-0724

http://www.linkedin.com/in/diegoaguirre3d
http://diegoa3d.com/


I suppose anything we do to `undef` presumes that we know what the API user wants it to mean.  The dev site has "do not include this field when state is unknown" for the `was_authorized` boolean, so that can be followed in the perl API instead of a special value for a key that means it isn't sent to the server.  False is good here.


Good! :)


1. return писать не обязательно
2. У тебя метод проверяет слова(в названии), а на деле еще и обновляет карточку, такого быть не должно.


Again, you maintain all of `item_tree` but you only ever look up content of `item_tree[top_leve]`. Why not only maintain the details relevant to `top_level` as you iterate over `matches?`


COMMITTER :)


Thanks for the reminder, I can make that update tomorrow.

For some additional context though: we share this gem between the CLI and server-side analysis code, so sometimes we'll make changes that don't impact the CLI and therefore we won't update the homebrew formula right away. Users can always do a `docker pull codeclimate/codeclimate` to get the latest image which tracks master a little more closely -- you can do this even if you've installed via homebrew.


Great! :)


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


merging graydon/stellar-core/bucket-apply-end-extra-read = ba59d586 into auto


graydon/stellar-core/bucket-apply-end-extra-read = ba59d586 merged ok, testing candidate = 1c1cc292


> IMO the current behaviour is fine as it is. The only problem is that current, researches are done very quickly, so indeed you might unlock a couple researches while designing your new tree, and that can be annoying if you would have prefered to put research points somewhere else. But for the time being it's good enough.

That's a true balancing issue, indeed. Maybe we could tweak that further after a few lans? ;)


Проще можно сделать, посмотри в документации/гайдах/гугле, без count и first и offset


This code works, but I think the readability could be improved. The nested if statement throws me off. Here's a suggestion:

```ruby
def attack! (enemy)
  if !self.dead? && !enemy.dead?
    enemy.damage(self.attack_power)
  else
    false
  end
end
```

Also do we really want this method to return false if the attack fails? If it is ok to have this method return nil (which is a falsey value anyway), we could shorten this to:

```ruby
def attack! (enemy)
  if !self.dead? && !enemy.dead?
    enemy.damage(self.attack_power)
  end
end
```


@bob-rove after reading about this I think that is not thread safe. Going back and forth and modifying instace variables doesn't look safe to me. I would need to do some proper tests to see if can verify it. Thanks for pointing this out.


Cool catch. Sorry, the code was mine and I didn't help a lot catching that one.


@pbrisbin awesome, thanks :+1:


Add newline/formatting after comma?


You could use the new expect syntax.

```ruby
expect(enemy).to receive(:damage).with(5)
```

I know this syntax is not in the RSpec cheat sheet, but you can find the docs here: https://www.relishapp.com/rspec/rspec-mocks/docs (see Message Expectations section)


It is :p because it's enable by default apparently


Please fix indentation.


The change looks simple enough and the comment makes it clear, so it fine to me.


```ruby
expect(@unit).to receive(:health_points).and_return(0)
```


Indeed, this is suboptimal. There is also a larger issue here at play: https://github.com/opnsense/core/issues/265

Long story short, we're going to have to disable translations of console parts until we have a proper unicode console, thereby circumventing the issue you describe here. I want to not burden the translators with this so this is something we'll do in the code itself. Sounds good?


I'd rather work more on the UI so if you can take care of it that would be great. Also I have no idea how to solve this yet, I haven't thought about it much.

Anyway do as you feel like. There might be other things to do with higher priority. Since there is a fix, it can wait.


Great job overall. Really clean code!


No pb. We all help at catching each other's bugs ^^


Let's avoid unclear, unsafe indexing like this, especially since you're only using the first two items of `possible_sub_levels`


LGTM. I hit the same bug today while testing audio only stream.


No idea what I meant by 75, sorry.


put back
this was shotgun debugging

you should git dt before CR


¡No fregués!


I don't understand your comment.  Can you further explain?

I think what you're saying is that even though that test runs successfully on Windows (msysgit 1.9.5) and Linux (git 1.7.1, git 1.7.10, git 1.8.1, git 2.1, and git 2.5.0), it should have failed because it is doing the wrong thing when it succeeds.

Have I understood correctly?


Oh I think I meant the 7% but now we're at 10% and its upstreamed so all is fine.


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


I think this change is slightly redundant with https://github.com/leapcode/soledad/commit/fa42397e13c5d7a70ceb961153d39e10a0d4c8f9
(and the new pkg/requirements-leap), although it might be worthy to have this redundancy. Didn't that work for you?


I think we should box this with some `" quotes "`


got to decouple this from libgdx. perhaps package a copy for now.


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


Typo: section 11.1 "to be execute directly"


If USB_OTG_DRIVER isn't variable only 4 (of 8) DIEPINT registers will be supported


Note: this is the wrong commit message.


EPx_WMAXPACKETSIZE depends on the usb descriptors


nitpick: if we're going alphabetically, 'o' < 'p' so "org.joda.time.DateTime would be before "play.api.Configuration"


@krainboltgreene Poor guy.  Sorry your problems won't just go away because you want them too.  No wonder you're so desperate to censor things that might offend someone.


Hey, the case here is that we wanted to have this automatically instead of leaving to the user to install the packages, so we kinda followed this [link](https://caremad.io/2013/07/setup-vs-requirement/) to always have the latest develop updated from git (or from the common folder in the case of common)


DO NOT MERGE!  THIS DESTROYS NEW CODE!


Drawing names bigger for larger cells is cool. But this way the name sometimes is obscured by the mass.


:+1: I like this rewording


@Skadabr Wondering if this should be an error instead of a warning.


This is safe now because we've split out all webhooks to not pause now ya? Before we evaluated whether webhooks used @step first right?


Sometimes the names are too long and do not fit into the box completely. Maybe we should draw them right-aligned and without a box?


After installing a new OpenGrok version I see the following columns headers but nothing gets displayed under it, is there a new configuration I am missing supporting this new table?

Project	   SCM type	       Parent (branch)


wasn't there a caveat to this somewhere? like when you need more than 3 members of that package, or it's long enough to be unwieldy (keeping in mind needing that many things may indicate too much coupling anyway)


fast-forwarding master to auto = 1c1cc292


No I definitely did not delete any lines, only changed words under the tr
tabs. Maybe what happened was, I already started editing the lines and when
I tried to sync this morning, it said there had been changes to the file
(you probably added more lines in the meantime). So maybe your progress was
lost when I synced the file that I was working on. Does that make sense?
Which files exactly are missing?

On Wed, Jul 29, 2015 at 11:27 PM, L Daniel Swakman <notifications@github.com
> wrote:

> @seratolgay <https://github.com/seratolgay> did you delete all these
> lines? There are all kind of translation strings that are not found now.
> Please be careful when you're editing these files...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Muhit-co/api/commit/fe868267278f3d59f268877233f0224447edb75e#commitcomment-12430331>
> .
>


first test runs in " No language dictionaries for the language :  "de""


Seems good and much more complex than I imagined hehe


that's why i created the example file. this way people have to explicitly make an action (copy the secrets.json-example to secrets.json) to make authentication work. without it, the app still works but authentication doesnt.


This code can be moved to after_save method on user model. will be cleaner that way...


1709-ComponentSetpointScheme (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)


wow i just noticed i didn't have the latest version. Do you by any chance
happen to know how to update my mraa library?
Thank you!

On Wed, Jul 29, 2015 at 12:29 PM, Mike H <notifications@github.com> wrote:

> http://iotdk.intel.com/docs/master/mraa/classmraa_1_1_i2c.html
>
> It's in the list of 7.3.0 and compiles fine for me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sparkfun/SparkFun_ADC_Block_for_Edison_CPP_Library/commit/2e985245d1d5daeae9b4ac737aa3c59644e831b5#commitcomment-12425035>
> .
>


add exception on constructor if user attempts to set field with null, and get rid of the null checks here. Whole class is geared towards field existing.


Retard is French for late. How can github be so Anglocentric? *smh*


This change is great. Thanks.


This breaks activating connector nodes. Should the essential fields be `type`-specific?


Maybe it is not a problem, but multiple booleans doesn't make sense to me. Isn't it either true or false?


the reason I asked for this is that I was not sure whether the output would contain bogus empty lines, and they might not show up in Hive. But the way you read it here, you would also not detect empty lines, because of the omitEmptyStrings()...


It is possible it is something inside my configuration, but my git fails with:

fatal: You are pushing to remote '/tmp/1438195900368-0', which is not the upstream of
your current branch 'master', without telling me what to push
to update which remote branch.

which I traced back to the specified commit.

I am surprised it worked on 2.5.0 given that is what I am using. The fix should have made it into 1.8.x line, not sure about MSYS git..

Though, it may be result of me pushing "push.default = upstream" in my git configuration....

We should be explicit where possible due to configuration settings that could be set on the CLI implementation which change behaviors of the non-explicit modes.

I am running again without my configuration file, to see if somehow changing that setting is what broke it.

At any rate, I'd rather the client plugin not depend on the stored configuration of the system running, and do everything possible using the plumbing commands and explicit settings which don't rely on configuration values as much. This would make a more robust experience for everyone.

It's highly possible my local configuration file was what has caused the trouble. This is true for how i discovered the issue with git describe.


looks great!


I think it is better to get the input stream, create a buffered reader and iterate over readLine(). That would give you all lines, including empty ones.


Nice job here. You could also incremented the great and greatest variable respectively as well as the totalVote variable.

And use the following formula to increment each bar. great/totalVote and greatest/totalVote.


I'd write this as:
if isinstance(data,(np.recarray,np.ndarray)):


Although the name of the method is rather clear,
I'd add to the Documentation that a ndarray is returned.


Since the thrift files are not regenerating for included sources, what happens if there is a version mismatch?

We have a project where we ship thrift-only JARs and expect the consumer to compile them, as they may be using a different version of Scrooge than the producer of the library. As of Scrooge 3.16.3 this worked, with 3.20.0 it is no longer compiling the files.


adding additional comments after speaking with customer


This is not showing up on my end. Do you have it working on your end?


Please remove debug code.


All bugfixes require a unit test.


This broke the build:

System/Environment.cs(656,4): error CS0152: The label `case 5:' already occurs in this switch statement
System/Environment.cs(645,4): (Location of the symbol related to previous error)


It still differs from the original version


Love the format and the way you created this function.


After re-running with HashMap, I think we can safley use it - the wierd performance-drop I was seeing was caused by Field missing equals/hashcode, filling the map with millions of entries in the benchmark. The benefits of ObjectMap is not as astronmical as advertised :wink:


Great way to make this work. Really simple and straight forward


1) Yeah that makes sense

2) I figured you got the UUID's through logins. But it still doesn't solve the massive CPU destroyer UUID support will add. I get 100% CPU during testing while it's inserting and it takes around a minute to insert 10,000 entries. I also have a pretty big player_data table so that's also slowing it down. The only way around this issue would be to implement hash checks, rather then equals/equalsIgnoreCase. Strings in JAVA are handled badly (As in just about every other launguage), so a hash based lookup would greatly optimize everything using player-data. Especially since strings are a "static object", so they retain the same hashcode rather then constantly re-generating it.

3) Hmm i see.. The multi-table query support on MySQL may provide a better outcome and would allow for me to move other things to their own tables (Text, large strings, ect). I really think i'm just going to have to re-code the entire database setup to be honest.. I may attempt this next month or when i have time.

4) Glad we agree on this. And if for whatever reason the server-software no longer supports ID's, we can just make an integrated list (To use with both the web interface, and the plugin). This will also allow for us to retrieve "cleaner" looking names and such for the UI.

Anyways, hopefully i'll be able to integrate some of your work soon. I REAALLY want to re-code all this database stuff and really take control of MySQL's cool features.


Lovely piece of code here! Fast and efficient.


Intellij generated code :blush:


The other translation says "Spitzname." Is that incorrect?


Let's keep the code for this function call on one line.


I dont't think this would help. The problem occurs, because after editing an offer, the offer gets deleted and re-created by the API. So we get a problem here in the App history, because we get an offer, which doesn't exist anymore. So this workaround just avoids an App crash and will after that show an error, that the item does not exist (404)


The other translations are "Pseudonyme." Is that incorrect?


Looks good.

I should probably also write a package.json file, which allows npm to install all the dependencies in one go.


90946736_SHRgt1_4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (745 of 745 tests passed)

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


I like where this is going.

One thought that occurred to me is that it might be interesting to make the generated methods instance methods.  Then your JsonHelper interface could require that the parser classes have the generated methods, and you could do:

```
Class <? extends JsonHelper<ResponseType>> cls;

JsonHelper<ResponseType> parser = cls.newInstance();
ResponseType response = parser.XYZ();
```

All strongly typed!

This is a backwards-incompatible change, however. :(


These are good tests to have.

Have you given thought to adopting units on these arrays, i.e.
to use Astropy Quantities?  I suppose the performance will take
a hit and you are unlikely to you use anything other than the
units you have specified here, but this can be another powerful
check on the coding..


I would like to see this TODO item addressed before merging the PR.


thanks


Sweet! You got a circle going on here!


Since converters and filters are required to implement ReporterInterface, they should register the messages. I this way reporters and filters and converters can work together to produce any type of message.
Set the reporter in the filter or converter before it runs, it will register whatever it wishes


Love that you provided various markers in your map.


Userspace code I had to add, in it's 'as is' state..

###  FIFO empty poll
```C
static inline
bool
usb_bulk_data_tx_ready() {
	return (REBASE(OTG_DIEPCTL(EP__BULK_DATA_TX&0x7f)) & OTG_HS_DIEPCTL0_USBAEP) && (!(((*(volatile uint32_t *)(((0x910 + 0x20*(EP__BULK_DATA_TX&0x7f))) + ((USB_OTG_HS_BASE))))) & OTG_FS_DIEPTSIZx_PKTMSK));
}

// used like :
if(usb_bulk_data_tx_ready() ) {
    usbd_ep_write_packet(...);
}
```

### some definitions were 'exported'
```C
// currently I use this ugly hack in my code
#include "libopencm3/stm32/otg_hs.h"
#define REBASE(x)        MMIO32((x) + (USB_OTG_HS_BASE))
#define OTG_GOTGINT_SEDET 0x0004
#define OTG_HS_GRSTCTL_TXFNUM_LSHIFT 6
volatile bool usb_cable_connected = 0;
#define OTG_FS_DIEPTSIZx_PKTMSK (0x3FF<<19)
```
### Enabled some interrupts for usb in my setup
```C
void usb_setup(void) {
	usbd_dev_local = usbd_init(&otghs_usb_driver, &dev, configs, usb_strings, 4, usbd_control_buffer, sizeof(usbd_control_buffer));
	/* fixme  enable additional interrupts */
	OTG_HS_GINTMSK |= OTG_HS_GINTMSK_OTGINT | OTG_HS_GINTMSK_SRQIM;

	usbd_register_set_config_callback(usbd_dev_local, usb_set_config);
	nvic_enable_irq(NVIC_OTG_HS_IRQ);
}
```
### Extensions to isr
```C
/**
 * Extended OTG hs isr that handles cable connect/disconnect..
 */
void otg_hs_isr(void) {
	uint32_t intsts = REBASE(OTG_GINTSTS);
	/* device connected */
	if (intsts & OTG_HS_GINTSTS_SRQINT) {
		usb_cable_connected = 1;
		REBASE(OTG_GINTSTS) = OTG_HS_GINTSTS_SRQINT;
	}
	/* device disconnected */
	if (intsts & OTG_HS_GINTSTS_OTGINT) {
		uint32_t gotgints = REBASE(OTG_GOTGIN);
		if (gotgints & OTG_GOTGINT_SEDET) {
			usb_cable_connected = false;
			usb_send_ready = false;
			usb_config_set = 0;
			usb_cable_disconnected();
			REBASE(OTG_GOTGIN) |= gotgints;
			return;
		}
	}

	usbd_poll(usbd_dev_local);
}
```

### added fifo flush handling
```C
/**
 * Cable disconnect (better name would probably be flush_usb_fifo or so..)
 * I hacked this function together 5 months ago.
 * Stopped working on it when USB stopped crashing.
 */
#define FLUSH_TIMEOUT 200000
volatile bool fail=false;
void usb_cable_disconnected(void) {

	volatile int timeout;

	/*
	 * disable the endpoint if it was enabled!
	 * this has to be done somewhere for all endpoints
	 * (eg. before set_config)
	 */
	/* IN-endpoints */
	//usbd_disconnect(&dev,true);
	//usb_setup();

	/*
	if (!(REBASE(OTG_GINTSTS) & OTG_HS_GINTSTS_GINAKEFF)) {
		REBASE(OTG_DCTL) |= OTG_HS_DCTL_SGINAK;
		while (!(REBASE(OTG_GINTSTS) & OTG_HS_GINTSTS_GINAKEFF));
	}
	*/

	if (!(REBASE(OTG_GINTSTS) & OTG_HS_GINTSTS_GONAKEFF)) {

		REBASE(OTG_DCTL) |= OTG_HS_DCTL_SGONAK;

		// stolen from nuttx stm32_otgfsdev.c
		#if 0
		  /* Wait for the GONAKEFF interrupt that indicates that the OUT NAK
		   * mode is in effect.  When the interrupt handler pops the OUTNAK word
		   * from the RxFIFO, the core sets the GONAKEFF interrupt.
		   */

		while (!(REBASE(OTG_GINTSTS) & OTG_HS_GINTSTS_GONAKEFF));

		#else
		  /* Since we are in the interrupt handler, we cannot wait inline for the
		   * GONAKEFF because it cannot occur until service th RXFLVL global interrupt
		   * and pop the OUTNAK word from the RxFIFO.
		   *
		   * Perhaps it is sufficient to wait for Global OUT NAK status to be reported
		   * in OTGFS DCTL register?
		   */

		  while ((REBASE(OTG_DCTL) & OTG_HS_DCTL_GONSTS) == 0);
		#endif

	}

	for (uint32_t addr=0; addr<8; addr++) {
		if (REBASE(OTG_DIEPCTL(addr)) & OTG_HS_DIEPCTL0_USBAEP) {

			/* disable out endpoints */

			//REBASE(OTG_DIEPCTL(addr)) |= OTG_HS_DIEPCTL0_SNAK;
			//while (!(REBASE(OTG_DIEPINT(addr)) & OTG_HS_DIEPINTX_INEPNE));

			REBASE(OTG_DIEPCTL(addr)) =  (REBASE(OTG_DIEPCTL(addr)) & ~OTG_HS_DIEPCTL0_USBAEP) | OTG_HS_DIEPCTL0_EPDIS | OTG_HS_DIEPCTL0_SNAK;
			while (!(REBASE(OTG_DIEPINT(addr)) & OTG_HS_DIEPINTX_EPDISD));

			// Don't care?
			// The application must read the OTG_FS_DIEPTSIZx register for the periodic IN EP, to
			// calculate how much data on the endpoint were transmitted on the USB.
			(void)REBASE(OTG_DIEPTSIZ(addr));

			/* flush out endpoints
			 * - is done in the end of this function..
			 * - removing this fixed a problem, where the device only
			 *   worked every second time it was connected for some reason
			 *   TODO: maybe because OTG_DIEPTSIZ(addr) shouldn't be 0???
			 */
			//REBASE(OTG_GRSTCTL) |= ((addr<<OTG_HS_GRSTCTL_TXFNUM_LSHIFT)&OTG_HS_GRSTCTL_TXFNUM_MASK) | OTG_HS_GRSTCTL_TXFFLSH;
			//timeout = FLUSH_TIMEOUT;
			//while ((REBASE(OTG_GRSTCTL) & OTG_HS_GRSTCTL_TXFFLSH) && timeout--);

			// CNAK?
		}
		//REBASE(OTG_DIEPTSIZ(addr)) = 0;

		/* disable in endpoints */
		if (REBASE(OTG_DOEPCTL(addr)) & OTG_HS_DOEPCTL0_USBAEP) {
			REBASE(OTG_DOEPCTL(addr)) =  (REBASE(OTG_DOEPCTL(addr)) & ~OTG_HS_DOEPCTL0_USBAEP) | OTG_HS_DOEPCTL0_EPDIS | OTG_HS_DOEPCTL0_SNAK;
			// this doesn't work :(
			//while (!(REBASE(OTG_DOEPINT(addr)) & OTG_HS_DOEPINTX_EPDISD));
			timeout = FLUSH_TIMEOUT; while (timeout--); // TODO use flag checking or so..
		}
	}

	/* flush IN/OUT-endpoints */
	REBASE(OTG_GRSTCTL) = (OTG_HS_GRSTCTL_RXFFLSH | (OTG_HS_GRSTCTL_TXFFLSH | (0b10000<<OTG_HS_GRSTCTL_TXFNUM_LSHIFT)));
	timeout = FLUSH_TIMEOUT;
	while ((REBASE(OTG_GRSTCTL)&(OTG_HS_GRSTCTL_RXFFLSH|OTG_HS_GRSTCTL_TXFFLSH)) && timeout--);

	REBASE(OTG_DCTL) |= OTG_HS_DCTL_CGONAK;

	/* wait a bit.. */
	timeout = 2000; while (timeout--);

	REBASE(OTG_DAINT) = 0xffffffff; // clear interrupts
}
```


I wasn't saying it was bad. I just didn't understand it.


Types of people in this thread:

1. \*grabs popcorn\* \*enables autoscroll\*
2. anti-github
3. funny GIF poster
4. pro-github
5. the hell is this place


lgtm


RadianceRefactor (macumber) - i386-Windows-7-VisualStudio-12: OK (2189 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.83%25-yellow.svg)


I do like it, but when there are only 2 I don't mind writing it out.

The main reason I changed this one is because after s/var/const, the spacing was off. So while I was fixing spacing I decided to just add the word.


Agreed, in hindsight we should've communicated this better and bumped the minor version. Apologies for the ongoing hiccup folks <3.


Do you prefer to do these methods in separate files as opposed
to a class method?  I can see arguements for each but would have
been inclined to do something like this under the Class.


Do we need it for compiler?


This looks good to me! :+1:


@aviau oops. :disappointed:

This is why I'm not a lawyer!


This doesn't actually make the test fail, but this holder type doesn't match the first vector type UInt4Vector, that is created in the vectors list. Should be be enforcing this type agreement so that this would fail?


@krainboltgreene I don't get your position. What changes for you if the repos of people you don't like are on github or on a different hoster? You don't see them in any case.

Besides, please don't act as if you were some kind of representative of anything. You are certainly not a representative of people that face the same disabilities as me and @g7823564  because you don't have a neurodegenerative disease. You're not a representative of github.

You're at best alienating people from the very real issues people with disability face, so please just stop, or be civil.


Fixes peerDependency issue #1


I think will be better if extract common part to property


Keep these.


Do these work under Travis?
Am just curious..


this is neither Golang otr nor our Fragment we are not using base64 encoding anywhere


why `pwd`?


Should Inaccuracy be computed over actual primary users or virtual primary users?


Keep this.


Great job with the carousel. Really like the images you used.


1709-ComponentSetpointScheme (kbenne) - 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-87.68%25-red.svg)


Good to know; I support dropping virtual and using override for children.


Yes, for example currently `kotlin.reflect.jvm.java` is used in several places in the compiler


It might make more sense to comment on the pull request, not on individual commits.


@Soll можно в 1 строчку, просто
```ruby
original_text.mb_chars.downcase == review_text.mb_chars.downcase
```


They released a apk with both libs.


relic - let me remove it


Great use of the carousel. Hint: if you like to see all the images on the same format. You must make sure that all images have the same dimensions and then the bootstrap carousel will take care of the rest.


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 4](https://build.fubar-dev.com/viewLog.html?buildId=53&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:17


Great work. Adding markers and popup messages.


Ah, well originally it was to effectively spread it over the arguments of `provider.set` cleanly. But it was bad code anyways!


what about http://faggit.xyz/?


Hah: https://github.com/ybur-yug/hobos/commit/8541c66516ad6cfb18e26e0c068b30457150680f

I guess it would be OK if I make my own name generator called "Niggers" just as long as I make it clear in the README that racism is very real problem in the United States and I also include links to the relevant charities (which you should all go donate to now btw)


If you look at this error it says that is trying reference a partial view which doesn't exist (or is possibly invalid).  It is unlikely that this is related to an eager loading change you made.

Look at `git diff` or compare commits in GitHub to see what else changed.

I can walk you through all of this on Friday.. I just wanted to make sure you're not spinning your wheels until then.


Why serialize needs to depend on Conversation?


Reverted it as it broke the build.


Since we don't want any `pid` files, you can change this to `*.pid`


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 5](https://build.fubar-dev.com/viewLog.html?buildId=56&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:14


See https://github.com/babel/babel/issues/2119#issuecomment-126112875


@jmendeth

More like anti and pro censorship.


it's just unit test for processing, we didn't have validation for that time being


I don't get it. Is this re-added?


1709-ComponentSetpointScheme (kbenne) - 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)


hey whats  the word on this one? looks like you had a good stab at it?


You're making an additional assumption here that the value you've derived from the 90s applied before them (back to day zero). This is not obviously the case (e.g. sunk public sector investment in housing?) and results in the 'explained by value:debt ratio from the 90s' line being a bit low I think?

You can get the same thing but add in the intercept using
```
lm(Value2015 ~ Debt2015, data=rw[1:45,])
```
which also means you don't have to regress on the differences.  This would position your plot potentially more 'correctly' - won't change the conclusions.

The other thing is there's a whole heap of positive autocorrelation in the data, so you're coefficient for the 90s may well be over-estimated.

Regarding the value:debt ratio, a plot of value vs debt is quite interesting as well.


@alavers

So political correct, much appreciated!


andrew, i just noticed this, after i commited some changes to the media classes. they now behave more as expected, with correct inheritance and reactive-properties.  just fyi.


Hi,

Still looking the same, although the code looks cleaner!

<img width="1025" alt="screen shot 2015-07-30 at 00 18 35" src="https://cloud.githubusercontent.com/assets/858793/8971440/cf204fa8-3650-11e5-8843-4061873585df.png">


Creating repo on Github


hey man,  are you going to submit this? what exactly does it do?


TeamCity Bender :: Deploy [Build 3.0.12.0](http://build.mikeobrien.net/viewLog.html?buildId=2441&buildTypeId=bender&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:17


Example AKT1 hive plot - acting as a kinase. edges connect to distinct peptides that the AKT1 protein phosphorylates; which belong to substrate proteins (edge to Substrate). In the absence of known target peptides, edges connect directly from kinase to substrate.

![akt1_hive_kin](https://cloud.githubusercontent.com/assets/7833961/8971475/a1232b5c-3605-11e5-8a46-6ff4e6e57b1c.jpg)


I remember having the exact issue on Linux back in May, but I don't remember what caused it and what I did (if anything) to fix it...


Burada AdapterAbstract mı extend olacaktı yoksa repository e mi baglayacaz bunu?


Well, MyDocuments is the same as Personal.


We phased out (or were supposed to have phased out) ```requires_step()```. You're right, webhooks should never block now. ```is_pause``` only blocks on ```wait_*``` class of ruleset types.

Actually, there's no reason ```requires_step()``` should be in the codebase anymore, I'll remove it.


Is that an extra [ ] around request id?


I also remember that I matched newer OF API more closely, maybe it is the same case now too


Actually, deriveJSON wasn't done for UpdateStatus in ide-backend.  But that is a better place to put it - I've done that in https://github.com/fpco/ide-backend/commit/5bb01e65dbb4b3dd5fc6f8b30a585fc084b87a39


Just one question. This change adds to the :per-turn map a key/value pair of card-id/true when :runner-install is fired, right? And then in :pre-install it checks for the presence of this value and doesn't give you the effect if so. Doesn't this only block you from using Kate's ability twice in a turn if you try to install the same card id twice? ie. the second time through if you are installing a different card, (:cid card) will be different, that key won't show up in the map and it will let you install with the bonus.

Or am I reading this code incorrectly?


https://gtmetrix.com/compare/MwLObVII/hQfsrcH4

Cool, I'll try to have a look when I have some time and improve Bootswatch page a little more :)


They aren't required to implement the interface. There are a few where it doesn't make much sense to implement it.


TeamCity Bender :: Deploy [Build 3.0.13.0](http://build.mikeobrien.net/viewLog.html?buildId=2443&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3240, ignored: 5 Build time: 00:01:12


Work in progress


Everything looks good to me. +1


This does a good job of accomplishing the "Unimplemented feature" of "Command-line auto-complete."
However, it needs to be tested by maintainer, and brought into setup.py as well if it's going to be official. Could just be an optional component described in the readme as well.


It's switching from a single pair of () to two pairs of [ ]s, to disambiguate from X-Socrata-Resource.

I could also try to figure out the correct number of backslashes to get parens working.


At some point, we should should add an abstraction to manage this (e.g., `VersionedValue`) . We do similar behavior in multiple places, if I recall correctly.


add @tailrec


Thanks for the reminder, I'd gotten busy with other things and forgot about this.  Yes, I'll tidy things up a bit and then submit this.

I rewrote some things to allow for displaying impedance Smith charts with non-normalized impedances, e.g. with 50 Ohms printed in the center.  Since this means the text labels are no longer fixed sizes, I also rewrote the label positioning to handle these changes.


This could also be a candidate for a feature branch to be merged in later, but I see no branching in the existing git so I'd like to respect that.


@sjorge: What's happening here is that `ctypes` is looking for the OpenSSL crypto library using `ldconfig`, and it's not finding it.

What output do you get from `ldconfig -p|grep libcrypto`? You should see something like
```
        libcrypto.so.1.0.0 (libc6,x86-64) => /usr/lib/libcrypto.so.1.0.0
        libcrypto.so (libc6,x86-64) => /usr/lib/libcrypto.so
```


Would this arbitrary message be exposed all the way to users?  Is that OK?


I haven't been thinking of thrift too much lately but if I remember correctly, you need to use a compatible version of Scrooge across the board. This is a trade-off in order to solve the bigger problems that occur if every downstream project recompiles all the upstream thrift sources all the time.


@alavers @onodera-punpun  I find it a fair compromise TBH. I'm on the anti-censorship side of the fence, and it was my proposal.

I think it is worth it to be offensive, if one at the same time raises awareness for the very problem.
Jokes, about rape can be funny if they make fun of the rapists. Jokes about race can be funny if they make fun of the racism.

There is nothing worse for a problem than silence.

I have MS, and I'd rather have people use cripple more as an insult, because it would at least raise some awareness for a horrible disease, that is still under-researched.


"generator" arg is unused


It will have an icon in the Qt interface.  GTK+ doesn't support icons on toggle menu items.


@krainboltgreene

![kfo](https://cloud.githubusercontent.com/assets/512714/8971673/74905e82-3607-11e5-9dec-143213cec634.jpg)


This got fixed in PR #25830 not sure it got merged in dev yet.


should this rule have Generator: true?


So, there's a design pattern here. It does not affect correctness, AFAICT, but it may affect performance for different implementations of pthreads. In particular, all the other uses of CV in grpc C++ that I can find use the notify_one/notify_all inside the same lock as the boolean variable being set. There is no race here. There may be a performance issue, with different sources recommending different design patterns based on the specifics of the pthreads implementation. Although I can't offer a specific rule here, can you put notify inside the lock for now? I will file an issue to test this out for performance, but let's make it all one way for now. For now, our inertia-based policy should be to have notify inside the lock.


cool, sounds good!

On Wed, Jul 29, 2015 at 6:30 PM, Brian Gibbons <notifications@github.com>
wrote:

> Thanks for the reminder, I'd gotten busy with other things and forgot
> about this. Yes, I'll tidy things up a bit and then submit this.
>
> I rewrote some things to allow for displaying impedance Smith charts with
> non-normalized impedances, e.g. with 50 Ohms printed in the center. Since
> this means the text labels are no longer fixed sizes, I also rewrote the
> label positioning to handle these changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gibbons/scikit-rf/commit/6c2b6af1905e6a05bbfd6244e032f5227cb19338#commitcomment-12432517>
> .
>


@bob-rove I can answer already this: is not thread safe. But I think I have a fix (for the fix). I'll put it as soon as I do more tests to be sure.


See above comment about notify inside the lock.


You can use a monochrome icon theme if you find the red color distracting.  And you can set the toolbar icon size larger (or set the mouse speed lower) if you have trouble clicking on the buttons accurately.


Why not just add a label, id:?


@tasilvar


cool, thanks!


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|suite_setup|1|0|0|0
basic_test|basic/test_create_vm|1|0|0|0
basic_test|basic/test_stop_vm|1|0|0|0
basic_test|basic/test_start_vm|1|0|0|0
basic_test|basic/test_reboot_vm|1|0|0|0
basic_test|basic/test_add_volume|1|0|0|0
basic_test|basic/test_add_vol_to_stopvm|1|0|0|0
basic_test|basic/test_add_volume_reboot_vm|1|0|0|0
basic_test|basic/test_add_multi_volumes|1|0|0|0
basic_test|basic/test_add_volume_negative|1|0|0|0
basic_test|basic/test_crt_temp_from_volume|1|0|0|0
basic_test|basic/test_crt_vm_with_volume|1|0|0|0
basic_test|basic/test_vm_securitygroup|1|0|0|0
basic_test|basic/test_create_iso_vm|1|0|0|0
basic_test|__account/test_crt_vm_by_normal_account__|0|1|0|0
basic_test|__account/test_crt_vm_by_normal_account2__|0|1|0|0
basic_test|__account/test_vm_ops_by_normal_account__|0|1|0|0
basic_test|__account/test_multi_accounts_vm_ops__|0|1|0|0
basic_test|suite_teardown|1|0|0|0
virtual_router_test|suite_setup|1|0|0|0
virtual_router_test|host/test_reconnect_host|1|0|0|0
virtual_router_test|account/test_vm_ops_by_normal_account|1|0|0|0
virtual_router_test|account/test_multi_accounts_vm_ops|1|0|0|0
virtual_router_test|novlan/test_create_vm|1|0|0|0
virtual_router_test|novlan/test_reboot_vm|1|0|0|0
virtual_router_test|novlan/test_ping_vms|1|0|0|0
virtual_router_test|host/test_reconnect_host|1|0|0|0
virtual_router_test|vlan/test_create_vm|1|0|0|0
virtual_router_test|vlan/test_reboot_vm|1|0|0|0
virtual_router_test|vlan/test_snat_create|1|0|0|0
virtual_router_test|vlan/test_dnat_reboot|1|0|0|0
virtual_router_test|vlan/test_ping_vms|1|0|0|0
virtual_router_test|vlan/test_create_3vms|1|0|0|0
virtual_router_test|vlan/test_start_vm|1|0|0|0
virtual_router_test|vlan/test_3l3s_vm|1|0|0|0
virtual_router_test|vlan/test_remove_dhcp|1|0|0|0
virtual_router_test|vlan/test_2l3_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.0|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.1|1|0|0|0
virtual_router_test|vlan/test_start_vms_simultaneously|1|0|0|0
virtual_router_test|vlan/test_crt_static_ip_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vm_with_hostname|1|0|0|0
virtual_router_test|vlan/test_attach_nic|1|0|0|0
virtual_router_test|vlan/test_attach_nic_stopped_vm|1|0|0|0
virtual_router_test|vlan/test_attach_nic_reboot_vm|1|0|0|0
virtual_router_test|volume/test_add_volume|1|0|0|0
virtual_router_test|volume/test_add_m_vols|1|0|0|0
virtual_router_test|volume/test_add_volume_vr|1|0|0|0
virtual_router_test|volume/test_crt_vm_volume|1|0|0|0
virtual_router_test|volume/test_crt_vm_2_volumes|1|0|0|0
virtual_router_test|volume/test_crt_vm_24_volumes|1|0|0|0
virtual_router_test|__volume/test_volume_ops_on_stopped_vm__|0|1|0|0
virtual_router_test|volume/test_crt_vm_from_dvt|1|0|0|0
virtual_router_test|template_image/test_crt_temp_image|1|0|0|0
virtual_router_test|template_image/test_del_temp_image|1|0|0|0
virtual_router_test|sg/test_ingress|1|0|0|0
virtual_router_test|sg/test_egress|1|0|0|0
virtual_router_test|sg/test_icmp|1|0|0|0
virtual_router_test|sg/test_udp|1|0|0|0
virtual_router_test|sg/test_stopped_vm|1|0|0|0
virtual_router_test|sg/test_vm_ops|1|0|0|0
virtual_router_test|sg/test_2vm_egress|1|0|0|0
virtual_router_test|sg/test_2vm_ingress|1|0|0|0
virtual_router_test|sg/test_4vm_complx|1|0|0|0
virtual_router_test|sg/test_detach_l3|1|0|0|0
virtual_router_test|pf/test_2l3s_pfs|1|0|0|0
virtual_router_test|pf/test_attach_pf|1|0|0|0
virtual_router_test|pf/test_delete_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_del_vip_before_add_pf_to_vm|1|0|0|0
virtual_router_test|pf/test_multi_pfs_sg_tcp|1|0|0|0
virtual_router_test|pf/test_multi_rules|1|0|0|0
virtual_router_test|pf/test_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_pf_sg_tcp|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_udp_same_vip_ports|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_with_same_vip|1|0|0|0
virtual_router_test|pf/test_tcp|1|0|0|0
virtual_router_test|pf/test_udp|1|0|0|0
virtual_router_test|pf/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_2_eips|1|0|0|0
virtual_router_test|eip/test_2l3s_eip_pf|1|0|0|0
virtual_router_test|eip/test_2l3s_eips|1|0|0|0
virtual_router_test|eip/test_attach_eip|1|0|0|0
virtual_router_test|eip/test_create_eip_vm|1|0|0|0
virtual_router_test|eip/test_del_vip_before_add_eip_to_vm|1|0|0|0
virtual_router_test|eip/test_eip_sg_tcp|1|0|0|0
virtual_router_test|eip/test_pf_eip_attach|1|0|0|0
virtual_router_test|eip/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_vm_ops2|1|0|0|0
virtual_router_test|snapshot/test_backup_snapshot|1|0|0|0
virtual_router_test|__snapshot/test_create_100_snapshots__|0|0|1|0
virtual_router_test|__snapshot/test_create_snapshot__|0|0|1|0
virtual_router_test|snapshot/test_create_sp_after_delete|1|0|0|0
virtual_router_test|snapshot/test_create_image_by_snapshot|1|0|0|0
virtual_router_test|snapshot/test_crt_sp_for_both_root_...|1|0|0|0
virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm2__|0|0|1|0
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm__|0|0|1|0
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|snapshot/test_crt_sp_from_vol_crt_f...|1|0|0|0
virtual_router_test|__snapshot/test_change_max_sp_depth__|0|0|1|0
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|regression/delete_sg_with_2_attache...|1|0|0|0
virtual_router_test|regression/rm_nic_sg_on_stopped_vm|1|0|0|0
virtual_router_test|regression/test_data_volume_with_vm...|1|0|0|0
virtual_router_test|__vr/test_ping_vms_reboot_vr__|0|1|0|0
virtual_router_test|__vr/test_reconnect_vr__|0|1|0|0
virtual_router_test|__vr/test_start_vrs_simultaneously__|0|1|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|__suite_teardown__|0|0|1|0
local_ps_virtual_router_test|suite_setup|1|0|0|0
local_ps_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_ps_virtual_router_test|account/test_vm_ops_by_normal_account|1|0|0|0
local_ps_virtual_router_test|account/test_multi_accounts_vm_ops|1|0|0|0
local_ps_virtual_router_test|novlan/test_create_vm|1|0|0|0
local_ps_virtual_router_test|novlan/test_reboot_vm|1|0|0|0
local_ps_virtual_router_test|novlan/test_ping_vms|1|0|0|0
local_ps_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_create_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_reboot_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_snat_create__|0|1|0|0
local_ps_virtual_router_test|vlan/test_dnat_reboot|1|0|0|0
local_ps_virtual_router_test|__vlan/test_ping_vms__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_create_3vms__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_start_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_3l3s_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_remove_dhcp__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_2l3_vm__|0|1|0|0
local_ps_virtual_router_test|vlan/test_crt_vms_with_limited_ips|1|0|0|0
local_ps_virtual_router_test|vlan/test_crt_vms_with_limited_ips2.0|1|0|0|0
local_ps_virtual_router_test|vlan/test_crt_vms_with_limited_ips2.1|1|0|0|0
local_ps_virtual_router_test|vlan/test_start_vms_simultaneously|1|0|0|0
local_ps_virtual_router_test|__vlan/test_crt_static_ip_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_crt_vm_with_hostname__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_attach_nic__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_attach_nic_stopped_vm__|0|1|0|0
local_ps_virtual_router_test|__vlan/test_attach_nic_reboot_vm__|0|1|0|0
local_ps_virtual_router_test|__volume/test_add_volume__|0|1|0|0
local_ps_virtual_router_test|__volume/test_add_m_vols__|0|1|0|0
local_ps_virtual_router_test|__volume/test_add_volume_vr__|0|1|0|0
local_ps_virtual_router_test|__volume/test_crt_vm_volume__|0|1|0|0
local_ps_virtual_router_test|__volume/test_crt_vm_2_volumes__|0|1|0|0
local_ps_virtual_router_test|__volume/test_crt_vm_24_volumes__|0|1|0|0
local_ps_virtual_router_test|__volume/test_volume_ops_on_stopped_vm__|0|1|0|0
local_ps_virtual_router_test|__volume/test_crt_vm_from_dvt__|0|1|0|0
local_ps_virtual_router_test|__template_image/test_crt_temp_image__|0|1|0|0
local_ps_virtual_router_test|__template_image/test_del_temp_image__|0|1|0|0
local_ps_virtual_router_test|__sg/test_ingress__|0|1|0|0
local_ps_virtual_router_test|__sg/test_egress__|0|1|0|0
local_ps_virtual_router_test|__sg/test_icmp__|0|1|0|0
local_ps_virtual_router_test|__sg/test_udp__|0|1|0|0
local_ps_virtual_router_test|__sg/test_stopped_vm__|0|1|0|0
local_ps_virtual_router_test|__sg/test_vm_ops__|0|1|0|0
local_ps_virtual_router_test|__sg/test_2vm_egress__|0|1|0|0
local_ps_virtual_router_test|__sg/test_2vm_ingress__|0|1|0|0
local_ps_virtual_router_test|__sg/test_4vm_complx__|0|1|0|0
local_ps_virtual_router_test|__sg/test_detach_l3__|0|1|0|0
local_ps_virtual_router_test|__pf/test_2l3s_pfs__|0|1|0|0
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|__pf/test_delete_pf_after_destroy_vm__|0|1|0|0
local_ps_virtual_router_test|pf/test_del_vip_before_add_pf_to_vm|1|0|0|0
local_ps_virtual_router_test|__pf/test_multi_pfs_sg_tcp__|0|1|0|0
local_ps_virtual_router_test|__pf/test_multi_rules__|0|1|0|0
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|__pf/test_pf_sg_tcp__|0|1|0|0
local_ps_virtual_router_test|__pf/test_pfs_tcp_udp_same_vip_ports__|0|1|0|0
local_ps_virtual_router_test|__pf/test_pfs_tcp_with_same_vip__|0|1|0|0
local_ps_virtual_router_test|__pf/test_tcp__|0|1|0|0
local_ps_virtual_router_test|__pf/test_udp__|0|1|0|0
local_ps_virtual_router_test|__pf/test_vm_ops__|0|1|0|0
local_ps_virtual_router_test|__eip/test_2_eips__|0|1|0|0
local_ps_virtual_router_test|__eip/test_2l3s_eip_pf__|0|1|0|0
local_ps_virtual_router_test|__eip/test_2l3s_eips__|0|1|0|0
local_ps_virtual_router_test|__eip/test_attach_eip__|0|1|0|0
local_ps_virtual_router_test|__eip/test_create_eip_vm__|0|1|0|0
local_ps_virtual_router_test|__eip/test_del_vip_before_add_eip_to_vm__|0|1|0|0
local_ps_virtual_router_test|__eip/test_eip_sg_tcp__|0|1|0|0
local_ps_virtual_router_test|__eip/test_pf_eip_attach__|0|1|0|0
local_ps_virtual_router_test|__eip/test_vm_ops__|0|1|0|0
local_ps_virtual_router_test|eip/test_vm_ops2|1|0|0|0
local_ps_virtual_router_test|__snapshot/test_backup_snapshot__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_create_100_snapshots__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_create_snapshot__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_create_sp_after_delete__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_create_image_by_snapshot__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_for_both_root_...__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm2__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm3__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_delete_snapshot_on_ru...__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_in_live_vm__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_crt_sp_from_vol_crt_f...__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_change_max_sp_depth__|0|1|0|0
local_ps_virtual_router_test|__snapshot/test_change_max_sp_depth_l...__|0|1|0|0
local_ps_virtual_router_test|__regression/delete_sg_with_2_attache...__|0|1|0|0
local_ps_virtual_router_test|__regression/rm_nic_sg_on_stopped_vm__|0|1|0|0
local_ps_virtual_router_test|__regression/test_data_volume_with_vm...__|0|1|0|0
local_ps_virtual_router_test|__vr/test_ping_vms_reboot_vr__|0|1|0|0
local_ps_virtual_router_test|__vr/test_reconnect_vr__|0|1|0|0
local_ps_virtual_router_test|__vr/test_start_vrs_simultaneously__|0|1|0|0
local_ps_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_ps_virtual_router_test|suite_teardown|1|0|0|0
local_nfs_virtual_router_test|suite_setup|1|0|0|0
local_nfs_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_nfs_virtual_router_test|account/test_vm_ops_by_normal_account|1|0|0|0
local_nfs_virtual_router_test|account/test_multi_accounts_vm_ops|1|0|0|0
local_nfs_virtual_router_test|novlan/test_create_vm|1|0|0|0
local_nfs_virtual_router_test|novlan/test_reboot_vm|1|0|0|0
local_nfs_virtual_router_test|novlan/test_ping_vms|1|0|0|0
local_nfs_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_create_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_reboot_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_snat_create__|0|1|0|0
local_nfs_virtual_router_test|vlan/test_dnat_reboot|1|0|0|0
local_nfs_virtual_router_test|__vlan/test_ping_vms__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_create_3vms__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_start_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_3l3s_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_remove_dhcp__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_2l3_vm__|0|1|0|0
local_nfs_virtual_router_test|vlan/test_crt_vms_with_limited_ips|1|0|0|0
local_nfs_virtual_router_test|vlan/test_crt_vms_with_limited_ips2.0|1|0|0|0
local_nfs_virtual_router_test|vlan/test_crt_vms_with_limited_ips2.1|1|0|0|0
local_nfs_virtual_router_test|vlan/test_start_vms_simultaneously|1|0|0|0
local_nfs_virtual_router_test|__vlan/test_crt_static_ip_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_crt_vm_with_hostname__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_attach_nic__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_attach_nic_stopped_vm__|0|1|0|0
local_nfs_virtual_router_test|__vlan/test_attach_nic_reboot_vm__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_add_volume__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_add_m_vols__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_add_volume_vr__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_crt_vm_volume__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_crt_vm_2_volumes__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_crt_vm_24_volumes__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_volume_ops_on_stopped_vm__|0|1|0|0
local_nfs_virtual_router_test|__volume/test_crt_vm_from_dvt__|0|1|0|0
local_nfs_virtual_router_test|__template_image/test_crt_temp_image__|0|1|0|0
local_nfs_virtual_router_test|__template_image/test_del_temp_image__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_ingress__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_egress__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_icmp__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_udp__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_stopped_vm__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_vm_ops__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_2vm_egress__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_2vm_ingress__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_4vm_complx__|0|1|0|0
local_nfs_virtual_router_test|__sg/test_detach_l3__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_2l3s_pfs__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_attach_pf__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_delete_pf_after_destroy_vm__|0|1|0|0
local_nfs_virtual_router_test|pf/test_del_vip_before_add_pf_to_vm|1|0|0|0
local_nfs_virtual_router_test|__pf/test_multi_pfs_sg_tcp__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_multi_rules__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_pf_after_destroy_vm__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_pf_sg_tcp__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_pfs_tcp_udp_same_vip_ports__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_pfs_tcp_with_same_vip__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_tcp__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_udp__|0|1|0|0
local_nfs_virtual_router_test|__pf/test_vm_ops__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_2_eips__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_2l3s_eip_pf__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_2l3s_eips__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_attach_eip__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_create_eip_vm__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_del_vip_before_add_eip_to_vm__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_eip_sg_tcp__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_pf_eip_attach__|0|1|0|0
local_nfs_virtual_router_test|__eip/test_vm_ops__|0|1|0|0
local_nfs_virtual_router_test|eip/test_vm_ops2|1|0|0|0
local_nfs_virtual_router_test|__snapshot/test_backup_snapshot__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_create_100_snapshots__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_create_snapshot__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_create_sp_after_delete__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_create_image_by_snapshot__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_for_both_root_...__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm2__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm3__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_delete_snapshot_on_ru...__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_in_live_vm__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_crt_sp_from_vol_crt_f...__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_change_max_sp_depth__|0|1|0|0
local_nfs_virtual_router_test|__snapshot/test_change_max_sp_depth_l...__|0|1|0|0
local_nfs_virtual_router_test|__regression/delete_sg_with_2_attache...__|0|1|0|0
local_nfs_virtual_router_test|__regression/rm_nic_sg_on_stopped_vm__|0|1|0|0
local_nfs_virtual_router_test|__regression/test_data_volume_with_vm...__|0|1|0|0
local_nfs_virtual_router_test|__vr/test_ping_vms_reboot_vr__|0|1|0|0
local_nfs_virtual_router_test|__vr/test_reconnect_vr__|0|1|0|0
local_nfs_virtual_router_test|__vr/test_start_vrs_simultaneously__|0|1|0|0
local_nfs_virtual_router_test|__host/test_reconnect_host__|0|1|0|0
local_nfs_virtual_router_test|suite_teardown|1|0|0|0
Total|295|120|163|6|6


Yep it's in 2015.8 already, I suppose dev will follow eventually. Seems there is more to fix before SmartOS works again though :(


The distinction between bootstrap and bootstrap_core is still confusing.  Everything in blueprint/ is now bootstrap_core_go_package, and everything in the build logic is now bootstrap_go_package.  Maybe these should be renamed to blueprint_go_package?  The build logic could then be renamed to build_logic_go_package, possibly supporting bootstrap_go_package for backwards compatibility?


In general I agree with Jc's (implied) point that SafeDownCast is the more standard way to do this in VTK and it has a different implementation that is probably more robust.

https://github.com/Kitware/VTK/blob/28aebaa3c3a722edd57275c35c4e7dbe1cf8cabc/Common/Core/vtkTypeTemplate.h#L47


card refers to the card whose ability is being triggered, in this case
Kate. target is the card that caused the ability to fire, in this case the
card being installed.
On Jul 29, 2015 3:26 PM, "Justin Liew" <notifications@github.com> wrote:

> Just one question. This change adds to the :per-turn map a key/value pair
> of card-id/true when :runner-install is fired, right? And then in
> :pre-install it checks for the presence of this value and doesn't give you
> the effect if so. Doesn't this only block you from using Kate's ability
> twice in a turn if you try to install the same card id twice? ie. the
> second time through if you are installing a different card, (:cid card)
> will be different, that key won't show up in the map and it will let you
> install with the bonus.
>
> Or am I reading this code incorrectly?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nealterrell/netrunner/commit/256b7c94e5be06bd2d020a7dd9d4225a72c0c265#commitcomment-12432467>
> .
>


This might be simpler as TargetStage()


Some translation should be generalized later on.


Consider `AccountID lastAccountID (beast::zero);` so that we don't rely on the "base_uint is zero-initialized" behavior. The change has zero overhead.


I'll get this one in with the IS_BLOCK_SERIES fixes...


Looks like a nice improvement - thanks!


Could not find an issue tracker, so reporting here...

This commit looks partly broken. Here is a test case where it causes indexing errors:

    class C {
     protected:
      /// \c bar
      int foo();
      int bar();
    };

    error: 'bar' is a protected member of 'C'

I've also managed to trigger this which I think is related -- but couldn't find a simple repro:

    indexer: /home/delroth/src/kythe/third_party/llvm/llvm/tools/clang/lib/AST/DeclBase.cpp:1341: DeclContext::lookup_result clang::DeclContext::lookup(clang::DeclarationName) const: Assertion `DeclKind != Decl::LinkageSpec && "Should not perform lookups into linkage specs!"' failed.


What's the point of `_Valid` here? Why do we care if `T<U, V...>` fails substitution in `_Valid` instead of failing substitution in `_Is`?


This appeared to be a copy/paste error. Tygon's fix looks correct.


Theoretically it is also possible that it is some other issue with java8..

I tried to find a java7 installation for Fedora22, but I wasn't even able to recompile the java7 src rpm because it depends on java7 implementation to compile javac!

So, I hacked the pom.xml to download jenkins 1.560 for testing, which appears to have gotten most of the tests working, but maybe there are issues related with this?

At any rate the git code that I have has the warning above, caused by the push command we provided to it.. I haven't figured out why it would work sometimes and not others. The tests take a bit to complete, so I will get back to you on this after running with a clean config, and I will try to figure out what option in my git-config caused the misbehavior.

I don't believe the behavior should be supported, unless maybe somehow it setup the wrong default origin so "git push <remote>" is failing because it isn't the default remote?


Do we want this change to be merged and isn't there a way of adding this with a .private file?


Why the special case for arrays? If `std::vector<int>&&` is a valid range, why should `int(&&)[N]` not be a valid range?


Steve,
Sorry about that, I was not thinking about your comprehension when I was writing it.  I was thinking, "Steve should be able to understand it if I can!!".  Kind of like when we write our own code, i.e., "Anybody can figure out what I am doing!"  Separate comments to follow.


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


Sorry @danielmarbach but I have just noticed the name change and I really dislike it :(
Yes it may be more descriptive of what it does but IMO that can be conveyed using API doco.
:-1: for this name


I just realized that we never fixed this. Looking at it again, I think you were right initially. Do you have an appropriate notation for asymptotically equal to? Otherwise, I think I must have intended that the limit of k_n - (2 / (drspr(n) - 1)) = 0.


LGTM


According to X.690, the initial length octet cannot be 0xFF, so you would need a check "if (i == 0xff) return 0" here.


yanlis extend etmisim bence :dart:


commit message is bad


Will do.

On Wed, Jul 29, 2015 at 3:37 PM, Vijay Pai <notifications@github.com> wrote:

> See above comment about notify inside the lock.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yang-g/grpc/commit/4f61528b96fff1210f90e1be07f2d4e16c0d6659#commitcomment-12432582>
> .
>


you should probably allow None for the ConsolePrinter first and then simply pass it


It's not unused. It's set during StageMain -- that's the only stage that we need to have Generator: true enabled. It shouldn't make a huge difference with the two other stages, but it shouldn't be necessary either.


@johnsimons so you agree it is more descriptive but still dont like it. can u give some more reasoning?


Pq usar a variável $LOCALIP?


With `T&&` in general, we don't know whether the returned iterator is valid or not. (`T` could be `tagged_pair<begin(I), end(I)>` for instance.) With `const T(&&)[N]`, we can be 100% certain that the returned pointer will dangle.


Does it not have any effect on subsumption relationships?


Will this still load modules based on sys.path ?


`state` isn't defined at this point, right? Should be fully qualified like the two lines lower `cm.state.lint.onMouseOver`.


@dpgeorge here is the appending of the MAC address to the default AP SSID. Thanks!


Lets move the mode to a constant?


I could use the built-in constant `File::APPEND` instead.


#1 Add version requirement in the Readme


Könnte man machen. Aber in dem Fall entkoppelt das die Sourcen etwas. D.h. der header muss nur noch in der CPP drin sein, nicht mehr in der h


Too technical, just say "CPU and GPU", I guess.


This is missing one profile zzrelax, it is number 11


Okay!  I'll try using a .private file on my machine.  Once I know it's working, I'll remove this commit from the PR.


*It's very likely your computer is simply not fast enough ...


Indeed! Crazy I missed this! :-) I think this should be `type`-specific, yes. I can change this right away.


*full speed, even if it can run even demanding PC games just fine.


1718_setpoint_managers (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2262 of 2288 tests passed)

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


*Technically, the only actual hard..


Something seems to have broken the status text like Playing, duration etc. The tooltip seems to show up fine though.


Hi Lodash community!!

I've had an idea to improve zipObject function. My English level is very low, sorry, but I can show and example of my idea.

[ ['test', 1], ['test', 2], ['test4': 3] ] => // before returns {test: 2, test4: 3}

I think would be useful that when a prop array is repeated, the result return a property with an array values.

[ ['test', 1], ['test', 2], ['test4': 3] ] => // now returns {test: [1,2], test4: 3}

With this commit, I've modified code to work this.

I hope that my contribution will be useful.


you calling the first param self is a bit weird don't you think?


oh, i see the "generator" arg is special, setting it to true is the build args is equivalent to Generator: true in the rule params.


Much better!


please make this a property, use the `@property` decorator


MemberArray_Reduce (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1771 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-100.0%25-green.svg)


actually this might have side effects or do some heavy calculations, I'm not sure what you're doing here is a good idea


Maybe `!string.empty()` would be clearer here?


There's a typo.


@keithbostic, this looks right to me, but what was the motivation?


Where does this put the compiled versions?


No reason to add these empty files


No helpers needed either


I think you mean "This is gitted"


hee ok o zaman kafam karıstı cunku :confused:
alkışlar sana o zaman :clap:


hey you're removing our nicely colored log messages silently in this commit don't you?


This commit LGTM.


Steve,
Let us first see if I can summarize my understanding of ActionCable and see if you agree:
* The purpose of ActionCable is to serve up WebSockets
* In ActionCable a connection instance instantiates a WebSocket connection
* In ActionCable multiple tabs pointing to the application represent multiple connection instances as could different browsers pointing to the application (in the case of Good Clean Games we could have 4 different people on 4 different browsers playing a Old Maid Game, this would be a case where ActionCable would be serving up 4 different connections which means it is serving up 4 different WebSockets).
* The client of a WebSocket connection is called a consumer.
* Each connection instance can have multiple channels.
* A consumer of a connection instance can subscribe to any number of channels on that connection instance.
* After reading DHH's introduction and the example a number of times, the channel is architected to encapsulate a logical unit of work, much like a controller in the Rails MVC architecture.  This explains why there was more code (comparatively) in the comments_channel.rb and comments.coffee files when looking at the actionable-examples repository.


Maybe, but there isn't any reason why blueprint couldn't have a bootstrap_go_package today, it just happens that all of the packages are required for minibp right now. We do have a mix of bootstrap_core_go_binary and bootstrap_go_binary modules.

I was half-tempted to just special-case 'minibp' and have it automatically mark all of it's dependencies as 'core', but 'gotestmain' and 'choosestage' would also need to have that done to them. Maybe do that for packages, but keep the bootstrap_core_go_binary and bootstrap_go_binary difference?


Thank you for the hint.
How do I make the same dimensions for the images? Do I open the images in a photo editor and change the dimensions there?


estava quebrando o menu no ipad mini. O menu ficava muito grande e os itens left desapareciam


Check out [number_to_currency](http://devdocs.io/rails/activesupport/numberhelper#method-i-number_to_currency)


Agreed, implemented in 6051df0e260a17f0b831758d5681746bdf4ca92f to be squashed into this change once the pending discussions have been resolved.


Oh! I understand now. Duh.


I have no problem with that test being removed. I wanted to assure that I understood what you were seeing.  Your commit message says there are issues running with git versions greater than 1.7, yet I run many different git versions and the test passes on all of them.

The global configuration setting is an interesting idea of a source of the difference between your configuration and mine.

The test itself is somewhat nonsensical, since a push without an explicitly declared ref seems unexpected.


Yes, i completely agree hah, that's very important.

How about more like [this](https://github.com/aaivazis/react-wrapper/blob/d5937a5c4400484687dcb56e39a2312f07905c9d/src/decorators/hoverable/index.js)?

I have the feeling that the way I determine if the wrapped component has implemented an `onMouseEnter` method or not is not the best way to go about it, but it works at the moment as a proof of concept.


This is the expected result...

`<!--[if lt IE 9]> <!-- <script type="text/javascript" src="scripts/that.js"></script> --> <![endif]-->`


Nobody wants any of you here, you're toxic, you're ableist, and you're harmful to communities without remorse.

For what it's worth, I'm glad this thread happened. It's a showcase of what kind of shit goes on in developer culture.

You're going to be alone and cynical in your little MRA "free speech" camps so busy being offended by the concept of offensiveness that you wither.


this commit breaks cases where $default_group is not 'root' because $default_group cannot be reassigned.


:+1: for reverting this.


TeamCity Bender :: Deploy [Build 3.0.14.0](http://build.mikeobrien.net/viewLog.html?buildId=2445&buildTypeId=bender&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3241, ignored: 5 Build time: 00:00:44
### Failed tests
```
Tests.dll: Tests: Serializer: Xml: ObjectTests: should_set_type_namespace_prefix_from_xml_attribute: Tests.Serializer.Xml.ObjectTests.should_set_type_namespace_prefix_from_xml_attribute(Tests.Serializer.Xml.ObjectTests+XmlRootNamespacePrefix): <no details avaliable>

```


@krainboltgreene Funny how this push is all about inclusiveness, while you use it to exclude those who you disagree with.


Yes. I suppose vcov could return a submatrix of cov.scaled.

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

> In fact shouldn't we basically discard gamma after fitting the model and
> use alpha ('Intercept') for everything after?
>
> On Wednesday, July 29, 2015, Jonah Sol Gabry <jgabry@gmail.com> wrote:
>
> > Also, I might be wrong, but I think users would probably expect us to
> > include alpha instead of gamma in cov.scaled when it's used by predict,
> > right?
> >
> > On Wednesday, July 29, 2015, bgoodri <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> >> Maybe the best thing would be to make covmat in stanreg be cov.unscaled?
> >> The only downside I see is that if someone does vcov() they will get the
> >> variance-covariance matrix for the unconstrained parameters rather than
> >> the
> >> parameters of interest.
> >>
> >> On Wed, Jul 29, 2015 at 3:23 PM, Jonah Gabry <notifications@github.com>
> >> wrote:
> >>
> >> > Yeah. Not a big deal, but just wanted to make sure we were on the same
> >> > page.
> >> >
> >> > On Wednesday, July 29, 2015, bgoodri <notifications@github.com>
> wrote:
> >> >
> >> > > 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
> >> > > <javascript:_e(%7B%7D,'cvml','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
> >> > > <javascript:_e(%7B%7D,'cvml','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');>
> >> > > > > <javascript:_e(%7B%7D,'cvml','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');>
> >> > > > > <javascript:_e(%7B%7D,'cvml','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');>
> >> > > > > <javascript:_e(%7B%7D,'cvml','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
> >> > > >
> >> > > > .
> >> > > >
> >> > >
> >> > > —
> >> > > Reply to this email directly or view it on GitHub
> >> > > <
> >> >
> >>
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12428967
> >> > >
> >> > > .
> >> > >
> >> >
> >> > —
> >> > Reply to this email directly or view it on GitHub
> >> > <
> >>
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12429067
> >> >
> >> > .
> >> >
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12429690
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/4dbf0063656745825898950d710bebd7bc5f7cad#commitcomment-12430366>
> .
>


indentation


Since it's always a binary issue, not a package issue, then maybe a single type of package with multiple binary types would make more sense?


@j-pb I'm on the anti-censor side as well. I was pointing out how silly it is that @ybur-yug defended github shutting down one repo because it had a disparaging term in the readme while he's hosting his own repo named "hobos".

And in an act of acknowledging his own hypocrisy, he quickly makes a readme change to promote the cause of solving homelessness... in a project that exists to generate funny hobo names.... It's just priceless.


switch order to: `bindings` or `viewBindings`


ver melhor isso


@dcs619 , FYI, this is a small update to a previous migration. If you've already run this migration in prod/beta, you may want to make this update manually (it's for the html block at the top of the Event Registration admin page to display navigation icons).  We updated it manually in or prod system.


hahaha `it ('should be tested')` made me laugh for a good while.

This is pretty close to what I had in mind. If it's okay with you I'm going to make a PR with some nazi-esque changes and we can move the discussion of the actual implementation there


rename


Wow, 22 days ago. That's a long time between commits.


@krainboltgreene

I like how you literally just keep posting rants without responding to anyone, for example @j-pb who says he has an actual disability, I'll quote if for you.

>@krainboltgreene I don't get your position. What changes for you if the repos of people you don't like are on github or on a different hoster? You don't see them in any case.

>Besides, please don't act as if you were some kind of representative of anything. You are certainly not a representative of people that face the same disabilities as me and @g7823564 because you don't have a neurodegenerative disease. You're not a representative of github.

>You're at best alienating people from the very real issues people with disability face, so please just stop, or be civil.


Probably should do this for number indexers as well.


Ok, pushed new changes, try again


Prefer not to use __defineSetter__ as it's been [deprecated](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/__defineSetter__), and isn't fully supported in all browsers we test on.


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 6](https://build.fubar-dev.com/viewLog.html?buildId=59&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:09


@krainboltgreene One of the most ableist people here is you. Stop objectifying us. We're not pawns for your political agenda.


One thing I do want to add is the ability to optionally specify the names of the handlers in a clean format. I need to finish up some other work before doing that so if you want to give that a stab then go ahead


Yes, of course. So you're suggesting it may be useful to overload `_Is<U, T, V...>` vs. `_Valid<T, U, V...>`? I suppose that's not outside the realm of possibility. I can certainly see `_Is<...>` vs. `_IsNot<...>` being useful but that has nothing to do with subsumption since `!_Is<...>` is an atomic predicate.


indent 2


krainboltgreene:
"Nobody wants any of you here"
"You're going to be alone"

Basic logic fail.

Also, I'm not so sure krainboltgreene doesn't have some kind of neurogenerative disease. How would we know? Are angry internet rantings a symptom?


saw approval from javajolt
at https://github.com/basho/lager/commit/9c0e141d30b052bea9dc40fdecb86f23046d063e


merging basho/lager/test/mra/gh280 = 9c0e141d into borshop-integration-282-test/mra/gh280


basho/lager/test/mra/gh280 = 9c0e141d merged ok, testing candidate = ca280a2a


know -> known


This was a nice surprise. Totally broke our build system. The bower.json file should NOT assume that less or scss compilation need to take place after copying files (with bower-main for example). It should be a packaged library!


I think this change caused btcd to not compile:

# github.com/FactomProject/btcd
C:\GoogleAppEngine\gopath\src\github.com\FactomProject\btcd\factompeer.go:60: msg.ECBlock.Header.Hash undefined (type *common.ECBlockHeader has no field or method Hash)
FAIL    github.com/FactomProject/btcd [build failed]
?       github.com/FactomProject/btcd/addrmgr   [no test files]
?       github.com/FactomProject/btcd/blockchain        [no test files]
# github.com/FactomProject/btcd
C:\GoogleAppEngine\gopath\src\github.com\FactomProject\btcd\factompeer.go:60: msg.ECBlock.Header.Hash undefined (type *common.ECBlockHeader has no field or method Hash)
ok      github.com/FactomProject/btcd/btcjson   5.283s
?       github.com/FactomProject/btcd/btcjson/btcws     [no test files]
?       github.com/FactomProject/btcd/btcjson/v2/btcjson        [no test files]
?       github.com/FactomProject/btcd/chaincfg  [no test files]
?       github.com/FactomProject/btcd/cmd/addblock      [no test files]
?       github.com/FactomProject/btcd/cmd/dropafter     [no test files]


splittedd -> split


@krainboltgreene
Why are you accusing other people of being toxic and harmful to communities when this applies to you right now? If you think that GitHub Support action was a good thing then state it without being toxic yourself.


Fixed it. Thanks!


@krainboltgreene
I'm seeing a whole lot projection here.  If this thread and the people who have replied to you are any indication, you're the one who isn't wanted.  Every response you've made is the equivalent of, "lalalalalalalala i'm not listening stop having different opinions that me."  It's pathetic.

Telling people to go away isn't going to make them go away.  Grow up and learn to deal with things you disagree with.  You look like a child throwing a temper tantrum right now.  This is a natural effect of getting so deep into PC rhetoric that any challenge to your beliefs becomes a challenge to your very existence.

I mean if this article isn't an example of delusional, self-flagellating white guilt then I don't know what is.


@alavers Thats not how it happened though :)
He offered talking about it, if somebody opens an issue on the project, [which I did.](https://github.com/ybur-yug/hobos/issues/1).

I then proposed the change, which he implemented.

Do you prefer to build more walls or tear them down :)?
Both sides have some valid points. And we need civil conversations to find valid solutions.

The worst we can do is scold people for acknowledging their mistakes.


Also new version of ServiceStackIDEA - 1.0.6 updated to include new version of net.servicestack.client + android when using `Add ServiceStack Reference`.


It is curious, thtat the bug happened on Ubuntu 12.04 (x86_64) + tcc 0.9.25, but not happened with Ubuntu 12.05 (x86_64) + also tcc 0.9.25. May be different compiler version claiming same version (?). Scary bug, anyway.


This one makes no sense this way. In #3 this change was on line [463](/itsmeront/squeakssl/pull/3/files#diff-9e4c3c0adfa49a82c47f778e7555b185R463), which is on line 410 here. The "(null)" is required when the format parameter is %s and the argument is null. So this should only be used with the property variable as argument.


I think you forgot to not make this a header.


I've always seen ~ for asymptotic equality (see the three-author tanglegram paper) though CS folk might use something else.


RadianceRefactor (macumber) - Win64-Windows-7-VisualStudio-12: OK (2192 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.96%25-yellow.svg)


Don't mention bug numbers in test names. Rename to something like "testEffectAllowedExceptionIsCaught"


Steve, I might only post one or two more comments tonight, as it takes a while for me to word things in a way that even a Marine might understand :).  I can say one thing, even though I cannot write the code I want right now, I think I am starting to pick up on this.  I do not feel too bad as Thomas Ingram said this was a big thing to take on as a beginning Rails developer.


@jgrigonis actually yes ;)
Multiple sclerosis can cause a variety of physical and psychological symptoms.
Including changes in personality and depression. (More depression than just explainable from the fear and stress it causes.)


Could use some `override` keywords.


Note the `if` here uses `'strong'` but the choice is from `'weak'`.


Could use `override`.


Ah, thanks for noticing this slip


(this is what I meant to refer to, when I said "I'll get this one in with the IS_BLOCK_SERIES fixes.")


nit: no space between switch and paren. I noticed you fixed one in the previous commit. Trying to maintain cosmic balance? :)


@AllPurposeName awww yeah


Ha, Android Studio must have fixed that one, but not the one I added. Fixed.


Actually, none of sensors plugins uses namespace


Notice how similar the blocks of code are above. Lots of very similar code is usually a good sign that there should be a function that implements the common functionality or a `for` loop that performs the same operation for each case (or both). Can you think of a way to write a `for` loop that asks each question in turn?

Note you can iterate over all the keys in a dictionary as follows:

```python
for key in questions:
    print key, '->', questions[key]
```


Assuming you're cross-compiling with Eclipse, you can go to [this somewhat hidden](http://iotdk.intel.com/sdk/mraa-update/) website and download the latest version (0.7.3.9 I think). Extract the file to the appropriate location in the Eclipse tool chain folder and you should be ready to go.

If you're NOT cross-compiling, I don't really have any advice.


Github won't stop doing this, it's not a one off situation. Outrage yourselves straight out the door to some other git host. Pleeeeeease.


I guess I should have deleted this branch instead of merging it in and then deleting it. Anyway, your `issues/9` branch should fix all your code here.


TeamCity Bender :: Deploy [Build 3.0.15.0](http://build.mikeobrien.net/viewLog.html?buildId=2446&buildTypeId=bender&guest=1) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 3239, ignored: 5 Build time: 00:02:05
### Failed tests
```
Tests.dll: Tests: Nodes: Object: Tests.Nodes.Object.ValueNodeTests.should_be_of_node_type_value: <no details avaliable>

Tests.dll: Tests: Nodes: Object: Tests.Nodes.Object.ValueNodeTests.should_be_of_type_value: <no details avaliable>

Tests.dll: Tests: Nodes: Object: Tests.Nodes.Object.ValueNodeTests.should_get_the_name: <no details avaliable>

```


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/288


@krainboltgreene
>Every response you've made is the equivalent of, "lalalalalalalala i'm not listening stop having different opinions that me." It's pathetic.

lol


This branch was behind; you fixed this in `issues/9` as part of #9.


Yes it is, the if-statement on FragmentSize and length of b64 is is completely copied from Golang OTR, and it isn't necessary here.